[ISSUE] Service does not define any selectors

Describe the bug
Right now, svc-timescaledb.yaml is not defining any selectors. Hence, the resulting timescaledb service will not select any pods to forward the request to, and will not work. I suggest to add the following lines to the Service.
  selector:
    app: {{ template "timescaledb.fullname" . }}
    cluster-name: {{ template "clusterName" . }}

I also suggest changing the name of the service to release-name-timescaledb to avoid conflicts with other charts in the same release that also setup a service using only the release name.
# This file and its contents are licensed under the Apache License 2.0.
# Please see the included NOTICE for copyright information and LICENSE for a copy of the license.

apiVersion: v1
kind: Service
metadata:
  name: {{ template "clusterName" . }}
  labels:
    app: {{ template "timescaledb.fullname" . }}
    chart: {{ template "timescaledb.chart" . }}
    release: {{ .Release.Name }}
    heritage: {{ .Release.Service }}
    cluster-name: {{ template "clusterName" . }}
    role: master
...

becomes
# This file and its contents are licensed under the Apache License 2.0.
# Please see the included NOTICE for copyright information and LICENSE for a copy of the license.

apiVersion: v1
kind: Service
metadata:
  name: {{ template "timescaledb.fullname" . }}
  labels:
    app: {{ template "timescaledb.fullname" . }}
    chart: {{ template "timescaledb.chart" . }}
    release: {{ .Release.Name }}
    heritage: {{ .Release.Service }}
    cluster-name: {{ template "timescaledb.fullname" . }}
    role: master
...

(setting name and app field to {{ template "timescaledb.fullname" . }})
To Reproduce
On minikube, run:
helm install prototype timescaledb/timescaledb-single

Then use the minikube dashboard or kubectl get endpoints prototype-timescaledb to verify that no pods match the service.
Expected behavior
Three pods should match the service and be listed under pods. (Command output after applying my fixes:)
PS C:\Users\Tobias Jacob\git\Prototype> kubectl get endpoints prototype-timescaledb
NAME                    ENDPOINTS                                            AGE
prototype-timescaledb   <IP_ADDRESS>:5432,<IP_ADDRESS>:5432,<IP_ADDRESS>:5432   8m31s

Deployment

What is in your values.yaml: Standard values
What version of the Chart are you using?: 0.13.1
What is your Kubernetes Environment (for exampe: GKE, EKS, minikube, microk8s): minikube

Deployment
Please share some details of what is in your Kubernetes environment, for example:
kubectl get all,secret,configmap,endpoints,pvc -L role -l release=$RELEASE

PS C:\Users\Tobias Jacob\git\Prototype> kubectl get all,secret,configmap,endpoints,pvc -L role -l release=prototype
NAME                                                     READY   STATUS    RESTARTS        AGE    ROLE
pod/prototype-prometheus-alertmanager-5876cff56d-xvq7r   2/2     Running   2 (4h21m ago)   17h
pod/prototype-prometheus-node-exporter-mfg44             1/1     Running   1 (4h21m ago)   17h
pod/prototype-prometheus-pushgateway-868698c6bd-l295s    1/1     Running   1 (4h21m ago)   17h
pod/prototype-prometheus-server-75b4c68767-5wm7z         2/2     Running   2 (4h21m ago)   17h
pod/prototype-timescaledb-0                              2/2     Running   0               170m   master
pod/prototype-timescaledb-1                              2/2     Running   0               170m   replica
pod/prototype-timescaledb-2                              2/2     Running   0               169m   replica

NAME                                         TYPE        CLUSTER-IP       EXTERNAL-IP   PORT(S)    AGE    ROLE
service/prototype-config                     ClusterIP   None             <none>        8008/TCP   170m
service/prototype-prometheus                 ClusterIP   <IP_ADDRESS>     <none>        9187/TCP   170m
service/prototype-prometheus-alertmanager    ClusterIP   <IP_ADDRESS>   <none>        80/TCP     17h
service/prototype-prometheus-node-exporter   ClusterIP   <IP_ADDRESS>    <none>        9100/TCP   17h
service/prototype-prometheus-pushgateway     ClusterIP   <IP_ADDRESS>   <none>        9091/TCP   17h
service/prototype-prometheus-server          ClusterIP   <IP_ADDRESS>   <none>        80/TCP     17h
service/prototype-replica                    ClusterIP   <IP_ADDRESS>    <none>        5432/TCP   170m   replica
service/prototype-timescaledb                ClusterIP   <IP_ADDRESS>   <none>        5432/TCP   124m   master

NAME                                                DESIRED   CURRENT   READY   UP-TO-DATE   AVAILABLE   NODE SELECTOR   AGE   ROLE
daemonset.apps/prototype-prometheus-node-exporter   1         1         1       1            1           <none>          17h

NAME                                                READY   UP-TO-DATE   AVAILABLE   AGE   ROLE
deployment.apps/prototype-prometheus-alertmanager   1/1     1            1           17h
deployment.apps/prototype-prometheus-pushgateway    1/1     1            1           17h
deployment.apps/prototype-prometheus-server         1/1     1            1           17h

NAME                                                           DESIRED   CURRENT   READY   AGE   ROLE
replicaset.apps/prototype-prometheus-alertmanager-5876cff56d   1         1         1       17h
replicaset.apps/prototype-prometheus-pushgateway-868698c6bd    1         1         1       17h
replicaset.apps/prototype-prometheus-server-75b4c68767         1         1         1       17h

NAME                                     READY   AGE    ROLE
statefulset.apps/prototype-timescaledb   3/3     170m

NAME                                          DATA   AGE    ROLE
configmap/prototype-prometheus-alertmanager   2      17h
configmap/prototype-prometheus-server         6      17h
configmap/prototype-timescaledb-patroni       1      170m
configmap/prototype-timescaledb-pgbackrest    1      170m
configmap/prototype-timescaledb-scripts       8      170m

NAME                                           ENDPOINTS                                            AGE     ROLE
endpoints/prototype                            <IP_ADDRESS>:5432                                     124m
endpoints/prototype-config                     <IP_ADDRESS>:8008,<IP_ADDRESS>:8008,<IP_ADDRESS>:8008   170m
endpoints/prototype-prometheus                 <IP_ADDRESS>:9187,<IP_ADDRESS>:9187,<IP_ADDRESS>:9187   170m
endpoints/prototype-prometheus-alertmanager    <IP_ADDRESS>:9093                                     17h
endpoints/prototype-prometheus-node-exporter   <IP_ADDRESS>:9100                                    17h
endpoints/prototype-prometheus-pushgateway     <IP_ADDRESS>:9091                                      17h
endpoints/prototype-prometheus-server          <IP_ADDRESS>:9090                                     17h
endpoints/prototype-replica                    <IP_ADDRESS>:5432,<IP_ADDRESS>:5432                    170m    replica
endpoints/prototype-timescaledb                <IP_ADDRESS>:5432,<IP_ADDRESS>:5432,<IP_ADDRESS>:5432   6m50s   master

NAME                                                           STATUS   VOLUME                                     CAPACITY   ACCESS MODES   STORAGECLASS   AGE    ROLE
persistentvolumeclaim/prototype-prometheus-alertmanager        Bound    pvc-96d01889-1eeb-4a20-a93d-11cab2159d21   2Gi        RWO            standard       17h
persistentvolumeclaim/prototype-prometheus-server              Bound    pvc-81120dba-99be-4031-af57-630bc66209e3   8Gi        RWO            standard       17h
persistentvolumeclaim/storage-volume-prototype-timescaledb-0   Bound    pvc-5cf64a4e-28f2-485e-8519-3435bcdec439   2Gi        RWO            standard       18h
persistentvolumeclaim/storage-volume-prototype-timescaledb-1   Bound    pvc-4a0dd9ac-36a2-43b1-a1fd-99e295a6059d   64Gi       RWO            standard       170m
persistentvolumeclaim/storage-volume-prototype-timescaledb-2   Bound    pvc-b6a06c3b-adcb-45f2-822a-d1a0846d8ea0   64Gi       RWO            standard       169m
persistentvolumeclaim/wal-volume-prototype-timescaledb-0       Bound    pvc-78dadd26-6a2b-4211-96db-97d276de56b3   1Gi        RWO            standard       18h
persistentvolumeclaim/wal-volume-prototype-timescaledb-1       Bound    pvc-0f05273a-3749-4547-bb30-36def91212d6   1Gi        RWO            standard       170m
persistentvolumeclaim/wal-volume-prototype-timescaledb-2       Bound    pvc-f8f98486-5318-4a61-acc9-59395d73f86f   1Gi        RWO            standard       169m

Logs
No logs
Additional context
No additional context

Right now, svc-timescaledb.yaml is not defining any selectors.

Patroni manages the Endpoints belonging to the Service directly, the lack of label selectors is by design for this service
See for details the patroni project, or this comment

I also suggest changing the name of the service to

That sounds reasonable, however that would be a breaking change for all users currently using the Chart, so I'm afraid we're not eager in touching that.

All right, thank you a lot! Setting clusterName: release-name-timescaledb allowed me to change the service endpoint.


Add support for renaming an index

Renaming a table without renaming its indexes creates a mismatch. Add support for renaming indexes also.
This PR does not handle renaming foreign keys with ALTER TABLE "people" RENAME CONSTRAINT "persons_team_id_fkey" TO "people_team_id_fkey"; that could be a good next feature.

Thanks for the PR. I think we should try to keep the built-in adapters consistent and either add the command if it exists or raise if not. Would you have any room to do the same for MySQL/SQL Server?

Good point. Yes, I think so.

Funny I just ran into the need for this.

@greg-rychlewski Updated. Note that the SQL Server example is based on their docs, but I've never used SQL Server so this is just my best attempt.

Note that the SQL Server example is based on their docs, but I've never used SQL Server so this is just my best attempt.

Sorry to ask for one more thing :(. But we could make sure it's working adding an integration test similar to the rename table one:
https://github.com/elixir-ecto/ecto_sql/blob/master/integration_test/sql/migration.exs#L616

@greg-rychlewski How does that look? Again I haven't actually run it against SQL Server. I was hoping CI would do that but I only see PostgreSQL integration tests. I'm trying to avoid having to install SQL Server. 😆

Thanks!


Get rid of ensureCorrectCaller

It's a holdover from the JavaScript days.
A few places that will have to be changed are...

[ ] Movement
[ ] Collisions (give a scope to ThingHittr)
[ ] Macros (give a scope to MapsCreator as scope instead of an argument)
[ ] ...more to come!

Sister issue to https://github.com/FullScreenShenanigans/FullScreenMario/issues/349

Closed by #148


Grpc Stream Error: 13

3 zeebe brokers and 2 zeebe gateways have been installed on a k8s platform. To enable external access to the gateway from outside k8s, an ALB ingress has been used.
When connecting to the publicly exposed endpoint getting this error ERROR: Grpc Stream Error: 13 INTERNAL: Received RST_STREAM with code 2 triggered by internal client error: Protocol error
But using zbctl i am able to connect and get the status from that endpoint. Please let me know if anything needs to be changes

after sending the useTLS to true, issue is resolved.


Added never Mutate State

Added Never Mutate State

[X] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

please add reason

Hi @AkshayBabbar,
Do add proper code formatting to your article and please dont just add solution. Explain it too.

Hi @AkshayBabbar,
I think the invalid tag was added by mistake.
However, do add proper code formatting to your article and please don't just add solution. Explain it too.

Hi Can you please check others pull request also since all of them is marked invalid without a reason.

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.

@AkshayBabbar Please make sure any solution you add includes all original seed code, plus any changes necessary for the solution to pass the tests.
Thank you.

Hi @AkshayBabbar,
As there has been no response from your end for a while, we are closing this PR.
Feel free to request this PR to be reopened or create a new PR, if you still wish to make the changes you originally added plus any additional change requests asked of you.  Also, remember to update your fork, so you have the latest version of master in case it has changed significantly.
Happy Coding!


disable feature level publication info for now

We need to figure out how this should really work.  At the moment this is not working anyway because of the thrift spec.

Had to close and reopen the PR to get travis to kick in.

Something weird happened with my fork, need to fix that and resubmit.


[feat] Proposal on web style

Describe the problem
In the process of using it, I found many css default settings that are different from Chromium.
For example, the :focus property outline is not defaulted to none in tauri, which causes some controls like buttons to appear as a dotted line when clicked.
This made it difficult for me to migrate my software from electron to tari (after all, I don't know the difference between the web kernel used by tari and Chromium in terms of css defaults)
Describe the solution you'd like
This will be fixed in subsequent releases
Alternatives considered
No response
Additional context
No response

You can use some css resets like Eric Myer's reset or normalize.css and use tools like auto-prefixer.


Deal with settings and analysers in Mapping and Document.init()

Currently Mappings is being configured by Mapping (Document,put_mapping)
It is also needed to put_settings of each field analyser.
for EACH field:
   analysers.append(field_analyser)

es.indices.put_settings(analysers=....)

#17


binder v2 for statements

#705 introduced a new binder for egg expressions. But it only covered SELECT queries. We need to do the same for the rest statements.

/assginme

@eliasyaoyc do you have any overview of how it would goes in mind? If not, I think it's better for Runji and me to move forward on this issue.


feat: adopt 4.2.0, Drop 4.1.0

:tada: This PR is included in version 3.0.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Ignore the notch

On the built in Retina display, is there a way to ignore the notch and expand to the full height of the screen?
Currently I have gaps.outer.top = [{ monitor."Studio Display" = 8 }, 6] and the 6 is counted from the notch's bottom.
A negative value doesn't seem to work and it the same as 0

macOS doesn't allow to put arbitrary windows above the notch's bottom line

got it, let's close this then

@nikitabobko: Alacritty is able to ignore the notch with the SimpleFullscreen option (documentation). Is that something that AeroSpace can mimic or does it have to be implemented at the app level?


Move HRIR files to gstatic server

Eventually HRIR files will be served via: https://www.gstatic.com/external_hosted

With #72, this issue is now obsolete.


Trying to access process.env

Describe the bug
Hey there,
using diary in Frontend environments bundled by webpack, vite or other without defining the process.env variable the library crashes.
I've identified that it tries to access it in the index.ts file when it thinks it's in a node porcess. Actually it should correctly identify that it's getting used in a browser.
Best
Alex
To reproduce
Use diary in a browser project which doesn't use process.env.
Expected behavior
It should not crash - by detecting it's running in a browser instead of a node.js process

I've found out that the __TARGET__ var is used for your build step and the problem is in the exports map of the package.json.
"exports": {
		".": {
			"browser": {
				"types": "./diary/browser/index.d.ts",
				"import": "./diary/browser/index.mjs",
				"require": "./diary/browser/index.js"
			},
			"types": "./diary/node/index.d.ts",
			"import": "./diary/node/index.mjs", // <- the browser selects this one
			"require": "./diary/node/index.js"
		},
		...
	}

I'm not sure but it seems that the bundle tools like vite don't pick the correct import here. Do you have any idea why that could be?

Hi @BraunreutherA mega sorry for leaving this for like 6months!!! 😢
Wondering how you went? Not sure what the defaults are for vite but looks like if you set the config build.target: 'browser' it sets the correct conditionals.
👉 https://stackblitz.com/edit/vitejs-vite-kewgxu


fixed warning: ... range patterns are deprecated to ..=

i'm beginner, but when i tried match section, i found warning: ... range patterns are deprecated. and then, the rust help show ..= for solved it. i used rustc 1.40.0 (73528e339 2019-12-16) for tired match section. thank you.

Hi @mandasialearn,
Thanks for reporting this issue. I was also working on some overall update these days. But, would like to push your PR. Can you update the example in https://github.com/mandasialearn/site/blob/patch-1/source/docs/e2.panicking.md#unreachable as well?
Thanks

I'm sorry for misplaced report. My pleasure.

Merged! Thanks for the PR! Feel free to report issues and make PRs, if you found any issues.


Need ability to add new jar to class path on nodes

Hi,
selenium grid extra provides ability to add a jar file to path as below on hub.
"hub_additional_classpath": [
"./extension-proxy-0.6.jar"
]
Similar ability is not provided at node. Can you pls add something like below to add a jar file on nodes aswell?
"node_additional_classpath": [
"./all-node-extensions-0.6.jar"
]
Thanks
Shankar

Good idea @shankarkc .

https://github.com/groupon/Selenium-Grid-Extras/pull/234 . This will get merged in before next release (maybe next week).

Thanks Shawn. It will be useful to run sikuli extension client on grid. A
nice to have feature as part of our automation framework.
On Mon, Jun 27, 2016 at 10:09 AM, Shawn McCarthy<EMAIL_ADDRESS>wrote:

#234 https://github.com/groupon/Selenium-Grid-Extras/pull/234 . This
will get merged in before next release (maybe next week).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/groupon/Selenium-Grid-Extras/issues/228#issuecomment-228652649,
or mute the thread
https://github.com/notifications/unsubscribe/AEb7V6PHLpLYbHOjOjP2RK2iylbqhYyDks5qP1QfgaJpZM4IvoDC
.

hi Shawn referring to your change line number 193 in below file
SeleniumGridExtras/src/main/java/com/groupon/seleniumgridextras/grid/GridStarter.java
You should add this to class path only if this is node right ?
As of now it get added even for hubs. We will have this jar only on nodes
and we need to add it to class path ONLY IF THIS IS a NODE.
On Mon, Jun 27, 2016 at 10:09 AM, Shawn McCarthy<EMAIL_ADDRESS>wrote:

#234 https://github.com/groupon/Selenium-Grid-Extras/pull/234 . This
will get merged in before next release (maybe next week).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/groupon/Selenium-Grid-Extras/issues/228#issuecomment-228652649,
or mute the thread
https://github.com/notifications/unsubscribe/AEb7V6PHLpLYbHOjOjP2RK2iylbqhYyDks5qP1QfgaJpZM4IvoDC
.

My bad. I just reffered the code snippet . Your code is right...You have
modified getWebNodeStartCommand method. Sorry for the confusion.
On Mon, Jun 27, 2016 at 11:44 AM, shankar KC<EMAIL_ADDRESS>wrote:

hi Shawn referring to your change line number 193 in below file
SeleniumGridExtras/src/main/java/com/groupon/seleniumgridextras/grid/GridStarter.java
You should add this to class path only if this is node right ?
As of now it get added even for hubs. We will have this jar only on nodes
and we need to add it to class path ONLY IF THIS IS a NODE.
On Mon, Jun 27, 2016 at 10:09 AM, Shawn McCarthy<EMAIL_ADDRESS>
wrote:

#234 https://github.com/groupon/Selenium-Grid-Extras/pull/234 . This
will get merged in before next release (maybe next week).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/groupon/Selenium-Grid-Extras/issues/228#issuecomment-228652649,
or mute the thread
https://github.com/notifications/unsubscribe/AEb7V6PHLpLYbHOjOjP2RK2iylbqhYyDks5qP1QfgaJpZM4IvoDC
.

@shankarkc Can you verify this works for you now, using Selenium-Grid-Extras 1.12.3 ?

This works great..You can close this....I am able to run sikuli now with this change


Fix: Remove unused async from getAuthorizatinoUrl and other minor fixes

getAuthorizationUrl() should not return a promise because there are no asynchronous operations under the hood, it only structures the URL object.

I extracted the types from the generics in provider() and exported them to the user. This may be useful for some people who want to create their own provider.

I removed the unnecessary nested function getHeadersValue(), which doesn't really make sense.

If I have forgotten or misunderstood something, let me know. 🤍

While not implemented yet, getAuthorizationUrl is asynchronous to make sure it works with OAuth 1.0(a) as well, since those require an extra API call when generating a URL.

I'm still torn on exposing OAuthProviderConfig since it's only used inside provider, and I don't think it's necessary to expose OAuthConfig since it's just there for consistency sake between the "official" providers.

Agree that it's unnecessary!

It'd be much easier to keep track of if these were separated into their own PRs as well


How to force reconnection on redis timeout?

We are using swoole with hundreds of workers, sometime when workers are idling for too long, redis go away. than a request come and redis do not reconnect to DB. It just throw error on GET method. So how we can keep alive the connection or force the service to reconnect?

@Webonaute hi, we resolving same problem, have you reasonable solutions for this? thank you!

Same problem, any one have solved this?


chore(master): release 0.11.0-alpha

:robot: I have created a release beep boop
0.11.0-alpha (2024-01-14)
Features

新增 (8fc12aa)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/mafqla/yalis-blog/releases/tag/v0.11.0-alpha :sunflower:


Reference to the wrong section

In the sentence "We will explain in section 3.3.1 in more detail how the referencing of METS.xml files must be implemented if this alternative is chosen." (page 13), I don’t think this section number is correct. I’m not sure which section is the correct one.

Reference updated.


local dev keys expired

these are only used for local dev, and the old ones expired

Summary by CodeRabbit

New Features

Updated cryptographic keys and certificates for enhanced security and authentication processes.

Bug Fixes

Resolved potential issues related to outdated keys that could affect secure communications.

Chores

Modified key values and configurations to comply with updated security protocols and standards.

Do we need to have this keys in CI?

there is a way to generate them with 2+ year expiration, maybe for local dev it is enough

Do we need to have this keys in CI?

it's not for CI, it's just for local dev. you could have the user generate them on their own, but then you also have to have them build the network.toml file too. local dev doesn't need to be secure so this makes setup easier.

there is a way to generate them with 2+ year expiration, maybe for local dev it is enough
good call out. let me push one with longer expiration


Logo from url

Can you add the option to load a logo from the url and not just from resources? please
Thank you for your work, is very nice this library.

You can load logo drawable by yourself, for example using Glide and create custom DrawableSource. Is it public interface.
But don't load logo inside DrawableSource.get method.
It is suspend now, but is runs blocking in QrCodeDrawable and it will not more be suspend in next version

thanks a lot!


Secure Decentralized finance 🔐

@Munzalimuazuuk  Please fork this repo and create a PR for your project for submission

Thank you


Prettier

⚠ Do not merge yet ⚠
We will wait until all of the documentation PRs from the nf-core hackathon are merged before handling this PR. That way, lots of different contributors don't all have to reformat their docs mid-way through their PR.

Tried running prettier again locally to see if my change would get around the bug but can't get it to run 😕
Did npm install -g prettier @prettier/plugin-php and then prettier --write . but it makes no changes - only loads of warnings:
public_html/test.md 15ms
[warn] Ignored unknown option { braceStyle: "1tbs" }.
[warn] Ignored unknown option { phpVersion: "7.4" }.

..and seems to only be checking the markdown files 🤔  Not sure what I'm doing wrong here..

Ah, I think that's the last thing, we need to document linting. For whatever reason the global install with the php-plugin doesn't work for me, but using it in an npm script does so run npm run lint @ewels!

ok, I just merged upstream and accepted all incoming with checking much, then ran prettier afterwards. Hope that's ok. I'm assuming that you haven't changed much code in this PR except for npm / stuff related directly to linting..

assuming that you haven't changed much code
That was the goal, at least!


CLN can we not ignore E0401 for pylint?

Now that pylint isn't incompatible with the rest of the dependencies, is it OK to remove
    "E0401",  # import error: I run pylint in a separate environment so it's OK

from pyproject.toml ?

Try removing it and run pre-commit run -a, if it passes it should be fine.

If this rule is removed from the pylint conf, we would get import error. pytest and other import dependencies in the source/test files won't be available to the pylint module since this would be run in an isolated environment by the pre-commit.
To fix this issue we have two options.

We need to make all our source and test dependencies available to pylint during pre-commit check using the additional_dependencies key .pre-commit-config.yaml for pylint would look like below. But this rule would keep failing as and when we take dependency on some new other 3rd party package.

  - repo: https://github.com/pycqa/pylint
    rev: pylint-2.6.0
    hooks:
      - id: pylint
        exclude: "docs"
        additional_dependencies:
          - "pytest"

Alternatively, we can make pylint as a local hook as suggested by creator the pre-commit. This is a generic change. We can freely add new dependencies and no need to change the pre-commit config everytime.

pre-commit runs pylint from an isolated virtualenv. Many of pylint's checks perform dynamic analysis which will fail there. You may find configuring pylint as a local hook more useful.

I tried making pylint as a local hook with language: system. This will make the pylint picked up from our virtual environment or some other local environment based on PATH variable. When we develop, usually we have the virtual environment activated, so pre-commit will have not issue finding and running pylint. In github actions, we need to install all the dependencies for pylint to work under system flag.

Don't remove the rule E0401 from pylint configuration.

I could not think of any other alternate solution. I will raising PR. If you folks don't like the PR, feel free to close the PR without merging. PR

PR merged, so closing this issue


feat(www): show github stars

Fetch and Display GitHub Star Count in Nav section.

This pull request introduces changes to the githubStarButton.astro component to fetch and display the star count from a GitHub repository
Fixes #46

The changesets should be removed because this doesn't change the CLI and won't result in a new package to NPM.

Sorry for that. I removed the changeset.


Add validation for educator full_name, add names to Student factory as well

Who is this PR for?
developers
What problem does this PR fix?
This came up from https://rollbar.com/somerville-teacher-tool/somerville-teacher-tool/items/140/, a bug where an educator name was nil.
What does this PR do?
Adds a presence validation to the Educator model, updates factory and specs that expected nil names.  Adds a presence validation to the Student model for first and last names, updates factories and specs.
In the process of doing this, I removed the Faker gem and added FakeNames to generate first and last names.  Both factory classes use this.
Checklists

[x] Author improved specs for code in need of better test coverage

hmm, this is stuck somehow.  Two test files fail in ways that make sense in https://travis-ci.org/studentinsights/studentinsights/builds/387824521?utm_source=github_status&utm_medium=notification, but locally when I removed the Faker gem I get errors like:

I18n::InvalidLocaleData:
       can not load translations from /Users/krobinson/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/locale/en.yml: expects it to return a hash, but does not

So what was happening with the translation exceptions was:

I removed the faker gem
That resulted in a version change for i8n, which faker had depended on
ActiveSupport also uses the i8n gem, which happens to call File.open to read in translations
The iep_storer_spec.rb file was mocking File.open so that it always returned a dummy string
So depending on how the spec was run, this might raise the translation exception when ActiveSupport loaded translation files

https://github.com/studentinsights/studentinsights/pull/1755/commits/484978752f16a1728068bb34f2a099659782d25a fixes the mocking.

selfie

This triggered a validation error for NB because of a rename that happened in the upstream SIS.  I fixed that manually, but we need to solve it more generally.  It's on the backlog.
For Somerville there are several of these, also all from name changes.  For now backing out that bit in https://github.com/studentinsights/studentinsights/pull/1774 and then separately we'll need to improve how the import process handles drift like this more generally.


Pod admission errors due to security policies should not uninstall Helm Release

Currently when a Deployment is failing due to PodSecurityPolicy admission error, these errors are not checked. This issue is logged to bubble that error on a "ReplicaSet" FailedCreate condition so the error can be correctly reported back to the user.

Hi @anandpartha. It's unclear what you are suggesting that should be changed in Helm. Can you please provide an example?

ping.

hi @bacongobbler sorry for the delay.

Currently say there is a PodSecurityPolicy (Disallow Privilege Pods) in the k8s cluster HelmOperator is deployed.
HelmRelease deploys a Deployment with Privilege Pods which are rejected by PodSecurityPolicy admission controller
HelmRelease also has the "Wait" configured to wait for resources
** This will be shown as "FailedCreate" condition on the ReplicaSet
HelmRelease will now never come-up wait

I think if someone could take advantage of this - they could just inject PrivilegePods into the cluster with PodSecurityPolicy and just make the helmOperator overwhelmingly busy.
Let me know what you think.

My apologies - when I ask for an example, I mean to ask if you could please provide a set of steps to reproduce the issue. Many of us are unaware of the edge cases surrounding certain Kubernetes object types and we don't pretend to know or have all the answers. :-)
When providing an example, we usually ask for a sample chart or a set of steps to demonstrate the issue. That way we can attempt to reproduce the issue you are explaining. We find that practical examples provide more context about an issue. It also provides us a test case to use against any proposed fixes.
Thanks.

I also noticed you mentioned "HelmOperator". Are you referring to another project? If so can you please provide a link?

Sorry about that, steps to see:

Prerequities:

PodSecurityPolicy admission controller is enabled
Create the following PodSecurityPolicy
apiVersion: policy/v1beta1 kind: PodSecurityPolicy metadata: name: no-priv-pods spec: privileged: false  # Don't allow privileged pods!

Create a HelmRelease with this chart: https://github.com/anandpartha/helm-chart/blob/main/pause-app-0.1.0.tgz

apiVersion: helm.fluxcd.io/v1
kind: HelmRelease
metadata:
  name: pause-app
  namespace: default
spec:
  releaseName: pause-app
  targetNamespace: default
  timeout: 300
  resetValues: false
  wait: true
  chart:
    repository: https://github.com/anandpartha/helm-chart
    name: pause-app
    version: 0.1.0
  values:
    privileged: true
    allowPrivilegeEscalation: true

The HelmRelease would never come-up Ready and HelmOperator will continue to keep retrying every 300seconds.

Okay. Since you're using a third-party project (HelmOperator), can you please file a ticket there.
Worth noting from their README:

:warning: Notice
The Helm Operator is in maintenance mode,
while we are investing into the future of GitOps by
re-architecting our tooling from the ground up.

If you can reproduce this issue with vanilla Helm, please provide the relevant steps necessary to reproduce the issue, including which version was used and what steps were necessary to reproduce the issue.
Thanks.

Closing as something that falls out of our support scope. We do not provide support for third-party projects. As such please open a ticket with the maintainers of the HelmOperator project. Thanks.


Websockets keep disconnecting

1. What version of Caddy are you running (caddy -version)?
0.9.5
2. What are you trying to do?
 What is your entire Caddyfile?
http://chat.myserver.io {
        redir 301 {
                / https://{host}{uri}
        }
}

https://chat.myserver.io {
        proxy / <IP_ADDRESS>:3000 {
                transparent
                websocket
        }
}

4.
Command:
sudo sh -c "ulimit -n 65535 && sudo /usr/local/bin/caddy -agree -log=stdout -conf=/etc/caddy/Caddyfile"
OS:
$ cat /etc/issue
Ubuntu 14.04.5 LTS \n \l

5. Please paste any relevant HTTP request(s) here.
Was seeing this output frequently:
2017/04/04 13:55:44 http2: server: error reading preface from client <IP_ADDRESS>:7630:
read tcp <myserverip>:443-><IP_ADDRESS>:7630: i/o timeout
2017/04/04 13:55:44 http: TLS handshake error from <IP_ADDRESS>:7631: read tcp <myserverip>:443-><IP_ADDRESS>:7631: i/o timeout

6. What did you expect to see?
7. What did you see instead (give full error messages and/or log)?
Websockets would disconnect about every 3-4 seconds indefinetely.  Not matter how many times I restarted it.
8. How can someone who is starting from scratch reproduce the bug as minimally as possible?
My setup is: caddy with the above config which reverse proxy to a docker container running rocket.chat.
docker run -d --name db mongo
docker run -d --link db:db -e MONGO_URL=mongodb://db:27017/rocketchat -e ROOT_URL=https://chat.myserver.io --name rocketchat -p 3000:3000 rocketchat/rocket.chat:0.54.2

You'll see the connection banner flash every few seconds, if you open the dev console on browser you'll see the socket drop and come back every few seconds.
If I downgrade to 0.9.4 it works fine.

Hello @geekgonecrazy,
thank you for your report!
You need to extend the default timeouts or disable them
timeouts none
It is really important to read our release notes (https://github.com/mholt/caddy/releases/tag/v0.9.5)

@elcore ah I remember seeing this now.  Might want to consider adding to notes that this effects websockets too.  I realize its just an upgraded http request, but still would help make this clearer.
Thanks for the quick response :+1:

Thank you very much @geekgonecrazy!!!

Might want to consider adding to notes that this effects websockets too.

"Because timeouts apply to an entire HTTP server which may serve multiple sites defined in your Caddyfile, the timeout values for each site will be reduced to their minimum values (with 0, or disabled, being the lowest) across all sites that share that server."
https://caddyserver.com/docs/timeouts

@geekgonecrazy The next release of Caddy will not have default timeouts.

@elcore thanks for the link to docs.
I definitely understand the need for timeouts.  Waiting around for too long is just wasting resources. No complaints for helping me save resources
I went ahead and disabled timeouts on mine.
But.. Per defaults I'm still unsure which would be effecting my websockets.
Read: 10s - this being a websocket is it expecting to handle all frames regardless in this amount of time before hanging up?  Since the request is upgraded and happens in less then this.. I would assume does not apply.
Read Header: 10s - this one I can't see really effecting websockets much.
Write: 20s - again is this 20 seconds to write all frames?
Idle: 2m - this is the one after being reminded of this change(I saw a tweet when it was being made :+1:) I assumed would effect websockets. But it never waited for 2 minutes.  It has a ping/pong frequency of like 10-15seconds.  So it was never idle this long.
@mholt I assume this is because of tickets like mine.  Is there a ticket you have discussed remove vs improve defaults?  Would be curious as to the decision.
I know most other web server / proxies have defaults.  So this definitely isn't new to have them.  Usually we only hit them when we are doing very edge case scenarios like servers that take too long to respond etc.
Thanks again for the responses guys!  We love caddy over on the Rocket.Chat project.  Our users love it too.

@geekgonecrazy Glad to hear, thank you. :)
Default timeouts were removed in part because, yes, issues of confusion (partly my fault for documentation fails, but it was very clear in the release notes) - despite the security benefits. Go doesn't let us customize timeouts per-request (yet, there are issues about it so maybe for 1.9 or 1.10) and since we lose that kind of flexibility, maybe best to turn them off by default. At least for now.
I didn't realize Rocket.Chat had docs for using it with Caddy. That's cool. Is there a reason this section is in your docs? https://rocket.chat/docs/installation/manual-installation/ubuntu/snaps/autossl#redirecting-http-to-https - Caddy does this for you automatically, no need to manually create a redirect (unless you are redirecting differently than Caddy does).

Go doesn't let us customize timeouts per-request

Ah!  Ok that makes sense.  Only done basic rest API's in go.  So was unaware of that limitation.

I didn't realize Rocket.Chat had docs for using it with Caddy. That's cool.
We use it in our snaps trying to make it simple to get full setup going.  So simplicity of caddy fit right in
I believe we also have directions for manual installs as well.

Caddy does this for you automatically, no need to manually create a redirect (unless you are redirecting differently than Caddy does).

I think that was oversight.  Came from nginx where such verbose syntax is required.  :grin: I'll update and simplify!  Thanks!


Exception

except Exception as e:

instead of

except Exception, e:

in init file

Is except Exception, e: valid for any version of Python?
#Abandonware

Same issue here.

Yes version 2 https://docs.python.org/2/tutorial/errors.html#defining-clean-up-actions

I've built a simple fix since no response from the owner:
https://github.com/davebulaval/careerjet-api/tree/3.2
PyPi: https://pypi.org/project/careerjet-api/3.2/


let the browser render the DOM before resizing the chart

This commit uses setTimeout() in order to wait until the browser has rendered the DOM. This fixes an issue in my application where the chart isn't rendered on page load.

Do you have any idea why the test is failing? It also fails with the current master (b4445bbddf402df9dde7380b1e77e557bd0fbc86).

Perhaps a dependency has been updated?

The current version seems to be passing: https://travis-ci.org/carlcraig/tc-angular-chartjs/builds although I will check if there is some issue with packages for tests.
If you could also look into the tests to see if we can work out where they are breaking it would be great.
You can run grunt watch locally to run all the tests.
Other than the tests failing, it looks like a good fix.

Theres definitely something going wrong with the builds, as I have re-run the latest commit and it now fails, even with all tests passing:

I don't know what's wrong here, can you investigate the issue? Thank you!

the new 2.1.X version is using chart.js 2.X ... if you still have this problem with updated libraries, please re-open this.
going to close this for now as there are conflicts.


Add descriptions for input parameters if they exist (Complement #164)

After correction of #164 the descriptions for input parameters are not used.
Proposed to add the descriptions if they exist.
Joined proposed code modifications to add these descriptions.
graphql-maven-plugin-logic\src\main\java\com\graphql_java_generator\plugin\DocumentParser.java
graphql-maven-plugin-logic\src\main\resources\templates\client_query_mutation_executor.vm.java
graphql-maven-plugin-logic\src\main\resources\templates\client_subscription_executor.vm.java
proposed-modifications.zip

I'll check that.
If you've done the code modification, can you do a Pull Request on github for that ?
It would be easier for me to check the changes.
Etienne

Released in the 1.18.10 version


Prevent multiple To recipients

Change the mailer interface to prevent leaking of possible hidden email addresses when sending to multiple recipients.

🎺 🤖


[enhancement] Publish FreeFlow on Maven

Publishing a Maven artifact would allow Gradle based builds to quickly and efficiently setup a project that uses this library.

Btw, you can currently use it as a Gradle/Maven dependency from:  https://jitpack.io/#Comcast/FreeFlow/d4bdd5edca

@jitpack-io Thank you very much, that is one of the most useful websites I have seen in a while.


StreamFactoryInterface::createStreamFromFile does NOT throw ValueError on PHP 8

Follow-up #45
fopen behavior changed with PHP 8, but not PSR-17. Function StreamFactoryInterface::createStreamFromFile still NOT throw ValueError, but only RuntimeException or InvalidArgumentException.
This PR revert #45

This PR is correct, and should be merged.

Released in version 0.9.0.


the CameraComponent how to convert screen position to world position

i can't find the api  screenToWorld  in CameraComponent

Most of the time you don't have to do that, because you'll get that information from the gesture events. For example, add the TapCallbacks mixin on the world instead of on the game.
What is your use-case? And I can tell you what the correct way of doing it is :)

I want to drag the characters purchased from the store into the world in the game

Create a point in the world at the beginning of the drag and drop, and when this point collides with an item in the store, copy an item and add it to the world for drag and drop. Is this feasible? ：)

Create a point in the world at the beginning of the drag and drop, and when this point collides with an item in the store, copy an item and add it to the world for drag and drop. Is this feasible? ：)

Depending on how your store works (if it is in the world or on the viewport for example) you might have to do camera.globalToLocal if the drag doesn't start on the world.

When I click on an item, I pass its name to the world and create it for drag and drop in the world. thanks ：）


Commented out some safe and useful apps

Removed entries related to creating recovery media and/or reimaging machine since these may actually be needed if system is not repairable.
Removed some Cyberlink entries for DVD/BD playback since Windows 8/8.1 does not come with first party software to play physical media movies.
Removed some Adobe CS tools that are still being used by some older suites.
Removed Dell Quickset and Asus ATK since they are hotkey drivers for laptops that control volume, brightness, keyboard LED backlighting (have been known to cause error codes in DevMan if missing).
Removed MS mouse and keyboard center since this is the driver and macro button configure tool for microsoft mice and keyboards.
Removed Intel Rapid Storage Technology (not to be confused with Rapid Start Technology) since this is a necessary driver in most Vista/7 systems. Systems may not be able to use Windows Update without this installed. Known bug in Windows 7 pre SP1.
Removed ClickFree since this is a legit external backup software.
Removed Hp SimplePass since this is the fingerprint reader driver and can produce an error code in DevMan if missing.

Accepted. There are a couple I'm going to re-enable, but for the most part these are good changes. Thanks.


Release planner automation to release SDK packages

SDK release apps in release planner still requires user to manually verify change log, API approval, package version and enter these details correctly in the app and also need to run the prepare release script manually. User needs to run the release pipeline manually once all the action items are checked in the release planner to release the package.
Main issues with this approach:

Too many manual steps that can be confusing and require additional help and follow up.
Prepare release script needs to be triggered manually from the repo which ends up with incorrect details in some cases.
Running prepare release script is not enforced so we don't have the package information for all the packages released.
Package work item is created from release planner and also attempted by prepare release script.

Solution:
We can add an automation in release planner that will validate all the checks like,

API approval
change log verification
package version
package name approval

If any checks are failed, then it will show a description next to it to show what needs to be done to resolve the error.
User can also click a button in the release planner to release the package instead of manually running the pipeline to release the package.
There won't be any need to run prepare release script with approach since it's taken care by automation itself and they also do not need to know the package version in advance.
How to implement it:

Change in release build pipeline
Prepare release script currently performs a series of checks and shows the failure on console. We can modify these checks to be integrated in build pipeline that gets triggered when someone attempts to release a package. If any of these checks fail, it will update package work item in DevOps to show the failure and an explanation with details on how to resolve it.  Release pipeline will have a parameter to just run this validation without releasing a package to perform dry run of validations as well.

Package work item:
New fields to show validation status and details.

Release planner

Instead of checklist for validations, release planner will have a button to run the release checks. This will trigger release pipeline with a flag indicating it's to verify package preparedness (dry run of release). Release planner will poll the status of pipeline and show validation details from package work item when available.

Release planner will also have a button to release the package, and this will run the release pipeline without dry run flag.

Overall SDK release workflow from release planner for service partner

Complete API readiness milestone
Open SDK release app and select the package to release for their product and select languages to release.
Generate SDK(not yet automated), get API approval, package name approval.
Once SDK code is in the language repo, run release readiness validation by clicking the button in release planner.
If validation fails, then fix the code and run the validation again.
Click the button to release the package.

Question: Data plane SDK release pipeline requires approval to run the release stage. This might still be required manually (since only few people can approve the release), and we can provide information on release planner if that's the case.
### Tasks
-[ ] 7750
- [ ] https://github.com/Azure/azure-sdk-tools/issues/7750

I think we will want some way to also query information like the version for a given package from the repo so it can be automatically displayed in the release plan and folks don't need to provide it. As part of the validation if they see the version isn't correct we can point them at instructions on how to update that in the repo.

Question: Data plane SDK release pipeline requires approval to run the release stage. This might still be required manually (since only few people can approve the release), and we can provide information on release planner if that's the case.

We can make adjustments to this once we start kicking off these pipelines from the release planner.

Remaining issue for this work is https://github.com/Azure/azure-sdk-tools/issues/8222
I am going to close this epic and we will follow up with that issue separately


Make compatible with Manifest v3

Fixes #156
NOTE: This should be carefully tested, as some things might still be broken. I'll continue to test on my end throughout this week.
This follows Chrome's migration guide and:

[x] changes manifest version to v3
[x] changes background script to service worker
[x] separates permissions from host permissions
[x] removes content security policy
[x] renames browser action to action
[x] removes web accessible resources
[x] uses scripting API to inject content scripts
[x] uses scripting API to inject functions (all functions that need to be injected now have to be added to Shared.functionsToInject on first run, because they need to be accessible from the service worker)
[x] removes ports, as they're not compatible with service workers (the new method to identify when a content script has connected is to send a message to the service worker, and the new method to identify when a content script has disconnected is to add a listener to browser.tabs.onRemoved)
[x] removes axios, as XMLHttpRequest isn't compatible with service workers (I had to implement our own queuing system for rate limiting, so it needs to be tested)

There might be other changes that need to be made.
These changes should be fully backwards compatible with manifest v2. Webpack is configured to build v2 for Firefox and v3 for Chrome, as v3 isn't accepted in the Firefox store yet, as far as I could tell.

@MrMamen It's a WIP, but if you already want to take a look.

@MrMamen Can we try to land this? I didn't test too much (also mostly just Netflix), but we can always refine it later. I'd like to release it on Chrome soon.

Yes. I'll test it a bit.

NRK doesn't work with scrobbeling. This is due to the injection script not working. (Only tested on Chrome).

Got it, thanks, I'll see if I can fix it.

@MrMamen Script injection for functions should be fixed.

@MrMamen Script injection for functions should be fixed.

It seems to me that the scrobbling part needed to be disabled and re-enabled after switching to this branch. (for NRK at least.) If there is no way to prevent this, perhaps the migrate script should simply disable the scrobbling for injection-script sites?

@MrMamen Script injection for functions should be fixed.

It seems to me that the scrobbling part needed to be disabled and re-enabled after switching to this branch. (for NRK at least.) If there is no way to prevent this, perhaps the migrate script should simply disable the scrobbling for injection-script sites?

But that's for Chrome, right? I don't think it will be possible to migrate from v2 to v3 on Chrome, because they have different IDs, so they should be two separate extensions when installed. That wasn't the case for you because you probably don't have the ID configured in your .env. Unfortunately, I think Chrome users will have to reconfigure their options.

I'll see if I can add an import/export functionality to allow users to move their data.

I'll see if I can add an import/export functionality to allow users to move their data.

Om not sure how important this is. More important to get this to shipped. We should be more careful with these things after it hits the Web Stores.

Ok, I've actually already uploaded it to the Chrome store because I wanted to test something: https://chrome.google.com/webstore/detail/universal-trakt-scrobbler/mbhadeogepkjdjeikcckdkjdjhhkhlid
But before we add it to the README I want to land another PR that I'll open later today that allows users to do a full initial sync.

I experience that at least netflix history keeps loading/spinning forever, with no obvious error..


Run tests on CI

You need to use a "logging command" to make changes to PATH persistent in azure pipelines. See here: https://docs.microsoft.com/en-us/azure/devops/pipelines/scripts/logging-commands?view=azure-devops&tabs=bash#prependpath-prepend-a-path-to-the--path-environment-variable
echo ##vso[task.prependpath]$PWD

Also, I think the instructions here https://packages.lunarg.com/ will work, but you picked the wrong version of Linux (bionic instead of xenial).

You need to use a "logging command" to make changes to PATH persistent across steps in azure pipelines.

Ah ... thanks for that!

... but you picked the wrong version of Linux (bionic instead of xenial)

Right ... that explains a lot, lol. Rolling back to this approach now.


Added Anime4K-GUI project

Hello,
I created GUI application which allows you to save the enhanced anime as higher-quality video files rather than real-time preview
Added link to application in Projects section in README

This is a great alternative to waifu2x-ios. Now I can utilize my Nvidia graphic card to upscale with Anime4K shaders. Thanks!


[MXNET-656] Update documentation for count_tokens_from_str

Description
count_tokens_from_str() throws an error when user tries to use special characters such as * . ? etc as delimiters. This occurs because the function makes use of re.split() which considers the delimiters to be Regular Expressions.
Checklist
Essentials

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant [JIRA issue]
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] Code is well-documented:
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes
Changed the documentation to let the the user know about the use of special characters as delimiters, along with an example of the correct usage.

@astonzhang


Differentiating between multiple grids on same view

Looks like in-spite of naming my multiple ajax grids on the same view (with search) I can't seem to have anything(like id / name/etc) to differentiate between the two.
When I explore the dom I see both grids with same html
How can I select or What would be correct Java script to select mvc-grid named "allgrid" and another mv-grid named "limitedgrid".
Posting here as I am not if it's a limitation at this time or if I am not grasping this correctly.
Thanks in advance

You can add different classes on tables using Css(...) method and select it from that.
I guess, I should not hide grid's data-name attribute, to make this easier.

Thanks for looking at tit right away. Will it be better to  use the "Named" attribute as an DIV ID ?...since it already works like a charm for sorting and filtering in the same scenario. Not sure if that makes any sense.

Naah populating id's could result in duplicates, which are not supported per html spec.

OK I am a beginner don't have comprehensive knowledge but this communication teaches me and gives me confidence. So if I do .Css("onegrid"), .Css("twogrid") how will it impact JS selector statement....var grid = $('.mvc-grid').data("mvc-grid")

It would be
var grid = $('.onegrid').parent().data('mvc-grid');

Thanks. I believe it's safe for most normal operation you want to do the two different/multiple grids. Depending so much on style attributes to handle complex client actions in multiple grid scenario scares me though.

You can use your own grid partial to render custom grids, if that helps. Too bad I still didn't add an example for that, but I will, for nested grid one.

Sure I got the idea. Thanks for all the hard work... it's a joy to see this grid show, filter, sort your data


Components loses props when navigating

I have app which have two pages both have form or forms created with subschema.
When i go first time to page which have multiple forms, all props gets right to component.
BUT when i navigate off the page and then back, all props are reset to default values!
I noticed this with at least Select component. I have made custom select component with listener property.
All props are defined in schema and schema holds right values in navigation but for some reason component doesn't get props anymore. Here is my select component:
import React from 'react';
import BaseComponent from './BaseComponent.jsx';
import Subschema from "Subschema";
const {PropTypes, types} = Subschema;
const {Select} = types;
export default class SelectListener extends BaseComponent {
static displayName = "SelectListener";
static propTypes = {
...Select.propTypes,
options: PropTypes.listener,
size: React.PropTypes.number,
};
static defaultProps = {
    size: 1
};

static contextTypes = {valueManager: PropTypes.valueManager};

constructor(...rest) {
    super(...rest);
    this._bind('render');
}

render() {
    var {size, ...restProps} = this.props;
    var value = this.props.value;
    // Here size is 1 when second navigation occurs, in schema it's defined to 5
    console.log(size);
    if (value == null && this.props.options) {
        value = this.props.options[0].val;
    }

    return (
        <Select {...restProps} value={value} size={size} />
    );
}

}
and part of schema:
schema: {
"Application": {
"type": "SelectListener",
"multiple": false,
"options": "appOptions",
"size": 5
},
I'm using version 1.3.0 of subschema because dynamic population of select's options thru value doesn't work on newer versions.

Just noticed that this is fixed on 1.6.0 version.


JUnit tests failure in Netty 4.0.56.Final

Expected behavior
Successful tests passing when build latest 4.0.x (4.0.56.Final) release.
Actual behavior
Tests failure.
Tests run: 747, Failures: 183, Errors: 315, Skipped: 61

Steps to reproduce
git clone https://github.com/netty/netty.git
cd netty
git checkout netty-4.0.56.Final
mvn test

Furthermore, it looks like the same problem (at least the number of failed tests is the same) occurred on the CI-server when I tried to open a pull-request for merging into the 4.0 branch: see here.
Strange, but, according to this page, there are no problems with the nightly builds of the 4.0 branch.
Netty version
4.0.56
JVM version (e.g. java -version)
1.8.0_171
JDK version (e.g. javac -version)
1.8.0_171
OS version (e.g. uname -a)
4.4.0-130-generic Ubuntu SMP x86_64 GNU/Linux

works here without problems. That said 4.0 is EOL please upgrade


prost-build: Support generating single include file

In order to support use-cases where many protocol buffer
packages are parsed and generated, support the configuration
containing an optional include_file() which allows the build
script to name a file to be created to automatically include!()
all the generated content with the right module structure.
Signed-off-by: Daniel Silverstone<EMAIL_ADDRESS>

This is just a rebase of #123

Bump

This is highly desirable behaviour for me. We have many hundreds of interdependent .proto files, so including them all is the only way to use any of them.
@danburkert Are there things about the existing attempts (this PR, https://github.com/danburkert/prost/pull/294, etc.) that make you unsure about whether you'd want to merge them? Equivalently, are you able to offer some guidance on what a PR to address this would look like for you to be confident in merging it?

Closing in favor of #433, which fixes some issues here


io.grpc.StatusRuntimeException: UNAVAILABLE: The service was unable to fulfill your request. Please try again. [code=8a75]

We started seeing this issue today. Is there a service outage happening?

What functionality is throwing this exception? Does it happen systematically? Our integration tests run smoothly 5 hours ago.

This was when creating a subscription. We are not hitting any quota limit for that just confirmed. Just started 30 mins ago.

One more bit of information on this.
We receive this exception when creating a subscription to a topic that was created with the older 0.3.0 release of the Java PubSub client. The topic was created many, many months ago.
I just tried to create a new topic and then a subscription to that topic using the 0.4.0 release and I did not get this error.

This is very strange. I see not reason why creating a subscription for an "old" topic should fail. I would also be surprised if google-cloud version had something to do with this as well. Can you confirm this happens every time you try to create the subscription?

I can confirm, yes. Would it help to provide the project name and topic that is having the problem?

The Pub/Sub google group could be a better place to receive help on this.
Just to have some more information: can a subscription to that topic be created via the web console?

Good question.
The answer is no. This is what I received when trying to create the subscription to the topic via the webconsole.

The answer is no. This is what I received when trying to create the subscription to the topic via the webconsole.

This seems to rule out an issue with our library. Can you confirm you created the topic with 0.3.0? I am asking because in previous versions (<= 0.2.7) we were using the experimental endpoint. This should have no impact but I just want to get more information.
@eschapira @kamalaboulhosn do you guys have some idea on this?

@mziccard We used the google client pubsub api before. Not the Google cloud java api

One more update Our production code which is running the old google client pubsub api is also hitting this issue. So this is a service related issue.

@r-tock @jjlueck-tock And you are not hitting the 10000subscriptions-per-topic limit, right? [https://groups.google.com/forum/#!topic/cloud-pubsub-discuss] is the best place for this issue, support can also help given your project id and topic name.

This appears to be service related. We have a support ticket going instead. You may close this issue as we're pretty sure that this is not a Java client bug.

We have a support ticket going instead.

Hope this gets fixed soon.


Fix dnslink lookup

Previously domain was used but that is tokio-0.1 library. Now domain seems to have upgraded to latest tokio in https://github.com/NLnetLabs/domain.
Our needs for the DNS are:

stub resolving
TXT records to get dnslink 1 2

rust-libp2p should already handle resolving names in /dns4 and /dns6 multiaddrs through threadpooled (stub?) resolver (haven't checked which library).
Being able to resolve dnslinks is the desired outcome. The async-std dependency seems a bit difficult. Alternatives on tokio would include trust-dns. Even if we decide to add back tokio support, we should be considering the what ever way libp2p is using as something we could leave for async-std.

I think this is now fixed with some of lukasz latest dep update PRs.

Ah yeah, the related tests pass (when enabled with tokio) after https://github.com/rs-ipfs/rust-ipfs/pull/290/commits/cb0fa7b3a4734d7bc379a225d548f2c80464d934.

And now we are fully tokio so this is always enabled.


Sync cdsdashboards conda environments

Within the recent changes from #1003, the original behaviour for syncing and inspecting available environments (e.g cdsdashboards included) was modified, as conda-store is now responsible for managing the creation and orchestrations of the environment.

I new way of adding cush a feature needs to be included

closing this as duplicate of #1019


v0.11.2: exposed serialization methods (issue #56)

Added

Added TransactionMapping (fixes #56 )
Added CreateTransactionFromPayload (fixes #56 )
Added SerializeTransactionToJSON
Added several toDTO() methods for serialization (fixes #56 )
Added multisigAccountAdded in Listener
Added accountAddedToMultisig in Listener

Pull Request Test Coverage Report for Build 274

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.6%) to 75.11%

Totals

Change from base Build 266:
0.6%

Covered Lines:
1298

Relevant Lines:
1581

💛  - Coveralls


multi_squad_noans_infer: output empty answer for no-answer text

Hi Guys,
For config (multi_squad_noans_infer), I wonder how the model determines if there is no answer in a text? and if we can reset the probability (if there is such a parameter)? Can anyone explain how squad_noans works?
In the model file, here : but not sure how it work.
As pointed out here (https://github.com/HKUST-KnowComp/R-Net/issues/48),

R-net uses boundary model presented in "Machine Comprehension Using Match-LSTM and Answer Pointer". It will give an answer whether the true answer exists or not.

So back to the question ...
Thanks so much.
Luke

Hi!
R-Net model finds an answer by predicting answer_start and answer_end positions with Pointer Networks. We add special no_answer token to the beginning of a text (actually, we add this token only before Pointer Network layer) and model should predict this token if question doesn't have an answer in a given text.
SQuAD v1.1 does not have training examples without an answer, so we added such examples to the training set. Some details on how we build training set with examples without an answer are given here: http://docs.deeppavlov.ai/en/master/features/models/squad.html#squad-with-contexts-without-correct-answers

Hi @yoptar , thanks and good to know this, though I think this didn't get to the point on

how the model determines if there is no answer in a text?

probably it is about Pointer network. Can you share insights on it or point me to the line of the code on how no answer is determined?

Maybe I was not so clear on answering this:

how the model determines if there is no answer in a text?

Pointer Network is an output layer of R-Net, the task of Pointer Network is to find two positions in text: answer_start and answer_end. no_answer is used as usual token and is appended to every text here: https://github.com/deepmipt/DeepPavlov/blob/a95149294bc7fa30baaa15c629adad4c1cba264e/deeppavlov/models/squad/squad.py#L191-L196
Probability of an answer is computed as 1 - P(no_answer is start token) * P(no_answer is end token), no_answer token index is 0:
https://github.com/deepmipt/DeepPavlov/blob/a95149294bc7fa30baaa15c629adad4c1cba264e/deeppavlov/models/squad/squad.py#L209-L210
Similar way of doing this is described in this paper (Section 3.3): https://arxiv.org/abs/1710.10723


Shouldn't be allowed to search groups if nothing selected

Acceptance Criteria

[ ] It shouldn't allow you to search for groups if there is no specific filter, campus, or zip code selected. One of the three has to be chosen in order for the search to function.

Context
In the current app if you do not pick a filter (or type one in), a campus, or a zip code then you are not allowed to search for groups. The button to search is disabled. The idea is that it could potentially be too many groups to pull back (since you are literally pulling every group). That's a large query that currently breaks the app.
Part of the reasoning behind having the campus list have an empty slot and not the "All Locations" option was to have a visual way to show that you hadn't selected a campus at all.
Side Note: It looks like (at least from https://beta-my.newspring.cc/groups) that the campus and zip code don't default to my personal campus and zip code when I am logged in as they do in our current app. That's something to check on anyway. I can do more testing.
UI Screenshot or Wireframe (optional)

I WANT TO SEE EVERY GROUP MAN! 😎

@conrad-vanl so after investigating this and talking to @richarddubay about it, the button isn't disabled because of the auto-selection of "Search My Location" to yes but in Holtzman this option is defaulted to no. I guess this is a question of should we have this auto-selected to yes or if this should be considered when disabling the button. Any thoughts?

My thoughts on this is that if it's auto selected to no you have the ability to type in a zipcode instead, and auto-selected to yes probably creates a faster experience for most users...

@IsaacHardy do you have an opinion one way or the other about this?

I'm like the experience @conrad-vanl explained above.

@conrad-vanl I'm having some issues trying to create the conditional validation with Yup that I think would work with the disabled={!isValid} on the submission button.
I'm using
Yup._____().when('useDeviceLocation', {     is: false,     then: Yup.____().required(),     else: Yup.____(), }
But isn't quite what we need for this.
Is this the right line of thinking or would you go about it another way?


The 2.12.1 version source package is missing

Follow the operations in the documentation and find that the source package of version 2.12.1 is missing

The screenshot of the page in the apache website is as follows：

Can we close this issue? @juzhiyuan


Change font size on website

from 12px to 13px

playing around

this issue is a subtask from issue #2

attachement

this is a mention @Fatimarouqa


Add appVersion for namerd

The key "appVersion" is needed for ci testing, it's missing in this yaml. That sometimes will cause testing failure.

/retest


如何更改菜单里的About 和 docs？

您好，我用您的主题搭建博客时，发现配置文件 config.toml 里没有更改 About 菜单和 docs 菜单的配置项。请问应该在哪修改呢？

I have the same exact problem: a docs menu and an about page are shown in the main menu even though they are defined nowhere in config.toml
I am still figuring out all the code.

I have the same exact problem: a docs menu and an about page are shown in the main menu even though they are defined nowhere in config.toml.
I am still figuring out all the code.
EDIT:
They are both defined in markdown front matter:

in /content/about.md for the about page
in various posts in /content/post

You can edit them directly here.

Many thanks. I got it.

Same here. I'd like to remove that link to about and docs.
Noticed there are configuration parameters for docs on https://github.com/xianmin/hugo-theme-jane/blob/master/dev-config.toml#L44-L55 . Seems like a work in progress

在post文章中，把
menu:
main:
parent: "docs"
 weight: 1

去掉就好了


Set up Google Analytics to track outbound links

So we can track which local websites people are going to.

got it

Working as expected on the site now, closing issue
Side discussion on capturing search progression I will look at some more, but can likely handle solely on the Tag Manager side

This is working!


cgo: undefined reference

I test the building of github.com/google/gopacket
mkdir -p /tmp/test-gb/src
cd /tmp/test-gb/src
mkdir -p github.com/google/gopacket
git clone https://github.com/google/gopacket github.com/google/gopacket
gb build github.com/google/gopacket/examples/arpscan

Got lots of undefined references. Tail of the log:
/var/tmp/go-link-EU9Hef/000001.o: In function `_cgo_c0f3dbecc2d2_Cfunc_pcap_freecode':
/tmp/test-gb/src/github.com/google/gopacket/pcap/pcap.go:267: undefined reference to `pcap_freecode'
collect2: error: ld returned 1 exit status
FATAL command "build" failed: # /usr/local/go/pkg/tool/linux_amd64/6l -o /tmp/test-gb/bin/arpscan -L /tmp/gb935557792 -L /tmp/test-gb/pkg/linux/amd64 -extld=gcc /tmp/gb935557792/github.com/google/gopacket/examples/arpscan/main.a: exit status 1
/usr/local/go/pkg/tool/linux_amd64/6l: running gcc failed: unsuccessful exit status 0x100

Similar problems with github.com/mattn/go-sqlite3:
/tmp/demizer/go-link-sr5BXt/000000.o: In function `unixDlClose':
/home/demizer/src/budgethound/vendor/src/github.com/mattn/go-sqlite3/sqlite3-binding.c:29970: undefined reference to `dlclose'
/tmp/demizer/go-link-sr5BXt/000000.o: In function `unixDlSym':
/home/demizer/src/budgethound/vendor/src/github.com/mattn/go-sqlite3/sqlite3-binding.c:29966: undefined reference to `dlsym'
/tmp/demizer/go-link-sr5BXt/000000.o: In function `unixDlOpen':
/home/demizer/src/budgethound/vendor/src/github.com/mattn/go-sqlite3/sqlite3-binding.c:29925: undefined reference to `dlopen'
/tmp/demizer/go-link-sr5BXt/000000.o: In function `unixDlError':
/home/demizer/src/budgethound/vendor/src/github.com/mattn/go-sqlite3/sqlite3-binding.c:29939: undefined reference to `dlerror'
collect2: error: ld returned 1 exit status
DEBUG build duration: 780.497828ms map[compile:32.292872ms link:739.231147ms]
FATAL command "build" failed: # /usr/lib/go/pkg/tool/linux_amd64/6l -o /home/demizer/src/budgethound/bin/bhcli -L /tmp/demizer/gb926072060 -L /home/demizer/src/budgethound/pkg/l
inux/amd64 -extld=gcc /tmp/demizer/gb926072060/github.com/demizer/budgethound/cmd/bhcli/main.a: exit status 1
/usr/lib/go/pkg/tool/linux_amd64/6l: running gcc failed: unsuccessful exit status 0x100

Workaround for go-sqlite3: gb build -v -ldflags="-extldflags=-ldl" all

@lazyuser I think i've got a fix for this first issue, although I cannot find a deb that gives me pfring.h. I'm going to submit it anyway.

This issue seems to be fixed for me. Thanks!


[loc][Query] Context needed for "contribution point"

For string below community members are asking about more context? Can you please help? Thanks!
Original comments
BG ISSUE - This is unclear. And what do the 'contribution point' and 'conributed' mean here?
by lyubomirv, 6 hours ago
ES - @johosek , can you help me to understand on what means "contribution point" in the file source in https://github.com/Microsoft/vscode/raw/master/src/vs/workbench/api/browser/viewsContainersExtensionPoint.ts line: 44
Thanks.
by AleksSyntek, 11 hours ago
Context
src/vs/workbench/api/browser/viewsContainersExtensionPoint:English (en):plaintext:vscode.extension.contributes.views.containers.id
Resource
vs_workbench
String Hash
bb64f8d530c8baeb8d85950114a52101
String text
Unique id used to identify the container in which views can be contributed using 'views' contribution point

Same concern about "contribution" applies to next string from the same file.
String - '{0}' contribution is only available when running out of dev or with the following command line switch: --enable-proposed-api {1}
Comment - What is 'contribution' here? by lyubomirv, 6 hours ago

Contribution refers to those that an extension contributes to VS Code.

@sandy081 I'm sorry but it's not clear. I still do not understand what is a 'contribution POINT'?
And for the other concern - '{0}' contribution. What does this mean? What could {0} be replaced with?

@lyubomirv Contribution point is an extension point in package.json to which extensions contribute to. For eg:
"contributes": {
	"viewsContainers": {
		"activitybar": [
                        {
			    "id": "azure",
			    "title": "Azure",
			     "icon": "resources/azure.png"
		        }
                  ]
	},
        "views": {
		"azure": [
			{
				"id": "cosmosdb",
				"name": "Azure Cosmos DB"
			},
                         {
				"id": "docker",
				"name": "Docker"
			}
		]
	}
}

In the above example viewsContainers and views are contribution points.
I just updated the message to replace {0} directly with viewsContainers because it should not be translated.

Thanks @sandy081 for the explanation on that. I will recheck the translation item.

@sandy081 Thank you!


Checklist

[x] documentation is added or updated

This pull request is ready to merge

Merged to the 3.0 branch. Thank you for the reviews.


Problem with collision detection addEventlistener

cmpRigidbodyfussel= fussel.getComponent(ƒ.ComponentRigidbody);
    cmpRigidbodyfussel.addEventListener(ƒ.EVENT_PHYSICS.COLLISION_ENTER, fusselCollides);

The function never really goes into the fusselCollides-Function, even though it collides many times with static, kinematic and dynamic objects. The rigidbody itsself is dynamic.
Full code is here:
https://cleokaty.github.io/Jetpackbear/
https://github.com/CleoKaty/Jetpackbear
Would be gratefull for some insight.

@Dev-MarkoF Ich konnte auf den ersten Blick in Janas Code keinen Fehler erkennen und habe erst einmal mit einem Minimaltest das Feuern des COLLISION_ENTER-Events geprüft. Siehe anbei. Dabei fällt auf, dass das Event nicht mehr kommt, wenn der springende Ball ein paar mal gedozt war und sich nicht mehr weit vom Untergrund hebt.

Hey,
first of JumpBall seems to be a timing issue and possibly something i cant really determine, because i'm not sure what's at play in an empty scene in Fudge, at some point there is no "leaving" event.
If the f.Time.setScale is changed to e.g. 1,5 it registers more bounces, 16 instead of 11, with scale of 2 it registers over 90 bounces. But it never registers bounces all the time, in that jump ball example.
In general, the Collision_Enter is called once for each collision between rigidbodies, if there is no Collision_Exit happening, there can't be a new "Enter", since it never registered the leaving.
What makes this so unpredictable for me is that, if i completly deactivate the "background" in jetpackbear it works as intended, btw. not sure why you have a rigidbody on the background. The odd behaviour here is that if you leave it activated but remove the rigidbody component, it does not work, it has to be removed from the node tree.
When the background is removed some bodies turn sideways, i guess you wanted to stop that with the "background", the better way to achieve that would be to turn the rotation factor of the bodies in Y direction of to not have the all bodies "smear" along the background.
I will look into it, but i'm not sure this is an actual problem within the physics calculation.

Thanks a lot! This is really helpfull!


Dynamic panel name according to edge name

Component

UI

Description
I have about 200 edge instances deployed and they all have the same dashboard deployed, tailored to the edge data. I want the name of the panel to be for example: "${EdgeName} - Energy".
How could I do it without creating a panel for each instance?
Thanks in advance.

Disclaimer
We appreciate your contribution whether it is a bug report, feature request, or pull request with improvement (hopefully). Please comply with the Community ethics policy, and do not expect us to answer your requests immediately. Also, do not treat GitHub issues as a support channel.

@Borch97
Currently, there is no built-in functionality to display the Edge name in a specific widget out-of-the-box. However, this is a great feature idea for the future.
For now, you can use the following workaround: create an alias to fetch all edges entities and display the name of the fetched edges. On the cloud, this alias will randomly select one of your edges and display its name. On the edge, this alias will select the exact name of the dedicated edge.
Here are screenshots of the configuration of the dashboard and alias on the cloud:

Screenshot of the first edge:

Screenshot of the second edge:

As you can see, the title of the dashboard differs from edge to edge. I hope this approach can meet your goals for the moment.


Change Log In link to Support Us

Lily asks:

Can we add a button saying "support us" to replace the "log-in" button at the top right hand of the home page (which will eventually link to the CF page when it's live)?

CF == "Crowd Funder"

I've added a theme custom setting support_us_url which if present will enable a "Support Us" button which links to this URL.

@wu-lee 'log in' is still there for me, should this be removed? It has no function.

On 23/09/2022 20:25, Mel wrote:

@wu-lee https://github.com/wu-lee 'log in' is still there for me,
should this be removed? It has no function.

It was restored when I reverted the "Support Us" button.  It has a
function, but we don't currently use it. Wasn't sure whether to clobber
it entirely.

Is it possible to have it so that it only appears if the function is turned on?
I know there's a member section, but I'd turned it off as it's not necessary right now, surely ghost should know that's not required anymore


log4cxx svn fetch

Previous issues: #390, resolved on 3 Jun 2016 commit  b4baa4a
Failed to fetch log4cxx provided in recipe.
Warning:
log4cxx-svn-r0 do_fetch: Failed to fetch URL svn://svn.apache.org/repos/asf;module=incubator/log4cxx/trunk;protocol=http, attempting MIRRORS if available
Error:
Fetcher failure for URL: 'svn://svn.apache.org/repos/asf;module=incubator/log4cxx/trunk;protocol=http'. Unable to fetch URL from any source.
Can be reproduced by using: bitbake log4cxx -c cleanall && bitbake log4cxx

@BilalArain I presume you're behind proxy. Then you need to configure your local subversion client. Bear in mind subversion doesn't respect http_proxy= and https_proxy= environment variables.

Your issue looks like a duplicate of https://github.com/bmwcarit/meta-ros/issues/413.

@bulwahn
I can checkout the repository successfully using svn co https://svn.apache.org/repos/asf/incubator/log4cxx/trunk. Therefore it is not a problem of proxy. The build configuration is as follows:
Build Configuration:
BB_VERSION        = "1.33.2"
BUILD_SYS         = "x86_64-linux"
NATIVELSBSTRING   = "universal"
TARGET_SYS        = "i586-poky-linux"
MACHINE           = "qemux86"
DISTRO            = "poky"
DISTRO_VERSION    = "2.2+snapshot-20170324"
TUNE_FEATURES     = "m32 i586"
TARGET_FPU        = ""
meta
meta-poky         = "master:49c2df5652d25e6e9b2a4976658ab9bb17fe6694"
meta-oe
meta-networking
meta-python       = "master:d63e5687c7bf6b054c6ed2fc4790e4970b023219"
meta-ros          = "master:eb4f9edf2af38a854a2a8d28e45ea02f6bdb7f12"
I have also changed the protocol in log4cxx recipe from 'http' to 'https' but no avail. The log
log_do_fetch.txt
log_do_fetch.txt
file of the error is attached.
I can confirm that the work around suggested in #413 using http://cgit.openembedded.org/openembedded/tree/recipes/log4cxx works but then I end up compile errors due to gcc mentioned in #390.
Is it possible to download the trunk and use SRC_URI = file://log4cxx.tar.gz? I have tried but then end up in License issues? Thanks for your help.

When I get work around this failure I end up in #460.
I close this issue and will work out to compile it directly on the development board.


[CaptureLocation & DropPin task] Fix crashes on config change

Complete Detail https://github.com/google/ground-android/pull/2761

@gino-m tried test
Idea was to mock the instance of drawAreaTaskMapFragment, so that whenever drawAreaTaskMapFragment.requireParentFragment() invokes, our mockDataCollectionFragment instance can be use and get the viewModel of DrawAreaTaskViewModel
but, not able to mock the instance of DrawAreaTaskMapFragment.

@gino-m ca we merge this?


add get all keys - GetListKeys, GetHashKeys, GetSetKeys, GetZsetKeys

since conflict, i create new pr.

add Get all keys

List, Hash, Set, Zset

related issues : https://github.com/flower-corp/rosedb/issues/179

hi @roseduan  could you pls take a look when you are free?


[WebSub] Refactor integration tests

Purpose

Add subscriber availability check before each publishUpdate call

Codecov Report

Merging #13390 into master will decrease coverage by 3.81%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master   #13390      +/-   ##
============================================
- Coverage     63.98%   60.17%   -3.82%
  Complexity      659      659
============================================
  Files          2147     2153       +6
  Lines         94210   104376   +10166
  Branches      12406    13361     +955
============================================
+ Hits          60281    62807    +2526
- Misses        29035    36308    +7273
- Partials       4894     5261     +367

Impacted Files
Coverage Δ
Complexity Δ

...rg/ballerinalang/test/context/BServerInstance.java
72.25% <100%> (ø)
0 <0> (ø)
:arrow_down:

...a/org/ballerinalang/bre/bvm/WorkerDataChannel.java
80.86% <0%> (-7.41%)
0% <0%> (ø)

...ain/java/org/ballerinalang/net/websub/hub/Hub.java
69.53% <0%> (-1.57%)
0% <0%> (ø)

...main/java/org/ballerinalang/net/http/HttpUtil.java
72.32% <0%> (-1.03%)
0% <0%> (ø)

...nalang/compiler/parser/antlr4/BallerinaParser.java
24.5% <0%> (ø)
0% <0%> (?)

...ler/parser/antlr4/BallerinaParserBaseListener.java
19.46% <0%> (ø)
0% <0%> (?)

...nalang/net/http/clientendpoint/InitGlobalPool.java
100% <0%> (ø)
0% <0%> (?)

...inalang/compiler/parser/antlr4/BallerinaLexer.java
27.57% <0%> (ø)
0% <0%> (?)

...er/parser/antlr4/BallerinaParserErrorListener.java
57.14% <0%> (ø)
0% <0%> (?)

...er/parser/antlr4/BallerinaParserErrorStrategy.java
36.5% <0%> (ø)
0% <0%> (?)

... and 8 more

Continue to review full report at Codecov.

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


pindexer: dex: implement /api/blockTimestamps related events and tables

We want to augment pindexer to support indexing for https://github.com/penumbra-zone/dex-explorer/blob/2b70ae60529284309f8748f6ca6a154a2bab82a9/src/pages/api/blockTimestamps/[...params].js, adding in any events where necessary.

Closing as previously completed - verified the data is already there


Make bitfields larger than type opaque

@fitzgen r?
Fixes #1007 by ensuring that bitfields larger than type will be opaque, ensuring the layout is correct.

@bors-servo r+

:pushpin: Commit aa27283 has been approved by fitzgen

:hourglass: Testing commit aa27283d7ed482613897ea115345c6531fb24db3 with merge 8387c7f5888e4e9b17828cfc6803dbbbd4fb6e47...

:sunny: Test successful - status-travis
Approved by: fitzgen
Pushing 8387c7f5888e4e9b17828cfc6803dbbbd4fb6e47 to master...


Rename janulator to swarmer.

Closes #1.

Please run ag janulator just to be sure.

I did grep and search in IntelliJ :)


dimension iotas not equal to itself

using Equality Distillation on dimension iotas always be False.
using Locator's Distillation will receive -1.

Fixed in dev


dynamic title for arrays

Hi thanks for the excellent library. Is there a way to have dynamic title for the array ? That would be useful when the array is collapsed, to know whether it contains any item or how many.
In your Children array of Child objects you show how headerTemplate can be used to make the items title dynamic. I have tried this on the Children array:
{
  "type": "array",
  "title": "Children",
  "headerTemplate": "Children {{ self.items.length }}",
  "items": {
    "type": "object",
    "title": "Child",
    "headerTemplate": "{{ i1 }} - {{ self.name }} (age {{ self.age }})",
    "properties": {
      "name": { "type": "string" },
      "age": { "type": "integer" }
    }
  }
}

but it does not seem to work. Am I using an incorrect synthax or is this a missing feature ? Paolo

https://github.com/jdorn/json-editor#dynamic-headers

Hi, I'm having the same issue, any ideas ?
Thanks.
Léo


[PRODUCTION] update production

merge at will

FYI We don't automatically deploy to production from master for Hours.

well, that is inconvenient... This is by choice?

Tested, works fine :)


Add a option to decide the way items are ordered (By Type then by Name, By Name, same as Domiticz (Default))

Add a option to decide the way items are ordered (By Type then by Name, By Name, same as Domiticz (Default))

Will be available in V<IP_ADDRESS>.


Revert "IIdSerializer to get result type hint (#2663)"

This reverts commit 19b061f00d6999c179332d264fd897ee6000a09f.

/AzurePipelines run


Setup project

Meal Gallery: Set up project
In this branch, I set up a JavaScript project for the Meal Gallery  app, using webpack and ES6 features, notably modules.
Project requirements

Create an npm project with webpack.
Write plain HTML markup with minimum styling.
Do not make API calls.
Use ES6 syntaxe to import CSS properties.
Set up Jest framework for testing.

How should this app be tested?
Clone this repository:
git clone https://github.com/mariegrace31/Meal_Gallery.git

Install all dependencies:
npm init -y

Fix Javascript linter errors:
npx eslint . --fix

Build the project:
npm run build

Open the project:
npm start

Hello @mariegrace31.
I finished setting up webpack and Jest framework. However, there is a problem with Lighthouse.
Please, can you help me fix it? We need it for the rest of the app creation.


Could not change default view for middleware datasources

Details:
The default view for middleware datasources is list view on Settings->My Setings->Default View page. If user wants to change it to other views like Grid view or tile view, the views are not getting selected ( The icon does not change to light blue color when clicked). For all other middleware entities, user can change the default view.
Product: Latest upstream
Steps:

Login to manageiq.
Navigate to Settings->My Settings->Default View page.
Click on Grid view or Tile view icon for 'Middleware Datasources'.
Grid view or Tile view icon for 'Middleware Datasources' does not get selected. The icon does not change to light blue color
Click on Grid view or Tile view icon for other entities, the color changes to light blue color.

@miq-bot add_label providers/hawkular
cc @abonas

@miq-bot add_label ui, bug

@miq-bot assign @Jiri-Kremser
@Jiri-Kremser please look at #8160 on where default views are configured, this fix should be similar. thanks!

@miq-bot assign @abonas
@Jiri-Kremser nevermind, I saw you're not available this week, I'll handle it.

Closed by https://github.com/ManageIQ/manageiq/pull/9036


Diegesis/dev

Task: https://github.com/etenlab/docs/issues/203

yes, I have seen changes in mvh/dev branch and it's updating the header, footer, and side nav menu as per GraphQL but after implementing Diegesis components it will allow you to set content in the diegesis component by passing props so it's mean I can't directly use markdown to render the content it would be nice if we receive GraphQL data in props form.
for other pages some of the functionalities not working as per the previous implementation and it's a little bit difficult to understand existing code and write it for the new design most of the functionality I covered. most probably I need some help understanding existing functionalities so that I can implement the same with the new design.
I'm planning to pick one page at a time and create a branch for that page and write functionality as per the new design and then raise the PR for the same.
The older design has more controls and functionality but I feel in the new design some of the controls are not available will discuss them once start working on each page.

yes, I have seen changes in mvh/dev branch and it's updating the header, footer, and side nav menu as per GraphQL but after implementing Diegesis components it will allow you to set content in the diegesis component by passing props so it's mean I can't directly use markdown to render the content it would be nice if we receive GraphQL data in props form.

Why can't we pass a Markdown string to a component? If we don't do that, and if people want ever more variety in what appears in the footer, I think we end up reinventing our own version of Markdown anyway.

for other pages some of the functionalities not working as per the previous implementation and it's a little bit difficult to understand existing code and write it for the new design most of the functionality I covered. most probably I need some help understanding existing functionalities so that I can implement the same with the new design.

Happy to help. The existing code is not very modular. We do need to decide whether it will be easier to bring the components into the project so that they can assume the Diegesis workflow. I don't have a firm opinion either way, but in some cases I think that the complexity of properties for a reusable component will make the code more complicated than just rewriting a component to do one thing, inline.

I'm planning to pick one page at a time and create a branch for that page and write functionality as per the new design and then raise the PR for the same.

That sounds good. So shall we close this PR?

yes, I have seen changes in mvh/dev branch and it's updating the header, footer, and side nav menu as per GraphQL but after implementing Diegesis components it will allow you to set content in the diegesis component by passing props so it's mean I can't directly use markdown to render the content it would be nice if we receive GraphQL data in props form.

Why can't we pass a Markdown string to a component? If we don't do that, and if people want ever more variety in what appears in the footer, I think we end up reinventing our own version of Markdown anyway.

For the markdown string as I can see ReactMarkdown react plugin will render the entire string as per the markdown syntax let's take an example of a footer section as per the diegesis component it has two parts on the left part it's showing the content of the footer and on the right side, it's showing branch name and also footer section responsive as per the mobile view then how to react markdown string going to handle these things. for better understanding I will recommend to check PageFooter component in diegesis ui-kit repo.

for other pages some of the functionalities not working as per the previous implementation and it's a little bit difficult to understand existing code and write it for the new design most of the functionality I covered. most probably I need some help understanding existing functionalities so that I can implement the same with the new design.

Happy to help. The existing code is not very modular. We do need to decide whether it will be easier to bring the components into the project so that they can assume the Diegesis workflow. I don't have a firm opinion either way, but in some cases I think that the complexity of properties for a reusable component will make the code more complicated than just rewriting a component to do one thing, inline.

It will be easier to bring the components into the project so that we can directly apply the props and add the functionality and it also helps us to make changes as per requirement. I know Diegesis ui-kit exporting both pages and its child components and on most of the pages, I'm directly using page components and passing all props.

I'm planning to pick one page at a time and create a branch for that page and write functionality as per the new design and then raise the PR for the same.

That sounds good. So shall we close this PR?

Yes, we can close this PR but I will create multiple PRs based on these current changes because most of the functionalities are working.


Doesn't work with inputs containing square brackets [ ] in the name

When the inputs contain square brackets [ ] (ex: <input type="email" name="ContactForm[email]">), it creates a duplicated error message each time the input is validated. In the code, field.form.querySelectorAll('[name="' + field.name + '"]') wont' work because it will look for [name=ContactForm[email]] and it doesn't like the double [ ].

The issue is only with IDs and querySelector() for the error messages. The inputs themselves seem to be fine. I pulled in your escape function in https://github.com/cferdinandi/bouncer/releases/tag/v1.2.1, gave you attribution, and only use it in getFieldID().
This carries over to all the places where it's used without have to include it explicitly, keeping the code more DRY. Thanks for your help here!

I tried with the new version (1.2.2) and I have the same problem as before. The error message is created each time you validate as you can see in my screenshot. Thanks

Please provide a reduced test case

You can test with this html file and click multiple times on submit without filling anything. And after that, try filling the radio buttons and email input. You can see that there are multiple error messages and that they don't disappear. Thanks
<!doctype html>

        <meta charset="utf-8">
        <title>Test Validation</title>

        <form>
            <input id="category-1" type="radio" required="required" name="ContactForm[category]" value="category-1">Category 1
            <input id="category-2" type="radio" required="required" name="ContactForm[category]" value="category-2">Category 2
            <input id="category-3" type="radio" required="required" name="ContactForm[category]" value="category-3">Category 3

            <input type="email" required="required" name="ContactForm[email]" placeholder="Email">

            <button>Submit</button>
        </form>

The CDN hasn't flushed it's cache yet, but when it does, v1.2.3 should finally address this. Reduced test case: https://codepen.io/cferdinandi/pen/rQoxyM
Also, thank you so much for your help and for providing a reduced test case to work with!

And CDN now has the latest version if you want to check the demo

It works now, thanks!


fix change log fatal to log warning to avoid code dump on decimal type

Proposed changes
Issue Number: close #xxx
Problem summary
cherry-pick from: #14150
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

buildall

I think we should use log fatal here, because it will return a wrong result when decimal overflow. If user is using insert into select to do ETL jobs, then data is wrong.