[WIP] feat: upgrade ApisixRoute CRD version to v1

Signed-off-by: Jintao Zhang<EMAIL_ADDRESS>Please answer these questions before submitting a pull request

Why submit this pull request?

[x] Bugfix

[ ] New feature provided

[ ] Improve performance

[ ] Backport patches

Related issues

fixes #653

Bugfix

Description

How to fix?

New feature or improvement

Describe the details and related test reports.

Backport patches

Why need to backport?

Source branch

Related commits and pull requests

Target branch

Codecov Report

Merging #693 (0adec5c) into master (3e9bdbf) will decrease coverage by 1.32%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #693      +/-   ##
==========================================
- Coverage   35.37%   34.04%   -1.33%
==========================================
  Files          60       61       +1
  Lines        5967     6144     +177
==========================================
- Hits         2111     2092      -19
- Misses       3612     3801     +189
- Partials      244      251       +7

Impacted Files
Coverage Δ

pkg/apisix/plugin.go
80.00% <0.00%> (-20.00%)
:arrow_down:

pkg/apisix/cluster.go
27.25% <0.00%> (-3.50%)
:arrow_down:

pkg/apisix/route.go
35.66% <0.00%> (-2.10%)
:arrow_down:

pkg/ingress/controller.go
1.06% <0.00%> (-0.01%)
:arrow_down:

pkg/ingress/secret.go
0.00% <0.00%> (ø)

pkg/kube/translation/apisix_ssl.go
0.00% <0.00%> (ø)

pkg/kube/translation/translator.go
47.00% <0.00%> (ø)

pkg/ingress/compare.go
0.00% <0.00%> (ø)

pkg/ingress/pod.go
32.46% <0.00%> (+0.88%)
:arrow_up:

Continue to review full report at Codecov.

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

Title shows only ApisixRoute CRD was upgraded to v1 but actually all resources were. So I changed the title, changes look good to me.

just skip the Flaky e2e tests.


playback-error not work

Hi, i have tested last version of plugin 1.1.2
and event playback-error is not called if network connection stopped.
APP => IOS
and if app played and i have refresh app with cmd - R, plugin don't stop music.
And i receive paused event playback-state, while this music is playing

I noticed the same at version 1.1.4
I had no error message when the internet connection has been stoped or when the source file missed, pointed to wrong place or was a not supported format.


Documentation for mentioning github username and password should be included

https://github.com/carla-simulator/carla/blob/8854804f4d7748e14d937ec763a2912823a7e5f5/Util/Docker/Prerequisites.Dockerfile#L5
We have to mention our username and github key here for the command to execute otherwise it gives
Cloning into '/home/carla/UE4.26'... remote: Support for password authentication was removed on August 13, 2021. Please use a personal access token instead. remote: Please see https://github.blog/2020-12-15-token-authentication-requirements-for-git-operations/ for more information. fatal: Authentication failed for<EMAIL_ADDRESS>this error,
I do not find any mention of this in any documentation, should I submit a PR with the same? Or is it something that is already mentioned in some documentation?
Regards.

Hey @dt1729 how did you solve the login issue? I'd love to file a constructive PR if I knew how to authenticate.
For non-docker I know how to create this GitHub auth token stuff, but for the ue4-docker tooling, it's really a black box for me :sweat_smile:
Thanks for your help in advance :wink:

I think I figured it out. When prompted, you need to enter the GitHub access token as "password". This should fix it.

I did not get any prompt perhaps because it was a university PC and someone with University had a github info entered there. I had to edit the dockerfile and hardcode that information for my account. If it works on a fresh Linux install this issue shall be closed, we'd need someone to test this maybe?

I had to edit the dockerfile and hardcode that information for my account.

I think you're confusing ue4docker with Prerequisites.Dockerfile. You don't need to hardcode the username and password btw. Rather use the --build-arg argument of the docker build command to set an ARG in a Dockerfile 😉
I'm working on a new Dockerfile and wanna propose it soon. Stay tuned 😄

I can help write a test for it if you want.

@dt1729 Here are my Docker tools as PR: #5750. You're welcome to test it if you want :wink:


small fix for imports

remove duplicate imports

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: irisdingbj
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ozevren
If they are not already assigned, you can assign the PR to them by writing /assign @ozevren in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

galley/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@irisdingbj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
4f3e3dc124dd8768c6f8a478fb41d9706a734c6b
link
/test istio-pilot-multicluster-e2e

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.

@irisdingbj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
4f3e3dc124dd8768c6f8a478fb41d9706a734c6b
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
4f3e3dc124dd8768c6f8a478fb41d9706a734c6b
link
/test istio-integ-k8s-tests

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.

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: irisdingbj, ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

galley/OWNERS [ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@irisdingbj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
4f3e3dc124dd8768c6f8a478fb41d9706a734c6b
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
4f3e3dc124dd8768c6f8a478fb41d9706a734c6b
link
/test istio-pilot-multicluster-e2e

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.

@irisdingbj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
4f3e3dc124dd8768c6f8a478fb41d9706a734c6b
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
4f3e3dc124dd8768c6f8a478fb41d9706a734c6b
link
/test istio-integ-k8s-tests

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.


Update pillow

Fix issue #2201

Thanks, Set the minimal requirement to >= 3.4.2 (security issues) and 4.0.0 in requirements


Improve validation error quality

Invalid file: sites.yml
version: v0.1
sites:
  ghjobs:
    base_url: https://jobs.github.com
    endpoint_urls:     # <- here is the error
      p:
        route: positions.json
        timeout: 10
        methods: [ GET ]

Error shown:
(RESTEasyCLI) ➜  RESTEasyCLI git:(master) recli list-sites
error: ./sites.yml: sites: !!python/object/apply:collections.defaultdict
  args:
  - !!python/name:builtins.dict ''
  dictitems:
    ghjobs:
      value:
        endpoint_urls:
        - Unknown field.
        endpoints:
        - Missing data for required field.

^ This should look like:
(RESTEasyCLI) ➜  RESTEasyCLI git:(master) recli list-sites
error: ./sites.yml: while reading this file below errors were found:
ghjobs:
  endpoint_urls: Unknown field.
  endpoints: Missing data for required field.

Hint: Here is the error formatting logic: https://github.com/rapidstack/RESTEasyCLI/blob/a30b62fd963ec364657cc4b8d851c8a68384c070/resteasycli/lib/workspace.py#L166
There may be a need of using recursive algorithm to format the string. error list can be joined using ,  (comma followed by space)

fixed


Link converter truncating link

We used a link:
http://static.tp-link.com/TD-W9970(EU)_V2_Quick_Installation_Guide_1482286347761d.pdf
but the link converter changed to:
https://link.domain.com/xxxxx/EgoNq30c(EU)_V2_Quick_Installation_Guide_1482286347761d.pdf
Looks like () break that. Any way to ensure that the whole string is changed when there is no space?

We should be able to update the regex used on the link replacer to avoid a) breaking things like this when there's an invalid character and b) support brackets in URLs.

Just another example, although this was more where the link went than the link itself.
Original: https://xxxxx/xxxx/views.php?action=view&id=12345
Converted: https://xxxxx/xxxx/views.php?action=view&id=12345
Causes the software behind it to error because it is expecting to see the &.

Was that from an HTML email or a plain text one?

HTML.

I see. The e-mail is likely being presented as & in the HTML and is just being copied into the links table as is. I'll see if I can reproduce.

I've been looking into this. It's actually not quite as simple as I would have hoped. We need to look at the regular expression used to detect links to avoid the situation. We want to make sure that links like this are terminated properly.
Please see our website (http://somesite.com) for full details.

If we were to allow brackets in the URL the final bracket closure would be included in the link and not included in the email. Postal has actually done exactly what it was supposed to do with that URL (even though it's not really desirable).
Unfortunately, links aren't always followed by a space which is why this occurs.
We'll need to look at this regex in a little more details to resolve this.

Maybe allowing (  and ) in the URL string but to ensure that ) is always after a ( if included after a http*://*/
It shouldn't then fall foul of the scenario you've mentioned.
It may still miss the odd use case where a URL string only has ( or ) instead of both but that could be  few and far between and then wouldn't cause any issues with a lone ) at the end due to the requirement of having a ( before it after a final slash (as it would never be part of a domain name).

Any fix should be tested against the duplicate tickets referenced.

I think that "non commun" characteres in url like "(" or ")" or even "]", etc. is not a priority, but a character like the query parameters separator "&" should be a priority to solve this bug because most of the urls in emails will have some kind of utm/tracking parameter...

This also happens to links with ports.
https://server.com:1234
gets turned in to:
https://link.domain.com/ref345/sasgsf:1234

I think I've managed to resolve this issue, see the screenshot for all the testing links mentioned so far being intact in the database.

Are there any additional testable links or should I make a pull request?

Thanks! If you've tested this against all links mentioned in these tickets, please go ahead and make a pull request and I'll get this in. Even if it's not perfect (i'm sure someone will find a new way to break it), I'm happy to commit it as an incremental improvement.

Yes please! many thanks!

Yes please too. Thanks for your work on this @willpower232!

I've discovered its not a complete solution but theres only one fail for plain text messages when the URL is surrounded by brackets but its a vast improvement if it matches everything else.

and no worries! We rely a lot on postal so getting the key issues fixed is important

@willpower232 I made those changed manually in our codebase and restarted postal, but the problem still persists. Our "&" are converted into "&" and thus rendered invalid. Any solution to this?
Thanks.

@alkanna can you share an example of a link that isn't working?

This kind of link ends up stripped near the end, I couldn't figure out where exactly since we do not store the original anywhere (it's a jwt token): https://xxx.xx.com/ticket/approve/eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJhcHByb3Zlcl9pZCI6NCwicmVxdWVzdG9yX2lkIjo3MTExLCJ0aWNrZXRfaWRzIjpbNjg1M10sImxpbWl0IjoiMjAyMDA5MjVUMDU1MjAwIiwiYXBwcm92ZWQiOmZhbHNlfQ.vzjOqRHzSlxe4zzOcbxIDh_nQq-W9_RoGJ_tF-QI

The example you've posted is very long so its likely getting clipped by the database if your Postal is older than the merging of https://github.com/postalhq/postal/pull/683, you might have to manually apply converting that database field to a text type for the extra character length.
Regarding &, you can see them being converted successfully in my much older screenshot, is it possible they got encoded to & before Postal processed them?

Same problem. E-Mails in my Laravel app are not working properly. & is converting to &. Seems there are several open issues around this problem. Is there a known workaround for that?

We too are experiencing the issue with & being stored in the tracking links table in the database. The issue is that the tools that send them to Postal first encode all characters within links, specifically HTML email. A link of https://test.com/test?test=1&parameter=1 will get html encoded to https://test.xx/testing?test=1&parameter=1 before sending to Postal via SMTP or API, then Postal takes that URL and stores it as such with Track Clicks being enabled. Which then the link is invalid if you were to visit a URL with & in it instead of just &.


test/k8s: remove unused migrate-svc manifests

These are unused since the K8sUpdates ginkgo test was removed in commit a6d866e6fff5 ("ginkgo: Get rid of K8sUpdates").

/test


fail to run program: exit status 1

I want to build https://github.com/shadowsocks/shadowsocks-go
but got this error
E:\shadowsocks-go\cmd\shadowsocks-local>gopm install
D:\Go\gopath\src\github.com\shadowsocks\shadowsocks-go\shadowsocks\encrypt.go:14:2: cannot find pack
age "golang.org/x/crypto/blowfish" in any of:
        D:\Go\src\golang.org\x\crypto\blowfish (from $GOROOT)
        E:\shadowsocks-go\cmd\shadowsocks-local\.vendor\src\golang.org\x\crypto\blowfish (from $GOPA
TH)
        D:\Go\gopath\src\golang.org\x\crypto\blowfish
D:\Go\gopath\src\github.com\shadowsocks\shadowsocks-go\shadowsocks\encrypt.go:15:2: cannot find pack
age "golang.org/x/crypto/cast5" in any of:
        D:\Go\src\golang.org\x\crypto\cast5 (from $GOROOT)
        E:\shadowsocks-go\cmd\shadowsocks-local\.vendor\src\golang.org\x\crypto\cast5 (from $GOPATH)

        D:\Go\gopath\src\golang.org\x\crypto\cast5
D:\Go\gopath\src\github.com\shadowsocks\shadowsocks-go\shadowsocks\encrypt.go:16:2: cannot find pack
age "golang.org/x/crypto/salsa20/salsa" in any of:
        D:\Go\src\golang.org\x\crypto\salsa20\salsa (from $GOROOT)
        E:\shadowsocks-go\cmd\shadowsocks-local\.vendor\src\golang.org\x\crypto\salsa20\salsa (from
$GOPATH)
        D:\Go\gopath\src\golang.org\x\crypto\salsa20\salsa
[GOPM] 11-28 15:35:50 [FATAL] fail to run program: exit status 1

OK, I found due to gfw,  but at least gopm need tell me the network error, not not found on local


compilation error on RHEL >= 9.4

hi there!
we are facing scap module compilation error on RHEL 9.4:
**_CC [M]  /var/lib/dkms/scap/7.0.0+driver/build/main.o
In file included from ./include/linux/linkage.h:7,
from ./include/linux/kernel.h:8,
from ./include/linux/list.h:9,
from ./include/linux/smp.h:12,
from ./include/linux/tracepoint.h:15,
from ./include/trace/syscall.h:5,
from /var/lib/dkms/scap/7.0.0+driver/build/main.c:14:
/var/lib/dkms/scap/7.0.0+driver/build/main.c: In function 'scap_init':
./include/linux/export.h:17:22: error: passing argument 1 of 'class_create' from incompatible pointer type [-Werror=incompatible-pointer-types]
17 | #define THIS_MODULE (&__this_module)
|                     ~^~~~~~~~~~~~~~~
|                      |
|                      struct module *
/var/lib/dkms/scap/7.0.0+driver/build/main.c:2877:36: note: in expansion of macro 'THIS_MODULE'
2877 |         g_ppm_class = class_create(THIS_MODULE, DRIVER_DEVICE_NAME);
|                                    ^~~~~~~~~~~
In file included from ./include/linux/device.h:31,
from ./include/linux/node.h:18,
from ./include/linux/cpu.h:17,
from ./include/linux/static_call.h:135,
from ./include/linux/tracepoint.h:22:
./include/linux/device/class.h:230:54: note: expected 'const char *' but argument is of type 'struct module '
230 | struct class * __must_check class_create(const char name);
|                                          ^~
/var/lib/dkms/scap/7.0.0+driver/build/main.c:2877:23: error: too many arguments to function 'class_create'
2877 |         g_ppm_class = class_create(THIS_MODULE, DRIVER_DEVICE_NAME);
|                       ^~
./include/linux/device/class.h:230:29: note: declared here
230 | struct class * _must_check class_create(const char *name);
|                             ^~~~~~~~~~~~
cc1: some warnings being treated as errors
As we understand class_create patch backported to ReadHat, if we make the following change the compilation succeeds:
if (LINUX_VERSION_CODE >= KERNEL_VERSION(6, 4, 0)) || (PPM_RHEL_RELEASE_CODE > 0 && PPM_RHEL_RELEASE_CODE >= PPM_RHEL_RELEASE_VERSION(9, 4))
g_ppm_class = class_create(DRIVER_DEVICE_NAME);
else
g_ppm_class = class_create(THIS_MODULE, DRIVER_DEVICE_NAME);
endif
is it possible to add the above to scap driver code?

Good day. We have the same issue. Is it possible to fix RHEL >= 9.4 compilation error?

Hi! Thanks for opening this issue!
This build failure should've been already addressed by https://github.com/falcosecurity/libs/pull/1452 in 7.2.0+driver, that is part of Falco 0.38.0.
Can you test with it?

I see that there are configure folder with test modules starting from 7.1.0+driver. And changes are there in image starting from sysdig 0.37.0. This should work. Thanks a lot for help

I see that there are configure folder with test modules starting from 7.1.0+driver

Exactly :)

/milestone 0.18.0


Update http4s-circe, http4s-dsl to 1.0.0-M41

About this PR
📦 Updates

org.http4s:http4s-circe
org.http4s:http4s-dsl

from 1.0.0-M38 to 1.0.0-M41
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.http4s" }
}]

labels: library-update, early-semver-pre-release, semver-spec-pre-release, version-scheme:early-semver, commit-count:1

Superseded by #319.


Do not force push to secp256k1-zkp branch

We're using this repository as a submodule in trezor-firmware and we are bound the secp256k1-zkp branch. We are currently locked to commit 9ecd8bf38a77a97b44e8c830b286c65051f2f707, but this commit is not in the git tree anymore. Try clean clone and check out this commit:
git clone git@github.com:ElementsProject/secp256k1-zkp.git
git checkout 9ecd8bf38a77a97b44e8c830b286c65051f2f707
fatal: reference is not a tree: 9ecd8bf38a77a97b44e8c830b286c65051f2f707

It seems to me someone forced-push into this branch removing any reference to this commit. Is that correct?

Please do not force push to this branch
Which commit should we use? What is the new corresponding one?

Uh, hm. It's certainly our practice to rebase secp256k1-zkp from time to time onto secp256k1, and then to force-push branches, and in the best case we would like to keep it that way but I see that this creates problems.
First we thought that the problem is that the commit is not in the repo anymore..., so I pushed the branch 2019-05-rebase-previous-head which currently is at 9ecd8bf38a77a97b44e8c830b286c65051f2f707. That'll make sure that the commit is not lost.
However I think the problem is that your submodule points to secp256k1-zkp branch and the commit is not on that branch. Or does your submodule work again now?

Okay, so after reading your comment again, I think the problem was just that the commit was not in the repo anymore. The commit should be there in the repo again, even though the branch has been force-pushed to. Can you confirm that this solves the problem? In the future we'll then need to make sure that we don't remove existing commits from the repo.
Otherwise we need to think about a different solution.

Can you confirm that this solves the problem?

Yes, it does.

In the future we'll then need to make sure that we don't remove existing commits from the repo.

Maybe consider naming the branch secp256k1-zkp-trezor or similar so it's clear it is no to be deleted?

I think in general a policy of not blowing away branches is a good idea.
On Fri, May 31, 2019, 8:56 AM Tomas Susanka<EMAIL_ADDRESS>wrote:

Can you confirm that this solves the problem?
Yes, it does.
In the future we'll then need to make sure that we don't remove existing
commits from the repo.
Maybe consider naming the branch secp256k1-zkp-trezor or similar so it's
clear it is no to be deleted?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ElementsProject/secp256k1-zkp/issues/68?email_source=notifications&email_token=ABMAFUY5OCDWQRISDXD7VA3PYEN6JA5CNFSM4HRZPQK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWVEOJI#issuecomment-497698597,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABMAFU5JPRDL5M3UYNIXPD3PYEN6JANCNFSM4HRZPQKQ
.

Maybe consider naming the branch secp256k1-zkp-trezor or similar so it's clear that it is not to be deleted?

Yes, it's just a quick fix for now. We'll need to discuss in detail how we do this for future updates, and then we can rename the branch to a proper name or make it tag or something else.

Sounds good. We need the commit there with a reference, other than that we don't care much. Let me know in case you decide to rename.

I would suggest forking ElementsProject/secp256k1-zkp under https://github.com/trezor/ (similar to how it's done with vendor/micropython dependency, IIUC).
This way secp256k1-zkp dependency updates will be coordinated with the TREZOR team via PRs to the trezor/secp256k1-zkp and trezor/trezor-firmware repositories.
What do you think?

@romanz we can do that, but then you will need to file PRs to both trezor/secp256k1-zkp and trezor-firmware to bump the submodule. But yes, it's possible as well.

Yeah so actually we were happy that we could all changes necessary for Trezor cleanly in secp256k1-zkp so that we don't need to maintain yet another fork. I think we'll want to keep that model. I'll discuss with @apoelstra and come back here.

Sorry if I was ambiguous, I means as long as there was a canonical way of keeping old(renamed?) branches that's an improvement, even if the default branch is force pushed.

I vote that every time we rebase we simply rename the old branch from secp256k1-zkp to secp256k1-zkp-YYYY-MM-DD.

@real-or-random did you push the old secp256k1-zkp somewhere? can you push it again under the name secp256k1-zkp-2019-05-31?

Yep, it's here:

I pushed the branch 2019-05-rebase-previous-head which currently is at 9ecd8bf.

Okay, now there is a secp256k1-zkp-2019-05-30 branch:
https://github.com/ElementsProject/secp256k1-zkp/tree/secp256k1-zkp-2019-05-30
Sorry again for all the trouble!

Ok! No prob.


Filter out bad words

Add bad word filters, fixes #2
Surprisingly, after looking through https://npms.io/search?q=profanity I found out most of it traces back to just 3 or 4 sources, and most of it extremely bad, especially in terms of false positives.
After all, I used leo-profanity and badwords-list(the package that uses it runs so slow it didn't have enough patience to find how long it will take to process our word list).
I applied some custom magic on top of badwords-list to increase hit rate while reducing false positives to reasonable levels.

I looked through the diff of what's being removed and there seems to be way too many false-positives. For example, all these were removed:
aerospace
backspace
debugger
meatballs
workspace
woodpecker
wristwatch

@sindresorhus It's just 0.82% total positives(some of which are false). What rate you find acceptable?

But I don't understand why words like aerospace and workspace are matching the filter? Seems like there's some large flaw in the matching algorithm.

@sindresorhus because they include spac.
Removed spac substring test, added fetch-words -v option which shows why words were excluded.
However the point was that we have almost 275k words, we only exclude around 2k of them. Does it really matter if some of them are excluded as collateral damage? Also at this point it should be possible to just go through all the 2k words we exclude and check them manually for someone who is really fluent in bad words(I am not, sorry).

I guess my question is, why are you even matching substrings? Can’t we only match the full words? Seems like that would reduce the false-positives to 0.

Because there is no such premade list.
Anyhow, added custom list inside, filters 284 out of 274926 words now(no false positives if we trust leo-profanity and badwords-list).

One more thing you could do to catch some more words is to add a s to the detected bad word and see if it matches a word. I see a lot of singular words with plural versions.

It's the same as matching substrings though, we get false positives. (And in fact almost only false positives, there was only a single non-false-positive word it filtered.).
So, I just added this word to the exception list instead.


[User Story-Resource Explorer] Update the resources tree to contain both v1 and beta endpoints

As a user,
I want to see all the available resources on Graph under the resources tree both v1.0 and beta

Removing from milestone as spec might change


example for leaflet js

Hi,
is there also a "mountain-biking.php" example using the OpenStreetMap services with leaflet.js instead of Google maps?
Regards

Hi, I have used it with both Leaflet and GMaps - found switching between the two fairly straightforward. I had a quick look but couldn't find an example I could quickly share but will have another look later.

Hey, I just knocked a quick example together and committed to the demo folder in 3a445ff
Note the code is quite old now - the examples still use Bootstrap 3

Thanks a lot


References to ABC methods

Would you mind adding the literature references to both ABC algorithms to the documentation?
In particular, I could not find the reference to the SMC-ABC paper by Turner.

done :+1:


Add UI to edit time zones

Make drop down list all available time zones.
https://www.tooloop.org/Cookbook/Configure Time and Date
https://pypi.org/project/pytz/

added in dev branch


Support to specify path match type for authorization policy.

Currently, the regex match is not supported in authorization policy. In our user experience, we need to do authorization on routes with regex match.
How about introduce path match type for authorization policy api.@howardjohn @hzxuzhonghu

Do we have a plan to support this?


Execute head script tag

When I have turbo enable, my script tag in head doesn't execute.
Is there anyway to "force" it ?
Thank's !

No directly.
But you can use Stimulus to detect page visits and force you code to run.

You can execute your function by listening on events:
document.addEventListener('DOMContentLoaded', () => {
  yourFunction()
})

document.addEventListener('turbo:render', () => {
  yourFunction()
})

The DOMContentLoaded is triggered on the initial page, and the turbo:render is triggered when the Turbo drive kicked in (clicking on links etc.)

document.addEventListener('turbo:render', () => {
yourFunction()
})
but using this method makes the page like (not responding) until it runs the js, even tho the js isn't so big and doesn't need much time to load, which is unacceptable, or maybe i've done something wrong...


fix code scanning alert

https://github.com/yurake/k8s-3tier-webapp/security/code-scanning
java quarkus application

add code scanning tools by #3020


readme: add yarn install instructions to dev install instructions

When running the pip install .[...] install instruction, it fails with an error:
      > yarn run build:lib && yarn run build:nbextension && yarn run build:labextension

      Internal Error: @widgetti/jupyter-react@workspace:.: This package doesn't seem to be present in your lockfile; run "yarn install" to update the lockfile

This is fixable if you just yarn install before running the pip install command.

This fix got me to the next step, but still having trouble getting setup following the dev install instructions, specifically in the yarn run build:prod command that is called in some of the  steps.
I'll spare you the details, but am I missing any critical install steps?


[3.0]  The problem collection of Triple with ApplicationDiscovery

We have found these issues which are related to Triple with ApplicationDiscovery:

[ ] https://github.com/apache/dubbo/issues/8599
[ ] https://github.com/apache/dubbo/issues/8462
[ ] https://github.com/apache/dubbo/issues/8657

What we should do  for further promotion:

[ ] Complete the docs: Triple协议迁移指南 as soon as possible
[ ] The logic of preferredProtocol was not user-friendly, we maybe should optimize it.
[ ] Optimize the dubbo-spring-boot-starter, especially the autoconfigure.
[ ] Give a demo about SpringBoot integrate with Dubbo 3.0 which cover ApplicationDiscovery, Triple Protocol

Agreed on triple doc construction, I will start this task ASAP

I will also assist in these tasks

The logic of preferredProtocol was not user-friendly, we maybe should optimize it.
What's the problem?
When we only use the triple protocol in dubbo with setting provider's register-mode as instance, the consumer can't find the service.
The source is in org.apache.dubbo.config.metadata.ServiceInstanceHostPortCustomizer, let's have a look:
@Override
public void customize(ServiceInstance serviceInstance) {
    String host = null;
    int port = -1;
    // ......
    Set<URL> urls = writableMetadataService.getExportedServiceURLs();
    if (CollectionUtils.isNotEmpty(urls)) {
        ApplicationModel applicationModel = serviceInstance.getApplicationModel();
        String preferredProtocol = applicationModel.getCurrentConfig().getProtocol();
        //  The default value of preferredProtocol is dubbo.
        if (preferredProtocol != null) {
            for (URL exportedURL : urls) {
                // The protocol of exportedURL is tri
                // because of tri != dubbo, the host and port will not be assigned values.
                if (preferredProtocol.equals(exportedURL.getProtocol())) {
                    host = exportedURL.getHost();
                    port = exportedURL.getPort();
                    break;
                }
            }
        } else {
            URL url = urls.iterator().next();
            host = url.getHost();
            port = url.getPort();
        }
        // If not the port of instace > 0, the application instance will not be registered to RegistryCenter(like Nacos, Zookeeper).
        // As a result, the consumer can't find the avaliabe providers when our settging is registry-mode=instance
        if (serviceInstance instanceof DefaultServiceInstance) {
            DefaultServiceInstance instance = (DefaultServiceInstance) serviceInstance;
            instance.setHost(host);
            instance.setPort(port);
        }
    }
}

For the moment, if you want both consumer and provider to force using Application Discovery in tri protocol, just change one line config of provider as follows:
dubbo:
  application:
    metadata-type: remote
    register-mode: instance
    protocol: tri  <==== HERE, set preferredProtocol as tri

The possible solution
change the default application-protocol as TRIPLE in the future

Why not now? Because most users are still trying to migrate from 2.0 to 3.0

// => org.apache.dubbo.config.ApplicationConfig#checkDefault
protected void checkDefault() {
    super.checkDefault();
    if (protocol == null) {
        protocol = DUBBO;   // <======= HERE
    }
    // ......
}

In ServiceInstanceHostPortCustomizer, If the default protocol not exists, just try to get the first protocol and log an error log.

Doc of migration to triple is constructing in https://github.com/apache/dubbo-website/pull/941/files


Consider using Gradle APIs to download JBR instead of DownloadAction

The issues with DownloadAction are:

It does not support dependency verification (see https://docs.gradle.org/current/userguide/dependency_verification.html)
It does not play nicely with caching and removing stale cached artifacts

I guess the relevant Gradle API is to pretend that JBR distribution site is Ivy repository with a custom pattern: https://docs.gradle.org/current/userguide/declaring_repositories.html#sub:defining_custom_pattern_layout_for_an_ivy_repository

Hey, Vladimir!
Thanks for putting that on the table. I've been recently looking for some alternatives to the DownloadAction, but I didn't even consider Ivy repositories as a way to go.
I've just got rid of it and replaced it with Ivy, and it seems to be working well.
Thanks for that!

That was fast!


MacOS .dmg app fails on opening

Operation System: macOS Sierra (version 10.12.6)
Beaker Version: 0.7.5
Installed the app but when I try to open it I get the error:

Hi @Fraser-Greenlee, sorry I just saw this.
I'm not quite sure how this is happening. It must be some kind of electron bug, because the stack trace you have suggests that the screen API is null. Doesn't seem possible.
Can you try building from source, at the 0.7.5 tag?

Confirmed this is still happening.

Note the screen API issue also causes an alert almost every time I scroll on a MacBook. Filed a new issue #753 for that.


Angular devkit 18.0.7 not compatible with Bootstrap 5.3.3

Command
serve
Is this a regression?

[X] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
13.0
Description
I updated from Angular 13 to latest 18.0.0 and this bug has arisen.
./libs/shared/styles/src/index.scss?ngGlobalStyle - Error: Module build failed (from ./node_modules/mini-css-extract-plugin/dist/loader.js):
HookWebpackError: Module build failed (from ./node_modules/sass-loader/dist/cjs.js):
From line 30, column 13 of node_modules/bootstrap/scss/_grid.scss:
   ╷
30 │             @extend %responsive-container-#{$breakpoint};
   │             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
You may not @extend selectors across media queries.
   ╷
30 │             @extend %responsive-container-#{$breakpoint};
   │             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
  node_modules/bootstrap/scss/_grid.scss 30:13               @content
  node_modules/bootstrap/scss/mixins/_breakpoints.scss 62:7  media-breakpoint-up()
  node_modules/bootstrap/scss/_grid.scss 19:5                @import
  node_modules/bootstrap/scss/bootstrap.scss 16:9            @import

Minimal Reproduction

Install Angular 18.0
Install Bootstrap 5.3.3
Run ng serve --ssl

Exception or Error
No response
Your Environment
ng version                                                                       1 ✘  wct_socketmix_38 3.8.18   20.11.1 

     _                      _                 ____ _     ___
    / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
   / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
  / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
 /_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
                |___/

Angular CLI: 18.0.7
Node: 20.11.1
Package Manager: yarn 1.22.22
OS: linux x64

Angular: 18.0.6
... animations, cdk, common, compiler, compiler-cli, core, forms
... language-service, localize, material-moment-adapter
... platform-browser, platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1801.0
@angular-devkit/build-angular   18.1.0
@angular-devkit/core            18.0.7
@angular-devkit/schematics      18.0.7
@angular/material               18.1.0
@schematics/angular             18.0.7
rxjs                            7.8.1
typescript                      5.4.5
zone.js                         0.14.7

Anything else relevant?
No response

This seems like a bug but we'll need to look at a reproduction to find and fix the problem. Can you setup a minimal repro please?
You can read here why this is needed. A good way to make a minimal repro is to create a new app via ng new repro-app and adding the minimum possible code to show the problem. Then you can push this repository to github and link it here.
This might be related to your directory structure so its really important to get an accurate repro to diagnose this.

I'm sorry, but we can't reproduce the problem following the instructions you provided.
Remember that we have a large number of issues to resolve, and have only a limited amount of time to reproduce your issue.
Short, explicit instructions make it much more likely we'll be able to reproduce the problem so we can fix it.
If the problem persists, please open a new issue following our submission guidelines.
A good way to make a minimal repro is to create a new app via ng new repro-app and add the minimum possible code to show the problem. Then you can push this repository to github and link it here.

I can't reproduce it either in a fresh repo, please don't close the issue as I've been trying to have the problematic repo for this week. @angular-devkit/architect is really unstable and has a lot of problems in all versions, but it seems to work well enough in small simple apps. That's why I keep developing the repo to have a more complex app.


Auto-Format on save cannot be disabled

Describe the bug
Whenever a .al-File in VSCode is saved, it is automatically formatted. This behavior is often failing due to performance issues and/or analyzers behaving badly. This renders it impossible to save the file to disk. VSCode has an option to auto-format files on save, but this option is disabled (it is by default). Yet, the AL extension ignores this setting.
To Reproduce

Check the "Format on save" setting. It should be disabled by default. If not, disable it.
Open a AL file and make some changes that required re-formatting.
Save the file.
Watch as the file is formatted.

AL Code to reproduce the issue
Expected behavior
The file should only be automatically formatted if configured.
Screenshots
/
5. Versions:

AL Language: 6.1.362735

Business Central: 17.1

Do you have any other extensions installed other than the AL Language extension?

I do. Lots. I have the following installed:

AL Code Actions
AL Object Designer
AZ AL Dev Tools
Azure Account
Azure Pipelines
C#
GitHubSharp Theme
GitLens
Hex Editor
Insert GUID
Material Icon Theme
OpenAPI
PowerShell
SQL Server
CRS AL
XML Tools
YAML

I have tried to disable them all and reproduce it with only the AL extension installed. Does not change anything.

@atoader I can see the "input-needed" tag is still there. Do you need any more info?

Try the following settings inside your user/workspace settings:
    "[al]": {
        "editor.formatOnSave": false,
    }

@dannoe : Ok, sure. This works fine, thanks for the hint. Yet, I do think AL should respect the "global" setting.


remove nested methods

Multiple stores seems to provide a better method of composition.
I haven't enjoyed this feature as much as I thought I would -- it sort of breaks referential transparency on the state parameter.
This would also allow for simpler logging. And it would encourage flat / modular code over hierarchical code.

In the end of the day this is more flexible. Will leave it in unless it leads to bugs.

Implemented in 1.0


Conflict with langchain-cohere package

I love the semantic router but a drawback is that it's cohere dependancy is cohere 4.57 whereas if you want to use langchain-cohere (in order to leverage the reranker for example), it requires cohere 5.3 and up! It seems like we essentially need to choose between the SR and the reranker... it would be great if they could work in tandem!

This should be resolved with #280 — I'm pushing this change to version 0.0.41
I'll close the issue for now, let us know if you run into any problems!

update: 0.0.41 is out https://github.com/aurelio-labs/semantic-router/releases/tag/v0.0.41

Awesome!


[retirement receipt] ensure fallback data works

Description
RELATED TO https://github.com/KlimaDAO/klimadao/issues/1096
To render a retirement receipt page, the subgraph is queried to get the retirement data for a specific index.
In some cases, the subgraph is not fast enough on this flow:

Successfully made an retirement from a project pool ( e.g. here)
Click on "Show Receipt" link in Success Modal
=> Redirect to new page with the receipt https://www.carbonmark.com/retirements/<address>/<retirement-index>
This page queries the subgraph on the server for the retirement data
If there is no data from the subgraph => get fallback data from the contract
=> Ensure to render a page with fallback data
See the code for the page here:  https://github.com/KlimaDAO/klimadao/blob/staging/carbonmark/pages/retirements/[beneficiary]/[retirement_index].tsx

PROBLEM

[ ] some users reported that this receipt page ended in a 404 page (not a page with fallback data)
[ ] Investigate when this happens (specific tokens? What does actually throw on the server?)
[ ] Ensure that the fallback data is always working to properly render a receipt page even when the subgraph is not returning any data

@psparacino @Atmosfearful
I was able to repeat getting a 404 page again for the retirement receipt.
This was the transaction details
Everything is good to the Retirement Successful modal:

When I click SEE YOUR RETIREMENT RECEIPT I get a 404 page:

If I refresh the page a couple of times the receipt then does render properly: https://www.carbonmark.com/retirements/0x6476edf98Bdc5EE62f9D4E4BF8932d69Ed685483/24

@psparacino is working on some refactors that will simplify this & improve performance, we will simplify the code so we never check the RPC for anything. This means, intentionally, 404s will happen when the subgraph is behind #1018
we will create a simple poller util that waits for the subgraph to catch up to a specified block number (which we know in advance, from the txn receipt), before navigating the user to the page #1096
Finally, in the future, we can enhance the retirement page and do a formal fallback design this time. I created #1290 to document this task, low prio.
I'll close this ticket and #578 in favor of #1096 #1018 and #1290


added tic tac toe

PR Description 📜
Tic Tac Toe
Fixes #18
I have added tic-tac-toe game where alternate chance is given to X and O.
Game can also be reset if required.
The chance of player each time and the winner is displayed.
Sound occurs on clicking the the box.
Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Added all files and image in assets folder updates readme.md in main and resolved conflicts.
Kindly review my PR @kunjgit

Hey @SimranLahrani842 👋!
@kunjgit here,
We have this annoucement to make 📢!!
During the time of PR don't alter main index file ! 🙂
In main Readme file just use * as a game number during the time we are merging we will add number !! 👀
Make sure you join our Discord🕹 if you haven't 😇
Thank you 💗!

Hey @SimranLahrani842 wave!
@kunjgit here, We have this annoucement to make loudspeaker!! During the time of PR don't alter main index file ! slightly_smiling_face In main Readme file just use * as a game number during the time we are merging we will add number !! eyes Make sure you join our Discordjoystick if you haven't innocent Thank you heartpulse!

Ok Sir it was not mentioned anywhere so I wrote number.
Do I have to change number and write * now?
@kunjgit

Hey @SimranLahrani842 👋!
@kunjgit here, We have this annoucement to make 📢!! During the time of PR don't alter main index file ! 🙂 In main Readme file just use * as a game number during the time we are merging we will add number !! 👀 Make sure you join our Discord🕹 if you haven't 😇 Thank you 💗!

added * instead of number..please review pr and merge it @kunjgit

@SimranLahrani842 Please resolve the branch conflicts

@SimranLahrani842 Please resolve the branch conflicts

Done sir kindly review and merge the PR... @Arun9739

@SimranLahrani842 LGTM

@SimranLahrani842 LGTM

@kunjgit please add the labels of level and gssoc and merge ..thanking you and  @Arun9739

Thank you  @SimranLahrani842 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Running Multiple Instances of bot() in a Threaded Application.

Before submitting an issue, make sure you have:

[x] Updated to the latest version
[x] Read the README and docs
[x] Searched the bugtracker for similar issues including closed ones

Purpose of your issue?

[ ] Bug report (encountered problems/errors)
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

Describe your issue
Working on an application that will instantiate n number of instances of bot() in a single directory with multiple IG account credentials.
However, I'm concerned that it will run into issues since the followed.txt, skipped.txt, and unfollow.txt don't contain a prefix to distinguish them from one account to another. For example in follow_users() you have:
    followed_list = self.read_list_from_file(
        "followed.txt")   # Read followed.txt file
    skipped_list = self.read_list_from_file(
        "skipped.txt")  # Read skipped.txt file

Questions:

Am I correct that this will be an issue?

Is there an easy solution if this is an issue or do I need to do some work by extending the codebase with my own methods?

yes , you will have problem.
you have to run each instance from different directory.

Thanks @BobZombiE69. I've been digging through the code and have come to this conclusion. I've been working on overriding the Bot class methods to support something like this.


[SPARK-12640][SQL] Add simple benchmarking utility class and add Parquet scan benchmarks.

[SPARK-12640][SQL] Add simple benchmarking utility class and add Parquet scan benchmarks.
We've run benchmarks ad hoc to measure the scanner performance. We will continue to invest in this
and it makes sense to get these benchmarks into code. This adds a simple benchmarking utility to do
this.

Test build #48738 has started for PR 10589 at commit 22afd1f.

Test build #48738 has finished for PR 10589 at commit 22afd1f.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class Benchmark(name: String, valuesPerIteration: Long, iters: Int = 5)
  case class Case(name: String, fn: Int => Unit)
  case class Result(avgMs: Double, avgRate: Double)

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/48738/
Test PASSed.

@nongli You don't need to repeat the title in the description of the PR (or it get duplicated in commit message).

Test build #48876 has started for PR 10589 at commit f231e9a.

Test build #48876 has finished for PR 10589 at commit f231e9a.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds the following public classes (experimental):
  case class Case(name: String, fn: Int => Unit)
  case class Result(avgMs: Double, avgRate: Double)

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/48876/
Test PASSed.

Test build #2341 has started for PR 10589 at commit 4f682c5.

Test build #48885 has started for PR 10589 at commit 4f682c5.

Test build #2341 has finished for PR 10589 at commit 4f682c5.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  case class Case(name: String, fn: Int => Unit)
  case class Result(avgMs: Double, avgRate: Double)

Test build #48885 has finished for PR 10589 at commit 4f682c5.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  case class Case(name: String, fn: Int => Unit)
  case class Result(avgMs: Double, avgRate: Double)

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

Merged build finished. Test PASSed.

I've merged this. Thanks!


Rackspace URLs don't work

Using a Rackspace URL, ViewerJS will not load the resource. Rackspace resource URLs are structured as https://[key].ssl.cf2.rackcdn.com/[filename]_[ext]-[id].
Could the lack of the .[ext] in the URL be throwing off ViewerJS?

Yes, the lack of .[ext] at the end of the path threw off ViewerJS.
Today ViewerJS 0.5.5 got released, and it features an initial HEAD request to learn about the content type. Additionally the type can be set by passing an argument to ViewerJS itself, e.g. like /ViewerJS/index.html?type=pdf#https://[key].ssl.cf2.rackcdn.com/[filename]_[ext]-[id].
This should solve the issue, thus closing it now.


chainlink-nodes: arbitrum gas estimator

Documenting core node env var changes from:

https://github.com/smartcontractkit/chainlink/pull/7225

Rebased to main


Add a fail test for exray

Add a test that will immediately fail regardless of input. Use case:
try {
   function_that_is_expected_to_fail();
   X.fail(); // if it reaches here, assure a failure
}
catch (e) {
   ...
}

It might not be necessary. Just wanted to put the note here before I forget.

Not planned at this current time.


JavaScript

JavaScript
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I can see the four topics I learned listed along with a suggestion on what I should study first

Closed this issue to early before adding functions to the console


Speedloaders don't Speedload

Description

Name explains itself. The only way to use the "speed"loader in its current state is to unload it, and put every bullet in the gun manually. I don't call that speedloading.
Happy 1000th open issue
Reproduction

Get a revolver (any revolver, as long as it has a speedloader)
Get a speedloader for said revolver
Unload revolver
Attempt to use speedloader on revolver
If done correctly, the speedloader will do absolutely nothing.
Additional context

This has been an issue ever since the gun rework I believe, but... Nobody's reported the issue. This says a lot about our society

I'm gonna attempt to fix this, as (far as I know) nobody has attempted to.


How to specify spec file by relative url

Background
I have a web server that returns swagger json (under /api), on the same domain that hosts my docusaurus website.
I have a following config in docusaurus.config.js.
It is supposed to fetch swagger spec file from ${BASE_DOMAIN}/api/swagger-doc/v1.
       specs: [
           {
             route: "/api_doc/v1",
             spec: "/api/swagger-doc/v1",
           },
         ],

Problem
I have been using 0.6.0 and I am trying to upgrade to 1.6.0. But after the upgrade I got the following error when I ran docusaurus start.
It seems it is trying to find a file on the filesystem but I want it to get resource from the path on the web server.
I may be missing something, but is this because the spec field behavior changed after some version? Is there any workaround?Thanks in advance.
 [ERROR] Error: ENOENT: no such file or directory, open '/api/swagger-doc/v1'
[run]     at BaseResolver.<anonymous> (/home/user/my-doc/node_modules/@redocly/openapi-core/lib/resolve.js:119:23)
[run]     at Generator.throw (<anonymous>)
[run]     at rejected (/home/user/my-doc/node_modules/@redocly/openapi-core/lib/resolve.js:6:65)

but is this because the spec field behavior changed after some version?

Yeah, it changed in v1. We now do build time bundling of api docs to match how docusaurus builds other types of pages (md files). So if you want to use a url it should be an absolute url and be resolvable at build time.

Is there any workaround?

Alternatively, if you want to maintain the old behavior then you can use the package as just a react component in a custom page like here - https://github.com/rohit-gohri/redocusaurus/blob/main/website/src/pages/examples/client-only/index.jsx Relative urls should work here AFAIK.

Seeing this same thing -- my openApiDocument is built in a previous step and exists in the filesystem at build time.
With
 specs: [
                    {
                        spec: `${__dirname}/__build__/openApiDocument.json`,
                        route: '/api/',
                    },
                ],

or
 specs: [
                    {
                        spec: `__build__/openApiDocument.json`,
                        route: '/api/',
                    },
                ],

I get the error:
Loading of api failed for "<redacted file path>/docs-api/openapi.json"
[ERROR] Error: ENOENT: no such file or directory, lstat '<redacted file path>/docs-api/openapi.json'

If i copy the file into openapi.json things work fine. Looks like the configuration isn't being read correctly.

Debug output looks ok
[REDOCUSAURUS] Options: {
  debug: true,
  specs: [
    {
      spec: '<redacted file path>/docs-api/__build__/openApiDocument.json',
      route: '/api/'
    }
  ],
  theme: { primaryColor: '#1890ff' }
}
[REDOCUSAURUS] Specs: [
  {
    spec: '<redacted file path>/docs-api/__build__/openApiDocument.json',
    route: '/api/'
  }
]
[REDOCUSAURUS] Final: {
  "themes": [
    [
      "<redacted file path>/node_modules/docusaurus-theme-redoc/dist/index.js",
      {
        "id": "theme-redoc",
        "primaryColor": "#1890ff"
      }
    ]
  ],
  "plugins": [
    [
      "<redacted file path>/node_modules/docusaurus-plugin-redoc/dist/index.js",
      {
        "spec": "<redacted file path>/docs-api/__build__/openApiDocument.json",
        "route": "/api/",
        "themeId": "theme-redoc",
        "id": "plugin-redoc-0",
        "debug": true
      }
    ]
  ]
}
[REDOCUSAURUS_PLUGIN] Opts Input: {
  config: undefined,
  spec: '<redacted file path>/docs-api/__build__/openApiDocument.json',
  route: '/api/',
  themeId: 'theme-redoc',
  id: 'plugin-redoc-0',
  debug: true,
  layout: {}
}
[REDOCUSAURUS_PLUGIN] Options: {
  layout: {},
  debug: true,
  config: undefined,
  spec: '<redacted file path>/docs-api/__build__/openApiDocument.json',
  route: '/api/',
  themeId: 'theme-redoc',
  id: 'plugin-redoc-0'
}

Ok nevermind -- mine is unrelated. I was switching over from docusaurus-preset-openapi and had left some config lying around

Alternatively, if you want to maintain the old behavior then you can use the package as just a react component in a custom page like here

@rohit-gohri Thank you so much. A relative URL in a React component in a custom page worked as expected!


Unusable storage container placed

When I placed a blue belt (also tried placing a stacker) a huge storage container gets placed right on top. cannot access the storage and cannot delete it either, yet it is in the way. Please advise. Thank you

Another issue found it removes your construction drones permanently and breaks that save.

What mods are you using?

Thank you for your reply. This issue is related to the better machines mod.
It didn't happen again after I removed the mod before opening a previous
save.
On Thu, 18 Jan 2024, 21:36 kremnev8, @.***> wrote:

What mods are you using?
—
Reply to this email directly, view it on GitHub
https://github.com/kremnev8/DSP-Mods/issues/93#issuecomment-1898927355,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY57HMVDPF3TBIV7TY6MAGLYPFMSNAVCNFSM6AAAAABB6CGJDKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJYHEZDOMZVGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Experiencing the same issue, Please see attached screen shot of floating immortal containers.
unable to build or delete after placing anything.
Enabled Mods:

AssemblerVerticalConstruction
BeltSpeedEnhancement
BepInEx
BetterMachines
Bulldozer
BulletTime
CloseError
CommonAPI
Dark_Fog_Information
DSPBigInventory
DSPModSave
DSPMultiplyStackSize
FastAbsorption
GalacticScale
GigaStationsUpdated
GS2_Cheats
IlLine
LDBTool
PlanetMiner
PlanetWormhole
PowerNodeMultipliers
RailgunsRetargetMini
RebindBuildBar
SmartTank
SmelterMiner
SplitterOverBelt
TelePotter
UniverseGenTweaks
UXAssist
WaterElectrolysis

I also have this problem.  The only mods we both have in common is Better Machines and Galactic Scale so this is likely an incompatibility between those two mods.

Disabling Galatic Scale (and GS-Cheats) and starting a new (sanbox) game produces the same results unable to build and when building it creates a path blocking container on top of Icarus, when in the air or on the ground.

Tested a new game with only this mod and its required components.
Steps to reproduce:

started new sandbox game
set Sandbox tools -> Sandbox tools (6) -> Fast build /frame to 'All'
successfully built a line of conveyor belts
disabled setting above, setting Fast build /frame to 'OFF'
placed another line of conveyor belts and ran into issue mentioned and seen in screen shot below.

Enabled Mods:

BepInEx

BetterMachines

CommonAPI
DSPModSave
LDBTool
RebindBuildBar

Any updates on this one? Would love to use tier 4 belts again


Show browser tab title instead of the browser name

It would be really great if the tab titles are shown instead of the the name of the application like in the example below:

The reason for it is that I'm using most of the (meeting) applications like Teams, Zoom and other within the browser and if I would like to mute one of the channels I have to try first which one is it and if I have multiple tabs with some audio open, then it is really a challenge to find the right one.
Beside that the icon of the Brave browser is not shown as it can been seen in the screenshot above. Brave is installed as Flatpak package.

I've added the ability to show the audio stream description in the settings menu of v11.

In Firefox, I submitted a patch to show the tab name on certain audio streams; I'm not sure of Brave's behavior here. Ultimately, it's up to the browser to provide a name to the audio stream, so if Brave does not set that property correctly there isn't much I can do on my end.


Version 0.10 has a bug

First run, creating autocomplete index...

Traceback (most recent call last):
  File "/Users/wanglei/.virtualenvs/pyenv/bin/aws-shell", line 11, in <module>
    sys.exit(main())
  File "/Users/wanglei/.virtualenvs/pyenv/lib/python2.7/site-packages/awsshell/__init__.py", line 47, in main
    write_index(index_file)
  File "/Users/wanglei/.virtualenvs/pyenv/lib/python2.7/site-packages/awsshell/makeindex.py", line 63, in write_index
    index_command(current, help_command)
  File "/Users/wanglei/.virtualenvs/pyenv/lib/python2.7/site-packages/awsshell/makeindex.py", line 55, in index_command
    index_command(child, sub_help_command)
  File "/Users/wanglei/.virtualenvs/pyenv/lib/python2.7/site-packages/awsshell/makeindex.py", line 55, in index_command
    index_command(child, sub_help_command)
  File "/Users/wanglei/.virtualenvs/pyenv/lib/python2.7/site-packages/awsshell/makeindex.py", line 43, in index_command
    arg, arg_obj.argument_model)
TypeError: generate_shorthand_example() takes exactly 4 arguments (3 given)

I check the code, and find this bug has been fixed. But the package in pypy is still not working. Please release a new version.

In case it helps anyone.. To fix this issue I did the following:
Installed the correct versions of rsa and colorama:
sudo pip install 'rsa<=3.3.0,>=3.1.2'
sudo pip install 'colorama<=0.3.3,>=0.2.5'
then removed aws-shell and reinstalled:
sudo pip uninstall aws-shell
sudo pip install aws-shell
And finally it ran with:
aws-shell

Version 0.1.1 is now available: https://pypi.python.org/pypi/aws-shell/0.1.1
You should be able to just run pip install --upgrade aws-shell (or sudo pip install --upgrade aws-shell depending on how you have python set up) to pull in the latest changes.
Let me know if that doesn't work for you and I'll reopen and take another look.

@jamesls It works, thanks a lot.


[FEAT]: Better handling for thinking models

What would you like to see?
Been experimenting with Gemni 2.0 Flash Thinking (awful name) lately and it seems like the anythingllm frontend handles the api answer as such:
[Thought process][Answer]
It puts all the text in the same "space" as if it was one unique answer, but it's not.
This is an example of an answer showcased in anythingllm:

the highlighted part is where I think both thought and answer are concatenated, not only because it clearly looks like, but also because that's the only part of the whole answer where a period is missing its respective subsequent space.
This is an example of the same prompt in google ai studio

If I expand that Thought box, it shows all the previous process.
I'm not sure if the api response body provides the possibility for implementing this, but I sure hope it does! Also, I wonder if this a mistake also happening with OpenAI's o1.
Edit: One workaround I found is to specify on the workspace's prompt to structure its responses like so:
[Thought process]
[Enter]
[Response]

This a good idea. I like how OpenAI had the thinking process as an unordered list that one would open and close.


Problem with image include during site build

Example of builds: https://github.com/lutaml/lutaml.org/actions/runs/480515518, https://github.com/lutaml/lutaml.org/actions/runs/480515516. This is the page that has an image inserted in it, but during build process the following error is thrown and the build is halted:
[✓] HTML Exported (4.5s)
Copying media docs/uml-domain-specific-languagesimple-tutorial/tutorial_diagram_1.png dist//uml-domain-specific-languagesimple-tutorial/tutorial_diagram_1.png
  Error: Trace: Error: ENOENT: no such file or directory, copyfile 'docs/uml-dom  ain-specific-languagesimple-tutorial/tutorial_diagram_1.png' -> 'dist//uml-dom  ain-specific-languagesimple-tutorial/tutorial_diagram_1.png'
      at Object.copyFileSync (fs.js:1991:3)
      at _callee2$ (/home/runner/work/lutaml.org/lutaml.org/lutaml-dot-org/node_  modules/@riboseinc/react-static-plugin-aperis-doc-pages/node.api.js:195:44)
      at tryCatch (/home/runner/work/lutaml.org/lutaml.org/lutaml-dot-org/node_m  odules/regenerator-runtime/runtime.js:63:40)
      at Generator.invoke [as _invoke] (/home/runner/work/lutaml.org/lutaml.org/  lutaml-dot-org/node_modules/regenerator-runtime/runtime.js:293:22)

This issue belongs to Aperis, let me move it there. Thanks!

This issue belongs to Aperis, let me move it there. Thanks!

Closing this in favor of aperis-docs/react-static-plugin-aperis-doc-pages#7

Closing this in favor of aperis-docs/react-static-plugin-aperis-doc-pages#7

Actually, this probably belongs in lutaml.org repoistory, but plugin repo is better than Paneron anyway, thanks!

Actually, this probably belongs in lutaml.org repoistory, but plugin repo is better than Paneron anyway, thanks!


Golang Binding

I am using Porcupine with Go and want to use Cobra too.
But Cobra does not have Go binding yet. (I can use C directly in Go, but I don't want to :))
I wonder if there is any roadmap for this.

Thank you.

If you submit a PR for it, we would love to look into it and merge it in :)

closing due to inactivity. re-open if needed.


Fix for out of space for CodeQL

Description
Github Runner is running out of space for default runner. Created new pool and image to use with more space.
Related issues
Addresses [issue #].
Testing
FHIR Team Checklist

Update the title of the PR to be succinct and less than 65 characters
Add a milestone to the PR for the sprint that it is merged (i.e. add S47)
Tag the PR with the type of update: Bug, Build, Dependencies, Enhancement, New-Feature or Documentation
Tag the PR with Open source, Azure API for FHIR (CosmosDB or common code) or Azure Healthcare APIs (SQL or common code) to specify where this change is intended to be released.
[ ] CI is green before merge
Review squash-merge requirements

Semver Change (docs)
Patch|Skip|Feature|Breaking (reason)

/azp run


Why olm checks all CatalogSource objects health when apply operators by Subscription

Bug Report
What did you do?

two CatalogSource objects in the same namespace
one CatalogSource is healthy, another one is unhealthy
apply an operator in the healthy CatalogSource by create Subscription object

What did you expect to see?
apply the operator in the healthy CatalogSource successfully. Because in the Subscription object, we specify the healthy Catalogsource object by name and namespace

What did you see instead? Under which circumstances?

operator apply failed
Subscription's condition unhealthy, message is:
 [failed to populate resolver cache from source community-operators/openshift-marketplace: failed to list bundles: rpc error: code = Unavailable desc = connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:50051: connect: connection refused", failed to populate resolver cache from source certified-operators/openshift-marketplace: failed to list bundles: rpc error: code = Unavailable desc = connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:50051: connect: connection refused", failed to populate resolver cache from source redhat-marketplace/openshift-marketplace: failed to list bundles: rpc error: code = Unavailable desc = connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:50051: connect: connection refused", failed to populate resolver cache from source redhat-operators/openshift-marketplace: failed to list bundles: rpc error: code = Unavailable desc = connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:50051: connect: connection refused"]
It seems the unhealthy CatalogSource effects the deployment of the healthy CatalogSource
We view the codes. When reconcile the Subscription objects, it will check all the CatalogSource Objects in the same namespace of the CatalogSoure  which the Subscription is subscribed

This isn't a bug, it's an intentional design decision. OLM considers the context of all the catalogs that are available in the context of the namespace (so, all the catalog sources in the namespace + the global catalog sources) in order to get a deterministic result when resolving all the operators in a namespace and their potential dependencies. When OLM installs an operator, it considers the potential context of everything installed in a namespace, and fails to resolve if it can't.

@kevinrizza thanks for taking the time and providing this valuable feedback.

This isn't a bug, it's an intentional design decision. OLM considers the context of all the catalogs that are available in the context of the namespace (so, all the catalog sources in the namespace + the global catalog sources) in order to get a deterministic result when resolving all the operators in a namespace and their potential dependencies. When OLM installs an operator, it considers the potential context of everything installed in a namespace, and fails to resolve if it can't.

I'm going to close this issue as Kevin has addressed the question and the official documentation discusses this concept as well. Thanks for using OLM @sqwaseda.


To support 0.38.1, we have changed style

Change styles of flex1 from {flex: 1} to {flexGrow: 1}
const styles = StyleSheet.create({
  flex1: {
    flexGrow: 1
  }
});

keyboardShouldPersistTaps = {true} is deprecated in RN 0.40
Use keyboardShouldPersistTaps        = "always"


Upgrade to 1.123.0+: Preview overwrites public/ directory

What version of Hugo are you using (hugo version)?

$ hugo version
hugo v0.123.0-3c8a4713908e48e6523f058ca126710397aa4ed5+extended linux/amd64 BuildDate=2024-02-19T16:32:38Z VendorInfo=gohugoio

Does this issue reproduce with the latest release?
Yes

When migrating from v1.121.2 to v.123.x, publication with hugo command works very well, but then, when doing hugo server --watch --port 1313 -DEF --printI18nWarnings the preview version overwrites the content of public/ (previously returned by the hugo command). This means that all URL are all wrong (mainly because using the port 1313) and that all drafts are published in production (which is kind of dangerous). Fortunately, I noticed that very quickly.
I did not have this issue with v1.122.0. It just started with v1.123.0 and is still there with v1.123.8.
I noticed that before 1.123.x, the server mode always served pages from memory, but now the default mode seems to be from disk. I assume that's why I got this unexpected behavior now.
I succeeded to avoid this problem with 2 solutions:

Add option --renderToMemory ; but since this service is running all the time and that content will continue to grow, I do not want to spoil memory resources on this server which is already used for many other things
Set -d something-else-than-public ; I will probably choose this option for the longer term.

Anyway, it would certainly worth it to write this quite important change in the change log of v1.123.0. I have not found it, but maybe I've missed it (it was a long and technical changelog).

And finally, I also wanted to thank you very much for this amazing tool that Hugo is :tada:  It offers so much freedom in customization of our website, for the production version, but also for the development/preview/server one (which has some differences, including links to internal documentation and few other things for website editors).

The release notes for v0.123.0 contains this:

If you follow the breaking changes link...

Which brings you to https://github.com/gohugoio/hugo/issues/11987.
I'm sorry we broke your workflow, but this was an intentional breaking change. In addition to the alternatives you mentioned, you can also just clear you public directory before publishing your production site. In CI/CD workflows that happens by default.

Ok. I read that very quickly. I did not expect this message would mean that by default the server would run from disk. I should have been more careful. Thank you for your time to explain me this.


Code formatting and indentation corrected as per swift v5 standard

Thanks for suggestion of using SwiftFormat to apply code formatting - I've taken onboard your suggestion and integrated this into the build process now. I'm going to close this merge request rather than merge it in because I wanted to customize the rules that SwiftFormat applies to the code rather than accepting all of the defaults but wanted to say thanks for taking the time to raise a PR 👍🏻


Default 'Copy' on ContextMenu on DataView is bust

The Copy Item appears to just copy out an '[Object] object'  string.
See Toolbox 'News' example and 'DataView' example in grids

Yes -- absolutely.  Brendan -- can you whack the default context menu here on DataViewModel?  Think it probably was just a copy error from the very original version.

Sure thing


Kotlin Polishing

Use better Kotlin style in general
Updates from Detekt analysis

Coverage remained the same at 100.0% when pulling ca6b474b9653e5ccae0bd325bc83b35ddad91908 on jeffgbutler:kotlin-polishing into f8ad7be44f24d2591e3f258785e86d175a8ec986 on mybatis:master.


how to neded

Basic Information
I'm wanting to know some more information kind of a how to guide to adding different units to the build menu, I am wanting to add the ffaa transport ship/boat to the build menu for tanoa, might just make it a bit more interesting, I'm clueless to when it comes to editing missions
Mission version:
liberation tanoa
latest update
Map used:
Tanoa
Mods used:
FFAA and a few others in the future
Dedicated or local game?
GitHub or Workshop mission file?
Individual Things
Did you edit anything within the mission files?
If yes, what did you edit exactly?
Short Issue Description
looking for a 'how to' guide to adding units to the build menu
Expected Behaviour
Steps to Reproduce the Issue
Screenshots, Serverlog or any other helpful Information

Please look in the provided wiki:
Unpbo your mission, edit it and pbo it again.
https://github.com/Wyqer/kp_liberation/wiki/EN:ImportantHints
https://github.com/Wyqer/kp_liberation/wiki/EN:ConfigFiles


upgrade node version to current

The node-sass version 3.4.1 doesn't exist anymore and therefore the whole plugin doesn't work anymore.
I upgraded the version to the newest one which appears to work.
See also issue: https://github.com/GitbookIO/plugin-styles-sass/issues/6

Let's get it merged!

Will close this due to inactivity of their part.


Doesn't work (even show) on VSCodium 1.46.1

Can't even see it on Markplace at VSCodium 1.46.1

The reason for this is that VSCodium has changed from the VS-Code Marketplace to open-vsx.org with Version 1.46 (https://github.com/VSCodium/vscodium#extensions-and-the-marketplace, https://github.com/VSCodium/vscodium/issues/418).
In my opinion Settings Sync should also be published on open-vsx.org in the future.
But then there is still the problem that only extensions on the respective marketplace can be synchronized.
For now, my recommendation is to change the marketplace from VSCodium back to marketplace.visualstudio.com. How to do this is described here: https://github.com/VSCodium/vscodium/blob/master/DOCS.md#extensions-marketplace.

Duplicate of #1217


Support-Matrix für Office-Konfigurationen veraltet

Link to "Support-Matrix für Office-Konfigurationen" does link to an outdated document. Recent document should be this:
https://query.prod.cms.rt.microsoft.com/cms/api/am/binary/RE2OqRI

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

ID: 35e85436-141e-3d54-7743-365fba243861
Version Independent ID: c9afa702-9001-e0cb-3508-ab4b0d97cd46
Content: Windows Server end of support and Microsoft 365 Apps - Deploy Office
Content Source: DeployOffice/endofsupport/windows-server-support.md
Service: o365-proplus-itpro
GitHub Login: @DHB-MSFT
Microsoft Alias: danbrown

@matthiaseckerle   We're still working on resolving this issue. In the meantime, we've updated the URL for the matrix in our articles to point to the English version of the matrix (https://query.prod.cms.rt.microsoft.com/cms/api/am/binary/RE2OqRI). That's not ideal, but at least customers will be seeing the most current version of the matrix.
(It might take a few days for our localization process to pick up and implement this change to the URL in the articles.)
Once we get the issue resolved with the localized versions of the matrix, we'll go back to using fwlink 2111390 so that customers will once again be taken to the appropriate localized version of the matrix, where available.
Thanks again for bringing this issue to our attention.

@matthiaseckerle  FYI, The marketing team (who is responsible for the matrix) has decided to only publish the matrix in English. The most current version (10 January 2023) is available here: https://query.prod.cms.rt.microsoft.com/cms/api/am/binary/RE2OqRI. This is the URL we're using in our "end of support" articles.


Text buttons

In writing design documentation for components I'm outlining the use cases for links vs. buttons and realized I missed text buttons in my initial pass for button components. Text buttons are visually similar to links, but they're button elements (with the spacing/padding of button elements), and their use case is distinct + most often paired with another higher-emphasis button.

Nice catch! Thanks @shea-fitz

Closed by #474


Cancel stalled tile requests when they are no longer necessary

If you move a lot around a map (or zoom in) a lot of tile request are generated in quick succession which won't be visualized since you care only about the ones covering your view. Contrary to basemap requests, these requests are never cancelled so to finish rendering you need to wait for the tiler to process all of them.
The current behaviour overloads the tiler unnecessarily and, at least in big datasets, requires using a lot of memory to handle the responses. I've had several crashes in chrome because of this. It'd really nice if we cancelled the tile queries when they are no longer necessary, specially the stalled ones.

Stale


[C++] grpc crashes when using encrypted key

Please answer these questions before submitting your issue.
What version of gRPC and what language are you using?
1.7.x/C++
What operating system (Linux, Windows, …) and version?
Tested on Windows 10.
What runtime / compiler are you using (e.g. python version or version of gcc)
VS 2015, OpenSSL - 1.0.1-h
What did you do?
Create a server with security credentials. Used cert.pem and key_encrypted.pem.
What did you expect to see?
Server starts up successfully or fails gracefully.
What did you see instead?
The server crashes at startup when requesting an async rpc. The crash goes away when key_uncrypted.pem is used.
Callstack
Myappserver.exe!grpc_server_request_registered_call(grpc_server * server, void * rmp, grpc_call * * call, gpr_timespec * deadline, grpc_metadata_array * initial_metadata, grpc_byte_buffer * * optional_payload, grpc_completion_queue * cq_bound_to_call, grpc_completion_queue * cq_for_notification, void * tag) Line 1497	C
 	Myappserver.exe!grpc::ServerInterface::RegisteredAsyncRequest::IssueRequest(void * registered_method, grpc_byte_buffer * * payload, grpc::ServerCompletionQueue * notification_cq) Line 666	C++
 	Myappserver.exe!grpc::ServerInterface::PayloadAsyncRequest<Myapp::ComponentMetricsRequest>::PayloadAsyncRequest<Myapp::ComponentMetricsRequest>(void * registered_method, grpc::ServerInterface * server, grpc::ServerContext * context, grpc::ServerAsyncStreamingInterface * stream, grpc::CompletionQueue * call_cq, grpc::ServerCompletionQueue * notification_cq, void * tag, Myapp::ComponentMetricsRequest * request) Line 187	C++
 	Myappserver.exe!grpc::ServerInterface::RequestAsyncCall<Myapp::ComponentMetricsRequest>(grpc::RpcServiceMethod * method, grpc::ServerContext * context, grpc::ServerAsyncStreamingInterface * stream, grpc::CompletionQueue * call_cq, grpc::ServerCompletionQueue * notification_cq, void * tag, Myapp::ComponentMetricsRequest * message) Line 244	C++
 	Myappserver.exe!grpc::Service::RequestAsyncUnary<Myapp::ComponentMetricsRequest>(int index, grpc::ServerContext * context, Myapp::ComponentMetricsRequest * request, grpc::ServerAsyncStreamingInterface * stream, grpc::CompletionQueue * call_cq, grpc::ServerCompletionQueue * notification_cq, void * tag) Line 96	C++
 	Myappserver.exe!Myapp::MyappControllerSlaveService::> > > > > > > > > > > > > > > > > > >::RequestgetComponentMetrics(grpc::ServerContext * context, Myapp::ComponentMetricsRequest * request, grpc::ServerAsyncResponseWriter<Myapp::GetComponentMetricsGrpcResponse> * response, grpc::CompletionQueue * new_call_cq, grpc::ServerCompletionQueue * notification_cq, void * tag) Line 777	C++
 	[External Code]
 	Myappserver.exe!??@0fa2038ac62cf9040b4dd3324bc6a549@(> > > > > > > > > > > > > > > > > > > > > > > > > > > * service, grpc::ServerCompletionQueue * cq, std::function<void __cdecl(grpc::Service *,grpc::ServerCompletionQueue *,Myapp::Grpc::GrpcEndpoint *)> createRpc, std::function<void __cdecl(Myapp::Grpc::RpcBase &,google::protobuf::Message const *)> processIncomingRequest, <Myapp::MyappControllerSlaveService::Service> > > > > > > > > > > > > > > > > > > > > > > > > > > *,grpc::ServerContext *,Myapp::ComponentMetricsRequest *,grpc::ServerAsyncResponseWriter<Myapp::GetComponentMetricsGrpcResponse> *,grpc::CompletionQueue *,grpc::ServerCompletionQueue *,void *)> requestRpc, unsigned short componentId, unsigned short commandId, Myapp::Grpc::GrpcEndpoint * endpoint) Line 390	C++
 	Myappserver.exe!Myapp::creategetComponentMetricsGrpc(grpc::Service * service, grpc::ServerCompletionQueue * cq, Myapp::Grpc::GrpcEndpoint * endpoint) Line 118	C++
 	[External Code]
 	Myappserver.exe!Myapp::Grpc::GrpcEndpoint::startGrpcServer() Line 406	C++

Crashes at
  size_t cq_idx;
  for (cq_idx = 0; cq_idx < server->cq_count; cq_idx++) {
    if (server->cqs[cq_idx] == cq_for_notification) {
      break;
    }
  }

What does the label "transient/fixit candidate" mean? Why is it not a "priority/P0" issue (I assume that is the highest).
I ask because in the last community meeting, I brought up the topic of many stability issues with grpc that are not getting enough attention and I was told that stability issues are the highest priority as long as there is a repro. Are you having any troubles with the repro here?
As an added info, the crash is reproducible on Linux with OpenSSL 1.0-2.
Thanks.

Actually, please disregard the bug. Turns out I was using the unique_ptr returned by serverbuilder without checking for it's validity so it was a bug on my end.
Apologies for the hassle (Though it'd still be helpful if the bug label and their meaning is documented somewhere).
Thanks.


Kafka producer and consumer not working in TLS enabled cluster

The 3 node kafka cluster is setup with TLS with inter broker communication protocol as SSL. I have created a test topic with 3 partitions and 3 replication factor. The kafka-node producer is able to send message to one of partition, but gives error for other two. Similarly kafka-node consumer is not getting any messages from two of the brokers. kafka console producer and consumers are working fine with the setup.
Please note that kafka-node producer and consumer are working fie with PLAINTEXT kafka listeners.
I am using following code to produce a test message -
var fs = require('fs');
var kafka = require('kafka-node');

var partition = process.argv[2] || 0;
var Client = kafka.Client;
var options = {
    pfx: fs.readFileSync('<PKCS_FILE>'),
    passphrase: '<PASSPHRASE>',
    requestCert: true,
    rejectUnauthorized: false
};

var kafkaClient = new Client('<ZK_SYSTEM>:<ZK_PORT>', 'kafka-node', {}, {}, options);
var producer;
var Producer = kafka.Producer;

function createProducer() {
    producer = new Producer(kafkaClient);
    producer.on('error', function (err) {
        throw new Error(err);
    });

    producer.on('ready', function () {
        console.log("producer is ready !!!");
        console.log('sending data to partition- ' + partition);
        producer.send([
            {topic: 'test3', messages: 'message', partition: partition}
        ], function (err, result) {
            if (err) {
                throw new Error(err);
            }
            console.log(err, result);
        });
    });
}

createProducer();

It gives following error for partition 0 and 1.
$ node ./producer.js 0
producer is ready !!!
sending data to partition- 0
/opt/nodetests/kafka-ssl/producer.js:34
                        throw new Error(err);
                        ^

Error: Error: UnknownTopicOrPartition
    at /opt/nodetests/kafka-ssl/producer.js:34:10
    at /opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/client.js:194:17
    at null.<anonymous> (/opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/client.js:244:31)
    at Client.handleReceivedData (/opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/client.js:622:18)
    at TLSSocket.<anonymous> (/opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/client.js:594:14)
    at emitOne (events.js:77:13)
    at TLSSocket.emit (events.js:169:7)
    at readableAddChunk (_stream_readable.js:153:18)
    at TLSSocket.Readable.push (_stream_readable.js:111:10)
    at TLSWrap.onread (net.js:531:20)

For partition 2, it is able to send the message with no error.
$ node ./producer.js 2
producer is ready !!!
sending data to partition- 2
null { test3: { '2': 6 } }

Sometimes with some other topic, we are getting following errors (with stack).
$ node ./producer.js
producer is ready !!!
Error: NotLeaderForPartition
    at Object.<anonymous> (/opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/protocol/protocol.js:417:29)
    at Object.self.tap (/opt/nodetests/kafka-ssl/node_modules/kafka-node/node_modules/binary/index.js:248:12)
    at Object.decodePartitions (/opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/protocol/protocol.js:415:14)
    at Object.self.loop (/opt/nodetests/kafka-ssl/node_modules/kafka-node/node_modules/binary/index.js:267:16)
    at Object.<anonymous> (/opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/protocol/protocol.js:60:14)
    at Object.self.loop (/opt/nodetests/kafka-ssl/node_modules/kafka-node/node_modules/binary/index.js:267:16)
    at decodeProduceResponse (/opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/protocol/protocol.js:407:10)
    at Client.handleReceivedData (/opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/client.js:620:22)
    at TLSSocket.<anonymous> (/opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/client.js:594:14)
    at emitOne (events.js:77:13)
    at TLSSocket.emit (events.js:169:7)
    at readableAddChunk (_stream_readable.js:153:18)
    at TLSSocket.Readable.push (_stream_readable.js:111:10)
    at TLSWrap.onread (net.js:531:20)
/opt/nodetests/kafka-ssl/producer.js:44
                if (err) throw new Error(err);
                         ^

Error: Error: NotLeaderForPartition
    at /opt/nodetests/kafka-ssl/producer.js:44:18
    at /opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/client.js:194:17
    at null.<anonymous> (/opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/client.js:244:31)
    at Client.handleReceivedData (/opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/client.js:622:18)
    at TLSSocket.<anonymous> (/opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/client.js:594:14)
    at emitOne (events.js:77:13)
    at TLSSocket.emit (events.js:169:7)
    at readableAddChunk (_stream_readable.js:153:18)
    at TLSSocket.Readable.push (_stream_readable.js:111:10)
    at TLSWrap.onread (net.js:531:20)

Also no issues noticed with kafka single machine setup with TLS.

Thanks for the report. I will look into this.

@kumar-b could you attach your producer send block to  connect event of your kafkaClient instead of ready and let me know how that works for you.

It gave same error as reported above.

Hmm are the pfx and passphrase required? We don't use them to connect to our cluster. Also what version of kafka are you running?

pfx file contains the key and cert of the client and it is password protected.  Since the client is able to connect to other brokers, pfx part may not be the cause. We are running kafka 0.10

Try calling kafkaClient.refreshMetadata(topic, callback) before you send. You can also echo out the contents of kafkaClient.topicMetadata that may provide more information.

Refreshed metadata before call and the topic metadata for three different run is echoed out.
$ node ./producer-connect.js 1 test3
sending data to topic- test3
sending data to partition- 1
topic metadata: {"test3":{"0":{"topic":"test3","partition":0,"leader":2,"replicas":[2],"isr":[2]},"1":{"topic":"test3","partition":1,"leader":3,"replicas":[3],"isr":[3]},"2":{"topic":"test3","partition":2,"leader":1,"replicas":[1],"isr":[1]}}}
producer is ready !!!
sending data to partition- 1
/opt/nodetests/kafka-ssl/producer-connect.js:44
                                throw new Error(err);
                                ^

Error: Error: UnknownTopicOrPartition
    at /opt/nodetests/kafka-ssl/producer-connect.js:44:11
    at /opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/client.js:194:17
    at null.<anonymous> (/opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/client.js:244:31)
    at Client.handleReceivedData (/opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/client.js:622:18)
    at TLSSocket.<anonymous> (/opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/client.js:594:14)
    at emitOne (events.js:77:13)
    at TLSSocket.emit (events.js:169:7)
    at readableAddChunk (_stream_readable.js:153:18)
    at TLSSocket.Readable.push (_stream_readable.js:111:10)
    at TLSWrap.onread (net.js:531:20)
$ node ./producer-connect.js 0 test3
sending data to topic- test3
sending data to partition- 0
topic metadata: {"test3":{"0":{"topic":"test3","partition":0,"leader":2,"replicas":[2],"isr":[2]},"1":{"topic":"test3","partition":1,"leader":3,"replicas":[3],"isr":[3]},"2":{"topic":"test3","partition":2,"leader":1,"replicas":[1],"isr":[1]}}}
producer is ready !!!
sending data to partition- 0
/opt/nodetests/kafka-ssl/producer-connect.js:44
                                throw new Error(err);
                                ^

Error: Error: UnknownTopicOrPartition
    at /opt/nodetests/kafka-ssl/producer-connect.js:44:11
    at /opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/client.js:194:17
    at null.<anonymous> (/opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/client.js:244:31)
    at Client.handleReceivedData (/opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/client.js:622:18)
    at TLSSocket.<anonymous> (/opt/nodetests/kafka-ssl/node_modules/kafka-node/lib/client.js:594:14)
    at emitOne (events.js:77:13)
    at TLSSocket.emit (events.js:169:7)
    at readableAddChunk (_stream_readable.js:153:18)
    at TLSSocket.Readable.push (_stream_readable.js:111:10)
    at TLSWrap.onread (net.js:531:20)
$ node ./producer-connect.js 2 test3
sending data to topic- test3
sending data to partition- 2
topic metadata: {"test3":{"0":{"topic":"test3","partition":0,"leader":2,"replicas":[2],"isr":[2]},"1":{"topic":"test3","partition":1,"leader":3,"replicas":[3],"isr":[3]},"2":{"topic":"test3","partition":2,"leader":1,"replicas":[1],"isr":[1]}}}
producer is ready !!!
sending data to partition- 2
null { test3: { '2': 8 } }

the above run was with topic having 1 replication factor and 3 partitions. But same behavior was observed with 3 partitions and 3 replication factor.

Hi @hyperlink
With some debug, I found out the root cause.
Since from kafka 0.9 the host and port property were deprecated and listeners was used as single point of truth, we have setup kafka brokers without host and port. So metadata of a broker looks like this way
//  metadata for leader- 1
{"jmx_port":-1,"timestamp":"1470139709693","endpoints":["SSL://<HOST1>:<PORT1>"],"host":null,"version":3,"port":-1,"sslHost":"<HOST1>","sslPort":<PORT1>}

Here host and port are null and -1. In a cluster all the brokers will have host and port as null and -1.
But in kafka-node after connect, it gets the broker metadata, extracts the host and port and stores as key value pair in self.brokers.
https://github.com/SOHU-Co/kafka-node/blob/master/lib/client.js#L88
var broker = brokers[key];
self.setupBroker(broker.host, broker.port, false, self.brokers);

Since all brokers has same null and -1 as host and port, it will be overwritten. Then Client.prototype.brokerForLeader gives the wrong broker information. So producer and consumer fails.
https://github.com/SOHU-Co/kafka-node/blob/master/lib/client.js#L525
I hope the analysis will be helpful in fixing the issue. Also please analyze the impact of the same on other parts of the module.

Actually this is not at all any configuration issue. host and port will be null and -1 for SSL mode and sslHost, sslPort will have real values. So setup brokers part should be changed based on self.ssl

Great debugging @kumar-b. I hope to have this fixed soon.

@kumar-b could you temporarily point your package.json to point to the fix-ssl-only-clusters branch and see if this issue is resolved for you?

It works with positive tests. I didn't try other negative tests.


Shortcut keys

Add shortcut keys to control the viewer like f to toggle fullscreen.

I think needs a simple system to register them, unless openseadragon provides something for this,.

Awesome, thanks. Let me know if you have any questions.

Awesome, thanks. Let me know if you have any questions.

Alright, thanks.

@Nargonath are you still working on this ?

Yes I am. I just had some other stuff to take care in other open source projects I maintain. 😃 I wrote some code, now I just need to test it. I should be able to do it during the weekend.

@Nargonath Oh lol, dont worry just checking

No worries. I might ask you some questions about testing using the examples if I cannot figure it out myself.

Isn't this issue closed by the PR mentioned above?

@Nargonath No, needs improvements and more shortcut keys

I see. I deleted the work I did because it was covered by the above MR.

Openseadragon Shortcut keysw:
https://github.com/openseadragon/openseadragon/blob/92904b7ac9f46583a3ba9237446c0608a0eb2ca0/src/viewer.js#L2583


Revert "Simplify children handling."

Reverts trueadm/inferno#56

@tejacques Did you find a solution on this, and / or confronted @trueadm with it?


~ not resolving to root directory

I'm attempting to get to my base styles they're relative to the context directory specified in webpack's config (link).
context
├── base
│   ├── _colors.scss
│   ├── _fonts.scss
│   ├── _layout.scss
│   └── _panel.scss
│   └── _colors.scss
└── admin
    └── components
        └── modal
            └── user
                └── preview
                    └── preview.scss

// preview.scss
@import "~base/colors"
.preview {
  color: $red
}

Please use Stackoverflow for questions using the webpack tag. This way, a much larger user-base is able to benefit from the answer and there are more people how actually answer it. If this is a bug report, please provide a small example project.


Set total_len when stripping a VLAN from pkt-in

The ofp_packet_in total_len field is not adjusted by any library
functions when we strip a VLAN tag from a data packet.  In addition,
deserializing and reserializing a packet might cause it to lose some
padding bytes.  To account for this, just set the total length to
whatever the length of the serialized packet is.

Thanks for the fix!


Problem training specific model

@Plachtaa ,
Thank you for sharing a brilliant repository, I have some questions.
After many test with your huggingface spaces below
https://huggingface.co/spaces/Plachta/VITS-Umamusume-voice-synthesizer
I found that the japanese intonation on the japanese only version (the one with only Umamusume) is better than the trilingual one.
Could you give me more detailed information about the base model you use for the Japanese only version, or any specific details for fine-tuning them.
I tried using the same config with the G_jp model on above Huggingface space repository but an error occurred
RuntimeError: Error(s) in loading state_dict for SynthesizerTrn:
        size mismatch for enc_p.emb.weight: copying a param with shape torch.Size([40, 192]) from checkpoint, the shape in current model is torch.Size([68, 192]).

Any advice is appreciate.
Thank you in advance

size mismatch is because the trilingual model use a different phoneme vocabulary (with 68 phonemes) from the pure Japanese one (with 40 phonemes).
If you intend to use the Japanese model, you should cony its corresponding config file (config.json) as well, which states the g2p tool used and its phoneme vocabulary.

@Plachtaa
Thank you for your advice. I have been able to finetune model mention above with good result!


Is it possible to express enqueue job B once job A completes?

As titled.
Similar idea to celery (a python job queue tool) canvas.

There's nothing explicit, but any job can call this.queue.enqueue() within itself


on login - javascript error Cannot read property 'identities' of undefined

TypeError: Cannot read property 'identities' of undefined
at Scope.$scope.identityExists (account.js:52)

duplicate of #27


[Bug] Inconsistent time_to_sec return value with timediff

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
2.0.1
What's Wrong?
When using time_to_sec on a value of type TIME returned by timediff, the result is always 0 if the timediff has been made between two date-like strings, and the result is the good one when the timediff has been made between 2 date objects.
What You Expected?
We would expect time_to_sec to always return the amount of seconds in the TIME object returned by timediff
How to Reproduce?
select time_to_sec(timediff('2024-01-22', '2024-01-15')) as seconds; returns 0 (wrong)
select time_to_sec(timediff(to_date('2024-01-22'), to_date('2024-01-15'))) as seconds; returns 604 800 (right)
while both timediff('2024-01-22', '2024-01-15') and timediff(to_date('2024-01-22'), to_date('2024-01-15')) return 168:00:00 which is the right value
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

fix  in https://github.com/apache/doris/pull/30248