fix: update constraints commands syntax for pcs-0.12

Enhancement:
Update the role to pcs-0.12 command syntax
Reason:
New version of pcs brings updated syntax of some commands. Previous syntax is deprecated but it still works. It will be dropped eventually, though, and the role should be ready for that.
Result:
Role uses new pcs syntax if new enough pcs is available.
Issue Tracker Tickets (Jira or BZ if any):
https://issues.redhat.com/browse/RHEL-45303

[citest]


sql: Note that system databases accessible only to root

In a customer Slack thread, there was some confusion as to why the system database didn't show up in the SHOW ALL response. In this case, the user was not root, and the system database is accessible only to root.
Eventually, we'll need to add docs on system tables, but for now, we should update the SHOW DATABASES docs to mention that system only shows for the root user.
@andreimatei

Close in favor of https://github.com/cockroachdb/docs/issues/1291.


Add lambda support

Resolves #4
Although the hubverse_transform package in this repo can be installed and run anywhere, our most immediate need is to run it as an AWS Lambda function (Lambda is AWS's function-as-a-service offering).
To get this deployed to the hubverse-transform-model-output Lambda that already exists in the Hubverse's AWS account*,  this PR adds two things:

A handler function that is invoked whenever an AWS event triggers the lambda (when a hub's bucket receives a new model-output file, for example)
A deployment script that packages the code into the .zip structure required by lambda

Approved via joint review session.


provide pre-built releases of xhyve

make it easy for people to download a single archive with CoreOS and xhyve bundled together

we should make a brew for it

@polvi agreed homebrew should have it. It would be cool to have a single archive with everything you need for less sophisticated users.

:+1:

@philips @polvi I have made such App which has everything prebuilt and also is Status Bar App, where from the menu you can control boot, halt and etc. It's only external dependency is iTerm2.
https://github.com/rimusz/coreos-xhyve-ui

@rimusz really GREAT work! (and thanks for the mention of my humble hacks in the credits)

@AntonioMeireles thanks
I had to credit you, for your awesome work there :)


az keyvault secret list --id expects key vault uri instead of resource id

Describe the bug
Command Name
az keyvault secret list
Errors:
The  --id parameter, but it expects the key vault uri instead of a resource Id.
This is inconsistent with other commands.
Expected Behavior
--id should accept a resource id and there should be an optional/alternative input for key vault uri.
Environment Summary
Linux-5.4.72-microsoft-standard-WSL2-x86_64-with-glibc2.2.5
Python 3.8.9
Installer: HOMEBREW

azure-cli 2.22.0 *

Extensions:
azure-devops 0.18.0
aks-preview 0.5.5
import-export 0.1.1

thanks for reporting, this is a legacy design. WIll need to revisit it.


Update rshapes.c

Add a much more efficient GetCollisionRec implementation

@manuel5975p Please, let me know if you review this PR as proposed on Discord for merging! :)


Windows 10 search is not working

Describe the bug
Windows 10 Search is not working. Not able to type in
To Reproduce
Creating an ISO without appx packages with microwin
Expected behavior
The search bar should work.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
CTF loader and windows search is running in task manager. But I found that I can't click on Indexing Options Control Panel.

I found out that creating an iso with microwin is causing this issue. I have created another iso without removing any packages and keeping both edge and defender. Somehow microwin was still able to brick the function.

https://www.reddit.com/r/ChrisTitusTech/comments/1bx41wl/used_microwin_with_windows_10_search_doesnt_work/

Any update?

Any update?

it still dosent work.


Create coder6586.html

Your pull request needs further work because of the following reason(s):

Your file does not clearly show a link to your github profile.

Once you have resolved the issues, comment on this request to be re-judged
This is an automated response, beep boop.
if you think there is a mistake, contact a maintainer

ce4a63e72fe246bc43cffa3656bd4ca528d4af3e


Manage access interface shows ability to delete owner for editors where there are multiple owners.

The following sequence illustrates this bug, which is a manage access appearance bug (not an actual access control bug)

User A creates a resource
User A assigns user C as Owner
User A assigns user B as editor
User B logs in and edits the resource
User B enters manage access.  At this point the interface displays an X next to each owner giving the impression that they can be deleted.
User B clicks the X from user A's access.
User A's entry in manage access disappears giving the impression that their access has been removed.  However after close and refresh user A is back on the list indicating that access was not removed.  However User B should not have been presented with the X to suggest they can delete users with owner settings.

I've made changes accordingly that disables parts of the interface for granting access that the current user doesn't have.

We need to change the API to avoid the ability to make the change.

I plan to make API change.

This is fixed.

This is fixed.


Default config processing images into the wrong distribution folder

On a new repo with no changes to package.json or imagemin.js I am seeing files placed in /assets/_src/images going to /assets/images/assets/_src/images. SCSS and JS seem to be compiling to their proper distribution folders. Changing config.image.dist in package.json does not appear to have any effect on the final location of the image files.

I did run NPM audit fix after install but the only package it appears to have upgraded is BrowserSync according to the packages.json file

Hi @lmerriam, I couldn't see this bug. npm run dev, npm run build both minify images from /assets/_src/images/ to /assets/images/ and copy them to /sites/assets/images.
I also tested changing $npm_package_config_image_dist to "assets/img". Files seem to be minified to the proper /assets/img folder and copied to /sites/assets/img.
Would you tell me your OS?


Remove-PnPTeamsChannel throws an exception, the team channel is deleted though

Reporting an Issue or Missing Feature
Issue about cmdlet "Remove-PnPTeamsChannel".
Expected behavior
The team channel to be removed, without throwing any exceptions.
Actual behavior
The team channel is actually removed, but the cmdlet returns an exception:
"Remove-PnPTeamsChannel: Channel not found".
Steps to reproduce behavior
Remove-PnPTeamsChannel -Team $team.GroupId -Identity $channel.Id
What is the version of the Cmdlet module you are running?
2.4.0
Which operating system/environment are you running PnP PowerShell on?

Windows

I'll open a PR to fix this, the message "Channel not found" is shown when the channel has been deleted successfully, so you can ignore that error for the time being.

Great! Thanks for your help.

@frbori - this issue has been fixed by @jackpoz , will be available in tomorrow's nightly builds as well as the next major release.
Thanks for raising this and thanks for fixing it :)


Feature: Expor/Import as curl command

Originally reported on Google Code with ID 301
Curl is a popular HTTP debugging tool. If we could export and import request settings
as curl commands, it would integrate better with command line workflows (not everyone
uses chrome-rest-client, hehe).

Reported by apennebaker.ni on 2015-02-03 15:58:36

This will be done.
It should be possible to save/copy the request as a cURL command.
There also should be possible to save data for other shell commands. Need to determine list of available commands.

This issue is outdated and probably will be removed as wont fix.
New design assumes that the app can export the request in different languages / frameworks / notation.

+1

As a proof of concept:

Would love this idea! Where just going to suggest this but saw this issue.
Would be so easy to go into the web inspector, copy a network request as curl and try it out in adv rest client.
Also able to generate whatwg fetch from a curl command would be grate


PROBLEMS ESTIMATING PANEL DATA

Hi!
I had no problems using a cross-sectional model. But when I try to use the panel model, it says that "The sfpanel() and/or sfdist() are not specified correctly."
var"#sfmodel_spec#43"(::Bool, ::typeof(SFrontiers.sfmodel_spec), ::Tuple{Symbol, Vector{Symbol}}, ::Vararg{Tuple{Symbol, Vector{Symbol}}})@SFmainfun.jl:212
sfmodel_spec@SFmainfun.jl:99[inlined]
top-level scope@Local: 1[inlined]
Everything seems correct in terms of coding. It seems that there is a problem with the sfpanel code, because the cross-sectional works just fine.
I am a beginner in Julia, so I may be missing an easy solution here.
This is my code:
sfmodel_spec(sfpanel(TFE_WH2010), sftype(prod),  sfdist(trun),
@timevar(year), @idvar(fips),
@depvar(lprod),
@frontier(lfertilizer_index1, lchemical_index1),
@μ(_cons),
@hscale(precip),                # h(.) function
@σᵤ²(_cons),
@σᵥ²(_cons))
Thanks in advance!

Unbalanced panel data is not a problem. I noticed that you had @IdVar(...) ; the macro should be in lower cases as @idvar(...).  Please see if the correction would fix the problem.

I just ran with the correct name and it is still not working. I had the same problem as before.
"The sfpanel() and/or sfdist() are not specified correctly."
var"#sfmodel_spec#43"(::Bool, ::typeof(SFrontiers.sfmodel_spec), ::Tuple{Symbol, Vector{Symbol}}, ::Vararg{Tuple{Symbol, Vector{Symbol}}})@SFmainfun.jl:212
sfmodel_spec@SFmainfun.jl:99[inlined]
top-level scope@Local: 1[inlined]
Code:
sfmodel_spec(sfpanel(TFE_WH2010), sftype(prod), sfdist(trun),
@timevar(year), @idvar(fips),
@depvar(lprod),
@frontier(lfertilizer_index1, lchemical_index1),
@μ(_cons),
@hscale(precip, precip2),                # h(.) function
@σᵤ²(_cons),
@σᵥ²(_cons))

I corrected the name and still had the same problem.
"The sfpanel() and/or sfdist() are not specified correctly."
var"#sfmodel_spec#43"(::Bool, ::typeof(SFrontiers.sfmodel_spec), ::Tuple{Symbol, Vector{Symbol}}, ::Vararg{Tuple{Symbol, Vector{Symbol}}})@SFmainfun.jl:212
top-level scope@Local: 1[inlined]
code:
sfmodel_spec(sfpanel(TFE_WH2010), sftype(prod), sfdist(trun),
@timevar(year), @idvar(fips),
@depvar(lprod),
@frontier(lfertilizer_index1, lchemical_index1),
@μ(_cons),
@hscale(precip, precip2),                # h(.) function
@σᵤ²(_cons),
@σᵥ²(_cons), message= true)

I corrected the name and still had the same problem.
"The sfpanel() and/or sfdist() are not specified correctly."
var"#sfmodel_spec#43"(::Bool, ::typeof(SFrontiers.sfmodel_spec), ::Tuple{Symbol, Vector{Symbol}}, ::Vararg{Tuple{Symbol, Vector{Symbol}}})@SFmainfun.jl:212
top-level scope@Local: 1[inlined]
code:
sfmodel_spec(sfpanel(TFE_WH2010), sftype(prod), sfdist(trun),              @timevar(year), @idvar(fips),              @depvar(lprod),               @frontier(lfertilizer_index1, lchemical_index1),               @μ(_cons),              @hscale(precip, precip2),                # h(.) function              @σᵤ²(_cons),              @σᵥ²(_cons), message= true)

WH2010T.csv
I couldn't reproduce the problem. Please use the attached dataset (WH2010T.csv) and run the following code and let me know if problems still occur. I have no problem running it with Julia 1.8.3 in Windows. Which version of Julia are you using and in which plateform (Windows, Mac, or Linux)?
df = DataFrame(CSV.File("WH2010T.csv"))

df[!, :_cons] .= 1.0

sfmodel_spec( sfpanel(TFE_WH2010), sftype(prod), sfdist(trun),
              @timevar(time), @idvar(id),
              @depvar(yit),
              @frontier(xit),
              @μ(_cons),
              @hscale(zit),
              @σᵤ²(_cons),
              @σᵥ²(_cons))

sfmodel_init()

sfmodel_opt( warmstart_solver(NelderMead()),
             warmstart_maxIT(200),
             main_solver(Newton()),
             main_maxIT(2000),
             tolerance(1e-8))

res = ()

res = sfmodel_fit(useData(df))

Sorry for the late response. I opted to work in Stata and R. Thank you for helping me!


Deeplinking web validator (Android)

A deep link is a special type of URL that takes users directly to a specific page or section within an app. It contains a scheme, host, and path to open the app and navigate to the desired location.
Deep links allow end-users to easily navigate to the relevant content in the mobile app from a mobile website. Because mobile apps generally provide better user experience, users who’re directed to an app are twice more likely to convert (e.g. make a purchase) than users who are directed to the mobile web.
However, deeplinking setup is complicated and often error-prone. We hope to improve this deeplinking implementation experience by first providing a validator to help developers determine if their implementation is accurate.
In Flutter applications, a deeplink setup consists of app setup and web setup. Assuming a developer starts with a fresh Flutter application, the app setup includes the adding certain settings or tags in native Android/iOS files and the dart code to handle incoming urls. The web setup involves maintaining a website that hosts a file to associate its web domain to a flutter app.
This particular issue tracks the web setup validation for Android app developed using Flutter. We will also have separate issues tracking web validation for iOS app using Flutter, and later app set up validation for both iOS and Android app using  Flutter.

Base on youtube video this is implemented in web and android https://youtube.com/shorts/iDxjec_DcjQ?si=K_YotG0SkS4Nv3pM ? Though I didn't find much about it in official doc

https://medium.com/flutter/whats-new-in-flutter-3-19-58b1aae242d2#:~:text=Deeplinking web validator
https://docs.google.com/document/d/1fnWe8EpZleMtSmP0rFm2iulqS3-gA86z8u9IsnXjJak/edit

@at15 This validator is for mobile only, and currently only works for Android's web setup, i.e. assetLinks.json. We are still working on adding more checks.


https://get.docker.io/ubuntu breaks apt-cacher

Following #7422
Since the recent switch from http to https, it broke any setup relying on apt-cacher-ng.
Your get HTTP code 403 from proxy after CONNECT when performing an apt update.
In my case, with ansible, it breaks any further apt commands.
Removing the apt-cacher is not an option (orchestrating hundreds of machines).

Same issue on Trusty following with a simple curl -sSL https://get.docker.com/ubuntu/ | sudo sh.

Same issue on Trusty following with a simple curl -sSL https://get.docker.com/ubuntu/ | sudo sh.

I think this has been resolved right?

or is it an actual problem with the install script

This is still a problem. It is something with how the apt repo is configured. It isn't a problem with the curl install script, just when you're pulling through an apt proxy like apt-cacher-ng.

gotcha thanks for explaining

Just as a quick example of this working in action if anyone wants to look for any sort of testing:

Start apt-cacher-ng:

docker run -d --name acng tianon/apt-cacher-ng

Start a docker container and add the docker repo and perform an apt-get update while using the proxy:

docker run -it --link acng:acng debian:jessie /bin/bash

Run the following commands in your debian container:

apt-get update && apt-get install -y apt-transport-https
echo 'deb https://get.docker.com/ubuntu docker main' > /etc/apt/sources.list.d/docker.list
echo 'Acquire::http::Proxy "http://acng:80";' > /etc/apt/apt.conf
apt-get update

Enjoy your 403 errors

I don't believe apt-cache-ng supports ssl.

I cheated and switched deb https://get.docker.com/ubuntu docker main to deb http://get.docker.com.s3-website-us-west-1.amazonaws.com/ubuntu docker main

@lancehudson - Ah yes, you're correct.  I am not sure why I didn't even think about that.
So there are three possibly solutions.  These must be done per server as there isn't anything that can be done to fix it Docker side as there isn't anything wrong:

Disable your apt proxy for HTTPS:
Acquire::http::Proxy "http://<url-to-apt-cacher-ng>:3142";
Acquire::https::Proxy "false";

Configure apt to bypass the proxy just for get.docker.com (works for get.docker.io too but it is probably a good idea to update to .com anyway...):
Acquire::http::Proxy "http://<url-to-apt-cacher-ng>:3142";
Acquire::HTTP::Proxy::get.docker.com "DIRECT";

Change your /etc/apt/sources.list.d/docker.list to use http instead of https:
deb http://get.docker.com/ubuntu docker main

Or even better. Follow the notes in the apt-cacher-ng config file that show how SSL can be passed through.
In your apt-cacher-ng config file (acng.conf) add the following line:
PassThroughPattern: get\.docker\.com

Well huh...  I could have sworn that I had tried that previously without any luck but here I am with that working perfectly.  I think I attempted to use the patterns I found here unsuccessfully:  https://github.com/saltstack-formulas/docker-formula/issues/6 but for whatever reason didn't just try get\.docker\.com apparently.  No matter, thanks much for what I would call a real solution @mandoonandy :+1:

I would think this issue can be closed as it is a misconfiguration of apt-cacher-ng essentially and nothing that can be fixed by docker.

@mbentley thanks!
I'm closing this now. @mgcrea if you disagree and still see this issue, feel free to comment and we'll reopen.

I'm getting this error while running the wget -qO- https://get.docker.com/ | sh provided in the docs on Ubuntu 12.4.
I don't think this is the appropriate place to report the issue since i'm not using apt-cacher.
 They have been ignored, or old ones used instead.

  For kicks, could you try to add Acquire::HTTP::Proxy::get.docker.com "DIRECT"; to your /etc/apt/apt.conf file and then doing an apt-get update to see if it still happens?

@mbentley You're a gentleman and a scholar. This was in a vagrant box that had some wonky proxy config set up. Thanks for your insight. For anybody who stumbles upon this problem be sure to check your /etc/apt/apt.conf :)

In case this helps anyone, here is a workaround to configure apt-cacher-ng to work with SSL:
http://blog.packagecloud.io/eng/2015/05/05/using-apt-cacher-ng-with-ssl-tls/

possibly the quickest solution is to add a line into your
           /etc/apt-cacher-ng/acng.conf

PassThroughPattern: .*

            Allow data pass-through mode for certain hosts when requested by the client
            using a CONNECT request. This is particularly useful to allow access to SSL
            sites (https proxying).

(this hint is also noted in goodwillcoding's reference)

In my situation, with the latest install instructions, adding Acquire::HTTP::Proxy::apt.dockerproject.org "DIRECT"; to my apt.conf.d/proxy01 solved the problem.


https://community.meteorite.bi/ Service Unavailable

Service Unavailable
The server is temporarily unable to service your request due to maintenance downtime or capacity problems. Please try again later.
Apache/2.4.18 (Ubuntu) Server at community.meteorite.bi Port 443
Can you help me？

Hi @xiepeichen it's back. Try again...


[Feature request] Non persistent option for highlight_node

I like how it looks and works, but keeping track of what is highlighted to remove it later can be cumbersome.
So maybe there could be an additional option, and when enabled, I have no call highlight_node on every tick, otherwise the effect disappears. Similar how  io_debug_geometry->draw.. works.

Will be available in the next build.


chore: release main

:robot: I have created a release beep boop
tools/kystrap: 1.4.0
1.4.0 (2024-03-12)
Features

new feature (35ae1cc)
new feature (65d03dc)
new feature (00f1108)

Bug Fixes

small change (c30d96c)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/shifty11/kyve-rdk-fork/releases/tag/tools/kystrap%401.4.0 :sunflower:


FocusScope: If a focused element unmounts, focus moves to body, breaking focus containment

🐛 Bug Report

FocusScope exhibits confusing behavior when a focused element unmounts, potentially allowing focus to travel outside of a FocusScope even when contain is set to true.
🤔 Expected Behavior

I would expect that when a focused element is unmounted, focus travels to a different element within the FocusScope, when contain is true.
😯 Current Behavior

When a focus element within a FocusScope unmounts, focus travels to  rather than another element within the focus scope.
🔦 Context

I am implementing a modal that renders a form with radio buttons on mount. Prior to the radio buttons being returned from the fetch, I show a loading indicator. I attempted to use the loading indicator as the focus target for the modal, assuming that when the loading indicator unmounts then the first form option would be rendered. Instead, focus traveled to body which is outside of my FocusScope.
💻 Code Sample

Sandbox with issue reproduced: https://codesandbox.io/s/thirsty-mahavira-dof7z?file=/src/App.js
Code in case sandbox becomes unavailable:
import React from "react";
import { FocusScope } from "@react-aria/focus";

export default function App() {
  const [focused, setFocused] = React.useState();
  const [buttonIsVisible, setButtonIsVisible] = React.useState(true);

  React.useEffect(() => {
    window.addEventListener(
      "DOMSubtreeModified",
      () => {
        setFocused(document.activeElement.tagName);
      },
      true
    );

    window.addEventListener(
      "blur",
      () => {
        setFocused(document.activeElement.tagName);
      },
      true
    );

    window.addEventListener(
      "focus",
      () => {
        setFocused(document.activeElement.tagName);
      },
      true
    );
  }, []);

  return (

      <FocusScope contain>

Focused Element Tag: {focused}

        <button>Focus me next please</button>
        {buttonIsVisible && (
          <button
            onClick={() => {
              setButtonIsVisible(false);
            }}
          >
            Press me to delete me
          </button>
        )}
      </FocusScope>

  );
}

🌍 Your Environment

Software
Version(s)

react-spectrum
"@react-aria/focus": "^3.1.0"

Browser
Chrome: 84.0.4147.105

Operating System
MacOS 10.15.5

🧢 Your Company/Team

ButcherBox/Engineering

Thanks for reporting! Where would you expect focus to go in your case? To the next focusable element after the button that was deleted? What if there is no focusable element inside the scope anymore? Feels like it may be challenging to answer those questions in a general way...

I think at a baseline, I would expect for the FocusScope to contain focus so long as it contains another focus-able element. The least surprising behavior here would probably be for the next element in the focus order to be focused. I think if there were no focus-able elements within the scope, then it probably makes sense for focus to break out of the scope in some way.
Thanks so much for your work on this! Your components are life-savers for our team :)

Is anybody started to work on this? I can try to handle this in 2-3, just want to know if anybody tried to implement this, so I can just continue not from 0, or any tips/comments would be useful too. Thanks!

@AleshaOleg I don't believe anyone has picked this up yet, feel free to give it a shot! I think it would be a good idea to first come to an agreement for the expected behavior first. IMO I think shifting the focus to the next element in focus order makes sense if it exists, but the question of where the focus should go if there are no focusable elements within the scope still stands. Perhaps in that case it should break out of focus containment and focus the body? Or should it find the last focusable element prior to the one that unmounted? Any suggestions?

Possibly resolved by https://github.com/adobe/react-spectrum/pull/3323

I've confirmed, this has been fixed in recent releases.


Fixed variable name in initWithUrlParams

The initWithUrlParams wasn't processing the facets, I found out the name of the variable was wrong.

Thanks @reinaris!


gdrdrv does not build on bleeding edge distros because HAVE_UNLOCKED_IOCTL has been dropped in linux kernel v5.9

build error is:
xxx/gdrdrv.c:xxx:6: error: ‘struct file_operations’ has no member named ‘ioctl’
xxx |     .ioctl    = gdrdrv_ioctl,
|      ^~~~~
Linux kernel commit is https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=4e24566a134ea167441a1ffa3d439a27cf400880

unlocked_ioctl has been around since v2.6.11, and recent GPU drivers like r418 require v2.6.32 at least, so we can safely assume that it is supported without checking for HAVE_UNLOCKED_IOCTL


Configure 2M huge pages (setup_permissions.sh) only configures 1024 2M hugepages

the script setup_permissions.sh has a function set_hugepages . This function currently configures only 1024 2M hugepages
################################################################################
# Configure 2M huge pages
#
# Hugepages are used to improve performance of the DMA operations between host
# and FPGA.  The OFS oneAPI-ASP requires a minimum of 4 2M hugepages per FPGA.
# However, there is additional optimization that allows user to pre-pin memory
# using hugepages.  The default setting in this script configures 2048 2M
# hugepages.  This means 4G of host memory is allocated for hugepages.
# This can be changed by configuring OFS_ASP_ENV_NUM_HUGEPAGES before running
# this script.
################################################################################
set_hugepages()
{
  NUM_HUGE_PAGES=${OFS_ASP_ENV_NUM_HUGEPAGES:-1024}

https://github.com/OFS/oneapi-asp/blob/51ccd10edb739581ace2a00f02d2e0dd26e8041f/common/linux64/libexec/setup_permissions.sh#L101
so this line should be
  NUM_HUGE_PAGES=${OFS_ASP_ENV_NUM_HUGEPAGES:-2048}

Hi @umairsiddiqui-digitek,
Thanks for the observation, this will be fixed in a future release, please see the 2024.2-1 release notes for the workaround for this issue.


CLOUDSTACK-8934 - Default routes not configured for rVPC and RVR

This PR fixes the default routes for redundant VPCs and isolated networks. New tests were introduced in order to make sure that the routers are working properly.
During the tests, I found out that the Firewall Egress was not working properly when creating the network offering with default allow. The bug has been fixed and tests for redundant and non-redundant isolated networks were added.
Test reports will follow in a separate comment.

Ping @remibergsma @karuturi @DaanHoogland @miguelaferreira @bvbharat @wido @borisroman
All, the tests were executed successfully! We still have the problems with cleaning the networks, as you can see below. For that there is an issue which I will work on next (https://issues.apache.org/jira/browse/CLOUDSTACK-8935)
Please, have a look and let me know if it all looks okay.
Cheers,
Wilder
== Hardware required tests ==

Management Server + MySQL running on CentOS 7.1
One KVM host running on CentOS 7.1
ACS Agent and Common RPMs built from source

Stop existing router, add a PF rule and check we can access the VM ... === TestName: test_isolate_network_FW_PF_default_routes | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_RVR_Network_FW_PF_SSH_default_routes | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network ... === TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network and check default routes ... === TestName: test_02_redundant_VPC_default_routes | Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policy on RouterVM ... === TestName: test_02_routervm_iptables_policies | Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policies on VPC router ... === TestName: test_01_single_VPC_iptables_policies | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test nics after destroy ... === TestName: test_01_VPC_nics_after_destroy | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test default routes ... === TestName: test_02_VPC_default_routes | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 8 tests in 6855.736s

OK

== No Hardware required tests ==

Management Server + MySQL running on CentOS 7.1
Two KVM hosts running on CentOS 7.1
ACS Agent and Common RPMs built from source

Test advanced zone virtual router ... === TestName: test_advZoneVirtualRouter | Status : SUCCESS ===
ok
Test Deploy Virtual Machine ... === TestName: test_deploy_vm | Status : SUCCESS ===
ok
Test Multiple Deploy Virtual Machine ... === TestName: test_deploy_vm_multiple | Status : SUCCESS ===
ok
Test Stop Virtual Machine ... === TestName: test_01_stop_vm | Status : SUCCESS ===
ok
Test Start Virtual Machine ... === TestName: test_02_start_vm | Status : SUCCESS ===
ok
Test Reboot Virtual Machine ... === TestName: test_03_reboot_vm | Status : SUCCESS ===
ok
Test destroy Virtual Machine ... === TestName: test_06_destroy_vm | Status : SUCCESS ===
ok
Test recover Virtual Machine ... === TestName: test_07_restore_vm | Status : SUCCESS ===
ok
Test migrate VM ... === TestName: test_08_migrate_vm | Status : SUCCESS ===
ok
Test destroy(expunge) Virtual Machine ... === TestName: test_09_expunge_vm | Status : SUCCESS ===
ok
Test router internal advanced zone ... === TestName: test_02_router_internal_adv | Status : SUCCESS ===
ok
Test restart network ... === TestName: test_03_restart_network_cleanup | Status : SUCCESS ===
ok
Test router basic setup ... === TestName: test_05_router_basic | Status : SUCCESS ===
ok
Test router advanced setup ... === TestName: test_06_router_advanced | Status : SUCCESS ===
ok
Test stop router ... === TestName: test_07_stop_router | Status : SUCCESS ===
ok
Test start router ... === TestName: test_08_start_router | Status : SUCCESS ===
ok
Test reboot router ... === TestName: test_09_reboot_router | Status : SUCCESS ===
ok
Test reset virtual machine on reboot ... === TestName: test_01_reset_vm_on_reboot | Status : SUCCESS ===
ok
Test router internal advanced zone ... === TestName: test_02_router_internal_adv | Status : SUCCESS ===
ok
Test restart network ... === TestName: test_03_restart_network_cleanup | Status : SUCCESS ===
ok
Test router basic setup ... === TestName: test_05_router_basic | Status : SUCCESS ===
ok
Test router advanced setup ... === TestName: test_06_router_advanced | Status : SUCCESS ===
ok
Test stop router ... === TestName: test_07_stop_router | Status : SUCCESS ===
ok
Test start router ... === TestName: test_08_start_router | Status : SUCCESS ===
ok
Test reboot router ... === TestName: test_09_reboot_router | Status : SUCCESS ===
ok
test_privategw_acl (integration.smoke.test_privategw_acl.TestPrivateGwACL) ... === TestName: test_privategw_acl | Status : SUCCESS ===
ok
Test VPN in VPC ... === TestName: test_vpc_remote_access_vpn | Status : SUCCESS ===
ok
Test VPN in VPC ... === TestName: test_vpc_site2site_vpn | Status : SUCCESS ===
ok
Test to create service offering ... === TestName: test_01_create_service_offering | Status : SUCCESS ===
ok
Test to update existing service offering ... === TestName: test_02_edit_service_offering | Status : SUCCESS ===
ok
Test to delete service offering ... === TestName: test_03_delete_service_offering | Status : SUCCESS ===
ok
Test create VPC offering ... === TestName: test_01_create_vpc_offering | Status : SUCCESS ===
ok
Test VPC offering without load balancing service ... === TestName: test_03_vpc_off_without_lb | Status : EXCEPTION ===
ERROR: Test VPC offering without load balancing service
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/data/git/cs2/cloudstack/test/integration/component/test_vpc_offerings.py", line 218, in tearDown
    raise Exception("Warning: Exception during cleanup : %s" % e)
Exception: Warning: Exception during cleanup : Execute cmd: deletenetworkoffering failed, due to: errorCode: 431, errorText:Can't delete network offering 15 as its used by 1 networks. To make the network offering unavaiable, disable it

Test VPC offering without static NAT service ... === TestName: test_04_vpc_off_without_static_nat | Status : EXCEPTION ===
ERROR: Test VPC offering without static NAT service
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/data/git/cs2/cloudstack/test/integration/component/test_vpc_offerings.py", line 218, in tearDown
    raise Exception("Warning: Exception during cleanup : %s" % e)
Exception: Warning: Exception during cleanup : Execute cmd: deletenetworkoffering failed, due to: errorCode: 431, errorText:Can't delete network offering 16 as its used by 1 networks. To make the network offering unavaiable, disable it

Test VPC offering without port forwarding service ... === TestName: test_05_vpc_off_without_pf | Status : EXCEPTION ===
ERROR: Test VPC offering without port forwarding service
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/data/git/cs2/cloudstack/test/integration/component/test_vpc_offerings.py", line 218, in tearDown
    raise Exception("Warning: Exception during cleanup : %s" % e)
Exception: Warning: Exception during cleanup : Execute cmd: deletenetworkoffering failed, due to: errorCode: 431, errorText:Can't delete network offering 17 as its used by 1 networks. To make the network offering unavaiable, disable it

Test VPC offering with invalid services ... === TestName: test_06_vpc_off_invalid_services | Status : SUCCESS ===
ok
Test update VPC offering ... === TestName: test_07_update_vpc_off | Status : SUCCESS ===
ok
Test list VPC offering ... === TestName: test_08_list_vpc_off | Status : SUCCESS ===
ok
test_09_create_redundant_vpc_offering (integration.component.test_vpc_offerings.TestVPCOffering) ... === TestName: test_09_create_redundant_vpc_offering | Status : SUCCESS ===
ok
Test start/stop of router after addition of one guest network ... === TestName: test_01_start_stop_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test reboot of router after addition of one guest network ... === TestName: test_02_reboot_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test to change service offering of router after addition of one guest network ... === TestName: test_04_chg_srv_off_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test destroy of router after addition of one guest network ... === TestName: test_05_destroy_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test to stop and start router after creation of VPC ... === TestName: test_01_stop_start_router_after_creating_vpc | Status : SUCCESS ===
ok
Test to reboot the router after creating a VPC ... === TestName: test_02_reboot_router_after_creating_vpc | Status : SUCCESS ===
ok
Tests to change service offering of the Router after ... === TestName: test_04_change_service_offerring_vpc | Status : SUCCESS ===
ok
Test to destroy the router after creating a VPC ... === TestName: test_05_destroy_router_after_creating_vpc | Status : SUCCESS ===
ok

=====================================================================

----------------------------------------------------------------------
Ran 47 tests in 8317,912s

overall code looks good, not sure if I need to test something special, @wilderrodrigues

@wilderrodrigues I will spend some time on testing this today!

Testing now, will post results later today!

@wilderrodrigues Thanks, looking good! LGTM
Tested on 2-node KVM cluster on CentOS 7.
I started by running the same tests:
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=true \
component/test_vpc_redundant.py \
component/test_routers_iptables_default_policy.py \
component/test_routers_network_ops.py \
component/test_vpc_router_nics.py

Results:
Create a redundant VPC with two networks with two VMs in each network ... === TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network and check default routes ... === TestName: test_02_redundant_VPC_default_routes | Status : SUCCESS ===
ok
Stop existing router, add a PF rule and check we can access the VM ... === TestName: test_isolate_network_FW_PF_default_routes | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_RVR_Network_FW_PF_SSH_default_routes | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 4 tests in 4592.786s

OK

I have seen these tests failing, but trying again and even doing it manually works.
Next test:
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=false \
smoke/test_routers.py \
smoke/test_network_acl.py \
smoke/test_privategw_acl.py \
smoke/test_reset_vm_on_reboot.py \
smoke/test_vm_life_cycle.py \
smoke/test_vpc_vpn.py \
smoke/test_service_offerings.py \
component/test_vpc_offerings.py \
component/test_vpc_routers.py

Results:
Test router internal advanced zone ... === TestName: test_02_router_internal_adv | Status : SUCCESS ===
ok
Test restart network ... === TestName: test_03_restart_network_cleanup | Status : SUCCESS ===
ok
Test router basic setup ... === TestName: test_05_router_basic | Status : SUCCESS ===
ok
Test router advanced setup ... === TestName: test_06_router_advanced | Status : SUCCESS ===
ok
Test stop router ... === TestName: test_07_stop_router | Status : SUCCESS ===
ok
Test start router ... === TestName: test_08_start_router | Status : SUCCESS ===
ok
Test reboot router ... === TestName: test_09_reboot_router | Status : SUCCESS ===
ok
test_privategw_acl (integration.smoke.test_privategw_acl.TestPrivateGwACL) ... === TestName: test_privategw_acl | Status : SUCCESS ===
ok
Test reset virtual machine on reboot ... === TestName: test_01_reset_vm_on_reboot | Status : SUCCESS ===
ok
Test advanced zone virtual router ... === TestName: test_advZoneVirtualRouter | Status : SUCCESS ===
ok
Test Deploy Virtual Machine ... === TestName: test_deploy_vm | Status : SUCCESS ===
ok
Test Multiple Deploy Virtual Machine ... === TestName: test_deploy_vm_multiple | Status : SUCCESS ===
ok
Test Stop Virtual Machine ... === TestName: test_01_stop_vm | Status : SUCCESS ===
ok
Test Start Virtual Machine ... === TestName: test_02_start_vm | Status : SUCCESS ===
ok
Test Reboot Virtual Machine ... === TestName: test_03_reboot_vm | Status : SUCCESS ===
ok
Test destroy Virtual Machine ... === TestName: test_06_destroy_vm | Status : SUCCESS ===
ok
Test recover Virtual Machine ... === TestName: test_07_restore_vm | Status : SUCCESS ===
ok
Test migrate VM ... === TestName: test_08_migrate_vm | Status : SUCCESS ===
ok
Test destroy(expunge) Virtual Machine ... === TestName: test_09_expunge_vm | Status : SUCCESS ===
ok
Test VPN in VPC ... === TestName: test_vpc_remote_access_vpn | Status : SUCCESS ===
ok
Test VPN in VPC ... === TestName: test_vpc_site2site_vpn | Status : SUCCESS ===
ok
Test to create service offering ... === TestName: test_01_create_service_offering | Status : SUCCESS ===
ok
Test to update existing service offering ... === TestName: test_02_edit_service_offering | Status : SUCCESS ===
ok
Test to delete service offering ... === TestName: test_03_delete_service_offering | Status : SUCCESS ===
ok
Test create VPC offering ... === TestName: test_01_create_vpc_offering | Status : SUCCESS ===
ok
Test VPC offering without load balancing service ... === TestName: test_03_vpc_off_without_lb | Status : EXCEPTION ===
ERROR
Test VPC offering without static NAT service ... === TestName: test_04_vpc_off_without_static_nat | Status : EXCEPTION ===
ERROR
Test VPC offering without port forwarding service ... === TestName: test_05_vpc_off_without_pf | Status : EXCEPTION ===
ERROR
Test VPC offering with invalid services ... === TestName: test_06_vpc_off_invalid_services | Status : SUCCESS ===
ok
Test update VPC offering ... === TestName: test_07_update_vpc_off | Status : SUCCESS ===
ok
Test list VPC offering ... === TestName: test_08_list_vpc_off | Status : SUCCESS ===
ok
test_09_create_redundant_vpc_offering (integration.component.test_vpc_offerings.TestVPCOffering) ... === TestName: test_09_create_redundant_vpc_offering | Status : SUCCESS ===
ok
Test start/stop of router after addition of one guest network ... === TestName: test_01_start_stop_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test reboot of router after addition of one guest network ... === TestName: test_02_reboot_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test to change service offering of router after addition of one guest network ... === TestName: test_04_chg_srv_off_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test destroy of router after addition of one guest network ... === TestName: test_05_destroy_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test to stop and start router after creation of VPC ... === TestName: test_01_stop_start_router_after_creating_vpc | Status : SUCCESS ===
ok
Test to reboot the router after creating a VPC ... === TestName: test_02_reboot_router_after_creating_vpc | Status : SUCCESS ===
ok
Tests to change service offering of the Router after ... === TestName: test_04_change_service_offerring_vpc | Status : SUCCESS ===
ok
Test to destroy the router after creating a VPC ... === TestName: test_05_destroy_router_after_creating_vpc | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 40 tests in 7144.018s

FAILED (errors=3)

The 3 errors are due to cleanups failing. Could be a timing issue, as I could successfully remove the resources manually. It's a known issue, CLOUDSTACK-8935, as mentioned above.
Finally I did a site-to-site VPN test with real hardware and real VMs.
VPCs look like this:

Then logging in to each VM, and I could ping the VM in the other VPC over its internal ip:

thanks for running the tests @remibergsma
should we merge this PR since we have two reviews from @DaanHoogland and @remibergsma ?

Let's wait for the test results of @borisroman as he said he was testing as well.
@borisroman if you are no longer testing, please let us know.

@remibergsma @karuturi @wilderrodrigues @DaanHoogland I tested this PR today. Tests ran fine except for the cleanups (which are known by CLOUDSTACK-8935). I'll spare the space of this comment as all test results already have been posted twice by Wilder and Remi. I also ran the test_loadbalance.py which turned out positive.
Test to create Load balancing rule with source NAT ... === TestName: test_01_create_lb_rule_src_nat | Status : SUCCESS ===
ok
Test to create Load balancing rule with non source NAT ... === TestName: test_02_create_lb_rule_non_nat | Status : SUCCESS ===
ok
Test for assign & removing load balancing rule ... === TestName: test_assign_and_removal_lb | Status : SUCCESS ===
ok

Ran 3 tests in 633.239s
OK
/tmp//MarvinLogs/test_loadbalance_HD34OP/results.txt
Code (style) changes look good to me.
To conclude: LGTM :+1:


Wrong B64_PAT generated by bash command from doc because of trailing newline char

In the reference docs for creating Basic authentication tokens based off PAT, there is a problem with the following bash script
:
MY_PAT=yourPAT		# replace "yourPAT" with your actual PAT
B64_PAT=$(echo ":$MY_PAT" | base64)
git -c http.extraHeader="Authorization: Basic ${B64_PAT}" clone https://dev.azure.com/yourOrgName/yourProjectName/_git/yourRepoName

When generating the B64_PAT, a trailing newline character issued by the echo command would render the resulting encoded string unusable. The correct command should be:
B64_PAT=$(echo -n ":$MY_PAT" | base64)

When using the -n flag, echo won't append a newline at the end of the output to stdout.
For me, after using the modified command, I was able to generate a usable encoded token for communicating with Azure Devops Rest API.

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

ID: 53f99d2d-363b-d01d-039d-4fdc636fa2df
Version Independent ID: f47330d5-e2de-36a4-e5b3-5bc77505b161
Content: Authenticate with personal access tokens - Azure DevOps
Content Source: docs/organizations/accounts/use-personal-access-tokens-to-authenticate.md
Product: devops
Technology: devops-security
GitHub Login: @chcomley
Microsoft Alias: chcomley

@gchamon  Thank you for the feedback.
This issue was previously reported under https://github.com/MicrosoftDocs/azure-devops-docs/issues/9262 and is currently under review.  Please refer there for further updates.


copyedits to doc/developer/repoproviders.rst

Thank you for contributing @Daniel-Mietchen :tada: :heart:


outspectra flag not working as of v2.03

Hi Arjan,
For a simple test file after building v2.04, I noticed that the outspectra y flag in an input file was not actually producing any of the outgoing composite particle spectra as intended.  The test file used was:
#
# General
#
projectile d
element Cu
mass 0
energy 1 40 1
#
# Output
#
outspectra          y

The particle spectra were not output even with multiple variations on mass, energy, element, and projectile.
I went and checked out prior commits for v2.00, v2.01, v2.02, v2.03, v2.04, and the most recent commit.  Using the same input file across all 6 versions, particle spectra are output in 2.00, 2.01, and 2.02, as of commit 65b2dc874dbd32d3ed649287ec35ed2edae3ba90.
However, starting in v2.03 (commit d5703833e0434ac69251721e6237218632cc37d5), the output file instead returns:
 7. Composite particle spectra

 Spectra for outgoing gamma

 Spectra for outgoing neutron

 Spectra for outgoing proton

 Spectra for outgoing deuteron

 Spectra for outgoing helium-3

 Spectra for outgoing alpha

 Average emission energies

when running both a single energy, or a multi-energy input file.  This behavior remains in both v2.04 (commit d269fa215357f80b6c7566cab7fa516d6f41a0b9), as well as the current commit (aa4753c0e02cb732de6580e8b3b58a14fc7521ee).
I'm not sure exactly what happened, but my best guess is that in between these commits, lines 111-132 of source/spectraout.f90 were deleted - this seems to control the writing of these composite spectra to the output file. A new line 156
write(*, '(/" LAB spectra for outgoing ", a8/)') parname(type)

was added for Lab spectra, but no equivalent version of the composite particle spectra seems to have been added ti replace the writing block which was removed.
Any help with a patch ehre would be greatly appreciated. Thanks!

Hi Andrew,
Latest version should have solved this. As you may have noticed, I am making all the output of TALYS more uniform, with consistent Metadata etc. to enable more automated processing for AI/L, processing codes etc and sometimes this goes wrong, apparently. Instead of printing everything in the output I now mention the specific files where the info can be found. However, if you want to go back to the old situation giving 'outall y' makes huge output files again.
Cheers,
Arjan

Hi Arjan,
Fantastic!  can confirm the spectra are now outputting properly - I had indeed seen the shift towards the new output format, and absolutely love it!!  Thanks so much for the quick patch!


merge develop to main to get latest changes from tet to staging

Description :sparkles:
Issues :bug:
Testing :alembic:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

With the same disclaimer as before: https://github.com/City-of-Helsinki/yjdh/pull/1524#pullrequestreview-1184308598

Codecov Report

Merging #1578 (5f932b6) into main (44b62d5) will increase coverage by 2.15%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main    #1578      +/-   ##
==========================================
+ Coverage   65.39%   67.54%   +2.15%
==========================================
  Files        1041      903     -138
  Lines       28513    25778    -2735
  Branches     2973     2533     -440
==========================================
- Hits        18645    17412    -1233
+ Misses       9414     8012    -1402
+ Partials      454      354     -100

Impacted Files
Coverage Δ

backend/benefit/helsinkibenefit/settings.py
95.03% <ø> (ø)

backend/tet/events/linkedevents.py
52.63% <ø> (ø)

backend/tet/tet/views.py
42.30% <0.00%> (-3.53%)
:arrow_down:

.../src/components/jobPostings/JobPostingsSection.tsx
100.00% <ø> (ø)

...src/hooks/backend/useEventPostingTransformation.ts
0.00% <0.00%> (ø)

frontend/tet/shared/src/utils/get-ui-end-date.ts
0.00% <0.00%> (ø)

...end/tet/shared/src/__tests__/utils/fake-objects.ts
56.25% <20.00%> (+3.12%)
:arrow_up:

...src/components/jobPostings/JobPostingsListItem.tsx
95.23% <80.00%> (-4.77%)
:arrow_down:

...ontend/tet/shared/src/utils/get-server-end-date.ts
83.33% <83.33%> (ø)

backend/tet/events/services.py
71.05% <100.00%> (+1.05%)
:arrow_up:

... and 149 more

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


Improve Staking page loading time (#1446)

@marcinbodnar took some notes on potential issues below:

Load staking dashboard using an account that has a lockup. Then switch account. Notice that main account on staking dashboard is switched, but the lockup account still shows, even though the account you just switched to does not.
On staking dashboard: Radio button hides/shows when loading - maybe use skeleton loading to avoid. Also the radio button shows when there’s only one account (should only show if there are multiple)
Not able to withdraw available tokens from staking pool. Confirm modal closes without any visible error.
Staking dashboard visually loads if you navigate to a different page and back again. Can we avoid that (at least visually) like before?
Navigate from staking dashboard to profile and back, and notice that only total amount staked and rewards earned load. It looks a bit off that pending release and available for withdrawal are not loading. Can we make them all load?
On profile: refresh the page using account that doesn’t have any lockup. Console: Error in background action: Error: No contract for account. Pointing to line 301 in staking.js
On profile: refresh the page using account that has a lockup. Console: TypeError: Cannot read property 'accountId' of undefined. Pointing to line 270 in staking.js
The globalAlert shows for staking/unstaking - worth setting to onlyError
Tried unstaking 0.1N from a lockup account. And got the error below. It actually succeeded, and the 0.1N was unstaked. However, the validator box says 6.41157 NEAR but should be ~6.3XXXX NEAR. I don't think it's getting to balance refresh because of the error.

@marcinbodnar would it be OK for you if we ship https://github.com/near/near-wallet/issues/1538 before this? Or do you want me to wait?

Looks good overall but we need to make sure this is tested well with different edge cases:

normal account
2FA account
account with lockup

(I didn't test thoroughly this time).

@vgrichina I tested it using testnet account, @Patrick1904 tested with mainnet account. But it would be good if Igor also tested it thoroughly. @ihaid could you please test this PR trying every possible action for Staking?

Found one issue, steps:
Go to stake on account with 2fa
Submit and confirm staking and on the 2fa confirm window click "Cancel"
Then you will see that it is hung in this state and the only refresh will help

Found one issue, steps:
Go to stake on account with 2fa
Submit and confirm staking and on the 2fa confirm window click "Cancel"
Then you will see that it is hung in this state and the only refresh will help

Thanks @ihaid it should be good now, please check.


Could you please update the code to use the latest version of PyTorch Geometric?

First of all thank you very much for your excellent work, I tried to run your program on my own computer. I am using the latest version 2.0.3 of pyg, and version 1.9.0 of pytorch.
As you know, there are some differences between the 2.0.3 version of pyg and the 1.6.3 version used in your project.
After my own modification, I have been able to run HyperGCN, CEGAT, CEGCN and AllSetTransformer, but when I try to run HGNN, HNHN and HCHA, the code always reports an error, and the error occurs here：
layers.py  line405
self.flow = 'target_to_source'
out = self.propagate(hyperedge_index, x=out, norm=D, alpha=alpha,   size=(num_edges, num_nodes))
or
layers.py  line303
 self.flow = 'target_to_source'
 out = self.propagate(hyperedge_index, x=out, norm=data.D_v_alpha_inv, size=(num_edges, num_nodes))
and it shows :
ValueError: Encountered tensor with size 4287 in dimension 0, but expected size 2708.
with 'cora' dataset.
I really find it very difficult to solve this problem, could you please update your code to help me out. Thank you very much

Hi @liyongkang123 ,
Thanks for being interested in our work!
Regarding your issue, have you tried to modify the code in the way that AllSetTransformer define propagations? It can be found at
https://github.com/jianhao2016/AllSet/blob/0d0e399a9168829fa898dd56f3d32bee36953b04/src/models.py#L455
Basically, instead of using

Try to create a reverse hyperedge_index (reverse_hyperedge_index[0] = hyperedge_index[1] and reverse_hyperedge_index[1] = hyperedge_index[0]). Also remember to make the smallest hyperedge id to 0 (like in L453-L454).
Currently, I do not have time to rewrite things and test them. Nevertheless, please let me know if my suggestion works or not. I think it should work...
Thanks,
Eli

Thank you very much for your explanation and inspiration. I carefully compared the documentation of the latest version of pyg with the documentation of version 1.6.3 and found that your code was changed from
 self.flow = 'source_to_target'
out = self.propagate(hyperedge_index, x=x, norm=B, alpha=alpha,   size=(num_nodes, num_edges))
self.flow = 'target_to_source'
 out = self.propagate(hyperedge_index, x=out, norm=D, alpha=alpha, size=(num_edges, num_nodes))

to
out = self.propagate(hyperedge_index, x=x, norm=B, alpha=alpha,size=(num_nodes, num_edges))
out = self.propagate(hyperedge_index.flip([0]), x=out, norm=D,alpha=alpha, size=(num_edges, num_nodes))

Now the original problem has been solved, I am currently using pyg2.0.4 version,
and then I found that if I update to version 2.0.5, your AllSetTransformer will not work properly, I am trying to solve this problem and it should not be a problem. I am concerned about the constant updates of pyg and its backward compatibility. This is really kind of funny.
Anyhow, thank you very much for your help, at least I can run all the models smoothly now.
Thanks,
Li

Hi @liyongkang123 ,
Great! Glad to hear that you work it out. Indeed, PyG compatibility is a bit annoying... My personal take is to stay with some "stable" PyG version and not keep updating it.
Anyway, I'll close the issue for now. Please feel free to reopen it if you have further questions.
Thanks,
Eli


[microTVM]Refactor crt_config.h header file generation

This PR:

refactors crt_config.h generation to a single template file and use CMake configure_file func to generate in build
Refactor Zephyr.cmake, CRT.cmake and Arduino.cmake to Micro.cmake

cc @alanmacd @mkatanbaf for review


Shouldn't assume lexicographical ordering for key Range

it is possible using a non-default comparator, e.g., reverse comparator.

PTAL @zhangjinpeng1987 @yiwu-arbug


maybe a schema for Zone Redundant Storage Premium File Share could be interesting also

[Enter feedback here]

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

ID: f618f40d-ae53-5c3b-d8bf-0e80614f66d3
Version Independent ID: 93083407-8f1e-b3e2-6a62-e30be5e1fbcb
Content: Data redundancy - Azure Storage
Content Source: articles/storage/common/storage-redundancy.md
Service: storage
Sub-service: common
GitHub Login: @tamram
Microsoft Alias: tamram

@ehsf973
Thanks for the feedback! Could you be a little more specific on what you mean by schema? If so, I am happy to pass along this feedback to the document author.

hello sorry for the misunderstanding "schema" is a french word i do mean a
diagram like visio file of the architecture, like those existing for ZGRS,
etc ...
Le lun. 19 avr. 2021 à 20:40, DerekHerman-MSFT @.***> a
écrit :

@ehsf973 https://github.com/ehsf973
Thanks for the feedback! Could you be a little more specific on what you
mean by schema? If so, I am happy to pass along this feedback to the
document author.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/73980#issuecomment-822689215,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHFJQUFNVXCTFYW346SUY5DTJR2JPANCNFSM43FMDEAA
.

@ehsf973 There is a diagram in the ZRS section. Is there something you feel should be added?
https://docs.microsoft.com/en-us/azure/storage/common/storage-redundancy#zone-redundant-storage

@ehsf973 I am going to closeout this issue for now. If you still feel there is something missing from the doc please tag me and let me know and we can reopen the issue.


docs: add Pearl UI website to showcase

Motivation
Adding another example website to the Docusaurus showcase
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

Thanks, nice site, just remember to sign the CLA

Friendly ping @agrawal-rohit we would definitely close PRs without a CLA after two weeks of inactivity.

Hi @agrawal-rohit we are closing this PR because we can't merge PRs without CLA. If you are back, feel free to sign the CLA and reopen the PR, and we will happily merge it :D

Hey, sorry about that. I have signed the CLA, which should be reflected in your systems after an hour or so.
Could you reopen this PR and merge it now?

Yep! @agrawal-rohit It's all good now. 👍 Thanks!


fix(material/slider): aria-valuetext host binding should be onPush co…

…mpatible
MatSlider updates _valuetext without marking for check. The easiest way to fix these types of issues when values are not part of the public API is to switch them to signals.

TESTED=cl/633330517


Please provide a dist version in repo

Sometimes you just want to download a built version and just add it via script tags. Or maybe use it in JSBin via https://rawgit.com/
It would be great if you could maintain a 'dist' version in the repo, meaning a version of most.js that is ready to use in the browser by using script tags.

Hi @sporto.  Most.js works great in RequireBin.  Also, you can create a script tag version using browserify:
browserify -s most most.js --no-detect-globals -o most.build.js

I made this remark (as side effect...) in #146 too...
If it is so easy to do with browserify, might I suggest to include this step in your build process, and to add the result to the project? This way, we can refer to the raw listing directly on GitHub from JSFiddle, Plunker, JSBin or CodePen... These sites can be used to show off code without asking users to clone the project and run it on their machines! And can be embedded in blog posts and such. So it would be good at least for marketing purpose. :grin:
RequireBin is useful, but it doesn't allow debugging (inlined code as base64!), and being able to use alternatives would be great.

@PhiLhoSoft good points :)  There are lots of good tools that, like it or not, still want script tags.  I didn't really think about RequireBin's lack of debugging ... that's also unfortunate.

include this step in your build process

Right now, there is no build process :)  which is really nice.  Commit, push, tag, npm publish, done.  That said, adding a single step like npm run build probably wouldn't be too bad.  I'll probably do it :)

Actually, I'm thinking of adding a built+minified version on jsdelivr.  What do you think?

See #150

:+1: :smile:

This is done in #150, and it'll go out in the next release (within the next day).


Proxy support

Hello, would it be possible to add HTTP(S) proxy support to Procurement? Right now, it seems to always try to connect to PoE servers directly. Thanks!

Procurement has had proxy support since version 1, check settings.xml!

No problem, now that you mention it I don't think it is actually documented anywhere.


// C# record type for items in the container public record Product(     string id,     string category,     string name,     int quantity,     bool sale );

Closing as spam


SOAP Autodiscover WSDL

A client which developes for .NET enviroment needs the same targetNamespace for the production and test enviroment.
For now using the Autodiscover feature, thats not possible. Becouse once you define the URI, the same value is set for targetNamespace and soap:address "location".
I would need a way to leave the targetNamespace the same, and only change the soap:adresss location in the <service><port><soap:address></></></> to distinguish between production and test enviroment.

This issue has been moved from the zendframework repository as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html
New issue can be found at: https://github.com/zendframework/zend-soap/issues/24


✨ Allow resetting visibility window of not visible messages

#335  👈
main

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

Join @RTLS and the rest of your teammates on  Graphite


multi root

Issue created from code comment with imdone.io

TODO @Ben multi root id:36
src/vs/code/electron-main/windows.ts:769
@imdone - Efficiently manage your project's technical debt. imdone.io

Issue closed by removing a comment.

TODO @Ben multi root id:36 gh:46
src/vs/code/electron-main/windows.ts:769
@imdone - Efficiently manage your project's technical debt. imdone.io


Fix $clientSecret passing as true when $client->isConfidential() is false

…alse

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Tests pass?
✔

Fixed issues
Fix $clientSecret passing as true when $client->isConfidential() is false

Thanks 👍


MQTT Pub to Sub Latency not shown when pool is large

Hi (again)
I'm now getting sensible results, but have a query... e.g. these should be broadly similar:
mzbench  run --nodes=4 --env poolsize=250 --env rate=40 mqtt-ramp.bdl
mzbench  run --nodes=4 --env poolsize=500 --env rate=20 mqtt-ramp.bdl
mzbench  run --nodes=4 --env poolsize=1000 --env rate=10 mqtt-ramp.bdl
All three scenarios execute successfully, and I see the expected throughput in the graphs. There is one small fly in the ointment. In the final run where poolsize = 1000, the MQTT Pub to Sub Latency graph is empty.
This is completely repeatable. If I drop back to 500 / 20, the graph is drawn again. The pub/sub latency is very interesting for us so I'd really like to know if it can appear more reliably.
Oddly I found that the 1000/10 run was the least stressful on VerneMQ of all of the three runs!

From left to right, the graphs are 250/40, 1000/10, 500/20. Similarly the amount of network traffic was much less on the 1000/10 run - is this MZBench 'scaling back' its reporting and causing less stress on VerneMQ (or simply a pleasant side-effect of the pub/sub latency graph not being drawn) ?
We are using two VerneMQ nodes in a cluster. (Yea, two is a bad number for quorum...)

What type of scenario is it? fan-in?
The most current reason for not seeing an end to end latency is simply that there are no consumers connected. You can see this in the Messages pane of MZBench, where you see the number of published and consumed messages.
The reason for this is that the underlying client for the MQTT workers is completely asynchronous. So sometimes you have to play around with wait times during the setup of consumers.

It's fan-in. Thank you for the tip about timing.. I've jiggled the wait() values around (currently wait 10 before the connect of the 2000 poolsize.. and another wait 10 before the loop... and that seems to be working :)
Thank you!


Error in res_imp$imputed[gene.expression == 0]

Hi,
I am trying VIPER on different datasets but it fails for one of them with the following error message:
Error in res_imp$imputed[gene.expression == 0] :
  invalid subscript type 'S4'

.. with the default parameters. I tried with an alpha of 0.5 and 1 to see if it is dependent on the method used but it leads to the same error. I also tried to feed in raw counts as well as CPM.
The dataset is of size  16k features x 900 cells, not that big, so I guess it shouldn't be an issue either. The datasets that worked have a similar size.
And as for #3, the function runs really slowly on my datasets (SMARTer and 10x protocol).
Do you have an idea to resolve this ?
Thanks,
Anthony

Hi Anthony,
Can you check the zero rate of your datasets? It looks like after filtering, the matrix becomes empty. Maybe some rows or columns have all zeros.
I know it is slow on 10X datasets. We now don’t have implementation that optimizes for low coverage large number datasets. Also our method fits regression model based on log transformed datasets. For some UMI, it might not be the optimal choice.
Thanks,
Mengjie
On Sep 25, 2019, at 2:28 AM, ansonrel <notifications<EMAIL_ADDRESS>wrote:
Hi,
I am trying VIPER on different datasets but it fails for one of them with the following error message:
Error in res_imp$imputed[gene.expression == 0] :
invalid subscript type 'S4'
.. with the default parameters. I tried with an alpha of 0.5 and 1 to see if it is dependent on the method used but it leads to the same error. I also tried to feed in raw counts as well as CPM.
The dataset is of size 16k features x 900 cells, not that big, so I guess it shouldn't be an issue either. The datasets that worked have a similar size.
And as for #3https://github.com/ChenMengjie/VIPER/issues/3, the function runs really slowly on my datasets (SMARTer and 10x protocol).
Do you have an idea to resolve this ?
Thanks,
Anthony
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/ChenMengjie/VIPER/issues/4?email_source=notifications&email_token=AD7DL3NB2HASV7TR5HZKD4LQLMHJNA5CNFSM4I2I22U2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HNQNRFA, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AD7DL3PK5UUQOZ7RA6FLOJ3QLMHJNANCNFSM4I2I22UQ.

Thanks for your quick answer Mengjie.
Apparently the dataset contains no empty row/column, and there are approximately 46% of zeros in the matrix.

How soon did you get the warning? Did it run for a while before the warning?
Sent from my iPhone
On Sep 26, 2019, at 12:54 AM, ansonrel <notifications<EMAIL_ADDRESS>wrote:
Thanks for your quick answer Mengjie.
Apparently the dataset contains no empty row/column, and there are approximately 46% of zeros in the matrix.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/ChenMengjie/VIPER/issues/4?email_source=notifications&email_token=AD7DL3IM3I7A4NPKGRAIDXLQLRE7FA5CNFSM4I2I22U2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7UMMBI#issuecomment-535348741, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AD7DL3OPD4ATQODDDUKVDBTQLRE7FANCNFSM4I2I22UQ.

Indeed, it ran for several (>2) hours before I got the message.


Override SSLContext._load_windows_store_certs with a no-op

SSLContext has a _load_windows_store_certs method which is called by SSLContext.load_default_certs to load CA certs on Windows. We could replace it with a method that does nothing to avoid unnecessary work.

At a high level that makes sense. We'll need to think carefully about how to name it to avoid confusion. Because load_default_certs is kind of already a way to say "we want system certs," except that of course it doesn't work on Mac and it doesn't switch away from using OpenSSL as the way to verify the certs that have been loaded. I don't have a specific proposal yet but it seems like we want something that's more about what system to use for verification, rather than "where the CA certs come from" (though that's probably the mental model that most users have about what needs to be fixed).


[AutoDiff upstream] Add flag-gated AdditiveArithmetic derivation.

Add AdditiveArithmetic derived conformances for structs, gated by the
-enable-experimential-additive-arithmetic-derivation flag.
Swift Evolution pitch thread.
Structs whose stored properties all conform to AdditiveArithmetic can derive
AdditiveArithmetic:

static var zero: Self
static func +(lhs: Self, rhs: Self) -> Self
static func -(lhs: Self, rhs: Self) -> Self
An "effective memberwise initializer":

Either a synthesized memberwise initializer or a user-defined initializer
with the same type.

Effective memberwise initializers are used only by derived conformances for
Self-returning protocol requirements like AdditiveArithmetic.+, which
require memberwise initialization.
Resolves TF-844.
Unblocks TF-845: upstream Differentiable derived conformances.

Example:
struct Vector2: AdditiveArithmetic {
  var x, y: Float

  // Compiler synthesizes:
  //
  // static var zero: Self {
  //   return Self(x: Float.zero, y: Float.zero)
  // }
  //
  // static func +(lhs: Self, rhs: Self) -> Self {
  //   return Self(x: lhs.x + rhs.x, y: lhs.y + rhs.y)
  // }
  //
  // static func -(lhs: Self, rhs: Self) -> Self {
  //   return Self(x: lhs.x - rhs.x, y: lhs.y - rhs.y)
  // }
}

One unimplemented pitch idea is requiring an explicit @memberwise attribute
on AdditiveArithmetic conformances to show that derivation is memberwise:
struct Vector2: @memberwise AdditiveArithmetic { ... }

Explicit @memberwise annotation is nice because memberwise AdditiveArithmetic
derivation does not make sense for some types:
// Memberwise derivation doesn't make sense here, so we don't want derivation to
// silently happen without explicit `@memberwise` annotation.
struct Fraction: AdditiveArithmetic {
  var numerator, denominator: Int
}

If general AdditiveArithmetic derivation is controversial, we can limit
derivation to TangentVector member structs synthesized during Differentiable
derived conformances, which is the use case we care about.

@CodaFi: would you like to review this patch, as a derived conformances code owner?
Our use case for AdditiveArithmetic derived conformances is differentiable programming. We've been incubating AdditiveArithmetic derived conformances on the tensorflow branch since 2018 - it should be pretty stable.

I think a controversial change is introducing “effective memberwise initializers”, which are necessary for synthesizing Self-returning protocol requirements:
struct Vector2: AdditiveArithmetic {
  var x, y: Float

  // Compiler synthesizes:
  //
  // static var zero: Self {
  //   return Self(x: Float.zero, y: Float.zero)
  //          ^~~~
  //          memberwise initializer is necessary for derivation
  // }
  // ...
}

Synthesized memberwise initializers are also public for public types whose stored properties are all public. This breaks Swift precedent, where synthesized struct memberwise initializers are at most internal.
However, it's important for supporting use cases like TF-1077: memberwise initializers for public synthesized types (e.g. TangentVector member types) should be public because manually declaring public memberwise initializers for these types produces a redeclaration error.

Feedback on the AdditiveArithmetic derivation pitch thread is mixed, so I'm not sure it can pass Evolution.
However, since AdditiveArithmetic derivation exists to support differentiable programming, we can limit AdditiveArithmetic derivation specifically for TangentVector member structs synthesized during Differentiable derived conformances. This seems less controversial, to support upstreaming differentiable programming.

@swift-ci Please test

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test

Merging to unblock progress. Happy to address feedback later!


Fix keybinds not working outside feedlist (alternative, reverting changes from PR 2337)

This is an alternative to PRs:

https://github.com/newsboat/newsboat/pull/2358
https://github.com/newsboat/newsboat/pull/2355

It reverts the changes from PR 2337 and makes the minimal changes necessary to still fix https://github.com/newsboat/newsboat/issues/2336.
I prefer the other options (less STFL magic in C++ code), but this solution should work fine.
Tested with stfl versions 0.24 and 0.22.

Coverage: 59.487% (+0.05%) from 59.437% when pulling 343a7e96a13fddeacaae34e9b3e0d9f53a5962ad on dennisschagt:revert-pr-2337-with-alternative into 43751b47b82330f2da2c07c8fdab7bce60aed6b4 on newsboat:master.

No longer relevant, since https://github.com/newsboat/newsboat/pull/2358 is merged


[Bug]: download model on colab

What happened?
Hello, how can I download the models with api on colab ؟
Steps to reproduce the problem

Go to ....
Press ....
...

What should have happened?
Hello, how can I download the models with api on colab ؟
What platforms do you use to access the site?
Linux
What browsers do you use to access the site?
Mozilla Firefox
Additional information, context and logs
No response

What have you tried?
This was working previously and now you need to add your civitia api key on some models/loras.

What have you tried?
This was working previously and now you need to add your civitia api key on some models/loras.

I know this, but I use colab, so I have to download the models every time I want to use it, so I want to know how to add the api to colab so that there is no problem when downloading, and I will add

@wolfkingal2000 Hi!
You can download the file by giving the URL parameter as follows
https://civitai.com/api/download/models/xxxxx?token={civitai_api_key}

To get a Civitai API Key, you need to create a Civitai account.

@wolfkingal2000 Hi!
You can download the file by giving the URL parameter as follows
https://civitai.com/api/download/models/xxxxx?token={civitai_api_key}

To get a Civitai API Key, you need to create a Civitai account.

Reall? Thanks for answering

I think we will need a fix to add the api key to a config or cli argument so that if any calls to the civitai.com api are used, it gets passed to all of them as it only seems to affect the ones where the model owners have enabled that feature, eg they required the user be logged in to download the model.

I think we will need a fix to add the api key to a config or cli argument so that if any calls to the civitai.com api are used, it gets passed to all of them as it only seems to affect the ones where the model owners have enabled that feature, eg they required the user be logged in to download the model.

Thank you, it would be great if you made this command or make cell command  send this issue to use that

I think we will need a fix to add the api key to a config or cli argument so that if any calls to the civitai.com api are used, it gets passed to all of them as it only seems to affect the ones where the model owners have enabled that feature, eg they required the user be logged in to download the model.

Thank you, it would be great if you made this command or make cell command send this issue to use that

My fork/branch has it in notebook as a variable but it is a little manual currently depending on what type of stuff I am creating.

It would be nice if we could use the Authorization: Bearer {token} header instead of a GET param.
For example, when using bash + wget, it's very easy to add this header, but non-trivial to tack-on a GET param to an arbitrary URL that may or may not already have existing GET params

 https://github.com/civitai/sd_civitai_extension


Invoking choice from Show CodeLens Commands For Current Line requires 2 Enters or clicks

Type: Bug

Set "typescript.referencesCodeLens.enabled": true
Open a  .ts file.
Put cursor on a line that has an N references CodeLens above it.
Open Command Palette and run Show CodeLens Commands For Current Line.
Press Enter to try and run the selected (only) entry (N references).

:bug: nothing happens until you press Enter a second time.
Results of using a mouse-click to try and run the CodeLens from the quickpick is the same. First click does nothing, and a second click is required.
This also happens in 1.81.1 so it is not a 1.82 regression.
VS Code version: Code - Insiders 1.82.0-insider (e7756c8870ee1df7360e6624e220534174039b02, 2023-08-31T05:33:59.698Z)
OS version: Windows_NT x64 10.0.22621
Modes:

I can reproduce. The code lens model is disposed because the focus lost/gain cycle retriggers its computation and then the picker restarts (to make sure a fresh item is selected)...

/verified

Added verified tag, as this was author-verified


Cannot connect WSL2 vEth interfaces in Mirrored mode even hostAddressLoopback is true.

Windows Version
Microsoft Windows [Version 10.0.22631.2506]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-1
Distro Version
Ubuntu 22.04.3 LTS
Other Software
.wslconfig:
[wsl2]
memory=4GB
localhostForwarding=true
#networkingMode=bridged
#vmSwitch=vNAT
#networkingMode=mirrored
guiApplications=true

[experimental]
autoMemoryReclaim=gradual #
networkingMode=mirrored
dnsTunneling=true
firewall=true
autoProxy=true
sparseVhd=true
hostAddressLoopback=true

Repro Steps

Enter WSL2 and create a network space and veth device with commands below

ip netns add ns1

Create a pair of veth devices and activate them with commands below

ip link add type veth
ip link set up dev veth0
ip link set veth1 netns ns1
ip a a <IP_ADDRESS>/24 dev veth0

Enter ns1 and allocate an IP.

ip netns exec ns1 bash
ip link set up dev veth1
ip a a <IP_ADDRESS>/24 dev veth1

Ping each other to make sure that netns is working

ping  <IP_ADDRESS> # in ns1
ping <IP_ADDRESS> # in wsl2 root

PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=64 time=0.034 ms

Ping either <IP_ADDRESS> or <IP_ADDRESS> in Windows cmd.

ping <IP_ADDRESS>

Windows firewall is disabled.
Expected Behavior
Windows can ping and connect the veth network normally.
Actual Behavior
Windows can't ping or connect the veth network at all.
22:08:29 -1 ❯ ping <IP_ADDRESS>

Pinging <IP_ADDRESS> with 32 bytes of data:
Request timed out.
Request timed out.
Request timed out.
Request timed out.

Ping statistics for <IP_ADDRESS>:
    Packets: Sent = 4, Received = 0, Lost = 4 (100% loss),

15:28:32 ERROR ❯ ping <IP_ADDRESS>

Pinging <IP_ADDRESS> with 32 bytes of data:
Request timed out.
Request timed out.
Request timed out.
Request timed out.

Ping statistics for <IP_ADDRESS>:
    Packets: Sent = 4, Received = 0, Lost = 4 (100% loss),

Diagnostic Logs
No response
### Tasks

My fault. It seems like that only Windows Insiders are permitted to use hostAddressLoopback option and it doesn't work in 23H2 stable.

I was also trying to do something similar to @kenvix and am unable to get the Windows host to be able ping the veth (192.168.99.x addresses).  I don't believe this issue is related to hostAddressLoopback, having it enabled or disabled both yield the same failure.

Hi. Can you please collect networking logs by following the instructions below?
https://github.com/microsoft/WSL/blob/master/CONTRIBUTING.md#collect-wsl-logs-for-networking-issues

Hey @chanpreetdhanjal, can you elaborate on what kind of networking logs should be collected?  The link you sent indicates to "reproduce the issue", but there isn't really traffic to reproduce in this case since it's more of an issue with the mechanism.

hello. thanks for reporting the issue and sorry for the delay following up on this
the hostAddressLoopback setting is intended for communication between Windows and Linux using IP addresses assigned to Windows, and only using TCP and UDP traffic.
for example if Windows has IP A, you can have a TCP/UDP server on Windows listening on IP A and connect to it from Linux, or a TCP/UDP server in Linux listening on IP A and connect to it from Windows
In this case you are trying to communicate with an IP that is assigned exclusively to Linux. for this scenario you will need some sort of iptables/nftables configuration to redirect traffic to the Linux vEth interfaces.

@CatalinFetoiu One of the issues I ran into is that routes are mirrored from windows -> linux, but don't appear to be mirrored from linux -> windows.  In this particular case, and excuse my potential miss-understanding here, but the veth in linux creates a route for <IP_ADDRESS>/24 pointing to veth0, however this is not visible in Windows.  If I create a route in windows pointing <IP_ADDRESS>/24 to the interface that's mirrored for WSL2, I end up with two conflicting routes on the linux side, and traffic destined to that interface from Windows does not appear to make it to the veth.  Perhaps a clarifying question here might be how I can get traffic from a windows application to be routed using the linux nftables, such that it can be routed to the veth.

@brett19, regarding mirrored mode, you are right, it is intended to mirror IP properties, such as addresses, routes from windows to linux, but not the other way around.
regarding configuration of nftables/iptables, you might look at what rules Docker configures when you run "docker run -d -p 8080:80 nginx:alpine", since Docker also creates veth interfaces and configures nftables rules to send traffic to/from them
for example you might try the following scenario:  enable the hostAddressLoopback setting in wslconfig, run "docker run -d -p 8080:80 nginx:alpine", then connect to :8080 from Windows, where  is an ipv4 address assigned to your Windows machine. Please make sure to have the latest wsl version, since we made some fixes for those types of scenarios

Hey @CatalinFetoiu, do you have any suggestions on how I might be able to get windows to route an alternative ip address into WSL in the context of mirrored networking?  I broadly attempted to do what you are describing, the issue I ran into (in the context of the issues veth setup) was that if I create a route in windows for <IP_ADDRESS>/24 pointing to the mirrored interface, this route ends up being mirrored onto the linux side, making it challenging to then route the packets differently from there.  For reference, the actual use-case here is that when using docker, I'm broadly able to set up containers to have their own IP addresses with the default bridge network, and this largely works correctly from the linux side of things (there is connectivity between WSL linux using the containers IP address), however those container IPs are not routable from Windows.  Note that I believe this is slightly distinct from the hostAddressLoopback scenario you described where the packets are already being routed through the mirrored host interface.
P.S. The more I'm discussing this issue, the more I'm realizing this may not actually be anything fundamentally "broken" with WSL, but rather a confluence of my lack of understanding of how WSL2 intends to have this kind of setup routed, along with more of a feature-request for enabling WSL mirrored networking to natively support correctly representing veth interfaces on the Windows side.

@brett19 thanks for following up and for your explanation
mirrored mode supports communication between the Windows host and Linux using either <IP_ADDRESS> or an IPv4 address assigned to Windows (for which case you need to enable the hostAddressLoopback setting). If you are trying to communicate from Windows with an IP address that is not assigned to Windows (such as the IP address assigned to the Linux veth interface), that traffic will be sent out of the Windows host, it will not be sent to Linux.
if you want to communicate from Windows or from an external machine with the IP assigned to the Linux veth interface, then the common way to enable this would be the following:
Let's say Windows has an Ethernet interface with IP X. Then we'll mirror that interface as a Linux eth0 interface and assign it the same IP X.
Then Windows or an external machine will communicate with the "external facing" IP of Linux (in this case X) and Linux will receive that traffic on the eth0 interface and forward it between eth0 and the veth interface. a way to do this would be using iptables/nftables
hope this helps. Let me know if you have any questions


Blueprint $table->foreign('firmware')->constrained() does not pluralize firmware correctly

Laravel Version: 7.2.1
PHP Version: 7.3.13
Database Driver & Version: Postgres 12.2

Description:
The core issue is that $table->foreign()->constrained() does not pluralize  correctly in some contexts.
Specific error:
Given Blueprint $table in a migration
$table->foreignId('vendor_firmware_id')->constrained(); becomes
$table->foreign('vendor_firmware_id')->references('id')->on('vendor_firmwares');
instead of
$table->foreign('vendor_firmware_id')->references('id')->on('vendor_firmware');
firmware should not be pluralized. It is not pluralized correctly in other migration contexts.
Steps To Reproduce:
class CreateVendorFirmwareTable extends Migration
{
    public function up()
    {
        Schema::create('vendor_firmware', function (Blueprint $table) {
            $table->id();
            $table->string('firmware')->nullable();
        });
    }
}

And a second table
class CreateDriveFirmwareTable extends Migration
{
    public function up()
    {
        Schema::create('drive_firmware', function (Blueprint $table) {
            $table->id();
            $table->foreignId('vendor_firmware_id')->constrained();
        });
    }

The migration fails
php artisan migrate:fresh
Dropped all tables successfully.
Migration table created successfully.
Migrating: 2020_03_19_100200_create_vendor_firmware_table
Migrated:  2020_03_19_100200_create_vendor_firmware_table (0.01 seconds)
Migrating: 2020_03_19_103100_create_drive_firmware_table

   Illuminate\Database\QueryException

  SQLSTATE[42P01]: Undefined table: 7 ERROR:  relation "vendor_firmwares" does not exist (SQL: alter table "drive_firmware" add constraint "drive_firmware_vendor_firmware_id_foreign" foreign key ("vendor_firmware_id") references "vendor_firmwares" ("id"))

  at vendor/laravel/framework/src/Illuminate/Database/Connection.php:669
    665|         // If an exception occurs when attempting to run a query, we'll format the error
    666|         // message to include the bindings with SQL, which will make this exception a
    667|         // lot more helpful to the developer instead of just the database's errors.
    668|         catch (Exception $e) {
  > 669|             throw new QueryException(
    670|                 $query, $this->prepareBindings($bindings), $e
    671|             );
    672|         }
    673|

      +9 vendor frames
  10  database/migrations/2020_03_19_103100_create_drive_firmware_table.php:27
      Illuminate\Support\Facades\Facade::__callStatic("create")

      +35 vendor frames
  46  artisan:37
      Illuminate\Foundation\Console\Kernel::handle(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))

The plural is coming from this package: https://github.com/doctrine/inflector

Please open a ticket on the doctrine package.

For completeness, this should be resolved by https://github.com/doctrine/inflector/pull/120


KAFKA-5692: Change PreferredReplicaLeaderElectionCommand to use Admin…

…Client
See also KIP-183.
The contribution is my original work and I license the work to the project under the project's open source license.
This implements the following algorithm:

AdminClient sends ElectPreferredLeadersRequest.
KafakApis receives ElectPreferredLeadersRequest and delegates to
ReplicaManager.electPreferredLeaders()
ReplicaManager delegates to KafkaController.electPreferredLeaders()
KafkaController adds a PreferredReplicaLeaderElection to the EventManager,
ReplicaManager.electPreferredLeaders()'s callback uses the
delayedElectPreferredReplicasPurgatory to wait for the results of the
election to appear in the metadata cache. If there are no results
because of errors, or because the preferred leaders are already leading
the partitions then a response is returned immediately.

In the EventManager work thread the preferred leader is elected as follows:

The EventManager runs PreferredReplicaLeaderElection.process()
process() calls KafkaController.onPreferredReplicaElectionWithResults()
KafkaController.onPreferredReplicaElectionWithResults()
calls the PartitionStateMachine.handleStateChangesWithResults() to
perform the election (asynchronously the PSM will send LeaderAndIsrRequest
to the new and old leaders and UpdateMetadataRequest to all brokers)
then invokes the callback.

Note: the change in parameter type for CollectionUtils.groupDataByTopic().
This makes sense because the AdminClient APIs use Collection consistently,
rather than List or Set. If binary compatiblity is a consideration the old
version should be kept, delegating to the new version.
I had to add PartitionStateMachine.handleStateChangesWithResults()
in order to be able to process a set of state changes in the
PartitionStateMachine and get back individual results.
At the same time I noticed that all callers of existing handleStateChange()
were destructuring a TopicAndPartition that they already had in order
to call handleStateChange(), and that handleStateChange() immediately
instantiated a new TopicAndPartition. Since TopicAndPartition is immutable
this is pointless, so I refactored it. handleStateChange() also now returns
any exception it caught, which is necessary for handleStateChangesWithResults()

@ijuma I've rebased this PR for the Protocol refactoring.

Rebased for merge conflict. Please could you review this when you have time @ijuma?

@cmccabe can you please review this?

Thanks for the patch, @tombentley.  There are some issues here, however.  It seems like all the async APIs are actually sync, because the code to retrieve the future is calling Future#get.  This obviously defeats the point of an async API.
Another issue I see is that you modify a map in the results object concurrently with users accessing the map.  If you want to have your code inserting elements into the map, you need locking to do that.
You also need to handle error conditions like the server sending the wrong thing, etc.

Tom, there is a patch here which shows what I'm talking about: https://github.com/apache/kafka/pull/4183

@cmccabe thanks going to the trouble of pointing out what must have seemed blindingly obvious.
Your patch raises a broader question in my mind: If it's OK to just use a Future<Map<Key, Value>> for this Result class why is that approach also not used in the other result classes, such as CreateTopicsResult, which use a Map<Key, Future<Value>> but likewise get a single response from a broker?
It was the precedents set by the existing Results which set me on the path of using a Map<Key, Future<Value>>. Personally I couldn't see a great benefit in a Map<Key, Future<Value>> when all the results are available at once. I feel l must be missing some subtlety.

@cmccabe a problem with your patch is that it doesn't work when the electionFuture completes abnormally (e.g. the Call times out and the future is completed via handleFailure()). thenApply() only invokes the Function when the receiver future completes normally, so abnormal completion of the electionFuture means the result future is never completed. We could fix this by exposing the BiConsumer and addWaiter(). wdyt?

@cmccabe any thoughts about the preferred way to deal with abnormal completion of then electionFuture?

bq. @tombentley wrote: thanks going to the trouble of pointing out what must have seemed blindingly obvious.
It's OK.  It wasn't obvious to me when I started doing async programming.  But over time it became easier :)
bq. a problem with your patch is that it doesn't work when the electionFuture completes abnormally (e.g. the Call times out and the future is completed via handleFailure()). thenApply() only invokes the Function when the receiver future completes normally, so abnormal completion of the electionFuture means the result future is never completed. We could fix this by exposing the BiConsumer and addWaiter(). wdyt?
Good catch.  I think this shortcoming is why I didn't use this approach for things like createTopics.
bq. @cmccabe any thoughts about the preferred way to deal with abnormal completion of then electionFuture?
Sorry, this got buried in my email box.  But I fixed my email settings to make notifications more visible...
I think @tombentley's idea of exposing the BiConsumer as public is the best way to go.  We should probably make it a pure interface to make things look nicer in Java 8. (Function should have been a pure interface, too.....)

Rebased onto current trunk. This PR now assumes that KIP-218 gets accepted and merged, since it uses KafkaFutureImpl.addWaiter() (using a dummy patch in place of the real KIP-218 work). This will probably need to be rebased again soon due to "KAFKA-5647: Use KafkaZkClient in ReassignPartitionsCommand and PreferredReplicaLeaderElectionCommand (#4260)".

Thanks for being patient, @tombentley.  We're working on getting the KafkaFuture change(s) in.

@cmccabe, @ijuma I have rebased this onto latest trunk. Would be grateful for review.

Thanks, @tombentley !  I took a look.  We should also ask someone familiar with the controller code to take a look at those parts.

@ijuma are you able to review this?

@tombentley Sorry for the delay. I probably won't have time to review this until late next week. Not sure if @junrao has time before that.

No problem @ijuma. It would great if @junrao could look at it.

If I rebase these changes is there any chance of getting this reviewed in the near future, @ijuma, @junrao?

@tombentley : sorry for the delays.  I was on vacation for a few weeks, and I think a lot of other folks are busy with the KIPs for 2.0.  I'm not sure if this PR will make 2.0 at this point.  But please do rebase when you get the chance!