Wrong solutions extracted

Describe the bug
Instead of the solutions being inserted I got the following:
#[cfg(test)]
mod tests {
    use super::*;

    use rstest::rstest;

    #[rstest]
    #[case("()", true
)]
    #[case("()[]{}", true
)]
    #[case("(]", false
)]
    #[case("([])", true
)]
    fn case(#[case] s: String, #[case] expected: bool) {
        let actual = Solution::is_valid(s);
        assert_eq!(actual, expected);
    }
}

Leetcode problem
https://leetcode.com/problems/valid-parentheses/description/

To Reproduce
cargo leet gen https://leetcode.com/problems/valid-parentheses/description/

``
Expected behavior
#[cfg(test)]
mod tests {
    use super::*;

    use rstest::rstest;

    #[rstest]
    #[case("()", true)]
    #[case("()[]{}", true)]
    #[case("(]", false)]
    #[case("([])", true)]
    fn case(#[case] s: String, #[case] expected: bool) {
        let actual = Solution::is_valid(s);
        assert_eq!(actual, expected);
    }
}

Seems we need to add another case to the regex

Also happens on https://leetcode.com/problems/valid-anagram

Another https://leetcode.com/problems/flood-fill

Another https://leetcode.com/problems/lowest-common-ancestor-of-a-binary-search-tree


Allow smudge to listen to the zero-address

This PR is up for discussion.
I wanted to allow Smudge to listen to the zero address (<IP_ADDRESS> or [::]) so that it is not required to configure a specific IP address and (maybe) also allow a node to bridge between different IP ranges.
The problem I faced was that a broadcast message defaults gets the listenIP address as the origin address, in case Smudge listens to the zero-address this would not be right.
The way I'm trying to solve this is, when the zero-address is used, the origin address on the broadcast gets updated with the local IP address used to send the broadcast message to another node.
Does this make sense? Is there another approach to solve this?

This is an interesting problem, and a good idea in theory. Certainly some kind of listening IUP customization would/could be useful for different scenarios, such as listening on different network adapters.
I haven't looked at the code yet, so I'll do that now. I may need to think about how to handle the origin address for the zero address though, or even if we want to do that at all? Would it make more sense to specify a particular adapter? I don't know. I'm rolling ideas around at this point.

From a user perspective it is very convienient to not have to configure a specific interface or IP address.
I guess in most cases, where smudge connect to nodes via only 1 source address my committed solution should work fine.
As soon as a smudge node connect to other nodes via multiple source addresses the result seem unpredictable to me.

Maybe it would suffice to create an error in the latter case?
Let smudge pick a local address once and use that troughout the lifetime of the program.

I'm closing this as I do not think this its viable.


Field Service Administrator Role can not Create Work Orders

A user with the Field Service Administrator role is unable to create a Work Order. The Field Service Administrator mentions support for CRUD abilities, but is unable to create a Work Order. What is the proper role a user would need to create a Work Order?

Field Service—Administrator is designed for service managers and IT administrators. This role has access to all field service entities including work orders, scheduling, and inventory. Additionally, this role has full Create, Read, Update, Delete (CRUD) abilities. For example, a field service administrator can create new work order types for the organization, which other people can then use on work orders that they create.

When creating a Work Order I get this error:
Exception Message: User with ID bf3faa1d-bea5-ea11-a812-000d3a33f58e does not have Create permissions for the msdyn_subtotalamount attribute in the msdyn_workorder entity. Count secured attributes in entity 11. User has 0 secured attribute privileges. callerAp=null

ErrorCode: -2147158782
HexErrorCode: 0x8004f502

Activity Id: 77afebd4-1589-4500-b2be-26a1a5c6f968

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

ID: f03acb64-01b5-8e4e-a745-b567afa9b48e
Version Independent ID: 43e5d33d-4ed3-4cfd-16af-86e0df59d354
Content: Set up field service users and security roles
Content Source: ce/field-service/view-user-accounts-security-roles.md
Service: dynamics-365-customerservice
Technology: field-service
GitHub Login: @FieldServiceDave
Microsoft Alias: daclar

The fastest solution I can think of is disabling the field security in all of the Work Order Columns through a solution package. Is there a recommend way to do this or some way I can modify the security role of the Field Service Administrator to get this to work?

Hello Greg @ugreg   - thank you for sharing the feedback and apologies for the delay in getting back to you.
We're looking into this issue and will circle back with an update.
Thanks!
Michael

Hello @ugreg - I checked this in my test environment and Field Service - Administrator seems to be able to create work orders. Since this issue is almost two years old, I will close it. Please reopen or create a new issue if you can reproduce this error again.
Thanks!
Michael


ReadNextAsync and ActivityId

Is there any reason why the ActivityId is not returning from a call to ReadNextAsync?
            FeedIterator<string> it = container.GetItemQueryIterator<string>("SELECT VALUE c.id FROM c");

            FeedResponse<string> result = await it.ReadNextAsync();

            Assert.IsNotNull(result.ActivityId);

Could it be fixed by passing the CorrelatedActivityId in CosmosCrossPArtitionQueryExecutionContext
 CosmosQueryResponseMessageHeaders responseHeaders = new CosmosQueryResponseMessageHeaders(
                continauationToken: continuation,
                disallowContinuationTokenMessage: null,
                resourceType: this.queryContext.ResourceTypeEnum,
                containerRid: this.queryContext.ContainerResourceId)
            {
                ActivityId = this.queryContext.CorrelatedActivityId.ToString()
            };

Why do you want the activity id?
The reason it doesn't return an activity id is there is no way to combine an activity id for aggregate queries. The correlated activity might be useful for client side traces, but it is not sent to Cosmos DB. Which makes it useless for trying to troubleshoot any issues with the Cosmos DB service. The next release will have the Diagnostics which will have all the metrics with all the different responses, and should have the activity id for each response.

@meechy79 can you please respond to the above question?


how to use codellama instead of openai api?

how to use codellama instead of openai api?
how to make it use llama.cpp?

I'm working on a solution to integrate the HuggingFace inference API into the program! I'll consider adding an additional feature for this. This may take some time, but I use other developer agents to speed up the small tasks. There will be a alternate version created by me soon!

It might be more efficient to integrate LiteLLM https://github.com/BerriAI/litellm
It already has access to basically all LLM api's, including Huggingface and Openrouter

It might be more efficient to integrate LiteLLM https://github.com/BerriAI/litellm It already has access to basically all LLM api's, including Huggingface and Openrouter

any example that we can use LiteLLM inside this project


Add Katello 4.4 + Foreman 3.2 test mapping

Katello 4.4 is branching today.

I'm double checking about future 4.1 releases. I'm pretty certain we won't have any more though.

Okay, yeah, no more 4.1 releases are planned.

For future reference, you can submit a PR to just add 4.4 / 3.2 and that can be merged even before branching. An additional mapping doesn't hurt.

I can add something in tool_belt to suggest doing this earlier.

In Foreman we typically have a policy where 2 releases are supported and you can have 1 in RC stages. So now 3.0 and 3.1 are supported while 3.2 will soon be RC1. Once 3.2 goes GA, 3.0 is EOL. There can also be unexpected releases, such as with a security update. That's why Foreman only has instructions to add, not to remove. Removal happens after the GA. I'd recommend doing the same.
Right now we don't have it in our procedures, but you can see there are some checks for GA vs RC vs .z releases:
https://github.com/theforeman/tool_belt/blob/master/procedures/foreman/release.md.erb#L112-L123
That would probably be the place to capture it.

@ekohl that sounds fine, so I'll just add in the new 4.4 definitions for now and then the older ones can be removed later when 4.4 is GA'd.
Our branching docs say to remove the oldest one even before the GA release, so I'll fix that.

Excellent. I like that we're aligning more and more processes.


Fixes #2253 and fixes #2283

Fixes #2253: refine logical tree check and populate parents of insert…ion points with logical info only if necessary.
Fixes #2283: when a node is removed, we need to potentially distribute not only its host but also its parent.

lgtm :+1:


No code generated with JetBrains Rider

Don't know if it is CsWin32's or Rider's fault. There is also a ticket at JetBrains: https://youtrack.jetbrains.com/issue/RIDER-57026
Actual behavior
Opening a project with CsWin32 in JetBrains Rider on Windows won't generate any code at all.
All references to CsWin32 generated code wills be marked as errors in red.
Build works; it is done via dotnet build and everything is set up correctly.
Expected behavior
Similar behaviour to Visual Studio. After opening a project or at least after a build code should be generated and available to the IDE. The IDE shouldn't show any errors.
Other source generators work with Rider, the integration of source generators seems to be not so perfect though.
Repro steps

Open a Project with CsWin32 with JetBrains Rider. For example this repository and one of the samples
Build the project and run it - CsWin32 will generate correct code, project will run
Rider won't see any generated code (all references red)
Rider doesn't show any errors regarding source generators

Context
Exact versions don't seem to matter. Maybe Linux or MacOS will work, as Rider is using .NET Core on those platforms as a base, but .NET 4.6.x on Windows.
I tried a lot of combinations of those versions/settings:

CsWin32

0.1.476-beta (winsdk nightly)
0.1.422-beta (nuget.org)
0.1.378-beta (nuget.org)
0.1.319-beta (nuget.org)

JetBrains Rider

2021.1.3
2021.2 EAP3

.NET SDK (switched via global.json)

5.0.301
6.0-preview4

TargetFramework

net5.0
net6.0
netstandard2.0

LangVersion

9
omitted

Windows 10 21H1

@andi0b it's rider's fault for not using roslyn and invoking the source generator apis before it actually does the "indexing" which is their way to have their IDE show up any errors in the code.

it's rider's fault
@AraHaan good to know. I was trying different things for hours to get it work. Maybe this information can be added to the README.

Done in 2c05bc0

@AraHaan What I actually still don't understand what makes CsWin32 different. All other source generators I tried in Rider work perfectly fine. They generate code and everything works. CsWin32 just doesn't seem to do anything. Not even fail.

The symptoms you describe make it sound like maybe Rider isn't initializing the source generators with msbuild's AdditionalFiles items, which CsWin32 absolutely requires but perhaps other source generators do not.

The symptoms you describe make it sound like maybe Rider isn't initializing the source generators with msbuild's AdditionalFiles items, which CsWin32 absolutely requires but perhaps other source generators do not.

I used to use AdditionalFiles myself until I later decided to use compiler exposed MSBuild properties. Perhaps what could be done is the files currently marked under AdditionalFiles could be opened up in an task and then output to an exposed MSBuild property if that file exists within the $(MSBuildProjectDirectory) (which is the directory of the project itself).
Perhaps that could be a test to see if the generator would start working within Rider.

@arahaan, I appreciate the brainstorming, but I'm not going to stop using AdditionalFiles for what they were designed for and dump entire file content into a scalar property value to workaround what ought to be a simple bug fix in Rider.
If you're interested in doing this as an experiment to inform JetBrains as to what they need to do to fix this scenario in their IDE, you're of course welcome to, but I'm afraid our limited resources require me to spend time fixing and enhancing CsWin32 without much regard to working around bugs in Rider.

I got email today that the rider issue that is opened for it was closed as fixed in branch. So a new release hopefully will fix it for this, and other source generators.

https://youtrack.jetbrains.com/issue/RIDER-57026 related jetbrains issue.
Works in Rider 2021.2 EAP6 although I have to reload the solution everytime I modify NativeMethods.txt. Rider also hides the txt file after loading the solution again. So there are some rough edges

I can confirm everything works for me now on Rider 2021.2 EAP 6 (pre-release build).

Works in Rider 2021.2 EAP6 although I have to reload the solution everytime I modify NativeMethods.txt. Rider also hides the txt file after loading the solution again. So there are some rough edges

@TorbenJ: true. This is generally an issue with Rider and source generators, but I can live with that. Unloading and reloading just the project (right click in solution explorer) does the trick. And if I add this to the csproj, the file stays visible in solution explorer:
<ItemGroup>
    <File Include="NativeMethods.txt" />
</ItemGroup>

Works in Rider 2021.2 EAP6 although I have to reload the solution everytime I modify NativeMethods.txt

New related issue at JetBrians: https://youtrack.jetbrains.com/issue/RIDER-65302


sir i succesfully installed ghos it is good but im unable to connect to a device pls try to contact and help me solve this problem

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

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

Additional context
Add any other context about the problem here.

#285

Can u tell me how to fix sir
On Sun, 8 Aug 2021, 6:24 pm Ivan Nikolsky, @.***> wrote:

Closed #289 https://github.com/EntySec/Ghost/issues/289.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/EntySec/Ghost/issues/289#event-5126630055, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AUZROUFYPEYHWH3DAQ2EZSTT3Z5AVANCNFSM5BYONPYQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

View this issue - #285


Add readme documentation for all the hooks

For every hook, there should be proper readme documentation.

@adityapradhan10 Let me take this up


pixelmon refresh

i am wondering if i could get a refresh on the server as i just took over the account carr_dala aka Warsong which has another account so please get back to me either here or in game

Can I get some proof of the OG owner relinquishing his account? As far as I know, Mojang doesn't offer a transfer

Can I get some proof of the OG owner relinquishing his account? As far as I know, Mojang doesn't offer a transfer

Going to close this, as it's nearing a month old. Reopen if you can provide what Ras requested.


dashboard/config: refactor config system

Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

For upstream-kasan.config I managed to get almost no diff. And it tested both upstream-kasan.config and upstream-kasan-base.config, both work fine.

For KCSAN there are ~1000 lines changed, but it boots/works and the changes look reasonable, so I propose to merge it as is, hopefully in future we will have more control and understanding over this.
I've looked at one USB config -- USB_DWC3_ULPI, it is now disabled because it depends on USB_GADGET. @xairy is it fine change? You seemed to want to disable configs that depend on USB_GADGET.
Some filesystem configs become enabled, but that's good, we just did not sync with KASAN config.
@melver

@xairy If we merge USB configs into upstream-kasan.config, are there any reasons to keep separate upstream-usb.config?
Also, should we enable all USB? Rather then only the ones that are enabled in distros?

For KCSAN there are ~1000 lines changed, but it boots/works and the changes look reasonable, so I propose to merge it as is, hopefully in future we will have more control and understanding over this.
I've looked at one USB config -- USB_DWC3_ULPI, it is now disabled because it depends on USB_GADGET. @xairy is it fine change? You seemed to want to disable configs that depend on USB_GADGET.
Some filesystem configs become enabled, but that's good, we just did not sync with KASAN config.
@melver

LGTM. I think it looks fine, most changes are only related to specific drivers or subsystems.

@xairy If we merge USB configs into upstream-kasan.config, are there any reasons to keep separate upstream-usb.config?

The intention was to not include unnecessary stuff for USB fuzzing config. There was (is?) a lot of configs enabled in upstream-kasan.cfg for which no one knows where those came from. Currently the USB config generation is a fully scripted based on distro configs.
But generally I don't mind merging.

Also, should we enable all USB? Rather then only the ones that are enabled in distros?

I think we'll be getting way too many bugs that no one cares about this way.

For KMSAN there is also ~1000 lines changed, similar to KCSAN, so not bad in itself.
Some things we can now do even better. E.g. dashboard/config/bits-kmsan.config contains:
# CONFIG_PARAVIRT is not set

but upstream-kmsan.config actually contains:
CONFIG_PARAVIRT=y

Now we disable it for real. Though, since current config works, perhaps we don't need to disable it anymore...
For both new and old KMSAN configs I see the following warning when booting kernel.
The ioremap error went away, I don't an explanation for it...
[   18.134145][    T1] ------------[ cut here ]------------
[   18.135669][    T1] WARNING: CPU: 3 PID: 1 at drivers/gpu/drm/drm_gem_vram_helper.c:279 drm_gem_vram_offset+0x168/0x180
[   18.135688][    T1] Kernel panic - not syncing: panic_on_warn set ...
[   18.135718][    T1] CPU: 3 PID: 1 Comm: swapper/0 Not tainted 5.9.0-rc8+ #52
[   18.135733][    T1] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS rel-1.13.0-44-g88ab0c15525c-prebuilt.qemu.org 04/01/2014
[   18.135746][    T1] Call Trace:
[   18.135782][    T1]  dump_stack+0x21c/0x280
[   18.135807][    T1]  panic+0x4c8/0xea7
[   18.135848][    T1]  ? kmsan_get_shadow_origin_ptr+0x81/0xb0
[   18.135868][    T1]  __warn+0x416/0x6a0
[   18.135898][    T1]  ? drm_gem_vram_offset+0x168/0x180
[   18.135921][    T1]  report_bug+0x7d4/0x9d0
[   18.135948][    T1]  ? drm_gem_vram_offset+0x168/0x180
[   18.135980][    T1]  handle_bug+0x6c/0xd0
[   18.136006][    T1]  __exc_invalid_op+0x34/0x80
[   18.136032][    T1]  exc_invalid_op+0x30/0x40
[   18.136056][    T1]  asm_exc_invalid_op+0x12/0x20
[   18.136088][    T1] RIP: 0010:drm_gem_vram_offset+0x168/0x180
[   18.136120][    T1] Code: ff ff 31 c0 31 c9 eb ae 8b 7d d4 e8 b2 c9 73 fb e9 4b ff ff ff 8b 3a e8 a6 c9 73 fb 4d 85 ff 0f 85 63 ff ff ff e8 a8 8e f7 fa <0f> 0b 31 c0f
[   18.136133][    T1] RSP: 0000:ffff88813d7326a8 EFLAGS: 00010293
[   18.136157][    T1] RAX: ffffffff86bd55b8 RBX: ffff888133d7a580 RCX: ffff88823da90000
[   18.136172][    T1] RDX:<PHONE_NUMBER>000000 RSI:<PHONE_NUMBER>0004c0 RDI: ffff888133d7a580
[   18.136190][    T1] RBP: ffff88813d7326e0 R08: ffffea000000000f R09: ffff88813fffa000
[   18.136205][    T1] R10:<PHONE_NUMBER>000004 R11: 00000000ffffffff R12: ffff888133d7a400
[   18.136220][    T1] R13:<PHONE_NUMBER>000000 R14: ffff88823da909e0 R15:<PHONE_NUMBER>000000
[   18.136247][    T1]  ? drm_gem_vram_offset+0x168/0x180
[   18.136274][    T1]  ? drm_gem_vram_offset+0x168/0x180
[   18.136300][    T1]  bochs_pipe_enable+0x1a4/0x480
[   18.136333][    T1]  drm_simple_kms_crtc_enable+0x138/0x1b0
[   18.136356][    T1]  ? bochs_connector_get_modes+0x210/0x210
[   18.136382][    T1]  ? drm_simple_kms_crtc_check+0x280/0x280
[   18.136409][    T1]  drm_atomic_helper_commit_modeset_enables+0x41a/0x12b0
[   18.136440][    T1]  drm_atomic_helper_commit_tail+0xe2/0x960
[   18.136468][    T1]  ? kmsan_get_metadata+0x116/0x180
[   18.136492][    T1]  commit_tail+0x7c9/0x9f0
[   18.136523][    T1]  drm_atomic_helper_commit+0xecc/0xf60
[   18.136551][    T1]  ? drm_atomic_commit+0xa7/0x220
[   18.136585][    T1]  ? drm_atomic_helper_async_commit+0x900/0x900
[   18.136609][    T1]  drm_atomic_commit+0x1c3/0x220
[   18.136643][    T1]  drm_client_modeset_commit_atomic+0xac9/0xd60
[   18.136681][    T1]  drm_client_modeset_commit_locked+0x16b/0x990
[   18.136711][    T1]  ? __msan_metadata_ptr_for_load_8+0x10/0x20
[   18.136740][    T1]  drm_client_modeset_commit+0x9e/0x120
[   18.136774][    T1]  __drm_fb_helper_restore_fbdev_mode_unlocked+0x200/0x350
[   18.136800][    T1]  ? kmsan_get_metadata+0x80/0x180
[   18.136830][    T1]  drm_fb_helper_set_par+0x1b6/0x2a0
[   18.136862][    T1]  ? drm_fb_helper_fill_pixel_fmt+0x7f0/0x7f0
[   18.136891][    T1]  fbcon_init+0x1de0/0x2e80
[   18.136921][    T1]  ? process_slab+0x14d0/0x18a0
[   18.136951][    T1]  ? fbcon_startup+0x1780/0x1780
[   18.136975][    T1]  visual_init+0x412/0x8b0
[   18.137004][    T1]  do_bind_con_driver+0x129c/0x1c40
[   18.137039][    T1]  do_take_over_console+0xf41/0x10e0
[   18.137071][    T1]  ? kmsan_get_shadow_origin_ptr+0x81/0xb0
[   18.137102][    T1]  fbcon_fb_registered+0xa05/0xdd0
[   18.137131][    T1]  register_framebuffer+0xe96/0x10d0
[   18.137169][    T1]  __drm_fb_helper_initial_config_and_unlock+0x1e71/0x26d0
[   18.137204][    T1]  drm_fbdev_client_hotplug+0xb88/0xf40
[   18.137229][    T1]  drm_fbdev_generic_setup+0x39d/0xa00
[   18.137253][    T1]  bochs_pci_probe+0x8ab/0x8c0
[   18.137278][    T1]  ? qxl_gem_prime_mmap+0x30/0x30
[   18.137300][    T1]  pci_device_probe+0xae6/0xe80
[   18.137325][    T1]  ? pci_uevent+0x820/0x820
[   18.137353][    T1]  really_probe+0xdd2/0x2420
[   18.137386][    T1]  driver_probe_device+0x293/0x390
[   18.137416][    T1]  device_driver_attach+0x25a/0x490
[   18.137445][    T1]  __driver_attach+0x78c/0x840
[   18.137471][    T1]  ? kmsan_get_metadata+0x116/0x180
[   18.137498][    T1]  bus_for_each_dev+0x210/0x340
[   18.137523][    T1]  ? driver_attach+0xb0/0xb0
[   18.137551][    T1]  driver_attach+0x89/0xb0
[   18.137577][    T1]  bus_add_driver+0x688/0xc70
[   18.137602][    T1]  driver_register+0x485/0x8e0
[   18.137627][    T1]  __pci_register_driver+0x216/0x370
[   18.137658][    T1]  bochs_init+0x135/0x17d
[   18.137684][    T1]  do_one_initcall+0x371/0x9c0
[   18.137706][    T1]  ? qxl_init+0x1d3/0x1d3
[   18.137741][    T1]  ? kmsan_get_metadata+0x116/0x180
[   18.137769][    T1]  ? kmsan_get_shadow_origin_ptr+0x81/0xb0
[   18.137789][    T1]  ? qxl_init+0x1d3/0x1d3
[   18.137817][    T1]  do_initcall_level+0x1e5/0x3c6
[   18.137844][    T1]  do_initcalls+0x127/0x1cb
[   18.137870][    T1]  ? cpu_init_udelay+0xcf/0xcf
[   18.137893][    T1]  do_basic_setup+0x33/0x36
[   18.137918][    T1]  kernel_init_freeable+0x23d/0x390
[   18.137941][    T1]  ? rest_init+0x1f0/0x1f0
[   18.137960][    T1]  kernel_init+0x1f/0x840
[   18.137979][    T1]  ? rest_init+0x1f0/0x1f0
[   18.138001][    T1]  ret_from_fork+0x1f/0x30
[   18.138349][    T1] Kernel Offset: disabled
[   18.144021][    T1] Rebooting in 86400 seconds..

@ramosian-glider

@xairy If we merge USB configs into upstream-kasan.config, are there any reasons to keep separate upstream-usb.config?

The intention was to not include unnecessary stuff for USB fuzzing config. There was (is?) a lot of configs enabled in upstream-kasan.cfg for which no one knows where those came from. Currently the USB config generation is a fully scripted based on distro configs.
But generally I don't mind merging.

Also, should we enable all USB? Rather then only the ones that are enabled in distros?

I think we'll be getting way too many bugs that no one cares about this way.

OK, let's leave everything as it is now for now.

@xairy please take a look at:
https://github.com/google/syzkaller/pull/1975/commits/56aea639afd7306370514c61ab4ef836f5dd830e
It removes HID. I don't see anything obviously useful among disabled things. Some mouses and LEDs. At least no config has USB in the name.

@xairy please take a look at:
56aea63
It removes HID. I don't see anything obviously useful among disabled things. Some mouses and LEDs. At least no config has USB in the name.

This does not look good. It's the thing I mentioned, HID configs don't depend on USB directly.
E.g. this change disables CONFIG_HID_PRODIKEYS, which is reachable over USB, and we've actually found a bug in it.

Would it make sense to add tests against the generated configs? E.g. make sure that KASAN config always has KASAN enabled, and USB config always has Raw Gadget and Dummy HCD?

@xairy please take a look at:
56aea63
It removes HID. I don't see anything obviously useful among disabled things. Some mouses and LEDs. At least no config has USB in the name.

This does not look good. It's the thing I mentioned, HID configs don't depend on USB directly.
E.g. this change disables CONFIG_HID_PRODIKEYS, which is reachable over USB, and we've actually found a bug in it.

I see. I dropped this commit.
But I disabled at least HYPERV, which is enabled via HID_HYPERV_MOUSE. And HYPERV brings in lots of other configs.

e sure that KASAN config always ha

Would it make sense to add tests against the generated configs? E.g. make sure that KASAN config always has KASAN enabled, and USB config always has Raw Gadget and Dummy HCD and make some selected drivers?

This is one of the main points of this config system. If something is specified and has a wrong value in the resulting config, it produces an error.
The problem is that none of USB configs are spelled out explicitly. But we can spell at least few major ones explicitly.

Would it make sense to add tests against the generated configs? E.g. make sure that KASAN config always has KASAN enabled, and USB config always has Raw Gadget and Dummy HCD and make some selected drivers?

This is one of the main points of this config system. If something is specified and has a wrong value in the resulting config, it produces an error.
The problem is that none of USB configs are spelled out explicitly. But we can spell at least few major ones explicitly.

Well, we actually now do. In the recent versions we now have in usb.yml:
config:
 - USB_SUPPORT
 - USB_GADGET
 - USB_RAW_GADGET: [v5.7]
 - USB_DUMMY_HCD

so if any of these are not present, the generation process will fail.

Found another issue: the new config enables some USB configs that aren't enabled in any of the distro configs, e.g. CONFIG_RTLWIFI_DEBUG and probably some others. In the python script we disable those as a separate step. For normal config these flaky ones don't really matter, but that step was specifically added to not enabled anything unneeded for Android fuzzing.

CONFIG_HID_REDRAGON is an example of the same issue, but for a driver config.

CONFIG_HID_REDRAGON is an example of the same issue, but for a driver config.

CONFIG_HID_REDRAGON is provided by defconfig.
Yes, the new code only enables configs.
Question is: are there any defconfig's that are not important? Are there enough to care to disable them?
E.g. CONFIG_HID_REDRAGON is enabled on my desktop and I see it enabled in ClearLinux:
https://github.com/clearlinux-pkgs/kernel-config/blob/master/base-5.4
So perhaps the majority we got disabled this way just point to our incomplete set of distro configs?

I guess it's OK for upstream-kasan.cfg, but I'd not do that for Android configs.
Looking at android-5.4 config we end up enabling e.g. CONFIG_HID_STEAM and CONFIG_USB_UAS which aren't enabled by defconfig nor by any of our android-* distro configs.
Also noticed that android-5.4 should have CONFIG_USB_RAW_GADGET enabled, but this change disables it.

Looking at android-5.4 config we end up enabling e.g. CONFIG_HID_STEAM and CONFIG_USB_UAS which aren't enabled by defconfig nor by any of our android-* distro configs.

Both of these come from gki_defconfig.

Ah, I see. I wonder why does Pixel not have them enabled? OK, enabling stuff that comes from GKI config makes sense.
Alright, except for CONFIG_USB_RAW_GADGET missing, the rest looks good.

I've pushed a new version.

Alright, except for CONFIG_USB_RAW_GADGET missing, the rest looks good.

Found the problem. It's because it was cherry-picked into android-5.4, but is not present in stable-5.4, so I did not get declaration right. Now it is:
 # It's introduced in v5.5 or so, but we want it in android-5.4, but it's not present in stable-5.4.
 # A better way to express this would to add "or" operator (v5.5||android-5.4), but for now we have this.
 - USB_RAW_GADGET: [v5.4, -stable-5.4]

https://github.com/google/syzkaller/pull/1975/commits/552b65eb5705df4855cf0c5db21cda4a876fd07b

Found another issue: the new config enables some USB configs that aren't enabled in any of the distro configs, e.g. CONFIG_RTLWIFI_DEBUG and probably some others. In the python script we disable those as a separate step. For normal config these flaky ones don't really matter, but that step was specifically added to not enabled anything unneeded for Android fuzzing.

Fixed. This was die to a more general problem: we run olddefconfig to fix up things, but it may enable some things because they are not present in the config and have default value y. This is what happened with CONFIG_RTLWIFI_DEBUG, but also affected lots of other configs.
This is now fixed here:
https://github.com/google/syzkaller/pull/1975/commits/d4236248dab8d79005f8473e53f1fb01735dbe12
(we effectively do something like "oldnoconfig")

Doh! this now doesn't boot with some issue in USB gadget:
WARNING: SOFTIRQ-safe -> SOFTIRQ-unsafe lock order detected
5.9.0+ #54 Not tainted
-----------------------------------------------------
v4l_id/5309 [HC0[0]:SC0[0]:HE0:SE1] is trying to acquire:
ffff888041b38e38 (&trig->leddev_list_lock){.+.?}-{2:2}, at: led_trigger_blink_setup drivers/leds/led-triggers.c:403 [inline]
ffff888041b38e38 (&trig->leddev_list_lock){.+.?}-{2:2}, at: led_trigger_blink_setup drivers/leds/led-triggers.c:392 [inline]
ffff888041b38e38 (&trig->leddev_list_lock){.+.?}-{2:2}, at: led_trigger_blink_oneshot+0x39/0xe0 drivers/leds/led-triggers.c:427

and this task is already holding:
ffff8880131a87e0 (&udc->lock){....}-{2:2}, at: spin_lock include/linux/spinlock.h:354 [inline]
ffff8880131a87e0 (&udc->lock){....}-{2:2}, at: get_gadget_descs+0x183/0x570 drivers/usb/usbip/vudc_sysfs.c:42
which would create a new lock dependency:
 (&udc->lock){....}-{2:2} -> (&trig->leddev_list_lock){.+.?}-{2:2}

but this new dependency connects a SOFTIRQ-irq-safe lock:
 (udc_lock){+.+.}-{3:3}

... which became SOFTIRQ-irq-safe at:
  lock_acquire kernel/locking/lockdep.c:5442 [inline]
  lock_acquire+0x1a8/0x6e0 kernel/locking/lockdep.c:5407
  __mutex_lock_common kernel/locking/mutex.c:956 [inline]
  __mutex_lock+0x156/0x13c0 kernel/locking/mutex.c:1103
  usb_add_gadget+0x21f/0x660 drivers/usb/gadget/udc/core.c:1228
  usb_add_gadget_udc_release+0x29/0x70 drivers/usb/gadget/udc/core.c:1281
  dummy_udc_probe+0x6a0/0xa00 drivers/usb/gadget/udc/dummy_hcd.c:1076
  platform_drv_probe+0xce/0x1a0 drivers/base/platform.c:761
  really_probe+0x290/0xc10 drivers/base/dd.c:554
  driver_probe_device+0x26b/0x3d0 drivers/base/dd.c:738
  __device_attach_driver+0x1d1/0x290 drivers/base/dd.c:844
  bus_for_each_drv+0x15f/0x1e0 drivers/base/bus.c:431
  __device_attach+0x26d/0x3f0 drivers/base/dd.c:912
  bus_probe_device+0x1e4/0x290 drivers/base/bus.c:491
  device_add+0xbd6/0x1af0 drivers/base/core.c:2938
  platform_device_add+0x35c/0x820 drivers/base/platform.c:611
  instrument_atomic_read include/linux/instrumented.h:71 [inline]
  test_bit include/asm-generic/bitops/instrumented-non-atomic.h:134 [inline]
  init+0x7cf/0xb93 arch/x86/crypto/camellia_aesni_avx_glue.c:279
  do_one_initcall+0x103/0x650 init/main.c:1207
  do_initcall_level init/main.c:1280 [inline]
  do_initcalls init/main.c:1296 [inline]
  do_basic_setup init/main.c:1316 [inline]
  kernel_init_freeable+0x612/0x696 init/main.c:1516
  kernel_init+0xd/0x1b8 init/main.c:1405
  ret_from_fork+0x1f/0x30 arch/x86/entry/entry_64.S:296

https://gist.githubusercontent.com/dvyukov/a11ea3f493da48bc4405dcb73209d424/raw/ee16ebfaab3157fd6f4f874613fd4d775618c42c/gistfile1.txt

Seems to be due to USB_LED_TRIG.
We did not have it in upstream-kasan.config, but we had it in upstream-usb.config, so I am not sure why we did not see it before...
@xairy maybe you can figure out something from the stack trace? What other config may lead to this error?

 AFAIR we also needs some of these, so Raw Gadget actually becomes usable, otherwise we default to one of the other legacy gadgets.

Seems to help. Disabled these gadgets on all instances:
https://github.com/google/syzkaller/pull/1975/commits/23e9f109273e9e1b369635bc6d6cf7f78810fddc
Now for more testing.

Could you also check that USB runtests still work? AFAIR we also needs some of these, so Raw Gadget actually becomes usable, otherwise we default to one of the other legacy gadgets.

vusb_ath9k were failing, I noticed we never explicitly enabled ATH9K configs and they are enabled accidentally on some instances and inconsistent across instances. I've enabled them consistently:
https://github.com/google/syzkaller/pull/1975/commits/60c3410b5df45b8c565822b00f3a67e90ae3308b
But it did not help. These 2 fail with the new kasan and usb configs:
vusb_ath9k none/thr                   : FAIL: run 0: call 0 is not executed
vusb_ath9k none/thr/cover             : FAIL: run 0: call 0 is not executed
vusb_ath9k none/thr C                 : FAIL: run 0: call 0 is not finished

vusb_hid namespace/thr                : FAIL: run 0: call 4 is not executed
vusb_hid namespace/thr/cover          : FAIL: run 0: call 4 is not executed
vusb_hid namespace/thr C              : FAIL: run 0: call 4 is not finished

But I tested with the old usb config and they fail in the same way. So I consider this is not a regression.

@hogander-unikie FYI is the kernel config overhaul.
In particular this is new config minimization logic:
https://github.com/google/syzkaller/pull/1975/commits/f7a39f95ec63990b47e55d8beb84b16623386101
I am obviously biased, but I can understand it (as compared to config-bisect.pl which I could only almost understand). The actual minimization logic is around 10 lines and is tested. Now we can evolve it to make it smarter. Some direction are outlined in the comment there.
And this change switches pkg/bisect to this new logic:
https://github.com/google/syzkaller/pull/1975/commits/ab73f70c335875ee696ae33e73a3014a026782a4


bundle_adjust with num-passes > 1... issue writing final condition files

If num-passes > 1, but bundle_adjust converges quickly, the final condition files are not written.
i.e., after the first pass:
No new outliers removed, and the algorithm converged. No more passes are needed.
... then it simply writes the adjust/tsai files, without writing the final condition files (which are useful for using with pc_align).

I put a fix. Now bundle_adjust will save all residual files after each pass since it is not guaranteed to reach the last one. This is some overhead, but normally one calls this tool with maybe two or three passes. Thanks for the report.


Discover serial number

Is your feature request related to a problem? Please describe:

It seems like it would be really annoying to have to manually find the bridge's serial number, then run a script, just to get the certs.
Describe the solution you'd like:

On my HWQS system, it exposes a MDNS TXT record SERNUM, and I was wondering if it would be possible to use this to generate the certificates? I don't have a Caseta system, so I can't test that part.
Describe alternatives you've considered:

Would be OK if it just made you enter the serial number, but ran the script itself, not manually.
Additional context:

In the Caseta case, you have to initiate a certificate signing request, and then press a button on the back of the hub that is right next to the serial number. It's always bugged me that you have to know the IP address of the device (which may be harder for some users to figure out). Even if I used mDNS to discover the new hub, it may be necessary to require either the serial number or the IP address for disambiguation, but that is probably a pretty rare case.
I suspect the cert generation script could:

If no serial number supplied, do an mDNS lookup for hubs
If multiple hubs discovered w/o provided s/n, fail
Connect in physical access mode, prompt user to press button
Generate certificates
Emit s/n alongside certificates

Right now I use a regex to extract the s/n from the hostname, and a TXT record is a much, much better solution.
I don't think this would be super hard to write.

And perhaps if you could detect whether the link button is pressed and there are multiple hubs, pick the one with the link button pressed

@JJTech0130 I'm working on this UI right now. Any interest in being a tester? 😃

While I'm more than willing to help, I don't actually have a Caseta system, so I don't think I'd be the best tester 😂

I just dropped a beta that adds bridge discovery. What happens when you try to use it?  😂

Okay, so thanks to some awesome users testing this for me, the bridge discovery code is looking pretty good. The Homebridge settings UI will now populate any bridges it finds (that answer to _lutron._tcp PTR lookups). It then asks for a SRV record to extract the serial number. Caseta bridges don't have the TXT record you mention:
Lutron\032Status._lutron._tcp.local. 10	IN SRV	0 0 22 Lutron-032e7e88.local.
Lutron\032Status._lutron._tcp.local. 10	IN TXT	"MACADDR=90:70:65:e3:8f:25" "CODEVER=08.07.12f000" "DEVCLASS=08040100" "FW_STATUS=1:NoUpdate" "NW_STATUS=InternetWorking" "ST_STATUS=good" "SYSTYPE=SmartBridge"
Lutron-032e7e88.local.	10	IN	A	<IP_ADDRESS>
Lutron-032e7e88.local.	10	IN	AAAA	fe80::9270:65ff:fee3:8f25

I'm curious to know if other device types have similar records.
In any case, the bridge finder now returns the systype along with IP and serial number information. The user of the BridgeFinder is responsible for managing secrets and constructing the SmartBridge object. Presumably it would be the same for some other bridge type.
I'm going to go ahead and close this up, since I think it fulfills the request (no need for users to discover their serial numbers and run a script to get the cert).


Inline functions can't invoke members of module-private types

In the below code, the compiler wrongly indicates that the method to be inlined (here, X::op_Addition) is inaccessible, but this is not accurate since the calling code is also in the same module.  Manually inlining the call works fine.
module Repro =
  type private X() =
    static member (+)(_:X,_:X) = 0

  let i = X() + X()

Expected behavior
No error.
Actual behavior

error FS0043: The member or object constructor 'op_Addition' is not public. Private members may only be accessed from within the declaring type. Protected members may only be accessed from an extending type and cannot be accessed from inner lambda expressions.

Known workarounds
Manually inline the method.

There is a language suggestion open for this somewhere.

@dsyme if this is really not considered a bug it would be nice to at least make the error message more accurate - op_Addition is public and so the rest of the message is wrong (as is plainly obvious since inlining the code by hand works fine).

op_Addition is public

I believe it's computed as private because it's in a private type? (in F#, private types can't have public members)

But then why can I call it explicitly from elsewhere in the module?  That is, if it were truly private in the .NET sense, this wouldn't work:
module Okay =
    type private X() =
        static member (+)(_:X,_:X) = 0

    let i = X.op_Addition(X(), X())

The error message makes it seem as though this won't work, but it will.

"private" on a type means "private to the enclosing namespace declaration group or module".
So this gives an error:
module Okay =
    type private X() =
        static member (+)(_:X,_:X) = 0

open Okay
let i = X.op_Addition(X(), X())```

Right, I get that (it's why I think this behavior is a bug).  My point is that in my initial repro the inlining would occur within the same module as the type definition (and so would be fine, as my latter version of the Okay module shows).  However the compiler doesn't allow it and the error message states:

The member or object constructor 'op_Addition' is not public. Private members may only be accessed from within the declaring type. Protected members may only be accessed from an extending type and cannot be accessed from inner lambda expressions.

This is clearly not true: op_Addition can be used outside of the declaring type X as long as it's used in the module where X is defined, which is all I was trying to achieve anyway.

@kbattocchi Yes, you're right, the error message is poor
However right now all solutions to statically resolved calls must currently be public.  There's an open language suggestion to lift that limitation, in which case this code would indeed compile.

@sdsyme, I've been going over all issues that mention 'inline' but can't find an (open or closed) issue that directly or indirectly addresses this.
Just saying, perhaps we need a new language suggestion for this one.

@abelbraaksma @kbattocchi This is the one https://github.com/fsharp/fslang-suggestions/issues/230


[DOC]: Include minimum hardware requirements in the Helm README

How would you describe the priority of this documentation request
None
Please provide a link or source to the relevant docs
github.com/nvidia/nv-ingest
Describe the problems in the documentation
The helm chart deployment needs to note the minimum hardware and software requirements necessary.
(Optional) Propose a correction or improvement
No response

Closed with https://github.com/NVIDIA/nv-ingest/pull/84


Using .pageSheet or .formSheet strangly resizes view

I think, on iPad, neither modalPresentationStyle = .pageSheet nor modalPresentationStyle = .formSheet do work with a heroModalAnimationType like for example .pageIn(direction: .up).
I'm using Hero 0.3.6.

Sorry, those presentation styles are not supported.


paper code inquiry

is this the source code for Exploiting Cognitive Structure for Adaptive Learning？

Not exactly, here we publish three simulators, KSS is the source code of the KSS used in our paper


opentracing-backend is no longer being built

The latest version of opentracing-backend is 3.5.2 (https://mvnrepository.com/artifact/com.softwaremill.sttp.client3/opentracing-backend) but every other package under com.softwaremill.sttp.client3 is up to 3.7.1 (example https://mvnrepository.com/artifact/com.softwaremill.sttp.client3/async-http-client-backend-cats-ce2).
Is it deliberate that opentracing-backend doesn't have new versions anymore? Or is this a bug?

Yes, that general implementation was broken. It's not possible unfortunately to create a "general" opentracing backend, as the way the context is propagated depends on the effect library. Currently we have an implementation for ZIO only (using their FiberLocals)

@adamw so does that mean opentracing-backend won't be getting new versions anymore? Or will it still get new versions but with a different release cycle to the other packages?

Yes, it's simply not possible to create an opentracing backend generically so that it would work with any effect (Future, sync, cats-effect, zio). There's a ZIO implementation (https://sttp.softwaremill.com/en/latest/backends/wrappers/opentelemetry.html#tracing-zio), and trace4cats provides sttp integration (also in the same doc). But I don't think there's a working solution e.g. for Future (although maybe somehow using Kamon?)


wrap long usernames/teamnames in convo menu header

r? @keybase/react-hackers

@keybase/react-hackers bump


TTree indices awareness in uproot for faster data access from a file?

We're trying to use ROOT files as a cache for cut information, and right now we're selecting the data we need by iterating through the entire file.
I was wondering if uproot is aware of TTree indices and might be able to grab data more efficiently, or if that is on the uproot roadmap.
Adding @bloer since he's interested in this issue as well.
Thanks!

I don't think I know about this. Try printing out ttree.all_members or ttree["some_branch"].all_members, maybe with minimal_ttree_metadata=False passed to uproot4.open. If you see what you're looking for, we can build convenience methods around it.

This has been quiet for a while, but I'm going through issues now.
By "TTree indexes," do you mean any of the following?

fIndexValues
fIndex
fTreeIndex

Currently, we're not reading these values because they come at the end of the structure and we're not using them. But the minimal_ttree_metadata parameter controls that:
>>> import uproot4
>>> import skhep_testdata
>>> f = uproot4.open(skhep_testdata.data_path("uproot-Zmumu.root"),
...                  minimal_ttree_metadata=False)
>>> f["events"].member("fIndexValues")
<TArrayD [] at 0x7fc53d3e8df0>
>>> f["events"].member("fIndex")
<TArrayI [] at 0x7fc53d3e8e50>
>>> f["events"].member("fTreeIndex")
None

If these are useful, I'll consider them non-minimal. What would you do with these arrays? Can you show me an example?

Let me know if you want to pursue this. By default, the fIndexValues, fIndex, and fTreeIndex quantities are not read; the minimal_ttree_metadata=False argument can turn them on.
If you know of a way that these quantities can be used, tell me and I'll (1) read them by default and (2) do whatever you have in mind.
For now, though, I'm going to close this issue because of the silence. (I use the open issues to look for work to do, so clutter is a problem. If you respond to an issue, even if it's closed, I do see it in my email inbox and will reopen the issue.)


Improve TypeScript declarations file

What's Changing and Why
Addresses but does not fully close #581
Proper way to import is now import * as X from 'jimp' wherein X can be
whatever the end-user wants, in most cases Jimp.
As stated on the aforementioned issue another PR might come with a more thorough fix not requiring // @ts-ignore
What else might be affected
Method of importing with TypeScript, possible requires doc updates.
Tasks

[ ] Add tests
[x] Update Documentation
[ ] Update jimp.d.ts
[x] Add SemVer Label

@DanielRosenwasser @hipstersmoothie I've reworked it so it no longer requires the ts-ignore to satisfy the IDEs. Would appreciate another review. Copying from my Git commit message:
I have losely based this on knowledge gained from analyzing other declarations files, primarily @types/better-sqlite3. Using this formatting for declaration file allows for import * as Jimp from 'jimp' importing which is a proper way when using TypeScript as it primarily recommends ES6 imports. It will require setting the allowSyntheticDefaultImports compiler option to true, however this is a very small issue as many packages on npm (including aforementioned better-sqlite3) require this.

Like previous commit this ensures we no longer need to use import Jimp = require('jimp'). As has been agreed upon, using require` when
working with ES6 modules is just wrong in code style.

Unlike previous commit no more ts-ignore, yay!

It is maintainable. New types or interface come at the bottom and any
new or modifications to functions and constants go in the topmost
interface. We'll never have to touch the declared const or the export.

Lastly, I would really appreciate a new version to be pushed to npm if you guys decide to merge this. Due to your building setup I can't just yarn add favna/jimp#fix/typedeclarations to install my own branch so until a new proper build is released I'll have to use the old method of importing which also blocks me from updating my documentation as the jsdoc2md compiler cannot handle the = require structure.

Oh and one final note, if you guys decide to merge this I think #581 can be closed since the importing and exporting should be entirely fixed but I'll let that upon your judgement.

@Favna I will release once @DanielRosenwasser gives a 👍

Okay, I think that this change is appropriate. Users can use
import jimp = require("jimp")

to import the module.
Alternatively, if they have esModuleInterop they can use a default import:
import jimp from "jimp";

For the record, because Jimp is callable, import * as jimp from "jimp" shouldn't be used.

@Favna can you add this to the docs in the "jimp" package readme? After that I'll get it merged and released.

Ow.. I see you merged already. Sorry I didn't add the usage doc to the readme yet. Long work days means I only just got home and you guys commented at my 2AM :\
I'll add the usage docs right now and create a new PR I guess.


v0.5.0 MaxListenersExceededWarning / errors deploying while there are browsers using the FGT wallets

Before even running npm run environment-test, we have error:
(node:1093) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 close listeners added to [Socket]. Use emitter.setMaxListeners() to increase limit
(Use `node --trace-warnings ...` to show where the warning was created)

Diagnostic:
With haproxy on:
pharmaledger@fgt-dev-pl:~$ docker logs --timestamps fgt > fgt.log 2>&1
pharmaledger@fgt-dev-pl:~$ less fgt.log
...
2021-09-06T14:18:57.921594927Z [API-HUB] listening on port :8080 and ready to receive requests.
2021-09-06T14:18:57.921597728Z
2021-09-06T14:18:58.546383204Z undefined:8080 - [2021-09-06T14:18:58.530Z] GET:/mq/receive-message/did:demo:MAH135315170 500 3.868ms
2021-09-06T14:18:58.547681052Z undefined:8080 - [2021-09-06T14:18:58.547Z] GET:/mq/receive-message/did:demo:PHAZCGVW7A7 500 0.36ms
2021-09-06T14:18:58.548591417Z <IP_ADDRESS>:8080 - [2021-09-06T14:18:58.548Z] GET:/mq/receive-message/did:demo:MAH135315170 500 0.193ms
2021-09-06T14:18:58.549074525Z <IP_ADDRESS>:8080 - [2021-09-06T14:18:58.548Z] GET:/mq/receive-message/did:demo:PHAZCGVW7A7 500 0.157ms
2021-09-06T14:19:00.529263307Z <IP_ADDRESS>:8080 - [2021-09-06T14:19:00.528Z] GET:/mq/receive-message/did:demo:MAH135315170 500 0.198ms
2021-09-06T14:19:00.529605048Z <IP_ADDRESS>:8080 - [2021-09-06T14:19:00.529Z] GET:/mq/receive-message/did:demo:PHAZCGVW7A7 500 0.205ms
2021-09-06T14:19:02.528358208Z <IP_ADDRESS>:8080 - [2021-09-06T14:19:02.527Z] GET:/mq/receive-message/did:demo:PHAZCGVW7A7 500 0.404ms
2021-09-06T14:19:02.529287195Z <IP_ADDRESS>:8080 - [2021-09-06T14:19:02.528Z] GET:/mq/receive-message/did:demo:MAH135315170 500 0.261ms
2021-09-06T14:19:04.527204506Z <IP_ADDRESS>:8080 - [2021-09-06T14:19:04.526Z] GET:/mq/receive-message/did:demo:MAH135315170 500 0.226ms
2021-09-06T14:19:04.527452742Z <IP_ADDRESS>:8080 - [2021-09-06T14:19:04.527Z] GET:/mq/receive-message/did:demo:PHAZCGVW7A7 500 0.129ms
2021-09-06T14:19:06.527772915Z <IP_ADDRESS>:8080 - [2021-09-06T14:19:06.527Z] GET:/mq/receive-message/did:demo:MAH135315170 500 0.452ms
2021-09-06T14:19:06.528336348Z <IP_ADDRESS>:8080 - [2021-09-06T14:19:06.528Z] GET:/mq/receive-message/did:demo:PHAZCGVW7A7 500 0.148ms
2021-09-06T14:19:08.534583138Z <IP_ADDRESS>:8080 - [2021-09-06T14:19:08.533Z] GET:/mq/receive-message/did:demo:PHAZCGVW7A7 500 0.445ms
2021-09-06T14:19:08.535099308Z <IP_ADDRESS>:8080 - [2021-09-06T14:19:08.534Z] GET:/mq/receive-message/did:demo:MAH135315170 500 0.275ms
2021-09-06T14:19:10.539753304Z <IP_ADDRESS>:8080 - [2021-09-06T14:19:10.539Z] GET:/mq/receive-message/did:demo:PHAZCGVW7A7 500 0.427ms
2021-09-06T14:19:10.540734351Z <IP_ADDRESS>:8080 - [2021-09-06T14:19:10.540Z] GET:/mq/receive-message/did:demo:MAH135315170 500 0.344ms
2021-09-06T14:19:12.541304916Z <IP_ADDRESS>:8080 - [2021-09-06T14:19:12.540Z] GET:/mq/receive-message/did:demo:MAH135315170 500 0.423ms
2021-09-06T14:19:12.541933805Z <IP_ADDRESS>:8080 - [2021-09-06T14:19:12.541Z] GET:/mq/receive-message/did:demo:PHAZCGVW7A7 500 0.329ms
2021-09-06T14:19:14.538792217Z (node:49) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 close listeners added to [Socket]. Use emitter.setMaxListeners() to increase limit
2021-09-06T14:19:14.538826408Z (Use `node --trace-warnings ...` to show where the warning was created)
...
2021-09-06T14:21:23.194178294Z Caught event type [uncaughtException]. Shutting down... uncaughtException Error [ERR_STREAM_WRITE_AFTER_END]: write after end
2021-09-06T14:21:23.194187414Z     at writeAfterEnd (_http_outgoing.js:668:15)
2021-09-06T14:21:23.194190214Z     at write_ (_http_outgoing.js:680:5)
2021-09-06T14:21:23.194192444Z     at ServerResponse.write (_http_outgoing.js:661:15)
2021-09-06T14:21:23.194194634Z     at /fgt-workspace/privatesky/psknode/bundles/pskWebServer.js:3492:8
2021-09-06T14:21:23.194196824Z     at deliverMessage (/fgt-workspace/privatesky/psknode/bundles/pskWebServer.js:4443:5)
2021-09-06T14:21:23.194211804Z     at NotificationsManager.sendMessage (/fgt-workspace/privatesky/psknode/bundles/pskWebServer.js:4504:11)
2021-09-06T14:21:23.194214184Z     at /fgt-workspace/privatesky/psknode/bundles/pskWebServer.js:3451:24
2021-09-06T14:21:23.194216314Z     at IncomingMessage.<anonymous> (/fgt-workspace/privatesky/psknode/bundles/pskWebServer.js:6344:9)
2021-09-06T14:21:23.194218714Z     at IncomingMessage.emit (events.js:327:22)
2021-09-06T14:21:23.194220794Z     at endReadableNT (internal/streams/readable.js:1327:12)
2021-09-06T14:21:23.194222984Z     at processTicksAndRejections (internal/process/task_queues.js:80:21) {
2021-09-06T14:21:23.194225154Z   code: 'ERR_STREAM_WRITE_AFTER_END'
2021-09-06T14:21:23.194246514Z }

The MaxListenersExceededWarning occurs early and causes api-hub to shutdown.
Workaround so far: Shutdown (inhibit fgt-dev) site on haproxy during the docker startup + npm build-all + npm run environment-test and run it again later.
root@haproxy:~# vi /etc/haproxy/haproxy.cfg
-------- COMMENT BY PREFIXING WITH #
       #use_backend fgt-dev-pl if { hdr(host) -i fgt.dev.pharmaledger.pdmfc.com }
       #use_backend fgt-dev-pl if { hdr(host) -i fgt-dev.pharmaledger.pdmfc.com }
--------

root@haproxy:~# systemctl reload haproxy ; less /var/log/haproxy.log

With haproxy stopped, re-deployed the docker image, and watch log again:
pharmaledger@fgt-dev-pl:~$ docker logs --timestamps fgt > fgt2.log 2>&1

Workaround seems to work.
Enabled haproxy again. (remove the # prefix for fgt-dev sites, and reload haproxy).

There seems to be no way to disable/enable published ports at runtime, without stopping/restarting the docker...
https://stackoverflow.com/questions/19335444/how-do-i-assign-a-port-mapping-to-an-existing-docker-container

Since v0.8.0 this is no longer an issue (a decent health check solves this issue)


[WIP]add filesystem using starlet interface

What type of PR is this：

[ ] bug
[x] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

@harui7890 Please resolve the conflicts.

@harui7890 Please resolve the conflicts.

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

rerun ut

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)


Update task 7 for SOFE

For this task as the CAD files wont be able to be converted to a mesh within the codespace environment without SALOME, however we could provide an example script for how this may be done once salome is installed locally?
Furthermore, for the H transport, for the workshop i'm guessing we should only run a steady state simulation for the 3D case, not sure how long the sims take to run. Otherwise we could provide a mesh for a 2D slice and run a simple transient sim on that?
However we will likely have to have GIFs and images of the XDMF outputs too
what do you think @RemDelaporteMathurin ?

I would like FESTIM to pivot towards a GMSH based workflow and wouldn't want us to dig ourselves into SALOME.

Furthermore, for the H transport, for the workshop i'm guessing we should only run a steady state simulation for the 3D case, not sure how long the sims take to run. Otherwise we could provide a mesh for a 2D slice and run a simple transient sim on that?

Yes! a steady state 3D is good. We could base it from my 3D monoblocks simulations (we have the repo for this)

However we will likely have to have GIFs and images of the XDMF outputs too

Yes


Discussion: How to use Ion in CI/CD

So, I don't know whether this is known or even intentional, but with an Ion version of SST you can't do

npx sst deploy
in CI/CD environments anymore, or rather you can't use SST over Npx anywhere at all anymore.

So are you supposed to just install the new SST CLI in CI/CD environments as well?

Dear @thdxr, Any update on the same.


oh, there's closed issue that i looking for.. passing trigger information when job execute

          oh, there's closed issue that i looking for.. was this issue solved?

In my case, i created Kafka reader spring batch which is triggered by KEDA topic.
and i wanted to intergrate project and looking for switch~case logic.
ex)
topic-1 triggers app-1
topiic-2 triggers app-2
so i thought this below works, but closed..
template:
  spec:
    containers:
      - name: containerjobname
        image: imagefoofoo
        env:
          - name: APP_NAME
            value: "{{metametadata.foo}}"

triggers:
  - metadata:
      bootstrapServers: 'kafkabroker'
      consumerGroup: topic-1-consumer-group
      lagThreshold: '1'
      offsetResetPolicy: latest
      topic: topic-1
      metametadata:
          foo : app-1
    type: kafka

Originally posted by @inha18minseokkim in https://github.com/kedacore/keda/issues/5218#issuecomment-2426597127

i just think it would be nice for job to know which topic triggers the job in runtime, although somethings like metadata is not available. topic would be enough


[BUG] iinfo not returning a non-zero return code

Describe the bug
oiiotool --info returns a non-zero return code when run on a file it doesn't recognize. This is what I would expect. However, iinfo returns zero when run on the same file.
To Reproduce
(The shell is bash, and the command echo $? shows the return value of the most recent command.)
jdietrch@saturn:~/a$ cat abc.txt
This is a text file.
jdietrch@saturn:~/a$ oiiotool --info abc.txt
oiiotool ERROR: read : OpenImageIO could not find a format reader for "abc.txt". Is it a file format that OpenImageIO doesn't know about?

Full command line was:
> oiiotool --info abc.txt
jdietrch@saturn:~/a$ echo $?
255
jdietrch@saturn:~/a$ iinfo abc.txt
iinfo ERROR: OpenImageIO could not find a format reader for "abc.txt". Is it a file format that OpenImageIO doesn't know about?

jdietrch@saturn:~/a$ echo $?
0
jdietrch@saturn:~/a$

Expected behavior
Along with the error message, I expect iinfo to return a non-zero return code as oiiotool --info does.
Evidence
See above.
Platform information:
I am running Debian testing and have version  <IP_ADDRESS>~dfsg0-3 of the openimageio-tools package installed.
Let me know if I can provide any further information.
Thank you,
James Dietrich

Quite right. Fix coming.


Add fading effect to timelapse

https://twitter.com/mildthing99/status/1492480129016246274
https://ottverse.com/crossfade-between-videos-ffmpeg-xfade-filter/?utm_source=pocket_mylist

It has been implemented.


What is the font use in the IDE screenshots?

Sorry for asking a (near) non project related question, but what is the gorgeous font use on all the screenshots? And I am sure that I am not the only one wondering! thx

I believe it's Operator Mono, if I'm not mistaken?

oh god, a 200$ font!
Wonder if I can convince my boss that it will increase, if not my productivity, my enjoyment at the office!

Found this as a free alternative<EMAIL_ADDRESS>

That's my editor, and it is indeed Operator Mono. I got it mostly because I take many screenshots of code and the italics impress people 😉
That being said, if you don't take screenshots often Fira Code is a great font with beautiful ligatures!


fixes to docs

remove docs/theory
update links to figures in testing theory doc

Codecov Report

Merging #172 (2d98b1a) into master (af5b836) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #172   +/-   ##
=======================================
  Coverage   56.09%   56.09%
=======================================
  Files          20       20
  Lines        3264     3264
  Branches      519      519
=======================================
  Hits         1831     1831
  Misses       1314     1314
  Partials      119      119

Continue to review full report at Codecov.

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


未能正确加载Microsoft.VisualStudio.Editor.lmplementaion.EditorPackage包

Does this issue occur when all extensions are disabled?: Yes/No
yes

VS Code Version: Visual Studio 2022 version 17.8.6

OS Version: win11

Steps to Reproduce:

[severity:It’s more difficult to complete my work]
I installed Visual Studio 2022 version 17.8.6 for the first time under my new computer win10 without any problems for the first time, but when I reopened VS again, it prompted that the Microsoft.VisualStudio.Editor.lmplementaion.EditorPackage package was not loaded correctly and popped up several times in a row, trying to remove C: The VisualStudio folder under \Users\18990\AppData\Roaming\Microsoft does not solve, nor does reinstall and repair vs. After that I tried to upgrade the operating system to win11. This problem still occurs after installing VS after formatting all hard drives,
This issue caused me to be unable to save the zoom of the code-editing page and rendering a white background in dark mode. Hopefully someone can have a good way to solve it. It’s been bothering me for a week. Thank you!!!

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.86.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

This seems to be for visual studio


Fix unchecked reading boolean values

While building a prototype with bitbuffer I noticed that the BitReadBuffer::read_bool_unchecked method doesn't take the endianess into account.

Good catch, thanks


Cannot read property 'profile' of undefined

Hi RallySlack,
Just trying out your framework today but getting errors. I think it is either a bug or a miss understanding on my part. I am getting a 'Cannot read property 'profile' of undefined'. I have included my config (Figure 1) and the error (Figure 2).
I could not figure out what the slackWebKey was for simply creating a Bot. Even with the following issue that was closed: https://github.com/paypal/rallyslack/issues/1. So i created a new Slack App via (https://*.slack.com/apps/new/RANDOMNUMBER) and was supplied with

Bot Oauth Access Token (slackBotKey?)
But User OAuth Access Token (slackWebKey?)

When typing in Slack it would trigger the error in Figure 2. However was not able to enable information coming from Rally.
Please correct the error of my ways.
Thanks!
-matt
Figure 1) Config
`"slackbot": {
"bot": "./bot/bot",
"slackBotKey": "xoxb-",
"slackWebKey": "xoxp-"
},
"config": {
"modules": {

  "capture": {
      "path": "./bot/capture/capture",
      "loadIgnore": true
  },

  "rally": {
      "path": "./lib/rally/rally",
      "options": {
        "aliasPath": "./lib/rally/lib/alias.json",
        "configPath": "./lib/rally/config.json",
        "rallyKey": "_***"
      }
  },

*** omitted for brevity ***
`
Figure 2) Error:
error:  TypeError: Cannot read property 'profile' of undefined
at Request.slackWeb.userInfo.err [as _callback] (C:\CE\rallyslack\bot\logger.js:25:38)
at Request.self.callback (C:\CE\rallyslack\node_modules\request\request.js:186:22)
at emitTwo (events.js:106:13)
at Request.emit (events.js:191:7)
at Request. (C:\CE\rallyslack\node_modules\request\request.js:1163:10)
at emitOne (events.js:96:13)
at Request.emit (events.js:188:7)
at IncomingMessage. (C:\CE\rallyslack\node_modules\request\request.js:1085:12)
at IncomingMessage.g (events.js:292:16)
at emitNone (events.js:91:20)

@Xerxous and @rootid23, do you have some idea why user would come undefined.
https://github.com/paypal/rallyslack/blob/master/bot/logger.js#L25
Can you help Matt here?

Just created an issue for a refactor on the code so that the missing slackWebKey won't cause the bot the crash. For further clarification on the slackWebKey, it was necessary to retrieve the detailed information of the user such as first/last as registered in Slack. Without it, the bot is only able to show the username.
This fix will make the slackWebKey optional for those who don't wish to use it.

Hi,
I am also getting the same issue!
I have taken out the slackWebKey, but still the same issue persists.
Cheers!


Consider nullable annotations in explicit nulls

The @NotNull and @NonNullable type annotations are currently used by explicit nulls to not nullify (or add flexible type to) a reference type from Java signatures.
In addition to these annotations, I suggest we also consider the @Nullable annotation. It indicates a value can indeed be null. In this case, we will always nullify the type (adding | Null), instead of adding a flexible type.
// in Java
@Nullable String f(String s)

// in Scala with explicit nulls currently
def f(s: (String)?): (String)?

// in Scala with explicit nulls considering the annotation
def f(s: (String)?): String | Null

A list of @Nullable annotations we may want to consider:
javax.annotation.Nullable
org.jetbrains.annotations.Nullable
org.jspecify.annotations.Nullable
...

There is already some support for this (or was it disabled at some point?). See NotNullAnnots in Definitions.scala and uses of that. def hasNotNullAnnot in JavaNullInterop.

Not sure about the current status of Java ecosystem. I guess jspecify is the latest standard for this kind of annotations?

Perhaps this could be a good introductory task for @HarrisL2

This is a list of annotations considered by Kotlin:
https://github.com/JetBrains/kotlin/blob/a9ff22693479cabd201909a06e6764c00eddbf7b/core/compiler.common.jvm/src/org/jetbrains/kotlin/load/java/JvmAnnotationNames.kt

Hi @noti0na1 , I want to work on this issue can you please assign me this one ?

Hi @rajRishi22 , feel free to work on this issue.

Reassign to @HarrisL2, since he is already working on this.

Reassign to @HarrisL2, since he is already working on this.

Got it.


Unnecessary libraries?

Why is the paulscode stuff downloaded?
Could these potentially be ommited from a future release when setting up versions that don't need them?

BetterJSONS. This is the reason why it's downloaded. Also, if you downloaded it once, it shouldn't download again. Unless you constantly clear libraries folder

Oh it fixes sound in older infdev, that's cool.

Ykw I'm not wasting this issue. Updating with weird bug.


Export-DBAUser does not contain permissions granted to a user created database role

Verified issue does not already exist?
No
What error did you receive?
No error shown, but the permission grant/deny t-sql is not generated.
Steps to Reproduce
# provide your command(s) executed pertaining to dbatools
# please include variable values (redacted or fake if needed) for reference

To replicate the issue, we assume our localhost has a sql server default instance (any version above 2012 will do)
use TempDB
create role Test_role;
create table dbo.t (id int);
grant select, delete to object::dbo.t to [Test_role];
Now we can do the PowerShell part
import-module dbatools
$svr = 'localhost';
$pth = 'c:\temp\permission.sql';
export-dbauser -server $svr -database tempdb -filepath $pth;
After you open $pth in notepad, you will only find the following t-sql for Test_role
USE [tempdb]
GO
IF NOT EXISTS (SELECT * FROM sys.database_principals WHERE name = N'Test_role' AND type = 'R')
CREATE ROLE [Test_role]
GO
But there is no such t-sql
grant select, delete to object::dbo.t to [Test_role];
Are you running the latest release?
Yes
Other details or mentions
No response
What PowerShell host was used when producing this error
PowerShell Core (pwsh.exe), Windows PowerShell (powershell.exe), Windows PowerShell ISE (powershell_ise.exe)
PowerShell Host Version
Name                           Value

PSVersion                      5.1.17763.1971
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.17763.1971
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
SQL Server Edition and Build number
sql server 2012 to sql server 2019 all tested

I think this is by design. Only directly granted permissions are exported.
To export the permissions granted to a database role, we have Export-DbaDbRole.
@potatoqualitee please reopen if I'm wrong.


Problem with modal size when using custom slot in modal inside component

I'm submitting a ...
[x] bug report

Greetings,
I'm trying to create a general "Edit" template, in which i want to use modals. Template is like this
<template>

      <form>
        <va-card :title="$t('forms.inputs.title')">
          <slot></slot>

            <va-button  color="danger">Back</va-button>
            <va-button color="warning">Close</va-button>
            <va-buttoncolor="success">Edit</va-button>

        </va-card>
      </form>

<va-modal
      v-model="editModal"
      :size='cModalSize'
      :title="editModalTitle"
      :label="editModalLabel"
      hideDefaultActions
      noOutsideDismiss
    >

      <slot name="editSlot"></slot>
      <template v-slot:actions>
        <va-button @click.prevent="editModal=false" color="danger">Back</va-button>
         <va-button @click.prevent="editEvent" color="success">Submit Modifications</va-button>
      </template>
    </va-modal>
</template>

When imported, everything works fine except modal size, I tried passing size as a prop, setting it directly but to no avail. No matter what I do the size is always the default(medium), which I think is not the expected behavior.
This how I'm calling the component (cn-edit is the name of the imported component)
<template>
  <cn-edit
    v-if="renderComp"
    @expDateSet="setExirationDate"
    modalSize="large"
    editModalLabel="Edit Consent"
    editModalTitle="Edit Consent"
  >

    <template v-slot:editSlot>

        <va-input
          :disabled="!canEdit"
          v-model="myModel"
        ></va-input>

        <va-select
          label="Form Title"
          v-model="selectedForm"
          keyBy="_id"
          textBy="formTitle"
          :options="forms"
        ></va-select>

        <va-input
          type="textarea"
          :disabled="!canEdit"
          v-model="anotherModel"
        ></va-input>

    </template>
  </cn-edit>
</template>

Did more test, this behavior occurs only when using a modal inside a slot. If i use modal in the same component but outside of th "" (see above) I get the expected behavior.

Using flex md12 class on modal fixes this. I was under the impression that specifying size was enough. Sorry for the commotion.


Enable use of Labels as Values feature in Clang when not wrapped by GCC

Split off from #77233. Tested on top of #92316, using the benchmark:
@tool
extends EditorScript

func _run() -> void:
	static_run()

static func static_run() -> void:
	var time_before := Time.get_ticks_msec()

	var r : int
	for i in range(10000000):
		var next : int
		var first := 1
		var second := 2
		for c in range(40):
			next = first + second
			first = second
			second = next
		r = next
	print(r)

	var total_time := Time.get_ticks_msec() - time_before
	print("Time taken: " + str(total_time))

In dev build, no notable difference. But in release build with lto=full, times went down from 16-18 seconds, to 13 seconds, i.e. ~20% improvement in speed.

Thanks!


Support offline installs through command-line

I discovered on the extension marketplace page that downloading .vsix files for offline installation is now possible. This new feature in VSCode is great, but it would be more convenient if it could also be done via code.exe --install-extension <path>.

It is already supported. You can pass the VSIX path for install-extension


Finish updating user cases we did from workshop

@K-Beicher you can add more detail here about that.

@K-Beicher will upload csv of user cases

User stories - draft.csv
First version of the user stories in csv format.

@signekb, @lwjohnst86 when you have time, could you have a look through the attached.  I'm particularly interested in hearing if you have any ideas as to the naming of the users.

@K-Beicher fabulous work!! Very extensive. I'll review it in more details over the next days :relaxed:

Very nice, @K-Beicher!
I have some ideas for naming the "user entering data" (with some help from ChatGPT):

Data Contributor (Pros: Clear and concise, emphasizes contribution; Cons: Generic, lacks specificity)
Data Uploader (Pros: Simple and direct; Cons: Somewhat informal, might downplay the significance of the role)

@K-Beicher I so far have only looked at the "admin" and the "data contributor" rows and they look great!
I agree with @signekb about the names. I like "data contributor", though it doesn't always fit with how people would actually be using it. But it does give a nice feeling of "contributing to something" rather than "told to do it" :stuck_out_tongue:

@lwjohnst86 Right? I also think it's nice with contributing rather than uploading 😋
In what ways would it not fit with how people would be using it?

Contributing mostly implies someone is giving something that they don't have to give. But if the technician or data entry personnel is paid to do that as their job, they have to do it. It's less contributing and more paid for it, if that makes sense.

Ahhhhhhh, in that sense.. I see... hmm...

We'll go with those.  @lwjohnst86 do we want to keep them as a csv file for now, or do we want to store them somewhere different?


CPU Fan doesn't spin on sartup

Describe the bug
Very often on system startup CPU fan doesn't spin even if the program says it's spinning at some RPM
To Reproduce
Steps to reproduce the behavior:

Start the system
AlienFX GUI starts itself with windows on tray
I see cpu temperature slowly rise and fan doesn't fisically spin even if the program shows some RPMs
If i restart AlienFX GUI or change "Power Mode" from manual to any other value and the go back to manual everything works perfectly

Expected behavior
AlienFX GUI should be able to control CPU fan without the need from me to restart it or force the cange of Power Mode
System (please complete the following information):

Model: Alienware X17 R2
Model details: i7-12700H, RTX 3080Ti Laptop, 16GB RAM
OS: Windows 11 Home 22H2

Additional context
I use a custom fan curve and it works perfectly fine if I do that thing to make AlienFX get unstuck

Try this one - it should help - fan_gui.zip

Ok, I've been using this version you gave me for some days with no problems BUT as I said the problem already was intermittent
In fact this morning it happened again, as always switching mode and going back to manual did the fan start spinning correctly as always

I do some into latest release version.
If this didn't help again, seems like issue into BIOS, and you should configure out the system other way - i tell you how.

So I should download the latest version?
(By the way in the last 2 weeks or so fans have worked fine)

It has some other fixes about it, so try.

Thank you, i'll try it and let you know <3

I've tested latest version for some days and I can tell that the problem is still there; as always it is discontinuous, sometimes it work, sometimes fans are blocked on 700rpm (sometimes is CPU1, other times is CPU2 or GPU 1 and so on; it seems to be random)
As always changing between profiles does work and restores function

Hmm....
Latest version ALWAYS switch between power modes then resume from sleep. So i really don't understand how it is possible to still hang fans. Maybe timing is an issue....

Today I've noticed something:
It's NOT that the program doesn't control the fan spinning, actually it does
The thing is that Fan Max RPMs are lower then actual maximum until I change Power Mode to another profile and then switch back to Manual
For example CPU2 fan will spin at 2000/2500 RPMs on manual mode 100% boost BUT if I switch to another profile and then go back to manual it will spin at 3000 RPMs
And I can tell this difference matters in terms of temperatures (and also is audible even without looking at RPMs)
I wish this could be helpful

Oh... Look at this - https://www.reddit.com/r/Alienware/comments/147sckl/fan_speed_not_working/
Seems like something broken into BIOS for some models...
About you issue... All i can do is delay power mode switch - MAYBE this can help. Now i switch mode to Balanced then app start/resume, then switch to mode you define into profile. But now it will be done immediately, so i have an idea some delay between can help.


Billing - Stripe UI - payment methods - REVIVAL

@cabch : created a new branch with most of your changes.
Only the latest ones (starting from the deletion of the stripe sandbox form) are missing!

@LucasBrazi06 : I think you can merge this first version.
Can you just advise regarding golden rule 14 - Avoid tree of i18n in json documents?

@LucasBrazi06 : I think you can merge this first version.
Can you just advise regarding golden rule 14 - Avoid tree of i18n in json documents?

I don't want that:
  "asset": {
    "title": "Asset",
    "description": "Manage assets (buildings, solar panels, batteries)",
    "tabs": {
      "assets": "Assets",
      "in_error": "In Error"
    },

I prefer tabs_assets, tabs_in_error...
When I do a search, I should find only one occurence if possible.

@LucasBrazi06 : I'm not allowed anymore to approve this one because I did some changes!!!!

@LucasBrazi06 : I'm not allowed anymore to approve this one because I did some changes!!!!

I think Caro can approve it :-)

@cabch : I had to merge again with master-qa (after a fix from Jerome) and the Card Creation dialog cannot be closed anymore (clicking on the cross buttion). Can you check?

@LucasBrazi06 : We have some doubt on how to check which user is the current one when creating/deleting a payment method.
Caroline has put some comments on the corresponding lines. Please advise if you think there is a better way to get the 'current user'.
Thanks,
Claude

User id search is window object is a security issue. Using the built in helper to get the logged user is should be enough.

as Claude said I had a doubt on it and let it as is for making it work for now knowing there must be a better way to get it.
"current user" refers to "current user shown in the UI" not to the loggedUser

Hi,
@LucasBrazi06: Based in Jerome feedback, Caroline did some changes on how to get the current user.
I think this is now OK.
We still have some issues in the Payment Method list (like the delete button is enabled also for the default payment method) but with no critical impact .
It would be helpful to have this on master-qa very soon to start validating the complete payment flow.
Thanks

@cabch since you seems to have unusual issue with your linter setup in VSCode, before doing or updating a PR, you can do a npm run eslint:fix to fix most of the formatting issue and see linter errors list;

FYI: i've just pushed some eslint fixes because eslint doesn't work as expected on @cabch machine right now.


Attestation-Service: Add vTPM PCRs to claims in az-snp-vtpm

The vTPM PCRs in the az-snp-vtpm verfier are currently part of the evidence, but they're not added to the claimset. The PCR measurements should be also in the claimset, so we can consider them in OPA policies.

cc @surajssd

@surajssd I think you can pick this up w/o formal assignment. I wouldn't expect people lining up for this.


Downloads are resetting for some reason

I've tried to teach a friend how to use ddrive to download some stuff from our discord server, but it doesn't seem to work for some reason
He boots up the program on Termux and can acess the main page just fine, but whenever he tries to download a file the download progress goes up to around 80% and then the download pauses automatically, and after restarting it, the download goes back to the beggining and starts to download again
Here's a video trying to reproduce the issue:
https://user-images.githubusercontent.com/88287554/182630166-68fc3386-dcbd-45a2-8726-f81070d6f9ce.mp4
I don't think the issue is related to the use on Termux because I've tried to reproduce the same steps on my phone and it works just fine
Weird thing is that my friend is actually able to download any files directly through Discord, said issue does not happens there
If there's more information I should provide, just ask for it
and thanks!!!

Sorry for late reply, This is known issue with google chrome on android. I tried to debug but couldn't figure out the issue. I will keep you posted on this if I find any solution.

This bug has been fixed in version v4.0


Simplified length limited string

I often want constr(max_length=XXX, strip_whitespace=True). Could we add a shortcut for this that worked better with mypy?
E.g. something like
class Foobar(BaseModel):
    name: StrLimit[63]

This would be equivalent to constr(max_length=63, strip_whitespace=True), e.g a field with a max length of 63, no min limit, strip_whitespace enabled.
The name StrLimit is debatable. I think using ConStr might be confusing, but perhaps would make more sense?
We could also allow StrLimit[10, 63] which would be equivalent to constr(min_length=10, max_length=63, strip_whitespace=True). However this would not be very pythonic in that the first argument meaning would change. Also I find myself wanting this very rarely.
thoughts?

You can't really do this with typing in Python, I don't think.  The issue here is that to parametrise ('subscribe') a class for type checkers, it needs to be made generic.  But a generic type can only be parametrised with other types - 63 is an instance of int and would be invalid.  The only exception to this is Annotated.

Perhaps this would be possible/clean with #2129 (working prototype)?
class Foobar(BaseModel):
    name: Annotated[str, MaxLen(63), StripWhitespace()]

@layday StrLimit doesn't need to be a class, it can be a subscribable object.
@samuelcolvin If we want to support both upper and lower bounds, perhaps StrLimit[:63] makes sense as syntax?

I don't think this is necessary any more.


Remove code to probe for 'saxon' script

The code that probed for the 'saxon' script did not actually make
sure that the script is the expected wrapper, even though the comment
claimed it is.  This makes xspec.sh fail when there is a 'saxon' script
in place that does not change the command line argument parsing behavior
like the expected 'saxon' script seems to.  I'm proposing to remove this
altogether and always use Saxon's native command line parsing routines
so that xspec.sh works on more systems.

Thanks for your pull request and sorry for the late reply. I personally use Saxon directly via the java command but other people may use the saxon script and the removal may break someone else's code.
I think @fgeorges is using the saxon script and the EXPath packaging system. @fgeorges, would you be able to reply to @hanshuebner ?

The test:
which saxon > /dev/null 2>&1

should probably read like:
which saxon > /dev/null 2>&1 && saxon --help && saxon --help | grep "EXPath Packaging"

That means "if the saxon command exists, and it supports the option --help, and that option's output contains "EXPath Packaging"," then it is the expected script.

I've opened a new, improved PR #121


Problem with using multiple TPU cores and the WandbLogger

Query from User on Lightning Slack
There seems to be a problem with using multiple TPU cores and the WandbLogger. When I run with a single TPU core everything works fine & the results from my experiment get logged. However, if I then switch to 8 cores everything seems to run ok but nothing gets logged to WandB.

@kaushikb11 I'll give a try and fix it.


Create jump-search.cpp

Assigned Issue Number?
#20
Language Opted?
C++
Number of files added or updated?
1
Name of these files?
Algorithm => Searching => Jump_Search => jump-search.cpp
Checklist

[x] I was assigned for this otherwise my PR will be rejected.
[x] I've read the contribution guidelines.
[x] I've referred the correct issue number.
[x] I've filled up this entire template correctly.

:star2: this repository and support.

@ashima0699 👍🏼


readme comments

Merge @puzrin README updates

I'll continue to improve the README using your changes.


github action testdata might have a issue.

What happened:
There might be several issues.
1 testdata action might not fail because there is no command checking if the PR includes all wasm files, like git diff --exit-code.
2 However, introducing a command like git diff --exit-code might lead to a new problem. wasm files seem to depend on developer's environment. For example, I am using mac environment. Even though I don't change any codes, wasm file I build is different from the file attached in the github repo. If we implement a command like git diff --exit-code, we might need to standardize the development environment.
Please let me know if I misunderstand something.
(Previously I talked this issue with @kazukousen)
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:
github_actions

github action testdata might have a issue

@Gekko0114 Can you investigate if it does have or not when get a chance?

/remove-lifecycle rotten
/assign


Validate South Korea land numbers

In South Korean land numbers could have 7 or 8 digits.
Here you can see valid patterns for land numbers:

https://www.numberingplans.com/?page=plans&sub=phonenr&alpha_2_input=KR&current_page=10

Coverage increased (+0.0007%) to 99.4% when pulling a3f8121c14f1cc642b8405d3651e3fe99596e326 on jhbabon:validate-south-korean-land-numbers into 7a4722c67fdd98cd0973e7b673cc540ee6dd9cd8 on floere:master.

Coverage increased (+0.0007%) to 99.4% when pulling a3f8121c14f1cc642b8405d3651e3fe99596e326 on jhbabon:validate-south-korean-land-numbers into 7a4722c67fdd98cd0973e7b673cc540ee6dd9cd8 on floere:master.

Coverage increased (+0.0007%) to 99.4% when pulling a3f8121c14f1cc642b8405d3651e3fe99596e326 on jhbabon:validate-south-korean-land-numbers into 7a4722c67fdd98cd0973e7b673cc540ee6dd9cd8 on floere:master.

Coverage increased (+0.0007%) to 99.4% when pulling a3f8121c14f1cc642b8405d3651e3fe99596e326 on jhbabon:validate-south-korean-land-numbers into 7a4722c67fdd98cd0973e7b673cc540ee6dd9cd8 on floere:master.

cc @floere

@jhbabon Thanks for the excellent PR 🎉… and sorry for the long wait 😔Released in 2.16.4 🚀

@floere Don't worry about the delay. Thank you very much for merging this!


Adding testing for deserialization of G1 and G2 points

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

awesome, thanks for the PR 👍


File Manager has an error where the loading bar will automatically complete itself.

For visual reference.

This happens immediately when you select a file, however if you monitor network traffic in task manager you can clearly see when it peaks / finishes and these times do not correlate with the bar. It finishes immediately while the upload seems to take 10 or so seconds.

Can you provide the logs from your browser?

Unable to reproduce.

Same thing happens to me. Using Chrome on Windows 10. Browser console log:
Uncaught DOMException: Failed to execute 'readAsArrayBuffer' on 'FileReader': The object is already busy reading Blobs.
    at x (<redacted>/js/vendor/upload/client.min.js:9:231)
    at Array.<anonymous> (<redacted>/js/vendor/upload/client.min.js:11:449)
    at n.<anonymous> (<redacted>/js/vendor/upload/client.min.js:15:179)
    at n.e.emit (<redacted>/js/vendor/socketio/socket.io.min.js:23:1606)
    at n.onevent (<redacted>/js/vendor/socketio/socket.io.min.js:22:31333)
    at n.onpacket (<redacted>/js/vendor/socketio/socket.io.min.js:22:30955)
    at n.<anonymous> (<redacted>/js/vendor/socketio/socket.io.min.js:23:2238)
    at n.e.emit (<redacted>/js/vendor/socketio/socket.io.min.js:23:1606)
    at n.ondecoded (<redacted>/js/vendor/socketio/socket.io.min.js:21:23673)
    at s.<anonymous> (<redacted>/js/vendor/socketio/socket.io.min.js:23:2238)
x @ client.min.js:9
(anonymous) @ client.min.js:11
(anonymous) @ client.min.js:15
e.emit @ socket.io.min.js:23
n.onevent @ socket.io.min.js:22
n.onpacket @ socket.io.min.js:22
(anonymous) @ socket.io.min.js:23
e.emit @ socket.io.min.js:23
n.ondecoded @ socket.io.min.js:21
(anonymous) @ socket.io.min.js:23
e.emit @ socket.io.min.js:21
s.add @ socket.io.min.js:21
n.ondata @ socket.io.min.js:21
(anonymous) @ socket.io.min.js:23
e.emit @ socket.io.min.js:22
n.onPacket @ socket.io.min.js:22
(anonymous) @ socket.io.min.js:21
e.emit @ socket.io.min.js:22
n.onPacket @ socket.io.min.js:22
n.onData @ socket.io.min.js:22
ws.onmessage @ socket.io.min.js:22

Reported to plugin repo at https://github.com/vote539/socketio-file-upload/issues/63
This might be due to the fact that we're on 0.5.0 and the latest release is 0.5.2 which included changes to blob handling in vote539/socketio-file-upload#60

Same for Google Chrome (stable) 55.0.2883.87 on Ubuntu 16.04.1 (64 bit)

since this have been reopend i comment here
ive had issue with filemanagers green bar with my when i upload a file it dose nothing it make empty file with right name but no green bar nothing file empty and is 0kb in size when file uploading is 1.5mb in size
on google chrome on windows server 2016

Happens on 0.5.7

This is all with these stable version correct? Not with the prerelease versions.

Correct Stable Version 0.5.7


New version: DataFrames v0.19.2

Registering package: DataFrames
Repository: https://github.com/JuliaData/DataFrames.jl
Created by: @bkamins
Version: v0.19.2
Commit: d3412e94159213d61b7851375c44fc47cd7d053f
Reviewed by: @bkamins
Reference: https://github.com/JuliaData/DataFrames.jl/issues/1913

@fredrikekre It seems that the build failed due to time-out in trying to access the server. Can I re-run the CI myself? Thank you.

Hopefully that kicked off the CI again.

Thank you!


Update gulp

I also noticed that gulp-webserver hasn’t been updated in a year so maybe we need a newer solution for that?

@varffs webserver is fine, doesnt do much, it probably doesnt need update.


reflect: cannot call *T methods on addressable Values of type T

I would expect this program to work:
package main

import "reflect"

type T int
func (*T) M() {}

func main() {
        var t T
        v := reflect.ValueOf(&t).Elem()
        v.MethodByName("M").Call(nil)
}

Currently, it produces panic: reflect: call of reflect.Value.Call on zero Value panic, because M is not in the method set of T (only *T).
This seems overly strict to me. The Go spec allows calling t.M() where t is an addressable value of type T; it's just implicitly executed as (&t).M(). I would expect package reflect to handle this implicit dereference, but it does not.
For comparison, the spec also allows an implicit dereference to call value-receiver methods on pointer types, and package reflect does perform this implicit dereference.
/cc @ianlancetaylor @dsnet

This appears to be the root cause of #24153:
// If v is a named type and is addressable,
// start with its address, so that if the type has pointer methods,
// we find them.
if v.Kind() != reflect.Ptr && v.Type().Name() != "" && v.CanAddr() {
	v = v.Addr()
}

The code needs to explicitly indirect values to get their full method set, and then needs to undo that operation, which then conflicts with the fix implemented for #22031.

I have wanted reflect to perform this implicitly. However, I have a suspicion that some things will break that were depending on the previous behavior.

@dsnet Yeah, I'm somewhat nervous about breakage too. @ianlancetaylor argued though that types generally don't have both value and pointer methods, so perhaps in practice this wouldn't actually come up very often.
One way forward would be to prepare a CL that changes package reflect to include the implicit indirection, and see how much (if anything) breaks.

If you craft the CL, I can run it through regression testing to measure the damage (if any).
As for https://golang.org/cl/97796, even though fixing this issue would avoid the need for CL/97796, should we still move forward with as it fixes the Go1.10 regression? We certainly won't be able to cherry-pick this issue into the 1.10 branch.

@dsnet Yeah, I think CL 97796 is still worth separately pursuing until we make a decision here.


Make DataTable vertically resizeable

Support vertically resizable components with resizableHeightProp
Make DataGrid component resizable

https://user-images.githubusercontent.com/10789765/181601843-ea00c7ef-c9b0-4d68-8a10-8d02c747782d.mov
(cursor while dragging is wrong in the video but fixed already)

Nice 👌


⚠ Bofrost.lu has degraded performance

In c4db2a0, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2256 ms

Resolved: Bofrost.lu performance has improved in c160012 after 29 minutes.