password generator

This is a simple password generator project using Python.

Developer's checklist

[ ] My PR follows the style guidelines of this project
[ ] I have performed a self-check on my work

If changes are made in the code:

[ ] I have followed the coding guidelines
[ ] My changes in code generate no new warnings
[ ] My changes are breaking another fix/feature of the project
[ ] I have added test cases to show that my feature works
[ ] I have added relevant screenshots in my PR

Hey @vishal14072002
Thanks for raising the issue. Seems like you have missed some important steps before sending a PR.

You first have raise a Issue on what changes you want to the code.
Secondly proceed with these guidelines https://github.com/Abbhiishek/Python-Resources/blob/main/CONTRIBUTING.md
For now i will close this PR, you raise a issue explaining  you want to add this functionality.
Happy Contribution ✨


Use the real path for resolving dependencies

Node uses the real path of modules (except when executed with --preserve-symlinks) to resolve dependencies. That is why pnpm's symlinked node_modules work fine with Node applications.
However, resolve is a really popular and widely used package and it does not use the real path of modules to resolve dependencies. As a consequence, a lot of tooling fails with pnpm. See the most recent issue: https://github.com/pnpm/pnpm/issues/857. And also many of these might be related: https://github.com/pnpm/pnpm/issues/801
Would you accept a pull request to change the behavior of resolve to work the way Node does? It won't change anything for projects that use npm/Yarn because they don't leverage symlinks but it would be a huge help for the pnpm community!

Sure, that sounds good - let's ensure that a) existing tests keep passing (ofc), b) we add a "preserve symlinks" option, so users can control the behavior if they like, and c) let's think carefully about if it's possible to avoid a semver-major.

@zkochan Did you forget to change this condition when resolving real path for basedir ?
https://github.com/browserify/resolve/blob/fed3f0fb1eee4e44655b572dc183b6eef8bf44e1/lib/sync.js#L29
E.g. this line may fail require('resolve').sync('../../path/to/another/package', { basedir: '/path/to/a/symlink' }).
FYI @ljharb

@malash could you file a separate issue for that, with a test case?

I'v create a new issue with bad case example https://github.com/browserify/resolve/issues/177


Update docs related to Onboarding

Update docs related to Onboarding
:recycle: Current situation & Problem
Outdated docs for the new release of the SpeziOnboarding module.
:gear: Release Notes

Adjusted docs for the newest release of the SpeziOnboarding module.

:books: Documentation
Only merge when StanfordSpezi/SpeziOnboarding#26 is merged.
:white_check_mark: Testing
--
:pencil: Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

[x] I agree to follow the Code of Conduct and Contributing Guidelines.

@PSchmiedmayer Ready for review after merging https://github.com/StanfordSpezi/SpeziOnboarding/pull/26 🚀


Fire when any file under a directory changes

Hi there! We're using this plugin, which we really like, and running a monorepo, so we have several modules under a single root:

a

src

main

scala

b

src

main

scala

c

src

main

scala

etc. We'd like to trigger separate processes whenever any file under a, or b, or c changes. Our directory trees could be 12 or so levels deep (not one of the things I like about the JVM but it is what it is) and setting our paths as

a/*
a/*/*
...
a/*/*/*/*/*/*/*/*/*/*/*/*

gets boring quite quickly. :) I looked in the globbing library you're using and it looks like it should support a/** but that didn't work for me. Do you have any suggestions to make this easier to accomplish?

Hey! The default built-in globbing in Golang does not support **, but fortunately we already have a resolution to this problem thanks to @benweissmann who submitted https://github.com/telia-oss/github-pr-resource/pull/59 which I think does exactly what you want 🎉
It's currently available under the dev tag here:
https://hub.docker.com/r/teliaoss/github-pr-resource/tags/
Would be great if you could try it out and report back 👍

This worked great, thanks so much - deeply appreciate the fast response. 💯


add types

Description
Linked issue: #4587
Problem

Changes

Based on @types/react-datepicker, incorporating changes from v6.3.0

add types (d.ts)
add test
add CI
fix md

Screenshots

To reviewers
Merging and releasing this PR will allow react-datepicker type completion to work without installing @types/react-datepicker.
You will also see the TS icon (painted blue) on npmjs.com.
Contribution checklist

[x] I have followed the contributing guidelines.
[x] I have added sufficient test coverage for my changes.
[x] I have formatted my code with Prettier and checked for linting issues with ESLint for code readability.

Thanks for the review.

My only other suggestion would be to find any instances in the README or elsewhere (e.g., documentation) that involve notes on contributing, and mention that TypeScript types should be updated when making relevant changes. A good example of this is actually your recent PR (https://github.com/Hacker0x01/react-datepicker/pull/4553).

I agree and will edit the md file.

The primary maintainers of this repository will need to have the final say on supporting TypeScript at this level. I think it seems like a beneficial addition to the project, but we will have to wait for their feedback before finalizing this.

I agree with you.
I would be happy if the main maintainer of this repository would understand the advantages and disadvantages of this PR and adopt it.

@yuki0410-dev I'm not an expert, but I understand the benefits of supporting this. Is there any reason why these definitions are also maintained in a separate repo like you linked? I guess that's the same of what we're now adding in this repo.

@martijnrusschen
Thank you for your understanding regarding TS support.
When this PR is released, users of later versions will no longer need to use @types/react-datepicker. However, the current version of @types/react-datepicker must be retained for users of versions prior to that release. (No further updates will be made.)
Initially there is no difference between the two repositories, but each time a parameter such as Props is added in this repository, it will be reflected in the types in this repository, and the difference between the two repositories will become larger and larger.
In this case, we are adding the type usePointerEvent as an example.
https://github.com/Hacker0x01/react-datepicker/pull/4592/commits/363cbb6b6abeb96e15d18f70e91d4353ddfc973a

Thanks, will wait for the recommended updated.

@martijnrusschen
Thank you for your understanding.
I will make the corrections pointed out by linter and add the documentation.

I did CI tweaks, Lint error handling, and md file corrections.

Sorry, but I'm going to have to rethink the idea once and I'll make it a draft.

This is a bold idea, but maybe we should consider adopting TS instead of just merging the repo with .d.ts?
This is almost impossible to do at once, but with a proper community road map, we can do it for all files individually. Moreover, the most important part is the datepicker itself, and docs and even tests can stay written in js for any amount of time

Yes agree, I'm open to it but don't have the time to get this going. Open to suggestions.

What path I see:

create an issue similar to the mui team https://github.com/mui/material-ui/issues/41273
prepare rollup and the rest tools to work with typescript files
from my experience, we should adapt ts from small modules to the bigger ones

I can prepare a draft of the migration issue

@mirus-ua sounds good!

@martijnrusschen here is the draft, let's discuss it inside the issue https://github.com/Hacker0x01/react-datepicker/issues/4700

#4700 is supported so that the type definitions will be output, and this PR will be closed.


Change Screwed Edition to Screwed Engine in screw you text

What feature do you want to get added on the base engine?
this is more better
To test your sight, and reliability, please select the option of what should NOT be requested.
Stage Editor, 6K+ support, and winning icons.

Already done

ok


Incorrect column label color in data grid Show Columns popover when on Dark theme

Describe the bug
A clear and concise description of what the bug is, and the conditions when it occurs.

How to reproduce
Please provide a link to a reproduction scenario using one of the Clarity Stackblitz templates. Reports without a clear reproduction may not be prioritized until one is provided.
https://stackblitz.com/edit/clarity-dark-theme-v013-5p45lh
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.
Versions
App

Angular: [e.g. 6]
Node: [e.g. 8.10.0]
Clarity: [e.g. 0.12.5]

Device:

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

Additional notes
Add any other notes about the problem here.

This looks like a regression bug related to the checkbox control that was recently updated.
Please subscribe to this issue, I'll update here when I know more about the fix for it.

The issue is with class "clr-control-label" added to  which was not there earlier. Not sure if other style changes of "clr-control-label" are required if so we can just remove color property from class.

Thanks @GeekAb - I'll be pushing a pull request to address the root cause of the issue soon.


How to use local, cloned repo or a forked project?

I am confused about the statement If the path is not a canonical import path, but rather a local path (starts with a dot . or a dash /), xgo will use the local GOPATH contents for the cross compilation. in README.md.
I have forked a Git repo and am providing the path to the xgo command, which causes the build to fail as the import path inside go files need to be changed everywhere.
To avoid this, what I ususally do is to clone the repo from my fork to the name is was supposed to have originally. For example, I would clone github.com/my-user/library to github.com/original-user/library and then build as usual.
xgo does not provide a way to do that. As a workaround, it does provide a way to build from local repository information, but I am unable to work with this.
Error:
Checking for required docker image karalabe/xgo-latest... found.
Cross compiling ....
Building locally ....
Compiling for darwin-10.6/amd64...
can't load package: package .: no Go files in /
2019/02/11 13:23:07 Failed to cross compile package: exit status 1.

No matter which directory path I specify, it fails with a similar error. It would help if we had some examples or for the documentation to be expanded.

Thank's for your answer. Kinda hacky. Think I'm going to wait for a fix :)


Implement special register access mode to write to AVR protected registers

The newer AVR Attiny chips contain a mechanism Microchip calls CCP: Configuration Change Protection.
There are a few special registers that configure system-level peripherals like the clock- or the flash-controller which should not be modified by accident. To prevent that, a special unlock byte has to be written to the CCP unlock register and then the protected register has to be written within the next four executed instructions. See chapter 8.5.7 on page 56 on the linked datasheet for more details.
I tried doing this using the regular register access mechanisms, but the rust compiler always reordered and optimized code so that this didn't work out. Even if it would have, it wouldn't be exactly reliable.
This patch adds a new Target type for AVRs and if it's selected the new file generic_avr_ccp.rs is emitted when generating code. This file contains a bunch of traits to define an unlock register, protected registers and blanket implementations that implement protected writes in case a register implements the aforementioned marker traits.
This has to be used in conjunction with atdf2svd to get an SVD. After generating the pac, you still need to manually define the list of protected registers like this:
pub use crate::generic::ProtectedWritable;

#[cfg(feature = "attiny817")]
pub mod attiny817 {
    use crate::generic::{UnlockRegister, Protected};

    // Mark the CPU.CCP register with the UnlockRegister trait so that it can be used to unlock the below defined registers
    impl UnlockRegister for crate::attiny817::cpu::ccp::CCP_SPEC { const PTR: *mut u8 = 0x34 as *mut u8; }

    // Configuration change protected registers in NVMCTRL
    impl Protected for crate::attiny817::nvmctrl::ctrla::CTRLA_SPEC { const MAGIC: u8 = 0x9D; type CcpReg = crate::attiny817::cpu::ccp::CCP_SPEC; }
    impl Protected for crate::attiny817::nvmctrl::ctrlb::CTRLB_SPEC { const MAGIC: u8 = 0xD8; type CcpReg = crate::attiny817::cpu::ccp::CCP_SPEC; }

    // Configuration change protected registers in CLKCTRL
    impl Protected for crate::attiny817::clkctrl::mclkctrlb::MCLKCTRLB_SPEC { const MAGIC: u8 = 0xD8; type CcpReg = crate::attiny817::cpu::ccp::CCP_SPEC; }
    impl Protected for crate::attiny817::clkctrl::mclklock::MCLKLOCK_SPEC { const MAGIC: u8 = 0xD8; type CcpReg = crate::attiny817::cpu::ccp::CCP_SPEC; }
    impl Protected for crate::attiny817::clkctrl::xosc32kctrla::XOSC32KCTRLA_SPEC { const MAGIC: u8 = 0xD8; type CcpReg = crate::attiny817::cpu::ccp::CCP_SPEC; }
    impl Protected for crate::attiny817::clkctrl::mclkctrla::MCLKCTRLA_SPEC { const MAGIC: u8 = 0xD8; type CcpReg = crate::attiny817::cpu::ccp::CCP_SPEC; }
    impl Protected for crate::attiny817::clkctrl::osc20mctrla::OSC20MCTRLA_SPEC { const MAGIC: u8 = 0xD8; type CcpReg = crate::attiny817::cpu::ccp::CCP_SPEC; }
    impl Protected for crate::attiny817::clkctrl::osc20mcaliba::OSC20MCALIBA_SPEC { const MAGIC: u8 = 0xD8; type CcpReg = crate::attiny817::cpu::ccp::CCP_SPEC; }
    impl Protected for crate::attiny817::clkctrl::osc20mcalibb::OSC20MCALIBB_SPEC { const MAGIC: u8 = 0xD8; type CcpReg = crate::attiny817::cpu::ccp::CCP_SPEC; }
    impl Protected for crate::attiny817::clkctrl::osc32kctrla::OSC32KCTRLA_SPEC { const MAGIC: u8 = 0xD8; type CcpReg = crate::attiny817::cpu::ccp::CCP_SPEC; }

    // Configuration change protected registers in RSTCTRL
    impl Protected for crate::attiny817::rstctrl::swrr::SWRR_SPEC { const MAGIC: u8 = 0xD8; type CcpReg = crate::attiny817::cpu::ccp::CCP_SPEC; }

    // Configuration change protected registers in CPUINT
    impl Protected for crate::attiny817::cpuint::ctrla::CTRLA_SPEC { const MAGIC: u8 = 0xD8; type CcpReg = crate::attiny817::cpu::ccp::CCP_SPEC; }

    // Configuration change protected registers in BOD
    impl Protected for crate::attiny817::bod::ctrla::CTRLA_SPEC { const MAGIC: u8 = 0xD8; type CcpReg = crate::attiny817::cpu::ccp::CCP_SPEC; }

    // Configuration change protected registers in WDT
    impl Protected for crate::attiny817::wdt::ctrla::CTRLA_SPEC { const MAGIC: u8 = 0xD8; type CcpReg = crate::attiny817::cpu::ccp::CCP_SPEC; }
    impl Protected for crate::attiny817::wdt::status::STATUS_SPEC { const MAGIC: u8 = 0xD8; type CcpReg = crate::attiny817::cpu::ccp::CCP_SPEC; }

    // Configuration change protected registers in TCD0
    impl Protected for crate::attiny817::tcd0::faultctrl::FAULTCTRL_SPEC { const MAGIC: u8 = 0xD8; type CcpReg = crate::attiny817::cpu::ccp::CCP_SPEC; }
}

I have not found a way yet to figure out the information that a register is protected by using the ATDF/SVD.
There will be a follow-up PR on avr-device which uses svd2rust to generate PACs for all kinds of AVR devices. I added support for the attiny817 which depends on this new CCP register access code.
As soon as I submitted that PR, I'll cross-reference it.
I am open to suggestions on how to make the code nicer. Especially the parts written in assembler. It was very finicky to make this work and I am still not 100% satisfied with how it looks. I tried getting rid of the ldi instruction, but couldn't make it work.

I have not found a way yet to figure out the information that a register is protected by using the ATDF/SVD.

If ATDF contains this information, svd2rust could use atdf2svd as library dependency and take it directly.

I just checked again. There is a key link missing.
The CCP register is defined and the two magic values that are required to be written into it to either unlock an IOREG or perform a flash memory write.
However, if we look at the register description for an IOREG-protected register like CLKCTRL.MCLKCTRLB, I don't see mentioned that it is actually IOREG protected.
So I think I was wrong. I have this stuff laying around for a while now and must've forgotten that this link is missing.


Update edge-secured-core-devices.md

update links to Asus devices, change from Asus to Intel based on acquired NUC business

@dimatx - can you update the author value to your GitHub alias? then add #sign-off  again? thanks

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

#sign-off

Hi @dimatx,

Microsoft authors of articles need to make updates through the private repository, so the updates can be staged and validated by current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.
It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:
Public repo URL: https://github.com/MicrosoftDocs/{docs-repo}  Private repo URL: https://github.com/MicrosoftDocs/{docs-repo-pr}
We have closed this pull request. Please resubmit the updates through the private repo. For more information, see the contributor guide.


.Net core2.0 console application

When i tried doing same with .Netcore console Application it is throwing a runtime exception as follows
Unhandled Exception: System.NullReferenceException: Object reference not set to an instance of an object.
at Microsoft.Owin.Hosting.Utilities.SettingsLoader.FromConfigImplementation..ctor()
at Microsoft.Owin.Hosting.Utilities.SettingsLoader.b__0()
at System.Threading.LazyInitializer.EnsureInitializedCore[T](T& target, Func1 valueFactory)    at Microsoft.Owin.Hosting.Utilities.SettingsLoader.LoadFromConfig(IDictionary2 settings)
at Microsoft.Owin.Hosting.Engine.StartContext..ctor(StartOptions options)
at Microsoft.Owin.Hosting.Starter.DirectHostingStarter.Start(StartOptions options)
at Microsoft.Owin.Hosting.Starter.HostingStarter.Start(StartOptions options)
at OwinSelfhostSample.Program.Main() in C:\Users\BOMBIE\source\repos\OwinSelfhostSample\OwinSelfhostSample\Program.cs:line 14
What should i do, i have searched about it a lot but couldn't find the solution.

Hello @Arun17Gupta ... This is the documentation repo, and only a handful of doc authors are monitoring the issues here. You should be able to get help with that error on a support forum, such as Stack Overflow.

You tried using katana in a .net core console app and it failed? Katana isn’t supported on.net core and there are no plans to port to .net Framework.

I want to Selfhost my .Net core console application. What should I do?

Use ASP.NET Core on .NET Core

Thank you, I think that will work.

@guardrex
Since the issue has been clarified by @davidfowl , we should close this issue.


[BUG] /personal-data - Wording for validation errors is grammatically incorrect.

Describe the bug
The validation errors for all text fields on /personal-data are grammatically incorrect. The validation errors seem to be automatically generated, and use the field titles in order to specify to the user what needs to be filled out. This becomes an issue with the address fields, because the individual text boxes do not have titles. The validation messages for all address fields say "address" instead of the information that needs to be entered (ex. city, state, etc.).
Currently:
You need to fill your [field title]
Suggested wording:
Please enter your [field label]
URL: https://reach4help-staging-web-client.firebaseapp.com/personal-data
To Reproduce
Steps to reproduce the behavior:

Reach the /personal-data page.
Erase auto-filled information in Name and Display name text fields.
Without filling in any of the required information, click "Continue"
Notice that the validation errors that appear under the text fields are all formatted the same, and are grammatically incorrect.

Expected behavior
Validation messages should be concise and use proper grammar.
Screenshots

Desktop (please complete the following information):
Windows 10 - Chrome - Incognito mode.

ZenHub Information
This information is updated automatically. To modify it, please use ZenHub.

Fixed and deployed to https://app.reach4help.org/


[Feature Request] (area), and (area 2D), no layers.

The forum here (http://www.godotengine.org/forum/viewtopic.php?f=11&t=1911&p=9138#p9138), as a result of the conversation found out that (area 2D) and (area) do not have the properties of layers it is so obvious that they are would be there in demand.

good point.. wonder why...
On Fri, May 1, 2015 at 1:59 PM, OlexiyKravchuk<EMAIL_ADDRESS>wrote:

The forum here (
http://www.godotengine.org/forum/viewtopic.php?f=11&t=1911&p=9138#p9138),
as a result of the conversation found out that (area 2D) and (Zone) do not
have the properties of layers it is so obvious that they are would be there
in demand.
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/1771.

will have to add
On Fri, May 1, 2015 at 2:00 PM, Juan Linietsky<EMAIL_ADDRESS>wrote:

good point.. wonder why...
On Fri, May 1, 2015 at 1:59 PM, OlexiyKravchuk<EMAIL_ADDRESS>wrote:

The forum here (
http://www.godotengine.org/forum/viewtopic.php?f=11&t=1911&p=9138#p9138),
as a result of the conversation found out that (area 2D) and (Zone) do not
have the properties of layers it is so obvious that they are would be there
in demand.
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/1771.

added, closing


AttributeError: 'DataFrame' object has no attribute 'dtype'

Enviroment:
Python 3.10.11 (main, May 15 2023, 19:29:30) [Clang 14.0.6 ]
Type 'copyright', 'credits' or 'license' for more information
IPython 8.12.0 -- An enhanced Interactive Python. Type '?' for help.
pandas                   2.0.2
Function:
Create a Power BI report from your data
PBI_visualize = QuickVisualize(get_dataset_config(df_pbi_), auth=interactive_auth) # auth=device_auth
Render the new report
PBI_visualize
Error:
AttributeError: 'DataFrame' object has no attribute 'dtype'

@sagazluiz ,
It's a generic error message to investigate the possible reason. Plese check the Considerations and Limitations for the data frame passing to the function.
If the issue still exists, please provide more information.


Tag definitions as "new", "updated", "deleted".

In the locust summary, it should be clear to viewers which functions/classes have been added for the first time in the code base, which ones are merely updated, and which were removed.

For new vs. updated, if changed_lines == total_lines, the definition is new. Otherwise, it is merely updated.
deleted entries is a little bit more complicated, as we are currently only analyzing files in the terminal_ref. The only role of the initial_ref is to generate the git diff.


macos binary codesign script should sign old bucket

Until the stable release of Flutter downloads cached binaries from the new storage bucket, the codesign script should codesign binaries from both buckets.

fixed in https://github.com/christopherfujino/codesign.py/commit/c4acf7151ff673d9dbd63d84e7ac1d102ac188e2


Make incsearch CTRL-L work for final character in file

When searching with incsearch set, CTRL-L adds the next character to the search, except for the final character in the file, when there hasn't ever been anything after it.
Invoke a clean Vim with incsearch:
% vim -u NONE -c "set incsearch"

Then press a and type:
abcd
efgh
ijkl

Press Esc immediately after the l (don't press Enter at the end, nor type and then erase any other character).
Then start a seach with /i and press CTRL-L; Vim will add the j into the search, turning it into /ij. Press CTRL-L again; Vim adds the k, making /ijk.
But press CTRL-L a third time, and Vim doesn't do anything: the l doesn't get added into the search.
Now add and remove a blank line at the bottom, with o, Esc, dd, and repeat the search. This time pressing CTRL-L three times does yield the expected /ijkl.
Discovered with version 8.1 including patches 1-490, on Ubuntu Linux.

Hi,
Could you please confirm the following patch?
diff --git a/src/ex_getln.c b/src/ex_getln.c
index 31e646dd3..b5c36f734 100644
--- a/src/ex_getln.c
+++ b/src/ex_getln.c
@@ -747,26 +747,22 @@ may_add_char_to_search(int firstc, int *c, incsearch_state_T *is_state)
     if (is_state->did_incsearch)
     {
 	curwin->w_cursor = is_state->match_end;
-	if (!EQUAL_POS(curwin->w_cursor, is_state->search_start))
+	*c = gchar_cursor();
+	if (*c != NUL)
 	{
-	    *c = gchar_cursor();
-
 	    // If 'ignorecase' and 'smartcase' are set and the
 	    // command line has no uppercase characters, convert
 	    // the character to lowercase.
 	    if (p_ic && p_scs && !pat_has_uppercase(ccline.cmdbuff + skiplen))
 		*c = MB_TOLOWER(*c);
-	    if (*c != NUL)
+	    if (*c == firstc || vim_strchr((char_u *)(
+				p_magic ? "\\~^$.*[" : "\\^$"), *c) != NULL)
 	    {
-		if (*c == firstc || vim_strchr((char_u *)(
-				   p_magic ? "\\~^$.*[" : "\\^$"), *c) != NULL)
-		{
-		    // put a backslash before special characters
-		    stuffcharReadbuff(*c);
-		    *c = '\\';
-		}
-		return FAIL;
+		// put a backslash before special characters
+		stuffcharReadbuff(*c);
+		*c = '\\';
 	    }
+	    return FAIL;
 	}
     }
     return OK;

--
Best regards,
Hirohito Higashi (h_east)

Could you please confirm the following patch?

Thanks. I shall try to take a look, but I have never compiled Vim before so will have to look up how to do that, and it may be a while till I get back to you.

Thank you for fixing, and sorry for not confirming this fix sooner.


Users/anjanich/tg2

Added sample for TaskGroup.

@chandan-anjani,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


add check script for reflector

@cleverhu https://github.com/clusterpedia-io/clusterpedia/pull/210 has been merged, you could rebase code.

/retest

/LGTM


Add<EMAIL_ADDRESS>article

Shall we release the article or you guys want to do more changes? @benStre @unyt-org/admin

Shall we release the article or you guys want to do more changes? @benStre @unyt-org/admin

Looks good to me. Go!


Deleted identifier packages

Closes #18

Pull Request Test Coverage Report for Build 76

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 73:
0.0%

Covered Lines:
524

Relevant Lines:
524

💛  - Coveralls

Si, pero por ahora soy el único.


Added data-gs-id to item attributes list

Coverage remained the same at 70.525% when pulling ba41f9a885c83d99a6d55013e97760dec53d1609 on luckylooke:patch-3 into fa4f2d0d076120f380f475eb71cbc585c4e4197b on troolee:develop.


x:expect/@test is evaluated in confusing context, if result is not wrapped in document node

do-nothing.xsl
<xsl:stylesheet version="2.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform" />

test.xspec
<x:description stylesheet="do-nothing.xsl" xmlns:x="http://www.jenitennison.com/xslt/xspec">
	<x:scenario label="When the transformation result is an empty sequence">
		<x:context />
		<x:expect label="result is empty" select="()" />
		<x:expect label="no node" select="()" test="node()" />
	</x:scenario>

	<x:scenario label="When the transformation result is a sequence of atomic values">
		<x:call function="one-or-more">
			<x:param select="1, 2" />
		</x:call>
		<x:expect label="result is the sequence of values" select="1, 2" />
		<x:expect label="no node" select="()" test="node()" />
	</x:scenario>
</x:description>

You may expect all the x:expect to result in Success. But actually no node ones result in Failure.
The report says that the outermost node of the test.xspec file is the result:

This is by design as of now, because the result is not wrapped in a document node and thus you can't rely on the context in @test. But whatever the architectural design is, the behavior is confusing.

For the record, XSpec 0.3 behaves in the same way.

I think x:expect/@test is one of the pain points of XSpec and will be mitigated only very partially (#174) in the near future. Wiki would need to be clearer about @test. And maybe it should advocate $x:result a bit more strongly.

#437 fixes the behavior.
Wiki still needs to be updated:

Wiki would need to be clearer about @test. And maybe it should advocate $x:result a bit more strongly.

Added a short sentence in Wiki. It still isn't clearer, but I think it's better than nothing.


Getting sporadic SocketExceptions/ExtendedSocketExceptions / Connection Refused using Docker

Getting the below exception every now and then, any suggestions?
Versions:
ASP.Net Core 2.1.3
Npgsql... <IP_ADDRESS>
Docker-compose 3.4
Docker-compose:
postgres_image:
    image: mdillon/postgis
    restart: always
    volumes:
      - pgdata:/var/lib/postgresql/data
      - ./postgres/:/docker-entrypoint-initdb.d/
    ports:
      - "5432"

Exceptions:
First Exception:
Type: ExtendedSocketException
An error occurred using the connection to database 'db' on server 'tcp://postgres_image:5432'.

Following Exception:
An exception occurred in the database while iterating the results of a query for context type 'ApplicationDbContext'.
System.Net.Internals.SocketExceptionFactory+ExtendedSocketException (111): Connection refused <IP_ADDRESS>:5432
at System.Net.Sockets.Socket.EndConnect(IAsyncResult asyncResult)
at System.Threading.Tasks.TaskFactory`1.FromAsyncCoreLogic(IAsyncResult iar, Func`2 endFunction, Action`1 endAction, Task`1 promise, Boolean requiresSynchronization)
--- End of stack trace from previous location where exception was thrown ---
at Npgsql.TaskExtensions.WithCancellation(Task task, CancellationToken cancellationToken)
at Npgsql.TaskExtensions.WithTimeout(Task task, NpgsqlTimeout timeout)
at Npgsql.NpgsqlConnector.ConnectAsync(NpgsqlTimeout timeout, CancellationToken cancellationToken)
at Npgsql.NpgsqlConnector.RawOpen(NpgsqlTimeout timeout, Boolean async, CancellationToken cancellationToken)
at Npgsql.NpgsqlConnector.Open(NpgsqlTimeout timeout, Boolean async, CancellationToken cancellationToken)
at Npgsql.ConnectorPool.AllocateLong(NpgsqlConnection conn, NpgsqlTimeout timeout, Boolean async, CancellationToken cancellationToken)
at Npgsql.NpgsqlConnection.<>c__DisplayClass32_0.<<Open>g__OpenLong|0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at Microsoft.EntityFrameworkCore.Storage.RelationalConnection.OpenDbConnectionAsync(Boolean errorsExpected, CancellationToken cancellationToken)
at Microsoft.EntityFrameworkCore.Storage.RelationalConnection.OpenAsync(CancellationToken cancellationToken, Boolean errorsExpected)
at Microsoft.EntityFrameworkCore.Query.Internal.AsyncQueryingEnumerable`1.AsyncEnumerator.BufferlessMoveNext(DbContext _, Boolean buffer, CancellationToken cancellationToken)
at Npgsql.EntityFrameworkCore.PostgreSQL.Storage.Internal.NpgsqlExecutionStrategy.ExecuteAsync[TState,TResult](TState state, Func`4 operation, Func`4 verifySucceeded, CancellationToken cancellationToken)
at Microsoft.EntityFrameworkCore.Query.Internal.AsyncQueryingEnumerable`1.AsyncEnumerator.MoveNext(CancellationToken cancellationToken)
at System.Linq.AsyncEnumerable.FirstOrDefault_[TSource](IAsyncEnumerable`1 source, CancellationToken cancellationToken)
at Microsoft.EntityFrameworkCore.Query.Internal.AsyncLinqOperatorProvider.TaskResultAsyncEnumerable`1.Enumerator.MoveNext(CancellationToken cancellationToken)
at System.Linq.AsyncEnumerable.SelectEnumerableAsyncIterator`2.MoveNextCore(CancellationToken cancellationToken) at System.Linq.AsyncEnumerable.AsyncIterator`1.MoveNext(CancellationToken cancellationToken)
at Microsoft.EntityFrameworkCore.Query.Internal.AsyncLinqOperatorProvider.ExceptionInterceptor`1.EnumeratorExceptionInterceptor.MoveNext(CancellationToken cancellationToken)
at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.ExecuteSingletonAsyncQuery[TResult](QueryContext queryContext, Func`2 compiledQuery, IDiagnosticsLogger`1 logger, Type contextType)

System.Net.Internals.SocketExceptionFactory+ExtendedSocketException (111): Connection refused <IP_ADDRESS>:5432

If you increase your connection timeout (+30 or so), can you still reproduce this?

I am going to try that and will let you know, it happens sporadically so it will take a few days to establish the result. Thanks much.

This issue seems to be under control now. Will re-open if it happens again. Thanks

If anyone ends up searching for this, check your connection string. I faced a the following exception as well:
Error Message:
 System.Net.Internals.SocketExceptionFactory+ExtendedSocketException : Resource temporarily unavailable
Stack Trace:
   at System.Net.Dns.InternalGetHostByName(String hostName)
   at System.Net.Dns.GetHostAddresses(String hostNameOrAddress)
   at Npgsql.NpgsqlConnector.Connect(NpgsqlTimeout timeout) in C:\projects\npgsql\src\Npgsql\NpgsqlConnector.cs:line 663
   at Npgsql.NpgsqlConnector.RawOpen(NpgsqlTimeout timeout, Boolean async, CancellationToken cancellationToken) in C:\projects\npgsql\src\Npgsql\NpgsqlConnector.cs:line 555
   at Npgsql.NpgsqlConnector.Open(NpgsqlTimeout timeout, Boolean async, CancellationToken cancellationToken) in C:\projects\npgsql\src\Npgsql\NpgsqlConnector.cs:line 414
   at Npgsql.NpgsqlConnection.<>c__DisplayClass32_0.<<Open>g__OpenLong|0>d.MoveNext() in C:\projects\npgsql\src\Npgsql\NpgsqlConnection.cs:line 274

It was because I made a stupid mistake of having SQL Server like connection string Server={host}:{port} instead of the correct Host={host};Port={port} format.

I still see this with the latest version of Npgsql. It happens if the database engine shuts down during the command. Npgsql should wrap it and treat it as a transient error.

@admilazz opened #901 to have connection errors wrapped in NpgsqlException.

Thank you.


[Feature request] Create managed classes for CertAdm.OCSPAdmin COM class

Implement CertAdm.OCSPAdmin into managed .NET code, if possible even PowerShell cmdlets in PSPKI module.
And if possible also implement methods for creating/managing an OCSP array, forcing a controller and force synchronization of its members and of revocation data.
I did not find any such method in the COM classes, only in the MMC snap-in so it would be good to have scripted way of doing it!
What I did on past projects with this COM class is the following:

Change OCSP audit filter
Create/modify several OCSP configurations for various CAs with either automatically retrieved from the CA configuration or manually entered BaseCrlUrls property, with given signing algo, signing flags, and so on...
Test the configurations

Added in 3.7: https://www.pkisolutions.com/apidocs/pkix.net/html/N_SysadminsLV_PKI_Management_CertificateServices.htm


xmap_masked and ymap_maksed

pt0 = (ymap_masked - self.cam_cx) * pt2 / self.cam_fx         pt1 = (xmap_masked - self.cam_cy) * pt2 / self.cam_fy
look likes xmap_masked and ymap_masked in a wrong order?

Don't mind about the naming.


Use span's recordException to record Errors

This follows the recommendations of Open Telemetry and will solve two outstanding issues for us.

Currently stacktraces are not appearing in Cloud Trace.
We need a nice normalized message and stacktrace for the dev ui.

Genkit Trace Store (AFTER)
"status": {
  "code": 2,
  "message": "Error: Got an error!"
 },
"timeEvents": {
  "timeEvent": [
    {
      "time":<PHONE_NUMBER>038.1628,
      "annotation": {
        "attributes": {
          "exception.type": "Error",
          "exception.message": "Got an error!",
          "exception.stacktrace": "Error: Got an error!\n    at /Users/michaeldoyle/repos/src/genkit/js/samples/dev-ui-gallery/lib/main/flows.js:148:23\n    at Context.<anonymous> (/Users/michaeldoyle/repos/src/genkit/js/flow/lib/context.js:71:27)\n    at Generator.next (<anonymous>)\n    at /Users/michaeldoyle/repos/src/genkit/js/flow/lib/context.js:36:61\n    at new Promise (<anonymous>)\n    at __async (/Users/michaeldoyle/repos/src/genkit/js/flow/lib/context.js:20:10)\n    at Context.memoize (/Users/michaeldoyle/repos/src/genkit/js/flow/lib/context.js:67:12)\n    at Context.<anonymous> (/Users/michaeldoyle/repos/src/genkit/js/flow/lib/context.js:105:58)\n    at Generator.next (<anonymous>)\n    at /Users/michaeldoyle/repos/src/genkit/js/flow/lib/context.js:36:61"
        },
        "description": "exception"
      }
    }
  ]
}

Cloud Trace (BEFORE)
Caught Errors

Uncaught Errors

Cloud Trace (AFTER)
Caught Errors

Uncaught Errors

Oh, I just noticed the screenshots... so allows errors to be rendered nicely in Cloud Tracing?

Oh, I just noticed the screenshots... this allows errors to be rendered nicely in Cloud Tracing?

It does. :) I talked to Kman yesterday about recording exceptions and apparently this was the expected behavior, but maybe got lost in the shuffle somewhere. I am going to run this by him just to make sure it matches his expectations.
One thing I am not 💯  on is that telemetry also has a concept of recording exceptions (via logging); not sure if there is any interplay between that and the trace data.


Update MD file on new branch 2017-25-21_21-11-29

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Handle strong_parameters installed as a gem

@ksimmons  thank you for your note. I tested your code and it worked on the rails 3.2 app. Sending a pull request.

Merged. Thanks for helping.


XP strategy is highlighting incorrect recipe

Not sure if they changed the xp rates, but im getting multiple wrong suggested potions when my strategy is set to XP

XP rates were changed as part of today's patch, yeah.
This has the table:
https://secure.runescape.com/m=news/rewards-and-xp-improvements---varlamore-the-rising-darkness

Not sure if it's related to this or separate, but happy to file a separate report if needed -- I'm having a similar issue; a mismatch between the highlighted recipe in the minigame overlay and the table that's highlighted

In this case, the homogenized MMM is highlighted in the overlay but the Alembic is highlighted, which crystallizes the potion. Which one should I do? Homogenize it, or crystallize it? I have an order for both a crystallized MMM and a homogenized MMM.

 Fix was waiting on this most recent update as Jagex has made many changes in the last week to it.

I couldn't wait any longer, need to abuse these xp rates before they get nerfed.
https://github.com/runelite/plugin-hub/pull/6708

omg its perfect now. GOAT


README

User Story
As a Bootcamp student,
I want the prework notes to be structured on a webpage,
So that I can easily find and read the information.
Acceptance Criteria
GIVEN the prework project repository,
WHEN I view the README for the repository,
THEN I can see a professional README that provides information on the project and how to use it

updated the README file in the project.


feat: display message after running mention each command on Discord

Date: 20-09-2024
Developer Name: Rishi Chaubey

Issue Ticket Number : 240

Description
Implemented a feature to display the message "users with example-unverified  roles:" following the execution of the mention each command.

Documentation Updated?

[ ] Yes
[x] No

Under Feature Flag

[ ] Yes
[x] No

Database Changes

[ ] Yes
[x] No

Breaking Changes

[ ] Yes
[x] No

Development Tested?

[x] Yes
[ ] No

Screenshots

Screenshot 1

Test Coverage

Screenshot 1

Additional Notes

here is the issue link for the PR
https://github.com/Real-Dev-Squad/discord-slash-commands/issues/240

Is your feature under the Feature flag?
and, the message says this The users with @developers role are: xyz so what about only a single user with that role

Is your feature under the Feature flag? and, the message says this The users with @developers role are: xyz so what about only a single user with that role

no its not under feature flag

please add Issue link
Please add test coverage

Is your feature under the Feature flag? and, the message says this The users with @developers role are: xyz so what about only a single user with that role

no its not under feature flag

Please put it under feature flag first, once it stays in production for more than 2 weeks without any apparent issue then you can remove it, every new feature goes behind dev flag first

Can you add a video showing if dev flag affecting the outcome? as you are hard coding  the value to true


Uvicorn import paths different than main.py’s, difficulties debugging

(I hope I describe this correctly, it is possible that I have a misunderstanding of how imports are resolved)
Problem: The FastAPI documentation on debugging suggests to call uvicorn directly from main via the IDE/Debugger. However, in this case, fastapi-nano’s imports do not seem to work, as the folder of main.py is the root folder for the module imports with this method of executing the code. So from app.core import auth fails, , from core import auth would work. The imports work when calling uvicorn directly since uvicorn app.main:app (as documented in the readme) seems to make the module resolution uses the folder uvicorn was called from as its root (that is, the project’s main folder with the folder app in it)
Possible solutions:

Assume calling from the /app folder and change paths accordingly
Document another way to debug
Dont fix/not a significant problem

from core import auth comes with its fair share of issues. Can you try python -m main instead of running the main module as a script? Running python main.py puts the path of the nested package to the top of the sys.path which can have other unintended consequences. You can learn more about it here:
https://rednafi.github.io/reflections/how-not-to-run-a-script-in-python.html

Not an issue.


jQuery UI: scrollX true - tbody center

I have a problem with the auto alignment of the table content. The header and content alignment changes diffrently when I change the size of the table.
example: jsFiddle
The thead and tbody are aligned to the left, if I set scrollX to true and maximize the window (this only happens if table content is bigger than table width). The thead alignmend is left but the tbody alignmend changes to center.
the root cause might be found in the css file "dataTables.jqueryui.css"
table.dataTable {
  margin: 0 auto;

it might be caused by the auto setting

It is caused by the assignment of a fixed pixel width to every column. I would suggest using either percentage values (since absolute values don't really make must sense if you want wanted the table to resize horizontally) or leaving one or more columns without a static with assigned so the browser can take up the remaining space.
I take the point that the result is unexpected, but I think the input is also unexpected, so it likely isn't a case that DataTables will cope with. If it is fundamentally important for you, you could remove the margin from your local stylesheet.


Crash when I use JFXPanel

I want to use JavaFX in Compose-jb,  but Compose-jb only support swing, so I use JFXPanel
But,,,,, kotlin.UninitializedPropertyAccessException: lateinit property layout has not been initialized when I use JFXPanel

My Code

    @Composable
    private fun createMainSwingScreen() {
        SwingPanel(
            background = Color.White,
            modifier = Modifier.fillMaxSize(),
            factory = {
                JFXPanel().also { jfxPanel ->
                    PlatformImpl.startup {
                }
            }
        }
    )
}

Error log

    Exception in thread "AWT-EventQueue-0 @coroutine#5" kotlin.UninitializedPropertyAccessException: lateinit property layout has not been initialized
	at androidx.compose.desktop.ComponentInfo.getLayout(SwingPanel.desktop.kt:113)
	at androidx.compose.desktop.SwingPanel_desktopKt$SwingPanel-euL9pac$$inlined$onGloballyPositioned$1.onGloballyPositioned(OnGloballyPositionedModifier.kt:60)
	at androidx.compose.ui.node.LayoutNode.dispatchOnPositionedCallbacks$ui(LayoutNode.kt:1095)
	at androidx.compose.ui.node.OnPositionedDispatcher.dispatchHierarchy(OnPositionedDispatcher.kt:51)
	at androidx.compose.ui.node.OnPositionedDispatcher.dispatchHierarchy(OnPositionedDispatcher.kt:55)
	at androidx.compose.ui.node.OnPositionedDispatcher.dispatch(OnPositionedDispatcher.kt:44)
	at androidx.compose.ui.node.MeasureAndLayoutDelegate.dispatchOnPositionedCallbacks(MeasureAndLayoutDelegate.kt:253)
	at androidx.compose.ui.node.MeasureAndLayoutDelegate.dispatchOnPositionedCallbacks$default(MeasureAndLayoutDelegate.kt:249)
	at androidx.compose.ui.platform.DesktopOwner.measureAndLayout(DesktopOwner.desktop.kt:227)
	at androidx.compose.ui.platform.DesktopOwner.render(DesktopOwner.desktop.kt:207)
	at androidx.compose.ui.platform.DesktopOwners.onFrame(DesktopOwners.desktop.kt:121)
	at androidx.compose.desktop.ComposeLayer$1.onRender(ComposeLayer.desktop.kt:123)
	at org.jetbrains.skiko.SkiaLayer.update$skiko(SkiaLayer.kt:117)
	at org.jetbrains.skiko.redrawer.Direct3DRedrawer.update(Direct3DRedrawer.kt:42)
	at org.jetbrains.skiko.redrawer.Direct3DRedrawer.redrawImmediately(Direct3DRedrawer.kt:37)
	at org.jetbrains.skiko.SkiaLayer$redraw$1.run(SkiaLayer.kt:82)
	at java.desktop/java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:316)
	at java.desktop/java.awt.EventQueue.dispatchEventImpl(EventQueue.java:770)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:721)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:715)
	at java.base/java.security.AccessController.doPrivileged(AccessController.java:391)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:85)
	at java.desktop/java.awt.EventQueue.dispatchEvent(EventQueue.java:740)
	at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:203)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:124)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:113)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:109)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
	at java.desktop/java.awt.WaitDispatchSupport$2.run(WaitDispatchSupport.java:188)
	at java.desktop/java.awt.WaitDispatchSupport$4.run(WaitDispatchSupport.java:235)
	at java.desktop/java.awt.WaitDispatchSupport$4.run(WaitDispatchSupport.java:233)
	at java.base/java.security.AccessController.doPrivileged(AccessController.java:312)
	at java.desktop/java.awt.WaitDispatchSupport.enter(WaitDispatchSupport.java:233)
	at javafx.embed.swing.JFXPanel.initFx(JFXPanel.java:241)
	at javafx.embed.swing.JFXPanel.(JFXPanel.java:267)
	at top.gtf35.nekohelper.ui.file_manager.FileManagerScreen$createMainSwingScreen$1.invoke(FileManagerScreen.kt:92)
	at top.gtf35.nekohelper.ui.file_manager.FileManagerScreen$createMainSwingScreen$1.invoke(FileManagerScreen.kt:91)
	at androidx.compose.desktop.SwingPanel_desktopKt$SwingPanel$4.invoke(SwingPanel.desktop.kt:84)
	at androidx.compose.desktop.SwingPanel_desktopKt$SwingPanel$4.invoke(SwingPanel.desktop.kt:83)
	at androidx.compose.runtime.DisposableEffectImpl.onRemembered(Effects.kt:81)
	at androidx.compose.runtime.ComposerImpl$RememberEventDispatcher.dispatchRememberObservers(Composer.kt:1344)
	at androidx.compose.runtime.ComposerImpl.applyChanges$runtime(Composer.kt:1409)
	at androidx.compose.runtime.CompositionImpl.applyChanges(Composition.kt:414)
	at androidx.compose.runtime.Recomposer.composeInitial$runtime(Recomposer.kt:699)
	at androidx.compose.runtime.CompositionImpl.setContent(Composition.kt:304)
	at androidx.compose.ui.platform.Wrapper_desktopKt.setContent(Wrapper.desktop.kt:39)
	at androidx.compose.desktop.ComposeLayer.initOwner(ComposeLayer.desktop.kt:257)
	at androidx.compose.desktop.ComposeLayer.access$initOwner(ComposeLayer.desktop.kt:50)
	at androidx.compose.desktop.ComposeLayer$Wrapped.init(ComposeLayer.desktop.kt:83)
	at org.jetbrains.skiko.HardwareLayer.checkIsShowing(HardwareLayer.kt:30)
	at org.jetbrains.skiko.HardwareLayer.access$checkIsShowing(HardwareLayer.kt:7)
	at org.jetbrains.skiko.HardwareLayer$1.hierarchyChanged(HardwareLayer.kt:22)
	at java.desktop/java.awt.Component.processHierarchyEvent(Component.java:6781)
	at java.desktop/java.awt.Component.processEvent(Component.java:6400)
	at java.desktop/java.awt.Component.dispatchEventImpl(Component.java:4990)
	at java.desktop/java.awt.Component.dispatchEvent(Component.java:4822)
	at java.desktop/java.awt.Component.createHierarchyEvents(Component.java:5628)
	at java.desktop/java.awt.Container.createHierarchyEvents(Container.java:1466)
	at java.desktop/java.awt.Container.createHierarchyEvents(Container.java:1466)
	at java.desktop/java.awt.Container.createHierarchyEvents(Container.java:1466)
	at java.desktop/java.awt.Container.createHierarchyEvents(Container.java:1466)
	at java.desktop/java.awt.Container.createHierarchyEvents(Container.java:1466)
	at java.desktop/java.awt.Component.show(Component.java:1680)
	at java.desktop/java.awt.Window.show(Window.java:1056)
	at java.desktop/java.awt.Component.show(Component.java:1717)
	at java.desktop/java.awt.Component.setVisible(Component.java:1664)
	at java.desktop/java.awt.Window.setVisible(Window.java:1028)
	at androidx.compose.desktop.ComposeWindow.setVisible(ComposeWindow.desktop.kt:85)
	at androidx.compose.desktop.AppWindow.show(AppWindow.desktop.kt:446)
	at androidx.compose.desktop.AppWindow.show$default(AppWindow.desktop.kt:432)
	at top.gtf35.nekohelper.ui.file_manager.FileManagerScreen.dispatch(FileManagerScreen.kt:78)
	at top.gtf35.nekohelper.ui.file_manager.FileManagerScreen$dispatch$3.invoke(FileManagerScreen.kt)
	at top.gtf35.nekohelper.ui.file_manager.FileManagerScreen$dispatch$3.invoke(FileManagerScreen.kt)
	at androidx.compose.runtime.RecomposeScopeImpl.compose(RecomposeScopeImpl.kt:91)
	at androidx.compose.runtime.ComposerImpl.recomposeToGroupEnd(Composer.kt:2252)
	at androidx.compose.runtime.ComposerImpl.skipCurrentGroup(Composer.kt:2499)
	at androidx.compose.runtime.ComposerImpl.recompose$runtime(Composer.kt:2625)
	at androidx.compose.runtime.CompositionImpl.recompose(Composition.kt:406)
	at androidx.compose.runtime.Recomposer.performRecompose(Recomposer.kt:724)
	at androidx.compose.runtime.Recomposer.access$performRecompose(Recomposer.kt:100)
	at androidx.compose.runtime.Recomposer$runRecomposeAndApplyChanges$2$2.invoke(Recomposer.kt:437)
	at androidx.compose.runtime.Recomposer$runRecomposeAndApplyChanges$2$2.invoke(Recomposer.kt:411)
	at androidx.compose.runtime.BroadcastFrameClock$FrameAwaiter.resume(BroadcastFrameClock.kt:41)
	at androidx.compose.runtime.BroadcastFrameClock.sendFrame(BroadcastFrameClock.kt:62)
	at androidx.compose.ui.platform.DesktopOwners.onFrame(DesktopOwners.desktop.kt:116)
	at androidx.compose.desktop.ComposeLayer$1.onRender(ComposeLayer.desktop.kt:123)
	at org.jetbrains.skiko.SkiaLayer.update$skiko(SkiaLayer.kt:117)
	at org.jetbrains.skiko.redrawer.Direct3DRedrawer.update(Direct3DRedrawer.kt:42)
	at org.jetbrains.skiko.redrawer.Direct3DRedrawer.access$update(Direct3DRedrawer.kt:11)
	at org.jetbrains.skiko.redrawer.Direct3DRedrawer$frameDispatcher$1.invokeSuspend(Direct3DRedrawer.kt:20)
	at org.jetbrains.skiko.redrawer.Direct3DRedrawer$frameDispatcher$1.invoke(Direct3DRedrawer.kt)
	at org.jetbrains.skiko.FrameDispatcher$job$1.invokeSuspend(FrameDispatcher.kt:32)
	at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
	at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106)
	at java.desktop/java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:316)
	at java.desktop/java.awt.EventQueue.dispatchEventImpl(EventQueue.java:770)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:721)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:715)
	at java.base/java.security.AccessController.doPrivileged(AccessController.java:391)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:85)
	at java.desktop/java.awt.EventQueue.dispatchEvent(EventQueue.java:740)
	at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:203)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:124)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:113)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:109)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
	at java.desktop/java.awt.EventDispatchThread.run(EventDispatchThread.java:90)

Hello! It looks like the initialization of the JFXPanel happens before the initialization of the SwingPanel, because both Swing and JavaFX have their own thread to dispatch events. So, if you try the following approach, it should solve your problem:
import androidx.compose.desktop.LocalAppWindow
import androidx.compose.desktop.Window
import androidx.compose.foundation.layout.Box
import androidx.compose.foundation.layout.Column
import androidx.compose.foundation.layout.fillMaxSize
import androidx.compose.foundation.layout.fillMaxWidth
import androidx.compose.foundation.layout.height
import androidx.compose.foundation.layout.padding
import androidx.compose.foundation.layout.Spacer
import androidx.compose.foundation.layout.size
import androidx.compose.material.Button
import androidx.compose.material.Text
import androidx.compose.runtime.Composable
import androidx.compose.runtime.DisposableEffect
import androidx.compose.runtime.mutableStateOf
import androidx.compose.runtime.remember
import androidx.compose.runtime.SideEffect
import androidx.compose.ui.Alignment
import androidx.compose.ui.geometry.Offset
import androidx.compose.ui.graphics.Color
import androidx.compose.ui.layout.Layout
import androidx.compose.ui.layout.onGloballyPositioned
import androidx.compose.ui.Modifier
import androidx.compose.ui.platform.LocalDensity
import androidx.compose.ui.unit.dp
import androidx.compose.ui.unit.IntSize
import androidx.compose.ui.unit.round
import java.awt.Container
import java.awt.BorderLayout
import javafx.embed.swing.JFXPanel
import javafx.application.Platform
import javafx.scene.Group
import javafx.scene.Scene
import javax.swing.JPanel
import javafx.scene.paint.Color as JFXColor
import javafx.scene.text.Font as JFXFont
import javafx.scene.text.Text as JFXText

fun main() = Window(
        title = "MyApp",
        size = IntSize(600, 550)
) {
    // JavaFX components
    val jfxpanel = remember { JFXPanel() }
    val jfxtext = remember { JFXText() }

    // The current container (depending on how you are using the CFD,
    // this could be ComposeWindow or ComposePanel)
    val container = LocalAppWindow.current.window

    val counter = remember { mutableStateOf(0) }
    val inc: () -> Unit = {
        counter.value++
        // update JavaFX text component
        Platform.runLater {
            jfxtext.text = "Welcome JavaFX! ${counter.value}"
        }
    }

    Box(
        modifier = Modifier.fillMaxWidth().height(60.dp).padding(top = 20.dp),
        contentAlignment = Alignment.Center
    ) {
        Text("Counter: ${counter.value}")
    }

    Box(
        modifier = Modifier.fillMaxSize(),
        contentAlignment = Alignment.Center
    ) {
        Column(
            modifier = Modifier.padding(top = 80.dp, bottom = 20.dp)
        ) {
            Button("1. Compose Button: increment", inc)
            Spacer(modifier = Modifier.height(20.dp))
            // The "Box" is strictly necessary to properly sizing and positioning the JFXPanel container.
            Box(
                modifier = Modifier.height(200.dp).fillMaxWidth()
            ) {
                JavaFXPanel(
                    root = container,
                    panel = jfxpanel,
                    // function to initialize JFXPanel, Group, Scene
                    onCreate = {
                        Platform.runLater {
                            val root = Group()
                            val scene = Scene(root, JFXColor.GRAY)
                            jfxtext.x = 40.0
                            jfxtext.y = 40.0
                            jfxtext.font = JFXFont(25.0)
                            jfxtext.text = "Welcome JavaFX! ${counter.value}"
                            root.children.add(jfxtext)
                            jfxpanel.scene = scene
                        }
                    }
                )
            }
            Spacer(modifier = Modifier.height(20.dp))
        }
    }
}

@Composable
fun Button(text: String = "", action: (() -> Unit)? = null) {
    Button(
        modifier = Modifier.size(270.dp, 40.dp),
        onClick = { action?.invoke() }
    ) {
        Text(text)
    }
}

@Composable
public fun JavaFXPanel(
    root: Container,
    panel: JFXPanel,
    onCreate: () -> Unit
) {
    val container = remember { JPanel() }
    val density = LocalDensity.current.density

    Layout(
        content = {},
        modifier = Modifier.onGloballyPositioned { childCoordinates ->
            val coordinates = childCoordinates.parentCoordinates!!
            val location = coordinates.localToWindow(Offset.Zero).round()
            val size = coordinates.size
            container.setBounds(
                (location.x / density).toInt(),
                (location.y / density).toInt(),
                (size.width / density).toInt(),
                (size.height / density).toInt()
            )
            container.validate()
            container.repaint()
        },
        measurePolicy = { _, _ ->
            layout(0, 0) {}
        }
    )

    DisposableEffect(Unit) {
        container.apply {
            setLayout(BorderLayout(0, 0))
            add(panel)
        }
        root.add(container)
        onCreate.invoke()
        onDispose {
            root.remove(container)
        }
    }
}

Looks so coooool!
Running well!
Thank you very much for providing the code that is easy to reuse 💖
Thanks a lot!

Can JavaFXPanel be used in each item in the Compose list?
That will create a lot of JFXPanel().
Is it okay to do this?

I think this is ok, but also I think that if you need to create a list from a large number of JavaFX items, you can use one JavaFXPanel as a container for a JavaFX ListView (or any other suitable component, because I am not very good at JavaFX framework) and fill it with JavaFX components with the proper styling to match your current CFD style.

I see ~, thanks a lot 💕

how do i solve "LocalAppWindow.current.window" in compose 1.0.0-beta5?

@Rsedaikin
Thank you .

@Rsedaikin
Great work with this example - currently using it with a WebView and Mapbox.js - works very well and I'm pleased with the performance.
I have noticed an implicit problem with using the JavaFXPanel.
Scenerio :

Display a JavaFXPanel composable using above code.
Remove the composable (feasible if replacing window content)
DisposableEffect::onDispose called as expected, JPanel which hosts the JFXPanel removed from the ComponentWindow
Display the JavaFXPanel again back in the same application session.

Expected : JFXPanel displayed as before
Result : JFXPanel is not displayed, only the JFrame
Digging deeper :
It appears the first time you add a JavaFXPanel it will attempt to register a PlatformImpl.FinishListener method call : JFXPanel::registerFinishListener.  This is then proxied to PlatformImpl::addListener(finishListener).
So far all good, however when this is called root.remove(container) when from the onDispose lambdais invoked the JPanel will notify the JFXPanel that it has been removed from the window.  This then invokes JFXPane::deregisterFinishListener which proxies  a call to : PlatformImpl.removeListener(finishListener).
Once this method is called it will remove the listener, which makes sense for the lifecycle of this JFXPanel.  However in removing the listener this method is called PlatformImpl::checkIdle.  If the PlatformImpl.FinishListener count hits 0 PlatformImpl::tkExit is called and this exits all event loops.
The end result is the compose application is still running but any calls to Platform::runLater do nothing for the rest of the application session.
Workaround :
Register a PlatformImpl.FinishListener manually for the lifetime of the application to prevent the event loop from exiting when finished listeners are unregistered keeping the count of registered listeners at least 1 example :
fun main() = application(exitProcessOnExit = true) {
    val finishListener = object : PlatformImpl.FinishListener {
        override fun idle(implicitExit: Boolean) {}
        override fun exitCalled() {}
    }
    PlatformImpl.addListener(finishListener)

    Window(
        title = "My App",
        icon = icon,
        resizable = false,
        state = WindowState(
            placement = Floating,
            size = size),
        onCloseRequest = {
            PlatformImpl.removeListener(finishListener)
            exitApplication()
         },
        content = { ..... })
}

It's work when i run the project via IDEA
but after build distributive seems like webview doesn't work at all =( Maybe you know how to fix it?

It's work when i run the project via IDEA but after build distributive seems like webview doesn't work at all =( Maybe you know how to fix it?

Sorry, I've found that I'll have a lot less trouble using swing , so I've given up on JavaFX

It's work when i run the project via IDEA but after build distributive seems like webview doesn't work at all =( Maybe you know how to fix it?

Sorry, I've found that I'll have a lot less trouble using swing , so I've given up on JavaFX

but anyway it will use WEbView from JavaFX, or you know some other webview implementations without javafx ? (i wonna create chat with youtube player or site preview inside, but javaXF webview only one solution what i found... =( )

It's work when i run the project via IDEA but after build distributive seems like webview doesn't work at all =( Maybe you know how to fix it?

Same behavior happens to me. @theone55 Did you find solutions for this?

Hi @aro311 @theone55
It seems that the problem was some ClassNotFoundException. However, Compose simply swallows any exception, which is why your app won't crash and it would simply appear that the WebView didn't load.
To fix that, I followed the tutorial at, Configuring included JDK modules | Native distributions & local execution | JetBrains/compose-jb, and as suggested by that tutorial, I ran the suggestModules gradle task on my project. It suggested that I add modules("java.instrument", "java.net.http", "jdk.jfr", "jdk.jsobject", "jdk.unsupported", "jdk.unsupported.desktop", "jdk.xml.dom") under compose.desktop.application.nativeDistributions.

Demo
Using the provided desktop-template, I made the following modifications to ./build.gradle.kts and ./src/main/kotlin/main.kt, and then I added a new file ./src/main/kotlin/JFXWebView.kt (sources below):
For ./build.gradle.kts:
import org.jetbrains.compose.desktop.application.dsl.TargetFormat

plugins {
    kotlin("jvm")
    id("org.jetbrains.compose")
    id("org.openjfx.javafxplugin") version "0.0.13"
}

repositories {
    mavenCentral()
    maven("https://maven.pkg.jetbrains.space/public/p/compose/dev")
    google()
}

javafx {
    version = "19"
    modules = listOf("javafx.controls", "javafx.swing", "javafx.web")
}

dependencies {
    // Note, if you develop a library, you should use compose.desktop.common.
    // compose.desktop.currentOs should be used in launcher-sourceSet
    // (in a separate module for demo project and in testMain).
    // With compose.desktop.common you will also lose @Preview functionality
    implementation(compose.desktop.currentOs)
}

compose.desktop {
    application {
        mainClass = "MainKt"
        nativeDistributions {
            modules("java.instrument", "java.net.http", "jdk.jfr", "jdk.jsobject", "jdk.unsupported", "jdk.unsupported.desktop", "jdk.xml.dom")
            targetFormats(TargetFormat.Dmg, TargetFormat.Msi, TargetFormat.Deb)
            packageName = "KotlinJvmComposeDesktopApplication"
            packageVersion = "1.0.0"
        }
    }
}

For ./src/main/kotlin/main.kt:
import androidx.compose.foundation.layout.fillMaxSize
import androidx.compose.ui.Modifier
import androidx.compose.ui.awt.SwingPanel
import androidx.compose.ui.window.singleWindowApplication

fun main() = singleWindowApplication {
     SwingPanel(
          factory = { JFXWebView() },
          modifier = Modifier.fillMaxSize(),
     )
}

For ./src/main/kotlin/JFXWebView.kt:
import javafx.application.Platform
import javafx.embed.swing.JFXPanel
import javafx.scene.Scene
import javafx.scene.web.WebView

class JFXWebView : JFXPanel() {
     init {
          Platform.runLater(::initialiseJavaFXScene)
     }

     private fun initialiseJavaFXScene() {
          val webView = WebView()
          val webEngine = webView.engine
          webEngine.load("https://html5test.com/")
          val scene = Scene(webView)
          setScene(scene)
     }
}

Then run ./gradlew packageDistributionForCurrentOS, install the resulting native distribution, and upon running, you should get something like:

P.S. Not sure if we should also worry about firewall settings, which may prevent the JavaFX WebView from also working, but then other parts of your app that connects to the internet might also stop working.

Any idea Apple Sign In shows "Failed to verify your identity. Tray again." with JFX Webview?

This is how i am using webviews, but its not getting disposed properly it opens for the first time but the second time it shows blank screen.

@Composable
fun DesktopWebView(
    modifier: Modifier,
    url: String,
) {
    val jPanel: JPanel = remember { JPanel() }
    val jfxPanel = JFXPanel()

    SwingPanel(
        factory = {
            jfxPanel.apply { buildWebView(url) }
            jPanel.add(jfxPanel)
        },
        modifier = modifier,
    )

    DisposableEffect(url) { onDispose { jPanel.remove(jfxPanel) } }
}

private fun JFXPanel.buildWebView(url: String) {

    Platform.runLater {
        val webView = WebView()
        val webEngine = webView.engine

        // Set the user agent to simulate a browser for YouTube
        webEngine.userAgent =
            "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.110 Safari/537.3"

        // Enable JavaScript support for YouTube embed player
        webEngine.isJavaScriptEnabled = true

Thanks, the solve my probleam @Rsedaikin
And if someone take version hight, maybe find
val container = LocalAppWindow.current.window
compile fail
You can replace to
val container = this.window
Then is run
@ximplia-paolo-pasianot

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


All rational numbers are parsed as Decimals.Decimal instead of Float

I have noticed unexpected behaviour. It might be intended but I haven't found any reference in the documentation.
Expected behaviour: rational numbers like 1.0 and 0.1 will be represented as Float
Actual behaviour: all non-integer numbers are represented as Decimals.Decimal
MWE:
using LibPQ
# mocking up a download with rational numbers
sql="""
select avg(1.0) as average, 1.0 as float_number, 0.0000000001 as really_small_number
from bap_aed.aed_metrics
"""
result = execute(conn,sql);

# you can see that LibPQ creates Decimals.Decimal types
result.column_types
# 3-element Vector{Type}:
#  Decimals.Decimal
#  Decimals.Decimal
#  Decimals.Decimal

Package version (but I've seen the same behaviour on 0.9)
[194296ae] LibPQ v1.14.0
versioninfo() (but it was the same on 1.7)
Julia Version 1.8.0
Commit 5544a0fab76 (2022-08-17 13:38 UTC)
Platform Info:
OS: macOS (arm64-apple-darwin21.3.0)
CPU: 8 × Apple M1 Pro
WORD_SIZE: 64
LIBM: libopenlibm
LLVM: libLLVM-13.0.1 (ORCJIT, apple-m1)
Threads: 6 on 6 virtual cores

Only columns with type numeric/decimal will appear this way; float4/real and float8/double precision will not.
The numeric type is an arbitrary-precision number that cannot be represented fully in a Float64. However, Decimals.Decimal is not a great type, and a replacement (or improvements to that package) would be welcome.
You can either force the type in Julia:
julia> execute(conn, sql; type_map=Dict(:numeric => Float64)).column_types
3-element Vector{Type}:
 Float64
 Float64
 Float64

or in SQL:
julia> sql="""
       select avg(1.0)::float8 as average, 1.0::float8 as float_number, 0.0000000001::float8 as really_small_number;
       """
"select avg(1.0)::float8 as average, 1.0::float8 as float_number, 0.0000000001::float8 as really_small_number;\n"

julia> execute(conn, sql).column_types
3-element Vector{Type}:
 Float64
 Float64
 Float64

psycopg2 in Python does the same thing, using the decimal module in Python's standard library.
https://www.psycopg.org/docs/faq.html#faq-float


[Bug]: NewsRequest classs param:symbols designed to take List[str] for symbols but can only take a single symbol as str

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Error when using NewsRequest with multiple symbols.
from alpaca.data.historical.news import NewsClient
from alpaca.data.requests import NewsRequest

NewsClient().getnews(request_params=NewsRequest(symbols=['AAPL','GOOGL'])

results in error.
Expected Behavior
No response
SDK Version I encountered this issue in
alpaca-py v0.31.0
Steps To Reproduce
ValidationError: 1 validation error for NewsRequest
symbols
  Input should be a valid string [type=string_type, input_value=['AAPL', 'GOOGL'], input_type=list]
    For further information visit https://errors.pydantic.dev/2.6/v/string_type

Filled out the Steps to Reproduce section?

[X] I have entered valid steps to reproduce my issue or have attached a minimally reproducible case in code that shows my issue happening; and understand that without this my issue will be flagged as invalid and closed after 30 days.

Anything else?
No response

https://github.com/hiohiohio/alpaca-py/blob/791ce40d01914ad967ae14a2f11d66449af119d4/alpaca/data/requests.py#L389
symbols (Optional[str]): The comma-separated list of symbols to query news for.

@neili02 the parameter currently does not support list. Please convert to comma separated str.
NewsClient().getnews(request_params=NewsRequest(symbols=",".join(['AAPL','GOOGL']))

Okay, thanks.


Crashes when using small icon preview

Description
Hello, I'm a Japanese Files user.
The text may be incorrect due to the use of translations. note that.
This is the main topic, but I would like to report that there were some parts that did not work properly when I was using it.
When accessing a folder that is about 2GB in size and has 134 items in the small icon layout, it seems to crash while loading.
Steps To Reproduce

Prepare various files in one folder. (From light files such as txt to heavy files such as bmp, mp4, font files, dll files, and zip files.)
2.Create multiple folders within the folder and prepare various similar files within them.
Return to folder "1" and change the layout to small icons.
If it loads normally, close Files and reopen it.
Access folder “1”.
If you try to preview with small icons, it will crash here.
*It may not be possible to reproduce depending on the environment. It may depend on the file and what you can preview.

Requirements
Prevent crashes when accessing folders that are set to display in (but may not be limited to) the small icon layout
Files Version
<IP_ADDRESS>
Windows Version
10.0.19045.3930
Log File
debug.log

Thanks for the report, can you see if this is an issue in the preview version. https://files.community/download

Thank you for your reply.
When I tried the Preview version, it seems that this problem does not occur.
I guess they were in the process of fixing it...I'm sorry I didn't check carefully before posting...
We support the development!

Sorry, I have another problem with the Preview version...
It seems like I can't open the file.
"A device connected to the system is not functioning."
It seems like an error message is displayed...

By the way, you can't open it by double-clicking it, but if you select "Open with" and select Notepad, etc., it will appear.

I'm happy to hear the crashing issue has been resovled.
Regarding the issue with opening text files, does it work if you uninstall/reinstall Files?

When uninstalling, should I also uninstall the stable version?  Or is it okay to uninstall only the Preview version in question?

I reinstalled it and it opened!
For reference, what I did after installing the Preview version and before I encountered the problem of not being able to open files was as follows, but is there something to do with it?

Import settings from stable version
Confirmed the problem that occurred when this topic started (small icon preview crash)
Layout display settings for each folder (download folder, picture folder, etc.)

I'm happy to hear it's been resolved, I'm not really sure what caused it but it's possible that it's a bug in Windows.

That's right!
I was wondering if importing the settings from the stable version was a bad idea, but (when I changed the settings manually without using the import function, I didn't have the problem of not being able to open the file). It's an amateur's idea, so I don't really understand it...lol
Anyway, I solved it successfully!
Thank you for your response!
I will continue to use the Preview version for the time being until there is a stable update.
Good luck with your development!


fix(osx): add support for high sierra

OK I think it made it this time! https://github.com/JamieMason/ImageOptim-CLI/commit/e25bfcefe3d4ae6bd8b3a823f6583228691362f0
Thanks a lot.

Works perfectly 😊


Node.js hook errors with direnv

What happened?
I installed nodejs@20 via devbox and added direnv with devbox generate direnv and now when I cd into that directory I get this error:
direnv: loading PATH/.envrc
direnv: using devbox
PATH/.devbox/gen/scripts/.raw-hooks.sh:1: DEVBOX_COREPACK_ENABLED: unbound variable
direnv: error exit status 1

Steps to reproduce
mkdir devbox-node-repro
cd devbox-node-repro/
devbox init
devbox add nodejs@20
# Info: Adding package "nodejs@20" to devbox.json
#
# nodejs NOTES:
# Devbox automatically configures Corepack for Nodejs when DEVBOX_COREPACK_ENABLED=1. You can install Yarn or Pnpm by adding them to your `package.json` file using # `packageManager`
# Corepack binaries will be installed in your local `.devbox` directory
#
# This plugin creates the following helper files:
#
# To show this information, run `devbox info nodejs`
#
#
devbox generate direnv
# ✓ Computed the Devbox environment.
# Success: generated .envrc file
# Success: ran `direnv allow`
# direnv: loading ~/Documents/devbox-node-repro/.envrc
# direnv: using devbox
# devbox-node-repro/.devbox/gen/scripts/.raw-hooks.sh:1: DEVBOX_COREPACK_ENABLED: unbound variable
# direnv: error exit status 1

Command
add, generate
devbox.json
{
  "$schema":  "https://raw.githubusercontent.com/jetpack-io/devbox/0.10.1/.schema/devbox.schema.json",
  "packages": ["nodejs@20"],
  "shell": {
    "init_hook": [
      "echo 'Welcome to devbox!' > /dev/null"
    ],
    "scripts": {
      "test": [
        "echo \"Error: no test specified\" && exit 1"
      ]
    }
  }
}

Devbox version
0.10.1
Nix version
nix (Nix) 2.20.5
What system does this bug occur on?
macOS (Apple Silicon)
Debug logs
No response

Probably worth noting that I’m using fish shell.

@savil fish 3.7.1 and direnv 2.34.0

Hey @evanleck, I've been trying to reproduce this but I can't get the error to replicate with Fish + Direnv. Can you try the following:

Update to the latest version of Devbox using devbox version update
Delete the contents of your .devbox directory, then regenerate them using devbox shell.

If those don't fix your issue, can you share the contents of the file at devbox-node-repro/.devbox/gen/scripts/.raw-hooks.sh

@Lagoja thanks for looking into the issue.

Done, I'm on 0.10.4 now.
Done.

Unfortunately, that didn't seem to fix the issue. The contents of .raw-hooks.sh is:
test -z $DEVBOX_COREPACK_ENABLED || corepack enable --install-directory "/Users/evan/Code/devbox-node-repro/.devbox/virtenv/nodejs/corepack-bin/"
test -z $DEVBOX_COREPACK_ENABLED || export PATH="/Users/evan/Code/devbox-node-repro/.devbox/virtenv/nodejs/corepack-bin/:$PATH"

I don't see DEVBOX_COREPACK_ENABLED anywhere in the output of env if that's of any help.

FWIW changing the contents of .raw-hooks to use test -v seems to fix the issue, though I'm not 100% sure it's functionally equivalent. Worth noting, test -v was introduced in bash 4.2, so this won't work on versions prior to that.
test -v DEVBOX_COREPACK_ENABLED || corepack enable --install-directory "/Users/evan/Code/devbox-node-repro/.devbox/virtenv/nodejs/corepack-bin/"
test -v DEVBOX_COREPACK_ENABLED || export PATH="/Users/evan/Code/devbox-node-repro/.devbox/virtenv/nodejs/corepack-bin/:$PATH"


Generate Histograms for 2D field

Description
generating a PDF for a 2D field looping over vertical levels
Fixes #453
Contribution checklist
Aim to have all relevant checks ticked off before merging. See the developer's guide for more detail.

[x] Documentation has been updated to reflect change.
[x] New code has tests, and affected old tests have been updated.
[x] All tests and CI checks pass.
[x] Added an entry to the top of docs/source/changelog.rst
[ ] Conda lock files have been updated if dependencies changed.
[x] Marked the PR as ready to review.
[ ] has ensemble capability
[x] AI was used

Old branch head: 2d9ead7edb58ee1338ec2ff506ba324688014746

Note to self: There are a bunch of conflict markers in this branch's code that need cleaning up.

code working and now need to add unit test.

GitHub Copilot used to debugfilename in  assert Path("test_473718.0.png").is_file()

@daflack @jfrost-mo I suggest that James does the technical review first and once that has passed David can do the science review.

@jfrost-mo  @daflack Many thanks for your reviews. I have addressed them all. Examples of line style plots are above in the respective comments. I added a trigger field in rose-meta to allow the user to configure the histogram type. Selection of postage stamps or all members in a single plot is not configurable in rose meta, but instead fixed in the yaml recipe file.

@jfrost-mo over to you to take a final look. Thanks both @daflack and @jfrost-mo for the review :-)

Plotting code changes look good. One of the include files has an unnecessary loop, and more tests need writing for the new plotting functionality.

It has the wrong statement in the loop. IT should read:
{% for model_field in PRESSURE_LEVEL_MODEL_FIELDS %}
Corrected now in 11fe6d7..8344e50

Plotting code changes look good. One of the include files has an unnecessary loop, and more tests need writing for the new plotting functionality.

This has been changed in the code and this version is outdated.

Test coverage look reasonable. There are still a few lines added that are uncovered, but it'll do for now.


Fix services registration, and adding schema util to sonos

Description:
Currently sonos_* services aren't registered if netdisco discovery is used for devices. This PR moves service registration into a def, sets up a global list of all known devices, and tries to invoke service registration as little as necessary. This PR also includes adding schema use to sonos.

Looks good! :dolphin:


Highlighter?

Which languages are supported with the Highlighter? And where do we set which type of Highlighter we use? (I want to highlight C++ and C code but doing ~~~ c++ doesnt work)

Hi @ImReallyShiny,
It uses the default Jekyll syntax highlighter - Rouge. I haven't tried this but if you use cpp it might work.


Redo pyyaml workaround for travis

Codecov Report

Merging #319 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #319      +/-   ##
==========================================
- Coverage   98.17%   98.13%   -0.05%
==========================================
  Files         123      123
  Lines       14159    13841     -318
==========================================
- Hits        13901    13583     -318
  Misses        258      258

Impacted Files
Coverage Δ

stix2/datastore/filesystem.py
92.85% <0%> (-0.12%)
:arrow_down:

stix2/v21/observables.py
97.16% <0%> (-0.02%)
:arrow_down:

stix2/patterns.py
98% <0%> (-0.02%)
:arrow_down:

stix2/test/v21/test_datastore_filesystem.py
99.17% <0%> (-0.02%)
:arrow_down:

stix2/test/v20/test_datastore_filesystem.py
99.19% <0%> (-0.02%)
:arrow_down:

stix2/test/v21/test_relationship.py
100% <0%> (ø)
:arrow_up:

stix2/test/v20/test_campaign.py
100% <0%> (ø)
:arrow_up:

stix2/test/v21/test_core.py
100% <0%> (ø)
:arrow_up:

stix2/v20/observables.py
100% <0%> (ø)
:arrow_up:

stix2/test/v21/test_attack_pattern.py
100% <0%> (ø)
:arrow_up:

... and 58 more

Continue to review full report at Codecov.

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


feat: add promotion and demotion for blob fee updates

Implements demotion and promotion based on the new blob fee for each block. This follows the same general pattern as update_base_fee, except without extra prioritization. The PoolSize struct now includes the blob subpool size as well. Blob txs are now removed from the all set of txs.
The blob subpool now does not include pending blob transactions.
Two tests are added - one tests promotion, the other tests demotion from the pool.

can we merge this @Rjected


 in a blank/white screen after being idle over the long weekend

https://buganizer.corp.google.com/issues/168047392

Upstream issue diagnosed as not a Flutter engine issue. Closing.


[JS-to-C++ test] Fix flaky USB transfer exception

One code path in TestingSmartCardSimulation - the USB interrupt transfer that's intended to notify about card insertions/removals - was returning values of incorrect format, causing assertion failures down the stack.
Presumably this never caused observable test failures, since the tests happened to coincide with the CCID driver's internal task timing (which would mean the interrupt transfers are undertested as of now).

Nice, I actually stopped seeing those consol errors from 'callCpp' with this patch.

Nice, I actually stopped seeing those consol errors from 'callCpp' with this patch.

Yeah it was my first suspect last week, but I took extra time in attempts to get a reliable regression test for this. Unfortunately still haven't got it, hence landed as is...


Add a new page just to show different Outreaches

We currently don't have a way in the app to view all your Outreaches.
This issue is for creating a page that lists all Outreaches, similar to the home page that lists all Clients. Create an Outreach Card that has the following information:

Client name
Client type
Type of Outreach (phone, email, etc)
Date
Whether the client responded
Whether the client responded saying yes to the project

To whomever intends to tackle this, please feel free to include any other relevant information.
Make sure to maintain styling consistent with the rest of the app. Refer to Issue #6 for the design discussion.

Hey Lara, i made a rough sketch about the outreaches page, hopefully to discuss it together with the group and help with the design of it

This looks great, @AhmedHHamdy! I posted this in our thread on Discord for discussion. I love it, my only concern would be the accessibility and functionality of the icons. Labels might be the better option. I'm excited to see what others think! :D Thank you for creating this!

Hi, I'd like to give this task a shot!


SafeRange bug restricts certain ranges

Version

Phaser Version: 3.55.2

Operating system:

Browser:

Description
The condition startIndex + endIndex > len, in SafeRange.js is not necessary, and it restricts valid ranges.
/* src/utils/arrays/SafeRange.js */
var SafeRange = function (array, startIndex, endIndex, throwError)
{
    var len = array.length;

    if (startIndex < 0 ||
        startIndex > len ||
        startIndex >= endIndex ||
        endIndex > len ||
        startIndex + endIndex > len)
    {

The purpose of this function is to validate ranges in the form of [startIndex, endIndex). Where [ includes startIndex and ) excludes endIndex.
The fix will allow the following ranges to be validated:
[n, Array.length), n = {1, ..., Array.length-1}, Array.length > 1
[n, m), n = {1, ..., Array.length - 1}, m = {n+1, ..., Array.length}, Array.length > 1

For example,
var arr = [0, 1, 2, 3, 4];
console.log(Phaser.Utils.Array.SafeRange(arr, 0, arr.length)); // true, since [0, 5) is a valid range for arr that gives [0, 1, 2, 3, 4]
console.log(Phaser.Utils.Array.SafeRange(arr, 1, arr.length)); // false, but should be true since [1, 5) is a valid range for arr that gives [1, 2, 3, 4]
console.log(Phaser.Utils.Array.SafeRange(arr, 2, 4)); // false, but should be true since [2, 4) is a valid range for arr that gives [2, 3]

It restricts the following static methods in Phaser.Utils.Arrays: CountAllMatching, EachInRange, GetAll, GetFirst, RemoveBetween, SetAll.
The proposed fix will allow these methods to be able to operate on the aforementioned ranges.
For example, this code snippet demonstrates the change for GetAll (Phaser.GameObjects.Group::getMatching)
https://jsfiddle.net/z7w96k53/1/

  create() {
    var gameObjects = [];
    for (let i = 0; i < 5; i++) {
      var gameObject = new Phaser.GameObjects.Container(this);
      gameObject.setData('num', i);
      gameObjects.push(gameObject);
    }
    var group = new Phaser.GameObjects.Group(this, gameObjects);
    var newGroup = new NewGroup(this, gameObjects)
    console.log("Before");
    console.log(group.getMatching('active', true, 0).map(element => element.getData('num'))); // [0, 1, 2, 3, 4]
    console.log(group.getMatching('active', true, 1).map(element => element.getData('num'))); // []
    console.log(group.getMatching('active', true, 2, 4).map(element => element.getData('num'))); // []
    console.log("After");
    console.log(newGroup.getMatching('active', true, 0).map(element => element.getData('num')));  // [0, 1, 2, 3, 4]
    console.log(newGroup.getMatching('active', true, 1).map(element => element.getData('num'))); // [1, 2, 3, 4]
    console.log(newGroup.getMatching('active', true, 2, 4).map(element => element.getData('num'))); // [2, 3]
  }

Example Test Code
This snippet contains the unit tests: https://jsfiddle.net/7x5Lpse1/16/

Thanks for opening this issue, and for submitting a PR to fix it. We have merged your PR into the master branch and attributed the work to you in the Change Log. If you need to tweak the code for whatever reason please submit a new PR.


Fix Actions version and API reference verbage

Description (required)

Fixes the Actions guide to say since v4.15
Update the API reference description to better capture the actions featureset
Add backticks to astro:actions on the API reference header

Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

en
guides/actions.mdx
Source changed, localizations will be marked as outdated.

en
reference/api-reference.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Cursor disappears

Hyprland Version
 System/Version info
Hyprland, built from branch main at commit 70ebc3add3b941a52f460f02129ef0842f580ec7 dirty (sessionLock: call wlr_surface_map/_unmap on the surface manually).
Date: Wed May 1 04:59:07 2024
Tag: v0.39.1-115-g70ebc3ad, commits: 4575

flags: (if any)

System Information:
System name: Linux
Node name: Arch
Release: 6.8.8-zen1-1-zen
Version: #1 ZEN SMP PREEMPT_DYNAMIC Sun, 28 Apr 2024 15:59:30 +0000

GPU information:
64:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Phoenix1 [1002:15bf] (rev c7) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

Bug or Regression?
Bug
Description
On some sites, the cursor disappears when placed on an image.
How to reproduce
test url
Crash reports, logs, images, videos
https://github.com/hyprwm/Hyprland/assets/39702328/f8778949-4bb0-4d6e-8113-2be60720f749

 well...
Looks like an issue with the cursor theme?

hyprland stable version in firefox and chromium cursor normal display, hyprland-git version firefox cursor display, chromium browser cursor disappears.

tested again with chromium (I am on git) and the cursor is not disappearing. Have you tried a different theme? Are you using hyprcursor?

Can repro with --enable-features=UseOzonePlatform --ozone-platform=wayland flags, happens only with some shapes like zoom_in/out or ew_resize

Can repro with --enable-features=UseOzonePlatform --ozone-platform=wayland flags, happens only with some shapes like zoom_in/out or ew_resize

yes

tested again with chromium (I am on git) and the cursor is not disappearing. Have you tried a different theme? Are you using hyprcursor?

chromium works under xwayland, it disappears in wayland, use chromium --enable-features=UseOzonePlatform --ozone-platform=wayland to enable wayland.

works on my end in both modes. Again, have you tried a different theme?

Happens on all themes I have installed (Bibata, Adwaita and Breeze). Hyprcursor is unset and disabled.

After using Hyprcursor themes, the cursor works, but chromium loses some cursor styles under wayland and falls back to the default styles. Cursor-Test
https://github.com/hyprwm/Hyprland/assets/39702328/03d32790-f6e1-4300-988f-7f0f5aacb12a
https://github.com/hyprwm/Hyprland/assets/39702328/7cdf7b38-60ab-4914-85f9-1974d34fa036

huh. What is the hyprcursor version you have installed?

huh. What is the hyprcursor version you have installed?

hyprcursor-git-0.1.8.r0.gcab47461-1

https://github.com/hyprwm/Hyprland/assets/107314235/430fd346-7da7-47b6-94c8-2082d83f6a08
Left half is Firefox (doesn't support hyprcursor) on the cursor test page and right half is kate (supports hyprcursor).
col_resize and row_resize are shown to fall back on default pointer on hyprcursor. The theme is extracted from the xcursor Adwaita and compiled from there.

this one is a problem with your theme. Make sure you have a shape that fits col_resize and row_resize. Worth noting that due to (ugh) web uses - and wl uses _.
Recommended overrides for the sidey side arrow:

analogous for toppy bottomy will be with n, s, and row.

I've committed<PHONE_NUMBER>4fd04e042821fd171be5ee59b0a8d0 which makes hyprland attempt - first before falling back to a pointer.

this one is a problem with your theme. Make sure you have a shape that fits col_resize and row_resize. Worth noting that due to (ugh) web using - and wl _.
Recommended overrides for the sidey side arrow:
analogous for toppy bottomy will be with n, s, and row.

Thanks, it's a problem with the Bibata-Modern-Ice theme, changing the - to _ in the filename works.

this one is a problem with your theme. Make sure you have a shape that fits col_resize and row_resize. Worth noting that due to (ugh) web using - and wl _.

That made me check implementation for wlroots and I found out they're using - in their shape_names. Is there a reason for using underscores in Hyprland's re-implementation?
Replacing underscores with dashes resolves initial issue with Xcursors. They do not disappear anymore.
diff.txt

hm, right. Maybe it was a mistake somewhere, I'll fix it.


o.DOM.injectStyles is not a function

aurelia.js:13 Uncaught (in promise) TypeError: o.DOM.injectStyles is not a function
at r.G (http://localhost:49849/dist/aurelia.js:13:17109)
at http://localhost:49849/dist/aurelia.js:6:1595
Downloaded skeleton-es2016-asp.net5 beta 1.2.2. Got everything running perfectly.
Pulled down the Kendo UI Bridge (KendoUI Core) and received error.  Ran dedupe and now getting the below error:
aurelia.js:13 Uncaught (in promise) TypeError: o.DOM.injectStyles is not a function
at r.G (http://localhost:49849/dist/aurelia.js:13:17109)
at http://localhost:49849/dist/aurelia.js:6:1595

Did you try jspm update after that? Also please check if you got any JSPM forks after installing.
Maybe the KendoUI bride folks should also know about it, so I ping @JeroenVinke :smile:

jspm update fixed it, thanks.


Use parking_lot::Mutex instead of std::Mutex in walreceiver

Making progress towards https://github.com/zenithdb/zenith/issues/956
Note that parking_lot mutexes don't poison. If a thread panics while holding a mutex, the mutex is released, without crashing the next thread to ask for it.

Huh, so that is fully API-compatible -- I expected more problems) Let's switch all mutexes to parking_lot then


Add the on_off call in the move_to_level_with_on_off command

Implement the on_off call in the move_to_level_with_on_off command for the MCU Tuya devices.
The previous implementation was removed in #1748. Originally introduced in #1489
After some discussion in https://github.com/zigpy/zha-device-handlers/pull/1748#issuecomment-1244931971 I really believe that the implementation must be done inside the handler library better than expect to add all the devices in the ZHA implementation. This way the handlers library would fully implement the move_to_level_with_on_off for the MCU devices.
Fixes: #1694

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 71.58%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

zhaquirks/tuya/mcu/init.py
0
3
0.0%

Files with Coverage Reduction
New Missed Lines
%

zhaquirks/tuya/mcu/init.py
1
53.66%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
5269

Relevant Lines:
7361

💛  - Coveralls

Codecov Report
Base: 71.60% // Head: 71.57% // Decreases project coverage by -0.02% :warning:

Coverage data is based on head (ec4bbb4) compared to base (a072ca3).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #1754      +/-   ##
==========================================
- Coverage   71.60%   71.57%   -0.03%
==========================================
  Files         239      239
  Lines        7358     7361       +3
==========================================
  Hits         5269     5269
- Misses       2089     2092       +3

Impacted Files
Coverage Δ

zhaquirks/tuya/mcu/__init__.py
53.98% <0.00%> (-1.02%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

Thanks David.


Fixed bug in docs for survival regression

Fixing #1036
First time contributing to a big open source project so I hope I did this right!

Thanks @jlim13!
The edit looks good, but you have also added a file filename.joblib. Can you remove this from the pull-request?

removed the file

Travis won't pass, unfortunately, as there's a test I need to fix. PR lgtm, thanks!


Adjust resource

Adjust required resources in base.config.
PR checklist

[x] This comment contains a description of changes (with reason)
[ ] CHANGELOG.md is updated
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] Documentation in docs is updated
[ ] If necessary, also make a PR on the nf-core/metapep branch on the nf-core/test-datasets repo

Is the boxplot commit in this PR intentional?

Is the boxplot commit in this PR intentional?

kind of ;) I just wanted to hide the outlier in the plot (....while thinking about how many data points we have)

kind of ;) I just wanted to hide the outlier in the plot (....while thinking about how many data points we have)