Get code coverage of several packages to 100%

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

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
1386ae970b6b6b08fe7eb48b93c82dd7fd9632a4
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
1386ae970b6b6b08fe7eb48b93c82dd7fd9632a4
link
/test istio-pilot-e2e

prow/e2e-simpleTests.sh
6a77f1fcc6491593138f83d9bd6552d6c2b6bfbc
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
1386ae970b6b6b08fe7eb48b93c82dd7fd9632a4
link
/test istio-pilot-e2e

prow/e2e-simpleTests.sh
6a77f1fcc6491593138f83d9bd6552d6c2b6bfbc
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
6a77f1fcc6491593138f83d9bd6552d6c2b6bfbc
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

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

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
6a77f1fcc6491593138f83d9bd6552d6c2b6bfbc
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
6a77f1fcc6491593138f83d9bd6552d6c2b6bfbc
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
6a77f1fcc6491593138f83d9bd6552d6c2b6bfbc
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

PTAL. All test failures appear to be flakes.

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

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
6a77f1fcc6491593138f83d9bd6552d6c2b6bfbc
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
6a77f1fcc6491593138f83d9bd6552d6c2b6bfbc
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
6a77f1fcc6491593138f83d9bd6552d6c2b6bfbc
link
/test istio-pilot-e2e

prow/istio-unit-tests.sh
e42ea589ac3f7f2c7cc3c291e244c68080492d7c
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

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

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
6a77f1fcc6491593138f83d9bd6552d6c2b6bfbc
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
6a77f1fcc6491593138f83d9bd6552d6c2b6bfbc
link
/test istio-pilot-e2e

prow/istio-unit-tests.sh
e42ea589ac3f7f2c7cc3c291e244c68080492d7c
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
e42ea589ac3f7f2c7cc3c291e244c68080492d7c
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
6a77f1fcc6491593138f83d9bd6552d6c2b6bfbc
link
/test istio-pilot-e2e

prow/istio-unit-tests.sh
e42ea589ac3f7f2c7cc3c291e244c68080492d7c
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
e42ea589ac3f7f2c7cc3c291e244c68080492d7c
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
e42ea589ac3f7f2c7cc3c291e244c68080492d7c
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

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

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
e42ea589ac3f7f2c7cc3c291e244c68080492d7c
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
e42ea589ac3f7f2c7cc3c291e244c68080492d7c
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
e42ea589ac3f7f2c7cc3c291e244c68080492d7c
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
e42ea589ac3f7f2c7cc3c291e244c68080492d7c
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


[iOS] Open PrivacyPolicy URL Link from AboutView

Issue

close [iOS] Open PrivacyPolicy URL Link from AboutView #560

Overview (Required)

Open PrivacyPolicy URL Link from AboutView

Links

noting.

Screenshot
https://user-images.githubusercontent.com/80329420/190916518-fcbaafa6-c6d4-40b7-ba7b-87dd45ee6e20.mov

@ry-itto
I fixed it.
Could you check it?
(I'm not confident.)

@onakama
Thanks!
How about place StaticURLs in another file and define to top level of file?
@Environment point is okey!

@ry-itto
Could you check it?

@onakama
Please fix conflicts 🙏🏼

@ry-itto
Could you check it?

@ry-itto
thanks!!
It was a good experience.


net/http: http.Client returns errors when re-using connections if the connection closes at the wrong time

What version of Go are you using (go version)?
go version go1.9.1 linux/amd64
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/vagrant/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build936671241=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"

What did you do?
I performed an HTTP POST using an *http.Client that had keepalives enabled and there was an idle connection available for re-use.
Specifically, I encountered this by using an *http.Client with an http.Transport set to have an IdleConnTimeout higher than the server's. Even a very small duration above can cause it. From my testing, I believe the connection gets taken for re-use just as the server closes it.
This program demonstrates the problem: https://play.golang.org/p/GTj6FEFWk-
Note if we change the request to be a GET, we don't see a problem.
What did you expect to see?
The request succeed. In particular, I would expect the *http.Client, being a connection pool and supporting connection re-use, would not raise an error when it finds a dead connection, but use a different connection. This assumes there was confidence the request indeed did not go through.
What did you see instead?
The sample program repeatedly makes HTTP requests. Sometimes they succeed, sometimes they fail with:
Post http://localhost:9999/test: EOF

Or:
Post http://localhost:9999/test: read tcp <IP_ADDRESS>:36589-><IP_ADDRESS>:9999: read: connection reset by peer

For example:
$ ./test-connection-reuse
2017/10/05 20:22:39 request succeeded
2017/10/05 20:22:40 request succeeded
2017/10/05 20:22:41 request succeeded
2017/10/05 20:22:42 request failed: error performing request: Post http://localhost:9999/test: EOF
2017/10/05 20:22:43 request succeeded
2017/10/05 20:22:44 request succeeded
2017/10/05 20:22:45 request failed: error performing request: Post http://localhost:9999/test: EOF
2017/10/05 20:22:46 request succeeded
2017/10/05 20:22:47 request succeeded
2017/10/05 20:22:48 request failed: error performing request: Post http://localhost:9999/test: read tcp <IP_ADDRESS>:36563-><IP_ADDRESS>:9999: read: connection reset by peer
2017/10/05 20:22:49 request succeeded
2017/10/05 20:22:50 request failed: error performing request: Post http://localhost:9999/test: read tcp <IP_ADDRESS>:36589-><IP_ADDRESS>:9999: read: connection reset by peer
2017/10/05 20:22:51 request succeeded

Further information
I searched for issues before reporting this. I found #8946 which I believe to be about this exact issue, except its example was an HTTP GET request. I believe the fix for #8946 was done for #4677 in 5dd372bd1e70949a432d9b7b8b021d13abf584d1.
From reading #4677, I believe the reason HTTP GET works is we retry in that case. Retrying an HTTP POST is unsafe depending on whether the request reached the server. But in the situation I'm describing, it didn't (although I don't know if we could reliably tell). See here in transport.go (comments closely below this line appear to be talking about this).
I noticed in #4677 that some of the basis for the behaviour was based on what Chromium does. I found its relevant code that handles what I think is going on here, and I don't see it talking about particular HTTP verbs (though of course I may not be looking in the right spot).
For my application, I've worked around this by ensuring the client's IdleConnTimeout is lower than the server's equivalent. I also added a retry at that level. I think a retry alone would not be sufficient, as presumably the retry could hit this very condition again.
It may be that nothing can be done. In that case, it might be good to document that this is expected, as it surprised me a little.
Thank you!

CC @tombergan

Thanks for the playground code. The relevant RFC is here:
https://tools.ietf.org/html/rfc7230#section-6.3.1
You've already pointed out the main challenge, which is determining if the POST had reached the server. This is impossible to know precisely. It looks like Chrome assumes the POST had not reached the server if the connection was reused and gets a RST before any response bytes are received. We could do the same thing.

Interesting, thank you for the RFC reference!
The heuristic from Chrome seems fairly reasonable. Presumably if it has been in use there the risk is less.
I also found #15723 and #18241 which sound similar in that they are about non-idempotent requests. From the latter I had the idea to try setting GetBody on the request in the sample program. The behaviour is the same.

I think it's too late to change to the Chrome behavior for Go 1.10, but we should probably do it for Go 1.11. Leaving NeedsInvestigation but @tombergan please feel free to make a decision and switch to NeedsFix.

I'll move it to Go 1.12.

In Go 1.12 we now have the fix for https://github.com/golang/go/issues/19943#issuecomment-421092421 ....

Change https://golang.org/cl/147457 mentions this issue: net/http: make Transport respect {X-,}Idempotency-Key header

That should be sufficient for this bug.
See the docs on the Transport type at https://tip.golang.org/pkg/net/http/#Transport

more TCP layer detail, if some guys want to know the reason:)
See the server side send the FIN packet

Chromium checked whether socket reused before sending retry request, I wonder why it is necessary for an ECONNRESET error, if the server is not handling on the port, you will get an ECONNREFUSED error instead, so maybe we can retry every time running into an ECONNRESET error. Or ECONNREFUSED is just Linux implementing detail? I didn't find any RFC describe its behavior. Correct me if I'm wrong.

Got explain from chromium dev

Servers can timeout idle connections at their leisure and restart at their leisure.  As a result, even if network conditions are perfect, reusing a stale socket can get you ERR_CONNECTION_RESET - so if we ever want to be able to reuse sockets, we need to retry on resets.  A server returning ERR_CONNECTION_RESET on a fresh socket, however, means that there is some sort of server (or middlebox) issue.  And yes, we can get a reset at that point (buggy SSL implementations seem to be big fans of resetting sockets during SSL negotiation when they're sad, for instance).


Unified Pagination

Many requests have a union { null, int } pageSize = null; and a union { null, string } pageToken = null;. We should consolidate these fields into a single type in common.avdl, so we don't have duplicate fields and comments everywhere.

This has been dormant since November. I am closing in 2 days.

These are great suggestions - both the original (encapsulate pagination) and the follow-up, to allow the return of multiple page tokens at once.
The former (encapsulation) would make implementing the latter easy, since a change to our pagination model would only need to be made in one place.

Thanks for the comment @macieksmuga I'll leave this open for now and revisit on 1 month.

@skeenan I appreciate it, thanks! If you haven't seen a PR from me on this topic by then, then it means that I was argued out of it :)

@ejacox what do you think? Less cruft in the proto for the paging elements.


Explore Eureka integration options

As a developer, I'd like to investigate integration options with Eureka. Following are few possible (SCS) integration plays.

[ ] Add support to store metadata in Eureka
As a user, I'd like to store module metadata in Eureka, so I can use the repository to determine the current state.
[ ] Add support for modules to register itself to Eureka
As a user, I'd like to have the modules self-register itself with Eureka whenever they're installed, so I can also discover the same modules using Data Flow Admin SPI and reuse them to create data pipelines.
[ ] Add Eureka Registration Capabilities
Can't a module just register with @EnableDiscoveryClient? Or do you want to add some special metadata for stream modules?
[ ] Make s-c-s modules searchable by its name
As a user, I'd like to search the modules by it's name aside from the default spring.application.name offered by boot, so I can also fetch modules by it's name.

This issue was moved to spring-cloud/spring-cloud-stream-app-starters#257


fix: dialtone-icons dependency

Fix dialtone-icons dependency
:hammer_and_wrench: Type Of Change

[x] Fix
[ ] Feature
[ ] Refactoring
[ ] Documentation

:book: Description
Removed dialtone-icons from dependencies and moved into devDependencies and peerDependencies.
:bulb: Context
Errors are being reported around dialtone-icons not found on clients using dialtone-vue.
:pencil: Checklist

[x] I have reviewed my changes
[ ] I have added tests
[ ] I have added all relevant documentation
[ ] I have validated components with a screen reader
[ ] I have validated components keyboard navigation
[x] I have considered the performance impact of my change
[x] I have checked that my change did not significantly increase bundle size
[ ] I am exporting any new components or constants in the index.js in the component directory
[ ] I am exporting any new components or constants in the index.js in the root

This will work but I'm still questioning whether dialtone-icons should be a peer dependency or not since DtIcon is part of the package and won't run without it. I guess the same is true of dialtone which is also a peer dependency...
Another option is to separate a different build target like we do for emoji. How big is the icons bundle in relation to the dialtone-vue package? If very large I think that will be the best option.

Dialtone-vue

Dialtone-icons

It almost as big a Dialtone-vue.
Why the peerDependency is not a good option? I think it would work perfectly fine and will not increase bundle size.

Why the peerDependency is not a good option? I think it would work perfectly fine and will not increase bundle size.

It only decreases bundle size if you are NOT using dialtone-icons, which most projects will be.
If we create a separate build target for icons like we did for emoji it would:

still not increase the bundle size of the core package as it's in a different file
consumer does not have to be responsible for dialtone-icons versioning
(most importantly) the icons bundle could be dynamically loaded in the app when it needs to be used rather than all loaded up front.

Think we can just close this one it's working fine in firespotter again. Can create additional tickets for bundle size / code splitting improvements.

Agree, closing this for now.


Bump kernel version to 4.7.0-m10

$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/1754360727

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/1754360727
Status: failure

locally tested and verified the Keystore cert renewal with updated samples-is (v4.3.8) and kernel 4.7.0-alpha10


Need dependencies offline download VSIX

Unlike the C++ plugin which offers full plugin offline support via VSIX, this plugin needs mandatory internet connection for dependency download.
Fix this please.

@xCyborg We have support so that you can create vsix on a computer with internet and once created you can use it on a computer without internet. Please follow the instructions here https://github.com/OmniSharp/omnisharp-vscode/wiki/Installing-the-C%23-extension-to-a-computer-without-internet-connectivity#installing-on-a-computer-that-cannot-connect-to-the-internet to do so

@xCyborg We haven't heard back from you, so I'm going to close this.

@akshita31 It's too much hassle to implement especially on Windows, can't you just package the VSIX for each major version and make it available for download, please?


Clean up permissions for Level 1 & Level 2

I’m able to access the Trail Alerts page as a Level 1 user and post closures (.../alerts/trail). Unlikely that a Level 1 user would figure out how to get to this page and post something, but if we can cut access completely, that would be good.

Other pages that should be cut off for both Level 1 and Level 2 users:
.../pointsofinterests
.../new_trails
.../trails_infos
.../trails_descs
.../pointsofinterests
.../activities
.../picnicgroves (I couldn’t get this page to load even with Admin account)

Looks resolved.


get rid of the VITE_REDIRECT_URI dependency

Load this variable (VITE_REDIRECT_URI) differently.  Instead of having to set it in github and loading it via the action since this is always matching the domain of the currently deployed app we should just use something else to determine that value (either from Azure deploy or locally).

should not even be needed in the first place as suspected: https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/dev/lib/msal-browser/docs/initialization.md#optional-configure-redirect-uri
we can simply remove this dependency in the github workflows and in our code.


Error due to Youtube restriction "Sign in to confirm you’re not a bot."

New Youtube login policy has broken the update of user playlists and audio/video player, making the app unusable for now.

yt-dlp issue : https://github.com/yt-dlp/yt-dlp/issues/10128
Potential workaround : https://github.com/jagrosh/MusicBot/issues/1588#issuecomment-2295305229


[ENHANCEMENT] STREAM DECK

I would love for there to be a version of this that i can use on my stream deck while in games, as when i use my Alexa i can say mayday mayday mayday and the lights turn red, it would be amazing if we could have preset keys on our stream decks to do it for us.
Describe alternatives you've considered
i have considered setting up a raspberry pi to connect instead of the box and just hosting a web controller from it. and then using an app on my pc to control it, ive thought about using a usb sniffer to make my own control panel from my pc, but then i found this.
do i have your permission to use your code and make a twitch bot that can control my lights from my pc depending on what the users ether type in chat or if they donate, or if they use channel points to make it strobe or something like that.

This repo is licensed with MIT license, so you can do whatever you want :)


Error when I installed and tried to launch project

Sorry I am completely new to this engine (and 3D development in general) but after I installed version <IP_ADDRESS> I received this error:
An error occurred while installing version <IP_ADDRESS>. Value cannot be null. Parameter name: path1 at System.IO.Path.Combine(String path1, String path2) at Xenko.Core.Packages.NugetStore.OnPackageInstalled(Object sender, PackageOperationEventArgs args) at Xenko.Core.Packages.NugetStore.d79.MoveNext() --- End of stack trace from previous location where exception was thrown --- at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) at Xenko.LauncherApp.ViewModels.PackageVersionViewModel.<>cDisplayClass550.<b_0>d.MoveNext()
And then when I launched a project it said it couldn't find Microsoft.Net.Sdk
This is probably something very simple and I'm just lost. (Plus I've worked 63hrs this week so my mind isn't functioning at it's best.)

try to install visual studio 2017 with c# support to get the .net sdk

Thanks I'll try. I thought I already had it installed when I installed
unity but I'll recheck.
On Thu, Aug 2, 2018, 11:15 AM pikachuk<EMAIL_ADDRESS>wrote:

try to install visual studio 2017 with c# support to get the .net sdk
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/xenko3d/xenko/issues/6#issuecomment-409982771, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AoFmZUyN-OHPF00hMv_Drjybs1WNFMy5ks5uMyWXgaJpZM4VsLNc
.

not really, since Unity Engine uses mono framework, it's not the same as xenko, xenko uses the native .net framework
make sure that you have installed it
you could try to uninstall and reinstall xenko 3.0 too, i had the same install error but it goes away after reinstalling it


Istio causes Prometheus memory leak caused by OOMKilled

Bug Description
Increased memory from 6GB to more than 70GB using Prometheus Operator to monitor Istio.
As shown in the figure below

Version
prometheus version
v2.11.0

# istioctl version
client version: 1.10.4
control plane version: 1.10.4
data plane version: 1.10.4 (9 proxies)

# kubectl version --short
Client Version: v1.16.1
Server Version: v1.16.1

# helm version --short
v3.5.3+g041ce5a

Additional Information
Prometheus istio dataplane monitoring configuration
# cat prometheus-oper-istio-dataplane.yaml
apiVersion: monitoring.coreos.com/v1
kind: ServiceMonitor
metadata:
  name: prometheus-oper-istio-dataplane
  namespace: monitoring
  labels:
    monitoring: istio-dataplane
spec:
  selector:
    matchExpressions:
      - {key: istio-prometheus-ignore, operator: DoesNotExist}
  namespaceSelector:
    any: true
  jobLabel: envoy-stats
  endpoints:
  - path: /stats/prometheus
    targetPort: http-envoy-prom
    interval: 15s
    relabelings:
    - sourceLabels: [__meta_kubernetes_pod_container_port_name]
      action: keep
      regex: '.*-envoy-prom'
    - action: labelmap
      regex: "__meta_kubernetes_pod_label_(.+)"
    - sourceLabels: [__meta_kubernetes_namespace]
      action: replace
      targetLabel: namespace
    - sourceLabels: [__meta_kubernetes_pod_name]
      action: replace
      targetLabel: pod_name

Prometheus istio controlplane monitoring configuration
# cat prometheus-oper-istio-controlplane.yaml
apiVersion: monitoring.coreos.com/v1
kind: ServiceMonitor
metadata:
  name: prometheus-oper-istio-controlplane
  namespace: monitoring
spec:
  jobLabel: istio
  selector:
    matchExpressions:
      - {key: istio, operator: In, values: [pilot,galley,citadel,sidecar-injector]}
  namespaceSelector:
    any: true
  endpoints:
  - port: http-monitoring
    interval: 15s
  - port: http-policy-monitoring
    interval: 15s

I see I have the same problem Prometheus Operator - OOM killed when enabling Istio monitoring, but I don't know how to deal with it. I need help, thanks.

@openstack-test would be great if you can look into your prometheus and get which dimension contribute the most dimension. It is likely that the destination_service label is unbounded. See How can I manage short-lived metrics? at https://istio.io/latest/about/faq/#metrics-and-logs.

@bianpengyuan  Thank you for your reply. I still do not understand how to deal with the problem of Prometheus OOMKilled caused by Istio


Microsoft.Azure.EventHubs.Processor

I believe the Get Started section should indicate we must include the package Microsoft.Azure.EventHubs.Processor to use EventProcessorHost.

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

ID: b109d8b1-4a76-6c2c-b7a1-f473c3723a68
Version Independent ID: 2ffb9a42-52e2-8aa1-86cc-d1033c42ec49
Content: Programming guide - Azure Event Hubs
Content Source: articles/event-hubs/event-hubs-programming-guide.md
Service: event-hubs
GitHub Login: @ShubhaVijayasarathy
Microsoft Alias: shvija

@Silverlight42 Thanks for your feedback! We will investigate and update as appropriate.

@Silverlight42  I have assigned the issue to the content author to review further and update the document as appropriate.

@Silverlight42 - The EventProcessorHost hyperlink takes me to the following API page that shows the namespace details about the class.
https://docs.microsoft.com/en-us/dotnet/api/microsoft.azure.eventhubs.processor?view=azure-dotnet.
I can include a note in the article but I am wondering if you looked at the hyperlink.
In the quick starts, we do have steps to add this package:
https://docs.microsoft.com/en-us/azure/event-hubs/event-hubs-dotnet-framework-getstarted-receive-eph#add-the-event-hubs-nuget-package
https://docs.microsoft.com/en-us/azure/event-hubs/event-hubs-dotnet-standard-getstarted-receive-eph#add-the-event-hubs-nuget-package

#in-progress

@Silverlight42 We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Api.Restapi().Account().Presence().Put() Does not Exist

The docs at this link say it should:
https://github.com/ringcentral/RingCentral.Net/blob/master/samples.md

No, the docs don't say it should.
The docs only mentioned GET /restapi/v1.0/account/{accountId}/presence
There is no PUT /restapi/v1.0/account/{accountId}/presence.
Feel free to re-open it if you have questions regarding this.

The doc do say that PUT /restapi/v1.0/account/{accountId}/extension/{extensionId}/presence is supported.
You should try Api.Restapi().Account().Extension().Presence().Put() instead of Api.Restapi().Account().Presence().Put()


Protect URLs from potentially leaking unpublished pages

Live pages have the possibility of linking to pages that are not yet live, which is fine. However, what's not fine is the information that's leaked from that link. This protects that information by replacing non-live page links on the live site with #. It also fixes the 404 issue for Featured Content modules.
Additions

context function that returns the protected URL

Changes

page links should use get_protected_url instead of get_page_state_url

Testing

Go to http://localhost:8000/data-research/research-reports/ and click on the See Full Report link on the featured content component to make sure that link is okay.
Go to that full report's edit page and unpublish it.
go back to http://localhost:8000/data-research/research-reports/ and verify that the link to the full report has been replaced with a #

Review

@richaagarwal
@rosskarchner
@kave

Good catch!  :+1:


[$500] MISSING_TRANSLATION is shown in pronoun field even without any pronouns set

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?: needs reproduction
Reproducible in production?: needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers): @quinthar
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @quinthar
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1711338168652399
Action Performed:

Login to an account in which no pronouns set
Open the profile page

Expected Result:
Pronoun field is empty
Actual Result:
Appears as ``Missing Translation" Also there are console logs
[alrt] pronouns.He/him was not found in the en locale - {"stack":""Error\n    at a.alert (https://staging.new.expensify.com/vendors-406b3bb5f059dfb66309.bundle.js:1:3900169)\n    at Module.B (https://staging.new.expensify.com/main-93cf1b44624c1fe66191.bundle.js:1:2335910)\n    at https://staging.new.expensify.com/main-93cf1b44624c1fe66191.bundle.js:1:1771873\n    at R (https://staging.new.expensify.com/main-93cf1b44624c1fe66191.bundle.js:1:3953628)\n    at Co (https://staging.new.expensify.com/vendors-406b3bb5f059dfb66309.bundle.js:1:5860880)\n    at Ms (https://staging.new.expensify.com/vendors-406b3bb5f059dfb66309.bundle.js:1:5881792)\n    at vl (https://staging.new.expensify.com/vendors-406b3bb5f059dfb66309.bundle.js:1:5918898)\n    at bc (https://staging.new.expensify.com/vendors-406b3bb5f059dfb66309.bundle.js:1:5907143)\n    at gc (https://staging.new.expensify.com/vendors-406b3bb5f059dfb66309.bundle.js:1:5907071)\n    at mc (https://staging.new.expensify.com/vendors-406b3bb5f059dfb66309.bundle.js:1:5906934)""}
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~019e0d665061e60d96
Upwork Job ID:<PHONE_NUMBER>176807424
Last Price Increase: 2024-04-01

I like going with solution 2, and I would like to recommend to use an Onyx migration instead of using the profile page to correct the pronouns.

Issue not reproducible during KI retests. (First week)

Issue not reproducible during KI retests. (Second  week)

Payment has been issued. @ntdiary, can you please complete the BZ list above?

I don't feel passionately about creating a regression test. 👍


TestTLSReloadAtomicReplace: grpc: timed out when dialing

via semaphore https://semaphoreci.com/coreos/etcd/branches/pull-request-8213/builds/2
--- FAIL: TestTLSReloadAtomicReplace (2.88s)
	v3_grpc_test.go:1790: expected no error, got grpc: timed out when dialing

TestTLSReloadCopy is hitting this too (https://semaphoreci.com/coreos/etcd/branches/master/builds/2143)
--- FAIL: TestTLSReloadCopy (3.29s)
	v3_grpc_test.go:1790: expected no error, got grpc: timed out when dialing

Should we just bump up the dial timeout (current 1-second)? I have been running this test over a week with slow CPU machine, but still cannot reproduce.

@gyuho ok let's bump to 5s and reopen if it's still failing


[ci] Upload coverage to Codecov

Let me know when you want me to merge it :)

Right now would be great. I don't believe I can test it on pr.

Oh, actually it worked.

Thank you!


Impro 1476 accumulate

Accumulate
Fully Clize
This new CLI is argparse-less
inputadvection
inputadvection is a function which will do the loading of u and v, this will be used for nowcast-extrapolate.
Test
old0 = 'old/nowcast-extrapolate/accumulation/kgo0_1_minute_fidelity.nc'
old0 = load_cube(old0)

old1 = 'old/nowcast-extrapolate/accumulation/kgo1_1_minute_fidelity.nc'
old1 = load_cube(old1)

new = 'new/nowcast-accumulate/basic/kgo.nc'
new = load_cube(new)

(old0.data.data == new[0].data.data).all()
True

(old1.data.data == new[1].data.data).all()
True

Testing:

[x] Ran tests and they passed OK
[x] Added new tests for the new feature(s)

Other
extrapolate
Reordered 2 loads so it does (u->v) x->y this is to avoid confusion. (And it was really buging me)
Acumulation frequency
accumulation_frequency is now declare as 1 (min).
This is compared to whe naccumulate was done by extrapolate and it can a kwarg for the user.
P.S
This was branched off of #1000 so it includes all of it's commits.

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 3

Complexity increasing per file
==============================
- lib/improver/cli/nowcast_accumulate.py  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- lib/improver/cli/nowcast_accumulate.py  1

See the complete overview on Codacy


⚠ Bofrost.de has degraded performance

In 93e2f11, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1165 ms

Resolved: Bofrost.de performance has improved in 7f22921 after 306 days, 16 hours, 13 minutes.


Handle structural conflicts

Related to #4
This might be a bit tricky.

It's indeed a bit tricky. Considering only a top-down diff algorithm, structural conflicts are easy to represent as whenever a structural conflict is encountered, you can be sure that none of the child nodes of either node in the conflict will match (because, you know, top-down diffing). So, when there's a conflict, you can simply pretty-print the conflicting LEFT nodes and their entire subtrees, and then the conflicting RIGHT nodes and their entire subtrees, delimited by conflict markers.
However, that's not the case with GumTree. Children of nodes involved in structural conflicts can themselves be matched and non-conflicting, so the approach outlined above won't work. Or, well, id isn't incorrect, it's just far less granular than optimal.

I think that #28 addresses this well enough that this can be closed. Separate issues for improvement to be opened later.


[DOCS] Adding Code of Conduct

Description
We propose adding a comprehensive Code of Conduct to our repository to ensure a safe, respectful, and inclusive environment for all contributors and users. This code will serve as a guideline for behavior, promoting diversity, reducing conflicts, and attracting a more comprehensive range of perspectives.
I want to work on this issue under GSSOC23, kindly assign this to me @LakshaySK106.
Screenshots
No response
Additional information
No response

I have added a code of conduct kindly review it! @LakshaySK106. Thank You!


Nullable

make code compatible with C# 8 nullable: https://docs.microsoft.com/en-us/dotnet/csharp/whats-new/tutorials/nullable-reference-types

Hi @VladislavAntonyuk
First of all, thank you for your feedback, I am always open to comments/feedback. I like the idea to support "Nullable Reference Types", I will include this feature ASAP.
Here is my proposal (Taking the string type as an example): If we try to assign "typeof(string?)" as "declaringType" parameter to the method "BindableProperty.Create", we will receive a compilation error, then based on the documentation the nullable reference types are a compile-time feature, that means we can set that parameter like "declaringType = typeof(string)" instead of "declaringType = typeof(string?)".
    using Maui.BindableProperty.Generator.Core;

    public partial class CustomEntry : ContentView
    {
        [AutoBindable]
        private string? _text;
    }

    public partial class CustomEntry
    {
        public static readonly Microsoft.Maui.Controls.BindableProperty TextProperty =
                                    Microsoft.Maui.Controls.BindableProperty.Create(
                                                            nameof(Text),
                                                            typeof(string), // Here the change
                                                            typeof(CustomEntry),
                                                            default(string?));

        public string? Text
        {
            get => (string?)GetValue(TextProperty);
            set => SetValue(TextProperty, value);
        }
    }

Hi @VladislavAntonyuk , I just sent a new release (Version 0.8.0) that includes the new functionality to support "Nullable Reference Types" Here is the commit. Again, I am always open to comments/feedback, Thanks !!

I receive this error:  CS8669	The annotation for nullable reference types should only be used in code within a '#nullable' annotations context. Auto-generated code requires an explicit '#nullable' directive in source.

I will create a PR

Hiya, I'm not seeing this feature functioning as intend in 0.9.2
I.e., I am running .NET 7 and I have the following class:
    public partial class OpenShiftViewModel : BindableObject
    {
        [AutoBindable()]
        private readonly string? _facility;
        [AutoBindable]
        private readonly string? _department;
    }

And the output looks like such:
    public partial class OpenShiftViewModel
    {
        [global::System.CodeDom.Compiler.GeneratedCode("Maui.BindableProperty.Generator.Core.BindableProperty.AutoBindablePropertyGenerator", "<IP_ADDRESS>")]
        public static readonly Microsoft.Maui.Controls.BindableProperty FacilityProperty =
                                        Microsoft.Maui.Controls.BindableProperty.Create(
                                                                nameof(Facility),
                                                                typeof(string),
                                                                typeof(OpenShiftViewModel),
                                                                defaultValue: default(string));

        [global::System.CodeDom.Compiler.GeneratedCode("Maui.BindableProperty.Generator.Core.BindableProperty.AutoBindablePropertyGenerator", "<IP_ADDRESS>")]
        [global::System.Diagnostics.CodeAnalysis.ExcludeFromCodeCoverage]
        public string Facility
        {
            get => (string)GetValue(FacilityProperty);
            set => SetValue(FacilityProperty, value);
        }

        [global::System.CodeDom.Compiler.GeneratedCode("Maui.BindableProperty.Generator.Core.BindableProperty.AutoBindablePropertyGenerator", "<IP_ADDRESS>")]
        public static readonly Microsoft.Maui.Controls.BindableProperty DepartmentProperty =
                                        Microsoft.Maui.Controls.BindableProperty.Create(
                                                                nameof(Department),
                                                                typeof(string),
                                                                typeof(OpenShiftViewModel),
                                                                defaultValue: default(string));

        [global::System.CodeDom.Compiler.GeneratedCode("Maui.BindableProperty.Generator.Core.BindableProperty.AutoBindablePropertyGenerator", "<IP_ADDRESS>")]
        [global::System.Diagnostics.CodeAnalysis.ExcludeFromCodeCoverage]
        public string Department
        {
            get => (string)GetValue(DepartmentProperty);
            set => SetValue(DepartmentProperty, value);
        }
}

Am I missing some here?

Hi, @tmose1106 thank you for your feedback, the fix is ready in version 0.9.3, you can find it here:
https://www.nuget.org/packages/M.BindableProperty.Generator/0.9.3


Documenting num_hexagon recurrence...

...because I messed it up today.

Coverage remained the same at 99.132% when pulling 2f0599d487da1fe2b5648a77c02aae7a4c1cdfa5 on ajfriend:recur into 56cafd158c2cf858246c7735e257ba33b88f6e20 on uber:master.

Aren't we renaming this in 4.0?

We are! At that point, the "actually" comment is no longer needed. In the meantime, why not?


Human reader stopped working with new API key

{
"detail": {
"status": "detected_unusual_activity",
"message": "Unusual activity detected. Free Tier usage disabled. If you are using a proxy/VPN you might need to purchase a Paid Plan to not trigger our abuse detectors. Free Tier only works if users do not abuse it, for example by creating multiple free accounts. If we notice that many people try to abuse it, we will need to reconsider Free Tier altogether. \nPlease play fair and purchase any Paid Subscription to continue."
}
}
it's because of this error, we should just tell the user

fixed: https://github.com/sebhs/human-reader-chrome-extension/pull/28


[DOCUMENATION] Backlog Update

Description
Making sure that all the user stories are well described

Update the user stories
add the acceptance criteria

https://github.com/DIT112-V21/group-17/wiki/User-stories-backlog


Fix Data Output Bug in the NN Testing

No data output even if told to do so through print.
Run the script on your machine in pycharm or another IDE, no list is output like it should be.
The program should output a list to the console, not even defaults are outputting properly.
OS:
Windows 10

Found main issue, no output due to no input prompt

Fixed in code, not posting fixed code due to invalid outputs though.


Reapply "Add Query tab (#113)" (#119)

This reverts commit 0f4ce796d38805d128777a9a690037c3984aadc4.

leaving this ready for when we want to land it


LOD Updates in Cruise Phase

I and a few others over at AVSIM here have observed that the LOD values stay the same when in cruise, which is fine if you are way up high in an airliner, but not so much when cruising lower over undulating territory like in the elevation profile attached.

I have experience with coding over the last 40 years, but am new to the github form of collaborative development. Nonetheless, I managed to download your source code, find the area where LODs are chosen and modified the code so that in cruise it will change LOD but only if it exceeds a hysteresis threshold for the next AGL bracket. I managed to compile it and get it to work with great results on my system but am unsure how to submit the suggested code changes, so I will just attach them here (had to put a txt extension on the file otherwise it wouldn't upload).
LODController.cs.txt
In short, I added two extra else if cases at line 152 of LODController.cs that don't care about VS trend but will only trigger once the hysteresis threshold has been breached, which I have currently set to +/-5% but this could always be user selectable in the GUI if you want to spruce it up a bit. The original climb and descent logic remains intact and that is what is executed first if applicable.

Ok I put aside for the Moment that I stated not being interested in Suggestions and that you have reported that Issue in the wrong Repo first.
Then still: I don't see any Benefits and only Drawbacks. The whole Idea to consider the VS (SimVar VERTICAL SPEED) is to NOT change the LOD just because the Terrain climbs and descends while the Plane does not. So ignoring that just leads to Situations of constantly changed LODs during Cruise.
And generally I just can't follow why hysteresis should be applied here at all?! There is no delay between Action and Cause . I'm not an Engineer ofc and only now that from Thermostats ... but: That type of Oscillation is exactly what was meant to be excluded by that.
Sure, there are Corner-Cases like shown in your Flight Profile. Descending "so near to the Ground" but still high above your Destination will reduce LODs earlier than it would be needed. BUT: The main Goal of having a reduced LOD on Ground (on your Destination) is given and each Decrement is exactly executed once and Increments are excluded.
I'm aware of Cases where DynamicLOD somehow skips Pairs respectively did not increase to the intended LOD for Cruise where I have also a Log for that. Have to look into this further, it almost seems as the ref Keyword would not work - which can't really be. It finds a higher Pair, still does not want to change to that. Maybe I just add a special Case for Cruise (zero Trend) or something like that.
When I have Time for that. Don't see it really as urgent Issue, since using a lower LOD on while Ground is still the Case.

Perhaps null zone is a better term to use than hysteresis in this case. The reason for its inclusion is precisely to NOT have LOD changing in cruise with only small AGL deviations. I've done a number of test flights with the change and it does not constantly change LOD like you suggest it would. The original climb/descent logic remains the same, it is only in cruise that it changes and even then it is only when cruising at low level as the null zone is quite high at high altitude and the LODs stay the same.
As such, IMO, and it seems a few others over at AVSIM, it makes it better for low level IFR and has no effect on your original objective of reducing LOD at airports on the ground or while climbing or descending. Perhaps you should try it before dismissing it entirely. I hope you do see the benefit and incorporate this change.

I will also add that the change I made also fixes an issue where you change profile mid flight while in cruise and it does not go to the correct LOD pair unless you descend or climb a little to trigger the recalculation. It also resolves the issue you stated where it is skipping pairs and not being in the right LOD pair at cruise because it is now active at cruise and will fix that right away.
If you are still worried about LODs toggling too often in cruise then just make the null zone higher than the 5% I chose. eg. if you make it 20% and you are cruising at 30000 ft and have a LOD trigger at 30000 ft, then it will only change if AGL changes by 6000 ft which would be extremely rare for all those conditions to exist at the same time. ie. for the most part in airliner cruising, this code lays dormant.

Just to illustrate how this change minimally impacts airliner ops, attached is a log file of an airliner flight I made today over north east USA and at no time during cruise (between 09:20:53.639 and 09:53:42.257) did my code changes activate.
DynamicLOD20240115.log

Was that a Step Descent?

Constantly writing to Memory, not Good.

Descent was normal, not stepped.
Those OLOD 30 lines do not come from my code changes as log entries from my code do not start with LODController:RunTik. They were in the climb phase or descent phase, where my code is not active.

Ah, I know what those OLOD 30 lines are from. When I am flying in VR and take the headset off, I Iimit FPS to 20 to save power and noise. For some reason that runtick code is triggering despite me not having FPS adaption selected as shown which is a bit strange, but not from my code changes.

Here's my log from 2 days ago before I made any changes to the code and it has quite a few runtick OLOD 30 lines too.
DynamicLOD20240113.log

I'll try and quantify what is triggering the OLOD 30 issue and will get back to you.

Sorry if the Exception in actually answering an Issue made a an false Impression ... I'm not really interested in "will get back to you".

I'm not really sure why you are getting offended. I just answered your questions with facts. I'm sorry you feel that way.
BTW, the OLOD 30 issue is happening in the climb in a test I am doing right now, even without FPS cap on.

I found what is causing the RunTick OLOD 30 line to keep occurring. For some reason when my aircraft is above 5000 ft but below 10000 ft, so is using the OLOD 30 at 5000 ft line, the olod value being compared on line 110 is actually 30.000002 against newlod value of 30, hence is not equal and is triggering the code to set OLOD to 30. All of the other OLOD values (100, 70, 50 and 10) show as whole numbers but 30 is being read as 30.000002, even when I reentered the data in the app for that level.
As a quick fix I modified the line 110 to be:
if (Math.Round (olod) != newlod)
and the erroneous log entry and OLOD setting goes away. While this fixes it, I'd be more worried what is setting olod to 30.000002 in the first place as that is not the value in the table.
Anyway, FYI.

I'm not really sure why you are getting offended. I just answered your questions with facts. I'm sorry you feel that way.

Uhm - I'm not getting offended, I just lost interest in the Discussion. Being disinterested is not the same as being offended?

I mean honestly, what exactly did you expect? To summarize: You have the Idea of making a Suggestion to a Tool where the very Description of that Tool already states that there is no Interest in Suggestions. Then you go ahead and formulate your Suggestion to the wrong Project/Repo first. Then the Suggestions is something to work-around the Tool's Core-Principle. A Suggestion based on a wrong Assumption (to quote: "... will attempt to modify it to continuously select LOD based on AGL if in cruise mode ..."). And which even actually enables said Behavior.
So your initial Conditions could not have been worse.
So in that Sense you where quite successful imho, since the standard Reaction to any Issue here is an instant-Close (sometimes not even with a Comment). But you certainly put considerable Effort in your Suggestion, so I don't wanted it to answer in that Way - albeit the Fact that it was getting late here and I was still working on other Stuff I just care more about.
And certainly it wasn't all in vain. Your Findings confirm my Idea that there needs to be some kind of an additional Checks/Cases (if it actually should attempt to change the LOD based on the AGL when in Cruise). And the Finding that we actually can have Precision Errors here is really really good. (Which is possibly a Regression of Simplification ...)
But what is also certain: I won't accept your (initial) proposed Changes. I won't give up on the Principle that the vertical Behavior of the Airplane has to be taken into Consideration.
After all it is an Open-Source Project under MIT License. So if you think that another Approach is more reasonable, there is nothing stopping you from doing a Fork, making the Changes and publishing that on your forked Repo. Heck, you can even do more welcoming Description and answer Issues & Suggestions of Users.


bootstrap-validatior automatically removes has-error class from form-group

I have a problem whenever after server side form fields validation if there are som errors return back and after that, I put has-error class to those form-group those having errors. When the page is fully loaded bootstrap-validatior automatically removes has-error class. How to prevent that?

This should now be fixed in v0.11.7. The validator won't automatically re-run validation of fields that already have errors upon plugin init.

Thanks it's working now. :smile:


Lockup Module

The lockup module limits what sort of messages are allowed to be sent while the chain is locked. It does so by exposing an AnteHandler to be run on every transaction entering into the mempool.
Currently the only types of messages blocked are the bank module's MsgSend and MsgMultiSend.

Happy with the implementation here. I think we still have some work to do on making sure the chain can start by default so that all the tests get back to working.

Codecov Report

Merging #16 (1b913da) into main (c0218e5) will decrease coverage by 0.39%.
The diff coverage is 11.42%.

@@            Coverage Diff             @@
##             main      #16      +/-   ##
==========================================
- Coverage   12.62%   12.23%   -0.40%
==========================================
  Files           9       13       +4
  Lines         713     1161     +448
==========================================
+ Hits           90      142      +52
- Misses        605      998     +393
- Partials       18       21       +3

Impacted Files
Coverage Δ

app/app.go
0.85% <0.00%> (-0.04%)
:arrow_down:

x/lockup/module.go
0.00% <0.00%> (ø)

x/lockup/types/genesis.pb.go
0.88% <0.88%> (ø)

x/lockup/ante.go
67.64% <67.64%> (ø)

x/lockup/types/genesis.go
70.27% <70.27%> (ø)

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 c63036a...1b913da. Read the comment docs.

@ChristianBorst
A few final changes
the commits "Adding protobuf target and script" and "Update dependencies" should probably be combined as the added dependencies are proto related.
Every lockup module related commit "Adding lockup module" through "Disable lockup module by default" should also be squashed and the lockup module commit message should expand on things like why it's disabled by default and a bit about the design decisions.


Create ledger if one doesn't exist

Add functionality to create a ledger csv db if one doesn't already exist.

if the ledger does not exist the open() function creates one.


Revert "[SPARK-36136][SQL][TESTS] Refactor PruneFileSourcePartitionsSuite etc to a different package"

This reverts commit 634f96dde40639df5a2ef246884bedbd48b3dc69.

Test build #141686 has started for PR 33533 at commit a733179.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46200/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46200/

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

Test build #141686 has finished for PR 33533 at commit a733179.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class PruneFileSourcePartitionsSuite extends PrunePartitionSuiteBase
class PruneHiveTablePartitionsSuite extends PrunePartitionSuiteBase
abstract class PrunePartitionSuiteBase extends StatisticsCollectionTestBase with TestHiveSingleton

Merged to master and branch-3.2.

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


Visdom posts to localhost server failed connect via proxy status 407 when not root usr

hello
I have run visdom server (python3 -m visdom.server)on my server. and i can open the link "http://localhost:8097".
but i try to excute python demo like:
import  visdom
import torch as t
vis = visdom.Visdom()
x = t.arange(0,30,0.01)
y = t.sin(x)
vis.line(X=x,Y=y,win='sinx')

i got this error from python running time:
failed CONNECT via proxy status: 407
failed CONNECT via proxy status: 407
Visdom python client failed to establish socket to get messages from the server. This feature is optional and can be disabled by initializing Visdom with `use_incoming_socket=False`, which will prevent waiting for this request to timeout.

I tried to use_incomming_socket=False option,but  when i refresh browser ,its still empty.
I guess python can not connect to visdom server.
Hope someone can help.Thanks

Oh, I can't open the link...

Are you running the demo on the same machine that you're running the server on? Do you have any errors in your javascript console?

Yes.i m running the demo on the same machine which i m  running the server on.
i do have some error in js console like follows, but i dont think thats the problem:

And if i run the demo with root usr, it works fine,i can see the visualization in browser. but if i run the demo with normal usr , i get the proxy error.

It's possible you need to specify username and password authentication into your session when not running as the root user, according to the requests library. Relevant code location for that would be here: https://github.com/facebookresearch/visdom/blob/db27b3483e118d76c320a7817b10c2f60e38b774/py/visdom/init.py#L405

I had a similar situation.
You need to specify a server.
Do not enter localhost, change it to enter ip.
vis = visdom(server='http://X.X.X.X')

This is properly supported with proper proxy handling since #538.


LaserMix with cuda 12.4 and Pytorch 2.4

On your official page you state that it possible to run LaserMix with newer versions of Pytorch. I have cuda 12.4 with Pytorch 2.4, but I am not able to run and receive the following error: ImportError: libGL.so.1: cannot open shared object file: No such file or directory.
I am not at the moment able to switch to the older version of cuda eg cuda 11. Please, help.

Hi @sputnik4, thanks for your interest in this work.
For your question:
We didn't try using PyTorch > 2.0 and CUDA > 11.8. Can you please attach more details about the error message, so I can try to provide some suggestions?

Thank you for quick response. I have had to switch to another machine (which has AVX cpu) and downloaded a docker container that follows your environment. I am now able to train on SemanticKITTI dataset. However, I now have a custom dataset and I want to know how you created the .pkl files for semi-supervised training?


🚀 Feature: Upload files with CURL

🔖 Feature description
Would be cool to have the ability to be able to upload files with CURL and just have the generated URL be returned.
Allow unauthenticated if unauthenticated shares are enabled, or authenticated through a token connected to your account.
🎤 Pitch
Be able to easily upload files through the terminal, integrate into scripts etc.. The sky's the limit!

It's actually already possible, though not documented.
There are a couple of gotchas, and it might break when the new features come in, but at this time I have a working script to upload files.

You need to prepend files with at least a comma ,, the exact format the frontend uses is data:application/octet-stream:base64,
You need to base64 files before sending
You need to specify the Content-Type header with the value set to application/octet-stream specifically
You need to use --data-binary @${fileName} to set the body.
For files larger than 10Mb, you have to chunk the files into exactly 10Mb chunks.
Example for the command: curl -v -X POST -sH "content-type: application/octet-stream" -b ${auth} --data-binary "@${fileName}" ${pingvinUrl}/shares/${shareId}/files$args

If you need a clearer example of this, you can upload a file using the frontend and inspect the network calls.
Hope this helps!

I am also looking to upload files through the terminal via CURL. Been looking for a simple, easy to setup, lightweight app that can accept files from CURL without any authentication. Seems to be a bigger ask than I would have figured though Pingvin seems to be the closest thus far!
@tdompierre
Since you've gotten CURL to work with Pingvin, I'm hoping you can provide some insight. I am not a developer so forgive me for my ignorance. I have no idea how to even begin with inspecting the network calls between the frontend and backend. I have Pingvin running in a Docker container on UnRAID. I just setup Netdata in another docker container and hope that I can use that for inspecting the network traffic.
Are there any tools or procedures you can recommend for inspecting the traffic between the frontend and backend? Things to look out for?

I've been exploring a potential feature that would enable file sharing from a remote server via CLI. If this aligns with the project's goals, I'd be eager to contribute a PR for this functionality.
Here's an overview of the proposed workflow:

File Upload: Users could upload a single, non-chunked file via a PUT request:
PUT https://example.com/api/shares/:shareId/files/:filename

If the shareId doesn't exist, it would be created
Requests without proper permissions would be denied
Existing files would be overwritten

Secure Link Generation: To enhance security, we could implement a feature in the Web GUI's Share page (/share/xxxx) that includes a "Copy Upload Link" button. This would generate a one-time file upload link:
https://example.com/api/shares/:shareId/files/:filename?token=xxxxx

Short Links: For improved usability, especially in clipboard-restricted environments, we could create shorter upload links. This would involve:

Implementing a simple URL shortening service
Using Next.js for forwarding
Resulting in concise links like: https://example.com/s/:uploadKey/:filename

Although this may not be a common requirement, it could potentially add significant value for certain use cases. Further discussion on these ideas and consideration of whether this functionality would be a good fit for the project could be beneficial.
There might be potential benefits and challenges with this approach. Any feedback or thoughts on how to refine or implement this feature would be appreciated.

@stonith404, in case you missed this, I'd like to hear your thoughts on this feature. What's your take on it?

@IvanLi-CN Sorry for my late reply. Yes, I like these suggestions and would love if you want to contribute to this feature.
Point 3 could get a bit complicated as currently it is possible to upload multiple files with the same name. But I think we could remove the ability to upload multiple files with the same name as this is probably not commonly used.

@stonith404 Thank you for the reminder. I will consider implementing logic in this part to prevent overwriting existing files with the same name.

That'd be an awesome addition, I searched for a replace for bashupload.com, which seems outdated since at least 2 years, beaing able to do this kind of upload/ download would be awesome!
Because many selfhosters often work in terminal-only enviroments where such things are just the best way to copy files, however trusting a public instance with potentionally personal files is always risky.
so a huge +1 from me

bashupload.com

I really like it! I just tried it out and found bashupload.com to be excellent for temporarily sharing individual files.
Thank you so much, this project has inspired me! I'll reference its implementation and include share and file information in the response.

@IvanLi-CN @stonith404 Hi
Is there yet any news to this?
Sounds like an amazing feature and seems like bashupload is long offline.

@CrazyWolf13
Sorry, I've been a bit unexpectedly busy in the last few months and I haven't been able to find the time to implement this feature yet.
I still wish Pingvin Share had such a feature, and I'm willing to implement it, but I probably won't be able to find the time in the next two months.


Adds vendoring

Towards https://github.com/giantswarm/prometheus-pingdom-exporter/issues/11

@boughtonp we're not reallllly using this project anymore, but I think I might have a solve for you.
This PR updates the vendoring, so the project builds correctly.
Using the following against this branch:
docker run --rm -v $(PWD):/go/src/github.com/giantswarm/prometheus-pingdom-exporter -w /go/src/github.com/giantswarm/prometheus-pingdom-exporter -e GOOS=windows golang:1.9.1 go build

(so you only need Docker installed, as opposed to a full golang installation)
gives me
$ file ./prometheus-pingdom-exporter.exe
./prometheus-pingdom-exporter.exe: PE32+ executable (console) x86-64 (stripped to external PDB), for MS Windows

which looks correct. i don't have a windows environment handy, so I can't check for you further. Does this help?

Thanks Joseph. Unfortunately Docker requires Windows 10.
Does Travis export executable snapshots anywhere?

Not right now. Can you install a golang toolchain?

Ok, so I installed Go and the following has given me an EXE:
git clone https://github.com/giantswarm/prometheus-pingdom-exporter.git prometheus-pingdom-exporter
cd !$
/c/Golang/bin/go get github.com/giantswarm/prometheus-pingdom-exporter/cmd
/c/Golang/bin/go build

Then running ./prometheus-pingdom-exporter server.exe <USERNAME> <PASSWORD> <API-KEY> gives what looks like correct data at http://localhost:8000/metrics - I'll check it properly on the Prometheus server tomorrow.
Thanks. :)


[BEAM-986] Improvements to ReduceFnRunner prefetching

add prefetch* methods for prefetching state matching existing methods
replace onTimer with batched onTimers method to allow prefetching across timers
remove deprecated TimerCallback usage
prefetch triggers in processElements

@kennknowles Ready for another look, this is the same as the previous PR but separated out the sdk changes by removing TimerCallback dependencies.

LGTM - merging. Nice again.


Deprecate repo

Updates README with a deprecation message
Updates index.md to automatically redirect to the new URL
Removes not needed files

Will we get 404 errors for existing links? Seems like only the root page (i.e., https://real-time-working-group.readthedocs.io/en/latest/) will show the "This website has been deprecated" message. It would be better to have all non-root pages link to the root page to avoid 404 errors (and let people know about the switch), or have each page redirect to the corresponding page on the new website.

Will we get 404 errors for existing links? Seems like only the root page (i.e., https://real-time-working-group.readthedocs.io/en/latest/) will show the "This website has been deprecated" message. It would be better to have all non-root pages link to the root page to avoid 404 errors (and let people know about the switch), or have each page redirect to the corresponding page on the new website.

Good point. I will do that.

@christophebedard I will try to do the trick by adding a custom 404.html redirecting to the new documentation. Like we do here but with a proper message https://github.com/ros-realtime/ros2_realtime_benchmarks/blob/main/docs/404.html.

We should add the custom 404.html yes.

@christophebedard @LanderU I used sphinx-reredirects to redirect all the pages. This will automatically redirect all the old URLs to the new root URL. I think this is a cleaner solution.


ci: Cleanup integration tests

What this PR does / why we need it:
This cleans up the following integration tests:

inventory_kubvirt
kubevirt_vm
kubevirt_vm_info

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

None

/lgtm

/approve
/lgtm


As a developer, I want to increase the test coverage of REL_Utils

The dev console says it's at 42% right now

**lurch: attach W-007354

Tracking W-007354

This has been marked as "Never" internally.


feat: update wallet connect

Updated to the latest Wallet Connect version.
The Vercel issue should be fixed; it was already updated by @truemiller in the previous PR. Thanks, Josh!
Also updated a few other packages such as ethers, reduxjs, wagmi etc.

https://github.com/valory-xyz/autonolas-contribution-service-frontend/assets/22061815/1749b83e-7355-458a-95eb-64aa1841b999

tweet proposal seems broken for me :[

@truemiller Could you help in reproducing this issue? A short video would be really helpful.

https://github.com/valory-xyz/autonolas-contribution-service-frontend/assets/31908788/c6c70eb0-c8a4-478b-9cd5-865a8e893f0b

I got the same issue as Josh @mohandast52


Test transaction with max CPU usage

pull request https://github.com/EOSIO/eos/pull/2651 created for fixing this issue.


preload is not executed unless html contain script tags

I have a problem with preload on webviews.
If the html page linked to in webview src does not contain the following:
<script>
</script>

somewhere on the page the preload script is not executed.
The following does not work.
<script></script>

I'm on OS X running Electron 0.25.2

This is caused by the same bug with #1117, I'm merging this issue to it.


Add Allergens Feature

PIVOTAL ID 152021104
I contacted code climate about the failing test.
What Works:

Adding allergens, both logged in and logged out. Delete needs to be implemented separately as I found out there is a scope issue which will most likely involve a bigger gmaps.js refactoring that I would rather do on another branch. (All functions in gmaps.js are defined within initAutoComplete(), so calling a function on a button 'onclick' action does not work since the function is technically undefined; or defined in another function). https://stackoverflow.com/questions/15637138/javascript-button-onclick-function-undefined

Moving the map around and searching and having markers remain in the right place.

Try it out! https://pure-temple-24010.herokuapp.com

Testing:

All Markers model methods tested.

All Markers controller models tested except markers#create. This was passing until the last time I ran the rspec test and all the functionality works on heroku so I commented it out in the interest of time.

Cucumber scenarios are unimplemented. This is because of several reasons but our coverage is fine and I manually tested each cucumber scenario in add_allergen.feature and labeled them as such. It's the best we can do in the interest of time.

Comments:
In summary you did a good job implementing add allergens but I am tentative in approving it due to the negative test coverage displayed by code climate. Once that's fixed I think we are almost good to go. There are still some potential issues with the code that i have detailed below.
Visual Bugs

After adding a marker, if we exit out of its auto generated text box, the marker disappears until we refresh the page.
Markers phase in and out of existence when I move the map. This differs from markers I had just placed which had no visual glitches when moving the map until i refreshed the page.

MarkersController

You use skip_before_action :verify_authenticity_token to avoid a CSRF token verification error. Is it not a problem when the CSRF token is invalid? Shouldn't we do error handling instead of avoiding the check altogether?

Database

Your db/schema.rb is creating a clients table. However we removed the client's model a couple of iterations ago. Why is it coming back?

Testing

So i guess there is no easy way to test the add_allergen button but will it be acceptable to Adnan?
It seems coverage issues for markers_controller stem from create, destroy, and marker_params functions. Could you do rspec unit tests for those to knock those coverage issues out?
It looks like the test coverage for the breathe_controller was never working up until now. it seems like it has only a few minor untested actions. Maybe you could write tests for it?


Create swarm0002 cluster

Describe the solution you'd like
Spin up a new production Docker Swarm cluster similar to swarm0001
Describe alternatives you've considered
We could repave hornet, but probably better to tear it down

Cluster created with 3 managers and 3 workers (worker pool can be expanded later)
Still needs:

[ ]  Shepherd
[ ] Seccheck
[ ] Cron jobs

@kumarnarendra701 can you please take care of seccheck and cron jobs (e.g. cert renewal) on swarm0002

@cpswan - Added below cronjobs.

seccheck
certcheck
desscheck(It's throwing dess.text: No such file or directory)
SSL renewal

Looks like this ticket is complete ... can it be closed?


Recent posts is empty if all the current posts are marked as featured

On my version of this (Smale.Codes there is only one post and it is featured. As such there is nothing in Recent and nothing to say 'no posts here' or similar.

I intentionally filtered Featured Posts under Recent Posts section.
In this way, I can avoid redundant posts in the index.
Maybe the best way to deal with this problem is to hide Recent section if there's no blog post for that.
Alternative method to solve this issue is displaying all posts without filtering featured posts.
To do so, for now, you can remove the filter logic in src/pages/index.astro

Recent Posts

        {
          sortedPosts
            .filter(({ data }) => !data.featured) // 👈🏻 remove this
            .map(
              ({ data, slug }, index) =>
                index < 4 && (
                  <Card
                    href={`/posts/${slug}`}
                    frontmatter={data}
                    secHeading={false}
                  />
                )
            )
        }

 ....

understood, I wonder if there needs to be some smarter logic to see if it is going to be empty then populate it as a message saying 'there are no posts' will be a bit odd.
something like, if there are posts and there is something after the filter, show that, if there are sortedPosts but nothing after the filter, remove the filter, and if there are no sorted posts, show 'no posts yet'

Made a PR for this issue.
Believe it solve this issue with no trade-offs.


fix: Correct syntax issue in generated log path for alt hec

:tada: This PR is included in version 2.6.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


how should i use  from @angular/cdk/scrolling

Bug, feature request, or proposal:
Error: Template parse errors:
Property binding cdkVirtualForOf not used by any directive on an embedded template. Make sure that the property name is spelled correctly and all directives are listed in the<EMAIL_ADDRESS>("

[ERROR ->]<div *cdkVirtualFor="let item of items">{{item}}</div>

What is the expected behavior?
What is the current behavior?
What are the steps to reproduce?
Providing a StackBlitz reproduction is the best way to share your issue.
StackBlitz starter: https://goo.gl/wwnhMV
What is the use-case or motivation for changing an existing behavior?
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Is there anything else we should know?

in .html i use like below

{{item}}

and i imported
import {ViewportRuler,ScrollDispatcher} from '@angular/cdk/scrolling'; in my component and also
import {ScrollDispatchModule ,ScrollDispatcher} from '@angular/cdk/scrolling';
import { VIEWPORT_RULER_PROVIDER } from '@angular/cdk/scrolling';  in my app.module.ts

Please keep GitHub issues for bug reports / feature requests. Better avenues for troubleshooting / questions are stack overflow, gitter, mailing list, etc.


[ECS] Fix PHP_CodeSniffer Loop inconsistency

Closes #1459
Closes #1449
Closes #1107

Finaly found some time to fix compatibility by using same architecture as PHP_CodeSniffer. The PHP_CodeSniffer active pattern is back now, but the code works and it's easier to maintain.
cc @ostrolucky

great. You can request me for review, I will review it in evening

Awesome! Thanks
I'll go through issues and make sure all are fixed
Todo

https://github.com/Symplify/Symplify/issues/1449

I've added you as collaborator, so I could ask you for review. After you confirm I'll ping you

Ready to review btw :+1:

Love it. I also like some simplifications you made. I tested this on some spots in my code I remembere there was an issue and this patch indeed solves the issue.

:+1: :heart: Awesome, thanks a lot.
I had to dig deep and re-learn whole PHP_CodeSniffer architecture over the night. It cost one energy drink, but was worth it :)
Thanks for feedback and kicking me with issues, it helped me a lot.


util: add type checking to CONTAINER_OF

Add a SAME_TYPE checking macro and use it to add type validation to CONTAINER_OF. This is inspired by the Linux container_of implementation.
The check is inhibited when using C++ as __builtin_types_compatible_p is not available there.

Linux version: https://elixir.bootlin.com/linux/latest/source/include/linux/container_of.h#L18
Deps:

https://github.com/zephyrproject-rtos/zephyr/pull/61936
https://github.com/zephyrproject-rtos/zephyr/pull/62023
https://github.com/zephyrproject-rtos/zephyr/pull/62024
https://github.com/zephyrproject-rtos/zephyr/pull/62018
https://github.com/zephyrproject-rtos/zephyr/pull/62019
https://github.com/zephyrproject-rtos/zephyr/pull/62020
https://github.com/zephyrproject-rtos/zephyr/pull/62021
https://github.com/zephyrproject-rtos/zephyr/pull/62022
https://github.com/thesofproject/sof/pull/8112

https://github.com/zephyrproject-rtos/zephyr/pull/61991
https://github.com/zephyrproject-rtos/zephyr/pull/62262

https://github.com/zephyrproject-rtos/zephyr/pull/62246

Push test run: https://github.com/zephyrproject-rtos/zephyr-testing/tree/vfabio-testing-branch
^^ there's some unrelated breakage in this one, but no type checking errors.
There may be more issues in the weekly CI run, but this is as much as I could fix before that.

Nice. I'm actually surprised this isn't finding more problems.

Yeah it wasn't too bad, sure there will be more in the weekly run.

Nice. I'm actually surprised this isn't finding more problems.

You asked, I delivered :-)

As ZRESTRICT does not seem to be related to CONTAINER_OF, I filed new bug

#62464


Spark ReadTask is expensive to serialize

In some Spark jobs, we see a substantial scheduler delay. I assume it happens in TaskSetManager when Spark serializes Iceberg ReadTask. The latter contains a couple of large strings (if you have a lot of columns) and an instance of FileIO (which can contain a full Hadoop conf).

I can confirm the issue is resolved if we avoid serializing FileIO. The main question is how to achieve that with minimum changes.

As a short-term solution, we can broadcast EncryptionManager and FileIO in IcebergSource. Then Reader and ReadTask can store references to the broadcasted values and fetch actual ones in createPartitionReader while creating TaskDataReader.
@rdblue thoughts?

Using a broadcast sounds good to me for now.
Can you open a PR for this?

Will open a PR today


[BUG] Typo in lib.js (Not sure if relevant any more)

Module Version: v2.2.1
There is a leftover-typo from removing .data in line 159
"charactera" instead of "character"
The section with this line is commented, so I'm not sure if this error is even relevant anymore.

It should be okay now with 2.2.3 and thanks again for the logs and consigni, unfortunately I am still not "skilled" with the new fields of 10. Thanks for your patience.


generate binaries for cli?

Would you consider a PR to use pkg as a prerelease script to generate binaries for non-node users
The binaries could be either available on:

https://unpkg.com/ (easy as we just need to publish the binaries)
github releases (a bit more complex as it will involve using https://developer.github.com/v3/repos/releases/#upload-a-release-asset and making sure there is a release on github (it could clash with np))

Nah, that's just creating lots of complications for a problem I don't have. There are many downsides in packaging a Node.js binary. I don't think this is something most people will need either. It's super easy to install Node.js. Would probably be a better solution to create a tool that can download Node.js CLI's and package them on-demand for people that refuse to install Node.js. It shouldn't be the responsibility of package authors.


Publication is "ARXIV.ORG" in the badge

https://arxiv.org/abs/2012.01598

Note: Accepted @ ARXIV.ORG (2020) -- [semanticscholar.org]

fixed with #158


[PDI-16252] - Normalise Rows step performance logic fix

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -f engine/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/trans/steps/normaliser/Normaliser.java
  engine/src/org/pentaho/di/trans/steps/normaliser/NormaliserData.java
  engine/src/org/pentaho/di/trans/steps/normaliser/NormaliserMeta.java
  engine/test-src/org/pentaho/di/trans/steps/normaliser/NormaliserTest.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.core.injection.bean.BeanInjector

Branch Change:  + .84%
Complexity Change:  + 1.39%

org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change:  + .39%
Complexity Change:  + .51%
Instruction Change:  + .08%

org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change:  + 2.02%
Complexity Change:  + 2.13%
Instruction Change:  + .34%

org.pentaho.di.job.entries.ftpsget.JobEntryFTPSGet

Branch Change:  + .53%
Complexity Change:  + .56%
Instruction Change:  + .07%

org.pentaho.di.job.entries.waitforsql.JobEntryWaitForSQL

Branch Change: -1.32%:small_red_triangle_down:
Complexity Change: -1.85%:small_red_triangle_down:
Instruction Change: -.27%:small_red_triangle_down:

org.pentaho.di.trans.steps.checksum.CheckSumMeta

Instruction Change:  + .11%

org.pentaho.di.trans.steps.csvinput.CsvInputMeta

Instruction Change:  + .05%

org.pentaho.di.trans.steps.datagrid.DataGridMeta

Instruction Change:  + .09%

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change:  + .11%

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change: -3.12%:small_red_triangle_down:
Complexity Change: -3.16%:small_red_triangle_down:
Instruction Change: -.14%:small_red_triangle_down:
Line Change: -.37%:small_red_triangle_down:

org.pentaho.di.trans.steps.ifnull.IfNullMeta

Branch Change:  + 2.00%
Complexity Change:  + 1.89%
Instruction Change:  + .17%

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change: -.06%:small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change:  + 1.18%
Complexity Change:  + 1.35%

org.pentaho.di.trans.steps.normaliser.Normaliser

Branch Change:  + 80.00%
Complexity Change:  + 73.68%
Instruction Change:  + 87.88%
Line Change:  + 89.71%
Method Change:  + 100.00%

org.pentaho.di.trans.steps.normaliser.NormaliserData

Complexity Change:  + 100.00%
Instruction Change:  + 100.00%
Line Change:  + 100.00%
Method Change:  + 100.00%

org.pentaho.di.trans.steps.normaliser.NormaliserMeta

Branch Change:  + 32.79%
Complexity Change:  + 25.67%
Instruction Change:  + 23.36%
Line Change:  + 25.98%
Method Change:  + 26.32%

org.pentaho.di.trans.steps.pgbulkloader.PGBulkLoaderMeta

Branch Change:  + 1.75%
Complexity Change:  + .93%
Instruction Change:  + .23%
Line Change:  + .28%

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Instruction Change:  + .08%

org.pentaho.di.trans.steps.singlethreader.SingleThreaderMeta

Instruction Change: -.10%:small_red_triangle_down:

org.pentaho.di.trans.steps.sort.SortRowsMeta

Instruction Change:  + .08%

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta

Instruction Change: -.18%:small_red_triangle_down:

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change: -.53%:small_red_triangle_down:

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change: -.18%:small_red_triangle_down:


Set limits of ocpp charging profiles to one decimal place

@LucasBrazi06 : seems ok to me. A customer requested to limit charging profiles to one decimal to make it compliant with the OCPP spec. Tobias wants to merge it as soon as possible. What do you think?

Okay for me.


routing error on Heroku for /assets/application.js

My Heroku logs report this:
2011-12-12T18:35:50+00:00 app[web.1]: ActionController::RoutingError (No route matches [GET] "/assets/application.js"):
I can't figure out why.
Thank you for your help.

assets? This looks like rails 3.1, we are using 3.0.3 now, it shouldn't use the assets path.
Are you using the master branch? check your gems versions

Wrong fork!  Sorry.


Issue #288

Actually this issue refers to match page indication.  However, if you think this is better that what we have now we can still merge this :)

😆 I know, it's not yet done

OK, just making sure since you tagged the issue in the commit rather than the PR :)

Unparsed match


 What am I missing? also how do I know the ID of the dude who pays?

Hi, i suffered from the same problem, did you have solved it?

Hi, i suffered from the same problem, did you have solved it?

No, this project's documentation is not satisfactory for newcomers. However, I guess that it works if you use proper IDs for everyone because it's obviously mandatory.

@jimfilippou @firma Go client just sends JSON to PayPal API, so if you follow PayPal logic and do. everything right, then it'll work - https://developer.paypal.com/docs/api/orders/v2/


Create isaric_malaysia.toml

first draft;
Comment from Sadie: 'For phase within observations, we will need to take into account the timing provided in ‘redcap_event_name’, as the same variable has been used for admission and daily measurements' => check this ...

adtl isaric/parsers/ncov-malaysia.toml ../isaric-data/malaysia/CVPRQTA_COVID19Malaysia_DATA_2020-December.csv --encoding=iso-8859-1

table
valid
total
percentage_valid

subject
6440
6476
99.444101%

visit
5265
6476
81.300185%

observation
160527
252445
63.588901%

subject

36: data must contain ['subject_id', 'country_iso3', 'admission_date', 'age', 'sex_at_birth', 'ethnicity', 'pathogen'] properties

visit

982: data must contain ['visit_id', 'subject_id', 'country_iso3', 'start_date', 'outcome', 'date_outcome'] properties
216: data.treatment_antiviral_type[0] must be one of ['Ribavirin', 'Lopinavir/Ritonvir', 'Interferon alpha', 'Interferon beta', 'Chloroquine/Hydroxychloroquine', 'Oseltamivir (Tamiflu)', 'Zanamivir', 'Casirivimab/Imdevimab', 'Remdesivir', 'IL6 inhibitor', 'Neuraminidase inhibitor', 'Convalescent plasma', 'Anti-influenza antiviral', 'Other']
13: data.treatment_antiviral_type[1] must be one of ['Ribavirin', 'Lopinavir/Ritonvir', 'Interferon alpha', 'Interferon beta', 'Chloroquine/Hydroxychloroquine', 'Oseltamivir (Tamiflu)', 'Zanamivir', 'Casirivimab/Imdevimab', 'Remdesivir', 'IL6 inhibitor', 'Neuraminidase inhibitor', 'Convalescent plasma', 'Anti-influenza antiviral', 'Other']

observation

90048: data must contain ['phase', 'date', 'name'] properties
1587: data.text must be string
283: data.value must be number

@ekamau thanks!

Some of the observation dates (like fatigue_date) refer to the date of onset, which should be captured in the start_date field of the schema, with date being the date of admission (observation record date)
{'treatment_antibiotics_type', 'treatment_delirium_type', 'treatment_antimalarial_type'} are not in the visit schema, should be added to make validation pass

I think this is ok, yes, to have a start date for a symptom separate to the date of observation/data collection, so a cough started 4 days before admission for example

More generally, you might want to pull in the latest version of the main branch to make sure your schemas are up to date (e.g. treatment_antibiotics_type is in the schema on the main branch, but isn't here which is contributing to some of your errors)

@ekamau I've merged all the main branch changes (from the mapping guide edits) into this branch and updated the parser as much as I can to match the new schema. I haven't gone through to check the actual mappings of existing stuff though, so you'll need to check you're happy with the changes and continue to make any updates you've got left on your end.

Thanks Pip - I will check if there were any edits or changes still pending.
On Wed, Jun 14, 2023 at 9:20 AM Pip Liggins @.***>
wrote:

@ekamau https://github.com/ekamau I've merged all the main branch
changes (from the mapping guide edits) into this branch and updated the
parser as much as I can to match the new schema. I haven't gone through to
check the actual mappings of existing stuff though, so you'll need to check
you're happy with the changes and continue to make any updates you've got
left on your end.
—
Reply to this email directly, view it on GitHub
https://github.com/globaldothealth/isaric/pull/135#issuecomment-1591592735,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABW7SJHJQLVVLFVVW3H6CP3XLHQEHANCNFSM6AAAAAAXY75JBA
.
You are receiving this because you were mentioned.Message ID:
@.***>

ncov-malaysia-comparison.csv- Malaysia comparison

Thanks very much Sadie - I will add the glasgow_coma_score attribute.
On Fri, Jun 16, 2023 at 8:36 AM sadiekelly @.***> wrote:

@.**** commented on this pull request.
On isaric/parsers/ncov-malaysia.toml
https://github.com/globaldothealth/isaric/pull/135#discussion_r1232410493
:
@ekamau https://github.com/ekamau the amendments all look ok to me.
from the comparison file, it looks like glasgow_coma_score in the
observations table may have been missed - this is available in the raw data
variable daily_gcs_sx.
—
Reply to this email directly, view it on GitHub
https://github.com/globaldothealth/isaric/pull/135#pullrequestreview-1483778625,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABW7SJBPTPQUDUE6O5OU5RTXLR4PJANCNFSM6AAAAAAXY75JBA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Implement Shopping Cart Feature Instead of My Orders

Currently, the My Orders section displays items that have not yet been ordered, which can create confusion for users. To enhance user experience and provide clarity, I propose implementing a dedicated Shopping Cart feature that separates items that are still in the cart from those that have already been ordered.
Could you please assign this to me?

@Varsha567 Any Update ??

HI @Varsha567 Please refer to #4 Once if you are working on this.

Hi @yuvrajsinghgmx
Apologies for the delay in providing an update. I was caught up with some other work, but I am now focusing on completing the task. I'll refer to #4 as you suggested and will provide a more detailed update soon.
Thank you for your understanding!

Hi @yuvrajsinghgmx
Apologies for the delay in providing an update. I was caught up with some other work, but I am now focusing on completing the task. I'll refer to #4 as you suggested and will provide a more detailed update soon.
Thank you for your understanding!

Okay No Worries.


[FLINK-8222] [build] Update Scala version

What is the purpose of the change
This is an incremental upgrade to the Scala security release 2.11.12.
"A privilege escalation vulnerability (CVE-2017-15288) has been identified in the Scala compilation daemon."
https://www.scala-lang.org/news/security-update-nov17.html
Brief change log
Updated scala version in both parent pom.xml and in flink-quickstart-scala pom.xml.
Verifying this change
This change is already covered by existing tests, such as Scala tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Should be good.
I have seen now and then warnings by the Scala compiler about Scala dependencies requiring different minor versions. It's only a warning, but I am wondering if there could be are compatibility issues?
I would go ahead and merge this for 1.5 soon to give it exposure.
Not sure if I would merge it into 1.4.1 because of the above. What do you think?

I'll merge this to 1.5 only.


Handle error cases better

The Dm3 client often assumes that requests and their responses are always successful and valid. That however might not always be the case.
If an error occurs, the client should still work and not crash.
Every request executed during "fetching contract" should be handled as it could fail.

@Bhupesh-mfsi please go through the code and compile a list of every interaction that could cause an error


MimeType in desktop file

Could MimeType for the supported formats be added to the desktop file so it can be easily integrated into the desktop environment?
https://specifications.freedesktop.org/desktop-entry-spec/latest/ar01s10.html
https://specifications.freedesktop.org/desktop-entry-spec/latest/apa.html

2.1 is on flathub now.


When using aggregate() or groupBy() to be able to use orderBy for the sum of two or more properties

Ive been looking around on forums and the docs and I have yet to see a way to do this. In mysql, the query would look like
SELECT int1 + int2 as summed ORDER BY summed DESC LIMIT 10;
or something related

I don't this is possible in Prisma right now without doing a raw query.
I think https://github.com/prisma/prisma/issues/5848 covers this issue is possibly a duplicate.


github: fix quoting in github workflow for jitter tests

Nested quoting got ignore previously. And this way one can specify
string name directly.
Successfully run with Jitter at
https://github.com/xnox/openssl/actions/runs/10223149419/job/28289017013

@t8m fix for workflow as mentioned in https://github.com/openssl/openssl/pull/24844

Should the 2 pod files be updated also?

Should the 2 pod files be updated also?

done

@nhorman @tom-cosgrove-arm if you agree with urgent, please merge this. IMO it is CI fix -> urgent.

Ok with urgent. Can merge shortly

merged, thank you!

merged, thank you!

Sorry, have been AFK - okay with urgent, FWIW!


demo links not working

hi guys,
have been waiting for the release for a long time. However the demo link is not working. Any clue?

@realsaraf Hey thanks for the heads up. We ran into a little hiccup with our build script. We've been aware of the issue and will resolve today

hey @realsaraf quick update. All the links, besides the old polymer demo should work. We are still in the midst of fixing that and should have it resolved soon. We also plan to provide new examples (similar to or exact replicas of the polymer examples) for 1.0

@Dwaynekj Thats awesome. Thanks for the quick turnaround.

All demo links are fixed


Add notEqual validator

Since there is already an equal validator I think having the negated version of it would be a useful addition.

Yes, we should add such a function for value, length, byte, and size. Also, we should consider how to validate a non-inclusive range. For example, if a string should have less than 2 or more than 4 digits.


Merge same lectures with different names and rename some other

There are several lectures with different names which are actually the same. These should be merged.
Some lectures should be renamed to a consistent naming scheme.
Merge
Mathe 1

Mathe-1
Mathematik für Informatiker I

Mathe 2

Mathe-2
Mathe 2 für Informatiker

Mathe 3

Mathe-3
Mathe 3 für (Bio-)Informatiker

Rename
Medizinische Bildverarbeitung

rename to Medizinische Visualisierung

Informatik I / II / III

rename to Informatik 1 / 2 / 3

I suggest to download all the protocols of these lectures. Then delete all these lectures and re-upload. Is a bit of work, but probably the most stable solution.

Merging has been done.
Renaming still to do.

Done


Sync with GCM 10.19.3

These updates are to bring the CTM inline with GEOSgcm v10.19.3.
This is possibly non-zero-diff. One of the updates in this version is MAPL 2.8.0 which had a bugfix that caused non-zero-diff in running Ops GOCART. It's possible this could manifest in the CTM as well. I'll label as both 0-diff and non-zero-diff and maybe @mmanyin or @JulesKouatchou can let us know which label is correct.

@tclune Please review the changes I've made and see if you agree with them.


error handling introducing c++ exceptions

enabled C++ exceptions
reworked exception flow to be handled synchronously
simplified exception handling / throwing

:+1:


compose: merge latest spec changes

Proposed changes
Manually merged latest changes from compose-spec/compose-spec:

tmpfs.mode: https://github.com/compose-spec/compose-spec/pull/252
build.tags: https://github.com/compose-spec/compose-spec/pull/255
env var secrets: https://github.com/compose-spec/compose-spec/pull/257
build.platforms: https://github.com/compose-spec/compose-spec/pull/267

Related issues (optional)
JIRA: EN-194

@dockertopia I diffed the upstream files spec.md and build.md and pulled all the changes over here.

I committed most of the suggestions and left comments on the others.
Reminder: We'll need to (manually) merge these changes back upstream into compose-spec/compose-spec once this PR is accepted.

I committed most of the suggestions and left comments on the others.
Reminder: We'll need to (manually) merge these changes back upstream into compose-spec/compose-spec once this PR is accepted.

Oops I had missed this notification. But I think everything is fine.


'?' in between shabad

This is in new db
@sarabveer

No, this is only an issue with the old database.

So interestingly this is in the second search result in old db (amrit kirtan). In semi-merged db it becomes the first search result. However, confirmed that this is an issue with old database's amrit kirtan results. Neither has the issue outside amrit kirtan.

Whoever supplied the old Database didn't know how to handle codepages/encodings properly.
This is/was also an issue with iGurbani and their database.


liveblog post module

is there any liveblog post module for orchard?

Not that I know of. Please keep github issues for bugs and feature requests. Prefer Gitter if you have questions, more responsive too.


NeomakeSh broken

:NeomakeSh ls
gives
Error detected while processing function <SNR>57_NeomakeSh..neomake#Make..neomake#GetMaker:
line   40:
E15: Invalid expression: 'neomake_'.a:name.'_'.key

I think I have a fix...

Fixed one bug, found another.
Now there's something about error formats. Down the rabbit hole!


Scrolling on feedback form is broken

In the below GIF I'm trying to scroll down using my mouse wheel.

Thank you @hellyeah for reporting this.

See #55


Quartz

Introducing Quartz Scheduler for KITE multi run
Addresses #11

Redundant! It has already been merged as part of the other pull requests.


Make menu.run a resolvable promise

This will enable a person to capture ussd result especially if you wantn to save it e.g. in a db:
let ussdResult = await menu.run(req.body);

@kechkibet Thanks for the contribution. Could you add a test case for the changes?

Hi,
We had a situation where we had to log all menus sent to client for support purposes

This is great @kechkibet - In addition to the test case, would you mind adding a small line to the README to note the change?
Also, we need to add a small change to the TS type definition file here to note that it is no longer a void return but rather a  Promise<string> now

Thanks @kechkibet! The TS change looks good to me :D
Would you mind making a small test case as @habbes mentioned? The codebase has very high test coverage and I think we'd like to keep it that way.

Thanks for the update @kechkibet, could you also mention this in the README as @yono38 suggested? And then we'll merge. Thanks.

Thanks @kechkibet


Is htmlunit affected by vulnerability CVE-2022-42889?

https://nvd.nist.gov/vuln/search/results?form_type=Basic&results_type=overview&query=CVE-2022-42889&search_type=all&isCpeNameSearch=false

The only usage of this is StringEscapeUtils at the moment - i think we are ok here.
Outside of this the current snapshot builds are already using 1.10.0

这是一封自动回复邮件。已经收到您的来信，我会尽快回复。

yes
maybe this weekend


CMS-735: Implement FE part of sidebar image settings

This PR implements the basic setup of FE UT infrastructure across the project

:+1: