text:
Add/Ignore

I noticed that your plugin has an "Ignore" feature, which is nice.
However, when you refresh the page these settings seem to be lost. It would be valuable to me to have some kind of persistence for these settings. I have two ideas:

LanguageTool's API supports adding words to a custom dictionary. Maybe this could be supported?
Another option would be to use local storage.

There are advantages to each of them; maybe both a worthwhile. What do you think @KnowZero ?

I think the best option would be a simple adapter that lets the user choose whether they wish to store it in localstorage or custom server.
localstorage is probably the least effort since all that requires is just dumping sgstore.ignore and retrieving it on initialization. But probably people would want custom servers cause users can use other devices. Also, ignore would probably would need fine tuning cause ignore in 1 document doesn't mean they want to ignore in all documents. or they might.
As for adding words, is that a normal feature or premium only?
That said, currently the development of this is in maintenance mode. Which means I am only doing bugfixes. I just added TinyMCE v5 to see if it still works properly and even then v5 is missing out on the progress bar which if you click in v4 will open a dialog and lets you fix things that way for those who prefer.
I have a few projects that I need to finish first before I can come back to this one. This entire thing needs to be refactored cause if you saw the source it is a bit of a mess.
The plan is to:

Remove dependence on jquery
Stop relying on lazy regexes
Spin off any UI into its own module so that it isn't tied down to TinyMCE v4 (Reason for why dialogs don't work in v5)
Support virtualdom based editors like ProseMirror and Slate (Personally, I actually planned to move onto ProseMirror over TinyMCE since it better meets my needs (Though Slate is good too, but I saw better performance with ProseMirror for large documents, which may have been fixed now its been a while), but I will support all of them)

But again, that will have to come after I finish the other stuff. Since the project that I was working this on for got delayed so until I finish the other stuff, I am only fixing bugs.
If you want you want, you can add it, as mentioned, just copy over sgstore.ignore into localstorage and then reload it back up on init.

I think the best option would be a simple adapter that lets the user choose whether they wish to store it in localstorage or custom server.

Makes sense.

As for adding words, is that a normal feature or premium only?

I believe that is a premium-only feature for LT.

If you want you want, you can add it, as mentioned, just copy over sgstore.ignore into localstorage and then reload it back up on init.

Yeah, I'm not super familiar with localstorage APIs, but I'll probably take a stab at that.

authors: KnowZero tooolbox


text:
Update .gitattributes file

These folders and files are not needed when fetching the package through Composer.
However, i'm not sure about the utils folder and phpstan-extension.neon.

Hi, I'm going through old opended issues and PRs to cleanup too-long-opened items and close them. Thank you for your contribution and time.
We needs a feature with test and passing CI to be able to merge it without letting a bug in.
If this is still relevant for you, please add code with test and make CI pass first. Then we'll able to merge your feature without further waiting :+1:
Thank for understanding.

Hello @TomasVotruba
I think your automated script closed this PR by mistake. This is not a new feature that require tests, but juste an update of the actual .gitattributes file which i think is still relevant!
Would you consider re-opening and merging this PR ? Or should I open a new one ?

It was not automated :)
The files in this PR were outdated. So new PR would be better, I can process it today :+1:
Thank you

source: gharchive/pull-request

repo: KnpLabs/DoctrineBehaviors

license: MIT

license_type: permissive

license_source: github-api


text:
Is there an automatic way to add a class to all  tags generated by the menu?

I use the Foundation CSS Framework and for the menu to look correct all the <li> tags need to have a class="menu-text" attribute. Also, the <ul> tags (top level and submenu) need to have a class="menu vertical" or class="menu horizontal" attribute. Is there a way to add this for all items at once? ->addAttribute after each and every link works. But what if there are a lot of them? It becomes just as much work as coding the menu by hand.
Thanks!

You can override the default template used to render the menu and add the new class. When using twig, you would do something like:
$menuRenderer = new \Knp\Menu\Renderer\TwigRenderer($twig, 'custom_menu.html.twig', $itemMatcher);

{# custom_menu.html.twig #}
{% extends 'knp_menu.html.twig' %}

{% block list %}
    {% set listAttributes = [listAttributes.class is not empty ? listAttributes.class : '', 'menu horizontal']|join(' ') %}
    {{ parent() }}
{% endblock %}

{% block item %}
    {% set classes = item.attribute('class') is not empty ? [item.attribute('class')] : [] %}
    {% do item.setAttribute('class', classes|merge(['menu-text'])|join(' ')) %}
    {{ parent() }}
{% endblock %}

Or when using the list renderer, you would override the renderItem() and renderList() methods.

source: gharchive/issue

authors: WouterJ adrianocr

repo: KnpLabs/KnpMenu

license: mit

license_source: bigquery


text:
MetSys restart failed on basically clean project

I just went to the asset store and downloaded the latest version into my barebone project, which consists of nothing but a Player with FSM and a tilemap.
I am using Godot 4.2.1 and I do not see how I can have naming conflicts.

What errors appear in MetroidvaniaSystem.gd?

When I try to set the checkmark in the Project Settings > Plugins tab, it will automatically close the IDE, reopen it and show the error popup from my original post.

I just tried installing in an empty project and it works after restart. I excluded the SampleProject though.
Try removing .godot folder maybe.

So it was unrelated to MetSys.
When you install or create an addon, the addons folder is created automatically. It's better to use that than to create it yourself.

authors: KoBeWi nyrrrr

repo: KoBeWi/Metroidvania-System


text:
Prevent page fault when uninstalling jumper

page fault occurs when running DARKSOULS.exe in 64bit wine prefix
something must reprotect that address after jumper is initialized
calling unprotect() again before uninstall() does the job!

looking at the code i would say unprotect is wrong..
auto address = (void *)((uintptr_t)src & ~(pagesize - 1));
mprotect(address, header.size(), PROT_READ | PROT_WRITE | PROT_EXEC);

the adress gets the nearest page, but the size isn't,
it only works when pagesize is a power of 2 .. and src+header.size() doesn't cross a second page..
wouldn't be something like this more correct ?
auto address = (void *)((uintptr_t)src / pagesize * pagesize));
auto size = ((uintptr_t)src + header.size() + pagesize - 1) / pagesize * pagesize - address;
mprotect(address, size, PROT_READ | PROT_WRITE | PROT_EXEC);

.. i think we should put unprotect() into jumper::install and jumper::uninstall

Thanks for your comments, you were right that unprotect() was fishy.
IMO assuming pagesize is a power of 2 is pretty safe, it's the only sane way to implement virtual memory mapping.  I've added an assert to document/check this assumption.
The length calculation was incorrect as you pointed out, it was only unprotecting pagestart+N instead of src+N.  If src+N crossed a page boundary, we would get a page fault.  I tested this fix on its own, but still got the page fault (reprotection is still needed to fix that).
Agreed that always unprotecting before install/uninstall is the best approach, this way it's bullet proof.  I've deduplicated the install/uninstall routines as swap_header(), inlined unprotect there, and added a sanity check on the ordering of install/uninstall just in case.

Oh, I also tested that mprotect really requires the address to be aligned to page start.  The documentation suggests this might not be needed, but testing confirms that it is (at least on my system).  Without this alignment I was instantly getting page faults on apps which normally work fine.
Aligning the length to page end as in your example shouldn't be necessary, the documentation is pretty clear that all pages on the range will be modified.

authors: KoKuToru flatwhatson

repo: KoKuToru/koku-xinput-wine


text:
Fix #334 - event name should be Kebab case

Event names should be kebab case, as per vue recommendations
To avoid breaking changes, we still emit the original event name.
This fix closes issue #334

@Cl3MM thanks! No worries about the coverage, is rather buggy at the moment. Merging


text:
Add "set/activate open window" feature

Also allows resetting the "open window" state back.
To check if a window is open in a zone, call
IsOpenWindowDetected() on a State object.
You need to add a using of KoenZomers.Tado.Api.Helpers to see the
extension method.

Finally got Tado to detect an open window.

repo: KoenZomers/TadoApi


text:
add S7 3p pubkeys

Add the 3rd party pubkeys for the Season hardfork, scheduled on or around 30th June.
@sillyghost TODO: Calculate approximate block height of AYA at June 30th, 2023 (13:37:33 UTC) For greater accuracy, this should not be done based on assumed chain blocktime, but on actual block time over the last 90 days. Please ensure chain is being mined regularly as the hardfork aproaches.
Source for pubeys: https://github.com/KomodoPlatform/dPoW/blob/season-seven/iguana/3rd_party
Source for hardfork date: https://github.com/KomodoPlatform/komodo/blob/master/src/komodo_hardfork.cpp#L15

Ready for review. Block height defined by Rack001 from AYA based on current block production


text:
Helper function to log incoming request

Fixes : #71

@astoycos  I tried this but this is currently not working.
But this was one approach i was thinking.
Other was adding this method inside :
https://github.com/Kong/blixt/blob/main/dataplane/ebpf/src/bindings.rs#L111 as one of the function.
But i thought it is not the right place and utility is only correct.

repo: Kong/blixt


text:
Helm install fails at kong.renderEnv

I was trying to get kong installed via our CD pipeline (fluxv2) and would always get the following error:
 Helm install failed: template: kong/templates/deployment.yaml:69:10: executing "kong/templates/deployment.yaml"
at <include "kong.controller-container" .>: error calling include: template: kong/templates/_helpers.tpl:383:4:
executing "kong.controller-container" at <include "kong.ingressController.env" .>:
error calling include: template: kong/templates/_helpers.tpl:599:15:
executing "kong.renderEnv" at <.>: wrong type for value; expected map[string]interface {}; got string   5s

After digging a bit more into helper.tpl I've managed to narrow the issue down to the following line:
{{- $_ := set $autoEnv "CONTROLLER_KONG_ADMIN_TLS_SKIP_VERIFY" "true" -}}

The above sets the value to "true" (string), which is then merged with default value (bool) from values.yaml
    kong_admin_tls_skip_verify: true

This makes mergeOverwrite fail silently and produce an empty string which, when passed down to renderEnv results in the original error message.
I still haven't figured out why the behaviour is different between helm CLI client and the library, it seems like they both call the same functions, however in helm CLI the above set command results in the actual boolean value being assigned to that key. Not sure why.
I thought it'd be good to have a fix here by simply changing the above set command to:
{{- $_ := set $autoEnv "CONTROLLER_KONG_ADMIN_TLS_SKIP_VERIFY" true -}}

This way the merge operation succeeds and helm chart gets installed correctly. Happy to do a PR if you think that solution is ok.

closing as a duplicate of #224

repo: Kong/charts


text:
fix(customize) sets permissions in line with Kong install

When using the customize feature, anything installed will be owned by root. This updates the files to be identically owned as the original Kong installed plugins.
Here's after a customize run (the readme example), installing 2 plugins:
s$ docker run -it --rm kong_custom /bin/bash
bash-5.1$ cd /usr/local/share/lua/5.1/kong/plugins
bash-5.1$ ls -la
total 172
drwxr-xr-x    1 1000     1000          4096 Nov 16 12:25 .
drwxr-xr-x    1 1000     1000          4096 Nov 13 05:49 ..
drwxr-xr-x    3 1000     1000          4096 Nov 13 05:49 acl
drwxr-xr-x    4 1000     1000          4096 Nov 13 05:49 acme
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 aws-lambda
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 azure-functions
-rw-r--r--    1 1000     1000          1288 Sep 28 18:54 base_plugin.lua
drwxr-xr-x    3 1000     1000          4096 Nov 13 05:49 basic-auth
drwxr-xr-x    3 1000     1000          4096 Nov 13 05:49 bot-detection
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 correlation-id
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 cors
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 datadog
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 file-log
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 grpc-gateway
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 grpc-web
drwxr-xr-x    3 1000     1000          4096 Nov 13 05:49 hmac-auth
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 http-log
drwxr-xr-x    2 root     root          4096 Nov 16 12:25 http-to-https
drwxr-xr-x    3 1000     1000          4096 Nov 13 05:49 ip-restriction
drwxr-xr-x    3 1000     1000          4096 Nov 13 05:49 jwt
drwxr-xr-x    3 1000     1000          4096 Nov 13 05:49 key-auth
drwxr-xr-x    2 root     root          4096 Nov 16 12:25 kong-upstream-jwt
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 ldap-auth
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 log-serializers
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 loggly
drwxr-xr-x    4 1000     1000          4096 Nov 13 05:49 oauth2
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 post-function
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 pre-function
drwxr-xr-x    3 1000     1000          4096 Nov 13 05:49 prometheus
drwxr-xr-x    3 1000     1000          4096 Nov 13 05:49 proxy-cache
drwxr-xr-x    4 1000     1000          4096 Nov 13 05:49 rate-limiting
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 request-size-limiting
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 request-termination
drwxr-xr-x    3 1000     1000          4096 Nov 13 05:49 request-transformer
drwxr-xr-x    4 1000     1000          4096 Nov 13 05:49 response-ratelimiting
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 response-transformer
drwxr-xr-x    4 1000     1000          4096 Nov 13 05:49 session
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 statsd
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 syslog
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 tcp-log
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 udp-log
drwxr-xr-x    2 1000     1000          4096 Nov 13 05:49 zipkin
bash-5.1$

Hey @Tieske. Is this change still needed? How does it relate to https://github.com/Kong/docker-kong/pull/546? Can one or both be closed if no longer required?

needs updating after https://github.com/Kong/docker-kong/pull/632 gets merged

Closing this due to lack of activity. Please re-open if needed.

This is still needed. It's just that #632 has been a moving target, so that needs fixing first

authors: Tieske gszr hbagdi

repo: Kong/docker-kong


text:
Simplify PR template

Attempting to make the PR template clearer.

Combined the summary and reason sections into one
Added changelog prompt
Added checklist - is this useful at all? If we want to use it, what else should be part of this checklist?
Left the info about labels in. It's a big block block of text that I don't think anyone reads, but the info needs to be there to cover why the builds fail.

Or does it? Should it be a bot comment instead, if someone doesn't add a label?

All feedback and ideas welcome.

Or does it? Should it be a bot comment instead, if someone doesn't add a label?

The label action just got the ability to add custom comments!

If the change log entry is in the PR it will conflict with other PR's that add a change log entry like Amy's monthly changelog PR. Not that we should discourage people from adding change log entries but I think we can omit that checkbox. I think the goal is for change log entries to be in the repo for the project's repo and Aha.
I would like to see a way to ask people if they are pointing to the correct branch and encourage them to ask for help if they don't know. That way if someone decides to follow the template the docs team won't merge something to main that is supposed to be for an upcoming release.
here's a rough example of what I mean:
** Feature work only **
[ ] Pointed to correct feature branch. (ex: release/gateway-3.2, deck-1.1.2)

What if we didn't hide this
 <!-- What did you change and why? -->

<!-- Include any supporting resources, e.g. link to a Jira ticket, GH issue, FTI, Slack, Aha, etc. -->

If someone doesn't notice it and doesn't add information when they submit a PR the text will appear and serve as a second notification to please fill that out.

If the change log entry is in the PR it will conflict with other PR's that add a change log entry like Amy's monthly changelog PR. Not that we should discourage people from adding change log entries but I think we can omit that checkbox. I think the goal is for change log entries to be in the repo for the project's repo and Aha.

I agree with that, but I'd also love for someone submitting this to add a changelog entry to the description. They don't need to add it to a changelog file. That way, if we have to compile a changelog after the fact, those descriptions are easy to track down.
However, I can also see the argument for saying that the PR summary itself is our changelog entry.

I would like to see a way to ask people if they are pointing to the correct branch and encourage them to ask for help if they don't know. That way if someone decides to follow the template the docs team won't merge something to main that is supposed to be for an upcoming release.

here's a rough example of what I mean:
** Feature work only **
[ ] Pointed to correct feature branch. (ex: release/gateway-3.2, deck-1.1.2)

Love the idea, let's do that.

Good point, I think I'd rather deal with the merge conflict potential and have an abundance of changelog entries than the opposite problem.

authors: Guaris lena-larionova mheap


text:
adding kind install instructions for KIC

Description
What did you change and why?
I added a section on installing Kong for Kubernetes with Kong Enterprise for KIC on kind. I feel we have make it easy for developers to deploy Kong enterprise and be able to play around so they can use and promote the platform internally.
Testing instructions
Netlify link: /kubernetes-ingress-controller/latest/deployment/kind/
Checklist

[x] Review label added
[x] PR pointed to correct branch (main for immediate publishing, or a release branch: e.g. release/gateway-3.2, release/deck-1.17)

Hey Lena,
I think all the changes are acceptable. I think I have committed most the
suggestions. Do you need anything else from me?
On Thu, May 18, 2023 at 8:04 AM lena-larionova @.***>
wrote:

@.**** requested changes on this pull request.
In app/_src/kubernetes-ingress-controller/deployment/kind.md
https://github.com/Kong/docs.konghq.com/pull/5365#discussion_r1197092198
:

@@ -0,0 +1,184 @@
+---
+title: Kong for Kubernetes with Kong Enterprise
+---

+This guide walks through setting up the {{site.kic_product_name}} with Kong
+Enterprise. This architecture is described in detail in this doc.
+
+### Set up Kind cluster
+
+The first thing we need to do is create a kind cluster to deploy {{site.kic_product_name}} to:
⬇ Suggested change
-The first thing we need to do is create a kind cluster to deploy {{site.kic_product_name}} to:
+First, create a kind cluster to deploy {{site.kic_product_name}} to:

In app/_src/kubernetes-ingress-controller/deployment/kind.md
https://github.com/Kong/docs.konghq.com/pull/5365#discussion_r1197093010
:

+nodes:
+- role: control-plane

kubeadmConfigPatches:

|

kind: InitConfiguration
extraPortMappings:

containerPort: 80

hostPort: 80
protocol: TCP

containerPort: 443

hostPort: 443
protocol: TCP
+EOF
+```

+We're going create our resources in the kong namespace. Create this now:
⬇ Suggested change
-We're going create our resources in the kong namespace. Create this now:
+Next, create a kong namespace for your resources:

In app/_src/kubernetes-ingress-controller/deployment/kind.md
https://github.com/Kong/docs.konghq.com/pull/5365#discussion_r1197093112
:

|

kind: InitConfiguration
extraPortMappings:

containerPort: 80

hostPort: 80
protocol: TCP

containerPort: 443

hostPort: 443
protocol: TCP
+EOF
+```

+We're going create our resources in the kong namespace. Create this now:
+
+```bash
+$ kubectl create namespace kong
⬇ Suggested change
-$ kubectl create namespace kong
+kubectl create namespace kong

In app/_src/kubernetes-ingress-controller/deployment/kind.md
https://github.com/Kong/docs.konghq.com/pull/5365#discussion_r1197093244
:

protocol: TCP
+EOF
+```

+We're going create our resources in the kong namespace. Create this now:
+
+bash +$ kubectl create namespace kong +
+
+### Create a Kong Enterprise License secret (optional)
+
+If you have a Kong Enterprise license, save it to disk as license.json and create a secret:
+
+```bash
+$ kubectl create secret generic kong-enterprise-license --from-file=license=./license.json -n kong
⬇ Suggested change
-$ kubectl create secret generic kong-enterprise-license --from-file=license=./license.json -n kong
+kubectl create secret generic kong-enterprise-license --from-file=license=./license.json -n kong

In app/_src/kubernetes-ingress-controller/deployment/kind.md
https://github.com/Kong/docs.konghq.com/pull/5365#discussion_r1197093475
:

+```

+### Create a Kong Enterprise License secret (optional)
+
+If you have a Kong Enterprise license, save it to disk as license.json and create a secret:
+
+bash +$ kubectl create secret generic kong-enterprise-license --from-file=license=./license.json -n kong +
+
+### Set your Kong Manager password
+
+The Kong Manager UI requires authentication. {{site.kic_product_name}} uses the kong-enterprise-superuser-password
+secret to set the default value for the default kong_admin user.
+
+Run the following, replacing cloudnative with a random password of your choice and note it down.
⬇ Suggested change
-Run the following, replacing cloudnative with a random password of your choice and note it down.
+Run the following, replacing cloudnative with a random password of your choice and note it down:

In app/_src/kubernetes-ingress-controller/deployment/kind.md
https://github.com/Kong/docs.konghq.com/pull/5365#discussion_r1197093557
:

+If you have a Kong Enterprise license, save it to disk as license.json and create a secret:
+
+bash +$ kubectl create secret generic kong-enterprise-license --from-file=license=./license.json -n kong +
+
+### Set your Kong Manager password
+
+The Kong Manager UI requires authentication. {{site.kic_product_name}} uses the kong-enterprise-superuser-password
+secret to set the default value for the default kong_admin user.
+
+Run the following, replacing cloudnative with a random password of your choice and note it down.
+
+```bash
+$ kubectl create secret generic kong-enterprise-superuser-password  -n kong --from-literal=password=cloudnative
⬇ Suggested change
-$ kubectl create secret generic kong-enterprise-superuser-password  -n kong --from-literal=password=cloudnative
+kubectl create secret generic kong-enterprise-superuser-password  -n kong --from-literal=password=cloudnative

In app/_src/kubernetes-ingress-controller/deployment/kind.md
https://github.com/Kong/docs.konghq.com/pull/5365#discussion_r1197093693
:

+bash +$ kubectl create secret generic kong-enterprise-license --from-file=license=./license.json -n kong +

+### Set your Kong Manager password
+
+The Kong Manager UI requires authentication. {{site.kic_product_name}} uses the kong-enterprise-superuser-password
+secret to set the default value for the default kong_admin user.
+
+Run the following, replacing cloudnative with a random password of your choice and note it down.
+
+bash +$ kubectl create secret generic kong-enterprise-superuser-password  -n kong --from-literal=password=cloudnative +
+
+Once these resources have been created, we are ready to deploy {{site.kic_product_name}}.
⬇ Suggested change
-Once these resources have been created, we are ready to deploy {{site.kic_product_name}}.
+Once these resources have been created, you're ready to deploy {{site.kic_product_name}}.

In app/_src/kubernetes-ingress-controller/deployment/kind.md
https://github.com/Kong/docs.konghq.com/pull/5365#discussion_r1197093815
:

+```

+### Set your Kong Manager password
+
+The Kong Manager UI requires authentication. {{site.kic_product_name}} uses the kong-enterprise-superuser-password
+secret to set the default value for the default kong_admin user.
+
+Run the following, replacing cloudnative with a random password of your choice and note it down.
+
+bash +$ kubectl create secret generic kong-enterprise-superuser-password  -n kong --from-literal=password=cloudnative +
+
+Once these resources have been created, we are ready to deploy {{site.kic_product_name}}.
+
+## Install Kong
⬇ Suggested change
-## Install Kong
+## Install {{site.base_gateway}}

In app/_src/kubernetes-ingress-controller/deployment/kind.md
https://github.com/Kong/docs.konghq.com/pull/5365#discussion_r1197094598
:

+Run the following, replacing cloudnative with a random password of your choice and note it down.
+
+bash +$ kubectl create secret generic kong-enterprise-superuser-password  -n kong --from-literal=password=cloudnative +
+
+Once these resources have been created, we are ready to deploy {{site.kic_product_name}}.
+
+## Install Kong
+
+bash +kubectl apply -f https://raw.githubusercontent.com/Kong/kubernetes-ingress-controller/v{{ page.kong_version | replace: ".x", ".0" }}/deploy/single/all-in-one-postgres-enterprise.yaml +
+
+This may take a few minutes if this is the first time you're running {{ site.base_gateway }} in Docker with a database.
⬇ Suggested change
-This may take a few minutes if this is the first time you're running {{ site.base_gateway }} in Docker with a database.
+This may take a few minutes.
not sure that you need to qualify the "few minutes". It may take a few
minutes for various reasons, right?
In app/_src/kubernetes-ingress-controller/deployment/kind.md
https://github.com/Kong/docs.konghq.com/pull/5365#discussion_r1197094742
:

+```

+Once these resources have been created, we are ready to deploy {{site.kic_product_name}}.
+
+## Install Kong
+
+bash +kubectl apply -f https://raw.githubusercontent.com/Kong/kubernetes-ingress-controller/v{{ page.kong_version | replace: ".x", ".0" }}/deploy/single/all-in-one-postgres-enterprise.yaml +
+
+This may take a few minutes if this is the first time you're running {{ site.base_gateway }} in Docker with a database.
+
+Once bootstrapped, you should see the {{site.kic_product_name}} running:
+
+```bash
+$ kubectl get pods -n kong
⬇ Suggested change
-$ kubectl get pods -n kong
+kubectl get pods -n kong

In app/_src/kubernetes-ingress-controller/deployment/kind.md
https://github.com/Kong/docs.konghq.com/pull/5365#discussion_r1197095000
:

+NAME                            READY   STATUS      RESTARTS   AGE
+ingress-kong-548b9cff98-n44zj   2/2     Running     0          21s
+kong-migrations-pzrzz           0/1     Completed   0          4m3s
+postgres-0                      1/1     Running     0          4m3s

Please split the result out into a separate codeblock.
In app/_src/kubernetes-ingress-controller/deployment/kind.md
https://github.com/Kong/docs.konghq.com/pull/5365#discussion_r1197095125
:

+This may take a few minutes if this is the first time you're running {{ site.base_gateway }} in Docker with a database.
+
+Once bootstrapped, you should see the {{site.kic_product_name}} running:
+
+bash +$ kubectl get pods -n kong +NAME                            READY   STATUS      RESTARTS   AGE +ingress-kong-548b9cff98-n44zj   2/2     Running     0          21s +kong-migrations-pzrzz           0/1     Completed   0          4m3s +postgres-0                      1/1     Running     0          4m3s +
+
+### Configure your ingress
+
+Kong's Admin API, Kong Manager UI and the proxy will all be exposed on the same port.
⬇ Suggested change
-Kong's Admin API, Kong Manager UI and the proxy will all be exposed on the same port.
+{{site.base_gateway}}'s Admin API, Kong Manager UI, and the proxy will all be exposed on the same port.

In app/_src/kubernetes-ingress-controller/deployment/kind.md
https://github.com/Kong/docs.konghq.com/pull/5365#discussion_r1197095523
:

+This may take a few minutes if this is the first time you're running {{ site.base_gateway }} in Docker with a database.

+Once bootstrapped, you should see the {{site.kic_product_name}} running:
+
+bash +$ kubectl get pods -n kong +NAME                            READY   STATUS      RESTARTS   AGE +ingress-kong-548b9cff98-n44zj   2/2     Running     0          21s +kong-migrations-pzrzz           0/1     Completed   0          4m3s +postgres-0                      1/1     Running     0          4m3s +
+
+### Configure your ingress
+
+Kong's Admin API, Kong Manager UI and the proxy will all be exposed on the same port.
+We use Kong to route to the correct internal port based on the host provided.
⬇ Suggested change
-We use Kong to route to the correct internal port based on the host provided.
+Use {{site.base_gateway}} to route to the correct internal port based on the host provided.

In app/_src/kubernetes-ingress-controller/deployment/kind.md
https://github.com/Kong/docs.konghq.com/pull/5365#discussion_r1197095960
:

ingressClassName: kong

rules:

host: manager.127-0-0-1.nip.io

http:

 paths:

 - path: /

   pathType: ImplementationSpecific

   backend:

     service:

       name: kong-manager

       port:

         number: 80

+" | kubectl apply -f -
+```
+
+Finally, apply the following patch to set the KONG_ADMIN_API_URI to the hostname we set in the ingress above:
⬇ Suggested change
-Finally, apply the following patch to set the KONG_ADMIN_API_URI to the hostname we set in the ingress above:
+Apply the following patch to set the KONG_ADMIN_API_URI to the hostname you set in the ingress above:

In app/_src/kubernetes-ingress-controller/deployment/kind.md
https://github.com/Kong/docs.konghq.com/pull/5365#discussion_r1197096848
:

     service:

       name: kong-manager

       port:

         number: 80

+" | kubectl apply -f -
+ + +Finally, apply the following patch to set the `KONG_ADMIN_API_URI` to the hostname we set in the ingress above: + +bash
+kubectl patch deployment -n kong ingress-kong -p "{"spec": { "template" : { "spec" : {"containers":[{"name":"proxy","env": [{ "name" : "KONG_ADMIN_API_URI", "value": "http://admin-api.127-0-0-1.nip.io" }]}]}}}}"
+```
+
+### Expose the proxy to your host machine
+
+We need to apply some kind specific patches to make our proxy accessible on our host machine:
⬇ Suggested change
-We need to apply some kind specific patches to make our proxy accessible on our host machine:
+Apply the following kind-specific patches to make the proxy accessible on the host machine:

In app/_src/kubernetes-ingress-controller/deployment/kind.md
https://github.com/Kong/docs.konghq.com/pull/5365#discussion_r1197097249
:

+Finally, apply the following patch to set the KONG_ADMIN_API_URI to the hostname we set in the ingress above:

+bash +kubectl patch deployment -n kong ingress-kong -p "{\"spec\": { \"template\" : { \"spec\" : {\"containers\":[{\"name\":\"proxy\",\"env\": [{ \"name\" : \"KONG_ADMIN_API_URI\", \"value\": \"http://admin-api.127-0-0-1.nip.io\" }]}]}}}}" +
+
+### Expose the proxy to your host machine
+
+We need to apply some kind specific patches to make our proxy accessible on our host machine:
+
+bash +kubectl patch deployment -n kong ingress-kong -p '{"spec":{"template":{"spec":{"containers":[{"name":"proxy","ports":[{"containerPort":8000,"hostPort":80,"name":"proxy","protocol":"TCP"},{"containerPort":8443,"hostPort":443,"name":"proxy-ssl","protocol":"TCP"}]}]}}}}' +
+
+It will take a few minutes to roll out the updated deployment. Once the new
+ingress-kong pod is up and running, visit http://manager.127-0-0-1.nip.io/ and you should be able to log
⬇ Suggested change
-ingress-kong pod is up and running, visit http://manager.127-0-0-1.nip.io/ and you should be able to log
+ingress-kong pod is up and running, visit http://manager.127-0-0-1.nip.io and you should be able to log

In app/_src/kubernetes-ingress-controller/deployment/kind.md
https://github.com/Kong/docs.konghq.com/pull/5365#discussion_r1197097584
:

+kubectl patch deployment -n kong ingress-kong -p "{"spec": { "template" : { "spec" : {"containers":[{"name":"proxy","env": [{ "name" : "KONG_ADMIN_API_URI", "value": "http://admin-api.127-0-0-1.nip.io" }]}]}}}}"
+```

+### Expose the proxy to your host machine
+
+We need to apply some kind specific patches to make our proxy accessible on our host machine:
+
+bash +kubectl patch deployment -n kong ingress-kong -p '{"spec":{"template":{"spec":{"containers":[{"name":"proxy","ports":[{"containerPort":8000,"hostPort":80,"name":"proxy","protocol":"TCP"},{"containerPort":8443,"hostPort":443,"name":"proxy-ssl","protocol":"TCP"}]}]}}}}' +
+
+It will take a few minutes to roll out the updated deployment. Once the new
+ingress-kong pod is up and running, visit http://manager.127-0-0-1.nip.io/ and you should be able to log
+in to the Kong Manager UI.
+
+As you follow along with other guides on how to use your newly deployed the {{site.kic_product_name}},
⬇ Suggested change
-As you follow along with other guides on how to use your newly deployed the {{site.kic_product_name}},
+As you follow along with other guides on how to use your newly deployed {{site.kic_product_name}},

In app/_src/kubernetes-ingress-controller/deployment/kind.md
https://github.com/Kong/docs.konghq.com/pull/5365#discussion_r1197097872
:

+bash +kubectl patch deployment -n kong ingress-kong -p '{"spec":{"template":{"spec":{"containers":[{"name":"proxy","ports":[{"containerPort":8000,"hostPort":80,"name":"proxy","protocol":"TCP"},{"containerPort":8443,"hostPort":443,"name":"proxy-ssl","protocol":"TCP"}]}]}}}}' +
+
+It will take a few minutes to roll out the updated deployment. Once the new
+ingress-kong pod is up and running, visit http://manager.127-0-0-1.nip.io/ and you should be able to log
+in to the Kong Manager UI.
+
+As you follow along with other guides on how to use your newly deployed the {{site.kic_product_name}},
+you will be able to browse Kong Manager and see changes reflected in the UI as Kong's
+configuration changes.
+
+## Making a request through the proxy
+
+Finally, let's setup an environment variable to hold the IP address of kong-proxy service:
⬇ Suggested change
-Finally, let's setup an environment variable to hold the IP address of kong-proxy service:
+Let's set up an environment variable to hold the IP address of kong-proxy service:

In app/_src/kubernetes-ingress-controller/deployment/kind.md
https://github.com/Kong/docs.konghq.com/pull/5365#discussion_r1197098108
:

+## Making a request through the proxy

+Finally, let's setup an environment variable to hold the IP address of kong-proxy service:
+
+bash +export PROXY_IP="proxy.127-0-0-1.nip.io" +curl $PROXY_IP +
+
+Output:
+
+ +{"message":"no Route matched with those values"}% +
+
+This $PROXY_IP variable will be used in future guides. Please follow our
⬇ Suggested change
-This $PROXY_IP variable will be used in future guides. Please follow our
+This $PROXY_IP variable will be used in future guides. Follow our
—
Reply to this email directly, view it on GitHub
https://github.com/Kong/docs.konghq.com/pull/5365#pullrequestreview-1431725098,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A26ZL4PBHXIXVTOEBVFKNXTXGVDN5ANCNFSM6AAAAAAWMUHMRE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Regards
https://konghq.com/ Rajeev Ramani (he/him)
APIs | Mesh | Microservices @ Kong http://konghq.com/
+61 431588242


text:
[WIP] feat: add wasm filter-chain entity support

Note: requires https://github.com/Kong/go-kong/pull/362
Summary
This adds support for the filter chain entity, added to Kong in 3.4.
Full changelog

Add gateway FilterChain

Issues resolved
KAG-4005
Testing

[ ] Unit tests
[ ] E2E tests
[ ] Manual testing on Universal
[ ] Manual testing on Kubernetes

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

Codecov Report
Attention: Patch coverage is 5.82878% with 517 lines in your changes are missing coverage. Please review.

Project coverage is 39.40%. Comparing base (599ea65) to head (96a88f6).

Files
Patch %
Lines

pkg/state/filter_chain.go
0.00%
159 Missing :warning:

pkg/file/types.go
5.26%
108 Missing :warning:

pkg/file/writer.go
17.34%
68 Missing and 13 partials :warning:

pkg/file/builder.go
9.09%
62 Missing and 8 partials :warning:

pkg/state/types.go
0.00%
48 Missing :warning:

pkg/dump/dump.go
0.00%
28 Missing :warning:

pkg/state/builder.go
0.00%
21 Missing :warning:

pkg/diff/diff.go
0.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #72      +/-   ##
==========================================
- Coverage   41.14%   39.40%   -1.75%
==========================================
  Files          73       74       +1
  Lines       10139    10653     +514
==========================================
+ Hits         4172     4198      +26
- Misses       5549     6018     +469
- Partials      418      437      +19

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

Looking at adding more tests, but in the mean time this is ready for code review.

authors: CLAassistant codecov-commenter flrgh

repo: Kong/go-database-reconciler


text:
[Feature Request] Save/Save As for workspace

Relates to #1638
I find myself repeating the process of Exporting the current workspace a lot. It becomes a tedious chore when you start to update your API frequently during development.

The process is lengthy - it requires you to click through several modals where you make repeated selections
The file name sometimes requires editing - for example: I might want to remove the date portion.
There is no direct key combo to invoke it - you need to open the Workspace settings first which in fact has its own combo, but it is an extra step.

I would love to see a traditional Save/Save As tandem of menu commands as in many other programs operating on documents, ie:

Save would only pertain to the current workspace
Save would retain the selected format and file location/name
at first invocation, Save would be equivalent to Save As

This would allow me to first Save As... into my project directory, from where it would be version controlled, and then I'd just Save whenever I feel like the file deserves an update.
IMO this would make the app more friendly and usable.
User should still be able to Export - the settings used for Export should not interfere with Save though.

Another nice-to-have related to this.
If the format used to store the workspace is JSON, you should consider not minifying the JSON. Because for VCS is better to have it line by line. For diffs and efficiency in merges.

supported

ill give it another push, my workmachine died, and guess what, everyhting in Insomnia is gone, because i am not able to "Sae as..." and Version Control my Files ...

authors: Chicane90 arturhun fjfnaranjo-wavelt

repo: Kong/insomnia


text:
text/plain responses with json are not displayed properly

Is there an existing issue for this?

[X] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Insomnia Version
2021.6.0
What operating system are you using?
macOS
Operating System Version
11.6
Installation method
dmg
Last Known Working Insomnia version
2021.5.3
Expected Behavior
Json with text/plain Content-Type responses to be displayed as beautified
Actual Behavior
Response is without format
Reproduction Steps
No response
Additional Information
No response

Can confirm this is affecting Windows 10, version 10.0.19043
JSON data that previously was syntax-highlighted and beautified is now coming across as plain white text. This appears to be the case for every response preview that had been returned historically as well.
Upon opening the application for the first time in ~week, all previously beautified JSON is formatted like plaintext.
I did not experience this issue in version 2021.5.3

Hi all, this appears to be a regression as we have seen several reports about it, but unfortunately I haven't been able to reproduce it. Could you please supply any specific URLs that are causing this behavior, and also a screenshot? The Content-Type response header is used to display a rich view, so there may be something in that check we have missed.

same here

append a SP after the "content-type:" solve my issues

Maybe my situation can provide some help

I was able to reproduce this using https://mockbin.org/bin/48236712-b20f-4f17-a11e-cf8dcfa6c2ae.

2021.5.3
2021.6.0

@develohpanda I think the problem you had with your example is that the endpoint you hit (httpbin.org/json) has a Content-Type of application/json, while the response the users are talking about here is when it's text/plain.
The other problem is that httpbin.org/json, for whatever reason, actually sends a response that's pre-formatted.

The mockbin above doesn't do either of those two things, which helps to reproduce.

authors: develohpanda dimitropoulos iceblair jmluang marcomrdagsasc


text:
insominia connetion is failing out

Expected Behavior
Trying to post query on hapi server but connection is taking too long to respond.
Actual Behavior
Doc1.docx
Insomnia is crashing
Reproduction Steps
connection is taking too long
Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
No response
Insomnia Version
2022.5.1
What operating system are you using?
Windows
Operating System Version
windows 11 home
Installation method
From Insominia Website
Last Known Working Insomnia version
2022.5.1

Hi @prakash5751, thanks for reporting this. We would need a bit more information before being able to do something with this issue.
I'm able to get a response from the server you show in your screenshot:

I would recommend the following, in order to check if there's some issue with Insomnia itself, or a network issue/problem with the server you are trying to connect with.

Try to issue a curl request of the same request. You can use Insomnia to get a curl request with the Copy as Curl option:

If you don't use curl, you could try the Generate Code and pick an option that is familiar to you.


text:
Kong prometheus plugin does not record 404 response codes from proxy

Is there an existing issue for this?

[X] I have searched the existing issues

Kong version ($ kong version)
3.6.1
Current Behavior
When we enable the Prometheus plugin globally, and the proxy returns 404 response codes, the Prometheus plugin does not log the 404 status codes.
This gives us no way to monitor for 404 errors that could be caused by scenarios such as invalid configurations pushed to Kong, resulting in the Kong proxy rejecting all requests.
Expected Behavior
Prometheus metric endpoint records 404 response codes
Steps To Reproduce
Declarative config(declarative/kong.yaml):
_transform: true

services:
  - host: httpbin.org
    name: test-service
    port: 443
    protocol: https
    routes:
      - name: get-route
        paths:
          - /get
        strip_path: false
      - name: post-route
        paths:
          - /post
        methods:
          - POST
        strip_path: false
  - host: api.chucknorris.io
    name: jokes-service
    port: 443
    protocol: https
    routes:
      - name: jokes
        paths:
          - /jokes/random
        strip_path: false
plugins:
  - name: prometheus
    config:
      status_code_metrics: true
      latency_metrics: true
      bandwidth_metrics: true
      upstream_health_metrics: true

Docker Compose:

networks:
  kong-net:
    external: false

services:
  kong:
    image: docker.io/library/kong:3.3.1-ubuntu
    environment:
      KONG_NGINX_WORKER_PROCESSES: 1
      KONG_LOG_LEVEL: "debug"
      KONG_DATABASE: "off"
      KONG_DECLARATIVE_CONFIG: /kong/declarative/kong.yml
      KONG_PROXY_ACCESS_LOG: /dev/stdout
      KONG_ADMIN_ACCESS_LOG: /dev/stdout
      KONG_PROXY_ERROR_LOG: /dev/stderr
      KONG_ADMIN_ERROR_LOG: /dev/stderr
      KONG_ADMIN_LISTEN: 0.0.0.0:8001
      KONG_ADMIN_GUI_URL: http://localhost:8002
      KONG_LICENSE_DATA: ""
      KONG_PLUGINS: bundled
    networks:
      - kong-net
    ports:
      - 8000:8000
      - 8443:8443
      - 8001:8001
      - 8444:8444
      - 8002:8002
      - 8445:8445
      - 8003:8003
      - 8004:8004
    volumes:
      - ./declarative:/kong/declarative/

Generate some random traffic http://localhost:8000/random that makes kong return 404, the hit http://localhost:8001/metrics, you will see no 404 reported by prometheus plugin.

hello @surenraju-careem , thank you for opening this issue.
I believe the behavior you describe is expected, some metrics are intentionally not incremented when no route/service is matched, including the http_requests_total, which are intended to count requests by status code "returned by upstream services" as documented.
Changing this behavior would be considered a feature, which we may consider for implementation based on demand and availability. If you would like to open a pull request to support this we would be more than happy to review it.
Let me know if you have any doubts, thank you!

authors: samugi surenraju-careem

repo: Kong/kong


text:
404 on documentation portal of Kong

NOTE: GitHub issues are reserved for bug reports only.
Please read the CONTRIBUTING.md guidelines to learn on which channels you can
seek for help and ask general questions:
https://github.com/Mashape/kong/blob/master/CONTRIBUTING.md#where-to-seek-for-help
Summary
When I clicked on Securing the Admin API on this url it show 404 error. This link can be found in given URL at the bottom of the page.
Steps To Reproduce

Go to this URL
Scroll to bottom of the page
Click on Securing the Admin API link. It will show 404 Error

I just tested the link and now (1st Nov) it seems to work fine. Closing this issue. Thanks for reporting it.

This was patched a few days ago via github.com/Kong/getkong.org. In the future any bugs or PRs about the docs (which are open source! :D ) can be sent to that repo. Thanks!

authors: kikito p0pr0ck5 sanketgandhi


text:
Can't update plugin by name

Summary
Can't update plugin by its name
Steps To Reproduce

Create api
Add plugin to it
Try to update a plugin with PATCH method, e.g.:

curl -XPATCH "http://localhost:8001/apis/138575d3-7714-44ea-a1f0-dca02baca397/plugins/response-transformer" --data "config.remove.headers=x-request-id"
{"id":"response-transformer is not a valid uuid"}

According to docs, https://getkong.org/docs/0.11.x/admin-api/#update-plugin, i should be able to update plugin by its name. It seems like here https://github.com/Kong/kong/pull/2252 it was fixed only for GET method.
Additional Details & Logs

Kong version 0.11.1

That's not possible, you need to use the plugin ID.

@AlexKovalevych This is a mistake on our side. It is indeed not possible to retrieve/update/delete a plugin by its name, since several instances of a same plugin can be configured simultaneously. One needs to use the id of such plugins to update them.
I have opened https://github.com/Kong/getkong.org/pull/552 on our documentation website, which should take care of fixing that.
In the meantime, I will be closing this issue, and considering it resolved. Sorry for the confusion!

@thibaultcha thank you for your response. But right now, adding a second instance of the same plugin gives me 409 status code response. That makes me think its not possible to add multiple instances of the same plugin (or i'm doing it wrong).
$ curl -XPOST "http://localhost:8001/apis/138575d3-7714-44ea-a1f0-dca02baca397/plugins" --data "name=response-transformer&config.remove.headers=x-request-id" -i
HTTP/1.1 409 Conflict
Date: Wed, 22 Nov 2017 08:36:29 GMT
Content-Type: application/json; charset=utf-8
Transfer-Encoding: chunked
Connection: keep-alive
Access-Control-Allow-Origin: *
Server: kong/0.11.1

{"name":"already exists with value 'response-transformer'"}

Same Kong version (0.11.1)

It is possible if they are added to the same API but different Consumers. This is detailed in the revert commit linked in the docs PR.

got it, thanks

authors: AlexKovalevych thefosk thibaultcha


text:
proxy_ssl_ciphers directive doesn't work

Summary
Specifying proxy_ssl_ciphers doesn't seem to have any impact on the CIPHERS offered during TLS Handshake. Tried specifying using custom nginx template as well as nginx_http_ directives. One caveat is that, if we change existing proxy_pass definition inside custom template from
proxy_pass   $upstream_scheme://kong_upstream$upstream_uri;
to
proxy_pass  https://ACTUAL_HOST.com
then proxy_ssl_ciphers starts working. This indicates that proxying via kong_upstream seems to ignore the directive.
Additional Details & Logs

Kong version: 1.1.2
Operating system: docker alpine and centos both

@thibaultcha : Looks like when we implmented https://github.com/Kong/kong/commit/c4b3b1a7be471ff97a60ecc56a8cb0d7749cf9b6, we started using our own SSL_CTX for balancer instead of default nginx one which supported proxy_ssl_ciphers directive. I see a TODO inside kong/init.lua that mentions
"-- TODO: copy proxy_ssl_* flags?"
Can you confirm if I am thinking in the right direction?

@mukgupta Yes, you are right. It's a known limitation on our end. We've planned to have a proper fix after 1.4.0

Thanks for confirming @guanlan . Is there a good workaround to this? I am currently using ngx.ctx.balancer_data.ssl_ctx:setCurvesList in a pre-function plugin to update the ciphers/curves

@guanlan in 1.2.1 I observe what I think is related: proxy_ssl_trusted_certificate is not respected (i.e. ssl handshake fails) while the same cert chain file works fine in nginx using proxy_ssl_trusted_certificate.  Am I conflating these things, or is it likely that this issue is the same for proxy_ssl_trusted_certificate?

Fixed in 1.4.0rc2 vis Service Mesh deprecation. Please give it a try!

authors: bneutra dndx guanlan mukgupta


text:
Use the manager client in the parser

What this PR does / why we need it:
This is a PoC to strip the store package out of the parser. Instead of using the store, the parser will List() resources directly from the manager client.
This WIP only does so for Ingress v1 resources. It does not yet attempt to filter these resources by class. It disables parser unit tests because Build() (which every parser unit invokes) now needs a manager client, and I didn't want to deal with converting a ton of test cases to https://pkg.go.dev/sigs.k8s.io/controller-runtime/pkg/client/fake for the PoC.
Special notes for your reviewer:
The relationship between UpdateKongAdminSimple, the KongUpdater function type, and things that use them remain confusing and somewhat annoying to refactor around. IDK if we have actual intent to implement alternative updaters (AFAIK, the original rationale for how this is structured), but we haven't so far. We may want to consider simplifying it.

Remaining failures should be resolved by https://github.com/Kong/kubernetes-ingress-controller/pull/2253
DB-less passed, and this change should be the same across both deployment types, so the PoC should be good to go.

It seems we've left this one for a while, given our current state and that we have a follow up issue are we OK to close this?

Copied to my fork in case we ever want to review the old POC

authors: rainest shaneutt

repo: Kong/kubernetes-ingress-controller


text:
Update timeout for .github/workflows/_conformance_tests.yaml

This is an automated pull request to update default timeouts for all jobs in .github/workflows/_conformance_tests.yaml that do not currently contain a default timeout property.
Please review these changes closely to ensure that all jobs have sufficient time to run. If a job needs more time, please add a timeout property to that job to override the organization default.
Additionally, please be attentive to unintended changes to the workflow YAML file, as the parser is not perfect.
Lastly, note that it is up to the reviewer to merge this pull request and delete the PR branch once they are satisfied with the changes.

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

Closing in favour of already merged #5733

authors: CLAassistant pmalek team-eng-enablement


text:
fix(proxy-wasm) improve instance recycling robustness

Allow for recycling the root instance (not managed by isolation modes)
Allow for smoother recycling of http instances (including when the root instance has trapped)
Also recycle trapped instances in the busy queue
Less verbose context creation of fresh and/or recycled instances starting in atypical steps (e.g. on_response_headers).

This uncovered an issue we have with SIGHUP and pending timers, which will need to be aborted through a triggered event. WIP.

This turned into what is currently a pretty heavy refactor of the proxy-wasm instance/context relationship, which was still unfinished (had lots of leftover code from before ictx, which makes the handling of root/filter contexts inconsistent).

Moved to #422.


text:
WebApp - Bug when connect extension

Describe the bug
Navigate incorrect screen in case connect with extension
To reproduce the bug

[x] Case 1:

Steps:

Import 1 account on extension
Connect successfully web-app with extension
On extension, import from seed phrase
On web-app, F5 and observe the displayed screen

Actual: Navigate welcome screen

Expect: Navigate Portfolio screen

[x] Case 2:
Steps:

On step 4: Perform re-connect with extension
Perform transaction and obverse the displayed screen

Actual: Do not show confirmation on extension

Expect: Show confirmation

Version test: https://1bb6eeb4.subwallet-webapp.pages.dev/home/tokens
Validation passed.

repo: Koniverse/SubWallet-Extension


text:
[Issue 3246] fix: update balance calculation for assets pallet

Related issues:
#3246

🚀 Deployed on https://pr-3247--sw-web-runner.netlify.app


text:
Mobile - Show incorrect withdrawal information of the Bifrost liquid staking when all accounts mode

Describe the task
Show incorrect withdraw information of the Bifrost liquid staking when all accounts mode

Check with address:

5CFh4qpiB5PxsQvPEs6dWAhzgAVLHZa8tZKxeE9XsHBg4n9t
5CAfi6MtzQBcw8iPniwxW8S8YHKEbGbqk8xSm6K9NzxxxLh7

@ThaoNguyen998 please check this issue again
=> I remember this issue was resolved in webrunner v1.2.22

Version test: v1.1.81(433)
Device test: IP 13
Can't reproduce this issue -> Close issue
Check with account:

15tge4dhLYenMCU9SpW5R8NZV1y4W4eJ91bmAEyNA6TCmhsv
5Cf9V61adqFFCVimge6pa2vST5QwWjHdDAFNP9tu58VyuaZ7

repo: Koniverse/SubWallet-Mobile


text:
version 1.0

Your plugin is not compatible any longer with the current version of VCV Rack. Are you planning on updating it and releasing a new version?

hi Gilbert, yes, i am planning to do a new version, but i havent the time right now :)
thanks for your interest... i would like to know how you are using it within the 0.6.2 release of vcv ? if you dont mind...

Great, thanks for taking a look at it!

thanks for your interest... i would like to know how you are using it

In the past I used to write electronic pieces using Pure Data mainly for 4- and 8-channel systems. It's been some time since I last did any multichannel serious work, but this is one of those 'nice-to-have' set of modules, particularly the quad output. Of course I could simply use two regular 2-channel outputs and route them through Jack, but it makes the patch clearer to have it as a single one. Also, I am considering to use Rack for a class of computer music and this might come in handy if any student wants to try using more than 2 channels.

Ah, and of course the quad mixer makes it much easier to pan things around the quad space.

yes, you are right... i m also planning to finish my 16 track sequencer to be able to control the matrix, using automations patterns, what do you think about it ?
i am also using VCv to teach people ;) great to ear that you are doing the same

That 16-track sequencer with quad output sounds like a great addition to VCV, I'd definitely be interested in it. Thanks again for taking a look at this and good luck with your classes too!

There has been a user going on the VCV community forum (netboy3 is his username) who has in the past been happy to help in the migration of plugins to v1.0 compatibility in VCV. Would you like a hand getting your plugin updated to the current version if he is keen and able?

authors: KoreTeknology gilbertohasnofb pgatt


text:
model failure

this is what i always got when i use animatediff nodes, can anybody tell me why?

this is backstage showed:

when i disconnected animatediff, everything worked fine.

Confirm that everything is working in your end properly by using the simplest workflow that's in the readme.

Confirm that everything is working on your end properly by using the simplest workflow that's in the readme.
I comfirm everything else works fine. The example in the issue picture only added a lcm lora but nothing else, and i certainly have tried the basic workflow using normal set sampler as shown below:

https://github.com/Kosinkadink/ComfyUI-AnimateDiff-Evolved?tab=readme-ov-file#txt2img
I meant this basic AD workflow here, just to make sure things are working as intended with AD

https://github.com/Kosinkadink/ComfyUI-AnimateDiff-Evolved?tab=readme-ov-file#txt2img
I meant this basic AD workflow here, just to make sure things are working as intended with AD

i have tried.it works fine.
does it mean that the clip text I gave was too short that the model did not know what to generate?

Hmm, it's possible, or perhaps you need to change some of the other settings in the sampler - lcm works a bit differently between normal SD and AD. I'd recommend adding LCM components/settings one at a time to an AD workflow so you can find the exact thing that causes things to break down, and then tweak the settings from there.

Hmm, it's possible, or perhaps you need to change some of the other settings in the sampler - lcm works a bit differently between normal SD and AD. I'd recommend adding LCM components/settings one at a time to an AD workflow so you can find the exact thing that causes things to break down, and then tweak the settings from there.

i have tried the lcm. It seems that the lcm lora will lead to a sharper image and the animation moves faster, which also causes blury background and misformed body and face,but still tolerable. However, when the clip text is too short like only "1girl",the animation will become a totally random noise.

authors: Kosinkadink zmlsunny

repo: Kosinkadink/ComfyUI-AnimateDiff-Evolved


text:
Is it possible to keep positiveButton, or any Button in Alert Dialog from closing the alert?

       alert {
            customView {
                oldList = listView {
                    adapter = MyAdapter(replies)
                }
            }
            negativeButton("No") {  }
            positiveButton("Yes"){}
        }.show()

In the above code, positiveButton and negativeButton both close the dialog, without me having a .dismiss() or .close() in their handlers. Is there any way to use anko for my alerts and keep my dialog open? Or would this require a pull request? (couldn't find anything in the wiki or online about this, so sorry if I overlooked something)

Hey Parker,
Been a long time since you posted this, but wanted to share my experience for anyone who comes after - Android dialogs, by design, dismiss themselves after those buttons are tapped.
This isn't great, and I've found myself using the following reference to work around it: https://stackoverflow.com/a/40261920
Cheers,
Akram

alert {
titleResource = R.string.title
messageResource = R.string.message
positiveButton(R.string.ok, { dialog ->
dialog.dismiss()
})
negativeButton(R.string.cancel, { dialog ->
dialog.dismiss()
})
show()
}

authors: AugustoFavretto ParkerK akram-aq1

repo: Kotlin/anko


text:
Fix for an Anko build error.

A fresh checkout of Anko from master does not currently build.
Anko makes (gratuitous?) use of a sun.plugin.dom InvalidStateException.
The required java-plugin.jar is not fetched from update_dependencies.xml,
nor is it downloaded when doing a gradle build.
It does not seem like the jar is used anywhere else, so I figured removing the
dependency makes more sense than trying to get the dependency working.

Fixed manually in cd2b30d0dc54ff8998a1c3a2159967515ac45498 due to the merge conflicts.
Anyway, thank you!

authors: steveyegge yanex


text:
Why doesn't scale work for area

Describe the bug
Please provide a description of the bug, including steps to reproduce it and the expected result.
To Reproduce
plot {
            area {
                x(listOf("January", "February", "March", "April", "May")) {
                    axis.name = "months"
                }
                y(listOf(200, 150, 300, 250, 420)) {
                    axis.name = "sales"
                    scale = continuous(min = 100, max = 500)
                }

                alpha = 0.5

                fillColor = Color.BLUE

                borderLine {
                    color = Color.BLACK
                    width = 1.5
                }
            }
        }

Screenshots

It's a lets-plot behavior. Same for bars. I consider that's bug.

https://github.com/JetBrains/lets-plot-kotlin/issues/219

Should be fixed via #358

authors: AndreiKingsley devcrocod

repo: Kotlin/kandy


text:
IME_ACTION_DONE is possible with empty credential fields

If you go into the password field and hit done with keyboard you are logged in...
Possible solution:
setOnEditorActionListener { _, actionId, _ ->
                when {
                    actionId == EditorInfo.IME_ACTION_DONE && login.isEnabled -> {
                        loginViewModel.login(
                            email.text.toString(),
                            password.text.toString()
                        )
                    }
                }
                false
            }

What are you talking about? 🤔
Wrong Repo? 🤔

authors: StefMa cryptedx

repo: Kotlin/kmm-sample


text:
How to build a jar

Hi!
Let's say I'm targeting only desktop and Android (so no iOS) which both support jvm. How would I build a jar file which I could add to my KMP app project? I don't care about publishing to Maven, etc., I just need a jar. Is this possible?
Thank you.

If anyone else has this problem, I found a solution. Maven Publish Plugin has a task publishToMavenLocal. Just running .gradlew publishToMavenLocal will generate jars in ~/.m2/repository/org/jetbrains/kotlinx/multiplatform-library-template/. You can then use them.

repo: Kotlin/multiplatform-library-template


text:
claude_api.send_message() error

response = claude_api.send_message(prompt, conversation_id)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/claude_api.py", line 129, in send_message
data = json.loads(json_str)
^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/json/init.py", line 346, in loads
return _default_decoder.decode(s)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/json/decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/json/decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

I tried to use US VPN and changed the cookie  but got the same error

It works for me though, can you give more detailed info?

That's the error that I get, I tried the API on google colab and got the same thing as well :
response = claude_api.send_message(prompt, conversation_id)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/claude_api.py", line 129, in send_message
data = json.loads(json_str)
^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/json/init.py", line 346, in loads
return _default_decoder.decode(s)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/json/decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/json/decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

That's the error that I get, I tried the API on google colab and got the same thing as well :
response = claude_api.send_message(prompt, conversation_id)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/claude_api.py", line 129, in send_message data = json.loads(json_str) ^^^^^^^^^^^^^^^^^^^^ File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/json/init.py", line 346, in loads return _default_decoder.decode(s) ^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/json/decoder.py", line 337, in decode obj, end = self.raw_decode(s, idx=_w(s, 0).end()) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/json/decoder.py", line 355, in raw_decode raise JSONDecodeError("Expecting value", s, err.value) from None json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

remove the "<conversation_id>" or

Possible causes: 1, this is because the proxy region is not allowed to provide services 2, the transmission of an incorrect parameter, such as "model" in "headers" should be updated to the same version as the web page.

repo: KoushikNavuluri/Claude-API


text:
xcodebuild action

Exposes xcodebuild CLI tool via fastlane action with defaults, environment variables & other nice things.
Usage:
lane :build do
  # Create an archive. (./build-dir/MyApp.xcarchive)
  xcodebuild(
    archive: true,
    archive_path: './build-dir/MyApp.xcarchive',
    scheme: 'MyApp',
    workspace: 'MyApp.xcworkspace'
  )

  # Export a signed binary (./build-dir/MyApp.ipa)
  xcodebuild(
    export_archive: true,
    export_path: './build-dir/MyApp'
  )
end

The action supports the following .env vars:

PROJECT
WORKSPACE
SCHEME
BUILD_PATH

For example, if your .env file looked like this:
PROJECT="./MyApp.xcodeproj"
WORKSPACE="./MyApp.xcworkspace"
SCHEME="MyApp"
BUILD_PATH="./build"

...then you can simplify the lane usage to:
lane :build do
  # Create an archive. (./build/MyApp.xcarchive)
  xcodebuild(
    archive: true
  )

  # Export a signed binary (./build/MyApp.ipa)
  xcodebuild(
    export_archive: true
  )
end

When archiving, the path to the .xcarchive file is cached in SharedValues for xcodebuild to use in a subsequent export.
When exporting, the path to the generated ipa is cached in SharedValues::IPA_OUTPUT_PATH for any deploy steps (i.e. Crashlytics) to use (should this also store in ENV["IPA_OUTPUT_PATH"] as the ipa action does now?).
I'd like to add even simpler alias actions for the most common xcodebuild operations; archive, export, build, & test.  Something like: 'xcarchive', 'xcbuild', 'xcexport', and 'xctest'.
This could reduce the usage to a single action verb, for simpler projects.
I was thinking of bundling these into the same module/file as alias classes. Thoughts?
Example usage of alias actions in a complete continuous deployment lane:
lane :inhouse do
  # Run tests
  xctest(
    destination: "name=iPhone 5s,OS=8.1"
  )

  # Generate XML code coverage
  gcovr(
    xml: true,
    output: "./code-coverage/report.xml"
  )

  # Clean the project
  xcclean

  # Create an archive
  xcarchive

  # Fetch/install provisioning profile
  sigh

  # Export a signed IPA
  xcexport

  # Deploy build to Beta
  crashlytics

  # Notify the HipChat room that the build was released
  hipchat(
    message: "iOS build successfully deployed to Beta!",
    channel: ENV["HIPCHAT_ROOM"],
    success: true
  )
end

I'd like to add even simpler alias actions for the most common xcodebuild operations; archive, export, build, & test. Something like: 'xcarchive', 'xcbuild', 'xcexport', and 'xctest'.
I was thinking of bundling these into the same module/file as alias classes. Thoughts?

^^ Yes yes, I agree with this!

Added alias/helper/companion actions:
xcarchive
xcarchive(
  archive_path: "./build-dir/MyApp.xcarchive",
  scheme: "MyApp",
  workspace: "MyApp.xcworkspace"
)

xcbuild
xcbuild(
  scheme: "MyApp",
  workspace: "MyApp.xcworkspace"
)

xcclean
xcclean

xctest
xcarchive(
  destination: "name=iPhone 5s,OS=8.1",
  scheme: "MyApp",
  workspace: "MyApp.xcworkspace"
)

Thank you so much @dtrenz for working on this. Great result! :rocket:

authors: KrauseFx dtrenz joshdholtz

repo: KrauseFx/fastlane


text:
🛑 💼 Plesk is down

In 6a19122, 💼 Plesk (https://krisnetics.biz:8443/login_up.php) was down:

HTTP code: 0
Response time: 0 ms

Resolved: 💼 Plesk is back up in a12a647.

repo: KrisKodira/uptime-monitor


text:
merge package template from arlac77/npm-package-template

package.json

chore(devDependencies): remove jsdoc-babel@^0.3.0
chore(devDependencies): remove jsdoc-to-markdown@^3.0.2
chore(devDependencies): add documentation@^5.3.5 from template
chore(devDependencies): update semantic-release@^11.1.0 from template
chore(scripts): update docs@documentation readme src/service-admin.js --section=API from template
chore(scripts): add lint@documentation lint src/service-admin.js from template

.travis.yml

chore(travis): merge from template .travis.yml

README.md

docs(README): update from template

Coverage remained the same at 14.035% when pulling ae11bf21f206787c250b55211330a73097300df3 on template-sync-1 into 3e389c1e9e888400f01aa911bf0f484cab956fbf on master.

Coverage remained the same at 14.035% when pulling ae11bf21f206787c250b55211330a73097300df3 on template-sync-1 into 3e389c1e9e888400f01aa911bf0f484cab956fbf on master.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #441   +/-   ##
=======================================
  Coverage   16.78%   16.78%
=======================================
  Files           3        3
  Lines         137      137
=======================================
  Hits           23       23
  Misses        114      114

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 3e389c1...d1c5afd. Read the comment docs.

Coverage remained the same at 14.035% when pulling d1c5afd98cdf66dde0723fd576c59f858e70b8cb on template-sync-1 into 3e389c1e9e888400f01aa911bf0f484cab956fbf on master.

authors: arlac77 codecov-io coveralls

repo: Kronos-Integration/kronos-service-admin


text:
Cannot download from Google Drive

Google prevents the download and states, that the files contain a virus. This applies to all versions.

Somehow it seems zipped Durazno is a trojan... =_="
https://www.virustotal.com/en/file/5dce4f610ae1f59bef663aedf1f7c7d03deb61437e4a9bb6e17449096e632585/analysis/1482775351/
I unzipped the files and ClamAV didn't complain, rezipped with Windows' instead of 7zip and it also complained about a trojan. If I use 7z instead, no problem. sighs

Thanks, it works now.

authors: KrossX vMeijin

repo: KrossX/Durazno


text:
Support None as fallback locale

Hi!
I've noticed a weird recursion error when using your improved version of python-i18n (I love what you did btw, thanks for the work).
Basically, in the t function, when a translation key cannot be found, your code checks if the used language is the same as the fallback one. If not, then it tries to find the translation with the fallback language. And that works fine as long as you do have a fallback language.
https://github.com/Krutyi-4el/i18nice/blob/7579c9da684f362ded7fa0acab7379098e539881/i18n/translator.py#L12-L21
BUT if the fallback language is set to None, then the condition at line 20 is always met (because locale cannot be None), and t() is called back with locale=None, which forces to use the locale config value, which is again different than the fallback value.
A simple fix should be to check if config.get('fallback') is None, and if it is, raise a proper error (a KeyError or anything more intuitive).
Hope this helps!

Thanks for the feedback!
In fact, fallback was never designed to be None, but I think it makes sense to add this feature. Will do in the next update.

Makes sense. Thanks for considering adding support to it then.
FYI I use None as fallback configuration because I want to receive an error if the translation is missing for any language I use. If I'm right, this is the only way (or will be?) to get a KeyError immediately, and don't... well, don't fall back to another working language.

You are almost right. You still won't avoid search_translation() which is terribly slow because of directory scanning (btw, it should be easy to change this as we have load_everything() now). I don't recommend relying on missing transactions in the first place. Of course, it's ok for debugging.

The new release is out


text:
Command 'elm.make' not found when pressing cmd+f5 in a non-Elm project

The command + f5 keybinding is used to run the project without debugging. vscode-elm has set the keybinding to run elm.make. The problem is that it does this even when you're not using Elm in your project.
I'm now forced to uninstall the Elm extension because VS Code doesn't have the ability to disable extensions yet.
This problem can probably be solved by using a when clause for the keybindings (https://code.visualstudio.com/docs/customization/keybindings#_when-clause-contexts).

Fixed in 0.11.1

authors: Krzysztof-Cieslak TimonVS

repo: Krzysztof-Cieslak/vscode-elm


text:
🛑 Kuerbis is down

In 6c7da13, Kuerbis (https://kuerbis.cf) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Kuerbis is back up in 987ed56.

repo: Kuerbis-HD/Kuerbis-Web-Status


text:
苹果安卓都会出现这种情况--->背景、图片不能显示？

麻烦作者看下是什么问题？（二维码、文字能正常显示）

真机上运行，需要添加网络图片地址为合法域名，或者你可以开启调试模式

远程调试已经启动不效验合法域名也会出现这个问题呢！不知道你那边能不能复现？

我们这边真机是没有问题的，可以说下您使用的微信版本和真机型号吗？

authors: CPPAlien heroFan

repo: Kujiale-Mobile/Painter


text:
Added begin and MTP_littleFS_Program sketch

This is what I did this morning along with the updated simplified sketch.  Note: usb_mtp is now part of the core so no need to address that in the sketch.

@mjs513 - glad you mentioned something about this PR as I did not get any notification about it...


text:
Predictive maintenance of industrial equipment

dataset link : https://data.nasa.gov/Aerospace/CMAPSS-Jet-Engine-Simulated-Data/ff5v-kuh6/about_data
This model aims to predict industrial equipment, in this case airplane engine before failure. This dataset provides time for failure during a particular duration and predicts the remaining useful life.
The model can be made for both classification and regression.
For classification, we can make 3 conditions of the machines : i) Good condition
ii) Moderate condition
iii) Warning condition
For regression, we predict the remaining useful life for the machine.
Different models will be implement in this project.

I don't think this is an level issue. Because I am going to implement multiple models.
First, The text file be converted to csv file by performing many data preprocessing steps.
Then, EDA is performed for the timeseries prediction.
Then, Implementation of multiple models
Plotting the graphs and results.
Please check @Kushal997-das


text:
Modified and simplified README.md file (#1471)

Project-Guidance! 🎊🎈
Welcome! 🎉
Thank you for visiting our repository! We're excited to have you here. If you find this project useful, please consider giving it a star! ⭐
🎉 Have you reviewed the Contributing Guidelines? 🤔 (Required)

[x] YES
[ ] NO

📝 Description

This is with reference to issue #1471
I read the whole README.md file of the repository. These parts of the document [The contributing, Project Admin, Engaging and Collaborating with, Stargazers over time 🤗🚀, Awesome contributors 🤩] are repeated two times in the README.md file which creates confusion for the new user to understand. So I will choose the best which fits and modify it for the user to simply understand about the repo and project. I have modified the "Introduction" segment which is simple and clear to understand for the new comers on this repo.
Please add the #hacktoberfest-accepted , #hacktoberfest2024 labels
🔧 Fixes #issue_no (#1471)

🏷 Initiative Participation (Required)
This pull request is associated with the following initiative. Please choose the one you are contributing to:

[ ] 🌸 GSSoC (Girl Script Summer of Code)
[ ] 🌼 GSSoC 2024 Extd.
[x] 🍂 Hacktoberfest 2024
[ ] ❄ Script Winter of Code (SWoC)
[ ] ⛄ Winter of Blockchain (WoB)
[x] 🙋♂ Personal
[ ] 📌 Other (please specify): _______________

Screenshots after changes:

Please review my pull request and merge it with your actual branch and add the labels #hacktoberfest-accepted, #hacktoberfest2024


text:
Title: To add ceil and floor in the sorted array of binary search in python

Initiative (Required)
Hacktoberfest 2024 🎃
Is your proposal related to a problem? Please describe.
To provide clear and concise code
Add any other context or screenshots about the proposal request here.
No response

@Kushal997-das  assign this to me please


text:
chore: remove kclvm python unit tests and move it to the kclvm-py repo.

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

chore: remove kclvm python unit tests and move it to the kclvm-py repo.

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[x] Other

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

Pull Request Test Coverage Report for Build 4751497635

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.988%

Totals

Change from base Build 4750129239:
0.0%

Covered Lines:
2300

Relevant Lines:
2614

💛  - Coveralls

authors: Peefy coveralls

repo: KusionStack/KCLVM


text:
fix : adjust the finalizer of CAPI client to make GC work normally

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

2. What is the scope of this PR (e.g. component or file name):

/pkg/service/c_api_call/proto_capi_kclvm_service_client.go
3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

adjust the finalizer of CAPI client to make GC work normally
4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[x] Other

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

Pull Request Test Coverage Report for Build 2838034229

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 48.014%

Totals

Change from base Build 2837304390:
0.0%

Covered Lines:
2539

Relevant Lines:
5288

💛  - Coveralls

authors: NeverRaR coveralls

repo: KusionStack/kclvm-go


text:
prisma migration issue

Unable to migrate the issue with prisma
Error: P3018
A migration failed to apply. New migrations cannot be applied before the error is recovered from. Read more about how to resolve migration issues in a production database: https://pris.ly/d/migrate-resolve
Migration name: 20240415100000_added_category_id_field_in_product_table
Database error code: 1146
Database error:
Table 'newEcommerce.product' doesn't exist
Please check the query number 1 from the migration file.

Hi @venkats-cr. Can you tell me more information about the error? Is this your first migration or have you run the project several times successfully before this? Also, did the error happen after your custom migration?

This is my first migration , got this error
Error: P3018
A migration failed to apply. New migrations cannot be applied before the
error is recovered from. Read more about how to resolve migration issues in
a production database: https://pris.ly/d/migrate-resolve
Migration name: 20240415100000_added_category_id_field_in_product_table
Database error code: 1146
Database error: Table 'newEcommerce.product' doesn't exist
Please check the query number 1 from the migration file.
On Fri, 19 Jul 2024 at 15:05, Aleksandar Kuzmanović <
@.***> wrote:

Hi @venkats-cr https://github.com/venkats-cr. Can you tell me more
information about the error? Is this your first migration or have you run
the project several times successfully before this? Also, did the error
happen after your custom migration?
—
Reply to this email directly, view it on GitHub
https://github.com/Kuzma02/Electronics-eCommerce-Shop-With-Admin-Dashboard-NextJS-NodeJS/issues/2#issuecomment-2238762752,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5N2B4JAA3ENWLIFXFXBH3TZNDMWLAVCNFSM6AAAAABLEAXMTWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZYG43DENZVGI
.
You are receiving this because you were mentioned.Message ID:
<Kuzma02/Electronics-eCommerce-Shop-With-Admin-Dashboard-NextJS-NodeJS/issues/2/2238762752
@github.com>

--
Regards
Venkatkumar S
Phone: +91 9840077575
*www.cloudripples.io http://www.cloudripples.io/ *

I think you should watch the tutorial on running the project again and try to repeat the steps. The tutorial link is in the README. If you didn't see it, here is the link: https://www.youtube.com/watch?v=Ry0aOMws0gE

@venkats-cr When you find a solution to the problem, please feel free to post it here.

Will do, thanks!
On Fri, 19 Jul, 2024, 21:24 Aleksandar Kuzmanović, @.***>
wrote:

@venkats-cr https://github.com/venkats-cr When you find a solution to
the problem, please feel free to post it here.
—
Reply to this email directly, view it on GitHub
https://github.com/Kuzma02/Electronics-eCommerce-Shop-With-Admin-Dashboard-NextJS-NodeJS/issues/2#issuecomment-2239503213,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5N2B4KWVABXSWQQLEM6YQ3ZNEZEBAVCNFSM6AAAAABLEAXMTWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZZGUYDGMRRGM
.
You are receiving this because you were mentioned.Message ID:
<Kuzma02/Electronics-eCommerce-Shop-With-Admin-Dashboard-NextJS-NodeJS/issues/2/2239503213
@github.com>


text:
Create pull request and merge develop to origin

Final release

Issue solved

repo: KwygonJin/homepage


text:
added few more details

new function for tokenControlInfo
fixed to reserve contract

LGTM

authors: secmask ssreelekhya


text:
Content tabs

Hi,
I've been using your plugin and it has been working great with Mkdocs and Material style admonitions. However, in my project I also use content tabs (https://squidfunk.github.io/mkdocs-material/reference/content-tabs) which are extremely similar to admonitions.
I've forked your repository and figured out that the only change would be to add === in the MARKERS tuple and corresponding tests. But, this made me realise that there would be so much code in common.
I am not sure how to approach this as you have already fully integrated this plugin and the name of does not really correspond to the content tabs. Do you have any suggestions or ideas? I would be more than happy to contribute.

That's a nice surprise that the admonition logic might just work!
So, I'm actually in the process of deleting that file now (#13) once my PR changes are released for the plugins package: https://github.com/executablebooks/mdit-py-plugins/pull/58
You could certainly implement just the parser in the same way I've included it here, but over the long-run, I would probably recommend:

Reviewing other markdown implementations to see if something like content blocks are supported so that the plugin can be sufficiently generally. You can also look at the specs that I sampled for admonitions: https://github.com/KyleKing/mdformat-admon/tree/726262380659718b5d17cb9e9080858a7fa346f9#caveats
Post an issue to https://github.com/executablebooks/mdit-py-plugins and let chrisjsewell know that you want to add support for content-tabs (hopefully I haven't added a can of worms by expanding the plugins to support MKDocs 😅)
Open a PR against mdit-py-plugins that adds a new content-block plugin and extracts a set of shared utilities from the admonitions package (maybe these could be called marked_blocks?), which accept a list of markers and returns some sort of useful data structure or modifies the state directly

I think having a dedicated plugin in mdit-py-plugins that generates useful HTML would be the best approach, but that would be the hardest. In the interim, I would still probably recommend having a prototype version of the code in your package like I did before #13, so you can start to find edge cases before mdit-py-plugins is updated
Whichever approach you take, I would be happy to add your plugin to mdformat-mkdocs!
I'm about to leave for vacation for over a week, so you caught me at a bad time, but hopefully this information is helpful! I'd be happy to add my input on code reviews or discuss other approaches as well

Also, to add, I'm not against supporting content-blocks in this plugin, but I agree that the plugin naming is unfortunate because it would be confusing to do both here

Hi Kyle,
Thanks a lot for the information. This is super helpful.

Turns out this was straightforward to implement!
There is now support for content-tabs in the latest version of mdformat-admon>=1.1.0 :tada:

I've moved the Content Tab logic to https://github.com/KyleKing/mdformat-mkdocs#mdformat-mkdocs along with a number of other improvements that should make the feature more reliable
I can release v2.0.0 as soon as mdformat-gfm is released (https://github.com/hukkin/mdformat-gfm/issues/31), but in the meantime you can try out the improvements with:
pip install mdformat-mkdocs@git+https://github.com/KyleKing/mdformat-mkdocs.git

authors: KyleKing leonardcser

repo: KyleKing/mdformat-admon


text:
mediapipe가 파이썬 3.11 버전에서는 지원하지 않음

파이썬 버전을 가장 최신버전에서는 지원하지 않으므로 다운그레이드 할 필요성이 있음

dependency에 추가하도록 하겠습니다.

https://google.github.io/mediapipe/getting_started/troubleshooting.html#:~:text=after running pip install mediapipe usually indicates that there is no qualified MediaPipe Python for your system. Please note that MediaPipe Python PyPI officially supports the 64-bit version of Python 3.7 to 3.10 on the following OS%3A 관련 웹페이지 내용


text:
[BUG] Error in the variable name

Thanks for the stubs!
Error in the variable name. You need to use max_commit_time_ms instead of max_commit_time_ns
In stubs
    def start_transaction(
        self,
        read_concern: typing.Optional[pymongo.read_concern.ReadConcern] = None,
        write_concern: typing.Optional[pymongo.write_concern.WriteConcern] = None,
        read_preference: typing.Optional[_ReadPreferences] = None,
        max_commit_time_ns: typing.Optional[int] = None,
    ) -> _MotorTransactionContext: ...

In motor
    def start_transaction(
        self, read_concern=None, write_concern=None, read_preference=None, max_commit_time_ms=None
    ):

Thanks for reporting this one! Your contribution is much appreciated! On it to verify and resolve the issue.

I've committed the resolution but can't push as I'm in a low connectivity area. Will push when I'm back.

Resolved as of Commit 3c2ce3c7eb98ebc5b41cad351b9b72bf54af97ce. Thanks again for notifying about the typo! Much appreciated! Thanks for your contribution!


text:
Corregir campo de edición de un registro

En un registro, cuando se edita el campo Nombre aceptado, en realidad se está editando el campo Nombre original. Hacer las modificaciones necesarias para que al api se envíe el campo acceptedNameUsage al momento de editar un registro

El ajuste quedó implementado en este commit. El Pull request se creará después de mezclar este otro debido a dependencia.


text:
Module/igv/1.0

Pull Request Checklists
Important: When opening a pull request, keep only the applicable checklist and delete all other sections.
Checklist for New Module
Required

[x] I used the cookiecutter template and updated the placeholder rules.

[x] The snakemake rules follow the design guidelines.

[x] All references to the rules object (e.g. for input files) are wrapped with str().

[x] Every rule in the module is either listed under localrules or has the threads and resources directives.

[x] Input and output files are being symlinked into the CFG["inputs"] and CFG["outputs"] subdirectories, respectively.

[x] I grouped the input symlinking rule to the next job that uses the input files.

[x] I updated the final target rule (*_all) to include every output rule.

[x] I explained important module design decisions in CHANGELOG.md.

[x] I tested the module on real data for all supported seq_type values.

[x] I updated the default.yaml configuration file to provide default values for each rule in the module snakefile.

[x] I did not set any global wildcard constraints. Any/all wildcard constraints are set on a per-rule basis.

[x] I ensured that all symbolic links are relative and self-contained (i.e. do not point outside of the repository).

[x] I replaced every value that should (or might need to) be updated in the default configuration file with __UPDATE__.

[x] I recursively searched for all comments containing TODO to ensure none were left. For example:
grep -r TODO modules/<module_name>/1.0

If applicable

[ ] I added more granular output subdirectories.

[ ] I added rules to the reference_files workflow to generate any new reference files.

[ ] I added subdirectories with large intermediate files to the list of scratch_subdirectories in the default.yaml configuration file.

[ ] I updated the list of available wildcards for the input files in the default.yaml configuration file.

Checklist for Updated Module
Important! If you are updating the module version, ensure the previous version of the module is restored from master.
If you want to restore a deleted file or directory from the remote master, you can use git checkout origin/master path/to/file,
then a git commit will ensure that file is tracked on your branch again.
Example:
mv modules/strelka/1.1 modules/strelka/1.2
git checkout origin/master modules/strelka/1.1

I have addressed most of the comments above, but I need to further investigate regarding:

if crams can be used for snapshots https://github.com/LCR-BCCRC/lcr-modules/pull/303#discussion_r1543446315
downloading IGV vs using conda environment https://github.com/LCR-BCCRC/lcr-modules/pull/303#discussion_r1543483073

plus i have made added some more changes since, so I will ask for an updated review once I'm finished

repo: LCR-BCCRC/lcr-modules


text:
Add MANIFEST.in to ensure package data is correctly installed

This correctly installs package data so that it is findable by importlib.resources. I also removed a .DS_Store file that should not have been in the repo in the first place.

yolo

repo: LEB-EPFL/eda-napari


text:
Adding unit tests to the operator

This PR bumps the operator/internal package from 35% test coverage to 67%

There is some commented-out tests for me to write later

repo: LF-Decentralized-Trust-labs/paladin


text:
runaway volume control

_roon-extension-rheos_logs.txt
_roon-extension-rheos_logs(1).txt

Am closing this as it seems this is no longer an issue.

authors: LINVALE markpinis

repo: LINVALE/RHEOS


text:
Temporarily switch from clang to gcc for sgx tests

Openenclave SDK samples tests fail with clang compiler. Temporarily switch to gcc to let the tests pass. Will switch back after new OE SDK package is released.

Test results -
[LISAv2 Test Results Summary]
Test Run On           : 04/21/2020 01:59:36
ARM Image Under Test  : Canonical : UbuntuServer : 18_04-lts-gen2 : latest
Initial Kernel Version: 5.0.0-1036-azure
Final Kernel Version  : 5.0.0-1036-azure
Total Test Cases      : 1 (1 Passed, 0 Failed, 0 Aborted, 0 Skipped)
Total Time (dd:hh:mm) : 0:0:7

   ID TestArea             TestCaseName                                                                TestResult TestDuration(in minutes)
-------------------------------------------------------------------------------------------------------------------------------------------
    1 SGX                  VALIDATE-INTEL-SGX-DRIVER-FOR-DC-VM                                               PASS                 3.73

[LISAv2 Test Results Summary]
Test Run On           : 04/21/2020 01:59:58
ARM Image Under Test  : Canonical : UbuntuServer : 16_04-lts-gen2 : latest
Initial Kernel Version: 4.15.0-1077-azure
Final Kernel Version  : 4.15.0-1077-azure
Total Test Cases      : 1 (1 Passed, 0 Failed, 0 Aborted, 0 Skipped)
Total Time (dd:hh:mm) : 0:0:7

   ID TestArea             TestCaseName                                                                TestResult TestDuration(in minutes)
-------------------------------------------------------------------------------------------------------------------------------------------
    1 SGX                  VALIDATE-INTEL-SGX-DRIVER-FOR-DC-VM                                               PASS                 3.93

authors: Francis-Liu LiliDeng