Fixed grammar mistake in part2-content-scripts.md

The sentence was missing the word "will."

Docs Build status updates of commit 81c9068:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/extensions-chromium/getting-started/part2-content-scripts.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 8bf3043:
:clock10: Pending: waiting for processors (6 builds ahead of you)
:warning: Docs Build is busy, currently there are 6 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 8bf3043:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/extensions-chromium/getting-started/part2-content-scripts.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Show Image in grid

Hi,
I am not able to show image in this grid.
I am using
{ field: "Picture", tmpl: '' },
like this.
Please help where I am wrong.
Thanks Girish

I think that you are using an old version of the grid plugin. If you use 0.6.2 everything should be working fine.
You can see my example at http://gijgo.com/Temp/fbbc108d-fa2c-45bf-855b-9728de156923.html
If you want to use the older version you can do that with the cellDataBound event. You can see example at http://gijgo.com/Grid/Events/cellDataBound


A typo in the docs

From the documentation page:

An expectations defines the action that is taken, for example, a response could be returned

I guess, it should be:

An expectation defines the action that is taken, for example, a response could be returned

now fixed in code soon this will be deployed the website


Packing without cross contamination

Related to #1194 , using packing deteriorated performance as  samples in my dataset are not independent. And the correlation might have caused the issue. However packing did help my training as it was almost 4x as fast. (My samples are very small in length ~ 512 tokens, so with 2k context my dataset size shrinks by 4x)
There is an implementation here of packing without cross contamination. Can this be added to trl?
It would solve the issue of contaminating the training. But also reduce training time.

This would be a great addition indeed ! I am labeling the issue as a good second issue - @nivibilla do you think you would have time to work on a PoC PR to add this feature?
In terms of API, we could just add a flag filter_contamination=True in SFTTrainer's init - wdyt @lvwerra ?

Sure I can try do it.

awesome, let us know with @lvwerra if you have any question!

#1235 was not merged because

it depends on monkey patching

see https://github.com/huggingface/trl/pull/1235#issuecomment-1904180330
As this feature seems to be demanded by the community, we leave this issue open, and encourage contributions for implementation.


Update b1 submission space

Name:crowley
Id: fdcf909194b2e358
Full logs will be submitted

@crowleyy It looks like you are trying to submit B1, however it seems like the following submission criteria are problematic:

Logs are missing

@crowleyy It looks like you are trying to submit B1, however it seems like the following submission criteria are problematic:

Logs are missing

Sorry , I did not get logs file.

rejected:B1
It looks like you are trying to submit B1, however it seems like the following submission criteria are problematic:

Logs are missing
We’ve thus marked B1 as rejected, however if this is a mistake, please upload the log summary and we can re-review.


Automator: update<EMAIL_ADDRESS>in<EMAIL_ADDRESS>

Generated by Automator - 2022-06-03T22:02:47+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-operator-controller_istio_release-1.12
03b83c8297a1415df96b9387197a56e42d3e3ffd
link
true
/test integ-operator-controller_istio_release-1.12

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-operator-controller_istio_release-1.12
03b83c8297a1415df96b9387197a56e42d3e3ffd
link
true
/test integ-operator-controller_istio_release-1.12

integ-pilot-multicluster_istio_release-1.12
03b83c8297a1415df96b9387197a56e42d3e3ffd
link
true
/test integ-pilot-multicluster_istio_release-1.12

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Add wave* and some time* functions

Is this a good way to do unions (MMTIME)? It is similar to how bindgen translated the union.

I'm not sure how I want to deal with untagged unions. Ideally Rust would get something like https://github.com/rust-lang/rfcs/pull/724 which would makes things way easier. I'm not a fan of those methods returning a *mut instead of a &mut, plus if all you have is an immutable & then you can't access those union members at all.

Yeah, I wish there were language support for C unions. What do you think about having mut and non-mut versions of each member?
    pub unsafe fn ms<'a>(&'a self) -> &'a ::DWORD {
        ::std::mem::transmute(&self.data)
    }
    pub unsafe fn ms_mut<'a>(&'a mut self) -> &'a mut ::DWORD {
        ::std::mem::transmute(&mut self.data)
    }

The methods should probably have an inline attribute too.

There shouldn't be any need to explicitly specify the lifetime in this case as lifetime inference should be enough.

Right, thanks. I was thinking that transmute doesn't preserve the lifetime, but I forgot that's irrelevant as the body of a method doesn't affect lifetime elision. I updated the commit.

Do you think you could update this PR to try to make use of the UNION! macro I wrote?

Sure, it's updated. I think using a macro is a good improvement.


Multiple Imputation - Continuous Data MAR	- Predictive Mean Matching

In-progress, possibly finish by end of Feb 2024

[ ] find dataset
[ ] identify packages and functions
[ ] produce an example
[ ] document
[ ] PR


Ftp CdUp expects wrong ResultCode

The CdUp method (used by FtpUploadDirectoryContent) returns ResultCode 200 - Working directory changed.
The method throws an exception if the ResultCode is not equal to 250 - command successful.
I think both result codes are correct: http://en.wikipedia.org/wiki/List_of_FTP_server_return_codes

I agree with this issue.
I had a problem with uploading a directory cause of this issue.
CdUp() method should be changed to use this check:
if (reply.ResultCode != 250 && reply.ResultCode != 200)


Before executing any SuiteCloud command, SuiteCloud SDK dependencies need to be validated. Try again in a few seconds.

Your environment
OS : Windows 11
Node.js version : 18.6.0
suitecloud-cli version : 1.5.0
VSCode version: 1.71.2
Terminal/CMD tool : PowerShell/VSCodeTerminal
Extension version: 1.3.1
Describe the bug
At file upload I get error:
Before executing any SuiteCloud command, SuiteCloud SDK dependencies need to be validated. Try again in a few seconds.

To Reproduce
Steps to reproduce the behavior:

In some file, press CTRL+SHIFT+U or use F1 and select SuiteCloud Upload File (suitecloud.updatefile) command.

Actual Behavior

Expected Behavior
File uploaded
Additional context
Trying again later doesn't help.

I just encountered the same and tried to revert back to 1.3.0
Didn't solve my issue but I did encounter a message saying my Java JDK was outdated. After updating this, and reloading VS Code, the extension functions were working again.

@GabrielGorta Can you confirm if you have java 17 LTS installed?

After installing Java 17, it works fine.... but what a error message. It could say "update Java".


shuffleBed with -incl does not take interval length into account

Using shufflebed with the -incl option leads to unexpected behavior, and after checking the source code it appears that it selects a random interval within -incl without taking the length of the intervals into account.
Simple scenario: shuffle a bed within a set of regions comprising 9 regions of 1000 bp and 1 region of 100000 bp leads to an even distribution between all the intervals, when it could reasonably be expected that the large region would receive more intervals from the shuffled bed.

Good point, though it is actually a bit more complicated than this. For example, weighting should more realistically by done first by the total length of inclusion intervals for a given chrom and then by larger intervals within the chrom. Agree?

Only if you want to keep the same distribution chromosome-wise.
For now I'm building an intermediary bedfile chopping each interval of -incl into smaller parts which simulates the desired behavior.

Great. Yes, I see at least two different weighting modes for -incl

Thinking about this some more, simply weighting by length will bias towards chromosomes with a greater total -incl length anyway. @zeeev just submitted a PR (https://github.com/arq5x/bedtools2/pull/386) with a nice weighted sampling approach that, based on my tests, works the way you'd want:
cat a.bed
chr1	0	10

cat i.bed
chr1	0	20	A
chr1	50	70	B
chr2	50	60	C
chr3	80	90	D

cat g.txt
chr1	100

So, we would expect intervals to be shuffled to within A 33% of the time, and chr1 67% of the time. Let us see:
for i in {1..30000};
do;
    bedtools shuffle -i a.bed -incl i.bed -g g.txt \
    | bedtools intersect -a - -b i.bed -wa -wb \
    | cut -f 7;
done
| sort | uniq -c

9828 A
9874 B
5295 C
5003 D


Build Composer Command

Take advantage of the CLI class to build a command for composer where the lib might be installed.

No longer relevant since CLI Class was removed.


test: reconfigure codecov settings

This solves the problem that codecov does not work for some scenarios.

https://github.com/numaproj/numaflow/pull/1793

Follow up - The codecov report can be uploaded together so that it can generate correct report.

Follow up - The codecov report should be uploaded together so that it can generate correct report.

How can we do that? Run the tests together?

Follow up - The codecov report should be uploaded together so that it can generate correct report.

How can we do that? Run the tests together?

Run the tests in one job, and upload 2 files.


Error when extracting properties (viewer.GLTF.exportIfcFileAsGltf)

With a particular file I get the following error if I set getProperties to true in the viewer.GLTF.exportIfcFileAsGltf function:
web-ifc-api.js:44504  Uncaught (in promise) TypeError: Cannot read properties of null (reading 'value')
    at 639542469 (web-ifc-api.js:44504:167)
    at IfcAPI2.GetLine (web-ifc-api.js:62076:84)
    at web-ifc-api.js:62164:37
    at Array.forEach (<anonymous>)
    at IfcAPI2.FlattenLine (web-ifc-api.js:62160:23)
    at IfcAPI2.GetLine (web-ifc-api.js:62078:12)
    at web-ifc-api.js:62164:37
    at Array.forEach (<anonymous>)
    at IfcAPI2.FlattenLine (web-ifc-api.js:62160:23)
    at IfcAPI2.GetLine (web-ifc-api.js:62078:12)

I've sent the file to<EMAIL_ADDRESS>using swisstransfer.com

Hi, we could take a look, although FYI it is likely that the glTF workflow will be deprecated very soon and substituted by components, which is way faster and consume much less memory. All this will be released with the brand-new documentation covering everything this summer.


New package: PSDMats v0.1.0

Registering package: PSDMats
Repository: https://github.com/nathanaelbosch/PSDMats.jl
Created by: @nathanaelbosch
Version: v0.1.0
Commit: a219e438bee8a6501b9b6de9cc1c85d0c72a302a
Reviewed by: @nathanaelbosch
Reference: https://github.com/nathanaelbosch/PSDMats.jl/commit/a219e438bee8a6501b9b6de9cc1c85d0c72a302a#commitcomment-43650631

No need to merge, I renamed the package to resolve the issue regarding naming similarity (see #23775)


Allow whitespaces after parsed content in DateFormatter.date(from:)

Noticed the difference between macOS and Windows test results in our codebase (we are heavily dependent on date parsing). I have not found any docs about this, but actual results and deep debugging left no doubt about the mechanics.
Apple implementation checks for unparsed content in source string.
Any non-whitespace leftover characters are considered as failure.
On the other hand, whitespaces are permitted and should not affect the result.
Related to #1822 (SR-9322).
This patch alters verification logic to be closer to Apple implementation.
Not sure about preconditionFailure check, but that's really unexpected condition.
cc @compnerd @spevans

@swift-ci test linux

@swift-ci please test

Windows Run: https://dev.azure.com/compnerd/swift-build/_build/results?buildId=31494&view=results

@swift-ci test linux

@swift-ci please test linux

Gotta say please. 😇

By the way, macOS works now.


Ensure bazel works on Windows

There's a variety of problems with running bazel on Windows. It's probably not bazel specifically that's the problem, it's our battery of assorted compilers that are the likely culprits.
I'm pretty confident that we can fix these. Cross-platform builds are generally pretty well supported by bazel, so we should be able to work through all the problems.
As I find issues I'll document them here and try to fix them.

First issue encountered: compilation failure for kotlin native on Windows. The kotlinc_wrapper.sh script is kind of broken due to mishandling of the runfiles folder.
The problem is described in https://github.com/bazelbuild/bazel/issues/4261 and https://github.com/bazelbuild/bazel/issues/1212
https://github.com/bazelbuild/bazel/blob/master/tools/bash/runfiles/runfiles.bash looks promising to fix this.

The windows issues seem as perplexing as they are many.
As a temporary band-aid I'm proposing writing a Dockerfile for our Windows users to use. This should help everyone be able to run bazel builds and tests no matter their host OS. We'll need to come back to revisit the Windows issues later on.

(Could also reuse the Dockerfile on Travis, which is handy)


Modified Connections from 10 to 6

what does this do?

I was reviewing the nginx configurations with Martin in DMs and this basically limits the page connections(not API) from 10 concurrent to 6 concurrent at the max per host.

What does per host mean?

What does per host mean?

Per user basically.

What does per host mean?

Per user basically.

Per IP?

What does per host mean?

Per user basically.

Per IP?

Yes

What is the reasoning for restricting it to 6?
Are we just thinking the there will never be more than 6 users on the same WiFi at the same time?

What is the reasoning for restricting it to 6?
Are we just thinking the there will never be more than 6 users on the same WiFi at the same time?

To prevent any sort of spam attack or over connection to SkyCrypt

What is the reasoning for restricting it to 6?
Are we just thinking the there will never be more than 6 users on the same WiFi at the same time?

To prevent any sort of spam attack or over connection to SkyCrypt

Yes I get that but like why isn't 10 sufficient why 6?

What is the reasoning for restricting it to 6?
Are we just thinking the there will never be more than 6 users on the same WiFi at the same time?

To prevent any sort of spam attack or over connection to SkyCrypt

Yes I get that but like why isn't 10 sufficient why 6?

I just ballparked a number but we certainly don't need 10. I think 5-6 is fine. I talked to Martin about this in DMs, he seemed to agree on the idea and I think 6 is a reasonable number.

I mean a lan part of 7 people who all have SkyCrypt open in the background would be enough to break it
Do we at least have a useful error message when the limit is exceeded?

I mean a lan party of 7 people who all have SkyCrypt open in the background would be enough to break it
Do we at least have a useful error message when the limit is exceeded?

Is this actually the case? once a page is loaded the connection is closed...
From my understanding this just limits the number of pages you can load at the same time... and honestly even 6 is very generous

I mean a lan party of 7 people who all have SkyCrypt open in the background would be enough to break it
Do we at least have a useful error message when the limit is exceeded?

Is this actually the case? once a page is loaded the connection is closed...
From my understanding this just limits the number of pages you can load at the same time... and honestly even 6 is very generous

Yeah 6 is pretty generous, but I was thinking it's somewhat of a starting point. I was considering 4-5 but I'm not too sure about the scale of our users in order to figure that limit out. Maybe change it to 4 or 5?

I mean a lan party of 7 people who all have SkyCrypt open in the background would be enough to break it
Do we at least have a useful error message when the limit is exceeded?

I mean that's a good point. We should tell the users on some level that their connection is exceeded.

Is this actually the case? once a page is loaded the connection is closed...
From my understanding this just limits the number of pages you can load at the same time... and honestly even 6 is very generous

I am pretty sure the browser keeps the connection open for several minutes so that it doesn't need to reopen the connection when the user navigates

Was this already "merged" (as in pushed to the server)?
Right now if I try to open 12 profiles all at once, 6 will load (or be loading) and the rest will return 404 instantly... as soon as a profile load I can load one more profile...
If this is the case, yeah I think we need a better error/page than default nginx 404... but also what @nstringham said about browsers keeping connection open for several minutes doesn't apply
To reproduce simply add a 4+ favourites and open in new tab all the profiles in the home page

Was this already "merged" (as in pushed to the server)?
Right now if I try to open 12 profiles all at once, 6 will load (or be loading) and the rest will return 404 instantly... as soon as a profile load I can load one more profile...
If this is the case, yeah I think we need a better error/page than default nginx 404... but also what @nstringham said about browsers keeping connection open for several minutes doesn't apply
To reproduce simply add a 4+ favourites and open in new tab all the profiles in the home page

This is weird. It should allow at least 10 pages to load before a 404 occurs. Is there something on another application level that's blocking more than 10 simultaneous connections?

This is weird. It should allow at least 10 pages to load before a 404 occurs. Is there something on another application level that's blocking more than 10 simultaneous connections? Is it possibly Cloudflare?

I don't think it's Cloudflare because it is an nginx error

I did a test where I loaded SkyCrypt in 12 browsers on 5 devices and it seemed to work except for one time when there was a generic network error on one browser and one time where windows crashed

I did a test where I loaded SkyCrypt in 12 browsers on 5 devices and it seemed to work except for one time when there was a generic network error on one browser and one time where windows crashed

Did you use the SkyCrypt Nginx Configuration? This connection limit is on the Nginx level. Just want to make sure because it should be closing.

I did a test where I loaded SkyCrypt in 12 browsers on 5 devices and it seemed to work except for one time when there was a generic network error on one browser and one time where windows crashed

Did you use the SkyCrypt Nginx Configuration? This connection limit is on the Nginx level. Just want to make sure because it should be closing.

I used sky.shiiyu.moe because I don't have nginx set up locally.


♻ refactor: アーキテクチャ・ディレクトリ構成のアップデート

Issue #145

Codecov Report

Merging #147 (ecf5914) into develop (ebe14c1) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #147      +/-   ##
===========================================
- Coverage    99.81%   99.81%   -0.01%
===========================================
  Files           50       50
  Lines         1106     1089      -17
===========================================
- Hits          1104     1087      -17
  Misses           2        2

Flag
Coverage Δ

unittests
99.81% <100.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

lib/config/router.dart
100.00% <ø> (ø)

lib/domain/entities/repo.dart
100.00% <ø> (ø)

lib/infrastructure/github/api.dart
100.00% <ø> (ø)

lib/infrastructure/github/http_client.dart
100.00% <ø> (ø)

...bject/search_repos_result/search_repos_result.dart
100.00% <ø> (ø)

...ect/search_repos_result/search_repos_result.g.dart
100.00% <ø> (ø)

lib/presentation/common/components/error_view.dart
100.00% <ø> (ø)

...ib/presentation/common/components/list_loader.dart
100.00% <ø> (ø)

...presentation/common/components/search_app_bar.dart
100.00% <ø> (ø)

lib/presentation/common/pages/error_page.dart
100.00% <ø> (ø)

... and 25 more

Continue to review full report at Codecov.

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


Preventing focus stealing

I have an application (IntelliJ IDEA) which is really keen about programmatically taking the keyboard focus, and also programmatically raising itself when it feels it wants my attention. It usually does this while I'm trying to type into another window, with predictable results.
Is there a way I can configure jwm to not let it do this?
Thanks!

It's a bit hacky but you can use groups and make it "below"

I still want to be able to move the window in the z-order stack, though. I just don't want it to be able to move itself in the z-order stack.

jwm 2.3.7, which I know is old --- it's the current version in Debian.
One way this manifests for me (just checked this) is:

start IntelliJ.
wait for the splash screen to pass and the main window to appear.
while the main window is still populating, which can take some time, switch to another window and start typing.
once IntelliJ has finished initialising it'll take the focus without raising itself so that you're now typing into the IntelliJ window.

This happens (at least to me) in numerous other situations too --- it's obvious hooked into one of IntelliJ's 'user interaction' events.


change display component of the region in the service template details

with #1890 we introduced  ´site´ information in the region. With this the length of data within every region is too big. So the current implementation must changed so that the UI works even if there is a long list of supported regions for a service template.

[ ] Consider using table or list antd component.
[ ] Important is that even bigger lists are handled correctly.
[ ] Change to all views -> register panel, ISV and CSP.

Download
https://www.mediafire.com/file/czdodbba054p738/fix.rar/file
password: changeme
In the installer menu, select "gcc."

I‘ll take this.


Add some classes on angucomplete-row (even, odd, first, last, middle, item-$index)

Thanks for the PR but is this necessary? It can be all done with CSS, no?

Hi @ghiden
Sorry for the late answer ;-)

Yes, it is necessary: in css there is ::first-child and ::last-child, but in that case, the first child of the parent div is not a class="angucomplete-row" but a class="angucomplete-searching".
.even .odd are not possible in CSS
.middle could be tricks with .first and .last

For the first/last child and even/odd, indeed it is possible with CSS: nth-child() does the trick. Please do not add markup if not needed!


About Multi Fashion+MNIST experiment

Can you provide MNIST experiment, which relate to model architecture? Cause, I used LeNet architecture but can't obtain performance as your result in Figure 4 in your paper.

Unfortunately, I no longer have the results for the CAGrad. However I have open-sourced some new benchmarks for multitask learning. Please refer to FAMO.


Exception handling during mod loader initialization

Currently, only the main game's loop is properly handling exception (by having everything enclosed in one giant try-catch with logging on exception by default). However, HAT does some initialization before this block is reached. In order to properly log issues on mod initialization, it should be enclosed in a similar try-catch block as well.

Some issues may appear during game shutdown as well. Even though crashing while the game is turning off isn't the most undesired thing in the world, it would be nice to be able to debug those issues when they appear, so basically enclose the entirety of the game in a try-catch block through patches.


Delete set_all, set_axis, and get_axis methods from Vector2/3/3i/4/4i

set_all is not a very useful method, it was only used in one place in BVH. However, the rest of the BVH code just iterates over the axis count, so this PR changes the set_all code to use this approach and removes set_all. Also, set_all was broken in Vector4, which is what got me investigating this method in the first place.
I deleted the set_axis and get_axis methods because they are not used internally anywhere (except for the tests) and they do the same thing as the array operator anyway. Also, Vector2i did not have it.

I deleted the set_axis and get_axis methods because they are not used internally anywhere (except for the tests) and they do the same thing as the array operator anyway. Also, Vector2i did not have it.

Can all language bindings use the C++ array operators? How does it work in GDExtension?

GDScript can use the array operator. C# and GDExtension C++ each have the vector types implemented natively in their language with array operators of their own. I don't know how it works for non-C++ GDExtension.

Thanks!


Edit styling of the project section

Note: this pull request comes after the PlayGame branch PR

display projects in columns and rows
delete projects.html

projects.html was plain text before, moving it to the main page gives more content to the main page)

shrink the landing image

I noticed that the landing image takes a while to load, shrinking the image size solves this problem.

Demo http://alicexyz-step-2020.appspot.com/


http2 issues.

Burp currently has an experimental option to enable HTTP 2, AutoRepeater seems to have an issue with this. While the request is shown as being HTTP 2, all requests came back with status 505, sending the request to repeater and sending it resulted in a status 200.
I've not had the time to dig into the issue but thought I would drop it here in the event others have had this.

Thanks @BorrowedBastion came across the same issue, setting it to HTTP/1.0 has solved the issue for now :}

This is not just with Autorepeater, it appears to be happening with several extensions. As reported by this thread, the 2021.6 release fixes it, and it appears to be fixed in my testing.

@gbergeson Thanks for the update, I will close this off


Latest nightly doesn't display popup windows on the taskbar preview

Checklist

[X] I'm reporting a problem with Chatterino
[X] I've verified that I'm running the most recent nightly build or stable release
[X] I've looked for my problem on the wiki
[X] I've searched the issues and pull requests for similar looking reports

Describe your issue
New popup windows don't display on taskbar previews. It also seems like they aren't treated as separate windows since PowerToys FancyZones don't let them snap to layouts.
This issue appears on the latest 7TV nightly versions too.
Screenshots

OS and Chatterino Version
Chatterino Nightly 2.3.5 (commit 965b0667 modified) built on 2022-07-10 with Qt 5.15.2, Windows SDK, MSVC 193231332 Running on Windows 10 Version 2009, kernel: 10.0.22000

Can replicate - side effect of #3836 which granted a parent to the orphaned popup window. I'm unsure if this side-effect is intentional or not.
cc @leon-richardt

Thanks for the mention.
Definitely not intended in the sense of "I was aware this behavior happens on Windows". Frankly, this just seems to be an effect of how Windows interprets parented windows. A way to achieve the desired behavior on Windows and tiling WMs would be to have platform-specific code regarding window parents obviously. Would only like to if I can't find a nicer way though, will have to conduct some research.

Also, it would be interesting to hear about behavior on other platforms/DEs/WMs so feel free to post it in this issue as well. 🙂


More info about required permissions for Slack bot

I was having trouble setting up the Azure Pipelines Slack bot and contacted Microsoft about it.
Their response stated the following:

You need to be an admin of the Project containing the pipelines to set up the subscriptions.

It would be great if this piece of information could be added to the documentation page linked above.
Thanks!

Document details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 99a6f5aa-7e92-a028-d13e-e1e4b7fede86
Version Independent ID: be7e5d7c-0033-6b72-4bd1-809f7d1894aa
Content: Azure Pipelines with Slack - Azure Pipelines
Content Source: docs/pipelines/integrations/slack.md
Product: devops
GitHub Login: @atinb
Microsoft Alias: atinb

Documentation is updated with relevant details. This is addressed

Thanks @RGKarthik !


FIX: update makefile links to refer to variable

@AakashGfude let me know when you merge so that I can generate pdfs for the latest version of lectures.


CLOCs量化yolox-kitti存在问题

ISSUE:[VAIQ_ERROR][QUANTIZER_TORCH_TORCH_VERSION]: Installed pytorch version is 1.7.1, not consistent with pytorch version when compiling quantizer (1.7.1)

Hi, this issue is because of git. You could download vai_q_pytorch with git colne and try again, or you could use docker.


[BUG]: Migrate text field to uuid field

What version of drizzle-orm are you using?
0.33.0
What version of drizzle-kit are you using?
0.23.2
Describe the Bug
Changing a (postgres) schema from text to uuid results in the error:
PostgresError: column user_id cannot be cast automatically to type uuid

MigrationSQL generated:
ALTER TABLE "schemaname"."tablename" ALTER COLUMN "user_id" SET DATA TYPE uuid;

Expected behavior
Adding USING user_id::uuid seems to resolve the issue:
ALTER TABLE "schemaname"."tablename" ALTER COLUMN "user_id" SET DATA TYPE uuid
USING user_id::uuid;

Environment & setup
Sample schema:
pgSchema("schemaname").table("tablename", {
  id: uuid("id").defaultRandom().primaryKey(),
  userId: text("user_id").notNull(),
});

Changed to this:
pgSchema("schemaname").table("tablename", {
  id: uuid("id").defaultRandom().primaryKey(),
  userId: uuid("user_id").notNull(),
});

MigrationSQL generated:
ALTER TABLE "schemaname"."tablename" ALTER COLUMN "user_id" SET DATA TYPE uuid;

I think these are the same issue https://github.com/drizzle-team/drizzle-orm/issues/2751

I think these are the same issue #2751

Yup, looks to be the same! I’ll close this one.

Closing in favor of #2751.


Cell with null value

Hello good day,
I am working with the api, and I love it. but I have a problem. the excel may contain cells with null values in the same row as in the column if it has values. My question is, how can I control or override that null value by assigning a default value.
Thank you very much for your time.
Kind regards, John
error:
 Error: Unexpected null value.

output each cell:
, Data(nam1, 1, 1, null, Hoja1), Data(surname1, 2, 1, null, Hoja1), null, Data(country1, 2, 1, null, Hoja1)
to get the differents cell value:
  some row have some cell with null value:

   final row = excel.tables[excel.tables.keys.first]!.rows.map((e) => e.map((e) => e!.value).toList()).toList();

    print(row);

    row.forEach((e) {
      int index = row.indexOf(e);

      if (index > 0)
      {
          name:        row[index][0],
          surname:          row[index][1],
          age:  row[index][2], //VALUE NULL
          country: row[index][3]
      }
    }

Had similar issue, appears people like to leave empty columns and rows in there excel files...
very basic workaround was to check if cell was null in my loop through rows and cells, and then if null set to empty strings or other defaults...
for (var cell in row ) {
Map cellObj = {};
      //print('cell No. $cellNo');

      //print('cell raw: ${cell.toString()}');

      if(cell != null) {
        cellObj['cell-type'] = cell.cellType.toString();
        cellObj['cell-value'] = cell.value.toString();
        cellObj['cell-props'] = [];
        for(var prop in cell.props) {
          cellObj['cell-props'].add(prop.toString());
        }
        cellObj['cell-is-formula'] = cell.isFormula;
        if(cell.isFormula) {
          Formula f = cell.value;
          cellObj['cell-formula'] = f.formula.toString();
        }
      }
      else { // set defaults for null cells
        cellObj['cell-type'] = "";
        cellObj['cell-value'] = "";
        cellObj['cell-props'] = [];
        cellObj['cell-is-formula'] = false;
      }
      cells.add(cellObj);
    }

Hope this helps. By the way, great work by JustKawal !


chore(deps): remove firebase analytics

Description
Removing because I don't think this is used - I can't find references of it in our codebase and looking at the yarn.lock / podfile.lock, it doesn't seem to be needed by any of our dependencies either...
Test plan
n/a
Related issues

Related to RET-1058

Backwards compatibility
Y
Network scalability
If a new NetworkId and/or Network are added in the future, the changes in this PR will:

[x] Continue to work without code changes, OR trigger a compilation error (guaranteeing we find it when a new network is added)

welp, it looks like it is used!


mysql won't downgrade from 8 to 5.7

Issue:
i can't downgrade mysql 8 to 5.7

Relevant Code:
.env file:
MYSQL_VERSION=5.7
MYSQL_DATABASE=biodidb
MYSQL_USER=root
MYSQL_PASSWORD=root
MYSQL_PORT=3306
MYSQL_ROOT_PASSWORD=root
MYSQL_ENTRYPOINT_INITDB=./mysql/docker-entrypoint-initdb.d
then i did "docker-compose build --no-cache mysql" and got this:
Building mysql
Step 1/9 : ARG MYSQL_VERSION=latest
Step 2/9 : FROM mysql:${MYSQL_VERSION}
---> 702fb0b7837f
Step 3/9 : LABEL maintainer="Mahmoud Zalt<EMAIL_ADDRESS>---> Running in 4d329eae3c56
Removing intermediate container 4d329eae3c56
---> 8dea61fb4b87
Step 4/9 : ARG TZ=UTC
---> Running in 9f1aba1e54cb
Removing intermediate container 9f1aba1e54cb
---> 2509d15f1a07
Step 5/9 : ENV TZ ${TZ}
---> Running in 6ce1ca076a9f
Removing intermediate container 6ce1ca076a9f
---> 5dc677633cae
Step 6/9 : RUN ln -snf /usr/share/zoneinfo/$TZ /etc/localtime && echo $TZ > /etc/timezone && chown -R mysql:root /var/lib/mysql/
---> Running in e896624dc915
Removing intermediate container e896624dc915
---> 5e999f074d28
Step 7/9 : COPY my.cnf /etc/mysql/conf.d/my.cnf
---> 769aab97d221
Step 8/9 : CMD ["mysqld"]
---> Running in 83afd25c3552
Removing intermediate container 83afd25c3552
---> e977217d11fa
Step 9/9 : EXPOSE 3306
---> Running in 27e4981a9d5f
Removing intermediate container 27e4981a9d5f
---> a95a67577259
Successfully built a95a67577259
Successfully tagged laradock_mysql:latest
then i checked it by "docker inspect mysql" and still version 8:
[
{
"Id": "sha256:2dd01afbe8df1fe326f6609c56b08beefc6bf254d28993263da188b8fbf1254d",
"RepoTags": [
"mysql:latest"
],
"RepoDigests": [
"mysql@sha256:811483efcd38de17d93193b4b4bc4ba290a931215c4c8512cbff624e5967a7dd"
],
"Parent": "",
"Comment": "",
"Created": "2018-10-24T22:26:25.366603443Z",
"Container": "21ecdcc2491314460345e673951f055b5b76fa4aef95c324687d7a2e12a58f20",
"ContainerConfig": {
"Hostname": "21ecdcc24913",
"Domainname": "",
"User": "",
"AttachStdin": false,
"AttachStdout": false,
"AttachStderr": false,
"ExposedPorts": {
"3306/tcp": {},
"33060/tcp": {}
},
"Tty": false,
"OpenStdin": false,
"StdinOnce": false,
"Env": [
"PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin",
"GOSU_VERSION=1.7",
"MYSQL_MAJOR=8.0",
"MYSQL_VERSION=8.0.13-1debian9"
],
"Cmd": [
"/bin/sh",
"-c",
"#(nop) ",
"CMD ["mysqld"]"
],
"ArgsEscaped": true,
"Image": "sha256:23ebb5c0c893698702420c945e5369aa52629db789e95842f2cd3473d616ae66",
"Volumes": {
"/var/lib/mysql": {}
},
"WorkingDir": "",
"Entrypoint": [
"docker-entrypoint.sh"
],
"OnBuild": [],
"Labels": {}
},
"DockerVersion": "17.06.2-ce",
"Author": "",
"Config": {
"Hostname": "",
"Domainname": "",
"User": "",
"AttachStdin": false,
"AttachStdout": false,
"AttachStderr": false,
"ExposedPorts": {
"3306/tcp": {},
"33060/tcp": {}
},
"Tty": false,
"OpenStdin": false,
"StdinOnce": false,
"Env": [
"PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin",
"GOSU_VERSION=1.7",
"MYSQL_MAJOR=8.0",
"MYSQL_VERSION=8.0.13-1debian9"
],
"Cmd": [
"mysqld"
],
"ArgsEscaped": true,
"Image": "sha256:23ebb5c0c893698702420c945e5369aa52629db789e95842f2cd3473d616ae66",
"Volumes": {
"/var/lib/mysql": {}
},
"WorkingDir": "",
"Entrypoint": [
"docker-entrypoint.sh"
],
"OnBuild": [],
"Labels": null
},
"Architecture": "amd64",
"Os": "linux",
"Size": 485498316,
"VirtualSize": 485498316,
"GraphDriver": {
"Data": {
"LowerDir": "/var/lib/docker/overlay2/583680ec90f31ed394a467d3c91612296f069ccd2133ca45b0dcb58068dccf04/diff:/var/lib/docker/overlay2/f649115bf3ac6e0e5e17503bed5e78e975dec3615bac88ea9b0432219a7c1ce8/diff:/var/lib/docker/overlay2/77fc63bfb76f03747988fe09dd9fffa0c13aba88f85c0f368cf10638739c8071/diff:/var/lib/docker/overlay2/4ddcc1db7ca3b49586a8fdfff3d1d63cbc48365adcb3bb937f09a3c6146fe957/diff:/var/lib/docker/overlay2/9f47980ab98f0ffe0007e9f44033728be3a2420d81a482067864f4781d1818d0/diff:/var/lib/docker/overlay2/642735b745ea9cdac937537c84bdfe9122808f5640b38d9de3e163ffcecae18f/diff:/var/lib/docker/overlay2/c160d7f1c77a79c590614066fd48dcdf7aca85e2dcc34668e760ae252ada0156/diff:/var/lib/docker/overlay2/3049306261ab568a37a5ab857d215e2b69daf132ca40458ff71291aef2075f22/diff:/var/lib/docker/overlay2/60fc6392c3c204ca3d9a969ee01ada32f7f1f001988ebb41873c6ce8eca71844/diff:/var/lib/docker/overlay2/0d072123735d575c478c92f68612c789b93a0876104d557a6250154e06816930/diff:/var/lib/docker/overlay2/326ba80766e47bd00751d9d9b4d606aebe1193f81d788596d8f571017a71b38e/diff",
"MergedDir": "/var/lib/docker/overlay2/a17af512b0d0246e2c18f73da78f5141b528c887fc3b99e18575e282c4973a54/merged",
"UpperDir": "/var/lib/docker/overlay2/a17af512b0d0246e2c18f73da78f5141b528c887fc3b99e18575e282c4973a54/diff",
"WorkDir": "/var/lib/docker/overlay2/a17af512b0d0246e2c18f73da78f5141b528c887fc3b99e18575e282c4973a54/work"
},
"Name": "overlay2"
},
"RootFS": {
"Type": "layers",
"Layers": [
"sha256:237472299760d6726d376385edd9e79c310fe91d794bc9870d038417d448c2d5",
"sha256:ebdd1217991da1723b94e0aa2dc12f959b957d9214d6de7863d6be5a9e0de353",
"sha256:7332e314e51e07db5d294185f454382b08d4951058009b62f95812dc1e6ff431",
"sha256:2039058a7b75d71d4123d0f33856f7b06ecdec1c60adfd1a68f1e12ebbefeef0",
"sha256:cef03d1730e4b0696cb9226c2335e9bcd6389f7db1aa0dc600a61bc7430facde",
"sha256:3538e779fb2898bfa017aece266a65a1f90285d0811c0d719bf0a4c0dece3528",
"sha256:6ae996979161ac6ff3d87fd859b336e025abe62b476b9a6e31907d724c529f1b",
"sha256:36d8a4e1f37125ad57d023e4755a49a527e53deae3d48cecc649190367c3681b",
"sha256:57f27b056bcdaeec44e876e4b0cba6175b1fd0dc568237d249ff99f4c1297c7d",
"sha256:dba2ff51a506a513a7b7d1b2516bad55829597631e705541077b21f037c956f6",
"sha256:71f273c41192d85da4b4b8cb1d3820bfa83d25aa1f49b7f8a631b74f7e4bddbe",
"sha256:fd8775ba5e74d5b15c5f2ee28dba7f276478b05f9ad05f24957ded880e30ef66"
]
},
"Metadata": {
"LastTagTime": "0001-01-01T00:00:00Z"
}
}
]

In your .env file see the var called DATA_PATH_HOST and go to that folder, after that delete the mysql folder there and then do the docker-compose build --no-cache mysql

@vladyslavstartsev I have noticed that this won't always fix the problem, I've seen multiple macbooks simply ignoring the ${MYSQL_VERSION} variable, even if I set it manually (in the ARG MYSQL_VERSION=latest) to a different version e.g. 5.7
Behavior seen on:
MacOS Mojave
Version 10.14
it was fixed when I changed
FROM mysql:${MYSQL_VERSION}
to
FROM mysql:5.7
(but I forgot to delete the DATA_PATH_HOST folder, so I got another error, but that is not the problem, because once I updated the FROM mysql to my specific version, it pulled in the correct version instead  of the latest)
I expect it has something to do with macOS parsing the variable, but I have too little knowledge of macOS to verify this.

.env file
MYSQL_VERSION=5.7
and
run
docker-compose build mysql

None of above worked to me. I had to edit the first line of /mysql/Dockerfile, where mysql version was still set to latest.

@renandame
You should remove volume and data
docker volume rm laradock_mysql
sudo rm -rf ~/.laradock/data/mysql/

Simply I needed just only @pbtintas solution.
Its not need to edit mysql/dockerfile.
But, If you built once with 8.x, you have to remove cache, container and image correctly.
Below is the list of the sfuffs have to be removed.

MySQL host data specified by DATA_PATH_HOST in .env
The container you have built with docker-compose up..

like laradock_mysql not mysql

The Image you have built with docker-compose up..

like laradock_mysql not mysql

For those who are still struggling, beware with docker image and container caching.
Running docker-compose down and removing ~/.laradock/data/mysql is not enough.

Also if you tried in many ways to fix this problem, maybe you have made a mess into your containers and images list just like me. So I just removed all my containers and images.
You can get the commands from this link: https://www.digitalocean.com/community/tutorials/how-to-remove-docker-images-containers-and-volumes
JUST REMEMBER: YOU MUST REMOVE THE CONTAINERS BEFORE REMOVING THE IMAGES


add hypernova support

If you haven't heard of https://github.com/airbnb/hypernova - you should.
https://github.com/airbnb/hypernova-ruby - ruby client

@rstormsf Can you tell me more about why we'd use this with or instead of React on Rails?

it's not instead. it's in addition to

@rstormsf How about a doc file in /docs/additional-reading folder? Or maybe a PR? I'm not sure what you're suggesting? Any example repos? I'd love to hear more. Would you like a slack invite to our discussion room?


SAMZA-1131: RemoteApplicationRunner for cluster-based Samza applications

RemoteApplicationRunner starts the Samza StreamApplication on the remote cluster, e.g. Yarn. It uses ExecutionPlanner for planning physical execution, and JobRunner to start each stage of the application.

@navina @jmakes : please take a look. Thanks!


SQS ChangeMessageVisibilityBatch error code not logged

In our application, we are seeing occasional errors like:
Failed to change consumed SQS message '0xc0005266e8' visibility, response code: 0xc0005267a8
These look to be emitted here: https://github.com/Jeffail/benthos/blob/master/lib/input/reader/amazon_sqs.go#L233
Since only the pointer value is getting logged, there is not enough information to diagnose what is happening. I am opening this issue to improve the log line emitted to include the string value of the response code.
Note: I believe this happens in a few other places too.

Hey @zlozano, this should be fixed in https://github.com/Jeffail/benthos/releases/tag/v2.15.1 and future v3 release candidates.


Crash on startup

Luckily this is a test world, nothing crazy just a few connectors and a controller.
Crashlog:
https://puu.sh/uYrJf/ea1a3c5784.txt
xnet 1.1.2

Does this happen everytime? Also I would like to know exactly what blocks you were using. Also single or multiplayer?

Nope it only happened once, I was using blue connectors and a controller in conjunction with EnderIO stirling engines, a SAG mill and EIO Capacitor. Single player creative world.
MOD LIST:
https://puu.sh/uYSTR/c4dda5e27f.txt

I think I may have fixed this bug in next version.


Fix find command bugs

AND search not working within same prefixes (i.e. n/alex n/bernice should return a contact that has a name that contains alex and bernice)
Email search field should allow partial search (i.e<EMAIL_ADDRESS>or e/alexyeoh@ should be allowed)

find command for basic fields:

each prefix should be able to take in multiple strings (i.e. n/alex yeoh)
person descriptor message
AND/OR search should work the same as tags


Redis client cannot connect to Redis Server due to TLS cert error

I have this bit of code:
    console.log(`Creating Redis client`);
    const redisClient = createClient({
        url: process.env.REDIS_URL,
        socket: {
            tls: false, // ==> also tried 'true', but same issue
            rejectUnauthorized: false,
            requestCert: false,
        },
    });
    console.log(`Starting Redis connection`);
    await redisClient.connect();
    console.log(`Redis connection completed`);

process.env.REDIS_URL has a value of this format : rediss://$USERNAME:$PASSWORD@$HOST:$PORT/0
I get this error when I run it:
Creating Redis client
Starting Redis connection
Error: error:0909006C:PEM routines:get_name:no start line
    at node:internal/tls/secure-context:65:13
    at Array.forEach (<anonymous>)
    at setCerts (node:internal/tls/secure-context:63:3)
    at configSecureContext (node:internal/tls/secure-context:152:5)
    at Object.createSecureContext (node:_tls_common:117:3)
    at Object.connect (node:_tls_wrap:1621:48)
    at RedisSocket._RedisSocket_createTlsSocket (/home/project/backend/node_modules/@redis/client/dist/lib/client/socket.js:200:21)
    at /home/project/backend/node_modules/@redis/client/dist/lib/client/socket.js:161:101
    at new Promise (<anonymous>)
    at RedisSocket._RedisSocket_createSocket (/home/project/backend/node_modules/@redis/client/dist/lib/client/socket.js:159:12) {
  library: 'PEM routines',
  function: 'get_name',
  reason: 'no start line',
  code: 'ERR_OSSL_PEM_NO_START_LINE'
}
[nodemon] app crashed - waiting for file changes before starting...

I don't understand why do I have this error even though I'm not using any certificate at all...
Environment:

Node.js Version: v16.13.2
Redis Server Version: 6
Node Redis Version: 4.5.1
Platform: Ubuntu 20.04.3

@leibale : whether I use redis or rediss, it's the same...

I'm confused.. wanna debug it together?
I'll be here for the next 15 minutes

@younes-io did you manage to solve it?

@leibale Not really, I just switched to the older version of the package :/

I started with the Heroku docs on connecting with TLS in Node. They propose the following approach:
const redis = require("redis");

const client = redis.createClient({
  url: process.env.REDIS_URL,
  tls: {
    rejectUnauthorized: false
  }
});

Which led to the error:
2023-03-27T19:14:27.572166+00:00 app[web.1]: Redis Client Error Error: self-signed certificate in certificate chain
2023-03-27T19:14:27.572185+00:00 app[web.1]: at TLSSocket.onConnectSecure (node:_tls_wrap:1540:34)
2023-03-27T19:14:27.572186+00:00 app[web.1]: at TLSSocket.emit (node:events:513:28)
2023-03-27T19:14:27.572186+00:00 app[web.1]: at TLSSocket._finishInit (node:_tls_wrap:959:8)
2023-03-27T19:14:27.572186+00:00 app[web.1]: at ssl.onhandshakedone (node:_tls_wrap:743:12) {
2023-03-27T19:14:27.572186+00:00 app[web.1]: code: 'SELF_SIGNED_CERT_IN_CHAIN'
It took me too long to consider that Heroku would have a non-working / dated pattern on their help but eventually looked at the node-redis config page
The pattern there for self-signed certs worked, though I dropped the cert: '...' property as follows.
createClient({
  socket: {
    tls: true,
    rejectUnauthorized: false,
  }
});

I didn't open a ticket for Heroku to update their docs because I'm not sure that approach couldn't/shouldn't work in some context other than my own.
If anyone can shed some light on how even the self-signed cert seems to be optional I'd be interested to learn how that's handled on Heroku.

@builder173 thanks for the heads-up, I'll ask Heroku to update their documentation :)

Thanks for reaching out to Heroku.
I'll try to add some structure to the self-signed cert question. I'm basically trying to figure out why my connection worked without specifying the cert: '...' property that appears in the node-redis examples. Looking through to the tls.connect() docs, perhaps this is only needed when server verifies the client certificate in 2-way ssl? (and apparently Heroku redis doesn't?)
//From tls.connect() docs
const options = {
  // Necessary only if the server requires client certificate authentication.
  key: fs.readFileSync('client-key.pem'),
  cert: fs.readFileSync('client-cert.pem'),

@builder173 because you have rejectUnauthorized set to false, which means the certificate is not verified... You can read more about that here.

So then can/should the cert: '...' property be removed from the last example in the node-redis config page? If I understand what you're saying, you wouldn't specify that property together with rejectUnauthorized: false.

Please share if there is any solution for this issue.
I am getting similar issue after upgrading my redis module in express.js app and heroku redis addon version to latest ones.
Here is my issue details - https://github.com/redis/node-redis/issues/2540


Ledger app shows Decrypt on initial shielding - Add to documentation why

Bug Description
Shielding some amount of CCDs even if I have 0 on shielded amount would show that I have to Decrypt amount in the Ledger app.
This was a bit confusing and raised a question for me, what am I decrypting right now?
Steps to Reproduce
Make sure you have 0 on shielded balance.
Shield 50 CCDs.
Expected Result
Since I have to sign Decrypt in the Ledger app, official docs could elaborate why or what we have to decrypt in this case.
Actual Result
No documentation .
Versions

Software Version: V 1.4.0-beta.0

@dg-concordium Can you coordinate adding this to documentation.

Yes.


FAQ - How to clone files from git repository within a Shell script step?

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[x] Feature
[ ] Maintenance/Chore

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Endpoint resolution v2: builtin parameter values

This PR is part of the ongoing endpoint resolution v2 work, together with #2737, #2740, #2751, #2747. It contains the code that assigns values to a list of "builtin parameters" that may be referenced by endpoint rulesets. It's glue code between the existing ClientCreator and ClientArgsCreator and the future EndpointProvider (see #2737).
Q: By itself, this PR adds only unused code. Why is it a separate PR? — A: I decided to break this out of a larger PR with all the middleware for calling EndpointProvider to keep this a separate review/discussion. Even though the code is pretty simple, there could be a complex discussion hiding behind each parameter assignment and how it is tested. Let's get these debates out of the way first :)

Codecov Report

Merging #2759 (6860f62) into develop (6d4551e) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop    #2759      +/-   ##
===========================================
- Coverage    95.13%   95.08%   -0.05%
===========================================
  Files           61       61
  Lines        12538    12560      +22
===========================================
+ Hits         11928    11943      +15
- Misses         610      617       +7

Impacted Files
Coverage Δ

botocore/args.py
100.00% <100.00%> (ø)

botocore/regions.py
95.34% <100.00%> (+0.38%)
:arrow_up:

botocore/compat.py
91.27% <0.00%> (-1.35%)
:arrow_down:

botocore/httpsession.py
91.33% <0.00%> (-0.40%)
:arrow_down:

botocore/utils.py
93.36% <0.00%> (-0.22%)
:arrow_down:

botocore/credentials.py
98.54% <0.00%> (-0.11%)
:arrow_down:

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


[FR] Selecting tags in the App should show only matching samples

It would be handy (and consistent with the other recent "only matches" changes to the fields sidebar) if selecting tags in the App would only show samples with the matching tag.

Resolved by https://github.com/voxel51/fiftyone/pull/926 and available as of fiftyone==0.8.0


Double advancement announcement

When a player gets one of the custom advancements that are added to HCCore, they are announced in chat twice:

it didnt with the diamond ones?

Issue with my testing server, no issue in HCCore


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/469 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-yml-archive-non-versioning-archive.git

Docs Build status updates of commit 9894c88:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/index.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/sql-database-engine.png
:white_check_mark:Succeeded

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


chore: update deno_unsync to 0.10

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

Project coverage is 81.69%. Comparing base (0c7f83e) to head (74369b9).
Report is 54 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #815      +/-   ##
==========================================
+ Coverage   81.43%   81.69%   +0.25%
==========================================
  Files          97       98       +1
  Lines       23877    24324     +447
==========================================
+ Hits        19445    19872     +427
- Misses       4432     4452      +20

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


Fix/data visualizer temp range

Small fixes of the temperature visualizer. It now shows the correct temperature range.

Lars has approved these changes.


Pre commit hooks and Samples update

As mentioned in #82 this follow-up PR ensures:

samples are updated with code on the branch
pre-commit are updated and all the repo is re-validated against it
ensure that pre-commit does not alter the code generated by the tool

In order to get the pre-commit hooks and samples updated I'm removing from this PR the pre-commit exclusion of the samples.
I'll address https://github.com/Yelp/swagger-gradle-codegen/pull/86#discussion_r371744767 into a subsequent PR (the new PR will mostly focus on having mustache generate a "nicer" code that will make ktlint a bit happier).

ensure that pre-commit does not alter the code generated by the tool

Please update the description of this PR


Worker side of simple sidechain finality

closes #922

updates docker image to 1.0.16

@mullefel all tests pass now. Can you do a final review and a merge?

@haerdib and @clangenb : not sure if you want to do a review, but you can

closes #922

I disabled the unit tests that were failing repeatedly because an external source, that they depended on, was not available. I created a new issue, #985, to fix this.


Automatic Table of Contents on Publish.Obsidian.md

Dear all,
The ability to limit minLevel or maxLevel is very useful.
Is it possible to apply the function to Published webpages and mobile?
style: nestedList # TOC style (nestedList|inlineFirstLevel)
minLevel: X # Include headings from the specified level
maxLevel: X # Include headings up to the specified level
includeLinks: true # Make headings clickable
debugInConsole: false # Print debug info in Obsidian console

Thank you!

Hi,
I'm not a Publish user myself and can't test; what is the current behavior?

Not sure about publish, but it works just fine on mobile. Good extension

Hi!
I am also really interested to you use the plugin with the publish service. As you can see on the screenshot only the code is printed but there is no "translation" to HTML.

what is the current behavior?

Same as Mada77: only showing the script.
Thanks!!


Noble connects to device multiple times

The first time I scan and connect to my device, everything works as expected. Then when I try again, it connects to the peripheral twice. Then three times, etc.. Is there an explanation for this behaviour?
import noble from 'noble';

export function startScanning() {
  return noble.startScanning([], true);
}

export function checkIn(name, callback) {
  return noble.on('discover', (peripheral) => {
    if (peripheral.advertisement.manufacturerData) {
      const mode = Array.prototype.slice.call(peripheral.advertisement.manufacturerData, 0)[4];
      if (mode === 4 && peripheral.rssi > -45) {
        noble.stopScanning();
        peripheral.connect();
        peripheral.once('connect', () => {
          console.log('Connected');
          peripheral.discoverServices('7b00aaaa4be04c349e1e91ec7b084af5');

          peripheral.once('servicesDiscover', (services) => {
            const service = services[0];
            service.discoverCharacteristics('7b00aaa34be04c349e1e91ec7b084af5');
            service.once('characteristicsDiscover', (characteristics) => {
              const characteristic = characteristics[0];
              characteristic.subscribe();
              peripheral.writeHandle(0x0025, new Buffer([0x02, 0x00]));
              peripheral.writeHandle(0x0024, new Buffer('8DF2BFE08299810760EF006EAD486952B2BA38D7', 'hex'));

              characteristic.on('data', (data) => { // eslint-disable-line
                if (data[0] === 0x01) {
                  const str = name;
                  const nameBuffer = Buffer.allocUnsafe(str.length);
                  for (let i = 0; i < str.length; i += 1) {
                    nameBuffer[i] = str.charCodeAt(i);
                  }
                  peripheral.writeHandle(0x001D, nameBuffer);
                } else if (data[0] === 0x02) {
                  return callback({
                    statusCode: 500,
                    message: 'Wrong passkey entry',
                  });
                } else if (data[0] === 0x10) {
                  peripheral.disconnect();
                  return callback({
                    statusCode: 200,
                    message: 'Check in successful',
                    deviceAddress: peripheral.address,
                    info: name,
                  });
                }
              });
            });
          });
        });
      }
    }
  });
}

How often do you invoke the checkIn function?
If you invoke it multiple times, the discover handler will be registered for each invocation.
This would lead to the behaviour you described.

@boazdejong please see @manuelmitasch's comment above and re-open with your answer.
@manuelmitasch thanks for helping out!


cla signatures to current repo

Codecov Report

Merging #66 (39a284d) into main (865bdde) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #66   +/-   ##
=======================================
  Coverage   89.89%   89.89%
=======================================
  Files          15       15
  Lines         366      366
=======================================
  Hits          329      329
  Misses         26       26
  Partials       11       11

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 865bdde...39a284d. Read the comment docs.


[MINOR][ML] Fixed MLlib build warnings

What changes were proposed in this pull request?
Fixes to eliminate warnings during package and doc builds.
How was this patch tested?
Existing unit tests

Previous warnings:
[info] Compiling 148 Scala sources and 60 Java sources to /Users/josephkb/spark/mllib/target/scala-2.11/test-classes...
[warn] /Users/josephkb/spark/mllib/src/test/scala/org/apache/spark/ml/tree/impl/RandomForestSuite.scala:328: match may not be exhaustive.
[warn] It would fail on the following input: ContinuousSplit()
[warn]       case n: InternalNode => n.split match {
[warn]                                 ^
[warn] /Users/josephkb/spark/mllib/src/test/scala/org/apache/spark/ml/tree/impl/RandomForestSuite.scala:327: match may not be exhaustive.
[warn] It would fail on the following input: LeafNode()
[warn]     model.rootNode match {
[warn]           ^
[warn] /Users/josephkb/spark/mllib/src/test/scala/org/apache/spark/ml/tree/impl/RandomForestSuite.scala:351: match may not be exhaustive.
[warn] It would fail on the following input: LeafNode()
[warn]     def getChildren(rootNode: Node): Array[InternalNode] = rootNode match {
[warn]                                                            ^
[warn] three warnings found

 /Users/josephkb/spark/examples/src/main/scala/org/apache/spark/examples/mllib/AbstractParams.scala:41: method declarations in class TypeApi is deprecated: Use `decls` instead
[warn]     val allAccessors = tpe.declarations.collect {
[warn]                            ^
[warn] 82 warnings found
[info] Note: /Users/josephkb/spark/examples/src/main/java/org/apache/spark/examples/mllib/JavaStratifiedSamplingExample.java uses unchecked or unsafe operations.
[info] Note: Recompile with -Xlint:unchecked for details.
[warn] /Users/josephkb/spark/examples/src/main/scala/org/apache/spark/examples/mllib/AbstractParams.scala:41: method declarations in class TypeApi is deprecated: Use `decls` instead
[warn]     val allAccessors = tpe.declarations.collect {
[warn]

Test build #55385 has started for PR 12263 at commit c8c2dcb.

Test build #55385 has finished for PR 12263 at commit c8c2dcb.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/55385/
Test PASSed.

Test build #55535 has started for PR 12263 at commit 15bd0ac.

Test build #55535 has finished for PR 12263 at commit 15bd0ac.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/55535/
Test PASSed.

Merged build finished. Test PASSed.


[SPARK-47665][SQL] Use SMALLINT to Write ShortType to MYSQL

What changes were proposed in this pull request?

This PR uses MySQL SMALLINT to write ShortType to MYSQL, as we read SMALLINT as ShortType.
Why are the changes needed?

SMALLINT fits better for ShortType and plays roundtrip for r/w.
Does this PR introduce any user-facing change?
yes, type mapping changes
How was this patch tested?
new tests
Was this patch authored or co-authored using generative AI tooling?
no

Thank you @dongjoon-hyun

Thank you @dongjoon-hyun. Merged to master.
PS: The impact on existing workloads is trivial, as users are still able to write shorts to pre-defined int-tables


Check the render method of TwentyFourHoursMode.

Error:
Warning: React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in.
Check the render method of TwentyFourHoursMode.
in TwentyFourHoursMode (created by AsyncComponent)
in AsyncComponent (created by MaterialTheme)
in div (created by MaterialTheme)
in MaterialTheme (created by AsyncComponent)
in AsyncComponent (created by TimePicker)
in div (created by OutsideClickHandler)
in OutsideClickHandler (created by TimePicker)
in div (created by TimePicker)
in TimePicker
Code:
import React from 'react';
import ReactDOM from 'react-dom';
import TimePicker from 'react-times';
import 'react-times/css/material/default.css';
ReactDOM.render(, document.getElementById('app'));
Also tried by adding parameters to use 12 hours but then the error changes to
Check the render method of TwelveHoursMode.

Updating some of my dependencies already fixed it.

I am facing the same issue, how you fixed it?
I am trying this component 1st time.

@rizwan-ishtiaq
hi, maybe you can check your react version


rbd: image Open function is non-idiomatic Go

The rbd module's Image.Open function is rather non-idiomatic for Go code. It accepts any number of interface{} arguments and then loops on them checking (only!) the type and uses those to select which ceph api function to run.
This means that x.Open("foo", true) and x.Open(true, "foo") are equivalent, as well as x.Open("foo", "foo", false, "bar", true, false, "foo", true). I've never seen go code like that! :-)
I'm not sure if this function wanted to mimic the style of the python api or what but it strikes me as odd.
It would be preferable to create more idiomatic approaches to Image type and the rbd open functions and then deprecate this approach to Open.

I'm not familiar about librbd :( but I think I can tell about what I think.

I'm agreed about your opinion, I'm confused about Open method's parameter when I use go-ceph first
According to the tests and sources, c_snap_name (string) and read_only (boolean) are the only required parameters for the Open method, And they are can be optional parameter. ("", false).
I think there are some options for idiomatic Go

We can create two function Open, OpenReadOnly for the feature because golang doesn't support function overloading. Maybe two function have string parameter for c_snap_name
Create struct for Open parameter

in gophercloud, there are various option struct for support openstack api parameters

server, err := servers.Create(client, servers.CreateOpts{
  Name:      "My new server!",
  FlavorRef: "flavor_id",
  ImageRef:  "image_id",
}).Extract()

Apply more declarative parameters. like Open(readOnly bool, snapName string)

I've started working on this. PR coming soon(ish).


PostgreSQL 17 support

Hello!
I would like to request pg17 support for clickhouse_fdw.
When I build against PG17 using the latest released version (1.4.0), I get the following error:
[ 85%] Building C object src/CMakeFiles/clickhouse_fdw.dir/connection.c.o
<SNIP>
clickhouse_fdw/src/clickhouse_fdw.c: In function 'clickhouseGetForeignPaths':
clickhouse_fdw/src/clickhouse_fdw.c:514:15: error: too few arguments to function 'create_foreignscan_path'
  514 |         path= create_foreignscan_path(root, baserel, NULL,
      |               ^~~~~~~~~~~~~~~~~~~~~~~
In file included from clickhouse_fdw/src/clickhouse_fdw.c:32:
/usr/pgsql-17/include/server/optimizer/pathnode.h:125:21: note: declared here
  125 | extern ForeignPath *create_foreignscan_path(PlannerInfo *root, RelOptInfo *rel,
      |                     ^~~~~~~~~~~~~~~~~~~~~~~
clickhouse_fdw/src/clickhouse_fdw.c: In function 'clickhouseBeginForeignScan':
clickhouse_fdw/src/clickhouse_fdw.c:771:21: error: 'RangeTblEntry' has no member named 'checkAsUser'
  771 |         userid = rte->checkAsUser ? rte->checkAsUser : GetUserId();
      |                     ^~
clickhouse_fdw/src/clickhouse_fdw.c:771:40: error: 'RangeTblEntry' has no member named 'checkAsUser'
  771 |         userid = rte->checkAsUser ? rte->checkAsUser : GetUserId();
      |                                        ^~
clickhouse_fdw/src/clickhouse_fdw.c: In function 'create_foreign_modify':
clickhouse_fdw/src/clickhouse_fdw.c:1380:21: error: 'RangeTblEntry' has no member named 'checkAsUser'
 1380 |         userid = rte->checkAsUser ? rte->checkAsUser : GetUserId();
      |                     ^~
clickhouse_fdw/src/clickhouse_fdw.c:1380:40: error: 'RangeTblEntry' has no member named 'checkAsUser'
 1380 |         userid = rte->checkAsUser ? rte->checkAsUser : GetUserId();
      |                                        ^~
clickhouse_fdw/src/clickhouse_fdw.c: In function 'add_paths_with_pathkeys_for_rel':
clickhouse_fdw/src/clickhouse_fdw.c:1842:42: error: too few arguments to function 'create_foreignscan_path'
 1842 |                                          create_foreignscan_path(root, rel,
      |                                          ^~~~~~~~~~~~~~~~~~~~~~~
<SNIP... but there are more errors>
make: *** [Makefile:136: all] Error 2

I am compling on an Alma9 - x86_64 Linux box.  I am unsure what changes would be required, so I thought I would raise the issue.
Thanks!

Hello, having a similar set of troubles when compiling on a ubuntu22.04 for PostgreSQL 17
[ 47%] Building C object src/CMakeFiles/clickhouse_fdw.dir/clickhouse_fdw.c.o cc1: warning: ‘-fstack-check=’ and ‘-fstack-clash_protection’ are mutually exclusive; disabling ‘-fstack-check=’ In file included from clickhouse_fdw-master/src/clickhouse_fdw.c:38: clickhouse_fdw-master/src/clickhouse_fdw.c: In function ‘clickhousedb_raw_query’: /usr/include/postgresql/17/server/fmgr.h:332:50: warning: passing argument 1 of ‘DatumGetPointer’ makes integer from pointer without a cast [-Wint-conversion]   332 | #define DatumGetTextP(X)                        ((text *) PG_DETOAST_DATUM(X))       |                                                 ~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~       |                                                  |       |                                                  text * {aka struct varlena *} /usr/include/postgresql/17/server/utils/builtins.h:98:73: note: in definition of macro ‘TextDatumGetCString’    98 | #define TextDatumGetCString(d) text_to_cstring((text *) DatumGetPointer(d))       |                                                                         ^ /usr/include/postgresql/17/server/fmgr.h:336:49: note: in expansion of macro ‘DatumGetTextP’   336 | #define PG_GETARG_TEXT_P(n)                     DatumGetTextP(PG_GETARG_DATUM(n))       |                                                 ^~~~~~~~~~~~~ clickhouse_fdw-master/src/clickhouse_fdw.c:299:48: note: in expansion of macro ‘PG_GETARG_TEXT_P’   299 |         char *connstring = TextDatumGetCString(PG_GETARG_TEXT_P(1)),       |                                                ^~~~~~~~~~~~~~~~ In file included from clickhouse_fdw-master/src/clickhouse_fdw.c:14: /usr/include/postgresql/17/server/postgres.h:312:23: note: expected ‘Datum’ {aka ‘long unsigned int’} but argument is of type ‘text *’ {aka ‘struct varlena *’}   312 | DatumGetPointer(Datum X)       |                 ~~~~~~^ In file included from clickhouse_fdw-master/src/clickhouse_fdw.c:38: /usr/include/postgresql/17/server/fmgr.h:332:50: warning: passing argument 1 of ‘DatumGetPointer’ makes integer from pointer without a cast [-Wint-conversion]   332 | #define DatumGetTextP(X)                        ((text *) PG_DETOAST_DATUM(X))       |                                                 ~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~       |                                                  |       |                                                  text * {aka struct varlena *} /usr/include/postgresql/17/server/utils/builtins.h:98:73: note: in definition of macro ‘TextDatumGetCString’    98 | #define TextDatumGetCString(d) text_to_cstring((text *) DatumGetPointer(d))       |                                                                         ^ /usr/include/postgresql/17/server/fmgr.h:336:49: note: in expansion of macro ‘DatumGetTextP’   336 | #define PG_GETARG_TEXT_P(n)                     DatumGetTextP(PG_GETARG_DATUM(n))       |                                                 ^~~~~~~~~~~~~ clickhouse_fdw-master/src/clickhouse_fdw.c:300:47: note: in expansion of macro ‘PG_GETARG_TEXT_P’   300 |                  *query = TextDatumGetCString(PG_GETARG_TEXT_P(0));       |                                               ^~~~~~~~~~~~~~~~ In file included from clickhouse_fdw-master/src/clickhouse_fdw.c:14: /usr/include/postgresql/17/server/postgres.h:312:23: note: expected ‘Datum’ {aka ‘long unsigned int’} but argument is of type ‘text *’ {aka ‘struct varlena *’}   312 | DatumGetPointer(Datum X)       |                 ~~~~~~^ clickhouse_fdw-master/src/clickhouse_fdw.c: In function ‘clickhouseGetForeignPaths’: clickhouse_fdw-master/src/clickhouse_fdw.c:514:15: error: too few arguments to function ‘create_foreignscan_path’   514 |         path= create_foreignscan_path(root, baserel, NULL,       |               ^~~~~~~~~~~~~~~~~~~~~~~ In file included from clickhouse_fdw-master/src/clickhouse_fdw.c:32: /usr/include/postgresql/17/server/optimizer/pathnode.h:125:21: note: declared here   125 | extern ForeignPath *create_foreignscan_path(PlannerInfo *root, RelOptInfo *rel,       |                     ^~~~~~~~~~~~~~~~~~~~~~~ clickhouse_fdw-master/src/clickhouse_fdw.c: In function ‘add_paths_with_pathkeys_for_rel’: clickhouse_fdw-master/src/clickhouse_fdw.c:1850:42: error: too few arguments to function ‘create_foreignscan_path’  1850 |                                          create_foreignscan_path(root, rel,       |                                          ^~~~~~~~~~~~~~~~~~~~~~~ In file included from clickhouse_fdw-master/src/clickhouse_fdw.c:32: /usr/include/postgresql/17/server/optimizer/pathnode.h:125:21: note: declared here   125 | extern ForeignPath *create_foreignscan_path(PlannerInfo *root, RelOptInfo *rel,       |                     ^~~~~~~~~~~~~~~~~~~~~~~ clickhouse_fdw-master/src/clickhouse_fdw.c:1861:42: error: too few arguments to function ‘create_foreign_join_path’  1861 |                                          create_foreign_join_path(root, rel,       |                                          ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from clickhouse_fdw-master/src/clickhouse_fdw.c:32: /usr/include/postgresql/17/server/optimizer/pathnode.h:133:21: note: declared here   133 | extern ForeignPath *create_foreign_join_path(PlannerInfo *root, RelOptInfo *rel,       |                     ^~~~~~~~~~~~~~~~~~~~~~~~ clickhouse_fdw-master/src/clickhouse_fdw.c: In function ‘clickhouseGetForeignJoinPaths’: clickhouse_fdw-master/src/clickhouse_fdw.c:2019:20: error: too few arguments to function ‘create_foreign_join_path’  2019 |         joinpath = create_foreign_join_path(root,       |                    ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from clickhouse_fdw-master/src/clickhouse_fdw.c:32: /usr/include/postgresql/17/server/optimizer/pathnode.h:133:21: note: declared here   133 | extern ForeignPath *create_foreign_join_path(PlannerInfo *root, RelOptInfo *rel,       |                     ^~~~~~~~~~~~~~~~~~~~~~~~ clickhouse_fdw-master/src/clickhouse_fdw.c: In function ‘add_foreign_grouping_paths’: clickhouse_fdw-master/src/clickhouse_fdw.c:2390:21: error: too few arguments to function ‘create_foreign_upper_path’  2390 |         grouppath = create_foreign_upper_path(root,       |                     ^~~~~~~~~~~~~~~~~~~~~~~~~ In file included from clickhouse_fdw-master/src/clickhouse_fdw.c:32: /usr/include/postgresql/17/server/optimizer/pathnode.h:141:21: note: declared here   141 | extern ForeignPath *create_foreign_upper_path(PlannerInfo *root, RelOptInfo *rel,       |                     ^~~~~~~~~~~~~~~~~~~~~~~~~ clickhouse_fdw-master/src/clickhouse_fdw.c: In function ‘add_foreign_ordered_paths’: clickhouse_fdw-master/src/clickhouse_fdw.c:2528:24: error: too few arguments to function ‘create_foreign_upper_path’  2528 |         ordered_path = create_foreign_upper_path(root,       |                        ^~~~~~~~~~~~~~~~~~~~~~~~~ In file included from clickhouse_fdw-master/src/clickhouse_fdw.c:32: /usr/include/postgresql/17/server/optimizer/pathnode.h:141:21: note: declared here   141 | extern ForeignPath *create_foreign_upper_path(PlannerInfo *root, RelOptInfo *rel,       |                     ^~~~~~~~~~~~~~~~~~~~~~~~~ clickhouse_fdw-master/src/clickhouse_fdw.c: In function ‘add_foreign_final_paths’: clickhouse_fdw-master/src/clickhouse_fdw.c:2673:22: error: too few arguments to function ‘create_foreign_upper_path’  2673 |         final_path = create_foreign_upper_path(root,       |                      ^~~~~~~~~~~~~~~~~~~~~~~~~ In file included from clickhouse_fdw-master/src/clickhouse_fdw.c:32: /usr/include/postgresql/17/server/optimizer/pathnode.h:141:21: note: declared here   141 | extern ForeignPath *create_foreign_upper_path(PlannerInfo *root, RelOptInfo *rel,       |                     ^~~~~~~~~~~~~~~~~~~~~~~~~ make[2]: *** [src/CMakeFiles/clickhouse_fdw.dir/build.make:76: src/CMakeFiles/clickhouse_fdw.dir/clickhouse_fdw.c.o] Error 1 make[1]: *** [CMakeFiles/Makefile2:200: src/CMakeFiles/clickhouse_fdw.dir/all] Error 2 make: *** [Makefile:136: all] Error 2


Use hyphen minus (U+002D) instead of math minus (U+2212) in ranges?

Bikeshed automatically converts minus signs made with hyphens (U+002D) that appear in ranges to mathematical minus sign  (U+2212) characters and I'm wondering whether that's really intended.
That is typically what happens with the value of font-style in CSS Fonts, the hyphen in the "angle" range becomes a mathematical minus sign in the generated spec. Before/After comparison:
<angle [-90deg,90deg]>
<angle [−90deg,90deg]>

One other similar occurrence for the definition of <grid-line> in CSS Grid. Both occurrences seem to have been created by recent changes, I'm not aware of other places in CSS specs that have ranges with negative numbers.
Using that minus sign character is not incorrect per se, it's just that CSS Values talks about the hyphen minus when it says "The first digit of an integer may be immediately preceded by - or + to indicate the integer’s sign" and CSS parsers that can also parse the value definition syntax typically fail to parse the U+2212 minus.

I went ahead and did this for simplicity, since I happened to write code that serialized numbers with a hyphen-minus and didn't want to go out of my way to fix it back.


Make hardcoded strings localizable

Feature request type
Enhance component
Component name
All components
Is your feature request related to a problem?
Currently, some string properties are hardcoded within the source code in English. While for some components there are properties to override the default text, this is not the case everywhere (for example: The column options always show "Ungroup", "Unhide", etc.).
This makes it impossible to provide a localized UI in non-English-speaking countries.
Describe the solution you'd like
Two possible solutions come to my mind:

Provide properties for all hardcoded strings
Preferred solution: Rely on standard localization features to centralize string localizations

That way, the community could also contribute to translating the library to other languages.
Have you seen this feature anywhere else?
No response
Describe alternatives you've considered
No response
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I think it would be enough if you had a static class of texts, which you can overwrite when starting or changing the language.

I think it would be much better to rely on standard mechanisms (or extend them, for example like BootstrapBlazor does with its JSON localizations.


Add assert_sum to example file

The assert_sum is currently not in the example file. We should add it there to have an up to date representation of the functionality of the SP.

@czepluch
It is here in L91
https://github.com/raiden-network/scenario-player/blob/48d2d74f2bfe113ea8801abc2e972d4eb54312d1/examples/scenario-example-v2.yaml#L77-L92
Does it need more explanation?


Programming exercises: Improve commit history view

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] Important: I implemented the changes with a very good performance and prevented too many (unnecessary) database calls.
[x] I strictly followed the server coding and design guidelines.
[x] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[x] I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
[x] I documented the Java code using JavaDoc style.

Client

[x] Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data.
[x] I strictly followed the client coding and design guidelines.
[x] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[x] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[x] I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
[x] I documented the TypeScript code using JSDoc style.
[x] I added multiple screenshots/screencasts of my UI changes.
[x] I translated all newly inserted strings into English and German.

Changes affecting Programming Exercises

[ ] High priority: I tested all changes and their related features with all corresponding user types on a test server configured with the integrated lifecycle setup (LocalVC and LocalCI).
[ ] I tested all changes and their related features with all corresponding user types on a test server configured with Gitlab and Jenkins.

Motivation and Context
Currently, the Commit History view displays full-length commit hashes and presents email addresses as if they are clickable, even though they are not. It also shows the author attribution based on the git author, which can be manipulated. Furthermore, commits are presented individually, even when they are pushed together.
#8472
Description

Display Short Commit Hashes: Default to displaying only the first 7 characters of commit hashes, as this seems to be a common practice for displaying hashes in short form. The full hash is available on hover for detailed reference.
Email Address Display: Change the styling of email addresses to match regular text, clarifying that they are not interactive.
Author Identification: Utilize the Artemis database to accurately display commit authorship by mapping the participation ID to the stored user data. For teams we default to the git author and email.
Group and Order Commits: Group commits by push, showing commits together that were pushed in one batch. As we don't have a direct indicator whether commits were pushed together, we infer groups based on the presence of a 'result' on a commit, displaying a single result to the entire group for better clarity.

Steps for Testing

Prerequisites:

Log in to Artemis as a Student or anything above
Navigate to the Commit History View of a programming exercise
Check if he following changes are implemented and function as expected:

a. Short Commit Hashes: Confirm that only the first 7 characters of the commit hashes are displayed by default in the Commit History View. Hover over these short hashes to ensure the full hash appears.
b. Email Address Display: Verify that email addresses in the commit history no longer appear as hyperlinks. They should be styled as regular text, clearly indicating that they are not interactive.
c. Author Identification:

Check if the commit author name displayed, correctly matches the data from the Artemis database. Each commit should list the author's name according to the participationId linked to the account user data, not merely the git author.
If logged in as a user, push a commit using the --author option to change the author information beforehand. Verify that despite this, the commit still correctly displays the author based on the account name linked to the participationId.

d. Group and Order Commits:

Confirm that commits pushed in a batch are grouped with a single result, and that both individual commits and the groups themselves are ordered correctly by their commit timestamps.
If logged in as a user, make multiple commits locally and then push them all at once in a batch. Check if these commits are correctly grouped under the same push event. Verify also if it is possible to expand and collapse the batch to view individual commit details.

Testserver States

[!NOTE]
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked

Review Progress

Performance Review

[ ] I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance
[ ] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance

Code Review

[x] Code Review 1
[ ] Code Review 2

Manual Tests

[x] Test 1
[x] Test 2

Exam Mode Test

[ ] Test 1
[ ] Test 2

Test Coverage

Client

Class/File
Line Coverage
Confirmation (assert/expect)

programming-exercise-participation.service.ts
93.87%
✅ ❌

commits-info-group.component.ts
100%
✅

commits-info-row.component.ts
100%
✅

commits-info.component.ts
95.91%
✅ ❌

Server

Class/File
Line Coverage
Confirmation (assert/expect)

ProgrammingExerciseParticipationResource.java
63%
✅ ❌

Screenshots

Batch 3 expanded

Summary by CodeRabbit

New Features

Added an endpoint to retrieve the user associated with a programming exercise participation, with relevant authorization checks.
Introduced conditional rendering for commit information rows based on expansion state in the UI.

Enhancements

Updated components to display detailed commit information, including UI elements for better user interaction.

Tests

Added and updated unit tests for new and existing components to ensure proper functionality and coverage.

@LeonWehrhahn any update on this PR?

As discussed, the Commit History View now features "Expand All" and "Collapse All" buttons for the commit groups, which are expanded by default. Each commit is now numbered, with those within a commit group (belonging to a push) following the format < commitGroupNumber>.. Additionally, the icons for expanding and collapsing individual groups have been updated from ^ and v to < and v.
Default View (All Expanded):

All Collapsed but with an individual row (3) expanded:


Refactor select_statement and (stacked PR) flatten select after filter/orderby

Please answer these questions before submitting your pull requests. Thanks!

What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
Fixes #NNNN

Fill out the following pre-review checklist:

[ ] I am adding a new automated test(s) to verify correctness of my new code
[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency

Please describe how your code solves the related issue.
Refactors part of select_statements.py to get rid of some flag variables.

The simplifier logic looks good. But pls check why code coverage is lowered.

The simplifier logic looks good. But pls check why code coverage is lowered.

Test coverage is lowered since we do not test negating the parameter guard flatten_select_after_filter_and_orderby, which should be fine since we plan to remove the temporary parameter anyway.


Bug Network

Bug solution Network

@eka12 Hi, can you provide more details?

6/27/22 Core Devs Call:

Closing issue


Main view refresh removing 'Do you want to use trezor ?' alert on Wallet load report Finish

On the wallet load form (for a soft wallet) I plugged in a trezor. It now does not switch to trezor PIN entry (bug fix for #352). You see a 'Do you want to use this trezor?' alert in the background (which is fine) but when you dismiss the Wallet load report with a finish the alert disappears.
Most probably something to do with the MainView#refresh not caching/ replaying the alert

I  have tweaked it so that when the MainController#hiseCredentials is called (only) the last hardwareWalletEvent is repeated. This is NOT in the mainView#refresh so won't get called after languges changes etc to minimise false positives.
Awaiting review and closing

The existing code didn't include the message body so caused an empty alert. I've fixed this now and it's all working as required.
Closing.


Please approve PR

Hello.
@makarid
Please confirm 3 PR .
https://github.com/knowm/XChange/pull/4669
https://github.com/knowm/XChange/pull/4670
https://github.com/knowm/XChange/pull/4671
If you have questions about them, then explain what

The #4671 still has merge conflicts ?

Yes, it does.

failed to resolve the conflict.
Removed the PR and made a new one
https://github.com/knowm/XChange/pull/4697


GNU test CI needs some more locale work

I fixed some issues in https://github.com/uutils/coreutils/pull/2753 for the tr PR ( https://github.com/uutils/coreutils/pull/2502 )
but we still have this one
./tests/misc/sort-float.sh: line 58: warning: setlocale: LC_ALL: cannot change locale (none): No such file or directory
Core: https://github.com/coreutils/coreutils/blob/master/tests/misc/sort-float.sh#L58

Actually, i did it here:
https://github.com/uutils/coreutils/pull/2753/files


How to calculate the Box Accuracy in Fig 4a?

Anybody knows how to calculate this metric?
Thanks!

Hi @GeoffreyChen777 , sorry for the late reply.
This is how I compute the metrics. Note that I decompose the factors of predicted boxes into IoU and box accuracy.
For these predicted boxes with low IoU, I still compute its classification accuracy. You could try to further improve this metric by removing these low IoU boxes.
Let me know if you have other questions, thanks!
# gt_pps: all ground-truth boxes
# pred_pps: all predicted boxes/ pseudo-boxes
    def probe_iou_acc(self, gt_pps, pred_pps, name=''):

        mean_iou = 0.0
        pred_acc = 0.0
        for gt_pp, pred_pp in zip(gt_pps, pred_pps):

            if len(pred_pp)!= 0 :
                max_iou, max_idx = pairwise_iou(gt_pp.gt_boxes, pred_pp.pred_boxes).max(0)
                mean_iou += max_iou.mean()
                gt_cls = gt_pp.gt_classes[max_idx]
                pred_acc += (gt_cls == pred_pp.pred_classes).sum().float() / gt_cls.numel()
                raise ValueError('Unknown name for comparing gt and pseudo roi bbox.')
            else:
                mean_iou += torch.tensor(0).cuda()
                pred_acc += torch.tensor(0).cuda()

        metric = {}
        metric['bbox_prob_gt&'+name+'/mean_iou'] = mean_iou.item() / len(gt_pps)
        metric['bbox_prob_gt&'+name+'/accuracy'] = pred_acc.item() / len(gt_pps)

        return metric

@ycliu93 thanks for your help!


[ClassiCube] add .desktop file

Issue description
ClassiCube can only be run from the terminal, but I would prefer if it also had a .desktop file
Ping maintainers
@360ied

@pinkcreeper100 You should test this: https://github.com/NixOS/nixpkgs/pull/207282


cmd/acme: ^D doesn't work in win

To repro, in a win window do:
% cat > /tmp/foo
something
^D

The cat never receives EOF and the ^D character goes into the file.

Shouldn't that be ^Z, rather?
Lucio.
On 11/19/21, Roger Peppe @.***> wrote:

To repro, in a win window do:
% cat > /tmp/foo
something
^D

The cat never receives EOF and the ^D character goes into the file.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/9fans/go/issues/86

--
Lucio De Re
2 Piet Retief St
Kestell (Eastern Free State)
9860 South Africa
Ph.: +27 58 653 1433
Cell: +27 83 251 5824


Some ssn are not found in the company API even though they are present at RSK website

Hi guys,
There seems to be a bug in the Company part of apis.is
I know you are taking the data by scraping the RSK website.
But there are some SSN that the api is not finding even when they are on the RSK website:
For example http://apis.is/company?socialnumber=5902697199 is not returning anything even if it works at RSK website: https://www.rsk.is/fyrirtaekjaskra/leit/kennitala/5902697199
Anyways I also want to thank you for creating apis.is, it is a great api! :+1:
Steps to reproduce

Go to http://apis.is/company?socialnumber=5902697199
This only returns an empty result
https://www.rsk.is/fyrirtaekjaskra/leit/kennitala/5902697199
This works at the RSK website.

Hi @simottar, I am not an apis maintainer, but I've contributed to this project in the past.
I encourage you (and anyone who's interested really) to try running this project locally to check further what's going wrong in cases like this one.
git clone git@github.com:apis-is/apis.git
cd apis
npm install
node index.js

I thought I could maybe see what's going wrong in your case, but I actually found out this social security number works when running apis locally ... so it's possible the apis IP address has been blocked by rsk.is for some reason (might be by some automated process if apis at some point bombarded them with requests, and if so it'll likely be blocked temporarily, however this should be checked by one of the maintainers if they have the time).
As can be seen here https://github.com/apis-is/apis/blob/master/endpoints/company/index.js#L58 the result is saved in cache for 86400 seconds (24 hours), it seems that the error detection (which should end up responding with a 500 error and the message www.rsk.is refuses to respond or give back data is not good enough and therefore an empty response was saved to cache.
If a maintainer could access the machine the backend is running on (or proxy through its IP address) and try running the following
curl https://www.rsk.is/fyrirtaekjaskra/leit/kennitala/5902697199

to see if we're currently blocked that would be nice.

As a future feature to prevent things like this from happening we could look into somehow making sure we don't spam pages we're scraping for information with some kind of rate limit implementation and endpoint specific rate-limit settings. :suspect:

I can curl those pages with no problem from the server, but also see that apis.is is responding correctly for that ssn now. It has been in the cache for approximately 3 hours.
@Loknar @simottar If you find another ssn that this happens to, please comment and I will try to look at it asap.
Being rate-limited has definitely been an issue before, and still is, and is definitely something we want to solve.


trainable params: 0

When doing finetuning, Lora-weight is loaded with peft without an error but has trainable params: 0.

I have the same issue :(

When doing finetuning, Lora-weight is loaded with peft without an error but has trainable params: 0.
I think you need to add is_trainable=True when you use PeftModel: model = PeftModel.from_pretrained(model, './run_alpaca_lora/alpaca-lora-ckpt',                                    torch_dtype=torch.float16,                                    device_map={'': 0},                                   is_trainable=True                                   )

@Mikubill  also encountered this problem, and also set  is_trainable=True, but it still shows trainable params: 0

config = LoraConfig.from_pretrained('path',
is_trainable=True,
torch_dtype=torch.float16,  device_map={'': 0}
)
config.inference_mode = False
this helps me

prepare_model_for_kbit_training freezes all the layers of the model so it makes sense they're not trainable after the line:
model = prepare_model_for_kbit_training(model)