Laggy settings window

ℹ Computer information

PowerToys version: v0.23.2
PowerToy Utility: The settings menu
Running PowerToys as Admin: Yes
Windows build number: [run "winver"]

📝 Provide detailed reproduction steps (if any)

Open the settings manager
Resize the window
Observe the lag

It's very random, it doesn't always happen, after I open the windows game bar to record it for example it's just gone until reboot.
✔ Expected result
I would expect a smooth resizing of the window.
❌ Actual result
The resizing is extremely slow and laggy

I think this is a known issue with XAML Islands (that we're currently using for Settings). Moving to WinUI should solve that issue.

Is there any help needed with this? I could contribute.

I think this is just one of those items that is dependent on the end user computer and we can't get away until we migrate to WinUI 3 for the tools more and more.
#6715 is the tracking item


Discussion about passthrough to parse object keeping extra props

Add  passthrough method inspired by Zod  to allow parse object() keeping extra props  that are present on original input object but not validated through Schema.
This maybe useful if you want to validate/trasform only "a portion of an object" that you receive from/send to and API, but want to keep all the original extra props present in the object that are not specified in object() schema.
PS: some more discussion points:

if you prefer other name please let me know, I just used the same name of Zod... but I'm open to alternative/refactor
do you think that it's correct to implement it as a "wrapper"  and my modification to unwrap or do you prefer other way to implement it ?
about this I was even thinking to implement it changing the object() implementation adding another overload with an optional strict: boolean param  before error - but to do it I need to change the implementation of getErrorAndPipe handling 3 params to discriminate boolean, string, Pipe[] ...  I don't know if you'll approve/prefer this kind of implementation, maybe the result is more clear because the real nature of passthrough is just an ObjectSchema that keep the extra params, but don't know if you like it...
Or maybe on the same "idea" i can create other 2 function objectWithExtraProps that return ObjectSchema but simply use my parse code to keep props...

This are all the ideas I came up when implementing it, what's your opinion?
Originally posted by @dmorosinotto in https://github.com/fabian-hiller/valibot/issues/15#issuecomment-1652418243

If we decide on a method. Can you think of alternative names besides passthrough?

About alternative names I can suggest keepExtras or keepExtraProps
But I have another question do you think that we can change the implementation of merge to make it works with Record + Object? Because actually it allow to merge only ObjectSchema[], but if we can make it works with Record+Object we can achieve the "passtrough" by simply merge( record(string(),any()) , object({...schema...}) );
I looked at merge code and record() constructor but the two are not compatible to simply join the parse as it's implemented now...
Or if you think that this idea maybe good, we can create a new method extends to specify merge Record+Object correctly handle the parse and input/output type ;-)
Waiting for your feedback @fabian-hiller  about how do you prefer to procede with this issue

The merge method is currently only for objects. I will check if the extension for records makes sense. However, I don't think we should implement the passthrough feature internally with merge.
Thanks for the alternative names. I will consider them. I haven't had a chance to think about this issue yet. I will try to answer you again here as soon as possible.

Yes I see the merge code and totally understand why it works only for object - and if you think that enable it to handle even record is a not desired design, I think that the more simple and effective ways to implement "passthrough" is a new method that enable "merging" ObjectSchema + RecordSchema and maybe calling it extends
with two overload one with optional RecordSchema? as you can decide to pass default record(string(), any()) if no RecordSchema is passed and extends is even "coherent" with Typescript type because a TypeX extends baseTObj if contains all the base props + enabling extras props to exist, and this is my "desired" behaviour of type Passthrough<T extends object> = T & Record<string, any>

Thanks for your further feedback. Since extends is used in TypeScript, not only for generics, but also for conditional types to check if a type matches or descends from another, the naming may not be perfect. For example, I had the idea of maybe using extends as a name for the type guard feature.
For passthrough it will be important to also investigate the other methods of Zod, such as strict, to then check what is the best overall approach for Valibot.

I added passthrough and strip in v0.14.0. Thank you for your contribution!


customize name of workspace '.vscode' directory

Use case: I'm sharing the same source directory (via volume mount) on two machines (one on mac, one on linux). I'm usually developing on mac and then testing the same code in a VM on linux. This is a cmake/c++ project so the build artifacts are already separated by specifying different build directories (outside the source tree). The only issue I have is with the workspace .vscode directory - I'd like the linux box to use a different directory than .vscode - for example .vscode-linux. This way I can keep separate workspace settings, launch, intellisense, etc. The name of the .vscode directory should be saved in the users settings.json (so it's global for this user) - once configured vscode should always try to open workspace .vscode-linux on that machine.
The source tree would than look like this:
mySrc/.vscode/*
mySrc/.vscode-linux/*
mySrc/src/
mySrc/test/
Hopefully the workspace name '.vscode' is not hardcoded in a million places and is always obtained via a function call ;)
The workaround I have is using symlinks:
myLinkedSrc/.vscode/
myLinkedSrc/src@ -> ../mySrc/src
myLinkedSrc/test@ -> ../mySrc/test
myLinkedSrc/CMakeLists.txt@ -> ../mySrc/CMakeList.txt
But that would not work on windows.
Sorry if this is a duplicate of some other request - I did my best, but searching for "vscode", ".vscode", "workspace vscode" is a bit challenging :)

There's currently no way of changing the '.vscode' folder name. In theory what's in there should work on all platforms. I think it's ok to share these across platforms, but maybe what you are looking for are more platform specific settings and launches. The launches have already the capability for platform specific exec paths and arguments.
Not sure what you mean by intellisense

Example of stuff that does not work on different platforms (if sharing the source folder):

.vscode/c_cpp_properties.json: contains compiler paths (I use clang on mac, gcc on linux)
.vscode/settings.json: the setting from mac should not apply on linux (for example on linux vscode added "C_Cpp.intelliSenseEngineFallback": "Disabled", I don't want that on the other system)

AFAIK any extension can put its settings in workspace .vscode - those settings might also be machine specific and won't work between systems.
The point here is to be able to share the code (same source tree), but not the settings.

What's in .vscode are the settings applying to that workspace and are typically also put into SCM (Git,...) and shared amongst all developers of the project. So the settings are expected to work for all machines and users.
User specific settings should go in the user settings. Maybe that would make sense for you to do for "C_Cpp.intelliSenseEngineFallback": "Disabled",
.vscode/c_cpp_properties.json should also be designed the way that it is platform neutral. If not, it should contain platform specific settings. Please file this as a request against the cpp extension.

Yes, c_cpp_properties.json has sections with names of the platform (name: Linux, Mac).. this does help a bit, but what if I have two "Linux" boxes - one using clang, the other gcc? :) How will vscode select the right one?
It seems that not all extension developers know that settings are "expected to work for all machines and users"... so an option to workaround such extensions would be welcome. By being able to specify a different instead of .vscode there would be no problems.
I never check-in .vscode into source control - the extensions I use always recreate the files they need. The only file I usually have to edit is launch.json - since the path of the binary is different depending on the system I work on.
As said... If there are too many places where the name '.vscode' is hardcoded and if extensions also hardcode that name - forget about it. There was a similar request for JetBrains products for the .idea folder and this was the reason why it couldn't be done.

I'd recommend to not do this, but @samodadela with the .workspace file that now exist and can include all definitions that are otherwise found in the .vscode folder you could move all entries there and have two files, don't you think?
So maybe that issue is up for closing?

/duplicate
https://github.com/microsoft/vscode/issues/29918


Unify p2p blocks processing (pubsub/sync)

Previously we had two separate mechanisms:

Initial syncing was done by the sync module.
After that new blocks were handled by the pubsub module.

The pull request unifies this logic by removing the pubsub module.

If I understand correctly, the test that wasn't working now is working. So now this is free to get merged.


400 time to next rebase real time

Closes #400

Does state.app.currentBlock get updated automatically after a rebase? If so I think this should work fine, if not, it might not change display from 'rebasing' if the user loads the page before the rebase and keeps it open until after the rebase.
Can't check the behaviour right now as it's just over 7 hrs to the next rebase :)
But if that is an issue, could do an onchain request to update the state when the countdown becomes negative?


Added table with links to website and repository created with Ignite

This change includes a link to a repository that only contains a README with a table of websites made with Ignite and its repositories.
I believe that the repositories shared here should share the Ignite project policy and be OpenSource.
Another repository has been created for as discussed in another pull request, so that the current one is not contaminated with many pull requests.
The ownership of this new repository should still be @twostraws
Whatever is decided.

Finally it will go into the original repository.


First pass at a get_file() function

Overview
Changes:

Added a function for collecting data files and caching them or, in the case of OPeNDAP links, opens and provides the data directly.

Related Issue / Discussion
https://github.com/Ouranosinc/raven/issues/185

@huard this is good to be merged

How do you see the next step ?
Is it to replace the links in common.TESTDATA by the urls ? Or use the links directly in the test files, without going through TESTDATA, or create pytest fixtures ?

The get_file approach helps us not have to deal with URLs and adding them in would likely be more complicated (as access via URLs is managed already, and examining the cache should be the priority). I would probably just replace from from xarray import open_dataset to from raven.tutorial  import open_dataset in our tests. We don't need to do much.


feat: add disableCORS option to server options

This feature allows you to disable CORS via the server options
Closes #34

Wholesale disabling of CORS may have its uses, but I think it would be a lot more useful if consumers of this package would be able to set the "allow origins" (for example) as a wildcard asterisk * typically gets rejected by HTTP clients with "credentials" (for example).
https://github.com/d-i-t-a/dita-streamer-js/blob/ff2a502d90d462fba474b9e33954afae6d0a841d/src/http/server.ts#L297-L311

I agree with you, but I think both are a useful feature.
CORS could perhaps be handled better using the CORS package as indicated in the express documentation.
If you agree I can refactor it with these changes.

With the latest changes, it works behind an API gateway with CORS enabled and allowedOrigins defined in the serverOptions. I've also updated the custom route methods (i.e. expressUse and expressGet) and the example in the README that shows how to enable CORS in these cases.


[5334] Bulk recommend fix errors renders incorrect page if user gets an error uploading

Context
https://trello.com/c/biSsSqUH/5334-bulk-recommend-fix-errors-renders-incorrect-page-if-user-gets-an-error-uploading
Changes proposed in this pull request

Update BulkUpdate::RecommendationsErrorsController to handle file uploads separate to BulkUpdate::RecommendationsUploadsController

Guidance to review

Log in as a provider
Visit "Bulk recommend"
Download CSV
Edit CSV and fill the column "Date QTS or EYTS standards met" with at least one cell containing an incorrect future date e.g. 7000-03-01
Upload CSV
View errors
Click the upload button without attaching a file
Form errors should appear on the current page

Screenshot

Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 Bulk recommend - fix-errors renders incorrect page if user gets an error uploading


Support "args" param in "@SpringBootTest"

Reflectively access SpringBootTestArgs(package private) to support args param in @SpringBootTest.

Thanks @snicoll, I got a picture.
Yeah, since MergedContextConfiguration is available at aot build time, it should pre-calculate the args. I'll update the change.

@snicoll Should we include that in 0.11.2? If yes, can I let you merge it with potential refinement if needed?

Thanks for the nudge. It looks good but considering we're going to release shortly, I've moved that one to the next release.

Thanks again @ttddyy


Adding xfce4-terminal in the list of preferred terminals

Install method (Flatpak/Binary/Other)
Flatpak
Distrobox Version
distrobox --version will tell you this.
1.4.0
Your Host OS
Debian 12 Bookworm
Describe your Issue
Hello, as first thanks for BoxBuddy that I just discovered today. I had issues with DISPLAY not being exported and audio failing. Just using BoxBuddy fix the two of them. I'm curious why, I guess my system doesn't pass some variables by default.
My "issue" here is that I use XFCE, and so I wanted to use xfce4-terminal instead of kconsole that I have installed for various reasons.
It's not a real problem, but if it doesn't make any issue, I think it could be nice to be able to make it work on XFCE out of the box.
Have a good day
Cheers

Hello, I wanted to add xfce terminal, but it does not work.
See https://dvlv.github.io/BoxBuddyRS/faqs#can-i-use-some_terminal-instead

Hello @Dvlv
Thank you, sorry I didn't look in the FAQ. I will use konsole it will be fine too ^^
I bit outside this issue, I didn't notice any export of variable in the source, so I don't really explain why it resolves my DISPLAY variable that it was not set by default. If you know why, I'm interested to know for which reason if not it's not an issue I will just be happy that it works on his own ^^
Have a good day

Sorry, I think that would be an issue with Distrobox itself, which I'm not able to help with.

Hello @Dvlv
Sorry to ping you again.
I did a little test by using a wrapper script instead of my konsole binary, and it looks like working. Do you remember what kind of issue you got with xfce4-terminal, so I can maybe do some test?
Here is the wrapper script I use. I'm on Debian 12 so perhaps it can be different on other distributions.
$ cat /usr/bin/konsole
#!/bin/bash

/usr/bin/xfce4-terminal.wrapper -x "$@"

The issue with xfce-term is that it expects its command to be a single argument, while all other terminals have the command pieces separate.
e.g.
gnome-terminal -- ls -la

vs
xfce4-terminal -e "ls -la"

Hello @Dvlv
You think it could maybe be used with like a wrapper in flatpak or perhaps a possibility to add a custom command?
I think the custom command is potentially better because it gives the possibility to use another script for specific use.
I still add the wrapper of Debian in case it can be related.
#! /usr/bin/perl -w
#
# Terminal.wrapper - Debian terminal wrapper script
#
# Copyright (c) 2004-2005 os-cillation
#

while ($opt = shift(@ARGV))
{
    if ($opt eq '-display')
    {
	$arg = shift(@ARGV);
	push(@args, '--default-display', $arg);
    }
    elsif ($opt eq '-name')
    {
	$arg = shift(@ARGV);
    }
    elsif ($opt eq '-n')
    {
	$arg = shift(@ARGV);
    }
    elsif ($opt eq '-T' || $opt eq '-title')
    {
	push(@args, '--title', shift(@ARGV));
    }
    elsif ($opt eq '-geometry')
    {
	$arg = shift(@ARGV);
	push(@args, "--geometry=$arg");
    }
    elsif ($opt eq '-fn')
    {
	$arg = shift(@ARGV);
    }
    elsif ($opt eq '-fg')
    {
	$arg = shift(@ARGV);
    }
    elsif ($opt eq '-bg')
    {
	$arg = shift(@ARGV);
    }
    elsif ($opt eq '-tn')
    {
	$arg = shift(@ARGV);
    }
    elsif ($opt eq '-e')
    {
	$arg = shift(@ARGV);
    if (@ARGV)
    {
        push(@args, '-x', $arg, @ARGV);
        last;
    }
    else
    {
        push(@args, '-e', $arg);
    }
    last;
    }
    elsif ($opt eq '-h' || $opt eq '--help')
    {
	push(@args, '--help');
    }
}
exec('xfce4-terminal',@args);

Thanks, but that feels a bit too hacky. I'd rather keep to the supported terminals.

Hello @Dvlv
I noticed for example for Konsole you pass -e distrobox enter debian so I did the same for xfce4-terminal and it works but I use -x instead of -e.
$ bash /tmp/b.sh -x distrobox enter debian
$ cat /tmp/b.sh
xfce4-terminal $@

I will try to make a PR but it looks like the compilation require much free space and for now I don't have enough.
Cheers

Xfce terminal support is now added in 2.1.5
Thanks for your investigations!

Hello @Dvlv
Sorry for not getting back to you sooner. It works like a charm.
Thanks again for your time for testing and updating.
Have a good day


Make minor fixes in Compiler to properly handle void type expression

PR Summary
Make some minor fixes in Compiler to properly handle void type expressions.
Before Fix
> [void]1,2,3
An error occurred while creating the pipeline.
+ CategoryInfo          : NotSpecified: (:) [], ParentContainsErrorRecordException
+ FullyQualifiedErrorId : RuntimeException

> [void]1 | Measure-Object
An error occurred while creating the pipeline.
+ CategoryInfo          : NotSpecified: (:) [], ParentContainsErrorRecordException
+ FullyQualifiedErrorId : RuntimeException

## The generated code for `[void](mkdir c:\test)` is discarded
> $s = @([void](mkdir c:\test))
> Test-Path c:\test
False

After Fix
> $a = [void]1,2,3
> $a.Count
3

> [void]1 | Measure-Object

Count    : 0
Average  :
Sum      :
Maximum  :
Minimum  :
Property :

> $s = @([void](mkdir c:\test))
> $s.Count
0
> Test-Path C:\test
True

PR Checklist
Note: Please mark anything not applicable to this PR NA.

[x] PR has a meaningful title

[x] Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] User facing Documentation needed

[ ] Issue filed - Issue link:

[x] Change is not breaking
[x] Make sure you've added a new test if existing tests do not effectively test the code changed

[x] Add [feature] if the change is significant or affectes feature tests

[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

@daxian-dbw Is the PR ready to merge or you will continue?

@iSazonov Sorry for the late response. I will address Jason's and your comments.

@iSazonov @lzybkr Your comments have been addressed. Thank you!


AP-601 New SCSS file with style for truncating email address

What
Link to story
New SCSS file with CSS to do the following:

max-width that (for narrower displays) prevents email containing field from overlapping with the service title.
overflow, vertical alignment added to stop other undesirable effects.

Checklist
Before you ask people to review this PR:

[x] Tests and rubocop should be passing: bundle exec rake
[x] Github should not be reporting conflicts; you should have recently run git rebase master.
[x] There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
[x] The PR description should say what you changed and why, with a link to the JIRA story.
[x] You should have looked at the diff against master and ensured that nothing unexpected is included in your changes.
[x] You should have checked that the commit messages say why the change was made.

Shall I close this and re-do the PR then?

Replaced with PR 564.


Allow weakref

[x] Closes #3317
[x] Tests added
[x] Passes black . && mypy . && flake8
[x] Fully documented, including whats-new.rst for all changes and api.rst for new API

Ready for review and merge

Any idea when a 0.13.1 release might be made to get this fix out in the wild?


State not passed to callback

In a SSO scenario, I redirect my users to https://myapp.auth0.com/authorize?state=HelloWorld&redirect_uri=https://myapp/signin-auth0....
When a user authenticates they get redirected to the /signin-auto0 middleware, which in turn sends them to my custom callback, /auth0/callback
Unfortunately, the state value seems to be lost when it gets to my callback. Is this a result of the commented out line at:
https://github.com/auth0/auth0-aspnet-owin/blob/master/src/Auth0.Owin/Auth0AuthenticationHandler.cs#L298
Without this, I can't send the users to the page that they want to go.

@BrianVallelunga This is a very old issue. Is this still relevant?

Closing this issue. Feel free to re-open if this is still an issue


Missing import statement for VersionNumber in init.gradle

brew install gradle (Mac Sierra)
run gradle

Expected Behavior
gradle should run
Current Behavior
When I run gradle, it failed because "Could not get unknown property 'VersionNumber'" at '/Users/ywang2/.gradle/init.gradle' line: 155
I added "import org.gradle.util.VersionNumber" to ~/.gradle/init.gradle, then it worked.
`TAG-32368:cc-indexer ywang2$ gradle
FAILURE: Build failed with an exception.

Where:
Initialization script '/Users/ywang2/.gradle/init.gradle' line: 155

What went wrong:
A problem occurred evaluating initialization script.

Could not get unknown property 'VersionNumber' for build of type org.gradle.invocation.DefaultGradle.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 0s
`
Context

Steps to Reproduce
$ gradle
FAILURE: Build failed with an exception.

Where:
Initialization script '/Users/ywang2/.gradle/init.gradle' line: 155

What went wrong:
A problem occurred evaluating initialization script.

Could not get unknown property 'VersionNumber' for build of type org.gradle.invocation.DefaultGradle.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 0s
Your Environment

Build scan URL:

This was an intentional change.
org.gradle.util was not part of Gradle's public API but used to be part of the default imports in build scripts.  This was removed in 5.0. Types in org.gradle.util may be removed or changed at any time as we treat it as an internal API.


esptool: 4.4 -> 4.5

Description of changes
Diff: https://github.com/espressif/esptool/compare/v4.4...v4.5
Changelog: https://github.com/espressif/esptool/releases/tag/v4.5

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Tested binary, able to read a chip ID.
Result of nixpkgs-review pr 216397 run on x86_64-linux 1

1 package built:

esptool


Resolve missing prop mock warning

Issue
Since react-native 0.58.0-rc.0 react-native-safe-module is throwing a missing prop mock warning defined here since getConstants mock is missing (getConstants become a part of native modules API in this commit).
Steps to reproduce
Require lottie-react-native in any project with react-native >= 0.58.0-rc.0

Hello! Can you please tell in what version this fix was released?

@yaroslavnikiforov it hasn't been released yet.


hcm: group access log options in a protobuf message

Commit Message: Group access log options in a protobuf message
Additional Description: Follow up to https://github.com/envoyproxy/envoy/pull/26155 request to group HCM access log options under a single protobuf message
Risk Level: Low
Testing: Unit tests
Docs Changes: none
Release Notes: none
Platform Specific Features: none

Was wondering what about similar PR for tcp_proxy? the difference is that for tcp_proxy, current release has access_log_flush_interval, would it be ok to apply the same refactor to it?

Same comment to https://github.com/envoyproxy/envoy/pull/26495#issuecomment-1493188381

/retest

/retest

/retest


Implement Metadata URI for Land NFTs

Task: Enhance the LandNFT contract to include a metadata URI for each NFT, allowing for off-chain storage of additional land information.
Definition of Done:
•  The LandNFT contract is updated to include a metadata URI for each token.
•  A function is added to update the metadata URI.
•  The tokenURI function is implemented to return the metadata URI.
•  Tests are written to verify the new functionality.
Note: Only applications through the OnlyDust platform will be considered.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I'm Poulav Bhowmick, a Starknet Wolf. I am a software engineer at Invisible Studios, and a blockchain engineer with a robust background in TypeScript, Rust, Solidity Cairo, fullstack development and blockchain technology. My experience includes building robust applications, optimizing functionalities and blockchain integration. I have actively participated in events and open source contributions, enhancing my capability to tackle real-world tech challenges. My projects can be viewed on my GitHub Profile and OnlyDust Profile. Plus I´m active member of Starknet, Ethereum, Stellar ecosystem.
How I plan on tackling this issue
To address this issue, I will enhance the LandNFT contract by adding a storage field that maps each token_id to its corresponding metadata URI. I will implement a new function that allows only authorized entities (likely the land registry contract) to set or update the metadata URI for each token. The tokenURI function will then be implemented to return the URI for a given token_id. I will ensure that the implementation is gas-efficient and integrates smoothly with the existing ERC721 logic. Finally, I will update the test suite to verify that the metadata URIs are correctly set, retrieved, and that only authorized updates are allowed.
ETA - 2 days

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Hello can I work on this

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Hi, I am a skilled cairo dev with lots of experience contributing to amazing projects, my profile is a witness to that statement. this is issue is similar to something i've worked on in the past so, it's a perfect fit for me.
How I plan on tackling this issue
I'll be using the standard erc4906 to achieve this to ensure compatibility. then i'll write test to ensure it functions as expected.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Hi, please can I be assigned this please? This would be my first time contributing to this project and I would love to be the given the opportunity to contribute. I have experience in html, css, JavaScript,TypeScript and solidity, and Cairo.
How I plan on tackling this issue
To solve this issue, I’ll take the following the steps:
1. I’ll add metadata URI storage-that is I’ll introduce a mapping to store metadata URIs for each token.
2. I’ll create a function to update the URI, restricted to the owner or authorized role.
3. Next, I’ll implement the  tokenURI  function to retrieve the stored URI.
4. Then I’ll create tests to validate the setting and retrieval of metadata URIs.
Please assign me, I’m ready to work.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Hello, i'm a frontend and blockchain developer, pls i will love to work on this issue
How I plan on tackling this issue

I will enhance the LandNFT contract to include a metadataURI variable that stores the URI for each NFT.
i will Implement Lock and Unlock Functions
-I will Update Transfer Function
i will Modify LandRegistry Contract
I will Implement Unlocking Mechanism

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I have contributed to several Cairo projects on OnlyDust and I've worked on very similar issues in the past.
How I plan on tackling this issue
I will carefully implement the feature as requested and add all the necessary tests.

@fishonamos thanks for the opportunity to contribute.
I noticed the issue was labeled as "Blocked", should I hold off on working on it for now?


Errno 111 Connection refuse

hi ,
sry for asking but i searched a lot for this issue !
i'm using ubuntu 16.0.4 LTS
i have to use proxy for connect to youtube
i tried :  tsocks youtube-dl "https://www.youtube.com/watch?t=4&v=BaW_jenozKc"
i am sure tsocks work correctly because i test it before .
Could u help me plz?
tsocks youtube-dl -v "https://www.youtube.com/watch?t=4&v=BaW_jenozKc"
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'https://www.youtube.com/watch?t=4&v=BaW_jenozKc']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.11.02
[debug] Python version 2.7.12 - Linux-4.4.0-45-generic-x86_64-with-Ubuntu-16.04-xenial
[debug] exe versions: ffmpeg 2.8.8-0ubuntu<IP_ADDRESS>, ffprobe 2.8.8-0ubuntu<IP_ADDRESS>
[debug] Proxy map: {}
[youtube] BaW_jenozKc: Downloading webpage
ERROR: Unable to download webpage: <urlopen error [Errno 111] Connection refused> (caused by URLError(error(111, 'Connection refused'),))
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 404, in _request_webpage
return self._downloader.urlopen(url_or_request)
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2001, in urlopen
return self._opener.open(req, timeout=self._socket_timeout)
File "/usr/lib/python2.7/urllib2.py", line 429, in open
response = self._open(req, data)
File "/usr/lib/python2.7/urllib2.py", line 447, in _open
'_open', req)
File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
result = func(*args)
File "/usr/local/bin/youtube-dl/youtube_dl/utils.py", line 1036, in https_open
req, **kwargs)
File "/usr/lib/python2.7/urllib2.py", line 1198, in do_open
raise URLError(err)

--proxy socks5://proxy:port.


f5 & f9 sometimes malfunction

hi, my os is win10 and version of vscode is 1.80. when i press f5 or f9 to run codes in text editor, codes could be sent to the terminal properly and sometimes executed. however, sometimes codes are just pasted to the terminal and i need to press Enter to execute them. i have no idea what triggers the malfunction. what should i do to fix the problem? lots of thx!

Hi, the code failed to execute sometime due to the race condition between sending text to the terminal vs. sending the newline character to execute the code.
Currently there are two ways the extension is mitigating this:

Increase the ExecutionLagMilliSec in the configuration which adds a delay between sending code to the terminal and sending the execution command to the terminal.  This is useful for those with a less responsive computer.
Switch SendCommandMethod from clipboard to file.  This is recommended for most modern computer with SSD hard drive.

The clipboard option will be deprecated in the future since it subverted the system clipboard
The file option is faster with SSD and can help reduce the effect of the race condition

A more permanent solution is for VSCode to formally support IPython where there is an npm API for IPython or VSCode extension has a python API.  Both are well beyond this extension scope.  I'll keep an eye for potential solution so the options above are currently the best I got.

Hi, I believe this issue is resolved.  Please reopen issue with a way to reproduce otherwise.


IPv6 problem

Is thie lib support IPv6?
PING fe80::250:56ff:fe95:d806(fe80::250:56ff:fe95:d806) 56 data bytes
64 bytes from fe80::250:56ff:fe95:d806%ens192: icmp_seq=1 ttl=64 time=0.129 ms
64 bytes from fe80::250:56ff:fe95:d806%ens192: icmp_seq=2 ttl=64 time=0.063 ms
^C
--- fe80::250:56ff:fe95:d806 ping statistics ---
2 packets transmitted, 2 received, 0% packet loss, time 1018ms
rtt min/avg/max/mdev = 0.063/0.096/0.129/0.033 ms
root@php-dev-cn:~/php-icmp-react# php examples/ping.php fe80::250:56ff:fe95:d806
Pinging "fe80::250:56ff:fe95:d806"...
Error: Timed out after 3 seconds

doesn't seem to work

I also check there is no function createIcmp6()

@louishot Thanks for your interest, this is indeed not currently supported by this project and I agree this would be a useful addition. There's also already a (somewhat dated) WIP PR #3 that aims to implement this.
I'm happy to accept PRs that implement support for IPv6 or if you need this for a commercial project and you want to help sponsor this feature, feel free to reach out and I'm happy to take a look. Other than that, there are currently no immediate plans to build this from my end (no demand at the moment and more important outstanding issues currently).
Let me know if you're interested in sponsoring this or want to work on this yourself 👍


upgrade purescript-react and use purescript-react-dom

introduces package.json to install react dependencies and
make testing easier.
also shims CommonJS require() for purescript-react and react-dom.
effectively upgrades ps-thermite to react 0.14.x. please let me know if you don't want some of these changes, i can remove them as needed.

:+1:
The current version of thermite/react throws warnings with newer versions of React, like:
Warning: input is a void element tag and must not have `children` or use `props.dangerouslySetInnerHTML`. Check the render method of null.

Maybe this will fix it, if not then I'll submit a pull request for purescript-react.

Looks good! Sorry for the delay, I'll test this out later today and put out a release.

Thanks!

:beers:


G-1

Bug
Part of the Visual Studio 2015 compatibility effort.
Fix ambiguous names. Cut down on the number of namespaces being used globally, boost should be the first to go.

Not worrying about vs compatibility.


Collapse static control

make the collapse-control pass take in static seq, static par as well as static repeat
It collapses nested static seq and static par in the same way as seq and par
For static repeat, it does the following:
Collapse
static repeat 0 { ** body ** }

into empty control
and
static repeat 1 {** body **}

into
** body **

This pr also gets rid of merge-static-par and static-par-conv altogether.

Looks great @paili0628 and thanks for reviewing it @calebmkim! I think merging it is good but also seems like we could add the change for #1351 here if you'd like. Up to you!

Wahoo!!!


Explain more about merge strategy for maintainers

Please verify the following:

[x] yarn test jest tests pass with new tests, if relevant
[x] yarn lint eslint checks pass with new code, if relevant
[x] yarn format:check prettier checks pass with new code, if relevant
[x] README.md (or relevant documentation) has been updated with your changes

Describe your PR
Updating docs to explain expectations for merging PRs.

:tada: This PR is included in version 10.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Corrected misspellings.

Some words were also missing.

Thank you!
Will update npm package when will be changing version.

I can work on the documentation, and sometimes improve the scripting if you give me write access :)

PR way is okay too.

Ok, I added more edits (two PRs to be exact)


Tfn solutions

Please don't make PRs with completed solutions to our repo, as it spams our PR tracker. 🙏


refactor: use image swiper in tech material component

Current dependencies on/for this PR:

main

PR #37

PR #39

PR #40   👈

This comment was auto-generated by Graphite.


Use the value calculated mindelay by the MPI min reduction

Also remove superfluous mpi wrapper function

https://www.neuron.yale.edu/neuron/static/new_doc/modelspec/programmatic/network/parcon.html#ParallelContext.set_maxstep

yes, i think this can be merged afaik


Resetting the space heating sliders in a new scenario affects the dashboard items

When a new scenario is created (and no sliders have been touched yet), ...

... resetting the space heating sliders results in a change in the dashboard items.

This is the case for both PRO and BETA.

This is because we merged the separate heating/hot water sliders in the front-end while leaving them separate in the back-end. The sliders show the starting value of the original space heating sliders. The starting value of the original hot water sliders may be different. Hitting reset sets them equal, resulting in a change in energy flows.
Since the differences are small and a fix is probably not very easy we decided to leave this for now. To fix this before the deploy is a bit ambitious I think 🐵
See Basecamp for some more background.

Thanks for your explanation, @michieldenhaan! I've removed the deploy milestone.


double-beep: Watch techindiainfotech.com

double-beep requests the watch of the watch_keyword techindiainfotech\.com. See the MS search here and the Stack Exchange search here.

techindiainfotech\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


Add Microsoft.AspNetCore.Razor.Design

Adding a new package for Razor MSBuild assets
/cc @natemcmaster @pranavkm

FYI - we used to have a package with this name - https://www.nuget.org/packages/Microsoft.AspNetCore.Razor.Design/1.1.0-preview4-final. Need to make sure we version this package to avoid conflicts.

Where was the change to add this package? CI is broken because the current submodule Universe is pulling from does not produce this package.

Did this change??? Do I need to create the package before adding it here?
I'm used to the requirement to add a package here before checking in any new code.

It changed a few weeks ago. Universe now asserts that all packages in artifacts.props and all packages in artifacts/build/ are an exact match.
FYI - this might change soon if we can get #686 done before you finish building this new package


Crash in default:log_player_action when woodcutter places sapling

Describe the bug
When a woodcutter places a sapling, the default game tries to log_player_action(), but player is nil.
To Reproduce
Create a villager. Set the job to woodcutter. Toss it a sapling. Woodcutter places it. Crash.
Expected behavior
The woodcutter places the sapling.
Log

2022-10-07 16:41:54: [Main]: Automatically selecting world at [/home/user/.minetest/worlds/test]
2022-10-07 16:41:56: ACTION[Main]: [MOD] Animalia [0.4] loaded
2022-10-07 16:41:56: WARNING[Main]: Not registering alias, item with same name is already defined: mushroom:brown_natural -> flowers:mushroom_fertile_brown
2022-10-07 16:41:56: WARNING[Main]: Not registering alias, item with same name is already defined: mushroom:red_natural -> flowers:mushroom_fertile_red
2022-10-07 16:41:56: ACTION[Main]: [working_villages] loading init
2022-10-07 16:41:56: ACTION[Main]: [working_villages] loading modutil portable
2022-10-07 16:41:56: ACTION[Main]: [working_villages] loading logging
2022-10-07 16:41:56: ACTION[Main]: [working_villages] loaded logging
2022-10-07 16:41:56: ACTION[Main]: [working_villages] loading LuaVenusCompiler
2022-10-07 16:41:56: ACTION[Main]: [working_villages] loaded LuaVenusCompiler
2022-10-07 16:41:56: ACTION[Main]: [working_villages] loading local_require
2022-10-07 16:41:56: ACTION[Main]: [working_villages] loaded local_require
2022-10-07 16:41:56: ACTION[Main]: [working_villages] setting modutil global
2022-10-07 16:41:56: ACTION[Main]: [working_villages] loaded portable
2022-10-07 16:41:56: ACTION[Main]: [working_villages] loaded init in 0.0188 s
2022-10-07 16:41:56: ACTION[Main]: [building_sign] loading...
2022-10-07 16:41:57: ACTION[Main]: [building_sign] loaded init in 0.0070 s
2022-10-07 16:41:57: ACTION[Main]: World at [/home/user/.minetest/worlds/test]
2022-10-07 16:41:57: ACTION[Main]: Server for gameid="minetest" listening on <IP_ADDRESS>:56271.
2022-10-07 16:41:58: WARNING[Main]: Texture "youngtree2branch.png" of youngtrees:youngtree2_middle has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]:   This warning can be a false-positive if unused pixels in the texture are transparent. However if it is meant to be transparent, you *MUST* update the nodedef and set use_texture_alpha = "clip"! This compatibility code will be removed in a few releases.
2022-10-07 16:41:58: WARNING[Main]: Texture "molehills_dirt.png" of molehills:molehill has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "ferns_fern_trunk_top.png" of ferns:fern_trunk has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "ferns_tree_fern_leave_big_end.png" of ferns:tree_fern_leave_big_end has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "ferns_fern_trunk_big_top.png^ferns_tree_fern_leave_big_cross.png" of ferns:fern_trunk_big_top has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "ferns_fern_trunk_big_top.png" of ferns:fern_trunk_big has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "bushes_leaves_1.png" of bushes:bushbranches1 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "bushes_leaves_2.png" of bushes:bushbranches3 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "bushes_leaves_1.png" of bushes:bushbranches2 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "bushes_leaves_2.png" of bushes:bushbranches4 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_twig_1.png" of trunks:twig_1 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_twig_2.png" of trunks:twig_2 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_twig_3.png" of trunks:twig_3 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_twig_4.png" of trunks:twig_4 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_twig_5.png" of trunks:twig_5 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_twig_7.png" of trunks:twig_7 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_twig_8.png" of trunks:twig_8 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_twig_9.png" of trunks:twig_9 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_twig_10.png" of trunks:twig_10 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_twig_11.png" of trunks:twig_11 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_twig_12.png" of trunks:twig_12 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_twig_13.png" of trunks:twig_13 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_moss.png" of trunks:moss_plain_0 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_moss_fungus.png" of trunks:moss_with_fungus_0 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_moss.png^[transformR90" of trunks:moss_plain_1 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_moss_fungus.png^[transformR90" of trunks:moss_with_fungus_1 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_moss.png^[transformR180" of trunks:moss_plain_2 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_moss_fungus.png^[transformR180" of trunks:moss_with_fungus_2 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_moss.png^[transformR270" of trunks:moss_plain_3 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "trunks_moss_fungus.png^[transformR270" of trunks:moss_with_fungus_3 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "default_tree.png^trunks_root_mask.png^[makealpha:0,0,0" of trunks:treeroot has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "default_jungletree.png^trunks_root_mask.png^[makealpha:0,0,0" of trunks:jungletreeroot has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "default_pine_tree.png^trunks_root_mask.png^[makealpha:0,0,0" of trunks:pine_treeroot has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "animalia_nest.png" of animalia:nest_song_bird has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "flowers_waterlily.png" of flowers:waterlily has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "default_sign_wall_wood.png" of working_villages:building_marker has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "default_sign_wall_wood.png" of building_sign:building_marker has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "flowers_waterlily_22.5.png" of flowers:waterlily_225 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "flowers_waterlily_45.png" of flowers:waterlily_45 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "flowers_waterlily_67.5.png" of flowers:waterlily_675 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "flowers_waterlily_small_1.png" of flowers:waterlily_s1 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "flowers_waterlily_small_2.png" of flowers:waterlily_s2 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "flowers_waterlily_small_3.png" of flowers:waterlily_s3 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "flowers_waterlily_small_4.png" of flowers:waterlily_s4 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "flowers_sunflower.png" of flowers:sunflower has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "flowers_seaweed.png" of flowers:seaweed has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "flowers_seaweed_2.png" of flowers:seaweed_2 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "flowers_seaweed_3.png" of flowers:seaweed_3 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "flowers_seaweed_4.png" of flowers:seaweed_4 has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "dryplants_grass.png" of dryplants:grass has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: WARNING[Main]: Texture "dryplants_hay.png" of dryplants:hay has transparency, assuming use_texture_alpha = "clip".
2022-10-07 16:41:58: ACTION[Server]: singleplayer [<IP_ADDRESS>] joins game. List of players: singleplayer
2022-10-07 16:42:01: ERROR[Main]: ServerError: AsyncErr: Lua: Runtime error from mod 'default' in callback luaentity_Step(): ...ds/working_villages/working_villagers/job_coroutines.lua:42: error in job_thread Runtime error from mod 'default' in callback item_OnPlace(): .../minetest/games/m
2022-10-07 16:42:01: ERROR[Main]: inetest_game/mods/default/functions.lua:730: attempt to index local 'player' (a nil value)
2022-10-07 16:42:01: ERROR[Main]: stack traceback:
2022-10-07 16:42:01: ERROR[Main]: 	.../minetest/games/minetest_game/mods/default/functions.lua:730: in function 'log_player_action'
2022-10-07 16:42:01: ERROR[Main]: 	...hare/minetest/games/minetest_game/mods/default/trees.lua:575: in function 'sapling_on_place'
2022-10-07 16:42:01: ERROR[Main]: 	...hare/minetest/games/minetest_game/mods/default/nodes.lua:734: in function <...hare/minetest/games/minetest_game/mods/default/nodes.lua:733>
2022-10-07 16:42:01: ERROR[Main]: 	[C]: in function 'resume'
2022-10-07 16:42:01: ERROR[Main]: 	...ds/working_villages/working_villagers/job_coroutines.lua:35: in function 'resume'
2022-10-07 16:42:01: ERROR[Main]: 	...minetest/mods/working_villages/working_villagers/api.lua:952: in function <...minetest/mods/working_villages/working_villagers/api.lua:937>: stack traceback:
2022-10-07 16:42:01: ERROR[Main]: 	[C]: in function 'place_node'
2022-10-07 16:42:01: ERROR[Main]: 	...ods/working_villages/working_villagers/async_actions.lua:223: in function 'place'
2022-10-07 16:42:01: ERROR[Main]: 	...s/working_villages/working_villagers/jobs/woodcutter.lua:63: in function <...s/working_villages/working_villagers/jobs/woodcutter.lua:43>
2022-10-07 16:42:01: ERROR[Main]: stack traceback:
2022-10-07 16:42:01: ERROR[Main]: 	[C]: in function 'error'
2022-10-07 16:42:01: ERROR[Main]: 	...ds/working_villages/working_villagers/job_coroutines.lua:42: in function 'resume'
2022-10-07 16:42:01: ERROR[Main]: 	...minetest/mods/working_villages/working_villagers/api.lua:952: in function <...minetest/mods/working_villages/working_villagers/api.lua:937>
2022-10-07 16:42:01: ACTION[Server]: singleplayer leaves game. List of players:
2022-10-07 16:42:02: ACTION[Main]: Server: Shutting down

Desktop (please complete the following information):
Minetest 5.6.1 (Linux)
Using Irrlicht 1.9.0mt8
Using LuaJIT 2.1.0-beta3
BUILD_TYPE=Release
RUN_IN_PLACE=0
USE_CURL=1
USE_GETTEXT=1
USE_SOUND=1
STATIC_SHAREDIR="/usr/share/minetest"
STATIC_LOCALEDIR="/usr/share/locale"

Mods activated:
animalia
anvil
biome_lib
bonemeal
creatura
mesecons
mobkit
mobs
mobs_npc
moreores
plantlife_modpack
working_villages
worldedit

Function that crashes (in default/functions.lua)
function default.log_player_action(player, ...)
	local msg = player:get_player_name()
	if player.is_fake_player or not player:is_player() then
		if not log_non_player_actions then
			return
		end
		msg = msg .. "(" .. (type(player.is_fake_player) == "string"
			and player.is_fake_player or "*") .. ")"
	end
	for _, v in ipairs({...}) do
		-- translate pos
		local part = is_pos(v) and minetest.pos_to_string(v) or v
		-- no leading spaces before punctuation marks
		msg = msg .. (string.match(part, "^[;,.]") and "" or " ") .. part
	end
	minetest.log("action",  msg)
end

This is probably a bug in the Minetest default game.  But maybe there is a way to provide a fake player.

As an update -- I dug into the minetest / default game code a bit and discovered that the issue is in the minetest default game.
The call minetest.place_node() does not include the placer as a parameter. The on_place() callbacks will always have a nil placer. I think that is the root bug.
The next layer bug is either in default.sapling_on_place() (trees.lua) or default.log_player_action().
default.sapling_on_place() checks placer for nil in a few places, but passes it to default.log_player_action().
default.log_player_action() doesn't check for nil.
This cannot be fixed in working_villagers without using a different place method.
I suppose this can be tagged as an upstream bug.


Samsung Layout changed - Android 13 - OneUI 5.1

Hi,
I have a Samsung Galaxy S20 FE with Android 13 and OneUI 5.1
I want to share with you the newest layout and possibilities that I have found. I don't know if they work to prevent killing the app.

Lock Recent App is not available anymore:

Battery optimization

Select app:

select battery:

select unrestricted:

Remove permissions if app is unused

Optimize battery usage is not available:

Already mentioned in the docu
Alarms and reminders
Maybe this is interessting for developers:

Auto-optimize daily + Adaptive power saving not available
Auto-optimize daily

Adaptive power saving

Adaptive battery

Lists of Sleeping apps, Unused apps, Deep sleeping apps, Never sleeping apps

I went through the Samsung issues and summarized them:
This issue covers: #689, #679, #618,  #451 & #430 (Android 12 but same layout?)
Empty issues: #524, #523,  #518, #516, #491, #473, #433, #350
Useless issues: #505, #499

Hi, I will edit the page, but I will probably need you to share the screenshots with me directly. Somehow, they are not downloadable as "private user content".
Could you please contact me at<EMAIL_ADDRESS>Thanks

Подскажите пожалуйста как обновить версию 9 до 12 андроид sm-A730FM для Казахстана

Hi, this benchmark app cannot update or alter your system. But there are guides that will help you upgrade the system (for example here .


[10.x] Add recursive sorting methods for a collection

I have added two new methods for sorting a collection recursively: one for ascending order sortRecursive and another for descending order sortRecursiveDesc.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Update npm package to 1.2

It's currently at 1.0.1: https://www.npmjs.com/package/c3-angular

Hmm, completely forgot about this one, I currently only use the bower stuff. Will do it as soon as possible.

This one is fixed, 1.2.0 is now in npm repo


Feature/Router

I've adjusted the router middleware interceptor, or at least made some attempt, in order to support multiple levels. In my personal case, I would only want to define middleware as meta in a parent route and have all child routes use that same middleware, including their own, if any. I am aware it's a bit longer than the original one, but I couldn't find a more efficient method without leaving the beforeEach() function.

@marknuyens thank you for this. I think I will need to have a play around with what you are suggesting. It's a difficult balance of trying to keep the examples and codebase simple to undertsand yet provide all the functionality you might need.

Closing for now to keep the codebase as generic as possible for all users.


Android: Update exported app and editor theme

Implements and Closes godotengine/godot-proposals#11071
Resolves https://github.com/godotengine/godot/issues/85110

This PR switches the theme for Android exported app to Theme.DeviceDefault.NoActionBar to better match the device’s default style. This makes the dialogs feel more consistent with the system theme, creating a smoother and more familiar experience for users.
Size comparission of old and new apk

Before and After images of native dialogs

Before this PR

After this PR, tested on a Xiaomi device

After this PR, tested on a Motorola device

Test Apk: demo app.zip
EDIT: Now It also updates the theme for Android editor.

Is it possible to set it through the theme instead, so that it's automatic for any UI element we create?

@m4gr3d Yes, I was trying to do that as well but encountered an issue with the navigation bar displaying a white color in light mode.

@m4gr3d I have updated the app theme in Themes.xml to now use Theme.DeviceDefault.NoActionBar.
<item name ="android:windowDrawsSystemBarBackgrounds">false</item>

This line removes the background of the navigation and status bars, addressing the white navigation bar issue that arose after the theme change.

A bit off-topic question but android specific. Can user force some specific version? I mean that you showed that motorola and xiaomi uses different default theme. Is it ever possible to force xiaomi use the same theme as motorola?

Tested on galaxy m14 4g, Android 14:
https://github.com/user-attachments/assets/963fe4c1-39c0-409a-98e7-ecd04941b2c6

@m4gr3d It is currently using a "fullscreen" theme, Do I need to apply a fullscreen variant of the DeviceDefault theme?

@m4gr3d I have updated the theme for Android editor and did the same hack for it too to solve the white navigation bar issue.

Thanks!


Convert raws from IMX519

Hi!
I want to capture raws as fast as possible with the imx519. As far as I understand it the fastest way is to capture raw bayer data with libcamera-raw and then process the files in a second step. I tried to use PiDNG and arducams utility script. Both don't seem to work.
Do you have a hint how to produce a DNG from the raws?
Thanks for your time!

Hi, thanks for the question. libcamera-raw aims to save raw files as fast as it can and therefore outputs the data to a giant flat file with no formatting information at all. You would have to convert this into a set of DNGs yourself afterwards.
However, libcamera-still will capture DNGS for you that you should be able to use directly. Try
libcamera-still -r -o test.jpg

which should save a test.dng file. But this method will not capture DNGs at anywhere near the same framerate.

Thanks, David!
I am just starting with bayer data in general, but my uneducated idea would be to simply store the options used for the capture in a separate file and then load both into dng_save(mem, info, payload->metadata, filename, app.CameraId(), options);
Does that make sense to you?

Ok, that sounds great! Thanks!


create-react-app logs error cipher operation failed

PS E:\VS projs\react-app> npx create-react-app app
Creating a new React app in E:\VS projs\react-apps\app.

Installing packages. This might take a couple of minutes.
Installing react, react-dom, and react-scripts with cra-template...
npm ERR! code ERR_SSL_CIPHER_OPERATION_FAILED
npm ERR! A4250000:error:1C800066:Provider routines:ossl_gcm_stream_update:cipher operation
failed:c:\ws\deps\openssl\openssl\providers\implementations\ciphers\ciphercommon_gcm.c:312:
npm ERR!

npm ERR!      C:\Users\ruturaj_k\AppData\Local\npm-cache\_logs\2022-01-30T09_11_05_521Z-debug-0.log

Aborting installation.
  npm install --no-audit --save --save-exact --loglevel error react react-dom react-scripts cra-template has failed.

Deleting generated file... node_modules
Deleting generated file... package.json
Deleting app/ from E:\VS projs\react-apps
Done.

I googled this seems no one has encountered this before can anyone explain me about this.
I tried clearing  cache "npm cache clean --force" reinstalled node js but seems no effect
node js and npm version
E:\VS projs>node -v
v17.4.0

E:\VS projs>npm -v
8.4.0

Node JS SSL ISSUE
The above solutions didn't work for me

I solved the issue , just go to windows settings -> Network reset -> reset now and hit the reset .. and restart your PC . it worked for me

I solved the issue , just go to windows settings -> Network reset -> reset now and hit the reset .. and restart your PC . it worked for me

did you solve the issue with this?

@JosueBCe  yes i did


Feature request: Extend PEM parsing to support additional formats

The PEM parsing and generation implemented in PEMDocument doesn't fully support the PEM format as defined in RFC1421.
In particular, fields such as Proc-Type and DEK-INFO are not supported, which means that formats as defined in RFC5915 cannot be processed:
-----BEGIN EC PRIVATE KEY-----
Proc-Type: 4,ENCRYPTED
DEK-Info: AES-256-CBC,A4E9C3555C149C675E03B622A42500D2

<snip>
-----END EC PRIVATE KEY-----

It would be advantageous if the processing of these additional fields were supported by the PEMDocument, as otherwise some PEM files cannot be processed properly.

Yeah this is totally an existing limitation. I believe we designed the type such that extending it for this use-case was possible, but the core feature set didn’t require it. We’d welcome a PR to add the function.


repDiversity -raref Error

Dear all, I am getting an error while using the repDiversity function with method set to raref, the issue is the same as https://github.com/immunomind/immunarch/issues/19, however I don't understand how to fix it . My data are processed after running MiXCR, and have tried in both downsampled and non-downsampled data. I have also created different objects for TCR alpha and TCR beta chain CDR3 sequences, the example given below is with TCR alpha.
I am using the 0.6.6 immunarch version.
Running  as:
imm_raref <- repDiversity(my_data$data, "raref", .verbose = T,.norm=T,.step = 1)
and the error I get is:

Error in if (Sest[1] == Sobs) { : missing value where TRUE/FALSE needed

Here is a snapshot of how my data look like:

I would appreciate any help/feedback on this!
ps. I cannot provide test data, since those  are not published yet.
Thank you! :)

Hi, @AimSchina!
My name is Aleksandr Popov, I am a developer of the Immunarch package. Thank you for using our software!
Unfortunately, we can't reproduce the bug using the dataset that comes with Immunarch. Could you provide a minimal reproducible example of your error with a small data sample? You can use this method to create a minimal reproducible example from your data: https://stackoverflow.com/questions/5963269/how-to-make-a-great-r-reproducible-example

I'm getting the same error. AimSchina where you able to fix it?

Hi, @licmn!
My name is Aleksandr Popov, I am a developer of the Immunarch package.
I see you have the same bug. Could you, please, provide me a reproducible example of your error? Small data sample and used commands with their arguments would be great. It will help me to fix the error.
Good luck,
Aleksandr

Thanks Aleksandr! This is the message I get after running the following line of code:
imm_raref <- repDiversity(my_data$data, "raref", .verbose = F)

Rarefaction analysis --- NOT WORKING---
imm_raref <- repDiversity(all_immdata_mixcr$data, "raref", .verbose = F)
Error in if (Sest[1] == Sobs) { : missing value where TRUE/FALSE needed

Hi, @licmn and @m-a-l-i-k!
Thank you for reporting this issue! Unfortunately, I still have no data sample to reproduce it. I've made an experimental fix, please install immunarch from diversity-fix branch and check it with your data:
install.packages(c("devtools", "pkgload"))
devtools::install_github("immunomind/immunarch", ref="diversity-fix")
devtools::reload(pkgload::inst("immunarch"))

When the fix will be finalized, it will be moved to dev branch; after that, to install it, use ref="dev" instead of ref="diversity-fix" in these commands.
Best regards,
Aleksandr


Where are cronical instances stored?

Related: #2 #18
I'm attempting to package Cronical with Scoop. For uninstallation, services would need to be removed with cronical.exe --remove --service-name foo. Where could I find a list of Cronical services?

Hi,
They're all registered in the Windows registry - the service-name parameter allows you to set a specific name for the service in place of the default name. Service names are visible in the Services view in Windows, if you right-click on a service, pull up properties and examine the "Service Name".
Unfortunately I don't really know of any way to enumerate these - if this is a requirement then you would probably have to enumerate all Windows services and check which ones link to a cronical.exe executable.
Thanks
Matt


Remove explanations from slo report tabbed and simple formats

Signed-off-by: Marc Perrien<EMAIL_ADDRESS>

cf #179

These messages are no longer in the output, probably removed when working on the update manifest branch that implied to refactor the output writers.
@mperrien  do you confirm we can close this one ?

@dmartin35 Yes, closing.


Adjustment of code

@qiangxue I'm sorry, but I forgot to commit to change the @cebe asked.

Please, ignore!


nixos/acme: fix subjectAltName in test snakeoil certs

Motivation for this change
nixosTests.acme has been broken since the bump to Go 1.15, as go's https client now requires subjectAltName in servers' certificates, and the snakeoil cert we were generating only had a CN, thanks to openssl's command line ~footgun~ interface. The generator script has been fixed and the certs regenerated.
Additionally, lego's passthru.tests has been set, since it was lacking before :)
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@aanderse @andrew-d @arianvp @emilazy @flokli @m1cr0man :)

Hmm, this still seems to fail for me:
webserver # [   20.170562] nixos[882]: finished switching to system configuration /nix/store/10c0lwmrh31p9ff14k4ss50y2iw8nzyd-nixos-system-webserver-20.09pre-git
(7.45 seconds)
client: must succeed: curl --cacert /tmp/ca.crt https://b.example.test/ | grep -qF 'hello world'
client #   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
client #                                  Dload  Upload   Total   Spent    Left  Speed
client #   0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
client # curl: (60) SSL certificate problem: self signed certificate in certificate chain
client # More details here: https://curl.haxx.se/docs/sslcerts.html
client #
client # curl failed to verify the legitimacy of the server and therefore could not
client # establish a secure connection to it. To learn more about this situation and
client # how to fix it, please visit the web page mentioned above.
client: output:
Test "Can add another certificate for nginx service" failed with error: "command `curl --cacert /tmp/ca.crt https://b.example.test/ | grep -qF 'hello world'` failed (exit code 1)"
error:
Traceback (most recent call last):
  File "/nix/store/yvccmvprvrds4dvahgy7ziava0bs2h38-nixos-test-driver/bin/.nixos-test-driver-wrapped", line 897, in run_tests
    exec(tests, globals())
  File "<string>", line 1, in <module>
  File "<string>", line 40, in <module>
  File "/nix/store/yvccmvprvrds4dvahgy7ziava0bs2h38-nixos-test-driver/bin/.nixos-test-driver-wrapped", line 421, in succeed
    raise Exception(
Exception: command `curl --cacert /tmp/ca.crt https://b.example.test/ | grep -qF 'hello world'` failed (exit code 1)
cleaning up
killing acme (pid 33)
killing acmeStandalone (pid 66)
killing client (pid 9)
killing dnsserver (pid 21)
killing webserver (pid 86)
(0.00 seconds)
builder for '/nix/store/kswx12850q147nkrl02d6qyhzcraafn6-vm-test-run-acme.drv' failed with exit code 1
error: build of '/nix/store/kswx12850q147nkrl02d6qyhzcraafn6-vm-test-run-acme.drv' failed

@flokli that's very odd... that's a new error i haven't seen before.
It passes for me on arch or nixos and on ofborg

@flokli your test run has a different store ptah to mine (/nix/store/kxzk2gl6kdaxv42b4hsp2myjmv7q61pv-vm-test-run-acme). Is there something different about how you're running it?

(actually weirdly mine doesn't match ofborg... not sure how that happens, we're on the same commit)

I wonder if there could be a race condition or transient failure where these lines:
      client.succeed("curl https://acme.test:15000/roots/0 > /tmp/ca.crt")
      client.succeed("curl https://acme.test:15000/intermediate-keys/0 >> /tmp/ca.crt")

Are producing the wrong output? curl doesn't have --fail so potentially they could be 404s or something

Is anyone else able to reproduce this? Or @flokli are you able to investigate?

Is anyone else able to reproduce this? Or @flokli are you able to investigate?

I had the same thing happen when working on #91121, and I can't narrow down what I did to fix it..
You could try adding acme.wait_for_unit("network.target") before the curl commands?

I've started encountering the same issue as above in #91121, despite my wait_for_unit trick and also using webserver.succeed("sync"). I believe the reason is that calling systemctl reload nginx does not imply that the config has been reloaded when it finishes, since all it does is send SIGHUP.
As explained here, a SIGHUP will cause the master process to delegate restarts of the worker processes. My bet is that this certainly takes longer than the time it takes the test scripts to go from reload -> run curl. Apachectl also sends signals, so it will not accurately block until the service is actually reloaded.
I'm not sure what the best solution is here. Adding a simple sleep would certainly solve the problem, but other than that I can only think of things like watching file descriptors on sockets or waiting for PIDs to change, which I don't know how to do cleanly.

I came up with a solution for the above problem in the other PR if anyone here is interested :)

Can we merge this for now and fix the nondeterminism later?

Sounds good to me
On Mon, Aug 31, 2020, 11:26 Jamie<EMAIL_ADDRESS>wrote:

Can we merge this for now and fix the nondeterminism later?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/96149#issuecomment-683670092, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEZNI7Z3YBKZNWDEDRYWCLSDNUDLANCNFSM4QJGZGJQ
.

Yeah grand. All the more pressure on people to merge #91121 too :wink:


Call Log History

Call log history API is available?

We don't have special API for it, but you can use this guide for implementation your feature using our API

No activity for a long time, so close.


Use Orleans with .NET Aspire

Summary
First pass at Orleans + Aspire doc
Fixes #702
Fixes #808
TODO:

[ ] Add section showing how to add multiple storage providers
[ ] Add deployment section? Unsure it's needed
[ ] Add section on extensibility: how to ensure that your Orleans providers work with .NET Aspire. The rough answer is that you add an assembly-wide Attribute and a config class to your provider library which understands how to read Config generated by Aspire and configure your provider with it.

Internal previews

📄 File
🔗 Preview link

docs/fundamentals/orleans.md
Use Orleans with .NET Aspire

This will need a TOC entry, and maybe even a hub page listing. Are the TODO's here indicating that this is a draft, or is it ready for review?

The TODOs are future improvements - they're not needed for the first version

Probably doesn't belong under fundamentals. Maybe we have another top level section for framework integrations.

@IEvangelist what are your thoughts on what the new section should be called?

The TODOs are future improvements - they're not needed for the first version

Good to know, it's probably worth placing those inside the article as well as a comment so that we have it for later - we could even create an issue to track those improvements too. Use HTML comments in the MD:
<!--
Ideas for future improvement:

- Add section showing how to add multiple storage providers
- Add deployment section? Unsure it's needed
- Add section on extensibility: how to ensure that your Orleans providers work with .NET Aspire. The rough answer is that you add an assembly-wide attribute and a config class to your provider library which understands how to read config generated by .NET Aspire and configure your provider with it.
-->

@IEvangelist what are your thoughts on what the new section should be called? "Frameworks"? Perhaps SignalR should also be in the same section. Currently, it has its own section, Real-time communication.

I like that, "Frameworks" as a new top-level node with those seems to fit nicely.

Since the SignalR page is specifically about Azure SignalR, maybe I should leave it where it is

Ok, I think this is ready, @IEvangelist. Please take another look when you're available

Thanks for getting this over the line, @IEvangelist!


Tweak giscus

Add tweaks to giscus implementation.

Will fix the merge conflicts I created

Looks good to go now, thanks @Remi-Gau.


(fix): Typo fix in CODE_OF_CONDUCT.md and README.md

I fixed some typos prepositional errors

Thanks for the PR @SinithH !! Can you sign your commit, here DCO check is failing

I signed it.

@SinithH , thanks. But you seem to have added multiple other unintended files. Can you raise a fresh PR with changes to just the CODE_OF_CONDUCT?

Done

Thanks for taking part in Hacktoberfest 2019. Please fill up this form to claim your swag https://hubs.ly/H0l5ldm0


UnparsedFlagAccessError: Trying to access flag cp_profile_file before flags were  parsed.

Tried installing or-tools on Windows and running an example. Did I miss anything?

Works only with 64-bit Python


drivers: usb_dc_nrfx: free memory block if event is intended to be dropped

Do not try to flush event queue in ISR.
Free memory block if event is intended to be dropped.
Resolves: #20953

I accepts this code change as it seems reasonable but from my pov it is fixing the board imperfection as it generates huge amount of unnecessary events what is strange to me.

No, the driver has a issue, it does not free the memory block. This patch fixes this issue, it is not a workaround!!!


mwc-textfield type date spinner alignment incorrect

Describe the bug
In chrome, mwc-textfield with type="date" displays up and down spinner buttons on hover/focus. These buttons do not line up with the date, drop arrow, or icons. This doesn't seem to be a bug when the textfield is filled.
To Reproduce
Steps to reproduce the behavior:

Modify any of mwc-textfield's in the demo (except the filled ones) to have type="date"
Hover or focus on the textfield.

Expected behavior
spinner buttons should align with other elements in the row
Screenshots

Browser Version (please complete the following information):

OS: Windows 10
Browser chrome
Version 80.0.3987.163

Additional context
Seems to have broken in 0.15.0
Can confirm it is not an issue in 0.14.1

Addressed in M3, though keep in mind that date text field types aren't fully supported.


Responsive Design - the main Login button is out of frame in Firefox + Linux MX

The Login button is out of view!

Same with Checkout.  Perhaps the Table also is unbounded, good to check.

No longer an issue, but great to make all proportions look good, over time.


[type: refactor] refactor for integration test

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #1940 (6cbf46d) into master (a05b105) will decrease coverage by 0.31%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1940      +/-   ##
============================================
- Coverage     63.74%   63.43%   -0.32%
+ Complexity     3956     3945      -11
============================================
  Files           574      574
  Lines         16739    16736       -3
  Branches       1595     1593       -2
============================================
- Hits          10670    10616      -54
- Misses         5216     5265      +49
- Partials        853      855       +2

Impacted Files
Coverage Δ

...ache/shenyu/client/tars/common/dto/TarsRpcExt.java
0.00% <0.00%> (-30.77%)
:arrow_down:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
64.70% <0.00%> (-29.42%)
:arrow_down:

...enyu/client/tars/TarsServiceBeanPostProcessor.java
36.76% <0.00%> (-19.12%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
63.10% <0.00%> (-11.66%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
56.84% <0.00%> (-6.32%)
:arrow_down:

...henyu/plugin/httpclient/NettyHttpClientPlugin.java
58.00% <0.00%> (-2.00%)
:arrow_down:

...che/shenyu/admin/model/vo/SelectorConditionVO.java
96.07% <0.00%> (-1.97%)
:arrow_down:

...t/zookeeper/ZookeeperClientRegisterRepository.java
70.90% <0.00%> (-1.82%)
:arrow_down:

...elimiter/handler/RateLimiterPluginDataHandler.java
69.69% <0.00%> (-1.52%)
:arrow_down:

... and 4 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 a05b105...6cbf46d. Read the comment docs.


Connection timeouts

What does this PR implement/change/remove?
Wraps the Intel AMT provider connection setup to return on context deadline.
Adds a connection timeout parameter for opening connection,
As of now, a user may choose to pass in a context deadline into the bmclib client, which
is shared between all the bmclib providers that is - when creating an a new connection/session on a BMC.
The shared timeout value entails that a single provider can attempt to connect for the the whole timeout deadline, and so leaving the other providers (which may be the successful ones) with much lesser time to connect and carry out their operation.
The WithConnectTimeout() client option sets the timeout when connecting to a BMC to create a new session.
When not defined the deadline value passed in (if any) through the context applies.
This timeout value will best work when its lesser than the passed in context deadline value.
Checklist

[ ] Tests added
[ ] Similar commits squashed

The HW vendor this change applies to (if applicable)

Intel AMT

What version of tooling - vendor specific or opensource does this change depend on (if applicable)
Description for changelog/release notes

@jacobweinstock let me know if you have any thoughts here.

@jacobweinstock let me know if you have any thoughts here.

Hey @joelrebel, thanks for tagging me. Looking now.

@jacobweinstock the latest changes pass the connection timeout as you suggested,
for the AMT changes, should I revert what I have in this PR?

@jacobweinstock the latest changes pass the connection timeout as you suggested, for the AMT changes, should I revert what I have in this PR?

Hey @joelrebel. Thanks, this looks good. Yeah, if you don't mind reverting the AMT changes I've got a PR ready to follow up with the library changes.

@jacobweinstock, I've reverted intelAMT changes in a72d5d6


[APIReview] Should OrderBy be a collection of strings?

Should the OrderBy field in MobileTableQuery[Options] be a collection of strings?

Dropped due to move back to LINQ provider.


Go To Definition Ctrl + Left - Click Not Working

VS Code Version:  1.90.0 89de5a8d4d6205e5b11647eb6a74844ca23d2573 x64
OS Version: Linux Mint 21.3 x86_64

Steps to Reproduce:

Disable all my extensions
Use F12 or Ctrl + F12 or Ctrl + Left - Click does not work
Enable all my extensions just work F12. Ctrl + Left - Click does not work in any case.

My vscode extension list:
cheshirekow.cmake-format
codeium.codeium
continue.continue
emeraldwalk.runonsave
equinusocio.vsc-material-theme-icons
franneck94.c-cpp-runner
jeff-hykin.better-cpp-syntax
josetr.cmake-language-support-vscode
lextudio.restructuredtext
llvm-vs-code-extensions.vscode-clangd
mhutchie.git-graph
ms-azuretools.vscode-docker
ms-dotnettools.vscode-dotnet-runtime
ms-iot.vscode-ros
ms-python.autopep8
ms-python.debugpy
ms-python.pylint
ms-python.python
ms-python.vscode-pylance
ms-toolsai.jupyter-keymap
ms-vscode-remote.remote-containers
ms-vscode-remote.remote-ssh
ms-vscode-remote.remote-ssh-edit
ms-vscode.cmake-tools
ms-vscode.cpptools
ms-vscode.cpptools-extension-pack
ms-vscode.cpptools-themes
ms-vscode.remote-explorer
pkief.material-icon-theme
redhat.vscode-xml
shakram02.bash-beautify
smcpeak.default-keys-windows
swyddfa.esbonio
trond-snekvik.simple-rst
twxs.cmake
vadimcn.vscode-lldb
vscode-icons-team.vscode-icons
wk-j.save-and-run
xaver.clang-format

/needsMoreInfo
What is language and in what code samples it was a problem. Please provide code and steps to reproduce.

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

C++ and Python are the programming Language that I could test.
Python File:
`class Prueba:
def init(self):
pass
def __str__(self):
    pass

def my_custom_function(self):
    print('I am a custom function')

def main():
my_class = Prueba()
my_class.my_custom_function()
if name == "main":
main()`
C++ File:
`#include
#include
class Prueba {
public:
Prueba() {} // Constructor
std::string toString() { return ""; } // Función equivalente a str
void myCustomFunction() {
    std::cout << "I am a custom function" << std::endl;
}

};
int main() {
Prueba myClass;
myClass.myCustomFunction();
return 0;
}`
In both cases If i use F12 on myCustomFunction in the main function, its works. But not Ctrl + Left-Click or Ctrl + F12

Any update of this bug?

Same problem here.

¿What other information do you need?


Clean up jupyter notebooks

Some can be scripted, some consolidated, some deleted

just fyi, fill_betweeen examples are in learning_curves/validation_curves


bngblaster DHCPv6 reporting wrong sessions established

HI there, thanks for the amazing tool you've released for free, been using it
Describe the bug
I still need to read into the RFCs to see if this is the expected behavior with dhcpv6 however dhcpv6 sessions are showing as completed when the dhcpv6 server sends a reply with a NoBinding (Option 13 - OPTION_STATUS_CODE, sub-option 3 "NoBinding" https://www.rfc-editor.org/rfc/rfc8415.html#section-21.13)
I was expecting the bngblaster to reattempt dhcpv6 if there is no binding but maybe I need to read up more on the RFC
To Reproduce
Setup any DHCPv6 server that would return a NoBinding in the v6 reply, specifically in my scenario I am authenticating v4 sessions on a Cisco ASR9k and CPE behavior is that it attempts dhcpv6 only after v4, I can't reproduce the same scenario on bngblaster because it seems to send both address families exactly at the same time however I'd at least expect a NoBinding to not officially think the dhcpv6 session is complete and reattempt it again
Version (bngblaster -v):
bngblaster -v
Version: 0.8.8
Compiler: GNU (11.3.0)
IO Modes: packet_mmap_raw (default), packet_mmap, raw

JSON configuration:
apologies, I've been trying different things to try to stagger the dhcp requests
{
        "interfaces": {
                "access": {
                        "interface": "ens20",
                        "type": "ipoe",
                        "outer-vlan-min": 1000,
                        "outer-vlan-max": 1002,
                        "inner-vlan-min": 2,
                        "inner-vlan-max": 4094,
                        "ipv4": true,
                        "ipv6": true,
                        "dhcp": true,
                        "dhcpv6": true
                }
        },
        "ipoe": {
                "arp-timeout": 10
        },
        "dhcp": {
                "enable": true,
                "broadcast": true,
                "timeout": 20,
                "retry": 30,
                "release-interval": 60,
                "release-retry": 10
        },
        "dhcpv6": {
                "enable": true,
                "timeout": 30,
                "retry": 10,
                "access-line": true,
                "rapid-commit": true
        },
        "access-line": {
                "agent-remote-id": "bng-demo-{session-global}",
                "agent-circuit-id": "<IP_ADDRESS>/<IP_ADDRESS> eth0:{session-global}",
                "rate-up": 1024,
                "rate-down": 16384
        }
}

Steps to reproduce the behavior:

Run the command bngblaster -C config.json -I -c 1000
DHCPv6 sessions will all complete upon receiving NoBinding in the dhcpv6 reply

Expected behavior
I expect the dhcpv6 to reattempt dhcpv6 process when receiving a NoBinding
Screenshots
bngblaster shows 4000 dhcpv6 sessions established (apologies example pictures are when I was testing 4k clients)

Dhcpv6 server returning NoBinding in the dhcpv6 advertise and also even the reply

Additional context
I can provide packet captures if required with and without rapid-commit, similar behavior if I leave that feature enabled or not, thanks!

A capture file would be great! You can attach the capture here or send to<EMAIL_ADDRESS>

This issue should be fixed in latest dev branch (765e797). Could you please verify?
Jan 19 13:56:13.527643 DHCPv6 (ID: 1) Start DHCPv6
Jan 19 13:56:13.527647 DHCPv6 (ID: 1) DHCPv6-Solicit send
Jan 19 13:56:13.530254 DHCPv6 (ID: 1) DHCPv6-Reply received
Jan 19 13:56:13.530285 DHCPv6 (ID: 1) IA_PD received with status code 6 (NoPrefixAvail)

Hi, apologies for not getting back with a capture file, I've just built the dev branch and can confirm I now see expected results, DHCPv4 creating the sessions and dhcpv6 reattempting and therefore successfully getting an IA prefix
Thank you so much, this will really help for testing in our lab and getting ipv6 deployed to all of our customers :-)

I just released a new version 0.8.11 including this fix.


UTF-8 encoding not compatible with RFC4180; not clear

RFC4180 states that the acceptable bytes of a CSV field are
TEXTDATA =  %x20-21 / %x23-2B / %x2D-7E
(also COMMA / CR / LF / 2DQUOTE if the field is quoted)
Any UTF-8 encoded string which is not equivalent to its (7-bit) ASCII encoding will not match this pattern. In addition, it is not clear whether UTF-8 encoded strings which contain control characters other than CR/LF are permissable or not.
--
It should be clearly documented that this change from the RFC has occurred. I would expect TEXTDATA to be redefined as something like "a valid UTF-8 code unit that is not COMMA, CR, LF, or DQUOTE"
This would also permit the use of control characters; i.e. characters 0-31 and 127, excluding CR and LF. Via the RFC, it is currently forbidden to use tabs in fields, for example.
It is also worth noting that the RFC also mandates:

commas as separators
using CRLF as a line ending
These are also potential areas where the dataprotocols spec permits the file to violate the RFC.

--
this is mostly nargling over details, mind.

@scraperdragon good point. Would you like to submit a pull request or do you have specific wording changes you would recommend?

@danfowler @rgrp can we move to clarify this? UTF-8 encoding and alternate separators are so important to the application of CSV and the data protocol specs in general, that we should be explicit in how we diverge from RFC4180.

@pwalsh @scraperdragon what do we think of changing the start of the CSV section to the following:
CSV files included in a Tabular Data Package package MUST conform to [RFC 4180
"Common Format and MIME Type for Comma-Separated Values (CSV) Files"][rfc4180]
subject to the following exceptions and additions.

Exceptions:

-   Files MUST be encoded as UTF-8 (the RFC requires 7-bit ASCII)
-   The standard line terminator character can be LF or CRLF (the RFC allows CRLF only)-   Files MAY (but SHOULD NOT) deviate from standard CSV in terms of delimiters
    (e.g. tab rather than ","), quote characters and other parameters (see below)

@rgrp LGTM


Border color

Is there a way to change the border color of the Login/Logout button? I would prefer it if it would look like a normal button.

You would have to modify the html code and submit a PR. Due to time contains not on my list of things to do our help with but would take a PR.


Button onClick prop updated.

I'm new in React Native and I couldn't run the alert command with onClick prop and I didn't find that prop in React Native documentation either.
http://facebook.github.io/react-native/docs/button.html#onpress
https://github.com/facebook/react-native/blob/master/Libraries/Components/Button.js

Thanks, you're right, onClick has been deprecated a long time ago!


Connection dialog not always connect signal to node

Godot version:
3.2.dev.custom_build. 197b65f32
OS/device including version:
Ubuntu 19.04
Issue description:
Sometimes when I try to connect signal to node, Godot shows error in Output tab and doesn't connect signal
ERROR: _make_or_edit_connection: Condition ' !it ' is true.
   At: editor/connections_dialog.cpp:475.

Steps to reproduce:

CC @YeldhamDev maybe since you poked at this class recently.

Been trying for a while, and I'm completely unable to reproduce this. Maybe my latest commit fixed this? If not, could you describe exactly how you're clicking on the tree item when it happens? Maybe it's deselecting it somehow?

I still have this error.
With the highest probability (maybe 10 - 20%) this error occurs when I try to connect signal with root of scene.
The steps to reproduce are rather simple
1 click at node
2 fast clicks at the signal
2 fast clicks at destination node

Does it only happen when double clicking?

Probably yes,
I found that sometimes Connect button just disable itself when connection window appear, and then this error occurs

Still unable to reproduce it, but here is a little analysis:

The _make_or_edit_connection method fails because there's no item selected in the tree.
That method is only called by ok_pressed, which needs an item to be selected in the tree to work.
So somehow, the item is being deselected between those two functions. Now to find out how.

Ok, let's try a solution: I assume that you know how to compile the engine, right?
So, put set_hide_on_ok(false); on the ConnectDialog::ConnectDialog() function in the "editor/connections_dialog.cpp" file. Then try reproducing the bug again.

Still error occurs.

Closing due to lack of update. Please comment if you can still reproduce this issue in the latest stable version of Godot.

I had this issue today in 3.5.stable when connecting signals in quick succession. Specifically, my setup is the following:

When connecting each of the pressed() signals to the MainMenu script one after another, I would sometimes get this error after clicking the Connect button. It seemed, qualitatively, like it happened if I double clicked the signal, then clicked Connect quickly.

Ah just spotted this issue has been re-opened at https://github.com/godotengine/godot/issues/63710.


Update packages; remove unused code

Pull Request Test Coverage Report for Build 72

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 22.043%

Totals

Change from base Build 70:
0.0%

Covered Lines:
41

Relevant Lines:
186

💛  - Coveralls


Render as Markdown

It would be nice to render markdown styles, at least the basics like links, bold ...

In normal text, being able to see the *test* render as italic, for instance?

Text with * are display in regular text.
No one formatation is displayed.

The goal would be to see it formatted as if it was in a markdown file, though, right?

Yes, right. Maybe basic format.
Most used are bold and links.

I've got this working, but the only question left is should completed or cancelled tasks also show the syntax highlighting for markdown.
(i'm using the afterglow-monokai syntax theme)
Plain ol' task:

Completed with highlighting:

Completed without highlighting:

My vote is on removing highlighting upon complete/cancel, what are your thoughts?

My vote is to removing :+1:

Also keen to see markdown highlighting (also vote for removing highlighting on done tasks)

I'll be posting the update as soon as I get a chance! I ran into some awkward issues with Atom's built-in markdown rendering reporting itself as gfm instead of markdown which also lead to some awkward styling decisions by many of the themes available. I'm not sure how to approach that bit, but my solution will work in the interim until a more clean plan is in place!