Istiod HPA scales in and out every few minutes

Description
We have 3 clusters with similar workload. On 2 clusters istiod runs on a single pod and HPA virtually never increases it to 2. On one of the three clusters however, HPA scales istiod out to 5 pods every few min. I.e. every few min we see istiod pods going from 1=>5 and then 5=>1.
What I noticed is that every few min it seems istiod CPU consumption jumps (triggering HPA scale out). Looking at istiod logs, it seems during such CPU spikes, we have 350+ logs per second that look as follows:
2021-07-20T18:35:34.853259Z	info	ads	CDS: PUSH for node:redacted-d6849b97f-hg959.team1 resources:310 size:315.7kB
2021-07-20T18:35:34.853480Z	info	ads	EDS: PUSH for node:redacted-5d54b56779-ks55r.team2 resources:278 size:132.5kB empty:0 cached:278/278
2021-07-20T18:35:34.854305Z	info	ads	EDS: PUSH for node:redacted-7548cbc45f-drfbz.team3 resources:278 size:132.5kB empty:0 cached:278/278
2021-07-20T18:35:34.858476Z	info	ads	LDS: PUSH for node:redacted-67d7b9858d-xc4qj.team4 resources:60 size:176.9kB
2021-07-20T18:35:34.853688Z	info	ads	CDS: PUSH for node:redacted-6bf54f6768-pwwx9.team2 resources:310 size:315.8kB
2021-07-20T18:35:34.859439Z	info	ads	LDS: PUSH for node:redacted-68678b758-9krj2.team3 resources:60 size:187.0kB
2021-07-20T18:35:34.859764Z	info	ads	LDS: PUSH for node:redacted-5d54b56779-ks55r.team2 resources:60 size:175.6kB
2021-07-20T18:35:34.860094Z	info	ads	EDS: PUSH for node:redacted-6f56d7d9f8-ftttm.team7 resources:278 size:132.5kB empty:0 cached:278/278
2021-07-20T18:35:34.865248Z	info	ads	RDS: PUSH for node:redacted-b7f7849b9-kf584.team5 resources:22 size:264.6kB
2021-07-20T18:35:34.867051Z	info	ads	CDS: PUSH for node:redacted-79f8746865-4fpww.team2 resources:310 size:315.7kB
2021-07-20T18:35:34.867771Z	info	ads	CDS: PUSH for node:redacted-synchronizationjob-7d88d5dxbfl2.team5 resources:310 size:315.8kB
2021-07-20T18:35:34.868481Z	info	ads	EDS: PUSH for node:redacted-79f8746865-4fpww.team2 resources:278 size:132.5kB empty:0 cached:278/278
2021-07-20T18:35:34.872880Z	info	ads	EDS: PUSH for node:redacted-6b46fc9c7d-x8sdj.team3 resources:278 size:132.5kB empty:0 cached:278/278
2021-07-20T18:35:34.853921Z	info	ads	CDS: PUSH for node:redacted-84dcc584c8-j4n74.team6 resources:310 size:315.8kB
2021-07-20T18:35:34.873420Z	info	ads	RDS: PUSH for node:redacted-9dcb99988-qm4t6.team5 resources:22 size:264.6kB
2021-07-20T18:35:34.874939Z	info	ads	EDS: PUSH for node:redacted-7d88d5dxbfl2.team5 resources:278 size:132.5kB empty:0 cached:278/278
2021-07-20T18:35:34.875879Z	info	ads	RDS: PUSH for node:redacted-5fb5b55bc5-9cnqb.team7 resources:23 size:262.9kB
2021-07-20T18:35:34.875993Z	info	ads	EDS: PUSH for node:redacted-6bf54f6768-pwwx9.team2 resources:278 size:132.5kB empty:0 cached:278/278
2021-07-20T18:35:34.876632Z	info	ads	LDS: PUSH for node:redacted-79f8746865-4fpww.team2 resources:60 size:176.9kB
2021-07-20T18:35:34.876803Z	info	ads	EDS: PUSH for node:redacted-6b9798545f-24vz6.team5 resources:278 size:132.5kB empty:0 cached:278/278
2021-07-20T18:35:34.882346Z	info	ads	EDS: PUSH for node:redacted.team7 resources:278 size:132.5kB empty:0 cached:278/278
2021-07-20T18:35:34.886385Z	info	ads	CDS: PUSH for node:redacted-78fcd86598-9tfg9.team3 resources:310 size:315.8kB

I don't see any error logs in istiod logs. The only logs that contain "error" in the description look as follows:
2021-07-20T17:34:32.726059Z	info	ads	ADS: "<IP_ADDRESS>:4067" sidecar~<IP_ADDRESS>~redacted.team1~team1.svc.cluster.local-80289 terminated rpc error: code = Canceled desc = context canceled

I see about ~900 such entries out of 46K logs from the past hours. Is this expected/anything we should do about it?
[ ] Docs
[ ] Installation
[ ] Networking
[X] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
Expected behavior
There is only one Istiod pod. HPA rarely (if ever) sees the need to scale out.
Version (include the output of istioctl version --remote and kubectl version --short and helm version --short if you used Helm)
❯ istioctl version --remote
client version: 1.10.2
control plane version: 1.9.5
data plane version: 1.9.5 (453 proxies)

❯ kubectl version --short
Client Version: v1.21.2
Server Version: v1.20.5

How was Istio installed?
istioctl install -s profile=default -f "./environments/test/istioctl-params.yml" -y

where the params file looks as follows:
apiVersion: install.istio.io/v1alpha2
kind: IstioControlPlane
spec:
  components:
    cni:
      namespace: kube-system
      enabled: true
  values:
    global:
      jwtPolicy: third-party-jwt
      proxy:
        holdApplicationUntilProxyStarts: true
        resources:
          requests:
            cpu: 10m
            memory: 64Mi
    sidecarInjectorWebhook:
      rewriteAppHTTPProbe: true
    gateways:
      istio-ingressgateway:
        serviceAnnotations:
          service.beta.kubernetes.io/azure-load-balancer-internal: "true"
          service.beta.kubernetes.io/azure-load-balancer-internal-subnet: "<Azure VNet subnet resource>"
        loadBalancerIP: "<Azure LB IP>"
    cni:
      excludeNamespaces:
        - istio-system
        - kube-system
        - kube-public
        - kube-node-lease
        - monitoring
      logLevel: info

Environment where the bug was observed (cloud vendor, OS, etc)

Azure AKS cluster with ~12-15 nodes, ~600 pods total (~450 with Istio/Envoy proxy)
Istio 1.9.5
K8S 1.20.5

Do you have any suggestions on how to further troubleshoot this?

We're still investigating this issue and we have a reproducible scenario:
Setup: We have a K8S cluster with some Istio workloads and some non-Istio workloads (i.e. pods that aren't part of the Istio service mesh, don't have Istio sidecar). Due to this, we have some nodes in the cluster that don't run any istio-enabled workloads (just "regular" K8S pods).
Repro: As soon as an Istio-enabled workload gets scheduled on such a node that doesn't have any Istio-enabled workloads yet, istiod starts consuming lots of CPU, HPA scales out istiod pod to 5 instances, and we see thousands of logs similar to the ones shown above.
Is this expected? I.e. is it expected that as soon as a cluster node gets any Istio-enabled workload, Istio needs to push some config/data to that node, resulting in all those logs and high CPU consumption?


Add unit tests for anonymizer/app/writer

Which problem is this PR solving?

Part of #5068

Description of the changes

Added unit tests for cmd/anonymizer/app/writer/

How was this change tested?

go test -cover -v ./cmd/anonymizer/app/writer
make lint test

Checklist

[x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
[x] I have signed all commits
[x] I have added unit tests for the new functionality
[x] I have run lint and test steps successfully

for jaeger: make lint test
for jaeger-ui: yarn lint and yarn test

Coverage 94.09% ==> 94.38%, nice! 🎉


[Feature] Add a new set of Rainbow Animation buttons

Description
I would like to add a new Button component consisting of 5 animated buttons with Rainbow hover effects.
Screenshots
Here's a sample screenshot of the buttons:

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.
[X] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

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

@Rakesh9100 I would like to work on this issue under JWOC, please assign it to me.

Proceed @zul132


New network request: ShimmerEVM Mainnet

Network information

Name: ShimmerEVM Mainnet
ChainId: 148
RPC http/s address: https://json-rpc.evm.shimmer.network/
Link to https://chainlist.org/: https://chainlist.org/chain/148
(Optional) Blockexplorer: https://explorer.evm.shimmer.network/
[x] I have run RPC='http://my-rpc-address' yarn estimate
[x] I have sent requiredFunds to 0xE1CB04A0fA36DdD16a06ea828007E35e1a3cBC37

https://explorer.evm.shimmer.network/tx/0x5e1393df213233d6c7e4bbaf652a5690c38fd526d9d7d7387066dab27c4439a7


Could not install typing files for JavaScript language features

VSCode Version: 1.14.1 64bit
OS Version: Windows 7
npm Version: 5.0.3

Steps to Reproduce: (I don't know the exact repro step, this does not repro on my Windows 10 machines)

Open any .js file (The following short file in my case)module.exports = __dirname;

VSCode says Could not install typings files for JavaScript language features. Please ensure that NPM is installed or configure 'typescript.npm' in your user settings
Set typescript.npm to C:\\Program Files\\nodejs\\npm.cmd or C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm.cmd but both don't help

Reproduces without extensions: Yes

VSCode Version: 1.14.2 (1.14.2)
OS Version: macOS 10.12.6 (16G29)
npm Version: 5.3

set to where /usr/local/bin/npm
and actual /usr/local/lib/node_modules/npm/bin/npm

Hey sorry I overlooked this issue. If you are still seeing this in VScode 1.16, please try collecting the typings installer logs:

Set "typescript.tsserver.log": "normal"
Restart VSCode and reproduce the issue
Run the TypeScript: Open TSServer log  command in vscode
Look in the ti-xxx.log file in there

Near the top of this file there should be an error message about why typings install failed. See https://github.com/Microsoft/TypeScript/issues/18344 for example

larsch@t420arch ~ % npm -v
5.4.2

larsch@t420arch ~ % node -v
v8.6.0

tsserver.log:
...
Info 32   [18:47:23.507] Sending request: {"projectName":"/dev/null/inferredProject1*","fileNames":["/opt/visual-studio-code/resources/app/extensions/node_modules/typescript/lib/lib.es6.d.ts","/home/larsch/src/pixel/pixel.js"],"compilerOptions":{"module":1,"target":2,"allowSyntheticDefaultImports":true,"allowNonTsExtensions":true,"allowJs":true,"jsx":1,"checkJs":false,"noEmitForJsFiles":true,"maxNodeModuleJsDepth":2},"typeAcquisition":{"enable":true,"include":[],"exclude":[]},"unresolvedImports":["buffer","crypto","events","fs","http","net","path","process","querystring","stream","tty","url","util","zlib"],"projectRootPath":"","kind":"discover"}
Info 33   [18:47:23.515] Received response: {"kind":"event::initializationFailed","message":"Command failed: npm install --ignore-scripts types-registry"}
Info 34   [18:47:23.515] event: {"seq":0,"type":"event","event":"typesInstallerInitializationFailed","body":{"message":"Command failed: npm install --ignore-scripts types-registry"}}
...

Output of npm install:
larsch@t420arch ~/src/pixel (git)-[master] % npm install --ignore-scripts types-registry                                                                                :(
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.

npm ERR! path /home/larsch/src/pixel/node_modules/.staging/types-registry-9f09bb8c/node_modules
npm ERR! code ENOTDIR
npm ERR! errno -20
npm ERR! syscall lstat
npm ERR! ENOTDIR: not a directory, lstat '/home/larsch/src/pixel/node_modules/.staging/types-registry-9f09bb8c/node_modules'

npm ERR!


Update stream-analytics-how-to-configure-azure-machine-learning-endpo…

…ints-in-stream-analytics.md

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:
Public repo URL: https://github.com/MicrosoftDocs/azure-docs
Private repo URL: https://github.com/MicrosoftDocs/azure-docs-pr
We have closed this pull request. Please resubmit the updates through the private repo. For more information, [see the contributor guide].(https://docs.microsoft.com/en-us/contribute/)


Wrong description about Typescript

I'm confused about the description of Typescript on the website, in the topic "JavaScript and More" on section "What is TypeScript?" says:

Because Typescript is a superset, not a language. I'm not certain if I understand correctly.

TypeScript has always called itself a language, and as a language it can be called a superset of JavaScript. Due to TypeScript having all the features of JavaScript and more (types)
Personly I prefer against talking in set theory on the landing page, which is why I use 'extends' or 'build upon' because the set theory audience is a slither of the general dev audience
Does that help clear it up?

cc: @matheusalbino the answer to your question.

Yes, it's clear. Thanks!


Add gardener extension submodule

What would you like to be added:
A new gardener extension golang submodule to be added to the project. It should have a binary that would

It should be like any other gardener extension (e.g. https://github.com/gardener/gardener-extension-shoot-dns-service)
It should

deploy the oidc-webhook-authenticator in the shoot namespace
create needed certificates for authentication
mutate kube-apiserver Pods and add the needed flags to them
add network policies

Hi @dimityrmirchev
Do you know a rough target date for this feature?


Hide coverage implementation details from the SDK

Now that the FFI has been fully separated, we would like to fully separate concerns.
These methods can still be accessed through the raw FFI and don't need to have wrappers exposed to SDK users.

Closing this while I double check some things


python3.pkgs.ale-py: add missing ninja dependency

Description of changes
This is detected when building this package using build, which we are switching Python packages to using.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 250211 run on x86_64-linux 1

4 packages built:

python310Packages.ale-py
python310Packages.ale-py.dist
python311Packages.ale-py
python311Packages.ale-py.dist


Publish a new version to depend on url 1.2

The current 0.4.0 on crates.io still depends on 0.2.38 so we get 2 urls crates pulled in for any non trivial program.

I didn't consider it important, but here's 0.4.1

thanks!


[Dark Heresy 1st Edition] Range Categories are Incorrect

As requested, I am resubmitting this.
Dark Heresy 1st Edition has the following Range Categories.
Point Blank (PB) - usually 3m or less
Short (S) - less than half the weapon's listed range (<R/2)
Medium (M) - Anything from half the weapon's listed range up to twice the weapon's listed range (R/2-R2)
Long (L) - greater than twice the weapon's listed range (>R2)
Extreme (E) - greater than three times the weapon's listed range. (>R*3)
This screen shot shows the incorrect categories on the ranged weapon "Not My Father's Bolt Pistol."

Point Blank (PB): 3m
Short (S): <R/2
Medium (M): <R2
Long (L): >R2
Extreme (E): >R*3
would be possible. There is not enough space to provide the range.

That looks great other than Medium Range. If there is enough room, I would suggest .5-2 or some variation of that (0.5-2x or .5-2x or whatever). If none of that fits, I would suggest leaving Medium as simply R.

@mperes something seems off with the rollout of the updates. Can you please verify when the changes will be rolled out? Thanks

@RobbaYaga please close it when it is fixed

Now it is mega confusing!
The fields contain the maximum values according to the rulebook.

@Tealk besides the medium range I disagree. But I also dont understand the problem with typing in text. So, I dont object to reverting it. Clarifing the issue in such a small place so that all parties are happy is probably impossible.
@RobbaYaga you can state your problem with it any maybe we can come to some sort of agreement (e.g. tooltip is more explaining)

I have no problem with the current change, however, if other people are having trouble with it, then just leave them blank and let people look up the ranges in the core book. My original post in this thread explains what the Ranges are according to the core rules and the "maximum values" change in each field depending on the weapon. (A weapon with a range of 30m is going to have vastly different maximums than a weapon with a range of 120m). Simply having the correct range abbreviations works for me. (They are correct now.)

Well, in my rulebook it is a little different than what you wrote in your first post:
Point Blank (PB) up to 3m
Short (S) to range/2
Medium (M) to range 2
Long (L) - to range3
Extreme (E) - to range*4
of course the minimum always refers to the value before; so the previous version was correct and the current one is faulty, because you can't shoot further than the extreme range according to the rules

What you have here is fine and is identical (if worded differently) to what I put in the post at the top of this thread. Regardless, the most succinct description of ranges comes from the Errata.
"Short range is below half of this number, long range is above double, extreme range is above three times and maximum is above four times. Below 3 metres is point blank range."

Extreme (E) - greater than three times the weapon's listed range. (>R*3)

But as far as I know, this would mean that the weapon can shoot infinitely far

Sure, that could be inferred from what is listed there. However the things you have listed are too many characters to fit into the available space as it has been described to me "(Extreme (E) - to range*4)." Perhaps yet another box that adds maximum range should be there, but probably not.
Honestly, Tealk, there never were ranges on the ranged weapons on the character sheet originally. So I honestly don't care what goes into the little boxes, since I can simply put the correct range in there anyway. I mostly wanted the range categories to be correct and they are now.
If this is really a problem, my best and final suggestion is to simply remove the range categories again and put the range descriptions on the character sheet in a static text box that reflects the errata wording and let the players / GM calculate them as needed.
We're all spending way too much time on this and I'm just thankful people are willing to work on the sheet at all this many years after the game went out of print. So thank you to you and Sexinator and do whatever seems best.

I mostly wanted the range categories to be correct and they are now.

no, as I have shown above it is not, because the statement placeholder="> R3" is wrong as well as placeholder="0.5-2R"

I feel like there is a real communication problem here.
The categories are correct: PB / S / M / L / E (as opposed to what was there before PB / N / M / F / E).
The details within each category that describes how to calculate each range are also, technically, correct. Extreme Range is, in fact, greater than the Range multiplied by 3 (>R3), just as Medium Range is from one half the Range to two times the Range (0.5-2R). The fact that there is a maximum Range is irrelevant.
HOWEVER, I don't care!!!
I offered my suggestion. Add a block of static text to the character sheet describing the Range categories and let the players / GM work them out. Leave the spaces within each category blank.
If that's not good enough for you, then change those spaces to say whatever you want. Just leave the categories as they are now.

just as Medium Range is from one half the Range to two times the Range (0.5-2R).

ok that was very ambiguous, it would be better to write 0.5R - 2R. I have assumed a mathematical calculation all the time

The fact that there is a maximum Range is irrelevant.

Why is that irrelevant? If you want to keep on shooting you ignore the rule?
House rules have no place in the bows in my opinion.

I will insert the following description text that will be displayed when you move the mouse over the fields

The input fields always show the maximum value for the distance category, the minimum value is always the previous category+1

then everything should be clear

While Short Range is technically incorrect (it's actually less than R/2), I'm fine with these changes as long as the category titles remain correct (PB / S / M / L / E).

I'm fine with these changes as long as the category titles remain correct (PB / S / M / L / E).

This is regulated by the translation, not by the programming.
You have to contact the Roll20 support and get an account.

I have an account on Roll20 you inconsiderate wretch. I'm sorry that your German translation is not what you want it to be, but on the English character sheet, the Range Categories should read PB/S/M/L/E. Change them back. That was the whole point of this ticket.

@Stexinator What options do I have for reporting this nonsense? Is this a Github issue or a Roll20 issue? I've been as accommodating as I can possibly be and this person just refuses to do anything except what they want and what they want is simply wrong based on the rules of the game.

Any translations you find in the folders below are not changed by GitHub, but by some other tool I don't know myself (I asked for an account once, but never got one)
https://github.com/Roll20/roll20-character-sheets/tree/master/Dark_Heresy/translations
What you say is wrong, because as you can see in the change, there is F and not L too; so this has nothing to do with my changes I mentioned.
https://github.com/Roll20/roll20-character-sheets/commit/d12cb5c5d40ba6bc40c45155c4a3b4680307a275

@Tealk there is no translation initalization yet for this sheet. One of the points of my change was, to give the english names for the ranges. Instead of reverting my changes you should make a commit which changes the placeholder according to the outcome of your discussion.
It is common to use the english language until that happens. If you want to have a certain language then feel free to start the translation of the sheet.

@Tealk @RobbaYaga by the way this PR is closed. @Tealk open a new issue which is linked to this one and post a summary into it. Then add your changes to the new issue

@Tealk there is no translation initalization yet for this sheet. One of the points of my change was, to give the english names for the ranges. Instead of reverting my changes you should make a commit which changes the placeholder according to the outcome of your discussion.
It is common to use the english language until that happens. If you want to have a certain language then feel free to start the translation of the sheet.

If these are not translations, what are they?
https://github.com/Roll20/roll20-character-sheets/tree/master/Dark_Heresy/translations

Right, I looked at 2nd edition. @RobbaYaga in this case I have to agree with @Tealk . Please try to get a Crowdin-Account (https://github.com/Roll20/roll20-character-sheets#internationalization--translations) and submit a fix for the english language there

Very well, I apologize to @Tealk for my harsh words. Given the sheet was completely in English when I came upon it, I (arrogantly) assumed it started out that way.
Thanks for being the intermediary, @Stexinator and I will join and ask for a fix.
Despite the mild battle of text we had, I really do appreciate all the efforts from both of you.

Diverse opinions have the potential to improve things.


Enable remote debugging

This fixes #16.
This PR provides some mapping functions in lib/util.js (extending one that was stubbed out, so I think this was a planned approach). They enable path translation between a local code base in Brackets and a remote instance of Xdebug (e.g., in a VM or Docker container). It also removes the binding to localhost that prevents Xdebug from connecting in the first place.
The user can set a preference in brackets.json that maps project roots to remote paths, e.g.:
"php-debugger.pathMap": {
  "/Users/admin/Documents/myProject/": "/var/www/myProject/"
}

Now, php-debugger can properly map breakpoints to a remote Xdebug instance, and can open local files while stepping through code. If the file is not available locally (because it is outside the mapping defined in the user preference), a note will be logged to the php-debugger console.
A note is also logged to let the user know whether or not a path map is being used.

Just a note that we could, in theory, go a little bit further for any situation (remote debugging or not) where Brackets can't open the file that Xdebug is operating on. Xdebug / DGBp allows retrieving the source code of files in a pretty straightforward way:
xdebug.execute("source", {
  f: util.toXdebugPath(fullPath)
}).done(function(response) {
  var sourceText = window.atob(response[0].textContent); // base64 decode
  /* do something with sourceText */
});

I would find this useful in many situations where the project I am working on is a plugin or module of a larger codebase contained within a VM / container that I am remote debugging. It would allow me to step through the entire codebase even if my project is only a small subset thereof.
The question is: How do we present this source code to the user in a way that is useful, straightforward, and unburdensome? These are files that, for whatever reason, we don't have access to locally. But Brackets is a code editor and doesn't provide, as far as I can tell, a construct for uneditable files. Perhaps a modal would work, or perhaps the use case isn't compelling enough for the work this would entail.
At any rate, this is an aside and shouldn't stand in the way of this PR. Thanks @spocke for a great plugin!

@chriszarate I know it has been nearly two years since you opened this PR (I'm only just now seeing it), but does your code still work for you? I seem to be able to connect to my remote instance properly, but it doesn't receive any data back from xDebug so nothing gets populated in the debug panel in Brackets.
I've tried to trace where things "break" in the plugin and it seems to be related to ./node/main.js. That gets loaded into _nodeDomain in ./lib/xdebug.js but none of the events in ./node/main.js appear to be firing off.

@d4mation Sad to say I switched to Atom, so I haven't tested this in some time. 😢  It definitely worked when I first opened the PR.

@chriszarate Bummer. Thanks anyway! I am giving Atom a shot. I've gone without proper debugging for too long 😆


Pass opencrud field properties to the form

I was trying to use SmartForm with a collection only using the new OpenCrud API (canRead instead of insertableBy, etc) and I found that no fields were showing because the schema arriving to Form.jsx did not contain those properties.

Oh, good call! Thanks!


[SP-6552] Backport of PDI-19937 - Database Password Field shows password in plain text when searching job's metadata (9.3 Suite)

Original PR:

https://github.com/pentaho/pentaho-kettle/pull/9063

@pentaho/tatooine

 Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

🐸 JFrog Frogbot

:white_check_mark: Build finished in 14m 13s
Build command:
mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl engine

:ok_hand: All tests passed!
Tests run: 2678, Failures: 0, Skipped: 6    Test Results

:information_source:  This is an automatic message


Implement collision margin

Objective
Collisions for thin objects such as triangle meshes can sometimes have stability and performance issues, especially when dynamically colliding against other thin objects.

Infinitely thin objects are more prone to numerical issues and tunneling. Two trimeshes colliding against each other can easily get stuck in an intersecting state or have other collision issues.
Collision algorithms for intersecting objects are typically more expensive than for the non-intersecting state. Trimesh collisions can be very expensive if the trimesh has dense geometry.

To allow users to mitigate these issues when necessary, engines such as Rapier and Bullet provide a collision margin or contact skin that forces the solver to maintain an artificial separation distance between objects. It can be thought of as a kind of shell that adds thickness to colliders, reducing the issues mentioned above.
Solution
Add a CollisionMargin component. It can be used to add artifical thickness to any collider for collisions.
commands.spawn((
    RigidBody::Dynamic,
    Collider::trimesh_from_mesh(&mesh),
    CollisionMargin(0.05),
));

Without a collision margin:
https://github.com/Jondolf/bevy_xpbd/assets/57632562/0364168b-81e9-4887-a2fa-10a23b87481e
With a collision margin:
https://github.com/Jondolf/bevy_xpbd/assets/57632562/b41a3cbc-0709-4c3f-9f20-1b73d2d48e76

Math checks out, just some code quality issues, nothing big :)


Move plotting to WolframModelPlot

Currently, many functions that output graphics (i.e., EventStatesPlotList, in some respects CausalGraph, etc.) are implemented as WolframModel properties, which breaks model/view separation between WolframModel and WolframModelPlot.
It would be better to move these into WolframModelPlot, which would accept an evolution object as the first argument, and a type of graph desired as the second argument.

Not sure it's a good idea. It would probably be better to restore HypergraphPlot name for WolframModelPlot.


Added a layer of self description for each Era

When one imports one of the Era modules qualified one has access to a number of properties of that Era. For example
import qualified Cardano.Ledger.Shelley as Shelley
then
Shelley.Tx   is the Tx for the Shelley era, this holds for all the Core.* type families
Shelley.Era is the Era for Shelley i.e. ShelleyEra c
Shelly.proxy is a proxy for that Era. etc.
In the file Cardano.Ledger.DescribesEra are a set of assertions of what properties hold for each era, and some tests that they actually do. One can see how an Era is constructed by looking at its description. For example here is the description for Mary.
type DescribesMary era =
( WellFormed era,
Core.Value era ~ Mary.Value era,
Core.TxBody era ~ Allegra.TxBody era,
Core.TxOut era ~ Shelley.TxOut era,
Core.Script era ~ Allegra.Script era,
Core.AuxiliaryData era ~ Allegra.AuxiliaryData era,
Core.PParams era ~ Shelley.PParams era,
Core.Tx era ~ Shelley.Tx era
)

Sounds like the current plan is not to merge this


Added TUC Sweden to se

@Jesperson Please provide us with an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course

@Jesperson Please provide us with an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course

https://www.tucsweden.se/utbildning/yh-utbildningar/systemutvecklare-net-distans/
This is my current education. There are more available.

@Jesperson I am sorry for a delay with processing your request.
Pull request merged. Thank you!

@Jesperson I am sorry for a delay with processing your request.
Pull request merged. Thank you!

No problem, I'm happy it got merged!
Looking forward to using Rider!


Is this required for the SS and SE task? assert abs(feat_list[i].size(0) - length_list[i]) < 5. I am getting this error for wav2vec

Traceback (most recent call last):                                                                                | 0/1738 [00:00<?, ?it/s]
File "/raid/home/rajivratn/hemant/superb/s3prl_1st/s3prl/run_downstream.py", line 220, in
main()
File "/raid/home/rajivratn/hemant/superb/s3prl_1st/s3prl/run_downstream.py", line 216, in main
eval(f'runner.{args.mode}')()
File "/raid/home/rajivratn/hemant/superb/s3prl_1st/s3prl/downstream/runner.py", line 308, in train
loss = self.downstream.model(
File "/raid/home/rajivratn/miniconda3/envs/SE/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "/raid/home/rajivratn/miniconda3/envs/SE/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
return forward_call(*args, **kwargs)
File "/raid/home/rajivratn/hemant/superb/s3prl_1st/s3prl/downstream/separation_stft/expert.py", line 258, in forward
features = match_length(features, feat_length)
File "/raid/home/rajivratn/hemant/superb/s3prl_1st/s3prl/downstream/separation_stft/expert.py", line 42, in match_length
assert abs(feat_list[i].size(0) - length_list[i]) < 5

This is not required, however this means something is off, since the expected length from the wav2vec is too different from our prepared target labels' length. I will check it. (It is weird since we ran wav2vec before and I do now remember this happened before.)

Hi, any update on this? I need to run the experiments for a paper. Thanks.

Hi @raotnameh ,
Sorry for the delay. However, I can not reproduce the error so far.
Is this error easy to identify or reproduce?

Specifically, does it happen at the middle/end of the training, or it happen at the start of the training?

@leo19941227 Apologies, I made a mistake in my code which caused the input from the upstream be of fixed length 1. I corrected the mistake and now it is not giving the error. Thank you for the help.

Oh, no problem. Thanks for letting me know. I am closing the issue.


Automatically link issue to milestone (on issue close)qs:

Functional Design

When github issue is closed it is automatically linked to the current milestone (latest if current is not found by date)
Action fail no milestone is not found

Technical Design

New Workflow action on Issue close event added.
It's triggered when an Issue is closed by any method.

gh api repos/org/repo/milestones --jq ".[] | .title"

Use latest milestone if current is not found


Import error sdk/metric/aggregation

After trying to run example from:
https://github.com/GoogleCloudPlatform/opentelemetry-operations-go/tree/main/exporter/metric
go: finding module for package go.opentelemetry.io/otel/sdk/metric/aggregation
go: xxx imports
        github.com/GoogleCloudPlatform/opentelemetry-operations-go/exporter/metric imports
        go.opentelemetry.io/otel/sdk/metric/aggregation: module go.opentelemetry.io/otel/sdk/metric@latest found (v0.41.0), but does not contain package go.opentelemetry.io/otel/sdk/metric/aggregation

We just need to update to the latest version to fix this (https://github.com/GoogleCloudPlatform/opentelemetry-operations-go/pull/728).  If you want to work around this, you can downgrade your sdk/metric package version in your go.mod to v0.41.0


bond_zh_cov response 500

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:
直接调用接口 ak.bond_zh_cov  报错 response 500
Expected behavior
正常调用接口并返回目标页 http://data.eastmoney.com/kzz/default.html 的数据
Screenshots
Desktop (please complete the following information):

OS: [Windows10]
Browser [e.g. chrome, safari]
Version [1.5.23]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

bond_zh_cov

感谢反馈问题，已修复该问题。升级到 AKShare 1.5.28 版本使用。


AWSSDK.Core version <IP_ADDRESS>

For a second I thought this was the dotnet core version coming out of beta 😞

Sadly no :(


Override test timeouts for stream reads

Merged build finished. Test FAILed.

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

Merged build finished. Test PASSed.

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

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.2 please


Allow has_closure_trees on polymorphic models

Not sure what the right term for this is, but let me explain my situation.
I have a Comment model. I understand I can change the parent_column_name for closure_tree, so that it points to commentable_id. I haven't tried this yet, but I'm assuming that will cause problems, because a Comment can duplicate commentable_ids because of different commentable_types. I'd like to limit closure_tree's methods to comments with commentable_type of "Comment" (except for root level comments, of course).
I had a brief search around and it seems like scopes on Rails 4 associations would help massively with this (though I expect that's also not going to be enough and I'm still on Rails 3 :disappointed:). Maybe I'm overthinking it. Maybe I just need to amend the parent method to return nil if the commentable_type is not the same as the model and a few other basic modifications.

Sorry, polymorphic parent tables aren't supported.

Yes, I know. That's why I created the issue. :smiley:
I don't mind maintaining my own fork for that. Do you think it would be difficult to implement?

Extremely difficult and, because the foreign keys can't have constraints
both in the model table and the closure table, I'd expect it to be
non-performant. You might as well not be using an rdbms if you're shipping
the r.
Not knowing your problem space, of course, but if you can rethink your
schema modeling to use sti polymorphism and just add  composition, you'll
be in a better place (just in general because of indexes, not just because
you can use this gem).


Callback on play finish

We need to figure out how to execute a callback on play complete.

I believe this is no longer an issue due to how the tracks autoloop now.


Setup phased transition away from PyTorch version-specific handling of cuda availability and device counting

What does this PR do?
Setup a phased transition away from PyTorch version-specific handling of CUDA availability (torch.cuda.is_available()) and device counting (torch.cuda.device_count())
This is a follow-up to #15110 and #85951 that prepares for removal of unnecessary PyTorch version-specific code once PyTorch 1.13 and 1.14 respectively are the minimum supported PyTorch.
Specifically:

All local NVML-based CUDA device counting code copied from upstream PyTorch can be removed once PyTorch 1.13 is the minimum required.
The _patch_cuda_is_available() context manager (and associated usage of it) and redirection of is_cuda_available to num_cuda_devices() can be removed once PyTorch 1.14 is the minimum required.

As of PyTorch 1.14, is_cuda_available() and num_cuda_devices() should be functionally aliases for torch.cuda.is_available() and torch.cuda.num_cuda_devices() respectively that do not require special PyTorch-specific handling and can take advantage of potential upstream enhancements to the functions (e.g., if those functions abstract ROCm CUDA availability checks using ROCm's SMI analog to NVML).
It likely makes sense to retain the is_cuda_available() and num_cuda_devices() wrappers in the code base to facilitate future PyTorch version-specific handling if it becomes necessary but theoretically those wrappers could be replaced with their associated PyTorch functions as well.
Also perhaps relevant, I've tested these changes locally on PyTorch 1.13.0rc3 and 1.14.0.dev20221013
cc @awaelchli @carmocca
Does your PR introduce any breaking changes? If yes, please list them.
None
Before submitting

[X] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[X] Did you read the contributor guideline, Pull Request section?
[X] Did you make sure your PR does only one thing, instead of bundling different changes together?
[X] Did you make sure to update the documentation with your changes? (if necessary)
[X] Did you write any new necessary tests? (not for typos and docs)
[X] Did you verify new and existing tests pass locally with your changes?
[X] Did you list all the breaking changes introduced by this pull request?
[X] Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
I made sure I had fun 🙃

@speediedan Not sure if I 100% understand the proposed change. Is it because partial changes are in 1.13, where device_count uses NVML yet is_available is not yet using the new implementation?

@speediedan Not sure if I 100% understand the proposed change. Is it because partial changes are in 1.13, where device_count uses NVML yet is_available is not yet using the new implementation?

Yep! It allows to use the relevant PyTorch functions directly whenever they are available, only using the temporary PL versions when necessary.  It also allows us to remove the PL NVML code from upstream once PT 1.13 is the minimum (instead of when PT 1.14 is the minimum).


Contributing improvements

Fix typos
Remove unused gulp-concat dependency
General HTML clean up
Add and show the styles for the %default-ol and %default-ul Sass placeholders that Bitters has
Restructure form elements

Don’t wrap the entire form improperly in a fieldset
Show two inputs wrapped in a fieldset
Show a disabled input
Show a disabled button
Show a legend inside a fieldset
Add a select with the multiple attribute

Improve styling for the welcome message

Typos? From me? inconcevable!

Trolling completed> :trollface:

@whmii Updated, per your comments! :100:

good to merge

Thanks!!
A lot of these changes should probably be carried over to the marketing site example page, as well.


Can't find on wiki anything relate to use .fla or .xfl files/folders as source

Hi,
I am salivating over compiling with VSCode, but even though I develop Flash and AIR since 15 years, I have always used Flash/Animate to compile and debug apps.
I would love to move to VS code+nextgenas, but I do not seem to find a way to build from a fla source file. could you point me in the right direction?
Thanks

You may be able to set up a coding environment in VSCode, but you will need to continue to compile and debug in Animate/Flash. There is no support for FLA/XFL files in the VSCode extension, and none is planned for the future.

Makes sense though. No choice than animate.
Thank you Josh.


feat: create landing page and refactor app and component folder structure

feat: create landing page and refactor app and component folder structure

Created landing page with the following sections:

Navbar
Home page
About Us
Gallery
Catalog
News
Footer

Refactored app directory by separating folders for different user roles:

Admin
Co-Admin
Staff
Lecturer
Student

Refactored component directory by organizing components  based on user roles.

https://github.com/user-attachments/assets/73aef43d-4af9-4a42-8c05-092552d07472

bagus2

merge aja

okey


Allow setting a custom column width programmatically

This is  a feature request.

Prerequisites

[x] Are you running the latest version?
[x] Are you reporting to the correct repository?
[x] Did you check the documentation?
[x] Did you perform a cursory search?

Description

Just like we can set custom values & meta values; allow setting the column programmatically.
Use case: a certain flag indicates if we need a 2/3 or half width field
Example implementation
$this->form->column($field, $value);

Please give me examples when this would be useful.

We have an informative column (shown during the editing process only) that usually requires a wider view (2/3); however, in some cases the (computed) data in it is quite brief which makes the interface look funky if that column is still shifted to the left.

For the moment I see too many implications and no clean implementation for something that I would probably never use.
Feel free to open a PR for this if you can find a clean implementation...


Try to escape all / in the args for project_id

Tries to really fix #125

Coverage increased (+0.17%) to 73.52% when pulling 223051eb59866d80e871fffc307b70426e06c346 on escape_project_id_namespace into 7224dcfb527db6a5ecc5a1e963f6f71ce8c9289f on develop.

Coverage increased (+0.17%) to 73.52% when pulling 223051eb59866d80e871fffc307b70426e06c346 on escape_project_id_namespace into 7224dcfb527db6a5ecc5a1e963f6f71ce8c9289f on develop.


edge-http: make fields in {Req,Resp}Headers non-optional

Ensures that the {Req,Resp}Headers is always complete and adheres to the HTTP spec. Saves a lot code from dealing with Options.
Also, send_status_line is removed, and inlined into send_request and send_status. This cuts ~80 lines from the code, which is nice.

Thanks for the review! I understand your point. I would like to know, in the first place, do you think that making the fields non-Option makes sense? If yes, then maybe I can spend some time to think about how to let the user allocate them before parsing.

Thanks for the review! I understand your point. I would like to know, in the first place, do you think that making the fields non-Option makes sense? If yes, then maybe I can spend some time to think about how to let the user allocate them before parsing.

It certainly makes the code simpler and easier to reason about. (My one little concern is the Method field, where I'm not sure we have - in the Method enum, all HTTP 1.1 methods, but I guess that can be double-checked and fixed if methods are still missing.  Or maybe we can introduce an Unknown method?)
Otherwise - between introducing something like RequestLine and wrapping this whole thing in an Option (but then, would the code become any shorter than it is now?) - and just using default values for http11 and Method (ditto for the status line case) I don't think there are that many options? And if we use default values, would that be a net win? Maybe, not sure.
Open to any other suggestions though.

This might indeed be unsolvable, other than using MaybeUninit, which is unsafe and does not look worthwhile to be used here.
How about this:

Remove Options as in this PR;
Give some default values in new, rather than None;
Let receive to be an instance method (same as the current master branch) to avoid allocation problem.

Did not know this about new vs default and the relationship to const. Really learned a lot :)


Design System - Buttons - Width and Height

The width and height of the buttons should be multiples of 4, with the following heights:

32px (small buttons)

40px (medium buttons)

48px (large buttons)

Thx,
Ismael

@jdrf-digitaldesigner Currently implementing the heights to the buttons. As for the widths, we'd advise against setting fixed widths on the buttons as we can't predict the content of the buttons. To ensure all content will fit inside the buttons without restrictions, let's leave the width defined by the padding and when used in projects, custom styles can overwrite on a case by case basis.
cc: @fuhton

@fuhton
PR submitted:
https://github.com/JDRF/design-system/pull/120

Merged

@jdrf-digitaldesigner Can you review and confirm?

@jdrf-digitaldesigner Can you review and confirm?

Confirmed. Thx!


New repo for Headless/Scheduled Notebooks

There is on-going work on supporting running notebooks as jobs from anywhere Jupyter is running. #957 showcases the concepts, components, and features of this extension. The goal here is to offer this to Jupyter (notebook, lab, hub) users as a new server extension under the jupyter-server org. Please vote here for setting up this repo.
Note: One of the comments in the server meeting was bundling the UI and the server api in a single extension (Currently, we have separate extensions for these), since the UI depends on the server api. We will go ahead with this option and bundle the UI along with the server apis.
cc @Zsailer

The proposed name for the new repo is jupyter-scheduler. @Zsailer Can you help with creating the new repo and adding permissions so I can push to it.

Yes! I'll do that now.
Closing the loop here—this move was officially approved by the Jupyter Server Team in a private vote. Thanks, @3coins, for the great work here!

https://github.com/jupyter-server/jupyter-scheduler
Done. 🚀
Closing this issue here 😎

Great! Thanks @Zsailer


Query: How book is generated?

Query: How book is generated? Could you please help me out. Is it autogenerated?

You can use ctl tool to generate the book. Ctl tool is in the ctl file folder. You need to build this tool first and use pdf command to generate the whole book, which is a markdown version. Finally, you can convert the markdown version to any format you want.


Rechunk array if smallest chunk size is smaller than depth

[x] Tests added / passed
[x] Passes black dask / flake8 dask
[x] closes #6597,  closes #6688

This is ready for review @dask/maintenance

I really like the idea of the dask framework handling the chunking issues under the hood.
One suggestion I may offer, and I could do in a follow-up if desired, is to allow an option for the small chunk to steal from a neighboring chunk.
I ran into this small chunk issue in the past, and tried to combine chunks like this PR.  It worked fine for most cases, but then I ran into an odd edge case.  My operation was using a library that was padding out my chunks dimensions to a power of 2 before loading it onto a GPU.  Using combined chunks, a corner chunk of a 3-d array was formed that was went over the power-of-2 threshold in all dimensions, leading to an 8x increase in size of the padded array.  Needless to day, I hadn't planned for that possibility.

One suggestion I may offer, and I could do in a follow-up if desired, is to allow an option for the small chunk to steal from a neighboring chunk.

I'm not quite sure I understand. Are you suggesting that rather than combining small chunks with neighboring big chunks, we should steal bits of big chunks to add on to small chunks? That seems like a fair amount of extra complexity.

One suggestion I may offer, and I could do in a follow-up if desired, is to allow an option for the small chunk to steal from a neighboring chunk.

I'm not quite sure I understand. Are you suggesting that rather than combining small chunks with neighboring big chunks, we should steal bits of big chunks to add on to small chunks? That seems like a fair amount of extra complexity.

The extra work could be worth it.  If someone has chunked data to a certain size, there could be a could reason.  A rechunking that makes bigger chunks will surprise a lot of people.  It can cause blow past the memory limits of a worker  --  this is exactly what happened to me.

Thanks for the idea @chrisroat! I implemented it and I think it's pretty neat!

Whoa..  and not a lot of additional complexity.  The test_ensure_minimum_chunksize tests are very helpful to see your algorithm in action.

Ok thanks for the second review @mrocklin! I'll merge this when green.


Add feature gate env to packages docs

Closes: https://github.com/aws/eks-anywhere-packages/issues/279
There didn't seem to be any other missing mentions
/cherry-pick release-0.10

/cherry-pick release-0.10


Cache

Laravel Version
10.3.1
PHP Version
8.2
Database Driver & Version
No response
Description
Cache not respecting TTL.
The project was initially done in Laravel 9 and later upgraded to 10.
I just noticed that TTL isn't respected.
I used Cache::remember and set TTL to 30 (seconds).
But the cache seems to remember for ever.
I tested it by going to tinker and setting Cache::put('name','value',10); but even after 20-25 seconds the data is still available when using Cache::get('name').
Note: I am able to clear the cache without issues, so it is definitely not a permission issue.
Steps To Reproduce
php artisan tinker
use Illuminate\Support\Facades\Cache

Cache::put('name','value', 10);

# wait 10-15 seconds

Cache::get('name'); # still returns value

Hey there,
Can you first please try one of the support channels below? If you can actually identify this as a bug, feel free to open up a new issue with a link to the original one and we'll gladly help you out.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks!


Developer guidelines

Browsing the code and wiki, I see that several things are getting inconsistent. In good faith of keeping the project readable and comprehensible for all (current and future) developers, I proposed some development guidelines. You can find them in the repository root.
Please read them, suggest anything you see appropriate (remember, we want to keep it reasonably short).
In case you have no objections, adhere to them when coding and running experiments. Also feel free to correct any inconsistencies you find in the code.

Code revised, tabs replaced with 4 spaces.


python38Packages.pywlroots: 0.14.9 -> 0.14.10

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.pywlroots is: "Python bindings to wlroots using cffi"
meta.homepage for python38Packages.pywlroots is: "https://github.com/flacjacket/pywlroots"
meta.changelog for python38Packages.pywlroots is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.14.10 with grep in /nix/store/nfyjlgppdb2qxd6wnrlir71fiq4i0qsj-python3.8-pywlroots-0.14.10

found 0.14.10 in filename of file in /nix/store/nfyjlgppdb2qxd6wnrlir71fiq4i0qsj-python3.8-pywlroots-0.14.10

Rebuild report (if merged into master) (click to expand)

6 total rebuild path(s)

3 package rebuild(s)

3 x86_64-linux rebuild(s)
3 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
python38Packages.pywlroots
python39Packages.pywlroots
qtile

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/nfyjlgppdb2qxd6wnrlir71fiq4i0qsj-python3.8-pywlroots-0.14.10 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python38Packages.pywlroots https://github.com/r-ryantm/nixpkgs/archive/3472d323d009fec952331ee64ebb36af4d536054.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/nfyjlgppdb2qxd6wnrlir71fiq4i0qsj-python3.8-pywlroots-0.14.10
ls -la /nix/store/nfyjlgppdb2qxd6wnrlir71fiq4i0qsj-python3.8-pywlroots-0.14.10/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

python38Packages.pywlroots
python39Packages.pywlroots
qtile

Maintainer pings
cc @chvp for testing.

Superseded by #146739


http/2 websocket

So i've just updated to the latest version that supports http/2
Would this break web sockets as I'm not getting an error of

failed: WebSocket opening handshake was canceled

and it's the only thing I can think of that would cause this.

I assume that error is coming from the client? Can you confirm from the network debug information in the browser what protocol was used? My understanding is that Websockets do not work over HTTP/2, and I would expect browsers not to try that, but I'm not certain.

I'm using socket.io with the websocket transport on the client.
Is it possible to disable http/2 until I can work it out?

Can you check the router log for any errors? There is not currently a flag, but I can add one today (ideally I'd like to understand the issue too). I'll do some local testing, what browser are you using?

I'm using chrome and just switched back to my old cluster for the time being, but I'll have a look at the logs in the next hour or so.

Ok this is the router log from 1 node on my cluster
root@cluster7:~# 2016/07/05 16:31:08 http: TLS handshake error from <IP_ADDRESS>:62280: EOF
-bash: 2016/07/05: No such file or directory
root@cluster7:~# 2016/07/05 16:34:31 http: TLS handshake error from <IP_ADDRESS>:57390: remote error: unknown certificate authority
-bash: 2016/07/05: No such file or directory
root@cluster7:~# 2016/07/05 16:34:31 http: TLS handshake error from <IP_ADDRESS>:57392: remote error: unknown certificate authority
-bash: 2016/07/05: No such file or directory
root@cluster7:~# 2016/07/05 16:34:40 http2: server: error reading preface from client <IP_ADDRESS>:57399: remote error: unknown certificate authority
-bash: 2016/07/05: No such file or directory
root@cluster7:~# 2016/07/05 16:34:40 http2: server: error reading preface from client <IP_ADDRESS>:57400: remote error: unknown certificate authority
-bash: 2016/07/05: No such file or directory
root@cluster7:~# 2016/07/05 16:34:40 http: TLS handshake error from <IP_ADDRESS>:57401: remote error: unknown certificate authority
-bash: 2016/07/05: No such file or directory
root@cluster7:~# 2016/07/05 16:35:13 http2: server: error reading preface from client <IP_ADDRESS>:57429: remote error: unknown certificate authority
-bash: 2016/07/05: No such file or directory
root@cluster7:~# 2016/07/05 16:35:13 http2: server: error reading preface from client <IP_ADDRESS>:57430: remote error: unknown certificate authority
-bash: 2016/07/05: No such file or directory
root@cluster7:~# 2016/07/05 16:35:13 http2: server: error reading preface from client <IP_ADDRESS>:57428: remote error: unknown certificate authority
-bash: 2016/07/05: No such file or directory
root@cluster7:~# 2016/07/05 16:35:13 http: TLS handshake error from <IP_ADDRESS>:57432: remote error: unknown certificate authority
-bash: 2016/07/05: No such file or directory
root@cluster7:~# 2016/07/05 16:35:13 http2: server: error reading preface from client <IP_ADDRESS>:57431: remote error: unknown certificate authority
-bash: 2016/07/05: No such file or directory
root@cluster7:~# 2016/07/05 16:35:26 http: TLS handshake error from <IP_ADDRESS>:57445: remote error: unknown certificate authority
-bash: 2016/07/05: No such file or directory
root@cluster7:~# 2016/07/05 16:35:26 http: TLS handshake error from <IP_ADDRESS>:57446: remote error: unknown certificate authority
-bash: 2016/07/05: No such file or directory
root@cluster7:~# 2016/07/05 16:35:26 http: TLS handshake error from <IP_ADDRESS>:57448: remote error: unknown certificate authority
-bash: 2016/07/05: No such file or directory
root@cluster7:~# 2016/07/05 16:35:26 http: TLS handshake error from <IP_ADDRESS>:57451: remote error: unknown certificate authority
-bash: 2016/07/05: No such file or directory
root@cluster7:~# 2016/07/05 16:35:31 http: TLS handshake error from <IP_ADDRESS>:52693: EOF
-bash: 2016/07/05: No such file or directory
root@cluster7:~# t=2016-07-05T16:39:45+0000 lvl=eror msg="heartbeat failed" component=discoverd service=router-http addr=<IP_ADDRESS>:80 err="unknown_error: Something went wrong"
root@cluster7:~# t=2016-07-05T16:39:45+0000 lvl=eror msg="heartbeat failed" component=discoverd service=router-http addr=<IP_ADDRESS>:80 err="unknown_error: Something went wrong"
root@cluster7:~# t=2016-07-05T16:39:46+0000 lvl=eror msg="heartbeat failed" component=discoverd service=router-http addr=<IP_ADDRESS>:80 err="unknown_error: Something went wrong"

so there's something not right by the looks of it but then I've just found that I'm seeing a lot of TLS handshake errors on my current cluster which I didn't realise.

You can ignore the handshake errors, those are probably just scanners or other bots that are hitting weird routes. I've got socket.io running on a local cluster and I'm testing it out now.

I've tested the latest socket.io in Firefox and Chrome with the new HTTP/2 code, and the browsers appear to be working properly. Two sockets are created, one for HTTP/2 and one for the HTTP/1.1 websocket upgrade, all communications in the socket.io chat example are going over websockets in both browsers.
Do you have a live example that I can look at? Are you doing any specific configuration of socket.io on the server or client?

I'm using sailsjs so could be something to do with that as it's not a standard socket.io setup, or I'm wondering if it's an ssl issue as I've read that any kind of mismatch would cause it to fail. I have a letencrypt/certbot certificate installed but there didn't seem to be any issue with that when I moved to the new cluster.
The site is https://www.baileyofsheffield.com but it's currently on the old cluster.

There is a known issue with restoring from backups in the most recent version that can cause the TLS certificate/key to be lost, I'd suggest running flynn route update with the cert/key that you want for the route and trying again. This issue will be fixed by #3049 today.

Yep that was it all working now.


Peyton Changes

Sorry it took a minute to look in detail, but the new code, readme, and examples are great! I'm reviewing in more detail now and can work on the visualization part and miscellaneous other components for the final report.


Standalone transpilation

So, I know this plugin is for Rollup, but I could really use the transpilation here standalone. I want to transpile files written in CommonJS to ES Modules, but I don't want any of the bundling to occur. My use cases require runtime resolution of modules using the standard ES Modules syntax and loader. Does anyone have any suggestions? Can this plugin be used standalone? I've gotten into the source code and tried to pull out the functionality that I need, but I ran into some blocking issues with the acorn parser...what would be really awesome is a babel plugin with just the functionality for transpiling CommonJS to ES Modules. Mostly I just want to know if anyone else sees this as a good idea, and for any pointers before I try to do it.

I think I've found a way to do this using Rollup itself programmatically, which will fit my use case: using the experimentalPreserveModules option. It's a very recent addition: https://github.com/rollup/rollup/pull/1922

It would be really nice to have this plugin do standalone transpilation for a single file though, without walking the dependency tree, dealing with the file system, and all of the extra complications that that is adding as I run into weird edge cases

Hey folks (this is a canned reply, but we mean it!). Thanks to everyone who participated in this issue. We're getting ready to move this plugin to a new home at https://github.com/rollup/plugins, and we have to do some spring cleaning of the issues to make that happen. We're going to close this one, but it doesn't mean that it's not still valid. We've got some time yet before the move while we resolve pending Pull Requests, so if this issue is still relevant, please @ me and I'll make sure it gets transferred to the new repo. :beer:


Add PodDisruptionBudgets to the Pinot Helm chart

I've set these by default at allowing 50% of the servers/brokers/controllers to be unavailable, and exposed these settings in the configuration so the user can change them.
These prevent outages when the underlying kuberenetes cluster undergoes an upgrade.
You might configure these like so in your helm yaml settings:
broker:
  replicaCount: 2
  disruptionBudget:
    enabled: true
    minAvailable: 50% # we could also set this to 1
controller:
  replicaCount: 2
  disruptionBudget:
    enabled: true
    minAvailable: 1 # we could also set this to 50%
server:
  replicaCount: 12
  disruptionBudget:
    enabled: true
    minAvailable: 11 # maybe we only have 2 replicas of our data, so we can only afford 1 server to be down at once

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

Project coverage is 0.00%. Comparing base (59551e4) to head (95eb6af).
Report is 443 commits behind head on master.

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #13153       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436     2440        +4
  Lines        133233   134216      +983
  Branches      20636    20785      +149
=============================================
- Hits          82274        0    -82274
- Misses        44911   134216    +89305
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <ø> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <ø> (ø)

java-11
?

java-21
0.00% <ø> (-61.63%)
:arrow_down:

skip-bytebuffers-false
?

skip-bytebuffers-true
0.00% <ø> (-27.73%)
:arrow_down:

temurin
0.00% <ø> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

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

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

Thanks for the contribution.
In general, one comment from K8s before v1.21 support, and to give other users better control over the PDB, can we have a boolean property say enabled to make the PDB optional?

Thanks for the contribution.
In general, one comment from K8s before v1.21 support, and to give other users better control over the PDB, can we have a boolean property say enabled to make the PDB optional?

Sure, done, and since the default replica counts of everything in values.yaml is 1 I've defaulted it to off

cc @xiangfu0 @zhtaoxiang

FYI, the zookeeper chart from bitnami (the one we depend on) also has this ability.

FYI, the zookeeper chart from bitnami (the one we depend on) also has this ability.

I'll adjust this to work with the same config settings as the zookeeper chart then, if people are already familiar with that one

FYI, the zookeeper chart from bitnami (the one we depend on) also has this ability.

I've adjusted this to be closer how to the zookeeper chart works, with pdb as the config name, and exposing both minAvailable and maxUnavailable


[#7] [Chore] [Android] Configure CD for the project

Close #7

What happened 👀

Setup CD workflow for distribution production app to Firebase with kaylabruce-kmmic app
Remove BuildKonfig.kt file and let it be generated by the workflow
Add necessary secret variables ANDROID_RELEASE_KEYSTORE  ANDROID_SIGNING_PROPERTIES  FIREBASE_APP_ID_PRODUCTION  FIREBASE_DISTRIBUTION_CREDENTIAL_FILE_CONTENT GOOGLE_SERVICES_JSON_PRODUCTION GOOGLE_SERVICES_JSON_STAGING

Insight 📝
The application is distributed (the CD workflow is triggered) whenever the release PR is merged into the main branch
Proof Of Work 📹
This POW is collected by changing the trigger branch of the workflow to the current branch for testing

I have tested and resolved your concerns in this commit https://github.com/nimblehq/ic-kmm-kayla-bruce/pull/86/commits/da2faafabdc7e0158ecf9d80a1685d4c174d3728 @nmint8m
Please check it again. Many thanks 👏


[fetch-core] credentials header is always overriden

Issue and Steps to Reproduce
credentials header is not properly passed when using fetchWithToken
Make a call to an API using fetchWithToken with
headers: {
credentials: 'include'
}
Versions
3.0.6
Screenshots
Expected
The value of 'credentials' header of the generated query should be  'include'
Actual
The value of 'credentials' header of the generated query is 'same-origin'

Altough being passed as a header in the query "credentials" setting has to be defined at the top level of fetchWithToken options and not in the "headers" part.


There is the dotted line around the flash player on some web pages.

There is the dotted line around the flash player on some web pages when you click on the Flash. I think it is kind of annoying. Common browser with general flash plugin doesn't show the dotted line.

Firefox shows dotted line outline around windowless plugins. Currently freshwrapper uses windowless mode for all wmode parameter values, while NPAPI Flash switches to windowed mode if wmode is direct and window. Once windowed mode gets implemented, dots won't appear.
(related to #72)

As a workaround, you can disable focus ring completely in Firefox by setting browser.display.focus_ring_on_anything to true and browser.display.focus_ring_width to 0 on about:config page.

With enabling windowed mode and XEmbed support by default in bd52c83c3814c69cf04040bd109770aa96cfeaa2 the behavior is now the same as for NPAPI plugin.
Firefox still shows dotted line if plugin is transparent or opaque, but that's how it's designed to behave.


ci: enable ccache for the clang-3.8 build

Fixes googleapis/google-cloud-cpp#3610

This change is

I swear, I tried this earlier and couldn't get it to work... glad it's working now. Not sure what I did wrong, but thanks.


Undefined variables when updating from v0.8.8 to v0.9.8

Hi guys, I just joined a team and I'm having problems updating from v0.8.8 to v0.9.8, after update I get a bunch of Undefined variable errors like this:

Module build failed:
color: $clr-black,
^
Undefined variable: "$clr-black".
in /node_modules/clarity-ui/src/utils/_variables.clarity.scss (line 81, column 12)

I figured out that all imports in _variables.clarity.scss are now removed. Do I need to do something else in order to make it work?
Thanks for your help.

@adityarb88 : Could you take a look and figure out what is happening?

@efgonz: We reorganized our CSS to avoid multiple imports after 0.8.8. You would have to refer to our main.scss file and make sure that all imports before variables.clarity.scss are imported before it too. Please note that this is not a recommended way to consume Clarity CSS because internal structural changes in the project are not considered & reported as breaking changes.

Closing based on the above comment. Please feel free to reopen the issues if you still face any issues.


[Core] Fix update_usage_summary when response doesn't have usage attribute

Why are these changes needed?

Related issue number

Close #1002 #984
Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Codecov Report
Attention: 10 lines in your changes are missing coverage. Please review.

Comparison is base (083f522) 26.63% compared to head (45a5358) 12.43%.

Files
Patch %
Lines

autogen/oai/client.py
0.00%
10 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1008       +/-   ##
===========================================
- Coverage   26.63%   12.43%   -14.20%
===========================================
  Files          28       28
  Lines        3777     3787       +10
  Branches      858      900       +42
===========================================
- Hits         1006      471      -535
- Misses       2700     3277      +577
+ Partials       71       39       -32

Flag
Coverage Δ

unittests
12.43% <0.00%> (-14.15%)
:arrow_down:

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

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

@kevin666aa  The PR LGTM overeall. Have you tested if this sufficiently address the issue raised in https://github.com/microsoft/autogen/issues/1002 and https://github.com/microsoft/autogen/issues/984. Can they be closed with this PR? Is it possible to add a test here?

@kevin666aa The PR LGTM overeall. Have you tested if this sufficiently address the issue raised in #1002 and #984. Can they be closed with this PR? Is it possible to add a test here?

I cannot replicate the error, so I can't put down a test for this. The bug is happening because response.usage may be a nonetype due to the use of a local model. The code I added in the  try catch block  would make sure that response.usage can be referenced, and it is not none type. Further checking of response.prompt_tokens is also added. So I think it would be sure to resolve the two issues.

@qingyun-wu People from  #1002 has confirmed this PR will solve the problem. I think we can merge this PR.

Could you fix the conflicts? @kevin666aa Thank you!

openai 1.5 support has been added by #1044 + #1071
There is no need to limit the openai version in this PR.

We can merge this once the conflict has been resolved. @kevin666aa

@kevin666aa failing tests.

@kevin666aa failing tests.

@kevin666aa failing tests.

Hello @ekzhu, can you help me run the openai test again?

#1110 needs to be merged first to prevent test failures.


Add tripleo_controllers tripleo_computes for DPA ci zuul nodeset

As part of [1] this add the tripleo_controllers tripleo_computes
groups to the zuul nodeset, to be used by the data plane adoption
ci playbooks in Related below. It also removes a redundant nodeset
no longer used by any job.
[1] https://issues.redhat.com/browse/OSPRH-8291
Related: https://review.rdoproject.org/r/c/rdo-jobs/+/53688
As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running
[x] Appropriate documentation exists and/or is up-to-date:

[x] README in the role
[x] Content of the docs/source is reflecting the changes

looks good, but we should add the groups for https://github.com/openstack-k8s-operators/ci-framework/pull/2018/files#diff-f3199dc676bf2b8d1ae52a4513c184798c9e9e9981404d6fe6d189097d35604cR66 as well, otherwise when changing the rdo-jobs playbook we'll break the WIP using that nodeset

ack good call can do thanks for reviewing

/recheck

recheck

/lgtm

/approve

recheck


Property:country

Definition

Dimension

Existing property

Existing class

Existing property identifier

Format
Text

Required

Repeatable
Single

Constraints

Examples

Notes

Proposed definition:

Definition
The country name associated with the address of the institution

Dimension

Existing property
country name

Existing class

Existing property identifier
http://www.w3.org/2006/vcard/ns#country-name

Format
String

Required

Repeatable
Single

Constraints

Examples

Notes

Can't we use ISO country codes https://en.wikipedia.org/wiki/ISO_3166-1_alpha-2

@nielsraes this is one of the address properties that we are going to defer to vcard properties such as http://www.w3.org/2006/vcard/ns#country-name


Wire amount formatting; winning bidder fix; media gallery fixes

git cherry-pick'd a bucket-list of items. Need to verify some of the image stuff in the morning, seems some commit competition that's direction rather than syntax; so I'll check in on that & ping here when GTG.

I think GTG. @afraser wanna take a look? BTW merge blocked pulling from lefnire/algomart, I don't have perms to this repo to create branch here


Missing parent to GO:0030544 Hsp70 protein binding?

Hi,
Should
GO:0030544 Hsp70 protein binding
not have the parent
GO:0051087 chaperone binding
?
Thanks

A question for @hdrabkin . Would the link always hold true?
(Let's leave aside the discussion of how the target - bound - protein should be represented in GO...)

No sure why I should know but my gut feeling is that the chaperone binding term should NOT even be in GO. The term chaperone is a description of what a variety of structurally unrelated proteins do (aid in folding, etc.). I vote get rid of GO:0051087

I think this term is meant to use with "co-chaperones", proteins that assist chaperones
https://en.wikipedia.org/wiki/Co-chaperone like J domain and  GroES
At one point these terms fell out of favour with GO in place of 'protein binding' but for LEGO  where  you need to connect by function, these are quite useful to describe the function of a gene product, if correctly defined but maybe should be restricted for GO annotation purposes to the role of a co-chaperone in assisting in protein folding..... It's also useful to have an MF term for these, because it is a biological role that people will search for and understand.
Had a few conversations with @ukemi and @pdthomas  wrt LEGO about this so they may have input.
If it goes, thats fine, but if we keep it "co-chaperone" is probably a better primary name.

To me it's like  enzyme binding. It implies a relatedness that does not exist structurally. Thus if something has enzyme binding I assume what is meant that it binds a specific protein that happens to have an enzymatic function. But what activity is that? If known, it would be annotated to that function as a separate annotation.
The protein-protein interaction that results int he binding may be located nowhere near an active site, etc..
The term "binding" implies a recognition of a structure (and thus binding to it), not a binding to an activity

Yeah I saw this on a co-chaperone gene page (SPAC144.08), and thought it looked like they should be a parent/child

The chaperone discussion relates to https://github.com/geneontology/go-ontology/issues/12252
Will arrange discussion with @ValWood and @ukemi in June.
I'll set to pending for now, thanks.

Note for self: Val suggested: "Let's leave this until later. The chaperone stuff might need broader discussion. Also I think the chaperones might have slightly different functional roles. I might be able to add some of these to the tickets based on some discussion with Antonia".

Hi @ValWood ,
You mentioned that this ticket is not urgent, so I'll assign it to you. Perhaps some of the chaperone-relevant issues could connect to other existing tickets e.g. https://github.com/geneontology/go-ontology/issues/13021. Thanks.

@ValWood I dont think all HSP are chaperones:
https://en.wikipedia.org/wiki/Heat_shock_protein#Role_as_chaperone
I would close this without action - let me know if you don't agree.
Thanks, Pascale

Could do. But I think the main problem here is that we can describe the same thing in 2  different ways, so bad for annotation consistency.
The same MF can be described as  hsp70 binding, or as chaperone binding. The meaning is identical (hsp70 are always chaperones).
I don't know why we need "hsp70 binding", it could merge into "chaperone binding" ? It only has 90 EXP annotations and the vast majority have "with" fields so no loss of info, but greater consistency.
Then I don't know why we have "heat shock protein binding", this one is too general to be useful because it does not apply to a single function, or family. Again most annotations have with fields....

hsp70 binding has two siblings:

GO:0051008 Hsp27 protein binding
GO:0051879 Hsp90 protein binding

This seems to be consistent with our practice to have 'protein class' binding- isn't it ?

I thought the longstanding idea was to get rid of protein class binding eventually if possible ?

Hsp90 protein binding
"Interacting selectively and non-covalently with Hsp90 proteins, any of a group of heat shock proteins around 90kDa in size."
87 manual annotations
these are hsp90 superfamily in pombe:
https://www.pombase.org/gene_subset/interpro:IPR036890
I'm not sure how this term is useful if used consistently for all hsp90-class

hsp27 binding has no annotations.
I would recommend to obsolete the unused one and merge the others   into protein binding and then only a small number without a "with" field would need to be fixed.
Most could be classed better as ATPAse or chaperone or co-chaperone (aka co-chaperone activity)
It's inconsistent - but it isn't critical.

FYI the UCL group has used the HSP90 terms to create 29 annnotations for 23 proteins. if a decision is taken to remove the HSP binding terms I would favour re-annotation of these to change them to chaperone binding or other function binding term. Information will be lost by just merging these with protein binding (around half of the annotations have no ID in the with field).
Ruth

@RLovering I would have merged to the parent 'heat shock protein binding' - would that be OK for everone ? @ValWood ?

 I would like that one to go too.
Again, impossible to annotate consistently (proof, only 346 EXP annotations).
 There definition within GO is unclear. Is it only a protein expressed in response to heat-shock, or is it any member of a family  which is related to a heat-shock protein.
Most annotations have "with" fields.
Personally, I would merge all into protein binding. This would make very little reannotation, but annotations without 'with fields' would need to be evaluated.
Or, obsoletion if people want to consider better concurrent annotations like "chaperone" "co-chaperone" "ATPase" etc.....
If we went this route, this isn't urgent so it wouldn't be a big deal if it took a while, to give people chance to deal reannotate.
In these cases it would be nice to be able to implement an "annotation block" so that no new annotations are added to a term which is "on the way out'....e.xistign annotations would still be valid, but we would not accumulate new ones.

+1

The 'heat shock protein binding'  merge would be fine with me, but I take the point that it would be better to have annotated to chaperone binding. I would be happy to start reannotation in this way but my main worry about re-annotation is that I am not convinced that the revised annotation information would be kept. It would be annoying to add annotations to chaperone binding and then find that this term is removed from the ontology. Personally I would rather people were encourage to capture PPIs with multiple terms: ie chaperone binding AND heat shock protein binding, rather than trying to force consistency by limiting term options. Tools can be developed to suggest terms that are often co-annotated to improve consistency. But I know that this is not the way we are heading.

Personally I would rather people were encourage to capture PPIs with multiple terms: ie chaperone binding AND heat shock protein binding, rather than trying to force consistency by limiting term options.

Co-annotation is fine to represent different activities, but it is really confusing for end-users to have  the same activity described in multiple ways. It also makes CAM-modelling impractical because  in this scenario the annotation would necessarily be interpreted as two different activities.
Maybe it is better to leave this for the time being. Chaperone binding has the same issues because again it is a heterogenous collection of proteins. Co chaperones bind to a chaperone and not the client. Includes DNAJ, and TPR for example. TPR are usually scaffold proteins. So co-chaperone or chaperone binding don't really  say anything about the molecular function.
However, the problem Antonia identified above is that some curators choose "chaperone binding" and some choose "hsp protein binding". This must be bad, and it wouldn't be practical (or useful in the long term) to tell people that they must use both (think transcription factors).

I'm no expert on HSPs, but are they structurally related? I think the name is based on a collective phenotype or function and not a structural family? are there more than one HSP70, 27, or 90? If not then the go term represents a single gene product yes? Should remove is yes?

Hi Harold
based on wikipedia:
https://en.wikipedia.org/wiki/Hsp70 structually conserved
https://en.wikipedia.org/wiki/Hsp27 Hsp27 is a member of the sHSP functional group; sHsps have some structural features in common:
https://en.wikipedia.org/wiki/Heat_shock_protein The principal heat-shock proteins that have chaperone activity belong to five conserved classes: HSP33, HSP60, HSP70, HSP90, HSP100, and the small heat-shock proteins (sHSPs).
does that help?
Ruth

So the articles don't actually address whether in any one organism, there can be multiple forms of 33, 60, 70, etc.?
To me conserved class means across different organisms, so when one is annotating for one organisms's gene, you would know it. Now in mouse, Hsp70 is a synonym for several different genes (Hspa1b, Hspa1b, Hspa4, , etc. (not sure if they are related other than they are heat-shock proteins). But if I would annotate the gene, I would know which one I am annotating for mouse.

Also, changed label 'chaperone binding' to 'protein folding chaperone binding' to match the term definition.


Why doesn't it retweet everything corresponding to the searchQueries?

Hello,
I've used your bot for the last couple days and noticed it only retweetd and followed a few dozens contest tweets a day.
If I do a manual search like https://twitter.com/search?q=rt win&src=typd
I find hundreds of contests that my bot hasn't retweeted and that are much more interesting than the ones it has retweeted.
Do you know why? and how to fix that?
Thanks!

Probably because you only searched for "rt win".
There are multiple parameters used now by the bot, mainly because it was spamming too much. We've filtered out retweets for starters. Secondly, it dismisses tweets that contain the word "vote". Third, it looks for both popular tweets and recent ones.
Also, as an improvement by someone in the community who used the app, there is a parameter for "minimum retweets needed by a tweet" in order for us to care about it (currently 10). This significantly reduces the amount of fake contests retweeted and stops retweeting other bots that retweet retweets of other bots.
Another aspect is the max. tweets the original poster has. If it has more than 20000, it's likely to be a bot so we dismiss it.
There are all configurable, though, You will find them at the top of the index.js file as constants, so you can play around with them to change outcomes of the bot.


Unable to echo in job_start's lambda callback function

Description
echo and echomsg do not display anything when invoked in the following fashion:
function! JobStartExample()
  let s:job = job_start(
    \'cmd /C dir',
    \{
      \'exit_cb':  { channel,status -> execute('echo "foo"') }
    \}
  \)
endfunction

Notes

With the same code, echoerr correctly displays foo
Passing the name of a function instead of a lambda makes echo/echomsg display fool correctly, ie:

function! JobExitCbExample(channel, status)
  echo "foo"
endfunction

function! JobStartExample()
  let s:job = job_start(
    \'cmd /C dir',
    \{
      \'exit_cb':  'JobExitCbExample'
    \}
  \)
endfunction

Expected Behaviour
I would have expected the same behaviour when using a lambda or when using a function name, ie displaying foo
Environment

on Windows 10 (gvim)
version: VIM - Vi IMproved 8.2 (2019 Dec 12, compiled Jul  4 2020 22:02:18), MS-Windows 64-bit GUI version with OLE support, Included patches: 1-1127

Try execute('echo "foo"', '').
See :help execute() for detail.

Thank you very much!

Note, that depending on your ewnvironment, you may not see the result. It is usually better to use :echomsg ...


Safari failed to render PDF

On safari (iPad IOS 8 / 8.2 / 8.3) PDF rendering stuck and spinner  keep displaying.
I am using PDF.js: 1.0.1040.
Getting exception: InvalidStateError: DOM Exception 11: An attempt was made to use an object that is not, or is no longer, usable. (pdf.js:6056)
Note that when i change zoom option to 50% or less then its able to display the pdf but when i again increase the zoom value from 50% to more its start throwing the above exception and spinner keeps moving.

Possibly a duplicate of #5839, but we need the PDF file that you are working with for verification. Also, try using the latest version of PDF.js, i.e., at least 1.1.114.

Yes, i did try with 1.1.114 version but same error.
Here the the sample pdf that will produce the error: http://pdf.site88.net/sample.pdf
Can we get some workaround until its get fixed in next pdfjs release ?

Desarrolladores y líderes de Mozilla, que son travestis culo!
Alguien debe colgar ustedes para arriba en sus propias bolas y dejar que su perro muerda su caricatura de una polla!
Posteriormente deben ponerlo tiempo que sea necesario por la garganta hasta que traga a sí mismos en el suicidio!
Ustedes Mozilla está ... Estúpido de mierda coños!
Developers and leaders of Mozilla, which are shemales ass!

They should then put it long as needed by the throat until he swallows themselves in suicide!
You Mozilla is...

Hi Guys, any update on this ? Its just stopping me to proceed further.
Any help will be highly appreciated.

Hi Guys, any update on this ? Its just stopping me to proceed further.
Any help will be highly appreciated.

Works fine on Safari 8.0.6 to Mac OSX. Could you also open issue with the vendor of iPad IOS 8?

Yes i am facing this issue only on iPad with OS version 8.1/8.2/8.3.
On Mac OS its work. The interesting thing is its work when i change the zoom option to 50% or below.
Not sure what is happening when zoom goes above 50%

I can confirm this issue on Safari 9 OSX 10.11 on longer page PDF's. If I zoom in greater than 100% it will render the entire PDF correctly. Tested on v1.1.469 and v1.2.109.
InvalidStateError: DOM Exception 11: An attempt was made to use an object that is not, or is no longer, usable.
drawImagepdf.js:5071
CanvasGraphics_endGrouppdf.js:5071
CanvasGraphics_executeOperatorListpdf.js:4066
InternalRenderTask__nextpdf.js:3218
InternalRenderTask__next

@adoming3 can you also attach steps to reproduce?

InvalidStateError: DOM Exception 11: An attempt was made to use an object that is not, or is no longer, usable.

There's a change that this particular error has been fixed by PR https://github.com/mozilla/pdf.js/pull/6551#issue-112704439, given the following:

I noticed that CachedCanvases are broken (global cache can be cleared in the middle of other page rendering) -- added local caches instead.

Based on reading this issue, it seems to me that it should have been fixed by PR #6551.
@Saket-Ranjan, @adoming3 Is this still an issue, using the latest version of PDF.js?

Closing as resolved until further information as requested above is provided.