Devops 012

Devops 012

Ran Plan for 4 projects:

dir: deployment/all-gcp workspace: default
dir: deployment/service/ols-svc-profile workspace: default
dir: deployment/service/ols-svc-talkgpt workspace: default
dir: deployment/service/ols-svc-transribegpt workspace: default

1. dir: deployment/all-gcp workspace: default
Show Output
Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
+ create

Terraform will perform the following actions:

  # module.gce-atlantis.local_file.ansible_vars[0] will be created
+ resource "local_file" "ansible_vars" {
      + content              = (sensitive value)
      + content_base64sha256 = (known after apply)
      + content_base64sha512 = (known after apply)
      + content_md5          = (known after apply)
      + content_sha1         = (known after apply)
      + content_sha256       = (known after apply)
      + content_sha512       = (known after apply)
      + directory_permission = "0777"
      + file_permission      = "0777"
      + filename             = "atlantis/ansible_vars.json"
      + id                   = (known after apply)
    }

  # module.gce-atlantis.local_file.private_key will be created
+ resource "local_file" "private_key" {
      + content              = (sensitive value)
      + content_base64sha256 = (known after apply)
      + content_base64sha512 = (known after apply)
      + content_md5          = (known after apply)
      + content_sha1         = (known after apply)
      + content_sha256       = (known after apply)
      + content_sha512       = (known after apply)
      + directory_permission = "0700"
      + file_permission      = "0400"
      + filename             = "atlantis/id_rsa.pem"
      + id                   = (known after apply)
    }

Plan: 2 to add, 0 to change, 0 to destroy.

:arrow_forward: To apply this plan, comment:

atlantis apply -d deployment/all-gcp

:put_litter_in_its_place: To delete this plan click here
:repeat: To plan this project again, comment:

atlantis plan -d deployment/all-gcp

Plan: 2 to add, 0 to change, 0 to destroy.

2. dir: deployment/service/ols-svc-profile workspace: default
Plan Failed: This project is currently locked by an unapplied plan from pull #3. To continue, delete the lock from #3 or apply that plan and merge the pull request.
Once the lock is released, comment atlantis plan here to re-plan.

3. dir: deployment/service/ols-svc-talkgpt workspace: default
Show Output
Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
+ create
-/+ destroy and then create replacement

Terraform will perform the following actions:

  # module.argocd_app.helm_release.helm will be created
+ resource "helm_release" "helm" {
      + atomic                     = false
      + chart                      = "argocd-apps"
      + cleanup_on_fail            = false
      + create_namespace           = false
      + dependency_update          = false
      + disable_crd_hooks          = false
      + disable_openapi_validation = false
      + disable_webhooks           = false
      + force_update               = false
      + id                         = (known after apply)
      + lint                       = true
      + manifest                   = (known after apply)
      + max_history                = 0
      + metadata                   = (known after apply)
      + name                       = "ols-talkgpt"
      + namespace                  = "cd"
      + pass_credentials           = false
      + recreate_pods              = false
      + render_subchart_notes      = true
      + replace                    = false
      + repository                 = "https://argoproj.github.io/argo-helm"
      + reset_values               = false
      + reuse_values               = false
      + skip_crds                  = false
      + status                     = "deployed"
      + timeout                    = 300
      + values                     = (sensitive value)
      + verify                     = false
      + version                    = "1.4.1"
      + wait                       = true
      + wait_for_jobs              = false
    }

  # module.workload_identity.google_project_iam_member.sa_iam must be replaced
-/+ resource "google_project_iam_member" "sa_iam" {
      ~ etag    = "BwYEn3HKRf0=" -> (known after apply)
      ~ id      =<EMAIL_ADDRESS>-> (known after apply)
      ~ role    = "roles/cloudkms.cryptoKeyEncrypterDecrypter" -> "roles/speech.admin" # forces replacement
        # (2 unchanged attributes hidden)
    }

Plan: 2 to add, 0 to change, 1 to destroy.

:arrow_forward: To apply this plan, comment:

atlantis apply -d deployment/service/ols-svc-talkgpt

:put_litter_in_its_place: To delete this plan click here
:repeat: To plan this project again, comment:

atlantis plan -d deployment/service/ols-svc-talkgpt

Plan: 2 to add, 0 to change, 1 to destroy.

4. dir: deployment/service/ols-svc-transribegpt workspace: default
Show Output
Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
+ create

Terraform will perform the following actions:

  # module.argocd_app.helm_release.helm will be created
+ resource "helm_release" "helm" {
      + atomic                     = false
      + chart                      = "argocd-apps"
      + cleanup_on_fail            = false
      + create_namespace           = false
      + dependency_update          = false
      + disable_crd_hooks          = false
      + disable_openapi_validation = false
      + disable_webhooks           = false
      + force_update               = false
      + id                         = (known after apply)
      + lint                       = true
      + manifest                   = (known after apply)
      + max_history                = 0
      + metadata                   = (known after apply)
      + name                       = "ols-transcribegpt"
      + namespace                  = "cd"
      + pass_credentials           = false
      + recreate_pods              = false
      + render_subchart_notes      = true
      + replace                    = false
      + repository                 = "https://argoproj.github.io/argo-helm"
      + reset_values               = false
      + reuse_values               = false
      + skip_crds                  = false
      + status                     = "deployed"
      + timeout                    = 300
      + values                     = (sensitive value)
      + verify                     = false
      + version                    = "1.4.1"
      + wait                       = true
      + wait_for_jobs              = false
    }

  # module.workload_identity.google_project_iam_member.sa_iam will be created
+ resource "google_project_iam_member" "sa_iam" {
      + etag    = (known after apply)
      + id      = (known after apply)
      + member  = (known after apply)
      + project = "ols-platform-dev"
      + role    = "roles/cloudkms.cryptoKeyEncrypterDecrypter"
    }

  # module.workload_identity.google_service_account.gsa will be created
+ resource "google_service_account" "gsa" {
      + account_id   = "ols-svc-transcribegpt"
      + disabled     = false
      + display_name = "Service Account for ols-svc-transcribegpt service"
      + email        = (known after apply)
      + id           = (known after apply)
      + member       = (known after apply)
      + name         = (known after apply)
      + project      = "ols-platform-dev"
      + unique_id    = (known after apply)
    }

  # module.workload_identity.google_service_account_iam_binding.workload_identity_binding will be created
+ resource "google_service_account_iam_binding" "workload_identity_binding" {
      + etag               = (known after apply)
      + id                 = (known after apply)
      + members            = [
          + "serviceAccount:ols-platform-dev.svc.id.goog[dev/ols-svc-transcribegpt]",
        ]
      + role               = "roles/iam.workloadIdentityUser"
      + service_account_id = (known after apply)
    }

Plan: 4 to add, 0 to change, 0 to destroy.

:arrow_forward: To apply this plan, comment:

atlantis apply -d deployment/service/ols-svc-transribegpt

:put_litter_in_its_place: To delete this plan click here
:repeat: To plan this project again, comment:

atlantis plan -d deployment/service/ols-svc-transribegpt

Plan: 4 to add, 0 to change, 0 to destroy.

:fast_forward: To apply all unapplied plans from this pull request, comment:

atlantis apply

:put_litter_in_its_place: To delete all plans and locks for the PR, comment:

atlantis unlock

atlantis apply

Ran Apply for 3 projects:

dir: deployment/all-gcp workspace: default
dir: deployment/service/ols-svc-talkgpt workspace: default
dir: deployment/service/ols-svc-transribegpt workspace: default

1. dir: deployment/all-gcp workspace: default
Show Output
module.gce-atlantis.local_file.private_key: Creating...
module.gce-atlantis.local_file.ansible_vars[0]: Creating...
module.gce-atlantis.local_file.private_key: Creation complete after 0s [id=ebd1b3819c3b50a3f97a1c51a49ceddc5a6d33a7]
module.gce-atlantis.local_file.ansible_vars[0]: Creation complete after 0s [id=31fd1b5aa743fed520ef01064082d8436d14129d]

Apply complete! Resources: 2 added, 0 changed, 0 destroyed.

Outputs:

blast_dns_id = "projects/ols-platform-dev/managedZones/ols-network-dns-blast"
blast_dns_managed_zone_id =<PHONE_NUMBER>21292270
blast_dns_name = "ols.blast.co.id."
blast_dns_name_servers = tolist([
  "ns-cloud-a1.googledomains.com.",
  "ns-cloud-a2.googledomains.com.",
  "ns-cloud-a3.googledomains.com.",
  "ns-cloud-a4.googledomains.com.",
])
blast_dns_zone_name = "ols-network-dns-blast"
blast_dns_zone_visibility = "public"
blast_private_dns_id = "projects/ols-platform-dev/managedZones/ols-network-dns-blast-internal"
blast_private_dns_managed_zone_id =<PHONE_NUMBER>158254910
blast_private_dns_name = "internal.ols.blast.co.id."
blast_private_dns_name_servers = tolist([
  "ns-gcp-private.googledomains.com.",
])
blast_private_dns_zone_name = "ols-network-dns-blast-internal"
blast_private_dns_zone_visibility = "private"
main_cluster_ca_certificate = "LS0tLS1CRUdJTiBDRVJUSUZJQ0FURS0tLS0tCk1JSUVMRENDQXBTZ0F3SUJBZ0lRZmRZSWo2Qlh6K1NvWm1JSmUxV29MVEFOQmdrcWhraUc5dzBCQVFzRkFEQXYKTVMwd0t3WURWUVFERXlSbE1tSTJaR1EzWVMwd01ERXdMVFE0WXpVdE9UY3laUzB6WlRSaU0yRmlOVGM1TjJFdwpJQmNOTWpNd09UQTBNRFUwTkRBNFdoZ1BNakExTXpBNE1qY3dOalEwTURoYU1DOHhMVEFyQmdOVkJBTVRKR1V5CllqWmtaRGRoTFRBd01UQXRORGhqTlMwNU56SmxMVE5sTkdJellXSTFOemszWVRDQ0FhSXdEUVlKS29aSWh2Y04KQVFFQkJRQURnZ0dQQURDQ0FZb0NnZ0dCQUpZeTZPRjh3T29BMjRlNjNocU10TEtJQm1HdmQvdlVJQ2lleTBuSQpvRUtYYUhYUm1RNXk1eU4xbXZDQ3Y2MTJicW1NOTMyY1Fsb3dncExOQzhVdkxscGRoUS9UbmtvRDUxOE9oTmx0Clo4a1p5MVQ4enFTS1VVVm9KNEpka0hEMDF0bEx0VlJya3hTclVFOUhkcFZyQVV0eFFJS0Zkb2JQZlNkYXVtWU8KcGw4NUZnRjdMbW11aEhDejVock1wSHhXMUxyVFUzVytoQTVGRllNZExSeVJ5WlIzQmtXc0xNRVk1ampMUW0ycQpzUTBZMEhnR2poekpMd3dmNExEdGVEOW5MeTFMaW9GaURpWWVadVBiQkZBN09IVWNjMlYyMWt0T2dhOExpS1U5CkhEVFlYOXIxMGxZSlN0RGFIWVJldDVMa1llMmFTYm53aXR3dmtCTnhLcE5Uek1TVlkwUTljOWtGRnhEREVNOUQKWFpzQ0NVTXE3akhzcWZYSWdqdWN2NmpVaExFVVozd3d4TW1EdmVhNUoxZUc2SEV0WDFCa2FKN0ZsZWhEbVg5MgpQbDZTQ1JrR0x4VVA2ZFBTWFhhQkZxUndMZzcxVzFjKzQyeWFTSTlKNHdUS0x6U05DVzVuS3U0Z3NDNk0xRlNTCi93Mzl0dE9iYmJSNEh4WmI0dWpGS2RvTSt3SURBUUFCbzBJd1FEQU9CZ05WSFE4QkFmOEVCQU1DQWdRd0R3WUQKVlIwVEFRSC9CQVV3QXdFQi96QWRCZ05WSFE0RUZnUVVyc2Urcm5LcVBEL0ErOXpIaVlteEQvRkJiNHN3RFFZSgpLb1pJaHZjTkFRRUxCUUFEZ2dHQkFCWjdHd1U1RnUvenRiMzVKVGY2KzdDWmtpYWFUdHBRTlNQdmVqVHNhSWpDCld1WENOTzZwWmJqeXE0a3UyZ2lPYk1yOGR3S3k3aXh4N1hLS2xUQzRQOG1Ob3J3MnE4K0dtcVJMemJmN1FYbXYKN0F4TmpjejIwTkFQVytDeEVFVUdRV25udFJTaWdmZ3NyM2swUWNLTzJ4N1hVSncyd3pkRWhxK3FGU1lOc0IzTAo1OTIxM2NOWGZGaHJzWUZHUW45cFQwRkpFNCtRcWpmTUdCVjJEODhNWmtDRGJPNC9UUkxYOVJzVFhpNTMxNSs5Cjd2cFZHZmNmblJKTXErdnpkRnZkc0Y1VlVwRCtYWTV3c2taYWZOMDNpYkUyTmQ1ZmVrbERMSHRqUVdPS1hrTEkKajFmL3lWbUlpUVhycThubVltU0ZwY1VJS2J2T3R4L3psNmZFRWhUbkxEY2RLU3AxUkVyWTh1VnV1a1ExQ2dFMQpjMXlmNGNPc0Q3aWp5L2J5NkpTQi9NMWhuQUNIQ0kvMm41aWl2L1RiOFpjQ09McjFlcnMvWEdmVVpCUzJzbEZRCldGck1QeDFVSi91REd4d3pteUdVUFR5WDBrR2Q0WEwzVEtCNDF0NE5LS0tHbGFPT1hBVll5QWhVRHc5NEZLSkoKWmpuREdiWjhvb0I4NDZTVzBxeUE1dz09Ci0tLS0tRU5EIENFUlRJRklDQVRFLS0tLS0K"
main_cluster_client_certificate = ""
main_cluster_client_key = <sensitive>
main_cluster_endpoint = "<IP_ADDRESS>"
main_cluster_id = "projects/ols-platform-dev/locations/asia-southeast2-a/clusters/ols-dev-compute-gke-main"
main_cluster_location = "asia-southeast2-a"
main_cluster_master_version = "1.27.3-gke.100"
main_cluster_name = "ols-dev-compute-gke-main"
main_cluster_self_link = "https://container.googleapis.com/v1/projects/ols-platform-dev/zones/asia-southeast2-a/clusters/ols-dev-compute-gke-main"
main_firewall_ids = {
  "icmp" = "projects/ols-platform-dev/global/firewalls/ols-dev-network-vpc-allow-icmp"
  "internal" = "projects/ols-platform-dev/global/firewalls/ols-dev-network-vpc-allow-internal"
  "rdp" = "projects/ols-platform-dev/global/firewalls/ols-dev-network-vpc-allow-rdp"
  "ssh" = "projects/ols-platform-dev/global/firewalls/ols-dev-network-vpc-allow-ssh"
}
main_firewall_self_links = {
  "icmp" = "https://www.googleapis.com/compute/v1/projects/ols-platform-dev/global/firewalls/ols-dev-network-vpc-allow-icmp"
  "internal" = "https://www.googleapis.com/compute/v1/projects/ols-platform-dev/global/firewalls/ols-dev-network-vpc-allow-internal"
  "rdp" = "https://www.googleapis.com/compute/v1/projects/ols-platform-dev/global/firewalls/ols-dev-network-vpc-allow-rdp"
  "ssh" = "https://www.googleapis.com/compute/v1/projects/ols-platform-dev/global/firewalls/ols-dev-network-vpc-allow-ssh"
}
main_nat_id = "ols-platform-dev/asia-southeast2/ols-dev-network-vpc-router/ols-dev-network-vpc-nat"
main_pods_secondary_range_name = "pods-range"
main_router_id = "projects/ols-platform-dev/regions/asia-southeast2/routers/ols-dev-network-vpc-router"
main_router_self_link = "https://www.googleapis.com/compute/v1/projects/ols-platform-dev/regions/asia-southeast2/routers/ols-dev-network-vpc-router"
main_secret_id = {
  "atlantis-password" = "projects/ols-platform-dev/secrets/atlantis-password"
  "github-oauth-client-secret-argocd" = "projects/ols-platform-dev/secrets/github-oauth-client-secret-argocd"
  "github-secret" = "projects/ols-platform-dev/secrets/github-secret"
  "github-token-atlantis" = "projects/ols-platform-dev/secrets/github-token-atlantis"
  "github-token-iac" = "projects/ols-platform-dev/secrets/github-token-iac"
  "openai-api-key" = "projects/ols-platform-dev/secrets/openai-api-key"
}
main_secret_version_data = <sensitive>
main_secret_version_id = {
  "atlantis-password" = "projects/687148684055/secrets/atlantis-password/versions/1"
  "github-oauth-client-secret-argocd" = "projects/687148684055/secrets/github-oauth-client-secret-argocd/versions/1"
  "github-secret" = "projects/687148684055/secrets/github-secret/versions/1"
  "github-token-atlantis" = "projects/687148684055/secrets/github-token-atlantis/versions/1"
  "github-token-iac" = "projects/687148684055/secrets/github-token-iac/versions/1"
  "openai-api-key" = "projects/687148684055/secrets/openai-api-key/versions/1"
}
main_services_secondary_range_name = "services-range"
main_subnet_ip_cidr_range = "<IP_ADDRESS>/16"
main_subnet_self_link = "https://www.googleapis.com/compute/v1/projects/ols-platform-dev/regions/asia-southeast2/subnetworks/ols-dev-network-vpc-subnet"
main_vpc_gateway_ipv4 = ""
main_vpc_id = "projects/ols-platform-dev/global/networks/ols-dev-network-vpc-main"
main_vpc_self_link = "https://www.googleapis.com/compute/v1/projects/ols-platform-dev/global/networks/ols-dev-network-vpc-main"

2. dir: deployment/service/ols-svc-talkgpt workspace: default
module.workload_identity.google_project_iam_member.sa_iam: Destroying<EMAIL_ADDRESS>module.argocd_app.helm_release.helm: Creating...
module.argocd_app.helm_release.helm: Creation complete after 1s [id=ols-talkgpt]
module.workload_identity.google_project_iam_member.sa_iam: Still destroying... [id=ols-platform-dev/roles/cloudkms.cryptoK...s-platform-dev.iam.gserviceaccount.com, 10s elapsed]
module.workload_identity.google_project_iam_member.sa_iam: Destruction complete after 10s
module.workload_identity.google_project_iam_member.sa_iam: Creating...
module.workload_identity.google_project_iam_member.sa_iam: Creation complete after 10s<EMAIL_ADDRESS>
Apply complete! Resources: 2 added, 0 changed, 1 destroyed.

3.  Resources: 4 added, 0 changed, 0 destroyed.

Locks and plans deleted for the projects and workspaces modified in this pull request:

dir: deployment/all-gcp workspace: default
dir: deployment/service/ols-svc-talkgpt workspace: default
dir: deployment/service/ols-svc-transribegpt workspace: default


Feature: Map

Summary
This branch adds the map functionality to the Shiny app.
Changes
The changes made in this PR are:

Loads in shapefiles for:

Hong Kong
District Councils

Uses ggplot2 to plot the shapefiles.
Highlights the district council the user has selected.
Enables search by typing.

...
Check

[x] The map of HK and district councils is clear.
[x] The right district is highlighted when the user chooses a district.
[x] The user can search by typing a district in the search box.
[x] The R CMD and travis.ci checks pass.
[x] MIT License badge is on the README.md
[x] Links to prod and pre-prod web-hosted version of apps are on README.md and work.
[x] @ocean0609's logo is now in the top-right of the webpage.
...

Note
This "fixes #13" and "fixes #24".
The shapefiles for the maps are stored in the repo. This is because they cannot be imported from GDrive.
Is currently deployed on pre-prod.

Now need to be able to join the HK DC shape file, shape_district with the Gsheet, data_master_raw. This is so we can link the user input with what's displayed on the map.
@martinctc and @justin-yim, will you be able to do this? I'm not entirely sure how to do it myself.

Now need to be able to join the HK DC shape file, shape_district with the Gsheet, data_master_raw. This is so we can link the user input with what's displayed on the map.
@martinctc and @justin-yim, will you be able to do this? I'm not entirely sure how to do it myself.

Actually, think I managed to join it. Looks like can do so via constituency code which are provided in both datasets!

Build is failing because it doesn't know where the shapefiles are. Cannot link to GDrive so will have to include the shapefiles into the repo.

The corresponding .dbf and .shx files have to be in the same directory of the .shp files in order for the shape files to work.
See: https://stackoverflow.com/a/58820909/5539702
Will add these back in the next commit!

I'm experiencing an issue where I can't run the map properly, even after adding the associated shape files:

I'm experiencing an issue where I can't run the map properly, even after adding the associated shape files:

Figured out what the issue is. Need all the shapefiles for this. Re-included them back in with the latest commit.

Fantastic stuff! Looks good. One potential enhancement in the future would be to make the search more 'fuzzy', e.g. for "HO MAN TIN", some might put "HOMANTIN" (both are known acceptable spellings).


internal to upstream 2016/4/26

Can one of the admins verify this patch?

@tensorflow-jenkins test this please


DLS-10273: [MRC] Test Pack Review - Readme and Dependencies

//  TODO: UPGRADING CAUSES ERROR - NEED TO INVESTIGATE
private val gatlingVersion = "3.6.1"


Fix for #244

What title says.

Codecov Report

Merging #245 (e477f6c) into develop (25d0c69) will decrease coverage by 0.09%.
The diff coverage is 37.50%.

@@             Coverage Diff             @@
##           develop     #245      +/-   ##
===========================================
- Coverage    87.34%   87.25%   -0.09%
===========================================
  Files           71       71
  Lines         5469     5470       +1
  Branches      1232     1235       +3
===========================================
- Hits          4777     4773       -4
- Misses         414      415       +1
- Partials       278      282       +4

Files Changed
Coverage Δ

src/instrumenter/interpose.ts
88.88% <37.50%> (-2.61%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


HDDS-9076. Recon UI OMDB Insights Changes and Improvements

What changes were proposed in this pull request?

Take user to Key Deletion page on click of "Overview" pending deletion card.
Take user to Pending Open Keys page on click of "Overview" pending open keys.
API param "prevKey" of deletePending keys and open keys seems not passing value.
First Time in API Call no need to call prevKey as we sending empty.
Pending Key Deletion from Overview Page need to delete.

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-9076
How was this patch tested?
Manually
1.webm

could you take a look @devmadhuu, @ArafatKhan2198 and @devabhishekpal? thanks in advance!

hi @smitajoshi12, thanks for working on this! could you please check if there are any necessary changes after #5023? if yes, maybe it should be separated and solved in another jira, once that's done you can continue on this.

@dombizita  Yes I will check with @ArafatKhan2198

hi @smitajoshi12, thanks for working on this! could you please check if there are any necessary changes after #5023? if yes, maybe it should be separated and solved in another jira, once that's done you can continue on this.

I ensured that the frontend was changed to align with the modifications made to the backend API endpoints. Therefore, I am confident that there shouldn't be any issues 👍

@devabhishekpal Could you take a look at it once

Thanks for working on this @smitajoshi12 LGTM +1

thanks for addressing the review comments @smitajoshi12! I tried your latest patch with pnpm run dev and the Overview page was not loading for me, this was the error message on the UI:
Error while fetching data
AxiosError: Request failed with status code 404

here is a screenshot:

could you please check it?
also on the OM DB Insights I saw this, this is because the db.json has some keys duplicated?

@smitajoshi12 Could you pull in the latest changes from upstream and see if everything is working out fine here?

thanks for addressing the review comments @smitajoshi12! I tried your latest patch with pnpm run dev and the Overview page was not loading for me, this was the error message on the UI:
Error while fetching data
AxiosError: Request failed with status code 404

here is a screenshot:

could you please check it?
also on the OM DB Insights I saw this, this is because the db.json has some keys duplicated?

@dombizita I believe the error which was not thrown earlier is happening due to the changes brought in by this PR
The fix for this raised by @smitajoshi12 over here :- Fix

Thank you, @smitajoshi12, for rebasing to the latest changes! Once you've integrated Zita's latest comment, I believe we'll be ready to proceed.

thanks for working on this @smitajoshi12! thanks for the review @ArafatKhan2198 and @devabhishekpal!


fix encode_multipart_formdata (TypeError: 'float' does not have the buffer interface)

Allow request "fields" to have floating point values, otherwise an explicit cast to str is needed while making the request.

This is an interesting PR. I'm not sure we want to support it, though, at least not in this form. Does it make sense to send non-integer values ever?

Yes, it does makes sense whenever you need fractional values(eg. transaction amount, etc.). Floating point values exists for the same reason.

I strongly feel that the representation of that floating point value should be determined by the user. The precision, rounding, etc. are things we absolutely should not be guessing.

Sure, let the user determine. Let the user make that choice, whether to make use of say "1.5" or 1.5. The latter choice throws an exception, as of now.

@sigmavirus24 is arguing that the current behaviour is preferable because it forces the user to make a choice.

Also, it looks like the current statement only exists as a compatibility shim in the first place? If that's the case, I'd be against adding more features to something that's arguably already technical debt.

I think that amounts to consensus here. Thanks @kartiknagpal, but I'm afraid we will not be accepting this PR. Sorry!


Feature Request: Support additional resources in ClusterServiceVersion, like StatefulSet and DaemonSet

Currently in the csv, only deployment, role/rolebinding and clusterrole/clusterrolebinding are supported to describe an operator installation.
However, to gain more control of the application or for other perpose, the operator needs not only a deployment (the controller), but also a StatefulSet (for stateful thing) or a DaemonSet (something like a node agent).
We have an operator which is composed by one deployment and one daemonset, which can not be installed by a csv. It will be great if csv can support it, thanks.

Looking for some information on the requirements: when your Operator get's installed, it's controller, CRDs and required RBAC will be put in place. It's then ready to provide it's services via it's API (the CRDs).
Is your requirement that you start acting on cluster without user requests? Because what you describe falls outside the definition of what makes up an Operator and should rather be deployed by your Operator instead of with your Operator.

@dmesser
I want to do the following things in the DaemonSet:

Gathering some node info which will be used for all the instances of the CRDs (sounds like "start acting on cluster without user requests")
Prepare a node agent which the controller will talk to when an instance is created, so action happens after the instance creation.
What's your suggestion to create this DaemonSet? with my Operator or by my Operator?

With what's possible now you'd deploy the DaemonSet via your Operator. It could do this on start-up using an init-container or as part of a Configuration CR that "configures" your Operator. The Operator could also creates this CR on start up - this would have the benefit, that your Operator will lifecycle it, in case it get's deleted or modified.

@dmesser Yes, I can deploy it during the controller start-up, but it is just a workaround. I don't think "Configuration CR" is a good idea because it is part of the operator, and should be managed by the olm in openshift, not the operator itself.
So it will be great if csv can support to create resources other than Deployment, which make it possible to install more complex operators.

I think we would need to look at the consequences of supporting what you ask for.
What you describe (Node agents in DaemonSet, etc), falls outside of the definition of what an Operator is: a controller owning certain CRDs.
If all the other stuff would become part of OLM we would also need to support configuring those - e.g. could you just instantiate the DaemonSet or would it need some configuration from the user/admin?. Normally this would come via the CR. Since the CRDs are not yet reconciled at install time, OLM becomes responsible for providing a user interface. Also, could you reliably create the DaemonSet if the Operator Deployment is not active yet? I.e. do you need ordering?
Now the point where OLM transforms into a fully configurable, YAML manifest template solution is not that distant anymore.
Whereas with the definition now, the boundaries of responsibilities is very clear: OLM deploys your Operator and your RBAC, an operator being one or more controllers in pods owning CRDs.
Everything else is up to the logic and value-add of your Operator.  Default configurations can be achieved using Configuration CRs or init containers. It's difficult for OLM to own these since the requirements would be so broad, that OLM would become a general-purpose deploy-my-bag-of-YAML controller. And people would rely on it instead of building the Operator to a higher maturity level.

@dmesser , thank you for your explanation. I totally agree that the olm should not become a general-purpose deploy-my-bag-of-YAML controller. Now I understand why I should use a "Configuration CR" to deploy it. Thank you.

Hello @dmesser, is this still valid today? Operators have to deploy its DaemonSets if needed? Or has something change since then?... Just to confirm and thank you!.


Troubles updating to CUDA 8.0RC

I received a new Pascal Titan X today and haven't been able to get things running.
I installed the video driver, then CUDA 8.0RC and got a Torch error trying to run neural-style. So I reinstalled its dependencies, cloned the latest version of Torch, installed it and then got the error:

/home/prime/torch/install/share/lua/5.1/trepl/init.lua:384: module 'loadcaffe' not found:No LuaRocks module found for loadcaffe

Then I reinstalled loadcaffe but I am still getting the same error as above. Any suggestions?

I'm running a Pascal Titan X on a fresh install of Ubuntu 16.04 with the CUDA 8.0 RC and I didn't have any problems like that, so I don't think it has to do with the new GPU or the new version of CUDA.
Is it possible that you have multiple torch installs floating around, and that luarocks installed loadcaffe into one torch install, while th points at a different torch install? Check which th and which luarocks and make sure they point to binaries in the same directory.

Thanks for the info. I just checked and they are both in the same directory:
/home/neural/torch/install/bin/th
/home/neural/torch/install/bin/luarocks
Hmm guess I will have to to a clean install of Ubuntu. When you installed yours, just curious if anything deviated from the instructions?
Just curious; how does is the performance compared to your last Titan X?

Ubuntu 16.04 ships with a newer version of gcc than the CUDA installer expects, so you need to hack it a bit during CUDA installation:
https://www.pugetsystems.com/labs/hpc/NVIDIA-CUDA-with-Ubuntu-16-04-beta-on-a-laptop-if-you-just-cannot-wait-775/
Apart from that I didn't have to do anything out of the ordinary to get it installed.
Funny you should ask about speed; I just updated the neural-style benchmarks:
https://github.com/jcjohnson/neural-style#speed
and I did more expensive benchmarking of various CNNs here:
https://github.com/jcjohnson/cnn-benchmarks
Overall the new Titan X is about 1.5x faster than the old one!

Ubuntu 16.04 ships with a newer version of gcc than the CUDA installer expects, so you need to hack it a bit during CUDA installation

Note that apparently they released a patch on August 8th to support gcc 5.4:
"This patch supports GCC 5.4 as one of the host compilers to use with CUDA 8 RC."

Overall the new Titan X is about 1.5x faster than the old one!

50% faster is definitely an improvement - but I guess I am a bit disappointed after buying into their marketing hype that deep learning could be accelerated 10-fold with the Pascal architecture. Thats what I get for believing marketing I guess!
;)

The 10x was absolutely a stretch - they arrived at this number by factoring in a 2x boost from moving to FP16 (which is supported on the P100 but not the Titan X) and also a multiplier due to better multi-GPU parallelism due to NVLink (which might help give more efficient data parallelism during training, but doesn't help neural-style). Overall I'm quite happy with a 1.5x boost; also note that cuDNN 5 is quite a bit more efficient than cuDNN 4 (thanks to winograd convolutions maybe?), so a Pascal Titan X with cuDNN 5 is almost 2x faster than a Maxwell Titan X with cuDNN 4.
Were you able to get your new card up and running?

Hey Jcj, how large of a picture can you process now with the 12GB?

Ah, I didn't know they crippled that feature. So if its 1.5 times faster now, another 2x would have made it roughly 3x faster? That would have been really nice! I guess the good news with that is time on a Pascal Quadro would be that much faster.
Dumb question. Does Torch utilize cuDNN?
Yes; I got it running on Ubuntu 16.04. Instead of using the hack you referenced, I used the --override flag to disable the GCC version check for the CUDA 8.0 install. Then I installed the patch I referenced above.

Fast PF16 on Pascal requires special hardware that for now is only present in the Tesla P100, so it's not like the new Titan X is artificially limited here.
Torch has cuDNN support through the cudnn.torch package. You can enable cuDNN in neural-style with the flag -backend cudnn; you should probably also add the -cudnn_autotune flag.

Torch has cuDNN support through the cudnn.torch package. You can enable cuDNN in neural-style with the flag -backend cudnn; you should probably also add the -cudnn_autotune flag.

Oh yeah duh, sorry I'm a bit rusty - I haven't used it without my shell scripts that add those flags for me in a while!

Has anyone tried using plain cunn with CUDA 8.0RC on a Pascal card? Just installed my 1070 on a fresh Ubuntu 14.04 yesterday (only four hours of struggling... mostly before I found out that 7.5 will not work, then the rest was getting around the login loop --as I want to use the onboard intel adapter for display and nvidia for cuda only.
Now, torch/cudnn/neural-style work fine while leaving cudnn out results in an error. Also cunn.test() gives similar errors. I probably should post on the torch.cunn project page but just wanted to check first if cunn works for you others?

@jcjohnson do you have your older Titan X working in the same machine as your new Pascal X?
When I boot up, the "GeForce GTX" green name lights up on both cards (last ten Titan X and Pascal Titan X), but as soon as I run nvidia-smi, the light turns off the older (non-Pascal) Titan X. I tried reinstalling NVIDIA-Linux-x86_64-367.35.run and the same thing happens. Any ideas?

@htoyryla I've used plain cunn with CUDA 8.0 RC on both the Pascal Titan X and on the 1080 without any problems.
@3DTOPO My Pascal X is in a new machine where it is the only card; unfortunately I don't have access to my Maxwell X at the moment to see if I have the same issue as you. However I'll eventually want to run them both in the same machine, so I'd love to know of any workaround you find to make that work.

@htoyryla same as jcjohnson - both are working here.
@jcjohnson I posted a question over at the nvidia forms, I will let you know what I learn.

Cunn now works for me too, after I re-installed from nn and further.


config/v1/types_cluster_version: Fix "represent from a" -> "represent a" typo

From 48f829bc53 (#185).

/lgtm


added saveman game

-Guess the word to save the man

@iamrahulmahato issue no #532

@iamrahulmahato please review my pr

@iamrahulmahato please review my pr #532


Update userscript dependencies and use CDN hosting

Switch all userscript dependencies to cdnjs hosting. Updated jQuery
version, jQuery-noty version. All scripts are now served over https.

Why are you changing the userscript hosting to a CDN, given they are readily hosted on your own server and I see no good reason to put it on someone else's servers?


Use PublishingAdapter in CliManualDeleter instead of using Publishing API directly

This is closely related to the work in #996.
This means the code can operate at a higher level of abstraction, passing instances of Manual rather than just some magic IDs. In doing this, I've extracted PublishingAdapter#discard which calls the already existing PublishingAdapter#discard_section.
In doing this I've also improved the overall level of abstraction used in this class, by using instances of Manual as much as possible rather than instances of ManualRecord. In doing so, I've extracted Manual#destroy which destroys the underlying ManualRecord and all associated SectionEditions.

@chrislo: Thanks!
Merging.


Error: Request failed with status code 502

Used to work like a charm, but encountering the below issue now. Any leads would be appreciated 🙏🏻.

A 502 bad gateway error is usually caused on the server side or some intermediate proxy. Does the error occur all the time? If not, a simple retry might already help.


Scheduling/editing an Activity - Setting the Alarm - too restrictive

The (GUI) setting of an alarm/reminder interval has the same limitation as the duration of an activity. Many times people will set an alarm for several days in advance or maybe in x minutes (or hrs).
We need a change in the GUI  to allow for setting of things like a 3 day or 13 minute.. or 7 hr.. type of reminder alarms.

Had a customer comment to me the LAN/Web approach is far better/easier/more flexiable to use than the current GUI in the mobile.

See Issue #9. There is now a duration field which is similar to the Web. It's a mixed autocomplete and picklist field enabling any custom values to be typed in various formats or you can use the picklist for the defaults.

Yep.. tested fine!
--
RJLedger – www.rjlsystems.net
Blog: blog.rjlsystems.net
Phone: +1<PHONE_NUMBER>
-----Original Message-----
From: Tony Skidmore<EMAIL_ADDRESS>Sent: Thursday, January 19, 2012 13:10
To: Robert Ledger
Subject: Re: [argos-saleslogix] Scheduling/editing an Activity - Setting the Alarm - too restrictive (#12)
See Issue #9. There is now a duration field which is similar to the Web. It's a mixed autocomplete and picklist field enabling any custom values to be typed in various formats or you can use the picklist for the defaults.

Reply to this email directly or view it on GitHub:
https://github.com/SageSalesLogix/argos-saleslogix/issues/12#issuecomment-3571292


HLSL Bindings don't work for Unity compute shaders

When trying to use the HLSL bindings in a Unity compute shaders it gives loads of errors. They are all "Cannot apply binary operator '+' on types 'FNLfloat' and 'FNLfloat'" or "Cannot apply binary operator '>=' on types 'FNLfloat' and 'int'" or similar. Do you know a way to fix this?

Replacing all occurences of FNLfloat with float solved the issue which is really strange, as the typedef should make them basically be the same thing 🤔


AutoUpdate Int Value

Hi,
I tried this out though when I create an NSManagedObject Subclass it automatically changed the integer 16 to NSNumber.
I tried running with the code (NSManagedObject.autoIncrementedIntegerAttribute("myCustomAutoID")  though it didn't seem to do anything.
Just wondering what I might be doing wrong.
Thanks in advance for your help.

Sorry I


[Returning 2.0] Game crashes after intros while using Ninja GFA patch

Hello,
I wanted to use GFA patch with Returning 2.0 ver 66,2.
After loading the game's two intros game crashes and I'm getting error shown bellow.

On the other hand bloodsplatter patch is working correctly

Thank you for the report.
This issue has been fixed already for a while, but the patch was never updated. It is expected to be updated in roughly three weeks (Mid-April).

Thanks for the answer. 3 weeks is a long time tho :/
Is it that complicated or you do not have time for it?


water_leak -> water_leak_rep

What does rep mean?

i think it is "repeat" like your "interval" change at my PR for the magnet sensor.

Yes, although I don't understand why the name of the attribute should be changed.

this attribute is only "true" ..
convert: (model, msg, publish, options) => {
return {water_leak: msg.data.data['65281']['100'] === 1};
on my water sensor... the other one is correct. therefore 2 attributes..

Then I think this converter can be removed (it has no use right?).
I guess this also fixes: https://github.com/Koenkk/zigbee2mqtt/issues/448

who has dev it ?? and why ??

I probably did as I assumed this was also needed, but the ias report is enough. Can you remove this converter?

ias was not implemented when this was first added @kirovilya tipped us in https://github.com/Koenkk/zigbee2mqtt/issues/17#issuecomment-387299734 to use ias (after a zigbee shepherd update)
Sent with GitHawk

I also have the same issue with one of my two water sensors.
Should I assume that this will be fixed in the next version?

@Notrial you can test the change by using the zigbee2mqtt dev branch

@Notrial you can test the change by using the zigbee2mqtt dev branch

I'm on hassio so I don't think i can :) Please correct me if I'm wrong.

You can, use the edge branch.


Windows are brought back to life after closing the settings window

Describe the bug

Any popup window that was closed before closing the settings window will appear.
To reproduce

Open a popup, like the Search popup.
Close it.
Open and close settings twice.
Observe that the window popped-up again, even though it should have been destroyed.

Screenshots

N/A
Chatterino version

Head is 10692354 + my own patches. Using old Makefile, so Version display is bugged on my build.
Operating system

Ubuntu 20.04.1 LTS
Additional information

Using an emote picker popup (the one brought up using :) with an empty text box can crash, because of an assert. Be careful :).

Can reproduce (see #2551).

Confirmed I can reproduce this. Will look into what causes this


What is the best alternative for stetho?

Summary:
Till some time back we were using Stetho for viewing and editing SQLite DB. But we removed it as the Android studio lets you go to the device file explorer and download the DB.
I am struggling to find a good Android Studio plugin that allows to view and edit real-time changes. Some of the plugins let you download the DB and then view it. I found another plugin called SQLScout that seems to support almost realtime viewing of DB but it isn't working for me.
What are you guys using for debugging the data in the DB? Tagging a few people @ashishkumar468 @misaochan @neslihanturan @macgills. Others also feel free to comment.
If the alternatives are not straightforward to use then it would be great to have Stetho back.

Android Studio is about to roll out an update which would support db browsing by default. In fact some of the canary versions already have it (4.1 canary 6 >)
https://developer.android.com/studio/preview/features

Thanks, @ashishkumar468.
Are there other options for people who are not using the canary builds?

This cropped up from  a google but not exactly a world of difference from Stetho. I would wait for the AS release, not that we have any guarantee that it will be very functional, sometimes AS releases stuff that just doesn't work (instant run, multiple simultaneous previews) but 4.0 and beyond is looking like good, they will actually have gutter icons for Dagger.
Mostly I find "needing to view the db" somewhat unnecessary as debugging can mostly solve the problem anyway, Evaluate Expression button is great for running arbitrary code which is way nicer than jury rigging sql commands yourself. Adding fake data is as easy as
dao.insert(listOf(0..100.map{ DbItem(it, "fake value%it")} ))

probably with a subscribe or Completable.fromAction  thrown in

Android Studio 4.1 is working well for me.


luci-app-hd-idle: convert to client side rendering

Signed-off-by: Van Waholtz<EMAIL_ADDRESS>

Good work! Just two little nitpicks as written above.

Updated.

Merged, thank you!


Question re: oculocutaneous albinism type 8

this term is meant to be added in the Orphanet sync:
[Term]
id: MONDO:0035364
name: oculocutaneous albinism type 8
subset: ordo_disease
synonym: "OCA8" EXACT [Orphanet:597733]
is_a: MONDO:0000001 {source="Orphanet:597733"} ! disease or disorder
is_a: MONDO:0018910 {source="Orphanet:597733"} ! oculocutaneous albinism
xref: Orphanet:597733 {source="MONDO:equivalentTo"} ! oculocutaneous albinism type 8
@ahamosh is this equivalent to MONDO_0035364 'oculocutaneous albinism type 8' (https://omim.org/entry/619165)?
Orphanet doesn't have much info about this term, so I can't be sure. I would guess at some point Orphanet will map this to OMIM?

Not entirely sure that I understand the questions, but OCA type VIII is the same as OCA8.  It looks like all the different OCAs are distinguished with the work Type.  OCA8 is due to DCT. It is mapped to 619965 in orphanet.

Thanks @ahamosh. I'll add this Orphanet xref as an equivalent xref on MONDO_0035364 'oculocutaneous albinism type 8'


Remove Redirect to GitHub Profile During Tool Installation

Issuie:
When installing the tool on Termux, I'm getting redirected to your GitHub profile. Although the installation process completes successfully when I return to Termux, this redirection is frustrating.
Proposal:
i suggest removing the redirection to the GitHub profile during the installation process. This will improve the user experience and prevent unnecessary frustration.
Advantages:

Smooth Installation:
Removing the redirection ensures a seamless installation process without interruptions.

User-Friendly:
Users won't get frustrated by being unexpectedly redirected to the GitHub profile.

Enhanced Experience:
a smoother installation process contributes to a more positive overall experience for users.

Considerations:

Purpose of Redirection:
Understand the reason behind the redirection and ensure its removal doesn't impact the project's goals.

User Feedback:
Gather feedback from other users to confirm if they also find the redirection frustrating.

pleazse considering this suggestion to improve the installation process of the tool.

ok.i understand


Added Pattern Syntax Checker Code in Java

This PR aims at addressing issue no. #9  adds the solution to Pattern Syntax Checker problem of Java module of hackerrank.

@swapnanildutta Could you please review this?


Exercises dependency graph

Generates a graph of dependencies between the exercises of a repository.
An exercise depends on:

Its declared backward exercises
The exercises that focusing on its required skills

This PR only includes the data structure and the graph generation, with a CLI option to dump the graph into its dot representation when building the repository. It should be useful to generate a set of exercises ordered by dependencies (a small algorithm is given as compute_exercise_set, but it is really naive).
For example, here is a generated repository with only 10 exercises and 3 skills (it should be clearer for a well-thought repository):

This could also be done client-side, with a button download exercise graph under "actions" in the teacher screen ? Or do we even have an easy way for web-display of .dot files ? The bonus is that it could handle "skills" defined from the web-interface.
Otherwise, maybe it would be best to keep the code outside of the client-reachable learnocaml_data module ?

There exists visualizers for dot files in JS, see https://github.com/mdaines/viz.js.


Is file download available for windows? 2) Is async storage available for windows?

Closing as not relevant to Renative. Feel free to reopen again


config base format

Description

format config.base by URL
Additional context

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[x] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.
#10229

Lack of detailed explanation of what this PR does. Maybe you can create an issue for this feature first?

That may be not necessary,  actually, there may be little users set base url to  Chinese or Japanese ... The main purpose of this PR is to let the parse of base url stay the same as others.
And here is the discussion config base with chinese will make some mistakes

Thanks. I get that they are inconsistent.
But what are the exact bugs?
Is this PR a bug fix, a new feature, or a refactoring?

refactoring

refactoring ^_^

#10534

Looks good to me. Thanks!

Thanks a lot


[JS]: Regular Expressions

Description
Introducing the topic of Regular Expressions and their significance in JavaScript.
Mode:  Documentation
Directory
JavaScript/Topics/13.Regular_Expressions
Domain
JavaScript
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign

/assign

@paramsiddharth @Creatoon @dakshsethi @s4-2 Kindly review this issue.
Thank you!

I would not be able to work on this issue please unassign it to me.

@subhendudash02 I'll work on this, please?

Ok
On Sun, Nov 28, 2021, 2:21 PM shreya-gb @.***> wrote:

@subhendudash02 https://github.com/subhendudash02 I'll work on this,
please?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/girlscript/winter-of-contributing/issues/2142#issuecomment-981040228,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVLGMNOLWDPDWENXXNPPPG3UOHUQZANCNFSM5ETADWQQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@subhendudash02 I'll work on this, please?

Ask @paramsiddharth


Add RBAC permission types for key value pairs

Add RBAC permission types for key value pairs so the changes at https://github.com/StackStorm/st2-rbac-backend/pull/55 to introduce RBAC to KVP can pass CI.

I added to the resource type to permission type map. The global permission type dynamically applies the list permission.


Can't select an option when used in combination with Fullpage.js

So it seems when Fullpage.js is being used on the page, you can't select a option in the dropdown. The only way I could select a value is by right clicking on it.
I've tracked the issue down to a pointerdown event listener attached by the scrolloverflow.js extension for Fullpage.js. I'm not sure if this is an issue with Choices, or Fullpage.js or both, let me know if I need to open an issue there instead.

I don't even know what Fullpage.js is so I'm not sure I'm going to be of much help - if you can track the issue down to an actual bug in this library, please open a new issue with an example 👍


Fix flicker with popup menu

When using the completion menu if a menu item has additional infos associated, a preview window is opened to show these.
When the preview window is shown the cursor is briefly switched to the preview window and back to its original position thus triggering a BufLeave and corresponding BufEnter command. This can cause a function call, that may trigger a redraw causing a flicker while moving between popup menu result. Only executing those function when the popup menu is not visible fixes the issue.

Excellent catch, thanks – I’ll merge it into the next release branch.


Update triaging guidelines

Description
Update the triaging guidelines to include open PR status check, and review of all "security" labelled issues in the project.
Check List

[ ] ~New functionality includes testing~
[ ] ~New functionality has been documented~
[x] Commits are signed 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.

@reta @willyborankin what would you think about this format change?

@reta @willyborankin what would you think about this format change?

I'm for "pull request discussion" which is an approach we use in our company. I personally find them vert useful.


[Docs] Add docstring.

Motivation
Add docstring to glip.py from mmdet/model/detectors/glip.py.
Modification
Add more docstring to help users get familiar with the code.
BC-breaking (Optional)
Does the modification introduce changes that break the backward-compatibility of the downstream repos?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects, like MMDet or MMCls.
The documentation has been modified accordingly, like docstring or example tutorials.

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


UICycleImage as public ?

Description
would it be possible/interesting to expose UICycleImage so we modders can reuse it instead of recoding it or copy/pasting the code in our project?
Steps to Reproduce
insert the following CS code in a separate mod project from tmodloader :
using System.Collections.Generic;
using Microsoft.Xna.Framework.Graphics;
using Terraria;
using Terraria.UI;
using Terraria.ModLoader.UI;
using System.Reflection;
using System.Linq;

namespace something
{
    public class Test
    {
        public UICycleImage TestCycleToggle;
    }
}

Expected behavior
UICycleImage is accessible for outside mods
Actual behavior
Visual Community shows an error :
Error	CS0122	'UICycleImage' is inaccessible due to its protection level
Versions
v<IP_ADDRESS> tModLoader
Additional Information
I believe it's because UICycleImage is declared as internal in tmodloader source code ?
line 8 : https://github.com/blushiemagic/tModLoader/blob/master/patches/tModLoader/Terraria.ModLoader.UI/UICycleImage.cs
I hope there's enough information, tell me if not.

I think @jopojelly made the entire thing (according to the blame), so let's have him decide.

It's a fairly simple and limited UIElement. I'm fine with it being public, but I'll probably want to make it more useful with an OnStateChanged event first, at the very least.
I'll try to do this soon.

thanks @JavidPack


Automated cherry pick of #9196: feat(region): add bucket cloud tags support

Cherry pick of #9196 on release/3.5.
#9196: feat(region): add bucket cloud tags support

Codecov Report

Merging #9199 (e469dd9) into release/3.5 (6a15e02) will increase coverage by 1.81%.
The diff coverage is 2.56%.

@@              Coverage Diff               @@
##           release/3.5   #9199      +/-   ##
==============================================
+ Coverage         5.98%   7.80%   +1.81%
==============================================
  Files              796     248     -548
  Lines           125693   33869   -91824
==============================================
- Hits              7522    2642    -4880
+ Misses          117504   30965   -86539
+ Partials           667     262     -405

Flag
Coverage Δ

aFlag
7.80% <2.56%> (+1.81%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/apigateway/handler/auth.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/jointbase.go
0.00% <ø> (ø)

pkg/cloudprovider/objectstore.go
2.06% <0.00%> (-0.51%)
:arrow_down:

pkg/compute/regiondrivers/managedvirtual.go
0.00% <0.00%> (ø)

pkg/hostman/hostinfo/hostovn.go
27.45% <0.00%> (-2.03%)
:arrow_down:

.../monitor/alerting/conditions/suggestrulereducer.go
0.00% <0.00%> (ø)

pkg/multicloud/esxi/host.go
0.77% <0.00%> (-0.01%)
:arrow_down:

pkg/multicloud/esxi/manager.go
0.70% <0.00%> (-0.01%)
:arrow_down:

pkg/monitor/alerting/conditions/suggestquery.go
2.00% <2.00%> (ø)

pkg/util/netutils2/netutils.go
21.62% <100.00%> (+0.35%)
:arrow_up:

... and 515 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 8dcfa2e...e469dd9. Read the comment docs.


Undocumented option SynchronizeSoftDeleteFormula

I'm hoping the SynchronizeSoftDeleteFormula is meant for external use to soft delete instead of hard deleting any un-matched entities, but I'm not sure what to put in it for SqlServer to change from a delete to an update.

Hello @npbenjohnson ,
Here are some example:
context.BulkSynchronize(list, option =>
{
                option.SynchronizeSoftDeleteFormula = "IsSoftDeleted = 1";
});

context.BulkSynchronize(list, option =>
{
                option.SynchronizeSoftDeleteFormula = "DeletionDate = GETDATE()";
});

context.BulkSynchronize(list, option =>
{
                option.SynchronizeSoftDeleteFormula = "IsSoftDeleted = 1 AND DeletionDate = GETDATE()";
});

It allows you to put any SQL after the SET statement on the update.
Let me know if that answer correctly to your question
Best Regards,
Jon

Sponsorship
Help us improve this library
Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Bulk Operations •  Dapper Plus
Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function • SQL Eval Function

That worked for me, thank you


Chetan UI branch

Here we go
https://github.com/anuragverma108/SwapReads/assets/109230049/85ed8146-3d63-4606-b7ab-bbcb6b6a33ee
New fetaure added

Write proper description along with issue number that what changes you did. Also attach screenshots & raise PR again

##Sorry for Inconvennience##
##Issue number 206
We have added feature of popup-box after submitting the Contact form
https://github.com/anuragverma108/SwapReads/assets/109230049/8718a030-b78d-4f50-891e-5d48c9745922


Failed to deploy cluster

k8s/tool/telegraf/telegraf-operator.yaml deployment fails
Kubernetes Version: v1.22.2
error: error validating "k8s/tool/telegraf/telegraf-operator.yaml": error validating data: [ValidationError(MutatingWebhookConfiguration.webhooks[0]): missing required field "sideEffects" in io.k8s.api.admissionregistration.v1.MutatingWebhook, ValidationError(MutatingWebhookConfiguration.webhooks[0]): missing required field "admissionReviewVersions" in io.k8s.api.admissionregistration.v1.MutatingWebhook]; if you choose to ignore these errors, turn validation off with --validate=false

change the file as below

Error gone cluster deployed, but now unable to start the telegraf container with following error.
Error: failed to start container "telegraf": Error response from daemon: OCI runtime create failed: container_linux.go:380: starting container process caused: process_linux.go:545: container init caused: rootfs_linux.go:76: mounting "/var/run/utmp" to rootfs at "/var/run/utmp" caused: mount through procfd: not a directory: unknown: Are you trying to mount a directory onto a file (or vice-versa)? Check if the specified host path exists and is the expected type

Hi @hkpanchani
I'am making a PR to update the rbac and the telegraf operator to avoid the v1beta deprecation.
I'am taking note of the change you provided, and locally, on an older version (1.15) this seems backward compatible. Thanks you for providing the solution.
For your second issue, it seems something recurrent (was already a problem back in 2019) because, the location / presence of this file change on the different linux distros. I'am reproducting your issue locally on docker-desktop on windows with debian sub-system for example.
Because the informations given by utmp are not that critical (system uptime, login, logout, some events). I have commented it by default. As there is no really fully generic way to make it work everytime and not everybody is fine with debugging this kind of thing, I would consider the simpler, the better.

@hkpanchani Coud you please checkout the #8 PR and try again with the change. Let me know if all work as expected and I would merge.
Have a nice day, thanks you again for your feedback

I had time to test it by myself and everything seems to work, even with the latest version on my minikube (1.22.3).
I merged the PR in the master branch. Let me know if things goes wrong on your side.
I'am closing the issue in the meantime


Can't get compressed assets

I'm trying to get my css and js files to be sent compressed to clients when in production mode, but that doesn't seem to be happening.
I've looked at the "Assets middleware server demo" and I can't see any difference to what I have in my app, and still the assets are not being compressed. Assets are being correctly sent otherwise.
So, besides this:
if (config.env === 'production') {
  environment.jsCompressor  = 'uglify'
  environment.cssCompressor = 'csso'
}

and the required uglifyjs and csso modules correctly installed what else is required to enable compression on assets returned by Mincer?
node.js 0.10.25
mincer 1.2.4
csso 1.3.11
uglifyjs 2.4.10

Nevermind. Found the problem. I was accessing the jsCompressor and cssCompressor properties of the function returned by createServer() instead of the object returned by Environment().


build failed

file:///xxxxxxxx/Main.storyboard: error: IB Designables: Failed to render and update auto layout status for TrafficViewController (Lfl-Vm-61S): dlopen(MKMagneticProgress.framework, 1): no suitable image found.  Did find:
MKMagneticProgress.framework: required code signature missing for 'MKMagneticProgress.framework'

Same here

Same here


Add DeleteItem functionality to code base

Add DeleteItem functionality to code base
Add testcase for DeleteItemCommand, DeleteItemCommandParser and AddressBookParser

Codecov Report

Merging #146 (daf52bc) into master (02ed72c) will increase coverage by 0.36%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #146      +/-   ##
============================================
+ Coverage     70.23%   70.59%   +0.36%
- Complexity      636      647      +11
============================================
  Files           105      107       +2
  Lines          2170     2190      +20
  Branches        254      256       +2
============================================
+ Hits           1524     1546      +22
+ Misses          552      550       -2
  Partials         94       94

Impacted Files
Coverage Δ

...main/java/seedu/address/commons/core/Messages.java
0.00% <ø> (ø)

...eedu/address/logic/commands/DeleteItemCommand.java
100.00% <100.00%> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
84.61% <100.00%> (+0.61%)
:arrow_up:

.../address/logic/parser/DeleteItemCommandParser.java
100.00% <100.00%> (ø)

...rc/main/java/seedu/address/model/ModelManager.java
89.41% <0.00%> (+2.35%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


sipp: 3.6.0 -> 3.6.1

Description of changes
version bump

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

To be frank: If you had linked the release notes you might have spotted a breaking change in this release, that should have also noticed while building this package on aarch64-darwin as you claimed.
https://github.com/SIPp/sipp/releases/tag/v3.6.1

To be frank: If you had linked the release notes you might have spotted a breaking change in this release, that you should have also noticed while building this package on aarch64-darwin as you claimed.
https://github.com/SIPp/sipp/releases/tag/v3.6.1

Oh, shoot! I actually didn't read the release notes but naively used nix-update to update the pkg. Looking at it again I now see that it fails on aarch64-darwin too and that I misinterpreted the output of nix-update.
Thank you for taking the time to look into it and obviously for pointing that out.
I'll try to fix it and update the PR then or if you prefer closing it I'll open another one once I reach a working state.

The ordering of the package is a bit wild. I usually do it chronologically.

src
patches
configure
build
install
test

@mweinelt thank you for all the input. Didn't expect that to be honest. Hope everything is fine now.

The ordering of the package is a bit wild. I usually do it chronologically.

src
patches
configure
build
install
test

That's true. I reordered it a bit. Hope it makes sense now.


[geom] The bounds function ignores transforms on groups

The bounds helper ignores transform matrices on groups, leading to weird results

Hmmm, that makes sense! I think the solution is calling something like transform manually. What is the difference between that and transformVertices?

Never mind, I figured it out! Should have paid more attention to the types :sweat_smile:


Change default keyboard shortcut

In apps with tabs, Command + Shift + T is usually the shortcut to re-open the most-recently-closed tab. This is also the case in VS Code (at least with the Atom keymap enabled).
A different default keyboard shortcut would be great. I'm aware I can change it, but it's still frustrating, especially for those of use who work on multiple computers. I've changed mine to Control + Option + Command + T for now.

Any ideas as to what shortcut would be better?

I don't mind Control + Option + Command + T. Could also use a chord. Shift + Command + K chord Shift + Command + T.
Might be good to do a poll or something.
I imagine once fix on save is possible, I'll disable it altogether as I'll rarely need it. Only if I have an unsaved file I'm mucking around with, in which case I'd just use the command palette.

Run on save is now possible, however I will look into adding a config option to change the default shortcut today.

Have you considered registering as a formatter instead of a command? https://code.visualstudio.com/blogs/2016/11/15/formatters-best-practices
Then you would just be able to use the default formatter shortcut of Shift+Option+F

Have you considered registering as a formatter instead of a command? https://code.visualstudio.com/blogs/2016/11/15/formatters-best-practices
Then you would just be able to use the default formatter shortcut of Shift+Option+F

I have, however this caused some issues last time I tried. I'm open to it (so long as it won't break anything).

Thinking about it again, I actually like being able to run the sort operation separate from formatting the whole document (especially when working with Vue files which can also include JavaScript).
ctrl+shift+h might be a good default shortcut, it's what the IDEA port uses.

As a temporary fix, add this to your keybindings.json (cmd + shift + p > keyboard shortcuts).
[
  {
    "key": "shift+cmd+t",
    "command": "workbench.action.reopenClosedEditor",
    "when": "editorTextFocus"
  }
]

+1 This was a pretty surprising keyboard rebinding. At least for me, CMD + SHIFT + T is a frequently used binding in both browsers and editors.


Help requested: Support for reference tags?

Hi there, I'm trying to figure out the best way to handle a whole bunch of embedded URLs in content that's been laid out in Markdown form with raw URLs embedded.
I'm using Doxter to manage the content, which allows me to reference assets directly in the text, but simply typing in {asset:3502} just gives me the raw URL for the JSON tag, which is obviously not what I'm looking for.
I can upload images as assets and get them to render in the text as  {asset:3502:img}. My question is, is there a similar method I could use for embedded assets? I'd specifically like to use this to handle tweets.
Just trying to get a handle on the functionality here.
Edit: I ended up using regex for this specific use case, but I would still be interested to hear how reference tags could be used with this plugin, if possible.

Sorry for the delay in this question being addressed. I don't think this is currently possible, unfortunately, since the embedded asset properties themselves aren't properties of the asset as required by the reference tags format. Would love to do this if it's made possible in some way in the future, though.


Tiles next to cities can be irrigated only if adjacent to ocean/river

Even though a city technically automatically has irrigation.

I thought I had added the city exception but apparently not. :-)

umm, yeah I havent had this::


try to be more compatible with hapi upgrades

Just trying be less fiddly when hapi upgrades come out... Thanks! L

Thanks for the PR and sorry it has taken me so long to review it.
I would love to reduce the specificity of HAPI modules versions. The issue is the only way I can get swagger to work is to reference some properties and objects in the HAPI modules which are not classed as part of public API. The smallest update can break hapi-swagger as they are not considering changes to some of the interfaces I use as a change to the API. I have to test everything by hand. After thinking about your pull request this applies to HAPI and JOI and I could reduce the specificity for Boom and Hoek.
I will update the versions of the HAPI modules later today and take this into account. Thanks


Add clear screen option

Add clear screen option when watching for changes before starting the build clear the screen if enabled.

Released as part of 0.21.0-rc.8


KinematicBody2D: test_move breaks when parent is offset from (0, 0)

Godot version:
3.1 beta 4
OS/device including version:
Windows 10
Issue description:
When a kinematic body 2d's parent does not have the position (0, 0), test_move behaves incorrectly.
Steps to reproduce:
Open attached project. Play the LevelBase scene by itself. Notice that it works normally. Now change LevelBase's position and test again. Notice that the platformer character will no longer behave correctly, such as refusing to jump, fast falling off ledges, etc. These have to do with the test_move checks in Player.gd, which is basic platforming logic.
you can also test
in TestRoom.tscn, and see how it impacts the game in Main.tscn.
Minimal reproduction project:
Spectrum.zip

This is a documentation issue, test_move() needs the global transform of the object.

Wow thanks for the clarification that test_move() requires the global transform! But this is seriously terrible documentation, I mean it's kind of alright for the most part, but there are so many things that are underdocumented in Godot, and this one just ruined a whole game jam for me. I'll definitely try to fix this in the documentation, although I have never edited it, so I hope it is not too hard.


Bugs in Adversarial Training.

Hi! I ran a few experiments on training the AOT-GAN on the places2 dataset. However, looking at the loss plots seems like the adversarial loss isn't affecting the learning. The magnitude remains the same during the whole course of training.
You can see the experiment results here: https://github.com/praeclarumjj3/AOT-GAN-Experiments#5-experiments.

Ohh never mind, the results come out nice if we use the testing pconv dataset anyways. So it's cool!


Automatically set failure domain of pools and portablility of osds based on available nodes.

Setting of failure domains also applies to the StorageClassDeviceSets. Failure domain in this case is set by podAntiAffinity and a topologyKey.


Syntax change for CSV options in copy from and external tables

enable new fields for csv external table and copy from csv file.
PRD: https://docs.google.com/document/d/1HyAsvRb0S8iv5bjIEAW424fCGyB3o-O3u9lITizw-OQ/edit?ts=5b83bb25

@soupi

Looks good. Thanks!

Thanks a lot for the review and the quick response, much appreciated!
Have a great vacation :)
On Tue, 4 Sep 2018, 23:54 Gil Mizrahi<EMAIL_ADDRESS>wrote:

Looks good. Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SQream/hssqlppp/pull/4#issuecomment-418514308, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Af3hGpmDOxOkqG3DslxrvVAdBhkkNWQZks5uXuhogaJpZM4WZU0C
.


Object Storage: Support Alibaba cloud OSS provider

[] CHANGELOG entry if change is relevant to the end user.

Changes
Alibaba cloud OSS(Object Storage Service) is widely used among China’s cloud users. This PR adds supports of OSS for thanos.
Verification

To run tests against this provider, you should set environment variables like below:
export OSS_ENDPOINT='OSS endpoint to connect to'
export OSS_ACCESS_KEY='Your access key'
export OSS_SECRET_KEY='Your secret key'
To use it in POC or production, you should add below to configuration file
type: OSS
config:
 bucket: "Your bucket name"
 endpoint: "OSS endpoint to connect to"
 access_key: "Your access key"
 secret_key: "Your secret key"

Some other configurations are http advanced configurations, will add documentation later.
I'm from Alibaba Cloud and would love to become the maintainer for the OSS provider. :)

Looking into tests failed.

@bwplotka @jojohappy @yeya24 Could you please help review this PR, our users need this support, thanks.:)

=== RUN   TestObjStore_AcceptanceTest_e2e
--- PASS: TestObjStore_AcceptanceTest_e2e (2.05s)
=== RUN   TestObjStore_AcceptanceTest_e2e/inmem
--- PASS: TestObjStore_AcceptanceTest_e2e/inmem (0.05s)
foreach.go:49: THANOS_SKIP_GCS_TESTS envvar present. Skipping test against GCS.
foreach.go:71: THANOS_SKIP_S3_AWS_TESTS envvar present. Skipping test against S3 AWS.
foreach.go:87: THANOS_SKIP_AZURE_TESTS envvar present. Skipping test against Azure.
foreach.go:103: THANOS_SKIP_SWIFT_TESTS envvar present. Skipping test against swift.
oss.go:341: created temporary OSS bucket for OSS tests with name test-testobjstore-acceptancetest-e2e-1ee0ded3a2691b90
=== RUN   TestObjStore_AcceptanceTest_e2e/Alibaba_OSS
--- PASS: TestObjStore_AcceptanceTest_e2e/Alibaba_OSS (0.86s)
foreach.go:135: THANOS_SKIP_TENCENT_COS_TESTS envvar present. Skipping test against Tencent COS.
PASS

@shaulboozhiao @wujinhu Thanks for your PR! Good job! But we could have only one implement of OSS, so could we make a discussion offline in slack?

Thanks @jojohappy @yeya24 for your suggestions，I will solve those issues. ”

sorry，i did not know you have worked on this before i created this pr.If you want to continue work on this，i will close this pr

i have talked with @jojohappy offline，i will close this pr，please calm down！


Bug located in booleans.py

🤓 Before submitting the issue

[X] I have visited the Troubleshooting section.
[X] I have searched among the existing issues.
[ ] I am using a Python virtual environment.
[X] I am using the latest version of PyMAPDL and its dependencies (fully updated virtual environment). You can update them using pip install --upgrade --upgrade-strategy eager ansys-mapdl-core.

🔍 Description of the bug
mapdl.asbl() seems to have a small bug in it's definition.
When issuing the command in pyansys and specifying that the initial area is to be deleted (keepa="DELETE"), the initial area is not deleted.
Workaround with specifing default values with BOPTN [ mapdl.boptn(value="NO") ] beforehand gets the job done.
When checking out the defintion of the asbl command, I think the error is in line 427 of file booleans.py (ansys-mapdl-core version 0.65.2)
Implementation is:
command = f"ASBL,{na},{nl},{keepa},{keepl}"
which should - according to my understanding - have an aditional komma before {keepa}:
command = f"ASBL,{na},{nl},,{keepa},{keepl}"
For reference, the APDL ASBL Command definition is:
ASBL, NA, NL, --, KEEPA, KEEPL
which has this blank space inbetween to presumably keep the format of for example the ASBA command that has the additional SEPO as a key:
ASBA, NA, NL, SEPO, KEEPA, KEEPL
The corresponding MAPDL Code would be for example:
FINISH
/CLEAR,NOSTART
/PREP7
k,1,0,0
k,2,0,1
k,3,1,1
k,4,1,0
a,1,2,3,4
l,1,3
asbl,1,5,,delete,keep
🕵 Steps To Reproduce
#Minimum working example
from ansys.mapdl.core import launch_mapdl
mapdl = launch_mapdl()

mapdl.prep7()
mapdl.k(1,0,0)
mapdl.k(2,1,0)
mapdl.k(3,1,1)
mapdl.k(4,0,1)
area_no = mapdl.a(1,2,3,4)
line_no = mapdl.l(1,3)

# mapdl.boptn(value="NO") # if issued, initial area is deleted

mapdl.asbl(na=area_no,nl=line_no,keepa="DELETE",keepl="KEEP") # keepa="delete" but area is still kept

mapdl.open_gui() # for visualization

mapdl.exit()

💻 Which Operating System are you using?
Windows
🐍 Which Python version are you using?
3.10
📝 PyMAPDL Report

PyMAPDL Software and Environment Report
Packages Requirements

Core packages
ansys.mapdl.core    : 0.65.2
numpy               : 1.23.3
appdirs             : 1.4.4
scipy               : 1.9.1
grpc                : Package not found
ansys.api.mapdl.v0  : Package not found
ansys.mapdl.reader  : 0.52.0
google.protobuf     : Package not found
Optional packages
matplotlib          : 3.5.3
pyvista             : 0.36.1
pyiges              : 0.3.1
tqdm                : 4.64.1
Ansys Installation

Version   Location
211       C:\Program Files\ANSYS Inc\v211
Ansys Environment Variables

ANSYS211_DIR                   C:\Program Files\ANSYS Inc\v211\ANSYS
AWP_LOCALE211                  de
AWP_ROOT211                    C:\Program Files\ANSYS Inc\v211
CADOE_LIBDIR211                C:\Program Files\ANSYS Inc\v211\CommonFiles\Language\de
📝 Installed packages
aiohttp==3.8.1
aiosignal==1.2.0
ansys-api-mapdl==0.5.1
ansys-api-platform-instancemanagement==1.0.0b3
ansys-mapdl-core==0.65.2
ansys-mapdl-reader==0.52.0
ansys-platform-instancemanagement==1.0.2
ansys-tools-path==0.3.1
appdirs==1.4.4
async-timeout==4.0.2
attrs==22.1.0
autopep8==2.0.1
certifi==2022.12.7
charset-normalizer==2.1.1
click==8.1.6
colorama==0.4.5
cycler==0.11.0
et-xmlfile==1.1.0
fatpack==0.7.7
fonttools==4.37.1
frozenlist==1.3.1
geomdl==5.3.1
googleapis-common-protos==1.56.4
grpcio==1.48.1
h5py==3.8.0
idna==3.3
imageio==2.21.3
importlib-metadata==4.12.0
kiwisolver==1.4.4
matplotlib==3.5.3
multidict==6.0.2
numpy==1.23.3
opencv-python==<IP_ADDRESS>
openpyxl==3.0.10
packaging==21.3
pandas==1.5.2
Pillow==9.2.0
platformdirs==3.10.0
protobuf==3.20.1
protoc-gen-swagger==0.1.0
psutil==5.9.5
pyansys-tools-versioning==0.4.0
pycodestyle==2.10.0
pyiges==0.3.1
pyparsing==3.0.9
python-dateutil==2.8.2
pytz==2022.7
pyvista==0.36.1
requests==2.28.2
scipy==1.9.1
scooby==0.6.0
six==1.16.0
tomli==2.0.1
tqdm==4.64.1
urllib3==1.26.14
vtk==9.2.0rc2
wslink==1.8.2
yarl==1.8.1
zipp==3.8.1
📝 Logger output file
No response

Hi @saschamk
Thank you very much for opening this issue. I'm working on it already!


Error running demo()

I am trying to run the demo and I get the following error
>>> saleae.demo()
Running Saleae connection demo.

Saleae connected.
Press Enter to continue...

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/saleae/saleae.py", line 493, in demo
    s.set_performance(PerformanceOption.Full)
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/saleae/saleae.py", line 267, in set_performance
    self._cmd('SET_PERFORMANCE, {}'.format(performance.value))
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/saleae/saleae.py", line 112, in _cmd
    self._send(s)
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/saleae/saleae.py", line 99, in _send
    self._s.send(bytes(s + '\0', 'UTF-8'))
TypeError: str() takes at most 1 argument (2 given)

Actually I just noticed your code is for Python 3. Do you plan to make it compatible with Python 2.7?

Someone else had an issue a little while back and I added some things that were able to fix Python2 support, see #5. I'm happy to merge in fixes for Py2k, but don't have the time to develop it myself.
Unfortunately, this isn't the easiest thing to add automated testing to since it requires an external program to be running.
Any particular reason you're still using Py2k? It is deprecated...

I did just test this locally with Python 2.7.11 and the demo seems to work fine.
Are you sure your installation is up to date, https://github.com/ppannuto/python-saleae/blob/master/saleae/saleae.py#L9 should fix this issue
Python 2.7.11 (default, Jan 11 2016, 00:48:43)
[GCC 4.2.1 Compatible Apple LLVM 7.0.2 (clang-700.1.81)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> type(bytes)
<type 'type'>
>>> bytes == str
True
>>> from builtins import bytes
>>> bytes == str
False
>>> type(bytes)
<class 'future.types.newbytes.BaseNewBytes'>

@ppannuto thanks for your replies. I am using Python 2.7 because many libraries use it and a many of my current projects were built in it. I decided to work with a virtual environment to use Python 3 and saleae.py. In any case my demo() never worked with Python 2.7 but I am not going to troubleshoot that any time soon.


Add Updated Conditon in Status

The health status in MySQLCluster.Status.Condition will be true even if all StatefulSet is not ready when updating MySQLCluster.
So if we use this status as a health check in ArgoCD, they proceed next wave in spite of the update of MySQLCluster is not finished.
As a solution to this problem, I added ConditionUpdated Condition in MySQLCluster.Status.Condition.
This condition will become true when an update of Statefulset is finished.

@YZ775
I guess UpToDate makes more sense than Updated.

@ymmt2005

@YZ775 I guess UpToDate makes more sense than Updated.

Thank you. I also think UpToDate is better.
I've renamed the field.

Please add documentation for this feature to reconcile.md.


Failed to build gem native extension: Your version of gcc is too old. You need at least version 4.2

Setting up a fresh install of El Capitan.
I get this error whenever I try to run brew cask.
⚡⇒  brew cask --config
Error: ERROR: Failed to build gem native extension.

    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby extconf.rb
-> sh /Library/Ruby/Gems/2.0.0/gems/dep-selector-libgecode-1.0.2/ext/libgecode3/vendor/gecode-3.7.3/configure --prefix=/Library/Ruby/Gems/2.0.0/gems/dep-selector-libgecode-1.0.2/lib/dep-selector-libgecode/vendored-gecode --disable-doc-dot --disable-doc-search --disable-doc-tagfile --disable-doc-chm --disable-doc-docset --disable-qt --disable-examples --disable-flatzinc
checking for the host operating system... Darwin
checking for g++... g++
checking whether the C++ compiler works... yes
checking for C++ compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C++ compiler... yes
checking whether g++ accepts -g... yes
checking for gcc... /usr/bin/gcc-4.2
checking whether we are using the GNU C compiler... no
checking whether /usr/bin/gcc-4.2 accepts -g... no
checking for /usr/bin/gcc-4.2 option to accept ISO C89... unsupported
checking for ranlib... ranlib
checking for diff... ok
checking for tar... ok
checking for make... ok
checking for sed... ok
checking for perl... ok
checking how to run the C++ preprocessor... g++ -E
configure: error: Your version of gcc is too old. You need at least version 4.2.
extconf.rb:98:in `block in run': Failed to build gecode library. (GecodeBuild::BuildError)
	from extconf.rb:97:in `chdir'
	from extconf.rb:97:in `run'
	from extconf.rb:104:in `<main>'

extconf failed, exit code 1

Gem files will remain installed in /Library/Ruby/Gems/2.0.0/gems/dep-selector-libgecode-1.0.2 for inspection.
Results logged to /Library/Ruby/Gems/2.0.0/extensions/universal-darwin-15/2.0.0/dep-selector-libgecode-1.0.2/gem_make.out

I've tried updating gcc:
⚡⇒  gcc -v
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/local/Cellar/gcc/5.2.0/libexec/gcc/x86_64-apple-darwin15.0.0/5.2.0/lto-wrapper
Target: x86_64-apple-darwin15.0.0
Configured with: ../configure --build=x86_64-apple-darwin15.0.0 --prefix=/usr/local/Cellar/gcc/5.2.0 --libdir=/usr/local/Cellar/gcc/5.2.0/lib/gcc/5 --enable-languages=c,c++,objc,obj-c++,fortran --program-suffix=-5 --with-gmp=/usr/local/opt/gmp --with-mpfr=/usr/local/opt/mpfr --with-mpc=/usr/local/opt/libmpc --with-isl=/usr/local/opt/isl --with-system-zlib --enable-libstdcxx-time=yes --enable-stage1-checking --enable-checking=release --enable-lto --with-build-config=bootstrap-debug --disable-werror --with-pkgversion='Homebrew gcc 5.2.0' --with-bugurl=https://github.com/Homebrew/homebrew/issues --enable-plugin --disable-nls --enable-multilib
Thread model: posix
gcc version 5.2.0 (Homebrew gcc 5.2.0)

Are you sure this is related to brew cask? What about just brew?
Also, there is no brew cask --config. Not sure where you got that from.

ugh, never mind, I had a rogue ENV var setting gcc to 4.2 for rvm. Going to purge my rvm in favor of rbenv and start over.

Not sure what's messed up here..
antic@antic:~/.dotfiles|atomantic⚡⇒  rvm use ruby-2.2.1
Using /Users/antic/.rvm/gems/ruby-2.2.1
antic@antic:~/.dotfiles|atomantic⚡⇒  brew doctor
Your system is ready to brew.
antic@antic:~/.dotfiles|atomantic⚡⇒  brew cask list

Error: ERROR: Failed to build gem native extension.

    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby extconf.rb
/Library/Ruby/Site/2.0.0/rubygems/ext/builder.rb:89:in `run': ERROR: Failed to build gem native extension. (Gem::Ext::BuildError)

    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby extconf.rb
/Library/Ruby/Site/2.0.0/rubygems/ext/builder.rb:89:in `run': ERROR: Failed to build gem native extension. (Gem::Ext::BuildError)

    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby extconf.rb
/Library/Ruby/Site/2.0.0/rubygems/ext/builder.rb:89:in `run': ERROR: Failed to build gem native extension. (Gem::Ext::BuildError)

    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby extconf.rb
/Library/Ruby/Site/2.0.0/rubygems/ext/builder.rb:89:in `run': ERROR: Failed to build gem native extension. (Gem::Ext::BuildError)

^^ repeats that about 50 times...
then starts repeating this:
extconf failed, exit code 128

Gem files will remain installed in /Library/Ruby/Gems/2.0.0/gems/ffi-yajl-1.1.0 for inspection.
Results logged to /Library/Ruby/Gems/2.0.0/extensions/universal-darwin-15/2.0.0/ffi-yajl-1.1.0/gem_make.out
	from /Library/Ruby/Site/2.0.0/rubygems/ext/ext_conf_builder.rb:37:in `block in build'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/tempfile.rb:324:in `open'
	from /Library/Ruby/Site/2.0.0/rubygems/ext/ext_conf_builder.rb:17:in `build'
	from /Library/Ruby/Site/2.0.0/rubygems/ext/builder.rb:161:in `block (2 levels) in build_extension'
	from /Library/Ruby/Site/2.0.0/rubygems/ext/builder.rb:160:in `chdir'
	from /Library/Ruby/Site/2.0.0/rubygems/ext/builder.rb:160:in `block in build_extension'
	from /Library/Ruby/Site/2.0.0/rubygems/ext/builder.rb:159:in `synchronize'
	from /Library/Ruby/Site/2.0.0/rubygems/ext/builder.rb:159:in `build_extension'
	from /Library/Ruby/Site/2.0.0/rubygems/ext/builder.rb:198:in `block in build_extensions'
	from /Library/Ruby/Site/2.0.0/rubygems/ext/builder.rb:195:in `each'
	from /Library/Ruby/Site/2.0.0/rubygems/ext/builder.rb:195:in `build_extensions'
	from /Library/Ruby/Site/2.0.0/rubygems/specification.rb:1436:in `block in build_extensions'
	from /Library/Ruby/Site/2.0.0/rubygems/user_interaction.rb:45:in `use_ui'
	from /Library/Ruby/Site/2.0.0/rubygems/specification.rb:1434:in `build_extensions'
	from /Library/Ruby/Site/2.0.0/rubygems/stub_specification.rb:60:in `build_extensions'
	from /Library/Ruby/Site/2.0.0/rubygems/basic_specification.rb:56:in `contains_requirable_file?'
	from /Library/Ruby/Site/2.0.0/rubygems/specification.rb:925:in `block in find_inactive_by_path'
	from /Library/Ruby/Site/2.0.0/rubygems/specification.rb:924:in `each'
	from /Library/Ruby/Site/2.0.0/rubygems/specification.rb:924:in `find'
	from /Library/Ruby/Site/2.0.0/rubygems/specification.rb:924:in `find_inactive_by_path'
	from /Library/Ruby/Site/2.0.0/rubygems.rb:185:in `try_activate'
	from /Library/Ruby/Site/2.0.0/rubygems/core_ext/kernel_require.rb:132:in `rescue in require'
	from /Library/Ruby/Site/2.0.0/rubygems/core_ext/kernel_require.rb:144:in `require'
	from extconf.rb:3:in `<main>'

extconf failed, exit code 1

ok, if anyone gets an issue like this, for some reason I had a bunch of old gems installed that needed cleaning. I purged (killed all of my installed gems):
sudo gem uninstall -aIx
gem cleanup

and now brew cask commands are happy

@atomantic you are my official hero of the day ;-)
I had troubles for hours getting cask up and running. Seems there was a gem installed that messed brews ruby environment  up. Pretty sure I will run into more troubles as my dev environment is now broken, but hey, at least cask is finally running...


Global loading state during route resolution

When switching between views, have a global loading state instead of partial loading states per component.

Consider using vuex.
router.beforeEach and router.afterEach might be a hint to get started.

I cannot recall the use case and why that might be better. Will keep it as soon s for now.


Working on Outlook for mac

How to make this plugin work in Outlook for mac ?
Thanks

I would like to know too.


OTA Configuration Updates

I have OTA firmware updates working, and am now trying to update the config.json file on the device thru the mqtt you have listed.  I must not understand the process you have listed in the readme, I am trying to read the config by using the $implementation/config but never get anything back.  Is there something specific I publish to that mqtt to get it to send the config?  From what I understand this can be done in normal mode, but I get no response.  I also tried to push json to change device name by itself to $implementation/config/set but didn't get any change or response there either.  My base_topic is set to sensor and the device ID is dev1, so I'm using sensor/dev1 for $implementation which I assume is correct.  Hopefully I'm just confused on the process.  Thanks, I love the project by the way.

Which Homie for ESP8266 version do you use?

I am using 1.5 from the public release.  From further digging thru the issues logs on here, I think I found my problem.  I have been using the homie-esp8266.readme.io documentation because everything was broken I tried to use on github and said it was now on there.  But that is for 2.0.0 isn't it.  So what I am trying to do isn't possible yet.
Sorry for the confusion.  I am new to github still, there's no where to beta test the new 2.0 that the public has access to is there?  Not a big deal, just curious.

So yes, the docs for v1.5.0 are here
You can beta-test the 2.0.0 (which is actually not even released as beta) by cloning the repository. The latest git available is the 2.0.0.


Testbed demos Can't resolve '@dimforge/rapier2d'

I'm trying to test out rapier.js for the first time, and according to the Getting started docs, I should be able to use the testbed3d/src/demos directory to get up and running.
Running npm install && npm run start from ${HOME}/rapier.js/testbed2d results in the following compilation error (and all subsequent files complain about being unable to find the module):
ERROR in ./src/index.ts 11:0-28

Module not found: Error: Can't resolve '@dimforge/rapier2d' in '${HOME}/rapier.js/testbed2d/src'

What is the entry point to working with testbed?

Similar (possibly related) issue here. I ran npm i @dimforge/rapier2d and tried to import it from my code and it's throwing a ERR_MODULE_NOT_FOUND. Can't get it to work at all.
Error [ERR_MODULE_NOT_FOUND]: Cannot find package '/home/runner/RapierTest/node_modules/@dimforge/rapier2d/' imported from /home/runner/RapierTest/index.js

I'm just getting this on a new 3D project:
Error: Cannot find module '@dimforge/rapier3d'

I have this problem, too:
Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'C:\repositories\<redacted>\node_modules\@dimforge\rapier2d\' imported from C:\repositories\<redacted>\<redacted>.js

I'm on Windows 10 if that matters.

@dsine-de -
I think the underlying issue with this error is in how the bundler works in your solution. For example, I could not get webpack to work at all.
In the end, I used the compat packages with node/typescript solutions and it works fine.
https://rapier.rs/docs/user_guides/javascript/getting_started_js#using-rapier-without-a-bundler
@sebcrozet - I think this should be closed as it isn't an error with the rapier project per say.


[RFC] 1Password (onepassword) support

Is your feature request related to a problem? Please describe.
I want to ensure that my secret data is accessed in a secure manner, from a secured enclave. The Vault integration is one solution to that, but I'm looking for a solution with other providers, e.g., 1Password.
Describe the solution you'd like
A clear and concise description of what you want to happen.
1Password provides a number of possible developer integrations. There are 2 potential solutions to this integration with 1Password

The Python SDK

https://developer.1password.com/docs/sdks
https://github.com/1Password/onepassword-sdk-python/
Benefits are being directly integrated with Python
Drawbacks are the Service Account requirement and Service Account limitations (namely that it's subject to rate limits and only certain 1Password Vaults)

Integration with the op CLI

https://developer.1password.com/docs/cli
There are two potential integration paths with op CLI

op read on each password lookup

This could potentially be done with a custom converter
The downside on this option would be the requirment of the op read lookup for each secret. Obviously not ideal for a large amount of secrets

op inject

The suggestion here would be to support the 1Password template syntax (possibly behind an existing @format token, or even a new @op token)
And then once the final file or data is compiled, run it through op inject to allow the op CLI to replace all the 1Password template references
This would require a deeper integration with Dynaconf, but would be provide the quickest parsing since it would be a one-shot and done

Describe alternatives you've considered
The alternative is essentially what I'm already going to start looking at, which is integrating with op read as a custom converter. And then I would add my custom converter to any code that I desire. It would seem, based on lack of existing issues, that perhaps I'm the only person interested in this integration, but I do think it would have wider community benefits ¯\_(ツ)_/¯
My implementation preference would be 2.b, the op inject solution. Realistically, that level of integration is beyond my grasp with the complexities of this solution. I will post back on the progress that I make at least with 2.a 👍

Here's my per secret op read and @op token solution
# settings.py
from subprocess import DEVNULL, PIPE, run
from dynaconf import Dynaconf, add_converter

def op_read(secret_reference):
    p = run(["op", "read", secret_reference], shell=False, stdout=PIPE, stderr=DEVNULL, text=True, check=False)
    return p.stdout.strip()

add_converter("op", op_read)

# settings.toml
PASSWORD = "@op op://Vault/website.com/password"

Tested and working 👍

Hey, thanks for sharing!
I'm glad you could make it work with converters, at least.
For 2.b, there are several approaches. This makes me think of something in the loading scope, like a general pre-parsing hook, or custom loaders.
Anyway, the integration details provided here are relevant to #1152 .