chown: cannot access '/dev/kvm': No such file or directory

Successfully built 126d21846d82
Successfully tagged windows-in-docker-container_win10:latest
Creating windows-in-docker-container_win10_1 ... done
Attaching to windows-in-docker-container_win10_1
win10_1  | chown: cannot access '/dev/kvm': No such file or directory
windows-in-docker-container_win10_1 exited with code 1
When i am comment out this line "chown root:kvm /dev/kvm" in startup.sh file
I got error
Successfully built 399b4fa3089d
Successfully tagged windows-in-docker-container_win10:latest
Recreating windows-in-docker-container_win10_1 ... done
Attaching to windows-in-docker-container_win10_1
win10_1  | Bringing machine 'default' up with 'libvirt' provider...
win10_1  | ==> default: Checking if box 'peru/windows-server-2022-standard-x64-eval' version '20231001.01' is up to date...
win10_1  | ==> default: Created volume larger than box defaults, will require manual resizing of
win10_1  | ==> default: filesystems to utilize.
win10_1  | ==> default: Uploading base box image as volume into Libvirt storage...
==> default: Creating image (snapshot of base box volume).
win10_1  | ==> default: Creating domain with the following settings...
win10_1  | ==> default:  -- Name:              _default
win10_1  | ==> default:  -- Description:       Source: /Vagrantfile
win10_1  | ==> default:  -- Domain type:       kvm
win10_1  | ==> default:  -- Cpus:              4
win10_1  | ==> default:  -- Feature:           acpi
win10_1  | ==> default:  -- Feature:           apic
win10_1  | ==> default:  -- Feature:           pae
win10_1  | ==> default:  -- Feature (HyperV):  name=relaxed, state=on
win10_1  | ==> default:  -- Feature (HyperV):  name=synic, state=on
win10_1  | ==> default:  -- Feature (HyperV):  name=vapic, state=on
win10_1  | ==> default:  -- Feature (HyperV):  name=vpindex, state=on
win10_1  | ==> default:  -- Clock offset:      utc
win10_1  | ==> default:  -- Memory:            6000M
win10_1  | ==> default:  -- Base box:          peru/windows-server-2022-standard-x64-eval
win10_1  | ==> default:  -- Storage pool:      default
win10_1  | ==> default:  -- Image(vda):        /var/lib/libvirt/images/_default.img, virtio, 100G
win10_1  | ==> default:  -- Disk driver opts:  cache='default'
win10_1  | ==> default:  -- Graphics Type:     spice
win10_1  | ==> default:  -- Graphics Websocket:
win10_1  | ==> default:  -- Graphics Port:
win10_1  | ==> default:  -- Graphics IP:
win10_1  | ==> default:  -- Graphics Password: Not defined
win10_1  | ==> default:  -- Video Type:        qxl
win10_1  | ==> default:  -- Video VRAM:        16384
win10_1  | ==> default:  -- Video 3D accel:    false
win10_1  | ==> default:  -- Sound Type:        ich6
win10_1  | ==> default:  -- Keymap:            en-us
win10_1  | ==> default:  -- TPM Backend:       passthrough
win10_1  | ==> default:  -- INPUT:             type=mouse, bus=ps2
win10_1  | ==> default:  -- CHANNEL:             type=spicevmc, mode=
win10_1  | ==> default:  -- CHANNEL:             target_type=virtio, target_name=com.redhat.spice.0
win10_1  | ==> default:  -- CHANNEL:             type=unix, mode=
win10_1  | ==> default:  -- CHANNEL:             target_type=virtio, target_name=org.qemu.guest_agent.0
win10_1  | ==> default:  -- RNG device model:  random
win10_1  | Error while creating domain: Error saving the server: Call to virDomainDefineXML failed: invalid argument: could not get preferred machine for /usr/bin/qemu-system-x86_64 type=kvm

ubuntu@instance-2:~$ docker-compose up
Creating network "ubuntu_default" with the default driver
Pulling win10 (ghcr.io/vaggeliskls/windows-in-docker-container:latest)...
latest: Pulling from vaggeliskls/windows-in-docker-container
aece8493d397: Pull complete
144679710325: Pull complete
619e9e7c184b: Pull complete
72a0c82c9b93: Pull complete
4895bb91dfcb: Pull complete
4a1083947222: Pull complete
4d5e7a535a97: Pull complete
Digest: sha256:d9915868b86f5415f008e9411d4582af8b000b56c5ecd75c94a8a5966730f5c0
Status: Downloaded newer image for ghcr.io/vaggeliskls/windows-in-docker-container:latest
Creating ubuntu_win10_1 ... done
Attaching to ubuntu_win10_1
win10_1  | chown: cannot access '/dev/kvm': No such file or directory
ubuntu_win10_1 exited with code 1

Thank you for reaching out, and I appreciate your feedback!
Based on the error log, it seems like you might be running the project on a Red Hat operating system. To ensure the project runs smoothly, it's essential to enable virtualization, specifically KVM (KVM kernel module).
Here are some references for further reading:

KVM Hypervisor Requirements: Link
Installing KVM packages on Red Hat: Link

P.S. I've successfully battle-tested this project on Ubuntu servers. If you have any specific questions or encounter further issues, feel free to ask for assistance. Good luck with your project!

i am running this container in ubuntu 20.04 lts GCP vm. I am using your docker image ghcr.io/vaggeliskls/windows-in-docker-container:latest i got same error
Question:
Is KVM package install in my host machine
@Thanks for responding

As of my last knowledge, cloud providers generally did not widely support nested virtualization on virtual machines (VMs). Nested virtualization refers to the ability to run virtual machines within virtual machines. This feature is more commonly associated with running virtualization hypervisors, such as VMware ESXi or Microsoft Hyper-V, libvirt (current project), etc, within a virtual machine.
If you are specifically interested in nested virtualization on VMs in a cloud environment, I recommend checking the documentation and the latest updates from the specific cloud provider you are using (GCP). The level of support may vary based on the hypervisor technology used by the cloud provider and the type of virtual machine instances they offer.
PS. I can also verify that shared virtualization is also not supported on AWS ec2 instances. That is the reason that face issue when you tried to deploy this project

i am build a docker image from your repo after that i am run that on ubuntu 22.04 lts
ragvendra@Macmini:~/Demo_project/windows-in-docker-container$ docker run     --env-file .env     --privileged     --cap-add NET_ADMIN     --cap-add SYS_ADMIN     --volume /sys/fs/cgroup:/sys/fs/cgroup     --device /dev/kvm     --device /dev/net/tun     --publish 3389:3389     --name win10     --interactive     --tty     window10:latestinternal:/usr/lib/ruby/vendor_ruby/rubygems/core_ext/kernel_require.rb:85:in require': cannot load such file -- winrm (LoadError)         from <internal:/usr/lib/ruby/vendor_ruby/rubygems/core_ext/kernel_require.rb>:85:in require'
from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/plugins/communicators/winrm/shell.rb:9:in block in <top (required)>'         from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/lib/vagrant/util/silence_warnings.rb:8:in silence!'
from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/plugins/communicators/winrm/shell.rb:8:in <top (required)>'         from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/plugins/communicators/winrm/communicator.rb:6:in require_relative'
from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/plugins/communicators/winrm/communicator.rb:6:in <top (required)>'         from <internal:/usr/lib/ruby/vendor_ruby/rubygems/core_ext/kernel_require.rb>:85:in require'
from internal:/usr/lib/ruby/vendor_ruby/rubygems/core_ext/kernel_require.rb:85:in require'         from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/plugins/communicators/winrm/plugin.rb:15:in block in class:Plugin'
from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/lib/vagrant/registry.rb:27:in get'         from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/lib/vagrant/machine.rb:267:in communicate'
from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/lib/vagrant/machine.rb:149:in initialize'         from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/lib/vagrant/vagrantfile.rb:81:in new'
from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/lib/vagrant/vagrantfile.rb:81:in machine'         from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/lib/vagrant/environment.rb:716:in machine'
from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/lib/vagrant/plugin/v2/command.rb:180:in block in with_target_vms'         from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/lib/vagrant/plugin/v2/command.rb:204:in block in with_target_vms'
from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/lib/vagrant/plugin/v2/command.rb:186:in each'         from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/lib/vagrant/plugin/v2/command.rb:186:in with_target_vms'
from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/plugins/commands/up/command.rb:87:in execute'         from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/lib/vagrant/cli.rb:67:in execute'
from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/lib/vagrant/environment.rb:290:in cli'         from /usr/share/rubygems-integration/all/gems/vagrant-2.2.19/bin/vagrant:226:in <top (required)>'
from /usr/bin/vagrant:25:in load'         from /usr/bin/vagrant:25:in '

Try providing more details about your running system (OS).
You mention that you try to run from Ubuntu22, but I see that your hostname called MacMini.

Is the ubuntu 22.04 lts version a VM ?
What your host OS, Architecture (example: Ubuntu/Linux, amd64) ?
Did you tried building from the provided Dockerfile ?

DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=22.04
DISTRIB_CODENAME=jammy
DISTRIB_DESCRIPTION="Ubuntu 22.04.2 LTS"
x86_64
Yes i am building using provided Dockerfile

Thank you for your valuable feedback.
Upon investigating the issue you reported, I was able to replicate the problem and understand that it emerged after the Dockerfile upgrade was carried out to be compatible with Ubuntu 22.04 (Jammy).
The current issue seemed to stem from a version that is packaged in the official Ubuntu repository, interfering with the generation of the VM.
To resolve the issue, I have manually installed the most recent version of Vagrant. This should have effectively fixed the problem.
Please give it a try and inform me if you continue to face any difficulties.
Thanks again for making me aware of the issue.


Add a delete button in teams card

Since the addition of a more contextual deletion modal in https://github.com/frzyc/genshin-optimizer/pull/2122, it may make sense to add it to the overview card

Can I take this issue?


Update version info

@yuemlu I made the version changes directly on the master branch of my fork. packageReleaseNotes were already updated but version in csproj was not. Also updated assemblyfileversion. Note that the diff of the PR shows all the files of the previous PR. I hope this is not a problem.

AzureApiTag in AzSdk.RP.props also needs to be updated, since you've change the api version

Otherwise it looks good to me @leonardbf

AzureApiTag in AzSdk.RP.props also needs to be updated, since you've change the api version

@isra-fel Ok, updated


DRBG does not support multiple modes

By that I mean that DRBG-HASH can only support one SHA mode, not SHA1-SHA512.
Same with counter and HMAC.

This issue was resolved a few weeks back


Spotify Windows and Web Client do not highlight currently playing song in playlist viewer

Compatible Device

[X] I'm running Raspotify on a compatible Device

Are you sure?

[X] I'm not trying to run Raspotify on a ARMv6 device

Compatible OS

[X] I'm running Raspotify on a compatible OS

Compatible Configuration

[X] I'm running Raspotify on a compatible Configuration

Latest Version

[X] I'm running the latest version of Raspotify

Are you sure?

[X] I'm not running Raspotify <IP_ADDRESS> on a Pi v1 or Pi Zero

Due Diligence

[X] I have done my due diligence

What happened?
The Spotify Windows and web client do not highlight currently playing song in playlist viewer when this song is played by raspotify connect client.
It works for the Android spotify client while raspotify is playing. This client shows the currently playing song while the windows and web clients do not.
When making the desktop client connect to any other official spotify client (like Android) then it highlights the currently playing song, only when connected to raspotify it doesn't.
Relevant log output and/or the contents of /etc/raspotify/crash_report if any ( sudo journalctl -u raspotify -b and sudo cat /etc/raspotify/crash_report )
do not have one

This is a known upstream issue with librespot. That part of the Spotify API isn't implemented yet.
See:
https://github.com/librespot-org/librespot/issues/1127
And also related I believe:
https://github.com/librespot-org/librespot/issues/913


Code signing + auto updater

I'm suggesting to have the menubar app code signed so that we can remove the security warnings when installing the app. We will also get the benefit of putting it on the App Store. Another by product is that we can then use the Squirrel.mac framework to update the app behind the scenes, making it ever green.
I can hook u guys up with the code signing and app store bit. Any OS X dev guys wanna chip in more ideas?
https://github.com/atom/electron/blob/master/docs/api/auto-updater.md 😍

:+1:


Avoid setting background for screens.

WHAT

WHY
Screens should sit naturally on any background behind them, setting a constant background makes this impossible without overiding MaterialTheme just for this component.
HOW
Checklist :clipboard:

[x] Add explicit visibility modifier and explicit return types for public declarations
[x] Run spotless check
[x] Run tests
[x] Update metalava's signature text files

Note: the tests don't show that background, they now apply the background inside the test.


[BUG] - Listbox typing issue when using dynamic collections

NextUI Version
2.2.9
Describe the bug
When I try to use the Listbox & ListboxSection via dynamic collections, typing issue will be reported as below screenshot but rendering without any error.

TS2322: Type (item: ListboxItemBaseProps) => Element is not assignable to type (ItemElement<ListboxItemBaseProps> | ItemElement<ListboxItemBaseProps>[] | ItemRenderer<ListboxItemBaseProps>) & (ReactI18NextChildren | Iterable<...>) Type (item: ListboxItemBaseProps) => Element is not assignable to type ItemRenderer<ListboxItemBaseProps> & string Type (item: ListboxItemBaseProps) => Element is not assignable to type string
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
import { Listbox, ListboxItem, ListboxItemProps, ListboxSection, ListboxSectionProps } from '@nextui-org/react'
import { useState } from 'react'

const settingList: Partial<ListboxSectionProps<ListboxItemProps>>[] = [
  {
    key: 'application',
    title: 'Application',
    items: [
      {
        key: 'appearance',
        title: 'Appearance'
      }
    ]
  }
]

function Setting() {
  const [selectedKeys, setSelectedKeys] = useState(new Set(['text']))
  return (
    <Listbox
      selectionMode="single"
      selectedKeys={selectedKeys}
      onSelectionChange={setSelectedKeys}
      items={settingList}
    >
      {
        section => (
          <ListboxSection {...section}>
            {(item) => (
              <ListboxItem {...item} />
            )}
          </ListboxSection>
        )
      }
    </Listbox>
  )
}

export default Setting

Expected behavior
Nah, please take a look
Screenshots or Videos
No response
Operating System Version
Windows
Browser
Chrome

and no issue if I don't use ListboxSection , like the example in document - Listbox -> ListboxItem
the issue only produce on Listbox -> ListboxSection -> ListboxItem

hey @Cowjiang, perhaps you have found a solution already?

hey @Cowjiang, perhaps you have found a solution already?

Not yet, just add @ts-ignore : (

I have the same issue when using ListboxSection with dynamic items.
<Listbox         aria-label='Page selection'       >         <ListboxSection items={items}>           {(item) => (             <ListboxItem               key={item.key}             >               {item.title}             </ListboxItem>           )}         </ListboxSection>       </Listbox>


commands-queue.ts: String length / byte length counting issue

Hopefully fixes #1628
Description
Fixes using string length instead of byte length when encoding command
Checklist

[x] Does npm test pass with this change (including linting)?
[x] Is the new or changed code fully tested?
[x] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)? Nothing but bugfix, not needed

Just got my laptop on, did some testing with \u{1f91e}, and it seems it's already picking it up as 2 bytes with the old method.
As I've already written the test I'll leave this open, but I'll revert 2fa5ef6.

@MCPE-PC I've changed back to Buffer.byteLength(), and set the tests to look for a bytecount of 4, but it's now failing the test on my machine:
       UTF-16 Byte length check (see #1628):

      AssertionError [ERR_ASSERTION]: Expected values to be strictly equal:
+ actual - expected

+ '*1\r\n$2\r\n🤞\r\n'
- '*1\r\n$4\r\n🤞\r\n'
          ^

I'm not quite sure yet, but it seems as if the different machines might be using different encoding standards (e.g utf-8 and utf-16)
Not sure how much time I'll have today, but I'll look into this further.
Sorry for dismissing you earlier.

Turns out there were two places that calculated the length of a string
https://github.com/NodeRedis/node-redis/blob/c72aab2fc29a9cc994f6b810aeb10a4fe1863f29/lib/commands-queue.ts#L47
and
https://github.com/NodeRedis/node-redis/blob/c72aab2fc29a9cc994f6b810aeb10a4fe1863f29/lib/commands-queue.ts#L53
The fix you wrote worked perfectly fine, I just didn't encounter it because I put the special character in the wrong place to encounter the specific issue you did.
Thank you for the help, re-opening PR for review and merging.

@NovaAtWarren @MCPE-PC thank you very much! :)


Fix boot2docker instructions

The volume mount for /var/run/docker.sock was missing.

Acutally, I just noticed that DOCKER_HOST is being passed... That's not the way I run under b2d, but I suppose it works.

In fact, I don't do anything special to run under b2d.
@jwilder Why did you introduce the previous commit? Using TCP to connect in the b2d case is unnecessary. The unix socket connection works fine.

@md5 Didn't even realize that b2d was also using the unix socket.

Under boot2docker, all --volume mounts except those from the use home directory (/Users under OS X and C:\Users on Windows) will mount files from the VM's filesystem. This includes /var/run/docker.sock.

Even in the case of the user home directories, it's a VM directory that's being mounted. It just happens to be a VM directory that exposes a host directory via the VirtualBox shared folders extension.

Also note that boot2docker (the CLI tools, not the Linux distro), is on its way out and will be replaced by docker machine.

I'm closing this in favor of #224.

@md5 I'm struggling with using docker-machine and nginx-proxy. Right now I'm using port forwarding to port from inside the virtual machine :80 (nginx) to my host :8080 to be able to use localhost:8080. Then I used pf/pfctl to forward from 8080 on my host to 80 on my host to be able to use only localhost. The problem is that I can't access the virtual hosts that I configured (like foo.localhost). Any idea?


[BP] Fatal Error while you kill many zombie

This almost happen every map that had many zombie_ragdoll you kill
which map are including
http://scmapdb.wikidot.com/map:robination
http://scmapdb.wikidot.com/map:infested also happen in infested2

Could it be an out-of memory error?

Fixed since BulletPhysics has been refactored.


[C++Interop] Allow Multiple SWIFT_CONFORMS_TO_PROTOCOL to C++ Classes

This change introduces the ability for C++ classes to conform to multiple swift protocols using the macro SWIFT_CONFORMS_TO_PROTOCOL.
This resolves https://github.com/swiftlang/swift/issues/75831.

@swift-ci please smoke test

@egorzhdan I've been on vacation for the past week and haven't been able to work on fixing these issues. I'll examine why the smoke tests fail in two days from today.

Sounds good, thanks @jz709u!

@egorzhdan I merged the main branch to attempt to fix the issues. Since then, the unit test checks have been stuck in yellow. Can you provide any insight as to why the current state is not changing? I appreciate any help you can provide.

Updating the PR branch resets the unit test status. Let me run the tests again.
Before merging this PR, could you please rebase it on the main branch, instead of having a merge commit? That would help us to have a cleaner history once the PR is merged.

@swift-ci please smoke test

Updating the PR branch resets the unit test status. Let me run the tests again.
Before merging this PR, could you please rebase it on the main branch, instead of having a merge commit? That would help us to have a cleaner history once the PR is merged.

@egorzhdan
Understood, I'll do that now.

@egorzhdan Please trigger the smoke test again. I just rebased and squashed the last commit into the test commit.
Thanks Again

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

Please run it once more.

@swift-ci please smoke test

I have been having issues creating a consistent dev environment so that I can test whether my code runs and works.
This makes development extremely difficult, and I have been working on this for quite a while. The build failures I run into are random, and it is hard to diagnose the cause.

@swift-ci please smoke test

@egorzhdan @Xazax-hun
What is the expectation here:
this is the error that's being output

`- error: redundant conformance of 'ConformsToCTwice' to protocol 'C'

but my test is expecting:

error: Redundant conformance of 'ConformsToCTwice' to protocol 'SwiftTest.C'

Do we expect the module to be included as a prefix to the protocol name?

I think ideally we would want the module name to be included in the output, e.g. MySwiftModule.MyProtocol.
But I don't think it's worth changing that in this patch. Let's preserve the old output format for now, and we can improve it in follow-up patches.

@swift-ci please smoke test

@egorzhdan @Xazax-hun  I won't be able to finish this PR.  Thanks for your support.

@egorzhdan can I finish this PR up? What's left to complete?


[hotfix] Prevent editing of previous file versions

Purpose
Fixes #3011
If a user navigated to the current file version from the versions table and then made some edits to create a new version, the user was not taken to the new version. As a result, the user could create new versions by editing this now out-dated file version.
Changes
Remove url params for current file version.

Looks fine to me. Steve, your'e up :)


Getting error while installing deps

Hi everyone. I'm trying to install Node.js deps including no-hassle package through heroku-buildpack-nodejs and getting this error log
-----> Installing dependencies
       Installing node modules
       npm ERR! Error while executing:
       npm ERR! /usr/bin/git ls-remote -h -t git://github.com/ShopBonsai/joi-to-swagger.git
      npm ERR!
      npm ERR! fatal: remote error:
       npm ERR!   The unauthenticated git protocol on port 9418 is no longer supported.
Seems the reason of this is no-hassle's joi-to-swagger dependency and it was installed as
"joi-to-swagger": "git://github.com/ShopBonsai/joi-to-swagger.git",
Is it possible to use standard version in package.json?

Fixed in https://github.com/ShopBonsai/no-hassle/pull/74


Add segments to Flow to update traffic on part of way

Issue

Targeting issue: the issue will be CLOSED once the PR merged. If there is no issue that addresses the problem, please open a corresponding issue and link it here. Uses the Closes keyword to close them automatically.
Closes #306

Any other related issue? Mention them here.

Description
A summary description for what the PR achieves.

Add segments to Flow to update traffic only on part of way.

Tasklist

[x] CHANGELOG-FORK.md entry (CHANGELOG)
[ ] profiles/CHANGELOG.md entry if any Lua changes
[ ] update relevant Wiki pages
[ ] add tests (see testing documentation

Prerequirements

Want to contribute? Great! First, please read this page Contribution Guidelines.
If your PR is still work in progress please attach the relevant label.

Have your tried it in your application already?

I'm working on the integration with our app. I've checked it manually via mali.js.org

How many segmented flows added compare with original way based flows(E.g., 50% more)? Does it significantly affect on performance?

At this moment we have a not big osm.pbf file and also there aren't a lot of speed limits in our app to check performance. But when the integration is finished I'd be able to generate a batch of Flows with and without segments to compare.

@wangyoucao577

Do you mind that we merge the codes after you finish the integration in your application? You may possible to show us some log to make sure it works well. Recently we'll not have such data to test it in real situation, so your integration will be the most important verifcation after unit test.

Yes, I do. It's a. good point.


Investigate how Mina does a proof of account inexistence.

Apparently it doesn't. We need to fix this.


[TC-LVL-2.1] OnLevel value is null instead of integer

Step 9 in TC-LVL-2.1 returns null value as OnLevel, while test plan says that is must be larger than MinLevel which is 1. Tested on 1bc660e70fd233dd0ee42b147e428ce8f8a3338d using x64 chip-tool and platform qpg.
TC_LVL_2_1.1660821562.log
TC_LVL_2_1.1660821562_DUT.log

null is a perfectly valid value for OnLevel.
Filed https://github.com/CHIP-Specifications/chip-test-plans/issues/2105

closing, as the test plan issue was created.


[WIP][SPARK-35396]Add AutoCloseable close to BlockManager and InMemoryRelation

This PR is proposing a add-on to support to manual close entries in MemoryStore and InMemoryRelation
What changes were proposed in this pull request?
Currently:
MemoryStore uses a LinkedHashMap[BlockId, MemoryEntry[_]] to store all OnHeap or OffHeap entries.
And when memoryStore.remove(blockId) is called, codes will simply remove one entry from LinkedHashMap and leverage Java GC to do release work.
This PR:
We are proposing a add-on to manually close any object stored in MemoryStore and InMemoryRelation if this object is extended from AutoCloseable.
Veifiication:
In our own use case, we implemented a user-defined off-heap-hashRelation for BHJ, and we verified that by adding this manual close, we can make sure our defined off-heap-hashRelation can be released when evict is called.
Also, we implemented user-defined cachedBatch and will be release when InMemoryRelation.clearCache() is called by this PR
Why are the changes needed?
This changes can help to clean some off-heap user-defined object may be cached in InMemoryRelation or MemoryStore
Does this PR introduce any user-facing change?
NO
How was this patch tested?
WIP
Signed-off-by: Chendi Xue<EMAIL_ADDRESS>

Can one of the admins verify this patch?

Jenkins retest this please

Test build #138835 has started for PR 32534 at commit 844f033.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43357/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43357/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43357/

Test build #138835 has finished for PR 32534 at commit 844f033.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138835/

Merged to master


SndBuf not supported

SndBuf isn't supported as a UGen type in Chunreal.

hmm, this definitely should be possible, though we might have disabled due to errors in our libsndfile code?
what happens when PublicDefinitions.Add("__DISABLE_SNDBUF__"); is removed from Chunreal.Build.cs?

Fully functional on MacOS. Importing a .wav file causes a system error on Windows 11.

@gewang The error looks something like this on Windows 11, UE 5.3.
LogChunreal: "<IP_ADDRESS> (chai)" :(string)
LogChunreal: [chuck](via SndBuf): sndfile error '1' opening 'full_path_to_soundfile.wav'...
LogChunreal: [chuck](via SndBuf): ...(reason: System error : T)

We found that issue was related to compiling with UNICODE-enabled compilation on Windows; SndBuf (more specifically libsndfile) has been modified to explicitly treat filename as ANSI, for now (we may or not look at UNICODE support in the future).
Thanks to @Amir-BK for tracking down the cause of this issue.


Give decks api ability to request tournament decks only.

Hey @bakert,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: f2d5d460-0151-11ea-82f1-cf3ba55cf6d0


[Editor] code editor doesn't let me scroll to the bottom of the script

Godot version:
v3.1.alpha.mono.official
OS/device including version:
Win10 64 bit
Issue description:
I just noticed that I could not scroll to the end of the script. Keep scrolling and the editor will stop 1 line before the end.
The only way I can get to the very bottom of the script, is by holding left click and dragging the scroll bar to the bottom. Not a big thing, could not find an existing issue.
Steps to reproduce:
gif showcasing the issue

Thank you @Mr-Slurpy !

Duplicate of #22744, which is now fixed on master.


check_last_maintenance : filter out small tables

The idea is to avoid false positives generated by empty or very small tables.

The last_vacuum & last_analyze services are supposed to check if the planned maintenances succeed.
If you don't have planned maintenances, you don't need to check them, you don't need to deploy this service, no matter how large are your tables.
This should probably be documented, so @Krysztophe just did it in PR #366.
However, when studying this issue, we found partitioned tables were raising errors even after being vacuumed. The issue #365 has been created to track this.

@Krysztophe insist to say:

on empty tables, it doesn't matter to run vacuum because it's almost cost free. So no need to optimize this use case.

Actually, I disagree with you. You can have planned maintenances, but some people have working tables which are created / destroyed regularly. We could avoid the false positives with an --exclude parameter (#369), but filtering out very small tables (with less than 50 tuples) would be beneficial.
(that being said, the default thresholds of this PR are too high)


Add an Expense Tracker

A Python script for tracking expenses.
This script allows users to record their expenses which makes tracking the expense easy.
The script uses a CSV file to store the data and the user can add, view and exit from the app.

@theadeyemiolayinka Please assign it to me under hacktoberfest, thank you!

Go ahead, @JuanitaCathy

@theadeyemiolayinka do i create a separate folder called python scripts or do it inside src?

All contributors should add their scripts inside a folder named after their username in the src folder and each project should have its own folder in the contributor's folder.
You can refer to the contributing guidelines for further explanation.

@theadeyemiolayinka Please review and let me know if there are any changes.


Wfly 6460 with jaxrs

Overview
This PR includes changes for JSON-P, JSON-B and JAX-RS with the Java EE 8 tech preview. It also replaces/includes #10819. @asoldano could you please give this your approval? @stuartwdouglas please let me know if the module configurations look correct as well.
JSON-P 1.1 WFLY-6460
This replaces JSON-P 1.0 with JSON-P 1.1 when the ee8.preview.mode system property is set to true. One thing to note about this is the system property only seems to work when the user specifies it in the JAVA_OPTS. This may be because elytron accesses the module before the property is set.
Also note that in the JMS client we hard-code the version to 1.0.4 since client itself still relies on JSON-P 1.0. This library is shaded into the client which is the reason for doing this.
Also note that JSON-P 1.0 implementation library includes both the API and the implementation which is my the module.xml exports only the javax.json.* packages.
JSON-B 1.0 WFLY-7185
This introduces the new JSON-B 1.0 API. The module itself will be added to all deployments however the resource will only be added if the ee8.preview.mode system property is set to true.
There are currently not tests for JSON-B however I did test manually and I will follow up with tests after #10833 gets merged.
JAX-RS Upgrade Information
This is a preview of the upgrade that I'm soon submitting for bringing JAX-RS 2.1 RESTEasy implementation in WildFly. I'd like the reviewers to start having a look and letting me know of any additional requirements / changes needed for merging before I tag 3.5.0.Final.
I'm dealing with the needed analysis / test plan in the mean time; besides JSR 370 (JAX-RS 2.1), the RESTEasy component upgrade also pulls few other planned new features.
Few JIRA links:
https://issues.jboss.org/browse/WFLY-9789
https://issues.jboss.org/browse/WFLY-6459
https://issues.jboss.org/browse/WFLY-9786
https://issues.jboss.org/browse/WFLY-9787
Dev analysis:
wildfly/wildfly-proposals#16
wildfly/wildfly-proposals#18
wildfly/wildfly-proposals#19

Looks like I must have messed something up in the rebase. These failures look like this PR caused the issue.

@jamezp @asoldano I guess what James is saying is the reason for JaxrsTestCase blowing up here

@jamezp I was not able to cherry-pick json-b commit on top of current master, json-p one went fine, there is conflict in pom.xml

@kabir @asoldano That's correct. Because of the hard dependency on JSON-B the tests fail because the JSON-B resource is not loaded.

@rsvoboda Yeah, it looks like the group ID change PR caused conflicts. I'll rebase it.

@rsvoboda I've rebased. I also have https://github.com/jamezp/wildfly/tree/WFLY-6460 which is just the two JSON commits.

@jamezp I have prepared test for json-p - look at https://github.com/rsvoboda/wildfly/commits/jsonp
I had to modify server.jvm.args to include -Dee8.preview.mode=true, otherwise JSON-P 1.1 was not loaded. I think it's the same as JAVA_OPTS you had to set in your case.
Feel free to cherry-pick https://github.com/rsvoboda/wildfly/commit/b17bc6970f5bb9aef6422c851908e34d3c0df387 commit or add comments if you spot something which should be fixed in the commit.

hmm, JSON-B stuff doesn't need server.jvm.args tuned, modify-ee8-config.cli with /system-property=ee8.preview.mode:add(value=true) seems to be enough.


Example for getting value to secret from ssm parameter store

Could you please write an example how to create ExternalSecret for ssm parameter store.
For example, I have value that have path /my/full_path/to/value
MY EXAMPLE:
---
apiVersion: external-secrets.io/v1beta1
kind: SecretStore
metadata:
  name: aws-ssm-paramstore
  namespace: ns_name
spec:
  provider:
    aws:
      service: ParameterStore
      # define a specific role to limit access
      # to certain secrets
      role: arn:aws:iam::{aws_account_id}:role/{role_name}
      region: us-east-1
---
apiVersion: external-secrets.io/v1beta1
kind: ExternalSecret
metadata:
  namespace: ns_name
  name: external_secret_name
spec:
  data:
  - key: /my/full_path/to/value
  target:
    name: k8s_secret_name
  secretStoreRef:
    kind: SecretStore
    name: aws-ssm-paramstore

Thanks for help. All examples on the page correlate Secret Store not parameter store.

An ExternalSecret is namespaced and creates secrets only in the very same namespace. You can however use a ClusterExternalSecret that allows you to distribute/create secrets across one or more namespaces if that is a thing you need to do.

I tried example but kubectl get SecretStores,ClusterSecretStores,ExternalSecrets --all-namespaces don't show that secret was created. Is any place where I can look for logs?

@igor-sidorov in your case you can check kubectl describe es external_secret_name -n ns_name for information.
You can check if the secret was created with the command kubectl get secret -n ns_name k8s_secret_name.
To use ClusterExternalSecrets, you need yet another manifest, and be sure to reference target namespaces with specific labels.

When I do kubectl get externalsecrets -A I can see SecretStore and ExternalSecret with state Valid and SecretSynced accordingly.
I see target name in description of externalsecret.external-secrets.io/external_secret_name but I can't find secret by  kubectl get secret -A .
How can I debug that?


[doc] Fix link in rank demo README.md .

Codecov Report

Merging #3759 into master will increase coverage by 0.1%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master    #3759     +/-   ##
===========================================
+ Coverage     51.99%   52.09%   +0.1%
  Complexity      196      196
===========================================
  Files           180      180
  Lines         14307    14350     +43
  Branches        489      513     +24
===========================================
+ Hits           7439     7476     +37
- Misses         6634     6637      +3
- Partials        234      237      +3

Impacted Files
Coverage Δ
Complexity Δ

.../scala/ml/dmlc/xgboost4j/scala/spark/XGBoost.scala
77.12% <0%> (+1.45%)
0% <0%> (ø)
:arrow_down:

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 9e73087...39adc7e. Read the comment docs.

Thanks!


Emphasize the entrypoint in the Project Files tree

Relies on the files API correctly returning isEntrypoint.

There are a few ways we could do this.
Bolding is one option:

but VS Code hardly ever bolds text.
Another is to use list.activeSelectionForeground and potentially list.activeSelectionBackground as well:

the explorer does something similar so this may look like the file is open, or selected in some other way.
VS Code typically uses color over font-weight or other tools to emphasize text so maybe another color could be used. Ideally it would be a VS Code color that themes could change (to avoid light vs dark mode issues)
We could also do something as simple as including it in the details of the tree item:

There are a lot of options.

I would suggest a combination of the second and third option (different color as well as annotated as the entrypoint). Confusion could occur if the filename contains the word "entrypoint", but the proposal also shows it slightly different than the file name, so I think it would be weird, but minimal there. If this is really an issue, we could try "<-- entrypoint" or something for the annotation.


Add highlights.scm

Context: https://github.com/ikatyang/tree-sitter-markdown/issues/19#issuecomment-762634320

🤩🤩

🤩🤩

Afaik, injections.scm is also universal to all tree-sitter implementations, and since Markdown's fenced code blocks are quite important I think it would make sense to also include this file in this repository.
There is already one in the nvim-treesitter plugin, see here, however this is specific to neovim. See this issue for a little more context.

Afaik, injections.scm is also universal to all tree-sitter implementations, and since Markdown's fenced code blocks are quite important I think it would make sense to also include this file in this repository.
There is already one in the nvim-treesitter plugin, see here, however this is specific to neovim. See this issue for a little more context.

Ref: https://tree-sitter.github.io/tree-sitter/syntax-highlighting
There are 3 official supported queries: highlights.scm, locals.scm, injections.scm.

Ref: https://tree-sitter.github.io/tree-sitter/syntax-highlighting
There are 3 official supported queries: highlights.scm, locals.scm, injections.scm.


Please refresh the screenshots in this doc.

Please refresh the screenshots in this doc.

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

ID: 469b31a6-10e1-1460-b20e-8f2c72561675
Version Independent ID: 50da61e1-e28a-63f2-2450-0ec5350b5c20
Content: Manage cluster vertical scaling (scale up) to match demand in Azure Data Explorer
Content Source: data-explorer/manage-cluster-vertical-scaling.md
Service: data-explorer
GitHub Login: @orspod
Microsoft Alias: orspodek

@ManojRaheja
Thanks for the feedback! We have assigned the issue to the content author to investigate this and update the document as appropriate.

@ManojRaheja
Thanks for the feedback! We have assigned the issue to the content author to investigate this and update the document as appropriate.

A work item has been created for this task.

A work item has been created for this task.

#please-close

#please-close


Fix webpack 4 migration guide link to split chunks docs

The link to optimization.splitChunks anchor was broken.

Thanks!


Clarify and simplify PSK

Description
In Mbed TLS, there are two types of PSKs:

Configured, aka static PSK (ssl->conf->psk...): Essentially used client-side although nothing prevents the server from using it. It is set by mbedtls_ssl_conf_psk()
Handshake PSK, aka dynamic PSK (ssl->handshake->psk...): Server-side only. That PSK is bound to the handshake hence doesn't survive renegotiation or session reset.
It is set by mbedtls_ssl_set_hs_psk().
Orthogonally, opaque PSKs used in PSA can be of any of the two aforementioned types (static or dynamic)

Since there are several types of PSKs, they are subject to a priority order. Dynamic PSKs precede static ones as showed in mbedtls_ssl_psk_derive_premaster() and ssl_compute_master().
All this makes PSK management difficult to understand and prone to error.
The goal of this PR is to clarify and simplify the use of PSKs:

The documentation is improved to explicit the precedence order described above
Two internal functions, mbedtls_ssl_get_psk() and mbedtls_ssl_get_opaque_psk(), are introduced to get the PSK with the highest priority. This guarantees that the right PSK is selected by virtue of code sharing and simplifies access to PSK

Requires Backporting
NO

@gbryant-arm The CI seems to fail. Could you take a look?

@gbryant-arm I'm not sure if you can access the CI results now, but in case you can't, the only failures now are in mbed-os testing, which is a know issue unrelated to your PR, and can be disregarded.

I forgot to sign off again, sorry about that...

The pr-merge job fully passed (disregarding broken mbed-os tests) and the pr-head job only failed in an instance of the reconnect test in ssl-opt.sh, a known issue that has been fixed in the meantime, so this does not block this PR.
I'm therefore labeling "ready for merge" and will merge on Tuesday (today's not a good day to merge).


Failed to add 1.9.0 to ios platform - exit code null for pod install

Expected Behaviour
succeed to install 1.9.0 to ios platform
Actual Behaviour
No scripts found for hook "before_plugin_add".
No version specified for phonegap-plugin-push, retrieving version from config.xml
No version for phonegap-plugin-push saved in config.xml
Attempting to use npm info for phonegap-plugin-push to choose a compatible release
Running command: /Users/yosi/code/yosi-app/platforms/android/cordova/version
Running command: /Users/yosi/code/yosi-app/platforms/browser/cordova/version
Running command: /Users/yosi/code/yosi-app/platforms/ios/cordova/version
Command finished with error code 0: /Users/yosi/code/yosi-app/platforms/android/cordova/version
Command finished with error code 0: /Users/yosi/code/yosi-app/platforms/browser/cordova/version
Command finished with error code 0: /Users/yosi/code/yosi-app/platforms/ios/cordova/version
Ignoring invalid version in phonegap-plugin-push cordovaDependencies: 2.0.0 (must be a single version <= latest or an upper bound)
Calling plugman.fetch on plugin<EMAIL_ADDRESS>Calling plugman.install on plugin "/Users/yosi/code/yosi-app/plugins/phonegap-plugin-push" for platform "android
Installing "phonegap-plugin-push" for android
Running command: /Users/yosi/code/yosi-app/platforms/android/cordova/version
Command finished with error code 0: /Users/yosi/code/yosi-app/platforms/android/cordova/version
Finding scripts for "before_plugin_install" hook from plugin phonegap-plugin-push on android platform only.
No scripts found for hook "before_plugin_install".
Install start for "phonegap-plugin-push" on android.
ANDROID_HOME=/Users/yosi/Library/Android/sdk
JAVA_HOME=/Library/Java/JavaVirtualMachines/jdk1.8.0_112.jdk/Contents/Home
Subproject Path: CordovaLib
Running command: /Users/yosi/code/yosi-app/platforms/android/gradlew clean -b /Users/yosi/code/yosi-app/platforms/android/build.gradle -Dorg.gradle.daemon=true -Dorg.gradle.jvmargs=-Xmx2048m -Pandroid.useDeprecatedNdk=true
Parallel execution is an incubating feature.
Incremental java compilation is an incubating feature.
:clean
:CordovaLib:clean

BUILD SUCCESSFUL

Total time: 1.039 secs
Command finished with error code 0: /Users/yosi/code/yosi-app/platforms/android/gradlew clean,-b,/Users/yosi/code/yosi-app/platforms/android/build.gradle,-Dorg.gradle.daemon=true,-Dorg.gradle.jvmargs=-Xmx2048m,-Pandroid.useDeprecatedNdk=true
Beginning processing of action stack for android project...
Installing Android library: push.gradle
Installing Android library: com.android.support:support-v13:23+
Installing Android library: com.google.android.gms:play-services-gcm:9.8+
Installing Android library: me.leolin:ShortcutBadger:1.1.4@aar
Action stack processing complete.
Updating build files since android plugin contained <framework>
Subproject Path: CordovaLib
Install complete for phonegap-plugin-push on android.
Finding scripts for "after_plugin_install" hook from plugin phonegap-plugin-push on android platform only.
No scripts found for hook "after_plugin_install".
Calling plugman.install on plugin "/Users/yosi/code/yosi-app/plugins/phonegap-plugin-push" for platform "browser
Installing "phonegap-plugin-push" for browser
Running command: /Users/yosi/code/yosi-app/platforms/browser/cordova/version
Command finished with error code 0: /Users/yosi/code/yosi-app/platforms/browser/cordova/version
Finding scripts for "before_plugin_install" hook from plugin phonegap-plugin-push on browser platform only.
No scripts found for hook "before_plugin_install".
Install start for "phonegap-plugin-push" on browser.
Error loading cordova-browser
Beginning processing of action stack for browser project...
Action stack processing complete.
Install complete for phonegap-plugin-push on browser.
Finding scripts for "after_plugin_install" hook from plugin phonegap-plugin-push on browser platform only.
No scripts found for hook "after_plugin_install".
Calling plugman.install on plugin "/Users/yosi/code/yosi-app/plugins/phonegap-plugin-push" for platform "ios
Installing "phonegap-plugin-push" for ios
Running command: /Users/yosi/code/yosi-app/platforms/ios/cordova/version
Command finished with error code 0: /Users/yosi/code/yosi-app/platforms/ios/cordova/version
Finding scripts for "before_plugin_install" hook from plugin phonegap-plugin-push on ios platform only.
No scripts found for hook "before_plugin_install".
Install start for "phonegap-plugin-push" on ios.
Beginning processing of action stack for ios project...
Action stack processing complete.
Adding pods since the plugin contained <framework>(s) with type="podspec"
pods.json found in platforms/ios
Podfile found in platforms/ios
Set pods.json for `GoogleCloudMessaging`
Added pod line for `GoogleCloudMessaging`
Set pods.json for `GGLInstanceID`
Added pod line for `GGLInstanceID`
Wrote to pods.json.
Wrote to Podfile.
Running `pod install` (to install plugins)
Running command: pod install --verbose
==== pod install start ====

  Preparing

Analyzing dependencies

Inspecting targets to integrate
  Using `ARCHS` setting to build architectures of target `Pods-Bringg Dispatcher`: (``)

Resolving dependencies of `Podfile`

Comparing resolved specification to the sandbox manifest
  A GGLInstanceID
  A GoogleCloudMessaging
  A GoogleIPhoneUtilities
  A GoogleInterchangeUtilities
  A GoogleSymbolUtilities
  A GoogleUtilities

Downloading dependencies

-> Installing GGLInstanceID (1.2.1)
  > Copying GGLInstanceID from `/Users/yosi/Library/Caches/CocoaPods/Pods/Release/GGLInstanceID/1.2.1-4a317` to `Pods/GGLInstanceID`

-> Installing GoogleCloudMessaging (1.2.0)
  > Copying GoogleCloudMessaging from `/Users/yosi/Library/Caches/CocoaPods/Pods/Release/GoogleCloudMessaging/1.2.0-f37ea` to `Pods/GoogleCloudMessaging`

-> Installing GoogleIPhoneUtilities (1.2.1)
  > Copying GoogleIPhoneUtilities from `/Users/yosi/Library/Caches/CocoaPods/Pods/Release/GoogleIPhoneUtilities/1.2.1-63f25` to `Pods/GoogleIPhoneUtilities`

-> Installing GoogleInterchangeUtilities (1.2.2)
  > Copying GoogleInterchangeUtilities from `/Users/yosi/Library/Caches/CocoaPods/Pods/Release/GoogleInterchangeUtilities/1.2.2-d5bc4` to `Pods/GoogleInterchangeUtilities`

-> Installing GoogleSymbolUtilities (1.1.2)
  > Copying GoogleSymbolUtilities from `/Users/yosi/Library/Caches/CocoaPods/Pods/Release/GoogleSymbolUtilities/1.1.2-631ee` to `Pods/GoogleSymbolUtilities`

-> Installing GoogleUtilities (1.3.2)
  > Copying GoogleUtilities from `/Users/yosi/Library/Caches/CocoaPods/Pods/Release/GoogleUtilities/1.3.2-8bbc7` to `Pods/GoogleUtilities`
  - Running pre install hooks

Generating Pods project
  - Creating Pods project
  - Adding source files to Pods project
  - Adding frameworks to Pods project
  - Adding libraries to Pods project
  - Adding resources to Pods project
  - Linking headers
  - Installing targets
    - Installing target `Pods-Bringg Dispatcher` iOS 8.0

  - Running post install hooks
  - Writing Xcode project file to `Pods/Pods.xcodeproj`
    - Generating deterministic UUIDs

Command finished with error code null: pod install,--verbose
Failed to install 'phonegap-plugin-push':Error: pod: Command failed with exit code null
    at ChildProcess.whenDone (/Users/yosi/code/yosi-app/platforms/ios/cordova/node_modules/cordova-common/src/superspawn.js:169:23)
    at emitTwo (events.js:87:13)
    at ChildProcess.emit (events.js:172:7)
    at maybeClose (internal/child_process.js:827:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:211:5)
Error: pod: Command failed with exit code null
    at ChildProcess.whenDone (/Users/yosi/code/yosi-app/platforms/ios/cordova/node_modules/cordova-common/src/superspawn.js:169:23)
    at emitTwo (events.js:87:13)
    at ChildProcess.emit (events.js:172:7)
    at maybeClose (internal/child_process.js:827:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:211:5)

Reproduce Scenario (including but not limited to)

cordova plugin rm phonegap-plugin-push
cordova plugin add phonegap-plugin-push --variable SENDER_ID="..." --verbose

Cordova CLI version and cordova platform version
cordova --version    6.4.0
cordova platform version android      6.00
cordova platform version ios      4.3.0
pod --version  1.1.1
macOS sierra 10.12.1

Plugin version
Latest 1.9.0

I know this issue looks the same as: https://github.com/phonegap/phonegap-plugin-push/issues/1342
But I am getting exit code "null" on what whom seems like successful pod install.

@yosiat what version of CocoaPods do you have installed?
@shazron have you ever seen anything like this?

@macdonst 1.1.1
cordova --version    6.4.0
cordova platform version android      6.00
cordova platform version ios      4.3.0
pod --version  1.1.1
macOS sierra 10.12.1

At the time of implementation and testing, we tested with CocoaPods 1.0.1 because that was the latest released and stable version. Perhaps something changed with the newer version, and we need to re-test.

@yosiat not sure if this is a big hassle, but see if you can downgrade to CocoaPods 1.0.1 and re-test. If that succeeds, that isolates the problem to CocoaPods 1.1.1

@shazron It works with cocoapods 1.0.1

@shazron Odd, cause I was using 1.1.1

Just re-tested using this procedure, it succeeded (although there was a bunch of curl output to stdout for downloads that I didn't see before):
cordova create test test test
cd test
cordova platform add ios
cordova plugin add phonegap-plugin-push
cordova build

Versions:
cordova --version    6.4.0
cordova platform version ios      4.3.0
pod --version  1.1.1
macOS sierra 10.12.1 (16B2555)

This was the output I saw that I mentioned:
$ cordova plugin add phonegap-plugin-push
Fetching plugin<EMAIL_ADDRESS>via npm
Installing "phonegap-plugin-push" for ios
     % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current

                                    Dload  Upload   Total   Spent    Left  Speed

     0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
   100 1085k    0 1085k    0     0  1666k      0 --:--:-- --:--:-- --:--:-- 1673k

     % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current

                                    Dload  Upload   Total   Spent    Left  Speed

     0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
   100 1596k    0 1596k    0     0  3119k      0 --:--:-- --:--:-- --:--:-- 3124k

     % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                    Dload  Upload   Total   Spent    Left  Speed

   100  119k  100  119k    0     0   357k      0 --:--:-- --:--:-- --:--:--  359k

     % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                    Dload  Upload   Total   Spent    Left  Speed

     0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
   100 1939k    0 1939k    0     0  3745k      0 --:--:-- --:--:-- --:--:-- 3751k

     % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current

                                    Dload  Upload   Total   Spent    Left  Speed

     0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
   100 12789  100 12789    0     0  58099      0 --:--:-- --:--:-- --:--:-- 58397

     % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                    Dload  Upload   Total   Spent    Left  Speed

     0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
   100 1207k    0 1207k    0     0  2196k      0 --:--:-- --:--:-- --:--:-- 2198k

I upgraded cocoapods from 1.0.1 to 1.1.1 using:
gem update cocoapods

Also, my ruby is:
ruby 2.2.3p173 (2015-08-18 revision 51636) [x86_64-darwin15]

Other info:
$ gem update cocoapods
Updating installed gems
Updating cocoapods
Fetching: cocoapods-core-1.1.1.gem (100%)
Successfully installed cocoapods-core-1.1.1
Fetching: claide-1.0.1.gem (100%)
Successfully installed claide-1.0.1
Fetching: cocoapods-deintegrate-1.0.1.gem (100%)
Successfully installed cocoapods-deintegrate-1.0.1
Fetching: cocoapods-downloader-1.1.2.gem (100%)
Successfully installed cocoapods-downloader-1.1.2
Fetching: cocoapods-trunk-1.1.1.gem (100%)
Successfully installed cocoapods-trunk-1.1.1
Fetching: molinillo-0.5.4.gem (100%)
Successfully installed molinillo-0.5.4
Fetching: CFPropertyList-2.3.3.gem (100%)
Successfully installed CFPropertyList-2.3.3
Fetching: nanaimo-0.2.2.gem (100%)
Successfully installed nanaimo-0.2.2
Fetching: xcodeproj-1.4.1.gem (100%)
Successfully installed xcodeproj-1.4.1
Fetching: fourflusher-2.0.1.gem (100%)
Successfully installed fourflusher-2.0.1
Fetching: gh_inspector-1.0.2.gem (100%)
Successfully installed gh_inspector-1.0.2
Fetching: cocoapods-1.1.1.gem (100%)
Successfully installed cocoapods-1.1.1
[removed a bunch of doc install logs...]
Gems updated: CFPropertyList claide cocoapods cocoapods-core cocoapods-deintegrate cocoapods-downloader cocoapods-trunk fourflusher gh_inspector molinillo nanaimo xcodeproj

It's possible that some gems have been updated in the interval between when you installed and I installed 1.1.1, possibly fixing some bugs since some gemspecs have ranges: https://rubygems.org/gems/cocoapods/versions/1.1.1
Try uninstalling and re-installing the cocoapods gem perhaps to see if this is the case.

I didn't go too deep into the dependency rabbit hole, but xcodeproj had two updates since 1.1.1 release: https://rubygems.org/gems/xcodeproj

@shazron using:
$ pod --version
1.1.1
$ xcodeproj --version
1.4.1

and I can't reproduce.

My ruby version is:
ruby 2.3.1p112 (2016-04-26 revision 54768) [x86_64-darwin16]

And I install cocoapods 1.1.1 from homebrew brew install cocoapods

@yosiat did you try reinstalling cocoapods? My point above was that a dependency was updated that you may not have picked up from an older install.

cordova platform remove ios and cordova prepare ios solved this for me.

@shazron yes.

I used 1.1.1 too and worked fine.
Can you install with gem install cocoapods instead of using homebrew? (might need sudo)

@Kjens93 should there be a cordova platform add ios in the middle there after remove? Or I'm assuming you had the platform saved already in config.xml most likely thus prepare restored it.

I got the same issue, impossible for me to install the plugin on ios :
`
[!] The Podfile does not contain any dependencies.
Error: pod: Command failed with exit code 1 Error output:
[!] The Podfile does not contain any dependencies.
`
and then
Failed to install 'phonegap-plugin-push':Error: pod: Command failed with exit code 1
when I run ionic build ios I got this message :
Cordova/build-debug.xcconfig' because it could not be loaded. 	Reason: File could not be parsed due to preprocessing errors:     pods-debug.xcconfig line 2: Unable to find included file "Pods/Target Support Files/Pods-X/Pods-X.debug.xcconfig"
Hope this help

@tgensol what version of cordova-ios are you using?

Hi guys,
I have same kind of issue with this plugin on iOs(no problem with android). I try to compile my project and i got this error:
Linker command failed with exit code 1 (use -v to see invocation)
ld: library not found for -Pods-MyProject
clang: error: linker command failed with exit code 1 (use -v to see invocation)

I am a newbie in iOs environment and mac, so i don't what information do you need.
Cocoapods version is 1.1.1 (installed via gem install), cordova-ios version is 4.3.0.
I tried this command (i'm using ionic):
ionic platform rm ios
ionic state reset  --plugins
ionic platform add ios
cd /platforms/ios
pod install

And then open project in xcode and try to build and run it.
Sry if i'm not in the good thread, feel free to move it if needed (or ask me to create a new one).

Ok i was opening the .xcodeproj in xcode instead of the xcworkspace. Now the build is ok. Sry for the spam.

@macdonst  I have :
cordova 6.4.0
android 6.0.0
ios 4.3.0
pod 1.1.1

I tried with a new project, I got this again
Failed to install 'phonegap-plugin-push':Error: pod: Command failed with exit code 1     at ChildProcess.whenDone (path_removed/plugin/platforms/ios/cordova/node_modules/cordova-common/src/superspawn.js:169:23)     at emitTwo (events.js:106:13)

Anyone getting the error:
Installing "phonegap-plugin-push" for ios
Failed to install 'phonegap-plugin-push':Error: pod: Command failed with exit code 1

should go to the command line and run pod repo update then try to re-install the plugin.

Your command line was not working, I removed cocoapods with 'gem uninstall cocoapods', and then used homebrew to install cocoapods, and it is working now !

An update. After updating Ruby and stuff to get Jekyll working for another project on the same system, it somehow "corrupted" or upgraded some of my Ruby gems (fun!) and I got the same Command failed with exit code null error.
I uninstalled and reinstalled cocoapods (current release, not any pre-release) using the steps here then it all works again. Basically what others in this thread have found as well...


Unable to get Repetition Working

I can create new tasks, but I haven't been able to get repetition working.  Sample code below.  Windows 2003.  After running the code, the task appears in the Scheduled Tasks folder, but in the task's properties, Schedule tab, Advanced Schedule Options, the Repeat Task checkbox is not checked.  Am I missing something or is it broken?

        Dim ts As New TaskService
        Dim td As TaskDefinition = ts.NewTask
        td.RegistrationInfo.Description = "Test"
        Dim wt As New WeeklyTrigger()
        wt.DaysOfWeek = DaysOfTheWeek.Monday Or DaysOfTheWeek.Tuesday Or DaysOfTheWeek.Wednesday Or DaysOfTheWeek.Thursday Or DaysOfTheWeek.Friday Or DaysOfTheWeek.Saturday
        wt.StartBoundary = Now
        wt.Repetition.Duration = TimeSpan.FromMinutes(660)
        wt.Repetition.Interval = TimeSpan.FromMinutes(10)
        Dim t As Trigger = td.Triggers.Add(wt)
        't.Repetition.Duration = TimeSpan.FromMinutes(660)
        't.Repetition.Interval = TimeSpan.FromMinutes(10)
        td.Actions.Add(New ExecAction("notepad.exe", "E:\Temp\Test.log", Nothing))
        ts.RootFolder.DeleteTask("Test")
        ts.RootFolder.RegisterTaskDefinition("Test", td)
Originally posted: 2008-11-10T14:43:05

Thanks for the quick turnaround.  I'll test it out tonight.
Originally posted: 2008-11-10T15:05:50

Works great, thanks again.
Originally posted: 2008-11-10T17:22:30

I am trying to set the EndBoundry in xp. But it's not working.
Originally posted: 2009-02-25T08:42:21

Fixed with update to code today and in released version 1.2.
Originally posted: 2008-11-10T14:43:41


icons in buttons off-centre in "compact layout"

In none of the buttons-with-icons the icons sit comfortably in the centre.

I believe this is an issue in core. I opened a task for small buttons in the core stylesheet: https://github.com/atom/atom/issues/8412


Fixes destroy to destroy_all

Original commit for this migration had an error. Migration not yet run so fixed in place.

👍


PhaserRenderer small refactor

It now extends Phaser.Game instead of containing it; added settings for canvas size and resize callback interval

it looks better when you load in and stand still, but its still pretty bad when moving around.

I then changed the camera.startFollow() roundPixels param to false and maybe it was a bit better.

You are correct, current settings seem to have the best performance and look so I reverted it. I also added a couple of settings to phaser config, notice the one that defines resize interval (something we discussed regarding scaling and resize). the other one simply forces round values for canvas size, which can improve performance on old devices.

100 ms resize interval feels really nice


How to generate small size image with synthtext background?

Hi, I am trying to generate small size synthtext images, e.g. 64*64 output size.
I do not have any available segmentation images by myself, so I am trying to resize/crop the background data which downloaded from this repo.
The org image size is around (500, 500). So I resized the img, seg and depth to (64, 64). But I do not know how to make fake values for labels and area.
If I do not change them, no text is generated. If I fake them to zeros or ones, it does not work either.
I just need to generate 1~5 line words in (64,64) images. I do not need these texts to be generated on a safe area. Rrandom places is ok for my case. Do you have any suggestions?
Thanks a lot.

You can try using a constant valued image/matrix for seg and depth images, and set the area and label correspondingly (to scalar values).
You could also extract text regions using the bounding boxes after they have been generated on the original images as provided in this code repo.


Start using GitHub Projects again

GitHub Projects are a great way to keep us organized on what's done, what's being worked on, and what's available to be claimed. It'd be great to start organizing issues and PR's there again.

I agree, this will help us to get a little more organized. I think Github projects will be good enough, but if we want something a little more fancy we can use https://waffle.io/pricing, it is free for open source projects.

Was thinking about this early :)
I closed it since we weren't really relying on it earlier. Definitely open to using it once more if we all decide to rely on it nicely.
How are you guys envisioning it? Split it by version releases and group issues and PRs in each section?

Maybe have the name of the Project be the name of the release (so like 1.4.0) and within the project have To Do, In Progress, and Completed.

Done :)
I've added separate projects for v.1.4.0, v.1.4.1 and  v.1.5.0. I think once we have everything in for 1.4.1 for example for bug fixes, we can add a new one for v.1.4.2 and so forth.
Once we have 1.4.0 merged in, we'll align this project roadmap with separate branches in the repo as well to keep things streamlined.
Also please let me know if you have edit access to projects. I would really like all of you folks to be able to add/modify the project timeline @alejandronanez @machour @andrewda @lex111 @chinesedfan

@housseindjirdeh works like a charm!

Works fine for me :)

@housseindjirdeh No problem. I love tasks!

Beautiful <3
Feel free to move things to any of the releases you see fit folks. We can always clarify further in our gitter channe; 🎉


[WIP] Try to get faster builds

What is this PR trying to achieve?
Make our builds go faster!
Who is this change for?
Bored developers!

Notes:

Trashing the common/target and ~/.ivy2 directories on my local machine causes a compile to go from 22s to 446s (!!).

Plan is to get a couple of runs of a complete build on Circle to see if this makes a difference, and merge if it does.

Initial two runs, we save about 6 minutes from caching.
Some other ideas:

Cache the Docker images used in the tests
Don’t stop and restart the Docker containers between tasks
Don’t run a stack of separate sbt tasks each time; we’re paying the cost of starting the JVM multiple times

Take advantage of Circle’s parallelism?

If we intend to move to Jenkins, we should try and make sure changes here are portable.

Superseded by #188.


PubChem Substance needs to be secondary source

There is some confusion... there are two interpretations of this field in the datasources.txt...
One is that this is used for PathVisio to say if it should show up...
The second is that it reflects if the database is the creator of the matching identifiers... for example, HMDB00xxxxx has the HMDB as primary ("1") and NugoWiki as secondary ("0")... the latter also used (it no longer exists) the HMDB identifiers, but did not create them.
This needs to be resolved first. For now, I go with the first interpretation, and then it needs to be primary, as PubChem SIDs are minted by PubChem.


Update sdk-go for latest interaction API

What was changed

Adds UpdateWorkflowExecution to the client API and relevant interceptors
Adopts expectation that interaction invocations will be attached to a PollWorkflowTaskQueueResponse (had previously been attached to the WFT Started event).
Comment formatting for Go 1.19+

Why?

Part of sync update project.  Client api is made public only because there is an assertion that client.Client and internal.Client are compatible interfaces.
Checklist

Closes

How was this tested:

Unit tests & sdk-features

Any docs updates needed?

I'm going to pull down this PR and rework it in favor of something that supports async update invocation from day 1


[NetAppFiles] BREAKING CHANGE: Add commands for new features and updates for netappfiles 2020-06

Description

This PR is for new commands for new features in NetAppFiles version 2020-06 along with some updates.
The following was changed or added:

Added optional boolean parameter named --force-break-replication to 'az netappfiles volume replication suspend' to be able to force break a volume replication.
Added new command 'az netappfiles volume replication re-initialize' that is used to re initialise a volume replication.
Added new command 'az netappfiles volume pool-change' with a new parameter named new_pool_resource_id to indicate which pool to change the volume to.
Added new command group 'az netappfiles snapshot policy' with following commands:

show, list, volumes, delete, update and create.

Added new command group 'az netappfiles account backup' with following commands:

show, list and delete

Added new command group 'az netappfiles volume backup' with the following commands:

show, list, delete, update and create

Added new command group 'az netappfiles account backup-policy' with following commands:

show, list, delete, update and create

Added new command group 'az netappfiles vault' with command 'list'
Added optional parameters to 'az netappfiles account ad add'

kdc-ip, ad-name, server-root-ca-certificate and backup-operators

BREAKING CHANGE: Removed service-level parameter from command 'az netappfiles pool update'
Added optional parameter 'qos-type' to 'az netappfiles pool update' and 'az netappfiles pool create'
Added optional parameters to 'az netappfiles volume create'

snapshot-policy-id, backup-policy-id, backup-enabled, backup-id, policy-enforced, vault-id, kerberos-enabled, throughput-mibps, snapshot-directory-visible, security-style, kerberos5-read-only, kerberos5-read-write, kerberos5i-read-only, kerberos5i-read-write, kerberos5p-read-only, kerberos5p-read-write and has-root-access

Added optional parameters to 'az netappfiles volume update'

vault-id, backup-enabled, backup-policy-id, policy-enforced and throughput-mibps

Testing Guide

History Notes

[NetAppFiles] BREAKING CHANGE: az netappfiles pool update: Removed service-level from parameters.
[NetAppFiles] az netappfiles pool update: Added optional parameter qos-type.
[NetAppFiles] az netappfiles pool create: Added optional parameter qos-type.
[NetAppFiles] az netappfiles volume replication suspend: Added force-break-replication as optional parameter.
[NetAppFiles] Add az netappfiles volume replication re-initialize: New command added to re-initialise replication.
[NetAppFiles] Add az netappfiles volume pool-change: New command to change the pool of a volume.
[NetAppFiles] Add az netappfiles snapshot policy: New command group with list, delete, update, show, create and volumes commands.
[NetAppFiles] Add az netappfiles account backup: New command group with show, list and delete commands
[NetAppFiles] Add az netappfiles volume backups: New command group with show, list, delete, update and create commands.
[NetAppFiles] Add az netappfiles account backup-policy: New command group with show, list, delete, update and delete commands.
[NetAppFiles] Add az netappfiles vault list: New command added.
[NetAppFiles] az netappfiles account ad add: Added optional parameters kdc-ip, ad-name, server-root-ca-certificate and backup-operators
[NetAppFiles] az netappfiles volumes create: Added optional parameters snapshot-policy-id, backup-policy-id, backup-enabled, backup-id, policy-enforced, vault-id, kerberos-enabled, throughput-mibps, snapshot-directory-visible, security-style, kerberos5-read-only, kerberos5-read-write, kerberos5i-read-only, kerberos5i-read-write, kerberos5p-read-only, kerberos5p-read-write and has-root-access.
[NetAppFiles] az netappfiles volume update: Added optional parameters vault-id, backup-enabled, backup-policy-id, policy-enforced and throughput-mibps

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

NetAppFiles

@jsntcy @kairu-ms Is it possible to get this PR going?

@haroldrandom  @jsntcy @kairu-ms Can we please get this reviewed

@haroldrandom @kairu-ms @jsntcy please help with this PR.

/azp run

@haroldrandom rebase complete


Fatal error: Cannot use Config\Services as Services

Fatal error: Cannot use Config\Services as Services because the name is already in use in /system/CodeIgniter.php on line 38
Working after remove below line from /system/CodeIgniter.php
use Config\Services;

I ran into a similar issue and it had to do with my PHP version. I updated to 7.15 and it fixed the issue:
https://github.com/bcit-ci/CodeIgniter4/pull/582

@abldev What version of PHP are you using? Are you on Windows?

PHP Version 7.0.8
OS: OS X Yosemite, Ver : 10.10.5

I've seen that happen twice - once on a Windows setup where paths weren't configured properly (would have to scour the forums for the link), or as @daylightstudio said, for versions of PHP under 7.0.15. IIRC, the person with the previous issue was using 7.0.10.
Please try upgrading and let me know if that works for you.

Its works after upgraded PHP version to 7.1.4.
I test after remove 'use Config\Services;' line and works for me.

Does it work without removing that line, though?

Yes, it work

I was removing that line as well. Then I upgraded my PHP version to the newest I could for Wamp and it worked ever since.

Sweet. Will update the readme's minimum PHP version to 7.0.15, then, unless someone has it working on Windows with an earlier build?


Deprecate BaseProfiler in favor of Profiler

What does this PR do?

Fixes #11409
Does your PR introduce any breaking changes? If yes, please list them.

Before submitting

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

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

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

Did you have fun?
Make sure you had fun coding 🙃

@kaushikb11 Will do!

Oh, I didn't know! Then no stress. Thanks!


add zouyx as pmc

What's the purpose of this PR
add zouyx as pmc as is announced in https://github.com/ctripcorp/apollo/discussions/3762
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Read the Contributing Guide before making this pull request.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit tests to verify the code.
[x] Run mvn clean test to make sure this pull request doesn't break anything.
[x] Update the CHANGES log.

Codecov Report

Merging #3763 (54e6bda) into master (6463904) will increase coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3763      +/-   ##
============================================
+ Coverage     50.48%   50.52%   +0.04%
- Complexity     2392     2393       +1
============================================
  Files           458      458
  Lines         14472    14472
  Branches       1467     1467
============================================
+ Hits           7306     7312       +6
+ Misses         6662     6658       -4
+ Partials        504      502       -2

Impacted Files
Coverage Δ

...ervice/service/ReleaseMessageServiceWithCache.java
87.05% <0.00%> (+1.17%)
:arrow_up:

...mework/apollo/portal/component/PortalSettings.java
65.62% <0.00%> (+7.81%)
:arrow_up:

Continue to review full report at Codecov.

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

I have read the CLA Document and I hereby sign the CLA


Warning or ... ?

> The Indirect Display driver is a user mode only model with no support for kernel mode
> components, hence the driver is able to use any DirectX API's in order to process the
> desktop image. In fact, the IddCx provides the desktop image to encode in a DirectX
> surface.
(emphasis mine)
It's not clear whether this implies that such a user-mode driver is excluded from interacting with, say, OpenGL or Vulcan (thus making it incompatible)? Is this meant to be a warning (i.e. "a driver of this type can only use DirectX"), or is it just informational?
A sentence could be added to mention alternatives to DirectX and how the aforementioned paragraph relates to them, if at all.

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

ID: d294cf2c-339d-15ac-0170-9142903e772a
Version Independent ID: aaf94242-9e91-9b8e-9ecc-9e40e855d95d
Content: Indirect Display Driver Model Overview - Windows drivers
Content Source: windows-driver-docs-pr/display/indirect-display-driver-model-overview.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @lorihollasch
Microsoft Alias: loriwhip

@mr-intj , you raise a very important point here. I've added clarification that libraries such as OpenGL and Vulkan are not appropriate for driver use.


This is fairly uncontroversial to merge, so I'm just going to go ahead and merge it.


Fix warnings in new nightly

Fixes #1143.
I just went with the implementation @maksimsco suggested on discord (apologies if you are not they here as well), there doesn't seem to be a really nice way to move a mutable reference to a slice forwards through the slice now.

I don't think we should do this-- as I pointed out in https://github.com/rust-lang/rust/issues/52671, I think it should be fine for an empty type (like []) to be promoted and have mutable references to it shared.

Dropped the ReadExact change, now this is just removing some unnecessary muts.


why different thresholds

I run the LFW database and the best threshold for accuracy is 1.24, and the accuracy is 99.2%
But when I run my own Asian people database, the best threshold is 0.76, and the accuracy is 94.2%
I am quite confused!!
NEED SOME HELP!
THANK YOU!

you can train with CASIA_Webface datasets, that maybe help a little.

What do you mean by 'run'? Did you train or just validate?@LiuNull

数据集不一样得到的结果不一样是完全没问题的，因为每个数据集的特征不一样，门限是根据这些特征找到的最佳结果。实际上lfw不能当训练集，所以tengerye问你为啥用run

The lfw images are totally collected from the the wild, so the distance between two images of the same person could be farther. while your data_set maybe collected under some constrains which would make the distance closer. The facenet get the best threshold from range(0,4,0.01) to get the highest accuracy.


Node#before is deprecated. Use Node#raws.before

Hello
When I use postcss-import and then postcss-next, there is this warning in the console:
Node#before is deprecated. Use Node#raws.before
I made a repo to reproduce here:
https://github.com/francoisromain/test-postcss-import-cssnext

This is likely a duplicate of #374.


Create image failed,:jenkins is preparing [REPLACEMENT ISSUE]

The original issue
Id: 1611
Title: Create image failed,:jenkins is preparing

could not be created.
This is a dummy issue, replacing the original one. It contains everything but the original issue description. In case the gitlab repository is still existing, visit the following link to show the original issue:
TODO

In GitLab by @luotian-github on Nov 8, 2019, 07:57
please Gu and Cui confirm the test version

In GitLab by @guyingyan on Nov 8, 2019, 07:57
1.0-767-g9cca511 is not last version , the last version 1.0-780-gb2f8e94.tgz, please use the laster version

In GitLab by @cuiaq123 on Nov 8, 2019, 07:57
resolved in 1.0-780-gb2f8e94

In GitLab by @sokril on Nov 8, 2019, 08:26
closed

In GitLab by @sokril on Nov 8, 2019, 08:26
assigned to @cuiaq123


Always show all selected facet rows

Codecov Report

Merging #82 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #82   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files         101    101
  Lines        2332   2332
  Branches      514    513    -1
=====================================
  Hits         2332   2332

Impacted Files
Coverage Δ

src/components/facets/Facet.tsx
100% <100%> (ø)
:white_check_mark:

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 3944216...4dd3a2f. Read the comment docs.


Upgrade terraform-provider-aws to v5.82.1

This PR was generated via $ upgrade-provider pulumi/pulumi-aws --kind=provider --target-version=5.82.1.
New resources:
apigateway/domainNameAccessAssociation.DomainNameAccessAssociation
cloudfront/vpcOrigin.VpcOrigin
memorydb/multiRegionCluster.MultiRegionCluster
networkmanager/dxGatewayAttachment.DxGatewayAttachment
rds/clusterSnapshotCopy.ClusterSnapshotCopy
New functions:
servicecatalog/getAppregistryAttributeGroupAssociations.getAppregistryAttributeGroupAssociations

Upgrading terraform-provider-aws from 5.81.0  to 5.82.1.
Fixes #4953

"aws:elasticache/replicationGroup:ReplicationGroup":
🟡 inputs: "atRestEncryptionEnabled" type changed from "boolean" to "string"
🟡 properties: "atRestEncryptionEnabled" type changed from "boolean" to "string"
That seems bad at a first glance. Checking why that happened

https://github.com/hashicorp/terraform-provider-aws/pull/40514 changed atRestEncryptionEnabled of aws:elasticache/replicationGroup:ReplicationGroup to a nullable bool which we model as a string in the bridge (https://github.com/pulumi/pulumi-terraform-bridge/issues/1514).
The ReplicationGroup is a rather widely used resource and atRestEncryptionEnabled is a setting almost everybody running Redis/Valkey in production would set. This is a rather severe breaking change that we cannot ship in a minor IMO

regress-4457 is failing because a new property was added to autoscaling.Group. That test is executing pulumi import and then editing the code to add the necessary changes. The problem is that our python tests use the latest release version of the SDK and that does not include newly added properties, but pulumi import will include them in the generated code. Chicken/Egg problem...
We should rewrite that test to use nodejs (or yaml?) because they are not susceptible to this issue.

Added override for ReplicationGroup atRestEncryptionEnabled to be a boolean and added a test for it

This was in https://github.com/hashicorp/terraform-provider-aws/releases/tag/v5.82.0


Don't hold a lock when calling a callback in ossl_namemap_doall_names

We don't want to hold a read lock when calling a user supplied callback.
That callback could do anything so the risk of a deadlock is high.
Instead we collect all the names first inside the read lock, and then
subsequently call the user callback outside the read lock.
Fixes #14225

Fixup pushed addressing the feedback. Please take another look.

Fixup pushed addressing a CI failure.
Please can someone re-approve this for inclusion in the current sprint?

approval holds

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged to master.


oqsprovider external test failing on master

The last two github CI master builds have failed in the oqsprovider external test:
https://github.com/openssl/openssl/runs/7551684870?check_suite_focus=true
https://github.com/openssl/openssl/runs/7554971041?check_suite_focus=true
But the one before that was fine:
https://github.com/openssl/openssl/runs/7541090009?check_suite_focus=true
The PR that got pushed where the state seemed to change (#18878) is a documentation only change so it doesn't seem like it could be the culprit. So I'm not sure what's changed to cause this.

@baentsch

I am already working on this - at least with temporary fix.

Fix in #18899 hopefully.


[vcpkg] Reduce dependencies and contents of vcpkgpaths.h

vcpkg/vcpkgpaths.h is omnipresent throughout the codebase; every dependency of this file slows compilation and makes circular dependencies more difficult and more common when writing new code.
This PR reduces the omniscience of this file to only use vcpkg/base/* files.
As a drive-by improvement, this PR (nearly) removes all uses of global, pre-defined triplets from the main tool. An exception still remains for per-platform default triplets, but this is logically the only place which should require hard-coded definitions.
(per-commit review is recommended)
@strega-nil

[1/111] Building CXX object CMakeFiles\vcpkg-test.dir\src\vcpkg-test\dependencies.cpp.obj
FAILED: CMakeFiles/vcpkg-test.dir/src/vcpkg-test/dependencies.cpp.obj
"C:\PROGRA~2\Microsoft Visual Studio\2019\Enterprise\VC\Tools\MSVC\14.26.28801\bin\Hostx86\x86\cl.exe"  /nologo /TP -DVCPKG_USE_STD_FILESYSTEM=1 -IC:\agent\_work\1\s\toolsrc\include /DWIN32 /D_WINDOWS  /GR /EHsc /MDd /Zi /Ob0 /Od /RTC1   -FC -permissive- -utf-8 -W4 -analyze -WX -std:c++17 /showIncludes /FoCMakeFiles\vcpkg-test.dir\src\vcpkg-test\dependencies.cpp.obj /FdCMakeFiles\vcpkg-test.dir\ /FS -c C:\agent\_work\1\s\toolsrc\src\vcpkg-test\dependencies.cpp
C:\agent\_work\1\s\toolsrc\src\vcpkg-test\dependencies.cpp(104): error C2039: 'X86_WINDOWS': is not a member of 'vcpkg::Triplet'
C:\agent\_work\1\s\toolsrc\include\vcpkg\vcpkgpaths.h(48): note: see declaration of 'vcpkg::Triplet'
C:\agent\_work\1\s\toolsrc\src\vcpkg-test\dependencies.cpp(117): error C2039: 'X86_WINDOWS': is not a member of 'vcpkg::Triplet'
C:\agent\_work\1\s\toolsrc\include\vcpkg\vcpkgpaths.h(48): note: see declaration of 'vcpkg::Triplet'
C:\agent\_work\1\s\toolsrc\src\vcpkg-test\dependencies.cpp(125): error C2039: 'X86_WINDOWS': is not a member of 'vcpkg::Triplet'
C:\agent\_work\1\s\toolsrc\include\vcpkg\vcpkgpaths.h(48): note: see declaration of 'vcpkg::Triplet'
C:\agent\_work\1\s\toolsrc\src\vcpkg-test\dependencies.cpp(132): error C2039: 'X86_WINDOWS': is not a member of 'vcpkg::Triplet'
C:\agent\_work\1\s\toolsrc\include\vcpkg\vcpkgpaths.h(48): note: see declaration of 'vcpkg::Triplet'
C:\agent\_work\1\s\toolsrc\src\vcpkg-test\dependencies.cpp(139): error C2039: 'X86_WINDOWS': is not a member of 'vcpkg::Triplet'
C:\agent\_work\1\s\toolsrc\include\vcpkg\vcpkgpaths.h(48): note: see declaration of 'vcpkg::Triplet'
C:\agent\_work\1\s\toolsrc\src\vcpkg-test\dependencies.cpp(147): error C2039: 'X86_WINDOWS': is not a member of 'vcpkg::Triplet'
C:\agent\_work\1\s\toolsrc\include\vcpkg\vcpkgpaths.h(48): note: see declaration of 'vcpkg::Triplet'