'The application cannot be started because it is currently updating" - No WSL2 Commands Work at All

Windows Version
Microsoft Windows [Version 10.0.19044.3448]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
No response
Distro Version
Ubuntu 22.04
Other Software
No response
Repro Steps
Any time I run a WSL command in powershell, I am met with
The application cannot be started because it is currently updating.
There are no other obvious issues, and when I attempt to start WSL in another window with Terminal Preview I get
The application cannot be started because it is currently updating.

[process exited with code 1 (0x00000001)]
You can now close this terminal with Ctrl+D, or press Enter to restart.

There seems to be no way out of this, I have tried restarting my PC, killing WSL related tasks, etc. And I cannot really troubleshoot as all WSL commands in powershell return The application cannot be started because it is currently updating.
Expected Behavior
Ideally some WSL commands should work, such as version reporting, but The application cannot be started because it is currently updating. is all I get.
Actual Behavior
I get the following from WSL commands:
PS C:\Users\bebebib> wsl.exe --version
The application cannot be started because it is currently updating.

And starting a new window in Terminal Preview returns:
The application cannot be started because it is currently updating.

[process exited with code 1 (0x00000001)]
You can now close this terminal with Ctrl+D, or press Enter to restart.

There seems to be no other mention of people having a similar issue.
Diagnostic Logs
No response

Was able to solve it, noting this for others as it may be helpful, I went to Settings -> Apps -> Windows Subsystem for Linux -> Advanced Options -> Repair, after I did this restarting the WSL terminal seemed to work

Was able to solve it, noting this for others as it may be helpful, I went to Windows Settings -> Apps -> Windows Subsystem for Linux -> Advanced Options -> Repair, after I did this restarting the WSL terminal seemed to work


Controller lookup doesn't work for Piraeus deployments

The kubectl command kubectl get linstorcontrollers.linstor.linbit.com --all-namespaces doesn't work for our Piraeus deployment. However, if you use kubectl get linstorcontrollers --all-namespaces instead, it works perfectly.
Locally I've changed https://github.com/piraeusdatastore/kubectl-linstor/blob/master/kubectl-linstor.go#L124. After that the kubectl plugin works as expected :-)
Thank you @WanzenBug for another great and useful tool!

Thanks for pointing that out.
That's what you get for only testing in-house :sweat_smile:

Your PR was faster than mine ;-)
Thanks for the super fast response!


Share example in PS rclone config

Can you share exaple how you configure PS value for rclone config

Usually I create the config locally first with https://rclone.org/docs/#configure
rclone config create "source" ...
rclone config create "destination" ...

This creates a local file that looks like this
$ cat ~/.config/rclone/rclone.conf
[source]
...

[destination]
...

This content is the actual value stored in SSM

Either using the console or this one-liner
aws ssm put-parameter --region eu-west-3 --name rclone-config --type SecureString --value file://$HOME/.config/rclone/rclone.conf --description "Entire rclone.conf for rclone-lambda"

Let me know if this is enough information, I'll add these details in the README

Closing this now, feel free to re-open if more information is needed


Connection not work in node-oracledb

I run this docker in a Azure VM and now I want to access from my personal computer (Not from azure VM) with node-oracledb.
I replace this:
hostname: localhost
port: 49161
sid: xe
username: system
password: oracle

To this:
{
    "user":  "system",
    "password": "oracle",
    "connectString": "//myIP:49161/xe"
}

Replacing myIP to my Azure Virtual IP
But not work... Any suggestions? Thank you very much! 😊

Ok. My fault!!! I need to open the 49161 port in my VM... 😅 😅 😅


Allow overloading of JSX element constructors

Search Terms
jsx overload conditional component react
Suggestion
Allow for some kind of overload mechanism for JSX element construction, that is: determine contextual types for the allowable attributes for a JSX expression based on attributes already passed in.
Use Cases
There are times when we want to conditionally render based on the type of a certain property in a props object, or the existence/non-existence of a certain property in a props object. A very common use case is rendering clickable elements as anchor tags if a href attribute is present:
type AnchorProps = JSX.IntrinsicElements['a']
type ButtonProps = JSX.IntrinsicElements['button']
declare function Buttonish(props: ButtonProps): JSX.Element
declare function Buttonish(props: AnchorProps): JSX.Element
declare function Buttonish(props: AnchorProps | ButtonProps): JSX.Element

const myAnchor = <Buttonish href="typescriptlang.org" onClick={event => {}} />
// `event` is inferred to be React.MouseEvent<HTMLAnchorElement> because `href` is already there

The above is one suggestion though I don't know we could resolve that with class-based components.
I know right now that this is partially solved with generic components + conditional types, but it's kind of odd DX:
type EitherProps<T> = T extends string ? { href?: T } & AnchorProps : { href?: undefined } & ButtonProps
declare function Buttonish<T>(props: EitherProps<T>): JSX.Element

const myAnchor = <Buttonish href="typescriptlang.org">
// generic type inferred, but looks weird since T = string which doesn't tell you much

Examples
Unsure what the best approach would be. I think one good DX I was hoping for was for the autocomplete list to be narrowed as attributes are added to a JSX expression, when the compiler gets more contextual information to infer, so that after href is added in the above example, the event handlers are narrowed to just be handlers related to anchor elements, for example.
Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

So this actually works as requested in 3.2. While some things broke in the great JSX migration, this is something that was fixed by it.

Are you using 3.2?

I just ran into this. @ferdaber What needs to happen so we can support overloading class components?

Removing the ElementAttributesProperty interface declaration in the JSX namespace of React's type definitions should do the trick, like my above comment.

Is this something we need to fix upstream in the React types, or are you suggesting doing this by patching the types locally?

Either will work, but if you choose to patch it it does need to be a literal patch, since you can't remove declarations via declaration merging

Thinking about this some more, why do you need overloads in your class example? Isn't the union parameter sufficient? Related: https://www.typescriptlang.org/docs/handbook/declaration-files/do-s-and-don-ts.html#use-union-types

Union parameter types are subtly different, and it may be a bug honestly (unsure). Consider the following:
type AElement = HTMLAnchorElement
type BElement = HTMLButtonElement

interface A {
    a: 'a'
    onClick(el: AElement): void
}

interface B {
    onClick(el: BElement): void
}

declare function AorB(props: A | B): JSX.Element
declare function AorBOverload(props: A): JSX.Element
declare function AorBOverload(props: B): JSX.Element

// TS error: `el` is inferred to be `any` even though it should be narrowed to `AElement`
const foo = <AorB a='a' onClick={el => {}} />

// this works fine and `el` is inferred to be `AElement`
const bar = <AorBOverload a='a' onClick={el => {}} />

@ferdaber Interesting, thanks. Opened an issue: https://github.com/microsoft/TypeScript/issues/34590.


Support Oracle Database 12.1 (or later)

Description
Support Oracle Database 12.1 (or later).
How Has This Been Tested?
Run unit test with command: pytest tests/test_conn_oracle.py
docker run -d -p 1521:1521 -e ORACLE_PASSWORD=oracle gvenzl/oracle-xe:21
sqlplus system/oracle
Snapshots:
Include snapshots for easier review.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have already rebased the commits and make the commit message conform to the project standard.
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] Any dependent changes have been merged and published in downstream modules

@duke147 Thanks， but too many unrelated code updates, you need to rebase the main and resubmit.

Too many incompatible return type with base class
dbgpt/datasource/rdbms/conn_oracle.py:83: error: "Generator[str, None, None]" has no attribute "difference"  [attr-defined]
dbgpt/datasource/rdbms/conn_oracle.py:90: error: Incompatible types in assignment (expression has type "Generator[str, None, None]", variable has type "set[str]")  [assignment]
dbgpt/datasource/rdbms/conn_oracle.py:188: error: Return type "list[dict[Any, Any]]" of "_get_sample_rows" incompatible with return type "str" in supertype "RDBMSConnector"  [override]
dbgpt/datasource/rdbms/conn_oracle.py:199: error: Return type "list[dict[Any, Any]]" of "_get_table_indexes" incompatible with return type "str" in supertype "RDBMSConnector"  [override]
dbgpt/datasource/rdbms/conn_oracle.py:248: error: Incompatible return value type (got "Any | None", expected "str")  [return-value]
dbgpt/datasource/rdbms/conn_oracle.py:250: error: Return type "tuple[str, str]" of "get_collation" incompatible with return type "str | None" in supertype "BaseConnector"  [override]
dbgpt/datasource/rdbms/conn_oracle.py:263: error: Incompatible return value type (got "tuple[Any | None, Any | None]", expected "tuple[str, str]")  [return-value]

@csunny Please take a look and let me know if there's anything else I need to do.

@csunny Please take a look and let me know if there's anything else I need to do.

@duke147 docker hub cannot pull  gvenzl/oracle-xe:21, now we cannot test oracle, so we will install oracle manually soon.

you can try this : docker pull dockerproxy.com/gvenzl/oracle-xe:21

你可以试试这个：docker pull dockerproxy.com/gvenzl/oracle-xe:21

作者你好，有什么办法能支持 Oracle 11.2 版本呢？

@codermast ，Currently, the oracle docker image cannot be pulled down and tested.


Error when trying to install 9.0.5

I'm getting following error when I'm trying to update to 9.0.5
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Root composer.json requires symplify/easy-coding-standard 9.0.5 -> satisfiable by symplify/easy-coding-standard[9.0.5].
    - symplify/easy-coding-standard 9.0.5 requires symplify/coding-standard ^9.1 -> found symplify/coding-standard[dev-master, 9.1.x-dev (alias of dev-master)] but it does not match your minimum-stability.

And in packagagist I'm seeing that version ^9.1 is on requirements - should those be 9.0.5 ? https://packagist.org/packages/symplify/easy-coding-standard

Hi,
thank you for reporting and useful informations :+1:
That's definitely bug, not sure how that happened.
I've released 9.0.6 with correct mutual dependencies https://packagist.org/packages/symplify/easy-coding-standard#9.0.6

Thank you for quick answer, I tried 9.0.6 version but it seems that there is same error still in symplify/autowire-array-parameter package https://packagist.org/packages/symplify/autowire-array-parameter

There was a nasty bug in monorepo splitting itself. It was correctly tagged in monorepo, but on split there were 2 concurent jobs splitting the same tag. Dependending on CI job, sometimes the correct won, sometimes the broken one.
Now split should happen for tagging just once.
This should fix it: https://github.com/symplify/symplify/commit/f5c2517ea573b2c253df0ae858f95a0f93c676bb
Could you confirm? Last version is 9.0.8

hmm, not seeing 9.0.8 on - https://packagist.org/packages/symplify/easy-coding-standard#9.0.8

Job got stuck... should be ok now

And yeah, I confirm that 9.0.8 works like expected! Really big thanks to you @TomasVotruba great job!

Glad to hear that :)


[Bug]: ProcessOutput Releases w/o Line Breaks

Replication of the Problem
const { Listr } = require('listr2');

const tasks = new Listr({
  task: () => {
    console.log('one');
    console.log('two');
  },
})

tasks.run();
// Outputs "onetwo"

Problem
Hi there,
I'm trying to use Listr2@6 in my CLI, but when tasks have console output, line breaks are lost during release(). The output comes out all glued together in one line, which is difficult to read.
(Also, is there a way to opt out of output during release? I'd prefer to hide it entirely, but I couldn't see a related option in the docs / source.)
Let me know if you need further details!
Environment
MacOS Ventura 13.2.1
Node.js 16.20.0
iTerm2 3.4.19
<EMAIL_ADDRESS>Log Output
onetwo

Preflight

[X] Read/Search existing issues and the provided documentation.
[X] Be sure to not open an issue in the looking for contributions section.
[X] Checking the latest version of the software to confirm it has not been fixed already.

:tada: This issue has been resolved in version 6.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Please don't rush on my account. Feel better soon!

:tada: This issue has been resolved in version 6.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

6.2 resolved my issues! Streams are releasing with no extra blank lines :)
I'm using new ProcessOutput(null, null, { dump: [] }) rather than my custom subclass now. Thanks again!

No problems at all. I am glad it was a quick fix for you and thank you again for reporting the issue.
Have a very nice day!


Rename to rules_shell

Context https://github.com/bazelbuild/bazel/pull/23660

/cc @fmeum @comius @Wyverald @aherrmann

I think that it could be confusing if sh_binary lives in rules_shell, not rules_sh. But I guess renaming the rule itself to shell_binary (which would be more accurate, it even defaults to bash) would be even more annoying?

Nvm, I just noticed that rules_python and rules_py are precedent for this and everyone seems to be doing okay. I will send a PR.

Yeah, and we also have https://bazel.build/reference/be/shell


⚠ Bofrost.nl has degraded performance

In 3836c4e, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 2393 ms

Resolved: Bofrost.nl performance has improved in f213318.


Type "Promise<{}>" cannot be assigned to type "Promise".

UnoCSS version
lastest
Describe the bug
presetIcons({
        extraProperties: { display: 'inline-block', 'vertical-align': 'middle' },
        collections: {
          'material-symbols': () => import('@iconify-json/material-symbols/icons.json').then(i => i.default)
        }
      })

Type "() => Promise<{}>" cannot be assigned to type "(() => Awaitable) | CustomIconLoader | InlineCollection | undefined".
Can't type "() = > Promise < {} >" assigned to type "() = > Awaitable < IconifyJSON >".
Type "Promise<{}>" cannot be assigned to type "Awaitable".
Type "Promise<{}>" cannot be assigned to type "Promise".
Type "{}" is missing the following attributes in type "IconifyJSON" : prefix, icons
Reproduction

System Info
No response
Validations

[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

You can use an as any or ignore the error. This is due to that @iconify-json/material-symbols/icons.json doesn't have a type, but not UnoCSS's issue.


Remove deprecated check for server compatibility via the list of endpoints

This check has been forgotten for some time now and blocks us from cleaning up the /ping response.
The PR removes it altogether as it had no use now 🙃

Oh no, I added one line! I hoped this would be just about removing :/

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 79.122%

Totals

Change from base Build<PHONE_NUMBER>3:
0.06%

Covered Lines:
3244

Relevant Lines:
4100

💛  - Coveralls


Use taurusgroup/coreth fork

Jira
https://taurusgroup.atlassian.net/browse/DEV-25617
Goal of the PR
Use the taurusgroup fork of the coreth EVM plugin
Technical details

Please also link this to the correct Jira, to give some context/documentation why the fork was needed and how the dependency on coreth changed.

Done 👍


added the option to pass 'bundleCommand' to react.gradle

This will enable Android app to leverage the unbundle option of the packager. The default value will be bundle which is the default behaviour today.
Motivation
We use react native in a production application and we noticed (by using production monitoring) that about 2 seconds of our app bootstrap time is from MainActivity.onResume() to the first line of the js bundle.
Our bundle size is about 4.5MB and most of it is not needed in order to load the first view of the application. After some investigation we found that using unbundle as our bundling strategy gives way better results (bundle load time was reduced from ~2 seconds to ~1 second)
What existing problem does the pull request solve?
The standard way to create Android react native app is by gradle assmebleRelease that among the other tasks execute the bundleReleaesJsAndAssets task from react.gradle
This task in running the bundle command on the packager and does not expose an option to run unbundle. This PR adds a bundleCommand option (similar to other override options that available on react.gradle)
Test Plan
This test scenario was performed manually to verify that everything is working as expected:

create new react application with react-native init
run assembleRelease
verify on app/build/intermediates/assets/release that index.android.js file exists
run the app and verify the main page is loading
go to build.gradle and add:
project.ext.react = [  bundleCommand: "unbundle" ]
run assembleRelease again
verify on app/build/intermediates/assets/release that single index.android.js file exists and js-modules folder exists with multiple chunks
run the app and verify the main page is loading

By doing all these steps I managed to run the app in both scenarios

Is there anything I can do to accelerate the approval of this PR?


Make script fail for test purpose

Codecov Report

Merging #167 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master      #167   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          471       469    -2
=========================================
- Hits           471       469    -2

Impacted Files
Coverage Δ

django_property_filter/filters.py
100.00% <0.00%> (ø)

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 ab82769...2392a8c. Read the comment docs.


Add localization to watch app

Rename watch target
Clean up watch files
Add localization to watch
Add SwiftGen to generate string for watch
Bump build version

Codecov Report

Merging #119 into development will increase coverage by 0.32%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           development     #119      +/-   ##
===============================================
+ Coverage        70.03%   70.36%   +0.32%
===============================================
  Files               33       33
  Lines             1532     1532
===============================================
+ Hits              1073     1078       +5
+ Misses             459      454       -5

Impacted Files
Coverage Δ

MyWeight/Services/MassService.swift
51.11% <0%> (+3.7%)
:arrow_up:

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 18962c7...9e2d6b1. Read the comment docs.


balena-yocto-scripts: Update balena-yocto-scripts to v1.10.3

Change-type: patch
Changelog-entry: Update balena-yocto-scripts to v1.10.3
Signed-off-by: Kyle Harding<EMAIL_ADDRESS>

@resin-jenkins retest this please

@resin-jenkins retest this please

@resin-jenkins retest this please

@resin-jenkins retest this please

@balena-ci I self-certify!


add polymer transformer group

I did not add any explicit tests here, but it is effectively tested by the transformed tests themselves if that makes sense? I also wanted to avoid adding a code_transformers dependency. Let me know if you disagree :).

lgtm


Don't log embedding output from EmbeddingWrapper

Just remove the output for now. We can add a flag for it later if it turns out to be useful.

Make sure to update the typescript SDK as well


[feature] price tracker - alert on a new "lowest" or "highest" price value, get alerts when products reach a new low (or high) value

Price tracker - alert on a new "lowest" or "highest" price value, get alerts when products reach a new low (or high) value
So it would look in all history, find the lowest, if the new price is the lowest, send a trigger

To track and alert on new lowest or highest prices, the system will:

Price Extraction: Continuously monitor and extract the current price using XPath or other selectors.
Historical Comparison: Compare the new price with stored historical data to check if it's a new lowest or highest value.
Optimization: When a new lowest or highest price is found, trigger an alert and replace the stored value. Previous irrelevant prices will be deleted to save storage and computation time is also optmised.

This approach ensures efficient monitoring and storage by keeping only the most relevant price data while providing timely alerts.


Wunderlist component

Description:
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#7448
Example entry for configuration.yaml (if applicable):

wunderlist:
  client_id: YOUR_CLIENT_ID
  access_token: YOUR_ACCESS_TOKEN

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Yes, please open a new PR.

Think this PR is missing the new-platform tag


Added Kendriya vidyalaya domain for delhi region

the school official website URL : https://rodelhi.kvs.gov.in/
the school street address, including city and country : Delhi Region, India
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course : https://rodelhi.kvs.gov.in/en
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students : N/A

@rudraksh-git

a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students

Please provide us with a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students. You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

@rudraksh-git You might miss my previous comment. Please react.


Dell Latitude 5410 download link incorrect

Using version 6.4.9, unable to download Dell Latitude 5410 drivers due to the download url being incorrect with additional characters in the url. Latitude 5410 is also listed in the catalog twice.

Had the same issue with the Latitude 7410.  Might be a duplicate on Dell's side with their XML.  I didn't have enough time to look into this as this was a 1 off model for us.  Might be worth reaching out to dell in regards to their XML.

I tested the latest beta build (6.5.0) with the modified Dell XML fix and this sorted out the issue for me with the Latitude 7410 build - A03 was successfully packaged on my MECM console.  We don't normally use beta software in our environment, but as this model for us is our key model at the moment, i had to try to see if the issue was fixed with this update.


“Radio” Broadcasting Channel

This is a feature request which will add a type of broadcasting channel to HuskChat.  With this players will be able to select channels which will allow them to see broadcasts within a certain radius of blocks.

Any news on this?


Change of flash message for multiple added volunteers

If multiple volunteers were added for an event, there will be multiple flash messages whose numbers are tantamount to that of the volunteers.
This pull request ensures that there is a single flash message, regardless of the number of volunteers who were added. Hence, the user will be able to see all the names of the volunteers in a single flash message that will displayed at the top of the screen.

Changes volunteer.py file by using three lists that keep track of volunteers and triggering three conditional statements for this PR objective.
Resolves issue #1020

 Change the flash message when adding multiple volunteers to an event. #1020


fix(Tooltip): 支持mouseEnterDelay默认0.4s

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
What is the current behavior? (You can also link to an open issue here)
What is the new behavior (if this is a feature change)?
Does this PR introduce a breaking change?
Please check if the PR fulfills these requirements

[ ] Follow our contributing docs
[ ] Docs have been added/updated
[ ] Tests have been added; existing tests pass

Other information:

lgtm


Update KBM strings

Summary of the Pull Request
Updated some strings.
Hopefuly it will prevent confusion.

PR Checklist

[x] Closes: #17935
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Let's not forget #16093 though

Giving it a test, it looks like there are still some strings referring to the old columns:

Yep, missed one, thanks :)

Question: MS style guide says to use "select" when referring to pressing keys. We...use pretty much every word that's related. (press, type, select...)
Should we (or I) change more text to follow this?

Question: MS style guide says to use "select" when referring to pressing keys. We...use pretty much every word that's related. (press, type, select...) Should we (or I) change more text to follow this?

Do you mean everywhere or only KBM strings?

Do you mean everywhere or only KBM strings?

Probably both 😅

Any more input?

Can we get this in for 0.72 please?

Sorry, totally slipped. 0.72 is closed out already. But I'll up this in my priority review list. Thanks for the call out.


stage advancement

What all is required for declaring stage 1?
Anything more than bumping the number?

Yes, transfer the repo into https://github.com/tc39-transfer (an admin here, who's also a tc39 delegate or IE, can do it)

Maybe add a link to the process: https://tc39.es/process-document/


Allow to define service annotations to be used with external load balancers as alternative to ingress

Hello,
I am deploying uptrace on AWS EKS and providing external access using the AWS LB Controller which requires a set of service metadata.annotations to detect it and provision the LB.
It would be useful if the helm chart supported it.
Thanks.

Hey,
Would be glad to help, but I barely understand the half of what you just said :) If it is not too much trouble, consider sending a PR.

@alexandrujieanu thanks :+1: Could you please take a look at https://github.com/uptrace/helm-charts/pull/5? Any other hints we should add to the docs?


Accuracy not increasing with BERT Large model

I experimented with  BERT_base_cased and BERT_large_cased model for multi class text classification. With BERT_base_cased, I got satisfactory results. When I tried with BERT_large_cased model, the accuracy is same for all the epochs
Epoch: 01 | Epoch Time: 0m 57s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.200893470219204	2.790178544819355	4.977107011354887	3.6057692021131516
Epoch: 02 | Epoch Time: 0m 57s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.085730476038797	2.287946455180645	4.954357807452862	3.6057692021131516
Epoch: 03 | Epoch Time: 0m 58s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.019492668764932	2.901785634458065	4.961122549497164	3.6057692021131516
Epoch: 04 | Epoch Time: 0m 58s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.0052995937211175	3.57142873108387	4.9535566843473	3.6057692021131516
Epoch: 05 | Epoch Time: 0m 58s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.003523528575897	3.23660708963871	4.9652618261484	3.6057692021131516
Epoch: 06 | Epoch Time: 0m 58s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.010107040405273	3.29241082072258	4.96296108686007	3.6057692021131516
Epoch: 07 | Epoch Time: 0m 58s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.028377030576978	2.678571455180645	4.94510478239793	3.6057692021131516
Epoch: 08 | Epoch Time: 0m 58s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.04387321642467	2.901785634458065	4.9411917466383715	3.6057692021131516
Epoch: 09 | Epoch Time: 0m 58s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.027528064591544	3.18080373108387	4.940045246711144	3.6057692021131516
Epoch: 10 | Epoch Time: 0m 58s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.023407867976597	3.29241082072258	4.940378886002761	3.6057692021131516
Epoch: 11 | Epoch Time: 0m 58s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.015415557793209	3.125	4.939220135028545	3.6057692021131516
Epoch: 12 | Epoch Time: 0m 58s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.018008896282741	3.29241082072258	4.9386150653545675	3.6057692021131516
Epoch: 13 | Epoch Time: 0m 58s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.003824523517063	2.957589365541935	4.938107490539551	3.6057692021131516
Epoch: 14 | Epoch Time: 0m 58s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.003440124647958	3.069196455180645	4.93824944129357	3.6057692021131516
Epoch: 15 | Epoch Time: 0m 58s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.012082431997571	3.069196455180645	4.9383643590486965	3.6057692021131516
Epoch: 16 | Epoch Time: 0m 58s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.009286454745701	3.01339291036129	4.93832148038424	3.6057692021131516
Epoch: 17 | Epoch Time: 0m 58s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.006769972188132	2.901785634458065	4.937925778902494	3.6057692021131516
Epoch: 18 | Epoch Time: 0m 58s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.006464583533151	3.125	4.937762847313514	3.6057692021131516
Epoch: 19 | Epoch Time: 0m 58s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.004164610590253	2.957589365541935	4.937491783728967	3.6057692021131516
Epoch: 20 | Epoch Time: 0m 57s
*******train_loss,train_acc,valid_loss,valid_ac**********
5.013612789767129	2.957589365541935	4.937890896430383	3.6057692021131516
Epoch: 21 | Epoch Time: 0m 58s
*******train_loss,train_acc,valid_loss,valid_ac**********
4.997398240225656	2.511160634458065	4.937900726611797	3.6057692021131516

With BERT_base_cased, there is no such problem. But with BERT_large_cased, why accuracy is same in all the epochs? Any help is really appreciated............. @thomwolf  @nreimers

Hi, I experienced this also in several experiments. BERT large is extremely sensitive to the random seed. Try some other seeds and you will likely get a performance at least on oar with the base model.
I haven't it studied further why the large model is so sensitive to the random seed, but it appears that the gradient for some step destroys the model, from which on you only get bad scores. Might be some exploding gradient or some Nan issues.

I find this issue as well.


Question: How can I view files inside cjs folder? Thank you

https://github.com/ReactTraining/react-router/blob/fc91700e08df8147bd2bb1be19a299cbb14dbcaa/packages/react-router-dom/index.js#L6

This folder is created when bundling the package for release. You can see it in your node_modules or browse it on unpkg.


Janitor gameplay is still a slog and very frustrating, more so during ventclog events

Description
With more and more messes getting added into the game; puddle rework, pills now breaking into liquid, the janitor gameplay loop has become very frustrating to play, even more so as late game janitor. despite many merges added to help elevate the issue, I feel like it has only gotten worse. I get that as the round goes on it gets more chaotic and thus more work for the janitor, cleaning should almost always feel like an uphill battle. Even with a full janitor team, rounds can be hell, more so if you are a single janitor. we don't really have the equipment or any relevant upgrades to help counterbalance said chaos, aside from the Advanced mop. After playing a few janitor rounds here is what I'm most frustrated with, and my proposed solutions that I see can be in the right direction of making the job a bit more rewarding to play as.
Problems

The drain still drains way too slowly. Especially when more than one janitor is in there. It can take minutes trying to empty your bucket by yourself, even longer if there is another janitor in there doing the same thing.
Vent clog events with space lube are a NIGHTMARE! One vent spewing out lube literally takes 2-3 trips because your bucket gets full 30% of the way into cleaning it. Compound this with the drain draining slowly and the amount of vents that roll space lube you can easily see yourself doing nothing but cleaning space lube spills for the rest of the round and still not get it all.
Advanced mops are not worth it. For major spills, witch in the late game the majority of the spills are major, it is quicker to bring a mop bucket with you because the advanced mop takes too long to replenish water and purge other liquids. For a late game item, it feels backwards to ignore it and use regular mops and buckets because they are more efficient and time saving than the late game item. There have been times where I click on a single tile in the middle of a major spill, only to be forced to wait ~15 seconds for the mob to refill/purge. Imagine this but with 20 tiles of lube.
This is more of a mapping issue, but a lot of janitor closets in maps are not stocked properly. Either they are missing water tanks and buckets. Or they only have one regular water tank witch gets used up in the first 10 minutes of the round.

Solutions

Either add more vents to the janitor room, increase their capacity, or have them drain faster. (Perhaps an upgrade?) Alternatively, allow the janitor to again dump out their bucket(perhaps a long do after or in small units to avoid grief in the hallways) in the closet on top of the drain so they can get back to the action as the drain slowly does its job. Bonus points for this because the janitor closet will become a literal quagmire of chemicals, seeping under the airlock into the hallway, slowly getting drained out by the drain.
Allow the mop buckets to hold more liquid. The bucket can only hold 600 units i believe. One vent during the event can pump out 1000+ units of space lube. This will require many trips to clean. (Perhaps allow buckets to be upgraded? Bluespace mop buckets anyone?) Or reduce the amount of lube each event creates.
Advanced mops should get their water generation and liquid purge speed increased by at least 50%
Normalize all janitor closets(maybe with the exception of Detroit) to always have buckets, the same janitor vendors, light replacers/space lights, and one or more high-capacity water tank or water source.

Reproduction
Play janitor on extended.

This should have some discussion and testing but also this is fairly easy to change.

The only other thing remaining for janitor atm is prototypes for cleanfoam nades but blood + space lube seem to be common complaints due to how common they are.

blood is ok since you usally dont have several massive puddles of it soaking the station + it has no direct downside like lube which is denial of service

I think if you want to nerf vent clog the actual thing to do first is make smoke / foam not clone solutions which will reduce the volume significantly.

i think the main issue is not just volume but also the ridiculous spread
i will be nerfing both of these since spreading solutions exponentially is too hard for me idk

Closing as a majority of QOL updates went out since this was created. Janitor feels much better to play.


Heads-up: Projects (classic) will be sunset on August 23, 2024. Only cards that have been updated in the last year will be migrated.

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
Just a heads-up as I just noticed this warning on the v6 project and specifically this "Only cards that have been updated in the last year will be migrated."
So you probably want to manually migrate this to not lose anything, as I believe there are cards that are not updated from 2022.
Reduced test cases
https://codepen.io/team/bootstrap/pen/qBamdLj - will this make the bot shut up?
What operating system(s) are you seeing the problem on?
Linux
What browser(s) are you seeing the problem on?
Firefox
What version of Bootstrap are you using?
v6

Thanks for the heads-up @nextgenthemes
Appreciate it, :) the v6 project has been migrated and is available at https://github.com/orgs/twbs/projects/38

Sent from my Verizon, Samsung Galaxy smartphone
-------- الرسالة الأصلية --------من: nextgenthemes @.> التاريخ: 10/8/2024  6:45 م  (GMT+03:00) إلى: twbs/bootstrap @.> نسخة: Subscribed @.***> الموضوع: [twbs/bootstrap] Heads-up: Projects (classic) will be sunset on August 23, 2024. Only cards that have been updated in the last year will be migrated. (Issue #40713)
Prerequisites
I have searched for duplicate or closed issues
I have validated any HTML to avoid common problems
I have read the contributing guidelines
Describe the issue
Just a heads-up as I just noticed this warning on the v6 project and specifically this "Only cards that have been updated in the last year will be migrated."
So you probably want to manually migrate this to not lose anything, as I believe there are cards that are not updated from 2022.
Reduced test cases
https://codepen.io/team/bootstrap/pen/qBamdLj - will this make the bot shut up?
What operating system(s) are you seeing the problem on?
Linux
What browser(s) are you seeing the problem on?
Firefox
What version of Bootstrap are you using?
v6
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.>
[
{
@.": "http://schema.org",
@.": "EmailMessage",
"potentialAction": {
@.": "ViewAction",
"target": "https://github.com/twbs/bootstrap/issues/40713",
"url": "https://github.com/twbs/bootstrap/issues/40713",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
@.***": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]


Any tips on how to physically connect to suggested beacons?

(This is not an issue with the code, just asking since you might know the answer)
Hey,
Thanks for this work, seems very cool!
I'm just getting into BLE as a hobby and don't have much experience with hardware, so this is probably a silly question, but what tools do I need to program the hardware you suggested such as:
"AliExpress beacon" which can be purchased here https://www.aliexpress.com/item/32826502025.html
I have to solder the power supply and data pins for programming, right?
I have access to a soldering station, but I think that this might be too small to solder, no?
I have a similar beacon by Raytac:
https://www.amazon.com/dp/B081GYNM2G?ref_=cm_sw_r_cp_ud_dp_RFBYEBARPW5J6KK9CCJV
But its too small for me to solder and seems like it requires some more professional gear.
Thanks for the help!
Edan

Hi Edan!
Yes, soldering may be tricky. Specially on that module since the pads are very close together. I used a very similar module and I don’t even solder to program it!
However after that I had to connect a CR2032 battery so I had to solder the VCC pad and for the GND I used the metal on the cover. You should use a very tiny lead soldering wire and a small tip and if I could do it I’m sure you will too ;)

Thanks! Will share what I end up with :)

Thanks for the tips 😊  Just made an order for "AliExpress beacons". I've only flashed microcontrollers with usb interface before. Can you recommend me a low cost programming adapter from Aliexpress that I can use to flash the beacon?

Thanks for the tips 😊 Just made an order for "AliExpress beacons". I've only flashed microcontrollers with usb interface before. Can you recommend me a low cost programming adapter from Aliexpress that I can use to flash the beacon?

I used that one(is not available anymore but just search for "st link v2"):
https://www.amazon.de/gp/product/B086TWZNMM
how to connect: https://gist.github.com/mowtschan/d1956b4634a84698ab3a96472f710610

hey @mowtschan thanks for the gist. Very helpful. I don't understand base64 -D <<< your_base64_key_from_openhaystack_app | xxd -p | cut -c13-18 what is this used for. Can you please clarify?

@vekexasia , with the bluesee app (or similar) you see a lot of devices but you don't know which one is yours.
I used that command to extract some part of the key to be able to track/recognise my device with the bluesee app.


Update backlight file

The file to change display's backlight on paper pro was incorrect.
Also, the presets' percentages are not totally accurate as the scale is non-linear but it doesn't seem important enough to change.

cat /sys/class/backlight/rm_frontlight/scale returns non-linear
cat /sys/class/backlight/rm_frontlight/linear_mapping returns no

extra note:
cat /sys/class/backlight/rm_frontlight/device/name returns aw99703 and a datasheet  I found online says it's an lcd backlight controller with 11bit dimming (0-2047) and linear or exponential control modes so it might be using the latter

Feel free to add more comments to #74 or open another PR to add more information to the documentation. Even if it's just extra external links with more information.


chore: rename meaure to value recorder

Fixes #1112

just a quick question on a first pass, was there any documentation or examples that needs to be updated as well ?

I'm sure there is, but this touches enough places that I am going to do it as separate PRs so it isn't overwhelming to review.
@mayurkale22 that ok?

I'm sure there is, but this touches enough places that I am going to do it as separate PRs so it isn't overwhelming to review.
@mayurkale22 that ok?

https://github.com/open-telemetry/opentelemetry-js/tree/master/packages/opentelemetry-metrics#measure I think this is the only place we have a reference for Measure. I would prefer to handle that in this PR.

done

@dyladan Looks like this is good to merge now.


fix(runloop) init_worker not called on plugins that were not configured

Summary
Fixes issue with plugins_iterator not executing init_worker when starting Kong where the plugin was not already configured (in a database). This issue was introduced in 1.2.0rc1.
The PR also does some style changes and refactorings like:

localizing global variable access on plugins iterator
changed the order of phase and ctx arguments given to iterator on first call
loads the needed structures for iterator earlier (on iterator construction)
does not use mock_ctx anymore -> certificate passes ngx.ctx (which current version of OpenResty rewrite phase clears, but it still gives a table and future proofs that code path) and init_worker passes nil (as the ngx.ctx is not used for anything on init_worker with plugins iterator)
finally it adds tests that check that init_worker is called when pre-configured and runtime-configured

Issues resolved
Fix #4665

@hishamhm, take a look and comment, otherwise I will merge it on friday.

Thanks @bungle! The fix() commit was cherry-picked in release/1.2.0, we will merge the rest in next after rc2!


add GitHub test action, with cross-env CI update

resolves #17 - CI testing
with some rationale notes added to package.json

seem to be based on: https://github.com/actions/starter-workflows/blob/master/ci/node.js.yml


Splash Screen + Continuum = image resolution badness

When hooking a device up to a large monitor (24") via Continuum, the Splash Screen's image is still using the -100 version, which, when stretched to fill the launch window, pixelates very badly.
It seems that either

we need to check window sizes and make smart decisions to force -200, -400 images (if they exist) or
the OS needs an update to make this work better

to get devs up & running today i'm inclined to go for the first option. I'm not sure how the Office Apps are doing this so well, but it would be great to get some insight there if you can, @JerryNixon

Continuum is a big issue we'll have to start testing.

Agreed. However it's good news that, for the most part, things seem to be working in my usage of Template10 (albeit heavily customized from early versions)

I am not sure but that Template 10 cannot change the behavior you are seeing. Because the correct resource item should automatically be selected. But a phone has very high resolution, which means the issue should not be that the scaled image is not present, I think. The reason it is picking the wrong image seems, wrong. Do you think T10 could / should fix it?

This seems like it might be fixed in Mobile build 10586.29. I'd like confirmation from others before closing, though.

From my MS contact:
we’ve now heard from the engineering team on this and they report that the expected behavior should be as follows:

1.       Before connecting to an external screen, only assets for the phone will actually be on the device – we only download necessary assets.
2.       User connects phone to external screen.
3.       The connection triggers app updates for any apps with available assets better suited for that screen.  The update takes place in the next ~24 hours.

So appears that after utilizing Continuum, we shouldn't rush to any judgement for 24hr after we've loaded the app on a target device.

If this use case is important to you, I think you would be wise to ship a higher-quality splash image. Please remember, you can ship both the high and the low and the system will pick between them. You don't have to ship just one.

Yup the part that was alarming was that, although I am shipping all resolutions of splash screen imagery, it wasn't being picked up. We know now that this is because the app doesn't download that imagery w/ initial install on a mobile device. Instead it waits for a Continuum use of the app, then downloads those assets w/in the next 24 hours.
I don't like this (and have replied back to the MS rep) because the first impression of Continuum is then shoddy and not what a developer would ideally want his/her users to experience. Here's to hoping it's resolved in the future (eg: download high res assets all the time for Continuum capable devices)


==> ERROR: Missing 5.15.12-artix1-1 kernel modules tree for module evdi/1.10.0.

DKMS install...
Creating symlink /var/lib/dkms/evdi/1.10.0/source -> /usr/src/evdi-1.10.0
Building module:
cleaning build area...
make -j4 KERNELRELEASE=5.16.0-zen1-1-zen all INCLUDEDIR=/lib/modules/5.16.0-zen1-1-zen/build/include KVERSION=5.16.0-zen1-1-zen DKMS_BUILD=1....
cleaning build area...
evdi.ko.zst:
Running module version sanity check.

Original module

No original module exists within this kernel

Installation

Installing to /usr/lib/modules/5.16.0-zen1-1-zen/kernel/drivers/gpu/drm/evdi/
depmod...
:: Running post-transaction hooks...
(1/1) Install DKMS modules
==> dkms install --no-depmod evdi/1.10.0 -k 5.16.0-zen1-1-zen
==> depmod 5.16.0-zen1-1-zen
==> ERROR: Missing 5.15.12-artix1-1 kernel modules tree for module evdi/1.10.0.

Hi, please make sure you have the linux-zen-headers pacman package installed. I haven't tried the linux-zen myself, but I'll give it a shot on my artix.

they are installed and if you need this informations my init is runit and i installed evdi via paru

Sure, looks like our installer scripts aren't working with runit at all. I've fixed it, but it will be out in the next release. I'll take a look at compiling with zen now.

Thanks


freecodecamp using googleapis CDN which is blocked in China

@naibaben commented on Tue Jun 05 2018

Describe your problem and - if possible - how to reproduce it
Google Jquery is not accessible for China
https://ajax.googleapis.com/ajax/libs/jquery/3.3.1/jquery.min.js
Add a Link to the page with the problem
Tell us about your browser and operating system

Browser Name: Google chrome
Browser Version: 66.0.3359.181
Operating System: Windows 10

If possible, add a screenshot here

@QuincyLarson commented on Tue Jun 05 2018
Thanks for the feedback. We could refactor this to use a different font. Can you still pass the challenge without getting the CDN link to work?

@MartinKolarik commented on Tue Jun 05 2018
@QuincyLarson the issue is about jquery, not fonts.

@AbdulrahmanEng commented on Wed Jun 06 2018
This article might help: https://weblogs.asp.net/jongalloway/using-cdn-hosted-jquery-with-a-local-fall-back-copy. @QuincyLarson.

@AbdulrahmanEng commented on Wed Jun 06 2018
@naibaben, have you tried using the site through a VPN (virtual private network)?

@MartinKolarik commented on Wed Jun 06 2018
I'd suggest using a China-friendly CDN like jsDelivr instead. Using a local copy would be very slow for people in China and is unnecessary complicated.

@raisedadead commented on Wed Jun 06 2018
We can alternatively bundle jQuery

We should switch to this for the jQuery CDN or alternatively bundle it.
<script<EMAIL_ADDRESS>integrity="sha256-FgpCb/KJQlLNfOu91ta32o/NMZxltwRo8QtmkMRdAu8=" crossorigin="anonymous"></script>

CDN please, not bundle. The bundle is big enough already


Next SSR deploy fails with module not found

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
d3rmzs0e8of5du
Region
eu-west-1
Amplify Console feature
Build settings
Describe the bug
NextJS SSR deploy fails with: Error: Cannot find module 'sass'
It looks like it doesn't use the right version of Next?
I have setup the build overrides.

2021-08-08T09:23:07.381Z [INFO]: # Ensuring NPM package '@aws-amplify/cli' is version: 'latest' 2021-08-08T09:23:12.464Z [INFO]: # Patching NPM package '@aws-amplify/cli' from 4.29.4 to 5.3.0...
I see it patching Amplify in the log. But I don't see such a mention for next.
Expected behavior
SSR build being able to deploy succesfully
Reproduction steps
Deploy a version of NextJS 11 with SSR
Build Settings
version: 1
frontend:
  phases:
    preBuild:
      commands:
        - npm ci
    build:
      commands:
        - npm run build
  artifacts:
    baseDirectory: .next
    files:
      - '**/*'
  cache:
    paths:
      - node_modules/**/*

Additional information
No response

This is a duplicate of https://github.com/aws-amplify/amplify-console/issues/2098 . Please leave a comment on that issue with your setup.


update: ack-ecr-controller to v1.0.4

Issue #, if available:
Description of changes:
Update ack-ecr-controller to v1.0.4
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

We could implement similar automation as eksworkshop to auto create PRs for helm chart bumps
https://github.com/aws-samples/eks-workshop-v2/blob/main/hack/update-helm-versions.sh

Great idea. I'll review and see how we can apply that!


Ensure Discogs API terms of service are met for the demo

Translating action items from https://wiki.lyrasis.org/display/LD4P3/2020-09-25+Cornell+LD4P3+Meeting+notes:
Make sure the live demo meets the Discogs API terms of service in terms of linking and back.
Note: We already do include a link to the main Discogs but would need to add the disclaimer they request.
https://support.discogs.com/hc/en-us/articles/360009334593-API-Terms-of-Use

I don't think the notice is required for the demo. The terms state that the terms are required if the application is "public-facing." We're not demoing a live application. Also, the notice doesn't have to be on the item view page (though the link does). It just needs to be somewhere on the application.


completions hanging

fixed bug when iterating with empty BoundedWeightedStateTraverser was not ending
added appropriate unit test

LGTM

@narekgharibyan
bump the version to release 0.2.2?

Yup!
I'll do that once Travis is done, and will release it.

Coverage increased (+0.007%) to 88.222% when pulling 7186a5a7b3868e0012a15c0d833807c573564e24 on narekgharibyan:completions_hanging into c43fb325e183f71583261b3f8957b6435f0351c2 on cliqz-oss:master.


Java pubsub authenticated push example

Add java pubsub authenticated push example

Someone will review once Kokoro passes.

closing this in favor of #1404


resize_rendered_image_dimensions does not seem to work (properly)

What steps will reproduce the problem?
1. Enable the resize_rendered_image_dimensions filter
2. Visit a page with images that should be resized a few times
3. Check the actual image dimensions

What is the expected output? What do you see instead?
There are multiple examples of this filter not behaving as expected on my page.
One example is the image on the URL
https://slave.chainels.com/image/22x7x111340925320370007_91384881835938696.png.p
agespeed.ic.OZ7trmuR4w.png
As even can be seen in the URL's path (apparently the detection works), this
image is rendered 27x7. However, the cached image is still in its original
dimensions (64x19 in this case).

What version of the product are you using (please check X-Mod-Pagespeed
header)?
<IP_ADDRESS>-3609

On what operating system?
CentOS 6.5 x86

Which version of Apache?
2.2.15

URL of broken page:
https://www.chainels.com/111340925320370007/?p=about
Note: load balanced between 2 servers running exactly the same configuration

PS:
Is it possible to run the code that checks the rendered image dimensions
manually, e.g. after ajax navigation actions? It only seems to update/work on
first-access to a page right now.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 4 Dec 2013 at 10:59

Hello, I'm facing the same issue on nginx 1.63 with last page speed module <IP_ADDRESS>, any new on this?

We too experience this issue on Apache/2.2.29, mod Pagespeed <IP_ADDRESS>
Please let me know if you need anything else.

Hi guys,
Is there any update or a recommended workaround for this issue?
Thanks,
Mic.


[QT-Wallet] Display amount when ALL is selected for send

Brent has made a suggestion for something we can improve on in the wallet. "If someone wants to send the full amount in an account, and they paste or type in the number value manually they get the error (transaction couldn’t process because there wasn’t enough for the fee).  The reason someone might do that instead of clicking the ‘all’ button is because it just puts the word ‘all’ in the field, so you don’t really know if it’s sending ‘all’ from the account, or ‘all’ from ALL accounts.
The fix, which I think is standard on most wallets, would be to have it so when you click ‘all’ it puts in the mwc value, not the word ‘all’. Also, I’m hoping it would take the fee out of the account instead of failing? That part probably already works."
Now that we calculate the transaction fee, we can put in a check to see if someone is trying to spend all of their spendable coin, subtract the fee, and allow this type of send to proceed. What do you think?
chris-devToday at 12:03 PM
I like the idea of putting the value in when you click All instead of putting "ALL".
and since we have fee calculation, we can do that.
scasalettoToday at 12:40 PM
It seems like we should update the Send screen with the amount and also display the fee. Otherwise people might get confused about why the amount is less than their entire spendable coin.

chris-devToday at 12:54 PM
I don’t think we should change the amount entered.
I just think clicking all should display actual amount.
We want people to affirmatively enter the amount they want to send.
scasalettoToday at 12:59 PM
OK, but then on the confirmation screen, they would see the updated amount and the fee?
chris-devToday at 12:59 PM
Amount should not be updated
If you enter 5 it should remain 5
If you don’t have 5 plus fee it should be an error
I think what we have to today is good but when you click ALL it should show the amount minus fee
So if you have 5, click ALL it should show 4.998 plus fee (or whatever the correct value is)
scasalettoToday at 1:08 PM
Ok I think we are in agreement. We are only talking about if the user selects All. So when you click All, the Amount field should get updated with an amount that is equal to your current spendable coins minus the transaction fee. But we will not display the transaction fee to them on the Send screen. Correct? They will only see why there is a difference from what they see as an "Available" amount and what we have just populated the Amount with, when they hit Next, enter the Sent To address, and then hit Send.
chris-devToday at 1:11 PM
Yes, ideally we would show them the fee sooner too, but that can come later.

Clicking All on the Send screen will now fill the Amount field with the number of spendable coins minus the transaction fee.
Tested the following scenarios:
HODL Account:
All outputs locked except for 1 HODL output (0.492)
Sent unlocked HODL output (txn fee: 0.004)
Correct amount displayed (0.488)
Send goes thru successfully
Non-HODL Account With Locked Enabled:
1 output unlocked (1.00)
Send unlocked output (txn fee: 0.004)
Correct amount displayed (0.996)
Send goes thru successfully
Non-HODL Account With Locking Disabled:
3 outputs unlocked (total 0.282)
Sent all unlocked outputs (txn fee: 0.002)
Correct amount  displayed (0.28)
Send goes thru successfully

Verified with the latest build, hence closing this ticket.


Update Node js

SteamDB app page links to a few games using this
https://steamdb.info/app/793490/
Brief explanation of the change
Find the Node JS libraries for mac OS and Linux.

also, remove the unrelated commits from your pr

also, remove the unrelated commits from your pr

How?

After running changed rules in this pull request (this bot only tests rules.ini changes):
Changed technologies: SDK.NodeJS
5354 apps still match.
These apps now match: (26)<PHONE_NUMBER>40<PHONE_NUMBER>00<PHONE_NUMBER>90<PHONE_NUMBER>50<PHONE_NUMBER>90<PHONE_NUMBER>90<PHONE_NUMBER>90 1008020 1161330 1184330 1185340 1803530 1823490 1884070 2177610 2211180 2302100 2491950 3011320
Please confirm that all these changed apps are correct.

merged in c4fa73f4a55a74d538b2a8fbaf905ee2989c85d1


Implemented initial environment configuration from process environment

Additionally:

Updated some packages to address audit failures
Updated the license in package.json from Unlicense to Apache 2.0. Unsure whether Unlicense was the intended license here- it contradicts the license in the source files.

It's not subject to API authentication. So I think it should be possible to configure the service to create this env and have OAuth enabled. This is the first work I've done here so I can't answer the question with total confidence however.

It's not subject to API authentication. So I think it should be possible to configure the service to create this env and have OAuth enabled. This is the first work I've done here so I can't answer the question with total confidence however.


Not an Issue

Hey there!
I know Thanksgiving is still two months away, but it's never too early to give thanks where it's due! 🎉 Just wanted to say a huge thank you for this super useful script. It’s been a lifesaver, and I'm grateful in this very day - not just in November! 🦃🍂
Keep up the awesome work!

❤ Thanks! I really appreciate it!


Invalid JSON issue

For now runner.py breaks when it tries to retrieve the JSON created by AnsibleModule on any YAML playbook.
runner.py, line 25:
module = AnsibleModule(argument_spec=arguments, supports_check_mode=True)
Error message:
fatal: [localhost]: FAILED! => {"changed": false, "failed": true, "msg": "Error: Module unable to decode valid JSON on stdin.  Unable to figure out what parameters were passed"}

the key is to have
ANSIBLE_METADATA = {'metadata_version': '0.1',
                    'status': ['unstableinterface'],
                    'supported_by': 'students'}
from ansible.module_utils.basic import AnsibleModule

in module file, even if you are not using/calling AnsibleModule
seems that by this import core ansible interpreting it as ANSIBALLZMODULE, not as old and unsupported JSON module.


Adding partner section in community page

adding partner section with 5 new partners
adding steps in editdocs to enable partners follow and send requests themselves,

Created the PR by mistake


BUG: Fix minor baseline bug [skip azp] [skip actions]

@drammock I merged but did not do [circle deploy] since I think you're about to cut 1.1 (and push to maint/1.1 with a [circle deploy]), then bump this to 1.2.0dev0 and then do a [circle deploy] on that bump commit (or something similar)


Generating a user reply failed

TypeError: Cannot read properties of null (reading 'name')
    at doBotReplyInPlaceOfUser ((index):7082:48)
    at async HTMLButtonElement.sendButtonClickHandler ((index):5678:15)

Ah, thanks! This should be fixed now. Please let me know if not.


Output Format Problem

Yes i have run the code and it produce the output in Output folder with file extension .raw. How I open this raw format and save it back into .obj or .ply format.  Can any body help me in this regards? is there is any special software or have some function in the code to write into .obj format..
Thanks

There is a raw2obj script which you can use. Just pass it the image from the examples/scaled/ folder and the raw file.


selection behavior when" multi-selection" and no "selection-enabled"

it would be consistent with current "multi-selection" behavior if no "selection-enabled" that click on checkbox select the row like click on top/column checkbox select all/none rows. as side effect, this way one can focus rows without select them without "selecting-item" workaround.
IMHO normal, in all other grids i know, focus row/cell does not select row. the selection should be distinct act like clicking on checkbox or press key and such things. but practically when above would be implemented one would have the this behavior.

Hi!
Good point, haven't actually noticed before this inconsistent behaviour. Thanks!


config: Disable report_todo by default

I don't even think the feature belongs in rustfmt, but at the very least it should be off by default.

The motivation for this feature is parity with make tidy. Apart from that, I think it is good engineering practice to have an enforced separation of TODO and FIXME, the former meant to be temporary while a patch is written, the latter is allowed to land. This is a check on landing incomplete code.
I'm sympathetic to the argument that this is not really formatting, as such and so shouldn't be on by default, but I can't think of a better place for it to live and I don't believe there is more motivation for turning it off rather than leaving it on.
Do you have a use case for why it should be off by default?
I'm curious what others think about this too.

Why it should be off by default: TODO is a convention used in many ways by many projects. Forcing an interpretation* is not the place of a formatting tool. Some examples of projects / organizations with different interpretations from the one in rustfmt:

Google C++ style guide: ‘Use TODO comments for code that is temporary, a short-term solution, or good-enough but not perfect.’
Cap'n Proto style guide: ‘release scripts can error out if release-blocking TODOs are present.’

Why it shouldn't be in rustfmt:: rustfmt is a formatting tool: it takes code in, and spits out better formatted code that abides by conventions. Reporting style deviations belongs in a lint tool.

yes, a formatting tool is all about enforcing conventions. However, these are syntactic rather than semantic; enforcing an interpretation of TODO leans more towards semantic. :-)

Also, rustfmt currently gives 18 such warnings on its own code base!

I agree rustfmt is not the ideal place to have this kind of lints. Maybe it is better to have that is clippy ?

clippy requires nightly which makes it a more difficult tool to recommend. I think a separate tool would make sense. Not sure if there is an existing rustlint but it would make sense I think. In the meantime, I'd just like this disabled :-)

ping :-)

I think rustfmt probably should be the tool for doing this, but I'm ok with it not being a default.

@nrc, rereading Design.md, and I saw this in there:

My long term goal is that all style lints can be moved from the compiler to
rustfmt and, as well as warning, can either fix problems or emit refactoring
scripts to do so.

so apologies for ignoring that. I wonder if the right approach is a family of tools that share a lot of implementation?

Yeah, I have been considering a less monolithic vision recently. I'm not exactly how it should work  and how to make it ergonomic though.


Better functionality for partitioned tensor networks/graphs

For example, something we want to do is be able to flatten a doubly-partitioned tensor network into a singly-partitioned tensor network:
julia> tn = randomITensorNetwork(named_grid((4, 4)); link_space=2);

julia> tnp = partition(tn; npartitions=2);

julia> tnpp = partition(tnp; npartitions=2);

julia> tnp2 = DataGraph(underlying_graph(tnpp), map(tnp -> reduce(⊗, vertex_data(tnp)), vertex_data(tnpp)));

julia> tnp2[1] == tnp[1]
true

julia> tnp2[2] == tnp[2]
true

It would be nice to define a function that flattens a partitioned ITensorNetwork:
ITensorNetwork(partitioned_tn::DataFraph{<:Any,ITensorNetwork}) = reduce(⊗, vertex_data(partitioned_tn))

and then make sure map_vertex_data works for this case (which currently errors if the vertex data type changes), so we can do:
map_vertex_data(ITensorNetwork, tnpp)

Other ideas would be to define a PartitionedITensorNetwork so we can dispatch on that type:
const PartitionedITensorNetwork{V} = DataGraph{V,<:ITensorNetwork}

and distinguish between singly-partitioned vs. more nested partitionings. Having a function that flattens nested partitioned network into a single ITensorNetwork would be helpful as well.
A related function we have for dealing with nested partitioned tensor networks is nested_graph_leaf_vertices, which works like this:
vertex_groups = nested_graph_leaf_vertices(tnpp)

for finding groups of vertices but maybe the API should be rethought since it's a confusing name.

Another idea would be to add a way to call partition and specify that you want to make sure that certain vertices are grouped together. An interface could be:
julia> tn = randomITensorNetwork(named_grid((4, 4)); link_space=2);

julia> tnp = partition(tn; npartitions=2, group_vertices=[[(1, 1), (2, 1)], [(3, 4), (4, 4)]]);

to make sure that vertices (1, 1), (2, 1) are grouped together and [(3, 4), (4, 4)] are grouped together. This could be implemented by first partitioning the graph according to the grouping (treating unspecified vertices as groups of 1), partitioning that partitioned graph, and then regrouping based on the functionality in the first post of this issue.

Closed by #126.


REVAL support?

Are there any plans to support legacy disturbance formats such as ABB REVAL from Relion 5XX series?

Hi @slemadin I think this repo/module should stay as it is - supporting standardized comtrade files. In the future we can add support for non standard comtrade, but an entirely different format is not in the scope.
Nonetheless, I'd implement it on another module/repository if samples and some documentation of it are available.


org: implement a force delete

Currently when you delete an org, the org have to be empty (no project created)
Maybe we could provide a --force-delete flag in the cli to also delete project inside the org

@gaelL should not be done on the API side ?

We have thought several times about this, having this ?force=true on the API but I think it's "not" a good behavior for an API. We have the restriction because deleting Projects is "sensible" so it's better to be handled for the user for example.
Would make sense for the CLI to follow the same behavior, maybe with a helper (displaying the current Projects) or something but that's up to you :).

thanks @xescugc for the update on this point. :)
in this case, we could have something like:
$ cy org delete --org my-org
org is not empty, please delete your projects or use --force
$ cy org delete --org my-org --force
org and X projects, are going to be deleted. Proceed ? [Y/n]

Is this feature still relevant @xescugc @gaelL @talset ?


patch problem

I did everything yet when I click launch it says 'Could not patch!' You're trying to launch a version you don't have a patch for! Ensure you're using a valid game version, and have the patch for this version in your cultivation install folder. If this means nothing to you, YOU HAVE THE WRONG GAME VERSİON. YOUR GAME VERSİON: 5.1' Idk how to fix this. I would be happy if you could help!

As it says, you've got the wrong game version. 5.1 doesn't have a patch file as there is no server for it anyways. You should be following the setup guide for Grasscutter, which tells you to get an earlier supported game version and links you to where you get it.
https://github.com/Grasscutters/Grasscutter?tab=readme-ov-file#quick-setup-guide


Remove NBSP (U+00A0) character which breaks method type hinting

Method Trustly_Api_Signed->accountPayout(...) did not work for php v7.2 (probably other versions too) because of this character being identified as a type hint for $currency argument.

@anders-trustly
I've hit exactly the same problem
  Type error: Argument 6 passed to Trustly_Api_Signed::accountPayout() must be an instance of  , string given, called in /home/michal/projects/somtel-api/
  src/MS/TrustlyBundle/Service/TrustlyService.php on line 116

Can this be merged?
Best regards,
Michal


AZ CLI 2.65.0 upgrade via homebrew to version 2.65.0_1 caused multiplicity of errors

Describe the bug
AZ CLI version 2.65.0 stop working after upgrade on MacOS via homebrew to homebrew version 2.65.0_1. If homebrew version 2.65.0 is used, it works correctly, but upgrade to 2.65.0_1 brokes dependancies.
Related command
az aks list -otable
az network bastion ssh --ids /subscriptions/abc123/resourceGroups/NETWORK-RG/providers/Microsoft.Network/bastionHosts/BASTION --target-resource-id /subscriptions/abc123/resourceGroups/VM-RG/providers/Microsoft.Compute/virtualMachines/mylab --username my_user_name --auth-type password
Errors
myuser@ntb1234 ~ $ az aks list -otable
The command failed with an unexpected error. Here is the traceback:
'NoneType' object has no attribute '__dict__'
Traceback (most recent call last):
  File "/opt/homebrew/Cellar/azure-cli/2.65.0_1/libexec/lib/python3.12/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/Cellar/azure-cli/2.65.0_1/libexec/lib/python3.12/site-packages/azure/cli/core/commands/__init__.py", line 564, in execute
    self.commands_loader.load_arguments(command)
  File "/opt/homebrew/Cellar/azure-cli/2.65.0_1/libexec/lib/python3.12/site-packages/azure/cli/core/__init__.py", line 527, in load_arguments
    loader.load_arguments(command)  # this adds entries to the argument registries
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/uuser/.azure/cliextensions/aks-preview/azext_aks_preview/__init__.py", line 45, in load_arguments
    load_arguments(self, command)
  File "/Users/uuser/.azure/cliextensions/aks-preview/azext_aks_preview/_params.py", line 433, in load_arguments
    k8s_support_plans = self.get_models(
                        ^^^^^^^^^^^^^^^^
  File "/opt/homebrew/Cellar/azure-cli/2.65.0_1/libexec/lib/python3.12/site-packages/azure/cli/core/__init__.py", line 782, in get_models
    return get_sdk(self.cli_ctx, resource_type, *attr_args, mod='models', operation_group=operation_group)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/Cellar/azure-cli/2.65.0_1/libexec/lib/python3.12/site-packages/azure/cli/core/profiles/__init__.py", line 91, in get_sdk
    return _sdk_get_versioned_sdk(cli_ctx.cloud.profile, resource_type, *attr_args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/Cellar/azure-cli/2.65.0_1/libexec/lib/python3.12/site-packages/azure/cli/core/profiles/_shared.py", line 694, in get_versioned_sdk
    sdk_path = get_versioned_sdk_path(api_profile, resource_type, operation_group)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/Cellar/azure-cli/2.65.0_1/libexec/lib/python3.12/site-packages/azure/cli/core/profiles/_shared.py", line 686, in get_versioned_sdk_path
    api_version = getattr(api_version, operation_group)
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/Cellar/azure-cli/2.65.0_1/libexec/lib/python3.12/site-packages/azure/cli/core/profiles/_shared.py", line 481, in __getattr__
    self._resolve()
  File "/opt/homebrew/Cellar/azure-cli/2.65.0_1/libexec/lib/python3.12/site-packages/azure/cli/core/profiles/_shared.py", line 468, in _resolve
    for operation_group_name, operation_type in self._client_type.__dict__.items():
                                                ^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute '__dict__'. Did you mean: '__dir__'?
To check existing issues, please visit: https://github.com/Azure/azure-cli/issues
myuser@ntb1234 ~ $

myuser@ntb1234 ~ $ az network bastion ssh --ids /subscriptions/abc123/resourceGroups/NETWORK-RG/providers/Microsoft.Network/bastionHosts/BASTION --target-resource-id /subscriptions/abc123/resourceGroups/VM-RG/providers/Microsoft.Compute/virtualMachines/mylab --username my_user_name --auth-type password
The command failed with an unexpected error. Here is the traceback:
No module named 'pkg_resources'
Traceback (most recent call last):
  File "/opt/homebrew/Cellar/azure-cli/2.65.0_1/libexec/lib/python3.12/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/Cellar/azure-cli/2.65.0_1/libexec/lib/python3.12/site-packages/azure/cli/core/commands/__init__.py", line 666, in execute
    raise ex
  File "/opt/homebrew/Cellar/azure-cli/2.65.0_1/libexec/lib/python3.12/site-packages/azure/cli/core/commands/__init__.py", line 733, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/Cellar/azure-cli/2.65.0_1/libexec/lib/python3.12/site-packages/azure/cli/core/commands/__init__.py", line 703, in _run_job
    result = cmd_copy(params)
             ^^^^^^^^^^^^^^^^
  File "/opt/homebrew/Cellar/azure-cli/2.65.0_1/libexec/lib/python3.12/site-packages/azure/cli/core/commands/__init__.py", line 336, in __call__
    return self.handler(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/Cellar/azure-cli/2.65.0_1/libexec/lib/python3.12/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
           ^^^^^^^^^^^^^^^^^^
  File "/Users/uuser/.azure/cliextensions/bastion/azext_bastion/custom.py", line 145, in ssh_bastion_host
    _test_extension(SSH_EXTENSION_NAME)
  File "/Users/uuser/.azure/cliextensions/bastion/azext_bastion/custom.py", line 78, in _test_extension
    from pkg_resources import parse_version
ModuleNotFoundError: No module named 'pkg_resources'
To check existing issues, please visit: https://github.com/Azure/azure-cli/issues
myuser@ntb1234 ~ $

Issue script & Debug output
.
Expected behavior
az aks list -otable should list AKS clusters in subscription
az network bastion ssh --ids /subscriptions/abc123/resourceGroups/NETWORK-RG/providers/Microsoft.Network/bastionHosts/BASTION --target-resource-id /subscriptions/abc123/resourceGroups/VM-RG/providers/Microsoft.Compute/virtualMachines/mylab --username my_user_name --auth-type password should connect to VM via ssh via bastion-host-paas-service
Environment Summary
myuser@ntb1234 ~ $ az version
{
  "azure-cli": "2.65.0",
  "azure-cli-core": "2.65.0",
  "azure-cli-telemetry": "1.1.0",
  "extensions": {
    "aks-preview": "9.0.0b6",
    "azure-devops": "1.0.1",
    "azure-firewall": "1.2.0",
    "bastion": "1.3.1",
    "containerapp": "1.0.0b3",
    "ssh": "2.0.5"
  }
}

Additional context
It seems as root cause from several others issues, which are reported.

Thank you for opening this issue, we will look into it.

This is the fix: https://github.com/Azure/azure-cli/issues/30127#issuecomment-2425812364

I also ran into this problem. Was fixed with the latest update homebrew azure-cli (2.65.0_2).

I have a similar issue when (auto) installing ssh and bastion extensions. Really need a fixed az cli version soon.

brew update && brew upgrade fixed the issue as:
/opt/homebrew/Cellar/azure-cli/2.65.0_2
thanks a lot for the quick fix.


chore: release main

:robot: I have created a release beep boop
stdlib: 1.0.12
1.0.12 (2023-12-23)
Bug Fixes

upd renderer version (c57097e)

react-shader-fiber: 1.0.9
1.0.9 (2023-12-23)
Bug Fixes

upd renderer version (c57097e)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/pavophilip/react-shader-fiber/releases/tag/stdlib-v1.0.12 :sunflower:

:robot: Release is at https://github.com/pavophilip/react-shader-fiber/releases/tag/react-shader-fiber-v1.0.9 :sunflower:


Error on validation Array on server

where i try validate a response of users(and array of users)
[
{
id: 1,
name: 'Important Customer',
email<EMAIL_ADDRESS>},
{
id: 2,
name: 'Super Customer',
email<EMAIL_ADDRESS>}
]
const form = await superValidate(users, crudSchema);
I get the following error
Argument of type 'User[]' is not assignable to parameter of type 'RequestEvent<Partial<Record<string, string>>, string | null> | Request | FormData | Partial<Record<string, unknown>> | null | undefined'.
Type 'User[]' is not assignable to type 'Partial<Record<string, unknown>>'.

Hi, yes, the crudSchema only handles a single user (an object, not an array), so you cannot pass the whole users array to it.

So what happens if the data am returning is an array if users? Hodlw do I attach it to form?

Note that this pretty much requires you to use options.dataType = 'json' (and use:enhance on the form element), otherwise you have to do some complicated string-to-array conversions when posting.

The idea is that a schema encapsulates a single form as an object (reflecting FormData), so if you need multiple users represented in the form, you should add them as a property to the schema:
const crudSchema = z.object({
  users: z.object({
    name: z.string()
  }).array()
})

I understand this, for posting data from form to the server, this is right. My question is on load function in server when am returning the users to the form(server to client)

It's the same, you are representing a form in the load function as well. So you should add your users to an object, and it should work:
const form = await superValidate({ users }, crudSchema);

Did you manage to make it work when adding the users to an object?

The solution you gave meant that if i have a schema of user object, i cant use it to define an array of users.
I dont know whether i understood you correctly, but it meant that i have to write another schema to define an array of users incase i want to return.
My understanding of CRUD, i should be able to use the user schema to create one, read one,read many, update one and delete one or many.

The schema passed to superValidate is used to validate a single form, not any arbitrary schema.
CRUD can be regarded as CRUD<T> where T is the entity you want to apply the operations to, so if you want to do CRUD<T[]>, you cannot use superValidate for that schema because it does not map to validating a single form.

Exactly, if the data isn't going to be used in the form, just pass it as another field than form.

Closing this, if you don't mind. :)


Add Docker Compose Setup for Easier Deployment

I have designed a Docker Compose process based on the existing Docker build workflow, hoping to provide users with a more versatile and automated way to deploy and use Hybooru!
I tested it in Ubuntu within Windows 11 WSL2, and it works without issues. 😆

Hi, thanks for the PR, but I would rather not add bunch of files(templates/examples?) for some specific tool for docker. I'm not experienced in docker so I might be missing something here, but what would be the advantage of using these configs over just using the hybooru package in just docker or whatever compose file one is using for their project/setup?
I would also rather not include guides made just for very specific environment(like WSL2 on windows), try to make it platform agnostic. After all Windows is not even officially supported by hybooru.
If writing this sort of compose files is something that is done often and people would benefit from having a short template, I think README.md might be a better place for it, just like the one for nginx reverse proxy.

I completely understand your point. I believe the purpose of writing the docker-compose is to enable simple deployment on any platform and to package the hybooru package deployment steps (in this case, specifically simplifying the database setup process)
making it more user-friendly.
By the way, I am currently checking the hybooru package, and it seems that the README does not mention this package or how to use it, which is why I completely overlooked it, my bad.
Regarding the PR adjustments, do you mind if I add instructions for using the hybooru package in the README? I will also include examples for docker-compose along with some explanations.

I decided to keep the Docker support and documentation as is. Primary way of using Hybooru is by using production builds and the documentation includes how to use it. I personally do not use Docker and the support for it will be kept minimal, just so people who are well experienced in it can introduce Hybooru into their Docker workflow. If someone is looking for ease of development process, they should just install node and postrgresql on their local machine and work on hybooru without adding the complexity of Docker configuration and containerization.


Tapis CLI commands seems slower than their Bash analogues

Describe the bug
Several Tapis CLI commands appear take longer to run (or return a response) than their Bash analogues. See the table below. Times are in seconds and are the aggregate of 5 measurements.

Command
Tapis CLI
Bash CLI

apps-list -l 10
1.56+/-0.22
0.54+/-0.08

systems-list -l 10
1.42+/-0.05
0.45+/-0.13

jobs-list -l 10
2.03+/-0.05
0.95+/-0.09

metadata-list -l 10
1.33+/-0.07
0.43+/-0.09

There are a couple of potential sources of overhead. First, Tapis CLI proactively refreshes its token on startup. Second, it is using the built-in Python JSON parser, which may be slower than either jq or the custom JSON parser written in Bash.


Update 0.3 upgrade guide

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (cd88b44) 100.00% compared to head (b1ae5a9) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #445   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           73        73
  Lines         2591      2591
  Branches       418       418
=========================================
  Hits          2591      2591

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.