Defaults are not set if all fields are optional and unspecified

Bug Report

What did you do?

Create a CRD where all fields in the spec are optional and configure defaults via +kubebuilder:default comments. These defaults are different from Go's 0 value, hence the need for this:
// KafkaTopicSpec defines the desired state of KafkaTopic
type KafkaTopicSpec struct {
	// Replicas is the replication factor across the brokers
	//+kubebuilder:default=1
	//+optional
	Replicas int `json:"replicas"`
	// Partitions is the number number of partitions in the topic
	//+kubebuilder:default=1
	//+optional
	Partitions int `json:"partitions"`
}

With CRDs generated and installed, I applied the following manifest:
apiVersion: kafka.gympass.com/v1alpha1
kind: KafkaTopic
metadata:
  name: kafkatopic-sample
spec:
#  replicas: 3
#  partitions: 3

What did you expect to see?
The default values for all fields within .spec are set.
What did you see instead? Under which circumstances?
.spec does not exist:
❯ kubectl get -f config/samples/kafka_v1alpha1_kafkatopic.yaml -o yaml
apiVersion: kafka.gympass.com/v1alpha1
kind: KafkaTopic
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"kafka.gympass.com/v1alpha1","kind":"KafkaTopic","metadata":{"annotations":{},"name":"kafkatopic-sample","namespace":"default"},"spec":null}
  creationTimestamp: "2021-05-04T19:33:46Z"
  generation: 1
  managedFields:
  - apiVersion: kafka.gympass.com/v1alpha1
    fieldsType: FieldsV1
    fieldsV1:
      f:metadata:
        f:annotations:
          .: {}
          f:kubectl.kubernetes.io/last-applied-configuration: {}
    manager: kubectl-client-side-apply
    operation: Update
    time: "2021-05-04T19:33:46Z"
  name: kafkatopic-sample
  namespace: default
  resourceVersion: "2401287"
  uid: 3fb5a081-2193-4c8b-8f00-a5f3e7447a8c

Environment
Operator type:

/language go

Kubernetes cluster type:
❯ minikube version
minikube version: v1.17.0
commit: 7e8b5a89575945ba8f8246bfe547178c1a995198

$ operator-sdk version
❯ operator-sdk version
operator-sdk version: "v1.7.1", commit: "c984b00c1a1f75871788bb082ee8226b5118e4da", kubernetes version: "1.19.4", go version: "go1.15.5", GOOS: "linux", GOARCH: "amd64"

$ go version (if language is Go)
❯ go version
go version go1.15.6 linux/amd64

$ kubectl version
❯ kubectl version
Client Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.1", GitCommit:"c4d752765b3bbac2237bf87cf0b1c2e307844666", GitTreeState:"clean", BuildDate:"2020-12-18T12:09:25Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.2", GitCommit:"faecb196815e248d3ecfb03c680a4507229c2a56", GitTreeState:"clean", BuildDate:"2021-01-13T13:20:00Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"linux/amd64"}

Possible Solution

Additional context
When I provide either field default setting kicks in for the other.
Currently wondering if this has to do with .spec being empty, perhaps the default setting logic only kicks in if the parent object in the manifest exists (so it doesn't if .spec == null). If that's the case, not sure if there is a way around this.
Specifying .spec but not creating any fields within doesn't really do anything, I suppose the API server still interprets it as .spec == null (I believe rightfully so).
Am I missing something here? Thanks! :-)

What does your CRD's validation scheme look like?
/triage support

@estroz here's the entire CRD, please let me know if this is not what you were asking for

CRD

---
apiVersion: apiextensions.k8s.io/v1
kind: CustomResourceDefinition
metadata:
  annotations:
    controller-gen.kubebuilder.io/version: v0.4.1
  creationTimestamp: null
  name: kafkatopics.kafka.gympass.com
spec:
  group: kafka.gympass.com
  names:
    kind: KafkaTopic
    listKind: KafkaTopicList
    plural: kafkatopics
    singular: kafkatopic
  scope: Namespaced
  versions:
  - name: v1alpha1
    schema:
      openAPIV3Schema:
        description: KafkaTopic is the Schema for the kafkatopics API
        properties:
          apiVersion:
            description: 'APIVersion defines the versioned schema of this representation
              of an object. Servers should convert recognized schemas to the latest
              internal value, and may reject unrecognized values. More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#resources'
            type: string
          kind:
            description: 'Kind is a string value representing the REST resource this
              object represents. Servers may infer this from the endpoint the client
              submits requests to. Cannot be updated. In CamelCase. More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#types-kinds'
            type: string
          metadata:
            type: object
          spec:
            description: KafkaTopicSpec defines the desired state of KafkaTopic
            properties:
              partitions:
                default: 1
                description: Partitions is the number number of partitions in the
                  topic
                type: integer
              replicas:
                default: 1
                description: Replicas is the replication factor across the brokers
                type: integer
            required:
            - partitions
            type: object
          status:
            description: KafkaTopicStatus defines the observed state of KafkaTopic
            type: object
        type: object
    served: true
    storage: true
    subresources:
      status: {}
status:
  acceptedNames:
    kind: ""
    plural: ""
  conditions: []
  storedVersions: []

I suppose we can easily work around this with a mutating webhook, but I was wondering if it was possible using OpenAPI only.

If I specify spec: {} then I get a validation error, but not if spec is empty. This is because spec is not required, and null is an allowed value in that case but not the empty spec. If you mark the spec as required, you'll see validation errors in both cases:

updated CRD
```diff
---
apiVersion: apiextensions.k8s.io/v1
kind: CustomResourceDefinition
metadata:
  annotations:
    controller-gen.kubebuilder.io/version: v0.4.1
  creationTimestamp: null
  name: kafkatopics.kafka.gympass.com
spec:
  group: kafka.gympass.com
  names:
    kind: KafkaTopic
    listKind: KafkaTopicList
    plural: kafkatopics
    singular: kafkatopic
  scope: Namespaced
  versions:
  - name: v1alpha1
    schema:
      openAPIV3Schema:
        description: KafkaTopic is the Schema for the kafkatopics API
        properties:
          apiVersion:
            description: 'APIVersion defines the versioned schema of this representation
              of an object. Servers should convert recognized schemas to the latest
              internal value, and may reject unrecognized values. More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#resources'
            type: string
          kind:
            description: 'Kind is a string value representing the REST resource this
              object represents. Servers may infer this from the endpoint the client
              submits requests to. Cannot be updated. In CamelCase. More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#types-kinds'
            type: string
          metadata:
            type: object
          spec:
            description: KafkaTopicSpec defines the desired state of KafkaTopic
            properties:
              partitions:
                default: 1
                description: Partitions is the number number of partitions in the
                  topic
                type: integer
              replicas:
                default: 1
                description: Replicas is the replication factor across the brokers
                type: integer
            required:
            - partitions
            type: object
          status:
            description: KafkaTopicStatus defines the observed state of KafkaTopic
            type: object
        type: object
+        required:
+        - spec
    served: true
    storage: true
    subresources:
      status: {}
status:
  acceptedNames:
    kind: ""
    plural: ""
  conditions: []
  storedVersions: []
```

Ah gotcha! So even if children fields/structs are marked as required the parent may still be completely missing if it isn't required itself. Thanks a lot @estroz, closing the issue :-)


[Event Request] Table 330 Currency Exchange Rate - Function FindCurrency

Dear ALAppExtensions Team,
We would like to request the following event publisher

procedure FindCurrency(Date: Date; CurrencyCode: Code[10]; CacheNo: Integer)
begin
if (CurrencyCode2[CacheNo] = CurrencyCode) and (Date2[CacheNo] = Date) then
Rec := CurrencyExchRate2[CacheNo]
else begin
if Date = 0D then
Date := WorkDate;
CurrencyExchRate2[CacheNo].SetRange("Currency Code", CurrencyCode);
CurrencyExchRate2[CacheNo].SetRange("Starting Date", 0D, Date);
//event request
OnAfterCurrencyExchRate2Filter(CurrencyExchRate2,CacheNo,Date);
//event request
CurrencyExchRate2[CacheNo].FindLast;
Rec := CurrencyExchRate2[CacheNo];
CurrencyCode2[CacheNo] := CurrencyCode;
Date2[CacheNo] := Date;
end;
OnAfterFindCurrency(Rec, CurrencyExchRate2);
end;

local procedure OnAfterCurrencyExchRate2Filter(var CurrencyExchRate2: Record "Currency Exchange Rate",CacheNo,Date)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[rt.openssl.org #3866] [PATCH] Support for vc10 build

Migrated from rt.openssl.org#3866 (status was 'new')
Requestors:

<EMAIL_ADDRESS>
Attachments:

0011-Support-for-vc10-build.patch

From<EMAIL_ADDRESS>on 2015-05-26 19:08:53:
Hello OpenSSL Org:

This is a change that Akamai has made to its implementation of OpenSSL.

Version: master branch
Description: Support for vc10 build

Includes some changes for VC9 and VC7 compiles as well

Github link:
https://github.com/akamai/openssl/commit/dcd470b6ab92b8e2661872de298de78de1b5057b

And attachment.

Thank you.

--
-Todd Short
//<EMAIL_ADDRESS>// âOne if by land, two if by sea, three if by the Internet."

@tmshort is this still relevant?

Yes, but needs an update.

Scanned through the code, and the only relevant thing would be snprintf(), and the only place that is used is in apps/rehash.c, so changing that code to use BIO_snprintf() would be sufficient. The other things in the original commit do not appear to be required.


[Bug]: heracles:composite task with a sub-list longer than two items will cause quest to not render properly

Bug Description
First, the good case: A nicely rendered quest with a composite task having two sub-tasks:

Second, the bugged case: How it looks if there are three or more sub-tasks:

Version
1.0.5
Mod Loader Version
0.14.22
Mod Loader
Fabric
Code of Conduct

[X] I have searched the issue tracker and confirmed that the issue has not been reported
[X] I have checked the FAQ (if one is present) and my issue is not listed
[X] I have verified that I am using the latest version of the mod
[X] I have verified that I have installed all the required dependencies for the mod
[X] I have verified that I do not have Optifine installed. We will close issues if we detect Optifine is in your pack. We cannot fix Optifine issues. Please do not report issues with Optifine present.

Big 'ol nvm. Works this morning. Can only assume something else fishy with the instance of AoF7 I was testing with last night.
TY again, for all the work. Sorry to add any mental burden with an erroneous bug report.


Chat: Copy conversation link to clipboard

Adding a CopyToClipboard icon next to the Chat title, to copy the convo url.
The cursor transforms into a pointer when we hover the title and the icon.
The icon changes to "check" to confirm it was done

I added a commit to remove the component as it was not really reusable because of the "truncate title logic", and set it directly to AppLayout.


Avoid curried thunks (workaround SR-12115)

Motivation:
Calling function(otherFunction) sometimes allocates, but
function({ otherFunction($0) } does not.
See: SR-12115 and SR-12116.
Modifications:
Avoid passing functions directly; pass them in new closures instead.
Result:
Fewer allocations.

Whoa that's good to know 👀

As an FYI @ktoso, most instances of this pattern do not trigger extra allocations: however, we haven’t isolated the common trait of the kinds that do.

@ktoso we believe that you can find all the bad ones by

compile in release mode
otool -tV path-to-binary | swift demangle | grep “curry thunk”

If that matches something that might allocate, if not you’re good


[crash] [local co-op] Crash after sleeping in RDV hotel

I am having an issue where game crashes after sleeping at the hotel on RDV.
I cannot stay overnight in the hotel. After sleeping in the hotel the game is saved, but the game will crash immediately after saving before loading following day.
I am playing local coop with one of my friends. Below attached my latest smapi-crash log.
https://smapi.io/log/9759da8885ae4c31818886426f5d2e66
I noticed this inside the log, but I don't know how to fix. Can I get some help?
"The given key 'Component: indoor' was not present in the dictionary. "
PS: We can successfully sleep in any other part of the map like my farm, ginger island or in tents normally, but whenever me or my (local co-op) friend sleeps in RDV hotel we get that crash.
Steps to reproduce:

Initiate a farm with RDV with 1 extra cabin,
load in the farm and a new local co-op player.
Go to RDV (either host or player 2) and play 500 gold for overnight stay.
Go upstairs, sleep in the hotel room
Make other player also go to bed (doesn't really matter where, it could be in main farm or any other area). The game will save, but will crash when loading following day.

Hi, a bit late, but this should be fixed as of Stardew Valley's official 1.6.9 update, which made changes to how tilesheets are loaded.
Update Stardew Valley and SMAPI to the newest versions and the issue should be gone.


screen sharing doesn't work on chrome

STR:

open chrome (version 90) and visit https://www.webrtc-experiment.com/Pluginfree-Screen-Sharing
click "share your screen", pick any window or tab on the machine
copy URL and paste into a new tab
video loads, but it is all white
paste same URL into firefox (version 87), and video loads as expected

I get the same behavior on both macOS 10.15 and ubuntu linux.

I tried on Arch Linux with Chromium 89.0.4389.90 and I get the exact same behavior.

Replicated on big sur 11.16 and El Capitan 10.11.16.
Screen sharing is working well on other open source WebRTC platforms.


fix: update dependencies

:tada: This PR is included in version 6.3.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fixed NPE in Staff Portal when loading VT2 images

Description
This PR includes the following proposed change(s):
Fixed NPE in Staff Portal
The Staff Portal would not load VT2 images due to a mismatch of properties, namely the act_or_regulation_name_code field is only on VT1 images, not VT2. There is no such property on VT2 images called act_or_regulation_name_code and so this field may be undefined.
Note: I noticed that the Staff Portal may try to use this field for determining if the Count sections are valid or not (presumbably to pull the "MVA" text for the act portion of the Act/Sect/Desc fields. Currently the portal thinks the counts are always invalid.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Refactoring / Documentation
[ ] Version change

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Does the change impact or break the Docker build?

[ ] Yes
[x] No

If Yes: Has Docker been updated accordingly?

[ ] Yes
[ ] No

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[x] New and existing unit tests pass locally with my changes

Just asking which part was showing 'undefined' because it should be showing "" in frontend already if is null or undefined

The ocrViolationTicket.fields["counts.count_no_1.act_or_regulation_name_code"] field is undefined for VT2 images so the .value was throwing an error.

ok, can you try ocrViolationTicket.fields["counts.count_no_1.act_or_regulation_name_code"]?.value ? Thanks

ok, can you try ocrViolationTicket.fields["counts.count_no_1.act_or_regulation_name_code"]?.value ? Thanks

That's much cleaner, thanks. Done.


Test step stays "pending or cancelled" after build completes.

Following on from issue: 583
Thanks @adriencg for getting the build working again.
The build now runs to completion and has deployed.
The Test step, however, remains as a "grey clock" rather than a "green tick" and reports Test is still pending or has been canceled for this build.  I cannot find any additional logging or monitoring output about it.
Thanks
Tom

If you wouldn't mind providing me with just your appId and region that'd let me investigate what might be happening:
You can find your APP-ID and REGION by looking at the amplify console:
App Settings --> General --> AppArn: It will look something like this:
arn:aws:amplify:REGION:accountId:apps/APP-ID
Note: I do not need your AWS account ID; just the appId + region.

Hi,
Here you go.
 Make sure that the phases block of the yaml is indented relative to the top-level test block (just above artifacts). Pasting the entire "Build settings" this time for convenience
version: 0.1
backend:
  phases:
    build:
      commands:
        - '# Execute Amplify CLI with the helper script'
        - amplifyPush --simple
frontend:
  phases:
    preBuild:
      commands:
        - yarn install
    build:
      commands:
        - yarn run build
  artifacts:
    baseDirectory: build
    files:
      - '**/*'
  cache:
    paths:
      - node_modules/**/*
test:
  artifacts:
    baseDirectory: cypress
    configFilePath: '**/mochawesome.json'
    files:
      - '**/*.png'
      - '**/*.mp4'
  phases:
    preTest:
      commands:
        - npm ci
        - npm install -g pm2
        - npm install -g wait-on
        - npm install mocha mochawesome mochawesome-merge mochawesome-report-generator
        - pm2 start npm -- start
        - wait-on http://localhost:3000
    test:
      commands:
        - 'npx cypress run --reporter mochawesome --reporter-options "reportDir=cypress/report/mochawesome-report,overwrite=false,html=false,json=true,timestamp=mmddyyyy_HHMMss"'
    postTest:
      commands:
        - npx mochawesome-merge cypress/report/mochawesome-report/mochawesome*.json > cypress/report/mochawesome.json
        - pm2 kill

Thank you!

Hi team,
My test says "Test step stays "pending or cancelled" after build completes."
here is my build setting
version: 0.1
backend:
phases:
preBuild:
commands:
- yum install -y gcc openssl-devel bzip2-devel libffi-devel
- wget https://www.python.org/ftp/python/3.8.2/Python-3.8.2.tgz
- tar xzf Python-3.8.2.tgz
- cd Python-3.8.2
- ./configure --enable-optimizations
- make altinstall
- rm /usr/bin/python3
- ln -s /usr/local/bin/python3.8 /usr/local/bin/python3
- curl -O https://bootstrap.pypa.io/get-pip.py
- python3 get-pip.py
- pip install --user pipenv
- cd ..
build:
commands:
- '# Execute Amplify CLI with the helper script'
- amplifyPush --simple
frontend:
phases:
preBuild:
commands:
- npm ci
build:
commands:
- npm run build
artifacts:
baseDirectory: build
files:
- '/*'
cache:
paths:
- node_modules//*
test:
artifacts:
baseDirectory: cypress
configFilePath: '/mochawesome.json'
files:
- '/.png'
- '**/.mp4'
phases:
preTest:
commands:
- npm install
- npm install wait-on
- npm install<EMAIL_ADDRESS>mochawesome mochawesome-merge mochawesome-report-generator
- npx pm2 start npm -- start
- 'npm start & npx wait-on http://localhost:3000'
test:
commands:
- 'npx cypress run --reporter mochawesome --reporter-options "reportDir=cypress/report/mochawesome-report,overwrite=false,html=false,json=true,timestamp=mmddyyyy_HHMMss"'
postTest:
commands:
- npx mochawesome-merge — reportDir cypress/report/mochawesome-report/mochawesome*.json > cypress/report/mochawesome.json
Help me regarding the same

Hi @adriencg, we are facing the exact same problem:
Region: EU-central-1
AppId: d3tcjln4xx1iki
for testing purposes I added the exact script you pasted here. Still the Test stage shows: "Test is still pending or has been cancelled for this build."
Can you have a look please?


Harden kubectl proxy

The current default behavior of kubectl proxy is to listen on all addresses on port 8001. Thus anyone on the same network gets full access to the apiserver of my cluster.
It would be good to default to listen on localhost and maybe add a flag to override this behavior.
The second issue is that the proxy is vulnerable to dns rebinding. I.e. if I'm running a kubectl proxy and browse the web at the same time, a malicious website can interact with the local apiserver. The recommended way to protect against dns rebinding is to check the Host header for expected values if present.

/cc @jlowdermilk

The recommended way to protect against dns rebinding is to check the Host header for expected values if present.

Can you expand on this? What expected values would we be checking for?

The recommended way to protect against dns rebinding is to check the Host header for expected values if present.

Can you expand on this? What expected values would we be checking for?

If it's a legitimate request from the browser, the host header should either include localhost:8001 or an ip address, i.e. <IP_ADDRESS>:8001 or [::1]:8001 but other addresses might be possible. An empty host header is probably fine too.
Another issue with this proxy is that other web based attacks will apply, for example XSRF and XSS.
To protect against XSRF attacks I think it would be sufficient to rely on the fact that you can't post cross-origin with an application/json content type. We'd just need to add a check for that content-type in the api. However, there might also be sensitive endpoints behind GET requests.
http://localhost:8001/api/v1bet3/namespaces/default/pods/elasticsearch-logging-dr7bq/exec comes to mind, though that one doesn't work since it requires a spdy connection upgrade, but maybe there are others?
And then there's the /proxy/ endpoint that passes the requests through to services running in kubernetes. That means any XSS in kibana can be used to bypass other restrictions and e.g. read /api/v1beta3/secrets or post a new pod description.
What's the usual way this proxy is used, via the browser or through curl..? Maybe there's some better way to harden this service, e.g. require a special header on any request and reject an requests that don't have them (which would still need protection against dns rebinding)?

However, there might also be sensitive endpoints behind GET requests.

Found one:
http://localhost:8001/api/v1beta3/proxy/nodes/kubernetes-minion-brc6/run/default/elasticsearch-logging-dr7bq/elasticsearch-logging?cmd=ls -l&stdout=1
With that, if you have kubectl proxy running and visit http://evil.com, evil.com will be able to run arbitrary commands in the cluster.

The proxy is often used with a web browser, e.g. https://github.com/GoogleCloudPlatform/kubernetes/blob/master/examples/update-demo/README.md#step-one-turn-up-the-ux-for-the-demo

A part of my previous comment was wrong. It's actually possible to POST application/json cross origin, so checking the content-type wouldn't be enough to protect against XSRF.

So I still think the proxy is scary if someone actually tries to attack it.
The api needs some XSRF protection so that an attacker can't trigger it cross-origin. A custom header might be fine in this case.
However, I'd be happier if we could completely drop POST requests to the API since any XSRF protections can be bypassed with an XSS. That is, any XSS in kibana, elasticsearch or any other service reachable via the /proxy/ api.

cc @caesarxuchao

@brendanburns are you working on this issue? I can take a look if you are not.

I'm working on this in the context of
https://github.com/GoogleCloudPlatform/kubernetes/pull/9287
But I'm not doing anything else yet.
Maybe the next thing is to set a CORS header?
--brendan


Update AutoRest C# version to 3.0.0-beta.20240117.3

Update AutoRest C# version to 3.0.0-beta.20240117.3

close since it's covered by #41389


add automatic builds for nixos-20.09

Repository on docker hub is created: https://hub.docker.com/repository/docker/niteo/nixpkgs-nixos-20.09

Repo gets created automatically once in master btw :)


[REVIEW] my-wallet-app-ui

Project link
https://github.com/rukundob451/my-wallet-app-ui

Thank you for sharing, we will review on a live stream soon 👍

https://github.com/rukundob451/my-wallet-app-ui/community
add a release
hide packages
hide wiki
add more to the readme seems to be the initial flutter readme
use issues and Prs

https://github.com/rukundob451/my-wallet-app-ui/community
add a release
hide packages
hide wiki
add more to the readme seems to be the initial flutter readme
use issues and Prs

Agree with Tom.. If your not gonna use issues, please hide the tab.

Sure going to do this immediately

Looks good, reviewed on live stream https://www.youtube.com/watch?v=afo0E5RrJYA

only has 1 commit, suggest committing in stages
releases are great to use, but while not using them, suggest hiding them for now (same with packages)
suggest having a screenshot in the README
hide wiki if not used

Thanks for this surely, let me do this straight up.
Really appreciate your reviews. Possibly you could recommend me to some projects as a contributor @eddiejaoude. What do you think?


Try using @deprecated docygen command for deprecated APIs

Find out all the deprecated things in our SDK and try using @deprecated attribute, see if it looks good.
Currently, at least for ApplicationContext, we've done deprecated via @note, and I think it is not noticeable enough, for something that is deprecated, see https://azuresdkdocs.blob.core.windows.net/$web/cpp/azure-core/1.14.0-beta.1/class_azure_1_1_core_1_1_context.html (and also we shoud drop ":" after @note there).

Here is how @deprecated looks: https://azuresdkdocs.blob.core.windows.net/$web/cpp/azure-core/1.14.0-beta.1/class_azure_1_1_core_1_1_credentials_1_1_token_credential.html#a804d54f8e363ca038ee43578e1209b61
I'd say it is not that much more visible, just a little bit better, because it is titled as "Deprecated" instead of "Note".


Carthage Compatible!

Please make this carthage compatible. It would be so much better that way. Thanks.

@Dershowitz011
Sorry.
You can install by Carthage.
I've updated README.
Please, check it.


ON CLUSTER not working in latest update

Query:
CREATE TABLE shard1.facts_test_2 on cluster motiv_1s2r (
...
)
ENGINE = ReplicatedReplacingMergeTree('/clickhouse/tables/facts_test_2', '{server}') PARTITION BY (toYear(time), toMonth(time))
ORDER BY
(mpid, siteid, time) SETTINGS index_granularity = 8192
Returns:
click1	9000	49	Code: 49, e.displayText() = DB::Exception: Query id cannot be empty, e.what() = DB::Exception	1	0
click2	9000	49	Code: 49, e.displayText() = DB::Exception: Query id cannot be empty, e.what() = DB::Exception	0	0
This occurs both within Tabix and running on clickhouse-client. Running without ON CLUSTER is fine.

We have fixed it in 1.1.54370.


Replace Versions by scooby

Now that scooby exists we should, for simplicity, simply use it.

I can probably simplify Travis-testing without loosing coverage, as many edge-cases were due to the Versions (with mkl, IPython, matplotlib etc).


Strong reference on presentingViewController

Is there any reason why it is strong reference?

I'm using it because I want Harpy to own the reference to the presentingViewController. Also, I'm not trying to avoid retain cycles, so there's no need to use any other keywords.
https://developer.apple.com/library/ios/documentation/Cocoa/Conceptual/MemoryMgmt/Articles/mmPractical.html


background-clip: text in Safari 14+

Fixes #6106.
https://developer.mozilla.org/en-US/docs/Web/CSS/background-clip#browser_compatibility looks mixed, but https://wpt.fyi/results/css?label=master&label=experimental&aligned&q=background-clip seems fine, so…

Thanks!


4 HOURS AND COUNTING... PLEASE FIX

Time: 0ms
Server Execution Error:
The server timed out waiting for the request to be processed. This may be due to high server load. Please try your request again.
Request Error:
Request failed with status code 500
This error was caused due to an issue processing the web request, not because of an issue executing your code. You can retry the request.

I've been getting this error all day in multiple browsers and on multiple PCs.

i used different browsers and different computers as well. A macbook and a windows laptop. Same exact
error. Please fix this??

+1

+1

Same issue here, I checked the status page and it says there are no errors but this is happening been like this for 4 hours now.

+1
Same issue for me.

It's been going for a while now.. weird, it's like no one cares

It would be courtesy of someone just to say "sorry about this we are working on it"

++

+1. In case in anybody was thinking it could be an ISP/network issue not codewars': all other sites work fine, same latency I'm used to, no parental/work/school blockers here, and just to be extra sure switched ISPs (from CenturyLink DSL to Verizon hotspot) and got the same issue. Happens when trying to test a kata and kata attempts. Some are succeeding now but much slower than usual. Stalls on Status: Assigning a server. Everything else seems fine. Haven't checked kumites.

++

Any good codewars alternative ?

+ Please fix it

++
I'm trying to show codewars to a friend who just started programming :(

#1229

it's work

I've contacted @jhoffner and we're looking into it.
Sorry about the delay, everyone is off for thanksgiving.

We found the possible cause and the fix is being deployed.

Closing this as fixed. Seems to be working for the past 2 hours.


The rootiness of a data store is not updated correctly for aliased data stores

When a data store is aliased, it is marked as root by calling setRoot on the data store context which updates its isRootDataStore property. However, the following places still return the value from the initial snapshot details of the data store context:

isRoot() that is used by garbage collection to determine which data stores are root and should be marked referenced.
On summarizing a data store via summarizeInternal method.

Aliased data stores are created with isRootDataStore initially false and after aliasing, the isRootDataStore will remain false in the initial snapshot.
Find and fix any other isRootDataStore inconsistencies.

@andre4i FYI


Approximate EIP 1559 Gas price estimation

@koeppelmann suggested to build another gas price estimation source that approximates EIP-1559 optimised gas pricing. It would use either eth_feeHistory or eth_getBlockBy* to query the current baseFeePerGas  and then add a fixed miner tip (e.g. 10 gwei) to that value.
Given we already re-request gas fees quite frequently (~ every 15s) we would be able to react to rising base fees. Falling base fees would still add in increased miner revenue since we are not capping the tip (this would require a proper EIP1559 implementation).
If eth_feeHistory is used, we could even see if the baseFee is currently rising or falling and adjust our gasPrice with that prediction (multiplying or dividing by 1.125).
An overview of EIP-1559 JSON RPC Changes can be found here: https://hackmd.io/@timbeiko/1559-json-rpc

An additional thought, we can query the fee history to get the current base fee on every block as part of the "service maintenance" that we already run whenever we observe a new block.

I agree with this. Should save us fees and also make us not pay extreme amounts when gas price through gas now spikes as we would wait for the base fee to react. If we don't get our transactions included for a short amount of time in these extreme situations that's fine and maybe even preferable.

For reference, we could create a new base fee component that implements Maintaining trait and then gets registered to update on every block:
https://github.com/gnosis/gp-v2-services/blob/2d9aa1eb318f5168c8669c61caf816b28a50377b/orderbook/src/main.rs#L345-L352

I would start with BlockNative gas price estimates, which should be good enough for now. This would only be needed for EIP-1559 networks not supported by BlockNative.

This has become more important now that GasNow has announced they will shut down on October 15th. We will probably have the blocknative estimator at that point but it would be nice to not be reliant on an external service.

Do we want this to be open source? (it could become really smart estimator over time)

Do we want this to be open source?

Yes.

AFAIU this has been implemented! 🎉


Programmatic access to image after build

📚 Context
I'd like to automate the process of building and pushing the node image, through a GitHub Actions workflow. However, in order to push an image to GHCR, I have to tag it, but the image that cartesi deploy build builds is not tagged, and its ID is not easily accessible. I could, of course, process the output of the command, or do some text-processing magic with the output of docker images, but...
✔ Solution
A more consistent solution, in my view, would be to either...

Allow the user to pass a tag to cartesi deploy build
Write the image ID to a file, like Docker does with --iidfile, so that the user can run docker tag later or do whatever it wants with it, which could be exporting it to an archive file, etc.

I believe this has been addressed, right, @tuler?

Yes, using cartesi deploy build --json


Sign and verify support

This is currently a work-in-progress branch to implement sign and verify in the openssl crypto plugin. Opening pull request to get early feedback on the code.
NOTE: currently missing tests. Will add those soon.

Mostly LGTM - thanks very much.  Obviously needs unit tests, but mostly looking great.

I wonder whether I need to extend the function signatures to also include optional MessageAuthenticationCode and Algorithm parameters, so that we can use HMAC (https://www.openssl.org/docs/man1.0.2/crypto/hmac.html) or CMAC (http://rubenlaguna.com/post/2015-02-05-compute-aes-cmac-using-openssl-slash-libcrypto/) if required.

@chriadam For the HMAC and CMAC, I would suggest to deal with those separately after we managed to get the current stuff working.

Amended my previous commit.

After the plugin load fix, I am able to run it on my desktop like this:
LD_LIBRARY_PATH=../lib/Secrets:../lib/Crypto QT_PLUGIN_PATH=../tests/plugins/testopensslplugin:../tests/plugins/testopensslcryptoplugin:../tests/plugins/testsqlcipherplugin:../tests/plugins/testsqliteplugin ./sailfishsecretsd --test

And execute the tests like this:
LD_LIBRARY_PATH=/home/Timur/dev/Qt/5.6.3/gcc_64/lib:../../../lib/Secrets:../../../lib/Crypto ./tst_cryptorequests

How much overhead does the new plugin loading semantic imply?  That is, on Sailfish OS devices, how many more libraries are speculatively loaded as potential Secrets/Crypto plugins?

Closing this PR, will open a new one.


Fixes #12 to improve error message

After fix, error message gets displayed as
Rally tired waiting 300.00 seconds for Replication controller rally-f9154336-qrg2lh4e:4fc9c740-25e0-49db-a21c-04dedfda115a to become 2 replicas running current status 2 replicas running and 1 replicas ready


Document Azure AD SSO setup process

https://pulumi.io/reference/service/saml-okta.html but for Azure AD

Fixed with https://github.com/pulumi/docs/pull/995 (porting the doc you wrote to Markdown).


Add collaborators to this repo

Comment below to be added as a collaborator
We will all be using this repository today. Even though it's public right now, you will not be able to make changes until you're given the correct permissions. We've automated this process via the GitHub API. Once you comment, we will add you as a collaborator.
You'll start to receive a lot of emails. (❗) You can slow these down immediately by clicking the Unwatch button at the top of the page and selecting either the Participating and @mentions or Ignore option.

Hello world

Hello there

Hello!

Hey!

Hi there!

Hi

Hello!

Hi!

No Comment!

Hi

hellooo


dts: nordic: Remove no longer needed peripheral aliases

Aliases defined for peripheral nodes in the nRF SoC definitions were
used in drivers to properly match the hardware instances.
After the drivers were converted to use the new DT API, these aliases
became needless.
Signed-off-by: Andrzej Głąbek<EMAIL_ADDRESS>

I'd just want to double check with @mbolivar-nordic , whether there is a policy to keep the aliases for a little while for backwards-compatibility, in case there are out-of-tree drivers that require them.

Zephyr v2.3 (and NCS v1.3.0) are a way for users to move to the new API with the existing aliases and legacy macros still in place, so this seems OK to me, thanks for asking. We always meant to get rid of these aliases (@erwango nicely referred to them as pleonastic) as soon as we could.

Heads up to @erwango in case there is anything similar to do for STM32.

Txs @mbolivar-nordic  I think on this front we're good on STM32 side. But maybe @MaureenHelm will be interested to do similar clean up on NXP side


Teach configuration to pass through internal load balancer DNS for certificate generation

This makes it possible to do things like issue certs properly inside the cluster
for all the addresses of a master. Previously that was only possible for AWS
clusters because the integration was AWS only.

note: this will require downstream changes

cc: @alberts


Fix for #3859

Stop ignoring .gitnore files. Add a failing test which now passes..

r? @brson
(rust_highfive has picked a reviewer for you, use r? to override)

:pushpin: Commit c21fff6 has been approved by alexcrichton

:hourglass: Testing commit c21fff6 with merge d4d039f...

:broken_heart: Test failed - status-travis

Welp, it fails on my test but I'm puzzled how it works on almost every platform but one. Possibly a timing issue with flushing to disk? Or is that just a hunch from working with too many broken parallel file systems?

Hm that does seem odd, but it looks like Travis on the PR is also failing for the same reason? It looks like it may be related to a race though as the builder failing is changing?
Perhaps some more debugging information could be added to help diagnose?

Sorry, I think I wasn't clear. It is failing on travis and the test that fails is the new code I added. I can't reproduce it locally on linux or OS X using eatmydata (which is a nice program which makes fsync into no-ops). But I do see failures with all the tests using templates if I just run them as ./target/debug/new-.... so maybe that's a thread to pull on.

Interesting!
In the face of https://github.com/rust-lang/cargo/pull/3878 I think we may want to close this though, right?

:umbrella: The latest upstream changes (presumably #3878) made this pull request unmergeable. Please resolve the merge conflicts.

With templates pulled out until we have an RFC that's passed, this PR is no longer valid.


add site : eosis.space

Hello, I'd like to add my site :  eosis.space

Closed in favor of 0057035: Used By was replaced with Showcase.
Your site was added to the showcase!


Add python 3.12 to CI and metadata

Python 3.12 is not on docker yet, thus the extremely failing tests.


[SMALLFIX] Added debugging info when create in/out streams

Merged build finished. Test PASSed.

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

@calvinjia We have been using my current style across the codebase. I feel it's better to keep it for consistency.

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

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


first version of k2-sherpa inference example

Hi everyone, we are opening PR in the hope to get the huggingface supporting k2 and sherpa as a part of the inference API.
Please comment/suggest changes and thank you for the existence :)
//attn: @csukuangfj and @danpovey

@Narsil
Could you try again?
I just uploaded config.json to
https://huggingface.co/csukuangfj/icefall-asr-librispeech-pruned-transducer-stateless3-2022-05-13/tree/main
I think @jtrmal  has been using
https://huggingface.co/jtrmal/icefall-asr-librispeech-pruned-transducer-stateless3-2022-05-13/tree/main
for testing.

HI @csukuangfj ,
I created a PR on top of your PR (so you can see and vet the modifications) to finalize the PR I think.
I was able to test out both models, which work fine ( both are the same model right ?). And are fast ! (1s on CPU).
When you merge my PR, this one should become green, we can then merge, and it's very little work for me to enable this in the API (just need to register this new library).
Thanks again for this addition !
Feel free to share too if adding this was easy/hard for you, or if you found some things that you might have done differently.
Cheers.

I think I'm done.  Thanks, everyone for your help and assistance!

It's working now !
https://huggingface.co/csukuangfj/icefall-asr-librispeech-pruned-transducer-stateless3-2022-05-13
Cheers, thanks for the great work !

thank you, too!
y.
On Thu, Jul 21, 2022 at 12:37 PM Nicolas Patry @.***>
wrote:

It's working now !
https://huggingface.co/csukuangfj/icefall-asr-librispeech-pruned-transducer-stateless3-2022-05-13
Cheers, thanks for the great work !
—
Reply to this email directly, view it on GitHub
https://github.com/huggingface/api-inference-community/pull/74#issuecomment-1191706811,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUKYX3PSM3OVYPP7B6PPD3VVF4E3ANCNFSM54EQQZ6A
.
You are receiving this because you were mentioned.Message ID:
@.***>

Very cool work! There are a couple of extra steps we can take to maximize the impact of this :) as per https://huggingface.co/docs/hub/models-adding-libraries

Modify https://github.com/huggingface/hub-docs/blob/main/js/src/lib/interfaces/Libraries.ts with a minimal code sample. That way users get a "Use in k2 sherpa" button with your own code sample. Users will allow be able to filter at left of hf.co/models. I think this item would be nice and should not be too complex.
Do you have plans for upstream support? That is, to allow users to push models to the Hub and even automatically create model cards

Thanks a lot for the contribution! :fire:

Hi, sounds good to me -- I can volunteer to figure it out the first step.
Add the second step I agree it would be good, but the changes would have to
be in k2-fsa/icefall and I'm not high enough in foodchain there
:) @danpovey? @cskuangfj?
I can volunteer to implement the changes needed there as well, if they will
agree.
y.
On Thu, Jul 21, 2022 at 12:46 PM Omar Sanseviero @.***>
wrote:

Very cool work! There are a couple of extra steps we can take to maximize
the impact of this :) as per
https://huggingface.co/docs/hub/models-adding-libraries

Modify
https://github.com/huggingface/hub-docs/blob/main/js/src/lib/interfaces/Libraries.ts
with a minimal code sample. That way users get a "Use in k2 sherpa" button
with your own code sample. Users will allow be able to filter at left of
hf.co/models. I think this item would be nice and should not be too
complex.
Do you have plans for upstream support? That is, to allow users to
push models to the Hub and even automatically create model cards

Thanks a lot for the contribution! 🔥
—
Reply to this email directly, view it on GitHub
https://github.com/huggingface/api-inference-community/pull/74#issuecomment-1191715354,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUKYXZBRBPYM74LF7HRHMTVVF5H5ANCNFSM54EQQZ6A
.
You are receiving this because you were mentioned.Message ID:
@.***>


Newlines in namerecord do not roundtrip

Source Sans Pro has DOS-style newlines (CR+LF) in its namerecord (nameID=13), but they do not roundtrip properly. It may related to #54.

Original: SourceSansPro-Regular.otf, with CR+LF
$ ttx SourceSansPro-Regular.otf: dumps to SourceSansPro-Regular.ttx, with CR+LF
$ ttx SourceSansPro-Regular.ttx: compiles to SourceSansPro-Regular#1.otf, with LF
$ ttx SourceSansPro-Regular#1.otf: dumps to SourceSansPro-Regular#1.ttx, with LF

My environment is as follows:

OS X 10.10.3
Python 2.7.9
FontTools HEAD (a54b98f445)
Source Sans Pro 2.010

Hmm. I suspect it's the fontTools.misc.xmlReader which normalises the line endings while parsing the TTX file...
Thanks for reporting this. I'll investigate.

the culprit seems to be the xml.parsers.expat module, which reads universal newlines and translates all other line endings to Unix (\n) newlines. I couldn't find any option to disable this feature...
>>> from xml.parsers.expat import ParserCreate
>>> parser = ParserCreate()
>>> parser.StartElementHandler = lambda name, attrs: print('Start element:', name, attrs)
>>> parser.EndElementHandler = lambda name: print('End element:', name)
>>> parser.CharacterDataHandler = lambda data: print('Character data:', repr(data))
>>> data = '<?xml version="1.0" encoding="UTF-8"?>\n<ttFont sfntVersion="\\x00\\x01\\x00\\x00" ttLibVersion="2.5">\n<test>\nhello\r\nworld\r\n</test>\n</ttFont>\n'
>>> parser.Parse(data)
Start element: ttFont {'sfntVersion': '\\x00\\x01\\x00\\x00', 'ttLibVersion': '2.5'}
Character data: '\n'
Start element: test {}
Character data: '\n'
Character data: 'hello'
Character data: '\n'
Character data: 'world'
Character data: '\n'
End element: test
Character data: '\n'
End element: ttFont
1

one way to fix could be converting \r (CR)  and \n (LF) to XML character entities: respectively,
 and
. Then, the expat XML parser should keep the \r\n instead of automatically converting them to \n.
I'll try to see if I can do that.

@mashabow I submitted a patch (#327) that fixes the carriage return roundtrip issue. Please confirm this solves it for you. Thanks

Thanks! Now it works well for me :+1:


Optionally configure Header Name for OAuth Resource Server

When using header asbabvehicle for the token, would be great to be able to specify a custom name to not conflict with many cli tools which use own bearer or basic auth.n. schemes.  What we try to achieve is a perimeter authorization for external users with https based cli tools.  Ever considered?

I guess those https cli tools should either speak standard OAuth 2.0 or avoid abusing the header name standardized by OAuth 2.0 "Authorization: bearer" for their own purpose. I don't see why a client would present a standard OAuth 2.0 bearer token - which is the only token accepted by mod_auth_openidc - in a non-standard way.
Moreover you should be able to separate paths protected by other means from paths protected by mod_auth_openidc to avoid header collusion.
Having said all that, there's a way to pass it in a Cookie, which is non standard, see:
https://github.com/pingidentity/mod_auth_openidc/blob/master/auth_openidc.conf#L316


Incorrect calculation of column widths when th is wider than td (affects Bootstrap)

When a column's title is wider than the data it contains, the available width is sometimes calculated incorrectly.
The reason is that when the header is cloned, it does not include the associated <tr> element, and when the column headers are subsequently appended to it they are appended directly to the <thead>.
So the source table is <thead><tr><th>, but the clone is <thead><th>.
This is a problem when the former has different padding to the latter (as in the case of Bootstrap). In this case (if the data in the cells is small), the cloned table is narrower than the original, and thus too many columns are sometimes shown.
I was hoping to provide a patch, but have failed to produce a nice solution. I have fixed it as follows:
-    var clonedHeader = $( dt.table().header().cloneNode( false ) ).appendTo( clonedTable );
+    var clonedHeader = $( dt.table().header().cloneNode( true ) ).appendTo( clonedTable );
Which kind of works, but it results in the cloned table having 2 header rows. I couldn't work out how to ensure that the cloned header keeps its associated row.
Thanks,
Andy

Finally fixed here. There was an error in how the width calculation table was being constructed that in Bootstrap meant that the cell header padding wasn't being counted, thus it was particularly noticeable there.
Thanks for flagging this up and sorry for the slow response to the issue!

Oooh, thanks for that, I'd completely forgotten! I did mean to move the issue to the other repo, but appreciate you fixing it nonetheless. Many thanks.


Update docs to have accurate default for ownership_timeout

Disclaimer: I am not entirely sure which is accurate, but I noticed a discrepancy between the docs for:

https://hexdocs.pm/db_connection/2.2.2/DBConnection.Ownership.html#module-options
and
https://hexdocs.pm/ecto_sql/Ecto.Adapters.SQL.Sandbox.html#module-owner-timed-out-because-it-owned-the-connection-for-longer-than-nms

I am relatively sure the 120_000 is more up to date due to this line: https://github.com/elixir-ecto/db_connection/blob/436101ab9678f7173d8284086536ee27ad8c05c5/lib/db_connection/ownership/proxy.ex#L8
but feel free to disregard and close if I am wrong, thanks!

You are correct.

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Unity Catalog Server Fails to Start on First Launch with Authentication Enabled

Describe the bug
Due to the current implementation:
https://github.com/unitycatalog/unitycatalog/blob/967151eb4211363375de2c56b0487f917e34067d/server/src/main/java/io/unitycatalog/server/UnityCatalogServer.java#L209
https://github.com/unitycatalog/unitycatalog/blob/967151eb4211363375de2c56b0487f917e34067d/server/src/main/java/io/unitycatalog/server/UnityCatalogServer.java#L106
https://github.com/unitycatalog/unitycatalog/blob/967151eb4211363375de2c56b0487f917e34067d/server/src/main/java/io/unitycatalog/server/auth/decorator/UnityAccessUtil.java#L41
the Unity Catalog server attempts to access MetastoreDAO without initializing it. This leads to the following error:
io.unitycatalog.server.exception.BaseException: Problem initializing authorizer: io.unitycatalog.server.exception.BaseException: No metastore found. Please check if the server is initialized properly.

To Reproduce
Steps to reproduce the behavior:

Clone the unitycatalog repository.
Configure authentication settings to be enabled.
Run the command $ bin/start-uc-server without first starting the server with authentication disabled previously.

Expected behavior
The Unity Catalog server succeeds to start the very first time with authentication enabled.
System [please complete the following information]:

OS: macOS Sonoma 14.4.1

cc: @creechy
take

take


[Feature]support batch verification before applying manifests

Is your feature request related to a problem? Please describe.

manifests of a component are often related, so we need a feature. If the feature is turned on, it is allowed to dry run in batches before applying. once there is an error, it will be returned
Describe the solution you'd like

Describe alternatives you've considered

My little thoughts,add a DryRunAdmissionHandler

Additional context

I suspect if the raise solution could completely solved the issue mentioned since the Handler here still checks resource validity in resource batch level, not component level. Another thing to mention is that this might cause additional performance overhead, so this implementation is suggested to be feature-gated. You could try to work on this and see if the solution could solve your problem.


chore: bump project dependencies to latest stable

Motivation
Just to keep our packages up-to-date this pull request does nothing more than update to the recommended library using npm outdated for the main project along with the example we provide.

I have tested on Android and looks good!


Type checking failed for master branch

Hi Nishanth,
I'm setting up the development pipeline today, but the run_checks.sh is reporting an error in the current master branch (as of July 27th 11:00PM EST), I got the following check error:
bowenli2@bender:~/ws/NeuroSym/predicators_gnn$ ./scripts/run_checks.sh
Running autoformatting.
Skipped 2 files
Autoformatting complete.
Running type checking.
tests/test_utils.py:3426: note: By default the bodies of untyped functions are not checked, consider using --check-untyped-defs  [annotation-unchecked]
tests/test_utils.py:3427: note: By default the bodies of untyped functions are not checked, consider using --check-untyped-defs  [annotation-unchecked]
predicators/datasets/generate_atom_trajs_with_vlm.py:713: error: No overload variant of "zip" matches argument types "List[List[str]]", "List[ImageOptionTrajectory]", "bool"  [call-overload]
predicators/datasets/generate_atom_trajs_with_vlm.py:713: note: Possible overload variants:
predicators/datasets/generate_atom_trajs_with_vlm.py:713: note:     def [_T_co] zip() -> zip[Any]
predicators/datasets/generate_atom_trajs_with_vlm.py:713: note:     def [_T_co, _T1] zip(Iterable[_T1], /) -> zip[Tuple[_T1]]
predicators/datasets/generate_atom_trajs_with_vlm.py:713: note:     def [_T_co, _T1, _T2] zip(Iterable[_T1], Iterable[_T2], /) -> zip[Tuple[_T1, _T2]]
predicators/datasets/generate_atom_trajs_with_vlm.py:713: note:     def [_T_co, _T1, _T2, _T3] zip(Iterable[_T1], Iterable[_T2], Iterable[_T3], /) -> zip[Tuple[_T1, _T2, _T3]]
predicators/datasets/generate_atom_trajs_with_vlm.py:713: note:     def [_T_co, _T1, _T2, _T3, _T4] zip(Iterable[_T1], Iterable[_T2], Iterable[_T3], Iterable[_T4], /) -> zip[Tuple[_T1, _T2, _T3, _T4]]
predicators/datasets/generate_atom_trajs_with_vlm.py:713: note:     def [_T_co, _T1, _T2, _T3, _T4, _T5] zip(Iterable[_T1], Iterable[_T2], Iterable[_T3], Iterable[_T4], Iterable[_T5], /) -> zip[Tuple[_T1, _T2, _T3, _T4, _T5]]
predicators/datasets/generate_atom_trajs_with_vlm.py:713: note:     def [_T_co] zip(Iterable[Any], Iterable[Any], Iterable[Any], Iterable[Any], Iterable[Any], Iterable[Any], /, *iterables: Iterable[Any]) -> zip[Tuple[Any, ...]]
tests/approaches/test_oracle_approach.py:446: note: By default the bodies of untyped functions are not checked, consider using --check-untyped-defs  [annotation-unchecked]
tests/approaches/test_oracle_approach.py:447: note: By default the bodies of untyped functions are not checked, consider using --check-untyped-defs  [annotation-unchecked]
predicators/approaches/grammar_search_invention_approach.py:1010: error: Unsupported left operand type for | ("Dict[Predicate, float]")  [operator]
tests/explorers/test_active_sampler_explorer.py:34: note: By default the bodies of untyped functions are not checked, consider using --check-untyped-defs  [annotation-unchecked]
tests/explorers/test_active_sampler_explorer.py:182: note: By default the bodies of untyped functions are not checked, consider using --check-untyped-defs  [annotation-unchecked]
Found 2 errors in 2 files (checked 418 source files)
Type checking failed! Terminating check script early.

What's the case on your end? I think I'll need to make sure the current master branch is passing checks before proceeding with integrating my code.
Thanks.

it works for me, maybe try updating all versions of the developer dependencies? Also try just running mypy . --config-file mypy.ini and see if that works?

I see. Tried to re-create a conda env on my desktop and it worked. Thanks.


rosdep init if rosdep not initialized

$ ~/as2_ws$ rosdep update
ERROR: no sources directory exists on the system meaning rosdep has not yet been initialized.

Please initialize your rosdep with

	sudo rosdep init

If rosdep init sources are already initialized, a warning shows up:
$ sudo rosdep init
ERROR: default sources list file already exists:
	/etc/ros/rosdep/sources.list.d/20-default.list
Please delete if you wish to re-initialize

Done in #36


PyScript keep track of used attributes of global vars

PyScript keeps track of what variables are not defined by the JS itself, i.e. globals. The JSModule used this information to determine what variables from the Python module should be transpiled into the JS module. If PyScript would also keep track (and expose) what variables of a variable are used, users would not have to import the names from modules in order to use them.
# ------ Now
from bar import foo

...
    class JS:
         def some_method(self):
              foo()

# ------ What would be nice
import bar
...
    class JS:
         def some_method(self):
              bar.foo()

Done in #408


Link your GitLab account unclear error message

If the user sets up the Gitlab auth app incorrectly, and then try to connect to GitLab via Profile -> Integrations -> "Link your Gitlab account", they get an unhelpful error message "Not Found The requested resource was not found on this server."
This may be fixed when we enable native Gitlab auth.
Loom here:
https://www.loom.com/share/f78d7727a2b8444aa05341a02f6a0d1a

Fixed


Inner function error tests

working pr for https://github.com/a-synchronous/rubico/issues/25

Shoulda closed this ages ago. I'll probably open another pr to address this somewhere further down the line.


source control not showing changes or detecting for certain folders

VS Code Version: latest
OS Version: windows 10 pro

Steps to Reproduce:

make changes to files in vendor folder for php compoer
throw your computer accross the room because microsft decided it would be best to make you spend 3 hours for still nothign to work, realy ruined my day, relisign all my commits where broken, microsoft im realy disapointed

Does this issue occur when all extensions are disabled?: YES

i even did a whole fresh reinstall with deleting all the roaming data too


Add an 'Explain' Button in the Privates Section to have separate frame explain the specifics of the Private

Since I keep getting questions on how the Various Privates work, and especially the Benefits, I feel an Explain Button (or just button with a big Question Mark (?) be added that will present a separate dialog that will show in gory detail all of the written rules, and how each of the benefits work, for how long (ie. Private close in Phase X, when a Y Train is purchased)

Info Button added to each Certificate to provide this information.


⚠ MeeseOS has degraded performance

In 201163e, MeeseOS (https://aaronmeese.com) experienced degraded performance:

HTTP code: 200
Response time: 3385 ms

Resolved: MeeseOS performance has improved in 085e1d8.


Add Bank module #48

@David-Sod could you please resolve conflicts here.

@luis-stakeordie Just solved conflicts.


install cellpose on ubiuntu 18.04

I am facing issues with installaing cellpose on default https://mxnet.apache.org/get_started/validate_mxnet#docker-with-cpu container
Step 1/5 : FROM mxnet/python:latest
latest: Pulling from mxnet/python
0a01a72a686c: Pull complete
cc899a5544da: Pull complete
19197c550755: Pull complete
716d454e56b6: Pull complete
f001a21ec4c9: Pull complete
735c6460a04c: Pull complete
5ef019fbc840: Pull complete
82cb81df15a8: Pull complete
Digest: sha256:443180535a8cd3d01ce9de89983535c1b86dd1bbfdef1e5506477a3b0dcc0dec
Status: Downloaded newer image for mxnet/python:latest
 ---> e1a3a7d79c7e
Step 2/5 : RUN useradd -u 1000 -ms /bin/bash user
 ---> Running in 4529ebd0483b
Removing intermediate container 4529ebd0483b
 ---> c4f68551ffa6
Step 3/5 : RUN pip install cellpose
 ---> Running in 4dc86c5b1a28
Collecting cellpose
  Downloading cellpose-<IP_ADDRESS>-py3-none-any.whl (92 kB)
Collecting Pillow<=7.0.0
  Downloading Pillow-7.0.0-cp35-cp35m-manylinux1_x86_64.whl (2.1 MB)
Collecting scipy
  Downloading scipy-1.4.1-cp35-cp35m-manylinux1_x86_64.whl (26.0 MB)
Requirement already satisfied: numpy in /usr/local/lib/python3.5/dist-packages (from cellpose) (1.18.1)
Collecting scikit-image
  Downloading scikit_image-0.15.0-cp35-cp35m-manylinux1_x86_64.whl (26.2 MB)
Collecting numba
  Downloading numba-0.47.0-cp35-cp35m-manylinux1_x86_64.whl (3.7 MB)
Collecting tqdm
  Downloading tqdm-4.46.0-py2.py3-none-any.whl (63 kB)
Collecting opencv-python
  Downloading opencv_python-<IP_ADDRESS>-cp35-cp35m-manylinux1_x86_64.whl (28.2 MB)
Collecting natsort
  Downloading natsort-7.0.1-py3-none-any.whl (33 kB)
Collecting mxnet-mkl
  Downloading mxnet_mkl-1.6.0-py2.py3-none-manylinux1_x86_64.whl (76.7 MB)
Collecting matplotlib
  Downloading matplotlib-3.0.3-cp35-cp35m-manylinux1_x86_64.whl (13.0 MB)
Collecting imageio>=2.0.1
  Downloading imageio-2.8.0-py3-none-any.whl (3.3 MB)
Collecting PyWavelets>=0.4.0
  Downloading PyWavelets-1.1.1-cp35-cp35m-manylinux1_x86_64.whl (4.4 MB)
Collecting networkx>=2.0
  Downloading networkx-2.4-py3-none-any.whl (1.6 MB)
Requirement already satisfied: setuptools in /usr/local/lib/python3.5/dist-packages (from numba->cellpose) (45.2.0)
Collecting llvmlite>=0.31.0dev0
  Downloading llvmlite-0.32.1.tar.gz (104 kB)
Requirement already satisfied: requests<3,>=2.20.0 in /usr/local/lib/python3.5/dist-packages (from mxnet-mkl->cellpose) (2.22.0)
Requirement already satisfied: graphviz<0.9.0,>=0.8.1 in /usr/local/lib/python3.5/dist-packages (from mxnet-mkl->cellpose) (0.8.4)
Collecting kiwisolver>=1.0.1
  Downloading kiwisolver-1.1.0-cp35-cp35m-manylinux1_x86_64.whl (90 kB)
Collecting python-dateutil>=2.1
  Downloading python_dateutil-2.8.1-py2.py3-none-any.whl (227 kB)
Collecting cycler>=0.10
  Downloading cycler-0.10.0-py2.py3-none-any.whl (6.5 kB)
Collecting pyparsing!=2.0.4,!=2.1.2,!=2.1.6,>=2.0.1
  Downloading pyparsing-2.4.7-py2.py3-none-any.whl (67 kB)
Collecting decorator>=4.3.0
  Downloading decorator-4.4.2-py2.py3-none-any.whl (9.2 kB)
Requirement already satisfied: certifi>=2017.4.17 in /usr/local/lib/python3.5/dist-packages (from requests<3,>=2.20.0->mxnet-mkl->cellpose) (2019.11.28)
Requirement already satisfied: idna<2.9,>=2.5 in /usr/local/lib/python3.5/dist-packages (from requests<3,>=2.20.0->mxnet-mkl->cellpose) (2.8)
Requirement already satisfied: urllib3!=1.25.0,!=1.25.1,<1.26,>=1.21.1 in /usr/local/lib/python3.5/dist-packages (from requests<3,>=2.20.0->mxnet-mkl->cellpose) (1.25.8)
Requirement already satisfied: chardet<3.1.0,>=3.0.2 in /usr/local/lib/python3.5/dist-packages (from requests<3,>=2.20.0->mxnet-mkl->cellpose) (3.0.4)
Collecting six>=1.5
  Downloading six-1.15.0-py2.py3-none-any.whl (10 kB)
Building wheels for collected packages: llvmlite
  Building wheel for llvmlite (setup.py): started
  Building wheel for llvmlite (setup.py): finished with status 'error'
  ERROR: Command errored out with exit status 1:
   command: /usr/bin/python3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-42_hpshv/llvmlite/setup.py'"'"'; __file__='"'"'/tmp/pip-install-42_hpshv/llvmlite/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' bdist_wheel -d /tmp/pip-wheel-r7xptxbo
       cwd: /tmp/pip-install-42_hpshv/llvmlite/
  Complete output (26 lines):
  running bdist_wheel
  /usr/bin/python3 /tmp/pip-install-42_hpshv/llvmlite/ffi/build.py
  LLVM version... Traceback (most recent call last):
    File "/tmp/pip-install-42_hpshv/llvmlite/ffi/build.py", line 106, in main_posix
      out = subprocess.check_output([llvm_config, '--version'])
    File "/usr/lib/python3.5/subprocess.py", line 626, in check_output
      **kwargs).stdout
    File "/usr/lib/python3.5/subprocess.py", line 693, in run
      with Popen(*popenargs, **kwargs) as process:
    File "/usr/lib/python3.5/subprocess.py", line 947, in __init__
      restore_signals, start_new_session)
    File "/usr/lib/python3.5/subprocess.py", line 1551, in _execute_child
      raise child_exception_type(errno_num, err_msg)
  FileNotFoundError: [Errno 2] No such file or directory: 'llvm-config'

  During handling of the above exception, another exception occurred:

  Traceback (most recent call last):
    File "/tmp/pip-install-42_hpshv/llvmlite/ffi/build.py", line 192, in <module>
      main()
    File "/tmp/pip-install-42_hpshv/llvmlite/ffi/build.py", line 182, in main
      main_posix('linux', '.so')
    File "/tmp/pip-install-42_hpshv/llvmlite/ffi/build.py", line 109, in main_posix
      "to the path for llvm-config" % (llvm_config,))
  RuntimeError: llvm-config failed executing, please point LLVM_CONFIG to the path for llvm-config
  error: command '/usr/bin/python3' failed with exit status 1
  ----------------------------------------
  ERROR: Failed building wheel for llvmlite
  Running setup.py clean for llvmlite
Failed to build llvmlite
Installing collected packages: Pillow, scipy, imageio, PyWavelets, kiwisolver, six, python-dateutil, cycler, pyparsing, matplotlib, decorator, networkx, scikit-image, llvmlite, numba, tqdm, opencv-python, natsort, mxnet-mkl, cellpose
    Running setup.py install for llvmlite: started
    Running setup.py install for llvmlite: finished with status 'error'
    ERROR: Command errored out with exit status 1:
     command: /usr/bin/python3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-42_hpshv/llvmlite/setup.py'"'"'; __file__='"'"'/tmp/pip-install-42_hpshv/llvmlite/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record /tmp/pip-record-wybepq31/install-record.txt --single-version-externally-managed --compile --install-headers /usr/local/include/python3.5/llvmlite
         cwd: /tmp/pip-install-42_hpshv/llvmlite/
    Complete output (29 lines):
    running install
    running build
    got version from file /tmp/pip-install-42_hpshv/llvmlite/llvmlite/_version.py {'full': 'aa11b129c0b55973067422397821ae6d44fa5e70', 'version': '0.32.1'}
    running build_ext
    /usr/bin/python3 /tmp/pip-install-42_hpshv/llvmlite/ffi/build.py
    LLVM version... Traceback (most recent call last):
      File "/tmp/pip-install-42_hpshv/llvmlite/ffi/build.py", line 106, in main_posix
        out = subprocess.check_output([llvm_config, '--version'])
      File "/usr/lib/python3.5/subprocess.py", line 626, in check_output
        **kwargs).stdout
      File "/usr/lib/python3.5/subprocess.py", line 693, in run
        with Popen(*popenargs, **kwargs) as process:
      File "/usr/lib/python3.5/subprocess.py", line 947, in __init__
        restore_signals, start_new_session)
      File "/usr/lib/python3.5/subprocess.py", line 1551, in _execute_child
        raise child_exception_type(errno_num, err_msg)
    FileNotFoundError: [Errno 2] No such file or directory: 'llvm-config'

    During handling of the above exception, another exception occurred:

    Traceback (most recent call last):
      File "/tmp/pip-install-42_hpshv/llvmlite/ffi/build.py", line 192, in <module>
        main()
      File "/tmp/pip-install-42_hpshv/llvmlite/ffi/build.py", line 182, in main
        main_posix('linux', '.so')
      File "/tmp/pip-install-42_hpshv/llvmlite/ffi/build.py", line 109, in main_posix
        "to the path for llvm-config" % (llvm_config,))
    RuntimeError: llvm-config failed executing, please point LLVM_CONFIG to the path for llvm-config
    error: command '/usr/bin/python3' failed with exit status 1
    ----------------------------------------
ERROR: Command errored out with exit status 1: /usr/bin/python3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-42_hpshv/llvmlite/setup.py'"'"'; __file__='"'"'/tmp/pip-install-42_hpshv/llvmlite/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record /tmp/pip-record-wybepq31/install-record.txt --single-version-externally-managed --compile --install-headers /usr/local/include/python3.5/llvmlite Check the logs for full command output.
WARNING: You are using pip version 20.0.2; however, version 20.1.1 is available.
You should consider upgrading via the '/usr/bin/python3 -m pip install --upgrade pip' command.
The command '/bin/sh -c pip install cellpose' returned a non-zero code: 1

Do you have requrement.txt with appropriate wheels that installer won't compine dependences? It would help me to reduce size of the container

last version of llvmlite provide wheels only for python 3.6+. Use newer python or install pip install llvmlite==0.31.0

I recommend using conda instead of pip if running into install issues using our environment.yml file. if that doesn't work please let me know


Add environmental based token

It's really restricting to have to use the file based token storage. It would be nice and easier to debug to use environmental variables instead.

I can take a stab at this!

That would be great.

I'm finished up for this, but I need to do some testing, and I ran out of steam for today! I'll definitely get the PR in sometime tomorrow. Stay tuned!

hey @exp0nge I just opened a PR here --> https://github.com/Kaggle/kaggle-api/pull/57 note that we definitely need tests before this goes through, and I think the lead developer might be on holiday, so TLDR stay tuned!

Released as 1.4.6.  Closing this out - let me know if there are any problems!

@rysteboe can I help with docstrings? Have you decided on a format?

@vsoch what you were doing is fine.  Additional documentation is not high on my to do list atm, so anything you do in that area is appreciated.

okay cool! I'm working on some boring stuff this morning, but will want to contribute this soon :)


osActivate support

Do you support OS Activate call and OS Verify?
gNOI OS Activate RPC
gNOI OS Activate RPC
Thanks,
Asaf.

Not yet, but it's a unary RPC so pretty easy to add. Would you like to give it a go and send us a pull request?

Sure:)
I’ll try some next week
Get Outlook for iOShttps://aka.ms/o0ukef

From: Greg Dennis @.>
Sent: Thursday, February 15, 2024 8:32:06 PM
To: openconfig/gnoigo @.>
Cc: Asaf Sonis @.>; Author @.>
Subject: Re: [openconfig/gnoigo] osActivate support (Issue #23)
Not yet, but it's a unary RPC so pretty easy to add. Would you like to give it a go and send us a pull request?
—
Reply to this email directly, view it on GitHubhttps://github.com/openconfig/gnoigo/issues/23#issuecomment-1946889830, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A7VNKY22YFCHAHGT2AS2KGLYTZICNAVCNFSM6AAAAABDKVKASSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBWHA4DSOBTGA.
You are receiving this because you authored the thread.Message ID: @.***>


[FLINK-18878][python] Support dependency management for Python Stream…

What is the purpose of the change
Add dependency management for StreamExecutionEnvironment when Users need to specified third party dependencies in their DataStream Job. We purpose to add the following interfaces forPython StreamExecutionEnvironment:

add_python_file()
set_python_requirements()
add_python_archive()
set_python_executable()

And user might apply those interfaces arbitrarily, we must configure those dependencies to operators that depend on them correctly before executing the job.
Further more, some operation might consist of multiple operations,  take key_by() for instance:
>>> source_stream = env.from_collection(...)
>>> keyed_stream = upstream.key_by(...)
>>> mapped_stream = keyed_stream.map(...)

The implementation of key_by() is consist of a _stream_key_by_map operator and a _keyed_stream_values_map operator.
we need to make sure the _stream_key_by_map operator is chained with the source_stream operator,   the _keyed_stream_values_map operator chained with mapped_stream operator that they have the same parallelism, slot sharing group, co-location group and the ship_strategy should be FORWARD.
All this task should be accomplished before submitting the job by a PythonConfigUtil.
Brief change log
Add the following interfaces forPython StreamExecutionEnvironment:

add_python_file()
set_python_requirements()
add_python_archive()
set_python_executable()

Add util method getEnvConfigWithDependencies() to get StreamExecutionEnvironment configuration with dependencies.
Add a util method generateStreamGraphWithDependencies() to generate a StreamGraph with all operators have dependencies configured.
Verifying this change
This change has test cases covered as followed:

test_add_python_file()
test_set_requirements_without_cached_directory()
test_set_requirements_with_cached_directory()
test_add_python_archive()
test_set_stream_env()
test_generate_stream_graph_with_dependencies()

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): ( no)
The public API, i.e., is any changed class annotated with @Public(Evolving): ( no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? ( not documented)

@hequn8128 Thank you for your comments, I have updated the PR according to your suggestions, please have a look at it.


Hanging parentheses

There is an hanging parentheses at the end of this section:
Using the kit: Report your issue at https://aka.ms/Creator-Kit-Issues. Though this kit isn't supported by Microsoft Support, you can reach out with support requests for any related, underlying platform and feature problems.)

Thanks for the notification. We'll get this updated.


Dao Core v1. Staking, Withdraw

Withdraw dialog
https://zpl.io/agmMJZO

Withdraw confirmation
https://zpl.io/bJ8YJ7W

Error state of input field is here https://zpl.io/V0rGAXm

Make withdraw confirmation as in the design

Please display fee amount in second currency as "< 1 cent"

the transaction confirmation design is implemented inside the wallet
done

As for 2:

warning string also would be added if user is about to spend money.
right button will be always "confirm". If user receives money - blue, if spends - magenta.
case of multiple assets is being designed now

Please test confirmation dialogs on the latest build.

1, 3 - checked
2 - Withdraw confirmation dialog will be implemented and checked in #717 Dapps tx confirmations


Remove globbing

This PR implements the (pending!) proposal to remove globbing at https://github.com/solid/solid-spec/issues/145
It bypasses clarifying the definition of globbing (https://github.com/solid/solid-spec/pull/148) by just removing it altogether, given that currently seems to be what the majority wants.
Just putting it out here as a possible option, no rush.

Client-side globbing alternative implemented in https://github.com/solid/ldp-glob; live demo at https://solid.github.io/ldp-glob/demo.html?https://drive.verborgh.org/public/

This would be for version 0.8 of the spec then. We need to discuss the timeline for that. I agree with Ruben about removal of globbing in the next spec version, but I agree with Melvin about moving slowly and not breaking things every few weeks. About timeline, my gut says let's do a next spec version 0.8 in December, and not rock the boat before that. But let's discuss that in the next weekly meeting!

Discussed offline with @melvincarvalho; this should be on hold until he and @timbl can discuss.
@michielbdejong Yes, but we should avoid that people start implementing globbing if it is going to be removed, so a note or label in the spec would be useful. And of course https://github.com/solid/solid-spec/pull/148 which aligns the spec with the actual situation.

I'm using globbing to retrieve hundreds to thousands of metafiles https://github.com/linonetwo/solid-tiddlywiki-syncadaptor/issues/4#issuecomment-491118236
I can't afford to do this client side, because there will be hundreds to thousands of wiki pages in that container, so there will be a huge amount of client-side fetch running concurrently.

@linonetwo have you run benchmarks comparing globing approach to https://github.com/solid/ldp-glob with HTTP/2 enabled?

@elf-pavlik Do I need to start solid-server as a library, and use spdy to enable HTTP2 in my server?

I think you could also just run it behind nginx and enable HTTP/2 in your nginx config

I'm using globbing to retrieve hundreds to thousands of metafiles linonetwo/solid-tiddlywiki-syncadaptor#4 (comment)

Thanks for sharing this use case, it's good to know what's out there.
May I ask for a bit more detail here?
What you seem to be using is .meta.*; however, this is a kind of pattern that is not supported across Solid servers (see #147). The kind of globbing that is currently in use, is only /*, so all files in a directory.
How does this affect your use case? (For instance, could you put your files in a meta subfolder?)
Another question I have is about the necessity of this design choice: could you give us some insights into the motivation for splitting data across this many files? (There might very well exist a more generic motivation, so eager to learn about it.)
A concern I do have is that, even for the server, thousands of files would turn this into a very expensive request, which ties into my DDOS worries regarding globbing (#145).

I can't afford to do this client side, because there will be hundreds to thousands of wiki pages in that container, so there will be a huge amount of client-side fetch running concurrently.

Point taken—except for "concurrently"; the browser will take tare of this, and with HTTP/2 there should only be a very limited overhead. Emphasis on should, because the per-request cost of NSS is currently too high, so it will be significantly slower with the magnitude of files you are naming.
That said, whatever design decision we make, having thousands of files in a single folder is bound to cause trouble one way or another. Not just for Solid, but for *nix or Windows systems too. So I believe the information architecture here can likely be more optimal. But please feel free to further expand on your use case, so we understand where the scale comes from.

Well, I've reconsidered it:

I won't use xxx.meta to store metadata generated by user and my application, because https://github.com/solid/solid-spec/issues/168 can't GET and DELETE xxx.meta
I will use SPARQL to update and read a single index.metafile.ttl, and create all files using Link <http://www.w3.org/ns/ldp#Resource>; rel="type", <index.metafile.ttl>; rel="describedby"

The reason I choose to use globbing was "it's the easier way to get my POC app working, and document is simple and certain", but actually I can use SPARQL instead, while I'm not pretty sure if it will work.
I draw a picture while I'was considering this:


dirty tracking

@buren thanks for the thumbs up feedback. This is looking pretty awesome to me and will be merged to master presently, but if you take it for a spin and find any problems or challenges (or if you don't!), feedback is welcome!


Trait environment

This adds the environment, i.e. the set of where clauses in scope, when solving trait goals. That means that e.g. in
fn foo<T: SomeTrait>(t: T) {}

, we are able to complete methods of SomeTrait on the t. This affects the trait APIs quite a bit (since every method that needs to be able to solve for some trait needs to get this environment somehow), so I thought I'd do it rather sooner than later ;)

Awesome! I have some minor naming questions, but they can be sorted out separately, if at all
bors r+


UltraDNS company name change reflected in document links

UltraDNS company has changed. Updating document links to reflect new name

/cc https://vercara.com/blog/anatomy-of-a-brand-transformation-our-journey-to-vercara, linked from https://portal.ultradns.com/login


Upgrade versioner plugin

..to a version which is still available in Maven Central.
Fixes #2594

Looks like there's still an issue with the build, that version doesn't seem to have been found either:
> Could not resolve all artifacts for configuration ':classpath'.
   > Could not find com.pascalwelsch.gitversioner:gitversioner:0.6.3.
     Searched in the following locations:
       - https://dl.google.com/dl/android/maven2/com/pascalwelsch/gitversioner/gitversioner/0.6.3/gitversioner-0.6.3.pom
       - file:/root/.m2/repository/com/pascalwelsch/gitversioner/gitversioner/0.6.3/gitversioner-0.6.3.pom
       - https://maven.google.com/com/pascalwelsch/gitversioner/gitversioner/0.6.3/gitversioner-0.6.3.pom
       - https://plugins.gradle.org/m2/com/pascalwelsch/gitversioner/gitversioner/0.6.3/gitversioner-0.6.3.pom
       - https://repo.maven.apache.org/maven2/com/pascalwelsch/gitversioner/gitversioner/0.6.3/gitversioner-0.6.3.pom
     Required by:
         project :

Yeah, I think the group ID has changed to de.sharpmind as well. Trying that now.

Working now!


Unknown filter error with delete_between and after filters

Cookbook version
2.3.2
Chef-client version
13.8.5
Platform Details
Oracle Linux Server 7.4
Scenario:
I am trying to remove multiple lines from a file and then add in a particular set of lines,
Steps to Reproduce:
Pull down line cookbook as dependency and inside test cookbook insert the following into a recipe:
text_lines =
'
<security-domain name="test-domain" cache-type="default">

<login-module code="ConfiguredIdentity" flag="required">
<module-option name="username" value=""/>
<module-option name="password" value=""/>
<module-option name="principal" value=""/>

'
match_pattern = /.<subsystem xmlns="urn:jboss:domain:security:1.2">./
filter_lines 'test.xml xxx-xxx-xxx' do
path "#{conf_path}/#{src_filepath}/#{config_file}"
filters(
[
{ delete_between: [/./, /.</security-domains>/] },
{ after: [match_pattern, text_lines, :first] },
]
)
end
Expected Result:
I expected all lines in the xml to be deleted between  and </security-domains> and then have the lines added in from above.
Actual Result:
kitchen converge fails with the following error:
====================================================================
Error executing action edit on resource 'filter_lines[test.xml xxx-xxx-xxx]'
====================================================================
       ArgumentError
       -------------
       Unknown filter, delete_between, specified

       Cookbook Trace:
       ---------------
       /tmp/kitchen/cache/cookbooks/line/libraries/helper.rb:53:in `filter_method'
       /tmp/kitchen/cache/cookbooks/line/libraries/helper.rb:67:in `invoke_filter'
       /tmp/kitchen/cache/cookbooks/line/libraries/helper.rb:30:in `block in apply_filter'
       /tmp/kitchen/cache/cookbooks/line/libraries/helper.rb:29:in `each'
       /tmp/kitchen/cache/cookbooks/line/libraries/helper.rb:29:in `apply_filter'
       /tmp/kitchen/cache/cookbooks/line/resources/filter_lines.rb:55:in `block (2 levels) in class_from_file'
       /tmp/kitchen/cache/cookbooks/line/resources/filter_lines.rb:54:in `each'
       /tmp/kitchen/cache/cookbooks/line/resources/filter_lines.rb:54:in `block in class_from_file'

       Resource Declaration:
       ---------------------
       suppressed sensitive resource output

       Compiled Resource:
       ------------------
       suppressed sensitive resource output

       System Info:
       ------------
       chef_version=13.8.5
       platform=oracle
       platform_version=6.10
       ruby=ruby 2.4.3p205 (2017-12-14 revision 61247) [x86_64-linux]
       program_name=chef-solo worker: ppid=10532;start=15:44:57;
       executable=/opt/chef/bin/chef-solo

Thank you for the report.  I'll take a look.

This looks like a problem with the resource and the helper method using the same name.  The code works on chef 14, fails on 13 and 15.  Unfortunately the code was developed on 14.  A fix is on the way.

Should be fixed in 2.3.3


Auto-import in JSDoc always uses ts behaviour

Vue - Official extension or vue-tsc version
v2.0.26
VSCode version
1.91.0
Vue version
latest
TypeScript version
5.5.2
System Info
No response
Steps to reproduce
In reproduction repo:

Open exampleLangJs.vue
Trigger import suggestions on MyType JSDoc comment

Accept first entry

What is expected?
Imports from JSDoc in <script lang="js">/ <script> work like in regular js file:
You can check this in reproduction repo:

Without @import comment

With @import comment

What is actually happening?
Whole d.ts/ts file is imported

Link to minimal reproduction
https://github.com/ilanaya/volar-starter
Any additional comments?
No response

I think this is caused by TS getNewImportFixes function identifying the .js extension instead of LanguageServiceHost.getScriptKind.
https://github.com/microsoft/TypeScript/blob/903e82b368c8328dba95b70b21415e7a48339857/src/services/codefixes/importFixes.ts#L1168


Add checkboxes for draft, open, and merged PRs

Hi @colineckert — thanks for your work on this theme! This is a totally speculative PR, and you should feel free to close it (e.g. on the grounds that it adds checkbox styles unique to this theme).
Changes
Adds three new checkbox styles for typical GitHub workflows:

- [D] indicates a Draft pull request.
- [P] indicates an open Pull request. Maybe you'd prefer O?
- [M] indicates a Merged pull request.

Uses MIT-licensed icons ("Octicons") from GitHub Primer:

Unfortunately, these aren't captured in the README.md screenshot yet.
Motivation
I use Obsidian at work, and work involves submitting PRs. Draft and open PRs are both partially completed tasks, but they're meaningfully different workflow states (blocked on me, blocked on a reviewer).
Using - [/] obscures that distinction!
I realize exhaustively adding checkboxes — for every workflow, or even just for developer workflows — would clutter this library. Set the limit where you see fit!

I can also distribute this as a snippet. Feel free to close this PR!

Closing: stale.

@lukasschwab thank you for opening this PR! sry for the delay, i've been traveling lately but i really want this so am reopening this PR, and after testing will merge. thanks again!

@colineckert oh, sweet! No worries about the delay, glad this turned out to be useful 😁

Hi all, wonderful addition to the theme, I just raised this feature request https://github.com/colineckert/obsidian-things/issues/140 before noticing this PR and that this is a very recent addition. Would you mind making this so that the [M] checkbox text is grayed out similar to the [x] checkbox?


optimize: limit redis mode all sessions

[ ] I have registered the PR changes.

Ⅰ. Describe what this PR did
limit the number of queries in Redis storage mode
Ⅱ. Does this pull request fix one issue?

fixes #4429
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #4431 (b53c955) into develop (4c88984) will decrease coverage by 0.18%.
The diff coverage is 60.00%.

@@              Coverage Diff              @@
##             develop    #4431      +/-   ##
=============================================
- Coverage      48.67%   48.49%   -0.19%
+ Complexity      3973     3965       -8
=============================================
  Files            733      733
  Lines          25155    25158       +3
  Branches        3085     3085
=============================================
- Hits           12245    12201      -44
- Misses         11596    11639      +43
- Partials        1314     1318       +4

Impacted Files
Coverage Δ

...ava/io/seata/core/constants/RedisKeyConstants.java
0.00% <ø> (ø)

...rage/db/store/DataBaseTransactionStoreManager.java
59.03% <ø> (ø)

...rage/redis/store/RedisTransactionStoreManager.java
59.40% <60.00%> (+0.07%)
:arrow_up:

...java/io/seata/server/metrics/MeterIdConstants.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ava/io/seata/server/metrics/MetricsSubscriber.java
18.98% <0.00%> (-44.31%)
:arrow_down:

...n/src/main/java/io/seata/common/util/IdWorker.java
77.08% <0.00%> (-6.25%)
:arrow_down:

...r/console/impl/file/GlobalLockFileServiceImpl.java
94.87% <0.00%> (-2.57%)
:arrow_down:

...in/java/io/seata/server/session/SessionHelper.java
72.41% <0.00%> (-2.30%)
:arrow_down:

...in/java/io/seata/server/session/SessionHolder.java
50.00% <0.00%> (-1.45%)
:arrow_down:

...tasource/sql/struct/cache/MysqlTableMetaCache.java
81.60% <0.00%> (+1.14%)
:arrow_up:

... and 3 more

Does it conflict with the GlobalSessionParam pageNum parameter?

GlobalSessionParam calls readSession with the xid conditional argument, so there is no problem.


Is very slow on Windows

Perhaps causes #281 and also affects Yukon usability on Windows devices.

@silverv we should try increasing the resolution of the system timer as done in the PyCyphal test suite:
https://github.com/OpenCyphal/pycyphal/blob/029fdb5b5a158372103edf436298bc3559377dd8/tests/conftest.py#L63
More on this in the Windows API docs

It would probably make sense to go for a lower than 6 ms value, ideally a lower than millisecond value

I don't understand why setting the delay even even higher helped the tests

Not sure what delay you are referring to. There are no delays here, the referenced API function changes the system timer resolution. It should be possible to go down to 1 ms, I would try that.


Failed to encode as JSON: Null key for a Map not allowed in JSON

Hi,
I think that there is an error with the lab content as when we launch the docker container ("docker run -p 8081:8080 --rm --name quote-generator vertx-microservice-workshop/quote-generator"), then the next error appears when we access the following address  : http://dockerhost:8081
io.vertx.core.json.EncodeException: Failed to encode as JSON: Null key for a Map not allowed in JSON (use a converting NullKeySerializer?)
        at io.vertx.core.json.Json.encodePrettily(Json.java:73)
        at io.vertx.workshop.quote.RestQuoteAPIVerticle.lambda$start$1(RestQuoteAPIVerticle.java:49)
        at io.vertx.core.http.impl.ServerConnection.handleRequest(ServerConnection.java:286)
        at io.vertx.core.http.impl.ServerConnection.processMessage(ServerConnection.java:412)
        at io.vertx.core.http.impl.ServerConnection.handleMessage(ServerConnection.java:139)
        at io.vertx.core.http.impl.HttpServerImpl$ServerHandler.lambda$createConnAndHandle$1(HttpServerImpl.java:709)
        at io.vertx.core.impl.ContextImpl.lambda$wrapTask$3(ContextImpl.java:357)
        at io.vertx.core.impl.ContextImpl.executeFromIO(ContextImpl.java:234)

However, the config.json file is well packaged and available within the docker contener
docker exec -it c6649fa1f319 /bin/bash
root@c6649fa1f319:/opt# ls -la
total 42736
drwxrwxrwx  3 root root     4096 May 27 16:26 .
drwxr-xr-x 56 root root     4096 May 27 16:26 ..
drwxr-xr-x  3 root root     4096 May 27 16:26 .vertx
-rwxrwxrwx  1 root root      410 May 27 06:56 config.json
-rwxrwxrwx  1 root root 43742687 May 27 16:25 quote-generator-1.0-SNAPSHOT-fat.jar

Resolved after replacing ("string") key with ("name") here
 vertx.eventBus().<JsonObject>consumer(GeneratorConfigVerticle.ADDRESS, message -> {
        JsonObject quote = message.body();
        quotes.put(quote.getString("name"), quote);
    });
```

@cmoulliard The underlying APIs have changed a bit since I wrote this workshop, so you may find some small issue. Let me know !
Everything is going to be stabilized soon (with vert.x 3.3.0 release on its way).
BTW, we plan to remove Docker from the lab to focus on vert.x only.


Move component option to top-level

It should be a top-level option:
newsletter: {
  component: true,
  // mailchimp | revue | buttondown
  <YOUR_NEWSLETTER_PROVIDER>: {
    // options like apiKey
  }
}

Also, you could support the provider prop in the component to use one of the providers defined (default to the first one defined).

Hey @Atinux
Thanks for the recomendation.
The component option as the top level seems like a very good idea!
In terms of provider in components, I dont think it is possible as the provider is used to define the server middleware that should be used.


this fix when you try to compile with rebar/relx

The explanation, that we need {modules,[]} , even empty.
http://lookonmyworks.co.uk/2014/06/16/application-metadata-file-exists-but-is-malformed/

The explanation, that we need {modules,[]} , even empty.
http://lookonmyworks.co.uk/2014/06/16/application-metadata-file-exists-but-is-malformed/

Could you squash the two commits together please?

Closing all tickets/PRs. Thanks!


adapting values for GCP infrastructure, not all checks are covered by…

… the vars file and they need to be adapted in addition

@arbulu89  Some checks were not successful
Is there something I can fix?

implemented in https://github.com/trento-project/runner/pull/34


In ResourceWithPermissions override name property if resource prop is explicitly passed through

Just realised that this might be redundant since we can override the label name already via the Resource component.


Can't compile any of the examples

Bevy version
v0.9.1
v0.9.0
v0.8.1
v0.8.0
[Optional] Relevant system information
rustc 1.66.1
What you did
git checkout ${tag_above}
cargo run --example hello_world

What went wrong
Every build, regardless of git tag failed to resolve bytemuck_derive::ByteEq
$ cargo run --example hello_world
   Compiling proc-macro2 v1.0.50
[...]
error[E0432]: unresolved imports `bytemuck_derive::ByteEq`, `bytemuck_derive::ByteHash`
   --> /home/computermouth/.cargo/registry/src/github.com-1ecc6299db9ec823/bytemuck-1.12.4/src/lib.rs:129:18
    |
129 |   AnyBitPattern, ByteEq, ByteHash, CheckedBitPattern, Contiguous, NoUninit,
    |                  ^^^^^^  ^^^^^^^^ no `ByteHash` in the root
    |                  |
    |                  no `ByteEq` in the root

For more information about this error, try `rustc --explain E0432`.

Additional information
Debian Sid -- but producing the same error on Debian Stable.

this is due to a dependency that just had a release with an issue: https://github.com/Lokathor/bytemuck/issues/165
the problematic version is going to be yanked

https://crates.io/crates/bytemuck/1.12.4 has been yanked, you should be good if you clear your lock file 👍

Yep, works! Interesting that the Cargo.lock is not checked in. I'm relatively new to rust, is that normal?

Rust recommends to not check in the lock file for libraries: https://doc.rust-lang.org/cargo/faq.html#why-do-binaries-have-cargolock-in-version-control-but-not-libraries