Extension spawns many copies of kubectl proxy on startup

When I open up VS Code for the first time and open a Kubernetes YAML file, the Kubernetes extension seems to be spawning a lot of instances of kubectl proxy --port=0 for every Kubernetes API object in the file.
For instance, opening this file (one of cert-manager's resources), results in spawning 28 copies of kubectl:

cluster-role.yaml
# Challenges controller role
apiVersion: rbac.authorization.k8s.io/v1beta1
kind: ClusterRole
metadata:
  name: cert-manager-controller-challenges
  labels:
    app: cert-manager
    app.kubernetes.io/name: cert-manager
    app.kubernetes.io/instance: cert-manager
    app.kubernetes.io/managed-by: Tiller
    helm.sh/chart: cert-manager-v0.10.0
rules:
  # Use to update challenge resource status
  - apiGroups: ['certmanager.k8s.io']
    resources: ['challenges', 'challenges/status']
    verbs: ['update']
  # Used to watch challenges, issuer and clusterissuer resources
  - apiGroups: ['certmanager.k8s.io']
    resources: ['challenges', 'issuers', 'clusterissuers']
    verbs: ['get', 'list', 'watch']
  # Need to be able to retrieve ACME account private key to complete challenges
  - apiGroups: ['']
    resources: ['secrets']
    verbs: ['get', 'list', 'watch']
  # Used to create events
  - apiGroups: ['']
    resources: ['events']
    verbs: ['create', 'patch']
  # HTTP01 rules
  - apiGroups: ['']
    resources: ['pods', 'services']
    verbs: ['get', 'list', 'watch', 'create', 'delete']
  - apiGroups: ['extensions']
    resources: ['ingresses']
    verbs: ['get', 'list', 'watch', 'create', 'delete', 'update']
  # We require these rules to support users with the OwnerReferencesPermissionEnforcement
  # admission controller enabled:
  # https://kubernetes.io/docs/reference/access-authn-authz/admission-controllers/#ownerreferencespermissionenforcement
  - apiGroups: ['certmanager.k8s.io']
    resources: ['challenges/finalizers']
    verbs: ['update']
  # DNS01 rules (duplicated above)
  - apiGroups: ['']
    resources: ['secrets']
    verbs: ['get', 'list', 'watch']

$ ps aux | grep kubectl
jacobwgillespie  43516   0.0  0.2  4457000  26244   ??  S     1:25pm   0:00.10 kubectl proxy --port=0
jacobwgillespie  43515   0.0  0.2  4456488  26712   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43514   0.0  0.2  4458572  28016   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43512   0.0  0.2  4449332  28076   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43507   0.0  0.2  4455720  27448   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43506   0.0  0.2  4459352  27796   ??  S     1:25pm   0:00.12 kubectl proxy --port=0
jacobwgillespie  43505   0.0  0.2  4449332  27660   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43504   0.0  0.2  4454952  26996   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43503   0.0  0.2  4447540  27616   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43489   0.0  0.2  4464436  26236   ??  S     1:25pm   0:00.12 kubectl proxy --port=0
jacobwgillespie  43486   0.0  0.2  4456744  26544   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43484   0.0  0.2  4449880  26324   ??  S     1:25pm   0:00.12 kubectl proxy --port=0
jacobwgillespie  43481   0.0  0.2  4448052  27400   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43479   0.0  0.2  4457000  27504   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43476   0.0  0.2  4474176  26348   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43475   0.0  0.2  4459096  26452   ??  S     1:25pm   0:00.12 kubectl proxy --port=0
jacobwgillespie  43474   0.0  0.2  4449064  26500   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43473   0.0  0.2  4448284  26756   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43472   0.0  0.2  4439068  26340   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43470   0.0  0.2  4440908  26648   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43469   0.0  0.2  4449600  26896   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43468   0.0  0.2  4460656  26696   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43524   0.0  0.2  4442444  26692   ??  S     1:25pm   0:00.12 kubectl proxy --port=0
jacobwgillespie  43523   0.0  0.2  4439860  27396   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43521   0.0  0.2  4467544  26432   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43520   0.0  0.2  4449100  26244   ??  S     1:25pm   0:00.12 kubectl proxy --port=0
jacobwgillespie  43519   0.0  0.2  4449856  27456   ??  S     1:25pm   0:00.11 kubectl proxy --port=0
jacobwgillespie  43517   0.0  0.2  4440104  25984   ??  S     1:25pm   0:00.11 kubectl proxy --port=0

This appears to be related to the number of resources inside the file, so for instance opening cert-manager.yaml from https://github.com/jetstack/cert-manager/releases/download/v0.10.0/cert-manager.yaml, with all resources included, attempts to launch over 430 instances of kubectl proxy --port=0 before my computer stops responding.

Note that this happens when opening a file for the first time, so in terms of reproduction:

Open VS Code in a blank state where it doesn't have any open files
Everything is okay
Open cert-manager.yaml
100s of kubectl proxy --port=0 processes appear

I have not dug far into the code, but I would assume that this is related to #583 that loads Swagger API definitions from the live cluster.  I would have expected that code to ensure that only one process is fetching the schema at a time, but perhaps that's missing or something's preventing the cache / mutex from working properly.

Sorry for the delay in replying to this - I've been away.  I think your diagnosis is correct - this is trying to retrieve the Swagger for each resource, and not taking account of the fact that a Swagger download for the cluster has already been kicked off.  I will take a look asap.

Hmm, I'm not seeing the multiple spawn behaviour with cluster-role.yaml.  On my system it spawns kubectl proxy --port=0 once, and this terminates correctly.  I also tried a YAML file with multiple subdocuments and that seemed to act correctly too.
I'm on Windows, so it's possible this is an OS-specific issue.  I'll keep digging.

Reproduced it.  When the extension knows the active context before a document causes it to request the schema, we attempt schema download multiple times simultaneously (and much more often than is necessary).  I wasn't seeing it because in my environment it didn't get the active context quickly enough to trigger the problem, so it was doing all those hundreds of loads from the fallback Swagger bundled with the extension; then when it got round to retrieving the Swagger from the cluster, the storm of schema lookups had abated.
I will get onto a fix.  Thanks for identifying this, and special thanks for the detailed repro steps!


dask-sql escape character

column name in ("hello's world") is throwing error . what is the escape character in dask-sql??

Hi @adityakumar1990! I hope the answer above helped. I will close this issue now, but that does not mean you can not ask more questions! Feel free to ping me or re-open this issue.


Sidebar will only display one badge at a time

First off, thank you for you work on this! We were hoping to use this addon to help developers know what parts of the stack are used in each component and help them find good implementation examples to learn with.
After trying to fiddle a while with the configuration I finally realized that the Sidebar component will only show the first badge. This kind of defeat the purpose of what we were hoping to achieve with tags and feels like an arbitrary limitation. Would it be possible to make it configurable and allow multiple tags in the sidebar too?
Happy to help with a PR if needed!

Hey there!
I'm surprised this issue didn't get opened sooner :) It is an opinionated limitation indeed. Storybook use the sidebar to display other indicators, and I'm reserving space on the right and limiting myself to one badge to limit the footprint of the addon in the sidebar. As some of the UI was designed before Storybook Test came out, I'm open to reworks on spacing.
Regarding multiple badges, I'd like to see UI proposals with worst-case scenario data before changing the addon, to make sure people don't shoot themselves in the foot with it. Open to expand/collapse UI proposals too. I'll likely ask Storybook maintainers for an opinion before committing to something.
I'm also open to adding an order badge property to allow some level of dynamic reordering of badges so that the most important badge gets displayed.

As for your workflow, could you give tag filters a try? There's a rudimentary filter UI in the sidebar. There are plans to extend filters to URL query params too, and I plan on improving that UI (later! in a few months, not soon!). I'd be curious to know if it can help your team find what they need.

Thanks for the quick response!
That all makes sense and I figured it must have been the reason. I understand the concern of a cluttered UI but if it's not the default behaviour would it really be a problem? Maybe something like a maxBadgesDisplayed configuration combined with an order property on the badges?

I'm not completely opposed to it but it does open a pandora's box. I'm worried people will start asking about layout options (which is hard to do with SB8 as no CSS variables are exposed to outside code) or per-story ordering once they realise that multiple badges break the UI more frequently.
I don't just want a PR that adds a config key and ignores the side effects, because the side effects will become extra issues with extra work to do after the fact :smile: But I'll review in good faith any UI proposal that shows how worst-case behaviour is handled, and I'll accept a PR that handles worst cases elegantly.

The filters will do the work yes, it's just that it doesn't really offer the same at-a-glance feel that the sidebar does and also they don't have any kind of styling and will be hard to differentiate if we add a lot of tags.

We'll work on the filter UI and there will, at some point, be a tag grouping mechanism (using : as a separator). Though I appreciate you need a solution now, and I won't work on the UI now, but in a few months.

Another idea for this problem could be to have some kind of computed badges, so right now you can make it so a badge apply if one of the tags included in its configuration is present. But creating a way to make it appear only if all tags are included on the component could allow to have a single badge that cover multiple tags.

I'd love to discuss that (in a separate ticket ideally)! This could work by extending the tags config key to take a function or a data structure that allows and/or.
What I'm reading is that you want to highlight components that match several conditions? And you wanted to display several badges so that those components with several conditions are immediately visible because they have more badges than the others?
I'd love to see exactly what your use case is, by the way. It helps me to know how the addon will be used when making assumptions about usage.

So, here is what I imagine your ideal config would be

Nnote that I don't recommend using display, the default with inheritance is what most users need
I'm applying a grouping of tags with a uses: prefix, as tag grouping will eventually land in the SB UI and it will benefit you to have adopted it, plus it helps make the tags more self-explanatory to newcomers
And on top of that, you would need multiple sidebar badges for it to make the most sense
But then, in the sidebar, you'd likely want to display the single "uses" badges to appear when "Good example" doesn't as they'd be redundant; so you'd need dynamic display configuration too...

function hasAllRecommendedTags(tags) {
  return tags.includes('uses:tada')
    && tags.includes('uses:graphql')
    && tags.includes('uses:typescript')
    && tags.includes('uses:tailwind');
}

tagBadges: [
  {
      tags: (tags) => tags.includes('uses:tada'),
      badge: {
          text: '🪄',
          // ...
      },
  },
  {
      tags: (tags) => tags.includes('uses:graphql'),
      badge: {
          text: 'GQL',
          // ...
      },
  },
  {
      tags: (tags) => tags.includes('uses:typescript'),
      badge: {
          text: 'TS',
          // ...
      },
  },
  {
      tags: (tags) => tags.includes('uses:tailwind'),
      badge: {
          text: 'TW',
          // ...
      },
  },
  {
      tags: hasAllRecommendedTags,
      badge: {
          text: 'Good Example',
          tooltip: 'This component/story uses all the recommended features and can be used as an example for new components.',
      },
  },
],

Alternatively, if I didn't go ahead with multiple sidebar badges, you would likely want to display the single "uses" badges only on the toolbar.
function hasAllRecommendedTags(tags) {
  return tags.includes('uses:tada')
    && tags.includes('uses:graphql')
    && tags.includes('uses:typescript')
    && tags.includes('uses:tailwind');
}

tagBadges: [
  {
      tags: 'uses:tada',
      badge: {
          text: '🪄',
          // ...
      },
      display: {
        sidebar: false,
        toolbar: true,
      },
  },
  {
      tags: 'uses:graphql',
      badge: {
          text: 'GQL',
          // ...
      },
      display: {
        sidebar: false,
        toolbar: true,
      },
  },
  {
      tags: 'uses:typescript',
      badge: {
          text: 'TS',
          // ...
      },
      display: {
        sidebar: false,
        toolbar: true,
      },
  },
  {
      tags: 'uses:tailwind',
      badge: {
          text: 'TW',
          // ...
      },
      display: {
        sidebar: false,
        toolbar: true,
      },
  },
  {
      tags: hasAllRecommendedTags,
      badge: {
          text: 'Good Example',
          tooltip: 'This component/story uses all the recommended features and can be used as an example for new components.',
      },
      // default display.
  },
],

I need to take a moment and

[ ] Assess what data is always available for dynamic tags config
[ ] Assess what data is always available for dynamic display config
[ ] Develop relevant dynamic config keys with unit tests

I'll still wait for more thorough worst-case-scenario UI proposals for sidebar badges and wait for others to provide feedback before moving on that.

Thank you so much for taking the time to analyze this! I think they're both good solutions for us. I'll change the tags to use groups too.
Do you know if there's a Figma template for the Storybook design system? I could give you a worst case scenario proposal for the sidebar.

You could come ask for that on Discord (should be a dedicated channel, else ping Kyle or Charles), but I'm not sure there is.


close the popen stream

l think we should close the popen stream before function exits

 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.


MessageGenerator should be able to generate unicode strings

Most likely based on a locale or similar.

Yoiu can now control the jfairy locale of the generator.


your pretrained google driver does not have .ckpt file

your pretrained google driver does not have .ckpt file

you can check my repo here for a Pytorch implementation if you want


Add npm weekly and npm yearly downloads badge

closes #939

@Daniel15 Would you like the honors? Typically I'd squash something like this through the UI, reworking the commit message as needed. Since there are no automated tests of the badges, you should do some of your own testing as well. A minimal version could be filtering try.html to make sure the badges are generating right.

@paulmelnikow  Sure, I can do that tomorrow or Friday! I want to test it out thoroughly before merging 😃

Tested locally and confirmed it all works as expected!


org.iu.edu.tr added

Istanbul University Official Website
Faculty of Computer and Information Technologies Website
recognition of the student email domain

@emrekabalcii I am sorry, the domain ogr.iu.edu.tr is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Removed depth test for background

The z-position caused it to fail the depth test on certain gpus due to
different floating point accuracies.
Also removed old pidgeot file and added a square background
@alexwalterbos RFR

:+1:


please accept my commit. All test work.

tommorow will continue with more fixture to be more native. I eddit all no native meanings and merge some words , but sure need more .

I'll want to put on hold this PR until I preview it and test it as me and other translators were having difficult times explaining bulgarian grammar and spelling to @BossNeo.

I prefer to leave this here until other bulgarian colaborator[s] say something or someone else reject this PR.


win11 PRO ver. 10.0.22000.469 does not work even with the .ini patch

I can't have multiple users login at the same time.
Applied the patch here, it did not make any difference.
https://github.com/stascorp/rdpwrap/issues/1707
With the patch, it reads not listening
W/o the patch , it seems fine but it does not really work

I'm experiencing the same issue.  Listener fully supported and connection does work, but does not allow multiple connections.  RDP connection always kicks out local user.  Same is happening with sebaxakerhtc's mod.

Duplicate of https://github.com/stascorp/rdpwrap/issues/1698

@sebaxakerhtc Please reopen this issue. As @proactivepcc also stated, it is not fixed nor a duplicate of the other one.

@sebaxakerhtc Please reopen this issue. As @proactivepcc also stated, it is not fixed nor a duplicate of the other one.

1 version of termsrv.dll = 1 issue
https://github.com/stascorp/rdpwrap/issues/1698#issuecomment-1030679727

@sebaxakerhtc Please reopen this issue. As @proactivepcc also stated, it is not fixed nor a duplicate of the other one.


rsync_src is not specify to working dir, while using yml recipe

Deployer version: 7.0.0-rc.8
Deployment OS: Ubuntu 20.04

From documentation:
set('rsync_src', __DIR__);

If I dont define rsync_src in deploy.yml i got the following output:
[localhost] run rsync -rz -e 'ssh '-p' '522' '-A' '-o' 'ControlMaster=auto' '-o' 'ControlPersist=60' '-o' 'ControlPath=/dev/shm/deployer@example.com:522'' --delete --exclude='.git' --exclude='deploy.yml' --exclude='.gitlab-ci.yml' 'phar:///usr/local/bin/dep/contrib/' 'deployer@example.com:/srv/example.com/releases/1/'
[localhost] The source and destination cannot both be remote.
[localhost] rsync error: syntax or usage error (code 1) at main.c(1420) [Receiver=3.2.3]

While i define it with __DIR__ or  "__DIR__" got following output:
  rsync_src: __DIR__

[localhost] run rsync -rz -e 'ssh '-p' '522' '-A' '-o' 'ControlMaster=auto' '-o' 'ControlPersist=60' '-o'<EMAIL_ADDRESS>--delete --exclude='.git' --exclude='deploy.yml' --exclude='.gitlab-ci.yml' '__DIR__/' 'deployer@example.com:/srv/example.com/releases/1/'
[localhost] rsync: [sender] change_dir "/builds/deployer/test-deploy-project/__DIR__" failed: No such file or directory (2)
[localhost] rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1330) [sender=3.2.3]

  rsync_src: "__DIR__"

[localhost] run rsync -rz -e 'ssh '-p' '522' '-A' '-o' 'ControlMaster=auto' '-o' 'ControlPersist=60' '-o'<EMAIL_ADDRESS>--delete --exclude='.git' --exclude='deploy.yml' --exclude='.gitlab-ci.yml' '__DIR__/' 'deployer@example.com:/srv/example.com/releases/1/'
[localhost] rsync: [sender] change_dir "/builds/deployer/test-deploy-project/__DIR__" failed: No such file or directory (2)
[localhost] rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1330) [sender=3.2.3]

deploy.yml
import:
  - contrib/rsync.php

hosts:
  stage:
    hostname: "example.com"

config:
  rsync:
    exclude:
      - .git
      - deploy.yml
      - .gitlab-ci.yml
    exclude-file: false
    include: [ ]
    include-file: false
    filter: [ ]
    filter-file: false
    filter-perdir: false
    flags: rz
    options:
      - delete
    timeout: 30

tasks:
  deploy:
    - rsync

My fix is:
  rsync_src: './'
or
  rsync_src: '.'

With that output all working fine:
run rsync -rz -e 'ssh '-p' '522' '-A' '-o' 'ControlMaster=auto' '-o' 'ControlPersist=60' '-o'<EMAIL_ADDRESS>--delete --exclude='.git' --exclude='deploy.yml' --exclude='.gitlab-ci.yml' './/' 'deployer@example.com:/srv/example.com/releases/20/'

So, I'm not sure if this problem is related to the missing source!
If you look very closely at the command you will notice that there are quite a few single quotes.
The argument:
-e 'ssh '-p' '522' '-A' '-o' 'ControlMaster=auto' '-o' 'ControlPersist=60' '-o'<EMAIL_ADDRESS>
is actually an argument that belongs to rsync and tells rsync to establish an SSH connection with the arguments behind it. So everything starting with 'ssh' is part of the SSH command and is also enclosed at the end with another single quote. That's why there are two at the end!
I have the same problem with the rsync contrib. I can't establish a connection either. But from my point of view it is rather the function "escapeshellarg" which encloses all SSH arguments in the Host class with single quotes. Then in the rsync contrib more single quotes were used to enclose the complete SSH command.
src/Host/Host.php:
    public function connectionOptionsString(): string
    {
        return implode(' ', array_map('escapeshellarg', $this->connectionOptionsArray()));
    }

conrtib/rsync.php:
    $sshArguments = $host->connectionOptionsString();
    runLocally("rsync -{$config['flags']} -e 'ssh $sshArguments' {{rsync_options}}{{rsync_includes}}{{rsync_excludes}}{{rsync_filter}} '$src/' '{$host->connectionString()}:$dst/'", $config);

The release documentation for the v7.0.0-rc.8 release states:

●  Refactored ssh client and host connection option.

So this part is obviously new and doesn't work like that!

And maybe as a side note, __DIR__ is a predefined constant of php and it is not known in the yaml cosmos!
But the variable rsync_src is already set to this constant by default in the rsync contrib:
conrtib/rsync.php:
set('rsync_src', __DIR__);

Oh man, so if you look with your eyes....
Then you will see that it is not because of the strange single quotation marks.
You were right! It is indeed because of the source rsync_src. Which also makes sense, because if you leave the variable set in the contrib the default is not in the deploy.yaml directory (which I assumed) but of course in the directory of the rsync post.
If you then install the deployer.phar globally like I did, a totally confusing source is created:
phar:///usr/local/bin/dep/contrib/

and the delpoyment crashes!
So, you definitely need to set the variable yourself and __DIR__ is not an alternative in the yaml file!
You can import a php file and create the variable rsync_src there or you have to live with your solution. As said, the predefined php constant __DIR__ represents the current directory where your source code is executed and is only available in php!!!
If it would be possible for the maintainer of the project to save the path to the deploy.php/deploy.yaml file as a configuration variable (let's say "local_path") when initializing the deployer you could also do the following:
config:
  rsync_src: '{{local_path}}'

Let’s throw an exception in resync_src not set.

Hello @oheinemann
This work nice with deploy.php, so i clarified that problem while using yml file, where not  accessible DIR variable.
That would be nice to extract path to deployer into another defined variable. I seen some related issues with current problem but in different situations (example: https://github.com/deployphp/deployer/issues/3165)

@antonmedv Yes, I think throwing an error would probably be a better solution than using the wrong directory by default.
I looked a little deeper into the code and found out that in the dep file in deployer/deployer/bin the constant DEPLOYER_DEPLOY_FILE is set and the path to deploy.php/deploy.yaml/deploy.yml is stored there.
And in the file ProcessRunner.php in deployer/deployer/src/Component/ProcessRunner exactly this constant is used for the WorkingDirectory of the process.
Here also the constant DEPLOYER_ROOT is checked which according to the documentation can also be set to define the WorkingDirectory.
With all this information I created a function in php which generates a local path:
set('local_path', function () {
    $deployerRoot = getenv('DEPLOYER_ROOT') !== false ? getenv('DEPLOYER_ROOT') : (defined('DEPLOYER_DEPLOY_FILE') ? dirname(DEPLOYER_DEPLOY_FILE) : null);
    if ($deployerRoot === null) {
        throw new \RuntimeException('Could not determine a local path!');
    }

    return rtrim($deployerRoot, '/') . '/';
});

I personally would like to have some functions in the yaml file, which unfortunately I can currently only solve with php.
As an example: I change the release name in all my projects to the current timestamp:
set('timestamp', function () {
    return date('YmdHis');
});

Unfortunately this is not possible in the yaml file, so I import a php file in each case to implement exactly that.
To have predefined variables for such typical issues would be a dream!

And as another side note!
I really have no idea why that works with all the silly single quotes!
However, I use exactly this function of the host in a separate task where I have a problem with exactly these single quotes.
So I rewrote this function for my task:
    public function connectionOptionsString(): string
    {
         return implode(' ', array_map(function($value) {
            return substr($value, 0, 1) !== '-' ? escapeshellarg($value) : $value;
        }, $this->connectionOptionsArray()));
    }

In my opinion, not all arguments need to be enclosed in single quotes. Especially the argument identifiers do not.
Also, I would mask the single quotes in the rsync conrtib again with slashes. Since here the string is inside single quotes.
addslashes($sshArguments = $host->connectionOptionsString());

@antonmedv Maybe this solution finds a way into your source code.

PR?


Add Blog launch for RSE AUNZ fiscal sponsorship

Text: https://hackmd.io/YR-f50tIR4S3Q2ru8x3vYw
tagging @manodeep and @rowlandm

Debs - low priority, but also pretty fast to do.

Hi Yo,
I was thinking also adding the main reason for the fiscal sponsorship was
the RSE Asia Australia unconference.
El vie, 12 jul 2024, 11:56 p. m., Yo Yehudi @.***>
escribió:

Debs - low priority, but also pretty fast to do.
—
Reply to this email directly, view it on GitHub
https://github.com/open-life-science/open-life-science.github.io/issues/803#issuecomment-2225642780,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADGRUXJ7SRU7ULUADJNNCLZL7OBTAVCNFSM6AAAAABKY7KTK6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRVGY2DENZYGA
.
You are receiving this because you were mentioned.Message ID:
<open-life-science/open-life-science.github.io/issues/803/2225642780@
github.com>

Yep, Rowland doesn't sleep.

Haha!
Updated the text to include rse Asia Australia. Please have a look and
adjust as appropriate :)
El sáb, 13 jul 2024, 6:47 a. m., Deborah Udoh @.***>
escribió:

Yep, Rowland doesn't sleep.
—
Reply to this email directly, view it on GitHub
https://github.com/open-life-science/open-life-science.github.io/issues/803#issuecomment-2226332026,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADGRUXIFTPUMVQYMVS333DZMA6G3AVCNFSM6AAAAABKY7KTK6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRWGMZTEMBSGY
.
You are receiving this because you were mentioned.Message ID:
<open-life-science/open-life-science.github.io/issues/803/2226332026@
github.com>

Thank you! ✨

I made some edits and cleaned up the text a bit.

Thank you @manodeep and everyone else! Blog post is ready at PR #804. Feel free to make additional suggestions there. 🙏

she's live - thanks all!!


[Auto-generated] Update requirements.txt

Update requirements.txt
Automatically created PR from the ci_update_requirements.yml workflow.

Codecov Report

:exclamation: No coverage uploaded for pull request base (ci/dependabot-updates@4f29bd2). Click here to learn what that means.
Patch has no changes to coverable lines.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@                   Coverage Diff                    @@
##             ci/dependabot-updates     #304   +/-   ##
========================================================
  Coverage                         ?   69.15%
========================================================
  Files                            ?       18
  Lines                            ?      522
  Branches                         ?        0
========================================================
  Hits                             ?      361
  Misses                           ?      161
  Partials                         ?        0

Flag
Coverage Δ

pytest
69.15% <0.00%> (?)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


fix: Change the docs of JSONAgentOutputParser

I am merely making some minor adjustments to the function documentation. I hope to provide a small assistance to LangChain.

Description: Change the docs of JSONAgentOutputParser. It will be JSON better,
Issue: no,
Dependencies: no,
Tag maintainer: @hwchase17,
Twitter handle: Not worth mentioning.

thanks @zhaozhiming!


Roll engine c40d6dacc78b..b0370c13c302 (3 commits)

https://github.com/flutter/engine/compare/c40d6dacc78b...b0370c13c302
git log c40d6dacc78b8c25a2822ec8c6f47aa46f642c5f..b0370c13c302bc747f0cb3736a3a79e79ae4f7cd --no-merges --oneline
b0370c13c Decode using the last cached required frame (flutter/engine#7715)
4f3eb42df Update documentation for command line args in FlutterProjectArgs. (flutter/engine#7733)
4e13ede57 Roll src/third_party/skia 9564bf44e5fa..8619a34bb875 (10 commits) (flutter/engine#7732)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (chinmaygarde@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


Windows 10 stuck on "Just a moment..." loop.

Describe the bug
A clear and concise description of what the bug is. BEFORE SUBMITTING YOUR ISSUE, PLEASE LOOK AT THE PINNED ISSUES AND USE THE SEARCH FUNCTION TO MAKE SURE IT IS NOT ALREADY REPORTED. ALWAYS COMMENT ON AN EXISTING ISSUE INSTEAD OF MAKING A NEW ONE.
To Reproduce
Steps to reproduce the behavior:

Install on an iPhone XS with ios 13.5 fakesigned with unc0ver jb.
Create a new VM with 1024mb of ram, 2 cores, microsoft windows 10 iso, and a 50gb virtual harddisk.
Start the VM, and install windows.
At here you will be stuck on the "Just a moment..." screen, Until you get to a screen that says "Why did my PC restart? There's a problem that's keeping us from getting your PC ready to use, but we think an update will get things working again. Here's how to get the update, 1. Make sure your PC is plugged in. 2. If your PC uses wifi, select next and follow the instructions to connect it to a wifi network. 3. If this PC does not use wifi insert a network cable to connect to a wired network and select Next. 4. Once your connected, select Next, and the update will install."
Select next, and it will get stuck on "Just a moment..." again, and it will go back to the screen described in step 4.

Crash log
If the app crashed, you need a crash log. To get your crash log, open the Settings app and browse to Privacy -> Analytics & Improvements -> Analytics Data and find the latest entry for UTM. You should export the text and attach it here.
Debug log
For all issues, including crashes, you should attach a debug log. Open UTM, and open the settings for the VM you wish to launch. Near the bottom of the configuration options is Debug Log. Turn it on and save the VM. After you experience the issue, quit the VM and re-launch UTM. Open the VM settings again and select Export Log... and attach it here.
debug.log
Upload VM
If possible, upload the .utm you are experiencing an issue with. Since Github has an attachment size limit, you may want to upload to another service such as Google Drive. Link it here.
https://mega.nz/folder/hHwHUDbS#kBdzof5v_8ay1lXPYeuZHg

Describe the bug
A clear and concise description of what the bug is. BEFORE SUBMITTING YOUR ISSUE, PLEASE LOOK AT THE PINNED ISSUES AND USE THE SEARCH FUNCTION TO MAKE SURE IT IS NOT ALREADY REPORTED. ALWAYS COMMENT ON AN EXISTING ISSUE INSTEAD OF MAKING A NEW ONE.
To Reproduce
Steps to reproduce the behavior:

Install on an iPhone XS with ios 13.5 fakesigned with unc0ver jb.
Create a new VM with 1024mb of ram, 2 cores, microsoft windows 10 iso, and a 50gb virtual harddisk.
Start the VM, and install windows.
At here you will be stuck on the "Just a moment..." screen, Until you get to a screen that says "Why did my PC restart? There's a problem that's keeping us from getting your PC ready to use, but we think an update will get things working again. Here's how to get the update, 1. Make sure your PC is plugged in. 2. If your PC uses wifi, select next and follow the instructions to connect it to a wifi network. 3. If this PC does not use wifi insert a network cable to connect to a wired network and select Next. 4. Once your connected, select Next, and the update will install."
Select next, and it will get stuck on "Just a moment..." again, and it will go back to the screen described in step 4.

Crash log
If the app crashed, you need a crash log. To get your crash log, open the Settings app and browse to Privacy -> Analytics & Improvements -> Analytics Data and find the latest entry for UTM. You should export the text and attach it here.
Debug log
For all issues, including crashes, you should attach a debug log. Open UTM, and open the settings for the VM you wish to launch. Near the bottom of the configuration options is Debug Log. Turn it on and save the VM. After you experience the issue, quit the VM and re-launch UTM. Open the VM settings again and select Export Log... and attach it here.
debug.log
Upload VM
If possible, upload the .utm you are experiencing an issue with. Since Github has an attachment size limit, you may want to upload to another service such as Google Drive. Link it here.
https://mega.nz/folder/hHwHUDbS#kBdzof5v_8ay1lXPYeuZHg

try this:
Step 1. Create A windows 10 vm in virtualbox on PC or mac
step 1.1. Name the vm "windows 10"
step 1.2. if you are going to install windows 10 32-bit, use Atleast 1 GB of ram or the 64 bit use atleast 2 GB of ram.
step 1.3. select Create new Hard disk
step 1.4. Set the hd size to 32 gigs or if not, use 24 gigs.
step 1.5. select VDI option
step 1.6. click on Create
step 2. power on the VM
step 3. Install Windows 10 like you normally do and finish the entire setup till when it shows you the desktop
step 4. power off the VM
step 5. copy the hard disk file to your iPhone XS
step 6. go to UTM app and change the hard disk image to the hard disk file you've copied in the last step.
step 7. Boot up the VM in UTM app
I hope it helps!

Edited: Ignore what I said, it works fine, safari just tries to block the vdi download.

A small bug is that even if I create a new hda.img and attach it to the VM, it is unable to save snapshots to it.

Edited: Ignore what I said, it works fine, safari just tries to block the vdi download.

then use chrome, and if that browser doesn't work, use other iOS browsers in your iPhone XS Max

yea that's what I did, Also I tried converting the .vdi to the raw format(.img) using clone, but I still can't save snapshots.

wait I think I will make a download link for Win10.utm file
Download Link: testing the VM...


okcruby food for August 11 Meeting

group: okcruby
food type: ?
guest approximation: 15
meeting date: Thursday, Aug 11 2016 at 11:30 AM
For visibility,
organizers: @jagthedrummer @piisalie @datachomp
note: this is an automated issue, please close/reply/etc if something seems off.

Jimmy johns was ordered. 15 piece platter, a nice deluxe veggie sandwich,
and chocolate chip cookies. Jeremy said y'all will get the drinks.
On Wednesday, August 10, 2016, Rob Sullivan<EMAIL_ADDRESS>wrote:

group: okcruby
food type: ?
guest approximation: 15
meeting date: Thursday, Aug 11 2016 at 11:30 AM
For visibility,
organizers: @jagthedrummer https://github.com/jagthedrummer @piisalie
https://github.com/piisalie @datachomp https://github.com/datachomp
note: this is an automated issue, please close/reply/etc if something
seems off.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/techlahoma/techlahoma/issues/119, or mute the thread
https://github.com/notifications/unsubscribe-auth/AArvZB7YtH_waFJ4hHpGLmcFoE_qvxuhks5qed5JgaJpZM4JhMfi
.

  Thank you!


don't verify hostname in certificate (e.g. trust all certificates)

Solution the issue #1573

@Benoit2R the tests are failing (cannot find the type)

@Benoit2R the tests are failing (cannot find the type)

I correct it in some minutes, sorry

I dont think the failed test is due to my code, can you check it ?
I have some trouble to test on my local dev environment because I'm on Windows with docker in a vagrant box.

@kiview if Docker CLI and other popular Docker clients ignore it then it is fine by me too

Thinking about it, we should do what Docker CLI does and respect the DOCKER_TLS_VERIFY environment variable. The docs are alarmingly vague on what the default is.
I'm happy to go with this as-is, and raise an issue for making this configurable (perhaps both via DOCKER_TLS_VERIFY and a config file flag?)

I'm happy to go with this as-is, and raise an issue for making this configurable

Sorry, I've changed my mind. I don't want to relax the defaults - it seems wrong.
I think that we should really be reading the DOCKER_TLS_VERIFY and DOCKER_CERT_PATH env vars and making the TLS cert verification behaviour act accordingly.
Sorry for changing my mind.

I'm happy to go with this as-is, and raise an issue for making this configurable

Sorry, I've changed my mind. I don't want to relax the defaults - it seems wrong.
I think that we should really be reading the DOCKER_TLS_VERIFY and DOCKER_CERT_PATH env vars and making the TLS cert verification behaviour act accordingly.
Sorry for changing my mind.

Just an explanation : DOCKER_TLS_VERIFY is still taken into account and the http connection is secured. The only check that my PR disables is the hostname in the DOCKER_HOST that needs to match the Subject Alternative Name in the certificate: https://en.wikipedia.org/wiki/Subject_Alternative_Name
OkHttp library is not coded to take care of wildcard SAN. I don't check the code of other rest client libraries but practically it works with my wildcard certificate.


All API links are returning 404 .

All the API links are giving 404 not found due to the invalid API-version being used : 2021-01-01-preview
Changed version for all APIs from 2021-01-01-preview to 2021-08-01

@SherrySahni : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

@dlepow for review
Thanks.

@SherrySahni - Thanks for catching the issue and submitting the PR. We already have another PR in progress that should address the same issue, so I'm closing this PR as a duplicate. We appreciate your attention and are working to avoid this issue in the future.
#please-close


LiveLink stops when Ctrl+Z pressed

LiveLink stops when Pressing Ctrl+Z for Undo action on C4D,and sometimes C4D crashes by it.
Actually looks like UI is still working,but it can't send
C4D's bugreport file and screenshot(zip)
It needs to restart C4D to fix it.

Hi Flowing, unfortunately most of these bugs are caused by memory errors in the underlying library that HLAELiveLink uses. The only true fix would be to completely re-write a websocket library (which I have considered). But the rate at which crashes happen seem to be small (at least in my tests). If there is enough demand however I will look into re-writing a the websocket backend. In the meantime if you do find any more situations that immediately cause a crash feel free to leave them as a comment on this issue, this will help me further diagnose the underlying problem.

thanks <3 cheers!
also C4D => HLAE cam sync stoppes when C4D has keyframe on camera positions.
I'm not sure it's websocket related bug or not tho.

Are there keyframes on each frame or is just a few across the timeline? And does it stop only on the keyframes or just outright stop working completely?

I'm using that your "HLAE CamIO 2 C4D",So it has keyframes on every frames.
it stoppes completely when C4D cam has keyframes.

Hmmm interesting, I will definitely have to look into that then. Thanks for the info!


Add code documentation

Worked on Ski Resort Mapping and Information Service and Merged it to the man branch.


Add new matcher to generated template

New menu item which would add a matcher based on the selection to a previously generated template.

Duplicate of #10 (caused by network issue when creating the ticket)


Insertion of the AISWEB API which is the official source of aeronautical information in Brazil

Insertion of the AISWEB API which is the official source of aeronautical information in digital media produced by the Department of Airspace Control (DECEA).
Thank you for taking the time to work on a Pull Request for this project!
To ensure your PR is dealt with swiftly please check the following:

[x] Your submissions are formatted according to the guidelines in the contributing guide
[x] Your additions are ordered alphabetically
[x] Your submission has a useful description
[x] The description does not end with punctuation
[x] Each table column should be padded with one space on either side
[x] You have searched the repository for any relevant issues or pull requests
[x] Any category you are creating has the minimum requirement of 3 items

Do I have to make a new PR?

Now it's all right!

Thanks!


feat: Cube search does not use full text search engine

Removing full text search and use simple contain for search.

@adintegra I tried the approach of not using the full text search here : https://github.com/visualize-admin/visualization-tool/issues/683. I think it solves the problem of performance. Since I had no longer access to scores, I had to redo the scoring logic but it's good, I think we could even reintroduce the keyword highlighting by doing it ourselves.
@ortnever and @rdataflow Could you try some of your favorite searches on https://visualization-tool-git-feat-search-no-fulltext-ixt1.vercel.app/en/browse?dataSource=Prod to see if it matches your expectations ? Thanks

@ortnever and @Rdataflow Could you try some of your favorite searches on https://visualization-tool-git-feat-search-no-fulltext-ixt1.vercel.app/en/browse?dataSource=Prod to see if it matches your expectations ? Thanks
It looks very good. Even with a short abstract from the description, the result is displayed in 1-3 seconds. Thank you !


option for "template" in AOT webpack build

I'm using 3rd-party module (ng2-bootstrap) in my AOT webpack build.
At this time, this module's components do not reference "templateUrl", only "template". As a result, the html templates are not compiled to javascript in my AOT bundle output.
See https://github.com/valor-software/ng2-bootstrap/blob/8305a56cd7aaebfd33920fe6b7ab86813449fac0/src/pagination/pagination.component.ts
Is there an option to force AOT code generation when using inline template?

Hi. Unfortunately that's up to the library (in this case ng2-bootstrap) to publish javascript files without the decorators built in. The CLI does not have control on that code. We're working on a proper solution, but in the meantime there's nothing you can do. We'll try to communicate with ng2-bootstrap to help them publish a better version of their library.


Incorrect URL for /connect endpoint

When I tried to use the node.connect() method, the URL generated ended up as
...api/codex/v1/connect/16Uiu2HAmCwmepfRARSyWnSMAzUmGfkMr5pvvDC7dKRGVjTNSJBpz?/ip4/<IP_ADDRESS>/tcp/8070

i.e missing addrs= in the query

Can you try with version 0.0.13 ?

Seems to work perfectly!


(Pre-training model - cpc_3d_brats.json) ValueError: too many values to unpack (expected 4)

Hi there,
I have a folder containing numpy files of brats which are all (128,128,128,4) and I have written in the path to this folder in cpc_3d_brats.json config file. However, when I tried to run train.py self_supervised_3d_tasks/configs/train/cpc_3d_brats.json, I obtain this following error.
Traceback (most recent call last):
File "train.py", line 4, in
main()
File "/home/svu/e0310071/self-supervised-3d-tasks/self_supervised_3d_tasks/train.py", line 78, in main
init(train_model)
File "/home/svu/e0310071/self-supervised-3d-tasks/self_supervised_3d_tasks/utils/model_utils.py", line 67, in init
f(**args)
File "/home/svu/e0310071/self-supervised-3d-tasks/self_supervised_3d_tasks/train.py", line 69, in train_model
steps_per_epoch=len(train_data),
File "/home/svu/e0310071/self-supervised-3d-tasks/self_supervised_3d_tasks/data/generator_base.py", line 47, in len
self.get_multiplicator()
File "/home/svu/e0310071/self-supervised-3d-tasks/self_supervised_3d_tasks/data/generator_base.py", line 39, in get_multiplicator
self.index_multiplicator = DataGeneratorBase.get_batch_size(self.__data_generation_intern([self.list_IDs[0]])[0])
File "/home/svu/e0310071/self-supervised-3d-tasks/self_supervised_3d_tasks/data/generator_base.py", line 124, in __data_generation_intern
data_x, data_y = self.pre_proc_func(data_x, data_y)
File "/home/svu/e0310071/self-supervised-3d-tasks/self_supervised_3d_tasks/algorithms/cpc.py", line 127, in f
return preprocess_grid_2d(preprocess_2d(x, self.crop_size, self.patches_per_side))
File "/home/svu/e0310071/self-supervised-3d-tasks/self_supervised_3d_tasks/preprocessing/preprocess_cpc.py", line 34, in preprocess_2d
_, w, h, _ = batch.shape
ValueError: too many values to unpack (expected 4)
How do I resolve this error?

Hello, just to add on, when I tried to run the pre-training 3dcpc algorithm, I got the above error. I used this command train.py self_supervised_3d_tasks/configs/train/cpc_3d_brats.json. The json file which I only change the data directory is as follows:
{
"algorithm": "cpc",
"batch_size": 64,
"data_dir": "/hpctmp/e0310071/BRATS_datatrain_128_numpy",
"dataset_name": "brats",
"train3D": true,
"data_dim": 128,
"n_channels": 4,
"embed_dim": 128,
"split_per_side": 4,
"epochs": 500,
"lr": 1e-3,
"num_layers": 4
}
However, I got a valueerror as mentioned in the previous comment. Does anyone have this same error or does anyone know how to resolve this error?

Hi, I tried just using the cpc_3d.json file instead of cpc_3d_brats.json file and the configurations in cpc_3d.json appears to make the code to run finish with no errors.

I will assume that the current configurations in cpc_3d_brats.json do not work and we should use the configurations in cpc_3d.json. Will be closing for now.


Restore entrypoint

Restore the entrypoint as gapic_plugin.py. Artman currently expects gapic_plugin.py as the main class.

@lukesneeringer PTAL
What do you think about this? It just makes calling this protoc pluging from artman a tad easier.


Ordinal Regression Code

Hi!
Niu et al. propose ordinal regression with CNN.
Their loss function has w_t_i, that indicates the weight of the i-th image for the t-th
task.
This parameter from absolute cost matrix (paper formula 2).
Your loss function without this parameter in code. (e.g. afad-ordinal.py)
Can you tall me why?

Yes, they included that weight in the generalized formulation but they didn't explicitly use weights. I.e., just below Eq. 2, they say that they set all weights equal to 1, which is the same as using no weighting.

Thanks for your reply.

 Please let me know if you have any other questions


Feature Request: Detect user browser and modernize code only to where the browser currently has support

This way if a user is using a browser that utilizes ES2015+ you will have the most optimized/modernized code available for that browser. Would require more maintenance, but would make this tool infinitely more useful.

This is not something Lebab is meant for, and I would strongly advise against using Lebab this way.
A much better way to achieve this is to use BabelJS to write your source code to the latest ECMAScript version and compile it down to ES5+ features supported by browsers. But I think this kind of browser-detection feature even fits to the scope of BabelJS. Rather it would need to be a separate project, perhaps a Webpack plugin or something.


Resume complete

This is a complete implementation resumability for intermittent disconnections.
Also there is a new constructor for Reactive socket:
auto socket = ReactiveSocket::disconnectedClient(...);
auto socket = ReactiveSocket::disconnectedServer(...);
which allows clients to start creating streams (requestXXX)
and then connecting the socket via connectServer, connectClient.
A few of the more granular signals are commented out since they are not used at the moment.
The change has been tested on the tcpresume{client,server}.

Does tcpresumeclient talk to normal tcpserver and vice/versa?

tcpresumeclient typically talks to tcpresumeserver and they exchange expected messages. It would connect to tcpserver, but it is not gonna exchange the expected messages.

Does that indicate a need to carefully update both client and server at the same time. Mostly interested that they don't cause segfaults

oh, I see what you mean.
I didn't break the compatibility so we don't need to upgrade both servers and clients at the same time.


Matching against opt-in list is case-sensitive

Thanks for this cool action! ▶ 💝
The current matching technique used to determine if a user is opted in to SizeUp is case-sensitive:
https://github.com/lerebear/sizeup-action/blob/cb3a93a2d6c0c5983116f345086b8606863dbc60/src/initializer.ts#L43-L46
This means that if the configuration file casing does not match their official GitHub username casing, it will consider the user opted out instead.
As least within GitHub, most of our configurations are intentionally case-insensitive for similar "gotcha" reasons. It would be great if this action could be updated to allow for that as well. ❤
I'm imagining the needed code change is roughly as easy as settling on a consistent casing for both sides of the comparison, e.g.
  const pullAuthor = pull.user.login.toLowerCase()
  const usersWhoHaveOptedin = config.optIns || []
  const userHasOptedOut =
    usersWhoHaveOptedin.length &&
    !usersWhoHaveOptedin.find((login: string) => login.toLowerCase() === pullAuthor)

Alternatively, you could use something like String#localeCompare, though the many input options for that are a bit cumbersome IMHO.
Would also be ideal to add a test for this case-insensitivity to prevent later regression. 🧪

Thank you for the report @JamesMGreene!

I'm imagining the needed code change is roughly as easy as settling on a consistent casing for both sides of the comparison

Yes, I agree that this should be sufficient.


Update cats-effect-std to 3.2.7

Updates org.typelevel:cats-effect-std from 3.2.1 to 3.2.7.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel", artifactId = "cats-effect-std" } ]

labels: library-update, semver-patch

Codecov Report

Merging #263 (322cf79) into master (8dcfc47) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #263   +/-   ##
=======================================
  Coverage   84.61%   84.61%
=======================================
  Files          35       35
  Lines         221      221
  Branches       10       10
=======================================
  Hits          187      187
  Misses         34       34

Flag
Coverage Δ

unittests
84.61% <ø> (ø)

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

Continue to review full report at Codecov.

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


Refactor blacklists; add chat commands for ip/ns/asn (#3884 take two)

This contains a major refactoring of blacklists.py in order to accommodate the new blacklist types properly and expose a sensible and unified API towards the calling functions in chatcommands, findspam, etc.
This is take two of #3884 which was merged and then backed out last week. I have fixed two bugs and extended the code slightly.

NACK on the review - fix your Lint first.

I have now tried to rebase this 4-5 times and I end up with really weird results every time. Maybe I'll leave that for tomorrow.

I would suggest splitting the "New Commands" and "Refactoring' into completely separate PRs.  I say this because I'd rather review the commands themselves independently of the "Refactor blacklists".  So I'm actually going to NACK this PR as is until you can split the two into separate PRs and issues.

Set to pending: Thomas requested that I split off the blacklist refactoring into a separate PR which can be reviewed separately, so I will proceed to do that, in 6-8.

Set to pending: Thomas requested that I split off the blacklist refactoring into a separate PR which can be reviewed separately, so I will proceed to do that, in 6-8.


What should be split into submodules?

Probably makes sense to split output_writers.jl, diagnostics.jl, and poisson_solvers.jl into submodules.
They're turning into megafiles that will benefit from being split into smaller logical chunks.
And this will probably only get worse with time.

Yes, let's not have megafiles.
I think the question is what shouldn't be in a submodule!
Fields would make good one. It could even be split into a separate package, eg StaggeredFields.jl.

Probably we should just start with the most important modularizations and work our way to the less important once. I'd say output writers and poisson solvers are most important. We need the poisson solver functionality written more generally and abstracted a bit more because, if it were, we could use a poisson solver for implicit time-stepping

Probably Grids will need a submodule when we have more than one.

This is related to issue #456

Ah sorry I remember we talked about this but forgot there's already an existing issue!
Diagnostics and output writers could probably already be split into submodules.
Maybe the grids can be modularized at the same time as the vertically stretched grid is introduced, while the Poisson solvers can be modularized when the new FACR solver is introduced (which might nuke some existing solvers too).

I am wondering if physics should also go to submodules, eg Buoyancy, Coriolis, and soon, SurfaceWaves.
I think this will help us keep our parameter structs concise. For example, FPlane is a bit confusing when it appears in isolation. But Coriolis.FPlane is clear. Also, the word "Buoyancy" appears a lot. If all buoyancy-related things are encapsulated within submodules, naming becomes easier.
We can stop exporting so many names at the top-level, and instead export the submodules. So a user will write
buoyancy = Buoyancy.Seawater()

or
coriolis = Coriolis.FPlane()

or
surface_waves = SurfaceWaves.UniformStokesDrift()

What do we think?

By the way, we might take some inspiration from Documenter.jl's structure:
https://github.com/JuliaDocs/Documenter.jl/tree/master/src

Yeah I think it's starting to make a lot of sense for Buoyancy, Coriolis, and SurfaceWaves to have their own modules.
Actually I also think that boundary_conditions.jl is becoming painful to parse and edit and would also benefit from becoming a submodule with multiple files.


[FEATURE] Don't default to empty config when config parsing fails

Currently eww simply defaults to an empty config when the daemon initially fails to load the config. This means that the error gets printed out during daemon startup, but does not get reported on other interactions with the config, such as opening windows. For those cases, it currently just prints out "Window does not exist".
This could be solved by actually storing the configuration as a Result in the app struct, or by having some other mechanism to guarantee that the config is always valid when parts of eww that rely on it are accessing it.

This has been implemented a long time ago ^^


Account named _______ not found.

This tool works perfectly up until the "prepare" phase, when I get the following error:
$ python cloudmapper.py prepare --account production
Traceback (most recent call last):
  File "cloudmapper.py", line 146, in <module>
    main()
  File "cloudmapper.py", line 134, in main
    run_prepare(arguments)
  File "cloudmapper.py", line 109, in run_prepare
    account = get_account(args.account_name, config)
  File "cloudmapper.py", line 41, in get_account
    raise Exception("Account named \"{}\" not found".format(account_name))
Exception: Account named "production" not found

I used the same account name in the previous step without issue.

I got a similar error.  Do you have a config.json set up with that account name in it?  (The error might be more helpful if it identified the name of the file it couldn't find the account name in.)

Yeah, this looks like your config.json file must not have a production account configured in it.  I just added some error handling to look for this case: https://github.com/duo-labs/cloudmapper/commit/fc1b824542fd95854ba6b0000fd30bbdedc04a45
Please close the ticket @damianesteban if this resolves your problem.

I believed this has been resolved, so I'm closing this.


minus and plus infinity

In 5.2.3, it states  bounding  box  limits  for  that angular axis shall be set to minus and plus infinity.
How will that be implemented? I am not sure that NaN (Not a Number) which is defined in IEEE754, along with its behaviour with other numbers, for the standard representation of integer and floating point numbers in digital computers, is sufficient, as many programming languages do not allow such a value.
Or have I missed something? (I'm still a Fortran programmer at heart)

@chris-little IEEE754 specifically defines +/- infinity as separate values than NaN, doesn't it?
I am however wondering about the need to refer to infinity rather than simply the -180..180 degrees of longitude and -90..90 degrees of latitude, which I believe is actually what is done?
The bounding box is specified in projected coordinates anyway, as what is being tiled is the finited projected CRS plane, with any spherical coordinates already wrapped around before projection, and where explicit values are used in all current examples of polar (e.g. https://github.com/opengeospatial/2D-Tile-Matrix-Set/blob/master/schemas/tms/2.0/json/examples/tilematrixset/UPSArcticWGS84Quad.json#L7) and cylindrical projection (e.g. https://github.com/opengeospatial/2D-Tile-Matrix-Set/blob/master/schemas/tms/2.0/json/examples/tilematrixset/WorldMercatorWGS84Quad.json#L7).
This also applies to TMS based on geographic/plate carre projections, where -90..90 and -180..180 values are specified (e.g. https://github.com/opengeospatial/2D-Tile-Matrix-Set/blob/master/schemas/tms/2.0/json/examples/tilematrixset/WorldCRS84Quad.json#L8), as again the wrapping is already done before tiling the plane (otherwise we would have an infinite number of rows and columns, wouldn't we?).
Should this text be removed?

This is the text commented comes for OWS common. This is what I propose to solve this issue.
"Theoretically, there are cases where defining a bounding box could be problematic or impossible, such us angular axis of an Ellipsoidal, Spherical, Polar, or Cylindrical coordinate system. However, tiles need to be circumscribed to real coordinates and will deliberately avoid to enter into regions of the space where coordinates go to infinite or cannot be defined. For example, the WorldMercatorWGS84Quad tile matrix set (based on a cylindrical projection) deliberately avoids to get close to the poles. Since tiles are conterminous, it is always possible to define a bounding box that includes them all."


Customize menu icon

Maybe we can add a prop to customize the menu icon

This is now fixed with 69ece75ce81b


It's not really an issue - Mac install

I am truly sorry to bother a project with so much awesomeness but wget is not installed by default on mac.
brew install wget fix this issue.

Thanks you!


ip: add ip_darwin / ip_linux files

This ensures that the IP package is not compiled with the netlink dependency in
Darwin (Mac) environments. Packages are imported into the Ginkgo tests which
transitively imported pkg/ip, which caused running the Ginkgo tests locally to
not work:
$ K8S_VERSION=1.15 ginkgo --focus=K8sServicesTest -v -- -cilium.holdEnvironment=true -cilium.skipLogs=true -cilium.provision=false
Failed to compile test:

../pkg/ip/ip.go:781:36: undefined: netlink.FAMILY_ALL

Ginkgo ran 1 suite in 6.686508356s
Test Suite Failed

Factoring out the functions which depend on the netlink library into
a separate file for Linux / Darwin works around this problem.
Signed-off by: Ian Vernon<EMAIL_ADDRESS>

This change is

test-me-please

Coverage increased (+0.01%) to 44.241% when pulling 6f737a35809f577e716cd157e00d65b304d936d3 on pr/ianvernon/hack-to-remove-netlink-pkg-ip-dep into 41fe45c8c0abf905478098d509c324fc48d09731 on master.


Functional key is reserved to make visible touch bar

System Preferences Configuration
Modifiers
Control key : Command
Command key : Control
Settings -> Keyboard
Press Fn key to Show F1, F2 etc.
I have a Karabiner-Element configuration
Fn -> Left Command
Left Command -> Fn
After these settings I expect using Ctrl as Function key. The issue here is to use Fn key to change touch bar view. OS still uses Fn key (not modified by Karabiner-Elements). I this case I cannot use Ctrl+F1, F2 etc. combinations.

I wouldn't be surprised if there is no way to fix this.

This seems to be a duplicate of  #543.


Cloned lines issue

When drawing any image on led panel it draws "cloned" lines. I made a little sketch, that should fill display line by line and writes to console number of line, that is drawing right now. I don't relly know what's wrong.
link to sketch
link to video how it works
PS: my panel uses rul5358 (SHIFTREG_ABC_BIN_DE) decoder

Can you try an older version of this library ? 1.0.2  and see if this issue persists?

Can you try an older version of this library ? 1.0.2 and see if this issue persists?

Yep, issue still persists.

You'll need to use pxMatrix library sadly.

PS: my panel uses rul5358 (SHIFTREG_ABC_BIN_DE) decoder

You will need to use PxMatrix which supports this type of panel.

You'll need to use pxMatrix library.

PxMatrix library should support my panel, but it uses very odd cabling


Handling RuntimeException thrown by YouTubeStandalonePlayer.getReturnedInitializationResult

I've noticed that when you have more than one app that can launch a standalone player for YouTube, your app will present a dialog for you to choose which app do you wish to use. If you close the dialog without choosing an option, the YoutubeStandaloneModule throws a RuntimeException that is not handled by the lib causing the app to crash.
Therefore I've wrapped the block inside a try catch and rejected the promise if the runtime error occurs.
I also fixed a few type definitions.

Thank you @luizppa for your PR! Can you please resolve the conflicts it has so that I merge it for the next version?

@zabojad done!


Remove legacy email addresses

I apparently missed a few instances of the old email address. Remove them, and perhaps instead link to https://hgvs-nomenclature.org/stable/#contact-us .
docs/background/basics.md
22:However, if you prefer sending a private email, questions about the HGVS nomenclature can be addressed to **"Varnomen @ HUGO-int.org"** (remove spaces).
29:Comments to proposals should be addressed to **"Varnomen @ HUGO-int.org"** (remove spaces), subject: HVNC-xxx (xxx the proposal number, e.g., HVNC-011).
33:To make sure that you receive notification of any proposal published for Community Consultation, please register at the HUGO office (E-mail info @ variome.org), subject: "HVNC discussion list".

Please keep the option to send a personal email. Some people prefer to ask a question in private, without the whole world watching.


Using correct dest config property.

When starting with gulp-modular, I started with the styles first. But the sass task uses the wrong dest config property.

Thanks for your pull request. I had to look at it twice, because everything works as expected already, even it looks like a bug in the code.
In the tasks/sass.js file we do the follwing:
 // we do some processing
      ...
 // we write the generated style file to the specified directory
      .pipe(gulp.dest(config.styles.dest))
 // we trigger livereload
      .pipe(browserSync.stream({match: '**/*.css'}))
 // the we extend an existing rev-manifest.json file
      .pipe(gulpif(config.build.rev, rev.manifest({cwd: config.statics.dest, merge: true})))
 // and save it again.
      .pipe(gulpif(config.build.rev, gulp.dest(config.statics.dest)));

The rev-manifest.json is used by css and js files, thats why it is placed in another folder. It can be used to automate retrieving the generated files.
{
  "scripts.js": "scripts-4c4707b04e.js",
  "style.css": "style-3aa65abdcc.css",
  "vendor.css": "vendor-df9763975e.css",
  "vendor.js": "vendor-780593d794.js"
}

What problems do you experience with the styles task?

I do not have static files, so no static task. When that is not active, styles and scripts error out on the reference to config.static.dest.

I see your problem, the easiest solution would be to comment in the static task, if you don't specify any files it won't do anything.
But as a fallback we should use the styles/scripts folder, failing can't be a solution.


[DOCS] Update link of speckle webapp in readme

docs: update link of speckle webapp in Readme

@ayansengupta17, Thanks for your vigilance and the submission; in this case, your contribution was already covered in a more extensive one.


V4 - IncompatibleNativeProperty  -moz-column-gap

Describe the bug
When using nativewind ^4.0.16 use the gap-x-4 class and you get a log message with
IncompatibleNativeProperty  -moz-column-gap
The actual feature does work fine you do get a gap, you just also get your console spammed by the message above on every reload.
To Reproduce
Steps to reproduce the behavior:

Use the gap-x-4 class

Expected behavior
No log message is generated.

I have this error as well. Was not present in V2

me too "nativewind": "^4.0.1",

Same here

We are closing all pre-v4.1 issues now that v4.1 is released. If the problem is still happening with the latest v4.1, please open a new issue and reference this one in the description.


regress-sv_br_ml20181012b_iv

Surelog issue: https://github.com/chipsalliance/Surelog/issues/2552

Issue fixed.


Error on trying to install the package.

npm i discord-speech-recognition
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@discordjs/voice
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/discord-speech-recognition
npm ERR!   discord-speech-recognition@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See C:\Users\GT-ACE\AppData\Local\npm-cache\eresolve-report.txt for a full report.
npm ERR!      C:\Users\GT-ACE\AppData\Local\npm-cache_logs\2022-10-28T17_24_28_802Z-debug-0.log

I am having the same error!

Should be fixed now #44


Run tests on multiple Python versions

A quick approach to #22:  we can run the tests under Python 3 on Travis by inserting a transformer step, without (yet) requiring the code to be Python-3 compatible.
I've also added a pypy job, to catch any regressions on that side.

I checked the request history of Travis but unfortunately only the build requests from the last 24 hours are there. Several possibilities exist but I'm not sure which one leads to this issue... Maybe you can try to commit again and I'll check the build request?

If a build hasn’t been triggered for your commit, these are the possible build request messages:
“Could not authorize build request”, usually means that the account’s subscription expired or that it ran out of trial builds.

“GitHub payload is missing a merge commit”, please confirm your pull request is open and mergeable.
“Branch excluded per configuration” or “Branch not included per configuration”, please make sure your branch is not explicitly excluded or not included in your .travis.yml file.
Build type disabled via repository settings, please make sure your Push and Pull Request builds are still active.

OK! It turns out that certain syntax errors (I think omitting the quotes in python versions?) makes Travis silently ignore the whole branch.  Anyway, it's running now.

The good news is that only four tests fail under PyPy.
The less-good news is that Python 3 support won't be completely effortless.  I don't think it's too bad though; the ~100 failing tests remaining look mostly like I need to sprinkle some __hash__ methods on the various classes and then look at how the ast module has changed.
So this remains a work in progress, but it is in progress!

OK! It turns out that certain syntax errors (I think omitting the quotes in python versions?) makes Travis silently ignore the whole branch. Anyway, it's running now.
The good news is that only four tests fail under PyPy.
The less-good news is that Python 3 support won't be completely effortless. I don't think it's too bad though; the ~100 failing tests remaining look mostly like I need to sprinkle some __hash__ methods on the various classes and then look at how the ast module has changed.
So this remains a work in progress, but it is in progress!

Hey Zac if you merge in the latest commit from master, all tests should pass under pypy

Also @Zac-HD I think the reason why there are a lot of hashable errors is that we kind of throw everything into a set/dict. For example I sometimes collect a bunch of functions into a set() and use them as keys in dictionaries. It worked fine in python2 but I'm not sure what happened in python3.

Nothing wrong with that pattern! It's just that custom objects aren't implicitly hashable in python 3, so we need to add the hash method.

Nothing wrong with that pattern! It's just that custom objects aren't implicitly hashable in python 3, so we need to add the hash method.

Hmmm really? I just tried to hash some Connectable and function in python3.6 it worked fine. What exactly are the new rules on custom objects?

From memory, instances of custom types are hashable by default, but not if you define an __eq__ method (because "hashable objects which compare equal must have the same hash value", and so the default id()-based implementation is not valid).

Anyway!  I think this branch has grown enough, and is ready to be merged.

Travis must pass on both CPython 2.7 and PyPy
Travis runs against CPython 3.6, 3.7, and PyPy 3.5; but allows them to fail for now
Improved, though unfinished, Python 3 support (37 tests failed, 461 passed)

So this is a clear improvement on the status quo.
@cbatten - after merging, I'd be happy to keep working on Py3 support but I don't think I have a strong advantage for the remaining issues.  Let me know if you'd like me to keep going, or leave it to someone else.

Now down to just seven tests that fail on Python 3!  I've hit diminishing returns, but this is very close to working :smile:

Ready to merge!


Fix broken links

Looks like the suggested change didn't get into the commit
/kind cleanup
/kind documentation
NONE

/assign @robscott

/lgtm


Does "Set-SPOTenant -HideSyncButtonOnTeamSite $true" remove the sync button but allow the "add a shortcut button"?

Does this command remove the sync button but allow the "add a shortcut button" to function?
The command is Set-SPOTenant -HideSyncButtonOnTeamSite $true

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 367f3b91-1252-7bce-fd1e-21ceb1ae12f3
Version Independent ID: 66e3f10a-0231-e7e0-0659-0be66eaf13fa
Content: Sync in SharePoint and OneDrive - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/sharepoint-sync.md
Service: sharepoint-online
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

Thank you for the prompt reply.  Should it also appear in the list of other documented parameters here https://learn.microsoft.com/en-us/powershell/module/sharepoint-online/set-spotenant?view=sharepoint-ps?


Video Player sample /Navigating between controls

Hello there , I just started to read about Tizen and the ability to do so with Xamarin .
I just cannot find a sample of video player for the TV ,
I tried to clone  git://git.tizen.org/profile/tv/apps/dotnet/sample/videoplayer but it has no files.
the other topic that might be answered if I have managed to clone that solutoin  is : do you have a recommended pattern to handle the events coming from the remote to navigate between different controls ?

Hello,
We are trying to add more samples for Mobile/TV/Wearable.
In case of Video, you can refer to Player sample for mobile first from DEV branch.
Tizen C# Sample Page: https://developer.tizen.org/ko/development/sample/.net-application/tizen-samples
PlayerSample for Mobile: https://github.com/Samsung/Tizen-CSharp-Samples/tree/dev/Mobile/PlayerSample
The code of "dev" branch means it is not yet ready for public release. plz, consider that.
thanks
Ben

@ahmad-crossplatform  This sample app is helpful to you.

@sgchoi5  @rookiejava  thank you guys , how about handling the remote control directions ? Is there any best recommendation ?

hello again , does any one have this but using CustomRenderes instead of implementing it as  service ?

In order to handle key events, you can use ElmSharp.EcoreEvent as below.

// Register key event callbacks for the window using the EcoreEvent.
EcoreEvent<EcoreKeyEventArgs> _keyUp = new EcoreEvent<EcoreKeyEventArgs>(EcoreEventType.KeyUp, EcoreKeyEventArgs.Create);

 _keyUp.On += (s, e) =>
{
 // write your code here using  e.Keyname and e.KeyCode
}

please refer to KeyCode. for more keycode details.

Video Player
You can make your own custom renderer for either VideoView or VideoPlayer refer to the sample app mentioned above. There is no big difference. And, we are preparing to contirubute a popoluar open source Xamarin.Forms media player  plugins for tizen such as MediaManger. You can use these plugins for your video player app when it available. (I'm not sure the date but we hope soonish.)

@ahmad-crossplatform
Hi
This is not an answer for your question but I’d like to introduce TV video player sample and a lecture.
I think it’s quite helpful for making your own Video Player.
Xamarin Forms TV App Development for Tizen
http://tizenappschool.org/tutorial/71/contents/1
Sample code
https://s3-us-west-1.amazonaws.com/tizenschool/97/Lect3_Snapshot09_Add_PlayerButton_Final.zip
Actually the sample code is related with below lectures that is describe how to make a video player on Tizen TV in detail, but those are written as Korean language. I leave URLs of them for the information.
Cross Platform TV Application Development
http://tizenappschool.org/tutorial/95/contents/1
http://tizenappschool.org/tutorial/97/contents/1

@hallower  thanks a lot , but for some reason I cannot see the media samples ... like the videos ... where do you get that from ?


[Feature Request] Timed publishing of games

Is your feature request related to a problem? Please describe.
Online conventions with lots of participants might a) want to set up the games in advance as there are likely to be lots of games planned and b) might not want to allow sign ups for games before the convention starts. Currently there doesn't seem to be a reasonable way to do this.
Describe the solution you'd like
A feature to set up a game but publish it at a later time and date (or to prevent sign ups before that time and date) would solve this issue.

As a workaround: You could set in the Manage Server section -> Template you can set a role which players need to sign up for that game (Which is then active for all games which use that template). You can set that to the admin role, and at the time at which your players are allowed to sign up, you can remove the player role from the template, so that everyone can sign up. (Tested right now that it works that way).
This would help at least with the sign up problem.
But the feature is a good idea, and the feature will be implemented, as soon as I have the necessary free time and the necessary spoons for that.


Readme.md needed.

 The readme must have screen shots to reference the pre class project.

README.md was created with screen shots to deployed applications. credits were given to the leaders of my bootcamp do to their endless effort and direct assistance with the project!


Update sebastian/diff and phpunit

This updates the sebastian/diff dependency version so that the tool can be pulled in via composer when using a PHPUnit version > v4

Thank you @jackbentley 😊


:w, :q ... commands are not working

Hello,
First of, all thank you for developing a great extension.
As you know :w, :q ... are not working.

window/buffer management should be performed by vscode by design. At some time, i'll rebound :w, :q etc to perform vscode commands, but i have more important issues to do right now.


Fixed tokenizer and audio processing logic

This PR addresses the following changes:

Updated the tokenizer logic in WhisperTokenizerWrapper.
Fixed the startIndex validation in AudioChunker.swift and AudioProcessor.swift.
Corrected the embedSize and sequenceLength calculations in AudioEncoder.swift.
Added missing implementations for applyChatTemplate and encode methods.
Updated dependencies in Package.swift.

Hi @1amageek, I'm happy to see your enthusiasm to contribute to WhisperKit! I'd like to make a few recommendations to help guide your work in the future, since this PR has grown quite large.

Your initial changes identified a bug in the startIndex checking code, which was great to see! I've adjusted it slightly and added you as a co-author in this commit 47035a8 (#216)
I'd highly recommend running the test locally in Xcode to make sure they pass. I've been periodically running the CI in this PR and they have been failing, which is a required check for merging.
I'd also recommend breaking up this PR into smaller more focused PRs that fix a specific issue or add a single new feature.
Try to make a best effort to conform to the existing code style. This includes keeping any existing documentation in place, using english language for new documentation, and including the common file header when adding new files.

With this in mind, I would recommend closing this PR and separating out the changes that add new features or fix existing bugs into separate PRs with details on why they are necessary and solve an open issue listed in https://github.com/argmaxinc/WhisperKit/issues. Thanks again for your contributions so far, open to any discussion on these topics.


"cannot use sqliteDB"

go get is failing for me:
Justins-MacBook-Air:openbazaar-go justin$ go get -u github.com/OpenBazaar/openbazaar-go
# github.com/OpenBazaar/openbazaar-go
./openbazaard.go:393: cannot use sqliteDB (type *db.SQLiteDatastore) as type "github.com/OpenBazaar/openbazaar-go/repo".Datastore in field value:
	*db.SQLiteDatastore does not implement "github.com/OpenBazaar/openbazaar-go/repo".Datastore (missing Purchses method)
./openbazaard.go:416: cannot use sqliteDB (type *db.SQLiteDatastore) as type "github.com/OpenBazaar/openbazaar-go/repo".Datastore in argument to service.SetupOpenBazaarService:
	*db.SQLiteDatastore does not implement "github.com/OpenBazaar/openbazaar-go/repo".Datastore (missing Purchses method)
./openbazaard.go:418: cannot use sqliteDB (type *db.SQLiteDatastore) as type "github.com/OpenBazaar/openbazaar-go/repo".Datastore in argument to "github.com/OpenBazaar/openbazaar-go/net".NewMessageRetriever:
	*db.SQLiteDatastore does not implement "github.com/OpenBazaar/openbazaar-go/repo".Datastore (missing Purchses method)
./openbazaard.go:421: cannot use sqliteDB (type *db.SQLiteDatastore) as type "github.com/OpenBazaar/openbazaar-go/repo".Datastore in argument to "github.com/OpenBazaar/openbazaar-go/net".NewPointerRepublisher:
	*db.SQLiteDatastore does not implement "github.com/OpenBazaar/openbazaar-go/repo".Datastore (missing Purchses method)

The same error is happening with go run openbazaard.go start. Fails on both go 1.6.3 and 1.7.

This has been fixed.


Update check for updates

Added database migrations to 'check for updates' job.
Added logic to database migrations to better handle error cases.

create separate jenkins job to do repair


Migration status of mirror-siyuan

root@mirrors-sjtug-2:/mnt# fio -threads=8 -size=32G -bs=1m -direct=1 -rw=randwrite -name=test -group_reporting -filename=./io.tmp -runtime=30
test: (g=0): rw=randwrite, bs=(R) 1024KiB-1024KiB, (W) 1024KiB-1024KiB, (T) 1024KiB-1024KiB, ioengine=psync, iodepth=1
fio-3.12
Starting 1 process
Jobs: 1 (f=1): [w(1)][100.0%][w=33.0MiB/s][w=33 IOPS][eta 00m:00s]
test: (groupid=0, jobs=1): err= 0: pid=32523: Tue Nov  3 23:43:50 2020
  write: IOPS=40, BW=40.9MiB/s (42.9MB/s)(1228MiB/30017msec); 0 zone resets
    clat (msec): min=10, max=220, avg=24.36, stdev=13.15
     lat (msec): min=10, max=220, avg=24.44, stdev=13.15
    clat percentiles (msec):
     |  1.00th=[   12],  5.00th=[   14], 10.00th=[   16], 20.00th=[   18],
     | 30.00th=[   20], 40.00th=[   21], 50.00th=[   22], 60.00th=[   23],
     | 70.00th=[   26], 80.00th=[   29], 90.00th=[   34], 95.00th=[   44],
     | 99.00th=[   71], 99.50th=[  104], 99.90th=[  148], 99.95th=[  222],
     | 99.99th=[  222]
   bw (  KiB/s): min=20480, max=61440, per=99.97%, avg=41878.67, stdev=7207.17, samples=60
   iops        : min=   20, max=   60, avg=40.87, stdev= 7.03, samples=60
  lat (msec)   : 20=38.27%, 50=57.98%, 100=3.09%, 250=0.65%
  cpu          : usr=0.35%, sys=0.31%, ctx=1240, majf=0, minf=10
  IO depths    : 1=100.0%, 2=0.0%, 4=0.0%, 8=0.0%, 16=0.0%, 32=0.0%, >=64=0.0%
     submit    : 0=0.0%, 4=100.0%, 8=0.0%, 16=0.0%, 32=0.0%, 64=0.0%, >=64=0.0%
     complete  : 0=0.0%, 4=100.0%, 8=0.0%, 16=0.0%, 32=0.0%, 64=0.0%, >=64=0.0%
     issued rwts: total=0,1228,0,0 short=0,0,0,0 dropped=0,0,0,0
     latency   : target=0, window=0, percentile=100.00%, depth=1

Run status group 0 (all jobs):
  WRITE: bw=40.9MiB/s (42.9MB/s), 40.9MiB/s-40.9MiB/s (42.9MB/s-42.9MB/s), io=1228MiB (1288MB), run=30017-30017msec

Disk stats (read/write):
  vdb: ios=0/2457, merge=0/2, ticks=0/59598, in_queue=56160, util=93.49%

Read performance is weird. Seems there's some optimization done on hypervisor.
root@mirrors-sjtug-2:/mnt# fio -threads=8 -size=32G -bs=1m -direct=1 -rw=randread -name=test -group_reporting -filename=./io.tmp -runtime=30
test: (g=0): rw=randread, bs=(R) 1024KiB-1024KiB, (W) 1024KiB-1024KiB, (T) 1024KiB-1024KiB, ioengine=psync, iodepth=1
fio-3.12
Starting 1 process
Jobs: 1 (f=1): [r(1)][69.2%][r=10.7GiB/s][r=10.0k IOPS][eta 00m:04s]
test: (groupid=0, jobs=1): err= 0: pid=879: Tue Nov  3 23:46:15 2020
  read: IOPS=3339, BW=3340MiB/s (3502MB/s)(32.0GiB/9811msec)
    clat (usec): min=86, max=24364, avg=297.38, stdev=1090.14
     lat (usec): min=86, max=24365, avg=297.76, stdev=1090.19
    clat percentiles (usec):
     |  1.00th=[   87],  5.00th=[   87], 10.00th=[   87], 20.00th=[   87],
     | 30.00th=[   87], 40.00th=[   87], 50.00th=[   87], 60.00th=[   87],
     | 70.00th=[   87], 80.00th=[   87], 90.00th=[   94], 95.00th=[  102],
     | 99.00th=[ 5997], 99.50th=[ 6456], 99.90th=[ 7701], 99.95th=[ 8225],
     | 99.99th=[15926]
   bw (  MiB/s): min=  152, max=11192, per=92.37%, avg=3085.04, stdev=4908.12, samples=19
   iops        : min=  152, max=11192, avg=3085.00, stdev=4908.04, samples=19
  lat (usec)   : 100=94.94%, 250=1.29%, 500=0.02%, 750=0.01%
  lat (msec)   : 2=0.01%, 4=0.03%, 10=3.69%, 20=0.02%, 50=0.01%
  cpu          : usr=0.57%, sys=29.44%, ctx=1291, majf=0, minf=265
  IO depths    : 1=100.0%, 2=0.0%, 4=0.0%, 8=0.0%, 16=0.0%, 32=0.0%, >=64=0.0%
     submit    : 0=0.0%, 4=100.0%, 8=0.0%, 16=0.0%, 32=0.0%, 64=0.0%, >=64=0.0%
     complete  : 0=0.0%, 4=100.0%, 8=0.0%, 16=0.0%, 32=0.0%, 64=0.0%, >=64=0.0%
     issued rwts: total=32768,0,0,0 short=0,0,0,0 dropped=0,0,0,0
     latency   : target=0, window=0, percentile=100.00%, depth=1

Run status group 0 (all jobs):
   READ: bw=3340MiB/s (3502MB/s), 3340MiB/s-3340MiB/s (3502MB/s-3502MB/s), io=32.0GiB (34.4GB), run=9811-9811msec

Disk stats (read/write):
  vdb: ios=2467/0, merge=0/0, ticks=13694/0, in_queue=13480, util=69.26%

281G	centos
7.3G	csw
1.3T	debian
210G	debian-cd
155G	debian-security
633G	deepin
9.2G	deepin-cd
462G	docker-ce

We have 4.5T remaining for synchronizing.


Test/oe selftest grub

Is there anything holding this PR back? It happens to also include a fix for detecting the machine name for the qemu test since if you build for multiple machines the autodetection fails.

Nope, leeme try it

Tested with oe-selftest


Suggestion: File Editor

Now I know there is already the CFG Editor in txAdmin, however, personally I think it would be useful to add the ability for other files to be edited via txAdmin for example, resources.cfg or permissions.cfg - vMenu uses the permissions.cfg and it would be easier to manage via txAdmin than logging into the server and editing it that way.

Want this idea to be reality, I was just looking into if this was possible to even add a second config page for things like connectqueue resource which you can't edit any other way then through an editor.


Fix pulsar:version vars in site2 docs.

Motivation
pulsar:version shows up in the docs. http://pulsar.incubator.apache.org/staging/docs/standalone/#installing-pulsar
Modifications
Update to the new variable format.
Result
The correct version shows up.

run java8 tests


is istioctl manifest generate different than istioctl manifest apply?

I installed istio with istioctl manifest apply.
Then generated a manifest with istioctl manifest generate > generated.yaml
When I checked with istioctl verify-install-f generated.yaml it failed with
Error: Istio installation failed, incomplete or does not match "generated.yaml" - the required ServiceAccount:istio-reader-service-account is not ready due to: serviceaccounts "istio-reader-service-account" not found

That service account is indeed present on the YAML file but was not created by manifest apply.
When I removed the installation with `kubectl delete -f generated.yaml>, I got the following errors:
Error from server (NotFound): error when deleting "generated.yaml": serviceaccounts "istio-reader-service-account" not found
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete gateways.networking.istio.io istio-multicluster-egressgateway)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete virtualservices.networking.istio.io istio-multicluster-egressgateway)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete envoyfilters.networking.istio.io istio-multicluster-egressgateway)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete destinationrules.networking.istio.io istio-multicluster-destinationrule)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete policies.authentication.istio.io grafana-ports-mtls-disabled)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete gateways.networking.istio.io ingressgateway)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete sidecars.networking.istio.io default)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete meshpolicies.authentication.istio.io default)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete destinationrules.networking.istio.io istio-policy)
Error from server (NotFound): error when deleting "generated.yaml": deployments.apps "prometheus" not found
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete attributemanifests.config.istio.io istioproxy)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete attributemanifests.config.istio.io kubernetes)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete handlers.config.istio.io stdio)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete instances.config.istio.io accesslog)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete instances.config.istio.io tcpaccesslog)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete rules.config.istio.io stdio)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete rules.config.istio.io stdiotcp)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete instances.config.istio.io requestcount)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete instances.config.istio.io requestduration)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete instances.config.istio.io requestsize)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete instances.config.istio.io responsesize)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete instances.config.istio.io tcpbytesent)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete instances.config.istio.io tcpbytereceived)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete instances.config.istio.io tcpconnectionsopened)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete instances.config.istio.io tcpconnectionsclosed)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete handlers.config.istio.io prometheus)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete rules.config.istio.io promhttp)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete rules.config.istio.io promtcp)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete rules.config.istio.io promtcpconnectionopen)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete rules.config.istio.io promtcpconnectionclosed)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete handlers.config.istio.io kubernetesenv)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete rules.config.istio.io kubeattrgenrulerule)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete rules.config.istio.io tcpkubeattrgenrulerule)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete instances.config.istio.io attributes)
Error from server (NotFound): error when deleting "generated.yaml": the server could not find the requested resource (delete destinationrules.networking.istio.io istio-telemetry)
Error from server (NotFound): error when deleting "generated.yaml": deployments.apps "istio-telemetry" not found
Error from server (NotFound): error when deleting "generated.yaml": poddisruptionbudgets.policy "istio-telemetry" not found
Error from server (NotFound): error when deleting "generated.yaml": services "istio-telemetry" not found
Error from server (NotFound): error when deleting "generated.yaml": deployments.apps "istio-tracing" not found
Error from server (NotFound): error when deleting "generated.yaml": services "jaeger-query" not found
Error from server (NotFound): error when deleting "generated.yaml": services "jaeger-collector" not found
Error from server (NotFound): error when deleting "generated.yaml": services "jaeger-agent" not found
Error from server (NotFound): error when deleting "generated.yaml": services "zipkin" not found
Error from server (NotFound): error when deleting "generated.yaml": services "tracing" not found

Which makes me think that manifest apply is not applying the same things that are being generated with manifest generate
This is using 1.4 beta4.

@elfinhe this maybe related to the namespace apply issue?

@elfinhe this maybe related to the namespace apply issue?

It is related to the 'istio-reader-service-account' creation. But the kubectl delete sounds like a different issue.

Just verified that PR#18844 will fix the istioctl manifest apply and istioctl verify-install -f generated.yaml  errors.

the kubectl delete -f generated.yaml error seems like caused by the reversed order of deletion on CRD and CRs.

@howardjohn @ostromart @jwendell
I don't think kubectl delete -f in our doc is an appropriate way to uninstall Istio. I suggest we create an istioctl manifest delete or istioctl manifest uninstall to address the errors in kubectl delete.

+1 from me. Not sure if it's feasible for 1.4 though.
From a user perspective, I'd expect also to have istioctl manifest verify-install, not manifest apply to save to a temp file and then verify-install -f <temp-file> to verify.

@jwendell https://github.com/istio/istio/issues/18547 is track istioctl verify-install to accept the icp cr and is targeted for 1.5.  welcome your comments on that.

@howardjohn @ostromart @jwendell
I don't think kubectl delete -f in our doc is an appropriate way to uninstall Istio. I suggest we create an istioctl manifest delete or istioctl manifest uninstall to address the errors in kubectl delete.

Created https://github.com/istio/istio/issues/18900 to track this

Seems like this is now tracked in other issues, thanks everyone


Updating youtube-shorts filter to block new shorts window on normal videos on desktop

What filter does this issue target?
https://letsblock.it/filters/youtube-shorts
What changes should be made?
Youtube is now showing shorts on the left basically all normal videos.
I think we should add a new line to the youtube shorts filter to account for this.

I hope this helps :) I just accidentally got lost in shorts for 30minutes without even realizing
Did you write a uBlock filter solving this?
www.youtube.com##ytd-reel-shelf-renderer
Additional notes
The new line is almost identical to one that already exists for mobile see,
m.youtube.com##ytm-reel-shelf-renderer

Thanks a lot @SiggyPony:

could you please test this more selective rule and comfirm that it works?

www.youtube.com##ytd-watch-next-secondary-results-renderer ytd-reel-shelf-renderer

I'd like to give you attribution on the commit adding that rule, could you please give me the @users.noreply.github.com email that is shown in https://github.com/settings/emails, so that I can add it as co-author?

Hey,  yes that more specific rule also works.
If you want to give some attribution I think its<EMAIL_ADDRESS>Took a bit to figure out, I don't generally use github 😅

Cool! It's been merged and deployed, you can remove the custom ruke and manually refresh your list right now to grab it!