text:
passing information from child class to parent, using dynamic classes in kvlang (Kivy)

For more details, please look in the stackoverflow question:
http://stackoverflow.com/questions/28684165/passing-information-from-child-class-to-parent-using-dynamic-classes-in-kvlang

I don't think this is a bug. I see you've already managed to fix your original issue, and passing the label object from the child button to the parent tree does not make much sense given that the label id is already in the parent, ready to reference.
As a sidenote, please describe you issue here on github in the future and provide the relevant code in this place.

Got it. I'm just used to stackoverflow so I thought it would be the
platform for such questions.
But I might have not explained the problem clearly enough, as the problem
remains:
What I want to do (which I thought would be straightforward) is to create a
class (Button1) that holds an ObjectProperty. And I want that this property
will be initialized by instances of Button1, and this doesn't seem to work.
Obviously, the label/button example was just to simplify things, I'm aiming
at a more elaborate case with many such object properties. I just wanted to
see how to make the simplest case work.
What puzzles me is that if I use a StringProperty instead of an
ObjectProperty, I can have the instance of Button1 to use it.
To summarize, I'm trying to find the kv way of passing arguments to the
init function of the parent class.
Can you help me do that?
On Tue, Mar 24, 2015 at 12:27 PM, dessant notifications@github.com wrote:

I don't think this is a bug. I see you've already managed to fix your
original issue, and passing the label object from the child button to the
parent tree does not make much sense given that the label id is already in
the parent, ready to reference.
As a sidenote, please describe you issue here on github in the future and
provide the relevant code in this place.
—
Reply to this email directly or view it on GitHub
https://github.com/kivy/kivy/issues/3186#issuecomment-85456261.

authors: dessant yoelk

repo: kivy/kivy


text:
RstDocument error

@akshayaurora spotted an error which appears right after Kivy Catalog is started.
It was introduced by https://github.com/kivy/kivy/pull/4055.

@dessant seems like the word-wrapping feature has some bugs with creating newlines. I'd like to look into this in a day or two.
Btw, how did you trace it back to that PR? I couldn't find any direct link, except for building an older commit on which it works.

@udiboy1209, I was suspecting TextInput changes because CodeInput is subclassed from it, and I've tested commits from the module's commit history until I've seen that the pr commit breaks, while the pr's parent commit works.

@dessant I just tested this with a simple TextInput. Same bug occurs ie. The two newlines between 1.4 and RstDocument are removed. I will look into this later today.

source: gharchive/issue

license: MIT

license_type: permissive

license_source: github-api


text:
When will there be an update?

The last update was 3 months ago. When will a new version be available? I have to use a different browser while this one is out of date.

The last update was 3 months ago. When will a new version be available? I have to use a different browser while this one is out of date.

Yeah we need to do something to continue it, because the devs maybe left the project.


text:
Add deleteAccount() callback.

Not sure this is enough.

Not sure this is enough

I will test it in the feature branch before merging into master. I will ping you if something is not working.

source: gharchive/pull-request

authors: Blackhex tbergq

repo: kiwicom/mobile


text:
🛑 Mahasamana is down

In fd40e47, Mahasamana (https://mahasamana.org) was down:

HTTP code: 502
Response time: 222 ms

Resolved: Mahasamana is back up in d025fa7 after 9 minutes.

repo: kizzna/uptime


text:
🛑 Thainess is down

In 73eb040, Thainess (https://thainess.net) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Thainess is back up in 6afeb28.


text:
🛑 Sangharaja is down

In 078a89d, Sangharaja (https://sangharaja.org) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Sangharaja is back up in 7086d8e.


text:
Rendering issue

As you can see, i got rendering issues? What did i do wrong? :)

Hi @GuzzyMedia,
Looks like you're trying to render a block of Razor within a WebForms template? That won't work :)
Form Editor has no official support for WebForms. If you want to use it in a WebForms context, you'll have to create your own rendering from scratch.
Alternatively you might be able to move the Razor to a macro. Not sure if it'll work, nor how it'll perform (you'll efficiently be mixing two different rendering engines in one page rendering).
-Kenn

Hello
Thanks for the fast response!
The thing is, that i'm trying to implement your form into a content page, but i'm not sure where to put the razor.. The template, our content page is using is a webform, and I find it as the only place I can put it into... Do you have the time to help me out?

I've just tried doing it with a macro.. Can't see what i'm doing wrong.. :(

The log files should probably give you some clues as to what's wrong with your code. My guess is you need to close the code block before calling @Html.Partial(...).
I just tried creating a WebForms master page that calls a Razor macro with the Form Editor rendering. Looks like it works just fine.
Keep in mind though that you're still mixing WebForms and Razor in the same rendering, which might have a performance impact. Also there might just be Form Editor specific things that simply won't work with this solution, as it's never been tested like this.
Here's the master page:
<%@ Master Language="C#" MasterPageFile="~/umbraco/masterpages/default.master" AutoEventWireup="true" %>

<asp:Content ContentPlaceHolderID="ContentPlaceHolderDefault" runat="server">

<!DOCTYPE html>
<html>
<head>
  <title>Test WebForms</title>
  <link rel="stylesheet" href="http://getbootstrap.com/dist/css/bootstrap.min.css"/>
  <style>
    /* required field indicator on the field labels */
    div.form-group.required > label:after {
      content: ' *';
      color: #a94442;
    }
  </style>
</head>
<body>

<umbraco:macro alias="FormEditorMacro" runat="server" />

</body>
</html>

</asp:Content>

...and here's the macro (I've used the NoScript rendering for simplicity, but it can be whatever):
@inherits Umbraco.Web.Macros.PartialViewMacroPage

@* render the form with the NoScript partial *@
@Html.Partial("FormEditor/NoScript", Umbraco.AssignedContentItem)

You're amazing! You just saved my day..
It was my macro that had some difficulties..
As you can see, i'm still pretty new to Umbraco, so i learned alot from this! 👍
Thanks alot!

You're welcome. Glad you got it working.

authors: GuzzyMedia kjac

repo: kjac/FormEditor

license: mit

license_source: bigquery


text:
Improve save_history

See https://github.com/danburkert/ksql/blob/master/src/main.rs#L199

I don't understand, it's expected because the two calls don't output the same Error. BTW, since ReadlineError can contain a io::Error, you can do fs::create_dir_all(history_path.parent().unwrap()).map_err(Into::into).and_then(|_| readline.save_history(&history_path)).
A better improvement would be to use https://github.com/brson/error-chain. Waiting for https://github.com/brson/error-chain/issues/53.

Waiting for https://github.com/brson/error-chain/pull/55 to be merged.

@Yamakaky both https://github.com/rust-lang-deprecated/error-chain/issues/53 and https://github.com/rust-lang-deprecated/error-chain/pull/55 are merged.
Is this ticket still relevant?

@Yamakaky both https://github.com/rust-lang-deprecated/error-chain/issues/53 and https://github.com/rust-lang-deprecated/error-chain/pull/55 are merged.
Is this ticket still relevant?

error-chain doesn't seem to be used anymore so no !

error-chain doesn't seem to be used anymore so no !

@gwenn, @Yamakaky says this ticket isn't relevant any more.
Can it be closed?

@gwenn, @Yamakaky says this ticket isn't relevant any more.
Can it be closed?

authors: Yamakaky gwenn walles

repo: kkawakam/rustyline


text:
Run in background or asynchronously

Is there any possibility to make the ".run()" method run in background or asynchronously? Without directly using threads nor similar options (twisted, etc).
If not, is there are option to stop the ".run()" method? Or should I use threads or similar to achieve this? I want to avoid the threading because the code will run on a simple RPi.
Thanks in advance

See subprocess.
import subprocess
import ffmpeg

args = (ffmpeg
    .input('in.mp4')
    .output('out.mp4')
    .get_args()
)
subprocess.Popen(['ffmpeg'] + args)

authors: agjunyent kkroening

repo: kkroening/ffmpeg-python


text:
dev(indexer): verify event origin

Describe the enhancement request
Currently, we do not verify that origin of an event (present in the fromAddress field of events) is equal to the KakarotAddress. We should verify this as it could pose an issue if we ever allow arbitrary calls to Cairo contracts from Kakarot.
Time: 20 min

Can I take this up ?

Hello @greged93 , my name is Dilan. I´ve worked with rust a lot of times so i think I should be able to complete this issue approximately in a week. I would like to investigate more and try to solve it. ¿Could i take this issue? I´m a responsible person.

Hey sorry @DilanHern but I will assign this to Sweta as she commented first

authors: DilanHern greged93 swetshaw

repo: kkrt-labs/kakarot-rpc


text:
dev: improve logging using json formatting

Describe the enhancement request
In order to make logs processing easier, update the current tracing implementation to use JSON formatting.
See below tracing from DNA:

This can be achieved using RUST_LOG_FORMAT: json in the environment for DNA.

Checked out how they are doing it for DNA. Can you consider assigning it to me?

hey @bhavyagosai yeah sorry, little involved with multiple blocks in our infra, will be easier if I take this


text:
feat: enable registration checks

Pull Request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Resolves: #936
What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[ ] No

This change is

Closes #136

repo: kkrt-labs/kakarot-ssj


text:
Feat: be able to specify a nodePort in values when type is NodePort

It would be nice to be able to specify a nodePort when defined in values.yml.
Without values.yml, it generates the following:
service:
  type: ClusterIP
  port: 80

Whereas with this values.yml:
---
service:
  type: NodePort
  nodePort: 32001

It would generate the following:
service:
  type: NodePort
  port: 80
  nodePort: 32001

Thank you!

authors: DamienGarrido klausmeyer

repo: klausmeyer/helm-charts


text:
🛑 relaunch.gleistein.com is down

In 2ff9b29, relaunch.gleistein.com (https://relaunch.gleistein.com) was down:

HTTP code: 503
Response time: 630 ms

Resolved: relaunch.gleistein.com is back up in a79e31d.

repo: klitsche/upptime


text:
docs(install): add guidance for installing the operator via Helm

Description
This PR adds guidance for installing the flux-kluctl-controller via Helm.
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Already done

repo: kluctl/flux-kluctl-controller


text:
An initial PR

Just a few suggested changes to get started looking at the code.

Add in conda environment file
Explicitly have example_data folder
Update the readme to include the LSC/kriging method

Closing this as has already been resolved.


text:
ISO Get LMP Fails on 2018-11-04

Reproduce
import gridstatus

iso = gridstatus.ISONE()
iso.get_lmp(
    date="2018-11-04",
    market="DAY_AHEAD_HOURLY",
)

AmbiguousTimeError                        Traceback (most recent call last)
Cell In[38], line 4
      1 import gridstatus
      3 iso = gridstatus.ISONE()
----> 4 iso.get_lmp(
      5     date="2018-11-04",
      6     market="DAY_AHEAD_HOURLY",
      7 )

File ~/Documents/isodata/gridstatus/lmp_config.py:26, in lmp_config.__call__.<locals>.wrapper(*args, **kwargs)
     24 if len(args) > 0 and isinstance(args[0], ISOBase):
     25     bound_args = self._verify_bound_args(bound_args)
---> 26     return self._class_method_wrapper(func, bound_args)
     27 else:
     28     # This is a runtime check after method is called.
     29     # Possible improvement: move this to "compile"-time.
     30     raise ValueError("Must be class method on ISOBase")

File ~/Documents/isodata/gridstatus/lmp_config.py:61, in lmp_config._class_method_wrapper(func, bound_args)
     59 instance_args = bound_args.args
     60 instance_kwargs = bound_args.kwargs
---> 61 return func(*instance_args, **instance_kwargs)

File ~/Documents/isodata/gridstatus/decorators.py:78, in support_date_range.__call__.<locals>.wrapped_f(*args, **kwargs)
...
File ~/Library/Application Support/pypoetry/venv/lib/python3.10/site-packages/pandas/_libs/tslibs/tzconversion.pyx:313, in pandas._libs.tslibs.tzconversion.tz_localize_to_utc()

File ~/Library/Application Support/pypoetry/venv/lib/python3.10/site-packages/pandas/_libs/tslibs/tzconversion.pyx:553, in pandas._libs.tslibs.tzconversion._get_dst_hours()

AmbiguousTimeError: There are 3 dst switches when there should only be 1.

Likely a problem in the pandas date parsing code. This would be a good first issue.

I'll take a look at this one for a first task

Ah.
Pandas assumes that time data is sorted so it can tell when we are going into DST. i.e. times will be going up ascending in the series and then drop the hour go back up again. I'm surprised it works with some of the other DST dates in 2022/2021. It may be that it just infers incorrectly in those cases 🤷
Because we are looking at hourly data pandas can't infer what the non DST 1a is and what the DST 1a is. But because of the funny 02X hour ending notation, we can be explicit on which one is which.
I should have a PR up soonish.

we do the groupby location and once that happens, there should be no ambiguity when trying to do the DST infernce.
https://github.com/kmax12/gridstatus/blob/88c8082e99be78e3eb1c20e7997a8bc07b69229d/gridstatus/isone.py#L426
I looked more and the issue actually looks to be with the data the ISONE is returning. here is a snippet. you can see there all multiple rows for the same data.

i think the actual fix is to just do a drop_duplicates

Woof, barked up the wrong tree! hahha. I see, it infers that there is 2 "01:00" in each of the groupings and then can infer that the second is the 01:00 -5UTC. Very cool. I'll add the dedupe code and re-push.

no worries. i also got it wrong in the issue description. issue is with isone, not gridstatus code :)

Seems like we may have another data oddity.
After deduping I kept receiving the error and turns out Location ID can have multiple Locations in it. Is there a reason why we groupby Location ID? After grouping by Location I had successful test runs and our difficult DST day worked as well.

i think only because I assumed Location Id would be unique. If location works, let use that!


text:
During Kmesh daemon restart process, new connections will not go through waypoint

What happened:
I write E2E test case (#661 )  to make sure everything work fine before, during and after Kmesh daemon restart.
I check whether the managed application  is processed normally by the Kmesh by checking whether the request go through waypoint.
However, according to the test result, during the restart of Kmesh daemon, traffic does go pass through waypoint.
What you expected to happen:
Before, during and after Kmesh daemon restart, all traffic should go through waypoint.
How to reproduce it (as minimally and precisely as possible):
Deploy bookinfo and service granularity waypoint for reviews, ref: https://kmesh.net/en/docs/userguide/try_waypoint/
Keep an eye on waypoint log:
kubectl logs -f reviews-svc-waypoint-6884756fc5-524dz

Create continuous traffic:
kubectl exec deploy/sleep -- sh -c "for i in \$(seq 1 10000); do curl -s http://productpage:9080/productpage | grep reviews-v.-; sleep 1; done"

At this time, you will see that the waypoint continues to output access log.
Simulate the restart process of Kmesh daemon by change the Kmesh image to an unavailable image.
kubectl edit daemonset kmesh  -n kmesh-system
...
   image: ghcr.io/kmesh-net/kmesh:latest-a
...

After that Kmesh pod has been in unavailable state of "ImagePullBackOff" and waypoint no longer output access logs, indicating that subsequent traffic no longer go through waypoint.
Anything else we need to know?:
Environment:

Kmesh version:
Others:

This is most likely a problem with the kind environment used
In the K8S cluster environment set up by kind, the /sys/fs directory is read-only by default, and kmesh cannot operate the directory. Therefore, the bpf program is stored in the kmesh pod and is deleted during the restart.
If you go inside the node when setting up the k8s cluster and create a readable and writable bpf directory, you should be able to solve this problem.
docker exec -it ambient-worker /bin/bash
mount -t bpf none /sys/fs/bpf

Please retest @YaoZengzeng

This is most likely a problem with the kind environment used In the K8S cluster environment set up by kind, the /sys/fs directory is read-only by default, and kmesh cannot operate the directory. Therefore, the bpf program is stored in the kmesh pod and is deleted during the restart. If you go inside the node when setting up the k8s cluster and create a readable and writable bpf directory, you should be able to solve this problem.
docker exec -it ambient-worker /bin/bash
mount -t bpf none /sys/fs/bpf

After doing this, it will work fine

Tested and verified, now closed

authors: YaoZengzeng hzxuzhonghu lec-bit

repo: kmesh-net/kmesh


text:
feat: support kmesh daemon version command(#198)

What type of PR is this?
feature

What this PR does / why we need it:
support kmesh daemon version command(#198)
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

How should I use the command? And do I get the gitversion or the Kmesh version?

i followed the same settings of kurator, it should be the kmesh version?


text:
[WIP] #200 add --check option to login command

close #200

[x] #287 の後にrebaseしてからmerge (#287 とは独立だが衝突するので)
[x] テストを書く

[ ] LOGIN_WITH_COOKIES.mdの手順を--checkオプションを使ったものにする

はどうでしょう？

よさそう

TopCoder への対応がないのですが、これは面倒だったためです。彼らはJavaScriptが好きすぎる (https://github.com/kmyk/online-judge-tools/issues/292)

LOGIN_WITH_COOKIES.md の手順5に --check を足しました。
なお手順3のテンプレートを作る部分はこれでは置き換えできません。(必要最低限の通信しかしないのでcookieがsetされない)

@fukatani WIP取りました。reviewとmergeお願いします

コンフリクト解消しました

コンフリクト解消ありがとうございます。

authors: fukatani kmyk

repo: kmyk/online-judge-tools


text:
I currently want to make the service to be exposed so that 80 -> 443 -> https automatically with TLS Termination at Gateway level. However I couldn't able to do anything with config-networking, config-gateway, config-domain.

I currently want to make the service to be exposed so that 80 -> 443 -> https automatically with TLS Termination at Gateway level. However I couldn't able to do anything with config-networking, config-gateway, config-domain.
I was able to make it work with http-80 though the HTTPRoute generated are only using 80. I want to be able to generate using this format:
httpbin redirect httproute sample
---
apiVersion: gateway.networking.k8s.io/v1beta1
kind: HTTPRoute
metadata:
  name: 80-http-httpbin
  namespace: httpbin
spec:
  hostnames:
    - "httpbin.ts.anhdle14.me"
  parentRefs:
    - name: l7-ts-gateway
      sectionName: http
      namespace: istio-system
  rules:
    - filters:
        - type: RequestRedirect
          requestRedirect:
            scheme: https
            statusCode: 301
            port: 443
---
apiVersion: gateway.networking.k8s.io/v1beta1
kind: HTTPRoute
metadata:
  name: 443-https-httpbin
  namespace: httpbin
spec:
  hostnames:
    - "httpbin.ts.anhdle14.me"
  parentRefs:
    - name: l7-ts-gateway
      sectionName: https
      namespace: istio-system
  rules:
    - backendRefs:
        - name: httpbin
          port: 80

What being currently generated now.
# Please edit the object below. Lines beginning with a '#' will be ignored,
# and an empty file will abort the edit. If an error occurs while saving this file will be
# reopened with the relevant failures.
#
apiVersion: gateway.networking.k8s.io/v1beta1
kind: HTTPRoute
metadata:
  annotations:
    networking.internal.knative.dev/rollout: '{"configurations":[{"configurationName":"hello","percent":100,"revisions":[{"revisionName":"hello-00001","percent":100}],"stepParams":{}}]}'
    networking.knative.dev/ingress.class: gateway-api.ingress.networking.knative.dev
    serving.knative.dev/creator: anhdle14@icloud.com
    serving.knative.dev/lastModifier: anhdle14@icloud.com
  creationTimestamp: "2023-04-02T19:20:34Z"
  generation: 3
  labels:
    networking.knative.dev/visibility: ""
    serving.knative.dev/route: hello
    serving.knative.dev/routeNamespace: default
    serving.knative.dev/service: hello
  name: hello.default.knative.ts.anhdle14.me
  namespace: default
spec:
  hostnames:
  - hello.default.knative.ts.anhdle14.me
  parentRefs:
  - group: gateway.networking.k8s.io
    kind: Gateway
    sectionName: http # <----------------- CAN SEPCIFY THE LISTENER HERE. BUT MISSING.
    name: knative-gateway
    namespace: istio-system
  rules:
  - backendRefs:
    - filters:
      - requestHeaderModifier:
          set:
          - name: Knative-Serving-Revision
            value: hello-00001
          - name: Knative-Serving-Namespace
            value: default
        type: RequestHeaderModifier
      group: ""
      kind: Service
      name: hello-00001
      port: 80
      weight: 100
    filters:
    - requestHeaderModifier:
        set:
        - name: K-Network-Hash
          value: d5e6e76864e0485d5d3e37426c6f081d0efba24af58e6a9285cb6cc55cb64dc9
      type: RequestHeaderModifier
    matches:
    - headers:
      - name: K-Network-Hash
        type: Exact
        value: override
      path:
        type: PathPrefix
        value: /
  - backendRefs:
    - filters:
      - requestHeaderModifier:
          set:
          - name: Knative-Serving-Revision
            value: hello-00001
          - name: Knative-Serving-Namespace
            value: default
        type: RequestHeaderModifier
      group: ""
      kind: Service
      name: hello-00001
      port: 80
      weight: 100
    matches:
    - path:
        type: PathPrefix
        value: /

Could we provide anyway to allow config-gateway to allow some templates to be used when generating *Route resource?
Originally posted by @anhdle14 in https://github.com/knative-sandbox/net-gateway-api/issues/367#issuecomment-1493420989

@anhdle14 to confirm you just want clients that make a request using HTTP to receive a 301 redirect to HTTPS?

/remove-lifecycle stale

/reopen

repo: knative-extensions/net-gateway-api


text:
Update releasability

Signed-off-by: Francesco Guardiani francescoguard@gmail.com

cc @nak3


text:
[Automated] Update actions

Produced via:
# meta: knative-sandbox/.github
# main: knative-sandbox/eventing-camel
cp $(find "${GITHUB_WORKSPACE}/meta/workflow-templates" -type f -name '*.yaml') \
  "${GITHUB_WORKSPACE}/main/.github/workflows/"

Cron. -knative-prow-robot
/cc knative-sandbox/source-wg-leads
/assign knative-sandbox/source-wg-leads

/lgtm
/approve

authors: knative-automation nicolaferraro

repo: knative-sandbox/eventing-camel


text:
[0.21] Backport of #525

Backport of #525 (Runaway CPU due to timed-out secret watcher)

/ok-to-test

authors: eric-sap travis-minke-sap

repo: knative-sandbox/eventing-kafka


text:
[main] Update community files

Cron -knative-prow-robot
/cc knative-sandbox/networking-wg-leads
/assign knative-sandbox/networking-wg-leads
Produced by: knative-sandbox/knobots/actions/update-community
Details:

/retest

/retest
/lgtm
/approve

authors: ZhiminXiang dprotaso knative-automation

repo: knative-sandbox/net-istio


text:
[master] Upgrade to latest dependencies

Produced via:
./hack/update-deps.sh --upgrade
./hack/update-codegen.sh

/cc knative-sandbox/networking-wg-leads
/assign knative-sandbox/networking-wg-leads

/ok-to-test

authors: davidor knative-automation

repo: knative-sandbox/net-kourier


text:
Add cluster and clusterBindings to types

Expected Behavior
The following types should be added:

pipelineparams should have clusters
pipelines should have clusterBindings
tasks should have cluster inputs

(All should be optional)
This wouldn't be expected to actually do anything yet (similar to the rest of the current implementation).
Actual Behavior
We have references to clusters in our example tasks:
https://github.com/knative/build-pipeline/blob/3726a431cee21f9a88bc320339c978cca59a58e5/examples/deploy_tasks.yaml#L17-L18
Clusters defined in our example pipeline params:
https://github.com/knative/build-pipeline/blob/3726a431cee21f9a88bc320339c978cca59a58e5/examples/pipelineparams.yaml#L33-L39
Note that the clusterBinding example is incomplete:
https://github.com/knative/build-pipeline/blob/3726a431cee21f9a88bc320339c978cca59a58e5/examples/pipelines/kritis.yaml#L44-L45
This should be binding a cluster from PipelineParams to a cluster input in the Task, e.g.
          clusterBindings:
              -  inputName: clusterName # refers to the input in the Task
                 clusterName: test # refers to the cluster in the PipelineParms

Additional Info
@barney-s will be working on designs around deployment and k8s cluster interaction for this project

/assign @shashwathi

authors: bobcatfish shashwathi

repo: knative/build-pipeline


text:
[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative/operations-wg-leads
/assign knative/operations-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/caching 5a6403e...b2a3f59:
  > b2a3f59 Update community files (# 612)
bumping knative.dev/eventing 9590bc3...08ac6d0:
  > 08ac6d0 upgrade to latest dependencies (# 6266)
  > 19935b1 Set dead letter sink URI in the `Channel` status (# 6256)
  > fffe9c2 Update community files (# 6255)
  > f9498e8 upgrade to latest dependencies (# 6252)
  > 1d881c1 Update community files (# 6251)
bumping knative.dev/serving f52ae88...b5996a7:
  > b5996a7 Fix PVC test service name (# 12736)
  > 509c582 Update net-istio nightly (# 12725)
  > 035df85 Update community files (# 12730)
  > a93cf84 Update community files (# 12716)
  > 08fdf9a Update net-kourier nightly (# 12719)
  > 9fb7372 Update net-gateway-api nightly (# 12711)
  > 910e518 Update net-kourier nightly (# 12710)
  > dbe2fef Update net-contour nightly (# 12709)
  > 7932c0b Enable istio access logging during e2e (# 12714)
  > 3d71d98 Update net-certmanager nightly (# 12708)
  > 79d4908 Fix TestProbeRuntimeAfterStartup/periodSeconds=0 flake (# 12643)
bumping knative.dev/hack e178598...c9c3ea1:
  > c9c3ea1 Update community files (# 160)
  > aae7c22 Update community files (# 159)
  > 0687d6e Update community files (# 158)
bumping knative.dev/pkg 38af013...616f1ab:
  > 616f1ab Add genreconcile for NetworkPolicy. (# 2465)
  > 721abec Bump prometheus/client_golang to v1.11.1 (# 2450)
  > a5ba81d Update community files (# 2461)
  > 3c4dec9 Take pointer to map in AsOptionalMap for consistency (# 2460)
  > ce6f287 Initialize prefix remapping map to avoid panic (# 2453)
  > 78fdd53 Update community files (# 2456)
  > a850b9e Use t.Setenv instead of os.Setenv in tests (# 2454)
  > d2cdc68 Update community files (# 2451)

/lgtm
/approve

authors: houshengbo knative-automation

repo: knative/operator


text:
Add a keeper command?

Should we have a keeper command that prints the command following it, a la https://github.com/rntz/talon-keeper?
I'm not sure whether that implementation will work on dragon, can anybody comment?

I love having keeper. It's based on unofficial apis, but so is "cancel" I guess so probably fine. I'd lean towards "yes"

My primary concern is whether it will work with dragon or not. If it will, I'd say merge it; if not, I'd rather not.

what does keeper actually do? it looks like it behaves a bit like "phrase" or "say" but completely without processing the text like <user.prose> or  would?

It's meant for when you want to insert the text of a voice command, eg. "keeper clear all" will insert "clear all". An anchored version of "phrase" would also theoretically work, but keeper (a) guarantees that what you capture will actually be a command rather than a similar-sounding sequence of words, because it doesn't use  but instead post-processes the command, and (b) doesn't cause competition between  and the command grammar during decoding, so it is less likely to get misrecognized.

authors: pokey rntz timo


text:
Firebird

Environment
Knex version:
Database + version: Firebird 2.5/3
OS: Linux
I work constantly with firebird databases and I'd like to use knex as I've been doing is awesome.
There is an existing but old fork supporting firebird. https://www.npmjs.com/package/knex-node-firebird
Could it be possible to add firebird as a supported RDBMS in this repo? If that's in your plans it'll be great not to have duplicated code and old dependencies when forking.
EDIT [08/16/2018]
I'm working on a firebird implementation, is not completed yet but is currently functional with basic behavior.
It's based from knex-node-firebird
This is the repo Knex(0.15.2) with Firebird FWIW.
I'd like to send a pull request to include firebird support in this repo but i don't know if this is functional to someone else.

I'm working on a firebird implementation, is not completed yet but is currently functional with basic behavior.
It's based from knex-node-firebird
This is the repo Knex(0.15.2) with Firebird FWIW.
I'd like to send a pull request to include firebird support in this repo but i don't know if this is functional to someone else.

Info about setting up new dialects is found from CONTRIBUTING.md

@kerwynrg  not found dialect firebird in
This is the repo Knex(0.15.2) with Firebird FWIW.
please send

I did an updated version of knex firebird dialect, check out my package knex-firebird-dialect

@Tomas2D Would you like to send a PR, adding dialect to ECOSYSTEM.md doc?

Sure, PR #5003 created.

repo: knex/knex


text:
Unable to run migration in Typescript

Environment
Knex version: ^0.95.4
Database + version: sqlite3@^5.0.2
OS: Linux
Bug
@lorefnon, I trying to use the knex in my migrations, but when I run its raised a exception.
TSError: ⨯ Unable to compile TypeScript:
src/databases/migrations/20210401032203_createDatabase.ts:5:44 - error TS7006: Parameter 'table' implicitly has an 'any' type.

20210401032203_createDatabase.ts
import { Knex } from "knex";

export async function up(knex: Knex): Promise<void> {
    return knex.schema.createTable('teams',table => {
        table.text('id')
        table.text('name').unique().primary().notNullable()
    })
    .createTable('countries', table=>{
        table.increments('id')
        table.text('name').unique().notNullable()
    })
    .createTable('teamCoutries', table => {
        table.increments('id')
        table.text('teamId').references('teams.id').notNullable().onDelete('CASCATE')
        table.text('countryId').references('countries.id').notNullable().onDelete('CASCATE')
    })
    .createTable('users', table => {
        table.text('id').primary()
        table.text('teamId').references('teams.id').nullable().onDelete('SET NULL')

    })
}

export async function down(knex: Knex): Promise<void> {
    knex.schema.dropTable('users')
    knex.schema.dropTable('teamCountries')
    knex.schema.dropTable('countries')
    knex.schema.dropTable('teams')
}

knexfile.ts
// Update with your config settings.
import path from 'path'

const stages = {

  development: {
    client: 'sqlite3',
    connection: {
      filename: './dev.sqlite3'
    },
    useNullAsDefault: true,
    migrations: {
      tableName: 'knex_migrations',
      directory: path.resolve(__dirname, 'src', 'databases', 'migrations'),
      loadExtensions:['.ts']
    }
  },

  staging: {
    client: 'postgresql',
    connection: {
      database: 'my_db',
      user: 'username',
      password: 'password'
    },
    pool: {
      min: 2,
      max: 10
    },
    migrations: {
      tableName: 'knex_migrations'
    }
  },

  production: {
    client: 'postgresql',
    connection: {
      database: 'my_db',
      user: 'username',
      password: 'password'
    },
    pool: {
      min: 2,
      max: 10
    },
    migrations: {
      tableName: 'knex_migrations',
      extension: 'ts'
    }
  }

}

module.exports = stages

the command that I ran
npx knex --knexfile knexfile.ts migrate:latest

@LorhanSohaky you have a TypeScript error and that error interrupts your migration process. The reason of error is having noImplicitAny flag enabled in project's tsconfig.json file. More about the flag here.

I found out that the error was due to the typescript version.
It would be interesting to warn this type of error when run any action, as eslint does

I think the error was not related to the TypeScript version you had. noImplicitAny flag is a part of tsconfig.json file which is an integral part of any TypeScript project regardless of the TypeScript version the project uses.

But I changed the typescript version from 3.x to 4.y and it worked

authors: LorhanSohaky ziyaddin


text:
Does not run on windows 10

Nodejs: v20.12.2
Error when running duel:
Failed to compile: spawn D:\project\xxxx\node_modules.bin\tsc ENOENT
Adding "shell: true" to the option of the spawn() function call in duel.js seems to solve the problem.

Hi @byteatom thanks for reporting this. Did you want to open a PR?

I also ran into this issue with version 1.0.8 and can confirm that adding shell: true to the options argument of spawn fixes it. I would be happy to submit a PR, but am curious about the timeline for the next major version of 2.0.0. How stable is 2.0.0-rc.1? I branched off main and was a little surprised when I realized there was a release candidate out there. Is there any interest in releasing a fix as 1.0.9? Of course, the issue is present in 2.0.0-rc.1 too, so if the plan is to publish a 2.0.0 soon I would just make my PR against main as-is. Just let me know and thank you for your work on this.

repo: knightedcodemonkey/duel


text:
linkStyle now supports list of indexes with a few tests

For example:
linkStyle 0,1,2 interpolate basis stroke:#00f,stroke-width:2px;
Other variants of linkStyle syntax are also included

Pull Request Test Coverage Report for Build 739

43 of 48 (89.58%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.07%) to 54.324%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/diagrams/flowchart/parser/flow.js
31
36
86.11%

Files with Coverage Reduction
New Missed Lines
%

src/diagrams/flowchart/parser/flow.js
1
62.04%

Totals

Change from base Build 716:
0.07%

Covered Lines:
2058

Relevant Lines:
3762

💛  - Coveralls

Am I supposed to address codeclimate's issues? Most of them are about the code auto-generated from jison, a couple is about similarities in test code (indeed the code pieces are similar when they test similar situations), and one is Function updateLink has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring. To me updateLink looks fine.
I couldn't find contribution guide or something, so could you please advice what should I do next?

Thank you Ivan! I agree, we need to think about the codeclimate tests. If you still want to do something, consider adding some diagrams with this styling in the e2e tests. That will make sure the fix is not lost at some point in the future.

authors: coveralls ivan-danilov knsv

repo: knsv/mermaid


text:
ConfigFlow

Adds ConfigFlow (UI)

Official documentation

Closed by PR #69

repo: knudsvik/EnergyScore


text:
feat: optimizations and bug fixes

Optimizes gas usage by removing the signals tree, replacing with a mapping.
Fixes nullifier bug by upgrading to latest snarkjs. Adds negative tests for the bug.
Saves all the leaves.
Upgrades to Solidity 0.5.10.

@weijiekoh can you please review?

MerkleTreeTester.sol needs upgrading to 0.5.x too

The cached verifier.sol is causing the CI build to fail.

The cached verifier.sol is causing the CI build to fail.

Ah! That's what it is. Thanks.

Fixes https://github.com/kobigurk/semaphore/issues/16.

authors: kobigurk weijiekoh

repo: kobigurk/semaphore


text:
Use best options for Symfony, fix Cobalt+Joomla prototype containers

update to Symfony 4.0.4
fixes Joomla & Cobalt container in prototype mode (they were just non-shared or empty before)
turn on as_files and inline_class_loader options, that's the default for skeleton Symfony 4 apps

Awesome! I couldn't configure the file-based container myself properly when I tried to play with it so thank you very much for the help! I'll rerun the test suite soon.

ping us when you do, I think a lot of people will be excited to see the results as a lot of work has been done there :)

Having to restart my computer and shut down all the apps is blocking the re-run. :D But I'll be able to do that in an hour! :)

@nicolas-grekas, @theofidry, I've just generated the results (many-many times, even with different setups), but unfortunately Symfony seems to be a bit slower now. :( (though the difference is usually negligible)
I am a bit disappointed because I expected - and wanted to see - big performance improvements. How much difference do you have on your systems? What did Blackfire say?
Although I am not very familiar with the internals of autoloading, I wonder if it tricks us. I think so because the biggest change was that Composer's autoloader is replaced with include_once invocations. So my very wild guess is that either my tests don't measure autoloading time properly or maybe opcache does something very differently than what you reckoned on. What do you think?
I would be happy if you had a good explanation about this weird phenomenon. :/ :)

Haha :)
On my local setup, I identified 3 separate situations:

on cold tests, Symfony was usually 1st or 2nd
on non-shared (prototype) tests, Symfony generates too many include_once, fixed in 4.1
on warmed up tests, it cannot be fastest, because it does 2 more isset checks (which IMHO means these tests are mostly non representative, we're not at 2 isset checks to decide which DI container to use.)

on warmed up tests, it cannot be fastest, because it does 2 more isset checks (which IMHO means these tests are mostly non representative, we're not at 2 isset checks to decide which DI container to use.)

True and the benchmark is not lying: the difference between rank 7 (Symfony) and rank 1 (Zen) for those is around .2ms...
Also compared to before (3.4), the results are significantly better :)

For reference, if we make this change, Symfony comes always first in the 3rd ranking group where it ranks 6th/7th for now:
--- a/src/Symfony/Component/DependencyInjection/Container.php
+++ b/src/Symfony/Component/DependencyInjection/Container.php
@@ -217,6 +217,11 @@ class Container implements ResettableContainerInterface
      */
     public function get($id, $invalidBehavior = /* self::EXCEPTION_ON_INVALID_REFERENCE */ 1)
     {
+        return $this->services[$id] ?? ($this->factories[$id] ?? array($this, 'make'))($id, $invalidBehavior);
+    }
+
+    private function make($id, $invalidBehavior)
+    {
         if (isset($this->aliases[$id])) {
             $id = $this->aliases[$id];
         }

I also agree that the last two test suites are the least useful and I hardly ever pay attention to them. I'll only leave them there because still, they might help someone who wants to optimize $container->get() calls. But maybe the number of iterations could be increased because as you also noticed, the difference is too small between containers.
Initially, my main source of confusion was that Symfony is not much faster in the first 2 suites. But Nicolas is right, the test suite 3 and 4 are much more affected by the changes. And when I talked about the "autoloader's trick", I suspected that the include_onces are needlessly called all the time. So thanks Nicolas for reassuring this (and I've also found your commit: https://github.com/symfony/dependency-injection/commit/e6158523d19a04043ecf403400f6c5b4617001eb).
I happily regenerate the results with Symfony 4.1 if you want :) I am also curious!

Zen vs Symfony 4.0.4:

Zen vs Symfony 4.1:

@nicolas-grekas Big congratulations :) This is fascinating!

And Symfony might come always 1st within the error margin if https://github.com/symfony/symfony/pull/26161 is merged :)

I'll keep an eye on the PR and will publish the updated results as soon as I can :)

Now merged :)
Should I send a PR to upgrade to 4.1? Can you do it?

No need for a PR, I'll do that now!

Done! Symfony is really on the top for almost all the tests :)

Let's celebrate :)
https://twitter.com/nicolasgrekas/status/963365262840954880

🎉
A small note regarding the results. As you can see below, sometimes the ranking doesn't make sense, maybe it would be good to adjust the second ranking criteria:

In some cases, the difference is incredibly small (0.009ms = 20%) and this for up to rank 6 sometimes. I would suggest to for example calculate a delta and for the the rank 1 + delta add a color indicating that the results are sensibly the same. For example:

Rank
Container
Time (ms)
Time (%)
Peak Memory (MB)
Peak Memory (%)

1
Yaco
0.01
100%
0.649
100%

2
PHP-DI
0.011
110%
0.627
97%

3
Aura
0.011
110%
0.841
130%

4
Symfony
0.011
110%
0.619
95%

5
Zen
0.011
110%
0.679
105%

6
Zend ServiceManager
0.011
110%
0.713
110%

7
Yii2 Container
0.012
120%
0.768
118%

8
Dice
0.013
130%
0.855
132%

9
Pimple
0.019
190%
0.727
112%

10
Cobalt
0.023
230%
0.757
117%

11
Joomla
0.025
250%
0.747
115%

12
Opulence IoC
0.031
310%
0.729
112%

13
Laravel
0.039
390%
0.695
107%

14
Disco
0.04
400%
0.807
124%

15
PHPixie DI
0.045
450%
0.705
109%

16
Auryn
0.064
640%
0.795
123%

For above from rank 1 to rank 8, the difference is so small they should kind of all appear as "super good"

@theofidry Great idea! But what would you think about increasing the number of iterations in the last 2 test suites? They will become even more synthetic this way, but I guess it is not a problem in this case.
@nicolas-grekas I couldn't resist the temptation of trying to optimize Zen... After many months, I found a "nice" way to eliminate some opcodes in the $container->get() method today with the following change: https://github.com/woohoolabs/zen/compare/4976acd6a27264616f01cec99658c99ec1f1d91f...46753cce657f3136e15997ae4aaefcc2b8946046?diff=unified&name=46753cce657f3136e15997ae4aaefcc2b8946046#diff-f348ecacd0d17522912d6b2f2b91c1bbR27
So Zen is the first now in the last 2 test suites! But I'll be happy if you can squeeze out even more speed from Symfony and it comes first again! A little competition helps us to do our best :)

Haha how unexpected.
There is nothing else to squeeze on Symfony side, at some moment we have some features that are measurable in microbench. ;)

authors: kocsismate nicolas-grekas theofidry

repo: kocsismate/php-di-container-benchmarks


text:
Show on landing in latest sales & listing RMRK2 in card

To indicate type of network, RMRK2 items missing rmrk2 in corner as we have Kusama or Basilisk

Seems doesn't work, on screenshots are rmrk2 items

repo: kodadot/nft-gallery


text:
Opening sidebar resets scroll position

before

after

the scroll position should stay

👋

authors: exezbcz leo-anderson-x


text:
feat(collections): filters

I didn't find a way to remove empty collections from query yet
PR type

[x] Feature

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted screenshot of demonstrated change in this PR

What's new?

[x] filters in /collections

Screenshot

I didn't find a way to remove empty collections from query yet

Maybe @vikiival would have idea? would be amazing to close on this, collections are not great with empty collections so not wasting collectors attention, showing only those which can get would be interesting.
Like by default I would put something, show only collections where is at least 1 NFT for BUY

Maybe @vikiival would have idea?

you can't filter on the nested entity
these are the fields you can filter
input CollectionEntityFilter {
  version: StringFilter
  name: StringFilter
  max: IntFilter
  issuer: StringFilter
  symbol: StringFilter
  id: StringFilter
  metadata: StringFilter
  currentOwner: StringFilter
  events: JSONFilter
  blockNumber: BigFloatFilter
  createdAt: DatetimeFilter
  updatedAt: DatetimeFilter
  and: [CollectionEntityFilter!]
  or: [CollectionEntityFilter!]
  not: CollectionEntityFilter
}

the possible hack is to count already minted NFTs in the collection entity

Like by default I would put something, show only collections where is at least 1 NFT for BUY

It will skew the metrics

the possible hack is to count already minted NFTs in the collection entity

We got the totalCount of nfts in collection but can't query/filter on > 0.
Does this work? https://github.com/vikiival/magick/pull/10

It will skew the metrics

Buy now toggle will change metrics for sure. (as it filter nfts price > 0 in collection)

We got the totalCount of nfts in collection but can't query/filter on > 0.

Have you tried to run it on your local machine? 💻
Is think that if we make the field as required it won't allow us to filter on top of that anyway

It will skew the metrics

Which metrics? Items?
We can reformulate text to reflect what is shown as numbers. Let's try come up with a smart word for that

Just chatted w @vikiival let's remove empty collection on UI till we figure out other solutions, just to have this working 😄
Lets do hacky solution for now, same with BUY stuff

Bump, can we remove empty collections and let's merge this?

Bump, can we remove empty collections on UI and let's merge this @roiLeo ?

Bump, can we remove empty collections on UI and let's merge this @roiLeo ?

hotfix would be to remove card from row and it's not looking great

I'm sure something can be done with graphql

Yeah, but looking for empty collections is draining attention is worse than showing nothing 😄
Let's push it and we will "upgrade" it later, even now it's a huge step with functionality tho!

WFM continue to #1357

authors: roiLeo vikiival yangwao


text:
Audio NFT player

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[x] Bugfix
[ ] Feature
[ ] Refactoring

What's new?

[x] PR closes #3248
[ ] Requires deployment <rubick/magick_issue>
[ ] <brief_description_of_what_I've_added>

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[ ] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[ ] I've tested PR on mobile and everything seems works
[ ] I found edge cases
[ ] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

LGTM

@Jarsen136 one doubt though
https://deploy-preview-3268--koda-nuxt.netlify.app/rmrk/gallery/10249720-466e8b6338599eaf1e-GAVCUBE-GAV_CUBE_101-0000000000000101?page=1
is this also an audio nft?

@Jarsen136 one doubt though https://deploy-preview-3268--koda-nuxt.netlify.app/rmrk/gallery/10249720-466e8b6338599eaf1e-GAVCUBE-GAV_CUBE_101-0000000000000101?page=1 is this also an audio nft?

Hey, it's just a image 🚶

pay 50 usd

😍 Perfect, I’ve sent the payout
💵 $50 @ 57.73 USD/KSM ~ 0.866 $KSM
🧗 Caiv9TbPz68q5dC8EcHu5xKYPRnremimGzqmEejDFNpWWLG
🔗 0x7c842769c2375b71e825b089613f1cf732bf60cea5c84dab09f07a22a82deceb
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


text:
Update 🇪🇸 Spanish translation in KodaDot

Check on discord who is responsible (wagmedia?) for this translation, I just integrate it into the code base

got some attestation from discord

[14:13]johnrhodel: Hey my friend
the person who is translating is the @MrDeFi#0905
[14:16]CryptoX I💰I: @MrDeFi#0905 was the person who did it

do we have somebody speaking Spanish here? I'm thinking maybe @JustLuuuu 😄

Claro que sí, hablo muy bien español

@yangwao I assume the payment is being done through different channels here

authors: petersopko roiLeo yangwao


text:
🛑 clouflare ipfs is down

In 76a9e81, clouflare ipfs (https://cloudflare-ipfs.com/ipfs/bafybeidkhq7aynm2yuuvzimkqcabbqgg3w2uzvy2e7zq3vl5ojhf36kxea) was down:

HTTP code: 0
Response time: 0 ms

Resolved: clouflare ipfs is back up in e283ee9.

repo: kodadot/uptime


text:
[Feature Request] Show fields in list view

Right now when viewing a collection in list view, there are 3 columns for each item: Name, Visibility and Actions. It would be nice to have the option to have a column displayed for each custom field too.
For example, I have a rating field, and can now sort by rating, but the rating is not visible in the collection list view, only when going into the item. It we could have this, the overview (and sorting) would be much more useful.

Yes this one of the next feature I want to add.
I just need to find the best way to implement this because an item can have an infinite number of fields, the list would quickly become readable. I'm probably going to add a checkbox for each fields in the collection form so users can choose which ones to display.

Sounds great 😊

I was going to suggest the same thing, love this idea and I think the proposed implementation of having a checkbox next to fields to show them in the column view is a great idea as well.

authors: CorneliousJD batu-one benjaminjonard

repo: koillection/koillection


text:
Naughty Points

For each peer, there should be an error channel that reports all peer errors. The channel should be consumed by an error handler that for each error assigns a weighted score (naughty points) to the peer. Points should decay over time exponentially and be added linearly (leaky bucket). Once a peer's error score surpasses a specific threshold, they are disconnected. The scores need to be preserved after diconnection and decay continue in case the peers attempts to reconnect. (e.g reconnect should not be a reset) These points accrue for both sync and gossip services.

Usurps #5.

authors: mvandeberg sgerbino


text:
Fix error when migrating packs and scheduled query has been deleted.

Fixes #1850

It needs to go in this migration, because the error in #1850 prevents the migration from completing. Anyone who is running into that error will (attempt to) run the migration again each time they fleet prepare db.

Oh, great. This LGTM then.

authors: marpaia zwass

repo: kolide/fleet


text:
Fix ServiceBalancer greediness

This PR fixes a situation where stack deploy is in progress and ServiceBalancerJob decides to fix stack services because desired state does not match. Now balancer checks if stack has a deploy in progress and does not touch services until deploy has finished.

repo: kontena/kontena


text:
[Backport] Add tooltip to plan progress bar indicating the clickable plan name (#519)

Backports #519

Updated, ready for review again.

repo: konveyor/forklift-ui


text:
Mismatch of config.toml with git checkout and image in podman compose:   pydantic_core._pydantic_core.ValidationError: 1 validation error for KaiConfig -  Unable to extract tag using discriminator 'provider' [type=union_tag_not_found, input_value={'solution_detectors': 'n...'password': 'dog8code'}}, input_type=dict]

I think the below is due to a mismatch where we are tagging images to isolate all changes, but we allow the config to be modified and used from the local checkout.
So in my situation:'
-> running podman compose up that uses a :stable tag pointing to:  commit 65a5761d479d84c15ccf0769e2bf00ea71a4836c
-> My kai checkout of 'main' is at commit 11a28d32992d9888234c2ea3a630a52733a48842
With this combination, the code running inside of the :stable image is reading the local checkout to parse the kai/config.toml and its crashing as I suspect we changed structure of kai/config.toml
Error from podman compose up
kai-1     |   File "/kai/kai/server.py", line 265, in app
kai-1     |     config = KaiConfig.model_validate_filepath(
kai-1     |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
kai-1     |   File "/kai/kai/models/kai_config.py", line 125, in model_validate_filepath
kai-1     |     return KaiConfig.model_validate(model_dict)
kai-1     |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
kai-1     |   File "/opt/app-root/lib64/python3.12/site-packages/pydantic/main.py", line 551, in model_validate
kai-1     |     return cls.__pydantic_validator__.validate_python(
kai-1     |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
kai-1     | pydantic_core._pydantic_core.ValidationError: 1 validation error for KaiConfig
kai-1     | incident_store
kai-1     |    pydantic_core._pydantic_core.ValidationError: 1 validation error for KaiConfig
kai-1     |     For further information visit https://errors.pydantic.dev/2.7/v/union_tag_not_found

https://gist.github.com/jwmatthews/2619140e479c4029b5e90d9639d41a68
The workaround for now is:

git checkout stable
podman compose up

This problem will fade away when we bump the stable tag to match main, yet there is potential for it to reoccur as long as we allow a relationship of the container image reading local disk for kai/config.toml... in theory the config file does not change often so the problem isn't expected to occur a lot, but its a potential breakage we may want to isolate.

We plan to address this in https://github.com/konveyor/kai/pull/320
The idea is that most settings will be overridden via environment variables, if the user wants/needs to make config settings, they may copy the build/config_example.toml to build/config.toml and edit what they need and this will be used and take precedence from podman compose up.

repo: konveyor/kai


text:
Automatic Type Checking

We've seen a few issues where type checking would have helped catch them before a runtime failure. I've been using type checking as it's built in to the vs code python extension.
We should add a type check as a pre-commit and CI step. While we're just starting to use it, it should just be a warning, not an error. We may decide to move towards errors some day. Even if it's just for certain files, for starters.
We do need to decide between MyPy and Pyright though.

pyright
mypy

- more dependencies (I think it needs node.js as well as python)
+ easier environment set up

+ more featureful, on the forefront of some type checking evolution
- less featureful

+ the default choice for the vs code python extension
- not the default

I lean towards pyright personally, but we can experiment.

Really like the idea of type checking!
About the VS Code comparison, I believe it should not be part of discussion. But to use mypy with it, I just add
    "python.linting.mypyEnabled": true,
    "python.linting.mypyArgs": [
        "--show-column-numbers",
        "--strict",
    ],

to settings.json (and mypy should be installed in the virtual environment to it to work).

I want to focus on a good development experience, so I think defaults matter. That's the only reason I brought up VS Code. But it is definitely easy to turn on mypy instead!

Closed with #710 and #713.

repo: konveyor/pelorus


text:
:bug: Update tags model to include source for tags

Removes unused "category" field in applications form tag dropdown
Add 'source' field to tagRef so that it will not be overwritten on application edit
Fixes the issues we were seeing around duplicate tags being created with automated tags turned on during subsequent analysis attempts

Closes #688


text:
Add links to product documentation

Resolves https://github.com/konveyor/virt-ui/issues/127

@vconzola I think that's expected until the docs go live. That's what Avital was saying in this comment: https://docs.google.com/presentation/d/1xUFNQPsQAmIrZdZNoG0G1D-UlOMc-OWFnKuwd_FR8Ao/edit?disco=AAAAKWfrTDc

The splash page has not been published yet, so you will see a 404. However, this will be the permanent canonical URL for MTV.

We could hold off on merging this until the docs are released but we do know this will be the correct link.

I'm gonna go ahead and merge this to declutter open PRs, but if you disagree we can revert it.

@vconzola I think that's expected until the docs go live. That's what Avital was saying in this comment: https://docs.google.com/presentation/d/1xUFNQPsQAmIrZdZNoG0G1D-UlOMc-OWFnKuwd_FR8Ao/edit?disco=AAAAKWfrTDc

The splash page has not been published yet, so you will see a 404. However, this will be the permanent canonical URL for MTV.

We could hold off on merging this until the docs are released but we do know this will be the correct link.

@mturley Gotcha. Makes sense. Although I'm not sure if we'll even have a direct doc link in the UI if the MA integration is going away.

That's true. Merged anyway so it'll be there when we re-enable that code.

authors: gildub mturley vconzola

repo: konveyor/virt-ui


text:
[BUG] failed to calculate node topology

What happened:
#kubectl describe pod XXX
Events:
Type     Reason            Age   From             Message
Warning  FailedScheduling  23s   koord-scheduler  0/5 nodes are available: 2 node(s) invalid CPU Topology, 3 node(s) had taint {dedicated: master}, that the pod didn't tolerate.
#kubectl logs koordlet-2zx2f -nkoordinator-system
I0410 14:25:26.458142   11700 states_noderesourcetopology.go:295] start to report node topology
E0410 14:25:26.463152   11700 states_noderesourcetopology.go:306] failed to calculate node topology, err: failed to GetKubeletConfiguration, err: request http://xx.xx.xx.xx:10255/configz failed, code 404
How to solve 404？
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

App version: v1.1.1
Kubernetes version (use kubectl version): v1.21.3
Install details (e.g. helm install args):  - --kubelet-insecure-tls=true
Node environment (for koordlet/runtime-proxy issue):

Containerd/Docker version:
OS version:
Kernal version:
Cgroup driver: cgroupfs/systemd

Others:

It seems the kubelet config is not exported at :10255/configz. However, it is not required. Please try to add the -disable-query-kubelet-config=true in the args of the koordlet DaemonSet.

Thanks, I have got it.

authors: dykhliww saintube

repo: koordinator-sh/koordinator


text:
K.S.

Uploading robots.txt…

Uploading except.txt…

except.txt

repo: korrectiv/merge-conflicts


text:
z-index issues

There are at least 2 z-index issues, which probably need a combined solution, so I report them in one issue:

together with allow-overlap z-index is actually reversed as the objects are drawn in decreasing z-index order, and if allow-overlap is set the later object is drawn above the previous one (which has a higher z-index). This cannot be fixed using "globalCompositeOperation='source-over'" as this would draw the new object below everything on the canvas, i.e. also those objects with lower z-index (e.g. geometry). An example of the error can be seen on OpenRailwayMap, the z-index of the white signs corresponds to the number shown on it.
the code to render text and icons does one pass through one text/icon combination before rendering the next (see kothic.js), which means a bare icon with low z-index would be drawn first, and a bare text with high z-index would not be drawn at all if it collides with this icon.

styledFeatures.sort(function (a, b) {             return a.zIndex < b.zIndex ? -1 :                     a.zIndex > b.zIndex ? 1 : 0;         });
does fix the z-index problems. I don't understand why it was comparing the fill-color and color keys?

This could have been written even simpler as return a.zIndex - b.zIndex. But this does not solve the problem, as the sorting already works. It's the algorithm in which order things are drawn and how it interacts with "allow-overlap: true".

authors: DerDakon braandl

repo: kothic/kothic-js


text:
Add matchers to Klock.

Related to https://github.com/kotlintest/kotlintest/issues/1156.

@sksamuel I have created this draft. I wanted to get an opinion on the approach I have used. Also, what are the matchers that should be implemented.

I believe you're on the right track.
I recommend adding some matchers for numbers, like Time.shouldHaveMinutes(5) or something similar.
I'm not sure if you're using our current datetime matchers as inspiration, but that can be a good starting point if you feel lost!
Take a look

@Kerooker I think I have added all the required matchers. I haven't documented the matchers like done in existing code since, I didnt find any need for doing it, as its self documenting. But if you want me to add docs, then I can.
I have added some tests as well.
I dont know if publishing works on that. Need to check on that.
Other than that, I think the PR is ready to be reviewed.

If you can address the feedback @dragneelfps I will merge. This will be an excellent addition.

@sksamuel feedback taken and code changes done.

Build failing with:
* Where:
Build file '/home/runner/work/kotest/kotest/kotest-assertions/kotest-assertions-klock/build.gradle.kts' line: 70

* What went wrong:
Could not read script '/home/runner/work/kotest/kotest/publish.gradle' as it does not exist.

Any help?

The build was changed over the weekend so publish.gradle doesn't exist. That's ok, we can merge.

authors: Kerooker dragneelfps sksamuel

repo: kotlintest/kotlintest


text:
Change return type of arguments in checkAndDisplayValidUser()

Change the return type to the expected return type of the set... functions

What do you want to return in this function, since it's likely going to change the state instead of returning something since it uses promises.

repo: kpattaswamy/UpLink


text:
Update README.md

Add qgeminiserver to the programming section.

@doga Thanks!


text:
Incorrect sysctl settings on 2.6.x Centos

There are no kern.maxfiles and kern.maxfilesperproc.
#sysctl -w kern.maxfiles=40960 kern.maxfilesperproc=20480
error: "kern.maxfiles" is an unknown key
error: "kern.maxfilesperproc" is an unknown key

Linux rs200516 2.6.32-431.23.3.el6.x86_64 #1 SMP Thu Jul 31 17:20:51 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux

Is this in the main documentation? IIRC this is only on the wiki, in which case you could simply add the working settings for your platform.

I grepped mojo's repo for kern.maxfilesperproc and kern.maxfilesperproc and did not find any matches.

I guess sysctl parameters changed in the meantime. Docs are at https://github.com/kraih/mojo/wiki/Tuning-servers-for-high-concurrency-workloads

@jberger I've tried these settings in belief that it would improve hypnotoad performance => http://stackoverflow.com/questions/32170038/hypnotoad-performance

As I have said, the wiki is not part of the core documentation and can be edited without code changes. As a result I'm going to close this issue.
That said, why are you using the OSX instructions when using centos (linux)?

@jberger re:OSX; you're right; not enough coffee I guess.

authors: jberger mpapec zoffixznet

repo: kraih/mojo


text:
Support for the sandbox attribute on iframe

We are looking into ways of isolating components from our application, this sounds like a good approach to simplify communication. One of the requirement we have is using the sandbox='allow-scripts' attribute  on the iframes
https://www.html5rocks.com/en/tutorials/security/sandboxed-iframes/
Which would mean each iframe will have a uniquer origin, I suspect this should be compatible if this library is able to do cross domain communication it should also work with these unique origins via postMessage.
Is there any way to set the sandbox attribute on the containing iframe?

I was able to add the sandbox='alllow-scripts' attribute via a custom containerTemplate
 frame.sandbox = 'allow-scripts';

However, the page fails with the following error:
Error: Method 'initChild' domain "https://example.com" does not match origin file:// in https://other-example.com

I have done some research and the key here is that origin for sandboxed iframes is set to null.
Then, there is some code in
https://github.com/krakenjs/post-robot/blame/7710b5a4035e392edafe01d1b0c8ba999e2c654f/src/drivers/receive/index.js#L116
That sets the origin to "file://" if the origin is null.
Eventually the matchDomain will fail
https://github.com/krakenjs/cross-domain-utils/blob/2e0c771fcbb28938c3be0d28df78b1738171c563/dist/module/utils.js#L862
Because the origin is a string = "file://" but is not a wildcard and it does not match the domain.
I had a quick try and added an extra line to return true if the origin is =='file://' and the whole framework seems to work, however I realize this is a security hole.
https://www.html5rocks.com/en/tutorials/security/sandboxed-iframes/
Suggest checking the contentWindow matches the source instead, but it seems this is currently only based on the origin being set.
window.addEventListener('message',
    function (e) {
      // Sandboxed iframes which lack the 'allow-same-origin'
      // header have "null" rather than a valid origin. This means you still
      // have to be careful about accepting data via the messaging API you
      // create. Check that source, and validate those inputs!
      var frame = document.getElementById('sandboxed');
      if (e.origin === "null" && e.source === frame.contentWindow)
        alert('Result: ' + e.data);
    });

I'm thinking if another approach would be depending on a prop injected at first render and using that as validation?
@bluepnume could you take a look, it seems supporting sandbox iframes would require changes in some of the core libraries, is this something feasible? Any thoughts?

I tried a naive patch that seems to work and I think it's more secure.
I'm trying this in the dist version to quickly experiment, but essentially if I add this other condition checking if the source of the event matches one of frames in the current window (parent)
![...document.querySelectorAll('iframe')].some(iframeElement =>  iframeElement.contentWindow === _ref2.source)

into function_serializeFunction
 if (!matchDomain(domain, origin)
&& ![...document.querySelectorAll('iframe')].some(iframeElement =>  iframeElement.contentWindow === _ref2.source)){
                            throw new Error("Method '" + data.name + "' domain " + JSON.stringify(util_isRegex(meth.domain) ? meth.domain.source : meth.domain) + " does not match origin " + origin + " in " + getDomain(window));
                        }

then even if matchDomain fails, since its coming from one of the iframes we rendered I think it's safe to be allowed, similar to the example in https://www.html5rocks.com/en/tutorials/security/sandboxed-iframes/
With that tweak all other communication parent -> child via initial props and updating props, child-> parent via function calls seems to work.
I'm sure there is a correct way of doing this instead of querying the document for iframes, but I hope it helps illustrate the problem and a potential solution.

@bluepnume any thoughts on adding this alternate check, maybe as an optional fallback?
I could work on the PR, it would be nice if you can provide initial guidance. I think the original file that needs to be patched to enable support is

Hi @mstuart @mnicpt @bluepnume I have starting implementing a solution for this issue.
If merged, this is the first PR of 3 that I will make to solve this support for sandbox attribute on iframe issue:
https://github.com/krakenjs/cross-domain-utils/pull/19
If merged, my next PR will be for the post-robot repo to use the function made in the cross-domain-utils repo.
The last PR would be for the zoid repo to upgrade versions of post-robot and cross-domain-utils in package.json

I'm a little confused about this issue. I tried implementing a test page here: http://bluesuncorp.co.uk/icv4/frame-parent.htm
This shows event.origin correctly populated even though the iframe is sandboxed. What am I missing?
I have strong concerns about implementing any message acceptance logic which does not rely on domain checks. It's very easy for iframes to be redirected to a rogue domain, and for that domain to send messages as if it were the original domain.
But I'd still be interested to repro this.

Aaah, I see now. It only works with allow-same-origin.
That said -- I think adding allow-same-origin in this case is a lot safer than relaxing the iframe domain check. Otherwise this is a "robbing peter to pay paul" scenario. You gain some marginal extra security for your iframe, at the expensive of a huge loss of security for domain checks from any messages from the iframe.

Also for what it's worth (FYI @anthonykhoa) -- zoid already does essentially check the event.source of any messages sent from the child window. Any functions which are passed from window->window using post-robot have that window check baked in; so adding a new check would be redundant.
https://github.com/krakenjs/post-robot/blob/master/src/serialize/function.js#L90-L94
Here is where that check happens right now:

Ah, thank you for explaining everything @bluepnume! I agree that the solution I implemented would result in a robbing peter to pay paul scenario -- using allow-same-origin would be safer than relaxing an Iframe check. I will close my PR

No worries! Thank you so much for all of your hard work on these repos so far.
We should keep this issue open, regardless. I've been trying to think of alternate ways to solve this. For instance:

Receive a message from the frame with origin: null
Generate a secret UID
Send that UID to the frame with domain: "<expectedDomain>"
Receive a response with the same UID
If the UID matches, we know the frame is still on the expected domain
Accept the message from #331

Only problem is this breaks down at step 3. The frame won't even accept messages to its actual domain: you get Failed to execute 'postMessage' on 'DOMWindow': The target origin provided ('http://www.foobar.com') does not match the recipient window's origin ('null').
So as far as I can tell, unless any other interesting hack turns up, there's no way for the parent window to get some solid proof that the child window is still on the expected domain.

authors: anthonykhoa bluepnume vlopez

repo: krakenjs/zoid


text:
Rust example is out of date

The Rust code should use println! (with a !), not println.

Ah, it's already been fixed in the repo.

repo: krasnoukhov/helloworldquiz


text:
Concentration effect name is empty and acvtice effects delete concentration effects

When I cast for example bless, the concentration effect has no name

If I cast bless and apply the active effect from the chat card, it deletes the concentration effect,

Foundry v10 build 291
dnd5e 2.2.1
Modules of notes might be:
Dfred convenient effect (the effect on the bless spell came from dfred convenient effect and has such a flag so i can remove the effect with their build in remove effect button, without opening the sheet)
ready set roll (provides the chat card that applies the active effect from something like bless when i click the button in the chat card)

The second part of the issue looks to be ET. Possibly a clash given that it transfers by name (or label) unless the transfers are 'permanent'.

but one has no name and the other does?

It was a guess. If the concentration effects get randomly deleted while you have all other modules disabled, please open a separate issue and I will take a look (provide steps to reproduce it).

so tested it today with just midi and its dependencies

the issue still happened
Foundry v11 build 302
DnD5e 2.2.1

with those modules active, roll the guidance spell and concentration on it start

target the token and click the in chat button to apply the effect, which does apply the effect but removes the concentration effect, despite both having a different name

I would like to emphasize the "all other modules disabled" part. CN does not work with midi, and it should never be used with midi. It has always been, and will always be, incompatible with midi.
This is not an issue with CN.

Then what is it an issue with? like you need something to get a active affect in chat like that with a button to apply. Normally I use ready set roll, now I used the alternative, midi, and its still not good. Or is there some option in foundry i am not aware of that gives me a "apply active affect" button?

If it is not an issue with CN, then discussion of it does not belong here.

I am no expert on the tech side of things, so maybe I am wrong, but if its a problem with 1 other module, might be an issue with the other modules, but if the issue is still there when an alternative other module is used, would it be that weird to assume maybe the issue relies with CN, not with two different modules.
or do those other modules use the exact same code to get that apply active effect button, so they simply make the same mistake twice? feels to me like thats less likely and its just an issue with CN. though again, no expert on code and tech and i could be wrong, just from my current perspective it seems like a CN issue.


text:
verifyAuthTime throws InvalidToken exception when auth time is in the future

The problem
I am using Android library to generate firebase authentication token which I am sending to server, however due to time synchronization difference in my server and firebases server, the token is in the future, by couple of seconds..
a $allowFutureTokens parameter is provided to verifyIdToken, but it only helps with the issue when the token is issued in the future, not when the tokens auth_time is in the future.
If $allowFutureTokens is to be supported, it should perhaps work when auth_time is in future too. a possible fix would be just to make Verifier's verifyAuthTime function throw IssuedInTheFuture exception when auth_time is in future.
in this case IssuedInTheFuture isn't really a well describing name to the exception. Instead of having a hard no, there could also be a maximum amount of time the token can be in future for it to be accepted, however I don't even understand why its a security concern to accept future tokens?
Shouldn't we just trust the authentication server to provide sane tokens, or is this a common practice when verifying tokens?

Could you please try if https://github.com/kreait/firebase-php/tree/a-different-kind-of-future works for you? I don‘t like what this functionality has become, but for know I‘m fine with it if it gets the job done :)

Released with 4.19.0

authors: jeromegamez pasmat

repo: kreait/firebase-php


text:
Error while verify token.

i'm using this package it's working fine but When i update my firebase-php (version 5.1.1) i got this error
User Deprecated: Not providing the current time is deprecated. Please pass an instance of DateTimeInterface.
when i call this funtion $fire_base->verifyIdToken($token);
the lcobucci/jwt version is 3.4.0
i got this error.. how i can fix it?
Environment
OS: ubuntu 20.04
PHP version: 7.3
Firebase SDK Version: 5.1.1

Duplicate of #504

@jeromegamez  but i didn't create custom token

It's the same underlying problem - lcobucci/jwt 3.4 introduces deprecation warnings in several places and I need to handle them :)

so we have to wait for the new update of lcobucci/jwt 3.4 and for firebase-php as well?


text:
LocalFileAgent throws error for non existing file in write mode

Test details

tested on everything up to 0d4695ddf499c3639099394676803689e9da22ea

Description
IS

the LocalFileAgent in a 'writing' use case throws error when file in path does not exist

SHOULD

in write mode a non-existent file should be created before trying to write to it
no error should be logged for non-existent files in write mode

Steps to reproduce

set up CsvAgent in serialize mode with LocalFileAgent consuming the events emitted and writing the contents to a file where the file defined in the 'path' option does not yet exist
trigger scenario by serializing a CSV
error is logged. it is unclear if the content is written nonetheless

It should create file that do not exist already, is it possible the directory of the file path you used was not existing?
If that is the case what is the expected behavior, do we create a error log message or try to create the directory before writing the file?

fixed. closing.

authors: davidwabnitz dsander


text:
addIndicator and Kotlin

Hi, if you try to use MaterialScrollBar with Kotlin and try to call method
addIndicator(indicator, false) or other DragScrollBar methods you got an error:
java.lang.IllegalAccessError: Illegal class access: 'com.example.app.BaseListFragment' attempting to access 'com.turingtechnologies.materialscrollbar.Indicator'

A workaround is to wrap those calls using a java class.
public class MaterialScrollBarHack
{
    // Hack to make widget work
    public static void Helper(Context context, RecyclerView recyclerView)
    {
        DragScrollBar materialScrollBar = new DragScrollBar(context, recyclerView, true);
        materialScrollBar.setHandleColour(ContextCompat.getColor(context, R.color.colorPrimary));
        materialScrollBar.addIndicator(new CustomIndicator(context), true);
    }
}

// Call this in Kotlin
MaterialScrollBarHack.Helper(context, recyclerView)


text:
support for: keys with spaces, escaped values

support for: keys with spaces, escaped values
what are the characters that need escaping?

Done as of cdbee01d1e590c8f654b6ee2211b99a1f94b14ef


text:
how to stop dhcp server?

I'm using this library in my project on linux and I need to run server within goroutine and stop it after, so how can I do this?
here is a sample of my code, also I'm trying to setup server on a virtual interface here
pc, _ := conn.NewUDP4BoundListener(ifName, ":67")
go func() {
  if err := dhcp4.Serve(pc, handler); err != nil {
	  panic(err)
  }

}()

Change the panic to return/break and close pc when ready is one way.

wow thank you so much

authors: Gictorbit krolaw


text:
🛑 Assets Impulso is down

In a4e3a1f, Assets Impulso (https://branded.krown.one/impulso/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Assets Impulso is back up in c8b6adf.

repo: krown-one/status


text:
fix failed test cases and add changes to run using openshift-tuning namespace

This PR fixes issue #81
Signed-off-by: Saad Khan saakhan@redhat.com

Need to fix conflicts

Pls fix the conflicts

authors: dinogun khansaad

repo: kruize/hpo


text:
Fix select highlight after & operator

The error reported in #149 was in the line-continuation-operator pattern that was missing from both name-list and attribute-list scopes used in declaration statements.
Before this change:

After this change:

This does not fix #149. Here's a snippet which will cause this to fail:
      namelist /inputlist/
     $     Nprotons, Nneutrons, Nshell, fmass, hbomeg,
     $     Nmin, deltaN, Nmax, WTmax, TwoMj, parity, Hrank, NormOrd,
     $     TBMEfile, Trelfile, Hrelfile, Vnnfile,
     $     strTrel, lamHcm, strVnn,
     $     neivals, maxits, startit, selectpiv, tol,
     $     numTBops, obdme, IFLAG_mode, blksize,
     $     uniqueID, orbitalfile, partitioningfile,
     $     DistrDiagTiles, reduce_solver_threads,
     $     checkpoint, write_matrix, observables_only,
     $     initvec_index, initvec_smwffilename
!
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!
!     Reading input file
      inquire(file='mfdn.input', exist=input)
      if (input) then
         open(unit=inputunit, file='mfdn.input',
     $        status='old', action='read')
         read(inputunit, nml=inputlist)
         close(unit=inputunit, status='keep')
      else
         print*, 'No inputfile...'
         ! call cancelall(000)
         stop
      endif

Note, this is fixed-format Fortran, not free-format.

@kc9jud, Thanks for the feedback ... I think that's because you have a different line-continuation-operator . I was not especting that $ there.

Meh ... More stuff wrong, besides that $ over there that is confusing me, the namelist is just a keyword highlighted it does not behave like a declaration statement

@kc9jud, I'm not able to transform the namelist in a declaration that will recognize and highlight the \groups\  and variables. It should not be difficult tho, I just don't understand the scopes enough to make it.
Therefore I separated the select-construct scope into 3 different ones, restricting them a little bit more (similar to what you did before).
This will solve the select in variables but will not do anything about that $ or the namelist scope.
Could you check it now?

Is this good enough to be merged @pedro-ricardo? Even it it not cover all the cases it is already an improvement. Or I am missing something?

Yep, That's basicaly it :+1:

authors: kc9jud krvajal pedro-ricardo

repo: krvajal/vscode-fortran-support


text:
Prepare for initial (alpha) release on Terraform Registry

In order to release a provider on the Terraform Registry, some preparation is needed, full details are available here.
Below is a summary of the steps outstanding:

[x] Add a license. (#20)
[x] Provider Documentation, can be generated with the help of tfplugindocs. Fleshing out some example files will probably be needed. (#17)
[ ] Setup automatic release generation with goreleaser and GitHub Actions. This will also include generating a GPG signing key, which will be used to sign all released binaries. This is required as the terraform CLI tool will validate all binaries against the public key published on the Terraform Registry. (#18)
[ ] Add information to the README. (#16)
[ ] Publish the provider on the Terraform Registry. (#19)

I think the very first version should probably be a pre-release version, e.g. v0.0.1-alpha.0. As detailed here pre-releases are supported, but need to explicitly specified as the desired provider version. If no version is specified, Terraform will only pick the latest stable version available for a provider.

I'm closing this issue in favor of tracking progress in the Public Release (alpha) milestone instead.

repo: krystal/terraform-provider-katapult


text:
GBMDiscretizer fails when estimator is XGBClassifier

I get the following error with xgboost==1.6.2 and scikit-gbm==0.2.1:
    109 def xgboost_trees_to_dataframe(obj):
--> 110     trees_df = obj.get_booster().trees_to_dataframe()
    112     # Rename columns
    113     trees_df = \
    114         trees_df.rename(columns={
    115             'Tree'     : 'tree_index',
   (...)
    124             'Category' : 'category'
    125         })

AttributeError: '_XgboostWrapper' object has no attribute 'get_booster'

Reproducible example:
from sklearn.datasets import load_iris
from sklearn.model_selection import train_test_split
import numpy as np

iris = load_iris()
data = pd.DataFrame(
    data= np.c_[iris['data'], iris['target']],
    columns= iris['feature_names'] + ['target']
)
data.columns = data.columns.str[:-5]
data.columns = data.columns.str.replace(' ', '_')

X, y = data.iloc[:, :4], data.iloc[:, 4:]
X_train, X_test, y_train, y_test = \
    train_test_split(X, y, test_size=0.3, random_state=0)
X_cols = X.columns.tolist()

gbm_discretizer = GBMDiscretizer(XGBClassifier(verbose=-1), X_cols, one_hot=False)
gbm_discretizer.fit_transform(X_train, y_train)

I am facing the same issue as @kyle-clara. Also, if I try to use GradientBoostingRegressor\Classifier instead of XGBoost I get a different error:

I believe this library was written for an earlier version of XGBoost and sklearn as I found multiple posts on StackOverflow mentioning same errors with get_booster attr missing which were resolved by downgrading xgboost.
@krzjoa Could you please specify versions of sklearn and xgboost libraries this package was written for and (preferably) update the code to work with the latest versions? Thank you

repo: krzjoa/scikit-gbm


text:
播放http直接偶尔崩掉

.KSYMediaPlayer: Error (-10011,206)

这个是报错了，崩溃应该有堆栈信息吧？

authors: FirmianaRain shenshaoyan


text:
hack,test/e2e: Default to the 4.6 operator registry image CI imagestream.

This change doesn't affect much except for providing a sensible default value for running the e2e suite locally and don't need to provide an up-to-date operator manifest bundle image. We can default the operator's registry image that gets tagged into the ocp/4.6 imagestream by CI.

/retest

Okay, looks like these recent e2e failures are due to a recent rebase with the openshift apiserver. Just going to force-merge this change for now as it's purely for test debugging.

repo: kube-reporting/metering-operator


text:
Proxy Subresource

With #127 nearing completion, there's actually only one special subresource left; proxy (which has its own verb).
Described in kubernetes.io/../proxies it describes a method to connect to services from outside the cluster using the proxy verb. See also a TL;DR stackoverflow answer.
The allowed http methods are the same for Pod, Service and Node. See e.g. Node methods):

POST /api/v1/nodes/{name}/proxy
POST /api/v1/nodes/{name}/proxy/{path}
DELETE /api/v1/nodes/{name}/proxy
DELETE /api/v1/nodes/{name}/proxy/{path}
GET /api/v1/nodes/{name}/proxy
GET /api/v1/nodes/{name}/proxy/{path}
HEAD /api/v1/nodes/{name}/proxy
HEAD /api/v1/nodes/{name}/proxy/{path}
PUT /api/v1/nodes/{name}/proxy
PUT /api/v1/nodes/{name}/proxy/{path}

e.g. we, at the very least, need some Api methods protected by a Proxy trait and implemented by:
impl Proxy for k8s_openapi::api::core::v1::Pod {}
impl Proxy for k8s_openapi::api::core::v1::Service {}
impl Proxy for k8s_openapi::api::core::v1::Node {}

impl<K> Api<K>
where
    K: DeserializeOwned + Proxy
{
    //todo!("methods from above list")
}

in kube-client.
Prior Art
OpenEBS Mayastor control plane has a forwarding crate using the /proxy verb:

https://github.com/openebs/mayastor-control-plane/blob/92af9014102beeb2996f43ef1f0e3ba182ebe0f9/k8s/forward/src/http_forward.rs#L253-L255
original upstream PR: https://github.com/openebs/mayastor-control-plane/pull/288

Previous proxy-like work; how to setup kube-client with a custom proxy connector, but that's more for supporting config.proxy_url.
Discussion
Not sure how to approach this in kube. My gut feel is something similar to portforward.rs, but we have also left client proxying up to custom clients (via user-supplied connectors).
There's ultimately no discussion about it currently so putting in an issue to gauge interest (plus i want to stash some tabs). If people have thoughts or ideas about this, comments are welcome.

i was trying to figure out how to tackle this issue and as you said portforward seemed to be the closest thing, so i took a look there. i think we might need to go about this in two ways (note that this is very high level):

introduce a trait Proxy and implement it for Pod, Service and Node. then have something like pods.proxy(req) where req will be the HTTP request to forward to the pod. this will be helpful for consumers who just want to access a single resource on-demand.
to aid users who want to build a full blown proxying service similar to kubectl proxy, we should introduce a HTTPForwarder, which would accept a request, and using the URL figure out the target. it'd then forward the request to the intended target. this way consumers won't have to fetch the actual pods/services/nodes (as is required in the first way), avoiding unnecessary API calls.

repo: kube-rs/kube


text:
Optimize e2e scripts

fix bug
add
fix shell

What type of PR is this?

/kind feature
What this PR does / why we need it:

Optimize e2e scripts
Unified e2e running entry

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/lgtm

/approve

repo: kubean-io/kubean


text:
Segfault on policy handling

Signed-off-by: Eswar Rajan Subramanian eswar@accuknox.com

Kindly ignore this commit as I am facing similar issue in other scenario.

Codecov Report

Merging #644 (19928f5) into main (86954a6) will decrease coverage by 0.02%.
The diff coverage is 14.28%.

@@            Coverage Diff             @@
##             main     #644      +/-   ##
==========================================
- Coverage   43.52%   43.49%   -0.03%
==========================================
  Files          23       23
  Lines        8504     8507       +3
==========================================
- Hits         3701     3700       -1
- Misses       4354     4358       +4
  Partials      449      449

Impacted Files
Coverage Δ

KubeArmor/enforcer/appArmorProfile.go
38.32% <14.28%> (-0.02%)
:arrow_down:

KubeArmor/monitor/systemMonitor.go
49.34% <0.00%> (-0.38%)
:arrow_down:

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 0a98271...19928f5. Read the comment docs.

authors: codecov-commenter seswarrajan

repo: kubearmor/KubeArmor


text:
fix the online locaregistry in the cluster details only displays 'online'

Signed-off-by: liuying liu.ying@99cloud.net

What type of PR is this?
/kind bug

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #73
Special notes for reviewers:

Does this PR introduced a user-facing change?

Additional documentation, usage docs, etc.:

/lgtm
/approve

repo: kubeclipper-labs/console


text:
kubeedge-counter-demo:Kubeedge -pi-counter is always pending

I want to run 'kubeedge-counter-demo', but 'Kubeedge -pi-counter' is pending in the end

That means the pod is not scheduled. Did you edit the deviceInstance yaml file with your edge node name?

repo: kubeedge/examples


text:
Failed to create registry

Failed to create cluster with main branch (1.10.8):

Checking logs in argocd, we noticed a failure in the registry:

rpc error: code = Unknown desc = Manifest generation error (cached): rpc error: code = FailedPrecondition desc = Failed to unmarshal "aws-auth.yaml": <nil>

#856

repo: kubefirst/kubefirst


text:
use pytorchjob with yolov5 code in single pod

in yolov5 code, when i use pytorchjob to run non-distributed training like that
apiVersion: "kubeflow.org/v1"
kind: PyTorchJob
metadata:
  name: test
spec:
  pytorchReplicaSpecs:
    Master:
      replicas: 1
      restartPolicy: Never
      template:
        spec:
          containers:
            - name: pytorch
              image: 10.50.120.102/daocloud/dap_pytorch_1.7.1:cpu
              imagePullPolicy: IfNotPresent
              env:
              - name: LOCAL_RANK
                value: "-1"
              - name: RANK
                value: "-1"
              - name: WORLD_SIZE
                value: "1"
              command:
                - python3
                - train.py
              resources:
                limits:
                  cpu: 1
                  memory: 2Gi
              volumeMounts:
              - name: code
                mountPath: /code
              - name: result
                mountPath: /result
              - name: dataset
                mountPath: /dataset
          volumes:
          - name: code
            hostPath:
              path: /nfs/dap/test/code
          - name: result
            hostPath:
              path: /nfs/dap/test/result
          - name: dataset
            hostPath:
              path: /nfs/dap/test/dataset

it cause error
Using 0 dataloader workers
Logging results to /result
Starting training for 50 epochs...
Traceback (most recent call last):
  File "train.py", line 698, in <module>
    main(opt)
  File "train.py", line 591, in main
    train(opt.hyp, opt, device, callbacks)
  File "train.py", line 344, in train
    train_loader.sampler.set_epoch(epoch)
AttributeError: 'SequentialSampler' object has no attribute 'set_epoch'

i guess the reason is  env variables RANK not right, seems operator will set the RANK=0
https://github.com/ultralytics/yolov5/blob/b57abb17f2525ce60145038c751a7d8c614e6953/train.py#L59-L62

@gaocegege

Maybe you could follow the guide here https://github.com/ultralytics/yolov5/issues/475 and use PyTorch 1.9.x+ DDP to train the model.

authors: gaocegege goofy-z

repo: kubeflow/training-operator