AutoCommandBufferBuilder.build() panics with debug_assert! failure with ImageLayout::Undefined

Version of vulkano: 0.32.1
OS: Windows
GPU (the selected PhysicalDevice): NVidia GTX 1650

Vulkano panics with the following message:
thread 'main' panicked at 'assertion failed: !matches!(new_layout, ImageLayout :: Undefined | ImageLayout ::\n        Preinitialized)', C:\Users\user\.cargo\registry\src\github.com-1ecc6299db9ec823\vulkano-0.32.1\src\command_buffer\commands\sync.rs:348:21
The render pass:
let render_pass = RenderPass::new(device.clone(), RenderPassCreateInfo {
        attachments: vec![ AttachmentDescription {
            format: Some(swapchain.image_format()),
            load_op: LoadOp::Clear,
            store_op: StoreOp::Store,
            initial_layout: ImageLayout::Undefined,
            final_layout: ImageLayout::PresentSrc,
            ..Default::default()
        }],
        subpasses: vec![ SubpassDescription {
            color_attachments: vec![ Some(AttachmentReference {
                attachment: 0,
                layout: ImageLayout::ColorAttachmentOptimal,
                ..Default::default()
            })],
            ..Default::default()
        }],
        ..Default::default()
    }).unwrap();

The AutoCommandBufferBuilder:
let mut builder = AutoCommandBufferBuilder::primary(&self.cmd_allocator, self.graphics_queue.queue_family_index(), CommandBufferUsage::OneTimeSubmit).unwrap();

builder
    .begin_render_pass(render_pass_begin_info, SubpassContents::Inline).unwrap()
    .bind_pipeline_graphics(self.pipeline.clone())
    .set_viewport(0, [Viewport {
        origin: [0.0, 0.0],
        dimensions: [self.swapchain.image_extent()[0] as f32, self.swapchain.image_extent()[1] as f32],
        depth_range: 0.0..1.0,
    }])
    .set_scissor(0, [Scissor {
        origin: [0, 0],
        dimensions: self.swapchain.image_extent(),
    }])
    .draw(3, 1, 0, 0).unwrap()
    .end_render_pass().unwrap();

return builder.build().unwrap()

Issue looks very similar to #1894, but that should be already fixed in 0.32.1, right?
I will try to untangle the rest of my code and generate a minimal working example of the issue tomorrow.

As promised, here's a complete minimal working example that shows the assertion failing

Show/hide code
extern crate vulkano;
extern crate vulkano_win;
extern crate winit;

use vulkano::{
    VulkanLibrary,
    device::{DeviceCreateInfo, Device, DeviceExtensions, QueueCreateInfo},
    device::physical::{PhysicalDeviceType, PhysicalDevice},
    instance::{Instance, InstanceCreateInfo},
    swapchain::{SurfaceInfo, Surface, Swapchain, SwapchainCreateInfo, ColorSpace, PresentMode},
    image::{ImageUsage, ImageLayout, view::ImageView},
    format::{Format, ClearValue},
    sync::{Sharing, PipelineStages, AccessFlags},
    render_pass::{RenderPass, RenderPassCreateInfo, AttachmentDescription, LoadOp, StoreOp, SubpassDescription, AttachmentReference, SubpassDependency, Framebuffer, FramebufferCreateInfo},
    command_buffer::{RenderPassBeginInfo, AutoCommandBufferBuilder, SubpassContents, CommandBufferUsage},
    command_buffer::allocator::{StandardCommandBufferAllocator, StandardCommandBufferAllocatorCreateInfo},
};

use vulkano_win::{VkSurfaceBuild};

use winit::{event_loop::EventLoop, window::WindowBuilder};

fn main() {
    let library = VulkanLibrary::new().unwrap();
    let instance = Instance::new(library.clone(), InstanceCreateInfo {
        enabled_extensions: vulkano_win::required_extensions(&library),
        ..Default::default()
    }).unwrap();

    let event_loop = EventLoop::new();
    let surface = WindowBuilder::new().build_vk_surface(&event_loop, instance.clone()).unwrap();

    let physical_devices = instance.enumerate_physical_devices().unwrap().flat_map(|dev| {
        if !dev.supported_extensions().khr_swapchain { return None }
        if dev.surface_formats(&surface, SurfaceInfo::default()).unwrap().is_empty() { return None }
        if dev.surface_present_modes(&surface).unwrap().collect::<Vec<_>>().is_empty() { return None }
        let queue_family = dev.queue_family_properties().iter().enumerate().find(|(family, props)| {
            props.queue_flags.graphics && dev.surface_support(*family as u32, &surface).unwrap()
        }).map(|(i, _)| i as u32);
        queue_family.and_then(|family| Some((dev, family)))
    }).collect::<Vec<_>>();

    let (physical_device, queue_family) = physical_devices.iter().max_by_key(|(dev, _)| {
        if dev.properties().device_type == PhysicalDeviceType::DiscreteGpu { 1 } else { 0 }
    }).unwrap();

    let (device, _) = Device::new(physical_device.clone(), DeviceCreateInfo {
        enabled_extensions: DeviceExtensions {
            khr_swapchain: true,
            ..Default::default()
        },
        queue_create_infos: vec![QueueCreateInfo {
            queue_family_index: *queue_family,
            ..Default::default()
        }],
        ..Default::default()
    }).unwrap();

    let (swapchain, images) = Swapchain::new(device.clone(), surface.clone(), get_swapchain_info(physical_device, &surface)).unwrap();

    let render_pass = RenderPass::new(device.clone(), RenderPassCreateInfo {
        attachments: vec![ AttachmentDescription {
            format: Some(swapchain.image_format()),
            load_op: LoadOp::Clear,
            store_op: StoreOp::Store,
            initial_layout: ImageLayout::Undefined,
            final_layout: ImageLayout::PresentSrc,
            ..Default::default()
        }],
        subpasses: vec![ SubpassDescription {
            color_attachments: vec![ Some(AttachmentReference {
                attachment: 0,
                layout: ImageLayout::ColorAttachmentOptimal,
                ..Default::default()
            })],
            ..Default::default()
        }],
        dependencies: vec![ SubpassDependency {
            src_subpass: None,
            dst_subpass: Some(0),
            src_stages: PipelineStages {
                color_attachment_output: true,
                ..PipelineStages::empty()
            },
            src_access: AccessFlags::empty(),
            dst_stages: PipelineStages {
                color_attachment_output: true,
                ..PipelineStages::empty()
            },
            dst_access: AccessFlags {
                color_attachment_write: true,
                ..AccessFlags::empty()
            },
            by_region: true,
            ..Default::default()
        } ],
        ..Default::default()
    }).unwrap();

    let framebuffers = images.iter().map(|image| {
        Framebuffer::new(render_pass.clone(), FramebufferCreateInfo {
            attachments: vec![ ImageView::new_default(image.clone()).unwrap() ],
            ..Default::default()
        }).unwrap()
    }).collect::<Vec<_>>();

    let cmd_allocator = StandardCommandBufferAllocator::new(device.clone(), StandardCommandBufferAllocatorCreateInfo {
        primary_buffer_count: 24,
        secondary_buffer_count: 0,
        ..Default::default()
    });

    let mut render_pass_begin_info = RenderPassBeginInfo::framebuffer(framebuffers[0].clone());
    render_pass_begin_info.clear_values = vec![Some(ClearValue::Float([0.0, 0.0, 0.0, 1.0]))];

    let mut builder = AutoCommandBufferBuilder::primary(&cmd_allocator, *queue_family, CommandBufferUsage::OneTimeSubmit).unwrap();

    builder
        .begin_render_pass(render_pass_begin_info, SubpassContents::Inline).unwrap()
        .end_render_pass().unwrap();

    builder.build().unwrap();
}

fn get_swapchain_info(physical_device: &PhysicalDevice, surface: &Surface) -> SwapchainCreateInfo {
    let mut swapchain_info = SwapchainCreateInfo {
        image_usage: ImageUsage {
            color_attachment: true,
            ..ImageUsage::empty()
        },
        image_sharing: Sharing::Exclusive,
        ..Default::default()
    };

    let surface_formats = physical_device.surface_formats(surface, SurfaceInfo::default()).unwrap();
    let (format, colorspace) = surface_formats.iter().find(|(format, colorspace)| {
        *format == Format::B8G8R8A8_SRGB && *colorspace == ColorSpace::SrgbNonLinear
    }).unwrap_or_else(|| {
        &surface_formats[0]
    });
    swapchain_info.image_format = Some(*format);
    swapchain_info.image_color_space = *colorspace;

    swapchain_info.present_mode = physical_device.surface_present_modes(surface).unwrap().find(|mode| {
        *mode == PresentMode::Mailbox
    }).unwrap_or(PresentMode::Fifo);

    let surface_capabilities = physical_device.surface_capabilities(surface, SurfaceInfo::default()).unwrap();
    swapchain_info.image_extent = surface_capabilities.current_extent.unwrap();
    swapchain_info.min_image_count = surface_capabilities.min_image_count + 1;

    swapchain_info
}

Version of vulkano: 0.32.1
OS: MacOS Ventura (13.0.1)
GPU: AMD Radeon Pro 5500M (8GB VRAM)

I had the same issue, and I did a little digging to find a workaround.
The issue seems to be here: https://github.com/vulkano-rs/vulkano/blob/0.32.1/vulkano/src/command_buffer/synced/builder.rs#L666
That creates a memory barrier with:
old_layout: state.initial_layout,
new_layout: state.initial_layout,

That means we hit that validation error due to setting new_layout to ImageLayout::Undefined.
That made me curious about how the examples were able to run. I'm very new to rust macros, but I think it's because the examples use single_pass_renderpass, and that macro ends up setting initial_layout/final_layout to the same well-defined value: https://github.com/vulkano-rs/vulkano/blob/master/vulkano/src/render_pass/macros.rs#L87  . Here's another example without the macro: https://github.com/vulkano-rs/vulkano/blob/master/examples/src/bin/multiview.rs#L225
So my workaround for now is to set both the initial_layout and final_layout to ImageLayout::PresentSrc.
My concern is that I'm not sure whether or not this makes sense to do.

@lentz1 Thank you for investigating!

After thinking this over for a while, it doesn't seem that obvious how to fix this issue. From what I can tell, it happens in one specific case: the image is in the Undefined or Preinitialized layout before the command buffer (i.e. it's newly created) and the first command that uses the image also expects Undefined or Preinitialized. Since neither of these is valid as the new_layout of an image memory barrier, the barrier is incapable of preserving the image as it is; it has to transition it to something.
If the first command expects Preinitialized, then the image must be in that layout, there's no way around it. If it's not, then the command must simply fail. But if the command expects Undefined, then any layout should in principle be valid. So I think in this case, if the new_layout is chosen to be something random other than Undefined, then the error should go away.


Update prettier dev-dependency to v2.5.1 in Eslint plugin

Solves: https://github.com/Azure/azure-sdk-for-js/issues/9329 for packages under common/tools/eslint-plugin-azure-sdk.
Updated prettier dev-dependency version to latest 2.5.1.
Files were re-formatted as well. There are only format changes in this PR, no manual changes except for package.json files.
Main format changes with Prettier 2.x in this PR include:

Trailing commas by default.
Whitespace added after every function keyword.

The build script (tsc -p tsconfig.build.json && prettier --write dist/**/*.{js,json,md}) is causing a failure when trying to build the libraries in MacOS:
[error] No files matching the pattern were found: "dist/**/*.json".
[error] No files matching the pattern were found: "dist/**/*.md".

This is most likely due to the pattern lib being stricter in MacOS with Prettier 2. Anyway, after talking with @deyaaeldeen we concluded that it does not make sense to format the dist folder with prettier with the build script. So we could remove the whole prettier part from it. This should fix the CI error.

/azp run js - eslint-plugin - ci

/azp run js - core - ci

CI is passing now :)


Split mapstate keys into allow and deny

The first commit converts the MapState type from a newtype an interface (mostly Nate's work, rebased on newer main)
The second commit then splits the map in the struct into two - to track allows and denies seperately.
This allows for an optimization in the third commit: denyPreferredInserts for the allow case, now doesn't have to loop through all other allows (which we don't care about).
Commit msgs for convenience:
policy: Make MapState an interface
`MapState` has many methods, some of which are public.
In order to keep state to accomplish creating a cohesive
map state that can keep state besides what will be put into
an ebpf map, `MapState` can no longer be a type declaration
for a golang map, but must be a struct that keeps state.

The endpoint package must also be refactored to use `MapState`
as an interface rather than a instantiated type of a map.

mapstate: split allows and denies
Now that mapState is a struct, we can split the tracking of allows and
denies. This commit should not introduce functional changes, but
prepares us for an optimization in a later commit.

mapstate: optimize denyPreferredInsert
Since the denyPreferredInsert logic cleanly splits on whether the to be
inserted entry is a deny entry, we can avoid iterating over some
entries: If we are inserting a deny, we are mostly interested in
iteration over existing allows, and vice versa.

In the case where we have a FQDN policy for a hostname which resolves to
many IPs (for example with a fqdn policy covering Amazon's S3), this
becomes a hot function (without performing a lot of useful work). The S3
workload is expected to be heavily skewed towards allow entries for the
many CIDR identities allocated for S3. When we insert such an entry, we
only need to iterate deny entries, of which there are expected to be
much fewer. Now that the tracking is split, we can employ this fact and,
instead of iterating all entries, iterate only the deny entries.

/test

/test

/ci-ginkgo

/ci-gateway-api

/ci-gateway-api

/test

@bimmlerd Changed v1.12 backport label to needs-backport/1.12 to be able to complete the release process. Looks like we can't have any pending backports when doing the release.

removing backport-pending labels here, as the project has decided against backporting this change -- it's too invasive for extensive backporting.


Added platform check for arm recipe test

Attempting to run arm tests on a non-arm platform will fail, so check if the build machine actually uses arm before running the test.

Looks like AppVeyor build failed because of missing zlib dependency, nothing related to my changes right?

@FlorentP42 yes, don't worry about it, it's safe to ignore for now. I'll add missing configurations to zlib a bit later.


Add custom commandline code to transform the data to nexus and the like

We can now add the code, as you can see from chenhmongmien. I suggest we do the same for the MIT paper.

Pretty bare bones: https://github.com/lexibank/lieberherrkhobwa/commit/8cd82a44638fe70c787063a68236409385b8aac9
Anything else you'd like to see added?

Done


SOLID POD: Secure Key request when entered and press ENTER it should be same as OK

User Story
After I have typed the key here, as a user I am inclined to press ENTER. This should be the same a tapping the OK button.

Implemented in pull request https://github.com/anusii/solidpod/pull/164
To test, go to the keypod demo page and

Click "Forget Security Key"
Turn on the "Encrypt Data" switch
Click "Key Value Table Demo"
Add a key-value pair and click "Submit"
In the security key input widget, type your security key and press the enter key.

Known issue:

If you move the input focus off the text input after typing your security key, then press the enter key seems to do nothing (haven't been able to find a fix for this so far).

Tested and works as expected


local storage impl & hide dark mode toggle on section

fixes #63
fixes #64

@katherinepeterson i have implemented the local storage functionality . Try it out on the preview deployment url and let me know your thoughts about it and any issues/gaps you face while validating the feature/code . If you think there are issues / we can move this to draft mode , and keep fixing remaining issues . If you think , its good to go , still let me test it on the preview deployment for one more day to be sure its good to go !
Also , this PR also fixes the issue where now when a section is deleted , dark-mode-toggle button also gets hidden , as there is no editor to toggle upon , unless a user selects a section to work upon .

@saurabhchaturvedy looks pretty good, a couple notes:

It looks like some of the changes are just formatting. Looks like you either aren't using a formatter or it's using different settings. Could you run Prettier with the settings in .prettierrc?
Can you resolve the conflicts with main?

@katherinepeterson , sure will fix the formatting & resolve the conflicts . Thanks for reviewing .

I noticed a bug with respect to the state always having title and description section present even when i delete it. It resets to this. @katherinepeterson can you confirm if this is the behaviour we want?
Before Refresh:

After Refresh

@rohitjmathew @katherinepeterson  i actually kept it like this , to let user start off with a placeholder every time , as instead of showing empty selected sections (as also when first user clicks the editor page links he starts with 'title-and-description'),  as anyways user will try to select something to start with .  But if thats does not good , i'll change it .

closing this PR as merging changes for two changes clubbed together on PR #80


Collapse catchments initial function working

Fixes #52 still need to deal with flowlines

Coverage increased (+0.2%) to 95.124% when pulling 7fdb858057c79fd937c2cb27bc86d6818ac2c8f0 on collapse into e9289869f898b5cea8942788354a1175aeefdc79 on master.


Run profiler after success

Coverage remained the same at 100.0% when pulling df5304aa67b74e093f6cf1cc089420e172d628df on run-profiler into c92d26864444b5af55042bbc9d046d3f85b0571c on master.

Coverage remained the same at 100.0% when pulling df5304aa67b74e093f6cf1cc089420e172d628df on run-profiler into c92d26864444b5af55042bbc9d046d3f85b0571c on master.

Coverage remained the same at 100.0% when pulling df5304aa67b74e093f6cf1cc089420e172d628df on run-profiler into c92d26864444b5af55042bbc9d046d3f85b0571c on master.

Coverage remained the same at 100.0% when pulling df5304aa67b74e093f6cf1cc089420e172d628df on run-profiler into c92d26864444b5af55042bbc9d046d3f85b0571c on master.

Coverage remained the same at 100.0% when pulling df5304aa67b74e093f6cf1cc089420e172d628df on run-profiler into c92d26864444b5af55042bbc9d046d3f85b0571c on master.

Coverage remained the same at 100.0% when pulling df5304aa67b74e093f6cf1cc089420e172d628df on run-profiler into c92d26864444b5af55042bbc9d046d3f85b0571c on master.

Coverage remained the same at 100.0% when pulling df5304aa67b74e093f6cf1cc089420e172d628df on run-profiler into c92d26864444b5af55042bbc9d046d3f85b0571c on master.


node-version: 'latest' taking 20+ minuites

Description:
Node setup taking forever (20m+) when using latest
Exmaple:

Code:
            -   name: Install Node.js
                uses: actions/setup-node@v3
                with:
                    node-version: 'latest'

Is the latest version not caches somewhere?
Action version:
actions/setup-node@v3
Platform:

[ ] Ubuntu
[x] macOS
[ ] Windows

Runner type:

[x] Hosted
[ ] Self-hosted

Example of this failing.

I'm getting it too. I'm use latest as the node version and the Log is:
❯ gh run view<PHONE_NUMBER> --log-failed
test    Node setup      2023-10-18T08:48:29.8829257Z ##[group]Run actions/setup-node@v3
test    Node setup      2023-10-18T08:48:29.8829747Z with:
test    Node setup      2023-10-18T08:48:29.8830054Z   node-version: latest
test    Node setup      2023-10-18T08:48:29.8830412Z   always-auth: false
test    Node setup      2023-10-18T08:48:29.8830776Z   check-latest: false
test    Node setup      2023-10-18T08:48:29.8831318Z   token: ***
test    Node setup      2023-10-18T08:48:29.8831613Z ##[endgroup]
test    Node setup      2023-10-18T08:48:30.3387908Z getting latest node version...
test    Node setup      2023-10-18T08:48:30.3411536Z getting latest node version...
test    Node setup      2023-10-18T08:48:30.3413660Z Attempting to download v21.0.0...
test    Node setup      2023-10-18T08:48:30.6240758Z Not found in manifest. Falling back to download directly from Node
test    Node setup      2023-10-18T08:48:30.6557915Z Acquiring 21.0.0 - x64 from https://nodejs.org/dist/v21.0.0/node-v21.0.0-linux-x64.tar.gz
test    Node setup      2023-10-18T08:53:10.8209480Z aborted
test    Node setup      2023-10-18T08:53:10.8209977Z Waiting 14 seconds before trying again
test    Node setup      2023-10-18T09:02:35.6053574Z aborted
test    Node setup      2023-10-18T09:02:35.6054014Z Waiting 16 seconds before trying again
test    Node setup      2023-10-18T09:05:51.9376210Z ##[error]aborted

The same problem occurs when version 19.8.1 is specified.
At least as of two days ago I had the same problem.
    runs-on: ubuntu-latest
    steps:
      - name: Set up Node
        uses: actions/setup-node@v3
        with:
          node-version: 19.8.1

I can get the tar.gz file correctly with curl, so I wonder if there is a problem with the cache handling in the @actions/tool-cache library?

Same issue here using
- name: Set up Node.js
        uses: actions/setup-node@v2
        with:
          node-version: '17.6.0'

Is there a chance this is related to last week's temp download dir change by any chance? https://github.com/actions/setup-node/commit/2a017f350dbf6c4b6bb4508cc83809719115162e

Same with node-version: '19'

We are also seeing this issue. Anytime we try and install node v18, if Node's downloads are slow that day it always fails. We didn't have this issue on earlier node versions.

Hello @andyslack
Thank you for submitting this issue. We will investigate and come back to you as soon as possible 🚀

@marko-zivic-93 let me know if there is anything I can do to help!

Same problem here too.
jobs:
  test_frontend:
    runs-on: ubuntu-20.04

    steps:
    - uses: actions/checkout@v3

    - name: Use Node.js
      uses: actions/setup-node@v3
      with:
        node-version: 19.x

    - name: Build
      run: |
        yarn install

    - name: Test
      run: |
        npm test

I'm getting the same issue with 19.1 since yesterday

Same with node 21. The install or post-install steps fail almost every run

Builds are failing for us as well. Downloads taking upwards of 7 hours.

This is happening for us as well, using latest.

Had the same problem with v3, but it only happens intermittently. Tried v4 out, same problem.

I found some information that might help others. If you can upgrade your version of node to an LTS version found in the manifest , I believe the builds will use the cached version from actions/node-versions and skip the download step. This link has more information, specifically this part:
"current/latest/node always resolve to the latest dist version. That version is then downloaded from actions/node-versions if possible, or directly from Node.js if not. Since it will not be cached always, there is possibility of hitting rate limit when downloading from dist"

@bonniegrubbs I think you are correct here.
Might be a good idea to add options to the manifest for latest-cached and also root versions e.g. 18 19 rather than having to specify 18.18.2
Just an idea to make it more user friendly.

Seeing this, too, with pinned versions of node. Sometimes the download fails and the ci failed. Is there some way to add more verbosity to try and see whats breaking down? Also, would it make sense to add a retry mechanism? Things generally work after re-running the workflow, so if we can just automate that it would allow ci to pass.

What I'm seeing with setup-node@v4 is whether I use 'latest' or '>=21.2', for example, the "best" I get is v18.19.1 while the actual latest is v21.7.0 right now.
    strategy:
      matrix:
        node-version: ['>=21.2']
    steps:
      - name: Checkout
        uses: actions/checkout@v4

      - name: Setup Node ${{ matrix.node }}
        uses: actions/setup-node@v4
        with:
          node-version: ${{ matrix.node }}
          check-latest: true
          cache: npm

Example of this failing.

What I'm seeing with setup-node@v4 is whether I use 'latest' or '>=21.2', for example, the "best" I get is v18.19.1 while the actual latest is v21.7.0 right now.
    strategy:
      matrix:
        node-version: ['>=21.2']
    steps:
      - name: Checkout
        uses: actions/checkout@v4

      - name: Setup Node ${{ matrix.node }}
        uses: actions/setup-node@v4
        with:
          node-version: ${{ matrix.node }}
          check-latest: true
          cache: npm

Builds are failing for us as well. Downloads taking upwards of 7 hours.

Hello @andyslack Thank you for submitting this issue. We will investigate and come back to you as soon as possible 🚀

Hi @andyslack, Apologies for the delayed response. Could you please confirm if the issue is still relevant for you? I have tested from my end and was able to successfully complete the setup node step with in seconds . Please see the attached screenshot for reference.
Thank you!

Hi @gowridurgad, since switching to a specific node version, it works fine; I no longer use latest, so I cannot confirm if it's still an issue.

Hi @andyslack, Thanks for the update, Since you’re no longer using the latest  and there’s no active issue on your end, we are proceeding to close this issue. Please feel free to reopen this issue or create a new one if necessary. Thank you.


Windows 95 Install and Post-Setup Crashes (SUWIN, RUNDLL32, BSOD, Restart freezes)

This issue was reported previously but closed and brushed off with a this isn't a UTM problem response. I didn't know if commenting on a closed issue would result in a fix so I posted here. No harm, no foul.
There is a Qemu patch that hasn't been pulled to work with the latest version of UTM 2.5.0 beta. This bug has been present since 4.0.0+ versions.
This allows Windows 95 to run without additional 3rd party fixes, both during setup and within the OS while changing certain drivers and settings. Installing and using Windows 95 is halted and crashes consistently without this fix. This fix also affects Windows 98 Installs and post setup changes as well.
The fix detailed:
<EMAIL_ADDRESS>Pull
git fetch https://github.com/patchew-project/qemu<EMAIL_ADDRESS>Resolves the following 3 issues:
https://gitlab.com/qemu-project/qemu/-/issues/1759
https://gitlab.com/qemu-project/qemu/-/issues/1964
https://gitlab.com/qemu-project/qemu/-/issues/2012

Seeing the same issue with version 4.x exactly as described in the previously closed issue.

Seeing the same issue with version 4.x exactly as described in the previously closed issue (5760)

I was told if I wanted it fixed I should work on helping to fix the other bugs first. I feel like not being able to install an OS is a major bug. If you can’t install it, you obviously can’t use it. I get it, but it’s been like dozens of releases and this is still there. I supplied links to the fix and it’s incorporated in the latest versions of qemu. Maybe one day but I’ve done all I’m apt to do. Not a programmer

Please this on v4.6.0 https://github.com/utmapp/UTM/releases

Fixed in v4.6


Recruitment candidate user account(s)

Describe the feature request.
One or more user accounts on the Analytical Platform for use by non-MoJ users for the purposes of technical assessments used in recruitment.
The requirements would include:

The ability to run VSCode, Jupyter Lab, and R-Studio as does any typical AP user.
For admins to be readily and frequently able to clear the NAS storage for these accounts (so a fresh image is available to each candidate without any persistence of files between interviews). During interview periods this would be daily if not several times per day, depending on how many 'guest' accounts existed for use.
Having read access to only specific named repositories in the MoJ Github organisations (ie. unable to see organisation-level repos, and only be able to see repos specifically created for recruitment exercises - or public repos).
[Maybe] for recruitment managers to have access to these accounts, or otherwise have a means to create these images when designing the technical assessment exercises. In the interests of being user-friendly, it might just be that assessors could log-in and pre-clone a repo for the candidate, but really the candidate could be talked through this process at the start of an assessment too, so this is not an essential requirement.

IAM access to S3 appears to already be adequate for these accounts (ie. there is no access by default), but it would be worth confirming information security at some stage.
Describe the context.
During recruitment for Data Science and Data Engineering roles there is a need to administer technical tests for assessing candidates. This typically means providing access to computing resources to test the candidates' skills are appropriate for the role.
Currently we use free online computing resources such as Google Colaboratory or Binder, but this presents several difficulties:

As free services these are unreliable and may subject users to timeouts mid-assessment (Binder in particular does so after 10 minutes of inactivity).
Google Colab requires a Google account to use, which some applicants may not have, or not wish to use in these circumstances. Requiring them to sign up to a Google account to apply also means they must agree to Google's terms of service, which could be considered an unreasonable requirement to be considered for employment.
Any data used in the assessment must be made public (eg. via Github or Google Drive). Synthetically generated data is sometimes preferable to open source data as we can be sure candidates have not used it before and can have a full understanding of its content (the patterns and relationships within it).
We cannot curate the environment and ensure relevant tools are installed in advance, so candidates lose time waiting for their chosen software to install, or worse, since they might be unfamiliar platforms, find themselves unexpectedly unable to use their preferred tools during the assessment.
MoJ recruitment assessments are temporarily public and/or being exposed to third-party organisations

Value / Purpose
In summary:

Provides a fairer and more stable environment for recruitment
Enables higher-quality recruitment exercises to be set
Resolves some information security concerns

Guest accounts would effectively resolve all the concerns in 'context' above, and would also provide an environment that accurately represents the working environment of a data professional in the MoJ, so candidates would be better informed regarding the nature of role they have applied for.
User Types
MoJ job applicants, Recruitment managers

We'd also want to create a separate terms-of-use agreement for these accounts.

Request forum: Ask understood, will take to Refinement


Does this repo has the "this is a fake app " problem?

Does this repo has the "this is a fake app " problem?

not found


Remove polyfill(.io)

Which if any of the things we're polyfilling with polyfill.io do we still need based on our current browser support? Relates to #10696

What % of our users are on IE 11?
Could @taceybadgerbrook find out?

In the last month, less than 0.01% of users were using Internet Explorer.


Use standard deviation for conf mat cell background color

The background currently encodes the number of confused instances at epoch x. So you mean you want to replace this encoding with the standard deviation of the distribution of misclassifications over multiple epochs?

If I remember correctly we wanted to visualize the standard deviation of multiple runs with the same architecture, hyperparameters and dataset.

Obsolete, since we decided for the heatmap visualization in the conf matrix and the single epoch is encoded by an indicator.

It's not obsolete. If you want to aggregate repeated runs of the same architecture/parameter config you don't want to show each run as one row in the heat map.
Or we have to discuss this at least.

We could add a 'diff to reference' checkbox that encodes the difference to a reference run in the heatmap or linechart. In Ordino this is handled my treating the first selection as reference - and somehow mark it to make this clear to the user.

Or we can call the checkbox 'Difference to first run"

I misinterpreted the ticket. I thought its about encoding the difference. However, I think the solution might be able to address both. We could add a second checkbox that shows the diff to the average in the cell heatmaps. @gfrogat, would that solve the use cases you had in mind?


Fixing build issues

Temporary stabilizing the product build

Re-enabled commented test cases with https://github.com/wso2/product-is/pull/835, https://github.com/wso2/product-is/pull/836 and https://github.com/wso2/product-is/pull/837


Windows Insider Build

Yesterday, I successfully installed WSLg (Ubuntu 20.04 Distro) by upgrading to the Windows Insider Build program.
Basic GUI applications worked.
But today morning, after I booted my laptop, within 5 minutes, I encountered the GSOD (Green Screen of Death).
Stopcode was HYPERVISOR_ERROR, which I found out after some small research is related to virtualization.
The issue was that percentage of error information being collected by Windows never increased from 0%.
I did a hard reset and very fortunately, I was able to boot up. I reverted back to my previous windows version.
WSLg could have had a lethal impact on the system. Windows Insider Build is pretty much unstable currently.
I'm just sharing my experience and I don't think we're just there yet with respect to WSLg. I'm sure others might have had more pleasant experiences than me.
Anyone having good fixes for this is welcome to continue this thread.

Images for refernce.

@skandaprasad, please submit report using Feedback hub, thanks!

Thanks for your report. As Hideyuki suggest this issue should be tracked through the Windows Feedback Hub as it isn't directly related to WSLg. Resolving this thread.


Add goleak tests to cmd/collector/app/handler

Which problem is this PR solving?

Part of #5006

Description of the changes

Added TestMain to package folders
Added make goleak into unit test ci

How was this change tested?

please run make goleak

Checklist

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

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

Thanks!


Unused characters in ^money baltop

There is a star behind every name and infront of every coincount

Aware of this, don't know quite why. Investigating

Maybe a formatting error


Implements ContentPicker localization options discussed in #7352

As per discussion in #7352, in this commit I created a new feature of the Orchard.ContentPicker module to implement the localization options:

Try to translate the items selected in the ContentPickerField to the culture of the ContentItem we are saving.

Remove localizable selected items that do not have a localization to the "correct" culture.
Remove selected items that do not have a LocalizationPart.

Check whether all selected items have the "correct" culture, and prevent saving if they don't.

Consider items without LocalizationPart as being of the "wrong" culture.

We'll get a demo on tuesday.

@sebastienros
I pushed the changes to the view from your comments.
The data-controllerid attribute is nice to know.
Without the inline styles, the "sub-option" checkboxes are on the same level as the option they depend on. I think it's clearer to have the "sub-option" indented. I can insert   to get basically the same, but it feels kludgey to me.
We'll give this a look when we demo it next Tuesday.


Create ISSUE_TEMPLATE.md

Add issue template to make dealing with issues easier by having users supply relevant information upfront.

Some reference here if you are not familiar with Github's templates:
https://github.com/blog/2111-issue-and-pull-request-templates

Thanks @bellwood and sorry for the delay. This is great! It'll be really helpful.


How can I specify more than one codec when I invoke NewTrackLocalStaticRTP

Hi @ChenMoGe2
That isn't possible today. What are you trying to implement?
We could modify it to make your use case work, or maybe it is possible to implement your idea another way.

Hi @ChenMoGe2
That isn't possible today. What are you trying to implement?
We could modify it to make your use case work, or maybe it is possible to implement your idea another way.

Thanks for you reply.I want do this because the sender sends video codec either h264 or vp8.
Maybe I already resovle it.I create two video track to distinguish different video codec

Hi @ChenMoGe2
That isn't possible today. What are you trying to implement?
We could modify it to make your use case work, or maybe it is possible to implement your idea another way.

Hello,I made an SFU server and use different client to connect it.One client sends video using H264 and the other sends video using VP8.

Hi @ChenMoGe2
Sorry I never got around to responding to this. This is possible with pion/webrtc@v3 now! TrackLocal is an interface so as long as you satisfy it you can send w/e codec you want.
Here is an example of the pattern you are requesting in Broadcast box trackMultiCodec allows the caller to switch between H264/VP8/VP9/AV1 at anytime.
I think it would make sense to add this pattern to Pion itself! Make TrackLocalDynamicRTP and TrackLocalDynamicSample it would require some thought around the API design/error handling though!


Exception in Saga when using PostgreSQL as repository

Is this a bug report?
Yes
Can you also reproduce the problem with the lastest version?
Yes (4.0.1)
Environment

Operating system: Window 7
Visual Studio version:  2017
Dotnet version: Core 2.0

Steps to Reproduce

Use Postgres as Saga repository.
Try to send second second event to saga.

Expected Behavior
No exceptions. Normal work with Saga.
Actual Behavior
MassTransit.EntityFrameworkCoreIntegration.Saga.EntityFrameworkSagaRepository<Ma
ssTransitTest.Saga.BusinessTypes.ToAruBicchi> Error: 0 : SAGA:MassTransitTest.Sa
ga.BusinessTypes.ToAruBicchi Exception MassTransitTest.Saga.Events.Contracts.IGo
odNightHappen, Npgsql.PostgresException (0x80004005): 42601: syntax error at or
near "["
   at Npgsql.NpgsqlConnector.<DoReadMessage>d__155.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNot
ification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at System.Runtime.CompilerServices.ValueTaskAwaiter`1.GetResult()
   at Npgsql.NpgsqlConnector.<ReadMessage>d__154.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Npgsql.NpgsqlConnector.<ReadMessage>d__154.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNot
ification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at System.Runtime.CompilerServices.ValueTaskAwaiter`1.GetResult()
   at Npgsql.NpgsqlConnector.<ReadExpecting>d__161`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNot
ification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at System.Runtime.CompilerServices.ValueTaskAwaiter`1.GetResult()
   at Npgsql.NpgsqlDataReader.<NextResult>d__32.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNot
ification(Task task)
   at Npgsql.NpgsqlDataReader.<<NextResultAsync>b__31_0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNot
ification(Task task)
   at Npgsql.NpgsqlCommand.<Execute>d__71.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNot
ification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at System.Runtime.CompilerServices.ValueTaskAwaiter`1.GetResult()
   at Npgsql.NpgsqlCommand.<ExecuteNonQuery>d__84.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNot
ification(Task task)
   at Npgsql.NpgsqlCommand.<>c__DisplayClass83_0.<<ExecuteNonQueryAsync>b__0>d.M
oveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNot
ification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at Microsoft.EntityFrameworkCore.Storage.Internal.RelationalCommand.<ExecuteA
sync>d__17.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNot
ification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at Microsoft.EntityFrameworkCore.RelationalDatabaseFacadeExtensions.<ExecuteS
qlCommandAsync>d__13.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNot
ification(Task task)
   at MassTransit.EntityFrameworkCoreIntegration.Saga.EntityFrameworkSagaReposit
ory`1.<SendQuery>d__9`1.MoveNext()

More info
The thing is happen here, in place, where SQL Query composed by hand... And usabe only in Microsoft SQL Server.
https://github.com/MassTransit/MassTransit/blob/fbe2af4be707ada2906e57ab2213e99fec0e752a/src/Persistence/MassTransit.EntityFrameworkCoreIntegration/DbContextExtensions.cs#L31
The interesting thing is that the first time of sending event to saga — it completes successefully. All other fails with exception as show above.
Exception coming from this function:
https://github.com/MassTransit/MassTransit/blob/fbe2af4be707ada2906e57ab2213e99fec0e752a/src/Persistence/MassTransit.EntityFrameworkCoreIntegration/Saga/EntityFrameworkSagaRepository.cs#L195
And exception thrown by the wrong syntax, when sending query composed here:
https://github.com/MassTransit/MassTransit/blob/fbe2af4be707ada2906e57ab2213e99fec0e752a/src/Persistence/MassTransit.EntityFrameworkCoreIntegration/Saga/EntityFrameworkSagaRepository.cs#L225
So, if I turn optimistic on — exception will be not throwed.

I ran into this as well, and I don't understand why it was written this way. So I turned on optimistic, but that isn't an ideal solution.

Current implementation was copied from classic ef 6 implementation and it works only with mssql.
To my understanding a possible solution would getting ISqlGenerationHelper from dbcontext and using it to compose proper table name with schema.

This was added to the EF6 implementation to solve for a bug where the schema name was not the default one (dbo). The EF6  implementation was never meant to support anything other than mssql. If I remember correctly I think it should be solvable with some refactoring.

@stromblom yep. EF6 was added and really localdb (i.e. mssql), was the target. All integration tests were targeted on localdb.
And you are right, this could definitely be refactored to allow the schema + table name retrieval to have custom implementation if you are using a different relational db.
I could probably take a crack at this this weekend.

@faserg1 have you by chance been able to test with the develop packages to see if this resolved the issue for you?

I'm going to close this due to lack of follow-up, a change was made, hope it addresses the issue.

It seems relate to difference handling concurrency in SQLServer and postgreSQL. Maybe this link would be helpful


Web page phishing detection added in machine_learning

Pull Request for PyVerse 💡
Requesting to submit a pull request to the PyVerse repository.

Issue Title
Please enter the title of the issue related to your pull request.
Web page phishing detection

[x] I have provided the issue title.

Info about the Related Issue
What's the goal of the project?
Describe the aim of the project.
Many people get scammed by this Web page phishing technique. Detecting them can save people from getting scammed. Hackers usually blackmail the people to get their personal information back. Identifying this techniques can save millions.

[x] I have described the aim of the project.

Name
Please mention your name.
Deepanshu Baghel.

[x] I have provided my name.

GitHub ID
Please mention your GitHub ID.
@deepanshubaghel

[x] I have provided my GitHub ID.

Email ID
Please mention your email ID for further communication.
Enter your email ID here.

[x] I have provided my email ID.

Identify Yourself
Mention in which program you are contributing (e.g., WoB, GSSOC, SSOC, SWOC).
GSSOC-EXT

[x] I have mentioned my participant role.

Closes
Enter the issue number that will be closed through this PR.
Closes: #484

[x] I have provided the issue number.

Describe the Add-ons or Changes You've Made
Give a clear description of what you have added or modified.
Web page phishing detection added in machine_learning

[x] I have described my changes.

Type of Change
Select the type of change:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Describe how your changes have been tested.
Describe your testing process here.

[ ] I have described my testing process.

Checklist
Please confirm the following:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

@UTSAVS26


Code coverage not available for the repository after going public.

We need to look into why the codecov coverage is not available for the repository and PRs.
At first glance it looks like it does not even try and upload data.

It looks like coverage is only included in one test line in package.json in the front-end package group. I see the codecov config file in the root of the repository. However, it doesn't look to me like automation or back-end is calling out to enable coverage or to retrieve artifacts.

@rbarkerSL I was troubleshooting this issue a bit and tried a few things but when I try and access https://app.codecov.io/github/hashgraph/hedera-transaction-tool it says that the repo is disabled. Is this something edited through the codecov account or how do we re-enable it?

@rbarkerSL how could I get access to the codecov account so I can check whats going on in there?


Wrong output in training_curve.py

Issue description
I’m testing the training_curve.py script and found an odd behavior. The script output a plot for each metrics from tensorboard event files. However, some plots don’t have the “right” content in them.
Current behavior
For example, on rosenberg (master branch commit 93b9e73):

hausdorff_score.png contains the tensorboard curve of specificity
intersection_over_union.png contains the tensorboard curve of precision
precision_score.png contains the tensorboard curve of hausdorff
specificity_score.png contains the tensorboard curve of intersection_over_union

I also tried on my laptop and got different “bad” pairings (master branch commit 93b9e73):

accuracy_score.png contains the tensorboard curve of specificity
dice_score.png contains the tensorboard curve of accuracy
intersection_over_union.png contains the tensorboard curve of recall
recall_score.png contains the tensorboard curve of dice
precision_score.png contains the tensorboard curve of intersection_over_union
specificity_score.png contains the tensorboard curve of precision

Steps to reproduce

I’m using this command to plot curves from 2 models:
ivadomed_training_curve -i path_output_model1,path_output_model2 -o path_output_for_png
Then I compre the png outputs with the tensorboard display.

Note: I have the same issue when using only one model path in -i.

Mmmm interesting :-(
As it is related to metrics only, the issue probably comes from somewhere in this loop ?

As it is related to metrics only, the issue probably comes from somewhere in this loop ?

@charleygros, I tested it a bit further, the error is not in the loop directly but in the order of the events_df_list used to loop, this list comes from tensorboard_retrieve_event.

In tensorboard_retrieve_event, the code here expects the metrics to be in a specific (hardcoded) order: dice, multiclass dice, hausdorff, precision, recall, specificity, intersection_over_union, accuracy.
The order above is supposed to correspond to the order that they are written in training, buy it doesn't match. Order in training here is: dice, multi_class_dice, precision, recall, specificity, intersection_over_union, accuracy, hausdorff.
Moreover, regardless of the order in training, it seems that the events files are read in the order of os.listdir here. (funny story: it is now iterdir with Pathlib, PR #886 was merged during my debugging).
The os.listdir output is not always in the same order which explains the difference in behavior that I had between my laptop and rosenberg. For each machine, the mismatch between metrics corresponds to the difference between the expected hardcoded list vs. the order of os.listdir (and had nothing to do with the order written in training).

I'll re-test it now with Pathlib and try to fix the issue.


Update cats-effect, cats-effect-laws to 3.3.5

Updates

org.typelevel:cats-effect
org.typelevel:cats-effect-laws

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

Applied Migrations

github:typelevel/cats-effect/v3_0_0?sha=v3.0.0
github:typelevel/cats-effect/v3_3_0?sha=series/3.x

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

labels: library-update, early-semver-major, semver-spec-major, scalafix-migrations

Superseded by #231.


Kick from channel feature

Is there any possibility to kick a user from a channel but allow him to rejoin? Or maybe is there any list of blocked users related to the channel?

The currently supported feature is block_from_channel that kicks the user and permanently disallows the user from accessing the channel. (There is no way to even unblock the user.)
afaik there is no feature to kick the user and still allow the user to re-join the channel.
https://github.com/stypr/clubhouse-py/blob/v304/clubhouse.py#L577-L587


fix: stream result

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[ ] Y

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[ ] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

-44 of 71 (38.03%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 62.923%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/kcl/api.go
27
71
38.03%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
3143

Relevant Lines:
4995

💛  - Coveralls


Update hpke and hpke-pq to 0.11.0

This version of hpke moves x25519-dalek to a stable version, letting downstream users get the fix for https://rustsec.org/advisories/RUSTSEC-2024-0344.
The pinned version of hpke-pq also fixes the CI.

@martinthomson are you able to cut a release containing this? Absolutely no urgency for that, I have no problems pulling from git if you don't have the bandwidth for it.


Deprecate useDeleteButtonTooltip for Chips

This PR deprecates useDeleteButtonTooltip, for Chip widgets that support having a delete button. This includes the Chip, the InputChip and the  RawChip widget.
Instead, to disable the tooltips on the delete button of this chips, users should pass an empty string to deleteButtonTooltipMessage.
A dart fix is provided.
The migration code is shown below:
Before:
Chip(useDeleteButtonTooltip: false);
InputChip(useDeleteButtonTooltip: true);
RawChip rawChip = RawChip();
rawChip.useDeleteButtonTooltip;

After:
Chip(deleteButtonTooltipMessage: ‘’);
InputChip();
RawChip rawChip = RawChip();
rawChip.deleteButtonTooltipMessage;

The design doc for this change can be found here.
This is not a breaking change as it is only deprecating a property.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@gspencergoog The PR is (mostly) ready for review, I just have a few concerns:

Since dev channel is deprecated, what version should I provide for the deprecation notices?
I updated some comments and a test for delete button tooltip, which I committed here https://github.com/RoyARG02/flutter/commit/bf86261edd101d67a6bd09f2cb9c5a335da117c2. If it looks good to you, I'll submit a separate PR.

Hey @Piinks, I applied the fixes on a test project, and I'm getting the following behavior:
RawChip(
    label: const Text('No tooltip for this delete button!'),
    onDeleted: (){},
    useDeleteButtonTooltip: false,
)

Becomes
RawChip(
    label: const Text('No tooltip for this delete button!'),
    onDeleted: (){}, deleteButtonTooltipMessage: '',
)

Is there something I need to add so that the parameter is added to its own line?

This will need a second review, I've reached out on discord to find someone. :)

Looks like there are some conflicts to resolve before it can land, unfortunately.


Add a method to check if we have a signed in user

Supersede loicteixeira/gj-unity-api#50
Having to do GameJolt.API.Manager.Instance.CurrentUser != null seems to confuse people.
In addition, a user object could being present does not guarantee that he is signed in.

fixed in da9bc9e2ef77126ef75e09aeecbeb9f080e47ef5


Consider using zict instead of HistoryDict.

We should consider using c in this zict example instead of HistoryDict as the default RE.md. It backs each entry in the dict with a separate file, thus avoiding the sqlite locking issue that prevents multiple RE instances from referring to the same md.

If we do this, also update the recommended way to do this in docs.


Account mismatch after some time

Running the trading bot for 20 minutes or more leads to this:
trading_bot    | ERROR:root:Failed tx execution: failed to simulate tx : <AioRpcError of RPC that terminated with:
trading_bot    |        status = StatusCode.UNKNOWN
trading_bot    |        details = "account sequence mismatch, expected 73, got 61: incorrect account sequence"
trading_bot    |        debug_error_string = "{"created":"@1657809272.661877125","description":"Error received from peer ipv4:<IP_ADDRESS>:9090","file":"src/core/lib/surface/call.cc","file_line":966,"grpc_message":"account sequence mismatch, expected 73, got 61: incorrect account sequence","grpc_status":2}"

This error seems to be repeating and i'm sending one transaction per account every 10 seconds.

Can't be reproduced


init spatial audio

[x] fetching and caching of audio files
[ ] audio resource loader
[x] basic spatial audio
[x] triggering of audio from game thread
[ ] inform main thread of all player EIDs from game thread
[ ] connect peer voip datachannels to respective player entity panner nodes
[ ] api for syncing audio playback between peers

Seems to be working well 👍 Merging so we can fix / test networking bugs.


Please check regency and district

i have imported all data from mysql file,
there are some district have no regency
check for ID 9419 until 9471, what is the regency name..?

Hi @ndro-agus
I've checked on CSV and MySQL dump:

All regencies have name (CSV regencies.csv line 501-514 and MySQL dump line 637-650).
All districts have its own regency (CSV districts.csv line 6956-7093 and MySQL dump line 7754-7893).

Did you see any errors while you were importing the dump?

no, there is no error, but some data is missing,
there are some district have regency id, but the id was not exists in regency table.

I can't reproduce the issue and the dump looks good to me.
Could you please get a list of districts that have a non-existing regency to here and the query to get the list?
If it is possible, could you try to reimport the database?

well, I have fetched data for the rest regency from Central Agency on Statistics (BPS) - MFD and MBS Update (http://mfdonline.bps.go.id/).
regency ID from 9419 to 9471,
Check DISTINCT regency ID from district, and compare with DISTINCT regency ID from regency.
you will see the differences.

Most probably, there is an issue while importing MySQL dump line 637-650, which contains the missing regency's name. Since it is not reproducible, I will close this issue. Feel free to re-open this issue if anyone faces missing regency's name issue again.


Dart VM crash in Linux web_tests

https://ci.chromium.org/ui/p/flutter/builders/try/Linux web_tests_0/2628/overview
In packages/flutter From command:
../../bin/flutter test -v --platform=chrome --web-renderer=html --sound-null-safety test/gestures/multitap_test.dart test/widgets/reparent_state_harder_test.dart test/painting/shape_decoration_test.dart test/services/keyboard_key_test.dart test/widgets/list_view_relayout_test.dart test/widgets/flow_test.dart test/rendering/sliver_cache_test.dart test/material/outlined_button_theme_test.dart test/foundation/change_notifier_test.dart test/material/search_test.dart test/services/platform_views_test.dart test/widgets/app_navigator_key_test.dart test/cupertino/debug_test.dart test/widgets/image_headers_test.dart test/widgets/scroll_position_test.dart test/gestures/recognizer_test.dart test/material/popup_menu_theme_test.dart test/material/icon_button_test.dart test/foundation/debug_test.dart test/widgets/draggable_test.dart test/widgets/interactive_viewer_test.dart test/widgets/selectable_text_test.dart test/cupertino/segmented_control_test.dart test/material/chip_theme_test.dart test/widgets/scrollable_test.dart test/material/scrollbar_paint_test.dart test/scheduler/benchmarks_test.dart test/services/lifecycle_test.dart test/widgets/semantics_test.dart test/rendering/debug_overflow_indicator_test.dart test/painting/image_resolution_test.dart test/rendering/relative_rect_test.dart test/widgets/state_setting_in_scrollables_test.dart test/material/icons_test.dart test/widgets/overflow_box_test.dart test/widgets/restoration_scopes_moving_test.dart test/widgets/icon_data_test.dart test/rendering/platform_view_test.dart test/animation/animations_test.dart test/widgets/rich_text_test.dart test/widgets/semantics_tester_test.dart test/widgets/animated_padding_test.dart test/material/ink_splash_test.dart test/painting/fractional_offset_test.dart test/rendering/independent_layout_test.dart test/cupertino/context_menu_test.dart test/material/flexible_space_bar_collapse_mode_test.dart test/material/outline_button_test.dart test/widgets/invert_colors_test.dart test/widgets/range_maintaining_scroll_physics_test.dart test/rendering/viewport_caching_test.dart test/rendering/reattach_test.dart test/widgets/scroll_physics_test.dart test/widgets/fade_transition_test.dart test/harness/trivial_test.dart test/gestures/long_press_test.dart test/material/typography_test.dart test/material/about_test.dart test/material/bottom_navigation_bar_theme_test.dart test/material/navigation_rail_theme_test.dart test/material/checkbox_theme_test.dart test/widgets/composited_transform_test.dart test/widgets/animated_container_test.dart test/material/flat_button_test.dart test/widgets/editable_text_show_on_screen_test.dart test/widgets/icon_theme_data_test.dart test/material/theme_test.dart test/widgets/default_text_height_behavior_test.dart test/material/paginated_data_table_test.dart test/foundation/consolidate_response_test.dart test/widgets/scrollable_semantics_traversal_order_test.dart test/foundation/observer_list_test.dart test/material/range_slider_test.dart test/rendering/view_chrome_style_test.dart test/gestures/team_test.dart test/widgets/semantics_debugger_test.dart test/painting/border_radius_test.dart test/gestures/events_test.dart test/material/dialog_test.dart test/widgets/navigator_restoration_test.dart test/material/bottom_app_bar_test.dart test/rendering/offstage_test.dart test/widgets/scrollable_restoration_test.dart test/material/data_table_test.dart test/widgets/set_state_5_test.dart test/widgets/title_test.dart

===== CRASH =====
si_signo=Segmentation fault(11), si_code=1, si_addr=0xa7
version=2.14.0-60.0.dev (dev) (Thu Apr 29 03:44:20 2021 -0700) on "linux_x64"
pid=32222, thread=32502, isolate_group=main(0x373e400), isolate=main(0x373c660)
isolate_instructions=18a0a20, vm_instructions=18a0a20
  pc 0x0000000001a901ef fp 0x00007fe6dd4fe028 /b/s/w/ir/x/w/flutter/bin/cache/dart-sdk/bin/dart+0x18901ef
-- End of DumpStackTrace
[exit     : sp(0) fp(0x7fe6dd4fe1e8) pc(0)]
[stub     : sp(0x7fe6dd4fe1f8) fp(0x7fe6dd4fe228) pc(0x7fe704c8266e)]
[dart     : sp(0x7fe6dd4fe238) fp(0x7fe6dd4fe278) pc(0x7fe6ddc2b17c) dart:io__NativeSocket@14069316_write_<anonymous closure> ]
[dart     : sp(0x7fe6dd4fe288) fp(0x7fe6dd4fe380) pc(0x7fe6ddc38405) *package:stack_trace/src/stack_zone_specification.dart_StackZoneSpecification__run@934444339 ]
[dart     : sp(0x7fe6dd4fe390) fp(0x7fe6dd4fe3c8) pc(0x7fe6ddc05d71) *package:stack_trace/src/stack_zone_specification.dart_StackZoneSpecification__registerCallback@934444339_<anonymous closure> ]
[dart     : sp(0x7fe6dd4fe3d8) fp(0x7fe6dd4fe450) pc(0x7fe6f91a54d9) *dart:async_::__rootRun@4048458 ]
[dart     : sp(0x7fe6dd4fe460) fp(0x7fe6dd4fe4a0) pc(0x7fe6f91a70bb) *dart:async_::__rootRun@4048458__rootRun@4048458 ]
[dart     : sp(0x7fe6dd4fe4b0) fp(0x7fe6dd4fe550) pc(0x7fe6ddc34230) *dart:async__CustomZone@4048458_runGuarded ]
[dart     : sp(0x7fe6dd4fe560) fp(0x7fe6dd4fe588) pc(0x7fe6f91a8076) *dart:async__CustomZone@4048458_bindCallbackGuarded_<anonymous closure> ]
[dart     : sp(0x7fe6dd4fe598) fp(0x7fe6dd4fe690) pc(0x7fe6ddc38405) *package:stack_trace/src/stack_zone_specification.dart_StackZoneSpecification__run@934444339 ]
[dart     : sp(0x7fe6dd4fe6a0) fp(0x7fe6dd4fe6d8) pc(0x7fe6ddc05d71) *package:stack_trace/src/stack_zone_specification.dart_StackZoneSpecification__registerCallback@934444339_<anonymous closure> ]
[dart     : sp(0x7fe6dd4fe6e8) fp(0x7fe6dd4fe760) pc(0x7fe6f91a547c) *dart:async_::__rootRun@4048458 ]
[dart     : sp(0x7fe6dd4fe770) fp(0x7fe6dd4fe7b0) pc(0x7fe6f91a70bb) *dart:async_::__rootRun@4048458__rootRun@4048458 ]
[dart     : sp(0x7fe6dd4fe7c0) fp(0x7fe6dd4fe860) pc(0x7fe6ddc34230) *dart:async__CustomZone@4048458_runGuarded ]
[dart     : sp(0x7fe6dd4fe870) fp(0x7fe6dd4fe898) pc(0x7fe6f91a8076) *dart:async__CustomZone@4048458_bindCallbackGuarded_<anonymous closure> ]
[dart     : sp(0x7fe6dd4fe8a8) fp(0x7fe6dd4fe900) pc(0x7fe6f91b5753) *dart:async_::__startMicrotaskLoop@4048458 ]
[dart     : sp(0x7fe6dd4fe910) fp(0x7fe6dd4fe928) pc(0x7fe6f91b05f3) *dart:async_::__startMicrotaskLoop@4048458__startMicrotaskLoop@4048458 ]
[dart     : sp(0x7fe6dd4fe938) fp(0x7fe6dd4fe960) pc(0x7fe6f91ad6df) *dart:isolate__RawReceivePortImpl@1026248__handleMessage@1026248 ]
[entry    : sp(0x7fe6dd4fe970) fp(0x7fe6dd4fe9d8) pc(0x7fe704c825af)]
/b/s/w/ir/x/w/flutter/bin/internal/shared.sh: line 230: 32222 Aborted                 "$DART" --disable-dart-dev --packages="$FLUTTER_TOOLS_DIR/.packages" $FLUTTER_TOOL_ARGS "$SNAPSHOT_PATH" "$@"

@a-siva @aam Any thoughts on this failure in the command line VM embedder? This just happened once last week.

The stack trace doesn't seem to have much info other than dart:io__NativeSocket@14069316_write_, I presume the crash is in the native method _nativeWrite which maps to Socket_WriteList in socket.cc. Is there a possibility of a race where the socket is closed and we are still trying to write to the socket ?

@a-siva Sorry, just saw this question. IIRC, in the Dart VM, socket file descriptors are wrapped in a C++ object and are reference counted due to getting handed back and forth between the Dart thread and the eventhandler thread. The Dart _NativeSocket object should be holding a reference to the C++ object, which should keep the reference count from dropping to 0, but if there's a mistake/race in reference counting, then it could be a dangling reference.

Similar stack trace on https://ci.chromium.org/ui/p/flutter/builders/try/Linux web_tests_7_last/4024/overview, but with an assert message:
../../third_party/dart/runtime/vm/object.cc: 4411: error: Unable to use class 16827440 which is not loaded yet.
...

/cc @dnfield

Another instance: https://ci.chromium.org/ui/p/flutter/builders/try/Linux SDK Drone/257867/overview. Bumping priority after noticing this is somewhat frequent.

@a-siva won't be able to get to this in the near future, @aam is there  someone else on your team who could take a look?

@Hixie we are one big happy team :-)
The key is to be able to reproduce the failure somehow locally. I tried reproducing it by running the tests non-stop on my dev box, but I could not get the failure.

Seen again here: https://ci.chromium.org/ui/p/flutter/builders/prod/Linux SDK Drone/77200/overview

Siva is aware of this issue and looking into next steps.

Sure, we can lower the priority, and ping this issue if it comes up again.


Download Link for Superfish is Broken

Is there another link that gets you to the download page? The current one is broken. Thanks!

Unfortunately, it is still unavailable. I wonder if they revoked public access.

@congunter Perhaps try to contact them?

Is there another link that gets you to the download page? The current one is broken. Thanks!

Hi there, I know your comment is from a while ago so IDK if this is still relevant, but after 2 days of scouring the internet because I wanted to use Superfish for a university class project, I found this mirrorlink from a NASA spaceflight forum from 2015.  In no way do I officially endorse this link because it came from a random person on the internet, but I don't seem to have any malware yet LOL.  If you wanna roll the dice, this link worked for me
http://hallaweb.jlab.org/12GeV/SoLID/download/field/PoissonSuperfish_7.18.exe

This is noted now in the main readme: https://github.com/ChristopherMayes/PySuperfish?tab=readme-ov-file#pysuperfish


nix: fix build on latest nixpkgs

updated nixpkgs version in the flake lock file.
fixed error: 'pkgconfig' has been renamed to/replaced by 'pkg-config'

Thank you :)


How to prepare coco data in tfrecord format?

Your code does not contain the path for the original coco detection dataset.
And the downloading of tfrecord format coco is wrong:“E tensorflow/tsl/platform/cloud/curl_http_request.cc:610] The transmission  of request 0x9690640 (URI: https://www.googleapis.com/storage/v1/b/pix2seq/o?fields=items%2Fname%2CnextPageToken&prefi
x=multi_task%2Fdata%2Fcoco%2Ftfrecord%2F) has been stuck at 0 of 0 bytes for 61 seconds and will be aborted. CURL timing information: lookup time: 9.4e-05 (No error), connect time: 0 (No error), pre-transfer time: 0 (No er
ror), start-transfer time: 0 (No error)”
How to use your code?

I can download your tfrecord format coco manually, but I don't know how tospecify the path. Thank you!


How to escalate a forum issue and contact the moderator

Hi!
I posted a question in the SQL Server Reporting Services forum several days ago and so far, the only response I've gotten was one that just asked another question, which I answered days ago.  Further, there seems to be no link to contact a forum moderator anywhere on the screen.  According to what I read, the only way to escalate a question is by contacting the moderator (in this case, I think it's Dave Patrick, but I'm not sure).  But if you can't contact the moderator, you're STUCK!  Frankly, I'm getting VERY upset about the issue I've reported.  Export from SSRS to Excel has always been buggy, but this is absurd.
Here's the link to my question . . .
https://social.msdn.microsoft.com/Forums/sqlserver/en-US/9978c722-bb41-413f-ada9-aa70f4bd8bc0/issues-with-excel-export-from-ssrs?forum=sqlreportingservices
I hope you can help.
Thanks.

@klgrube MSDN forums have closed at https://docs.microsoft.com/en-us/teamblog/msdn-technet-english-forums-closure


Use isLoading to disable withdrawal button

Description
A small fix for #939 to prevent the user from being able to tap withdrawal twice triggering a double withdrawal.
Tested
Tested locally on iOS and Android Emulators
How others should test

Start a withdrawal and attempt to tap the button multiple times to trigger a double withdrawal
Start a withdrawal and when prompted for the pin entry click bank and see that withdrawal becomes enabled again.

Related issues

Fixes #939

Backwards compatibility
This is a minor change and should be backward compatible

https://valora.testquality.com requirement R79.

https://valora.testquality.com requirement R80.

Hi @MuckT, We have verified this issue on Latest Android internal build V1.18.0 and updated the status in test quality tool.
Test quality tool link:
Android (11):
https://valora.testquality.com/project/12128/plan/21657/run/149950/result/2469323
Verified on Devices: Google Pixel 4a(11.0)

Hi @MuckT, We have verified this issue on Latest Android Internal build V1.18.0 and updated the status in test quality tool.
Test quality tool link:
Android (10.0):

https://valora.testquality.com/project/12128/plan/21657/run/149950/result/2469312
https://valora.testquality.com/project/12128/plan/21657/run/149950/result/2469317

Verified on Devices: Redmi Note 8 (10.0)
Thanks..!!

Hi @MuckT, We have verified this issue on Latest  Test flight build V1.18.0 and updated the status in test quality tool.
Test quality tool link:

https://valora.testquality.com/project/12128/plan/21657/run/149950/result/2469268
https://valora.testquality.com/project/12128/plan/21657/run/149950/result/2469273

Verified on Devices: iPhone SE (13.6.1)
Thanks..!!


Query that runs in Airpal hangs in SQL Lab

The following query runs in 18 seconds in Airpal but hangs for 1.5 hours+ in SQL Lab:
SELECT a.ds, dau, overall_wau
FROM meta.dim_data_products_cube a
LEFT JOIN (
SELECT ds, dau as overall_wau
FROM meta.dim_data_products_cube
WHERE grouping__id = 0
) b ON a.ds = b.ds
WHERE grouping__id = 4

can't reproduce, query works on both prod and master.
Looks like it was transient error.


fix(management): check visibility on patch update

When performing a patch update on a page to modify its publication state, the request does not define a visibility. In that case, we must keep the existing visibility status of the page and apply it to the related pages (translations and links)
Fixes gravitee-io/issues#5559

Already on master with #1489


Adjusted Dialog Styling

ADO-XXX
Description
List of proposed changes:

Aligned dialog styling with Figma designs: added background blur and fixed height component

What to test for/How to test

Open Quiz Dialog to see changes

Additional Notes

Not associated with a story

FYI, I rebased on main and force-pushed.

Can the associated branch for this be deleted?


Docker compose revamp

CE-429
Currently blocked by: https://northerntech.atlassian.net/browse/MEN-7477
After that the links need to be updated master -> 1.0.1

Sorry to expand the scope here so much, but we should also consider the Mender Hub Update Module section:

https://hub.mender.io/c/update-modules/13?ascending=false&order=views

In particular there is a "competing" Kubernetes implementation that we made: https://hub.mender.io/t/kubernetes/1939
The problem is that it will confuse customers if we provide multiple Update Modules for the same update type.

@kjaskiewiczz FYI, ref the QA discussion.
My opinion is that we need to move the app-update module to https://github.com/mendersoftware/mender-update-modules, add a Mender Hub Update Module tutorial (maybe one for Docker-Compose and another for Kubernetes).
I don't understand why the app-update module was done separate? Was this a more "strategic" decision to split out the update modules we support or was it just done oneoff?

Thanks for the info.

I don't understand why the app-update module was done separate? Was this a more "strategic" decision to split out the update modules we support or was it just done oneoff?

I don't know, but maybe @merlin-northern knows something about it?

Thanks for the info.

I don't understand why the app-update module was done separate? Was this a more "strategic" decision to split out the update modules we support or was it just done oneoff?

I don't know, but maybe @merlin-northern knows something about it?

who me? let's face it my knowledge is a bit on a low level.
so it was a decision that was made, to keep it separate:

app-update-module is really an answer to the Mender for Applications, implemented as an update module, true, but we had in mind the release and it had been decided to release from a separate repo
it goes a bit beyond the update modules as it introduces so-called submodules
the mender-update-modules has or had slightly different idea

I can dig further. if you want to move it to the other repo we can discuss.

@estenberg
Thanks for the review, will go through it.

Have you seen that we already have a tutorial here: https://docs.mender.io/artifact-creation/create-an-artifact/docker-compose?
I think that one is mis-placed, and we should remove it after you add yours to Get started.

I've used that one as the base and describe the high level changes in the commit messages.
The current PR changes are doing the move.

The app-update-module 1.1.0 has been released. The links has been updated already


Governance. Delegations. Address string shortening works wrong on "Add delegation" screen

Description
https://github.com/user-attachments/assets/7629275a-b978-4428-82ae-736b4a699939
Environment
https://github.com/novasamatech/nova-spektr/commit/cb012730236d518d6ac8c00c662b47397ad0135d

Status verification: ✅


Molecule delegated driver is being ignored; unable to determine where to place Zuul variables

Greetings,
I have a working molecule test using "molecule verify" to run some tests with testinfra verifier. I'm trying to get pytest to run my ansible tests (so I can also make use of its reporting features, debugging, etc) using your pytest-molecule. I am using "delegated" driver (but currently have empty create/destroy yml files but want for future use). Running "pytest -m delegated" results in the following error:
Molecule delegated driver was not enabled because missing zuul.build variable in current inventory. We will tag scenarios using it with 'skip' marker.

Sure enough, my molecule tests are skipped:
molecule/default/molecule.yml s                                                                                [100%]

I see in your documentation that you state:

Due to the potential destructive nature of delegated driver, scenarios running it will run only if Ansible detects one of the following variables as defined: zuul, use_for_testing.

I'm trying to figure out where in my molecule.yml I need to include var "zuul: True" or "use_for_testing: True" or "zuul.build: /tmp/zuul" since I am not using an actual inventory file and am relying on molecule's provision section to build me a dynamic inventory file.
I have tried different permutations, but all have the same error that my molecule tests are being skipped because zuul.build is missing.
provisioner:
  name: ansible
  connection_options:
    host_key_checking: False
    ansible_ssh_user: ${vm_user}
    ansible_password: ${vm_pass}

  inventory:
    group_vars:
      all:
        zuul: True
        use_for_testing: True
        zuul.build: /tmp/zuul

Can you provide specific insight as to where exactly the "zuul" and/or "zuul.build" variable needs to be defined (and also what value)?
Current core pip requirements:
ansible==2.7.11
molecule==2.20.1
mock
pyqt5<=5.12
spyder
typed-ast
jinja2==2.10
pyyaml==3.13
pytest-bdd
pytest
testinfra==1.19.0
pytest-molecule==1.1.0

Full molecule.yml:
---
dependency:
  name: galaxy
driver:
  name: delegated
lint:
  name: yamllint
platforms:
  - name: ${vm_ip}
provisioner:
  name: ansible
  connection_options:
    #TO DO: host key checking does not appear to be working
    host_key_checking: False
    ansible_ssh_user: ${vm_user}
    ansible_password: ${vm_pass}

  inventory:
    group_vars:
      all:
        # pytest-molecule:  Due to the potential destructive nature of delegated driver,
        # scenarios running it will run only if Ansible detects one of the following
        # variables as defined: zuul, use_for_testing.
        # SOURCE: https://pypi.org/project/pytest-molecule/
        zuul: True
        use_for_testing: True
        zuul.build: /tmp/zuul

  # ansible.cfg
  #config_options:
  #  defaults:
  # ...

  log: True
  lint:
    name: ansible-lint
verifier:
  name: testinfra
  env:
    PYTHONWARNINGS: "ignore:.*U.*mode is deprecated:DeprecationWarning"
  directory: ../../steps
  lint:
    name: flake8
  options:
    v: 1

Thank you.

Version info reported by "molecule verify":
============================= test session starts ==============================
    platform linux -- Python 3.6.8, pytest-5.1.2, py-1.8.0, pluggy-0.12.0 -- /home/jack/virtualenv/venv_rhel_functional_tests/bin/python3
    rootdir: /home/jack/git/Azure/Tower/Ansible-Role-VMware-Create-VM/Create-VM/tests/functional, inifile: pytest.ini
    plugins: bdd-3.2.1, molecule-1.1.0, testinfra-1.19.0

This is running on Ubuntu 18.04 via WSL on Windows 10 work laptop with latest updates from dist-upgrade.
Linux DW202N2 4.4.0-43-Microsoft #1-Microsoft Wed Dec 31 14:42:53 PST 2014 x86_64 x86_64 x86_64 GNU/Linux

wheel and setuptools upgraded on system. Using virtual env with "pip install pip wheel setuptools --upgrade" also performed, although there was an error during bdist_wheel build when building the venv environment, so possible that something didn't compile for pytest-molecule. I'm using delegated mode because I'm not dealing with docker containers. Not sure why zuul is needed for my setup.

Oddly, if I run the following:
ansible localhost -e ansible_connection=local -o -m shell -a "echo {% if zuul is defined %}True{% else %}False{% endif %}" --extra-vars "zuul=1"

I can get the above to return True:
localhost | CHANGED | rc=0 | (stdout) True

But if I place the following in my provision section, it's ignored and reports the same error:
provisioner:
  name: ansible
  ansible_args:
    - --extra-vars "zuul=1"

Printing the exception gives insight:
Error code 2 returned by: ansible localhost -e ansible_connection=local-o -m shell -a exit {% if zuul is defined or use_for_testing is defined %}0{% else %}0{% endif %}

It appears that the command is failing to put a space between "ansible_connection=local" and "-o" due to missing comma at line 74:
                "ansible_connection=local" "-o",

should be:
                "ansible_connection=local", "-o",

Sadly, this still doesn't fix my issue. Still getting Error code 2 exception which appears to be a connection error from Ansible.
Then the issue goes away.

@jacklh9 I will take care of this, some work around drivers on molecule prevented me from finishing that and make new releases.


Mat/Pat - Review page syncing errors

We noticed that the parental leave start date and parental leave length does not match in the video below. This also caused validation errors because the allowance dates were not within parental leave dates in the review page.
Video - https://bcgov.sharepoint.com/:v:/r/teams/03991/Shared Documents/Maternity and Parental Leave/Test Screenshots %26 Recordings/Review Syncing issues.mov?csf=1&web=1&e=MaBT65
AC

[ ] All fields in the the review page should sync with the fields in the actual form

@ayushdamani @fazil-ey this change is ready to test on TEST env. let me know if you face any other issue while testing this one.

Not able to replicate the error anymore, thanks Bhumin!

Tested a bunch of scenarios changing data in both the review page and the tabs. Did not see any syncing errors


Add possibility to apply issue description without close editor

Currently when you edit JIRA issue you can save your changes and close editor.
Add button to "Apply" changes without close editor.

Don't see the need in this anymore.


feat: line selection highlighting

Description
Adds selection highlight support to Line charts
Related Issue
#374
Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Do you mind adding a story that demonstrates this feature?

retest

@marshallpete the existing story already demonstrates this.  the ChartPopover -> LineChart story has a UNSAFE_highlightBy control.  It can be set to item(default), series, or dimension.  This is the same as what we already have for the ChartPopover -> StackedBarChart story

I think "line highlighting" highlight the line, by drawing a stroke outline, rather than all the points in the line?
I'm thinking of the use case where there are hundreds of points in a condensed space, so that highlighted points would overlap.

Good callout @majornista . With spectrum 2 changes we will be re-evaluating all the hover, focus and select states. I think we can revisit then.


Feature.406

Description
Features, steps, and data files corresponding to the tutorial 406

Maybe, for every feature/step/40x....py instead of naming the initialization of the feature "step_impl_tutorial_203" it could be simply named as "step_impl_tutorial". This can be changed in the future and doesn't affect anything else.


Templetize report name

If Adaptive & pele control files have different report name the next error arises:
We could avoid that by templetazing pele control file
File "/apps/PYTHON/2.7.13/INTEL/lib/python2.7/runpy.py", line 174, in _run_module_as_main
"main", fname, loader, pkg_name)
File "/apps/PYTHON/2.7.13/INTEL/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "/gpfs/home/bsc72/bsc72893/test_mutations/AdaptivePELE/AdaptivePELE/adaptiveSampling.py", line 680, in
main(args.controlFile)
File "/gpfs/home/bsc72/bsc72893/test_mutations/AdaptivePELE/AdaptivePELE/adaptiveSampling.py", line 597, in main
initialStructures = simulationRunner.equilibrate(initialStructures, outputPathConstants, spawningParams.reportFilename, outputPath, resname, topology_file)
File "/home/bsc72/bsc72893/test_mutations/AdaptivePELE/AdaptivePELE/simulation/simulationrunner.py", line 429, in equilibrate
newStructure = self.selectEquilibratedStructure(self.parameters.processors, similarityColumn, resname, trajNames, reportNames, topology=topology)
File "/home/bsc72/bsc72893/test_mutations/AdaptivePELE/AdaptivePELE/simulation/simulationrunner.py", line 576, in selectEquilibratedStructure
report_values = report[:, cols]
IndexError: too many indices for array
~

Note that the error shown here it's not due to a mismatch in the report names but a known bug in the equilibration procedure if some report has only one line. This bug is fixed in the new release that will be soon available.
Nevertheless, this is an interesting feature and I have already included, again in the new version you will be able to find it.


sub layer toggle

I can't seem to get the sub layer toggle functionality to work. I'm fairly new to the API, but have understood the configuration thus far. Do I have to add my own custom layers to achieve this or can I still just use the generated webmap ID? Cheers

@berg8228  There isn't anything special you have to do. If your web map has layers that has sub layers the layer list will automatically display them. Here's an example.
http://www.arcgis.com/apps/MapTools/index.html?webmap=7a560116489a43e0ad699c3d35f1f5e1

Thanks for the speedy reply. That example is exactly what I'm trying to do. It seems as though I'm not loading the layers properly into the AGOL web map. To get the layer groupings I desire, I've been zipping multiple shapefiles together prior to uploading to the web map from my desktop. The problem is however, I can only toggle of the entire group and not each individual layer in said grouping as occurs in your demo. Do they need to published as feature services? I'm still getting acquainted with AGOL so bear with me...Could you quickly your procedure to adding sub layers to the web map? Thanks so much in advance. Cheers,
Brent

@berg8228 I'll have to test this approach of zipping multiple shapefiles to create a hosted service with sub layers. It sounds like it should work but there could be an issue.


version rework

This is the proposed rework of ConverterVersion that should be useful for all converters as early detection of version incompatibilities avoids many user errors. Version checking for games should be done by converters themselves as done here, as each setup has its own specifics, so eu4tovic2 likely won't bother with minSource and maxSource.
Please note we already have a ConverterVersion in commons that should/could be replaced by this one here when all converters are swapped.
Also, this PR solves f**n kintus, let them burn.

PS. This approach bruteforces rawVersion so no json library is required.


Not Needed

this is not needed because of https://10youtube.com/

If you're going to propose a "good" online YouTube downloader, I guess http://www.clipconverter.cc/ is a better choice. (It's the first item in a Google search.) It can download the 1080P version of https://www.youtube.com/watch?v=lqQg6PlCWgI while https://10youtube.com/ can't. And by the way, youtube-dl handles 500+ other websites than YouTube.


fieldName isn't always used with configuration generation

Given the following generation configuration it is observed that the fieldName is used for overview but not for storeProducts.
headers: &headers
  accept: "application/json, text/plain, */*"
  accept-language: "en-GB,en-US;q=0.9,en;q=0.8"
  appversion: "11.3.1-WEB"
  authorization: XXXX-XXXX-XXXX-XXXX-XXXX-XXXX-XXXX-XXXX-XXXX-XXXX-XXXX-XXXX-XXXX-XXXX
  cache-control: "no-cache"
  compatible_components: "CONVENIENCE_FEE,RAIN_FEE,EXTERNAL_COUPONS,STANDSTILL,BUNDLE,MULTI_SELLER_ENABLED,PIP_V1,ROLLUPS,SCHEDULED_DELIVERY,SAMPLING_ENABLED,ETA_NORMAL_WITH_149_DELIVERY,ETA_NORMAL_WITH_199_DELIVERY,HOMEPAGE_V2,NEW_ETA_BANNER,VERTICAL_FEED_PRODUCT_GRID,AUTOSUGGESTION_PAGE_ENABLED,AUTOSUGGESTION_PIP,AUTOSUGGESTION_AD_PIP,BOTTOM_NAV_FULL_ICON,COUPON_WIDGET_CART_REVAMP,DELIVERY_UPSELLING_WIDGET,ZEPTO_PASS,ZEPTO_PASS:1,ZEPTO_PASS_RENEWAL,NEW_FEE_STRUCTURE,NEW_BILL_INFO,RE_PROMISE_ETA_ORDER_SCREEN_ENABLED,MANUALLY_APPLIED_DELIVERY_FEE_RECEIVABLE,MARKETPLACE_REPLACEMENT,ZEPTO_PASS,ZEPTO_PASS:1,ZEPTO_PASS_RENEWAL"
  deviceid: "7155916704990858"
  platform: "WEB"
  pragma: "no-cache"
  priority: "u=1, i"
  requestid: "8933443057428010"
  sec-ch-ua: "\"Not/A)Brand\";v=\"8\", \"Chromium\";v=\"126\", \"Google Chrome\";v=\"126\""
  sec-ch-ua-mobile: "?0"
  sec-ch-ua-platform: "\"macOS\""
  sec-fetch-dest: "empty"
  sec-fetch-mode: "cors"
  sec-fetch-site: "cross-site"
  sessionid: "2301731813531412"
  storeid: "bc6a4a81-1c35-457e-a732-7ca7198ee2ef"

inputs:
  - curl:
      src: "https://cx.zepto.co.in/api/v1/pass/overview"
      fieldName: "overview"
      headers: *headers
  - curl:
      src: "https://api.zepto.co.in/api/v3/inventory/catalogue/store-products/?store_id=bc6a4a81-1c35-457e-a732-7ca7198ee2ef&subcategory_id=84f270cf-ae95-4d61-a556-b35b563fb947&page_number=2&user_session_id=4d8c9ce6-d3b2-45e3-a22c-d9bf0c99b589"
      fieldName: "storeProducts"
      headers: *headers
preset:
  mergeType: 0.9
  consolidateURL: 0.5
  useBetterNames: true
  treeShake: true
output:
  path: "./generated.graphql"
  format: "graphQL"
schema:
  query: "Query"

NOTE:

Create a authorization token by creating an account on the provided website and then copy it your chrome devtools by looking at the requests.

/bounty $30

/attempt #2543

Should be fixed with the LLM powered inference engine #2200


_write_pem assumes PKCS12

The _write_pem() function is assuming the pkey object is an OpenSSL.crypto.PKCS12 instance.  Obviously, this is a problem if pkey is an OpenSSL.crypto.PKey instance from a PEM file, which lacks the get_privatekey() method.

This was addressed in pypki3.


Improve command line support

Issue: #567 and #700
Requirements

[x]  issue was opened to discuss proposed changes before starting implementation. It is important do discuss changes before implementing them (Why should we add it? How should it work? How should it look? Where will it be? ...).
[x]  during development, node version specified in package.json was used (ie using nvm).
[x]  package versions and package-lock.json were not changed (npm install --no-save).
[x]  app version number was not changed.
[x]  all new code has tests to ensure against regressions.
[x] npm run lint reports no offenses.
[x] npm run test is error-free.
[ ]  README and CHANGELOG were updated accordingly.
[ ]  after PR is approved, all commits in it are squashed

Description of the Change
Fixes a regression bug introduced by #570
No more exception when reaching a long break
Add parsing duration features
Changes the way pause -d works from seconds, to parsing a duration like 1h20m. Examples added, and instructions updated.
Verification Process
For regression bug : let stretchly run until long pause (in progress)
For parsing duration, I added related tests
Sorry for this little bug !

IMO ‘60’ and ‘60m’ should be supported separately—it's just easier to send off a number =)
I can also contribute a bit in the testing department, for the sake of thoroughness. Give me a few hours, and I'll post some proposed changes, with both the parsing and the tests.

@decadent so, do you want me to restrict the PR to the bugfix ?

Nah, I'll just commit some changes in my branch so they can be pulled in the PR. Dunno if I should just make a separate PR later, but the changes are pretty small and quick, so I think it makes sense to jam them in and avoid multiplying entities further.

Sorry for the delay, couldn't complete the changes before the sleepytime yesterday. My proposed changes are in this PR—it's mostly tests, but I also permitted myself to edit descriptions for some options so they flow a bit better. Let me know if that's too much.
I like that style of tests better than just testing each feature once—in the spirit of ‘equivalence partitioning’. Found a couple small potential bugs this way.
BTW, I also learned you can do this:

make some changes, hit a linter error when committing
fix the issue but forget to add the fix to Git's staging cache
commit code with the original mistake, since linter checks the files on disk the second time and doesn't see that the staging cache is different.

LGTM, thanks for those new tests :+1:

Looking good, don't think it's necessary to add things to README/CHANGELOG since changes are quite small.
Will test it for the next few days, squash it and make @decadent as co-author and we're good to go.

BTW, it seems that the commands are only processed when invoked via the second instance, and that includes ‘help’ and ‘version’—which is rather weird for a program.
Also

and make @decadent as co-author

If you mean in the Readme or something permanent like that, then please skip me for now, as my input is quite minuscule so far.

BTW, it seems that the commands are only processed when invoked via the second instance, and that includes ‘help’ and ‘version’—which is rather weird for a program.

Yes, this was the assertion made in the previous discussion. For sure, I can't disagree, a socket listening for events would be better in some UX/UI points (like error return).
If you have the bandwith and the motivation, you can try it since it would be a great upgrades to what we have already done here.
For now, I have all the things I need to control stretchly from my window manager or cli, that's ok-ish for me.

If you mean in the Readme or something permanent like that, then please skip me for now, as my input is quite minuscule so far.

No, just as co-author of the squashed patch commit.

:+1:
this can be merged, then?
For now, I would keep it as it is, via second instance. Don't wanna add another complexity and future will tell us what would be useful, if anything.

Yes, this was the assertion made in the previous discussion. For sure, I can't disagree, a socket listening for events would be better in some UX/UI points (like error return).

I mean, the code can just process the argv when launched as the main instance—if there's no any logical conundrum with the second-instance handling. I'll take a look at doing that—I vaguely remember running into some slightly advanced stuff like ‘how do I exit before creating a Dock icon’, and I must have code for that somewhere.
I guess this can be postponed for later.

LGTM ! :+1:

Just popping in to offer my apologies for dropping the ball on the earlier command-line PR, and to congratulate everyone on making this feature even better than the original plan :)
In case it helps any fellow Arch users, my one tiny contribution is the stretchly-git AUR package, to supplement the existing stretchly-bin option for those of us wanting to track the master branch rather than releases.

way to go! Thanks @lkrms !


feat: switch to minotari containers

Description
Use containers prefixed with minotari_.
Motivation and Context
Update the code in accordance with Issue #225.
How Has This Been Tested?
It has not been tested yet.

Covered in #242


Use includes more intelligently.

Right now, we include all associations in the model controller mixin. Rather, we should allow filters to include what they need to avoid including associations unnecessarily.

Fixed by ac7092437237846b1b34da04b56dd1006e9d4110


What is my iosClientID

Hey there. I am using this module for my android and ios project. I don´t have any issues with my android version, it just worked on the fly. With my iOS version i can´t figure out what to put in my iosClientID. For Android i had to set my webClientID and that was all, now maybe someone has a hint for me what to use for my ios App. My code so far:
		if (Platform.OS === 'android') {
			GoogleSignin.configure({
				scopes: ['email', 'profile', 'https://www.googleapis.com/auth/plus.profile.emails.read', 'https://www.googleapis.com/auth/plus.login'],
				webClientId: Constants.GOOGLE_SIGN_IN_CLIENT_ID,
				offlineAccess: true // if you want to access Google API on behalf of the user FROM YOUR SERVER
			});
		} else {
			GoogleSignin.configure({
				scopes: ['email', 'profile', 'https://www.googleapis.com/auth/plus.profile.emails.read', 'https://www.googleapis.com/auth/plus.login'],
				iosClientId: Constants.GOOGLE_SIGN_IN_CLIENT_ID, // only for iOS
				webClientId: Constants.GOOGLE_SIGN_IN_CLIENT_ID,
				offlineAccess: true // if you want to access Google API on behalf of the user FROM YOUR SERVER
			});
		}

the iosClientID is simply the id you get when you download your plist file
to find it again: open google cloud console. under Api Manager -> Credentials, Section OAuth 2.0 client IDs take the client ID with a type of iOS