RISCV_FORMAL_VALIDADDR ignored for loads and stores

I'm setting a region of memory to be invalid using RISCV_FORMAL_VALIDADDR. This works as expected for instruction fetch (after fixing some bugs in my processor). Unfortunately I'm still failing some load instruction checks. The example trace it gave me for the test I looked at had a load from an invalid address with spec_trap = 0.
It looks like the following lines in rvfi_insn_check.sv
https://github.com/cliffordwolf/riscv-formal/blob/master/checks/rvfi_insn_check.sv#L104-L105
should be something like this:
        wire mem_access_fault = spec_valid && (spec_rs1_addr == rs1_addr) &&
                ((spec_mem_rmask && !mem_pma_r) || (spec_mem_wmask && !mem_pma_w)
                || ((spec_mem_rmask || spec_mem_wmask) && !`rvformal_addr_valid(spec_mem_addr)));

I tried this out on my processor and all the checks pass now.

This should be fixed now in current git HEAD.
(Some background: Until now we only had cores with RISCV_FORMAL_VALIDADDR that also had RISCV_FORMAL_PMA_MAP, and the PMA map only contains valid addresses. That's why this was not an issue before.)


Carousel Arrows

Hi guys, is there any plan to add prev and next arrows option on Carousel?
Thanks

You can easily achieve this by prepending and appending them with an onclick eventlistener and triggering prev/next.

I agree with @DanielRuf. It is easy enough to add it yourself and in addition to that not part of the specs.


CodeMirror without DOM

Is it possible to construct a CodeMirror object without a DOM (e.g. in Node)?

No. You can run modes using the addon/runmode/runmode.node.js, but constructing an editor without a DOM would not make any sense.


arkmanager help text should output ark-server-tools version

Issue
steam@ip-172-23-163-27:~$ arkmanager --help
Usage: arkmanager [OPTION]

Option                Description
backup                Saves a backup of your server inside the backup directory
broadcast <msg>       Sends a message to all users connected to server
saveworld             Saves the game world to disk
rconcmd <cmd>         Execute RCON command on server
checkupdate           Check for a new ARK server version
install               Install the ARK server files from steamcmd
installmod <modid>    Installs a mod from the Steam workshop
restart               Stops the server and then starts it
restart --all         Restarts all servers specified in configfile_xxxxx
run                   Runs the server without daemonizing
start                 Starts the server
start --all           Starts all servers specified in configfile_xxxxx
stop                  Stops the server
stop --all            Stops all servers specified in configfile_xxxxx
status                Returns the status of the current ARK server instance
update [OPTION ...]   Check for a new ARK server version, if needed, stops the server, updates it, and starts it again
upgrade-tools         Check for a new ARK Server Tools version and upgrades it if needed
uninstall-tools       Uninstall the ARK Server Tools
useconfig <name>      Use the configuration overrides in the specified config name or file

Update command takes the below options:
       --force        Apply update without checking the current version
       --safe         Wait for server to perform world save and update.
       --warn         Warn players before updating server
       --validate     Validates all ARK server files
       --saveworld    Saves world before update
       --update-mods  Updates installed and requested mods
       --backup       Takes a backup of the save files before updating

Expected
steam@ip-172-23-163-27:~$ arkmanager --help
Usage: arkmanager [OPTION]

Option                Description
backup                Saves a backup of your server inside the backup directory
broadcast <msg>       Sends a message to all users connected to server
saveworld             Saves the game world to disk
rconcmd <cmd>         Execute RCON command on server
checkupdate           Check for a new ARK server version
install               Install the ARK server files from steamcmd
installmod <modid>    Installs a mod from the Steam workshop
restart               Stops the server and then starts it
restart --all         Restarts all servers specified in configfile_xxxxx
run                   Runs the server without daemonizing
start                 Starts the server
start --all           Starts all servers specified in configfile_xxxxx
stop                  Stops the server
stop --all            Stops all servers specified in configfile_xxxxx
status                Returns the status of the current ARK server instance
update [OPTION ...]   Check for a new ARK server version, if needed, stops the server, updates it, and starts it again
upgrade-tools         Check for a new ARK Server Tools version and upgrades it if needed
uninstall-tools       Uninstall the ARK Server Tools
useconfig <name>      Use the configuration overrides in the specified config name or file
**version                     Outputs the current ARK Server Tools version**

Update command takes the below options:
       --force        Apply update without checking the current version
       --safe         Wait for server to perform world save and update.
       --warn         Warn players before updating server
       --validate     Validates all ARK server files
       --saveworld    Saves world before update
       --update-mods  Updates installed and requested mods
       --backup       Takes a backup of the save files before updating

arkmanager version can literally just return the following to assist in bug reporting.
steam@ip-172-23-163-27:~$ grep arkst /etc/arkmanager/arkmanager.cfg | cut -d '"' -f 2
1.2
master

Taken as a feature request to add a --version option.


Facing an error while creating users in MSSQL server (in aws) < Error: unable to read user [platform-test].[user]: mssql: 'STRING_AGG' is not a recognized built-in function name.>

Hi,
Sharing the script. Please let me know any findings. MSSQL version =  13.0.1601.5
terraform {
required_providers {
mssql = {
source = "betr-io/mssql"
version = "0.1.0"
}
}
}
provider "mssql" {
debug = "true"
}
resource "mssql_login" "example" {
server {
host = "xxx.xxx.xxx.xxx"
login {
username = "mssql_admin"
password = "Mynewpassword"
}
}
login_name = "test"
password   = "Hello"
}
resource "mssql_user" "example" {
server {
host = "xxx.xx.xxx.xx"
login {
username = "mssql_admin"
password = "Mynewpassword"
}
}
username = "test"
roles  = ["db_owner"]
database = "platform-test"
login_name = mssql_login.example.login_name
}
Error
< Error: unable to read user [platform-test].[user]: mssql: 'STRING_AGG' is not a recognized built-in function name.>
Note: Able to create in database but getting the above error. Please give your inputs.
Thanks and Regards,
Ishan Yadav

STRING_AGG function is available only since MS SQL 2017 while v13.0.1601.5 is MS SQL 2016

Following pr should fix it https://github.com/betr-io/terraform-provider-mssql/pull/48


Adding Saturn.Extensions.Authorization via paket.

I'd like to try the use_google_oauth functionality but I can't add it via paket. At the moment I have Saturn 0.7.4 added to my project but the Saturn.Extensions.Authorization.dll isn't included. Do I have to manually add it somehow?
.paket/paket.exe show-installed-packages
Paket version 5.176.1
Build Fake.Core.Target - 5.2
Build Fake.DotNet.Cli - 5.2
Build Fake.IO.FileSystem - 5.2
Build FSharp.Core - 4.3.4
Client dotnet-fable - 1.3.17
Client Fable.Core - 1.3.17
Client Fable.Elmish.Browser - 1.0
Client Fable.Elmish.Debugger - 1.0.1
Client Fable.Elmish.HMR - 1.0
Client Fable.Elmish.React - 1.0.3
Client Fulma - 1.0.0-beta-025
Client Fulma.Extensions - 1.0.0-beta-016
Server dotnet-fable - 1.3.17
Server Fable.Elmish - 1.0.3
Server Fable.Elmish.Debugger - 1.0.1
Server Fable.Elmish.HMR - 1.0
Server Fable.Elmish.React - 1.0.3
Server Fable.JsonConverter - 1.0.8
Server FSharp.Core - 4.5.2
Server Fulma - 1.0.0-beta-025
Server Fulma.Extensions - 1.0.0-beta-016
Server jose-jwt - 2.4
Server Saturn - 0.7.4
Performance:
 - Runtime: 604 milliseconds
sashan@deimos sashan/code/titan master ☺ >

I also tried various ways to add it via paket (e.g. .paket/paket.exe add Saturn.Extensions.Authorization -g Server -p src/Server/Server.fsproj)
but they didn't work.
My guess is the Extensions are not in nuget? My paket.dependecies refers to Saturn like this
nuget Saturn

I'll try and use a github reference and see if that works.

Saturn.Extensions.Authorization is not yet distributed through NuGet (as there exist couple of things I need to figure out before doing it - https://github.com/SaturnFramework/Saturn/issues/120). I think the simplest way will be using paket github reference to just add https://github.com/SaturnFramework/Saturn/blob/master/src/Saturn.Extensions.Authorization/OAuth.fs).

Saturn.Extensions.Authorization has been published to NuGet. I will be available when NuGet finishes validation and whatever else it's doing (probably some bitcoin mining) here - https://www.nuget.org/packages/Saturn.Extensions.Authorization/


Amending rollback to work with contentful

Rollback now deploys the files originally created in a given tag rather than rebuild them, adds better future proofing by using main as the initial checkout rather than the original tag which could have unintended side-effects with AWS

DEPLOY_URL=https://d2kxgpsar79iuc.cloudfront.net/
cloudfront_id=E16E3KB436FG8A


Cannot install on GKE autopilot cluster due to mutatingwebhookconfigurations access denied

Describe the bug:
Hi,
I am trying to install cert-manager on the "new" GKE cluster autopilot via Helm.
Unfortunately I ran into this error :
Error: rendered manifests contain a resource that already exists. Unable to continue with install: could not get information about the resource: mutatingwebhookconfigurations.admissionregistration.k8s.io "cert-manager-webhook" is forbidden: User "XXXX" cannot get resource "mutatingwebhookconfigurations" in API group "admissionregistration.k8s.io" at the cluster scope: GKEAutopilot authz: cluster scoped resource "mutatingwebhookconfigurations/" is managed and access is denied

Expected behaviour:
The helm script should install cert-manager.
Steps to reproduce the bug:

Create a cluster on GKE with autopilot
Run

helm install \
  cert-manager jetstack/cert-manager \
  --namespace cert-manager \
  --version v1.2.0 \
  --create-namespace

Anything else we need to know?:
Environment details::

Kubernetes version:  1.18.12-gke.1210
Cloud-provider/provisioner: GKE
cert-manager version: 1.2.0
Install method: helm

/kind bug

GKE Autopilot has no support for 3rd party webhooks in it's current state, this limits many Kubernetes plugins such as ourselves not to be able to operate correctly. This is reportedly (on Twitter: https://twitter.com/BagadeVivek/status/1365701217469534220 ) to be fixed in a next release.

Great thanks.
May I leave the bug open until this is fixed ?

We cannot action this so we will close it. However we can leave it pinned for informational purposes
/close

any timeline for support cert-manager on GKE autopilot? just tried out autopilot, and would like to implement it with all our environments, but this blocks us from moving forward.
anyone know a workaround?

any timeline for support cert-manager on GKE autopilot?

Autopilot doesn't let users create MutatingWebhookConfigurations as documented. These are required for cert-manager to work, so there is no timeline or workaround from our side.
Many other tools also cannot be installed by end-users of autopilot, such as istio. If you need to use cert-manager, or any tool that uses a mutating webhook you cannot currently use autopilot - this is the tradeoff you make when choosing the product which seems to be designed for security and ease of use over configurability.
If you have a commercial relationship with Google, reach out to them and ask them if they intend to add MutatingWebhookConfiguration support to autopilot. It seems like they may at some point?
Other low-config managed Kubernetes services are available from the other big cloud providers. I have no vested interest with any of them so won't recommend any 😅 .

any timeline for support cert-manager on GKE autopilot? just tried out autopilot, and would like to implement it with all our environments, but this blocks us from moving forward.
anyone know a workaround?

A possible workaround is to use Google Managed Certificates: https://cloud.google.com/kubernetes-engine/docs/how-to/managed-certs

@olaf-2 I have tried Google's managed certificates in the past, they were the reason I ended up switching to cert-manager in the first place.
In my 3 years of developing on the cloud (primarily AWS and GCP), it often seems to be the case that everything is advertised to "Just work", but that is rarely the case.

A possible workaround is to use Google Managed Certificates

A word of notice: HTTP(S) Internal Load Balancer doesn't support Google Managed Certificates so it won't work for such use case.
I'm using GKE Autopilot and an Internal HTTP(S) LB, so I can't use both Google Managed Certificates and cert-manager. I'm quickly going out of options :)

BTW, do you think it would be possible to set-up a separate GKE cluster (no autopilot) for the sake of cert-manager and then somehow share/mirror the generated certificates between the two clusters?

Hey everyone
We've now added support for mutating webhooks in Autopilot. More details at https://cloud.google.com/kubernetes-engine/docs/concepts/autopilot-overview#webhooks_limitations

Weve been itching to try this, since we are starting a new company that needs wildcard certificates, so we would like to run cert-manager under a new Autopilot cluster to achieve this.
However, when I try this, I still get
User "*******" cannot get resource "mutatingwebhookconfigurations" in API group "admissionregistration.k8s.io" at the cluster scope: GKEAutopilot authz: cluster scoped resource "mutatingwebhookconfigurations/" is managed and access is denied
Now when double checking my new autopilot cluster version is: 1.20.9-gke.701
How can we go about trying this out in Google Cloud?

For anyone getting here via Google, the chart now installs correctly on 1.21+. As of today (22nd Sep) to get onto 1.21+ you may need to be on the "Rapid" release branch, this can be configured under "Advanced" cluster settings on cluster creation and cannot be changed for existing clusters. Imagine it won't be that long until the Regular release branch gets 1.21 but not sure of the exact date.

That said, while the chart installs and the pods start, the log output of cert-manager-startupapicheck is:
Not ready: the cert-manager webhook CA bundle is not injected yet

repeated until it eventually times out, so not currently able to get it to work.

I've been looking forward to this for some time, thanks @vivekbagade and everyone else who helped with this update to Autopilot.
Out of the box, there's still an incompatibility with cert-manager on Autopilot. Cert-manager manages its leader election in configmaps in the kube-system namespace, but this is disallowed by Autopilot (seems like a reasonable choice). I used this Kustomization file to move those configmaps into the cert-manager namespace.
I thought I'd share in case it helps others. I'd also like to learn if anyone else would suggest a better approach.
---
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization

resources:
  - https://github.com/jetstack/cert-manager/releases/download/v1.5.3/cert-manager.yaml

patches:
  - target:
      kind: Role
      name: cert-manager:leaderelection
    patch: |-
      - op: replace
        path: /metadata/namespace
        value: cert-manager
  - target:
      kind: Role
      name: cert-manager-cainjector:leaderelection
    patch: |-
      - op: replace
        path: /metadata/namespace
        value: cert-manager
  - target:
      kind: RoleBinding
      name: cert-manager:leaderelection
    patch: |-
      - op: replace
        path: /metadata/namespace
        value: cert-manager
  - target:
      kind: RoleBinding
      name: cert-manager-cainjector:leaderelection
    patch: |-
      - op: replace
        path: /metadata/namespace
        value: cert-manager
  - target:
      kind: Deployment
      name: cert-manager
    patch: |-
      - op: replace
        path: /spec/template/spec/containers/0/args/2
        value: --leader-election-namespace=cert-manager
  - target:
      kind: Deployment
      name: cert-manager-cainjector
    patch: |-
      - op: replace
        path: /spec/template/spec/containers/0/args/1
        value: --leader-election-namespace=cert-manager

If you're using the standard Helm chart, you can simply pass helm the appropriate parameter, as in
helm install my-release --namespace cert-manager --version v1.5.4 jetstack/cert-manager --set global.leaderElection.namespace=cert-manager
Apart from the helm install command "failing" with the error message Error: failed post-install: timed out waiting for the condition and the startupapicheck job never completing successfully, this worked like a charm for me on a newly spun-up auto-pilot cluster ("Rapid" branch of course). In spite of these errors, cert-manager seems to work just as it should, creating certificates as expected. (Full disclosure:  I only did a quick test with self-signed CA/issuer and a certificate based on that, no fancier use cases yet)
Great work and thanks @vivekbagade !

You need to make sure you're using the "rapid" channel for GKE Autopilot. I've successfully installed cert-manager v1.6.1 using Helm to a cluster. As already noted in the thread, it's necessary to override the global.leaderElection.namespace from the default kube-system as this is a "managed namespace" on Autopilot (see https://cloud.google.com/kubernetes-engine/docs/concepts/autopilot-overview#managed_namespaces).

Thanks @mattbates  - the --set global.leaderElection.namespace=cert-manager --  is indeed the key.

Status update on this, It seems the rapid release channel is no longer required in GKE Autopilot.  Installed successfully with:
helm install --create-namespace --namespace cert-manager --set installCRDs=true --set global.leaderElection.namespace=cert-manager cert-manager jetstack/cert-manager

As at today (11/22/21), this works on the regular channel 1.21.5-gke.1302 as well.

For anyone else like me who's a little slow :upside_down_face:
helm repo add jetstack https://charts.jetstack.io
helm repo update
helm install --create-namespace --namespace cert-manager --set installCRDs=true --set global.leaderElection.namespace=cert-manager cert-manager jetstack/cert-manager

(Do this instead of kubectl apply -f https://github.com/cert-manager/cert-manager/releases/download/vX.Y.Z/cert-manager.yaml)
Now you can continue on with your kubectl apply -f issuer-lets-encrypt-staging.yaml

For anyone else who's a little slow like me 🙃
Instead of kubectl apply -f https://github.com/cert-manager/cert-manager/releases/download/vX.Y.Z/cert-manager.yaml, do:
helm repo add jetstack https://charts.jetstack.io
helm repo update
helm install --create-namespace --namespace cert-manager --set installCRDs=true --set global.leaderElection.namespace=cert-manager cert-manager jetstack/cert-manager

If you already did kubectl apply -f https://github.com/cert-manager/cert-manager/releases/download/vX.Y.Z/cert-manager.yaml, uninstall it first: https://cert-manager.io/docs/installation/kubectl/#uninstalling
Now you can continue on with your kubectl apply -f issuer-lets-encrypt-staging.yaml

This works for me on Gke Autopilot with k8s v1.24.100-gke.2300

Anyone has success with argocd? Somehow I still get the error message:
apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: cert-manager

  # You'll usually want to add your resources to the argocd namespace.
  namespace: argocd

  # Add a this finalizer ONLY if you want these to cascade delete.
  finalizers:
    - resources-finalizer.argocd.argoproj.io

spec:
  # The project the application belongs to.
  project: default

  # Source of the application manifests
  source:
    chart: cert-manager
    helm:
      parameters:
        - name: installCRDs
          value: "true"
        - name: global.leaderElection.namespace
          value: "cert-manager"
    repoURL: https://charts.jetstack.io
    targetRevision: v1.12.1

  # Destination cluster and namespace to deploy the application
  destination:
    server: https://kubernetes.default.svc
    namespace: cert-manager

  # Sync policy
  syncPolicy:
    automated: # automated sync by default retries failed attempts 5 times with following delays between attempts ( 5s, 10s, 20s, 40s, 80s ); retry controlled using `retry` field.
      prune: true # Specifies if resources should be pruned during auto-syncing ( false by default ).
      selfHeal: true # Specifies if partial app sync should be executed when resources are changed only in target Kubernetes cluster and no git change detected ( false by default ).
      allowEmpty: false # Allows deleting all application resources during automatic syncing ( false by default ).


PodSecurity Standards on microk8s v1.24

Hi I'm using microk8s version 1.24 on Ubuntu 20.04.
I've enabled PodSecurity admission using FeatureGate. Restrictions on a namespace using labels is working fine. However when I'm trying to apply a cluster wide policy I'm getting following error:
The admission control config file is as follows:
admission-control-config
apiVersion: apiserver.config.k8s.io/v1
kind: AdmissionConfiguration
plugins:
- name: PodSecurity
 path: cluster-level-pss.yaml

The cluster wide configuration file is as follows:
apiVersion: apiserver.config.k8s.io/v1
kind: AdmissionConfiguration
plugins:
- name: PodSecurity
 configuration:
   apiVersion: pod-security.admission.config.k8s.io/v1alpha1
   kind: PodSecurityConfiguration
   defaults:
     enforce: "baseline"
     enforce-version: "latest"
     audit: "baseline"
     warn: "baseline"
   exemptions:
     usernames: ["administrator"]
     runtimeClasses: []
     namespaces: ["kube-system","container-registry","gpu-operator-resources"]

Error is as follows:
Jun 27 21:25:32 administrator-virtual-machine microk8s.daemon-kubelite[658208]: Error: failed to initialize admission: couldn't init admission plugin "PodSecurity": no kind "AdmissionConfiguration" is registered for version "apiserver.config.k8s.io/v1" in scheme "vendor/k8s.io/pod-security-admission/admission/api/scheme/scheme.go:30"
Jun 27 21:25:32 administrator-virtual-machine microk8s.daemon-kubelite[658208]: F0627 21:25:32.233367 658208 daemon.go:67] API Server exited failed to initialize admission: couldn't init admission plugin "PodSecurity": no kind "AdmissionConfiguration" is registered for version "apiserver.config.k8s.io/v1" in scheme "vendor/k8s.io/pod-security-admission/admission/api/scheme/scheme.go:30"

Due to above error the kube-apiserver daemon isn't starting. Kindly let me know what am I missing here...
Thanks.

Following are steps to replica:

Add following line to kube-apiserver configuration file:

--feature-gates=PodSecurity=true
--enable-admission-plugins=PodSecurity
--admission-control-config-file=/var/snap/microk8s/current/args/admission-control-config

Restart kube-apiserver daemon
Update admission-control-config with following:

apiVersion: apiserver.config.k8s.io/v1
kind: AdmissionConfiguration
plugins:
- name: PodSecurity
  path: podsecurity.yaml

Add PodSecurityStandard labels on namespace.

pod-security.kubernetes.io/enforce: baseline
pod-security.kubernetes.io/enforce-version: v1.24

podsecurity.yaml has following snippet:

apiVersion: apiserver.config.k8s.io/v1
kind: AdmissionConfiguration
plugins:
- name: PodSecurity
 configuration:
   apiVersion: pod-security.admission.config.k8s.io/v1alpha1
   kind: PodSecurityConfiguration
   defaults:
     enforce: "baseline"
     enforce-version: "latest"
     audit: "baseline"
     warn: "baseline"
   exemptions:
     usernames: ["administrator"]
     runtimeClasses: []
     namespaces: ["kube-system","container-registry","gpu-operator-resources"]

But it doesn't seem to be working, I'm not able to apply exemptions on those namespaces. Kindly suggest.

Trying to apply cluster-wide defaults & exemptions as per the documentation: https://kubernetes.io/docs/tasks/configure-pod-container/enforce-standards-admission-controller/
Kindly suggest.

Resolved in Slack thread. There was an issue with the config file format.
For future reference:
# /var/snap/microk8s/current/args/admission-control-config
---
apiVersion: apiserver.config.k8s.io/v1
kind: AdmissionConfiguration
plugins:
- name: EventRateLimit
  configuration:
    kind: Configuration
    apiVersion: eventratelimit.admission.k8s.io/v1alpha1
    limits:
    - type: Namespace
      qps: 50
      burst: 100
      cacheSize: 2000
    - type: User
      qps: 10
      burst: 50
- name: PodSecurity
  configuration:
    apiVersion: pod-security.admission.config.k8s.io/v1alpha1
    kind: PodSecurityConfiguration
    defaults:
      enforce: "baseline"
      enforce-version: "latest"
      audit: "baseline"
      warn: "baseline"
    exemptions:
      usernames: ["administrator"]
      runtimeClasses: []
      namespaces: ["kube-system","container-registry","gpu-operator-resources"]

and then have the following lines appended to /var/snap/microk8s/current/args/kube-apiserver:
--feature-gates=PodSecurity=true
--enable-admission-plugins=PodSecurity
--admission-control-config-file=/var/snap/microk8s/current/args/admission-control-config

A restart is needed for the changes to take effect:
sudo snap restart microk8s

@m1nish1208 feel free to re-open if you have any extra issues. Thank you!


Fix SQLAlchemy warnings

Minor adjustments suppress SQLAlchemy warnings that would appear when running make tests. The following changes reduced the total number of warnings from 142 to 119.
Suppress warnings about .events relationships:
SAWarning: relationship 'User.events' will copy column users.id to column user_events.source_id, which conflicts with relationship(s): 'UserEvent.source' (copies users.id to user_events.source_id). If this is not the intention, consider if these relationships should be linked with back_populates, or if viewonly=True should be applied to one or more if they are read-only. For the less common case that foreign key constraints are partially overlapping, the orm.foreign() annotation can be used to isolate the columns that should be written towards.   To silence this warning, add the parameter 'overlaps="source"' to the 'User.events' relationship. (Background on this error at: https://sqlalche.me/e/14/qzyx)

Suppress warnings about many-to-many relationships:
SAWarning: relationship 'Project.users' will copy column users.id to column roles.user_id, which conflicts with relationship(s): 'Role.user' (copies users.id to roles.user_id). If this is not the intention, consider if these relationships should be linked with back_populates, or if viewonly=True should be applied to one or more if they are read-only. For the less common case that foreign key constraints are partially overlapping, the orm.foreign() annotation can be used to isolate the columns that should be written towards.   To silence this warning, add the parameter 'overlaps="user"' to the 'Project.users' relationship. (Background on this error at: https://sqlalche.me/e/14/qzyx)

Suppress warnings about or_() with dynamic args:
SADeprecationWarning: Invoking or_() without arguments is deprecated, and will be disallowed in a future release. For an empty or_() construct, use or_(False, *args).

Suppress warning about using subquery for in_():
SAWarning: Coercing Subquery object into a select() for use in IN(); please pass a select() construct explicitly

consider if these relationships should be linked with back_populates

I think we should probably do what the warning is suggesting here instead?

I resolved the merge conflicts but the tests are now failing. I'm happy to merge this if we can resolve those failures.

@miketheman and @di: I’ve fixed the tests for this PR, should be ready for merge.

@dstufft Thank you!


Peepee Poopoo moment

Username and Discrim: DanKSpooKLy#1337
User ID:<PHONE_NUMBER>48713217
Image Link: https://i.imgur.com/xSObeUv.gif

done :D


[Documentation] Add Table to Display Repository Statistics in README.md

Issue Description:
The issue involves adding a well structured table to display repository statistics, such as the number of stars, forks, issues, and pull requests, directly in the README.md file. This will help contributors and users track the activity and contributions at a glance.
Steps to Implement:
Create a table in the README.md file displaying the following repository statistics:
Stars
Forks
Open Issues
Open Pull Requests
Closed Pull Requests
Use markdown syntax to include badges or icons for each statistic.
Add dynamic links to the corresponding sections of the repository for easy access (e.g., link to the "Issues" or "Pull Requests" tabs).
Expected Behavior:
The README.md file will contain a neatly formatted table that displays real-time repository statistics, making the repository's activity more visible and engaging for contributors and users.
Screenshot (for reference):

@c0sm0void kindly assign me this issue with the labels of gssoc extended,hacktoberfest and level

@c0sm0void kindly assign the level label

@KGupta2601 assigned


Notebooks: Don't Display Notebook until Markdown Cells are Rendered

Don't display notebook (i.e. still show spinner) until markdown cells are rendered. When the notebook spinner disappears, the notebook should be fully loaded.

Doesn't meet the bar for March.


Path description needs a label formatter

Testing #3488
The path should be presented as it is in the above part of the picture:

Interesting. When I ran it I didn't see the review prefix and I kind of liked having the commit ID in there.

That information is quite redundant though, no? If at all I would show it next to the commit label as description.
Btw I was pushing changes to the label formatter yesterday when there is no formatter registered, so it is possible I caused an issue here, but nevertheless a label formatter makes sense here imho.


Separator for long/variable length field

This is a feature/enhancement proposition.
Bit field could be used to document binary packet format if there would be a variable length separator capability. Something like this :

Has this feature been added, trying to document a binary communications protocol and this is basically a requirement to make bitfield useful for that purpose?

No, this feature is not being implemented yet.
I have some questions:
Are we talking about this particular example with one variable length field in the end?
Can this field be in the middle?
How this would look like with multiple variable fields?
Can we have mix of Varible anf fixed fields?
@MattIhnen can you share your use case?


Proxy release 2024-12-26

Proxy release 2024-12-26
Please merge this Pull Request using 'Create a merge commit' button

reviewed for changelog


chore: remove most impl AsRef<str,Path>

These mostly only bloat code size and compilation times, which are already not great when we are already codegenning a billion structs and ASTs with serde derives
For AsRef, the only change is having to specify an extra 1 (one) character (&) when using String so it derefs to str, otherwise it's the same
For AsRef it's the same as AsRef when using Path/PathBuf, but it's a bit worse for string literals; but most of these functions are called with variables in the first place. except in tests which will have to use .as_ref() or Path::new
I've not removed these when using with &[] or impl IntoIterator since that does help quite a bit, e.g if you have an owning iterator of Strings, you cannot borrow inside of a map closure
I've also kept Into since that makes more sense, you want to avoid cloning if you can pass in an owned thing already, otherwise you'll clone it yourself
Also fixes some inconsistencies in function signatures for str vs path

Pending github waking up and running actions


Demo is broken

The demo depends on ../dist/downscale.js which vanished with 9c58a57.

Looks like this was fixed in https://github.com/ytiurin/downscale/commit/2eb4683caa9fb59fd43eeba2018477d1d3579fae

right


Test Traits

Description
I've been enjoying the new DatabaseTestTrait, but it reveals how awkward the $setUpMethods and $tearDownMethods properties are for actual use. There is no easy way to extend them, traits cannot use them without conflict errors, adjusting them in the framework would be a breaking change...
This PR moves toward a new approach: dynamic trait staging methods. I've borrow some helper functions from Laravel to pull all the traits from a test case and then check for a corresponding method name. For example:
trait AuthTrait
{
	protected setUpAuthTrait()
	{
		$user = $this->createFakeUser();
		$this->logInUser($user);
	}
...

class AuthenticationFeatureTest
{
	use AuthTrait;
...

This allows each trait to define its own set up and tear down methods without the test case needing to know about them. If no methods are present then nothing happens.
Checklist:

[X] Securely signed commits
[X] Component(s) with PHPdocs
[X] Unit testing, with >80% coverage
[ ] User guide updated
[X] Conforms to style guide

@lonnieezell I had the same thought but when I looked into it FeatureTestTrait doesn't actually do any staging at all, it handles everything during call().
I'm doing some massive test optimization on a project right now, leveraging traits instead of my previous class extension methods. I've bought my test time down to about 1/5 of what it used to be. When I tackle the feature tests part I will see if anything sticks out to me as needing improvements.

I've bought my test time down to about 1/5 of what it used to be.

@MGatner Wow. That's sweet. I'm always for faster tests. That means I'll run the full suite more often :)

Failures were because I forgot namespaces :O Fix incoming.

Would it be better to add those functions in common? I think usefulness of those will go beyond testing.

I'm open to it. At this point they weren't needed elsewhere and since they can be moved any time I figured I'd throw them in testing, but I'm glad to hear otherwise.


Name of submissions: missing?

We need to be able to add a meaningful name to submissions (in addition to the automatic labels 'File Name' and 'Username'). This name should appear on the leaderboard (in addition to or instead of Username)

Hello, I've been working on an update to the Fact Sheet system for v2. This system allows you to set questions to be attached to submissions. You can see the details in the PR #553. Does this solve your problem?

Hello @Logan-Ruf ,
I am not sure to understant how the Fact Sheet system works. Where can I find the factsheet edit form? How can I specify it?
Thank you very much.

Hi again, I found the fact sheet editor...
Seems to work pretty well, thank you!
However, I noticed that the metadata do not appear on the leaderboard .json file.
The idea would be to be able to identify different submissions when multiple submissions from a single user will be allowed on the leaderboard (I had a look on PR#542, looks good also!). What are your thoughts about this?

Thanks for pointing that out. The issue should be fixed now.


Different zero shot results, grad strides do not match butcket view strides

Thank you for your great paper.
I tried to train a zero-shot model (vitl16_384) and tested on PASCAL fold 0 and got the following problems:

After testing, it returned mIoU = 32.9 versus 61.3 reported in the paper.

This is my training script:
python train_lseg_zs.py \

--exp_name train_vitl16_pascal_fold0 --project_name lightseg \

--backbone clip_vitl16_384 \

--dataset pascal --data_path data/Dataset_HSN \

--fold 0 --nshot 0 \

--batch_size 4 --base_lr 0.0001 --max_epochs 200 \

--weight_decay 1e-5 --no-scaleinv --widehead

How does the max_epochs argument take part in the training process since there are only 4 epochs logged out?
Apart from changing the model from vitl16_384 to vitb32_384, is there anything wrong with my training script?

While training, this error is logged out:

[W reducer.cpp:283] Warning: Grad strides do not match bucket view strides. This may indicate grad was not created according to the gradient layout contract, or that the param's strides changed since DDP was constructed.  This is not an error, but may impair performance.
grad.sizes() = [512, 256, 1, 1], strides() = [256, 1, 256, 256]
bucket_view.sizes() = [512, 256, 1, 1], strides() = [256, 1, 1, 1] (function operator())

While training, DPP is enabled and I only used 01 GPU with batch_size = 4. I am not sure if this damages training. Does the argument accumulate_grad_batches probably make this happen?

Hi @ngfuong ,
Thanks for your interest in LSeg!
As has been reported, we only train a few epochs of LSeg on zero-shot for PASCAL and COCO experiments. I haven't met such a problem, you might need to check your system for details. Also, we have released our checkpoints, please feel free to use them.
Hope this helps.
Best,
Boyi

Hi @Boyiliee
Thank you for replying!
I'm trying to reproduce your results so whether your reported results matche mine is of critical importance.
Could you verify if my arguments and hyperparameters are reasonable? I'm training with 01 Tesla A100 80GB GPU.


Pin engine.version to cherrypicked change

Updating engine version to this cherry pick: https://github.com/flutter/engine/commit/040747ed404590aa25490e48b15db10a75e16cba

Does flutter-build broken message mean anything for a branch?

Nope


Define custom metric methods even if port is zero

If metrics_port is zero then custom metric methods needs to be defined too.
Cought in Ingress API with disabled metrics

@lindgrenj6 it seems there are some values kept between two specs. Any idea?

Hmm yeah we're back to the old "Can't modify frozen array" error that I ran into when initially writing them. Heres the changes I made to get back to that:
diff --git a/spec/lib/insights/api/common/metrics_spec.rb b/spec/lib/insights/api/common/metrics_spec.rb
index 4d47ddf..263bd7e 100644
--- a/spec/lib/insights/api/common/metrics_spec.rb
+++ b/spec/lib/insights/api/common/metrics_spec.rb
@@ -1,3 +1,6 @@
+require 'prometheus_exporter'
+require 'prometheus_exporter/client'
+
 describe Insights::API::Common::Metrics do
   let(:custom_metrics) do
     {
@@ -19,21 +22,26 @@ describe Insights::API::Common::Metrics do
   let(:metrics) { Insights::API::Common::Metrics }
   let(:prometheus) { PrometheusExporter::Client.default }

-  before do
-    # Dummy app doesn't listen here.
-    allow(Rails).to receive_message_chain(:application, :middleware, :unshift)
-  end

   describe ".setup_custom_metrics" do
+    around do |example|
+      PrometheusExporter::Client.default = PrometheusExporter::Client.new
+      metrics.instance_variable_set(:@metrics_port, port)
+      metrics.activate(nil, "dummy_prefix", custom_metrics)
+
+      example.call
+
+      metrics.instance_variables.each{|e|metrics.remove_instance_variable(e)}
+      PrometheusExporter::Client.default = nil
+    end
+
     context "with metrics port set" do
       before do
-        metrics.instance_variable_set(:@metrics_port, 9394)
-        metrics.activate(nil, "dummy_prefix", custom_metrics)
+        # Dummy app doesn't listen here.
+        allow(Rails).to receive_message_chain(:application, :middleware, :unshift)
       end

-      after do
-        metrics.instance_variable_set(:@metrics_port, nil)
-      end
+      let(:port) { 9394 }

       it "creates the singleton methods on the Metrics object" do
         expect(metrics.respond_to?(:message_on_queue)).to be_truthy
@@ -51,14 +59,7 @@ describe Insights::API::Common::Metrics do
     end

     context "with metrics port zero" do
-      before do
-        metrics.instance_variable_set(:@metrics_port, 0)
-        metrics.activate(nil, "dummy_prefix", custom_metrics)
-      end
-
-      after do
-        metrics.instance_variable_set(:@metrics_port, nil)
-      end
+      let(:port) { 0 }

       it "creates the singleton methods on the Metrics object" do
         expect(metrics.respond_to?(:message_on_queue)).to be_truthy

I'll look into it some more later, but here something that gets it along further.

Worked on this a bit more today @slemrmartin and I can't seem to get it consistent. This is the "most consistent" way I've found, it fails once every 5-10 test runs for some reason. It's annoying!
diff --git a/spec/lib/insights/api/common/metrics_spec.rb b/spec/lib/insights/api/common/metrics_spec.rb
index 4d47ddf..12b6a6b 100644
--- a/spec/lib/insights/api/common/metrics_spec.rb
+++ b/spec/lib/insights/api/common/metrics_spec.rb
@@ -1,14 +1,17 @@
+require 'prometheus_exporter'
+require 'prometheus_exporter/client'
+
 describe Insights::API::Common::Metrics do
   let(:custom_metrics) do
     {
       :custom_metrics => [
         {
-          :name      => "message_on_queue",
+          :name        => "message_on_queue",
           :type        => :counter,
           :description => "total number of messages put on the queue"
         },
         {
-          :name      => "error_processing_payload",
+          :name        => "error_processing_payload",
           :type        => :counter,
           :description => "total number of errors while attempting to messages put on the queue"
         }
@@ -19,21 +22,21 @@ describe Insights::API::Common::Metrics do
   let(:metrics) { Insights::API::Common::Metrics }
   let(:prometheus) { PrometheusExporter::Client.default }

-  before do
-    # Dummy app doesn't listen here.
-    allow(Rails).to receive_message_chain(:application, :middleware, :unshift)
-  end
-
   describe ".setup_custom_metrics" do
-    context "with metrics port set" do
-      before do
-        metrics.instance_variable_set(:@metrics_port, 9394)
-        metrics.activate(nil, "dummy_prefix", custom_metrics)
-      end
+    before do
+      allow(Rails).to receive_message_chain(:application, :middleware, :unshift)
+      metrics.instance_variable_set(:@metrics_port, port)

-      after do
-        metrics.instance_variable_set(:@metrics_port, nil)
-      end
+      metrics.activate(nil, "dummy_prefix", custom_metrics)
+    end
+
+    after do
+      metrics.instance_variables.each { |e| metrics.remove_instance_variable(e) }
+      PrometheusExporter::Client.default = nil
+    end
+
+    context "with metrics port set" do
+      let(:port) { 9394 }

       it "creates the singleton methods on the Metrics object" do
         expect(metrics.respond_to?(:message_on_queue)).to be_truthy
@@ -51,14 +54,7 @@ describe Insights::API::Common::Metrics do
     end

     context "with metrics port zero" do
-      before do
-        metrics.instance_variable_set(:@metrics_port, 0)
-        metrics.activate(nil, "dummy_prefix", custom_metrics)
-      end
-
-      after do
-        metrics.instance_variable_set(:@metrics_port, nil)
-      end
+      let(:port) { 0 }

       it "creates the singleton methods on the Metrics object" do
         expect(metrics.respond_to?(:message_on_queue)).to be_truthy

@lindgrenj6 I tried current specs version 30times in a line without error. Can you try it on your localhost pls, to be sure? It's hard to say what exactly caused random errors, but probably setting default client to different instance than the one in let


Update dependencies for v0.11.0 release

Signed-off-by: Nicko Guyer<EMAIL_ADDRESS>

Codecov Report

Merging #333 (6ce3752) into main (a3e5efe) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #333   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          232       232
  Lines        12721     12721
=========================================
  Hits         12721     12721

Continue to review full report at Codecov.

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


Having .po files in one folder for each language

Is there a possibility of having .po files in one place rather than having each one in a separate folder ?

Could you try to pass a string to gettext_compact?
https://www.sphinx-doc.org/en/master/usage/configuration.html#confval-gettext_compact

What do you mean by passing a string @tk0miya , would you mind expanding your comment a bit ? Thanks

The gettext_compact is a configuration of sphinx. So please add the following line to your conf.py
gettext_compact = 'filename'

Maybe my last question is, how can I separate the concerns of saving in file for English and one file for Germany for example when am translating @tk0miya

Thank you for explanation. Unfortunately, there is no way to do that. Please put .po file to the directory for each language.


Reduce http2 buffer slicing

Motivation
DefaultHttp2FrameReader currently does a fair amount of "intermediate" slicing which can be avoided.
Modifications
Avoid slicing the input buffer in DefaultHttp2FrameReader until necessary. In one instance this also means retainedSlice can be used instead (which may also avoid allocating).
Results
Less allocations when using http2. I will aim to write a benchmark when I get a chance, I'm not sure that any of the existing ones cover this. Anecdotally (possibly meaningless :)), there are more than 11k fewer slices done when running the codec-http2 unit tests, and more than 4k of the remaining slices done now use retainedSlice.

@njhill nice one!

@netty-bot test this please

@netty-bot test this please

@njhill thanks a lot!


Ft query endpoint #16

What does this PR do?
this is the PR for quesries endpoint, create, read and delete query
Description of Task to be completed?

[x] create a query
[x] delete a query
[x] read queries

How should this be manually tested?
clone the repository and test the end points from routes/index.js
Any background context you want to provide?
no
What are the relevant pivotal tracker stories?
-https://trello.com/c/IIKItnf0/26-queryendpoints16
Screenshots (if appropriate)

Web capture_31-1-2022_1297_davteams postman co
Questions:

the branch has no conflict ...now


How to pass props to route

FYI

Please search the docs and issue tracker before asking the question. We want to make sure that the GitHub issue tracker remains the best place to track bug reports and feature requests that affect the development of React Router.
Questions like yours deserve a purpose-built Q&A forum. Would you like to post this question to Stack Overflow with the tag #react-router? https://stackoverflow.com/questions/ask?tags=react-router.
We also have an active and helpful React Router community on Reactiflux, which is a great place to get fast help with React Router and with the rest of the React ecosystem. You can join at https://discord.gg/0ZcbPKXt5bYaNQ46.
Use React.cloneElement() - see the CHANGES and also see https://github.com/rackt/react-router/issues/1531


WIP: Initial 3.0.0 release

See the changelog
Still need to do more testing
Also need to update iOS (although that's not a blocker)
If anyone wants to try it out and provide feedback please do so :).
Thanks!

Codecov Report

Merging #58 into master will decrease coverage by 0.94%.
The diff coverage is 82%.

@@            Coverage Diff             @@
##           master      #58      +/-   ##
==========================================
- Coverage   55.38%   54.44%   -0.95%
==========================================
  Files         162      162
  Lines        9663     9397     -266
==========================================
- Hits         5352     5116     -236
+ Misses       4311     4281      -30

Impacted Files
Coverage Δ

src/enamlnative/android/android_adapter.py
100% <ø> (+10.71%)
:arrow_up:

src/enamlnative/widgets/tab_layout.py
80% <ø> (-1.58%)
:arrow_down:

src/enamlnative/android/android_checkbox.py
100% <ø> (ø)
:arrow_up:

src/enamlnative/ios/uikit_flexbox.py
56.96% <ø> (-0.26%)
:arrow_down:

src/enamlnative/android/android_toggle_button.py
100% <ø> (+39.13%)
:arrow_up:

src/enamlnative/android/android_view_animator.py
0% <ø> (ø)
:arrow_up:

src/enamlnative/android/android_toolbar.py
97.29% <ø> (+1.14%)
:arrow_up:

src/enamlnative/widgets/flexbox.py
57.44% <ø> (-0.89%)
:arrow_down:

src/enamlnative/android/android_yoga.py
0% <ø> (ø)
:arrow_up:

src/enamlnative/widgets/rating_bar.py
80.95% <ø> (-0.87%)
:arrow_down:

... and 93 more

Continue to review full report at Codecov.

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


Props, number of item to show

Hi,
Can we have as props the number of elements to display initially in the slide ?
Thanks

Hi!
the plugin properties have this option, for example:
        <Carousel           value={value}           plugins={[             {               resolve: slidesToShowPlugin,               options: {                 numberOfSlides: 4,               },             },           ]}         >

Hi!
the plugin properties have this option, for example:
        <Carousel           value={value}           plugins={[             {               resolve: slidesToShowPlugin,               options: {                 numberOfSlides: 4,               },             },           ]}         >


Delay after fetching instructions when not using icache

I have the following program:
and x3, x1, x12       # instr1
and x4, x5, x6         # instr2
and x7, x8, x10       # instr 3
and x12, x14, x16   # instr 4

and x13, x11, x2     # instr 5
and x20, x21, x22   # instr 6
and x23, x24, x25   # instr 7
and x26, x27, x28   # instr 8

The execution trace like this:

What I notice here is that it takes 10 cycles for the new instructions (5,6) to arrive at the decoder.
This is not beneficial for fast execution, since the CPU is kind of idle for 5 cycles between writing instructions 3 and 4 and reading instructions 5 and 6.
Are there configuration parameters to increase the instruction/fetch buffer? Such that it is possible to have more than 4 instructions in execution?
I also couldn't really find out if this is due to the instruction buffer size in the decoder (which seems to have 4 instruction slots but somehow only has 2) or the fetch buffer size in the IFU.
Any ideas?

this core should be able to execute 2 integer instructions per clock. Make sure you are running code from the cache and fast external memory or from ICCM.
the MRAC CSR register needs to be programmed to enable caching ...

I turned on the icache and now it works as expected after the respective instructions are cached. Thanks!
However, I do not need 16 kB of icache, i want to have way less.
Is it safe to change NUM_WAYS and NUM_BANKS (https://github.com/chipsalliance/Cores-SweRV/blob/7332edc0adaa7e9a0c842d169154429e8d987786/design/ifu/ifu_ic_mem.sv#L228:L229) to do so?

you can set cache size to 8K, I think - run swerv.config -help for possible build parameters ranges.
BTW, you can check out more modern smaller EL2 core ...

Ok, I managed to tweak it down to 2048 Byte. However, it is more of a hack...I had to add modules to mem_lib.sv and modify the config script of course.
The EL2 does not exactly fit my purposes, since I need a core as pipelined as possible, preferably dual issue.
However, thank you very much for your help. I really appreciate it.
All my questions are resolved now.


How do I catch/handle errors?

Hi!
This seems like a feature that has not yet been properly documented. My API can in some situations throw custom exceptions, how do I let hug handle these errors gracefully, rather than ending up in a http 500 state?
Best,
Erwin

Hi @ErwinHaasnoot,
In hug 2.0.0 you can do the following:
@hug.exception(Exception):
def my_handler(exception):
     return {'my_exception': 'format'}

Where Exception can be replaced with the kind of exception you want to handle.
hug < 1.9.9 requires local exception handling (try, catch) within your handler.
Unfortunately, hug 2.0.0 features are not yet well documented but I hope to expand the documentation at time of release. Questions like this help me decide what areas need priority :)
Thanks!
~Timothy

Cool, that waa pretty much what I was looking for. I'm currently running
the latest dev version, as I really need the Api Key handling. So will look
at it again on Monday!
Thanks so much!
On 26 Feb 2016 6:47 pm, "Timothy Edmund Crosley"<EMAIL_ADDRESS>wrote:

Closed #227 https://github.com/timothycrosley/hug/issues/227.
—
Reply to this email directly or view it on GitHub
https://github.com/timothycrosley/hug/issues/227#event-567901002.


Creating SQLite in memory fails

Attempt to create SQLIte database in memory as documented in docs, i.e.
environments:
    testing:
        adapter: sqlite
        memory: true     # Setting memory to *any* value overrides name

fails:
$ vendor/bin/phinx migrate -e testing
Phinx by Rob Morgan - https://phinx.org. 0.8.1

using config file ./phinx.yml
using config parser yaml
using migration paths
 - /<redacted>/db/migrations
using seed paths
 - /<redacted>/db/seeds
using environment testing
using adapter sqlite
Could not determine database name! Please specify a database name in your config file.

The workaround for this is to provide name regardless it is used or not:
environments:
    testing:
        adapter: sqlite
        name: /tmp/db
        memory: true

Are you able to provide a pull request with your suggested changes for documentation here?


Linking resources in draft form breaks display/deployment of starting point "published" resource

Adding related resources by linking "drafts" breaks the site when deployed.
Something to consider for future is whether we could link up resources and simply hide them until they are ready to be published. That way a lot of the work could be done proactively to place a resource within a topic or within another resource's related resources before the content is finalized and published. In an ideal world, this would allow us to move and iterate more quickly.
In the present, a "Draft resource" cannot be linked to any entries until published.

resolved

Stumbled upon this issue again.
Linking a draft resource to a topic causes the topic page to break.

This is still an issue. When a link to a draft is published, the initial page that links out will break completely.

Band-aid fix: temporarily hidden in production (by removing the link to draft in Contentful and redeploying)

Fixed


Add tests

In this branch, I added tests.

Hi @yodit93
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights
Used gitflow :+1:
Great effort in making the requested changes :+1:


CD.Spec.Platform.None => none-platform

Add a None platform type for ClusterDeployment for agent-based
installations. This results in the hive.openshift.io/cluster-platform
label being set to none-platform.
HIVE-1781

/hold for testing (by whom??)
/assign @abutcher

okay, nice, thanks @abutcher!
/hold cancel

Manual cherry-pick: #1719


Could not GET 'http://tokbox.bintray.com/maven/com/google/android/flexbox/1.0.0/flexbox-1.0.0.pom'

[x] I have searched this issues at (https://github.com/google/flexbox-layout/issues) and confirmed this is not a duplicate

Issues and steps to reproduce

Disable jcenter() and used mavenCentral()
Then update to latest flexbox that support mavenCentral()
Sync Gradle
Rebuild Project
Build: failed Could not GET 'http://tokbox.bintray.com/maven/com/google/android/flexbox/1.0.0/flexbox-1.0.0.pom'

Expected behavior
Rebuild project without error
Version of the flexbox library
com.google.android.flexbox:flexbox:3.0.0
Link to code
#1
...
//        jcenter()
mavenCentral()
...
#2
...
//    implementation 'com.google.android:flexbox:1.1.0'
implementation 'com.google.android.flexbox:flexbox:3.0.0'
...

The library is now hosted on google maven. Please try with the google maven repository.


Fix ProfilePrivacy modal styles

Before:

After:

@MathiasMoralesDixtra tested on web?

@MathiasMoralesDixtra tested on web?

I uploaded pictures of the web.


chore: use app/globals.css instead of styles/globals.css

Closes #2028
✅ Checklist

[✅] I have followed every step in the contributing guide (updated 2022-10-06).
[✅] The PR title follows the convention we established conventional-commit
[✅] I performed a functional test on my final commit

Changelog
This change would streamline development and configuration for the majority of developers who use both the T3 stack and Shadcn UI.

💯

Shadcn works fine with the /styles/globas.css for me. It auto finds the file


Allow more than one multicast server to bind on same port and address

I was trying to run the 'multicast_client_server' example, but was throwing 'EADDRINUSE' bind error.
I ended founding the explanation on this thread: https://github.com/mdns-js/node-mdns-js/issues/31
Just one change on 'lib/server.js' and the example worked. So i made this little pull request.
Hope that helps.
P.s.: running more than one server in different process also work
Note: i tested on node v0.12.7

Looks like node v0.10.x does not support passing an object as option in 'dgram.createSocket' (https://travis-ci.org/mcollina/node-coap/jobs/98028649) :/
Maybe there is some way to check the running node version (i don't know how).


feature:克隆流程添加处理人信息

Codecov Report

Merging #462 (e6c96f1) into v2.6.x_develop (ebe6a6b) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@               Coverage Diff               @@
##           v2.6.x_develop     #462   +/-   ##
===============================================
  Coverage           62.62%   62.63%
===============================================
  Files                 441      441
  Lines               27044    27045    +1
===============================================
+ Hits                16937    16939    +2
+ Misses              10107    10106    -1

Impacted Files
Coverage Δ

itsm/service/managers.py
57.26% <100.00%> (+0.18%)
:arrow_up:

itsm/workflow/validators.py
42.39% <0.00%> (+0.21%)
:arrow_up:

Continue to review full report at Codecov.

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


Add support to REST Module for new Search term highlights

Checklist

[x] the contributor license agreement is signed
[x] commit message follows commit guidelines
[x] tests are included

Description of change

This mainly adds support to oeq-ts-rest-api to support the new highlight array which comes back from the search2 endpoint. In doing this though I noticed the return type for search was the Common.PagedResult<T> which does not match the server - and even has the resumptionToken which search2 does not use.
So inline with the Server definition I also created a specialised interface (SearchResult), one that didn't have a resumptionToken and instead does have a highlight[]. I could have looked at changing PagedResult to have a base class without resumptionToken and build from there, however server side these are not related so it made no sense to build in that relationship this side of the fence.
Once that was also done I tweaked the front-end to use it. Interestingly, because the types structurally are very similar TS was happy and everything still worked. However if these diverge further in the future we want to make sure things line up. And to further support this I changed the mocked data to TS with types.

I'll merge on green unless anyone has anything else they'd like addressed.


Supports day format in ReadableDuration

What have you changed? (mandatory)
Make util::config::ReadableDuration supports day format for Serialize/Deserialize, such as 1d2h that means one day and two hours.
What are the type of the changes? (mandatory)
It's  a improvement.It make ReadableDuration more flexible for configuration.
How has this PR been tested? (mandatory)
I modified the test_parse_readable_duration test case in util::config module, and adds new assertions for parsing the day format, including only days format and parsing days with other time units.
Does this PR affect documentation (docs/docs-cn) update? (mandatory)
No.
Does this PR affect tidb-ansible update? (mandatory)
No.
Refer to a related PR or issue link (optional)
Benchmark result if necessary (optional)
Add a few positive/negative examples (optional)

@killme2008 Thanks for your contribution! Please sign-off the DCO.

@rleungx  OK, i do it right now. Thanks.

@overvenus  ok

@overvenus  Fixed it at https://github.com/tikv/tikv/pull/3687/commits/431c5e99bb95f1e3a7b11b5655954688102c8fa8

Good job! could you sign off your commits?


Add basic integration test for segment replication

Description
This PR adds an integration test to do sanity for segment replication.
Issues Resolved
https://github.com/opensearch-project/k-NN/issues/921
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Is this the breaking change? Seems it doesn't work with rolling upgrades, or that is work in progress?
Are you going to add more tests? Seems this one uses basic settings: 1 shard, 1 replica. Also do we need to check scores and order of doc ids in result? I'm not sure what can be a side-effect of segment replication not working correctly.

Thank you @martin-gaievski for reviewing this PR.
This PR on adding tests is work related to bug on core which impacts OS 2.7.0 version. These tests are added to provide basic validation that kNN indices works with segment replication and will also supplement testing for changes we are doing in https://github.com/opensearch-project/OpenSearch/issues/3881
Added more details in https://github.com/opensearch-project/k-NN/pull/927

@dreamer-89 looks like there are no more comments for code itself, could you please re-run all the CI checks

@dreamer-89 are you still working on this PR?

@dreamer-89 are you still working on this PR?

Thanks @navneet1v @martin-gaievski for checking in. Please let me know if you have any open questions/concerns regarding this change. If not, lets move forward with PR approval and code merge.

@dreamer-89 the build is broken please fix the build.


Move syslog daemon config

The syslog daemon config is not loggregator source code and as such has
no business in code.cloudfoundry.org/loggregator. This commit moves it
up as a top level entry in Bosh's src directory.
This commit also updates the configuration of Metron and Metron Windows
to ensure that both jobs consume a syslog daemon config package as part
of their configuration. Subsequent Metron package and job configuration
remains the same.

I would appreciate someone double-checking the code here to make sure I've not made a mistake moving the various pieces.

@enocom
We attempted to deploy this to a bosh lite however encountered the following error while creating the bosh release:
bosh create-release --force
Building a release from directory '/home/pivotal/workspace/loggregator':
  - Constructing packages from directory:
      - Reading package from '/home/pivotal/workspace/loggregator/packages/metron_agent_windows':
          Collecting package files:
            Missing files for pattern 'loggregator/src/code.cloudfoundry.org/loggregator/metron/syslog_daemon_config/*'
  - Constructing jobs from manifest:
      - Expected to find package 'loggregator_common' since it's a dependency of job 'doppler'
      - Expected to find package 'golang1.8.3' since it's a dependency of job 'loggregator-acceptance-tests'
      - Expected to find package 'loggregator_common' since it's a dependency of job 'loggregator_trafficcontroller'
      - Expected to find package 'loggregator_common' since it's a dependency of job 'metron_agent'
      - Expected to find package 'syslog_daemon_config' since it's a dependency of job 'metron_agent_windows'
      - Expected to find package 'reverse_log_proxy' since it's a dependency of job 'reverse_log_proxy'
      - Expected to find package 'loggregator_common' since it's a dependency of job 'syslog_drain_binder'

Exit code 1

@bradylove I've update the PR and tested that I can create a bosh release. Will you double check that this is correct?

@enocom I tried again to deploy these changes however metron fails to start:
I was able to get better logging by doing the following on the consul VM:
consul/bfad811f-5152-40de-84ea-c7423102ccc6:/var/vcap/jobs/metron_agent/bin# ./metron_agent_ctl start
metron: no process found
metron: no process found
metron: no process found
metron: no process found
./metron_agent_ctl: line 25: /var/vcap/packages/metron_agent/syslog_daemon_config/setup_syslog_forwarder.sh: No such file or directory

@apoydence Thanks for trying again. I'm going to close this PR and will open a new PR once I've sorted out these problems.


Use ecl library version with recent bug fixes and enhancements.

Bug Fixes:
https://github.com/PX4/ecl/pull/586 - EKF: fix covariance and output filter buffer initialization
https://github.com/PX4/ecl/pull/590 - EKF: Fix innovation in fuseDeclination()
Enhancements:
https://github.com/PX4/ecl/pull/543 - ekf_helper: add more useful methods to interface with the covariances
https://github.com/PX4/ecl/pull/588 - Add unit tests for DataValidator

Additional testing required?

Reversion testing with FW and MC platforms is recommended. Individual bugfix changes have been tested, but not flown as a complete package.

Build is failing - will investigate and fix.

@priseborough we seem to have some trouble regarding https://github.com/PX4/Firmware/pull/11092 - which brings usage to PX4/ecl#543.

@TSC21 I will add the minimum changes to ekf2_main to handle the revised access to the state variances.

@dagar and @tstellanova, https://github.com/PX4/ecl/pull/588 appears to be causing the CI SITL ecl data validation tests to fail.

Short flight on a multi-rotor (pixhawk cube) showed no issues with estimator performance or CPU load:
CPU Load:

Estimator time slip:

Magnetomer:

GPs velocity:

GPS Position:

Baro:

Quaternion variances:

Quaternion Reset Count:

Should be fixed now.

Tested on Pixracer v4;
Armed, tested in position, then changed to Altitude and a few seconds made a fliped and collapsed.
https://review.px4.io/plot_app?log=2ff0d9b0-58c5-4e2d-9993-2883e2d1fe7f

Tested on Pixhawk 4 v5; no issues were noted, good flight in general.
https://review.px4.io/plot_app?log=e0b0dbb2-c743-445e-a1fc-e17bc89c3d42
Master was tested as well; no noticeable differences.
https://review.px4.io/plot_app?log=c3e08940-4078-466e-aff1-853e28aad108
Tested on Pixhawk 4mini v5; no issues were noted, good flight in general.
https://review.px4.io/plot_app?log=dc3b8978-3879-4e5c-9f76-f059558fa7e7
Master was tested as well; no noticeable differences.
https://review.px4.io/plot_app?log=a6268987-ab5c-4d87-8bbf-e503b30964f1

Tested on Pixhawk cube v2;
It was tested in all modes Position, Altitude, Stabilized, Mission and RTL and it works very well in all modes.
https://review.px4.io/plot_app?log=27081981-026b-4c64-804a-cbad637f821e

@priseborough Sound like issues persist here.

@jorge789 Your crash had nothing to do with the estimator. Innovations were within limits throughout.

Tested on Pixracer v4;

Armed.
Takeoff on position mode and flew as expected.
Switched to altitude mode, a few seconds later vehicle made flipped and collapsed.

https://review.px4.io/plot_app?log=2ff0d9b0-58c5-4e2d-9993-2883e2d1fe7f

I'm not seeing an obvious issue in this log. Did your battery die? Is your power module (current/voltage sensor) calibrated? https://docs.px4.io/en/config/battery.html
https://review.px4.io/plot_app?log=2ff0d9b0-58c5-4e2d-9993-2883e2d1fe7f#Nav-Power


Community feedback before release 0.8

Dear All,
before officially releasing 0.8, we would like to get your feedback on the pre-release (check https://github.com/smartsdk/ngsi-timeseries-api/releases/tag/0.8-alpha), that provides a number of interesting new features and bug fixes, including initial support for NGSI-LD.
PR #373 introduced basic support for NGSI-LD relying on v2 API. In short this means that using the current endpoint QL can store NGSI-LD payloads with few caveats. More details here.
@agaldemas
@blackbearst
@Chandradeep-NEC
@cnoelle
@daminichopra
@ES-SC
@fisuda
@fmonsanto
@HaythemHorbit
@heikkilv
@HOP-Ubiquitous
@ekatsiri
@iamarnavgarg
@iaoiui
@jason-fox
@VeroCity-UG
@jcheca
@jlohmoeller
@juliozinga
@lmreyes
@luisdelacalle
@msgis
@ohylli
@PepeluDev
@pooja1pathak
@SakshiSharma-India
@Trigyn-Technologies
@vjain641
@ilarimikkonen
@pinkerltm
@pierre-josselin

hi, more than happy to give feedback. Which Crate version would you recommend to use? and which version of Redis? We had done some initial tests with redis-cache image and the results were pretty good. When we have time, we do some load tests.

@ilarimikkonen crate 4.1.4 is the base version used in the tests. We may do a version upgrade, but probably only after the release of 0.8. Redis we are using the last version: 6.0.9 (probably would be better to fix it)

@chicco785 few initial questions/observations:
In our initial tests (with redis-cache ) we got the following results on last run:
Setup: 100 000 messages approx. 300 messages / sec.
Messages send to Orion and then with a sub to QL. Running on Kubernetes.
There was ~50% message loss with one QL and ~20% with five.

Is 0.8 and redis-cache image any different in performance wise?
Could you describe the architecture and behaviour of Redis caching? We saw this behaviour during testing (this is write rate: rate(container_fs_writes_bytes_total{}[5m]):

as you can see, we ran few times, 1st with lower load and then increasing. We have trouble understanding what Redis is doing.

A bit of documentation is here:

https://quantumleap.readthedocs.io/en/latest/#cache-back-end
You can find also tests here:
https://quantumleap.readthedocs.io/en/latest/admin/benchmarks/

I don't think there are difference between the two images in term of performance. In short, as of today, redis is used to avoid to read every time from the db the metadata, which showed to slow down a lot inserts.
In general it is better to test ql load without orion, because that's the only way you can be sure that if you send 3000 messages, 3000 should reach ql. Recall that if for whatever reason a new message for an entity comes before it was sent the previous notification, that won't be sent.
with 1 ql and 30 sec timeout, we have no issue in processing 100 messages / sec on a laptop. surely, tweaking gunicorn configuration more can be done (but we haven't tested). Then to change significantly, probably we will need to move away from flask.
in general, as you can see here:
https://github.com/smartsdk/ngsi-timeseries-api/issues/349#issuecomment-704575150
avg response time of crate during insert is 50 msec, ql transformation and insert code adds to it only 30 msec. which lead us to think that the web server has a quite a role (see copied discussion from the comment above):

so what's adding the 3 seconds? a profiler would be of help to get deeper, but gunicorn + flask may have quite a role in that, as well as python garbage collection. surely, fine tuning gunicorn configuration may be quite important to achieve best performance.
as said, for performance, flask seems to be not ideal and a good alternative could be https://fastapi.tiangolo.com/
still, 105 reqs per second on a laptop does not look bad. at the time being, we will work on consolidating pr #373, but we won't invest on fastapi (we don't have a the moment a use case / customer that justify that investment).

avg response time of crate during insert is 50 msec, ql transformation and insert code adds to it only 30 msec

It looks like we spend the rest of the time doing network I/O---web server isn't necessarily at fault here. See https://github.com/smartsdk/ngsi-timeseries-api/issues/399#issuecomment-733708372


Interpolation inside attributes has been removed

When I try to run the Hello World example from the readme file, I get this error in the console:

[Vue warn]: class="col-{{col.key}} col-cell {{canOrderBy(col.key) ? 'ord' : ''}} {{orderClass(col.key)}}": Interpolation inside attributes has been removed. Use v-bind or the colon shorthand instead. For example, instead of <div class="{{ val }}">, use <div :class="val">.

My package.json has:
"vue": "^2.0.5",
"vue-router": "^2.0.1",
"vue-smart-table": "^2.6.0-beta5"

Any ideas on how to avoid this error and run the Hello World example?  Thanks.

Hi vote539, vue-smart-table is not compatible with Vue 2.0 I'm currently busy with other projects.


Development proxy ignores path in K8s API URL

Bug Report
What did you do?

set ~/.kube/config to a Rancher-managed cluster's API
(sth. like https://my-domain.tld/k8s/clusters/c-some-id)
kubectl apply […] to deploy CRD
operator-sdk run to run operator

What did you expect to see?
I expected both steps 2 and 3 to target the cluster configured in step 1.
What did you see instead? Under which circumstances?

kubectl apply works as expected
operator-sdk run ignores the path component when setting up the proxy and uses https://my-domain.tld instead

In my particular case this didn't actually surface as an error. Instead the modified API endpoint without a path happened to target the management cluster of my Rancher deployment instead of a development cluster. So it actually ran the operator against a different cluster without any warning at all.
Environment
Operator type:
/language ansible
Kubernetes cluster type:
Rancher, RKE
$ operator-sdk version
operator-sdk version: "v1.7.1-19-g3ce1eeef", […], kubernetes version: "v1.19.4", […]
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"21", […]}
Server Version: version.Info{Major:"1", Minor:"20", […]}
Possible Solution

To prevent people from shooting themselves in the foot I'd suggest that operator-sdk run (and any other commands that might behave similarly) fail with an error message if they encounter an API endpoint with a path component.
It would be nice if Operator SDK could be improved to work with clusters that have an API with a path component, such as the ones behind a Rancher instance or other proxies/multiplexers.

Additional context

https://github.com/kubernetes/kubernetes/issues/94280
https://kubernetes.slack.com/archives/CAW0GV7A5/p1620896783095000

Fix for this issue is dependent on kubectl PR. Once this PR is merged then only we will be able to move ahead with this fix as it has changes in apimachinery.

Created another issue #4980 which will fix this issue and throw an error for now. Once kubectl PR gets merged then we will move ahead with changes.

Removing this from v1.9.0 milestone as #4980 (in milestone v1.9.0) is tracking this issue.

As part of the Operator SDK v1.9.0 release, we have moved temporary fix for this issue. It will throw an error if API endpoint has Path in it. @rbq Could you please try with new version of Operator SDK. You can download the release and view the changes here: https://github.com/operator-framework/operator-sdk/releases/tag/v1.9.0

As part of the Operator SDK v1.9.0 release, we have moved temporary fix for this issue. It will throw an error if API endpoint has Path in it. @rbq Could you please try with new version of Operator SDK. You can download the release and view the changes here: https://github.com/operator-framework/operator-sdk/releases/tag/v1.9.0

Checked, throws an error as expected. :)

@rbq - thank you for you confirmation :)

Till now, we did the temporary fix for this issue as this issue was dependent on kubectl PR. kubectl PR got merged and we are moving ahead with the permanent fix for this issue.

This was waiting on k8s 1.23 release per the above comment. We merged k8s 1.23 in PR #5505

I'm getting the message "[...] contains a path component, which the proxy server is currently unable to handle properly. Work on this issue is being tracked here: [...]" in 1.19.0.

/cc @jmrodri

thank you for bringing this @rbq - We are working on it and will fix this soon.

@rbq - We removed the change that was throwing an error when there is a path in the proxy server. It will be available in the 1.20 operator-sdk release.

@laxmikantbpandhare That's great to hear, thanks a lot!

@laxmikantbpandhare Are releases cut based on schedule (the milestone says due today) or features?

@rbq - We planned to do it by EOD tomorrow. I will update you here.

@rbq - We have released Operator SDK v1.20.0. Could you please verify and let us know ur observations.

@rbq - that's great, thank you for the confirmation.


Adding support for earlier versions of Unity

Specifically tested in 5.2.3 but should work in earlier versions as well.

Thanks for fixing that! I was noticing that before but somehow forgot itxD You'll get the 100th commit ;)


New app: tempgraph

Description
Added new app TempGraph
Copilot

🤖 Generated by Copilot at c89f0b6
Summary
📄🚀🌡

This pull request adds a manifest.yaml and a README.md file to the tempgraph app, which is an app that displays a graph of the temperature history for a given location. These files provide the necessary metadata and documentation for the app to be compatible and comprehensible with the tidbyt platform and its users.

tempgraph app grows
manifest and README files
autumn harvest time

Walkthrough

Create a new app called tempgraph that displays a line chart of the temperature and humidity data from a specified location using the OpenWeatherMap API (link, link)
Add a manifest.yaml file that defines the app's metadata, such as its id, name, summary, description, author, file name, and package name (link)
Add a README.md file that provides an overview of the app, its features, its API details, its configuration options, its error handling, and its future improvements (link)

Several errors were generated because I inadvertently had some other projects in the community/apps directory and committed them, which affected this pull request. They have been removed and this app is the only thing I am submitting at this time.

Original Author got messed up and switched to a tidByt employee


For id: Gather defendants address should we add a checkbox for "Unknown" that writes "Unknown" for user rather than making them write it on the required lines?

They would still be able to do that. Right now it requires the user to input 3 parts of the address (Street, Town/City, State). Instead of having the user type out "Unknown" in these fields twice and choose a random state, I suggest having a checkbox (in addition to showing all address fields) that allows the user to say the address is entirely Unknown.


Validity of results

Good afternoon!
I'm sorry to bother you.
Could you please advise me what could supposedly be wrong with my results?
I saw a thread above with a similar case.
But I'm confused by the fact that my results are so different between bp and edges. Could this be due to the fact that I didn't use any masking for the satellite sequence?

Dear Anna, apologies for the late response--I have been on vacation. Edge and bp count have different properties, and that's why they also exhibit different growth curves. For Instance, it is expected that the core genome (coverage >= 1, quorum = 100%) of edges tends converges towards 0, as edges that are shared between all genomes should not exist, unless in presence of segmental duplication.
From the figures alone, I cannot spot any obvious mistake of your analysis.

Thanks so much for the reply! This has helped me a lot. I was just looking at the HPRC example where the common part is preserved, even though they have a lot more genomes represented and I was worried.
Thank you for your work!

You're welcome. Feel free to post further questions if you have!


Automatic detection of dark or light mode of page form user browser.

Is your feature request related to a problem? Please describe.
Automatic detection can be done from user browser and when user opens website same mode will be applied to page at first user can change it afterwards.
Describe the solution you'd like.
Addition of javascript code that detects which theme is currently applied by user .
Describe alternatives you've considered.
No response
Additional context.
No response
Show us the magic with screenshots
No response
Checklist

[X] I have checked the Existing Issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

sure go ahead
Make sure to join the discord server for further communication
Don't forget to star the repository
Complete the task in next 2 days and kindly mention your issue # in your pr like this:
Closes #IssueNumber
Note: the level of the issue may change based on the quality of the PR code

Its been weeks and no pull request was made so we are closing this issue.


 error during component test

I have a component <GlobalHeader/> that uses the <LoginLink/> component. When I attempt to test <GlobalHeader/> I get an error:
<GlobalHeader /> Component
    ✖ "before each" hook for "renders"
      PhantomJS 2.1.1 (Mac OS X 0.0.0)
    null is not an object (evaluating 'router.getLoginRoute')

This appears to trace back to <LoginLink/> but I can't determine why it's broken. This only happens during testing, it renders just fine outside of tests.

Thanks @mheisig for the report, can you also tell us which testing frameworks and patterns you are using?

This appears to be related to an older version of react-router that I was using. Updating to the latest appears to have solved the problem:
"react-router": "^3.0.0",
"react-router-redux": "^4.0.7",

FWIW, this was using Karma, Mocha, and Enzyme.

@mheisig Great to hear that updating your react-router version solved the issue!


fix(artifact): log the error that should not raise

Because

we do not have to raise error when the users plan to upload a file to a catalog

This commit

log error but not raise error

Will be done by another PR

will be done by another PR


Strange behavior when uploading mission while in mission mode

At least in SITL the VTOL_TAKEOFF command does not transition when the mission is uploaded while in mission mode, it also ascends much faster. I think more problems can arise when this happens.

I have a different behaviour on my mission code but the same basic problem. Will try to sync with master and solve this.

I believe this is resolved? Reopen if not.


Improve wasi support

Hello maintainer, I don't know if the maintenance team has considered improving the project's support for wasi ? Looks like it hasn't been updated for a while.

https://github.com/bytecodealliance/wasmtime-go/blob/main/wasi.go

In the past six months, the wasi support of wasmer-go has become better. Will it refer to the implementation of functions similar to GenerateImportObject ?

https://github.com/wasmerio/wasmer-go/blob/master/wasmer/wasi.go

Sincerely

Thanks for the report, but do you have a specific issue or problem that can be addressed? If not I'm not sure how best to help with this issue.

Can users refer to wasmer-go and use wasmtime-go invoke wasm program which has the wasi standard?
like this: https://github.com/wasmerio/wasmer-go/commit/49b556ba29cc8816ffe91f09e0846151b2dc0810
In the current code, I don’t seem to find an implementation similar to GenerateImportObject function, It may be very troublesome to call the wasi program.

There is a wasi example in the documentation and this crate has its own take on the API for support, inspired by the Rust crate support.
If you've got an issue or a problem I'm happy to help, but otherwise I think the example covers the question here so far.


Support arm64 builds

Provides arm64 binaries, and makes GOARCH configurable at build time for Docker builds.
fixes #393

I've made an arm64 build available here for testing.


Passwords are stored in plain text in the database.

Just installed this using the MySQL database and checked the users table and found that passwords are stored in plain text, can this be changed to encrypted passwords?

Yes, it can be... When I will realize how it do, and does not break existing passwords while updating

I was fixed it, update please to <IP_ADDRESS>


Create Services project

What is the problem that this PR is trying to fix?
We want to run the services for Travis CI to run separate from the jobs/workers. This PR should provide a GCP project and Kubernetes cluster for this.
What approach did you choose and why?

Initialize/configure project from Terraform
Use GCS for tfstate storage
New networking module to not include the complicated NAT as on the workers.
Refactored Kubernetes module.

How can you test this?
Setup a staging project first.
What feedback would you like, if any?
Sure, feel free to discuss.

I was able to set this up on the staging project. Once billing is set up for production project, this can move forward.

https://github.com/travis-ci/terraform-config/pull/684/commits/30b6bdcd567e99fadbac3b822a91c025b7541a8c adds a target to create the Terraform state bucket, this should be run once before running Terraform.


fix: fixed border radius issue in gradient border

Fixed border radius issue in gradient border section
🛠 Fixes Issue
Closes #213
👨💻 Changes proposed

Fixed border radius issue in gradient border section.
Now the border radius gets set to 0 when one uncheck the checkbox, or get the results

✔ Check List

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

:tada: This PR is included in version 2.4.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Publish Hystrix BOM POM

Please publish Hystrix BOM POM - a POM with dependency management section listing all of the Hystrix modules (contrib included) and their version, so it's easier to enforce consistent version of all Hystrix modules in projects using Hystrix.

Would you like to contribute a PR for this?

Closing due to inactivity


Upgrade Symfony to 5.3

Fixes #480

Todo:

[ ] Replace security layer
[ ] Refactor user providers services (providers no longer required, can be changed to query)


Update to Quick Start

editorial: two words

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


add maven into gitignore

add maven into gitignore
Reasons for making this change:
TODO
Links to documentation supporting these rule changes:
TODO
If this is a new template:

Link to application or project’s homepage: TODO

@HGuedes thanks for the contribution, but we already have a template for Maven at Maven.gitignore.


Novosibirsk, Russia

Data from their official ESRI server, without license information. The address data is in a "table" and is linked to buildings with geometry, so it was a clunky custom pyesridump script to download it. Let me know if you want that uploaded too.
Probably will not work straight away since csv requires lat/lon and this has WKT building geometry. Can get centroids instead of buildings, or maybe there is a way to provide WKT to the machine?

I have added the centroid coordinates to the file and kept the WKT polygons of the houses in case they are needed in the future.

Re-run this please.

This looks like it might be the city of Novosibirsk, not the oblast, but the addresses look good. The build failure looks like something we fixed a while back in master with JS parsing Python regexes. Maybe try merging in master?

It's a bit in the middle: it's much larger than the city itself, but only covers the Eastern third of the region (though the vast majority of the population): http://loi.sscc.ru/bdm/nso/big/dorogi/road_oblast.jpg

@trescube any ideas?

Re-run this please.


call useImage with conditional DataSource / uri

Since hooks cannot be called conditionally, I'd expect to be able to send null / undefined to useImage.

I'm caching images, and because of that I don't have the file url available on mount.
Right now I'm sending an empty string to useImage, causing it to do an unnecessary fetch, but preferably I'd use null or undefined.

What about instead of null giving an empty Uint8Array? then you have length=0 and you know that you didn't give anything.
Even better, you could create this empty array outside the component and to an identity check:
const NO_IMAGE = new Uint8Array(0);

if (image === NO_IMAGE) {
  // do this...
}

Closing for now but if the suggested solution is not good, please reopen this issue 🙏

Hi William,
Here's an example of how we're using useImage:
const Item = ({ remoteImage, localImage }: { remoteImage?: string; localImage?: DataSource }) => {
  const cachedImage = useImageCache(remoteImage);
  const skiaImage = useImage(remoteImage ? cachedImage : localImage);

  return (
    <TouchableOpacity>
      {!!skiaImage && (
        <Canvas style={{ width, height }}>
          <Image width={width} height={height} image={skiaImage}/>
        </Canvas>
      )}
    </TouchableOpacity>
  );
};

This resulted in empty 'GET' requests, so now we've implemented your suggestion:
const skiaImage = useImage(remoteImage ? cachedImage || NO_IMAGE : localImage);

This does cause useImage to do unnecessary work, processing NO_IMAGE and initially rendering the canvas while it's not yet needed.
Ideally when we pass null into useImage we get null as return value so the canvas will not render when using !!skiaImage.

Hi @laurens-lamberts - we'll check this out, we also have a PR regarding this topic #678 which might be related? I don't think we've made any changes between the revisions you are referring to - would it possible for you to test the PR that @JonnyBurger made?


[jsfm] upgrade weex-js-framework to 0.22.4

Upgrade weex-vue-framework to 2.4.2-weex.6, make sure the document can be cleared event the destroyed or beforeDestroy lifecycle hook is failed.

The file change is https://github.com/Hanks10100/vue/commit/dbebd36fa18809d744bb27242ac8741dbdb2cbe7


Feat/unverified project boost amount

https://github.com/Giveth/GIVeconomy/issues/739

@mohammadranjbarz I changed the migration file name to make it run again! do you think it can work? I thought it can be a better idea rather than having two migration files for single action

@mohammadranjbarz I changed the migration file name to make it run again! do you think it can work? I thought it can be a better idea rather than having two migration files for single action

Good idea, I think it's better

Ok, merged it. @mohammadranjbarz Please have a look on the deploy and migrations

Ok, merged it. @mohammadranjbarz Please have a look on the deploy and migrations

Thanks @aminlatifi everything seems good
@maryjaf  You can test it on UAT now


Open source JS navigator

It sounds so much simpler than UINavigationController bridging. cc @vjeux @jordwalke
Thanks!

http://facebook.github.io/react-native/docs/navigator.html
https://github.com/facebook/react-native/blob/master/Examples/UIExplorer/Navigator/NavigatorExample.js
https://github.com/facebook/react-native/tree/master/Libraries/CustomComponents/Navigator
Fortunately we got this in for the open source release. Let me know what you think!

With some custom nav bar implementations that simulate iOS's style, this could quickly surpass UINavigationController. Really great @ericvincenti. BTW I noticed some small issues causing crashes with the UIExplorer example and put up #200 as a fix.

:+1:
Thanks @ericvicenti
@tadeuzagallo is working on getting hardware touches to be aligned with the screen refresh rate/cycle which is going to make it even better!
Check out the F8 app which uses this for all of its transitions/gestures.


False positive unused variables warning inside CEs

I am using SqlHydra in one of my project. It uses CEs to build SQL queries and I found out that Rider report unused variables inside CEs even if they are used.
person and account are reported unused when they are actually used.

On simpler query without join the warning goes away:

I was able to reproduce the problem using Linq CE syntax:

Both place and place1 are reported unused.
let people = [
      ("Tomas", 1)
      ("Joe", 2)
      ("Don", 1)
  ]

  let places = [
      (1, "Cambridge")
      (2, "Wisconsin")
  ]

  let x = query {
      for person in people do
          join place in places on (snd person = fst place)
          join place1 in places on (snd person = fst place1)
          select (fst person, snd place)
  }

Same happens when using  SQLProvider

@MangelMaxime Thank you for reporting this. This warning is produced by FCS, and I've created an issue there: https://github.com/dotnet/fsharp/issues/14566

@edgarfgp Could you share any example, please? Preferably, in the F# repo.

I'll close this issue, let's discuss it in the linked one instead.

I've pushed a temporary workaround on our side, until the issue is fixed in FCS.

Awesome thank you @auduchinok


reshaping in detection heads crashes graph

Issue Type
Bug
Source
source
MCT Version
main
OS Platform and Distribution
ubuntu 20.04, pytorch 2, cuda 12
Python version
3.11
Describe the issue
In detection tasks, we often see this pattern at the end of heads:

# Permute output from (N, A * K, H, W) to (N, HWA, K)
N, _, H, W = cls_logits.shape
cls_logits = cls_logits.view(N, -1, K, H, W)
cls_logits = cls_logits.permute(0, 3, 4, 1, 2)
cls_logits = cls_logits.reshape(N, -1, K)  # Size=(N, HWA, K)

FX won't trace tensor.view(), so we replace it by tensor.reshape().

However, this block does work with MCT but with the code before it crashes with error like this:

File "model_compression_toolkit/core/pytorch/back2framework/pytorch_model_builder.py", line 94, in _run_operation
out_tensors_of_n_float = op_func(*input_tensors + op_call_args, **functional_kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
RuntimeError: shape '[-1, 9, 91, 25, 42]' is invalid for input of size 532350

Commenting this code, then MCT works.

So the issue seems to be about shape inference in Graph because FX ShapeProp does trace correctly.

Expected behaviour
Memory manipulations like view, reshape, permute don't change activation quantization.
MCT should infer shapes correctly.

Code to reproduce the issue
Any torchvision detection model such as FCOS, retina, SSD.

Log output
No response

Hi @mikeseven,
I managed to run Retinanet (without pre/post process) from torchvision with similar code for logits and box_regressions:
N, C, H, W = logits.shape
logits = logits.view(N, -1, C//self.num_anchors, H, W)
logits = logits.permute(0, 3, 4, 1, 2)
logits = logits.reshape(N, -1, C//self.num_anchors)  # Size=(N, HWA, 4)
Please provide a code example of your network to explore this issue.
Thanks, Elad.

Der Elad,
I synced to the latest code and tried on retinanet. Same error.
As you see below, the issue occurs during calibration on COCO dataset whose images have different sizes after pre-processing. This could be the reason.... gotta check.
Thanks.
  File "model_compression_toolkit/core/runner.py", line 248, in _prepare_model_for_quantization
    mi.infer(_data)
  File "model_compression_toolkit/core/common/model_collector.py", line 95, in infer
    tensor_data = self.fw_impl.run_model_inference(self.model, inputs_list)
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "model_compression_toolkit/core/pytorch/pytorch_implementation.py", line 178, in run_model_inference
    return model(*to_torch_tensor(input_list))
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/lib/python3.11/site-packages/torch/nn/modules/module.py", line 1519, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/lib/python3.11/site-packages/torch/nn/modules/module.py", line 1528, in _call_impl
    return forward_call(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "model_compression_toolkit/core/pytorch/back2framework/pytorch_model_builder.py", line 282, in forward
    out_tensors_of_n, out_tensors_of_n_float = _run_operation(node,
                                               ^^^^^^^^^^^^^^^^^^^^
  File "model_compression_toolkit/core/pytorch/back2framework/pytorch_model_builder.py", line 94, in _run_operation
    out_tensors_of_n_float = op_func(*input_tensors + op_call_args, **functional_kwargs)
                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
RuntimeError: shape '[-1, 9, 91, 25, 42]' is invalid for input of size 532350

Confirmed. It's an input size problem, so not an issue with mct. Closing.


[2.070][ndslice] upgrade win*.mak & uncomment unittest with dummyranges

@MartinNowak this PR upgrades the win*.mak files to build docs on windows and uncomments a unittest with dummyranges. The unittets was commented before #3806 was merged.

Sorry there is now a merge conflict b/c I merged back some missing changes from the stable branch.

Rebased

Rebased again.
I have found a bug for allocators on windows:
std\experimental\allocator\mallocator.d(277): Error: @nogc function 'std.experimental.allocator.mallocator.AlignedMallocator.alignedReallocate' cannot call non-@nogc function 'std.experimental.allocator.mallocator.AlignedMallocator.deallocate'

I will report it.

Yes, https://issues.dlang.org/show_bug.cgi?id=15508

I will check it again.

But workaround is removed https://github.com/D-Programming-Language/phobos/commit/4cf352c19db9fff49dc7dbbf3ea763c648735748

Fixed, it can be merged now.

Auto-merge toggled on

Thanks! Should I open the same PR for stable branch or it can be merged to stable another way?

The same PR for stable #3905


is there any solution when there are many observations per respondent

like the question titled, and when I want to segment respondent applying this stepmix package

thank you

@ericlacourse pointed out that you could consider RMLCA which can be achieved (I think) with StepMix. You would need to structure your n respondents with k repeated measurements as an n x k data frame and use this as input to StepMix.

Closing. Feel free to reopen if needed.