Terraform throws errors on subsequent deployments.

Terraform Version Details
I am getting errors when re-deploying to the same AWS subaccount and region. Screenshot of errors attached. I have deleted everything that I can find in the account that this originally deployed to. I have deleted the tfstate backup file in the application deployment.
Terraform Version Details:
{
"terraform_version": ""1.1.7"",
"terraform_revision": "null",
"terraform_outdated": "true",
"provider_selections": "{}"
}

### Terraform Variable File Details

# !NOTE! - These are only a subset of the variables in CONFIG-VARS.md provided
# as examples. Customize this file to add any variables from CONFIG-VARS.md whose
# default values you want to change.

# ****************  REQUIRED VARIABLES  ****************
# These required variables' values MUST be provided by the User
prefix                                  = "viya4aws"
location                                = "us-east-2" # e.g., "us-east-1"
# ****************  REQUIRED VARIABLES  ****************

# !NOTE!

# **************  RECOMMENDED  VARIABLES  ***************
default_public_access_cidrs = ["<IP_ADDRESS>/32"]  # e.g., ["<IP_ADDRESS>/32"]
ssh_public_key              = "~/.ssh/id_rsa.pub"
# **************  RECOMMENDED  VARIABLES  ***************

# Tags for all tagable items in your cluster.
#tags                                    = { } # e.g., { "key1" = "value1", "key2" = "value2" }

# Postgres config - By having this entry a database server is created. If you do not
#                   need an external database server remove the 'postgres_servers'
#                   block below.
#postgres_servers = {
#  default = {},
#}

## Cluster config
kubernetes_version                      = "1.21"
default_nodepool_node_count             = 2
default_nodepool_vm_type                = "m5.2xlarge"
default_nodepool_custom_data            = ""

## General
efs_performance_mode                    = "maxIO"
storage_type                            = "standard"

## Cluster Node Pools config
node_pools = {
  cas = {
    "vm_type" = "m5.2xlarge"
    "cpu_type"     = "AL2_x86_64"
    "os_disk_type" = "gp2"
    "os_disk_size" = 200
    "os_disk_iops" = 0
    "min_nodes" = 1
    "max_nodes" = 5
    "node_taints" = ["workload.sas.com/class=cas:NoSchedule"]
    "node_labels" = {
      "workload.sas.com/class" = "cas"
    }
    "custom_data" = ""
    "metadata_http_endpoint"               = "enabled"
    "metadata_http_tokens"                 = "required"
    "metadata_http_put_response_hop_limit" = 1
  },
  compute = {
    "vm_type" = "m5.8xlarge"
    "cpu_type"     = "AL2_x86_64"
    "os_disk_type" = "gp2"
    "os_disk_size" = 200
    "os_disk_iops" = 0
    "min_nodes" = 1
    "max_nodes" = 5
    "node_taints" = ["workload.sas.com/class=compute:NoSchedule"]
    "node_labels" = {
      "workload.sas.com/class"        = "compute"
      "launcher.sas.com/prepullImage" = "sas-programming-environment"
    }
    "custom_data" = ""
    "metadata_http_endpoint"               = "enabled"
    "metadata_http_tokens"                 = "required"
    "metadata_http_put_response_hop_limit" = 1
  },
  stateless = {
    "vm_type" = "m5.4xlarge"
    "cpu_type"     = "AL2_x86_64"
    "os_disk_type" = "gp2"
    "os_disk_size" = 200
    "os_disk_iops" = 0
    "min_nodes" = 1
    "max_nodes" = 5
    "node_taints" = ["workload.sas.com/class=stateless:NoSchedule"]
    "node_labels" = {
      "workload.sas.com/class" = "stateless"
    }
    "custom_data" = ""
    "metadata_http_endpoint"               = "enabled"
    "metadata_http_tokens"                 = "required"
    "metadata_http_put_response_hop_limit" = 1
  },
  stateful = {
    "vm_type" = "m5.4xlarge"
    "cpu_type"     = "AL2_x86_64"
    "os_disk_type" = "gp2"
    "os_disk_size" = 200
    "os_disk_iops" = 0
    "min_nodes" = 1
    "max_nodes" = 3
    "node_taints" = ["workload.sas.com/class=stateful:NoSchedule"]
    "node_labels" = {
      "workload.sas.com/class" = "stateful"
    }
    "custom_data" = ""
    "metadata_http_endpoint"               = "enabled"
    "metadata_http_tokens"                 = "required"
    "metadata_http_put_response_hop_limit" = 1
  }
}

# Jump Server
create_jump_vm                        = true
create_nfs_public_ip = false

Steps to Reproduce
To reproduce this, all I had to do was:

Use terraform to deploy (successfully)
Use terraform to delete (successfully)
Run the same terraform deploy command.

These errors have occurred ever since the first initial and successful deployment.
Expected Behavior
It should deploy/destroy successfully with no errors each and every time it's run.
Actual Behavior
On the second deployment, it will throw 3 errors.
Additional Context
No response
References
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Is the kubeconfig file the code is access the correct cluster? The endpoint the code is trying to access for the cluster in your code above is:

I can verify that the endpoint listed each time we attempt to deploy is matching what is shown in AWS. It is getting regenerated with a new prefix of characters each time. I have destroyed and redeployed since my last update, and I get the same error, with a different endpoint URL. That URL does match what AWS shows in the console as the API server endpoint.

In terraform.tfvars, is the value for default_public_access_cidrs the public IP of the machine you are deploying from?

Yes, it is. My public IP is currently "<IP_ADDRESS>" and that is what I've put into the default_public_access_cidrs parameter.

I didn't mean to imply that the actual parameter is <IP_ADDRESS> without the cidr range at the end. If you look at the screenshot in my original post, it includes the "/32" in the parameter value exactly as you have shown.

Ok, so the timeout is saying that from whatever machine you're running tf that it cannot access that endpoint for the cluster. I would try from that box setting KUBECONFIG to the kubernetes config file and running kubectl get node -o wide to see if you can access the cluster from that machine

@tclehman I have seen the three errors before during some testing. I was deploying from a bastion server in AWS when it occurred. I was able to work around the issue by adding the -eks_worker_sg security group to the bastion server, then rerunning the terraform apply. I know it doesn't solve the mystery, but it allows the infrastructure to build successfully (for me anyway) and the cluster has worked as I expect ever since.

@Technolero
I went in and checked the bastion host (on my deployment its called the jump-vm, I assume these are the same thing) and it seems to already have that security group attached to it. Please see attached screenshot.

@thpang  I am trying to capture the output but am unfortunately lacking in expertise here. I believe I need to be inside the cluster to get that kubectl command working, correct? I was having trouble getting the command to work. See screenshot below.

@Technolero I misunderstood before, makes sense.
I ran ifconfig on the terraform host that I am running the deployment from. For your reference, here is the output:

Then I added those IP's to the eks_worker_sg:

I then reran the deployment, but unfortunately still get the same errors:

@Technolero
As an additional test, I opened up all traffic on that security group. I still got the error on deployment:

From the <IP_ADDRESS>/32 machine were you able to run the kubectl command and look at your cluster?

Kubectl has been installed on the docker image, and not the host machine. With that in mind, it appears I am only able to run it from with the container. Would you know how I would be able to exec into the container to run the commands you are suggesting.

@Technolero  @thpang
I suspect the issue is due to network accessibility of the terraform host itself. Would it be ok if I added a public ALB in front of the terraform host, and pointed the default_public_access_cidr to that load balancer?

@tclehman when you did the ifconfig, that returned your private IPs, which were not what connects to the API endpoint, your public IP is. Not sure about your AWS VPC architecture, but either your terraform host has a public IP assigned to it or the NAT gateway in your VPC has one, and those are the public IPs that need to be added to the worker security group. You previously mentioned a <IP_ADDRESS> IP, what resource is that associated with?

@tclehman, were you able to resolve the load balancer issue that you were having?

I think so. It's been over a year I don't recall what this issue even was for.

Thanks, closing #167 as resolved.


[BUG]: Infinity/zero points not filtered out

Description
When running MSMs that include infinity/zero points, an error occurs and the resulting point is a zero point
Reproduce
Run msm with at least one point being a zero/infinity point
Expected Behavior
Zero points should be filtered out and a correct result should be returned

Just to comment on this one: the only protocol where I've seen zero points is Groth16, I don't think SRS-based schemes might have zero points in their MSMs. And in gnark's Groth16 zero points are filtered on CPU, right? I think that's the reason why we haven't implemented this yet, though I'm also not sure what's the best way to do this on GPU

Lurk-labs told us that they have this problem as well (many zero points) in Nova.
cc: @krakhit @ChickenLover

@omershlo : If I remember correctly he meant zero scalars, it affects the distribution from being uniform. I believe ICICLE algorithms already takes this into account. Zero points are not generally present in SRS based protocols.
@jeremyfelder You encountered zero scalars in ezkl right?

@krakhit yea we do have efficient handling of zero scalars in our MSM

Closing, completed in #372


Inconsistencies in docs and cookiecutter project

I've found a few inconsistencies in the docs or in the docs vs the cookiecutter widget project, probably caused by API changes at some point if I were to guess. I may be wrong, but my impression is that these are doing the same:
this.listenTo(...)
this.model.on(...)

and
this.save_changes()
this.touch()

The cookiecutter project uses this.model.on, while listenTo and save_changes are mentioned at:
https://ipywidgets.readthedocs.io/en/latest/examples/Widget Low Level.html
touch is mentioned in the docs here without explaining its signature:
https://ipywidgets.readthedocs.io/en/latest/examples/Widget Custom.html
If these are not the same, the documentation is lacking as to what the difference is.
Also, the cookie cutter project uses DOMWidgetModel while the docs only mention WidgetModel, also at
https://ipywidgets.readthedocs.io/en/latest/examples/Widget Low Level.html

Nevermind. Although I still think the documentation for ipywidgets is a bit thin, these are mostly from the Backbone.js API which is well documented.

It is definitely thin - thanks for bringing these up. Here is some info not in backbone:
save_changes just saves changes without providing a callback for output. touch does provide a callback for output, so that output produced as a result of the change can be seen by the user. Typically, a view will use .touch(). A model will use save_changes.
As for the widget models, if your model is representing an element in the DOM, it should probably use DOMWidgetModel. Some models do not represent elements on the page, and should use the more abstract WidgetModel (e.g., most of the models in pythreejs, which represent elements in the webgl scene graph, use WidgetModel).


Passing true/false for --{type}-bust in command line

I just realized that in order to set true/false in command line you have to use 0 or 1. Like this:
svg-sprite --css --css-bust=0 --css-sprite icons-sprite-css.svg svg/*.svg
If you use true/false it is treated as a string and it isn't parsed before https://github.com/jkphl/svg-sprite/blob/master/lib/svg-sprite/mode/base.js#L76 , getting always true as value ( !!'false' ).

Hey @arsuceno,
thanks for clarifying this. Yes, I already noticed that there must have been a change in yargs regarding the way it is dealing with boolean arguments / switches. I'll investigate this as soon as I find some time.
Cheers,
Joschi

@arsuceno The behavior of boolean arguments should be fixed now. Please use --switch or --switch=true to enable a boolean feature. Any other value (or omitting the argument altogether) will result in the default value (mostly false). Please let me know if this still causes you troubles.


load and remove different forms succesively on the same page (json communication via websocket)

Hi, I managed to make a small modification to ngformbuilder-full.js so that it pushes submissiondata to a websocket. I can load the formdefinitions json also via the websocket, after creating the necessary div elements in javascript, with the javascript part in https://github.com/formio/ngFormBuilder/blob/master/index.html in a function where the json is an input param and assign $rootScope.form = the input parameter. the formbuilder is loaded via
   if (typeof angular == 'undefined') {
    $.ajax({
      cache: true,
      url: '/js/ngFormBuilder-full.js',
      dataType: 'script',
      success: function () {
        document.getElementById('taskReferenceDiv').task.renderFormIOFormContents(resp.formfields[0])
      }
    });
    }

This works. My question is: after submit of a first form I would like load a next json with formdefinitions, while I stay on the same page. I can remove the divs I named above after the submit, maybe do $rootScope.$destroy(), but how can I prepare to show this next form? typeof angular != 'undefined' now, I cannot load ngFormBuilder-full.js a second time. I have no particular angular knowledge.
Appreciate your work, it is a great product!

This is what our wizard component does where it just swaps out the form= attribute with the new Form schema. The component will render the new form dynamically. Here is some code where this works.
https://github.com/formio/ngFormio/blob/master/src/directives/formioWizard.js#L91
Hope this helps.
Travis.

Thanks to your suggestion I have it working. I cannot use a formiowizard as the order of the forms must be determined runtime. Should there be no problem concerning memory leaks with the following solution?
After submission of a form's data I have
$rootScope.form.components = [];
$rootScope.form.components.length = 0;

To swap forms I use
$rootScope.form = formComponents; // SHO was incoming parameter
$rootScope.renderForm = true;
$rootScope.$digest()


(ENG-1217) Improve performance of CassandraEquivalentContentStore

Refactors logic to only query for content that is likely to be needed by traversing the equivalence graph before fetching content
Adjusts logic of EquivalenceGraphFilter to remove unintuitive behaviour of selecting nodes that don't belong to the requested graph sources

Abandoned in favour of ENG-1280


8344.3 fix github system integration

https://github.com/Shippable/pm/issues/8344
update github auth system integration to show following values in UI

Client Id (textbox, editable)
Client secret (textbox, editable)
WWW Url (textbox, non editable. Value is set from www system integration )

verify that client id and client secret values are correctly being set in vault
move the section to edit system configuration before auth provider section

complete. closing.


Docs feedback: windows grafana alloy arugments

URL
https://grafana.com/docs/alloy/latest/tasks/configure/configure-windows/
Feedback
Trying to migrate from grafana agent static to grafana alloy and the Arguments section is lack luster. Not seeing anything in regards to passing the equivalent of
-config.expand-env=true
Am I missing something or does this not exist with Alloy / should I be doing it another way?

Hi, -config.expand-env=true isn't necessary in Alloy because of the env function always being available.
For example:
logging {
  level = env("LOG_LEVEL")
}

Makes sense and I was able to get what I wanted with this. Closing it out


There always report errors when build service "smartkg"

It can be seen from the Docker-compose file that the application contains three services, namely aspnetcore, smartkg, and ui. When I  execute "sudo docker-compose build --build-arg DOCKER_HOST=${docker_host_ip}" , there always report errors about smartkg as follows,
=> ERROR [smartkg internal] load metadata for docker.io/library/aspnetcore:latest                            48.0s
failed to solve: DeadlineExceeded: DeadlineExceeded: DeadlineExceeded: aspnetcore: failed to do request: Head "https://registry.docker-cn.com/v2/library/aspnetcore/manifests/latest?ns=docker.io": dial tcp <IP_ADDRESS>:443: i/o timeout
It seems that the image 'docker.io/library/aspnetcore:latest' can not be accessed,
Can you help me fix this issue?

You'd better replace ${docker_host_ip} to your own server IP.
Meanwhile, you need a docker account. Then use it in docker login.
Otherwise, you will get error for "ERROR: Service 'aspnetcore' failed to build: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit
"


Add files via upload for metal chassis - 2nd code

This is Zackary's code for metal chassis robot that is trying to grab the rings to place on horizontal electrophoresis.

This code is not going to be merged because it is for the metal chassis robot, coded by Zackary.


zrender文件夹下npm run prepublish，生成的lib中缺少文件

General Questions

Issue Type

[ ] I have a question to ask about how to use ECharts to ...；我想提问如何使用 ECharts 实现某功能
[ x] I have a bug to report；我想要报 bug
[ ] I have a feature to request, e.g.: I'd like a new feature that ...；我需要一个新功能
[ ] I have a feature to enhance, e.g.: The current feature should be improved in the way that ...；我希望改进某个功能
[ ] There's something wrong with the documents；文档有些问题
[ ] Others, or I'm not sure which issue types to choose here；其他，或我不知道应该选什么类型

Issue Details

通过在incubator-echarts文件夹下npm install或者在zrender文件夹下npm run prepublish，生成的lib中缺少文件，造成运行时出现module not found的问题出现

Expected Behavior

Current Behavior

ERROR in ./lib/echarts/lib/util/graphic.js
Module not found: Error: Cannot resolve module 'zrender/lib/graphic/helper/subPixelOptimize' in E:\work-space\work\sh-isoc\sh-isoc-situation-web\src\main\develop\lib\echarts\lib\util
@ ./lib/echarts/lib/util/graphic.js 104:27-81

Online Example

Topics

Anything Else We Need to Know

Environment

ECharts version；ECharts 版本: {v4.2.0-rc.2}

[ ] It happens only on certain browsers or operating systems. 对于特定浏览器或操作系统才会出现的问题，请提供相应环境信息：{BROWSER_VERSION_OR_OS_INFORMATION_HERE}

I believe this has been fixed. Please check again and open a new issue if this problem still exists.


tagger: adapt severity

What does this PR do?
Change the severity of a tagger warning.
Motivation
Today it's too verbose for various reasons.
The main issue is in the Kubernetes kubelet code with non updated Static Pods:
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kubelet: container docker://10f1442e162659d1143b4ac333d9d77e0e07adf1d008b401c3d2e5f7e66558a9 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kube-service-collector: container docker://10f1442e162659d1143b4ac333d9d77e0e07adf1d008b401c3d2e5f7e66558a9 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kubelet: container docker://10f1442e162659d1143b4ac333d9d77e0e07adf1d008b401c3d2e5f7e66558a9 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kube-service-collector: container docker://10f1442e162659d1143b4ac333d9d77e0e07adf1d008b401c3d2e5f7e66558a9 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kube-service-collector: container docker://7c3d6b021ee8ae048a557595becf33e74fde5095ba31ed3f22b8ad317f3c32f2 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kubelet: container docker://7c3d6b021ee8ae048a557595becf33e74fde5095ba31ed3f22b8ad317f3c32f2 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kubelet: container docker://7c3d6b021ee8ae048a557595becf33e74fde5095ba31ed3f22b8ad317f3c32f2 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kube-service-collector: container docker://7c3d6b021ee8ae048a557595becf33e74fde5095ba31ed3f22b8ad317f3c32f2 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kubelet: container docker://7c3d6b021ee8ae048a557595becf33e74fde5095ba31ed3f22b8ad317f3c32f2 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kube-service-collector: container docker://7c3d6b021ee8ae048a557595becf33e74fde5095ba31ed3f22b8ad317f3c32f2 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kubelet: container docker://7c3d6b021ee8ae048a557595becf33e74fde5095ba31ed3f22b8ad317f3c32f2 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kube-service-collector: container docker://7c3d6b021ee8ae048a557595becf33e74fde5095ba31ed3f22b8ad317f3c32f2 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kubelet: container docker://7c3d6b021ee8ae048a557595becf33e74fde5095ba31ed3f22b8ad317f3c32f2 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kube-service-collector: container docker://7c3d6b021ee8ae048a557595becf33e74fde5095ba31ed3f22b8ad317f3c32f2 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kubelet: container docker://10f1442e162659d1143b4ac333d9d77e0e07adf1d008b401c3d2e5f7e66558a9 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kube-service-collector: container docker://10f1442e162659d1143b4ac333d9d77e0e07adf1d008b401c3d2e5f7e66558a9 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kube-service-collector: container docker://10f1442e162659d1143b4ac333d9d77e0e07adf1d008b401c3d2e5f7e66558a9 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kubelet: container docker://10f1442e162659d1143b4ac333d9d77e0e07adf1d008b401c3d2e5f7e66558a9 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kubelet: container docker://10f1442e162659d1143b4ac333d9d77e0e07adf1d008b401c3d2e5f7e66558a9 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kube-service-collector: container docker://10f1442e162659d1143b4ac333d9d77e0e07adf1d008b401c3d2e5f7e66558a9 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kubelet: container docker://7c3d6b021ee8ae048a557595becf33e74fde5095ba31ed3f22b8ad317f3c32f2 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kube-service-collector: container docker://7c3d6b021ee8ae048a557595becf33e74fde5095ba31ed3f22b8ad317f3c32f2 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kube-service-collector: container docker://10f1442e162659d1143b4ac333d9d77e0e07adf1d008b401c3d2e5f7e66558a9 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kubelet: container docker://10f1442e162659d1143b4ac333d9d77e0e07adf1d008b401c3d2e5f7e66558a9 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kubelet: container docker://7c3d6b021ee8ae048a557595becf33e74fde5095ba31ed3f22b8ad317f3c32f2 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kube-service-collector: container docker://7c3d6b021ee8ae048a557595becf33e74fde5095ba31ed3f22b8ad317f3c32f2 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kubelet: container docker://10f1442e162659d1143b4ac333d9d77e0e07adf1d008b401c3d2e5f7e66558a9 not found in podList
[ AGENT ] 2018-02-23 15:30:49 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kube-service-collector: container docker://10f1442e162659d1143b4ac333d9d77e0e07adf1d008b401c3d2e5f7e66558a9 not found in podList
[ AGENT ] 2018-02-23 15:30:50 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kube-service-collector: container docker://7c3d6b021ee8ae048a557595becf33e74fde5095ba31ed3f22b8ad317f3c32f2 not found in podList
[ AGENT ] 2018-02-23 15:30:50 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kubelet: container docker://7c3d6b021ee8ae048a557595becf33e74fde5095ba31ed3f22b8ad317f3c32f2 not found in podList
[ AGENT ] 2018-02-23 15:30:50 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kube-service-collector: container docker://10f1442e162659d1143b4ac333d9d77e0e07adf1d008b401c3d2e5f7e66558a9 not found in podList
[ AGENT ] 2018-02-23 15:30:50 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kubelet: container docker://10f1442e162659d1143b4ac333d9d77e0e07adf1d008b401c3d2e5f7e66558a9 not found in podList
[ AGENT ] 2018-02-23 15:30:50 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kubelet: container docker://7c3d6b021ee8ae048a557595becf33e74fde5095ba31ed3f22b8ad317f3c32f2 not found in podList
[ AGENT ] 2018-02-23 15:30:50 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kube-service-collector: container docker://7c3d6b021ee8ae048a557595becf33e74fde5095ba31ed3f22b8ad317f3c32f2 not found in podList
[ AGENT ] 2018-02-23 15:30:50 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kubelet: container docker://10f1442e162659d1143b4ac333d9d77e0e07adf1d008b401c3d2e5f7e66558a9 not found in podList
[ AGENT ] 2018-02-23 15:30:50 UTC | WARN | (tagger.go:246 in Tag) | error collecting from kube-service-collector: container docker://10f1442e162659d1143b4ac333d9d77e0e07adf1d008b401c3d2e5f7e66558a9 not found in podList

@xvello IMHO the change needs to be done where we cannot find the containerID: pkg/util/kubernetes/kubelet/kubelet.go
But we cannot use the custom collectors.ErrNotFound because of circular imports.
I propose to use a dedicated pkg errors for that.

this should fix #1447 and #1037


Add Dynamic Clipping methodology for impression measurement

Add documentation that if EDP uses the new methodology for impression, the maximum_frequency_per_user in measurement_spec will be ignored
Design doc: https://docs.google.com/document/d/1vboqbbHOqblYhooYCwAJ2fyqBCZOILFKmxtCm_O-kaY/edit?usp=sharing

This change is


Validate certificate of https://downloads.linux.hpe.com - stores the RPM GPG keys

In https://github.com/CSC-IT-Center-for-Science/ansible-role-hp-spp/blob/master/tasks/redhat.yml#L4
we get the RPM GPG keys over http.
With https:// you get this error:

msg: Failed to validate the SSL certificate for downloads.linux.hpe.com:443. Make sure your managed systems have a valid CA certificate installed.  If the website serving the url uses SNI you need python >= 2.7.9 on your managed machine.  You can use validate_certs=False if you do not need to confirm the server\s identity but this is unsafe and not recommended Paths checked for this platform: /etc/ssl/certs, /etc/pki/ca-trust/extracted/pem, /etc/pki/tls/certs, /usr/share/ca-certificates/cacert.org, /etc/ansible

Same here, seems like HP switched to a new domain for their repository but their certificate is incomplete.

HP scores an F!? Unbelievable.

I now get an A from sslabs and if I test with fetching them over https it works.
However, the benefits for doing this seems small. Closing this issue and keeping as http.


The Environment has not been configured yet!

Context

To Fix Firebase Leak, I have provided the code in #35 to use Environment Variables
It's expected by you, to put the Environment variables

Steps to do it
Repo > Settings > Secrets > and then Add Variables from .env.prod file I have provided you
Then please Trigger the NextJS Workflow to redeploy

Tried fixing using Commit 3f9eb8f05b


Assortment of improvements for CentOS 7

Hi,
Here are a handful of changes that I've had to make during my use of your module. No longer does it complain that I need to recreate the Ceph rbd datastore each time :)
Cheers,
Gavin.

hi there, i will have a closer look at it tomorrow. Generally looks good for me, but @tuxmea might have some comments for the types/providers.

Cool, thanks.
On the failed tests - I don't understand why it failed the Debian APT key tests, however the RedHat /sbin/brctl symlink makes sense. Commit 23d9c8e specifically removed the symlink for RedHat since otherwise you end up with a symlink to itself and no usable brctl binary :(

i don't see the point with brctl... this works in centos 6. the brctl package delivers only a binary in /usr/sbin but opennebula needs it in /sbin. do you use centos 7, was it changed there?
[rwaffen@srv ~]$ rpm -ql bridge-utils
/usr/sbin/brctl
/usr/share/doc/bridge-utils-1.2
/usr/share/doc/bridge-utils-1.2/AUTHORS
/usr/share/doc/bridge-utils-1.2/COPYING
/usr/share/doc/bridge-utils-1.2/FAQ
/usr/share/doc/bridge-utils-1.2/HOWTO
/usr/share/man/man8/brctl.8.gz

Thanks for taking the time to review and provide feedback :)
I've been using CentOS 7 for this build (my usual choice is Debian and I fancied a change). bridge-utils looks similar to CentOS 6:
[root@userver1 ~]# rpm -ql bridge-utils
/usr/sbin/brctl
/usr/share/doc/bridge-utils-1.5
/usr/share/doc/bridge-utils-1.5/AUTHORS
/usr/share/doc/bridge-utils-1.5/COPYING
/usr/share/doc/bridge-utils-1.5/FAQ
/usr/share/doc/bridge-utils-1.5/HOWTO
/usr/share/man/man8/brctl.8.gz

However the part which may differ are these symlinks in the root:
[root@userver1 ~]# ls -l /
total 32
lrwxrwxrwx.   1 root root    7 Mar 20 21:33 bin -> usr/bin
dr-xr-xr-x.   4 root root 4096 Mar 20 22:10 boot
drwxr-xr-x   22 root root 3440 Mar 22 18:26 dev
drwxr-xr-x.  94 root root 8192 Mar 22 16:19 etc
drwxr-xr-x.   2 root root    6 Jun 10  2014 home
lrwxrwxrwx.   1 root root    7 Mar 20 21:33 lib -> usr/lib
lrwxrwxrwx.   1 root root    9 Mar 20 21:33 lib64 -> usr/lib64
drwxr-xr-x.   2 root root    6 Jun 10  2014 media
drwxr-xr-x.   5 root root   37 Mar 22 20:41 mnt
drwxr-xr-x.   2 root root    6 Jun 10  2014 opt
dr-xr-xr-x  147 root root    0 Mar 21 18:19 proc
dr-xr-x---.  11 root root 4096 Mar 22 15:55 root
drwxr-xr-x   32 root root 1040 Mar 22 21:40 run
lrwxrwxrwx.   1 root root    8 Mar 20 21:33 sbin -> usr/sbin

Hence when Puppet installs a symlink at /sbin/brctl it's actually overwriting the binary at /usr/sbin/brctl - the symlinking of /bin and /sbin are new in RHEL7/CentOS7:
Ref: https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/7/html/Migration_Planning_Guide/sect-Red_Hat_Enterprise_Linux-Migration_Planning_Guide-File_System_Layout.html#sect-Red_Hat_Enterprise_Linux-Migration_Planning_Guide-File_System_Layout-New_layout_for_root_file_system
In that case, my change is incorrect since it should only apply when $::lsbmajdistrelease == 7

Interesting. My changes have somehow managed to break compatibility with Ruby 1.8.7.
Aside from why this might be the case, is that version of Ruby still considered important to support? Upstream abandoned it in 2013, and even Ubuntu's extended support stopped in July 2014.

Many people still use puppet on ruby 1.8
The error is within the version compare.
You are comparing string (facter) with integer.
Please make use of the versioncmp function.
Please change your code to:
diff --git a/manifests/compute_node/config.pp b/manifests/compute_node/config.pp
index 913fb65..636ae7b 100644
--- a/manifests/compute_node/config.pp
+++ b/manifests/compute_node/config.pp
@@ -76,7 +76,7 @@ class one::compute_node::config (
     }
   }

-  if $::osfamily == 'RedHat' and $::lsbmajdistrelease < 7  {
+  if $::osfamily == 'RedHat' and versioncmp($::lsbmajdistrelease, '7') < 0  {
     file { '/sbin/brctl':
       ensure => link,
       target => '/usr/sbin/brctl',

Then the tests will pass.

Thanks for the insight - I'm still fairly new with Puppet so I would never have realised the versioncmp method.
Was there something specific in the build logs which pointed to that conclusion, or simply greater experience and exposure?

When running the spec tests one saw the error message: "undefined method `<' for :undef:Symbol"
Something like this is always due to comparing incompatible data.

tests are green now :smile: ... looks good to me... @tuxmea are we ready to merge?

No. Merge conflicts! @gdhgdhgdh you need to rebase your branch to upstream master.

ah, okay, i see. there's also a big warning left to my (now grey) merge button.

Tonight was not the right time to try a rebase or merge for the first time.

;-)

so, merged... lets see what happen :smile:


Updated the size of DELETE button

I've changed the size of DELETE button. Now it is equal in size to that of EDIT button.
Fixes issue #145

What's this PR do? (related issue if exists)
Any context you want to provide on the implementation?
Screenshots? (necessary if an UI/UX change)

Duplicate of #147

@ineffable23 you can try #141


The ability to easily download an entire folder would be nice.

how do I easily get documents back out? I found how to download a single document, but that was hidden away pretty well. The ability to easily download an entire folder would be nice.

Feature added (well, UT are still missing... but that is a minor detail).
Will be shipped as part of 1.5.0.


Manage license through sensu_license type

Pull Request Checklist

Description

Manage licenses through sensu_license type.
Related Issue

Fixes #1217
Motivation and Context

Previously a failed license add would not retry if license wasn't added and only way to force Puppet to re-add failed Exec was to modify or delete the license file and let it get re-created and trigger the Exec. This ensures some better logic around license management and ensures changes to the license file still refresh the license inside sensuctl.

Great work!


publish<EMAIL_ADDRESS>

Requested by: @st0012
Quick links:

View changes
View check runs

Assign the accepted label to this issue to approve the release.

Since it's been late on Friday, let's wait till Monday so we can have time to release other integration libraries 🙂

@HazAT do you think we can release this today?


Some errors are occurred in the tutorial

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I'm convinced that this is not my fault but a bug.

Description
There are several error outputs in the tutorial.
planning simulation errors (no initial pose and goal)

[component_container_mt-2] [ERROR<PHONE_NUMBER>.717185575] [system.system_monitor.hdd_monitor]: Failed to execute findmnt. /dev/sda3 (getDeviceFromMountPoint() at /home/yukky/workspace/awf-autoware/src/universe/autoware/system/system_monitor/src/hdd_monitor/hdd_monitor.cpp:336)

[system_error_monitor-5] [ERROR<PHONE_NUMBER>.426385570] [system_error_monitor system_error_monitor/input_data_timeout]: [Single Point Fault]:  (logThrottledNamed() at /home/yukky/workspace/awf-autoware/src/universe/autoware/system/system_error_monitor/src/system_error_monitor_core.cpp:54)

planning simulation errors (after set initial pose and goal)

rosbag play simulation errors (before play rosbag)

[component_container_mt-3] [ERROR<PHONE_NUMBER>.399824469] [system.system_monitor.hdd_monitor]: Failed to execute findmnt. /dev/sda3 (getDeviceFromMountPoint() at /home/yukky/workspace/awf-autoware/src/universe/autoware/system/system_monitor/src/hdd_monitor/hdd_monitor.cpp:336)

[system_error_monitor-6] [ERROR<PHONE_NUMBER>.984191756] [system_error_monitor system_error_monitor/input_data_timeout]: [Single Point Fault]:  (logThrottledNamed() at /home/yukky/workspace/awf-autoware/src/universe/autoware/system/system_error_monitor/src/system_error_monitor_core.cpp:54)

rosbag play simulation errors (after play rosbag)

Expected behavior
There are No error.
Actual behavior
There are several error outputs in the tutorial.
Steps to reproduce
Planning simulation
ros2 launch autoware_launch planning_simulator.launch.xml map_path:=$HOME/Downloads/sample-map-planning vehicle_model:=sample_vehicle sensor_model:=sample_sensor_kit
https://autowarefoundation.github.io/autoware-documentation/main/tutorials/ad-hoc-simulation/planning-simulation/
Rosbag replay simulation
ros2 launch autoware_launch logging_simulator.launch.xml map_path:=$HOME/Downloads/sample-map-rosbag vehicle_model:=sample_vehicle sensor_model:=sample_sensor_kit
https://autowarefoundation.github.io/autoware-documentation/main/tutorials/ad-hoc-simulation/rosbag-replay-simulation/
Versions
No response
Possible causes
No response
Additional context
No response

@ito-san Can you investigate the error regarding hdd_monitor?

Are https://github.com/autowarefoundation/autoware.universe/issues/641 and https://github.com/autowarefoundation/autoware.universe/issues/643 are part of this issue?

Since the tasks need to be re-done, we'll close the issue and if errors occur again, this issue needs to be re-created.

cc @ito-san
Now, I want to reopen this issue because we get some errors when plannign simulator is launched.

@yukkysaito Please consider to close this issue if you don't have any problem with it after the fixes.

@yukkysaito is this issue still present?

I'm closing the issue since there are no updates for long time.
We can reopen this issue or create new one if we find issue with the tutorial.


Avoid warning on PHP 7

There is no need for the reference parameter here (its arguments are objects like “new QRNumber”), and expressions can’t be passed as references.

thanks a lot!


MultiProcessing removal

The next version, and thus the master branch will ship without the multiprocessing code.
It's huge and we just don't want to keep maintaining it.

@giulioungaretti any idea how long it will take until a replacement multiprocessing? Our group is quite reliant on the multiprocessing architecture, and we will probably have to separate branches until then


create review model

Issue was completed in the 23-create-review-controller and merged into that branch. Thus, merging using the branch assigned to this issue would be redundant.


Listing Multiple Departures in Dashboard

Hi Jason
Nice Integration!
Quick question.  If "Number of Departures to Track" is set to 3 then is it possible to show the next 3 departures on a card?  If so which one would you use?
SteveW

Sorted - I found the answer in the Wiki :)

There is an example here that might help: https://github.com/make-all/metlink-nz/wiki
I use an entities card to list all the departures.  The example uses two lines per departure, one with the name of the service and one with the time it is due, but it is also possible to combine these into a single line using the template-entity-row custom component.


fix(instrumentation-runtime-node): mixin default config in constructor

Which problem is this PR solving?
The constructor of the instrumentation runtime node does not mixin the default config. This means that if I pass an empty object, the eventLoopUtilizationMeasurementInterval will be undefined, which has the side-effect of the configured interval to be near 0.
Short description of the changes
Similarly to all other instrumentations, I've mixed in the default config to make sure default objects are always defined. Please note that the instrumentation file itself didn't have tests, so I haven't added any.

cc @d4nyll @pichlermarc @trentm


Improve IMSC time syntax output

Closes #102

@palemieux I've merged master to the branch and resolved conflicts.

@palemieux I fixed the issue with the "s".
I also sync up with master which changes the interfaces to imsc.from_model as it takes both a frame rate and progress bar. Please double check that implementation to make sure it is OK with you.
CI passes

I also sync up with master which changes the interfaces to imsc.from_model as it takes both a frame rate and progress bar. Please double check that implementation to make sure it is OK with you.

@valnoel @BobBrownConsulting I am thinking we need come up with a more general way of passing parameters to the readers and writers, e.g. through a config file and command line params. Thoughts?


Update package.json file with latest dependencies version

Update package.json file with latest dependencies version:

@sap/grunt-sapui5-bestpractice-build to version 1.3.64
@sap/approuter to version  5.7.0
Remove unneeded dev dependencies

 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.

Thanks!


<IP_ADDRESS> - custom field values are being assigned to all categories

Description
if we create a custom field, assign the field to categories. then, we set a value for the field in a category. we will see that the value will be set to all categories.
Possible Solution
it should not set the value for the field for all categories
Additional context
the issue is related to <IP_ADDRESS> version specifically.

I can confirm, but this seems to be a JS/Vue issue - the data does not actually get stored for all categories, but the form fields don't reset when switching to another category.
If the category that I'm switching too already has filled out values, it shows the correct values. If I switch to a new category with empty fields and do a hard refresh, I can also fill the fields normally, just switching without reloading breaks it (shows wrong values).

yes, most likely it is a JS/Vue issue. I was wrong about the origins of it in my first post here. thanx for pointing that out.

I can confirm, but this seems to be a JS/Vue issue

should be fixed with https://github.com/shopware/platform/pull/1566

should be fixed with https://github.com/shopware/platform/pull/1566


tiny problems

Please generate a gradlew file by running gradle wrapper
Please include sample projects.

A really cool and useful project.

Hi

I thought a less of code is better. However, many of other project have actually a gradle wrapper and it seems that having it will make it easier to use in travis CI. Okay - I will add it
Will also add a sample project

Wait for update today - tomorrow
Have a nice day

Hi,
This will be fixed the nearest time (today - tomorrow)
On 12 Feb 2015 04:45, "代码家"<EMAIL_ADDRESS>wrote:

Please generate a gradlew file by running gradle wrapper
Please include sample projects.

A really cool and useful project.
—
Reply to this email directly or view it on GitHub
https://github.com/shell-software/fab/issues/1.

@shell-software  Awesome.

This is done.
Sorry for the delay.


Discuss on class diagram design pattern implementation

Designed a draft for bridge design pattern.
Reason for adding bridge design pattern to the report export features is due to the fact that Bridge design pattern decouples the implementation from the abstract class. In this case the abstract class is the report which contains report generation methods. The implementation is the export report feature which allows the export of different file types such as XLS, DOCS and PDF

Converted the classes that are related to report generation using the abstract factory pattern where. The types of report that can be created are .pdf, .docx and .xls type reports

to add on what @shahzad1999  has posted for the image , .pdf .docs. xls will communicate with each respective file format factories.

proposed facade design pattern, where i combine some methods together in one facade class, not sure if will go ahead with the coding


[IBM] Fix formatting in HTTPS doc

This PR fixes the kubeflow-gateway YAML in the IBM authentication doc and removes any trailing whitespace from lines.
Closes: #2393

thanks!
/lgtm

/assign @animeshsingh @Tomcli
PTAL at this PR that introduces formatting changes and /approve
Related to https://github.com/kubeflow/website/pull/2409#pullrequestreview-551906091 (splitting a PR into smaller bits)

/lgtm
/approve


Refactor /tasks api to match with contracts

Updating the /tasks api to match contracts
ToDo

[ ] Update tests

Closing it for being a duplicate of https://github.com/Real-Dev-Squad/website-backend/pull/307


404 redirects to file name if path changes

As we build out our /learn pages we will probably continue to change folder structure as we have more documentation. This breaks existing URLs out in internet-land.
If there is a 404, we could check the file name for a match and then direct to that page. We can assume that the file names won't change.
However if the file name changes, then assume the default behavior with a redirect to the home page.

@mattgreg I make I quick research about possibilities and I think the more optimum way to solve this will be to use this plugin:
https://www.gatsbyjs.com/plugins/gatsby-redirect-from/#usage
So all you would have to do is to add redirect_from in your markdown files, where you will put an older path for this URL. Based on that we should get a nice server-side redirect.
It's not SEO perfect ( since it's not HTTP 301 redirect ) but to achieve that, we would have probably make some entries in azure/cloudfront ( don't remember where we handle old URL redirects ). Keep it on the azure/cloudfront isn't also perfect since it won't be so clean and easy to manage.

That looks like a great solution, even if a little mechanical.

@mattgreg I've added the plugin I've mentioned above. To handle normal pages ( not generated from MD files ) I have added different solutions. You can do this by adding an entry in the redirect.csv file. Check details in README in the pull request.

@miedziak
can you add these instructions to the readme?
is redirect_from part of the header in the markdown pages - if so what's the syntax on that line. Can you create an example in the readme? We have a section in the readme for markdown files metadata fields
For the redirect.csv where should this file live? in the repo root?


Remove most commented-out code lines across the codebase

Don't merge yet; I'll redo this pull request after the vulkan branch is rebased into master.
Some multi-line commented code blocks may remain here and there as they're more difficult to find in an automated manner.
This should be reviewed thoroughly, as I may have accidentally removed a handful of blocks that contained useful information somehow.

@Calinou Will you have time to rebase this? This blocks #38403. vulkan has been merged into master for awhile now. If not, I am willing to work on this (assuming it's approved by @akien-mga).

@Calinou Will you have time to rebase this? This blocks #38403. vulkan has been merged into master for awhile now. If not, I am willing to work on this (assuming it's approved by @akien-mga).

This should likely be redone from scratch, it's likely impossible to rebase.

This should likely be redone from scratch, it's likely impossible to rebase.


update RBAC file

The RBAC file was copied unmodified from driver-registrar. Is it still correct?
The introduction still refers to "external provisioner" (was already broken when creating that file initially for driver-registrar).

Looking through the code I think it only needs CSIDriver create and delete permissions
cc @gnufied who is working on testing this in our e2es

Michelle Au<EMAIL_ADDRESS>writes:

cc @gnufied who is working on testing this in our e2es

@gnufied: please make sure that the RBAC file in Kubernetes E2E stays in
sync with the one here.
I have a PR pending
(https://github.com/kubernetes/kubernetes/pull/71703) which pulls yaml
files via URLs, but that can only work when the right file content is
really available elsewhere.

/assign

The current RBAC definitions don't include permissions to create the CSIDriver CRD which are created on startup:
https://github.com/kubernetes-csi/cluster-driver-registrar/blob/e7051356d4202aecd8b1b37c95cbc0eec133854d/cmd/cluster-driver-registrar/k8s_register.go#L54
At the moment, the cluster-driver-registrar exits with os.Exit(1) if it doesn't have the permission to create the CRD. I'm not quite sure if it would make sense to ignore that error and only exit if it is unable to register the CSIDriver object.
From the current CSI documentation, it is unclear to me who the responsible party for creating the CRDs is. The HostPath example mentions that the CRDs need to be created manually by a kubernetes administrator.

The v1.0.1 release was tagged without fixing the RBAC rules contained in that release. We should prepare a release v1.1.0 which no longer installs the CRD and has correct RBAC rules.
@msau42 The master branch has that fixed and looks like it could be turned into v1.1.0.

There are other big changes coming to cluster driver registrar soon, namely that the crd is going to be replaced by an intree beta object. Do you think it's worth it to have a 1.1, or move straight to a 2.0? The crds were technically alpha so should we consider breaking them to warrant a major version bump?

Michelle Au<EMAIL_ADDRESS>writes:

There are other big changes coming to cluster driver registrar soon,
namely that the crd is going to be replaced by an intree beta
object. Do you think it's worth it to have a 1.1, or move straight to
a 2.0? The crds were technically alpha so should we consider breaking
them to warrant a major version bump?

Right now there is no "good" release that works on Kubernetes
1.13. V1.0.1 works but has the broken RBAC file. I guess we could fix
this with an updated release note where we quote the correct RBAC rules.
A v2.0 won't work on Kubernetes 1.13.
I would prefer to tag v1.1.0 on master before replacing the CRDs. If you
set a v1.1.0-rc1, then I can do some testing with it.

How about cherry picking https://github.com/kubernetes-csi/cluster-driver-registrar/pull/19 to release 1.0 and cut a new 1.0.x?

Michelle Au<EMAIL_ADDRESS>writes:

How about cherry picking https://github.com/kubernetes-csi/cluster-driver-registrar/pull/19 to release 1.0 and cut a new 1.0.x?

As there's no code change in that case, I think it's enough to add a
note to the 1.0.1 release note with the errata - basically just a link
to the PR and the resulting revision of the RBAC file.
Remember that the release process in 1.0.x is still manual, so it would
be more work to do a new release there compared to master.

"Patrick Ohly"<EMAIL_ADDRESS>writes:

Michelle Au<EMAIL_ADDRESS>writes:

How about cherry picking https://github.com/kubernetes-csi/cluster-driver-registrar/pull/19 to release 1.0 and cut a new 1.0.x?

As there's no code change in that case, I think it's enough to add a
note to the 1.0.1 release note with the errata - basically just a link
to the PR and the resulting revision of the RBAC file.

Actually, the RBAC rules from PR #19 do not work for v1.0.1, because
that release still needs permission to create CRDs.
I ended up backporting two more commits from master. See
https://github.com/kubernetes-csi/cluster-driver-registrar/pull/36


Update data_preprocessing.py

Mean/std normalization of input image before feeding to ANN as an additional preprocessing step proposal...

Same as https://github.com/clemkoa/tooth-detection/pull/17


Improve error popup for User Google Drive storage out of space

Error Code
Error: Forbidden

Error: Forbidden
    at Request.<anonymous> (https://homebrewery.naturalcrit.com/homebrew/bundle.js:75356:19)
    at Emitter.emit (https://homebrewery.naturalcrit.com/homebrew/bundle.js:1966:20)
    at xhr.onreadystatechange (https://homebrewery.naturalcrit.com/homebrew/bundle.js:75665:10)
{
  "status": 403,
  "method": "PUT",
  "url": "/api/update/g1LAcYR05zRg"
}

### Your description of what happened:

"Oops" error while Saving or Auto-saving on Google Chrome, Mozilla Firefox, and even Microsoft Edge.

The code "

auto-saved.
"
seems to be malfunctioning.

According to our logs, it was trying to save to your Google Drive, but your drive is full.

This is an opportunity to improve the error messaging for this particular google response. Example response from Google here (personal info removed):

 GaxiosError: The user's Drive storage quota has been exceeded.
     at Gaxios._request (/app/node_modules/gaxios/build/src/gaxios.js:130:23)
     at runMicrotasks (<anonymous>)
     at processTicksAndRejections (node:internal/process/task_queues:96:5)
     at async JWT.requestAsync (/app/node_modules/google-auth-library/build/src/auth/oauth2client.js:368:18)
     at async Object.updateGoogleBrew (/app/server/googleActions.js:137:3)
     at async updateBrew (/app/server/homebrew.api.js:215:19) {
   response: {
     config: {
       url: 'https://www.googleapis.com/upload/drive/v3/files/xxxxxxxxxxxxxxxxxxxx?uploadType=multipart',
       method: 'PATCH',
       userAgentDirectives: [Array],
       paramsSerializer: [Function (anonymous)],
       data: [PassThrough],
       headers: [Object],
       params: [Object],
       validateStatus: [Function (anonymous)],
       retry: true,
       body: [PassThrough],
       responseType: 'json',
       retryConfig: [Object]
     },
     data: { error: [Object] },
     headers: {
       'alt-svc': 'h3=":443"; ma=2592000,h3-29=":443"; ma=2592000,h3-Q050=":443"; ma=2592000,h3-Q046=":443"; ma=2592000,h3-Q043=":443"; ma=2592000,quic=":443"; ma=2592000; v="46,43"',
       connection: 'close',
       'content-length': '257',
       'content-type': 'application/json; charset=UTF-8',
       date: 'Thu, 17 Nov 2022 22:50:52 GMT',
       server: 'UploadServer',
       vary: 'Origin, X-Origin',
       'x-guploader-uploadid': 'xxxxxxxxxxxxxxx
     },
     status: 403,
     statusText: 'Forbidden',
     request: {
       responseURL: 'https://www.googleapis.com/upload/drive/v3/files/xxxxxxxxxxxxxxxxxxxxxxx?uploadType=multipart'
     }
   },
   config: {
     url: 'https://www.googleapis.com/upload/drive/v3/files/xxxxxxxxxxxxxxxxxxxxx?uploadType=multipart',
     method: 'PATCH',
     userAgentDirectives: [ [Object] ],
     paramsSerializer: [Function (anonymous)],
     data: PassThrough {
       _readableState: [ReadableState],
       _events: [Object: null prototype],
       _eventsCount: 2,
       _maxListeners: undefined,
       _writableState: [WritableState],
       allowHalfOpen: true,
       _flush: [Function: flush],
       [Symbol(kCapture)]: false,
       [Symbol(kCallback)]: null
     },
     headers: {
       'x-goog-api-client': 'gdcl/6.0.0 gl-node/16.11.1 auth/8.0.2',
       'content-type': 'multipart/related; boundary=ebb21e98-e8f2-422c-b489-2ef3ccb12a91',
       'Accept-Encoding': 'gzip',
       'User-Agent': 'google-api-nodejs-client/6.0.0 (gzip)',
       Authorization: 'Bearerxxxxxxxxxxxxxxxxxxxxxxxxx',
       Accept: 'application/json'
     },
     params: { uploadType: 'multipart' },
     validateStatus: [Function (anonymous)],
     retry: true,
     body: PassThrough {
       _readableState: [ReadableState],
       _events: [Object: null prototype],
       _eventsCount: 2,
       _maxListeners: undefined,
       _writableState: [WritableState],
       allowHalfOpen: true,
       _flush: [Function: flush],
       [Symbol(kCapture)]: false,
       [Symbol(kCallback)]: null
     },
     responseType: 'json',
     retryConfig: {
       currentRetryAttempt: 0,
       retry: 3,
       httpMethodsToRetry: [Array],
       noResponseRetries: 2,
       statusCodesToRetry: [Array]
     }
   },
   code: 403,
   errors: [
     {
       domain: 'global',
       reason: 'storageQuotaExceeded',
       message: "The user's Drive storage quota has been exceeded."
     }
   ]
 }

Yes, apologies for the misconception.  I was very confused.  I see now that
my phone backup caused an issue.
On Thu, Nov 17, 2022, 10:06 PM Trevor Buckner @.***>
wrote:

According to our logs, it was trying to save to your Google Drive, but
your drive is full.
—
Reply to this email directly, view it on GitHub
https://github.com/naturalcrit/homebrewery/issues/2510#issuecomment-1319486115,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AM4BUXCJGSBRU5ATF7ARB3LWI3XB3ANCNFSM6AAAAAASC6UNBU
.
You are receiving this because you authored the thread.Message ID:
@.***>

I believe this was solved by #3255


perf(ui/breadcrumb): optimize docs

Checklist

[ ] Fix linting errors
[ ] Tests have been added / updated (or snapshots)

Change information

Codecov Report

Merging #783 (b760982) into feat/breadcrumb (502ff62) will decrease coverage by 0.06%.
The diff coverage is 16.66%.

@@                 Coverage Diff                 @@
##           feat/breadcrumb     #783      +/-   ##
===================================================
- Coverage            94.97%   94.91%   -0.07%
===================================================
  Files                  245      245
  Lines                 5577     5582       +5
  Branches              1042     1043       +1
===================================================
+ Hits                  5297     5298       +1
- Misses                 280      284       +4

Impacted Files
Coverage Δ

packages/varlet-ui/src/breadcrumb/props.ts
100.00% <ø> (ø)

packages/varlet-ui/src/breadcrumb/Breadcrumb.vue
38.88% <16.66%> (-7.27%)
:arrow_down:

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


[ISPN-7535] Cache creation requires specific permissions when using s…

…ecurity manager
Move Util.getClassLoaders() and ReflectionUtil.invokeAccessibly() to doPrivileged() when security manager is enabled
Jira: https://issues.jboss.org/browse/ISPN-7535
PR to master branch: https://github.com/infinispan/infinispan/pull/4920

Requires the same changes as https://github.com/infinispan/infinispan/pull/4920

Closing. Please address all comments in https://github.com/infinispan/infinispan/pull/4920 and then reopen.

@slaskawi would you please reopen this PR? The https://github.com/infinispan/infinispan/pull/4920 has been merged, and this has the same fix with #4920, but to 8.2.x branch


Seen some memory leaks?

PyKafka version:  e9ac86a
Kafka version: 0.9.0
The code is something like:
import json
import threading
from concurrent.futures import ThreadPoolExecutor

from pykafka import KafkaClient

executor = ThreadPoolExecutor(10)
semaphore = threading.BoundedSemaphore(20)

kafka = KafkaClient(...)
topic = kafka.topics['some-topic']
consumer = topic.get_balanced_consumer(
    consumer_group='test',
    managed=True,
    fetch_message_max_bytes=10 * 1024 * 1024
)
for message in consumer:
    obj = json.loads(message.value)
    semaphore.acquire()
    executor.submit(process, obj)
    consumer.commit_offsets()

def process(obj):
    try:
        # do something
    except Exception:
        # do something
    finally:
        semaphore.release()

And I have seen a lot of memory(from ~12G to ~30G) consumed after running for a while.
So I try to debug with objgraph:
objgraph.show_most_common_types(limit=20)
Message                    47050
list                       18440
function                   12654
dict                       5630
tuple                      5319
builtin_function_or_method 3146
weakref                    2962
type                       1754
getset_descriptor          1700
wrapper_descriptor         1518
cell                       1503
method_descriptor          1096
property                   930
module                     748
set                        641
member_descriptor          513
frame                      459
MovedAttribute             332
_Condition                 323
classobj                   316

objgraph.show_growth(limit=20)
Message    47052      +215

The Message should be pykafka.protocol.Message instance.
Any idea how to debug this?

@messense You might want to adjust the queued_max_messages parameter to ensure you're not storing too many Message instances. If you lower it and still see huge memory usage, it's certainly possible there's a leak.

@emmett9001 Tried that. Lower it to 500 works better, but still see huge memory usage occasionally (way less often than default 2000 though).


Support for ORDER BY CASE WHEN?

I have a use case that does not appear to be supported.  Please observe this example:
var query = SQLiteConnection.Table<MyClass>()
              .OrderBy(x => x.SomeProperty == "Preferred Value" && x.AnotherProperty == "2nd Preferred")
              .ThenBy(x => x.SomeProperty)
              .ThenBy(x => x.AnotherProperty);

If I were to write this in plain SQL for a tool such as DBeaver it would look like this:
ORDER BY
CASE WHEN
SomeProperty = 'Preferred Value' AND AnotherProperty = '2nd Preferred'
THEN 0 ELSE 1
END ASC,
SomeProperty ASC,
AnotherProperty ASC

When I try to run the C# code I get a System.NotSupportedException: 'Order By does not support: [I believe this is the Expression.ToString() from the first OrderBy function in the C# code].  If there is a different way to achieve this sort or have I missed something?  Please let me know.  Otherwise, could we consider supporting something like this?
Thanks.

Adding more info.  One of my customers wants a list of work sites ordered by a secondary id then a primary id.  However, they want an exception to this sort where the users default work site sits on top.


Update app.js

Fixed error sending form data.
Previously, not the correct data was sent, if a radio button with the same name is used, several values are sent

This basically just reverts https://github.com/the-control-group/voyager/pull/3004.
I think we need to look for another solution for the problem you describe.
Also, we need to merge that into 1.x

Agreed with @emptynick.  I'm not opposed to reverting that fix so long as another fix is provided.  Or someone finds another solution to this problem.
Either way, I'm going to close this.  Feel free to reopen or open a new PR if a solution is discovered.


Can't grant access to all files

Describe the bug
Can't grant access to Internal memory root (or grant MANAGE_EXTERNAL_STORAGE permission) to app
So when I delete images I must grant access to folders containing them
To Reproduce
Steps to reproduce the behavior:

Select any image that is NOT in the Internal memory root (or any other Android-protected folder)
Press 'Delete' button
Dialog show up to grant access to 'xxx' folder of 'parent of xxx' folder, press OK
See error like on the screenshot

Expected behavior
I can grant access to Internal memory root (or grant MANAGE_EXTERNAL_STORAGE permission) and delete any images without requiring to grant access to all folders containing them
Screenshots

System information and logs:
Aves version: 1.5.7-play (Build 61)
Flutter version: 2.5.3 (Channel stable)
Android version: 11 (SDK 30)
Android build: RMX2155_11_C.13
Device: realme RMX2155
Google Play services: ready
Additional context
When I try to delete image that in Internal memory root (or any other Android-protected folder), appears Android dialog to delete it, and it deletes normally

This is not a bug. The dialog that appears in your screenshot is provided by the OS. Apps can request write access to directories but the OS restricts on which directory this access can be granted. Accessing a volume root directory is no longer possible from Android 11.
Aves is not considered a file manager, as it only deals with media files, so it would not get the MANAGE_EXTERNAL_STORAGE permission according to the Google Play policy. More on this topic at #106.
In other words, the "expected behavior" you describe is, as far as I know, not possible on Android 11. But in practice, media files should be within a few standard directories, like /DCIM/, /Pictures/, /Movies/. So you can grant access a couple of times and be done.
To modify a file in the root directory, as you describe in "additional context", the app switches to a file-level access when it cannot work with directory-level access. But as you can see, this requires user interaction for each operation. More on this topic at #64.


chore: refactoring public executor interface

Please read contributing guidelines and remove this line.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#9701  👈
#9565
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @dbanks12 and the rest of your teammates on  Graphite


Avoid calling count of undefined

Should address this:
error  in ./assets/img/press/logo-white.png
Module build failed: TypeError: Cannot read property 'count' of undefined
at _splitBoxes (/Users/.../node_modules/node-vibrant/src/quantizer/mmcq.ts:20:18)
at MMCQ (/Users/.../node_modules/node-vibrant/src/quantizer/mmcq.ts:58:5)
at Vibrant._process (/Users/.../node_modules/node-vibrant/src/vibrant.ts:55:33)
at /Users/.../node_modules/node-vibrant/src/vibrant.ts:71:35
at tryCatcher (/Users/.../node_modules/bluebird/js/release/util.js:16:23)
...

Fixes #48

V2 branch was deprecated. The issue was fixed in v3.0. Thanks anyway.


Incorrect resolution displayed when using DPI scaling

Microsoft PowerToys version
0.37.2
Running as admin

[ ] Yes

Area(s) with issue?
FancyZones, FancyZones Editor
Steps to reproduce
Open the Fancy Zones editor using a system using 125 (for example) display scaling with a monitor at 2560x1440 resolution (for example)
Note the displayed monitor resolutions.
Resize a zone. Note the displayed monitor resolutions.
✔ Expected Behavior
The displayed resolution values do not match the monitor resolutions. For example, at monitor running at 2560x1440 at 125% scaling, the Fancy Zones editor displays zones at 2048x1152.
From a user experience perspective, if I want to make a zone to be exactly 900x600 (relative to monitor resolution), the FZ editor shows the incorrect values to achieve this.
❌ Actual Behavior
The displayed coordinates appear to be scaled (i.e. multiplied by) the scaling factor instead of showing monitor coordinates.

Other Software
No response

Verified, believe this is mostly a UX issue, correct @SeraphimaZ

Verified, believe this is mostly a UX issue, correct @SeraphimaZ

Yes, that's right. This behavior is by design.

I guess from one developer to another: Why is that the design? It is an internal representation of pixels that no user of the software can see.  To any ordinary user of windows using (for example) 1920x1080 at a scale of 125%, windows does not display to the user anywhere in the system that their resolution is 1536x864
I propose a "design" change that shows to the user in the pixel values that are relevant to them.

@SeraphimaZ for 100% dpi, do we display taskbar in the calculation?  I did a quick test at 300 DPI so a 3840x2160 monitor showed 1280x672.  The vertical calc for a window at the bottom stopped right at the taskbar.

@madallday reason why the scaling is shown is we show zone size.  "if you show 3000px, why can't i create a zone 3000px wide?"

Respectfully, I disagree. In reality my monitor is at that resolution, and
all other windows apps use the coordinates 0-3000 (in your example).
If I have a program programmatically position a window that is 1000px wide
at 2000px it would be at coordinates 2000 through 3000.
Every other application is able to scale just fine, and since this
application is specifically used for positioning windows in the windows
environment, I especially expect this from the application.
On Thu, Jan 6, 2022 at 7:01 PM Clint Rutkas @.***>
wrote:

@madallday https://github.com/madallday reason why the scaling is shown
is we show zone size. "if you show 3000px, why can't i create a zone 3000px
wide?"
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/11751#issuecomment-1007056243,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIWR6HDJ5AQRGHXVLVEM7LUUY3PXANCNFSM46VGNWHA
.
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.
You are receiving this because you were mentioned.Message ID:
@.***>

@SeraphimaZ for 100% dpi, do we display taskbar in the calculation? I did a quick test at 300 DPI so a 3840x2160 monitor showed 1280x672. The vertical calc for a window at the bottom stopped right at the taskbar.

@crutkas , 3 x 672 = 2016, not 2160, do you know why this is happening? I'm curious as to why it doesn't show 1280 x 720?

I agree with @madallday on this one, I think the distinction is that px as location markers on a screen does not change irrespective of scaling, while the number of px used to render an image on a screen is proportional to the scale.
Adopting @madallday's expectation would align with how I understand scaling is actually working in windows, otherwise images would appear increasingly stretched as scale increases above 100%. However, as it appears, the number of px is upscaled (the same px aren't stretched over a greater surface area).

Verified, believe this is mostly a UX issue, correct @SeraphimaZ

Yes, that's right. This behavior is by design.

the current design choice seems to align with the scenario of the px being stretched. Thoughts @SeraphimaZ ?

@geoffkopaczewskyj Willing to bet the remainder is the taskbar.  The TLDR is we are factoring in DPI and this is acting as designed. I don't think we'll change the behavior other than maybe adding in "Scale factor" to make it more clear as even if you think you have 3840 horizontal pixels, you don't due to your scale.  We are showing effective, scaled pixels, not actual pixels that FancyZones deems usable.
Let's track work here to help add in that extra clarity to editor.

I don’t think you understand the issue of you believe this is working as designed.

From: Clint Rutkas @.>
Sent: Monday, January 17, 2022 10:33:57 PM
To: microsoft/PowerToys @.>
Cc: Adam Dylla @.>; Mention @.>
Subject: Re: [microsoft/PowerToys] Incorrect resolution displayed when using DPI scaling (#11751)
Closed #11751https://github.com/microsoft/PowerToys/issues/11751.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/11751#event-5907908331, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAIWR6CZF6BN7NMIAXZIOQLUWTUTLANCNFSM46VGNWHA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>

See what you mean re the taskbar occupying available px.
I think I'm still with @madallday.
@crutkas, if I asked you to make a cube shaped house twice as big within the same plot of land, how would you define the new locations of the vertices of the house's footprint, given one vertex of the plot of the land is the origin (0,0), after scaling according to the current behaviour design?


Welder Fuel Tanks don't work properly.

This was originally reported by hubismal#5912 on Discord. I have reproduced it on local.
How to reproduce:

Have a welder.
Attempt to refuel the welder from a tank. Nothing will happen, and you won't get the "You refuel the welding tool" message.
The tank will still explode if touched with a lit welder.
Interestingly, empty welder fuel tanks will still give you the "You refuel the welding tool" message.

On a side note, there's now two entries for "Fuel Tank [Empty]" in the spawn menu.

issue with new solution re factor, most likely


How do I generate a different prompt for each batch?

I'm using this for the wildcards feature, I used to use the wildcards script but it wasnt working so i found this.
It is properly generating prompts from my wildcards file. The problem is it only generates the prompt once, so if I do a batch of 10 images, all 10 will have the same prompt.
What I want is for every image to have a unique prompt. I think this is how wildcards used to work.
Is there a way to get dynamic prompts to do this?

you can use "prompts from file or textbox" script

Dynamic Prompts already generates a new prompt for every image. Could you share your prompt so that I can better understand your question.

@skeddles Um... Just to be sure, which extension are you running in particular and which version..? I don't think this extension adds "Wildcard prompt:" to the PNG info (and a git log -S doesn't show that in history either).

Ah I think you were right, I went in and deleted the wildcards script from the scripts folder and extensions folder. It seemed like both had failed to load, but they must have been working in the background.

Glad you got it sorted out!


Add release notes: v0.19.0-0.4.0

Add changelog entry / release notes for the upcoming release.
My first attempt at doing so, if you are happy with this @apoelstra we can probably release after this merges?

Ok to merge but the merge script is giving me a weird error
Traceback (most recent call last):
  File "/store/home/apoelstra/code/bitcoin/bitcoin-maintainer-tools/github-merge.py", line 484, in <module>
    main()
  File "/store/home/apoelstra/code/bitcoin/bitcoin-maintainer-tools/github-merge.py", line 393, in main
    first_sha512 = tree_sha512sum()
  File "/store/home/apoelstra/code/bitcoin/bitcoin-maintainer-tools/github-merge.py", line 166, in tree_sha512sum
    assert(metadata[1] == b'blob')
AssertionError

which I guess refers to some submodule weirdness when the script attempts to create a cryptographic hash of the git tree.

Yeah you need to add
        if metadata[1] == b'commit':
            continue

Right above the line throwing the error assert(metadata[1] == b'blob')


Automatic chunking of arrays ?

Hi there,
Hopefully this turns out to be a basic issue, but I was wondering why the chunks='auto' that dask seems to provide (https://docs.dask.org/en/latest/array-chunks.html#automatic-chunking) isn't an option for xarray? I'm not 100% sure of how dask decides how to automatically chunk its arrays, so maybe there's a compatibility issue?
I get the impression that the dask method automatically tries to prevent the issues of "too many chunks" or "too few chunks" which can sometimes happen when choosing chunk sizes automatically. If so, it would maybe be a useful thing to include in future versions?
Happy to be corrected if I've misunderstood something here though, still getting my head around how the dask/xarray compatibility really works...
Cheers!

so da.chunk({dim_name: "auto"}) works but da.chunk("auto") does not. The latter is a relatively easy fix. We just need to update the condition here:
https://github.com/pydata/xarray/blob/bd84186acbd84bd386134a5b60111596cee2d8ec/xarray/core/dataset.py#L1736-L1737

Oh ok I didn't know about this, I'll take a look and read the contribution docs tomorrow ! It'll be my first PR so may need a bit of hand-holding when it comes to tests. Willing to try though!

Awesome! Please see https://xarray.pydata.org/en/stable/contributing.html for docs on contributing

Agreed, this would be very welcome!
chunks='auto' isn't supported only because xarray support for dask predates it :)

Cheers! Just had a look, is it as simple as just changing this line to the following, @dcherian ?
if isinstance(chunks, Number) or chunks=='auto':
            chunks = dict.fromkeys(self.dims, chunks)

This seems to work fine in a lot of cases, except automatic chunking isn't implemented for object dtypes at the moment, so it fails if you pass a cftime coordinate, for example.
One option is to automatically use self=xr.decode_cf(self) if the input dataset is cftime? Or could just throw an error.

Also, the contributing docs have been super clear so far! Thanks! :)

is_scalar(chunks) might be the appropriate condition. is_scalar is already imported from .utils in dataset.py

This seems to work fine in a lot of cases, except automatic chunking isn't implemented for object dtypes at the moment, so it fails if you pass a cftime coordinate, for example.

Can we catch this error and re-raise specifying "automatic chunking fails for object arrays. These include cftime DataArrays" or something like that?

Nice, that's neater! Would this work, in the maybe_chunk() call? Sorry about the basic questions!
 def maybe_chunk(name, var, chunks):
     chunks = selkeys(chunks, var.dims)
     if not chunks:
            chunks = None
     if var.ndim > 0:
            # when rechunking by different amounts, make sure dask names change
            # by provinding chunks as an input to tokenize.
            # subtle bugs result otherwise. see GH3350
            token2 = tokenize(name, token if token else var._data, chunks)
            name2 = f"{name_prefix}{name}-{token2}"
            try:
                return var.chunk(chunks, name=name2, lock=lock)
            except NotImplementedError as err:
                raise Exception("Automatic chunking fails for object arrays. These include cftime DataArrays.")
    else:
        return var

The error message from dask is already pretty descriptive:
NotImplementedError: Can not use auto rechunking with object dtype. We are unable to estimate the size in bytes of object data
I don't think we have much to add on top of that?

I also thought that, after the dask error message it's pretty easy to then look at the dataset and check what the problem dimension is.
In general though, is that the type of layout you'd suggest for catching and re-raising errors? Using raise Exception() ?

If we think can improve an error message by adding additional context, the right solution is to use raise Exception(...) from original_error:
https://stackoverflow.com/a/16414892/809705
On the other hand, if xarray doesn't have anything more to add on top of the original error message, it is best not to add any wrapper at all. Users will just see the original error from dask.


Feature/glossis currents

Add  support for flowmaps that  have a 'b'  (blue) mask  channel.

I was planning  to submit this to my own fork. But it is not yet finished for merge in yoru  branch yet.
I am working  on extending your code  with the option to have a blue "mask" channel. That allows the code to be  used for ocean currents. I added an example in the  docs.
Please feel free to comment on how I can make  it suitable for a merge  into your version.


Foreign Keys not working as it should.

So I'm making an workout tracking app. For the database i'm using Moor latest version and following ResoCoders tutorials.
I have 2 tables I need to connect - Routines and their exercises.
Following is the code for defining the tables -
class Routines extends Table {
  IntColumn get id => integer().autoIncrement()();
  TextColumn get routineTitle =>
      text().customConstraint('REFERENCES exercises(routineTitle)')();
}

class Exercises extends Table {
  TextColumn get routineTitle => text()();
  TextColumn get exerciseName => text().nullable()();
}

class Workout extends Table {
  final Routine routine;
  final Exercise exercise;

  Workout({
    @required this.routine,
    @required this.exercise,
  });
}

@UseMoor(tables: [Routines, Exercises], daos: [RoutineDao, ExerciseDao])
class AppDatabase extends _$AppDatabase {
  AppDatabase()
      : super((FlutterQueryExecutor.inDatabaseFolder(
          path: 'db.sqlite',
          logStatements: true,
        )));
  @override
  int get schemaVersion => 2;

  @override
  MigrationStrategy get migration => MigrationStrategy(
        beforeOpen: (db, details) async {
          await db.customStatement('PRAGMA foreign_keys = ON');
        },
      );
}

This is my DAO -
@UseDao(tables: [Routines, Exercises])
class RoutineDao extends DatabaseAccessor<AppDatabase> with _$RoutineDaoMixin {
  final AppDatabase db;
  RoutineDao(this.db) : super(db);

  Stream<List<Workout>> get watchAllRoutines => select(routines)
      .join([
        leftOuterJoin(
          exercises,
          exercises.routineTitle.equalsExp(routines.routineTitle),
        )
      ])
      .watch()
      .map((rows) => rows.map((row) => Workout(
          routine: row.readTable(routines),
          exercise: row.readTable(exercises))));
  Future<List<Routine>> get getAllRoutines => select(routines).get();
  Future addRoutine(Insertable<Routine> routine) =>
      into(routines).insert(routine);
  Future deleteRoutine(Insertable<Routine> routine) =>
      delete(routines).delete(routine);
}

When I add in any routine or exercise, the Workout list is always null even if the routine and exercise have data in it. Nothing is showing on the screen and i get many debug errors. (no ui errors)

Thanks for the detailed description. There are two issues that might cause a problem:

the routines table contains a reference to the exercises table, but the routines table is created first (so before the exercises table exists). Can you try to switch the order in the @UseMoor annotation? (e.g tables: [Exercises, Routines])
you use routineTitle in the reference clause, but the generated columns will use snake_case. So REFERENCES exercises(routine_title) should work

and i get many debug errors

Would be good to have the error message on some of them :) If it complains about some table not existing or a violated constraint, it's probably because one of the problems above.

One minor problem: Workout shouldn't extend from table, because the workouts aren't stored as a table in sqlite (this can confuse the generator, but it's not a problem in this case).
I just tried your code locally, and I've found the problem: The "foreign key mismatch" error happens because Exercises.routineTitle is not unique in the Exercises table! This means that there could be two exercises with the same routineName. Now if we have a routine with the same name, which of the exercises would it reference? Sqlite just reports an error right away in that case.
There are two fixes possible here:

Make Exercises.routineTitle a primary key, which makes it unique. It would fix the error, but then each exercise can only belong to one routine, and each routine can only have one exercise. That's probably not what you want
To model a many-to-many relationship between routines and exercises (which means that a routine can consist of multiple exercises, and that an exercise can appear in more than one routine), you would have to introduce another table (this article explains this quite nicely). Moor has an example on many-to-many relationships that models a shopping-card system here. Maybe that example helps building your workout database as well.

I'll close this issue because moor is working as intended here, but if you run into any other problems or have any questions, please let me know :)


Implement active cropland marker in OpenEO-GFMAP

Phase I introduced the active cropland marker based on a simple growing season detection algorithm, based on which active pixels were identified. We need to translate this workflow into OpenEO-GFMAP so it can become part of the new WorldCereal system. In that process, we might also need to revise some thresholds/definitions/etc.

to discuss with Jeroen Dries who could work on this!

methodology is still being developed, so not working on this for now


Production deploys fail on context check

Error when executing a ansible-playbook pemm.yml, need to instead execute ansible-playbook pemm.yml -e '{"deploy_contexts": []}'
All the messy logs:
TASK [create_deployment : Create a deployment] *********************************
fatal: [cdh-pemm1.princeton.edu]: FAILED! => changed=false
  access_control_allow_origin: '*'
  access_control_expose_headers: ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type, Deprecation, Sunset
  connection: close
  content: '{"message":"Conflict: Commit status checks failed for master.","errors":[{"contexts":[{"context":"Travis CI - Branch","state":"success"}],"resource":"Deployment","field":"required_contexts","code":"invalid"}],"documentation_url":"https://developer.github.com/v3/repos/deployments/#create-a-deployment"}'
  content_length: '302'
  content_security_policy: default-src 'none'
  content_type: application/json; charset=utf-8
  date: Fri, 05 Jun 2020 14:50:23 GMT
  json:
    documentation_url: https://developer.github.com/v3/repos/deployments/#create-a-deployment
    errors:
    - code: invalid
      contexts:
      - context: Travis CI - Branch
        state: success
      field: required_contexts
      resource: Deployment
    message: 'Conflict: Commit status checks failed for master.'
  msg: 'Status code was 409 and not [201]: HTTP Error 409: Conflict'
  redirected: false
  referrer_policy: origin-when-cross-origin, strict-origin-when-cross-origin
  server: GitHub.com
  status: 409
  strict_transport_security: max-age=31536000; includeSubdomains; preload
  url: https://api.github.com/repos/Princeton-CDH/pemm-scripts/deployments
  vary: Accept-Encoding, Accept, X-Requested-With
  x_accepted_oauth_scopes: ''
  x_content_type_options: nosniff
  x_frame_options: deny
  x_github_media_type: github.v3; format=json
  x_github_request_id: BC8E:560E:668A2:ACFF7:5EDA5BAF
  x_oauth_scopes: repo_deployment
  x_ratelimit_limit: '5000'
  x_ratelimit_remaining: '4991'
  x_ratelimit_reset: '1591368768'
  x_xss_protection: 1; mode=block
	to retry, use: --limit @/Users/kmcelwee/cdh/CDH_ansible/pemm.retry

PLAY RECAP *********************************************************************
cdh-pemm1.princeton.edu    : ok=1    changed=0    unreachable=0    failed=1

I think this has been resolved on all our current deploys. The problem was that some of context checks have changed, and we may need to override/customize the list on a per-project basis in the settings. We may want to revisit the defaults at some point, but not sure how urgent that is (or how consistent we are with naming our checks).


Storage failed: '>' not supported between instances of 'float' and 'NoneType'

Describe the bug
codechecker store fails with
[INFO 2020-05-23 13:27] - Storage failed: '>' not supported between instances of 'float' and 'NoneType'

CodeChecker version
CodeChecker analyzer version:
---------------------------------------------------------------
Kind                 | Version
---------------------------------------------------------------
Base package version | 6.13.0
Package build date   | 2020-05-23T10:49
Git commit ID (hash) | 2934ac134ce141f7cc91a948cbba8e6389381fac
Git tag information  | 6.13
---------------------------------------------------------------

CodeChecker web version:
------------------------------------------------------------------------
Kind                          | Version
------------------------------------------------------------------------
Base package version          | 6.13.0
Package build date            | 2020-05-23T10:49
Git commit ID (hash)          | 2934ac134ce141f7cc91a948cbba8e6389381fac
Git tag information           | 6.13
Server supported API (Thrift) | 6.27
Client API (Thrift)           | 6.27
------------------------------------------------------------------------

To Reproduce
$ CodeChecker store codechecker-results/ -n firefox_$(date "+%Y%m%d%H%M") --url firefox
[INFO 2020-05-23 10:41] - Storing analysis results for run 'firefox_202005231041'
[INFO 2020-05-23 10:41] - Checking local passwords or tokens in /var/lib/jenkins/.codechecker.passwords.json
[INFO 2020-05-23 10:41] - Password file not found.
[INFO 2020-05-23 10:41] - Checking for local valid sessions.
[INFO 2020-05-23 13:27] - Storage failed: '>' not supported between instances of 'float' and 'NoneType'

Happy to help debugging
(it would be nice to show the backtrace)

@sylvestre Could you please run your store command in debug mode (--verbose debug) and send us your output.

I did it and it took more than 13h to get this error:
[DEBUG][2020-05-25 07:52:04] {system} [15707] <140713747179328> - store.py:366 assemble_zip() - Copying file '/var/lib/jenkins/workspace/firefox-codechecker/codechecker-results/DecryptJob.cpp_clang-tidy_9ee326367450416a9885397e67a737f1.plist' to ZIP assembly dir...
[INFO][2020-05-25 08:09:14] {system} [15707] <140713747179328> - store.py:646 main() - Storage failed: Can't close the ZIP file while there is an open writing handle on it. Close the writing handle before closing the zip.
Exception ignored in: <function ZipFile.__del__ at 0x7ffa7842fea0>
Traceback (most recent call last):
  File "/usr/lib/python3.7/zipfile.py", line 1789, in __del__
    self.close()
  File "/usr/lib/python3.7/zipfile.py", line 1798, in close
    raise ValueError("Can't close the ZIP file while there is "
ValueError: Can't close the ZIP file while there is an open writing handle on it. Close the writing handle before closing the zip.

which is of course different ... :)
My codechecker-results directory is 257 Go. I am trying to run it again if I get the same error again

Could the current way of extracting the zip file on the server side cause this problem?
It it written out to the disk and immediately read back for extraction.
https://github.com/Ericsson/codechecker/blob/9b25c841942b8992a808bda831f8eee8eab137f1/web/server/codechecker_server/api/report_server.py#L505-L525

I tried again in Debug and yeah, same pb again with zip.

on which platform do you try run it?

@gyorb on Debian

In the latest release v6.12.0 there were some clang-tidy configuration changes to show results from the header files.
It can increase the found reports. Are there a lot more reports after that change?
You can switch back to the old behavior with this analyzer configuration:
CodeChecker analyze compile_command.json --analyzer-config clang-tidy:HeaderFilterRegex=\"\"
Could you try to run an analysis with this configuration?

I faced with it.
Seems codechecker/web/client/codechecker_client/cmd/store.py:324
                if sc_handler.has_source_line_comments(file_path, report_line):
                    file_hash_with_review_status.add(file_hash)

reading every single file takes a long.

That part was not touched recently but maybe the increased report count causes to check a lot more files than before.
We will need to check what can be improved there.

I'm working on some performance improvements for the store command.

@gyorb sorry it took forever for me to get back to this (as it takes almost a full day to run)
Using
CodeChecker analyze compile_command.json --analyzer-config clang-tidy:HeaderFilterRegex=\"\"

indeed made the build completed.
I will try to run store in verbose mode but it takes several days to run :/

@sylvestre there were some performance improvements done for the store did you have time to try it out?


[integration]:  merging bfloat16 model failed

base-model: Weyaxi/Dolphin2.1-OpenOrca-7B
Scenario:

followed the following guidelines - https://github.com/intel-analytics/ipex-llm/tree/main/python/llm/example/GPU/LLM-Finetuning/QLoRA/alpaca-qlora#1-install
Fine-Tuning method alpaca-qlora (IPEX-LLM)
after FT when trying to merge the model have an torch data_type issue

(ft_Qlora) intel@imu-nex-sprx92-max1-sut:~/ritu/ipex-llm/python/llm/example/GPU/LLM-Finetuning/QLoRA/alpaca-qlora$ python ./export_merged_model.py --repo-id-or-model-path Weyaxi/Dolphin2.1-OpenOrca-7B --adapter_path ./out-dir-FT/tmp-checkpoint-1400/ --output_path ./out-dir-FT/tmp-checkpoint-1400-merged
/home/intel/miniconda3/envs/ft_Qlora/lib/python3.11/site-packages/torchvision/io/image.py:13: UserWarning: Failed to load image Python extension: ''If you don't plan on using image functionality from `torchvision.io`, you can ignore this warning. Otherwise, there might be something wrong with your environment. Did you have `libjpeg` or `libpng` installed before building `torchvision` from source?
  warn(
2024-05-24 14:37:05,078 - INFO - intel_extension_for_pytorch auto imported
2024-05-24 14:37:05,084 - WARNING - The installed version of bitsandbytes was compiled without GPU support. 8-bit optimizers, 8-bit multiplication, and GPU quantization are unavailable.
/home/intel/miniconda3/envs/ft_Qlora/lib/python3.11/site-packages/huggingface_hub/file_download.py:1132: FutureWarning: `resume_download` is deprecated and will be removed in version 1.0.0. Downloads always resume when possible. If you want to force a new download, use `force_download=True`.
  warnings.warn(
2024-05-24 14:37:05,713 - ERROR -

****************************Usage Error************************
Please use torch_dtype=torch.bfloat16 when setting load_in_low_bit='bf16'.
2024-05-24 14:37:05,713 - ERROR -

****************************Call Stack*************************
Failed to merge the adapter, error: Please use torch_dtype=torch.bfloat16 when setting load_in_low_bit='bf16'..

what else tried:
added 'torch_dtype=torch.bfloat16'  in utils code (in function -> merge_adapter) for e.g.
--> common/utils/util.py  +183
    try:
        base_model = AutoModelForCausalLM.from_pretrained(
            base_model,
            #load_in_low_bit="nf4", # should load the orignal model
            #torch_dtype=torch.float16,
            #ritu: added for DolphinOrca-7b
            **torch_dtype=torch.bfloat16**,
            #end
            device_map={"": "cpu"},
        )

        lora_model = PeftModel.from_pretrained(
            base_model,
            adapter_path,
            device_map={"": "cpu"},
            #torch_dtype=torch.float16,
            **torch_dtype=torch.bfloat16,**
        )

this doesn't solve the issue and gives an empty error.
Loading checkpoint shards: 100%|████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 2/2 [00:00<00:00, 11.78it/s]
2024-05-24 10:35:22,564 - INFO - Converting the current model to bf16 format......
[2024-05-24 10:35:22,912] [INFO] [real_accelerator.py:158:get_accelerator] Setting ds_accelerator to xpu (auto detect)
Failed to merge the adapter, error: .

Hi, @raj-ritu17 , I have reproduced error during merging model. We will try to fix it, update here once it is solved.

Hi, @raj-ritu17 .
We have fixed this bug. Please install the latest ipex-llm (2.1.0b20240527), no need to modify utils code and just run this script to merge model.
According to my local experiment, this merging process works and you could use this merged model do inference following https://github.com/intel-analytics/ipex-llm/tree/main/python/llm/example/GPU/HF-Transformers-AutoModels/Model/mistral


Make jquery dependency optional

I know this sounds a little weird because the name of this project is jquery-browser-plugin, but it doesn't seem to use jquery for anything. Currently I want to use this plugin to detect the browser before the app launches, therefor I need jquery (I use jquery version 2, old IE versions are not supported) which may break on non supported browsers.

@gabceb Maybe we don't need a separate repo after all...
https://gist.github.com/yckart/5563717#file-jquery-addrule-js-L37-L40
https://gist.github.com/yckart/5563717#comment-829398
And I don't think this would require any AMD/Node.js support changes either.

@jcbvm I've implemented this at #56, if you'd like to look at it.

@le717 This looks good to me. I think it's fine to run the code automatically on page load, so just assigning the result to jQBrowser instead of exposing it as a function. But I'm fine with both.

@jcbvm I'll update it to run on page load (as that matches the jQuery behavior), but I would still like to have a function available if someone wanted to detect a browser by an arbitrary UA. There are probably instances somewhere where this happens, and exposing a function allows that. @gabceb and I will, of course, pound out any details, but I'd like to hear your thoughts on that use case.

I think one solution would be to keep the jQuery.uaMatch and add the function window.jQBrowser.uaMatch with the same behaviour as jQuery.uaMatch. This way you will have full compatibility with the old version. Or you can move the uaMatch to the returned object (as a function), that way both the jquery and global versions are exactly the same.

Fixed by #56. Thanks @le717

@jcbvm This is now available in v0.0.8, now available on npm. :)

Cool, many thanks!


Import completion error

from llmengine import Completion
ImportError: cannot import name 'Completion' from partially initialized module 'llmengine' (most likely due to a circular import)
pip list:
scale-llm-engine   0.0.0b7

I can't replicate your import issue on my machine with 0.0.0b7, could you provide some more information?

I'm going to close this issue now assuming that there's not anything else that needs to be done here.


Enhancement: Escape now "escapes" Calendar event popups

Summary
Follows up from #579 which resulted from #551
Because I'm very shaky on React Hooks, I made a new branch with just the changes to CourseCalendarEvent.tsx for review and feedback specifically on this, since it isn't a Dialog & onClick!

Previously, you had to click out of Calendar Event popups (which may cover other events).
Implemented React Hooks that listens for the "escape" keyCode and sets the visibility of the Paper popup to 'none'.

Test Plan
Checked that on "escape", the Paper/popup disappears.

LGTM and works. I think that this is the idiomatic way of implementing an event listener.
I would just nitpick and say that event.keyCode === 27 should be documented, either with a comment or with const escapeKeyCode = 27...

Added in the documentation w/ a comment!

I should've said this last time, but I would rename isShown to something like paperRef. It's not a bool. After that, I think it's ready to merge. Sorry about that

Done! 💖


[Config] - Use Codeception for tests instead of plain PHPUnit

Codeception should be the primary testing framework in use in the whole SlaxWeb\Framework

Resolved.