Update playwright driver to 1.15.0-next-1630342926000

Request#sizes: https://github.com/microsoft/playwright/pull/8234/files
ElementHandle#checked=: https://github.com/microsoft/playwright/pull/8525/files

CI failure will be fixed by including https://github.com/microsoft/playwright/pull/8551/commits/2a2afc8fbadc193adf755624e4bc357d2209ddb9

CI failure is due to the change of latest Playwright updates.


Fixes #10924 - Trim the redundant permissions in the permission list

Describe your changes:
Fixes #10924
Trim the redundant permissions in the permission list

Type of change:

[ ] Bug fix
[x ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[x ] My PR title is Fixes 10924 : Trim the redundant permissions in the permission list
[x ] I have commented on my code, particularly in hard-to-understand areas.

Improvement

[x ] I have added tests around the new logic.

Thank you @pdeeparao for the patch


add profiling log for livy sessions

adding profiling log for:

opening a livy session
submitting a SQL and getting results in a livy session

Sample profling log:

grep "Spark adapter" logs/dbt.log
11:59:23.553348 [debug] [MainThread]: Spark adapter: Starting livy session
11:59:23.553949 [debug] [MainThread]: Spark adapter: Using HTTP auth
11:59:23.554056 [debug] [MainThread]: Spark adapter: Creating new livy session
11:59:25.683234 [debug] [MainThread]: Spark adapter: Obtained new livy session: 691
11:59:25.683733 [debug] [MainThread]: Spark adapter: Polling to check if session is idle
11:59:26.799749 [debug] [MainThread]: Spark adapter: Session state is starting
11:59:26.800161 [debug] [MainThread]: Spark adapter: Sleeping 2 seconds before next poll
11:59:29.967822 [debug] [MainThread]: Spark adapter: Session state is starting
11:59:29.968325 [debug] [MainThread]: Spark adapter: Sleeping 2 seconds before next poll
11:59:33.145060 [debug] [MainThread]: Spark adapter: Session state is starting
11:59:33.145422 [debug] [MainThread]: Spark adapter: Sleeping 2 seconds before next poll
11:59:36.370334 [debug] [MainThread]: Spark adapter: Session state is starting
11:59:36.370837 [debug] [MainThread]: Spark adapter: Sleeping 2 seconds before next poll
11:59:39.567247 [debug] [MainThread]: Spark adapter: Session state is starting
11:59:39.567808 [debug] [MainThread]: Spark adapter: Sleeping 2 seconds before next poll
11:59:42.779152 [debug] [MainThread]: Spark adapter: Session state is starting
11:59:42.779586 [debug] [MainThread]: Spark adapter: Sleeping 2 seconds before next poll
11:59:46.074194 [debug] [MainThread]: Spark adapter: Session state is starting
11:59:46.074718 [debug] [MainThread]: Spark adapter: Sleeping 2 seconds before next poll
11:59:49.313127 [debug] [MainThread]: Spark adapter: Session state is starting
11:59:49.313746 [debug] [MainThread]: Spark adapter: Sleeping 2 seconds before next poll
11:59:52.607048 [debug] [MainThread]: Spark adapter: Session state is idle
11:59:52.607644 [debug] [MainThread]: Spark adapter: Created new livy session: 691
11:59:52.607821 [debug] [MainThread]: Spark adapter: New livy session created 691
11:59:52.607988 [debug] [MainThread]: Spark adapter: Livy session connected
11:59:52.608132 [debug] [MainThread]: Spark adapter: 691: Submitting query: select version()
11:59:53.791065 [debug] [MainThread]: Spark adapter: 691: Awaiting query results for 0
11:59:54.985270 [debug] [MainThread]: Spark adapter: 691: Query status for 0: running
11:59:54.985694 [debug] [MainThread]: Spark adapter: 691: Waiting 2 seconds to poll next for: 0
11:59:58.126779 [debug] [MainThread]: Spark adapter: 691: Query status for 0: running
11:59:58.127159 [debug] [MainThread]: Spark adapter: 691: Waiting 2 seconds to poll next for: 0
12:00:01.311489 [debug] [MainThread]: Spark adapter: 691: Query status for 0: running
12:00:01.312023 [debug] [MainThread]: Spark adapter: 691: Waiting 2 seconds to poll next for: 0
12:00:04.614740 [debug] [MainThread]: Spark adapter: 691: Query status for 0: running
12:00:04.615296 [debug] [MainThread]: Spark adapter: 691: Waiting 2 seconds to poll next for: 0
12:00:07.747721 [debug] [MainThread]: Spark adapter: 691: Query status for 0: available
12:00:07.748369 [debug] [MainThread]: Spark adapter: 691: Query execution over
12:00:07.748606 [debug] [MainThread]: Spark adapter: SPARK VERSION 3
12:00:07.749427 [debug] [MainThread]: Spark adapter: 691: Sumitting query: select 1 as id
12:00:08.912563 [debug] [MainThread]: Spark adapter: 691: Awaiting query results for 1
12:00:10.150414 [debug] [MainThread]: Spark adapter: 691: Query status for 1: available
12:00:10.150785 [debug] [MainThread]: Spark adapter: 691: Query execution over
12:00:10.153810 [debug] [MainThread]: Spark adapter: Connection.close()
12:00:10.154249 [debug] [MainThread]: Spark adapter: Connection.close()
12:00:12.312976 [debug] [MainThread]: Spark adapter: cleanup_connections
12:00:12.313091 [debug] [MainThread]: Spark adapter: Closing the livy session: 691

Look good. few minor comments.


wsl --install -d Ubuntu still installs 20.04 instead of 22.04

Version
Microsoft Windows [Version 10.0.19044.1645]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu
Other Software
No response
Repro Steps
C:\>wsl --unregister ubuntu
Unregistering...

C:\>wsl -l -v
Windows Subsystem for Linux has no installed distributions.
Distributions can be installed by visiting the Microsoft Store:
https://aka.ms/wslstore

C:\>wsl --list --online
The following is a list of valid distributions that can be installed.
Install using 'wsl --install -d <Distro>'.

NAME            FRIENDLY NAME
Ubuntu          Ubuntu
Debian          Debian GNU/Linux
kali-linux      Kali Linux Rolling
openSUSE-42     openSUSE Leap 42
SLES-12         SUSE Linux Enterprise Server v12
Ubuntu-16.04    Ubuntu 16.04 LTS
Ubuntu-18.04    Ubuntu 18.04 LTS
Ubuntu-20.04    Ubuntu 20.04 LTS

C:\>wsl --install -d Ubuntu
Downloading: Ubuntu
Installing: Ubuntu
Ubuntu has been installed.
Launching Ubuntu...

C:\>wsl -- lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 20.04 LTS
Release:        20.04
Codename:       focal

Expected Behavior
I expected WSL to download the latest Ubuntu version for distribution Ubuntu, which is 22.04.
(Long-term support releases have dedicated distribution names like Ubuntu-16.04, Ubuntu-18.04 and Ubuntu-20.04.)
Actual Behavior
Distribution Ubuntu was on version 20.04.
Diagnostic Logs
No response

@elsaco Appreciate you found a workaround until it is fixed.

wsl --list --online doesn't show 22.04

I get this exact same issue, 22.04 is not in the online list of WSL distributions.
Unfortunately I can't use the "winget" workaround, as I can't install the extra package due to work's GPO settings.

Ubuntu 22.04 was added in the manual installation page. Direct link https://aka.ms/wslubuntu2204

Ubuntu 22.04 was added in the manual installation page. Direct link https://aka.ms/wslubuntu2204

Thanks for update Biswa96.
Our work uses GPO to block access to MS Store, this link allowed me to manually download it and install as my standard user account. WSL has already been enabled on our workstations by administrators.
The only issue I had with manual installation was NS not working in /etc/resolv.conf.    During the "Updating Settings" step, I had to drop to console to change NS ipaddress, then the installation was able to complete.. must be a step in the install to do some online step.
After reboot, all worked fine.

Any updates on this?
22.04 still doesn't show up when you run wsl --list --online. Why is that?

Is it expected that Microsoft or Canonical should update the Ubuntu (version-less listing) item in the MS Store ?
If Microsoft, then is there any road-map, estimated timeline, or needed pre-requisites that need to happen ?

For the record: the 22.04.1 release was postponed until August 11 and released, although the release page still lists it in the "Future" section, and wsl --list --online does not have Ubuntu-22.04 for me. Will see what happens next.
I personally hope it will become available by default before ~August 25 so I can update my course's curriculum by September 1.

22.04.1 is in the msstore already:
Name               Id                    Version         Match       Source
----------------------------------------------------------------------------
Ubuntu             9PDXGNCFSCZV          Unknown                     msstore
Ubuntu 22.04.1 LTS 9PN20MSR04DW          Unknown                     msstore

Hmm, it seems like this actually needs a PR? See https://github.com/microsoft/WSL/issues/8634 and https://github.com/microsoft/WSL/blob/66222226bd50d2780bc0c6d12e1928ebad6e5ff6/distributions/DistributionInfo.json

So? How to install and is fixed?

You can download and install the package manually:    https://aka.ms/wslubuntu2204
May be some more info here:   https://learn.microsoft.com/en-us/windows/wsl/install-manual

You can download and install the package manually: https://aka.ms/wslubuntu2204
May be some more info here: https://learn.microsoft.com/en-us/windows/wsl/install-manual

Your link Ubuntu2204-220620.AppxBundle is 22.04 or 22.04.1 ?
Your link gives 1,3GB file and super slow 50kbit/s

Store 22.04.1 gives 667,4mb and decent speed 4Mb/s

@danielniccoli use winget and the store ID "9PN20MSR04DW" to install, or the name "Ubuntu 22.04 LTS". It will pull the correct version.
PS C:\> winget show --name "Ubuntu 22.04 LTS"
Found Ubuntu 22.04 LTS [9PN20MSR04DW]

That previous command shows this in russian

This workaround has already been posted, thanks.

The link I provided is the workaround posted above, I had to use it as our work has locked down the Microsoft Store and winget in their GPO settings, so the link was the only way I could get 22.04 version.

still doesn't up for online !!

why it's still not up on --online !! ??

why it's still not up on --online !! ??

Because it's Microsuck!

I did a wsl --install --distribution Ubuntu and it pops up an Ubuntu window for 20.04. 22.04.1 is still listed in my Start Menu. So I end up with both 20.04 and 22.04.1 installed at once. It is weird. I end up with Ubuntu.exe and Ubuntu2204.exe in my AppData\Local\Microsoft\WindowsApps as well. I end up with both of these entries in my Apps & Features even after wsl --unregister Ubuntu.

Thanks for the response, @elsaco . You must be on a different version. I see the option.
    --install [Options]
        Install additional Windows Subsystem for Linux distributions.
        For a list of valid distributions, use 'wsl --list --online'.

        Options:
            --distribution, -d [Argument]
                Downloads and installs a distribution by name.

                Arguments:
                    A valid distribution name (not case sensitive).

                Examples:
                    wsl --install -d Ubuntu
                    wsl --install --distribution Debian

Additionally, the below description from the --help suggests at least a removal of the software, though it isn't completely clear of what it does. I suppose it keeps the bare bones software and only deletes the file system.
    --unregister <Distro>
        Unregisters the distribution and deletes the root filesystem.

Even still, the install process isn't all that straightforward even still as now I have two versions of Ubuntu in my Apps & Features. My main issue was that 22.04 was installed with Version 1 of WSL. Once I did wsl --set-version Ubuntu-22.04 2 , my installs worked correctly like for oh-my-posh and go within HomeBrew.
For completeness, here is my wsl --status output.
Default Distribution: Ubuntu-22.04
Default Version: 2

Windows Subsystem for Linux was last updated on 3/28/2022
WSL automatic updates are on.

Kernel version: <IP_ADDRESS>

@mattcargile you're correct! I'm using latest version:
PS C:\Users\elsaco> wsl --version
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>
WSLg version: 1.0.47
MSRDC version: 1.2.3575
Direct3D version: 1.606.4
DXCore version: 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp
Windows version: 10.0.19045.2311

Seems like Ubuntu 22.04 was added in January: https://github.com/microsoft/WSL/commit/efd1413b9689673f956b85c226f384b1591434cb
Not sure if it's the default for wsl --install -d Ubuntu, would anyone mind checking?

I upgraded from 20.04 to 22.04 and still all references show 20.04 but in ubuntu cli it shows 22.04. So any upgrades don't update the original install................

Seems like Ubuntu 22.04 was added in January: efd1413
Not sure if it's the default for wsl --install -d Ubuntu, would anyone mind checking?

A couple months late, but just for anyone else possibly reading this thread - running wsl --install -d Ubuntu installs Ubuntu 22.04.2 LTS.

I'm still getting 20.04:
PS C:\Users\g> wsl --list --online
The following is a list of valid distributions that can be installed.
Install using 'wsl --install -d <Distro>'.

NAME            FRIENDLY NAME
Ubuntu          Ubuntu
Debian          Debian GNU/Linux
kali-linux      Kali Linux Rolling
openSUSE-42     openSUSE Leap 42
SLES-12         SUSE Linux Enterprise Server v12
Ubuntu-16.04    Ubuntu 16.04 LTS
Ubuntu-18.04    Ubuntu 18.04 LTS
Ubuntu-20.04    Ubuntu 20.04 LTS
PS C:\Users\g> wsl --install -d Ubuntu
Downloading: Ubuntu
Installing: Ubuntu
Ubuntu has been installed.
Launching Ubuntu...

$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 20.04.6 LTS
Release:        20.04
Codename:       focal

@gabrielgrant Did you already have WSL installed? I believe a bunch of my friends got Ubuntu 22.04 installed by default with no problems.

@yeputons yes, i had WSL installed (and had been using it with Ubuntu 18.04 as Ubuntu-18.04) before installing the non-version-pinned Ubuntu distribution


Error during manual release through fastlane

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
We're trying to release an iOS app manually through fastlane once it's been approved. I could see automatic_release, that serves automatic release of app after approval. But dont find any option to release an app manually after it's been approved by Apple.
Our usecase is we submit our app 2 days before the release through fastlane tools. During release window, we just need to release the app through fastlane. I dont find any command that fulfills our requirement. Please suggest any command that does this action.
Complete output when running fastlane, including the stack trace and command used
I'm running below command and getting error as below:
CMD:
fastlane deliver --skip_binary_upload --skip_metadata --skip_screenshots --automatic_release -a "${PKG_NAME}" --build_number ${IOS_BUILD_NUMBER} -u ${IOS_USER} -x false -f true
ERROR:
ITC.apps.devReject.message.and.link.pendingDevRelease.canReject
The app will not be released. I had to login to iTunes manually and click on "Release" button. We want to automate this part also.
Environment
[20:56:20]: Get started using a Gemfile for fastlane https://docs.fastlane.tools/getting-started/ios/setup/#use-a-gemfile
[20:56:20]: Generating fastlane environment output, this might take a few seconds...
✅ fastlane environment ✅
Stack

Key
Value

OS
10.12.6

Ruby
2.0.0

Bundler?
false

Git
git version 2.13.5 (Apple Git-94)

Installation Source
/usr/local/bin/fastlane

Host
Mac OS X 10.12.6 (16G29)

Ruby Lib Dir
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib

OpenSSL Version
OpenSSL 0.9.8zc 19 Mar 2015

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
9.0.1

System Locale

Variable
Value

LANG

LC_ALL
en_US.UTF-8
✅

LANGUAGE

fastlane files:
No Fastfile found
`./Appfile`
XXXXXXXXXXXXXXXXX" # Your Apple email address
XXXXXXXXXXXXXXXXXX # Developer Portal Team ID

# you can even provide different app identifiers, Apple IDs and team names per lane:
# More information: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Appfile.md

fastlane gems

Gem
Version
Update-Status

fastlane
2.79.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

slack-notifier
2.3.2

CFPropertyList
2.3.5

claide
1.0.2

colored2
3.1.2

nanaimo
0.2.3

xcodeproj
1.5.2

rouge
2.0.7

xcpretty
0.2.8

terminal-notifier
1.8.0

unicode-display_width
1.3.0

terminal-table
1.8.0

plist
3.3.0

public_suffix
2.0.5

addressable
2.5.2

multipart-post
2.0.0

word_wrap
1.0.0

tty-screen
0.6.3

tty-cursor
0.5.0

tty-spinner
0.8.0

babosa
1.0.2

colored
1.2

highline
1.7.8

commander-fastlane
4.4.5

excon
0.59.0

faraday
0.13.1

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20170404

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
2.1.0

gh_inspector
1.0.3

json
1.7.7

mini_magick
4.5.1

multi_json
1.12.2

multi_xml
0.6.0

rubyzip
1.2.1

security
0.1.3

xcpretty-travis-formatter
0.0.4

dotenv
2.2.1

bundler
1.15.4

faraday_middleware
0.12.2

uber
0.1.0

declarative
0.0.10

declarative-option
0.1.0

representable
3.0.4

retriable
3.1.1

mime-types-data
3.2016.0521

mime-types
3.1

little-plugger
1.1.4

logging
2.2.2

jwt
1.5.6

memoist
0.16.0

os
0.9.6

signet
0.7.3

googleauth
0.5.3

httpclient
2.8.3

google-api-client
0.13.6

libxml-ruby
2.6.0

@hksna Hey 👋
Did you ever get this working? - I'm currently looking for a way to release an app (via fastlane/command line) that has already been reviewed, I don't want to supply any screenshots, release notes or builds either.
Seems that deliver doesn't have this functionality? - seems it is only able to turn on automatic release.

@lukewakeford ,
We used to perform submit + release which will be fulfilled by automatic_release. However, we're getting apps approved by Apple at least a day earlier so that we just need to push release button during our release window. I don't find this option where I can just release through command line.
I had to go to iTunes and release manually instead of our CI/CD process.


Design a better logo for the framework

just something high res and minimal like a single green leaf of some sort

Hey! I wanted to contribute to this project. I think this is a good place to start. I can share some ideas and you can pick out the logo you like. I would also like to get a brief on how I can contribute to this project!

Hi there!
I'll go ahead and make the contribution file, I'll let you know when it's ready this evening. :)

here you go,
https://github.com/Basilcss/core/blob/main/CONTRIBUTING.md
when it comes to adding a new logo to the framework, simply add it to the docs and make a pull request, we will then review and accept if it suits well.

Great! I would like to get started by kindly asking you to list out whatever you would like to see in your logo. You have already mentioned about it being minimalist and high res. Anything else? Do you want it to be similar to your current logo?

Good question @SrNightmare09,
I think having it a simple green leaf of some sort would work great. If you have any great ideas just go ahead and design as you want. We're open for all ideas.

Alright, on it!

Thanks for the assignment! And a bigger thank you for adding me to your organization!!

Hello Daniel,
here is a little something I have been working on, let me know your opinions about it!

@SrNightmare09 It looks good, could you make it a bit more minimal and green?

Can you guide me through as to in what ways I can make the logo more minimal?
As for making it green, I will make it green once we finalize the design.

Hi @SrNightmare09 , we have decided to keep the current logo for now, thanks for your contribution! Feel free to try another issue!


Minecraft 1.20.1 crashed on fabric modpack

crash-2024-08-18_<IP_ADDRESS>74-quilt_loader.txt
so minecraft just crashed

can anyone help?

uhhhhh, i don't think loader github issues are the best place for getting support with crashes, i'd try getting support in the discord server.
also, its a good idea to upload crash reports to https://mclo.gs/ when getting support.

Install a newer version of Java. Specifically Java 17.


Continue to next video on error in playlist

When I tried to download a playlist, the first video failed because it has been removed from YouTube but the rest of the videos in the playlist should download.
$ youtube-dl https://www.youtube.com/playlist?list=PLg4aSwO3XsLv8-qsyQeQcFDhL_k04wMPj -v
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'https://www.youtube.com/playlist?list=PLg4aSwO3XsLv8-qsyQeQcFDhL_k04wMPj', u'-v']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2016.01.23
[debug] Python version 2.7.10 - Windows-8-6.2.9200
[debug] exe versions: ffmpeg N-76860-g72eaf72, ffprobe N-76860-g72eaf72
[debug] Proxy map: {}
[youtube:playlist] PLg4aSwO3XsLv8-qsyQeQcFDhL_k04wMPj: Downloading webpage
[download] Downloading playlist: Punch Brothers WHO'S FEELING YOUNG NOW ? New Full Album
[youtube:playlist] playlist Punch Brothers WHO'S FEELING YOUNG NOW ? New Full Album: Downloading 12 videos
[download] Downloading video 1 of 12
[youtube] cgph-dr8W_A: Downloading webpage
[youtube] cgph-dr8W_A: Downloading video info webpage
ERROR: cgph-dr8W_A: YouTube said: "Punch Brothers - Movement A..."
This video is no longer available due to a copyright claim by Warner Music Group.
Traceback (most recent call last):
  File "youtube_dl\YoutubeDL.pyo", line 666, in extract_info
  File "youtube_dl\extractor\common.pyo", line 313, in extract
  File "youtube_dl\extractor\youtube.pyo", line 1226, in _real_extract
ExtractorError: cgph-dr8W_A: YouTube said: "Punch Brothers - Movement A..."
This video is no longer available due to a copyright claim by Warner Music Group.

use the " -i, --ignore-errors              Continue on download errors, for example to skip unavailable videos in a playlist" option
youtube-dl https://www.youtube.com/playlist?list=PLg4aSwO3XsLv8-qsyQeQcFDhL_k04wMPj -v -i

As mentioned by @AndroKev use -i to skip errors.

Could we add a hint to the output when an error occurs and there are queued videos?
Hint: use the -i flag to continue on download errors


Raises an error if password length is less than 8 chars

Signed-off-by: Kapil Chouhan<EMAIL_ADDRESS>Description

Raises an error if password length is less than 8 chars and not included: 1 uppercase, 1 lowercase, 1 digit, and 1 special character
Added test cases
Ensured chef-style on the code changes made

Issues Resolved
#610
Check List

[ ] New functionality includes tests
[x] All tests pass
[x] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco
[ ] PR title is a worthy inclusion in the CHANGELOG

Thanks @kapilchouhan99


Add support for FK switching between non-castable types

Description
This is a draft PR for adding support for FK switching between non-castable types. Draft PR because:

For now it only handles the case of bigint -> timestamp.
The logic should probably be structured a lot better.

Motivation
Solves https://github.com/stripe/pg-schema-diff/issues/51.
Testing
Docker test pass. (There are 4 errors though, but they were there before the changes as well)
Considering this test case of migration:
CREATE TABLE events (
	timestamp [BIGINT -> TIMESTAMP] PRIMARY KEY,
	replicated_timestamp BIGINT
);

CREATE TABLE effects (
	name VARCHAR(26),
	timestamp [BIGINT -> TIMESTAMP],
	FOREIGN KEY (timestamp) REFERENCES public.events(timestamp)
);

Before implementing the changes, planning the migration returns an error:
Error: generating plan: validating migration plan: running migration plan: executing migration statement: {ALTER TABLE "public"."effects" ALTER COLUMN "timestamp" SET DATA TYPE timestamp without time zone using to_timestamp("timestamp" / 1000) 3s 3s [ACQUIRES_ACCESS_EXCLUSIVE_LOCK: This will completely lock the table while the data is being re-written for a duration of time that scales with the size of your data. The values previously stored as BIGINT will be translated into a TIMESTAMP value via the PostgreSQL to_timestamp() function. This translation will assume that the values stored in BIGINT represent a millisecond epoch value.]}: ERROR: foreign key constraint "effects_timestamp_fkey" cannot be implemented (SQLSTATE 42804)

After the changes, here's the generated plan:
1. ALTER TABLE "public"."effects" DROP CONSTRAINT "effects_timestamp_fkey";
        -- Statement Timeout: 3s

3. ALTER TABLE "public"."effects" ALTER COLUMN "timestamp" SET DATA TYPE timestamp without time zone using to_timestamp("timestamp" / 1000);
        -- Statement Timeout: 3s
        -- Hazard ACQUIRES_ACCESS_EXCLUSIVE_LOCK: This will completely lock the table while the data is being re-written for a duration of time that scales with the size of your data. The values previously stored as BIGINT will be translated into a TIMESTAMP value via the PostgreSQL to_timestamp() function. This translation will assume that the values stored in BIGINT represent a millisecond epoch value.

4. ANALYZE "effects" ("timestamp");
        -- Statement Timeout: 20m0s
        -- Lock Timeout: 3s
        -- Hazard IMPACTS_DATABASE_PERFORMANCE: Running analyze will read rows from the table, putting increased load on the database and consuming database resources. It won't prevent reads/writes to the table, but it could affect performance when executing queries.

5. ALTER TABLE "public"."events" ALTER COLUMN "timestamp" SET DATA TYPE timestamp without time zone using to_timestamp("timestamp" / 1000);
        -- Statement Timeout: 3s
        -- Hazard ACQUIRES_ACCESS_EXCLUSIVE_LOCK: This will completely lock the table while the data is being re-written for a duration of time that scales with the size of your data. The values previously stored as BIGINT will be translated into a TIMESTAMP value via the PostgreSQL to_timestamp() function. This translation will assume that the values stored in BIGINT represent a millisecond epoch value.

6. ANALYZE "events" ("timestamp");
        -- Statement Timeout: 20m0s
        -- Lock Timeout: 3s
        -- Hazard IMPACTS_DATABASE_PERFORMANCE: Running analyze will read rows from the table, putting increased load on the database and consuming database resources. It won't prevent reads/writes to the table, but it could affect performance when executing queries.

7. ALTER TABLE "public"."effects" ADD CONSTRAINT "effects_timestamp_fkey" FOREIGN KEY ("timestamp") REFERENCES events("timestamp");
        -- Statement Timeout: 3s
        -- Hazard ACQUIRES_SHARE_ROW_EXCLUSIVE_LOCK: This will lock writes to the owning table and referenced table while the constraint is being added. Instead, consider adding the constraint as NOT VALID and validating it later.

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

I'm going to close this, since it's stale at this point. Feel free to re-open if you're still working on it!


Fix a bug for make deploy at HEAD

This bug is a warning of converting integer to rune not digits. Not sure
what was recently changed that introduced this bug. Fix it so the make
deploy can work.
Tested by make test.
Fix #975

/assign @GinnyJI
/assign @adrianludwin


Feature request: add sumIf, countIf, etc. formulas

This is not an issue, just new feature request.
Could you please add sumIf (SUM_IF), countIf (COUNT_IF) uniqIf (COUNTD_IF), etc. formulas to your connector?
Performance of native functions in ClickHouse is much better then nested formulas like SUM(CASW WHEN ...) and develoment in Tableau will be much easier.
Thanks

Hi @kuntzi and thank you for the feature request.
While waiting for the Aggregate Combinators to be implemented (written in the dialect.tdd of the Connector),
you may want to write them "as is" using the RAWSQLAGG functionality (Live Connection only, no extract).

@yurifal Thanks for excelent idea, I just forgot about this function.

Closed with this commit.


Issue with amount

For some reason the Sdk divides my amount by 100.E.g if i want to  pay 2000 it shows 200.

That's intentional. You're supposed to pass the amount in kobo.

What do you mean by Kobo?

It's the smallest subunit of the Naira, the Nigerian currency.

I would advice that you multiply the text field value by 100 before passing it to the plugin.

Thanks for your response.I think you should write this on the documentation or allow users to state they want to use kobo explicitly with a parameter  ,i am not from Nigeria  i didn't know that.
Thank you again for the response.

For future references, there's already documentation for this and it can be found here.


Rush Multi-Project Build Tool  - Some third-party scripts might produce unexpected results due to the change in macOS version from 10.x to 11. (62477208)

Package Name:  Rush Multi-Project Build Tool
Package Version: 5.28.0
Operating system: Mac Big Sur
Node.js version is 14.10.1 (pre-LTS)
Is the bug related to documentation in
Rush Multi-Project Build Tool 5.28.0
Describe the bug
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:
Rush Multi-Project Build Tool 5.28.0 - https://rushjs.io
Node.js version is 14.10.1 (pre-LTS)
Starting "rush link"
Linking projects together...
ERROR: Internal Error: Cannot load shrinkwrap at
"/azure-sdk-for-js/common/temp/pnpm-lock.yaml"
You have encountered a software defect. Please consider reporting the issue to
the maintainers of this application.
Expected behavior
Known Issues
Some third-party scripts might produce unexpected results due to the change in macOS version from 10.x to 11. (62477208)
Workaround: Set SYSTEM_VERSION_COMPAT=1 in the calling environment, for example: $ SYSTEM_VERSION_COMPAT=1 legacy_script.pl
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
https://developer.apple.com/documentation/macos-release-notes/macos-big-sur-11-beta-release-notes

@jay-most Can you please share more details on this issue? Is the issue specific to any Azure SDK package and if yes, can you please share SDK package name and any specific steps or action that triggered this issue? If this is an in general issue in Rush multi project tool then this must be moved to rushstack repo instead of azure-sdk-for-js.

@jay-most Closing this issue as we have not heard back from you with answers to the above questions. If you still see this problem, please log a new issue with details for questions from @praveenkuttappan above.


Dynamic import can be ignored by /* webpackIgnore: true */

What problem does this feature solve?
Support ignore dynamic import.
import(/* webpackIgnore: true */ "./a.js");

What does the proposed API of configuration look like?
Support ignore dynamic import.
import(/* webpackIgnore: true */ "./a.js");

bump


update amikolajczyk bio

👷 Deploy Preview for hearai processing.
🔨 Explore the source changes: 86a0b5ded94f77d6a15fce21f969d8cd9bbf3fa2
🔍 Inspect the deploy log: https://app.netlify.com/sites/hearai/deploys/613dece20da6800008e43fcf


feat(repo): allow id_attribute to be customized per method

The PR allows the id_attribute setting to be overridden without changing the entire class configuration.
This is useful if you want to quickly use the get or delete method by another candidate/surrogate key such as slug.
Pull Request Checklist

[ ] New code has 100% test coverage
[ ] (If applicable) The prose documentation has been updated to reflect the changes introduced by this PR
[ ] (If applicable) The reference documentation has been updated to reflect the changes introduced by this PR

Description

Close Issue(s)

Outside of the scope of this PR, but wdyt about supporting instrumented attributes e.g. for the id_attribute (and other places it might apply)?

Outside the scope of this PR, but wdyt about supporting instrumented attributes e.g. for the id_attribute (and other places it might apply)?

I'm all for it.  I think it could it could help make the order by method more flexible.


New version

The version of this repo on my devices and simulator of Firefox OS not work.
So i have edit and this work without problems :-)
document.addEventListener("DOMContentLoaded", function() {
// Only enable for chromeless window
  if (locationbar.visible) return;

// Selector matches external links, but allows https/http switching
  var selector = "a[href^='http']:not([href*='://" + location.host + "']):not([target='_blank'])";

  Array.prototype.forEach.call(document.querySelectorAll(selector), function(el) {
    var target = el.getAttribute('target');
    if (!target || target.substr(0, 1) === '_') {
      el.setAttribute('target', '_blank');
    }
  });

});

@Mte90 do you have an example where the snippet doesn't work? I want to add tests to the library before fixing unreproducible issues.

has been a long time and do not remember the details but i use in my wp plugin https://github.com/Mte90/Firefox-OS-Bookmark-for-Wordpress/blob/master/public/assets/js/public.js
As i have written in the comment my code is perfect to add the target at the start but it's not useful if in the page there are some link added in the fly.So i have written a mix of this snippet for a better code/optimization:


Proposal: Support inline "if" statement syntax

This is a brainstorming idea that came from https://github.com/golang/go/issues/32825#issuecomment-506707539.
To be honest, I'm not sure if a agree with my own proposal, since it goes against some Go conventions, which disallow one-line-ifs (https://github.com/golang/go/issues/27135) or the ternary operator, for example.
Even then, I think it's worth posting the idea here and raise some discussion.

Proposal
I propose that Go could allow inline if statements, just like Ruby does. This means that a code like this:
if value < 0 {
	value = 0
}

Could be written like this:
value = 0 if value < 0

Or, to give a more common example:
if err != nil {
	return errors.Wrap(err, "oops")
}

Could be written as:
return errors.Wrap(err, "oops") if err != nil

Some experiences from Ruby
On the Ruby world, inline if statements are used a lot. Some Rubysts think inline if`s shouldn't be used for non-return statements, though, for a similar reason why Go requires blocks on if statements: it makes it harder to follow code paths.
Based on that, it's also a possibility to only allow inline if on return statements:
// this would work
return foo.thing() if foo != nil

// this would raise a compiler error
foo.doSomething() if foo != nil

This restriction may prevent users to misuse this feature. Requiring a return in the beginning of the line would mitigate the raise on complexity while reading code searching for code paths.
Talking about errors
While this proposal has no relation to errors per se, it's true that the most common use case would be reducing the number of lines of code required to do error handling. In theory, in most cases this would allow 1 line to check + return an error instead of 3.
Pros
I think this proposal is way saner then https://github.com/golang/go/issues/32437 which proposes the try built-in:

try is black magic, and will be likely very confusing for beginners, while return is clear
try is implicit, returns are explicit
try disallow doing anything different than just returning the error, while with a plain return ... if people still have control to return a wrapped error or to modify the if statement:

return err if err != nil && err != context.Cancelled

I also think this is better than most errors proposals out there (including handle, etc).
Cons

Harder to follow code paths, since if blocks are easier find while scanning code
Can encourage programmers to write long lines (80+ characters) which reduces readability
This breaks with existing Go conventions (but not more than try or handle, IMHO)

if err != nil return err

Why would your proposal be better than this example?
I don't like either, though.

Would it also allow single-line case, for, else, var () ? I'd like them all, please ;-)
See also #32611 which suggests: on err, <single_statement>

Well, the obvious sequitur seems to me only a little stretch to Go syntax; not beautiful, but that is in the eye of the beholder:
return err != nil; arg1, arg2, arg3, fmt.Errorf("failed: %v", err)
as in - for the case in point:
return err != nil; err
Which then might just lead to the totally heretical:
return arg1, arg2, arg3, fmt.Errorf("failed: %v", err); err != nil
and an incompatible idiom for returning the now "special" error argument:
func action(a int) (arg1, arg2, arg3 string; error) { ... }
(note that I intentionally omitted the "err" return argument name to suggest that the elements either side of the semicolon may be treated differently), but that is starting to look like an unsightly mess.
Also, maybe the return needs parentheses to "escape" the semicolon:
 return (err != nil ; arg1, arg2, arg3, fmt.Errorf("failure: %v", err))
or
return (arg1, arg2, arg3, fmt.Errorf("failure: %v", err); err != nil)
I leave that to the referees.
Well, one needs to turn every stone...

if err != nil return err

Why would your proposal be better than this example?

@henvic I never saw a language that allow that. IMO that's harder to read, since you don't have a keyword to separate the if statement to the rest of the code. Yes, you have return, but the inline if could be used for anything, so this would look strange:
if value < 0 value = 0

Having the if after ensure you have the if to separate the code from the check. Ruby also does that.

Would it also allow single-line case, for, else, var () ? I'd like them all, please ;-)
See also #32611 which suggests: on err, <single_statement>

@networkimprov I disagree with supporting inline else and for, I think that would hurt too much the readability.
About inline case or var, I'm not even sure what you mean here.

Well, the obvious sequitur seems to me only a little stretch to Go syntax; not beautiful, but that is in the eye of the beholder:
return err != nil; arg1, arg2, arg3, fmt.Errorf("failed: %v", err)

[...]

@lootch This falls in the same problem other proposals do, it more complicates then simplify things. That statement is less readable than a plain if statement, and I'm sure it could be confusing to new Gophers.
Inline ifs are just code, as we've already been writing, but inline to safe a few lines of code.

If you really wanted to collapse error checks into a single line, a much smaller change would be for gofmt to allow:
if err != nil { return err }

That opens its own list of questions, such as "what about for". But at least it doesn't introduce new syntax.

@mvdan, ppl keep repeating that, but it's not actually trivial.
if err == io.EOF { continue }     // if this
else if err != nil { return err } // why not this?

if t { x() } else { y() } // or this?
v := a; if t { v = b }    // or that?

And would the new go fmt rewrite much existing error code?

I'm not saying it's a good idea. I'm saying that introducing new syntax generally has a higher cost.

Generally. But in this case, probably not.

I've always been an advocate of go fmt allowing if statements which contain a single statement to be written on one line. These are very common and are often written like this in many other languages.
However, I'd leave it at that as doing more would open up a can of worms. In particular,  I wouldn't allow it if there was an else clause and I wouldn't allow it for for statements either.

However, I'd leave it at that as doing more would open up a can of worms. In particular, I wouldn't allow it if there was an else clause and I wouldn't allow it for for statements either.

I think that's the point: by having the inline if we could keep the current behavior of gofmt, disallowing single line for else and for and other usage like that.
To be honest, I'm fine with our current state, and have upvoted https://github.com/golang/go/issues/32825 to reflect that. 🙂
But if we're going to make a change, I think it may be worth paying the cost to have a better and flexible syntax. A lot of other proposals also requires deeper changes, but this one is simpler.

I think this proposal would make it too easy to bury important side-effects.
	x = thingThatNeedsToHappenForSideEffect() if false  // Easy not to notice the 'if`'.

	x = y if thingThatModifiesY()  // Does x use the value of y from before the mutation, or after?

@bcmills I agree that code like that shouldn't be written.
But I think that while we should in general reduce the possibility of bad code, we can't 100% guarantee that the user won't shoot their own foot.
I mean: functions with side effects are almost always bad and should be avoided. The inline if per se don't encourages code like that.

In my opinion
return fmt.Errorf("my error: %v", err) if err != nil

is harder to read, because it buries the important part.  When skimming through the code, it looks like a return statement, so you think "wait, the function returns now?  What is all this other code after the return?"  Then you realize that this is actually a backward if statement.
I don't understand the benefit that is worth the cost.  The "pros" listed above are all about how try is bad.  But this proposal is not about try.  If we don't adopt the try proposal, this proposal still has to stand on its own merits.  And right now I don't see the benefit.

@ianlancetaylor

I don't understand the benefit that is worth the cost. The "pros" listed above are all about how try is bad. But this proposal is not about try. If we don't adopt the try proposal, this proposal still has to stand on its own merits. And right now I don't see the benefit.

To be honest, this is indeed more a counter-proposal to try than something I'd propose if the try discussion wasn't a thing. I actually agree with you that having blocks for ifs makes it way easier to read. I wouldn't be disappointed if this proposal get rejected, I'm just doing some brainstorming here. 😉
That said, the only real benefit here is reducing lines of code and boilerplate. (Actually, that's also the only benefit of try).

In my opinion
return fmt.Errorf("my error: %v", err) if err != nil

is harder to read, because it buries the important part. When skimming through the code, it looks like a return statement, so you think "wait, the function returns now? What is all this other code after the return?" Then you realize that this is actually a backward if statement.

Well, I think this would be specially surprising for people that are not aware of this syntax, but perhaps people would get used to it. On the Ruby world, most people doesn't complain about it because they're used to see code like this.
func WriteToFile() error {
	file, err := os.Open("file.txt")
	return errors.Wrap(err, "couldn't open file") if err != nil
	defer file.Close()

	_, err = file.Write([]byte("Writing something to this file"))
	return errors.Wrap(err, "couldn't write to file.txt") if err != nil

	return nil
}

While reading the third line of code above, you kinda know the return should have an if because you see more code below in the same function. That code would be dead code if the return was a simple return, and Go wouldn't even allow that since dead code triggers a compilation error.
That's how I read Ruby code with early returns: if I see code in the method after the return statement, then I know it might be a conditional return.

IMHO a ternary operator is a good idea.
I totally disagree with the Go convention here https://golang.org/doc/faq#Does_Go_have_a_ternary_form
Consider in C the following code:
printf("my friend%s", (nbFriends>1?"s":""));
or in C++ :
std::cout << "my friend" << (nbFriends>1?"s":"") << std::endl;
In Go it causes either huges repetitions which can cause mistakes, or very verbose and inefficient code, or both, for something which should be straightforward:
Solution 1:
// horribly repetitive, risk of divergence
if nbFriends > 1 {
  fmt.Printf("my friends\n")
} else {
  fmt.Printf("my friend\n")
}

Solution 2:
fmt.Printf("my friend")
if nbFriends > 1 { fmt.Printf("s") }
fmt.Printf("\n")

Solution 3:
var plural = ""
if nbFriends > 1 { plural = "s" }
fmt.Printf("my friend%s\n", plural)

Solution 4:
// dangerous (ifTrue and ifFalse are both evaluated, contrary to a real ternary operator),
// and not generic at all (works only for strings)
func ifStr(condition bool, ifTrue string, ifFalse string) string {
  if condition {
    return ifTrue
  }
  return ifFalse
}
fmt.Printf("my friend%s\n", ifStr(nbFriends > 1, "s", ""))

I think the problem with this is that you lose the visual cue that something special is happening.
if foo {
    return bar
}

Since everything in the if block is indented, it can be ignored when trying to understand the regular flow of the function. See Mat Ryer's presentation on why he doesn't use else.

I think breaking the convention of the if statement is an overhead because your logic will be not from top to down and left to right. You don't know the if condition at first sight. Besides, it doesn't solve the error handling problem.

As noted above, the benefits of this proposal seem to be: try is bad.  But we aren't implementing try, so this proposal does not seem to bring enough benefits for the cost.
Making this orthogonal would mean that every statement can have an optional condition, which is a poor fit for the language as it exists today.  As @bcmills said above, it is easy to bury important side-effects.
-- for @golang/proposal-review


[FR] Make frontmatter complement trigger on JSON frontmatter

I format my frontmatter as JSON for certain types of data. It would be great if the frontmatter complement autocomplete for JSON syntax as well as YAML!

Hi, @shemraz :)
I have never heard of the front matter of JSON. Can we write the front matter as JSON in Obsidian?

Hi @tadashi-aikawa!
Yep, we can! Perhaps I'm in the minority who use it haha. From the Obsidian Docs:

@shemraz
Thank you for sharing. I tried it, but it did not work yesterday. However, I have tried it on today, and it works well!

@shemraz
Released in v9.0.0-beta6 🚀
I couldn't support the JSON format perfectly; however, it would work better than before.

Amazing, thank you so much!


Errors on "TRANSCRIPTION_MODEL"

With   TRANSCRIPTION_MODEL = 'deepgram' I am getting:
pygame 2.5.2 (SDL 2.28.3, Python 3.11.6) Hello from the pygame community. https://www.pygame.org/contribute.html 2024-05-20 14:38:03,651 - INFO - Recording started 2024-05-20 14:38:09,461 - INFO - Recording complete 2024-05-20 14:38:09,592 - ERROR - An error occurred: can only concatenate str (not "NoneType") to str 2024-05-20 14:38:09,592 - INFO - Deleted file: test.wav 2024-05-20 14:38:10,733 - INFO - Recording started 2024-05-20 14:38:16,905 - INFO - Recording complete 2024-05-20 14:38:17,038 - ERROR - An error occurred: can only concatenate str (not "NoneType") to str 2024-05-20 14:38:17,039 - INFO - Deleted file: test.wav 2024-05-20 14:38:18,176 - INFO - Recording started
ANd with   TRANSCRIPTION_MODEL = 'groq'
pygame 2.5.2 (SDL 2.28.3, Python 3.11.6)
Hello from the pygame community. https://www.pygame.org/contribute.html
2024-05-20 14:37:08,207 - INFO - Recording started
2024-05-20 14:37:13,265 - INFO - Recording complete
2024-05-20 14:37:13,432 - ERROR - Failed to transcribe audio: 'Groq' object has no attribute 'audio'
2024-05-20 14:37:13,432 - INFO - You said: Error in transcribing audio
2024-05-20 14:37:13,997 - INFO - HTTP Request: POST https://api.groq.com/openai/v1/chat/completions "HTTP/1.1 200 OK"
2024-05-20 14:37:14,006 - INFO - Response: Sorry to hear that! If you're experiencing issues transcribing audio, can you please provide more context or details about the error, such as:

groq is due to:
2024-05-20 14:48:41,398 - ERROR - Failed to transcribe audio: Error code: 404 - {'error': {'message': 'The model whisper-large-v3 does not exist or you do not have access to it.', 'type': 'invalid_request_error', 'code': 'model_not_found'}}
Which i have no idea how to get access to whisper via groq i see nothing about it on their website.
But the deepgram error i have no idea why it is happening since the deepgram tts is working fine.

Groq's transcription model is only accessible in private beta, and Deepgram is a placeholder for a future implementation, this is why they do not work. At the moment, for the transcription model, your only choices are OpenAI Whisper or my local implementation using FastWhisperAPI. The API can run locally, in a Docker container, and even in a Google Colab if you do not want to install anything. Here is the link to the branch of the project that uses FastWhisperAPI: https://github.com/3choff/Verbi.git,and here is the link to the repository of FastWhisperAPI:  https://github.com/3choff/FastWhisperAPI.git . Keep in mind that if you decide to use Google Colab, you will have to replace the localhost address in trascription.py with the one provided by NGROQ when you run the last cell in the notebook. I hope this helps.

@mercuryyy as @3choff pointed out Groq STT is currently gated and support for Deepgram STT will be added soon.

@mercuryyy add the deepgram transcription model in #3


LFS-596: Update webpack dependencies

WIP- a little problem with react router - need to refresh page to be loaded.

Dev change log:
"dependencies":
"@material-ui/core": "^4.4.3" - > ^4.11.0
"@material-ui/icons": "^4.4.3" ->  ^4.9.1
"react": "^16.9.0" ->  ^16.13.0
"material-table": "^1.50.0" ->  ^1.69.0
"react-dom": "^16.9.0" ->  ^16.13.0
"react-router-dom": "^5.0.1" ->  ^5.2.0
"react-number-format": "^4.3.0" ->  ^4.4.0
"moment": "^2.24.0" ->  ^2.29.0
"uuid": "^3.3.3" ->  ^8.3.0
"formik": "^1.5.7" ->  ^2.1.7
"semantic-ui-react": "^0.83.0" ->  ^2.0.0
"yup": "^0.27.0" ->  ^0.29.0
"formik-material-fields": "0.0.4" ->  ^0.0.6
"devDependencies":
"@babel/core": "^7.4.5" ->  ^7.11.6
"@babel/plugin-proposal-class-properties": "^7.4.4" ->  ^7.10.4
"@babel/plugin-proposal-optional-chaining": "^7.8.3" ->  ^7.11.0
"@babel/preset-env": "^7.4.5" ->  ^7.11.5
"@babel/preset-react": "^7.0.0" ->  ^7.10.4
"babel-loader": "^8.0.6" ->  ^8.1.0
"eslint-plugin-react": "^7.13.0" ->  ^7.21.0
"eslint": "^5.16.0" ->  ^7.10.0
"webpack": "^4.33.0" ->  ^4.44.0
"webpack-bundle-analyzer": "^3.3.2" ->  ^3.9.0
"webpack-cli": "^3.3.2" ->  ^3.3.12
Upgrade of "history": "^4.9.0" ->  ^5.0.0 has to happen once "react-router-dom" release version 6... Now that upgrade is impossible due to explicit dependency of "react-router-dom" on "history"  version 4...


added 3 test case to token.rs

i did the make fix

@ankush1233 Thanks for the PR. Could you run "make fix" to fix up the formatting of the code?

why this is showing ? this Rust CI / fmt (pull_request) i did make fix everything ran successfully then i did cargo fmt -- --check
no error nothing can u please say me whats the problem? and how can i solve the problem?

Couldnit be possible that you run a different version of rust? I see a few small space issues in the github action. I think this would show up with a make fix (note thatif there are changes, you must commit them manually)

@ankush1233 I'm seeing something similar to what CI is showing:

Let me know if you'd like someone to push a fix to your branch.

Can u do it?

@ankush1233 I might not be able to push the change, now that I think about it, because the PR is generated from a branch in your own copy of the repo.  But here is the patch you can apply to your own code:
commit 35d79f299454eaed8366c8bc06bc4db53a91637d
Author: Eric Walker<EMAIL_ADDRESS>Date:   Mon Nov 6 20:10:52 2023 -0700

    Fix linting issues

diff --git a/src/html5/tokenizer/token.rs b/src/html5/tokenizer/token.rs
index 54c48be..49de83d 100644
--- a/src/html5/tokenizer/token.rs
+++ b/src/html5/tokenizer/token.rs
@@ -221,10 +221,10 @@ mod tests {
         let token = Token::Eof;
         assert_eq!(format!("{}", token), "EOF");
     }
-
+
     #[test]
     fn test_is_start_tag() {
-        let token = Token::StartTag{
+        let token = Token::StartTag {
             name: "div".to_string(),
             is_self_closing: false,
             attributes: HashMap::new(),
@@ -232,10 +232,10 @@ mod tests {
         assert!(token.is_start_tag("div"));
         assert!(!token.is_start_tag("span"));
     }
-
+
     #[test]
     fn test_is_any_start_tag() {
-        let start_tag = Token::StartTag{
+        let start_tag = Token::StartTag {
             name: "div".to_string(),
             is_self_closing: false,
             attributes: HashMap::new(),
@@ -244,11 +244,11 @@ mod tests {
         assert!(start_tag.is_any_start_tag());
         assert!(!other_tag.is_any_start_tag());
     }
-
+
     #[test]
     fn test_is_text_token() {
         let text_token = Token::Text("TestingText".to_string());
-        let other_token = Token::StartTag{
+        let other_token = Token::StartTag {
             name: "div".to_string(),
             is_self_closing: false,
             attributes: HashMap::new(),

You'll probably want to figure out what's going on in your environment.  I'd run these commands to see if they do anything:
$ make fix
$ git status

Ok, we got past one hurdle with the updating of patch-2.  Now it looks like you'll need to fix your branch and make sure the tests run.

@ankush1233 ok, it seems you've corrected the Rust syntax errors.  Now you'll want to do three more things:

Add commit signing in GitHub and Git, so that your commits are signed
Try that make fix command again, but this time run it on the patch-2 branch
Rebase your commits into a single commit so that the commit is signed

This pr has been idle a while. I am closing in favor for #276 which merges @ankush1233 work in the core.  If you happen to read this, thanks for the work on this PR!


Use css vars for theming

The Issue
To make a theme I need to compile  all scss into > all.css every time a change it's made.
Also I can't edit vars from developer-tools ( because its get compiled to css values ), so
this make a tedious process for testing / creating new themes.
Steps to reproduce

Copy all scss styles.
Modify vars.
Build / compile scss > all.css

Expected behaviour
Enable editing themes (vars) inside developer-tools
Actual behaviour

To make a theme I need to compile  all scss into > all.css every time a change it's made.
Also I can't edit vars from developer-tools ( because its get compiled to css values ), so
this make a tedious process for testing / creating new themes.

System Configuration

Configuration
Version

App
0.14.3

Daemon (lbrynet)
0.14.2

Wallet (lbryum)
3.1.4

Platform
Linux (Linux-4.10.0-28-generic-x86_64-with-elementary-0.4.1-loki)

https://developer.mozilla.org/en-US/docs/Web/CSS/Using_CSS_variables

Example:
:root {
  --red: #b00;
  --blue: #4679bd;
  --grey: #ddd;
}
body {
  color: var(--red);
  background: var(--grey);
}

We can override the current scss vars like this:
   $color-main: var(--color-main);
   $color-bg: var(--color-bg);

https://spee.ch/9/Custum-Create-Your-Own.png

Merged in branch v16, to be next release.


Failed to initialize WebGL context.

Steps to Reproduce

Execute flutter run
Launching lib/main.dart on Chrome in debug mode...

Expected results:
Render flutter web correctly

Actual results:

WARNING: failed to attach a stencil buffer. Rendering will be skipped.
WARNING: failed to attach a stencil buffer. Rendering will be skipped.
WARNING: Falling back to CPU-only rendering. Failed to initialize WebGL context.
Flutter Web Bootstrap: Programmatic
WARNING: failed to attach a stencil buffer. Rendering will be skipped.
══╡ EXCEPTION CAUGHT BY SCHEDULER LIBRARY
╞═════════════════════════════════════════════════════════
The following LegacyJavaScriptObject object was thrown during a scheduler
callback:
  RuntimeError: null function or function signature mismatch

When the exception was thrown, this was the stack:
<EMAIL_ADDRESS>178:52  _flush
<EMAIL_ADDRESS>12:244  flush
lib/_engine/engine/canvaskit/surface.dart 404:13                 flush
lib/_engine/engine/canvaskit/surface.dart 369:5
[_presentSurface]
lib/_engine/engine/canvaskit/surface.dart 115:14                 <fn>
lib/_engine/engine/canvaskit/surface.dart 32:26                  submit
lib/_engine/engine/canvaskit/rasterizer.dart 35:12               draw
lib/_engine/engine/platform_dispatcher.dart 586:7                render
lib/ui/window.dart 104:50                                        render
packages/flutter/src/rendering/view.dart 231:15                  compositeFrame
packages/flutter/src/rendering/binding.dart 508:18               drawFrame
packages/flutter/src/widgets/binding.dart 892:13                 drawFrame
packages/flutter/src/rendering/binding.dart 370:5
[_handlePersistentFrameCallback]
packages/flutter/src/scheduler/binding.dart 1146:15
[_invokeFrameCallback]
packages/flutter/src/scheduler/binding.dart 1083:9               handleDrawFrame
packages/flutter/src/scheduler/binding.dart 997:5
[_handleDrawFrame]
lib/_engine/engine/platform_dispatcher.dart 1090:13              invoke
lib/_engine/engine/platform_dispatcher.dart 160:5
invokeOnDrawFrame
lib/_engine/engine/initialization.dart 194:45                    <fn>
════════════════════════════════════════════════════════════════════════════════
════════════════════
WARNING: Falling back to CPU-only rendering. Failed to initialize WebGL context.
Flutter Web Bootstrap: Programmatic
WARNING: failed to attach a stencil buffer. Rendering will be skipped.
WARNING: Falling back to CPU-only rendering. Failed to initialize WebGL context.
Flutter Web Bootstrap: Programmatic
WARNING: failed to attach a stencil buffer. Rendering will be skipped.
WARNING: failed to attach a stencil buffer. Rendering will be skipped.
WARNING: Falling back to CPU-only rendering. Failed to initialize WebGL context.

Logs

flutter doctor -v

[✓] Flutter (Channel stable, 3.0.0, on Linux Mint 20.3 5.4.0-110-generic, locale en_US.UTF-8)
    • Flutter version 3.0.0 at /home/mahan/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ee4e09cce0 (9 days ago), 2022-05-09 16:45:18 -0700
    • Engine revision d1b9a6938a
    • Dart version 2.17.0
    • DevTools version 2.12.2

Chrome (web)    • chrome • web-javascript • Google Chrome 101.0.4951.64

neofetch

OS: Linux Mint 20.3 x86_64
Host: HP Compaq Elite 8300 SFF
Kernel: 5.4.0-110-generic
Resolution: 1366x768
DE: Cinnamon
CPU: Intel i5-3470 (4) @ 3.600GHz
GPU: Intel HD Graphics

I did not have this problem in Windows

All flutter web builds have this problem for me

Interestingly, this problem does not exist in Firefox (Firefox 100)

Dart dev  tools in chrome :

Dart dev tools in firefox :

@mdpe-ir
I think you are hitting https://github.com/flutter/flutter/issues/96919 and root-cause seems to be discussed in the underlying comments in it, so I suggest you to follow-up in it for further updates.
Regarding runtime null error, the same is captured in this open issue https://github.com/flutter/flutter/issues/91674 in which you can follow-up.
Closing this as a duplicate. If you disagree, write in comments and I'll reopen it.


redesign bmm-page

very open for writing something else in the attempt

On mobile so can't review properly. But from the screen shot: the bid amount is on the mainchain, I.e. regular btc not sbtc. Fees are collected on the sidechain though, so those are sbtc


feat: adds relationship-based filtering to GraphQL

Description

This PR adds the ability to filter documents in GraphQL on nested relationship fields. For example, if one has collection A with field r being a relationship to collection B with a text field t, it is now possible to do a { r__t: { equals: … } } where condition. Before, this wasn't possible.
Related:
#761
#957
https://discord.com/channels/967097582721572934/1146829720885919744/1146829720885919744
This is my first contribution to Payload, so let me know what edge cases I might have missed or any code-related changes you would like to see. I didn't find a good place in the documentation to add a description of this feature, but if there is a spot I missed let me know!

[x] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] I have added tests that prove my fix is effective or that my feature works
[x] Existing test suite passes locally with my changes

@JarrodMFlesch is there any chance this get accepted soon?

This would be an awesome addition. It's the only thing that blocks us from using graphql and payloadcms

Bump - sorely needed!

Also bumping. Really useful feature!

Tried the commit, but it seems to cause infinite recursive loops on a project using @payloadcms/plugin-nested-docs.

Bumping! We really need this.

I'm not really sure how I missed this. I agree this feature would be killer.
I'll try to give this a look next week! Feel free to ping me if I don't!

Bumping!

This still seems like a useful feature. However, it would need to be reworked for 3.0.

Still hoping @JarrodMFlesch will find time to re-implement this :D

following

Bumping!


Errors and poor performance using fastApi/uvicorn compared to falcon/gunicorn

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
def get_app() -> FastAPI:
    mycompnent_app = FastAPI(
        title="mycompnent",
        description="my app",
        openapi_tags=MYCOMPONENT_METADATA
    )
    mycompnent_app.include_router(api_routeur, prefix=MYCOMPONENT_PREFIX)
    mycompnent_app.add_event_handler("startup", start_app_handler(mycompnent_app))
    mycompnent_app.add_event_handler("shutdown", stop_app_handler(mycompnent_app))

    return mycompnent_app

app = get_app()

---router--
@router_m.post("/predict", response_model=ObjectEnrichiInference, name="prediction" )
async def prediction(object: ObjectDTO, request: Request) -> ObjectEnrichiInference:
    .....
    return msg

Description
Hello,
We have a machine learning use case where using uvicorn with fast api has bad performances.
You can find the results of the test via this link: (https://digitalean.com/performance_python_machine_learning_use_case.pdf)
Can you tell us why we have this performance please? And can you help us to have better performance with uvicorn and fast api please?
Thanks in advance.
Operating System
Other
Operating System Details
Kubernetes python pods
FastAPI Version
fastapi==0.65.1 uvicorn[standard]==0.13.4
Python Version
python:3.7
Additional Context
Hello,
We have a machine learning use case where using uvicorn with fast api has bad performances.
You can find the results of the test via this link: (https://digitalean.com/performance_python_machine_learning_use_case.pdf)
Can you tell us why we have this performance please? And can you help us to have better performance with uvicorn and fast api please?
Thanks in advance.

Highly likely you are blocking the event loop with whatever you are doing in prediction
gunicorn/gthread is mitigating this for you by putting things in a thread pool.
If you change your async def prediction to def prediction FastAPI will spawn it on a thread pool and things might speed up a bit  but you should not be doing intense computational work in async python because it is cooperative not preemptive multitasking.

Or maybe your workload is just not async (most ML workloads are CPU bound) so you're comparing the overhead of an async framework with a better performing (via Cython and less features) sync framework


Added Dockerfile

Fixes #685

@thestarsahil I have added the dockerfile.
Thankyou

Your branch is not up to date  , Pull the latest changes and then push it

Your changes showing the previous edition


web components and esm

hey hey! i've been looking for a mobx replacement because it's so bloated: and valtio looks really elegant and promising!
unfortunately, i can't consider valtio because my apps are esm-only and use true web components — no react — i'm currently using lit-element
are there any plans to decouple valtio from react, so it can be used in any application, like mobx?
to alleviate the incompatibility, in addition to the react-decoupling work, the esm distribution would need to be modified to include .js extensions on all non-bare-specifier esm imports (so that library can work natively in browsers), eg:

import { createMutableSource, useMutableSource } from './useMutableSource'
    becomes
import { createMutableSource, useMutableSource } from './useMutableSource.js'

:wave: chase

Hi, honestly I didn't expect this is even useful without React. Most of effort is to combine with React. So, there was no such plan.
That said, as you express strong interest, let me try decoupling and see how it goes.


Data Vault tables should push down the table definition options

SQL Aide code for the Data Vault
const hl7Hub1 = dvts.hubTable("entity", {
hub_entity_id: primaryKey(),
business_key1: tcf.unique(text()),
business_key2: tcf.unique(text()),
...dvts.housekeeping.columns,
});
Output generated
CREATE TABLE IF NOT EXISTS "hub_entity" (
    "hub_entity_id" TEXT PRIMARY KEY NOT NULL,
    "business_key1" TEXT /* UNIQUE COLUMN */ NOT NULL,
    "business_key2" TEXT /* UNIQUE COLUMN */ NOT NULL,
    "created_at" TIMESTAMP DEFAULT CURRENT_TIMESTAMP,
    "created_by" TEXT NOT NULL,
    "provenance" TEXT NOT NULL,
    UNIQUE("business_key1"),
    UNIQUE("business_key2")
);

So required the composite  option for the hubTable and satelliteTable also as below:
CREATE TABLE hub_entity (
  hub_entity_id NOT NULL,
  business_key1 NULL,
  business_key2 NULL,
  CONSTRAINT hub_entity_pkey PRIMARY KEY (hub_id),
  CONSTRAINT hub_entity_unq UNIQUE (business_key1, business_key2)
);

This appears to be done so I'm closing it. @vinodkalpaka if you still need to do some work on it please reopen.

Add unique constraints to the Data Vault link tables.


Modify EarlyStopping on_train_end

If restore_best_weights=True and the last epoch of training was reached without triggering the early stop (probably because to a very high patience), the best set of model weights should be restored anyway.

@lbosk are you able to address the comment? Note that for formatting you can just use black.

@lbosk Can you please check @fchollet's comments and keep us posted ? Thank you!

Hi @lbosk Any update on this PR? Please. Thank you!

Hi @lbosk Any update on this PR? Please. Thank you!


Cannot post a simple transaction of 209434 bytes

Running latest master branch of BigchainDB I cannot post a transaction of ~200 KB
Steps to reproduce:
From https://github.com/bigchaindb/project-jannowitz/tree/code-examples/js-examples
Run node differentPayloadSize.js
The logs of the server when inserting a transaction of this size is:
bigchaindb_1   | [2018-04-18 11:46:59] [DEBUG] (gunicorn.error) GET / (webapi     - pid: 23)
bigchaindb_1   | [2018-04-18 11:47:00] [DEBUG] (gunicorn.error) POST /api/v1/transactions (webapi     - pid: 23)
bigchaindb_1   | [2018-04-18 11:47:00] [DEBUG] (urllib3.connectionpool) Starting new HTTP connection (1): tendermint (webapi     - pid: 23)
bigchaindb_1   | [2018-04-18 11:47:00] [DEBUG] (urllib3.connectionpool) http://tendermint:46657 "POST / HTTP/1.1" 200 190 (webapi     - pid: 23)
bigchaindb_1   | [2018-04-18 11:47:00] [DEBUG] (bigchaindb.tendermint.lib) {'jsonrpc': '2.0', 'id': '35d5eb20-6ad3-431b-8c65-8943d8987439', 'result': {'code': 0, 'data': '', 'log': '', 'hash': '0FADA44F5302012E41D83B81AD1CD93841004196'}} (webapi     - pid: 23)

I do not see any log for creating a block, not any other warning.
Also cannot find the transaction searching for it with the id and the transactions endpoint

I found some interesting parameters in the Tendermint source code, specifically for:

0.12.1
0.19

The limits are 21MB for a block, and 10kB for a single transaction.
const (
	MaxBlockSizeBytes = 104857600 // 100MB
)

// DefaultBlockSize returns a default BlockSize.
func DefaultBlockSize() BlockSize {
	return BlockSize{
		MaxBytes: 22020096, // 21MB
		MaxTxs:   100000,
		MaxGas:   -1,
	}
}

// DefaultTxSize returns a default TxSize.
func DefaultTxSize() TxSize {
	return TxSize{
		MaxBytes: 10240, // 10kB
		MaxGas:   -1,
	}
}

There were some issues in py-abci (abci) which caused errors with larger transactions. Those were fixed and are currently available in bigchaindb-abci==0.4.4 (which is used by bigchaindb==2.0.0a3). We plan to make a pull request with our fixes to the main py-abci repo.
@manolodewiner can you check if this issue still exists?

Yes the problem seems solved using the latest version of the master branch of BigchainDB.
Great job team, thanks!


Exclude two features (time and amount) from fraud dataset

These two features are not used as predictors in Hummingbird. Excluding them makes the number of predictors/features 28. The other minor change is to fix a typo in the output file name.

The other minor change is to fix a typo in output file name.

Thanks Hong! Could you please give a link to the Hummingbird script that has the problem? Also, we do not need to keep everything exactly the same with HummingBird, I think. HummingBird is a great paper, but their experiments are not a standard benchmark. Using all the features in the real-world dataset may be more interesting. What do you think?

The Hummingbird script is here: https://github.com/microsoft/hummingbird/blob/bb87598466744aa5729d8b59e14e938c4871e7fa/benchmarks/datasets.py#L41:~:text=X %3D df[[col for col in df.columns if col.startswith("V")]]
The link below is an description of the Fraud dataset:
https://www.kaggle.com/datasets/mlg-ulb/creditcardfraud#:~:text=Features V1%2C V2,and 0 otherwise.
I also found that the number_features in the config.json is set to 28.

Hong,
In config.json, every dataset is set to 28 features, which looks incorrect to me. For example, if you download the airline dataset, it only has 14 features. Let's fix that!
Also, can you explain why to remove amount and time? Again, we do not need to follow what other papers do unless there is a solid reason.

Just had a discussion with Hong and agreed on several things:
(1) Some Kaggle studies showed that Amount is a good feature for XGBoost classification on the fraud dataset, and they also kept Time and achieved 98% recall on this skewed dataset: link
(2) We are not sure why HummingBird removed these two features, but we find it followed NVidia's gbm bench: link To keep consistent with the previous benchmarks may be a good idea for verification of our benchmark results.
(3) Now the scripts are only tested for a few benchmark datasets, so the config.json needs to be updated for setting up the correct values for features, query, y_col, categorical_col, etc, for other datasets. We will work on that in the next several days.


Serving static export from sub-folder

I've exported and deployed index.html and main.js to a sub-folder via FTP. Since it's not served from a sub-domain, once you go back and forth it removes the URL path and appends the #slide_no to the main domain.
Is there any setting for the base URL?
PS: Thank you for this great tool!

Thanks! This looks like a bug, I think pushState should only set e.g. #1 without the hash

Should be fixed in the latest release, but please reopen this if not


Rewrite readme

Rewrite and re-order readme.
Main changes:

Remove instructions on installing homebridge. There are more complete directions on the project page.
Add more detail on plugin setup and use.
Add examples for each accessory type.
Misc cleanup and reordering.

Hi, do you know how to change status of device (on or off) by command line? I plan change status after receive some hex ir, so this way i could "sync" tv remote or other device with homekit.. Today if you power on tv, then turn off with tv remote, homekit still think that tv is on.. I would like to run external software/script that after receive hex command, it change status of device..  First step is to know how change status by command.. Thank you

Hi,
This is a pull request for documentation and not the correct place for your question. Your question is also not specific to this plugin; homebridge is what dictates accessory states
Go to: https://github.com/nfarina/homebridge
Possible solution: https://github.com/nfarina/homebridge/issues/506

Thank you. I know that i am on not correct place, its just i dont see any activies last months.. This pull request was accepted?

Not yet, still open. I hope it will be reviewed soon

I think that owner abandoned this.. Last commit was july.. Last open pull request also was july.. It have really good potential.. But to be perfect should add one really important feature, sync online commands (siri) with offline commands (tv remote, air conditioner remote, etc), this way would make devices really smart.. Today status of device is aways wrong (because you dont use just homekit).. But to do that, it would envolve broadlink device to.. (to aways listen IR and change device status).. Lets Hope this project developer or somebody forks it.. Anyway i will do research of how can i do something like it..

Sorry but I am having trouble understanding what you mean. The Broadlink unit has no idea what the state of the device is (or even if the signal was received by the device). This is why the disableAutomaticOff parameters are there.
If you need this feedback then you should look at another solution that needs to physically or electrically interact with the target device.

yes, i know that can use disableAutomaticOff .. but this is no real solution.
Imagine that situation:
Today work this way:
Turn on tv  using homekit (homebridge)
Status of homebridge tv device turn "on"
Then, you turn off using tv remote control (offline device)
Status of homebridge tv device continues "on" because it cant know that tv was truned off using tv remote control
------------------------------------------------ What i would like to change (depend new features)
Turn on tv  using homekit (homebridge)
Status of homebridge tv device turn "on"
Then, you turn off using tv remote control (offline device)
New  --> Broadlink  would aways listen infrared  command, then  if detect tv remote control power hex signal, it would change status of homebridge tv device to "off"
This way, aways status of devices would be right..

@alexymik Hi, could you change the required status of key-values, into a separate column? Like this:

key
Description
value
default value
Required

name
A descriptor for the accessory that will show in HomeKit apps.
"TV"
-
Required

type
The type of accessory.
"switch"
-
Required

persistState
Determines whether the state of accessory persists after homebridge has been restarted.
false
true
Optional

resendHexAfterReload
When persistState is true (it is by default) this will resend the hex code for the last known state if homebridge is restarted.
true
false
Optional

disableLogs
Disables the log output for this accessory.
true
false
Optional

Thanks for all of the work you did on this. I've taken on board a lot of the input but unfortunately there were too many changes that I didn't want to move over and too much movement of text for me to be able to use this PR.


chore Refactor be CMakeLists option

Proposed changes
Refactor be CMakeLists option
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run clickbench-new

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 47.51 seconds
stream load tsv:          540 seconds loaded<PHONE_NUMBER>9 Bytes, about 132 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.2 seconds inserted 10000000 Rows, about 342K ops/s
storage size:<PHONE_NUMBER>6 Bytes

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 47.07 seconds
stream load tsv:          512 seconds loaded<PHONE_NUMBER>9 Bytes, about 139 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          32 seconds loaded 861443392 Bytes, about 25 MB/s
insert into select:          29.3 seconds inserted 10000000 Rows, about 341K ops/s
storage size:<PHONE_NUMBER>4 Bytes


[consensus] Remove unused dependency on parity-multiaddr

During dependency triage we realized this dep was unused.

@bors-libra r+

:pushpin: Commit 0546081 has been approved by bmwill

@bors-libra r+

:pushpin: Commit 6355d37 has been approved by bmwill

:hourglass: Testing commit 6355d3765f39ae1b7473b7320006b5239d5f3571 with merge 5e32118e37ba4f95c0548a009730c1da04d31c1d...

:broken_heart: Test failed - checks-circle_commit_workflow

@bors-libra retry

:hourglass: Testing commit 6355d3765f39ae1b7473b7320006b5239d5f3571 with merge b70fd1681d6b7d3a01f413d69175e2146eee60f0...

:broken_heart: Test failed - checks-actions_land_blocking_test

@bors-libra retry

:hourglass: Testing commit 6355d3765f39ae1b7473b7320006b5239d5f3571 with merge c0256d341364362194a014ef4a3b9f8803b362da...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: bmwill
Pushing c0256d341364362194a014ef4a3b9f8803b362da to master...


Automate composited meteo catalogue completion

Already work in progress. Please track the status also here.

fyi; a more long term item we are currently investigating for cropsar 2D
We are investigating the best way to automate updating STAC collections for large scale jobs.


We could decouple hs_discover builds from HS builds

Related to https://github.com/hydroshare/hydroshare/issues/5506
Currently, we build hs_discover every time that we build the Hydroshare image.
https://github.com/hydroshare/hydroshare/blob/2.15/Dockerfile-multistage-node#L1-L31
If we build and publish an hs_discover image separately, it will speed up our HS deployments/rebuilds--the discover/node image can be pulled and cached without having to run npm install and npm run build on every HS deployment

actually, I'm realizing this wont improve or deployment/spin up time. Just our build time, but that's of less interest


Make splash screen fade out duration configurable

Thanks for this!
I do worry that with adding more and more optional configurations, the configuration section of the script becomes harder to read and find what developers usually change. I created #64 to discuss that.
Having said that, LGTM, Thank you!


How to deriving OCaml inner type like Lexing.position?

type pos = Lexing.position
[@@deriving show]

In the way, it will still cause the Error: Unbound value Lexing.pp_position

https://github.com/whitequark/ppx_import/#combining-with-deriving

When using yojson, it shows yojson cannot be derived for [%import :Lexing.position]

Note that you need to require import before any deriving plugins, as otherwise deriving will not be able to observe the complete type.

Thanks


small css changes on homepage

🚀 Deployed on https://670829e5700eca3c66841350--cozy-fox-0defec.netlify.app


Highlighted icon overlapping on text

Highlighted icon overlapping on text.
URL: https://franklin.bevespi.com/how-to-use-bevespi-inhaler
Devices: Samsung S22 Chrome
Test Evidence:
Actual:

Duplicate issue https://github.com/hlxsites/bevespi/issues/466


Join declaration and assignment

Declaration and assignment can be joined
Use IDE static checker to identify and fix variables that
can be initialized rather than declared and assigned.
This was a tedious semi-automated process.
Followup commits were changes that could be found
by using the joined declaration and assignment.
PR Checklist

[x] No API changes were made (or the changes have been approved)
[x] No major design changes were made (or the changes have been approved)

Thanks for all the work put lately to make the style in ITK more streamlined with recommendations and more consistent @hjmjohnson :100: and sorry for not being able to contribute with PRs lately.
The dashboard has started complaining about changes related to this PR, e.g.
https://open.cdash.org/viewBuildError.php?buildid=10096096

@jhlegarreta  Weird. I did not see that on my builds.  ... Fixed by initializing in next PR related to const and constexpr.


packit should by able to read  packit.yml file

Based on comment https://github.com/dcantrell/pykickstart/pull/259 packit should be able to read also packit.yaml instead of .packit.yaml

please also do packit.yml; and let's add this info into /docs/configuration.md


Trying to update certificate, but it is unfortunately creating a new one with same name

PS C:\Users\username> .\update-iis-certificate.ps1 .\mydomain.no.pfx -CertSubject "CN=*.mydomain.no" -PFXPassword "secret_passwd"
Transcript started, output file is C:\windows\Temp\update-iis-certificate-2020-09-10T13-00-44.txt
Logging to C:\windows\Temp\update-iis-certificate-2020-09-10T13-00-44.txt:
+++ Start Certificate Update +++

Loading the Web Administration Module...

Locating the current(old) certificate in store...

Unable to locate current(old) certificate in store!

Importing certificate into store...

Imported certificate:
PSParentPath: Microsoft.PowerShell.Security\Certificate::LocalMachine\My

Thumbprint                                Subject

44D35E0954929AB4357404E81FB9EC8E35BBE82F  CN=*.mydomain.no

Locating the new certificate in store...
Unable to locate new certificate in store!
+++ Completed Certificate Update +++
Transcript stopped, output file is C:\windows\Temp\update-iis-certificate-2020-09-10T13-00-44.txt

PS C:\Users\username> dir Cert:\LocalMachine\My
PSParentPath: Microsoft.PowerShell.Security\Certificate::LocalMachine\My
Thumbprint                                Subject

F7DE5FABC06AB240E4A3DABB3D95360842E6ADFA  DC=Windows Azure CRP Certificate Generator
01F367EB2A4ABCBCC8364C8975D2919EE332AE8E  CN=.mydomain.no
44D35E0954929AB435740ABC1FB9EC8E35BBE82F  CN=.mydomain.no
As you can see, the certificate is imported, but the binding(s) in IIS is not happening.
Any suggestions would be helpful.  Thanks!

Update, the -Remove works:
`PS C:\Users\username> dir IIS:\SslBindings\
IP Address          Port   Host Name        Store            Sites

<IP_ADDRESS>             443                     My               Default Web Site
PS C:\Users\username>
PS C:\Users\username>.\update-iis-certificate.ps1 -CertSubject "mydomain.no" -Remove
PS C:\Users\username>.\update-iis-certificate.ps1  -PFXPath .\mydomain.no.pfx -CertSubject "CN=*.mydomain.no" -PFXPassword "secret_pass"
PS C:\Users\username> dir IIS:\SslBindings
PS C:\Users\username>
PS C:\Users\username>.\update-iis-certificate.ps1  -PFXPath .\mydomain.no.pfx -CertSubject "mydomain.no" -PFXPassword "secret_pass"
PS C:\Users\username> dir IIS:\SslBindings
PS C:\Users\username>`
But I have to install the certificate binding manually by clicking in the IIS manager GUI.
Am I using the command syntax wrong?? What is the recomended content to the -CertSubject parameter?

I'm having the same issue with a wildcard cert.. regular cert works fine and installs the bindings fine.

I'm having the same issue with a wildcard cert.. regular cert works fine and installs the bindings fine.

Looking at the code a little bit and it looks like all you have to do is use something like this for your -CertSubject flag:
-CertSubject *.example.com

No quotes. This worked for me!

Looking at the code a little bit and it looks like all you have to do is use something like this for your -CertSubject flag:
-CertSubject *.example.com

No quotes. This worked for me!


Total RAM in a cluster?

Sorry for the naïve question but I don't really understand how a cluster works, so here it goes:
If I have a cluster with 3 instances and each instance has 8 Gb of RAM, does my cluster have a total of 24 Gb of RAM available to the apps?
I guess it doesn't, and that the only reason for having several instances is high availability?

Additional cluster nodes provide both more capacity (i.e RAM/CPU for applications) and high availability.
So in your case yes, there should be roughly 24GB of RAM available to applications minus some RAM for Flynn and OS level system services.

Awesome! Thanks :)


Including a plot with Sphinx?

Hey, love the project; keep up the good work.
I'm curious as to how I might include an mpld3 plot in my RTD site. I cruised the code and saw plot_generator.py. Is including that as a Sphinx extension a way I could do it? I just couldn't find any documentation on it but it's apparently able to be done. Maybe a wiki page would help (or maybe I just couldn't find it)?
Thanks

Figured out one solution at least.
.. raw:: html
    :file: yourmpld3.html


Synapse prep in CDC Subscription

Additional configuration of synapse workspace in CDC subscription

This is now closed as of 5/1.


Update the spec parameters (MaxMemory, etc) for running workflow.

As discussed with Alan,

add the time stamp in couch db reqmgr when spec file is updated.
add new table with workflow id and updated timestamp.
Job updater compares the time stamp in wmbs (table above) and reqmgr2 couchdb.
if reqmgr2 couchdb record is newer, update the specs in the disk (JobCache, sandbox),
update the wmbs table.

See #8646 for further details

@vlimant I'm planning to work on this ticket in the coming weeks. Unless you think Unified mechanism is good enough and we don't need it. So your input and answer to the questions I asked above would be highly appreciated.

@amaltaro there are several other candidates for unified integration already (#8914, #8921, #8920, #8324, ...)  ; I believe those are the ones we put together as first thing of the integration.
The mechanism for classad tweaking in unified is all in https://github.com/CMSCompOps/WmAgentScripts/blob/master/Unified/equalizor.py and is depending on gwmsmon (although everything can be retrieved from ES directly). It will likely require further documentation of what is done exactly

@amaltaro , According to a small discussion with James today morning, these computations are a little expensive and increase the loads on the schedd. He mentioned that @todor-ivanov tried implementing something like this in CRAB3 schedds which made the schedds slower. So the best place could be to implement this directly at the condor level(probably in a schedd attached to negotiator) and can be a feature request to the condor developers. May be we can ask about this in the next condor developers meeting.

@sharad1126 - I’m not sure that comment makes much sense.  Without knowing the exact thing Alan is planning, it could be almost no load - or very expensive.
In fact, if done right, this could be much more efficient than the current system because one could affect all idle jobs in a single transaction instead of doing it one-by-one (like Unified does today).

@bbockelm I discussed about this with @amaltaro and then I discussed this with James Letts and James told me what I exactly mentioned in the above comment. Of course it is a good idea to get this done which would help us making the system more efficient.

What I have in mind is actually an update of the workflow spec file, such that jobs still waiting in the global workqueue (or waiting for the agent job splitting) could use the up-to-date parameters, thus stopping the usage of JobRouter.
In the next phase of this tunning, we could also update jobs pending in the local condor queue (basically the same process as done for RequestPriority/JobPrio).
I believe those 2 approaches are not tightly coupled and can be delivered in different stages.

JobRouter is using some legacy code currently which will be removed very soon.
https://github.com/CMSCompOps/WmAgentScripts/pull/489/commits/1c04a0b8429fa5e8d102044a11d85aed63118731


Manimlings? An easier and effective way of learning manim

Questions to probe the reader's understanding
Like they did for the Rust Documentation, they provide a quiz at the end of each session.

https://rust-book.cs.brown.edu/

How can the new feature be used?
This way the user can track progress and most important: increase retention when reading the doc
Which in some way, is not different from studying

https://docs.manim.community/en/latest/tutorials/quickstart.html ?

We should have a page with a clear outline of where people can find resources for learning programming with python and maybe integrate some terminology into the docs so people have an easier time understanding the differences between things like classes, objects, static methods, functions as parameters and so on.

Custom Mobjects and a streamlined way for creating them should be a part for experienced users.

The documentation should provide useful information for short and long time users of the library

An explanation on how to go past the documentation with information on how to actively use the docs correctly in order to achieve the wanted animations would also be a good idea. (How to search for keywords, and maybe going into the structure of Manim a little, because i think the deepdive is a bit to deep for the average user at the moment)

#1352

When I was learning how to use manim, the biggest problems I had with the documentation were:

The quickstart was far too basic for me as I already had programming experience
It was very annoying to find a method because you had to basically look through the inheritance tree

This could also be confusing for first-time programmers who have never heard of inheritance before
I feel like these should definitely be tackled.

There's some documentation inspiration that I think would be cool:
If you go to the old discord.js documentation (I haven't used the new one in a while), you'll see that when you search for a particular thing, it returns a list of classes, types, properties, methods etc that contain that word. The way the current manim docs work is that it's just a full-text search, returning any and all links that contain the word you're searching for. This is not particularly helpful. Say you're looking for the Arrow Mobject:

If I were a newcomer to the documentation (which I am), this would be daunting. Half of these results don't necessarily have to do with the Arrow Mobject. Just coincidentally, the mobject happens to be at the top of the list, but that isn't always the case. If we were to implement something similar to discord.js's old documentation, I think that could help users navigate easier.

A low hanging fruit would be the replacement of the current quickstart guide with the contents from our interactive tutorial notebook, https://try.manim.community

When I was learning how to use manim, the biggest problems I had with the documentation were:
* The quickstart was far too basic for me as I already had programming experience

* It was very annoying to find a method because you had to basically look through the inheritance tree

  * This could also be confusing for first-time programmers who have never heard of inheritance before
    I feel like these should definitely be tackled.

so do you suggest a "hands on" approach? would be cool
in the link that i shared they actually put some exercises

so do you suggest a "hands on" approach? would be cool

Personally, I think that a quiz at the end of each section or something similar would be overkill. Most people would start manim with some goal for a video in mind, I would call that hands-on experience. The docs should only really be used as a reference in my view.

What is the state of the art of this PR?
Is there something that I could help? Do we have some point at which we agree that this part of the docs has to be created?


anti-CRISPR proteins get assigned the phrog cluster "acr" instead of a number

phold version: version 0.1.4
Python version: 3.10.14
Operating System: Linux

Description
I was attempting to run Phold on a GenBank file (gbk) to run Phynteny on it subsequently. However, the anti-CRISPR proteins were assigned "acr" as their phrog value instead of the phrog cluster they belong to.  The same thing happens to vfdb hits and I imagine card hits as well.
What I Did
phold run -i pharokka.gbk -o phold/sequence_1/ -t 6 -d /phold/20240322 -f

 CDS             complement(33955..34404)
                 /ID="BUHCUXVW_CDS_0014"
                 /transl_table=11
                 /phrog="acr"
                 /locus_tag="BUHCUXVW_CDS_0014"
                 /function="moron, auxiliary metabolic gene and host
                 takeover"
                 /product="anti-CRISPR protein"
                 /source="Pyrodigal-gv_0.3.1"
                 /translation="MAKFIGVKMIEVVPMTAREANDKGHRIGNHSFEEDGYEVTYPNGY
                 KSWSPAKEFEKAYYKLEDPAGDVLKENDIKRFIKGIENVKVGTKTTNTTLTCLTGFEVH
                 GQAACVKPENFDLNVGSNYAQIKAEDKIWEGLGFVLQWAKYGLKK*"

Hi @Fabian-Bastiaanssen ,
This is by design in Phold - we want to represent the fact that this protein is not hitting a PHROG as its top hit but rather something in acrid. Same as CARD and VFDB.
The issue is more on the Phynteny side as it will break as it currently stands for acrs (there is a fix for CARD and VFDB) - Susie is working on a big update at the moment, so it might be a while before she gets to this, but she knows about it - thanks for altering me.
George


Unable to write a jest test case rendered by shallow

Anyone has any solution how can i write a Jest test case to check below?

whether the row is selected or not and how to simulate a row

assume, designed a customFilter on one of the columns and check whether the filter is enabled on a column or not. And, how to simulate the filter text box so that should check the table values.

assume i have a delete button, how to write a test case around it?
getCustomFilter = filterHandler => {
return (
<SearchField                  key="searchField"                  id="searchField"                  ref='search'                  className='search-field'                  placeholder='Search'                  onChange={this.dataSelected.bind(this, filterHandler)}></SearchField>
);
};
dataSelected = (filterHandler, e) => {
filterHandler(e.target.value)
};

<BootstrapTable id="Table" data={DataSource}                                 hover striped bordered={ false }                                 withoutTabIndex>             <TableHeaderColumn isKey dataField="Name" dataSort                                filter={{ type: 'CustomFilter', getElement: this.getCustomFilter.bind(this) }}>             </TableHeaderColumn> </BootstrapTable>);

HI @jani937 sorry for lately reply
I know there're a lots of thing I need to improve so that I already decide to make a new repo to rebuild react-bootstrap-table: https://github.com/react-bootstrap-table/react-bootstrap-table2
Anyway, something is hard to test as I know, so not sure if following answer can help you

just check the checkbox directly or check the state of react-bootstrap-table, because the select row key will saved in the state as selectedRowKeys

If you have a column filter, I think your test case should focus on itself. if you just wanna check if the filter appear on the specified header, just check the DOMNode or something

Same as above, each custom component should have itself tests, even some callback function, you can still verify a callback whether being called. Use sinon is a good point to test it


Automated cherry pick of #3773: fix: 调整存储容量后刷新调度缓存

Cherry pick of #3773 on release/2.13.
#3773: fix: 调整存储容量后刷新调度缓存

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.13@c1e9886). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.13   #3774   +/-   ##
==============================================
  Coverage                ?   6.94%
==============================================
  Files                   ?     499
  Lines                   ?   84205
  Branches                ?       0
==============================================
  Hits                    ?    5850
  Misses                  ?   77859
  Partials                ?     496

Flag
Coverage Δ

#aFlag
6.94% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/storages.go
1.12% <0%> (ø)

Continue to review full report at Codecov.

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

/lgtm
/approve


Build failures with GCC 6

The code fails to build with modern g++-6/gcc-6 releases. I had to revert to gcc-5/g++5 to make progress.

@vanvugt Can you share your output?

I can't, sorry. It's been two months and I am no longer attempting to use MSDK. But it's a trivial problem description so no doubt someone will fix it, if not already.

Ok, but which version of gcc did you try exactly? 6.4? I want to try to reproduce your problem.

It was Ubuntu 17.10 at the time, so on 14 Jun I estimate that would have been 6.3 ...
https://launchpad.net/ubuntu/artful/+source/gcc-6

Thanks @vanvugt for your feedback.
Before I waited your answer I tried to reproduce your bug with fresh compiled from source gcc 6.4 - building successfully finished.
I will try to reproduce it with gcc version 6.3

Maybe this commit had something to do with this :)
https://github.com/Intel-Media-SDK/MediaSDK/pull/11

@ishmael1985 thanks for the comment.
About gcc 6.4:
I understood that I installed latest version of gcc not correct so cmake used my 4.8.5 version of gcc.
I will set up my environment correctly and then will try to reproduce your problem again.

@vanvugt We have finally reproduced your bug with gcc 6.
That was errors with deprecated auto_ptr. Isn`t it?
We will think about possible way of fixing it.

No hurry. We have decided we don't need MSDK support in Ubuntu any time soon.

We switched to gcc 6.3.1 in our CI. @onabiull I think we can close this issue if @vanvugt is ok with it :)

@vanvugt Thanks for your feedback!