chore: support selenium 4.10.0+

SUMMARY
This is a proper fix for the PR https://github.com/apache/superset/pull/24538.
However we have to think about how the WEBDRIVER_CONFIGURATION should be. The selenium 4.10+ only allows options and services to be passed to the WebDriver constructor.
https://github.com/SeleniumHQ/selenium/blob/c14d9678990942b93cb421c5567d0da7fb29c7bd/py/selenium/webdriver/firefox/webdriver.py#L41-L46
https://github.com/apache/superset/blob/a698587e8c867f25f4cace6ec02259cccaa5986c/superset/utils/webdriver.py#L276-L279
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Thanks @gnought for the change. As your point out with Selenium 4.10+ the WebDriver signature changed which means that existing WEBDRIVER_CONFIGURATION definitions (including the default) could fail and thus this would be deemed a breaking change. It seems prudent to hold off this change until when the breaking window opens for Superset 4.0.

FYI: if the change is made it should be updated to 4.14.x (or newer) to fix a CVE finding (refer linked https://github.com/apache/superset/issues/26992)

Unfortunately, I think this missed the proposal window for breaking changes, and will have to go into 5.0. I'l add it to that project board now. I'm also not sure where we stand with the potential migration to Playwright (thus tagging @kgabryje here) and if that will become the standard by 5.0. If we want to proceed with this upgrade, which seems sensible, this PR will just need a rebase and bump to 4.14.x or greater by the time that breaking window re-opens.

This has also been updated to auto-close https://github.com/apache/superset/issues/26992 when it does become mergeable.

Hi @rusackas
the pull request here seems to update it to V4.10.x - only have in mind it should be updated to >= V4.14.x (newer as it is in this pull request)

FYI I fixed the merge conflicts, can we remove the hold on this one?

FYI I fixed the merge conflicts, can we remove the hold on this one?

@mistercrunch We can't remove the hold label if the PR contains a breaking change. That will only be possible during a breaking window.

@gnought we want to push this through for Superset 5.0 (we're approaching the breaking window during which this can be merged). Would you be able to rebase this PR and fix failing CI checks?

@gnought we want to push this through for Superset 5.0 (we're approaching the breaking window during which this can be merged). Would you be able to rebase this PR and fix failing CI checks?

@villebro done.


0.1.1.

Same here....not working with hacs

@sander030 Sorry for the I just released a Hotfix which should solve the installation issue

Cant find it in integrations in hassio

Cant find it in integrations in hassio

Dear @JePeSWE The integration doesn't come natively with hassio, but it's available through the Home Assistant Community Store (HACS). I've made a quick tutorial that walks you through how to install the integration using HACS: https://codingcyclist.medium.com/how-to-install-any-custom-component-from-github-in-less-than-5-minutes-ad84e6dc56ff


Belongs To Eager Loading Soft Deleted: withTrashed() is ignored when sorting

Summary & Code snippet of problem or feature request
There are 2 Eloquent models with the following relationship:

Post belongsTo Thread

Thread can be soft-deleted (https://laravel.com/docs/5.7/eloquent#soft-deleting).
I am using Datatables to display posts. I "eager load" threads (similarly to your example), but because threads can be soft-deleted - I had to add withTrashed():
        $posts = Post::select('posts.*')
            ->with(['thread' => function ($query) {
                $query->withTrashed();
            }]);

... to avoid errors. And everything works fine, 'thread.title' column is there and the posts that belong to the soft-deleted threads are also listed.
But when I try to sort by the 'thread.title' column - withTrashed()  is ignored and Datatables shows only posts that belong to threads that have not been deleted (but it still counts all the posts including those that belong to threads have been deleted).

For example, lets say that there are 9 posts stored in the DB (in posts table), of which 2 belong to thread that has been soft-deleted. Datatables will show all 9 posts, and at the bottom there will be:
Showing 1 to 9 of 9 entries

Then I click on the 'thread.title' column to order by the thread titles and Datatables will show 7 posts (that belong to threads that have not been deleted). The remaining 2 posts that belong to thread that has been deleted are not shown (loaded) anymore.
But it will still show:
Showing 1 to 9 of 9 entries

at the bottom.

Initially, these queries are executed:

select count(*) as aggregate from (select '1' as `row_count` from `posts`) count_row_table // 9
select `posts`.* from `posts` order by `id` desc limit 10 offset 0
select * from `threads` where `threads`.`id` in ('1', '2', '3') // '3' has been soft-deleted

... and all 9 posts were displayed ( including two belonging to the soft-deleted thread).

After sorting by the 'thread.title' column:

select count(*) as aggregate from (select '1' as `row_count` from `posts`) count_row_table // 9
select `posts`.* from `posts` left join `threads` on `posts`.`thread_id` = `threads`.`id` where `threads`.`deleted_at` is null order by `threads`.`title` asc limit 10 offset 0 // `withTrashed()` is ignored and "where `threads`.`deleted_at` is null" was added
select * from `threads` where `threads`.`id` in ('1', '2') // '3' is not specified anymore (like initially was) because `withTrashed()` is ignored

... only 7 posts were displayed (that belong to threads that have not been soft-deleted). But it will still display "Showing 1 to 9 of 9 entries"...

System details

Operating System: Windows 7 64 bit
PHP Version: 7.2 (XAMPP)
Laravel Version: 5.6.39
Laravel-Datatables Version: 8.9.1

@yajra I just tried suggested workaround and it did not work, withTrashed() is sill being ignored after sorting by the 'thread.title' column (the same as I described above).
    public function thread()
    {
        return $this->belongsTo(Thread::class)->withTrashed();
    }

@yajra withTrashed() method is ignored, aswell as onlyTrashed()
That's happens due to 1628 PR
In EloquentDataTable at the end of performJoin method we have
if ($deletedAt !== false) { $this->getBaseQueryBuilder()->whereNull($deletedAt); }
$deletedAt variable always has value with a field name if related model using SoftDelete. No matter what kind or relation is set.
So the problem is:

It works well when we don't want to have deleted records in results
If model relation is set with withTrashed(), $deletedAt in not false and we applying ->whereNull($deletedAt), but we shouldn't
If model relation is set with onlyTrashed() we still got $deletedAt with field name and applying ->whereNull($deletedAt) instead of ->whereNotNull($deletedAt)

This happens only with joining tables, not with counting records. That's why total records not equals to displayed results sometimes.
I've tried to remove this whole comparison part with $deletedAt. Relations with withTrashed() and onlyTrashed() works like a charm and just like they should to. Tested with Laravel 5.5 and Laravel-Datatables v8.13.6. Queries in Debugbar were correct.
Since as from v8.0.0 we got
On the fly support for SoftDeletes. No need to use withTrashed and onlyTrashed.
I think this part of a code looks redundant and should be removed.
Same issue with #1692

@selecod can you please submit a PR on the solution you've mentioned. Will try to check it as soon as I can. Thanks!

@yajra sure, here it is
#2103
The same can be applied to 9.0 branch, if everything is fine.


crypto/subtle: No Constant Time Comparison For int64

Several well-used programs in Golang use int64 as an identifier, but do not use constant time comparison when authenticating. This could be used to leak information to an adversary (potentially). Unfortunately, crypto/subtle does not have a constant-time comparison algorithm for int64, which would clearly be useful to have.
I do not have a clear understanding of how things work "under the hood" in Golang, so I do not trust myself to write a proper constant time int64 comparison algorithm for crypto/subtle. However, I think one could be easily implemented/adapted.
@FiloSottile @rsc @agl It looks like you guys know what you're doing on crypto/subtle. Could you help out here?

Do you just want eq/neq, or do you need ordering?
For eq/neq ConstantTimeEq(int32(x), int32(y)) & ConstantTimeEq(int32(x>>32), int32(y>>32)) will work.
for ordering, I think (ConstantTimeLessOrEq(int32(x>>32), int32(y>>32)) & ^ConstantTimeEq(int32(x>>32),int32(y>>32))) | (ConstantTimeEq(int32(x>>32),int32(y>>32)) & ConstantTimeLessOrEq(int32(x), int32(y))) might work.

Eq/neq is all I need (sorry for being ambiguous).


SimpleMarketplace.sol is  rejected by Blockchain Workbench

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [ x] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Download SimpleMarketplace.sol from [https://github.com/Azure-Samples/blockchain/blob/master/blockchain-workbench/application-and-smart-contract-samples/simple-marketplace/ethereum/SimpleMarketplace.sol]

Try to upload the sample smart contract in Azure Workbench to create a new application.
Any log messages given by the failure

We found these errors in your contract code. Resolve them and upload the file again.

{"Errors":["Workflow contract SimpleMarketplace should not derive from WorkbenchBase base contract. This is considered an error now. Update the SimpleMarketplace contract to remove WorkbenchBase and calls to "ContractCreated" and "ContractUpdated" functions. "],"Warnings":[]}
Expected/desired behavior

sucessfull creation of sample application.

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)
Windows 10 with Chrome

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

Greetings,
Thank you for your message.  I have not been able to reproduce the error with the existing application and on the latest version of Workbench.
It looks like you may be having an older version of the SimpleMarketplace application because the one in the link, which is also the current one, does not have Workbench base nor the function calls.  Please try with the latest versions and let us know how that goes.
Thanks.

Hello @amrikmahal,
I am writing to inform you that I will be closing this issue since it has been open for more than 30 days.  Please feel free to re-open it if required.
Thanks,
Ramya


⚠ Secret Website 3 has degraded performance

In ab176cf, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 21775 ms

Resolved: Secret Website 3 performance has improved in 5fdc4ff after 30 minutes.


Victor motors

Changed Talon to Victor motors

The code looks good but needs to be rebased and the merge conflicts need to be dealt with. The following should help:
git checkout main
git pull
git checkout  Victor-Motors
git rebase main

And then go through the process from last week of using git status to view the conflicting files, resolve conflicts in vscode, and use git add .


Optional actor-level causality tracking

Overview
The default for tracking happened-before relationships (= potential causality) of events is described in #68. Briefly, event-sourced actors on the same local event log share a vector clock entry. Consequently, the strong clock condition doesn't hold in this context. Concurrency of events can only be reliably detected if they have been emitted at different locations. Although this is reasonable default for many applications, others need a more fine-grained tracking of potential causality.
Individual EventsourcedActors (EAs) should be able to reserve their own entry in a vector clock, instead of sharing it with others on the same local event log. This should be possible by overriding a sharedClockEntry method to return false. When overridden, concurrency of events can be reliably detected even if they have been emitted at the same location.
Formalism
The causality tracking default, described in #68, satisfies the plausible clock condition which includes the weak clock condition. If all sources of concurrent activity (EAs) have their own entry in the vector clock, the strong clock condition holds. Given a function T that maps events to their vector timestamps, for any two events x and y, there is an isomorphism between vector timestamps and the causality relation between these events (see also VectorTime operators):

x = y ⇔ T(x) equiv T(y) (equivalent)
x → y ⇔ T(x) < T(y) (happened-before)
x ↔ y ⇔ T(x) conc T(y) (concurrent)

Given a subset of EAs that have their own entry in a vector clock and a subset of events E that is routed between these EAs, the strong clock condition also holds for all events in E, if these EAs do not additionally consume events for which the strong clock condition doesn't hold.

Optional actor level causality tracking has been disabled (see #280)


"let" declarations aren't part of UHDM model

Test: Assertions
let declarations are not part of UHDM model.

let expressions have to be inlined like macros.
They are not going to show in the UHDM model, they are going to be inlined with parameter substituted.

let expressions have to be inlined like macros.
They are not going to show in the UHDM model, they are going to be inlined with parameter substituted.

What is design::Let_decl supposed to be then? If all let expressions are inlined then this call would always return an empty vector.


Tutorial Request

Thank you for creating and sharing this code. In your description, you describe after scraping from Covers.com, etc., one applies your code to that scraped data. Would you be so kind as to describe or point to how to scrape, what tools, perhaps a tutorial? I'm really looking forward to experimenting with your tools but I am not sure of how to start. Thank you so much.

Adding myself to this as I am also interested in the samething.

Waaaazzzzzuuuuuupppppppp......  90s babie here would love to also know the same damn thing. :)

Awesome

OddsJam has an api that is affordable and well documented.
This library handles the quantitative functionality of data you have fetched, and I believe they should remain separate. this issue should be marked stale


[docs] azure_identity docs seem to be wrong.

I'm trying to do something very simple: Use use the Azure CLI to print out an access token.
In trying to get something going, I'm following the official example on the azure_identity docs on crates.io, it shows:
let credential = DefaultAzureCredential::default();
let response = credential
    .get_token(&["https://management.azure.com/.default"])
    .await?;

credential is an instance of DefaultAzureCredential, which does not have a *public* .get_token() method. Using default(), the resulting object looks like:
DefaultAzureCredential { sources: [Environment(EnvironmentCredential { http_client: Client { accepts: Accepts, proxies: [Proxy(System({}), None)], referer: true, default_headers: {"accept": "*/*"} }, options: TokenCredentialOptions { authority_host: Url { scheme: "https", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("login.microsoftonline.com")), port: None, path: "/", query: None, fragment: None } }, cache: TokenCache(RwLock { value: {} }) }), ManagedIdentity(ImdsManagedIdentityCredential { http_client: Client { accepts: Accepts, proxies: [Proxy(System({}), None)], referer: true, default_headers: {"accept": "*/*"} }, object_id: None, client_id: None, msi_res_id: None, cache: TokenCache(RwLock { value: {} }) }), AzureCli(AzureCliCredential { cache: TokenCache(RwLock { value: {} }) })], cache: TokenCache(RwLock { value: {} }) }

So I then look at the README.md for the azure_identity package in this repository and it instead shows:
let credential = azure_identity::create_credential()?;

The only create_credential() method I see is in the token_credentials::specific_azure_credential file. This example doesn't work either.
So what is the correct way to get a token? There are plenty of get_token() and get_access_token() methods all over, but they're all private.

get_token is exposed via the TokenCredential trait, which the azure-identity credential providers implement.
Import azure_core::auth::TokenCredential and you'll have access to get_token.

That's interesting. The following code works:
    let credential = DefaultAzureCredential::default();
    let token = credential
        .get_token(&["https://management.azure.com/.default"])
        .await?;
    println!("token: {:?}", token.token.secret());

But after creating the credential, rust-analyzer doesn't seem to know about any of that functionality:

"get_token" listed in completions is just sourced from existing text in the buffer.
Same with VS Code:

I don't have buffer text completion configured in VS Code so there's no completions available at all.
I don't know much about how rust-analyzer works, but can you reproduce this?

I'm not familiar with using that portion of VS Code + rust-analyzer.  With VS Code + rust-analyzer, I can confirm it is able to find the implementation using this example:
use azure_identity::DefaultAzureCredential;
use azure_core::auth::TokenCredential;

fn main() {
    let credential = DefaultAzureCredential::default();
    let _ = credential.get_token(&[".default"]);
}

As such, addressing issues in rust-analyzer+vscode is out of scope for the Azure SDK for Rust.

Fair enough, if your environment shows correct autocompletions then I suspect something is off on my end.
Thanks for the help!


Unable to render this definition

I'm using Unraid, and updated to the latest version, but still get the same error:

Could it be browser caching? Have you tried CTRL + SHIFT + R?

Could it be browser caching? Have you tried CTRL + SHIFT + R?

Oh yes, I forgot it. thx~


window.innerHeight breaks on Firefox and Edge

Probably the same root cause as https://github.com/Microsoft/PowerBI-visuals-tools/issues/81 (and #38) however in this case it also fails on Edge.
Microsoft Edge 38.14393.0.0
Microsoft EdgeHTML 14.14393
Mozilla Firefox 51.0.1 (32-bit)
    export class Visual implements IVisual {
        constructor(options: VisualConstructorOptions) {
            console.log("innerHeight test");
            try {
                console.log("innerHeight ok", window.innerHeight);
            } catch (e) {
                console.error("innerHeight err", e);
            }
        }
    }

Firefox:

TypeError: 'get innerHeight' called on an object that does not implement interface Window.

Edge:

TypeError: Invalid calling object

Closing due to age


Unable to use userFollowsBroadcaster (deprecated endpoint)

Bug Report
Code

const isFollower = await apiClient.users.userFollowsBroadcaster('userId', 'broadcasterId')

Expected behavior

The function uses the new endpoint GET https://api.twitch.tv/helix/channels/followed as detailed here: https://dev.twitch.tv/docs/api/reference/#get-followed-channels and successfully returns a boolean.
Actual Behavior

The function is using the deprecated endpoint GET https://api.twitch.tv/helix/users/follows as detailed here: https://dev.twitch.tv/docs/api/reference/#get-users-follows and giving a 404 error because I'm using a client ID created after February 17, 2023. This endpoint will also be decommissioned on August 3, 2023.
Environment

Version: 6.0.1
Node version: 18.13.0
Browser version: Chrome 110.0.5481.105
Operating system: Windows 11

The method will not be changed to use the new endpoint, as people who have used the endpoint before can still use it. Instead, there will be a new method.


chore(package): update travis-deploy-once to version 5.0.4

Coverage remained the same at 81.739% when pulling c950a00a3ddcfbc9eabee7da0c54a9d5f3d1973b on greenkeeper/travis-deploy-once-5.0.4 into ee90cd82ce583d24dd23314708be8ab5d990fb46 on master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[roles-0.1.0] Create new Top Hat

Closes #2034

@mudrila there's nothing even in this PR yet!

@mudrila there's nothing even in this PR yet!

Lol I thought that was the intention 🤣

Touche

This doesn't need to be open yet


nixos/tox-node: init

Motivation for this change
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] 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 nix-review --run "nix-review wip"
[x] 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)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Right now it uses the same uid as tox-bootstrapd, this probably should be changed.

r?

r?

You'll have to elaborate on that.

Github says “changes required” even though I've applied necessary changes.

Aside from my notes on hard-coded bootstrap nodes this PR looks fine.
A changelog entry for 19.09 would be great too!

Looks great now! Thanks for your contribution :)

This PR broke tarball build due to the missing descriptions on the module options.

The grahamcofborg-eval-nixpkgs-tarball was reported as successful, is that the same thing? If so, why the discrepancy in outcome?

@joachifm since it happens at build time no, ofborg doesn't build untrusted inputs automatically, only evals untrusted inputs automatically.

Whoops.. Descriptions added in 9a176d669a21a8dfccee07c8d4d81cc35cabcc39

Thanks @adisbladis!


Stack 296 , A10_timezone example

This is the exact same as the PR#76. It's just missing the bootimage playbook


Source Protection

Secrets (STACS + Gitleaks)
Commit Lint

:tada: This issue has been resolved in version 1.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


add h5py and eccodes

Description
This PR updates the Dockerfile for gnu and clang containers to:

build NCEPLIBS-bufr python package manually. I had to implement a hacky way to build it in dist-package instead of site-package. I'm not sure if there is a better way for building this package or why building it in site-package was unsuccessful.
build eccodes and h5py python packages. This can be replaced with the new method of building python packages in jedi-stack in the future.
add VOLUME /var/lib/docker needed by CodeBuild

The new containers are uploaded to AWS ECR and are currently used in all CI.
Issue(s) addressed
related to https://github.com/JCSDA-internal/jedi-stack/issues/146
Acceptance Criteria (Definition of Done)
Successfully build clang and gnu containers
Dependencies
None
Impact
None

Have you tried creating the site-packages directory manually and then adding it to the PYTHONPATH variable before running the install command?

@climbfuji I tested your suggestion and it didn't work. I get the error: ModuleNotFoundError: No module named 'ncepbufr. Even if I add site-packages  to thePYTHONPATH.
I added --install-lib=/usr/local/lib/python3.8/dist-packages to the install command and with that, I was able to build ncepbufr library in dist-packages, and all the tests passed with that.

@climbfuji I tested your suggestion and it didn't work. I get the error: ModuleNotFoundError: No module named 'ncepbufr. Even if I add site-packages to thePYTHONPATH.
I added --install-lib=/usr/local/lib/python3.8/dist-packages to the install command and with that, I was able to build ncepbufr library in dist-packages, and all the tests passed with that.

Thanks for trying. Your solution is fine, too.

@mer-a-o Please confirm that you are happy with this and I will merge. Thanks!

Yes! Please go ahead and merge :)


Not possible to use Symbol values for checkbox v-model values

Vue version
3.4.21
Link to minimal reproduction
https://play.vuejs.org/#eNqlkk9PAjEQxb/KpBcxQYjoiSwkaDhojBpRL9bDsgxY7LZN/6yQzX53Z7suYjRcvE3fe53+ZtKSTYzpFQHZkCUus8J4cOiDGXMlcqOthxIsLqGCpdU5HFH0iCuuMq2ch+fJzdP0FEYw2+ZzLTucNQpnxz8ig1+RQYy0oSKVASlDL3VeXslI+g0MYdDBY25k6pFOAIlQJnjwW4MjzrI3zN7nesMZFCe5XqAkMbYjZRgLEloq6P+vBVG3LRaiGJflF3lVJf1aqMH3YFmXeUcTLsWqt3Za0ZLL+jK9qXMjJNo74wVtgLMhRKf2Uin1x3XUvA3YbfXI+Ye+doQ+pOLeokNbEPbO86ldoW/s6ewWN1TvTJo1SEofMB/QaRlqxiZ2EdSCsPdykfYqfhWhVo9uuvGoXDtUDVonq5intQa8PDD6N+5Z7zze46pi1SdpaN8M
Steps to reproduce
Create multiple checkboxes with array v-model and pass a symbol value to each checkbox value prop.
What is expected?
That the v-model array contains the symbol value if the corresponding checkbox is checked.
What is actually happening?
Browser complains that it cannot convert Symbol to a string: Cannot convert a Symbol value to a string.
System Info
No response
Any additional comments?
No response

closing this in favor of your previous issue

OK. But I think the other one is maybe a special case of this one, but not necessary. I think codepath for array handling is different than true-value/false-value handling?

Link to the other issue: https://github.com/vuejs/core/issues/10597


Crazy JS to include author snippets

Before:

After:

👍

Hm, I am not sure. I don't mind I guess, but might be too much?

It's not as big as it looks on the screenshots when compared to amount of text in posts I think.
WDYT @patriknw

I don't have a strong opinion. I don't think it adds much value and wall of text can be an disadvantage.

Not adding until we do a proper blog design then I guess


rstudio: clean up and refactor

The first 2 commits should not cause any rebuilds.
The 3rd commit should only cause rstudio-server to rebuild. (Since it now has empty arrays as desktopItems and qtWrapperArgs instead of them being undeclared)

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
25.05 Release Notes (or backporting 24.11 and 25.05 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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

pinging @ciil @drupol just to make sure this PR didn't fly under your radar

I see that the rstudio server test is failing. I'm assuming it's not supposed to. Will check it out.

Turns out rstudioWrapper expected the empty $out/share directory of rstudio-server (which I removed during the refactor) to exist. Now only the non-server version symlinks the share directory.
Fixed and rebased on latest master

Rebased on master.
Is everything that worked before this PR still working?
I haven't checked since I can't really program in R and haven't used this program before.
If everything is OK, is there anything stopping us from merging?

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/2175


fix(QUV-2366): node modules imports paths have change to the root node modules

…de_modules

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: jguede:x: santiago.trigosantiago.trigo seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


docker plugin does not list stopped containers

This makes it impossible for users to start a container after they've stopped it via Wash.

Bug is in https://github.com/puppetlabs/wash/blob/master/plugin/docker/containersDir.go#L37 -- we should probably set All to true in ContainerList options.

It also makes it impossible to delete stopped containers.


Tags on individual metrics

Could yace be modified to add tags as labels on the individual metrics instead of (or as well as) the aws_<service>_info meta-metric?
For yace, graphing e.g. the request rate for all ELBs in a service would currently look like:
rate(aws_elb_requestcount_sum[5m]) + on (name) group_left(tag_kubernetes_io_service_name) aws_elb_info{tag_service="something"}

This is quite a long query to do filtering on a tag, and I expect that once there's a large amount of cloudwatch metrics ingested into Prometheus, this will be much slower than a query like rate(aws_elb_requestcount_sum{tag_service="something"}[5m]), since Prometheus will calculate the 5-minute rate for every ELB in its database before filtering down to a specific service.
(We had previously been using a machine_info metric similar to the aws_<service>_info metric for a while, but noticed there were performance problems when trying to graph metrics filtering by tags - e.g. graphing rate(node_vmstat_pgpgin{service="something",otherlabel="somethingelse"}[5m]) is an order of magnitude or 2 faster than graphing rate(node_vmstat_pgpgin[5m]) * on (instance) machine_info{service="something",otherlabel="somethingelse"}. It seems like when Prometheus is asked to join 2 metrics and only filter one of them, it isn't smart enough to optimise that join and it ends up reading data for every metric that exists for the unfiltered side.)
I could solve this with a recording rule in Prometheus for each metric that yace returns that joins the metric to the aws_<service>_info metric, but it would be cleaner if the metrics started out with the labels.
Is this a change you'd be interested in?

Its not difficult to implement and it was my initial plan.
During my research to write the exporter, i spoke with a very helpful prometheus contributor. He gave me a lot of tipps. One of them was to not do it that way.
"CAUTION: Remember that every unique combination of key-value label pairs represents a new time series, which can dramatically increase the amount of data stored. Do not use labels to store dimensions with high cardinality (many different label values), such as user IDs, email addresses, or other unbounded sets of values." - https://prometheus.io/docs/practices/naming/
So lets make an example of your idea vs the current implementation of this exporter:
Example:

You are monitoring 3x elbs with 10x metrics => 30x cloudwatch Metics + 3x aws info metrics
You add a 4x elb to your monitoring which has a new tag.

Current implementation:

There are now 4x new aws info metrics. As the new tag need to be added to all info metrics. As one metric should always have the same labels.
There are 10x new cloudwatch metrics.

Your idea:

40x new cloudwatch metrics as every metric needs consistent labels, therefore all metrics must be changed.

For me it feels like that tags are not changed very frequently. What is your opinion about it after hearing the reason why it was done that way?

Thanks for the background. I agree that adding new tags on one ELB shouldn't cause the others to turn into new metrics, that does sound like it would cause issues.
Since each tag is associated with a resource in AWS, I don't think tag values fall into the "unbounded sets of values" category - but you're right, it would be unwise to add all of the tags to each metric and blow out the metrics storage.
In my case I only really filter on 2 or 3 tags across my infrastructure, which identify the group/service the resources belong to. Would having configuration for a list of tags to associate with actual metrics make sense instead of all tags? That way adding extra ELBs with new tags won't cause the other ELB metrics to change, just the AWS info metrics, but Prometheus can still filter metrics by those tags.

Thats a nice idea i like that. @nhinds
Maybe something like that in the config file. What do you think @nhinds
staticTags:
  ec2:
    - Name
    - Environment
  rds:
    - Name
    - CostLocation
discovery:
 - region: eu-west-1
   type: "ec2"
   searchTags:
     - Key: Name
       Value: Jenkins.*
   metrics:
     - name: CPUUtilization
       statistics:
         - 'Maximum'
       period: 6000
       length: 6000

This makes sense for our company as well. Nice fix for the issue. Will try to implement this today or next friday. Having another issue i want to implement first, lets see, how it works out.

I've given this a go in #25

Ah oops, didn't intend to duplicate work -- just hadn't seen any activity and had some spare time. Feel free to close mine if you like.

@nhinds - Yeah already started with the topic last friday and should have told you. Sorry!
But i love all your contributions. Very happy about it!


Check that constructor calls follow a chaining discipline.

(Toned down my ambitions compared to #5019. This PR only checks the constructor chaining discipline as a first easier step.)

The constructor of jl.Object cannot call any other constructor.
Every other constructor must call exactly one other constructor, on this, either from the same class or the direct superclass.

There cannot be any other call to constructors.
These restrictions are also enforced by the JVM. Technically we are more restrictive because the JVM allows different code paths, each having a single constructor call, whereas we require exactly one at the top level. This is fine because both Scala and Java, as languages, enforce that more restrictive property.
Unfortunately, we used to generate an illegal super constructor call in the synthetic classes that extend DynamicImportThunk. We patch them with a deserialization hack.
In the first commit, we do not yet change the compiler to fix the above. In the second commit, we fix the compiler and only enable the deserialization hack on IR < 1.17.

The failure on the intermediate commit is only on AppVeyor, and was a timeout. I can't rebuild a non-last commit on AppVeyor without retesting everything, so I'll just leave it as is.

@sjrd, do you know if / how the JVM validates constructor calls if it allows different calls depending on control flow?
One one hand, disallowing stuff like:
if (...)
  super()
else
  super()

Seems restrictive for the IR level.
On the other hand:

Consistent with ES
Potentially a reasonable trade-off to allow validation.

They basically have a different validation type for an uninitialized object and an initialized one. Then they have stack typing at each instruction. Finally after the constructor call the type of the argument transitions from uninitialized to initialized. At merge points, the JVM checks that the stack types align, and that the type of the this value (which is tracked per instruction) align as well.
We could do it as well, but IMO it's overkill.

(rebased since we already have a commit changing the version on main)


Makes SPDX SBOM Reproducible

Writes custom documentNamespace and use SOURCE_DATE_EPOCH for created
timestamp when available

Resolves #368
Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[x] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

@sophiewigmore could you take a peek at this when you have a chance?

Bump @paketo-buildpacks/tooling-maintainers


Noty and VS2017 asp.net core 2.1 LibMan

Noty version: v3.2.0-beta.
Browser: any
OS: windows
jsFiddle link (if available):

Visual Studio 2017 .net core 2.1 now has LibMan (Library Manager). In my project I had noty installed via Bower and now I'm changing it to LibMan. This problem is when I'm installing noty via LibMan I have only 'noty.js' and 'noty.css' files. I can't choose 'themes' folder to add. So I had to get themes manually and add them to my project and not in noty installation folder. Can u add themes to LibMan?
PS:
the same problem with bounce.js file.

I think Libman is using CDNJS as default.
related: #425
For now, you can create a libman.json and choose a filesystem provider instead of CDN provider. https://github.com/aspnet/LibraryManager/wiki/libman.json-reference


az functionapp deployment crash

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az functionapp deployment source config-zip
Errors:
The command failed with an unexpected error. Here is the traceback:
'str' object has no attribute 'value'
Traceback (most recent call last):
  File "/opt/az/lib/python3.6/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 657, in execute
    raise ex
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 691, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/appservice/custom.py", line 405, in enable_zip_deploy_functionapp
    return enable_zip_deploy(cmd, resource_group_name, name, src, timeout, slot)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/appservice/custom.py", line 417, in enable_zip_deploy
    scm_url = _get_scm_url(cmd, resource_group_name, name, slot)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/appservice/custom.py", line 1948, in _get_scm_url
    webapp = show_webapp(cmd, resource_group_name, name, slot=slot)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/appservice/custom.py", line 607, in show_webapp
    return _show_app(cmd, resource_group_name, name, "webapp", slot)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/appservice/custom.py", line 736, in _show_app
    "Unable to find {} '{}', in RG '{}'".format(cmd_app_type.value, name, resource_group_name),
AttributeError: 'str' object has no attribute 'value'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
build dotnet 3 function app into a zip file
deploy infra with app service and function app (windows based)
az login to subscription I want to deploy into
az functionapp deployment source config-zip --resource-group validrg --name validfunc --src validzipfile.zip --debug

Expected Behavior
successfully deployed function app
Environment Summary
Linux-<IP_ADDRESS>-microsoft-standard-WSL2-x86_64-with-debian-bullseye-sid, Ubuntu 20.04.3 LTS
Python 3.6.10
Installer: DEB

azure-cli 2.29.0 *

Additional Context

Experiencing the same issue! We use this installation method   in our dockerfile. Any tips on how to install an older version of the az-cli?

Experiencing the same issue! We use this installation method  in our dockerfile. Any tips on how to install an older version of the az-cli?

We have managed to mitigate it by reverting to use 2.28.0 (it seems that there is no 2.29.1 for the "focal" ubuntu release available yet in the packages, despite there being ones for all the other versions.
Adding this to our pipeline downloaded the older version and allowed us to continue to deploy at least until this is corrected
sudo apt install -y --allow-downgrades azure-cli=2.28.0-1~focal

route to service team


Compare page file details dropdown

Description
The file dropdown in the compare page is getting obstructed by file titles below.

How to replicate the issue
I found the bug here https://github.com/x13a/Silence/compare/v1.11.1...master

Go on a compare page
Collapse files if the changes are long
Click on the dropdown icon

The dropdown should be obstructed
Extension version
22.7.11
Browser(s) used
Brave

We’re going to need to hotfix-disable this CSS feature. I’m not sure if we can actually avoid this. The only way this would be possible is if the open dropdown adds a class so we can kick it up the z-index — or if we had the :has() selector 🥲

This is https://github.com/refined-github/refined-github/issues/5800#issuecomment-1178779392 (kinda strange I missed the notification)

The only way this would be possible is if the open dropdown adds a class so we can kick it up the z-index

Maybe we could add that class ourselves, using a simple delegated listener for clicks on the dropdown button :thinking:


Add manifest for building Krogoth

Add manifest file for building Poky release Krogoth with meta-updater

Works for me. If you make Anton's suggested changes, I'll approve.

@OYTIS @patrickvacek thank you for the feedback. I have updated the xml file and I have added @OYTIS in suggested-by field in the latest git commit.
Thanks, Leon


Add SemVer pre-release version support for Ballerina packages

Update Ballerina package implementation to support SemVer pre-release and build metadata version support. This is required for Ballerina standard library modules.
https://semver.org/

Refer https://github.com/ballerina-platform/ballerina-spec/issues/667


Add streaming badges

ie. Disney+, Hulu, Amazon Prime, etc.

added in https://github.com/inttter/md-badges/commit/f865bd8333842a5424bdfda8d1b8f59ac6261712


Fix endless loop when executing insert SQL with mybatis select tag

Fixes #1603 .

Pull Request Test Coverage Report for Build 5990

3 of 5 (60.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.004%) to 71.673%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-jdbc/sharding-jdbc-core/src/main/java/io/shardingsphere/shardingjdbc/jdbc/adapter/AbstractStatementAdapter.java
3
5
60.0%

Totals

Change from base Build 5988:
-0.004%

Covered Lines:
9435

Relevant Lines:
13164

💛  - Coveralls


How to add odbc for sqlsrv?

How to add odbc for sqlsrv?
I recive error This extension requires the Microsoft ODBC Driver for SQL Server - php 8.2

We just provide the sqlsrv extensions but you have to install the MS ODBC driver yourself.
The MS ODBC can be installed easily in Alpine. Follow these steps.

I already know why doesn't work, I need to install a good gpg package RUN apk add gnupg. Thanks!


Broken links in the documentation

There are some broken links in the doc, e.g.: "jump into Parallelism" at the bottom here.
Also, grep docs/ shows there are more in the docs and READMEs.

It seems we broke a few links in #169, @diesalbla.

Thanks for pointing this out @n4to4.
Fixed by #175.


Strange behavior $scope.$watchCollection with the cloned objects (by angular.copy)

If collection Object is changed with use:

change not object property and then use obj = angular.copy(obj) - its work ok,
but if it's changed with:
change element of object property of collection Оbject,  with type (element of Array, for example) and then use obj = angular.copy(obj) - follow that oldVal, and newVal - is equal in watcherFn !!!

For example:
http://plnkr.co/edit/dMZih1fChn6GE3zXEG38?p=preview
if change obj.username then ok, but if changeobj.aaa[0], then old and new val are equal in watcher function

Strange indeed. I suspect it has something to do with watchCollection only watching one level deep. You can work around this quite easily by using $watch('obj', actionFn, true) instead. watchCollection should be used when you have an array of values and you are removing / adding whole objects. watch with third argument is true compares the whole object, which looks like what you want to do.
Also, why are you copying $scope.obj on ng-change?

angular.copy is a deep copy, but watchCollection without deep watch maintains only a shallow copy (for performance reason) and therefore sends you as old object items containing the new values.
It is normal behavior, when you don't deep watch, you must not rely on "pointers" in old value but only on first level property. The same is true for normal $watch.

Closing as this is the expected behaviour of $watchCollection.

Thanks for an explanation!
But ... if it's created a deep copy of "obj" then "obj"'s property "aaa" shall change and refer to other Array object and run watchFn.
obj.aaa 'before' != obj.aaa 'after' angular.copy
Why old value of obj.aaa doesn't refer to oldVal in watchFn?
to frfancha: following your explanation shall be:
First $digest watchFn -  oldVal_1 shallow equivalent newVal_1, and not trigger whatchFn
Second $digest watchFn (after deep copy) - newVal_2 != oldVal_1 and newVal_2.aaa != oldVal_1.aaa
and trigger watchFn with (newVal_2, oldVal_1),
but  instead occurs Second watchFn trigger with (newVal_2, newVal_1).
Or in each cycle watcher's Value redefined by newVal even if newVal shallow-eq oldVal?
if it so, whether that it was better to exclude redefined Value in that case?
It would allow to trace more precisely changes and to trigger watchFn correctly if object-like property of Collection changes in functions runs inside $timeout
to Narretz: i use angular.copy in ng-change for demo-example.
If i use $scope.obj.aaa = [$scope.obj.aaa[0], $scope.obj.aaa[1]]  (create a new value of "obj.aaa" then folow the same strange behavior
A lot of thanks for consideration and further explanations!

Here is a more comprehensive Plunker for you to play with: http://plnkr.co/edit/rc4s6le7ef3H85MjjHTp?p=preview

It's Very visually, thanks a Lot !
Whether correctly I understand that in case of the current implementation of $watchCollection, if only updating of properties of nested object (for example obj.aaa[0]) and its cloning happens in the different $digest-cycle (for example, by using $timeout), then  $watchCollection launches watchFn once (of two or more $digest-cycle) with (oldVal, newVal), that which satisfy to the following:
newVal !== oldVal  but newVal deep equivalent of oldVal ?

$watchCollection and $watch (shallow) will both be triggered if the main object being watched is replaced with something new
$watchCollection will be triggered if any of its top level properties changes
$watch (deep) will only be triggered if a deep comparison of the nested objects is different, whether or not the actual object itself changes.

@AbstractVisions You should read chapter one of the book "Build your own angularjs" (from Tero Parviainen).
It explains extremely clearly what is the scope, the digest cycle, watchers, ... after that everything is clear and all behaviors that could seem strange with only superficial knowledge become perfectly logical and predictable.
My life with angularjs has changed after reading this book, thanks to Tero!


fix typo in the readme files

Pull Request Test Coverage Report for Build 374

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 45.0%

Totals

Change from base Build 364:
0.0%

Covered Lines:
351

Relevant Lines:
780

💛  - Coveralls


Support for Android 8.0 and 8.1

Please add support for Android 8.0 and 8.1+

There is no "support for x or y version". This project just uses a protocol, which is implemented or not on your device'

Ok, Thanks!


Add contributor to release notes

Recognition for #21145

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

@bot-gradle test and merge


Extend the SeedPredicate of controllerregistration controller

How to categorize this PR?

/area quality
/kind enhancement
What this PR does / why we need it:
This PR enhances the SeedPredicate of the controllerregistration controller to return true if there is a change in spec, labels or annotations.
Which issue(s) this PR fixes:
Fixes https://github.com/gardener/gardener/issues/7732
Special notes for your reviewer:
Release note:

NONE

/test pull-gardener-e2e-kind-migration


[Warhammer 4e Character Sheet] Remove Pray Crits

[ ] The pull request title clearly contains the name of the sheet I am editing.
[ ] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[ ] The pull request makes changes to files in only one sub-folder.
[ ] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description

Removed critical rt display for Prayer spells, as they are not get dubs random criticals.

Character Sheet Info Roll20 Internal Use only.


Avoid duplication in make_standalone with multiple encoders

This corrects a problem in the recent 6d673f6 (Extend make_standalone
script to allow for more than 1 encoder, 2021-02-12) which could (and
did) result in having duplicated dependencies in the generated files.
While this didn't seem to result in any problems, it is still at best
completely unnecessary.
Keep track of the dependencies we had already seen to prevent this from
happening.

Sorry, my previous PR wasn't quite correct, so here is another one amending it slightly to avoid having duplicated sections in the output.

Thanks again.


Taiko NFT bridge solution

Describe the feature request
Why we need the taiko NFT Bridge?
An NFT bridge is a helping hand used to connect different networks throughout the blockchain ecosystem. Most of the time, you need to send NFTs from one test network to another on the Ethereum platform. NFT bridging is a helpful solution when you want to send an Ethereum test token from taiko  katla(testnet) to another test network (Goerli).
Note that every network (katla, Goerli, matic, etc) has its own protocols. So, an NFT bridge solution must meet their requirements to establish a trustworthy connection between them.
We need NFT bridging in order to reach the following benefits:
Facilitating the transfer of NFTs between different networks on Ethereum
Making NFTs accessible everywhere
Accessing all the features of different networks
if its available in testnet then it will be really good for the tests as well
Spam policy

[X] I verify that this issue is NOT SPAM and understand SPAM issues will be closed and reported to GitHub, resulting in ACCOUNT TERMINATION.

Plz open these kind of issues here: https://github.com/taikoxyz/taiko-mono/issues :)


Feature/add announcement subscriptions

Description

[x] New feature (non-breaking change which adds functionality)
[x] Complete, but not tested (may need new tests)
[x] My code follows the style guidelines of this project

Saw the code working over Zoom, looks good to go


range function python tutorial

Populate common-built-in-functions/range.py with a tutorial and challenge.

Can I contribute to this??

Hey! I would like to contribute to this as well.

hey @inspirezonetech please accept my PR


Electron 11 & sqlite3  support

Hi there,
thanks for your great work on node-sqlite3. I use it for my electron-app!
I was just wondering if there is support for Electron 11 planned for the future. I a lot of warning messages when I try to rebuild the sqlite3 using the electron-rebuild latest version that is 2.3.4 as of today.

I am getting the issue like FATAL ERROR: Error::Error napi_create_reference(https://github.com/mapbox/node-sqlite3/issues/1406)  with electron version 11 and  sqlite3 version 5.0.0.
with electron version 8.3.4 and sqlite3 4.2.0 it was working.Please provide the solutions.

Thank you for your dedication and great work on node-sqlite3.
I am using sqlite3 in my electron app, and I need to update it to electron 11, but it is not possible since sqlite3 is not supported in the latest versions of electron, only up to version 8.2 of electron, I was wondering if it is planned that in the next versions. Thanks in advance.

Thank you for your dedication and great work on node-sqlite3.
I am using sqlite3 in my electron app, and I need to update it to electron 11, but it is not possible since sqlite3 is not supported in the latest versions of electron, only up to version 8.2 of electron, I was wondering if it is planned that in the next versions. Thanks in advance.

+1

+1

I didn't want to do this, but I had to. I ended up switching to better-sqlite due to this compatibility error. Luckily, I was able to easily because I was utilizing TypeORM, so it was just a matter of changing a few strings to specify the new DB driver. I've had no other issues so far. So, if you are looking for an alternative, better-sqlite should work.
Also, as far as I know, node-sqlite3 is only technically compatible with Electron 8.x. I believe better-sqlite is compatible with the latest, v11

I didn't want to do this, but I had to. I ended up switching to better-sqlite due to this compatibility error. Luckily, I was able to easily because I was utilizing TypeORM, so it was just a matter of changing a few strings to specify the new DB driver. I've had no other issues so far. So, if you are looking for an alternative, better-sqlite should work.
Also, as far as I know, node-sqlite3 is only technically compatible with Electron 8.x. I believe better-sqlite is compatible with the latest, v11

Would you be able to try with v5.0.3? 🙂

Works with Electron 18.x for me!


New release changed some function. Unknown how to replace them with new version of GNU package

I'm trying to replace the old function calls with the new but I'm stuck.
It used to work like this:
await engine.RegisterDhtAsync(new MonoTorrent.Dht.DhtEngine(MonoTorrent.Dht.Listeners.DhtListenerFactory.CreateUdp(dhtPort)));
and
await engine.EnablePortForwardingAsync(CancellationToken.None);
and
e.TorrentManager.Torrent.Files.BytesDownloaded  // where e is PieceHashedEventArgs
and
torrent.Files[0].BitField.PercentComplete
Please help finding the new function to use to replace the above (old) way of getting the torrent information

await engine.RegisterDhtAsync(new MonoTorrent.Dht.DhtEngine(MonoTorrent.Dht.Listeners.DhtListenerFactory.CreateUdp(dhtPort)));

Just set the DhtPort property in EngineSettings` to enable DHT. Everything else, including port forwarding for the Dht port, will be automatic.
await engine.EnablePortForwardingAsync(CancellationToken.None);

Set EngineSettings.AllowPortForwarding = true. This is the default value so you should set it to false if you want automatic port forwarding to be disabled.

I updated the release notes to make it a little clearer as to where these methods have gone :) https://github.com/alanmcgovern/monotorrent/releases/tag/release-v2.0.0-alpha.unstable.rev0017

thank you for the fast reply! How about the DHT node files:
await engine.DhtEngine.StartAsync(nodes);
Is this also now different / automatic?

e.TorrentManager.Torrent.Files.BytesDownloaded
can't find Files.BytesDownloaded
Also
torrent.Files[cur_torFileIndexNum].BitField
Can't find BitField

So, should I now only user TorrentManager.Files instead of Torrent.Files?

Yes, i misread the original comment. This was a deliberate change to make the Torrent object immutable. All mutable state is in TorrentManager.Files now! This is also how you set the priority of a file too.

await engine.DhtEngine.StartAsync(nodes);

If you want to load pre-saved nodes into the DhtEngine you can call this method as per usual. I may make a few changes around how to start/stop things to separate the lifespan of Dht and active torrents. Just because you stop all the torrents doesn't mean you want to stop the Dht engine, but those lifespans are linked at the moment.
If that does change it'll be in the release notes!
Let me know if you need anything else!


[DOR-118][AO] add "What's the case reference number?" page

The new page is at /trader-services/pre-clearance/amend/case-reference-number URL


[otbn] Add timing accuracy to bus-functional model

As discussed here: https://github.com/lowRISC/opentitan/issues/2477
Timing is cycles===instructions, hence we can move forward with just returning the cycles.

[x] Add backchannel for this information, most easy is obviously a file
[x] Count down from this return until the done goes up

Ready for review in #2625

Did go into otbn-dev through 51ae3f3ac0b7a43af62ca751e70d4d487953bb9a


instanceof SerializableResource

After some consideration, I came up with the following idea to enable the usage of the instanceof keyword with SerializableResource. This approach leverages decorators and metadata.
We define SerializableResource as an abstract class. Abstract classes can be used similarly to interfaces through the implements keyword. Additionally, we apply a decorator to our class, which will be explained below:
Example:
@SerializableResource.register
export class Material extends Mutable implements SerializableResource {}

The abstract class SerializableResource  now allows us to achieve two things:

Encapsulate a decorator (register) that registers the class, using the class's metadata to indicate that it implements SerializableResource.
Override [Symbol.hasInstance] (instanceof behavior) to check the metadata of a given instance to verify whether it implements SerializableResource.

SerializableResource:
export abstract class SerializableResource implements Serializable {
  public abstract name: string;
  public abstract idResource: string;
  public abstract readonly type: string;

  // decorator
  public static register(_class: abstract new (...args: General[]) => SerializableResource, _context: ClassDecoratorContext): void {
    let meta: Metadata = _context.metadata;
    meta.implements ??= [];
    meta.implements.push(SerializableResource);
  }

  // instanceof
  public static [Symbol.hasInstance](_instance: unknown): _instance is SerializableResource {
    if (Function.prototype[Symbol.hasInstance].call(SerializableResource, _instance)) // default instanceof behavior
      return true;

    let meta: Metadata = _instance.constructor[Symbol.metadata]; // check the metadata
    return meta?.implements?.includes(SerializableResource);
  }

  public abstract serialize(): Serialization;
  public abstract deserialize(_serialization: Serialization): Promise<Serializable>;
}

To make this work, we need to decorate our class with @SerializableResource.register (see Example). Once the class is registered, we can now do the following:
Let instance: Material = new Material("Material");
// check instance
instance instanceof SerializableResource // true
// check class
Material.prototype instanceof SerializableResource // true

Note: We could expand this pattern to allow the registration of custom resources compatible with the editor. The abstract class could keep track of all registered SerializableResources, enabling creators to register their own custom SerializableResources.

Well... errr... sounds good to me.


Adopt automated code formatting

Benefits of automated code formatting:

We'll have a standard style, which helps make the codebase more readable for newbies and oldies alike
Automation drastically reduces back-and-forth over style; we just go with what the formatter provides
Reduces merge conflicts from intentional or accidental formatting changes
Reduces noise in PRs from people fixing formatting errors when they find them (or having to submit separate formatting PRs)

I propose we use ktlint.

Implements a combination of the official Kotlin and Android style guides.
Formatters available for major editors, including IntellIJ, Emacs, and Vim
Minimally configurable, which helps keep us from bikeshedding over small style choices
A number of options for Gradle integration. (I usually just re-use a simple buildSrc plugin I wrote.)

I can add this very quickly, but would like to wait for #26 to be merged first.

👍🏼for automating this.


Add missing property in LocalSessionStorageCollection

There is a missing property in LocalSessionStorageCollection class. Adding it.

I think this change is related to the .3000 server build. Some deps are pulling the more recent version even though the client is supposed to be on .2000. It is probably safest to either upgrade the server to .3000 or pin the client to .2000.


cannot update publicPath in remoteEntry.js

I use create-react-app and craco to use webpack 5 module federation
"react-scripts": "5.0.0-next.47",
"@craco/craco": "^6.4.0",
"craco-module-federation": "^1.1.0",

I want to know why set publicPath to 'auto' in code
At my project,I edit publicPath by craco.config.js like
module.exports = {
  webpack: {
    configure: (webpackConfig, { env, paths }) => {
      webpackConfig.output = {
        ...webpackConfig.output,
        publicPath: "https://abc.com",
      };
      return webpackConfig;
    },
  },
...

and run yarn build-->"build": "craco build", but it not works.
when I update it to "build": "PUBLIC_URL=https://abv.com craco build",it only works in index.html, not work in remoteEntry.js

I only use craco to avoid this problem


Consider using fully qualified test names

It is possible that someone could use two different test files and have the same test name, one in each. In that case, the results are going to get messed up. We should probably use a fully qualified testname, e.g.
test_input_file.py::test_number_one
instead of as we do currently:
test_number_one

Done in v0.9.0.


Drop remarks on setting the content root

Fixes #13057

@javiercn This might be a bug fix. It will only take 10 seconds to confirm with you.

@Rick-Anderson Approval is here :point_right: https://github.com/aspnet/AspNetCore.Docs/pull/13058#discussion_r310690947


Generic cloner interface

Currently we can setup a objectMapper in configuration, where now this is used only for cloning custom resources (serializing and de-serializing POJOs):
https://github.com/java-operator-sdk/java-operator-sdk/blob/5431289871baaf221745cbf7e522b70f7e69c9e2/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/config/ConfigurationService.java#L82-L90
The goal of this issue is to generalize this interface and have cloner interface in place, so not just sub-classes of object mappers can be provided.

Do we really think there'll be other cloner implementations, though?

To run the Operator using Kotlin required different implementation. Basially a different object mapper. So the idea is that if we need already to and it's quite easy to make this generically replacable this way.


[Question] I have a question about youtube-dl

Im using my custom gui for send comands to youtube-dl and im using a method for multiple download (Select 20 links and auto download it 1 by 1).
So i would like to know if is possible to make youtube-dl start in background or aleast not going foreground at the current app or anything else im using in that moment because the console keep popup every time a new download comand is sent to it
thank you!

It is nothing complex, i just send the command to the cmd with the gui, im not using any framework.
commandToExecute = "cmd.exe /c start "+ YOUTUBEDLDIRECTORY + ecttt....
I will try to find something related to windows console then, thank you !

Do you even aware what does your command mean? You're running start.exe process via cmd.exe that literally:

C:\Dev\youtube-dl\master>start /?
Starts a separate window to run a specified program or command.

and runs youtube-dl.exe in that window.
What you should do is run youtube-dl.exe directly.

My friend helped me and i just learned a very very basic java language :P but yeah i never noticed this and i will try it directly

i just solved addeding /min after /start:
commandToExecute = "cmd.exe /c start /min "+ YOUTUBEDLDIRECTORY + ecttt....
Also i cant run youtube-dl.exe directly i get some weird error probabily because the gui is scripted to work with this method, anyway thank you !


[experiment, wip] proposal - use wheels in pythonpath for bootstrapped pip

Motivation for this change
this is an experiment to clean up the python path of the bootstrapped pip
i accidentally started it, and it shows many python packages have implicit setup requirements
that just arent listed
rebase to master upcoming
Things done

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

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

@RonnyPfannschmidt since you mentioned you're not using Nix anymore I am closing this.


Fix the existing AKS cluster displays an error message when upgrading

rancher-server
Problem:
Change previous cluster agent pool name will cause AKS API error
Solution:
Don't change the previous cluster agent pool name
Issue:
https://github.com/rancher/rancher/issues/18165

LGTM


Python: applications are affected by PYTHONPATH

Issue description
pythonpath leaks into subprocesses all over the place
Steps to reproduce an example

nix-env -i nox
nix-shell -p python27Packages.enum34
nox -h

[nix-shell:~]$ nox -h
Traceback (most recent call last):
  File "/nix/store/6n255nladjbailvkkhs0ai0ls01iiphq-python2.7-enum34-1.0.4/lib/python2.7/site-packages/enum/__init__.py", line 371, in __getattr__
    return cls._member_map_[name]
KeyError: '_convert'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/nix/store/wf1za4si6rl5y45kcxhslyjq9wywcakr-nox-0.0.3/bin/.nox-wrapped", line 8, in <module>
    from nox.search import main
  File "/nix/store/wf1za4si6rl5y45kcxhslyjq9wywcakr-nox-0.0.3/lib/python3.5/site-packages/nox/search.py", line 4, in <module>
    import subprocess
  File "/nix/store/q17rbyq94md0z91g289d5p8rc2q2b60c-python3-3.5.1/lib/python3.5/subprocess.py", line 364, in <module>
    import signal
  File "/nix/store/q17rbyq94md0z91g289d5p8rc2q2b60c-python3-3.5.1/lib/python3.5/signal.py", line 8, in <module>
    _IntEnum._convert(
  File "/nix/store/6n255nladjbailvkkhs0ai0ls01iiphq-python2.7-enum34-1.0.4/lib/python2.7/site-packages/enum/__init__.py", line 373, in __getattr__
    raise AttributeError(name)
AttributeError: _convert

You know nix-shell sets PYTHONPATH, so you would know this will happen. Of course it is nice if it wouldn't but it does.
I can imagine with Nix you expect better, but this would happen on other systems as well. If you set PYTHONPATH, then other Python applications will break. How do you suggest to protect Python applications from PYTHONPATH? Maybe we should unset PYTHONPATH in the wrapper first?
https://github.com/NixOS/nixpkgs/pull/13721

the thing is, if all nix does for setting up environments is creating env vars, its pretty much not delivering its promise
it should pretty much set up different trees of symlinks for different working environments
this one is the same reason why the tox package on nix is completely broken for example

In https://github.com/NixOS/nixpkgs/commit/def310df459f649a4ce2df8b7ba19aa8e4d5af58 I fixed a bug with setuptools that affected tox. Now you should be able to just nix-env -i pythonPackages.tox. However, at this point it still comes with a wrapper. In https://github.com/NixOS/nixpkgs/pull/16668 I took out the PYTHONPATH wrapper, although the PYTHONHOME wrapper remains.
Regarding your example with nox. You are aware of the --pure flag for nix-shell?

See https://github.com/NixOS/nixpkgs/issues/16591 for a possible short-term fix.

#17749 fixes part of the problem, the nix-shell side needs more thought, perhaps off by one profiles are whats really needed

(in that case nix-shell is exactly correct)

@FRidh if i understand right, with the current python infrastructure, this one is fixed?

In principle yes, but we still have quite some applications and services that use --prefix.

Closing this, if you see find it in a specific application, please open an issue against it directly.


URL Cleanup

This commit updates URLs to prefer the https protocol. Redirects are not followed to avoid accidentally expanding intentionally shortened URLs (i.e. if using a URL shortener).
Fixed URLs
Fixed Success
These URLs were switched to an https URL with a 2xx status. While the status was successful, your review is still recommended.

[ ] http://www.apache.org/licenses/ with 2 occurrences migrated to:
https://www.apache.org/licenses/ (https result 200).
[ ] http://www.apache.org/licenses/LICENSE-2.0 with 141 occurrences migrated to:
https://www.apache.org/licenses/LICENSE-2.0 (https result 200).

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


Be able to set srcport (firewall friendly) mode of tftp.py through config

According to the RFC for tftp its not mandatory to use srcport 69 for the reply traffic back to the client.
The normal behaviour is something like this:

Client connects to tftp server using srcport >1023 (or >49152 depending on os and version) and dstport 69. In this example lets say the client used srcport 12345.

Server replies to the client using srcport >1023 and dstport (in this example) 12345.

The above confuses any firewalls you might have in between.
A workaround is to make the tftp server to use srcport 69 in its replies back to the client, that is something like this:

Client connects to tftp server using srcport >1023 (or >49152 depending on os and version) and dstport 69. In this example lets say the client used srcport 12345.

Server replies to the client using srcport 69 and dstport (in this example) 12345.

One can fix this manually by changing "self.sock.bind((self.ip, 0))" (which exists at two places in tftp.py) into "self.sock.bind((self.ip, 69))" but its a more clean way if this could be made through config.
That is a "firewall friendly" mode of the tftp server (for example firewall_friendly true/false).

From RFC1350:

A requesting host chooses its source TID as described above, and sends its initial request to the known TID 69 decimal (105 octal) on the serving host.  The response to the request, under normal operation, uses a TID chosen by the server as its source TID and the TID chosen for the previous message by the requestor as its destination TID. The two chosen TID's are then used for the remainder of the transfer.

In this context, TID is synonymous with UDP port. While that may not expressly forbid the server picking TID 69, using a standard ephemeral port is best practices. All firewalls have problems with TFTP, from any server software, unless the firewall has a layer-4 helper.

Given @icb-'s comments above, I don't think this will be implemented, as we want to stick to the RFC.

As referenced in RFC1350 there is nothing that says that the TID on the server side is not allowed to be 69.
Adding this as an option (firewall friendly / singleport use) will make PyPXE be able to be used in a firewalled environment where one currently is forced to use other software (such as tftpgui who supports using srcport 69 for the replytraffic) simply because PyPXE currently doesnt work.
So I would propose to have this issue reopened :-)

I'll accept a PR for this issue, as long as the default behaviour is not impacted in any way. I don't have the time to work on this myself at the moment.

There's no portable way to specify the ephemeral port range, and even the ways you can are system-wide. You would have to either use a fixed port, limiting you to one in-flight transfer, or loop through a range of ports finding an unused one. Either way you've increased your susceptibility to spoofed traffic.
The proper way to deal with TFTP through a firewall is with a layer-4 helper, like nf_conntrack_tftp in Linux or tftp-proxy in OpenBSD.

I've just had a look at the code again, and whilst we wouldn't explicitly be limited to one transfer, it would bring on a whole host of bugs unless we forced one at a time.
It would also require a lot of changes and for very minimal benefit, especially given the above firewall mitigations. We would essentially be maintaining two tftp main loops, or a complete rewrite.


Rating does not recompose if rating value changed extrenally

  @Composable
  fun RatingBar(
      modifier: Modifier = Modifier,
      value: Float = 0f,
      numStars: Int = 5,
      size: Dp = 26.dp,
      padding: Dp = 2.dp,
      isIndicator: Boolean = false,
      activeColor: Color = Color(0xffffd740),
      inactiveColor: Color = Color(0xffffecb3),
      stepSize: StepSize = StepSize.ONE,
      ratingBarStyle: RatingBarStyle = RatingBarStyle.Normal,
      onRatingChanged: (Float) -> Unit
  ) {
      var rowSize by remember { mutableStateOf(Size.Zero) }
      var rating by remember { mutableStateOf(value) }

var rating by remember { mutableStateOf(value) } knows nothing about external changes in value
I'm newbie in jetpack compose, but I guess you need to rewrite the code like this
   var rating by remember(value) { mutableStateOf(value) }

fixed in the v1.1.1 release. changed to  state hoist pattern. so, you can check latest updated readme.


Error in cluster mapping with new tagpacks

Hypothesis: contain too many invalid crypto addresses

BCH
FAILED
"['address', 'cluster_id', 'cluster_defining_address', 'no_addresses', 'in_degree', 'out_degree'] not in index"


GroverOperator import error

GroverOperator import error

Qiskit Aqua version: 0.8.0
Python version: 3.6
Operating system: Ubuntu 18.04

What is the current behavior?
When I run
python test/aqua/test_grover.py
I get the error:
Traceback (most recent call last):
File "test/aqua/test_grover.py", line 28, in
from qiskit.circuit.library import GroverOperator
ImportError: cannot import name 'GroverOperator'
And I search the "GroverOperator" in https://qiskit.org/documentation/, I get nothing about it.

Steps to reproduce the problem
python test/aqua/test_grover.py
What is the expected behavior?
No error return when running python test/aqua/test_grover.py
Suggested solutions
You should define GroverOperator before using it.

The master branch here is only guaranteed to work with the master branches of the the other dependent parts of qiskit, such as qiskit-terra in this case where you will find GroverOperator. If you install the latest from master you should find that works.


virt-launcher: Switch to virtqemud

Signed-off-by: akriti gupta<EMAIL_ADDRESS>Short description:
More details:
What this PR does / why we need it:
This PR update Virt Launcher Pod Hypervisor Connection URI socket to /var/run/libvirt/virtqemud-sock for 4.13
https://github.com/kubevirt/kubevirt/pull/8619
Release note:

virt-launcher: use virtqemud daemon instead of libvirtd

!verified

!verified

!verified

!verified

!cherry-pick v4.13


fix clearing function

this needs to be cleaned up and put in a proper unit test in the /test/ dir of the repo

i will work on this issue with Xiaojie

@lorenzokaaks  and I have a clue if you need

As @jeannesorin and @lorenzokaaks  suggested I changed the output of the function to be the squareroot of the sum of squares instead of being an array of errors

created new branch with new function and test

when you are ready to share this addition you need to create a pull request from that new branch.


Undeclared IPython dependency

Expected behavior
Package documentation / metadata should match actual dependencies.
Actual behavior
When installing apache-sedona along with its official requirements, IPython is not installed.
Importing sedona though also triggers an IPython import:
Traceback (most recent call last):
  File "/Users/xyz/Projekte/sedona-migration/run_migration_to_3.5.py", line 3, in <module>
    from sedona.spark import SedonaContext
  File "/Users/xyz/Library/Caches/pypoetry/virtualenvs/sedona-migration-M_RNyLtM-py3.9/lib/python3.9/site-packages/sedona/spark/__init__.py", line 43, in <module>
    from sedona.raster_utils.SedonaUtils import SedonaUtils
  File "/Users/xyz/Library/Caches/pypoetry/virtualenvs/sedona-migration-M_RNyLtM-py3.9/lib/python3.9/site-packages/sedona/raster_utils/SedonaUtils.py", line 18, in <module>
    from IPython.display import display, HTML
ModuleNotFoundError: No module named 'IPython'

This is very inconvenient especially since IPython seems to be around only for the limited use-case of displaying images in an IPython environment.
It feels like sedona python should be usable with ipython though - or the IPython dependency should be properly documented in the package.
Would you be open to accept a PR that moves the IPython import in
https://github.com/apache/sedona/blob/87104212e30f0ea571164d6952c62dbb69ec1428/python/sedona/raster_utils/SedonaUtils.py#L18-L23
as a local import into the display_image classmethod?
That way the python lib - expect for the this specific feature - would still be usable without IPython.
Steps to reproduce the problem
pip install apache-sedona==1.5.1 pyspark
python -c "import sedona.spark"

Settings
Sedona version = 1.5.1
Apache Spark version = 3.3.2
Apache Flink version = ?
API type = Python

thanks @jiayuasu  - any idea when this will be released?


pgp is incompatible with rsa 0.9.x

Given the following example Cargo.toml:
[package]
name = "pgpbuild"
version = "0.1.0"
edition = "2021"

[dependencies]
pgp = "0.10.1"
rsa = "0.9"

cargo build fails with:
error: failed to select a version for `rsa`.
    ... required by package `pgp v0.10.1`
    ... which satisfies dependency `pgp = "^0.10.1"` of package `pgpbuild v0.1.0 (/home/user/work/testing/pgpbuild)`
versions that meet the requirements `^0.9.0-pre.0` are: 0.9.0-pre.2, 0.9.2, 0.9.1, 0.9.0, 0.9.0-rc.0, 0.9.0-pre.1, 0.9.0-pre.0
the package `pgp` depends on `rsa`, with features: `expose-internals` but `rsa` does not have these features.
all possible versions conflict with previously selected packages.
  previously selected package `rsa v0.9.0`
    ... which satisfies dependency `rsa = "^0.9.0-pre.0"` of package `pgp v0.10.1`
    ... which satisfies dependency `pgp = "^0.10.1"` of package `pgpbuild v0.1.0 (/home/user/work/testing/pgpbuild)`
failed to select a version for `rsa` which could resolve this conflict

Looks like the expose-internals feature was removed after the prereleases of rsa 0.9.
This means that the pgp crate currently cannot be used with rsa 0.9.x

Thanks for the report, working on it in #240, looks like there is some issue with verification now, due to stricter checks in the latest release which I need to track down.


Taxon constraint: GO:0072487 MSL complex

Please provide as much information as you can:

GO term ID label:

MSL complex
Cellular Component
Definition (GO:0072487 )
A histone acetyltransferase complex that catalyzes the acetylation of a histone H4 lysine residue at position 16. In human, it contains the catalytic subunit MOF, and MSL1, MSL2 and MSL3. PMID:16227571 PMID:20018852

Request to add a taxon constraint:

**only in taxon: **
Metazoa

Supporting evidence if available (e.g PMID):

MOF= KAT8 (not in yeast)
MSL1,2,3 appear to be metazoan specific

https://github.com/geneontology/go-annotation/issues/4725


Confirm with an independent machine that APH-machine-specific tests are skipped

@eteq - could you confirm or deny this? The only thing you need to do is just execute the full test suite after checking out bd035cbf76875168f83e2ad4307951045216fc5c, which is part of the sim_manager_revamp branch on my personal Halotools fork.
(Do you know a ProTip for how I can provide a GitHub reference to a specific branch, e.g., a symbolic link to " the sim_manager_revamp branch on my personal Halotools fork"?)

It's actually not possible that this is behaving improperly, so no human confirmation is necessary


New Items: MP3 Encoder, Machine Learning

mpeg audio layer 3 encoder written in Fortran 90/95 (mp3 encoder)
https://github.com/cure-honey/uzura3
Machine learning scripts for Fortranner.
https://github.com/NaoMatch/FortLearner

Added uzura3. FortLearner was already present.


skip failing configure checks during cross-compile builds

Compile and run configure checks don't work in simple cross-compile
build environments so ignore failures when cross-compiling.

You are supposed to specify a terminal library using the --with-tlib=library argument.

This is not a problem of Vim's configure.ac as Bram described. Closing.


Gridcoin Chinese Translation

Hey guys！ Gridcoin to da moon！恭喜发财！

Sweet! Was this made on master or dev?

Sorry I‘m new to here and don't know there are different branches. I believe I choosed the master branch.

@mrip001 No worries, I'll merge this to dev. There will be a handfull of new translations there.

I have merged the translations to dev but there were a couple of errors. Can you please finalize the translations from dev in Qt Linguist instead of a manual editor? Check translations and tools here:
https://github.com/gridcoin/Gridcoin-Research/blob/development/src/qt/locale/bitcoin_zh_CN.ts
https://github.com/lelegard/qtlinguist-installers/releases

Ok， I‘ll try to fix it


sqlite parameta for shemata missing, presetting is wrong

the predefined parameter "$sqlite_schema_file" in case of redhat family is  wrong. it should be called: '/usr/share/doc/pdns-backend-sqlite-4.?.?/schema.sqlite3.sql' rather then '/usr/share/doc/pdns-backend-sqlite-4.?.?/schema.sqlite.sql'.
In addition is not possible to change this var as the class powerdns is missing the line in () where we could change this.

ping ...

@LooOOooM you do know that this is very unfriendly behaviour in any open source project, right? This is a project where we spend our free time on. It's not commercial, nobody earns anything or gets paid to make this.
We appreciate your work, but pinging or asking for updates is something we do not want to see in our projects.
We will be releasing this. I can't tell you when. For now, I will be closing this issue.

@LooOOooM I am going to reopen this. Not because it hasn't been merged, but because it's probably not fixed 100%.
Which OS have you checked this on? The check in params.pp is for the OS family RedHat, this includes RedHat itself and CentOS (and probably others in the future). I did a check on CentOS 7 & 8:

On CentOS 8 the file is here: /usr/share/doc/pdns-backend-sqlite/schema.sqlite3.sql. This is a different path.
On CentOS 7 the file doesn't exist at all.

If this is going to be fixed, it needs to be fixed for those operating systems too. This means:

The params.pp file path should be adjusted for CentOS 8 (I know this will be EOL by the end of the year, but other operating systems will take over and are likely to have the same location).
Creating tables should throw an error if the file is not set or the file does not exist.

If you do decide to fix this, please open a new PR in a new branch and PR it to our develop branch. Don't reuse your old branch as it will cause issues.

#cat /etc/redhat-release
CentOS Linux release 7.9.2009 (Core)
rpm -qa|grep pdns
pdns-backend-sqlite-4.5.0-0.alpha0.master.928.g2b3dbbb61.1pdns.el7.x86_64
pdns-recursor-4.5.0-0.alpha3.master.105.g2b3dbbb61.1pdns.el7.x86_64
pdns-4.5.0-0.alpha0.master.928.g2b3dbbb61.1pdns.el7.x86_64
#rpm -ql pdns-backend-sqlite-4.5.0-0.alpha0.master.928.g2b3dbbb61.1pdns.el7.x86_64|grep "/schema.sqlite3.sql"
/usr/share/doc/pdns-backend-sqlite-4.5.0/schema.sqlite3.sql

cat /etc/redhat-release
CentOS Linux release 8.2.2004 (Core)
rpm -qa|grep pdns
pdns-backend-sqlite-4.5.0-0.alpha0.master.1020.g84808ff64.1pdns.el8.x86_64
pdns-recursor-4.5.0-0.beta1.master.11.g84808ff64.1pdns.el8.x86_64
pdns-4.5.0-0.alpha0.master.1020.g84808ff64.1pdns.el8.x86_64
rpm -ql pdns-backend-sqlite-4.5.0-0.alpha0.master.1020.g84808ff64.1pdns.el8.x86_64|grep "/sche"
/usr/share/doc/pdns-backend-sqlite/schema.sqlite3.sql

This is locked. If someone runs into this issue and can create a PR that solves it properly and fully, please do.


How to freeze codebook ?

I'm trying to train a network with a predetermined codebook, but I can't get the codebook to stay the same. I have tried to set vq_module.eval() but I then get the error:
Traceback (most recent call last):
  File "/media/USRNAME/Storage/Code/VQCode.py", line 110, in <module>
    avg_loss = train_one_epoch()
  File "/media/USRNAME/Storage/Code/VQCode.py", line 24, in train_one_epoch
    loss.backward()
  File "/home/USRNAME/.local/lib/python3.10/site-packages/torch/_tensor.py", line 487, in backward
    torch.autograd.backward(
  File "/home/USRNAME/.local/lib/python3.10/site-packages/torch/autograd/__init__.py", line 200, in backward
    Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
RuntimeError: element 0 of tensors does not require grad and does not have a grad_fn

Right now I'm just resetting its value in the forward pass each time.

@BMontens could you try setting ema_update = False when instantiating?

It worked! Thank you!

@BMontens wait lol
ok, how would you ideally like to use it?
maybe something like vq(codes, ..., freeze_codebook = True) ?

Well, there already is an argument "learnable_codebook" so I thought that did it (could you explain what it does btw). But yes, having a way to make a frozen VQ would be great.
Right now, what I'm doing is:
self.vq = VectorQuantize(dim = dim, codebook_size = size, ema_update = False)
self.vq._codebook.embed = torch.tensor(codes, device="cuda:0")

also added a setter method for the codebook, so you can just do self.vq.codebook = torch.tensor(codes)

Yeah, it worked! Amazing!

If you do a two-stage training where you train a codebook in the first stage and then want to freeze your codebook for the second stage, does that mean you create another codebook with freeze_codebook = True and you copy the codes from your trained codebook into the second one you just created?
trained_codebook = VectorQuantizer()

# train codebook during first stage

new_codebook = VectorQuantizer(freeze_codebook = True)
new_codebook.codebook = trained_codebook.codebook

# second stage of training where I want to use the frozen codebook

Because I've tried something like this and it looks like my network doesn't learn anything while if I remove the VQ part the loss curves are going down so I don't know where I went wrong or if this is the proper way to do it


support ingesting  changes from tables with different schemas

Currently when using SQL API,  we must define table schema in the source table. So the tables in one flink cdc sql job must be with the same schema. Is there any way to ingest  changes from tables with different schemas in one flink job? Thank you.

Closing this issue because it was created before version 2.3.0 (2022-11-10). Please try the latest version of Flink CDC to see if the issue has been resolved. If the issue is still valid, kindly report it on Apache Jira under project Flink with component tag Flink CDC. Thank you!


How to download PDF data and split dataset?

The readme mentioned that:

Data from Open-source Libraries
For the open-source PDF data, you can directly download the frames, corresponding masks, description and narrative from StorySalon.

but I can't find PDF data from that link.
Also, after processing the data, how to split them to train and val dataset. What is the proportion between them?

Emm, we have already provided frames, masks, and descriptions in the zip files in the HuggingFace repository (https://huggingface.co/datasets/haoningwu/StorySalon/blob/main/StorySalon.zip). And the indices of test data are also included, or you can find them in https://github.com/haoningwu3639/StoryGen/blob/master/data/PDF_test_set.txt and https://github.com/haoningwu3639/StoryGen/blob/master/data/video_test_set.txt.

Thank you very much for your generous reply! Looking forward to your new work!


I am getting this below error

everytime i try to tun the example it says:
Error: While trying to resolve module @flyerhq/react-native-chat-ui from file C:\Users\sriva\Downloads\react-native-chat-ui-main (1)\react-native-chat-ui-main\example\src\App.tsx, the package C:\Users\sriva\Downloads\react-native-chat-ui-main (1)\react-native-chat-ui-main\package.json was successfully found. However, this package itself specifies a main module field that could not be resolved (C:\Users\sriva\Downloads\react-native-chat-ui-main (1)\react-native-chat-ui-main\lib\index.js. Indeed, none of these files exist:

C:\Users\sriva\Downloads\react-native-chat-ui-main (1)\react-native-chat-ui-main\lib\index.js(.native|.android.js|.native.js|.js|.android.json|.native.json|.json|.android.ts|.native.ts|.ts|.android.tsx|.native.tsx|.tsx)
C:\Users\sriva\Downloads\react-native-chat-ui-main (1)\react-native-chat-ui-main\lib\index.js\index(.native|.android.js|.native.js|.js|.android.json|.native.json|.json|.android.ts|.native.ts|.ts|.android.tsx|.native.tsx|.tsx)
at DependencyGraph.resolveDependency (C:\Users\sriva\Downloads\react-native-chat-ui-main (1)\react-native-chat-ui-main\example\node_modules\metro\src\node-haste\DependencyGraph.js:311:17)
at Object.resolve (C:\Users\sriva\Downloads\react-native-chat-ui-main (1)\react-native-chat-ui-main\example\node_modules\metro\src\lib\transformHelpers.js:129:24)
at resolve (C:\Users\sriva\Downloads\react-native-chat-ui-main (1)\react-native-chat-ui-main\example\node_modules\metro\src\DeltaBundler\traverseDependencies.js:396:33)
at C:\Users\sriva\Downloads\react-native-chat-ui-main (1)\react-native-chat-ui-main\example\node_modules\metro\src\DeltaBundler\traverseDependencies.js:412:26
at Array.reduce ()
at resolveDependencies (C:\Users\sriva\Downloads\react-native-chat-ui-main (1)\react-native-chat-ui-main\example\node_modules\metro\src\DeltaBundler\traverseDependencies.js:411:33)
at processModule (C:\Users\sriva\Downloads\react-native-chat-ui-main (1)\react-native-chat-ui-main\example\node_modules\metro\src\DeltaBundler\traverseDependencies.js:140:31)
at async addDependency (C:\Users\sriva\Downloads\react-native-chat-ui-main (1)\react-native-chat-ui-main\example\node_modules\metro\src\DeltaBundler\traverseDependencies.js:230:18)
at async Promise.all (index 2)
at async processModule (C:\Users\sriva\Downloads\react-native-chat-ui-main (1)\react-native-chat-ui-main\example\node_modules\metro\src\DeltaBundler\traverseDependencies.js:198:5)

Hey @srivastavaanurag79 why is there (1) in the path? Do you have 2 folders there? Can you remove everything and start from scratch?

cleaned everything cloned in a fresh folder same error

Sam error with (1) in the path?

Ah, did you do yarn command in the both root and example? Cause it can't find lib folder that means you didn't compile the lib

Yes tried everything, more than two times. Had this issue last month also thought would be resolved in the next update or commit. But the issue is still there.

Sam error with (1) in the path?

Yes

Ah, did you do yarn command in the both root and example? Cause it can't find lib folder that means you didn't compile the lib

It just won't work, tried in a windows laptop and a MacBook pro, for iOS and Android both.

How is that not working? You need to install dependencies to run the project. You mean yarn command not working?

I mean after running the yarn command it shows that error strace

But yarn just installs dependencies, and this is typical error when the project is running. Are you sure that you can't just simply install dependencies?

Just did clone and install on a new Mac (not used to develop the library), everything runs (I have latest node, yarn, android studio). Once again:

yarn to install dependencies in the root folder
yarn to install dependencies in the example folder
yarn android to run the app from the example folder

If for some reason example does not compile on Windows, better to help me understand what are the commands you're doing, rather than throwing random accusations. I don't have access to Windows, so I am sorry I can't test there.
Alternatively, rather than using example from the project, you can just add the library through npm and just use it in your project directly, which I believe is configured to work on Windows.
Have a nice day.

Just did clone and install on a new Mac (not used to develop the library), everything runs (I have latest node, yarn, android studio). Once again:

yarn to install dependencies in the root folder
yarn to install dependencies in the example folder
yarn android to run the app from the example folder

If for some reason example does not compile on Windows, better to help me understand what are the commands you're doing, rather than throwing random accusations. I don't have access to Windows, so I am sorry I can't test there.
Alternatively, rather than using example from the project, you can just add the library through npm and just use it in your project directly, which I believe is configured to work on Windows.
Have a nice day.

I have followed all the above steps with yarn and npm, it shows the same error.

Just did clone and install on a new Mac (not used to develop the library), everything runs (I have latest node, yarn, android studio). Once again:

yarn to install dependencies in the root folder
yarn to install dependencies in the example folder
yarn android to run the app from the example folder

If for some reason example does not compile on Windows, better to help me understand what are the commands you're doing, rather than throwing random accusations. I don't have access to Windows, so I am sorry I can't test there.
Alternatively, rather than using example from the project, you can just add the library through npm and just use it in your project directly, which I believe is configured to work on Windows.
Have a nice day.

I have followed all the above steps with yarn and npm, it shows the same error. Installing directly in the project throws the same error.

Have a nice day.

I was getting this same error, following the steps suggested revealed that I needed to update my node version.  Updating node, and following the procedure again fixed the problem.

Steps for total noobs:

yarn to install dependencies in the root folder
yarn to install dependencies in the example folder
yarn start in the example folder to run metro
yarn android to run the app from the example folder


ParallelCluster compute nodes not scaling down when cloud formation stack is in status 'UPDATE_COMPLETE'

Environment:

AWS ParallelCluster version aws-parallelcluster-2.0.2
OS: alinux
Scheduler: SGE
Master instance type: m5.2xlarge
Compute instance type: r5.2xlarge

Bug description and how to reproduce:
I noticed that after making any udpates to my cloud formation stack that was created for the parallel cluster then the scale down functionality handled by nodewatcher on the compute nodes would no longer work.  Even if the nodes didn't have any jobs for the period of inactivity threshold time (i.e. 10 min) the compute node still wouldn't be spun down.  This results in loss of valuable cash.  Looking at the nodewatcher logs it would always report the stack creation completion of False.
2019-02-06 12:45:07,040 INFO [nodewatcher:main] parallelcluster-fs-galprod1805-1a creation complete: False
I believe I have found why this is, but not sure the reasoning of being coded this way.  Looking in nodewatcher.py, in function stackCreationComplete, it would only ever return true if the stack is at status CREATE_COMPLETE, however if anyone makes an update to their stack, the status then changes to UPDATE_COMPLETE, thus breaking this functionality:
def stackCreationComplete(stack_name, region, proxy_config):
    log.info('Checking for status of the stack %s' % stack_name)
    cfn_client = boto3.client('cloudformation', region_name=region, config=proxy_config)
    stacks = cfn_client.describe_stacks(StackName=stack_name)
    log.info('Status %s' % stacks['Stacks'][0]['StackStatus'])
    return stacks['Stacks'][0]['StackStatus'] == 'CREATE_COMPLETE'

I would recommend this function be changed to handle this situation.  We often make changes to the stack, for example changing the compute node type.  Possibly change the return in that function to be (this is what I have done as a temporary patch):
return stacks['Stacks'][0]['StackStatus'] == 'CREATE_COMPLETE' or stacks['Stacks'][0]['StackStatus'] == 'UPDATE_COMPLETE'

This has been fixed, just not released yet:
https://github.com/aws/aws-parallelcluster-node/blob/develop/nodewatcher/nodewatcher.py#L159

Ok, perfect.  Is there an easy way I can get my compute nodes to use this version versus the current one?

yep, clone the repo and update the cluster to use a custom node package.
See: https://aws-parallelcluster.readthedocs.io/en/latest/custom_node_package.html
Note, this will only apply to newly launched compute nodes. You can scale the asg to 0 nodes, then back up. Basically:
pcluster stop [your_cluster]
# wait a few mins for scale down
pcluster start [your_cluster]

Resolving. Feel free to re-open


Question about upcoming sparse-view novel view synthesis and iterative novel view synthesis

Hi, thanks for releasing such amazing work!
I am curious about the upcoming sparse-view nvs and iterative nvs.
(1) What will be the differences between sparse-view nvs and released single-view nvs?
In my opinion, only difference is sparse-view nvs need Dust3R to reconstruct two views and maybe need iterative nvs for large baseline?
(2) The details of iterative nvs (progressive nvs in paper)?
The paper didn't tell any details of "we backproject and align the generated novel view onto the current point cloud" after "apply ViewCrafter along the path to generate a sequence of high-fidelity novel views".
I would like to ask what kind of model is used to get the depths of novel view, still Dust3R? how to align the point clouds of novel views to current point cloud?

Thanks!

Yes, single-view NVS and sparse-view NVS differ in that the latter requires DUSt3R to reconstruct two or more views.
We use the global alignment strategy  provided by DUSt3R to backproject and align the generated novel view onto the current point cloud.