Sometime lookup endpoint failed to give proper result in absence of prototype.

Below are the example of a few domains and there results

websiteName=lfcla.com
https://api-gw.dev.platform.linuxfoundation.org/cla-service/v4/company/lookup?websiteName=lfcla.com
{"Code":"404","Message":"EasyCLA - 404 Not Found - error occured while search orgname , websitename lfcla.com - error: [GET /v1/lookup][404] lookupNotFound  map[]","x-request-id":"15c51842-9776-448a-a604-4de569778954"}

websiteName=https://www.lfcla.com
https://api.dev.lfcla.com/v3/organization/search?websiteName=https://www.lfcla.com
Response :
{"Code":"404","Message":"EasyCLA - 404 Not Found - error occured while search orgname , websitename https://www.lfcla.com - error: [GET /v1/lookup][404] lookupNotFound  map[]","x-request-id":"681d0078-2bd0-4752-bbee-1318d61010be"}

websiteName=https://lfcla.com
https://api.dev.lfcla.com/v3/organization/search?websiteName=https://lfcla.com
Response :
{"list":[{"organization_id":"0012M00002NDacSQAT","organization_name":"cla linux","organization_website":"https://lfcla.com"}]}

This is a defect raised for #2351.

David Deal  11:54 AM
@niravpatel27 - Regarding the Clearbit API - can we work to find partial matches? We have integrated into EasyCLA and it is working as expected, however, we are concerned that the user will type:

https://lfcla.com => finds a match :white_check_mark:
http://llfcla.com => does not find a match :x:
lfcla.com => does not find a match :x:
www.lfcla.com => does not find a match :x:

Can we strip the protocol and other non-domain name items from the provided website URL and then search for that???  Ideally, this will be done in the Org Service API.
Nirav  12:19 PM on December 29th
Yeah let me see
Nirav  12:19 PM
Yeah let me see

David Deal  11:54 AM
@niravpatel27 - Regarding the Clearbit API - can we work to find partial matches? We have integrated into EasyCLA and it is working as expected, however, we are concerned that the user will type:

https://lfcla.com => finds a match :white_check_mark:
http://llfcla.com => does not find a match :x:
lfcla.com => does not find a match :x:
www.lfcla.com => does not find a match :x:

Can we strip the protocol and other non-domain name items from the provided website URL and then search for that???  Ideally, this will be done in the Org Service API.
Nirav  12:19 PM on December 29th
Yeah let me see
Nirav  12:19 PM
Yeah let me see

Waiting for Organization Service updates to support partial searches.

Waiting for Organization Service updates to support partial searches.

@vinod-kadam Resolved now so closing the ticket

@vinod-kadam Resolved now so closing the ticket


Vulnerable version of xmldom as a dependency

mathml-to-latex depends on an older version of xmldom which has a vulnerability.
https://github.com/advisories/GHSA-5fg8-2547-mr8q

@caleb-flores @asnunes can you please merge this pull request to resolve the vulnerabilities


chore: added a project

Related Issue
Closes #1480
Description
Added the UI design of a cat purchasing website
Screenshots

@priyankarpal I have added the changes. Please merge now

@priyankarpal Please add a GSSOC tag in this PR . I am  a GSSOC contributor. It is necessaary for me!

@priyankarpal Please add a GSSOC tag in this PR . I am a GSSOC contributor. It is necessaary for me!

previously we've added but currently, We are not providing any labels for project additions as they are not associated with solving any issues.


[kube-prometheus] cAdvisor metrics are unavailable with Kubernetes v1.7.3+

What did you do?
Successfully installed kube-prometheus in a Kubeadm cluster v1.7.5.
What did you expect to see?
The cAdvisor endpoints in the Prometheus kubelet job working correctly.
What did you see instead? Under which circumstances?
Several metrics are gathered correctly, but not the cAdvisor ones in the kubelet job.
Environment

Kubernetes version information:

Client Version: version.Info{Major:"1", Minor:"7", GitVersion:"v1.7.4", GitCommit:"793658f2d7ca7f064d2bdf606519f9fe1229c381", GitTreeState:"clean", BuildDate:"2017-08-17T08:48:23Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"7", GitVersion:"v1.7.5", GitCommit:"17d7182a7ccbb167074be7a87f0a68bd00d58d97", GitTreeState:"clean", BuildDate:"2017-08-31T08:56:23Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes cluster kind:

Kubeadm at v1.7.5.

Manifests:

kube-prometheus/manifests/prometheus/prometheus-k8s-service-monitor-kubelet.yaml
[...]
spec:
  jobLabel: k8s-app
  endpoints:
  - port: http-metrics
    interval: 30s
  - port: cadvisor
    interval: 30s
    honorLabels: true

Basically, as the cAdvisor metrics have been moved, that configuration is not working anymore.
The official Prometheus' Kubernetes configuration example has already been updated with the change.
A similar configuration should be applied to the prometheus-k8s-service-monitor-kubelet.yaml manifest too, e.g.
[...]
spec:
  jobLabel: k8s-app
  endpoints:
  - port: http-metrics
    interval: 30s
  # This is for cAdvisor in K8s 1.7.3+
  - path: /metrics/cadvisor
    port: http-metrics
    interval: 30s
    honorLabels: true

It worked in my environment, but of course it might not be backward-compatible.
Perhaps changing it in the kube-prometheus Helm chart (I'm assuming they are working the same way - but haven't tested it yet) and adding a configuration property to the chart o switch the behavior might be a better option.
However, I couldn't managed to find a way to express the cAdvisor metric link with the proxy, as done in the Prometheus official example (https://kubernetes.default.svc:443/api/v1/nodes/<NODE_NAME>/proxy/metrics/cadvisor instead of http://<NODE_IP>:10255/metrics/cadvisor which is the result of this configuration change).
Since it would be useful also to access the metrics endpoint for the kube-scheduler pod (which is only available through the master-proxy in my setup), I was wondering if it is possible to build such endpoint.

I'm not super familiar with kubeadm, but the change in 1.7.3 was unrelated to the cAdvisor metric endpoint exposed on port 4194, the metrics were only removed from the /metric endpoint of the kubelet and moved to /metric/cadvisor.
I think in terms of this issue this was actually a firewall issue, that your worker that runs the Prometheus pod doesn't have network access to the kubelet 4194 port.

@brancz I've dug into the actual kubelet configuration and you're right: the default kubeadm configuration disables the cAdvisor port (with the --cadvisor-port=0 flag).
Actually I've just found the original thread on the Prometheus repository, which already covered this explanation.
Personally, I'm not very willing to change the kubeadm settings and redeploy the cluster just to enable that port, while there is already another endpoint working (and changing a manifest is enough to exploit it). But that's just me 😄
Feel free to decide whether the alternative is worth being included or not; I can use the workaround for now (and perhaps other people might find this useful too).

metoo, with kube-aws. seems like eventually scraping the /metrics/cadvisor endpoint will be preferred? We'll maintain a fork till this is resolved (see commit pingback)

We'll probably move to the /metrics/cadvisor endpoint when we upgrade everything to be target 1.8.0+.

Is it meant to be adding in a kubelet/4194 target by default?
We have:  k8s 1.7.7 built with bootkube
prometheus-operator 0.14.0 (not meddled with the versions, to prometheus 2.0.0-rc1)
We, as the original poster described, get kubelet metrics but no cadvisor metrics.  Checking prometheus' targets, its scraping :10250 as per normal.
I don't really mind whether it would get cadvisor metrics from 4194/metrics or 10250/metrics/cadvisor - but how do we get it to do... either?
It's clear from the kubernetes issues that they don't really consider breaking them out and making the separate a big thing - so they're unlikely to put them back into 10250/metrics.  For people hand-rolling their prometheus config it's not too difficult, but when operator is doing it, we lack the control to add things in (one day, prometheus will get the hang of multiple config sources).
So, what have people who use prometheus-operator been doing to get those metrics back?

Can you explain more what you think is not working today? I’m saying this because I think all combinations are possible today, but possibly not well enough documented.
If one wants to use the 4194/metrics endpoint then that’s already reflected in the kube-prometheus manifests.
If one wants to use the 10250/metrics/cadvisor metrics then one has to modify the servicemonitor endpoint provided in kube-prometheus and set the explicit metrics endpoint.
Does that clear things up?

@ghostflame To use the 10250/metrics/cadvisor endpoint, it should be enough to edit the kube-prometheus/manifests/prometheus/prometheus-k8s-service-monitor-kubelet.yaml manifest the way I reported it in the issue description:
change
- port: cadvisor
    interval: 30s
    honorLabels: true

to
- path: /metrics/cadvisor
    port: http-metrics
    interval: 30s
    honorLabels: true

and redeploy.
Hope this helps...

@lorenzo-biava

you don't have to redeploy kubernetes by change kubeadm settings, just change address from <IP_ADDRESS> to <IP_ADDRESS> in etc/kubernetes/manifests/kube-scheduler.yaml, and kube-controller-manager.yaml and restart docker & kubelet works
as you have mentioned the correct servicemonitor configs - path: /metrics/cadvisor  port: http-metrics  interval: 30s  honorLabels: true
or `[...]
spec:
jobLabel: k8s-app
endpoints:

port: http-metrics
interval: 30s

This is for cAdvisor in K8s 1.7.3+

path: /metrics/cadvisor
port: http-metrics
interval: 30s
honorLabels: true`
I did not get it work, the alarm manager still reports k8skubeletdown.
Can you give some hints?

@lzbgt thks

@lzbgt Sorry I've missed this; I've not been working with Kubernetes and Prometheus Operator lately, but it's definitely good to know that there's a dedicated doc for kubeadm configurations now. Thanks for pointing that out :wink:

Closing as sample configurations are available and specific documentation for certain platforms as well.

Thanks @lorenzo-biava


Misplaced comment after minification

Description:
Looks like the bug I ran into had nothing to do with the changes in #7856, since it's reproducible without it. Looks like it might have only surfaced now because https://github.com/swc-project/swc/pull/7853 changed the default value of jsc.minify.format.comments? Added a minimal test case here with the expected result.
Here's the actual output:
 export var padding = '';
 export function exec2({ commands }) {
     return __awaiter(this, void 0, void 0, function*() {
         for(let i2 = 0; i2 < commands.length; i2++){
             let command = commands[i2];
             yield // some-comment
             function({ command }) {
                 command();
             }({
                 command,
                 handleError
             });
         }
     });
 }

The comment ends up getting added after the yield, which makes the output invalid.
Going to see if I can figure out a fix tomorrow, but let me know if you have any ideas on where to start looking in the meantime!

We may need need_paren like in https://github.com/swc-project/swc/blob/5ae2e810d8d8a2d23c9e81eae128fb72c5c1f09c/crates/swc_ecma_codegen/src/lib.rs#L2936-L2940 for emit_yield_expr too.
https://github.com/swc-project/swc/blob/5ae2e810d8d8a2d23c9e81eae128fb72c5c1f09c/crates/swc_ecma_codegen/src/lib.rs#L1882-L1886

@kdy1 thank you so much for the suggestion, it worked great!
Added the fix and updated the expected output accordingly in https://github.com/swc-project/swc/pull/7858/commits/7d1bc9f0f56c02c57d92bbfeed45a4c3114b5a49


Multiple responsive sliders break when new dots event fired

I was able to replicate this with the latest version (1.5.2), 1.4.1 doesn't seem to have an issue with the bug though, where as 1.5.0 does.
If you have multiple sliders on a page which some by default have dots enabled and others don't, if a new breakpoint happens and dots are set to true where they previously weren't then it will kill other sliders dot interactions. One way to restore the behaviour back to the dots is to either use arrows or swipe action to move to the next pane, after this the behaviour is back until the next breakpoint.
I've setup an external demo to show, if you reduce the page width to where the bottom slider has dots appear you'll notice the top slider dot interaction stops working. Same happens if you have dots enabled by default and remove them at a later breakpoint, when you scale back up the dot event being fired breaks other image sliders.
http://jsfiddle.net/rr0zxoeL/
$('.responsive').slick({
  dots: true,
  infinite: true,
  speed: 300,
  slidesToShow: 2,
  slidesToScroll: 2
});

$('.responsive2').slick({
  infinite: false,
  speed: 300,
  slidesToShow: 1,
  slidesToScroll: 1,
  responsive: [
    {
      breakpoint: 500, // this breakpoint kills .response dots
      settings: {
        slidesToShow: 1,
        slidesToScroll: 1,
        infinite: true,
        dots: true
      }
    }
  ]
});

great slider so far! this is really one of the best. I am having the same problem with multiple sliders and breakpoints/dots. purely i can´t fix it by myself. :(

I can see the bug, thanks for the description and the JSFiddle. - helps a lot! :+1:
Think I already have a patch for it, gonna test a little more and then push to my branch :wink: :sparkling_heart:
Si.

@simeydotme There still seems to be some issues, it's generally when scaling back up. I've updated the jsfiddle with the latest code source and added a few more breakpoints, where it removes and adds.
I believe the issue is happening when the dots are set to false and then back to true. http://jsfiddle.net/rr0zxoeL/1/
New response code
responsive: [
  {
    breakpoint: 500,
    settings: {
      slidesToShow: 1,
      slidesToScroll: 1,
      infinite: true,
      dots: true
    }
  },
  {
    breakpoint: 400,
    settings: {
      slidesToShow: 2,
      slidesToScroll: 2,
      dots: false
    }
  },
  {
    breakpoint: 300,
    settings: {
      slidesToShow: 1,
      slidesToScroll: 1
    }
  }
]

Really @nbankzy ? 😱 Jeez, let me see... Would imagine this is unrelated, though, as logically the patch I did was solid and oobvious... Thats for the effort of creating the fiddle and contributing 🙇 I'll see what I can find.
Si.

haha, sorry @nbankzy ... you had not taken the current master branch.
http://jsfiddle.net/simeydotme/rr0zxoeL/2/
I updated the slick.js file to the current, unminified, master branch. it works fine? - I'm afraid you'll have to wait for @kenwheeler the plugin author to collate the recent changes into a tag-release, or minify that file yourself. :smiley:
Simon.

@simeydotme looks like your solution was solid, sorry about that! was sure I had selected the newest code release, oh well. Thanks for the fix!


Update README.md to include the CLA

Updated README to include the CLA (Contributor Licensing Agreement) for this repo

Checklist

[x] I have made a material change to the repo (functionality, testing, spelling, grammar)
[x] I have read the [Contribution Guide] and my PR follows them.
[x] I updated my branch with the master branch.
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation about the functionality in the appropriate .md file
[x] I have added in line documentation to the code I modified

Short description of what this PR does:

Adds a CLA (Contributor License Agreement in README.md section

If you have questions, please send an email to Sendgrid, or file a Github Issue in this repository.

Hello @Bharat123rox,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


#93 Add PPC64 lines

Added pp64 lines . Please push to real-CI to see how it pans out and advise on proper TOOLCHAIN_ARCH variable.
Signed-off-by: Tisham Dhar<EMAIL_ADDRESS>

Looks like you might need to send this as 3 seperate pull request so the deps are available.

In compiler, binutils, stdlib sequence ?

binutils
compiler
stdlib

Is the other I think?

Replaced by https://github.com/litex-hub/litex-conda/pull/96, https://github.com/litex-hub/litex-conda/pull/97 and #98


this.setState is not a function on clean component render

React-Quill version

[x ] 1.2.3

I am importing quill through import Quill from 'react-quill'

The setState method would be provided by the React component prototype; it wouldn't necessarily appear in the devtools as part of the object. Is your component perhaps unmounting before the editor setup is complete?

It's not unmounting, it's just a normal render with no props. What information can I provide for you to check further?

A Codepen to demonstrate the issue would help.
By the way, it looks like you're trying to use the editor as a form element. The editor contents won't be captured on submit because it does not wrap a DOM element like a textarea or input. You need to provide a change handler and synchronize with form state yourself. (Another strategy is to sync with a hidden input).

I'm aware I need to handle change events from Quill. I just didn't add any props to make sure they weren't the issue and confirmed that those errors are still appearing, I will be checking on a clean repo and if I can reproduce it, I will let link you. Maybe it could be the way the parent component behaves.

Closing this issue since I can't reproduce on a clean project. Thank you!


Atom asks to save changes when quitting (OS X)

When quitting Atom, unsaved documents ask to save changes which is unexpected behaviour on OS X.
Other apps that are able to restore windows such as Pages, Numbers, Keynote and Sublime Text 3 don't ask to save changes, they simply open the unsaved documents the next time the app is launched.

Good news, @nuno-agostinho!  The functionality you're asking for has just been implemented in the 1.4 betas, so feel free to try those out and report back :smile:.

@50Wliu Thanks! Works like a charm in the 1.4 beta 3! :)


Avoid invocation of cvmfsexec distros building if not necessary

As identified in #308, this PR brings in a change to the existing use of subprocess module (via the GWMS subprocessSupport module) for efficiency purposes.
Currently, the subprocessSupport module is used to execute /usr/bin/create_cvmfsexec_distros.sh always, regardless of whether cvmfsexec distributions are required to be built or rebuilt. If CVMFS is not needed and the cvmfsexec_distro tag is not defined in the factory configuration (disabled mode), then calling subprocessSupport to execute the script is basically unnecessary. Additionally, since subprocess spawns a new process to execute the script, this might result in a memory-intensive operation in a production setting.

@mmascher Upon reviewing this PR, I realized that the suggestion you made to cgWParamDict.py can be modified such that the invocation of /usr/bin/create_cvmfsexec_distros.sh can be added to the existing if cfgs: condition, which is right after the lines corresponding to the invocation of subprocessSupport.iexe_cmd, instead of adding an extra if condition. I modified accordingly and added a message to inform the user about the building/rebuilding being disabled since that was previously being handled inside the create_cvmfsexec_distros.sh script. I also modified the description you had originally added to the list of Bug Fixes in the CHANGELOG file.
Thanks for catching this as well as sharing your suggestions for improvement!


Compiler fails if there are two fragments in a single file

On https://github.com/sjwilczynski/graphitation/tree/user/stwilczy/fragmentRepro I added another fragment to two components in apollo-watch-fragments example. They mimic setup we have in 1JS, where we very often have more than one fragment (view data and domain data, and sometimes even another domain data one). In such case even if the fragment is very simple compiler fails with:
➜  apollo-watch-fragments (user/stwilczy/fragmentRepro) yarn duct-tape-compiler
yarn run v1.22.15
$ duct-tape-compiler --schema ./data/schema.graphql --src ./src --emitQueryDebugComments

Writing ts
DeprecationWarning: 'createTypeAliasDeclaration' has been deprecated since v4.8.0. Decorators are no longer supported for this function. Callers should switch to an
overload that does not accept a 'decorators' parameter.
DeprecationWarning: 'createImportDeclaration' has been deprecated since v4.8.0. Decorators are no longer supported for this function. Callers should switch to an ove
rload that does not accept a 'decorators' parameter.
ERROR:
Error writing modules:
Invariant Violation: Expected to find a @refetchable directive on TodoList_viewDataFragment
    at invariant (C:\Users\stwilczy\Repos\graphitation\node_modules\invariant\invariant.js:40:15)
    at printWatchNodeQueryReExport (C:\Users\stwilczy\Repos\graphitation\packages\apollo-react-relay-duct-tape-compiler\src\formatModule.ts:118:12)
    at C:\Users\stwilczy\Repos\graphitation\packages\apollo-react-relay-duct-tape-compiler\src\formatModule.ts:98:33
    at C:\Users\stwilczy\Repos\graphitation\node_modules\relay-compiler\lib\codegen\writeRelayGeneratedFile.js:153:22
    at Generator.next (<anonymous>)
    at asyncGeneratorStep (C:\Users\stwilczy\Repos\graphitation\node_modules\@babel\runtime\helpers\asyncToGenerator.js:3:24)
    at _next (C:\Users\stwilczy\Repos\graphitation\node_modules\@babel\runtime\helpers\asyncToGenerator.js:25:9)
    at C:\Users\stwilczy\Repos\graphitation\node_modules\@babel\runtime\helpers\asyncToGenerator.js:32:7
    at new Promise (<anonymous>)
    at C:\Users\stwilczy\Repos\graphitation\node_modules\@babel\runtime\helpers\asyncToGenerator.js:21:12
error Command failed with exit code 100.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

@alloy in latest commit to linked branch I added all the things that I was able to found. I wasn't able to write unit tests as I don't know compiler or duct tape code well enough but I pushed changes to schema in apollo-watch-fragments example with issue. To allow you to easier navigate, I added for each issue a comment in code like [issue 1]: .... which is either immediately seen at app runtime or triggered by uncommenting/adjusting fragment

Fixed by #341


Add Logo Result, Name Result, and TVNetwork item

This adds the following:

Logo Result (for Images attached to a TVNetwork; their format differs from other Image Results)
Name Result (for Alternative Names)
TVNetwork Item

I've included tests for each as well. The only "non-standard" method I added was getBestFilePath() on a Logo result. The API always returns a .png file path, but for some Logos there is an .svg available which is indicated in the file_type field. This method checks for the existence of an svg image and returns that, otherwise returns the png.

Odd, not sure why the tests failed there, they all pass locally.

In complement of my comments, tests code on Items\TVNetwork missing:

I assume the non-covered lines it is complaining about are tests for when the API would be returning an "empty" search results list (for instance such as companyEmptyOk.json). Since I didn't see a way to search for Networks with the API, I'm not sure we'll ever have empty results like that. What I did was find one where it returns empty strings for each attribute, but that doesn't seem to satisfy the code-coverage since the properties will still be set on the object and the return ''; will never run. Maybe the solution is to remove those checks in the getters, since if we call for a Network that doesn't exist it'll throw an API exception instead of returning an empty object. What do you think?

Also, how do you want me to handle the @author tag? Obviously I am reusing a lot of your code here. Do you want me to also add an @author tag for you on the class/method-level or is the one in the header sufficient?

Also, how do you want me to handle the @author tag? Obviously I am reusing a lot of your code here. Do you want me to also add an @author tag for you on the class/method-level or is the one in the header sufficient?
edit: Will add the new commits once your test-fixing commits are merged.

I think that if you create a new method/class you must mentionned only your name. If you complete an existant class / method, you add you name in comment.
My name is at the beginning of each files to specify the package it is sufficient.

In complement of my comments, tests code on Items\TVNetwork missing:

I assume the non-covered lines it is complaining about are tests for when the API would be returning an "empty" search results list (for instance such as companyEmptyOk.json). Since I didn't see a way to search for Networks with the API, I'm not sure we'll ever have empty results like that. What I did was find one where it returns empty strings for each attribute, but that doesn't seem to satisfy the code-coverage since the properties will still be set on the object and the return ''; will never run. Maybe the solution is to remove those checks in the getters, since if we call for a Network that doesn't exist it'll throw an API exception instead of returning an empty object. What do you think?

You should not remove these tests in the getters because, if you check  the documentation all response fields are optional !

In consequence, possibly you can have a valid response (200) with no information inside. You must simulate these response in your json response to test this.


Lava Player Loading But Not Playing Tracks

I have been using Lava Player 1.3.19 for a while now with my Discord Bot, but unfortunately in the past few days I noticed it will not play most YouTube tracks. I sends a message saying it loaded the track, but no audio plays. I tried going back to a previous branch of my program to see if updating my bot to JDA 4 broke it, but my old program on JDA 3 had the same issue. Anyway I have tried to fix it for a while but cant seem to, if anyone else has this same issue please let me know. I am very interested to find out the cause of this bug. Thanks!

Have you tried updating to the latest version?

Yeah I switched to the newest version of Lava Player and the problem still persists. I went to several other versions too.

turn on debug logs and look for anything unusual. for example, a timeout while connecting to discord voice servers can mean that your IP is blocked (which is the case for all hetzner-based hosts).

I seem to have a similar bug started just recently. I've noticed that if you restart discord (I mean completely from tray) while the bot is playing, it starts working again.

I seem to have a similar bug started just recently. I've noticed that if you restart discord (I mean completely from tray) while the bot is playing, it starts working again.

Yeah that seems to have fixed it for at the moment. Thanks for the help man, I have been befuddled by this for days.

The problem is that if you stop the bot and start another track the same behaviour repeats.

I contacted Discord about the issue. They sent me an automated response saying that YouTube is limiting bots from playing music. This doesn't explain why sometimes I can't hear the bot playing music, but my friends can. I am waiting for a response from Discord. Thanks everyone for the help, I will post here when I get any new information.

This is most definitely a discord bug, then. I'm pretty sure i've seen this months ago in their trello board, though i can't find it (the board) atm.
you might want to ask on the discord testers server: https://discord.gg/discord-tester

This is most definitely a discord bug, then. I'm pretty sure i've seen this months ago in their trello board, though i can't find it (the board) atm.
you might want to ask on the discord testers server: https://discord.gg/discord-tester

Here seems to be an issue with describes the exact opposite https://trello.com/c/UkNEavqc/1243-discord-does-not-keep-a-bot-muted-after-restart
Btw I suppose that's the board you were looking for? https://trello.com/b/AExxR9lU/desktop-bugs

Yeah I joined the discord bot developers discord and it appears YouTube is block some music bots IPs. That’s not my issue. I believe my issue is a bug with discord itself. Anyway. There’s not much more to do so I think I’m going to close this issue. Thanks for all the help guys I really appreciate it. Have a good day!

Seems like bots of middle up to large scale are addressed by this limitation. They do this by analyzing traffic and blocks them if they match bot like behavior. That would explain why our smaller bots still work, even though by working around.


React-Router docs contribution required

Is your feature request related to a problem? Please describe.
I often find that new users struggle to understand the nuances of nested routes in React Router. Specifically, the current documentation does not provide enough real-world examples or clear explanations of how to effectively implement nested routing in various scenarios. This can lead to confusion and incorrect implementations, which ultimately affects user experience.
Describe the solution you'd like
I propose enhancing the React Router documentation by including detailed examples of nested routes. These examples should cover common use cases, such as dynamic routes with parameters, route-based code splitting, and protecting routes with authentication. Additionally, a step-by-step guide or video tutorial could greatly benefit users who are visual learners.
Describe alternatives you've considered
While there are some third-party tutorials and articles available online, they often lack the authority and up-to-date information that comes from official documentation. Additionally, community forums can provide some insights, but they are not always comprehensive or reliable.
Additional context
Having clear and thorough documentation will not only help new users adopt React Router more easily but also improve overall community support by reducing the number of questions related to nested routing. Screenshots and code snippets could further illustrate the concepts effectively.

Hi there,
Can you please assign  this to me ?

Sure  , happy contributions 😊

Hi @AdityaSrivastava185 , I have a small doubt there is already a react-router  documentation in the src.

Yes @Nitheesh-2003 , there are lot of things which needs to be improved , keep the things simple , easy to understand and structured for learning

Sure @AdityaSrivastava185, Will work on it

Hi @AdityaSrivastava185
I have updated the documentations of React Router.
Can you please review it and merge it.
Thank you


Fix E2E tests, make service account outputs depend on IAM grants

Make outputs of service account depend on the IAM grants given within the module, so SA has all necessary permissions when used in other contexts.

Checklist

I applicable, I acknowledge that I have:

[ ] Read the contributing guide
[ ] Ran terraform fmt on all modified files
[ ] Regenerated the relevant README.md files using tools/tfdoc.py
[ ] Made sure all relevant tests pass

FAST E2E run
Fabric modules E2E run

This is one more of the "I'm not sure this won't introduce circular deps" PRs...

I know :-)

I'm not yet sure. I'm waiting for E2E tests to go :boom: :-)

I'm not yet sure. I'm waiting for E2E tests to go 💥 :-)

well, seeing cycles pop up in the plan tests is a red flag enough for me :)


Support for Min_p

I would like to have min_p as a ollama modelfile parameter.
See the link for context of this request: https://www.reddit.com/r/LocalLLaMA/comments/17vonjo/your_settings_are_probably_hurting_your_model_why/

Duplicate of #1142 (corresponding PR: #1825)

Thanks for the request @twalderman, just resolving this one for now to keep things organized in the the one issue mentioned above.


Hardware Specs for running the service

Hi,
What hardware specs does the machine that cloud.science-miner.com/grobid/ is running on have?
We have deployed the docker container on an AWS c5.xlarge instance (4vCPUs and 8GB RAM) but it is taking more than a minute to process (processFulltextDocument) individual PDFs with like 5-10 pages whereas the hosted service is able to achieve the same in less than 5 secs.

Hi @samanyougarg !
The demo on science-miner runs on a Linode instance 6 CPU Cores, 16 GB RAM together with another demo and it hardly uses more than 6GB in total. I am not using a docker container, but on another deployment I did and I had very similar performance.
Maybe such a super long runtime might be due to the consolidation option set for all extracted bibliographical references? If CrossRef is used, it happens that their API is lagging and with many reference to match, it's possible that we have something very slow from time to time.
I have never used a AWS instance with grobid myself, but other Grobid users are and they got usually faster runtime than the science-miner demo, because their machine doesn't host other services.

Thank you very much for the quick reply @kermitt2!
I am using the docker image without any change and don't think I have consolidation option set for all extracted bibliographical references.
Thank you for letting me know. Will test on a different server again with/without docker.

Quick update: I installed the service using the docker container on a Digital Ocean CM with 4vCPUs and 8GB RAM. It is working well now.


Create flake.nix

This will make it far easier to get started with eframe/egui. Simply running the following command will (hopefully) set up a working environment: nix --experimental-features 'nix-command flakes' develop (if Nix is installed)

See also #27.


Support ensembles of models as well as cascading models

Wide (Spark ML) + Deep (Tensorflow AI)

also, cascading models can help control latency-accuracy trade-off.
can configure - or turn dials at runtime - to force accuracy vs. performance (gpu's, etc)

Good inspiration:  https://github.com/riga/tfdeploy/


Plugin doesn't respect tsconfig.json exclude files.

I have a simple react project and I want to prevent vite from transforming certain files from within my src directory. I have an exclude line in tsconfig.json and tsc ignores these files, but vite doesn't. It complains that it can't find modules imported in those files. I don't want those files transformed at all.
How do fix?

That's how bundler works, they try to resolve imports that are not type only. If your import is a runtime one, what would it mean to no transform it?


Connection reset by peer - SSL_connect (Errno::ECONNRESET)

What did you do?
> bundle exec jekyll serve --drafts
What did you expect to happen?
Start jekyllrb server (using Mac OS X 10.9.5)
What happened instead?

> bundle exec jekyll serve --drafts
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/net/http.rb:918:in `connect': Connection reset by peer - SSL_connect (Errno::ECONNRESET)
	from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/net/http.rb:918:in `block in connect'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/timeout.rb:52:in `timeout'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/net/http.rb:918:in `connect'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/net/http.rb:862:in `do_start'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/net/http.rb:851:in `start'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/open-uri.rb:313:in `open_http'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/open-uri.rb:709:in `buffer_open'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/open-uri.rb:210:in `block in open_loop'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/open-uri.rb:208:in `catch'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/open-uri.rb:208:in `open_loop'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/open-uri.rb:149:in `open_uri'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/open-uri.rb:689:in `open'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/open-uri.rb:34:in `open'
	from ~/git/techgraphics.github.io/Gemfile:7:in `eval_gemfile'
	from /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler/dsl.rb:32:in `instance_eval'
	from /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler/dsl.rb:32:in `eval_gemfile'
	from /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler/dsl.rb:10:in `evaluate'
	from /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler/definition.rb:25:in `build'
	from /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler.rb:155:in `definition'
	from /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler.rb:118:in `setup'
	from /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler/setup.rb:8:in `<top (required)>'
	from /Library/Ruby/Site/2.0.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /Library/Ruby/Site/2.0.0/rubygems/core_ext/kernel_require.rb:54:in `require'

**> bundle install**

Error details

    Errno::ECONNRESET: Connection reset by peer - SSL_connect
    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/net/http.rb:918:in `connect'
    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/net/http.rb:918:in `block in connect'
    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/timeout.rb:52:in `timeout'
    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/net/http.rb:918:in `connect'
    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/net/http.rb:862:in `do_start'
    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/net/http.rb:851:in `start'
    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/open-uri.rb:313:in `open_http'
    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/open-uri.rb:709:in `buffer_open'
    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/open-uri.rb:210:in `block in open_loop'
    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/open-uri.rb:208:in `catch'
    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/open-uri.rb:208:in `open_loop'
    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/open-uri.rb:149:in `open_uri'
    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/open-uri.rb:689:in `open'
    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/open-uri.rb:34:in `open'
    ~/git/techgraphics.github.io/Gemfile:7:in `eval_gemfile'
    /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler/dsl.rb:32:in `instance_eval'
    /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler/dsl.rb:32:in `eval_gemfile'
    /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler/dsl.rb:10:in `evaluate'
    /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler/definition.rb:25:in `build'
    /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler.rb:155:in `definition'
    /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler/cli/install.rb:78:in `run'
    /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler/cli.rb:157:in `install'
    /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
    /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler/vendor/thor/lib/thor/invocation.rb:126:in `invoke_command'
    /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler/vendor/thor/lib/thor.rb:359:in `dispatch'
    /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler/vendor/thor/lib/thor/base.rb:440:in `start'
    /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler/cli.rb:10:in `start'
    /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/bin/bundle:20:in `block in <top (required)>'
    /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/lib/bundler/friendly_errors.rb:7:in `with_friendly_errors'
    /Library/Ruby/Gems/2.0.0/gems/bundler-1.9.2/bin/bundle:18:in `<top (required)>'
    /usr/bin/bundle:23:in `load'
    /usr/bin/bundle:23:in `<main>'

Environment
Bundler   1.9.2
Rubygems  2.4.7
Ruby      2.0.0p481 (2014-05-08 revision 45883) [universal.x86_64-darwin13]
GEM_HOME  /Library/Ruby/Gems/2.0.0
GEM_PATH  /Library/Ruby/Gems/2.0.0:/Users/gstaylor/.gem/ruby/2.0.0:/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/gems/2.0.0
Git       2.3.5

>ping -c5 rubygems.org
PING rubygems.org (<IP_ADDRESS>): 56 data bytes
Request timeout for icmp_seq 0
Request timeout for icmp_seq 1
Request timeout for icmp_seq 2
Request timeout for icmp_seq 3
--- rubygems.org ping statistics ---
5 packets transmitted, 0 packets received, 100.0% packet loss
――― TEMPLATE END ――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――

Note: they block ping so that’s not a good test.

Ugh, we really need to be catching ECONNRESET :(

@techgraphics unfortunately, this error has nothing whatsover to do with Bundler. You have put Ruby code into your Gemfile that causes this error:
require 'json'
require 'open-uri'
versions = JSON.parse(open('https://pages.github.com/versions.json').read)

Have a good day!


NFS Provisioner

Following the instructions in Azure Stack HCI Quickstart I have a cluster deployed and AKS Hybrid installed. AKS Hybrid is connected to Arc for Kubernetes. Generally the cluster is operating fine.
Wanted to try out NFS as a Persistent Storage. So, i setup NFS on a Ubuntu machine outside the Cluster. Able to mount the nfs drive on the Host. Validated the nfs server is reachable from within a Pod in the Cluster.
So overall networking and access to NFS is fine. However the provisioner is unable to provision the volume...
Log from csi-nfs-controller-xxxxxxx-xxxx
I0727 00:51:02.833479       1 controller.go:1337] provision "default/hci-nfs-pvc" class "nfs-csi": started
I0727 00:51:02.834322       1 event.go:285] Event(v1.ObjectReference{Kind:"PersistentVolumeClaim", Namespace:"default", Name:"hci-nfs-pvc", UID:"f05ee6d2-97a6-47b9-91d2-807374b971c1", APIVersion:"v1", ResourceVersion:"553997", FieldPath:""}): type: 'Normal' reason: 'Provisioning' External provisioner is provisioning volume for claim "default/hci-nfs-pvc"
I0727 00:51:02.906455       1 controller.go:1075] Final error received, removing PVC f05ee6d2-97a6-47b9-91d2-807374b971c1 from claims in progress
W0727 00:51:02.906495       1 controller.go:934] Retrying syncing claim "f05ee6d2-97a6-47b9-91d2-807374b971c1", failure 28
E0727 00:51:02.906528       1 controller.go:957] error syncing claim "f05ee6d2-97a6-47b9-91d2-807374b971c1": failed to provision volume with StorageClass "nfs-csi": rpc error: code = Internal desc = failed to mount nfs server: rpc error: code = Internal desc = mount failed: exit status 32
Mounting command: mount
Mounting arguments: -t nfs -o hard,nfsvers=4.1 <IP_ADDRESS>:/srv/nfs /tmp/pvc-f05ee6d2-97a6-47b9-91d2-807374b971c1
Output: mount.nfs: Operation not permitted
I0727 00:51:02.906657       1 event.go:285] Event(v1.ObjectReference{Kind:"PersistentVolumeClaim", Namespace:"default", Name:"hci-nfs-pvc", UID:"f05ee6d2-97a6-47b9-91d2-807374b971c1", APIVersion:"v1", ResourceVersion:"553997", FieldPath:""}): type: 'Warning' reason: 'ProvisioningFailed' failed to provision volume with StorageClass "nfs-csi": rpc error: code = Internal desc = failed to mount nfs server: rpc error: code = Internal desc = mount failed: exit status 32
Mounting command: mount
Mounting arguments: -t nfs -o hard,nfsvers=4.1 <IP_ADDRESS>:/srv/nfs /tmp/pvc-f05ee6d2-97a6-47b9-91d2-807374b971c1
Output: mount.nfs: Operation not permitted
To setup the NFS provisioner and Storage Class followed these instructions:
Install-AksHciCsiNfs -clusterName
apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
name: nfs-csi
provisioner: nfs.csi.akshci.com
parameters:
server:  <IP_ADDRESS> # NFS server endpoint
share: /srv/nfs # NFS share path
reclaimPolicy: Retain
volumeBindingMode: Immediate
mountOptions:

hard
nfsvers=4.1

PVC
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
name: hci-nfs-pvc
spec:
accessModes:

ReadWriteOnce
storageClassName: nfs-csi
resources:
requests:
storage: 1Mi
kubectl describe pvc
Name:          hci-nfs-pvc
Namespace:     default
StorageClass:  nfs-csi
Status:        Pending
Volume:
Labels:
Annotations:   volume.beta.kubernetes.io/storage-provisioner: nfs.csi.akshci.com
volume.kubernetes.io/storage-provisioner: nfs.csi.akshci.com
Finalizers:    [kubernetes.io/pvc-protection]
Capacity:
Access Modes:
VolumeMode:    Filesystem
Used By:
Events:
Type     Reason              Age               From                                                                     Message

Normal   Provisioning        6s (x5 over 22s)  nfs.csi.akshci.com_moc-lika3uh6pyg_2901a927-84d5-455f-864a-a2412a741e9f  External provisioner is provisioning volume for claim "default/hci-nfs-pvc"
Warning  ProvisioningFailed  6s (x5 over 22s)  nfs.csi.akshci.com_moc-lika3uh6pyg_2901a927-84d5-455f-864a-a2412a741e9f  failed to provision volume with StorageClass "nfs-csi": rpc error: code = Internal desc = failed to mount nfs server: rpc error: code = Internal desc = mount failed: exit status 32
Mounting command: mount
Mounting arguments: -t nfs -o hard,nfsvers=4.1 <IP_ADDRESS>:/srv/nfs /tmp/pvc-6880efc7-e44e-43ae-a257-181aac64f146
Output: mount.nfs: Operation not permitted
Normal  ExternalProvisioning  2s (x3 over 22s)  persistentvolume-controller  waiting for a volume to be created, either by external provisioner "nfs.csi.akshci.com" or manually created by system administrator
  Again validated the Windows Explorer on the Host is able to mount and able to add a file and view it on the NFS server.

As i further dug into the problem, i found the issue is with the SRV records. I tried the upstream CSI driver and still was running into the same issues...
So played with SRV records for the NFS - updated as follow for now.... (need ot learn more about NFS configurations) will work to make this secure...
/srv/nfs  *(rw,sync,no_root_squash,insecure)
Storage Class
apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
name: nfs-csi
provisioner: nfs.csi.akshci.com
parameters:
server: x.x.x.x # NFS server endpoint
share: /srv/nfs # NFS share path
reclaimPolicy: Retain
volumeBindingMode: Immediate
mountOptions:

nfsvers=4


Terraform: add stage params to Jenkins build job to bring into line with other Jenkins build jobs

Closing this issue because it was done last year.

Closing this issue because it was done last year.


Clarify that diagnostic settings retention only applies for Storage

The az monitor diagnostic-settings doc should clarify the retentionPolicy schema as it's now only shown in an example, and most importantly it should call out that retention policies can only be used when targeting a Storage account. That example which now shows retention policy furthermore has a --event-hub-rule {eventHubRuleID} parameter which doesn't make sense in this case as the diagnostics target is a Storage account, not Event Hub.

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

ID: b9ae3afa-74ff-12e8-a32c-c01240bd37da
Version Independent ID: 522b7e55-2510-8e51-021a-2fc134fabfbe
Content: az monitor diagnostic-settings
Content Source: latest/docs-ref-autogen/monitor/diagnostic-settings.yml
Service: monitoring-and-diagnostics
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

monitor

@yonzhan @kairu-ms I'm not sure what the process is for CLI repo contributions but is there any action required on me please? Can we look to incorporate this small change to remove confusion for customers?

Hi @jelledruyts, are you talking about this command? https://github.com/Azure/azure-cli/blob/086e8934ed2c18c65debfb8a867e05961bd3521a/src/azure-cli/azure/cli/command_modules/monitor/_help.py#L822-L876

Hi @kairu-ms, yes those commands include either --log or --metrics which contain diagnosticSettings but the schema isn't mentioned, only examples. I'd say the following actions would be good:

Under the Optional parameters header, the --logs and --metrics parameters should link to the schema definitions, respectively here and here.
That API doc has the RetentionPolicy schema defined, but it should mention that retention only applies when the target is a storage account. Perhaps that doc is owned by a different team, but if that's the case then I'm not sure how to approach them to ask for this clarification?
Ideally, the CLI doc we're talking about here would also note the fact that retention only applies when the target is a storage account to avoid customer confusion.
The CLI doc should remove the --event-hub-rule {eventHubRuleID} parameter from the example as it doesn't make sense.

Hope that helps!

Thanks @jelledruyts, we will fix it.


{% load static %} for django app

Hi,
I need to add this line {% load static %} for using jade in a django application. Unfortunately, compile hero throws an error "head unexpected text "{% lo"
Is there a way to have compile hero only copying a line without trying to compile it?

just comment it at the top
// {% load static %}
doctype html
html(lang='en')


Flush positions table. Purge Old positions Postgresql

Please help my harddrive is getting full 8 % per day.
I am trying to delete old records from database. PostgresSql, and I get this error.
Query
delete from positions where time <= (now() - '20 day'::INTERVAL);
error
ERROR: update or delete on table "positions" violates foreign key constraint "devices_fkey_position_id" on table "devices"
SQL state: 23503
Detail: Key (id)=(13) is still referenced from table "devices".

I find the similar error but I cannot make the query for posgresql.
#1122
UPDATE devices SET latestPosition_id = NULL WHERE latestPosition_id IN (SELECT id FROM positions WHERE time < (now() - '20 day'::INTERVAL);

Well I think I got it. Missed a ")".
UPDATE devices SET latestPosition_id = NULL WHERE latestPosition_id IN (SELECT id FROM positions WHERE time < (now() - '20 day'::INTERVAL));
delete from positions where time <= (now() - '20 day'::INTERVAL);
Should done the task.

This is what I usually do in MySQL database:
SET foreign_key_checks = 0;
DELETE FROM positions WHERE serverTime < DATE_ADD(NOW(), INTERVAL -1 MONTH);
UPDATE devices SET positionId = NULL WHERE positionId NOT IN (SELECT id FROM positions);
SET foreign_key_checks = 1;

There must be a way to do something similar in PostgreSQL.

@warlockopy, have you resolved the issue ? If yes please post your query for testing purpose.

IMHO, We should have the "data retention interval" in the configuration file which can be set from traccar admin UI.
Then based on database configured, traccar should select the appropriate query to delete the records.
@tananaev, does it make sense ?
Thanks.

@kirantpatil, maybe you should do some research before making comments? There is already such configuration parameter.

You mean, "database.historyDays" ? But does not delete or clear the old data.

That's exactly what it does.


Add new user

#872

@VincentHch - Since you might be unfamiliar with the process of adding a new team in our codebase, I have pushed a commit to address the same. You can get familiar with our codebase to solve more issues 😄 .


Replace header dropdown with buttons

Also decreased vertical divider indents to make it more visible and look better.

it seems kIsWeb is removed.
Does this change handle web?

it seems kIsWeb is removed.
Does this change handle web?

Just noticed that! I forgot about the web. I Will add it

@singerdmx  I think removing kIsWeb will not effect the outcome. Because I'm using index to get the values with separate Lists for Attributes and Strings.
So, these two piece of code will output the same thing wether kIsWeb is true or not.
final Map<Attribute, String> _valueToText = {
    Attribute.header: 'Normal text',
    Attribute.h1: 'Heading 1',
    Attribute.h2: 'Heading 2',
    Attribute.h3: 'Heading 3',
  };

child: Text(
            !kIsWeb
          ? _valueToText[value!]!
          : _valueToText[value!.key == 'header'
              ? Attribute.header
              : (value.key == 'h1')
                  ? Attribute.h1
                  : (value.key == 'h2')
                      ? Attribute.h2
                      : Attribute.h3]!,
         ),

AND
List<Attribute> _valueAttribute = [Attribute.header, Attribute.h1, Attribute.h2, Attribute.h3];
List<String> _valueString = ['N','H1','H2','H3'];

child: Text(_valueString[index]),


"undefined" When i press the medal button but the lang is brainfuck

This is a bug that occurs because the default language is brainfuck but isn't properly loaded yet.

Fixed.


Upgrade upload/download-artifact in sync across all org

Because upload-artifact and download-artifact are not compatible across v3-v4 major bump, we need to upgrade them across all repos at the same time, instead of when Renovate opens a PR in one repo.
Probably also need to cut new releases for that.

Opened https://github.com/slsa-framework/example-package/issues/320 to also make sure the test repo is in the known.

It seems this is also affecting OSS project that have updated the actions (via Dependabot, etc.) so now are using upload-artifact@v4 in the release workflow. This is a ticking timebomb in all OSS projects :(

@mihaimaruseac We discussed this problem a month ago, which is still open: https://github.com/slsa-framework/slsa-github-generator/issues/3068
Indeed many releases have failed after upgrading download-artifact to v4.

Thank you. Closing this in favor of that one.


[Live] refactor AddLiveAttributesSubscriber

Q
A

Bug fix?
no

New feature?
no

Tickets
n/a

License
MIT

Cleanup from #220, this subscriber should not depend on the twig runtime.

I had been noticing this too :). Thanks!


"Summaries: Should generate summary tree with isolated channels disabled" test failing in real service

The real service tests are failing - https://dev.azure.com/fluidframework/internal/_build?definitionId=56&_a=summary.
The logs in the pipeline run don't have information on which test is failing but on running locally, I see this failure consistently:
"Summaries: Should generate summary tree with isolated channels disabled" -  Error: Should not have attempted summary with disposed summarizer.

so when result = summarizer.summarizeOnDemand({ reason: "test" }); is called in the test at line 231, i set breakpoint and stepped in, it seems neither summarizer or running Summarizer was disposed here if (this._disposed || this.runningSummarizer?.disposed) from summarizer.summarizeOnDemand
is this relevant?
@pleath since you worked on this mainly

looks like a timing issue. the test probably needs to wait longer after the stop for the summarizer to actually dispose


"Module compiled with swift 4.0 cannot be imported in swift 3.1" error on build

Yesterday I tried Xcode9 beta. Now I want to go back using Xcode 8 but I faced this error on project build for DeviceKit and only for DeviceKit:
Module compiled with swift 4.0 cannot be imported in swift 3.1
The framework installation is made by Carthage. Building DeviceKit project on Xcode 8 works fine. But not on my project. I've tried a lot of things from deleting all Carthage and rebuild all, cleaning project, removing Derived Data... But the problem is still here. And only on DeviceKit.
So I wonder if there is some configuration in DeviceKit to be made to solve it.

I think that is because DeviceKit uses the new build system and does not use the normal DerivedData folder (at least on my machine, I'm not sure whether this is a local/user or shared setting ... You may wanna try deleting any "Derived Data" folder inside DeviceKit's root dir ... Does that help?

I found the problem: it was related to Carthage. See the answer on StackOverflow.


Collateral tab doesnt sort by column

The Chore
The collateral tab in Console currently doesnt sort based on columns.
You can click the column header and it acts as if it is trying to sort ... but doesnt. We need to fix this so we can sort on any column.
Ideally we should also have some sort of default sorting.
Tasks

[ ] Fix sorting on the collateral tab
[ ] Decide if we want a default sort and if so, what is it?
[ ] Apply the default sort if applicable

Additional details / background info

if we have there so-called "infinite scroll", the only one reasonable possibility of doing column sorting is to do it on server side. This was already reported to core team.
I can create an issue for core team and temporary hide controls for sorting on the table.

@macqbat I don't think we should use infinite scrolling here, the list of assets isn't likely to be long enough to require pagination (at least in the short/mid term), so I'd say the need for filtering and sorting is greater. Can we change it to not use the infinite row model to allow filtering and sorting, client side only

Default sorting of ag-grid is the first column ascending


fix(TypedTransaction::decode): do not panic on bad rlp input

Motivation
I encountered an edge case where TypedTransaction::decode panicked from a malformed transaction.

Solution
Explicitly check the leading byte of the tx for the transaction type.

PR Checklist

[x] Added Tests
[ ] Added Documentation
[ ] Breaking changes

Pushed a fix that should get CI to pass (or at least get farther along)

Looks like CI is good other then just a flaky example


bug(evm): de-synchronization between wasm bank balance and EVM account state

bank send funds to a wasm account

nibid tx bank send \
nibi1zaavvzxez0elundtn32qnk9lkm8kmcsz44g7xl \
nibi14hj2tavq8fpesdwxxcu44rty3hh90vhujrvcmstl4zr3txmfvw9ssa9gcs \
1000000unibi \
--gas auto \
--gas-adjustment 1.5 \
--gas-prices 0.025unibi \
--yes | jq

query bank balances

nibid q bank balances nibi14hj2tavq8fpesdwxxcu44rty3hh90vhujrvcmstl4zr3txmfvw9ssa9gcs | jq
{
  "balances": [
    {
      "denom": "unibi",
      "amount": "1000000"
    }
  ],
  "pagination": {
    "next_key": null,
    "total": "0"
  }
}

query evm account

nibid q evm account nibi14hj2tavq8fpesdwxxcu44rty3hh90vhujrvcmstl4zr3txmfvw9ssa9gcs | jq
{
  "balance": "0",
  "balance_wei": "0",
  "code_hash": "0xc5d2460186f7233c927e7db2dcc703c0e500b653ca82273b7bfad8045d85a470",
  "nonce": "0",
  "eth_address": "0x5A8D648DEE57b2fc90D98DC17fa887159b69638b",
  "bech32_address": "nibi14hj2tavq8fpesdwxxcu44rty3hh90vhujrvcmstl4zr3txmfvw9ssa9gcs"
}

The EVM account should have the same balance as the bank balance.

Hello @k-yang  A support ticket has been opened for you. Kindly Talk to Support on the Live support page so that your issue can be looked into. CLICK HERE

Note that this is not related to any difference of actual funds. The GRPC query is simply retrieving the balance incorrectly by using StateDB.GetAccount instead of the bank balance query.
Easy fix here: https://github.com/NibiruChain/nibiru/pull/2134


Disable TX completely feauture

TX function is illegal in depending on the country,
I would like to have the ability to completely disable the TX function in all band.

There is a TX enable/disable option in the normally hidden menu, you can also set some country/TX setting there too, although there are limited options.
To access the hidden menu items to change the TX options and if you want to completely disable the TX ..

Turn the radio off
Press and hold down the PTT and the upper side button at the same time
Turn the radio back on whilst holding those two buttons down
You should now have the extra/hidden menu settings to play with

Wonder what country you're in ?  I could add an option to match your part of the world to prevent the radio TX'ing where it's not allowed. The radio being able to TX where it shouldn't can get people into serious trouble and/or taken in for questioning in some parts of the world.

There is a TX enable/disable option in the normally hidden menu, you can also set some country/TX setting there too, although there are limited options.
To access the hidden menu items to change the TX options and if you want to completely disable the TX ..

Turn the radio off
Press and hold down the PTT and the upper side button at the same time
Turn the radio back on whilst holding those two buttons down
You should now have the extra/hidden menu settings to play with

Wonder what country you're in ? I could add an option to match your part of the world to prevent the radio TX'ing where it's not allowed. The radio being able to TX where it shouldn't can get people into serious trouble and/or taken in for questioning in some parts of the world.

Probably means without a licence which is common throught the world....   So a generic disable everywhere could be useful for those without a licence etc

Wrong thinking. Nobody will give you a fine for TX feature in a radio.

Wrong thinking. Nobody will give you a fine for possessing TX feature in a radio. Without a licence you always transmit on your risk.

yes they will and DO in the UK and whats more you can even go to prison! Same with many countries if caught!
In the UK, it is illegal to use a two-way radio without an appropriate license. The penalties for doing so can vary depending on the circumstances of the offense, but can include:
Fines: If caught using a two-way radio without a license, you may be fined up to £5,000 or face imprisonment for up to 6 months.
Confiscation of equipment: Your two-way radio may be confiscated by the authorities if used without a license.
Prosecution: If found guilty of using a two-way radio without a license, you may face prosecution which can result in a criminal record.
Interference with emergency services: The use of unlicensed two-way radios can interfere with emergency services, such as the police or ambulance service. If this interference results in harm to a person or property, the offender may face a more severe penalty.
It is important to note that there are some exceptions to the license requirement, such as the use of certain types of radios in specific circumstances, or for use by licensed operators from other countries who are visiting the UK.
To avoid penalties and ensure compliance with the law, anyone who wishes to use a two-way radio in the UK should obtain the appropriate license and follow the rules and regulations set out by Ofcom, the UK communications regulator.

Thank you for reply.
I'm live in Japan,
In my country illegal for possession of unlicensed/uncertified transceiver with ready to tx'ing. (on any freqencies)
It's very complicated to get permission, so I modified to make it completely receive-only.

Thank you for reply. I'm live in Japan, In my country illegal for possession of unlicensed/uncertified transceiver with ready to tx'ing. (on any freqencies) It's very complicated to get permission, so I modify to make it completely receive-only.

That sucks buddy... lets hope 1of11 or someone else can help.... in the mean time there is away to do it on stock fws
https://whosmatt.github.io/uvmod/?hidden
https://uvmod.valek.net.ru/?hidden       (google will translate and you have al fws too)
downside is poor AM  but at least its a start for you

This firmware lets you completely disable the TX if you want, the hidden menu item "TX EN" lets you do it.

This firmware lets you completely disable the TX if you want, the hidden menu item "TX EN" lets you do it.

Shows you how little I explorer the menus properly...lol
Great news for the op!

Even in the USA where they maintain the illusion about how free they say they are they have rules about what to do with you if caught.


Change age categories for Malaria report

Add 0-11mois / 12-59mois instead of <5ans and add the following to the table:
Nombre de RDT négatif
Nombre de RDT positif
Nombre de RDT non fait

The changes have now been made, but there are a few remaining issues:

The RDT numbers do not make sense to me. Are they correct?
RDT numbers are in the table twice. Is this as intended?

A screenshot:

Hi Jyri - number of RDT positif (positive) equals the number of palu simply, grave (serious) and inconvnu (unknown) which is good.
The number RDT negative (negative) and non fait (Not done) should equal RDT effectues (Done) (which is 2,557) minus RDT positive (512) however. Clearly this is not right as they are both 0.
Otherwise all looks ok.

Thanks for the comments, I was suspecting something is fishy with the numbers. I'll investigate why they look like they do.

It appears that the numbers that are circled red in the attached screen shot are always zero. There are no cases where a confirmed malaria case would have an RDT test negative or Not done. Could these columns be removed?

According to discussion with @johnhaskew, the RDT columns "Negative" and "Not done" have been removed and the rows "Number of negative RDT tests" and "Number of RDT tests not done" are calculated using data from rows "Number of fever cases", "Number of RDT tests performed" and "Number of positive RDT tests".
This is how the report looks at the moment:

This is great Jyri thank you


Please Add the Orange Justice as an Emote

Orange Justice would be great to pair with the other emotes
This mod is the most clean and simple compared to the TooManyEmotes Mod which is super buggy

Orange Justice would be great to pair with the other emotes
This mod is the most clean and simple compared to the TooManyEmotes Mod which is super buggy

I am definitely looking to add emotes but still learning how it all works. This will most likely be a while away unless there is anyone who is familiar with unity development to help, sorry :/

https://thunderstore.io/c/lethal-company/p/FlipMods/TooManyEmotes/


Can you update CSync to 5.0.0

I really like this module, but the v50 doesn't work

Hi, this is dust, a friend of InnohVateur. He dosen't want to work on the mod anymore, at least not for a while. However, I'm going to work on it on his behalf. If I dont run into too many trouble, you can except a working version out by the end of the week. I'l keep you posted. And please don't hesistate to suggest anything we could add to upgrade the moon !

Hi, this is dust, a friend of InnohVateur. He dosen't want to work on the mod anymore, at least not for a while. However, I'm going to work on it on his behalf. If I dont run into too many trouble, you can except a working version out by the end of the week. I'l keep you posted. And please don't hesistate to suggest anything we could add to upgrade the moon !

Using csync3.1.1 in the v50 version resulted in a low frame count for the host
thx bro

Noted, I'l see what I can do.
I have a question : Did someone else try to host and ran into the same fps drop issue ?

Noted, I'l see what I can do.
I have a question : Did someone else try to host and ran into the same fps drop issue ?

yes,It is normal to remove this module, sometimes this module does not have the normal random resulting low frame count, and now I only need a version that can be used in v50, I will test it

I'm not sure that I understood your comment, sorry.

Update : I have a working v50 version of the mod, without lethal config nor Csync

The update is now on thunderstore. There may be bugs, but I think it should work fine in V50.
But, right now the only way to change the config is via R2Modman GUI. Sorry for the inconveniance, I will fix that ASAP

The update is now on thunderstore. There may be bugs, but I think it should work fine in V50. But, right now the only way to change the config is via R2Modman GUI. Sorry for the inconveniance, I will fix that ASAP

Thx bro for your work

You'r welcome ! Enjoy you time with the mod, and don't hesistate to report any bug you may encounter while playing !
And as always, if you have idea for improvement, feel free to share them as well !

Update : Lethal Config and Lethal Settings are working again, however I can't get Csync to well sync the config just yet for some reason. I will add all of the settings and see if it's works, and either way I will make a release out of that !

Update : With the help of @Owen3H , the creator of Csync, We were able (Well he was able to be honest) to fix the issue. Now I need to add all of the config that I want to be fixed and to test all of em. This will take a day or two, you can except 1.3.1 to release on thunderstore around wensday.


Fix #6288

Summary
Expands the scope of this topic from "constructing the complete tree behind a query using factory methods" to "dynamic querying based on runtime state".
I've tried to illustrate why you'd want to use local variables, when you'd want to call additional LINQ methods, when you'd want to vary the expression passed into those same methods; and when there's no recourse but to construct the expression tree by hand, either to pass into the LINQ methods, or to include the LINQ method call on the tree.
Ping @aaron-old @DeveloperLookBook @Miggleness @psxvoid -- as upvoters on the original issue, I'd be very interested in feedback from you.
Fixes #6288.
I also think some mention should be made of Dynamic LINQ, at least in it's current incarnation, and how much easier it is to construct expression trees by building strings vs calling the factory methods.

@BillWagner

There's a lot of great work here @zspitz I really like the information you've added.

So do I 😁. How is the structure of the article? I aimed to go from simpler technique to more complex ones.

In addition, the sample code is great. We've been adding samples to projects, and including them using the ::: extension. That helps us ensure that any changes don't break any code. Can you include the samples as a project that builds for us?

Where can I find guidance on how to do this?
Also, is there a standard source of sample data I can use?
Since this article talks about different queries based on runtime data, should I be prompting for data using Console.ReadLine? e.g. for the length variable:
Console.WriteLine("What length do you want to group by:");
Console.Write("?");
var sLength = Console.ReadLine();
var length = int.Parse(sLength);

Or something more elaborate, such as a textual UI using Gui.cs or similar?

@zspitz

How is the structure of the article?

I like it. You've done a great job of going from the simplest case to more complicated scenarios. That's great.

Where can I find guidance on how to do this (include code in docs)?

The syntax for the extension is here. Look at the second example. One difference is that instead of line numbers for the last parameter, we use snippet IDs.  See this example with the code here.

Also, is there a standard source of sample data I can use?

Not really. (It's something we're considering for the future, but nothing is ready now.)  What you've done is fine. I like that it is easy for the reader to understand.

@BillWagner

How do I use the ::: extension on code which doesn't actually compile?
bool sortByLength = /* ... */;

var qry = companyNamesSource;
if (sortByLength) {
    qry = qry.OrderBy(x => x.Length);
}

I think it's important to present the code this way (bool sortByLength = /* ... */;) in this and similar examples, to emphasize that we don't have the value of sortByLength at compile time, and at runtime it might go either way.

Once I'm finished with this article and written its' samples, I'd like to similarly rewrite the corresponding VB article.

@zspitz

How should I use the ::: extension on code which doesn't actually compile?

Don't. Code that doesn't compile should be shown inline. We haven't designed a good way to validate that code doesn't compile in a prescribed way. It's in the backlog, but a number of tasks are in front of it.

Once I'm finished with this article and written its' samples, I'd like to similarly rewrite the corresponding VB article.

That would be fantastic. Let's make that a separate PR.

I've added a section with specific information about ...

I gave it a quick look. I like what you've done.
Let me know when you're ready for a fresh review.

@BillWagner
I've specified comments that indicate variables which are initialized outside of the snippet:
// bool sortByLength = /* ... */;

Let's make that a separate PR.

The text of the article will be the same, aside from the code snippets and generics usage (Expression(Of TDelegate) instead of Expression<TDelegate>). Is it still worth a separate PR?
Other than that, I am ready for a fresh review.

should the filename be changed as well?

No. We'd have to add redirects, and it's not a huge value add.
I'll review now.

@BillWagner Accepted.


fastlane upgrade

fastlane 2.225.0 is available. You are on 2.223.1.
767
You should use the latest version.
768
Please update using bundle update fastlane.
not sure why this is happening and confused. Any help would be greatly appreciated

I have some problem.


Are the packet buffers big enough?

in Config there are the functions incoming_packet_capacity and outgoing_packet_capacity. They control the maximum size of the buffers used to hold packets to prevent unbounded memory usage. They reach their maximum when the server is sending packets faster than the client can receive it or vice versa.
Are the default values large enough? I recall stackotter saying he was kicked from terrain.rs with a render distance of 32 because of this. Is this even the right approach to the problem?

Just playing around with the examples. Can confirm that a render distance of 32 got me kicked.

2000 cows on the cow sphere example gets you kicked


Editable State of Children in External Component

Thank you @prevwong for your pull request. It indeed resolves the initial issue and now the children are being passed correctly. However, a new side effect has emerged that we need to address.
Given that 'UserSwiper2' is an external component, we can't edit it directly from the interface. But its child component is editable, implying that we should be able to edit it. Isn't this correct?
If I navigate to the component from the sidebar, I can edit the child, but the parent (which is non-editable) dominates, and we lose the selection of the child. This leads to a subpar user experience compared to a typical Reka component.
Can anyone suggest a workaround for this issue, so that we can achieve a seamless experience similar to a Reka component?
Thanks again for your help and looking forward to your suggestions.
Originally posted by @Sofianebenabdallah in https://github.com/prevwong/reka.js/issues/32#issuecomment-1547675299

Let's keep the discussion on the original thread.


can I move the window with lua

What is your question?
can I move the window with lua

setPropertyFromClass('lime.app.Application', 'current.window.x')
I would very much recommend getting the position before you move it and basically offsetting it


feat: Add namespace override settings in Helm Chart

PR description
This PR includes changes to allow the descheduler Helm chart to be deployed to a specified namespace.
Usage
You can deploy to a specific namespace by setting namespaceOverride in the values.yaml file as follows:
namespaceOverride: your-custom-namespace

/ok-to-test

/assign @jklaw90
seems fine to me, any thoughts?

/lgtm


Support for .netstandard2.0

I followed recommendations from Microsoft https://docs.microsoft.com/en-us/dotnet/core/porting.
This includes:

Migrating nuget references from using packages.config file to embedded references in .csproj files
Replaced incompatible LZ4PCL library with LZ4
Upgraded .net framework to version 4.7.2
Upgraded csproj files to compile using .netstandrard2.0

Fixes #14 and allows to fix #12 in future commits by adding more platforms.
Any comments/feedback are welcome. Thanks.

You might want to consider compiling against just netstandard2.0 instead of netcoreapp2.2;net472
netstandard2.0 is compatible with both.
https://devblogs.microsoft.com/dotnet/announcing-net-standard-2-0/

That's cool, I wasn't aware of that. Updated to use .net standard 2.0, but the CI is now failing...

Right it looks like CI server now needs another line (perhaps instead of nuget restore)
dotnet restore

based on https://stackoverflow.com/a/48440777/706456
Can someone help me with the CI server? I don't think I have access to it

yes, the build server should compile this using dotnet build (which in turn triggers dotnet restore) instead of using msbuild directly to compile this solution. However the build script is not part of the repository at this point, so only @poulfoged has access as far as I can see.
Adding a build script might be useful here. I usually write mine in Cake (which is C# as well), but that's a preference.

Looks great. I'll move AppVayor config to the repo.


Router Link

Codecov Report

Merging #41 into master will increase coverage by 0.54%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #41      +/-   ##
==========================================
+ Coverage   30.93%   31.48%   +0.54%
==========================================
  Files          28       29       +1
  Lines         375      378       +3
  Branches       43       43
==========================================
+ Hits          116      119       +3
  Misses        237      237
  Partials       22       22

Impacted Files
Coverage Δ

src/Main.js
100.00% <ø> (ø)

src/components/FeaturedPost.js
100.00% <ø> (ø)

src/components/Footer.js
100.00% <ø> (ø)

src/components/Header.js
95.23% <ø> (ø)

src/components/HomeHeroSection.js
100.00% <ø> (ø)

src/pages/SpeakerDetail.js
12.00% <ø> (ø)

src/components/ScrollToTop.js
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


[Feat]: Workflow for closing old PRs

This feature aims to automate the management of open PRs in a repository, ensuring that outdated or abandoned PRs are closed in a timely manner. By doing so, it helps maintain repository hygiene, improves workflow efficiency, and provides a better experience for both maintainers and contributors.

@abhisheks008 please assign me this issue

Hi @Asymtode712 in this repo, I used to review each and every PR. After reviewing them, if I found them duplicate or not so worth it, I close them. There is no need to add a bot for this problem. I am happy to review the PRs.


[macOS] Get the NSView for a window with NSWindow::contentView

Instead of holding a reference to the original NSView in Window, only hold the NSWindow and get its view by calling NSWindow::contentView. This allows the user to set a new contentView without breaking winit's resize handling. My specific use case is to replace the contentView with an MTKView allowing me to render with Metal.

I'm generally in favour of this but have some concerns regarding the lack of checks for null pointers.

Having more debug_asserts is always good.

@lloydac do you plan to continue working on this PR?

Wouldn't it make more sense to just make it possible to replace the view via winit's API?
(That said, once #518 merges, replacing the view would completely break keyboard handling.)


KIC: Warn that deploy manifests don't install the admission webhook

Where is the problem?
https://docs.konghq.com/kubernetes-ingress-controller/latest/deployment/admission-webhook/#verify-incorrect-credential-secrets
What happened?
@rkazak ran into an issue with these docs:
The Verify incorrect credential secrets section said that KIC would reject an invalid secret while, when tried, it did not (because KIC was installed from deploy manifests).
What did you expect to happen?
The docs saying that KIC will reject an invalid secret only if installed with the admission webhook (installed by default by the Helm chart but not by deploy manifests).
The scope of this issue is to put the warning in at least the below places:

Quick Start Guide
The Verify incorrect credential secrets section mentioned above (currently unclear to me which guide that is)
Other places that mention webhook rejection/validation

Code of Conduct and Community Expectations

[X] I agree to follow this project's Code of Conduct
[X] I agree to abide by the Community Expectations

Hi @mflendrich.  This has now been addressed in the documentation: https://docs.konghq.com/kubernetes-ingress-controller/latest/deployment/admission-webhook/. The related PR is https://github.com/Kong/docs.konghq.com/pull/6342. The instructions were tested and validated.
Last login: Thu Oct 19 09:57:19 on ttys001
rajakavitha@Rajakavithas-MacBook-Air ~ % orb start k8s
rajakavitha@Rajakavithas-MacBook-Air ~ % kubectl get nodes
NAME       STATUS   ROLES                  AGE   VERSION
orbstack   Ready    control-plane,master   13h   v1.27.4+orb1
rajakavitha@Rajakavithas-MacBook-Air ~ % kubectl apply -f https://raw.githubusercontent.com/Kong/kubernetes-ingress-controller/v2.12.x/deploy/single/all-in-one-dbless.yaml

error: unable to read URL "https://raw.githubusercontent.com/Kong/kubernetes-ingress-controller/v2.12.x/deploy/single/all-in-one-dbless.yaml", server reported 404 Not Found, status code=404
rajakavitha@Rajakavithas-MacBook-Air ~ % kubectl apply -f https://raw.githubusercontent.com/Kong/kubernetes-ingress-controller/main/deploy/single/all-in-one-dbless.yaml

Warning: resource namespaces/kong is missing the kubectl.kubernetes.io/last-applied-configuration annotation which is required by kubectl apply. kubectl apply should only be used on resources created declaratively by either kubectl create --save-config or kubectl apply. The missing annotation will be patched automatically.
namespace/kong configured
Warning: resource customresourcedefinitions/ingressclassparameterses.configuration.konghq.com is missing the kubectl.kubernetes.io/last-applied-configuration annotation which is required by kubectl apply. kubectl apply should only be used on resources created declaratively by either kubectl create --save-config or kubectl apply. The missing annotation will be patched automatically.
customresourcedefinition.apiextensions.k8s.io/ingressclassparameterses.configuration.konghq.com configured
Warning: resource customresourcedefinitions/kongclusterplugins.configuration.konghq.com is missing the kubectl.kubernetes.io/last-applied-configuration annotation which is required by kubectl apply. kubectl apply should only be used on resources created declaratively by either kubectl create --save-config or kubectl apply. The missing annotation will be patched automatically.
customresourcedefinition.apiextensions.k8s.io/kongclusterplugins.configuration.konghq.com configured
Warning: resource customresourcedefinitions/kongconsumergroups.configuration.konghq.com is missing the kubectl.kubernetes.io/last-applied-configuration annotation which is required by kubectl apply. kubectl apply should only be used on resources created declaratively by either kubectl create --save-config or kubectl apply. The missing annotation will be patched automatically.
customresourcedefinition.apiextensions.k8s.io/kongconsumergroups.configuration.konghq.com configured
Warning: resource customresourcedefinitions/kongconsumers.configuration.konghq.com is missing the kubectl.kubernetes.io/last-applied-configuration annotation which is required by kubectl apply. kubectl apply should only be used on resources created declaratively by either kubectl create --save-config or kubectl apply. The missing annotation will be patched automatically.
customresourcedefinition.apiextensions.k8s.io/kongconsumers.configuration.konghq.com configured
Warning: resource customresourcedefinitions/kongingresses.configuration.konghq.com is missing the kubectl.kubernetes.io/last-applied-configuration annotation which is required by kubectl apply. kubectl apply should only be used on resources created declaratively by either kubectl create --save-config or kubectl apply. The missing annotation will be patched automatically.
customresourcedefinition.apiextensions.k8s.io/kongingresses.configuration.konghq.com configured
Warning: resource customresourcedefinitions/kongplugins.configuration.konghq.com is missing the kubectl.kubernetes.io/last-applied-configuration annotation which is required by kubectl apply. kubectl apply should only be used on resources created declaratively by either kubectl create --save-config or kubectl apply. The missing annotation will be patched automatically.
customresourcedefinition.apiextensions.k8s.io/kongplugins.configuration.konghq.com configured
Warning: resource customresourcedefinitions/tcpingresses.configuration.konghq.com is missing the kubectl.kubernetes.io/last-applied-configuration annotation which is required by kubectl apply. kubectl apply should only be used on resources created declaratively by either kubectl create --save-config or kubectl apply. The missing annotation will be patched automatically.
customresourcedefinition.apiextensions.k8s.io/tcpingresses.configuration.konghq.com configured
Warning: resource customresourcedefinitions/udpingresses.configuration.konghq.com is missing the kubectl.kubernetes.io/last-applied-configuration annotation which is required by kubectl apply. kubectl apply should only be used on resources created declaratively by either kubectl create --save-config or kubectl apply. The missing annotation will be patched automatically.
customresourcedefinition.apiextensions.k8s.io/udpingresses.configuration.konghq.com configured
serviceaccount/kong-serviceaccount created
role.rbac.authorization.k8s.io/kong-leader-election created
clusterrole.rbac.authorization.k8s.io/kong-ingress created
clusterrole.rbac.authorization.k8s.io/kong-ingress-crds created
clusterrole.rbac.authorization.k8s.io/kong-ingress-gateway created
rolebinding.rbac.authorization.k8s.io/kong-leader-election created
clusterrolebinding.rbac.authorization.k8s.io/kong-ingress created
clusterrolebinding.rbac.authorization.k8s.io/kong-ingress-crds created
clusterrolebinding.rbac.authorization.k8s.io/kong-ingress-gateway created
service/kong-admin created
service/kong-proxy created
service/kong-validation-webhook created
deployment.apps/ingress-kong created
deployment.apps/proxy-kong created
Warning: resource ingressclasses/kong is missing the kubectl.kubernetes.io/last-applied-configuration annotation which is required by kubectl apply. kubectl apply should only be used on resources created declaratively by either kubectl create --save-config or kubectl apply. The missing annotation will be patched automatically.
ingressclass.networking.k8s.io/kong configured
rajakavitha@Rajakavithas-MacBook-Air ~ % curl -sL https://raw.githubusercontent.com/Kong/kubernetes-ingress-controller/main/hack/deploy-admission-controller.sh | bash

.....+++++++++++++++++++++++++++++++++++++++*........+.......+..............+....+.........+.....+.+..+...+....+++++++++++++++++++++++++++++++++++++++*.........+..........+........+...+....+.....+......+.........+......+....+........+.+.....+....+...............+...+...++++++
.....+........+...+...+...............+++++++++++++++++++++++++++++++++++++++*.+.............+++++++++++++++++++++++++++++++++++++++*.....+..............+...+.......+...+............+...+..+.............+......+..+.........+...+.......+...+........+...+.+...+..................+........+.........+...+.........+....+..+......................+..+...+.++++++
-----
secret/kong-validation-webhook created
deployment.apps/ingress-kong patched
validatingwebhookconfiguration.admissionregistration.k8s.io/kong-validations created
rajakavitha@Rajakavithas-MacBook-Air ~ % kubectl create secret tls kong-validation-webhook -n kong \
    --key tls.key --cert tls.crt

error: failed to create secret secrets "kong-validation-webhook" already exists
rajakavitha@Rajakavithas-MacBook-Air ~ % kubectl get deployments -A
NAMESPACE     NAME                     READY   UP-TO-DATE   AVAILABLE   AGE
kong          kong-gateway             1/1     1            1           13h
kube-system   coredns                  1/1     1            1           13h
kube-system   local-path-provisioner   1/1     1            1           13h
kong          kong-controller          1/1     1            1           13h
kong          proxy-kong               2/2     2            2           77s
kong          ingress-kong             1/1     1            1           77s
rajakavitha@Rajakavithas-MacBook-Air ~ % kubectl patch deploy -n kong ingress-kong \
    -p '{"spec":{"template":{"spec":{"containers":[{"name":"ingress-controller","env":[{"name":"CONTROLLER_ADMISSION_WEBHOOK_LISTEN","value":":8080"}],"volumeMounts":[{"name":"validation-webhook","mountPath":"/admission-webhook"}]}],"volumes":[{"secret":{"secretName":"kong-validation-webhook"},"name":"validation-webhook"}]}}}}'

deployment.apps/ingress-kong patched (no change)
rajakavitha@Rajakavithas-MacBook-Air ~ % >....
    - configuration.konghq.com
    apiVersions:
    - '*'
    operations:
    - CREATE
    - UPDATE
    resources:
    - kongconsumers
    - kongplugins
  - apiGroups:
    - ''
    apiVersions:
    - 'v1'
    operations:
    - UPDATE
    resources:
    - secrets
  clientConfig:
    service:
      namespace: kong
      name: kong-validation-webhook
    caBundle: $(cat tls.crt  | base64) " | kubectl apply -f -

validatingwebhookconfiguration.admissionregistration.k8s.io/kong-validations configured
rajakavitha@Rajakavithas-MacBook-Air ~ % echo "apiVersion: configuration.konghq.com/v1
kind: KongConsumer
metadata:
  name: harry
  annotations:
    kubernetes.io/ingress.class: kong
username: harry" | kubectl apply -f -

kongconsumer.configuration.konghq.com/harry created
rajakavitha@Rajakavithas-MacBook-Air ~ % echo "apiVersion: configuration.konghq.com/v1
kind: KongConsumer
metadata:
  name: harry2
  annotations:
    kubernetes.io/ingress.class: kong
username: harry" | kubectl apply -f -

Error from server: error when creating "STDIN": admission webhook "validations.kong.konghq.com" denied the request: consumer already exists
rajakavitha@Rajakavithas-MacBook-Air ~ % echo "
apiVersion: configuration.konghq.com/v1
kind: KongPlugin
metadata:
  name: request-id
config:
  foo: bar
  header_name: my-request-id
plugin: correlation-id
" | kubectl apply -f -

Error from server: error when creating "STDIN": admission webhook "validations.kong.konghq.com" denied the request: plugin failed schema validation: schema violation (config.foo: unknown field)
rajakavitha@Rajakavithas-MacBook-Air ~ % kubectl create secret generic some-credential \
  --from-literal=kongCredType=basic-auth \
  --from-literal=username=foo

secret/some-credential created
rajakavitha@Rajakavithas-MacBook-Air ~ % kubectl create secret generic some-credential \
  --from-literal=kongCredType=wrong-auth \
  --from-literal=sdfkey=my-sooper-secret-key

error: failed to create secret secrets "some-credential" already exists
rajakavitha@Rajakavithas-MacBook-Air ~ %

Closing this issue for now. Please feel to reopen if any other issue needs to be addressed. Related PR: https://github.com/Kong/docs.konghq.com/pull/6342


fix(exceptions): log certain APIExceptions tracebacks

[Current coverage][cc-pull] is 86.53%

Merging [#824][cc-pull] into [master][cc-base-branch] will increase coverage by <.01%

@@             master       #824   diff @@
==========================================
  Files            30         30
  Lines          2702       2704     +2
  Methods           0          0
  Messages          0          0
  Branches        438        439     +1
==========================================
+ Hits           2338       2340     +2
  Misses          242        242
  Partials        122        122

Powered by Codecov. Last updated by [91f94f3...3f980b8][cc-compare]
[cc-base-branch]: https://codecov.io/gh/deis/controller/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/deis/controller/compare/91f94f391bf6c682b3360b8ed7d54040bececfb3...3f980b89740f5b5f728b37c010e95f69193b9613
[cc-pull]: https://codecov.io/gh/deis/controller/pull/824?src=pr


how to use demo.py

Excuse,me. I ran demo.py.
The captions are very strange.
However,the detection part performs well.
Besides, the objects detected in the photo have no relationship with the corresponding caption. The objects don't appera in the caption.
Why the demo.py generates captions like this?
ID:184386 sentence: wipe [ bulldog ] doing wipe afraid form payer wipe afraid form
ID:231037 sentence: wipe [ ship ] undergoing wipe [ baseball bat ] outdoor heating rubs wipe animal
ID:467437 sentence: wipe [ ship ] serene outdoor wipe [ bulldog ] doing wipe [ ship ]
ID:258588 sentence: wipe [ ship ] corsage wipe afraid reflected payer cub housed stitch kids
ID:460251 sentence: wipe [ clock ] converses skating outdoor heating rubs wipe moped tried
ID:357978 sentence: wipe hull rubs [ bagel ] preparation carved wipe tried
ID:571641 sentence: wipe alive receiver doing wipe alive payer wipe [ pontoon ]
ID:163348 sentence: wipe [ schnauzer ] payer wipe [ pontoon ] outdoor wipe receiver
ID:298978 sentence: wipe [ jetliner ] skating corsage wipe thermos doing wipe [ macbook ]
ID:258628 sentence: wipe [ ship ] horns stitch kids outdoor wipe western
ID:118432 sentence: wipe hull rubs [ bagel ] preparation outdoor heating rubs wipe bins bubbles [ cockatoo ]
ID:205230 sentence: wipe hull rubs [ kids ] skating carved wipe [ cupcake ]
ID:345136 sentence: wipe civilians [ walker ] skating corsage wipe bins doing wipe prints payer afraid [ cockatoo ]
ID:404613 sentence: wipe [ kid ] horns sprout wipe [ foal ] corsage wipe bins
ID:131089 sentence: wipe chalk [ walker ] horns stitch doing wipe ceramics
ID:432570 sentence: wipe [ biker ] doing who corsage reception skating outdoor wipe tried
ID:214224 sentence: wipe [ jetliner ] horns preparation session wipe southeast doing wipe envelopes rubs [ skateboarded ]
ID:199764 sentence: wipe hull rubs [ bagel ] preparation carved wipe tried doing wipe rugs
ID:176312 sentence: wipe [ jetliner ] preparation corsage dandelion rubs wipe served sprout wipe [ firetruck ]
ID:453297 sentence: wipe [ firetruck ] horns skating outdoor wipe tried bathtub useful wipe terminal
ID:529668 sentence: wipe [ jetliner ] sprout wipe [ firetruck ] corsage dandelion rubs wipe rugs
ID:551107 sentence: wipe civilians [ traffic light ] preparation corsage dandelion rubs wipe [ macbook ]
ID:413120 sentence: wipe inspired [ biker ] fun doing prints removing uneaten
ID:369771 sentence: wipe straining rubs crotch doing wipe [ ski ] rubs crotch payer wipe [ ski ] rubs standing

I am sorry for late reply, have not noticed so far :( Did you solve the problem, I am running normally and I think may you did not run preprocessing step (or you just need to download the preprocessed data and put into data/coco/)

Download them from this: https://www.dropbox.com/s/1t9nrbevzqn93to/coco.tar.gz?dl=0

What I am getting now

When running evaluation

I ran into the same problem, but not the reason you mentioned above.

Here I see that the textual caption and visual caption are both wrong? If you just run demo.py without any modification, I dont have any clues why you get this..

The image can be detected normally but the title generation is not the detected one.for example
eg:a [ backpack ] on a [ cheesecake ] on a street。
But the picture does not detect the two words， [ backpack ] ， [ cheesecake ]，But it’s detected as a person。

Yeah, correct! I am re-running and see that textual caption is correct but visual caption is wrong, I put another here but I dont expect the answer from the author tbh, since he has not reply any issues for long
https://github.com/jiasenlu/NeuralBabyTalk/issues/43

I am trying to figure out, If any progress, I will let you know

thank you very much

@wlybug  hi, I encountered the same issue that correct sentence template but incorrect visual words by detector. So how can I fix it?

Hey @luulinh90s  !
Have you been able to run the demo on a custom image? Kindly let me know if you have.

I think if we use the custom image, the object in the image and the object proposals provided by the author are not match, then the result will be incorrect. Anyway, I still can't, you should contact the author for the answer tbh :(

Hi, can anyone help me to run the demo please?

I didn't understand what should I enter to continue

@luulinh90s Can you guide please? what  I'm doing wrong?

@RamatovInomjon you just need to press c then enter to continue the program

@luulinh90s thanks for your quick reply, it is started downloading .vector_cache/glove.6B.zip file,  I open the code but didn't get what this file for? thanks for your time again

This file is for word embedding, its nice to use this or you can choose what embedding tool you want rather than GloVE

@luulinh90s thanks for support


expose redis to outside with istio sidecar?

Hello, i want to expose redis to outside with istio, The server redis is well exposed via the Service
However, il does not work to expose the redis to outside with istio
export INGRESS_HOST=$(kubectl -n istio-system get service istio-ingressgateway -o jsonpath='{.status.loadBalancer.ingress[0].ip}')
redis-cli -h $INGRESS_HOST -p 11337
Here is the code below, Do you have any ideals pls
Thans
kind: Deployment
metadata:
  namespace: redis
  name: azure-vote-back
spec:
  replicas: 1
  selector:
    matchLabels:
      app: azure-vote-back
  template:
    metadata:
      labels:
        app: azure-vote-back
    spec:
      nodeSelector:
        "beta.kubernetes.io/os": linux
      containers:
      - name: azure-vote-back
        image: mcr.microsoft.com/oss/bitnami/redis:6.0.8
        env:
        - name: ALLOW_EMPTY_PASSWORD
          value: "yes"
        resources:
          requests:
            cpu: 100m
            memory: 128Mi
          limits:
            cpu: 250m
            memory: 256Mi
        ports:
        - containerPort: 6379
          protocol: TCP
          name: redis-port
---
apiVersion: v1
kind: Service
metadata:
  namespace: redis
  name: azure-vote-back
spec:
  ports:
  - name: redis-port
    port: 6379
    protocol: TCP
  selector:
    app: azure-vote-back
---
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: azure-vote-back
  namespace: redis
spec:
  hosts:
  - "*"
  gateways:
  - redis-gateway
  tcp:
  - route:
    - destination:
        host: azure-vote-back.redis.svc.cluster.local
        port:
          number: 6379
---
# voting-app-gateway.yaml
apiVersion: networking.istio.io/v1alpha3
kind: Gateway
metadata:
  name: redis-gateway
  namespace: redis
spec:
  selector:
    istio: ingressgateway
  servers:
  - port:
      number: 11337
      name: redis-port
      protocol: TCP
    hosts:
    - "*"

your service istio-ingressgateway should also have a port 11337 exposed, if not, kube-proxy will not proxy the traffic to port 11337 of istio-ingressgateway.