[provider-local] VPN tunnel check succeeds even if VPN is broken

How to categorize this issue?

/area networking testing
/kind bug
What happened:
In the provider-local HA setup (tested with single-zone but should also apply to multi-zone), kube-apiserver talks directly to the kubelet API instead of using the VPN connection.
With this, operations like kubectl logs and kubectl port-forward (for which the kubelet API is called by kube-apiserver) work even if the VPN connection is broken.
As the VPN tunnel check performed by gardenlet uses a port-forward operation (code), the shoot can be reconciled successfully and be marked as healthy even if the VPN connection is broken.
This problem might cause bugs and regressions in the VPN setup to go unnoticed.
E.g., in https://github.com/gardener/gardener/pull/9597 there was a problem in the HA VPN configuration (fixed in a later commit).
Nevertheless, most test cases of pull-gardener-e2e-kind-ha-{single,multi}-zone succeeded. I.e., shoot creations were successful although the VPN connection was never working.
The problem was only discovered by chance in the credentials rotation test case (ref).
What you expected to happen:
If the VPN connection cannot be established successfully

shoot reconciliations should fail
shoot status should be set to unhealthy
e2e tests should fail accordingly

How to reproduce it (as minimally and precisely as possible):

make kind-ha-single-zone-up gardener-ha-single-zone-up
Apply the following patch to example/provider-local/shoot.yaml

--- a/example/provider-local/shoot.yaml
+++ b/example/provider-local/shoot.yaml
@@ -8,6 +8,10 @@ metadata:
     shoot.gardener.cloud/cloud-config-execution-max-delay-seconds: "0"
     authentication.gardener.cloud/issuer: "managed"
 spec:
+  controlPlane:
+    highAvailability:
+      failureTolerance:
+        type: node
   cloudProfileName: local
   secretBindingName: local # dummy, doesn't contain any credentials
   region: local

kubectl apply -f example/provider-local/shoot.yaml
Wait for the shoot to be reconciled successfully and healthy.
Verify manually that the VPN connection works:

k -n kube-system logs deploy/metrics-server --request-timeout 2s
k -n kube-system port-forward svc/metrics-server 8443:443 --request-timeout 2s
k top no

Break the VPN connection: k -n shoot--local--local scale sts vpn-seed-server --replicas 0
Ensure there are no more open TCP connections from kube-apiserver to kubelet: k -n shoot--local--local delete po -l role=apiserver
Repeat the VPN verification from step 5. logs and port-forward work, while connection to the metrics-server (k top no) doesn't work.
Observe that the shoot status is healthy.

Anything else we need to know?:
This only applies to HA clusters, where routes to the shoot networks are configured explicitly in the kube-apiserver pods.
For non-HA clusters, there is an EgressSelectorConfiguration that connects to the envoy-proxy container in the vpn-seed-server using HTTPConnect instead of using explicitly configured IP routes.
E.g.:
$ k -n shoot--local--local exec -it deploy/kube-apiserver -c vpn-path-controller -- sh
~ # ip r
default via <IP_ADDRESS> dev eth0
<IP_ADDRESS>/16 via <IP_ADDRESS> dev bond0 # shoot pod network
<IP_ADDRESS>/16 via <IP_ADDRESS> dev bond0 # shoot service network
<IP_ADDRESS>/26 dev tap0 proto kernel scope link src <IP_ADDRESS> # VPN network
<IP_ADDRESS>/26 dev tap1 proto kernel scope link src <IP_ADDRESS> # VPN network
<IP_ADDRESS>/26 dev bond0 proto kernel scope link src <IP_ADDRESS> # VPN network
<IP_ADDRESS> dev eth0 scope link
~ # ip r get <IP_ADDRESS> # node IP
<IP_ADDRESS> via <IP_ADDRESS> dev eth0 src <IP_ADDRESS> uid 0
    cache

Note, that there is no route for the shoot node network. This is because Shoot.spec.networking.nodes is empty, as is overlaps with Seed.spec.networks.pods (provider-local starts pods in the seed as shoot nodes).
Hence, kube-apiserver can talk directly to the kubelet API via the seed pod network.
There are even multiple mechanisms for allowing this direct communication path from kube-apiserver to kubelet:

allow-machine-pods NetworkPolicy: https://github.com/gardener/gardener/blob/70fe495bc88d0df524b7016b1ce1d59854aa19e8/pkg/provider-local/controller/infrastructure/actuator.go#L61-L65
machines Service: https://github.com/gardener/machine-controller-manager-provider-local/blob/aa28b3aede72b45440183187c23db89ea76840d5/pkg/local/create_machine.go#L67-L86
webhook for adding networking.resources.gardener.cloud/to-machines-tcp-10250=allowed label to kube-apiserver: https://github.com/gardener/gardener/blob/70fe495bc88d0df524b7016b1ce1d59854aa19e8/pkg/provider-local/webhook/controlplane/ensurer.go#L88

To verify that kube-apiserver of local HA shoots talks directly to the kubelet API, use the following steps:

Create a HA shoot. Wait for the shoot to be reconciled successfully and healthy.
k -n shoot--local--local delete netpol allow-machine-pods
k -n shoot--local--local delete svc machines
Ensure there are no more open TCP connections from kube-apiserver to kubelet: k -n shoot--local--local delete po -l role=apiserver
Repeat the VPN verification from step 5 above. logs and port-forward don't work (don't use the VPN connection), while connection to the metrics-server (k top no) works (uses the working VPN connection).
Observe that the shoot status is unhealthy because the port-forward operation doesn't work.

Environment:

Gardener version: v1.93.0-dev

This issue does also affect non-HA scenarios in the local setup. As there is no node range defined for shoots in the local setup the network connectivity will be the following for the VPN check in the reconciliation:
kube-apiserver -> envoy proxy container of vpn-seed-server pod -> machine pod via seed cluster network -> kubelet

In real scenarios, it should be like this:
kube-apiserver -> envoy proxy container of vpn-seed-server pod -> local route to vpn device created by vpn-seed-server container in same pod -> vpn-shoot -> actual node -> kubelet

Fixing this can prevent regressions, but there are also validations in place preventing shoot/seed network overlaps, which may make this somewhat challenging.

You're right. In the non-HA scenario, kube-apiserver will always connect to vpn-seed-server because of the EgressSelectorConfiguration. This one, however, routes only pod and service IPs via the VPN, but nodes are routed via the seed's pod network (again, because Shoot.spec.networking.nodes is empty):
$ k -n shoot--local--local exec -it deploy/vpn-seed-server -c vpn-seed-server -- sh
~ # ip r
default via <IP_ADDRESS> dev eth0
<IP_ADDRESS>/16 via <IP_ADDRESS> dev tun0
<IP_ADDRESS>/16 via <IP_ADDRESS> dev tun0
<IP_ADDRESS> dev eth0 scope link
<IP_ADDRESS>/24 dev tun0 proto kernel scope link src <IP_ADDRESS>
~ # ip r get <IP_ADDRESS> # node IP
<IP_ADDRESS> via <IP_ADDRESS> dev eth0 src <IP_ADDRESS> uid 0
    cache

We can verify this route by breaking the VPN connection on the shoot-side this time:

k -n shoot--local--local annotate mr shoot-core-vpn-shoot resources.gardener.cloud/ignore=true
k -n kube-system scale deploy vpn-shoot --replicas 0
Ensure there are no more open TCP connections from kube-apiserver to kubelet: k -n shoot--local--local delete po -l role=apiserver
kubectl logs and kubectl port-forward still works (direct route to kubelet API without VPN) but k top no is broken as services are routed through the VPN.

To summarize:
Problem
In the provider-local setup, the VPN tunnel check performed by gardenlet (port-forward check) does not detect a broken VPN tunnel, because either kube-apiserver (HA clusters) or vpn-seed-server (non-HA clusters) route requests to the kubelet API directly via the seed's pod network.
When the VPN connection is broken, kubectl port-forward and kubectl logs continue to work, while k top no (APIServices, Webhooks, etc.) is broken.
We should strive towards resolving this discrepancy between the local setup and cloud setups regarding the VPN connection to prevent bugs by validating the real setup in e2e tests.
Proposal
1. Set Shoot.spec.networking.nodes
Setting Shoot.spec.networking.nodes ensures the VPN configures routes for the node network via the VPN tunnel.
Though, this network must not overlap with Seed.spec.networks.pods.
Right now, the Seed.spec.networks.pods field is only used for API validation to prevent obvious misconfigurations. I don't see a problem if the seed pod network is larger than what is configured in Seed.spec.networks.pods.
Hence, we could split the seed pod network into one default subnet (configured in Seed.spec.networks.pods) and add a dedicated calico IPPool for machine pods (configured in Seed.spec.networks.shootDefaults.pods -> Shoot.spec.networking.pods).
With this, the IP packets would still be routable to and between machine pods, but we would have disjoint networks to configure in the API objects and with this correct routes via the VPN tunnel.
2. Forbid direct communication of seed components with machine pods
We want to ensure that the VPN tunnel checks only succeed if the VPN is successfully established.
For this, we need to drop all NetworkPolicies allowing communication from seed components to machine pods.
E.g., when the VPN connection is broken in HA clusters, there will be no route to the shoot node network and hence packets will be routed via the seed pod network.
Additional Improvements
We might also consider augmenting gardenlet's tunnel check. In addition to testing an operation that talks to the node network for the kubelet API (e.g., port-forward), it could also test an operation targeting the pod/service network (e.g., metrics server).
This doesn't resolve the discrepancy in setups but might be useful in general to detect more failure cases in shoot health checks.
WDYT?

This problem might cause bugs and regressions in the VPN setup to go unnoticed.
E.g., in https://github.com/gardener/gardener/pull/9597 there was a problem in the HA VPN configuration (fixed in a later commit).
Nevertheless, most test cases of pull-gardener-e2e-kind-ha-{single,multi}-zone succeeded. I.e., shoot creations were successful although the VPN connection was never working.
The problem was only discovered by chance in the credentials rotation test case (https://github.com/gardener/gardener/pull/9597#issuecomment-2060681653).

@timebertt we have this exact thing happening on latest using provider-openstack
i can add some logs if it helps.

@Lappihuan this issue is only about provider-local, so provider-openstack is out of scope for this issue.
Please open a new issue for your case if necessary.

/assign @rfranzke @timebertt

@timebertt Is anything else needed for closing this issue?

The main part of this issue are done, but we could still have a look at:

We might also consider augmenting gardenlet's tunnel check. In addition to testing an operation that talks to the node network for the kubelet API (e.g., port-forward), it could also test an operation targeting the pod/service network (e.g., metrics server).

What is your opinion on improving the tunnel check?

Thanks, makes sense. Do you have ideas how to implement this check?

The only thing I have in mind would be to check the availability of the metrics-server APIService. This could be a good indicator of connectivity to the service network. If it's broken because of some other problem (unrelated to networking), reporting unhealthy system components would still be correct.

Sounds reasonable 👍🏻 I would be fine doing it this way.

Great. Then this issue should not be stale:
/remove-lifecycle stale
I don't expect to get to this this week. If somebody else has time for this, feel free to pick it up :)


Evaluate schemas in the test function.

Schema errors are kinda hard to deal with, because they occur during test collection rather than during the tests themselves.  This means they all have to be solved before any of the tests will run (which is especially annoying when a lot of the test is actually bing performed in the schema, e.g. eval()) and the error messages aren't as good.
But it just occurred to me that I could evaluate the schema in the test itself without the user needing to do anything different.  The basic idea is to make a wrapper test function parameterized with the dictionaries loaded from the file.  That wrapper function would then evaluate the schema and call the actual test function.  In pseudocode:
def parametrize_from_file(test_func):

    def wrapper(params_dict):
        params = schema(params_dict)
        test_func(**params)

    return pytest.mark.parametrize(wrapper)

On second thought, I think this will cause some intractable problems:

What if the test function has fixtures?
How can I confirm that all test cases have the same keys?


Launch Protein Viewer from File failed

"Start Protein Viewer' with PDB works,
but when select 'Launch Protein Viewer from Files(s)', vscode throw error msg:
"Command 'Launch Protein Viewer from File(s)' resulted in an error (Cannot read properties of undefined (reading 'map'))
vscode version:
Version: 1.67.1 (Universal)
Commit: da15b6fd3ef856477bf6f4fb29ba1b7af717770d
Date: 2022-05-06T12:37:16.526Z (4 days ago)
Electron: 17.4.1
Chromium: 98.0.4758.141
Node.js: 16.13.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 20.6.0

Hmm. I'm struggling to reproduce this @zirui. What sort of files are you trying to open? Maybe you can share them?

i'm trying to open a pdb file(download from rcsb.org),
but i think this error was not caused by the file itself, because it occurs before open files(when select 'Launch Protein Viewer from Files' in command palette)

Ah, gotcha. You shouldn’t use the command palette to open from files. If you right click on the file(s) in the file explorer & select ‘Launch Protein Viewer from Files’ it should do the trick. Check out the GIF in the readme for an example :)

@zirui How did you get on? Can I close the issue?

right click on the file in the file explorer & select 'Launch Protein Viewer from Files' works!
but i think  it will be more convenient if view pdb from command palette  is supported

I agree that would be a nice feature. Unfortunately, I don’t think I’m going to have bandwidth to add it in the foreseeable future. Apologies.
Happy to support if you want to contribute a PR.

On 16 May 2022, at 02:54, zirui @.***> wrote:

right click on the file in the file explorer & select 'Launch Protein Viewer from Files' works!
but i think it will be more convenient if view pdb from command palette is supported
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.


olpath directive doesnot create path if scope variable updates

Hello,
I am trying to build a path using <ol-path coords={{map.path}}></ol-path> and my controller looks like this:
app.controller('mapController', ['$scope','$timeout', function($scope,$timeout) {
    var map = this;
    $timeout(function(){
        map.path = [[[-58.38,-34.60], [-58.58,-34.60],[-58.38,-34.70], [-58.38,-34.80]]];
    }, 250);
}

Debugging your directive the coords array is empty. Instead If I remove the $timeout the path works. The problem is that I am fetching the path dynamically from an API and the building it.
Tried with $digest and $apply and also added $watch in the directive with no result..
Any help is appreciated, thanks

Hi,
I had the same issue (ol-path only has a link method, so it won't update once rendered), so I ended up with this workaround:
        <ol-path ng-if="ctrl.pathFlag" coords="{{ctrl.coords}}"></ol-path>
        <ol-path ng-if="!ctrl.pathFlag" coords="{{ctrl.coords}}"></ol-path>
    </openlayers>


The fate of existing backups if retention policy is shortened is not clear.

This isn't clear from the documentation.
Consider a database that is set for Weekly up to 10 years that has accumulated 2 years of backups. The policy is then changed to Weekly for only 1 year. What happens to those existing backups that exceed the new 1 year policy? Do they remain untouched or are they all automatically deleted after the policy is changed?

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

ID: 2b388152-5d9d-bfb9-1dd3-168b4877ac5a
Version Independent ID: 901abe21-41fd-c208-6301-9cc45acf0e4f
Content: Store Azure SQL Database backups for up to 10 years
Content Source: articles/sql-database/sql-database-long-term-retention.md
Service: sql-database
Sub-service: backup-restore
GitHub Login: @anosov1960
Microsoft Alias: sashan

@LuisGonzalezLT We are checking this and will get back to you.

@LuisGonzalezLT We have checked this our team and they responded:
"The new policy impacts the future backups only. In general, the backups are deleted based on the expiration date set for them by the policy in effect at the time of creation. So going forward, the weekly backups will be set to expire in 1 year. The old backups will expire in 10 years since the creation of each. If the 10-year backups are no longer needed, the customer would have to individually delete them, which is possible."
This document explains the same
https://docs.microsoft.com/en-us/azure/sql-database/sql-database-long-term-retention

Hope this helps. We are going to close this thread now. If any further questions get back to us and we will continue the discussion.


Copyrightform debug

fixed date input bug where the date would reset itself to the default during input
fixed UI bug where when non-transferable rights were unchecked, it would show up as undefined rather than false

Deployed to https://preview.teia.art/


Rewrite assert statements, refs #12

This is still a WIP.
I have several questions:

How does get_arg_offset suppose to work? I am bit a lost with this approach 🙂
Do we need to handle msg part of assert expr, msg? Because not all self.assert*() methods support it

Closes #12

How does get_arg_offset suppose to work? I am bit a lost with this approach

Yeah, it is not a very straight forward solution. But basically it is for adjusting comments appropriately when the initial version of the same call (e.g assertTrue(a < 10, msg="x") has 2 arguments) has more/less arguments than the refactored version (e.g assertLessEquali(a, 10, msg="x") has 3 arguments).
I don't think it is something that this refactor should worry about. Perhaps try returning 0?

Do we need to handle msg part of assert expr, msg? Because not all self.assert*() methods support it

I guess it serves the same purpose, so we can use it. I don't think we need to add it in this PR.


why can't get input line from stdin when this line end with EOF

What version of Go are you using (go version)?

$ go version
go version go1.9.7 windows/amd64

Does this issue reproduce with the latest release?
func main() {
reader := bufio.NewReader(os.Stdin)
inputBuffer, err := reader.ReadString('\n')
if err != nil && err != io.EOF {
fmt.Println(err.Error())
return
}
inputBuffer = strings.TrimRight(inputBuffer, "\r\n")
inputStrings := strings.Split(inputBuffer, " ")
fmt.Println(inputStrings)

}
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?
when i input 123 and Ctrl+D,  can't get the 123?

123^D
[]

What did you expect to see?
Ctrl+D is EOF, i should got the 123
What did you see instead?
i cat't got the 123
i wonder kown the reason

It should work, it likely has to do with how your terminal interprets Ctrl+D presses. You might have to press it twice, or do Ctrl+D followed by Enter. See https://superuser.com/a/169057 for more about those shortcuts and how they differ between terminals.
That said, our issue tracker is not the best medium for questions. More people will see your question, and you will get better answers, if you ask on a forum rather than on the issue tracker. See https://golang.org/wiki/Questions.

Hi there,
We have decided that our experiment to allow questions on the issue tracker has not had the outcome we desired, so I am closing this issue. I'm sorry that we can't answer your question here.
There are many other methods to get help if you're still curious:

Stack Overflow with questions tagged "go"

The Go Forum, a web-based forum

Gophers Slack, use the invite app for access. The #general channel is a good starting point.

Go Community on Hashnode with questions and posts tagged with "go"

The golang-nuts mailing list

IRC channel #go-nuts on Freenode

Thanks


fix: Make meta description longer

Going through ahrefs.com warnings

@yolile You'll just have one string to translate to Spanish and Russian (Learn about the data source's coverage, features and quality issues.).

Added (but I haven't pushed anything to transifex)


Elasticsearch::config doesn't support Hiera merge_behaviour: deeper

At the moment there is no support for the Hiera setting "merge_behaviour: deeper" when using the property "elasticsearch::config".
Please support this just like "elasticsearch::instances" and "elasticsearch::plugins" that DO have deeper support.

@tylerjl this would be useful for the init_defaults as well.


WQEX updates

Description
Minor updates to the WQ executor to address some usability issues.

The situations in which autocategory=False would be beneficial are rare (very large number of apps, each called rarely, very different resource utilization). It should be turned on by default in almost all cases, so this makes that the default.
Use new python_package_analyze to scan exec_parsl_function.py. The remote runner has dependencies of its own, so it's necessary to scan both the app body and the runner.
Add option to specify extra packages to pack up. Useful if e.g. the Python app exec()s a Perl program provided by Conda, which can't be picked up by analyzing imports.

Type of change

Bug fix (non-breaking change that fixes an issue)
New feature (non-breaking change that adds functionality)

cc @ZhuozhaoLi

Hmmm.... looks like this isn't safe to merge yet.
At some point the CCTools Ubuntu build broke (Ben's looking into it), so the tarball used for testing will 404. The version bump in 64d811b will break the WQ tests only after this is merged. I'll update when the CCTools build is sorted out

Ugh....looks like the new CCTools build doesn't match the Travis environment, this is turning into ugly dependency management. Could CCTools be obtained via Conda or by building from source?

Ugh....looks like the new CCTools build doesn't match the Travis environment, this is turning into ugly dependency management. Could CCTools be obtained via Conda or by building from source?

Looks more like a path issue to me?

@trshaffer  > Could CCTools be obtained via Conda or by building from source?
There aren't any particular requirements on how pre-reqs like cctools should be installed in CI. Some soft goals are: not complicated; fast; and close to how an end-user is told to install this stuff.
If you can do it in a better way than it is being done now, do that.

We discussed it at the meeting over at ND, and decided that continuing with a tarball build is the best way to go. We'll just need to make sure that we have a Travis-compatible build going forward.
So this is OK to review/merge


setup: add kind create and clean cluster

Added the part where the kind creates the cluster from the configuration file and the part where the cleanup the cluster.
Please point out any questions in my code as its my first time to write the command line program.

Also, please bear in mind to log with different verbosities as detailed as possible, some of them may be just debug level but can display more details when needed that help to diagnose why the commands are failed to run

Also, please bear in mind to log with different verbosities as detailed as possible, some of them may be just debug level but can display more details when needed that help to diagnose why the commands are failed to run

Otherwise look good to me, @wu-sheng @fgksgf want to recheck?
@Humbertzhang the cleanup step can be done in the next PR because when testing the setup, we need the corresponding step to clean it up :)

Otherwise look good to me, @wu-sheng @fgksgf want to recheck?
@Humbertzhang the cleanup step can be done in the next PR because when testing the setup, we need the corresponding step to clean it up :)

UPDATE:
Since your PR title includes cleanup, let's have it in this PR as well? It's very easy IMO
Ok, i will add cleanup in this PR today.

UPDATE:
Since your PR title includes cleanup, let's have it in this PR as well? It's very easy IMO
Ok, i will add cleanup in this PR today.


Add missing Studio Pro keyboard shortcuts

Ctrl+- for zooming out in a canvas editor.
Ctrl+Shift+- for zooming in in a canvas editor.

These shortcuts have been working in Studio Pro for forever, but they are not listed in the documentation.


Unable to broadcast if amazon ivs broadcast only exist inside DFM module

Broadcast not working if library only implemented inside DFM  (Dynamic Feature Delivery) module. It will works if base module implement it, and all SO files are inside Base APK.
Details:
Android broadcast SDK: 1.2.0
Gradle: 7.3
Android Gradle Plugin 7.0
Assume I have 2 modules, module A is application module and module B is Dynamic Feature Module.
Amazon Broadcast only implemented inside DFM module B, to help reduce our main APK size.
When starting broadcast, it show blank black screen with this repeated error:

022-01-17 16:33:23.662 16578-16578/com.traveloka.android.staging W/AmazonIVS: External cameras are not supported on this device, not listing USB devices
2022-01-17 16:33:23.663 16578-16578/com.traveloka.android.staging D/AmazonIVS: Using size 1280x720 (1,777778)
2022-01-17 16:33:23.663 16578-16578/com.traveloka.android.staging D/AmazonIVS: Using size 1920x1080 (1,777778)
2022-01-17 16:33:23.670 16578-29746/com.traveloka.android.staging E/AmazonIVS: FATAL: No opengl configurations:
2022-01-17 16:33:23.685 16578-29746/com.traveloka.android.staging I/AmazonIVS: Surface=0x9d821a60 window=0xa7acd008
2022-01-17 16:33:23.709 16578-29746/com.traveloka.android.staging I/AmazonIVS: Surface=0x9767c180 window=0xa7e63008 display=0x1 config=0x2
2022-01-17 16:33:23.752 16578-16578/com.traveloka.android.staging I/AmazonIVS: create http client okhttp
2022-01-17 16:33:23.759 16578-29746/com.traveloka.android.staging I/AmazonIVS: Surface=0xa0801f80 window=0xa7ad5008
2022-01-17 16:33:23.782 16578-29750/com.traveloka.android.staging D/AmazonIVS: Using fps range 5->30
2022-01-17 16:33:23.817 16578-16578/com.traveloka.android.staging D/AmazonIVS: Using size 1280x720 (1,777778)
2022-01-17 16:33:23.818 16578-16578/com.traveloka.android.staging D/AmazonIVS: Using size 1920x1080 (1,777778)
2022-01-17 16:33:23.819 16578-29746/com.traveloka.android.staging E/AmazonIVS: Shader compilation failed
2022-01-17 16:33:23.820 16578-29746/com.traveloka.android.staging E/AmazonIVS: Shader compilation failed
2022-01-17 16:33:23.821 16578-29746/com.traveloka.android.staging E/AmazonIVS: 337 glGetError=502
2022-01-17 16:33:24.124 16578-29746/com.traveloka.android.staging E/AmazonIVS: Shader compilation failed
2022-01-17 16:33:24.125 16578-29746/com.traveloka.android.staging E/AmazonIVS: Shader compilation failed
2022-01-17 16:33:24.133 16578-29746/com.traveloka.android.staging E/AmazonIVS: 337 glGetError=502
2022-01-17 16:33:24.156 16578-29746/com.traveloka.android.staging I/AmazonIVS: Surface=0x9767e520 window=0xa7b1a808
2022-01-17 16:33:24.159 16578-29746/com.traveloka.android.staging E/AmazonIVS: Shader compilation failed
2022-01-17 16:33:24.160 16578-29746/com.traveloka.android.staging E/AmazonIVS: Shader compilation failed
2022-01-17 16:33:24.163 16578-29746/com.traveloka.android.staging E/AmazonIVS: 337 glGetError=502
2022-01-17 16:33:24.179 16578-29746/com.traveloka.android.staging E/AmazonIVS: Shader compilation failed
2022-01-17 16:33:24.179 16578-29746/com.traveloka.android.staging E/AmazonIVS: Shader compilation failed
2022-01-17 16:33:24.185 16578-29746/com.traveloka.android.staging E/AmazonIVS: 337 glGetError=502
2022-01-17 16:33:24.198 16578-29746/com.traveloka.android.staging E/AmazonIVS: Shader compilation failed
2022-01-17 16:33:24.199 16578-29746/com.traveloka.android.staging E/AmazonIVS: Shader compilation failed
2022-01-17 16:33:24.219 16578-29746/com.traveloka.android.staging E/AmazonIVS: 337 glGetError=502
2022-01-17 16:33:24.257 16578-29746/com.traveloka.android.staging E/AmazonIVS: Shader compilation failed
2022-01-17 16:33:24.258 16578-29746/com.traveloka.android.staging E/AmazonIVS: Shader compilation failed
2022-01-17 16:33:24.262 16578-29746/com.traveloka.android.staging E/AmazonIVS: 337 glGetError=502

However, if we add amazon broadcast SDK dependencies to our application module the issue is gone. This is the logcat from it:
2022-01-17 16:30:57.253 31672-31672/com.traveloka.android.staging D/AmazonIVS: Found library broadcastcore
2022-01-17 16:30:57.256 31672-31672/com.traveloka.android.staging W/AmazonIVS: External cameras are not supported on this device, not listing USB devices
2022-01-17 16:30:57.267 31672-31672/com.traveloka.android.staging D/AmazonIVS: Using size 1280x720 (1.777778)
2022-01-17 16:30:57.268 31672-31672/com.traveloka.android.staging D/AmazonIVS: Using size 1280x720 (1.777778)
2022-01-17 16:30:57.282 31672-32010/com.traveloka.android.staging E/AmazonIVS: FATAL: No opengl configurations:
2022-01-17 16:30:57.294 31672-32010/com.traveloka.android.staging I/AmazonIVS: Surface=0x77c8c460 window=0x6b7ff008
2022-01-17 16:30:57.368 31672-32010/com.traveloka.android.staging I/AmazonIVS: Surface=0x77c8c1f0 window=0x6ad5c008 display=0x1 config=0x2
2022-01-17 16:30:57.441 31672-31672/com.traveloka.android.staging I/AmazonIVS: create http client okhttp
2022-01-17 16:30:57.473 31672-32010/com.traveloka.android.staging I/AmazonIVS: Surface=0x775d3dd0 window=0x6b7ff808
2022-01-17 16:30:57.531 31672-32005/com.traveloka.android.staging D/AmazonIVS: Using fps range 5->30
2022-01-17 16:30:57.559 31672-31672/com.traveloka.android.staging D/AmazonIVS: Using size 1280x720 (1.777778)
2022-01-17 16:30:57.561 31672-31672/com.traveloka.android.staging D/AmazonIVS: Using size 1280x720 (1.777778)
2022-01-17 16:30:57.866 31672-32010/com.traveloka.android.staging I/AmazonIVS: Surface=0x6b6f2d50 window=0x6b5ff808

In this case, all SO files and SDK from amazon broadcast are move to base apk, and probably because of that this log is showing up D/AmazonIVS: Found library broadcastcore.
However put amazon broadcast dependencies to our base APK will adding 2 MB to our download and install size.
Do you have some idea if it issue from amazon broadcast SDK(maybe not supporting DFM yet) or something I can do to make it works with only DFM module implementation?
Let me know if you need additional information.
Thank you in advance

Hi @doniwinata0309 , thanks for reporting this. We have not tested the Broadcast SDK with DFM so it's possible there are issues leading to lack of support. I've opened an internal ticket to track and investigate it.
In the DFM logs, do you see the following log? It would show up very early:
Failed to load library broadcastcore

I believe the issue has to do with a failure to find the fragment shaders (Shader compilation failed), which are simple raw resource text files. There is a previously-reported issue (https://github.com/aws-samples/amazon-ivs-broadcast-android-sample/issues/6) that led to the same problem, although a different cause.
Thanks for bringing this to our attention! I will update this post when I have more information.

Hi @caleighm I did not see this "Failed to load library broadcastcore" showing up on logcat. As additional info, this also happen on our debug environment where R8 is disabled.
Thank you for quick response and feedback, let me know if something I can help.

Hi @doniwinata0309 , thanks for this additional information. Could you please also reach out to your IVS business contact with this issue as well, for easier internal tracking?

Hi @caleighm we already send it to AWS support about this issue. Sure, let me know if something I can help, thank you for your help


No module named 'tensorflow_core.estimator' error

Gettting No module named 'tensorflow_core.estimator' error while running util.play_generated_song(text), though i have installed tensorflow

Most of the lab doesnt work

This sounds like you've installed an incorrect version of Tensorflow.
What version of Tensorflow are you running? This lab was developed for use in Google colab and is supported with TensorFlow 1.13.0

(Cross-posted from another issue with the same root cause.)
👋 Hello, future deep learning lab adventurer. Ideally, these notebooks are always up to date or maybe they could include a setup.py file 😉?
Until then Google Collab allows you to "Copy to drive" and modify the Notebook. In which case you can add a new code block to install a specific version !pip install tensorflow==1.14.


More informative commit messages

So far the vast majority of commit messages have been

Add files via upload

Is it possible to make the commit messages a little more informative, so it's easier to see at a glance what changes have been made. For example 06863a80e530673fbc4cf301aa28ddddd7cf824c, could have been something like

Created Dockerfiles for latest Java updates based on Alpine

We are practicing this now.


Documentation Inconsistency

There appears to be some inconsistencies in the documentation and it is leading to some errors. On https://docs.saltproject.io/en/latest/topics/tutorials/master-cluster.html#tutorial-master-cluster it lists the following:
Master Config:
id: <IP_ADDRESS>
cluster_id: master_cluster
cluster_peers:
  - <IP_ADDRESS>
  - <IP_ADDRESS>
cluster_pki_dir: /my/gluster/share/pki
cachedir: /my/gluster/share/cache
file_roots:
    - /my/gluster/share/srv/salt
pillar_roots:
    - /my/gluster/share/srv/pillar

In particular, note the file_roots and pillar_roots. If we run with this configuration, we end up with the following in the log file:
The file_roots parameter is not properly formatted, using defaults

The documentation on https://docs.saltproject.io/en/latest/ref/configuration/master.html#file-roots would seem to suggest that we need to use:
file_roots:
  base:
    - /my/gluster/share/srv/salt

Also, notice in the sample Master Config file, it uses cluster_pki_dir. Yet the documentation on https://docs.saltproject.io/en/latest/ref/configuration/master.html#cluster-pki-dir only shows cluster_pki. So which is it?

Sorry, wrong repo.


issue 2220 fixed

see https://github.com/twitter/bootstrap/issues/2220

Already resolved in 2.0.2-wip.

Yep, I know that's resolved in 2.0.2-wip. Why not to merge it to master branch?


Refactoring Miner finds refactorings but not CK

java.lang.RuntimeException: RefactoringMiner finds a refactoring for class 'com.readystatesoftware.mapviewballoons.BalloonItemizedOverlay', but we can't find it in DiffEntry: 'MOVE_SOURCE_FOLDER'. Check RefactoringAnalyzer.java for reasons why this can happen.
java.lang.RuntimeException: RefactoringMiner finds a refactoring for class 'com.commonsware.cwac.cache.AsyncCache', but we can't find it in DiffEntry: 'MOVE_SOURCE_FOLDER'. Check RefactoringAnalyzer.java for reasons why this can happen.
java.lang.RuntimeException: RefactoringMiner finds a refactoring for class 'vn.mbm.phimp.me.PhimpMe', but we can't find it in DiffEntry: 'RENAME_ATTRIBUTE'. Check RefactoringAnalyzer.java for reasons why this can happen.

I think the solution is easy here. Ignore move source folder refactorings,
as we don’t want to predict them anyways.
On Sat, 7 Mar 2020 at 08:46, Jan Gerling<EMAIL_ADDRESS>wrote:

java.lang.RuntimeException: RefactoringMiner finds a refactoring for class
'com.readystatesoftware.mapviewballoons.BalloonItemizedOverlay', but we
can't find it in DiffEntry: 'MOVE_SOURCE_FOLDER'. Check
RefactoringAnalyzer.java for reasons why this can happen.
java.lang.RuntimeException: RefactoringMiner finds a refactoring for class
'com.commonsware.cwac.cache.AsyncCache', but we can't find it in DiffEntry:
'MOVE_SOURCE_FOLDER'. Check RefactoringAnalyzer.java for reasons why this
can happen.
java.lang.RuntimeException: RefactoringMiner finds a refactoring for class
'vn.mbm.phimp.me.PhimpMe', but we can't find it in DiffEntry:
'RENAME_ATTRIBUTE'. Check RefactoringAnalyzer.java for reasons why this can
happen.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/refactoring-ai/predicting-refactoring-ml/issues/96?email_source=notifications&email_token=AAAYTTDVFOCY42R24OOGEW3RGH3VHA5CNFSM4LDNK2BKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ITI2X7A,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAYTTF6MRMVCMPLNOOZQJDRGH3VHANCNFSM4LDNK2BA
.

--
Maurício Aniche
Delft University of Technology
http://www.mauricioaniche.com

I think the solution is easy here. Ignore move source folder refactorings, as we don’t want to predict them anyways.

I think it is not that, because if we don't collect the Rename Package and Move Source Folder refactorings, then we get issues with the Process Metrics tracking, because the class files are not updated to a new file path. Furthermore, this will also create issues with saving the source code of stable commits.


Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
19 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
16 seconds
sanity-inspect-image

✅ Succeeded
15 seconds
deprecated-base-image-check

✅ Succeeded
18 seconds
clair-scan

✅ Succeeded
56 seconds
clamav-scan

✅ Succeeded
12 seconds
sbom-json-check

✅ Succeeded
11 seconds
sanity-label-check

✅ Succeeded
11 seconds
sanity-optional-label-check

✅ Succeeded
7 seconds
show-summary


Geometry gets strongly visually modified (simplified) when Camera3D's x frustum_offset is decreased below a certain negative number

Tested versions

Reproducible in: 4.2.2-stable, 4.2.1-stable

System information
Godot v4.2.2.stable - Windows 10.0.19045 - Vulkan (Forward+) - dedicated Quadro M1000M () - Intel(R) Core(TM) i7-6820HQ CPU @ 2.70GHz (8 Threads)
Issue description
When decreasing the x value of the Camera3D's frustum_offset (in "Projection Frustum" mode), from a certain value onwards (depending on the near plane and the size, with near plane = 1 and size = 1, a frustum_offset <= -0.89 creates the issue), the geometry of many 3D models gets over-simplified with a big visual impact.
Steps to reproduce
Add custom geometry to scene (.glb file). Set the Camera3D to Projection Frustum mode and decrease its frustum_offset.x value. If required, adjust camera's position to keep geometry in visible range. Effect seems to not appear on all geometries.
Minimal reproduction project (MRP)
Camera_Bug.zip

For tracking : #57416


Welcome: weird mac-only/windows-only/linux-only CSS rules

Search for mac-only, I do not understand these rules for the OS, they all seem to do the same?
Also, we already have rules for .mac, .windows and .linux

These are for showing/hiding parts of the Welcome page (and Interactive Playground) depending on which OS we are on. These are still in use.


Unit tests for AddRequestsCommandHandlerAsync #879

Implemented AbortOnNotFoundItinerary
Implemented AbortOnNotFoundRequests
Implemented NotAlterTheAlreadyAssignedRequests
Implemented AssignRequestsToTheItinerary
Implemented MaintainTheOrderingOfTheItineraryRequests

I created the unit tests for this class. Let me know what you think.

Good stuff @adam-gligor, thanks for this PR. :shipit:


OSM Tag Frequency Counter for UI Color Analysis

OSM Tag Frequency Counter for UI Color Analysis
TL;DR
I used empirical analysis to identify the most relevant colors on the map for the purpose of creating colorblind-friendly map color schemes.
Overview
In order to create data-informed colorblind-friendly color schemes for the map, I measured the frequency of OSM tags rendered within a specific map extent.
This PR includes code incorporated from #1365. However, the main contribution of this PR is the chunk of code in /modules/pixi/PixiLayerOsm.js which does the OSM tag counting. This code was instrumental in #1384, specifically in the creation of three (3) colorblind-friendly color schemes.
NOTE: For the most part, this conversation is going to be held independent of the color system proposed in #1384
Thought Process Summary
Rapid has 39 distinct HEX colors referenced in /modules/core/StyleSystem.js and /data/colors.js. The aim is create colorblind-friendly color schemes in which:

All colors are colorblind-accessible
All color interactions are colorblind-accessible

One approach to this would be to create new color schemes which replace all 39 colors while ensuring the two conditions listed above.
A much simpler approach would be to focus on 5-10 colors which are most relevant to a colorblind user. That is what this PR addresses. The core question being asked is:

"How does one identify the most relevant colors on the map?"

Approaches
This section covers three potential approaches to answering the question above. Approaches no. 1 and no. 2 were attempted in the course of this PR. Approach no. 3 was not attempted due to the amount of resources and time it would require.
These approaches are NOT conclusive (hence the ISSUEs highlighted) and can be built upon further.
Approach no. 1
"How many times is each color referred to in the StyleSystem.js file?"
PROCESS: Use the file search function in whatever file editor being used
ISSUE: Does not account for how often that specific definition is rendered on the map e.g. red is only referred to once in StyleSystem.js but the feature that uses it (building) occurs the most on the map (specific to the evaluation map location)
Approach no. 2
Builds on Approach no. 1 by asking: "How many times is each OSM tag and value RENDERED?"
PROCESS: Programmatically count during the render() function call in /modules/pixi/PixiLayerOsm.js and print the results in console
ISSUE: Does not account for how features change across the map with changes in location and zoom level. >>> NOTE: One cannot evaluate the entire map at once because Render only renders the features (areas/polygons, lines and points) at a certain zoom level
Approach no. 3
Builds on Approach no. 2 by asking: "How many times is each OSM tag and value rendered ACROSS THE ENTIRE MAP?"
PROCESS: Move across the map in tiles/extents at every possible zoom and evaluate at every possible location >> Very rigorous, very time-consuming
ISSUE:  Does not account for the relative size of the feature being rendered e.g. building is the OSM tag rendered the most BUT features like highway get rendered a lot bigger and take up more visual space
Results and Implications
This section is based on the data gathered in Approach no. 2. The frequency counter code chunk is dependent on the entities rendered in the current map extent (as specified in the URL used).
Research Context and Results

Map extent: map=15.53/6.6005/3.3352
URL: http://<IP_ADDRESS>:8080/#background=Bing&datasets=fbRoads,msBuildings&disable_features=boundaries&map=15.53/6.6005/3.3352
Console Output:

Link to Data + Sheet Descriptions: https://docs.google.com/spreadsheets/d/1M56K55LK4txRY_y68z0EESOMRL_y0974CLEvssZ4h3A/edit?usp=sharing

STYLE_SELECTOR Frequency (Specific): Contains the loaded STYLE_SELECTORS (osmkey and osmvalue), how many times each osmvalue was loaded, its assigned styleID, the color definitions attached to said styleID and previews of the defined colors (MOST IMPORTANT !!)
Color Definition (direct): Contains every hardcoded color value, the STYLE_DECLARATIONS in which it is used and how many times it is used
Color Definition (colors.json): Contains every color value defined by colors.json, the STYLE_SELECTORS in which it is used and how many times it is used

Research Implications
Based on the data gathered, I identified the 4 or 5 most frequently-used colors which heavily impacted accessibility. These identifications were further backed by visual confirmation using the colorblind emulators incorporated from #1365.
Once these colors were identified, the initial question - "How does one identify the most relevant colors on the map?" - was considered answered. You can check out the /data/color_schemes.json file in #1384 to see how these identified colors were redefined.

This is really interesting work, but I don't know what to do with it right now.  Thanks!
see also my comments on https://github.com/facebook/Rapid/pull/1384#issuecomment-2317859716


fix a small bug

The file "3ddfa/inference.py" line 83 should be "pts_2d_68 = preds[0]" not "pts_2d_68 = preds[0][0]".

Yes, I note this problem, and I take above suggestion to solve it.


harfbuzz: 1.7.5 -> 1.7.6

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/harfbuzz/versions.
These checks were done:

built on NixOS
Warning: no binary found that responded to help or version flags. (This warning appears even if the package isn't expected to have binaries.)
found 1.7.6 with grep in /nix/store/q29jj339iyq46gh6dx7zrpbgx07b7826-harfbuzz-1.7.6
directory tree listing: https://gist.github.com/ea2a4f299a574853f5fa5137ff8505ca

cc @edolstra for review

Success on x86_64-darwin (full log)
Attempted: harfbuzz
Partial log (click to expand)
/nix/store/qhnj5y32pxj42q7q9wzp0ck8i3af2kf6-harfbuzz-1.7.6

Failure on x86_64-darwin (full log)
Attempted: harfbuzz
Partial log (click to expand)
cannot build derivation '/nix/store/c0qm3jqdzk9mlg6hw9ap0mssfyx7cgc1-apple-framework-CoreVideo.drv': 8 dependencies couldn't be built
cannot build derivation '/nix/store/qyann91j9rcxz8x5nppn490b21jpbk2l-apple-framework-Foundation.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/lp6sj4d58qxnggcysv6l0ywfq6xzilpw-libuv-1.19.2.drv': 8 dependencies couldn't be built
cannot build derivation '/nix/store/x72zxr9k4hy4qj0mj1pqga4mgwbm2azh-apple-framework-QuartzCore.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/flric8r9hfasr06g1s9krg8hfrf5hk53-cmake-3.10.2.drv': 13 dependencies couldn't be built
cannot build derivation '/nix/store/g74ir1i9has082fckmn1w6lmnxcb6yhs-apple-framework-Carbon.drv': 10 dependencies couldn't be built
cannot build derivation '/nix/store/d7ca9yfjsm0mi6b919kasjw7lb30x74p-graphite2-1.3.6.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/7zc035vyysxy7fcjk169mmcsxqcxv6x9-cairo-1.14.10.drv': 20 dependencies couldn't be built
cannot build derivation '/nix/store/y72f174vhrj4kihl17mx2b6is7dsiai6-harfbuzz-1.7.6.drv': 8 dependencies couldn't be built
error: build of '/nix/store/y72f174vhrj4kihl17mx2b6is7dsiai6-harfbuzz-1.7.6.drv' failed

Success on x86_64-linux (full log)
Attempted: harfbuzz
Partial log (click to expand)
checking for references to /tmp/nix-build-harfbuzz-1.7.6.drv-0 in /nix/store/3rk6arqk5dgavsdkl7vd8lkccnkydjx1-harfbuzz-1.7.6...
shrinking RPATHs of ELF executables and libraries in /nix/store/isfmwk4gc6mpih8ikbb0nnibqv0kmjk9-harfbuzz-1.7.6-dev
shrinking /nix/store/isfmwk4gc6mpih8ikbb0nnibqv0kmjk9-harfbuzz-1.7.6-dev/bin/hb-shape
shrinking /nix/store/isfmwk4gc6mpih8ikbb0nnibqv0kmjk9-harfbuzz-1.7.6-dev/bin/hb-subset
shrinking /nix/store/isfmwk4gc6mpih8ikbb0nnibqv0kmjk9-harfbuzz-1.7.6-dev/bin/hb-view
shrinking /nix/store/isfmwk4gc6mpih8ikbb0nnibqv0kmjk9-harfbuzz-1.7.6-dev/bin/hb-ot-shape-closure
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/isfmwk4gc6mpih8ikbb0nnibqv0kmjk9-harfbuzz-1.7.6-dev/lib  /nix/store/isfmwk4gc6mpih8ikbb0nnibqv0kmjk9-harfbuzz-1.7.6-dev/bin
patching script interpreter paths in /nix/store/isfmwk4gc6mpih8ikbb0nnibqv0kmjk9-harfbuzz-1.7.6-dev
checking for references to /tmp/nix-build-harfbuzz-1.7.6.drv-0 in /nix/store/isfmwk4gc6mpih8ikbb0nnibqv0kmjk9-harfbuzz-1.7.6-dev...

Success on aarch64-linux (full log)
Attempted: harfbuzz
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/w0lyajj8y5s7mp4b0dy85gd6agplvdjq-harfbuzz-1.7.6-dev
shrinking /nix/store/w0lyajj8y5s7mp4b0dy85gd6agplvdjq-harfbuzz-1.7.6-dev/bin/hb-ot-shape-closure
shrinking /nix/store/w0lyajj8y5s7mp4b0dy85gd6agplvdjq-harfbuzz-1.7.6-dev/bin/hb-subset
shrinking /nix/store/w0lyajj8y5s7mp4b0dy85gd6agplvdjq-harfbuzz-1.7.6-dev/bin/hb-shape
shrinking /nix/store/w0lyajj8y5s7mp4b0dy85gd6agplvdjq-harfbuzz-1.7.6-dev/bin/hb-view
strip is /nix/store/j7d4mr0ikv974ig7yzhknpsq288js4bs-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/w0lyajj8y5s7mp4b0dy85gd6agplvdjq-harfbuzz-1.7.6-dev/lib  /nix/store/w0lyajj8y5s7mp4b0dy85gd6agplvdjq-harfbuzz-1.7.6-dev/bin
patching script interpreter paths in /nix/store/w0lyajj8y5s7mp4b0dy85gd6agplvdjq-harfbuzz-1.7.6-dev
checking for references to /build in /nix/store/w0lyajj8y5s7mp4b0dy85gd6agplvdjq-harfbuzz-1.7.6-dev...
/nix/store/p0mbawgk2bgb7cmk8jig3bc4bknq7xcg-harfbuzz-1.7.6


CLOUDP-113301: Create atlascli packages tasks

Proposed changes

Jira ticket: CLOUDP-113301

Checklist

[x] I have signed the MongoDB CLA
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added any necessary documentation in document requirements section listed in CONTRIBUTING.md (if appropriate)
[x] I have addressed the @mongodb/docs-cloud-team comments (if appropriate)
[x] I have updated e2e/E2E-TESTS.md (if an e2e test has been added)
[x] I have run make fmt and formatted my code

Further comments

The goal of this PR is to create the packaging tasks for atlascli. This ticket is the 1st of 4 tickets that aims to release atlascli (see CLOUDP-113173).

@gssbzn I have filed https://jira.mongodb.org/browse/CLOUDP-113304 for the docker smoke tests


VS code crashes when script has syntax error or once execution completes

OS: MAC OSX 10.12.6
vs code version: Version 1.17.2 (1.17.2)
launch.json
{
        "version": "0.2.0",
        "configurations": [

            {
                "name": "Bash-Debug (select script from list of sh files)",
                "type": "bashdb",
                "request": "launch",
                "scriptPath": "${command:SelectScriptName}",
                "commandLineArguments": "",
                "windows": {
                    "bashPath": "C:\\Windows\\sysnative\\bash.exe"
                },
                "linux": {
                    "bashPath": "bash"
                },
                "osx": {
                    "bashPath": "bash"
                }
            }
        ]
    }

Script used
#!/usr/bin/env bash
export NICE_VAR=10
export ANOTHER_VAR=20

for i in {0..25..5}
do
   export NICE_VAR=$((NICE_VAR + i))
   echo $NICE_VAR
done

for ((i=1;i<=25;i+=1))
do
    export ANOTHER_VAR=$((ANOTHER_VAR+i))
    echo $ANOTHER_VAR
done

if [ $ANOTHER_VAR -gt $NICE_VAR ]
then
    echo "ANOTHER bigger than nice"
else
    echo "Nice is bigger or equal"
fi

Steps to reproduce.

Directly run the script in debug mode, vs code exits after script finishes exec
Make a syntax error and run in debug mode, vs code crashes and exits

Script with errors
#!/usr/bin/env bash
export NICE_VAR=10
expor ANOTHER_VAR=20

for i {0..25..5}
do
   export NICE_VAR=$((NICE_VAR + i))
   echo $NICE_VAR
done

for i in {0..25..5}
do
   export NICE_VAR=$((NICE_VAR + i))
   echo $NICE_VAR
done

for ((i=1;i<=25;i+=1))
do
    export ANOTHER_VAR=$((ANOTHER_VAR+i))
    echo $ANOTHER_VAR
done

if [ $ANOTHER_VAR -gt $NICE_VAR ]
then
    echo "ANOTHER bigger than nice"
else
    echo "Nice is bigger or equal"
fi

@faizanahemad
Could you provide the output of following commands for me to guess the cause of the crash?

which -a pkill
pkill -V
brew info proctools

I think your crash comes from a bug in pkill installed by homebrew. If so, you will get an output like:
$ which -a pkill
/usr/local/bin/pkill
/usr/bin/pkill

$ pkill -V
pkill (proctools 0.4pre1) http://proctools.sourceforge.net

$ brew info proctools
proctools: stable 0.4pre1 (bottled)
pgrep, pkill, and pfind for OpenBSD and Darwin (macOS)
https://proctools.sourceforge.io/
/usr/local/Cellar/proctools/0.4pre1 (9 files, 113.1KB) *
...

I fixed the bug in proctools and sent PR, however,
probably it will never be merged.
We'll fix it by avoiding using /usr/local/bin/pkill on macOS.

Here's self explanatory gif.

Please verify with version 0.1.1 (just published).

I think this is fixed, please re-open if needed...


Typo in "onedrive-api-docs/sdk/js-v7/js-picker-overview.md"

In step 3.1
Current
Click Generate New Password to create an Application secret. While this value is not necessary for the picker, it must have been created.
Proposed change
Click Generate New Password to create an Application secret. While this value is not necessary for the picker, it must be created.
I am an FTE in USDX in the bay area and want to help contribute to our OSS docs in anyway that I can!
Greg

Please consider sending a pull request.

Understood will send once back at PC.


Added some I2C example code.

This example expects the RaspberryPiSenseHAT to be attached, as well a AS612 Temperature Sensor.
I've created a second firmware image because of this requirement.
Output from the sonata_everything_demo image:
bootloader: Loading software from flash...
bootloader: Booting into program, hopefully.
led walk compartment: Look pretty LEDs!
i2c example: EEPROM ID of device at address 0x50:
i2c example:    R-Pi | 0x52 0x2d 0x50 0x69
i2c example:    .... | 0x1 0x0 0x3 0x0
i2c example:    .... | 0xf0 0x3 0x0 0x0
i2c example:    .... | 0x1 0x0 0x0 0x0
i2c example:    -... | 0x2d 0x0 0x0 0x0
i2c example:    .X.. | 0xe7 0x58 0xd1 0x95
i2c example:    .V(. | 0xf1 0x56 0x28 0xab
i2c example:    .N.B | 0xcb 0x4e 0x99 0x42
i2c example:    .y.. | 0xc7 0x79 0xd6 0xa3
i2c example:    .... | 0x1 0x0 0x1 0x0
i2c example:    ..Ra | 0xc 0x9 0x52 0x61
i2c example:    spbe | 0x73 0x70 0x62 0x65
i2c example:    rry  | 0x72 0x72 0x79 0x20
i2c example:    PiSe | 0x50 0x69 0x53 0x65
i2c example:    nse  | 0x6e 0x73 0x65 0x20
i2c example:    HAT. | 0x48 0x41 0x54 0x7f
i2c example:    .... | 0x0 0x0 0x0 0x0
i2c example:    .... | 0x0 0x0 0x0 0x0
i2c example:    .... | 0x0 0x0 0x0 0x0
i2c example:    .... | 0x0 0x0 0x0 0x0
i2c example:    .... | 0x0 0x0 0x0 0x0
i2c example:    .... | 0x0 0x0 0x0 0x0
i2c example:    .... | 0x0 0x0 0x0 0x0
i2c example:    .... | 0x0 0x0 0x0 0x0
i2c example:    .... | 0x0 0x0 0x0 0x0
i2c example:    .... | 0x0 0x0 0x0 0x0
i2c example:    .... | 0x0 0x0 0x0 0x0
i2c example:    .... | 0x0 0x0 0x0 0x0
i2c example:    .... | 0x0 0x0 0x0 0x0
i2c example:    .... | 0x0 0x0 0x0 0x0
i2c example:    .... | 0x0 0x0 0x0 0x0
i2c example: The temporature sensor configuration readout is 0x40ff
i2c example: The temporature readout is 0xc35
i2c example: The temporature readout is 0xc34
i2c example: The temporature readout is 0xc34
i2c example: The temporature readout is 0xc35

Draft because it depends on https://github.com/lowRISC/cheriot-rtos/pull/1

This breaks the format check which I have fixed in first commit of #11


Add Filters to Trade logic

We should add some filters to the trade logic. Users should be able to filter trades by position as well as any player on their roster.

Closing in favor of #46


Relative humidity  cluster [TC-RH-2.2] - Test step shows operate on device to change the temperature significantly

In Relative humidity cluster [TC-RH-2.2] testcase mention we have to manually change the temperature,currently there is no provision to enter this value or to pre-configure this value. If there is a provision to con=figure this value, it will be helpful to verify this feature.
Here I attached the link for TE 8 Verification steps
https://docs.google.com/spreadsheets/d/1h4d4J90KAxFqUL_cXZHyM3pHjHCEXzFITkQhN2KMqDY/edit#gid=1806775375

Shift to test plan issue.


KV certificates API Implementation

Ignore any checkstyle issues (incoming commit with the fix)

@JonathanGiles
@srnagar
TThank you for the review. All the comments are addressed.
I'm merging the PR for release, continue to review the PR, if you want to (will address any additional feedback in a follow up PR)


Do we need to have collector_windows.go in this repo?

Do we need to have collector_windows.go in this repo?
please see this comment: https://github.com/SumoLogic/sumologic-otel-collector/pull/1560#discussion_r1579175092

It is related to this change https://github.com/SumoLogic/sumologic-otel-collector/pull/1443 but we need to synchronise collector_windows.go with upstream version


WindowBits and degree

The choice of the "windowbits" size is influenced by the length of the scalar or the size of the degree. In the case where the degree is very small, such as 2^13, how should "windowbits" be selected? Can you give me some advice? Thanks！！

Hi Miles, as mentioned on the Z-Prize discord, i suggest studying the slides here: slides https://www.slideshare.net/GusGutoski/multiscalar-multiplication-state-of-the-art-and-new-ideas
That should give you a general idea on how to arrive at some some ballpark number for "windowbits".
The concrete number also depends on the concrete implementation of the algorithm, hardware, configuration parameters (like how much precomputation was used) etc., usually it is chosen by running benchmarks with different windowbits values and choosing the fastes one.
Please let me know if that answers your question and i can close this issue.

yeah thanks


Does airline has problems with vim versions?

Everything works just fine with the default vim7.2 of CentOS6 in terminal.
And I don't need to install powerline fonts to display those special symbols.
These are my configurations for airline.
let g:airline_powerline_fonts = 1
let g:airline#extensions#tabline#enabled = 1

But after I complie a newest vim7.4 myself, airline seems to have some problems with displaying those symbols.

The statusline doesn't suit the screen size.And when I move the cursor, the number of cursor ruler doesn't show up in the right place.
Is airline ok with different vim versions?
PS: Still CentOS6, newly compiled vim7.3 also works fine.

Most likely your terminal settings are wrong so Vim thinks, the width is different. This is not an airline problem.

Found the setting that leads to this problem. Unselecting that option will solve it.
It's because I came across an interestomg command and had a try. Some characters were wider so I double check the settings.
curl wttr.in/boston

Thank you for replying : )

@ShiFengAway Thanks for reporting back!


Package output is missing .js extensions in imports when using Typescript

Describe the problem
This is a Typescript problem (see this, and this), but package is affected by this because it produces ESM javascript.
Say your src/lib folder has two files in it: index.ts and store.ts. Your index.ts looks like this:
export { myStore } from './store'

When you compile with svelte-kit package, the result is this:
export { myStore } from './store'

Because there's no .js file extension, this is will likely fail in ESM. SvelteKit projects seem to handle this fine (I'm not sure why), but projects using webpack for example will fail with the following error:
Module not found: Error: Can't resolve './store' in 'node_modules/your-lib'
Did you mean 'store.js'?
BREAKING CHANGE: The request './store' failed to resolve only because it was resolved as fully specified
(probably because the origin is a '*.mjs' file or a '*.js' file where the package.json contains '"type": "module"').
The extension in the request is mandatory for it to be fully specified.
Add the extension to the request.

Describe the proposed solution
Frustratingly, it seems Typescript's stance on this is to add .js to your file extensions in your .ts files. This works when compiling through package, but will not work if you want to consume src/lib yourself within the kit project (i.e under /routes, if you wanted to set up docs or something).
The only way around this that I can think of is to add .js extensions in a post-compilation script. Similar to how tsc-esm does it. It's dumb that kit has to care about this, but I think it's going to be a common gotcha for many library authors otherwise.
Alternatives considered

Compiling to commonjs but... no thanks

Importance
would make my life easier
Additional Information
In that tsc-esm package, it uses a utility file from grubber to add the .js extensions. I wrote up a script and ran it against my package output:
import patchJsImports from<EMAIL_ADDRESS>
patchJsImports('./package')

and it worked on everything except my index.js file, but I haven't looked much further into that.

For anyone running into the issue for the meantime, I was able to get .js extensions added by installing ts-patch and using typescript-transform-extensions. Obviously not the ideal solution, but it will work for now.

There's a lot to unpack in that TS issue, but I kind of agree with their final decision. There's also a summary of that thread from https://github.com/microsoft/TypeScript/issues/16577#issuecomment-703190339. I'm not sure what we should do for package as we are not doing any bundling step here, module resolution works as expected for almost everything else I know, except (which you mentioned) that Webpack template. Which makes me think that this should be resolved on Webpack's loader side. I'm also moving this to p2 for reasons that there's not a lot users that uses Svelte with Webpack, and it's plugin isn't as well maintained as Rollup or Vite.
Can you also elaborate how adding .js breaks usage within the kit project itself? What happens if you write it in index.js instead of index.ts? If that works, this might need to be resolved on Vite or the vite-plugin-svelte's side.

The reason .js imports won't work within the kit project itself is because that .js file doesn't exist yet (it will only be there post-compilation in the output folder). So if you run svelte-kit package, the .js file will exist in the output and that will all be fine when consumed. But if you're importing src/lib internally in your routes, since it's just .ts files, there won't be any .js file to import and vite will throw an error. This would go for any bundler, really.
Since ESM spec requires import extensions I think this could be another QoL thing package does for you even though it's a result of using typescript. However, like you said, rollup/vite handle this gracefully, so maybe it's fine to leave it up to the bundler. It's just a bit of extra burden on library maintainers using package to communicate this, even if the webpack + svelte userbase is low.

I don't understand. If I import src/libs from the same project

Here's a repo demonstrating the problem: https://github.com/mattjennings/svelte-kit-package-extensions-bug

Ok I understand - you're stuck in an either-or situation. Either make it work for webpack, or work internally, but not both.

exactly yeah.

According to their changelog this is scheduled to release in a few days when 2.7 is out https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md

Mhm TS 4.5 reversed adding ESM support through a flag due to having concerns about it: https://github.com/microsoft/TypeScript/issues/46452
Not sure how this affects resolving this ticket.

hey @dummdidumm @mattjennings what's your current approach on this?
We're running into the same issue when trying to build packages.
Example:
//./TestClass.ts
export default class TestClass {
  ...
}

//index.ts
export {default as TestClass} from './TestClass'

Becomes this after build:
//TestClass.js
export default class TestClass {
    ...
}

//index.js
export {default as TestClass} from './TestClass'

And it errors with:
import {TestClass} from 'test-package';
        ^^^^^^^^^
SyntaxError: The requested module 'test-package' does not provide an export named 'TestClass'

I'd love to help find a solution, as this basically prevents to build a simple package when using typescript with the package command :)
Or are there any known workarounds?

What about importing with .js in your source code?
//index.ts
export {default as TestClass} from './TestClass.js'

I don't think the package command should be prepending imports with .js. That sounds very risky.

So when using package command and the library is ts based, how is this intended to work?
The files get transpiled to js, but the references / import statements aren’t, so when importing the package later on it fails.
Maybe I’m missing a point here.

The problem is TypeScript: It doesn't rewrite import paths (doesn't append a .js). There's the experimental module: node12 setting which would make TypeScript shut up when such a ending is present (right now it errors), but that module setting is only available in nightlies right now. So right now we're in this limbo state where TS doesn't fully support the new ESM resolution style yet, but you need to use it as part of SvelteKit because we decided to go ESM since it's the new standard and therefore futureproof. Until TS makes this non-experimental workarounds like the one you mention (rewriting import paths) are the best solution.

I'd love to help find a solution, as this basically prevents to build a simple package when using typescript with the package command :)
Or are there any known workarounds?

Using ts-patch with typescript-transform-extensions is the easiest workaround I found. I use it for svelte-modals if you want to see how it looks in practice.
I haven't looked into this module: node12 thing but it sounds promising, hopefully that finds its way into a stable release soon.

Looks like TS 4.7 will finally have the nodenext/whatever setting which will enforce/allow the .js ending which should fix this

We've now upgraded to TypeScript 4.7 and we can use node16/nodenext now.

People authoring packages can solve this now by using node 16 / nodenext. They need to change the following settings in their tsconfig.json:
{
  "compilerOptions": {
    // ...
    "moduleResolution": "nodenext"
  }
}

I don't think we should change this setting for all users to this though, because then everyone is required to adhere to the new node resolution rules (which means explicit file endings for relative imports), not just package authors.
I therefore think we need another option in create-svelte which let's you chose if you want to create a package or an app. This would also solve the confusion around "what's this package command in my scripts, and why does it prompt me to install other stuff first in order to use it?"

Using "moduleResolution": "NodeNext", I have an error with import { page } from '$app/stores':

Cannot find module $app/stores or its corresponding type declarations.

Could you create a new issue for this with a reproduction?

I'm not sure that "moduleResolution": "Node16" is the best for us in the long run. Many people will be confused by that. I wonder if it's worth the trouble to use the transform API of ts.transpileModule to modify the relative imports of all Svelte/JS/TS files and rewrite them so that they satisfy the strict node resolution.

TypeScript will (in 5.0? don't know) ship more fine grained module resolution flags, which hopefully solve this issue.

We set NodeNext for libraries now:
https://github.com/sveltejs/kit/blob/b47794069dbec0b1b7d5af75b42614f3b46b0811/packages/create-svelte/shared/%2Bskeletonlib%2Btypescript/tsconfig.json#L13

See also https://github.com/sveltejs/kit/pull/12163


Updated to run on 0.5 with the updated release constants.

Coverage increased (+0.8%) to 68.736% when pulling b5c6bf969cf598485bfdd50d04f91455901dbbd4 on nightly-version into c0b8fc2b6650c3a12bb6dc9896916f20f4ddf73e on dev.


Notify users when thier data has been processed

Horreum handles data async by design. Most of the time this is not an issue, but when large transformations are occurring the processing of datasets can be slowed down.  Users are not aware that their data is still in the processing queue.
A notification bar would allow userse to see when their data was being processed, and when it was ready for reporting

We could have a panel with outstanding jobs per category


Fix web deployment jnlp issue

This change is

Freaky what needs to be done in gradle for this simple step :D


Documenting "Top level" Interfaces

I wonder if interfaces that are referenced by other interfaces could be put in a special section in the docs. So there would be something like top level interfaces and interfaces that can be used by these (reuse and combination of interfaces without redundancy).
interface part {
    a: string;
}

interface root {
    b: part;
}

So when creating the docs, I would like to hide part unless the user goes to root and looks up the type for its member b. part just by itself would never been used (only in combination with root).
For now, all the interfaces are listed in the same section which is not very clear.

Separating individual documented items can be done through the @category tag. I don't think it makes sense right now to do more than that.

Separating individual documented items can be done through the @category tag. I don't think it makes sense right now to do more than that.

I tried using the @category tag and it is great!
Will there be an option to ignore certain categories?

You can use @hidden or @ignore to hide reflections - there isn't any way to hide categories currently.

But that would always hide them, right? The nice thing about @internal is that there is the --stripInternal flag which make sit flexible.

Correct, maybe I'm missing something... what's your goal here? It might be worth opening a new issue to discuss adding more filtering options.

The goal is that I can produce API docs for different purposes, depending on the categories. There are two target groups: people that simply use my lib in their projects and people that would like to develop it further.
For someone that is simple using my lib, a lot of classes etc. are not relevant. So it would be handy to simply exclude some categories using an option for typedoc, without touching the source of my project.

If this seems reasonable to you, I will gladly file a feature request.

That makes some sense - and I think some of your needs will likely be met by the 0.20 upgrade. In 0.20, typedoc will only document what you export from a given file, so for users, you could generate documentation only from your entry point, while for developers you could generate docs for the source directory, which would include all files.
That said - a hiddenCategories option seems reasonable, feel free to open an issue :)

Ok, will do :-)

@Gerrit0 issue created: #1407


Cant Open (I have read that I have to modify the code but im new to this stuff so...)

How? If someone helps me, ill give this a star ⭐

MainForm.cs you find pretty much at the Top a variable, set it to false.

MainForm.cs you find pretty much at the Top a variable, set it to false.

ok

MainForm.cs you find pretty much at the Top a variable, set it to false.

Cant find the variable

Sorry Login.cs
Line 44

Sorry Login.cs Line 44

ok, ty


Add code coverage analyses for automated tests

We should find a way to analyze and publish the code coverage when doing a builds. Coveralls might be a good solution

done with #48


v7 Plan

Goal:
Migration should be find-and-replace of import paths only, and no change should be needed that's harder than that.

[x] Switch compilation to https://github.com/ef4/decorator-transforms
UPDATE: turns out this isn't needed because ember-resources doesn't actually use any decorators internally

Smaller dist
requires minimum ember-cli-babel 8.2.0

this version of ember-cli-babel is still compatible with ember-source @ 3.28 so consumers of ember-resources should be able to upgrade without an issue.
ember-resources can't actually have a peer on ember-cli-babel, because ember-cli-babel technically isn't required ... :upside_down_face: but folks with normal apps will care!

[x] Move (the current implementation of) class-based resources out of ember-resources
Ref: https://github.com/NullVoxPopuli/ember-resources/issues/1056

[ ] Provide documentation for managing classes with resources

[ ] Add to https://tutorial.glimdown.com

[x] Move to another library for folks who want to keep using the same APIs

This library will have the same 3.28+ support promise, just class-based resources won't be included in the RFC

[x] Move the utilities out to reactiveweb
Ref: https://github.com/NullVoxPopuli/ember-resources/issues/1046

more utilities already exist and are encouraged to be added and shared here
keeps ember-resources focused on the resource implementation and documentation

Diff

Example migration

if using the utils,npm add reactiveweb

if using class-based resources,npm add ember-modify-based-class-resource

task or trackedTask
replace ember-resources/util/ember-concurrency with reactiveweb/ember-concurrency
keepLatest
replace ember-resources/util/keep-latest with reactiveweb/keep-latest
map
replace ember-resources/util/map with reactiveweb/map
trackedFunction
replace ember-resources/util/function with reactiveweb/function
helper
replace ember-resources/util/helper with reactiveweb/helper
debounce
replace ember-resources/util/debounce with reactiveweb/debounce
UpdateFrequency or FrameRate
replace ember-resources/util/fps with reactiveweb/fps
RemoteData or remoteData
replace ember-resouces/util/remote-data with reactiveweb/remote-data

seems like it's completed?

yup! thanks!

Hey @NullVoxPopuli, I'm upgrading ember-resources in our deps from 5.4.0. As a first step, I'm upgrading through version 6.5.1 but I got an error in the browser:

importing from 'ember-resources/util/ember-concurrency' is deprecated and will be removed in ember-resources@v7. The exact same code and support is available at https://github.com/universal-ember/reactiveweb. pnpm add reactiveweb and then  import { task, trackedTask } from 'reactiveweb/ember-concurrency';. See also: https://github.com/NullVoxPopuli/ember-resources/issues/1061

That's obviously here 😅 So I followed the migration examples - added reactiveweb and replaced how we import trackedTask and trackedFunction.
Unfortunately, I got a problem building the project:

Build Error (PackagerRunner) in node_modules/.pnpm/reactiveweb@1.3.0_@babel+core@7.21.3_@ember+test-waiters@3.0.2_@glimmer+component<EMAIL_ADDRESS>Module build failed (from ../../../../../../../../Users/tomek/Projects/phorest/my-phorest-ember/node_modules/.pnpm/thread-loader@3.0.4_webpack@5.88.1/node_modules/thread-loader/dist/cjs.js):
Thread Loader (Worker 3)
$TMPDIR/embroider/040dc0/node_modules/.pnpm/reactiveweb@1.3.0_@babel+core@7.21.3_@ember+test-waiters@3.0.2_@glimmer+component<EMAIL_ADDRESS>Static class blocks are not enabled. Please add @babel/plugin-transform-class-static-block to your configuration.
37 |  * State container that represents the asynchrony of a trackedFunction
38 |  */
39 | class State {
| ^
40 |   static {
41 |     g(this.prototype, "data", [tracked], function () {
42 |       return null;

Fortunately, the error is quite informative so I installed @babel/plugin-transform-class-static-block and added that plugin to the configuration. It builds again 🎉
But, I get another issue in the browser:

Error while processing route: index Could not find module @ember/owner Error: Could not find module @ember/owner
at<EMAIL_ADDRESS>(http://localhost:4200/my-phorest/bb7892c76a4460f74421.js:100394:7)
at webpack_require (http://localhost:4200/my-phorest/292f8541880780ed8d9f.js:23:42)
at ./node_modules/.pnpm/ember-resources@6.5.1_@ember+test-waiters@3.0.2_@glimmer+component@1.1.2_<EMAIL_ADDRESS>(http://localhost:4200/my-phorest/1efb467e37333c449f0a.js:125501:197)
at webpack_require (http://localhost:4200/my-phorest/292f8541880780ed8d9f.js:23:42)
at ./node_modules/.pnpm/ember-resources@6.5.1_@ember+test-waiters@3.0.2_@glimmer+component@1.1.2_<EMAIL_ADDRESS>(http://localhost:4200/my-phorest/1efb467e37333c449f0a.js:126406:102)
at webpack_require (http://localhost:4200/my-phorest/292f8541880780ed8d9f.js:23:42)
at ./node_modules/.pnpm/reactiveweb@1.3.0_@babel+core@7.21.3_@ember+test-waiters@3.0.2_@glimmer+component<EMAIL_ADDRESS>(http://localhost:4200/my-phorest/1efb467e37333c449f0a.js:131519:73)
at webpack_require (http://localhost:4200/my-phorest/292f8541880780ed8d9f.js:23:42)
at ./services/recent-clients.js (http://localhost:4200/my-phorest/bb7892c76a4460f74421.js:89938:87)
at webpack_require (http://localhost:4200/my-phorest/292f8541880780ed8d9f.js:23:42)

It seems like it's pointing to ember-resources 🤔 Not sure what to do now. Have you seen this before?

Static class blocks are not enabled.

you will need

ember-cli-babel 8.2+
ember-auto-import 2.7+
if you're using embroider, a newer @embroider/core and @embroider/compat as well (I forget the exact versions)

I installed @babel/plugin-transform-class-static-block

You should be able to remove this once you upgrade the above two packages

Could not find module @ember/owner
Have you seen this before?

yeah, this happens when you node_modules are messed up -- ember-resources is compat with 3.28+, and @ember/owner didn't exist until 4.11, so there is a dependencySatisfies check here: https://github.com/NullVoxPopuli/ember-resources/blob/2754944a0bd71a7ac3161b3313682ac13c3a6974/ember-resources/src/function-based/manager.ts#L25 which chooses between @ember/owner and @ember/application to get getOwner and setOwner so that ember-resources can work without logging deprecations.
To fix, you'll probably want to delete your node_modules, and re-install.
Lemme know what you run in to next!

Thanks @NullVoxPopuli! I manually updated ember-cli-babel and ember-auto-import as you've mentioned but didn't want to guess with embroider packages 😅 I knew they change between Ember 5.1 vs 5.2 so I upgraded and it works 🎉.
Also not sure what are the exact embroider packages but the upgrade did this:
- "@embroider/compat": "^2.1.1",
- "@embroider/core": "^2.1.1",
+ "@embroider/compat": "^3.2.1",
+ "@embroider/core": "^3.2.1",
  "@embroider/router": "^2.1.3",
- "@embroider/webpack": "^2.1.1",
+ "@embroider/webpack": "^3.1.5",


Badge: compatibility with PF themes

See for example: https://primefaces.org/diamond/misc.xhtml
There the HTML for a button with a badge is:

  <button .../>
  5

To do so, I think we need to change from:

  <pe:badge content="1" />

to:
<pe:badge content="1" severity="warning">

</pe:badge>

I'm not sure if this makes much sense... I guess it depends on what the plans with the badge will be in terms of the basic templates. Because the only reasons to use this component is that the styling is included. If styling is included you could simply create a composite or custom tag.

Closing this as badge will move to PrimeFaces

Closing this as badge will move to PrimeFaces


Stroop word does not run

Fixed it. It could not find the thisExp structure. I added it and now it works.


Wrong version number

I am on 3d951150684573fc7c76e1c91ee54f0b14b09b21, however:
[         CommandParser.h:35]

        ╔═══╗╔═══╗╔╗  ╔╗     ╔═╗╔═╗
        ║╔═╗║║╔═╗║║╚╗╔╝║     ╚╗╚╝╔╝
        ║╚═╝║║║ ║║╚╗╚╝╔╝      ╚╗╔╝
        ║╔╗╔╝║╚═╝║ ╚╗╔╝ ╔═══╗ ╔╝╚╗
        ║║║╚╗║╔═╗║  ║║  ╚═══╝╔╝╔╗╚╗
        ╚╝╚═╝╚╝ ╚╝  ╚╝       ╚═╝╚═╝ HZB 2022.

[         CommandParser.h:44]    RAY-X Terminal Application <IP_ADDRESS>
         GIT: 368fde2
         BUILD: 2022-09-26T18:54:09

Also I've compiled it today …

Hi there,
We have tested this and cannot reproduce the error.
Make sure that you are running the correct exec.
The build is now under ../build/debug/bin or ../build/release/bin

Sorry, I cannot build anymore, because of #102.

We are working on fixing the dev-branch. Seems not to be working for some...

Dev branch should be working again, could you try this again, to see if the issue still persists?

Yes, this works now, issue is gone, thanks!


crypto/tls: On a failed call to tls.Dial the returned net.Conn evaluates to false on comparison of net.Conn == nil, but prints  when printing the value

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
I've reproduced the error on go 1.10.1 linux and 1.9.2 darwin
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
I've produced the error on my mac:
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/russell/gocode"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/d3/kzyj6l8s5j3fz4848dls2tjw0000gn/T/go-build303409938=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
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"
and on my linux docker container:
GOARCH="amd64"
GOBIN=""
GOCACHE="/root/.cache/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/opt/gt/gocode"
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
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"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build289060333=/tmp/go-build -gno-record-gcc-switches"
What did you do?
During a failed call to tls.Dial, the returned net.Conn fails to evaluate to true in a comparison of net.Conn == nil.
The error is reproducible here: https://play.golang.org/p/KlpsoDFmEnf
What did you expect to see?
If an error occurs during a call to tls.Dial, the returned net.Conn should evaluate to true in a comparison of net.Conn == nil
What did you see instead?
net.Conn == nil evaluates to false, even though fmt.Println(net.Conn) will print <nil>

Simplified repo:
package main

import (
	"fmt"
)

type foo struct{}
type bar interface{}

func main() {
	var f *foo = nil
	var b bar = f

	fmt.Println(b)
	fmt.Println(b == nil)
}

I think the faq explains it nicely here
tls.Dial is returning a nil *tls.Conn (pointer to struct), and you are assigning it to a variable of interface type, that will therefore be non-nil even if the pointer inside is nil.

That's right. You are assigning a *tls.Conn to net.Conn. So the underlying type will always be non-nil. Either do a type assertion to get the underlying type, or do not assign to an interface.
Closing this as "not a bug".

Ah, I see. That makes perfect sense now. Thank you.


React Native iOS: follow example for appcenter-post-clone.sh but got "sed: Podfile.lock: No such file or directory"

Issues
File: appcenter-post-clone.sh
As title, I followed the example and still got

sed: Podfile.lock: No such file or directory

Solution
I created appcenter-post-clone.sh beside my project package.json. Make sure that you already have Podfile.lock in your project's ios folder. I actually change Podfile.lock path then check cocoapod versions for general purpose. Hope to help you guys
#!/usr/bin/env bash
CUR_COCOAPODS_VER=`sed -n -e 's/^COCOAPODS: \([0-9.]*\)/\1/p' ios/Podfile.lock`
ENV_COCOAPODS_VER=`pod --version`

# check if not the same version, reinstall cocoapods version to current project's
if [ $CUR_COCOAPODS_VER != $ENV_COCOAPODS_VER ];
then
    echo "Uninstalling all CocoaPods versions"
    sudo gem uninstall cocoapods --all --executables
    echo "Installing CocoaPods version $CUR_COCOAPODS_VER"
    sudo gem install cocoapods -v $CUR_COCOAPODS_VER
else
    echo "CocoaPods version is suitable for the project"
fi;

Environment

MacOs version 10.14.6
Xcode 10.3
CocoaPods 1.7.5
React Native: 0.60.5

Thanks for the suggestion @tientruongvan1995! Do you mind making a PR with the additional changes you had to make as an example for others who may have the same issue?

thanks for notice, but this case is for React Native, I wonder if it works with another platform, so I think I cannot make a PR for it

Thanks for the suggestion @tientruongvan1995! Do you mind making a PR with the additional changes you had to make as an example for others who may have the same issue?

thanks for notice, but this case is for React Native, I wonder if it works with another platform, so I think I cannot make a PR for it, I need some like/agreement to do that


Add Alloy deployment to Ansible hosts

Adds an Ansible role to install the Alloy agent to ship journal, auth logs and
PostgreSQL database logs to Loki so that they can be browsed, monitored and
alerted on from within Grafana.
Draft for now as Alloy is not yet configured to ship logs anywhere, this
currently just installs the agent and deposits and empty configuration file.

That's not a bad shout.
I think we could definitely do ProtectHome.
As for read-only etc, I see no reason why we couldn't switch this to a whitelist
and just read-only / and selectively permit paths underneath that.
As for memory limits, I can do this but it's just going to be a pain to add,
if we have a service reaching high memory we should set up alerting to watch for
that. Let's protect against that type of problem when it happens.
As for the others, I'll add those now.


feat: handle audio from m3u8

hello, first of all i would like to say thank you for building this :) this is the only updated library that does this functionality now days, u saved me some time!
this PR aims to handle the audio from the m3u8 playlist, which wasn't being handle correctly previously, feel free to close this PR if you think this is not needed
God bless

Thank you very much for the PR! It's merged now


save fails 1 to 1 relation - null not allowed

hi,
i have a emplyee table that has a personID column, which is a foreign key to person table.  There is a 1-to-1 relationship.  When i hit save on a employee dialog, i get a "Cannot insert the value NULL into column 'PersonId' ..." exception.  I have personId column decorated as such:
[DisplayName("Person"), NotNull, ForeignKey("[x].[Person]", "Id"), InnerJoin("jPerson"), TextualField("PersonName")]
Am i missing some thing.  I considered MasterDetail and LinkSet relation but i don't think they apply.

Why is your foreign key table reference '[x].Person'? Look at the intellisense for the table and you'll clearly see youre calling it wrong. If anything, it should be ForeignKey(typeof(PersonRow),'Id'). Do a Find call on your project for 'foreignkey' and you'll find tons of samples on how to properly use it.

I believe it was auto generated by sergen.  Even in the developer guid it using this pattern: [DisplayName("Genre"), ForeignKey("[mov].Genre", "GenreId"), LeftJoin("g")]
public Int32? GenreId
{
get { return Fields.GenreId[this]; }
set { Fields.GenreId[this] = value; }
}
Therefore, i don't think that is the problem.

so your schema name is 'x'?

That's correct.  it's something else but for privacy reason i replaced it with x

Hi the issues are still unresolved. any help will be appreciated.

[DisplayName("Person"), NotNull, ForeignKey("[x].[Person]", "Id"), InnerJoin("jPerson"), TextualField("PersonName")]
You have NotNull attribute on your row property. If it's auto generated by sergen, you must ensure about your table column is nullable.

The column 'PersonId' on table 'Employee' is not nullable by design.  As all emplyees are considered person.  I was hoping there was some behavior like MasterDetail or LinkingSetBehavior, that would do insert in the Person table prior to inserting into Employee and using the Id from Person table insert for PersonId column in employee table.
I even tried modifying the database to make PersonId nullable, eventhough i don't get any error, however, only employee table was inserted into and nothing was inserted into person table.

I think I'm understanding the issue. You essentially want to create the the employee record BEFORE the person record exists. If you do not assign a value to the personId and it's set to null it wont take the value but if you just need it to create a fresh person record when saving the employee record, you could make the personId field on the employee table to allow null and then add behavior so that the dialog will trigger a new personId if you save the field as null. You should be able to add in the savehandler repository a
protected override void BeforeSave()
{
base.BeforeSave();
if (Request.Entity.personId === null) {
Request.Entity.personID = //create new dialog on person table and return id or give it a default id;
}
}

Yes i can follow the example in listing movies for a person (the hardway) to create the person record before saving employee in the database.  However, i was hoping there would be already some behavior implemented for 1 to 1 relationship.  I will wait for Volkan to voice his opinion on this before proceeding further.

Well, actually having trouble doing the manual way as well.  Unable to create a new PersonRow in SaveRequestHandler {
protected override void BeforeSave()

Dear Volken,
upon your convenience i would appreciate if you can have a look at my dilemma and advise me appropriately.  Basically all i am trying to do is save a Employee and Person Entity together.  Employee table contains a personID foreign key to person table.  The edit dialog is for the employee entity which has person columns such as first and last name using expression attribute.  When i try to save, i get an error saying PersonID doesn't allow null.  I tried to overwrite the before save logic on the repository but i can't instantiate the personrow object as that class is sealed.  I was hoping there would be an easier way to do all this as opposed to manually handling all that work, similar to MasterDetail or linkingSet relations.  Please advise.  thanks.

:) Sealed classes can be instantiated, "not subclassed"
You need to insert PersonRow yourself, with Connection.InsertAndGetID(new PersonRow { ... }), take its ID and set it to your Row.PersonID.
UpdatableExtension is as name implies, updates an extension table, it doesn't auto insert a record.

Thanks Volkan!.  I appreciate the help.  I have one general question.  Would you recommend with regards to serenity, database designs that implement inheritance such as Person table and Employee table or a single employee table that encapsulate both person and employee fields?

  Thumbs up!


Manual switch change does not update values automatically

Checklist

[X]  I am not using Home Assistant. Or: a developer has told me to come here.
[X] I have checked the troubleshooting section and my problem is not described there.
[X] I have read the changelog and my problem is not mentioned there.

Deploy method
PKG executable
Z-Wave JS UI version
8.9.0
ZwaveJS version
8.9.0
Describe the bug
I have Fibaro and Qubino switches and dimmers and switch or dim values doe not automatically change in Zwave-UI. When I click Refresh button, values as updated as expected.
Pooling (all) devices regularly is not an option for obvious reasons.
To Reproduce
Pres manual switch on ZWave device and change is not automatically updated
Expected behavior
Device's value should atuomatically update in Zwave-UI when change is initiated on device's site.
Additional context
No response

And perform a re-interview while the log is running.

Very, very weird. Today value change works as it should. I'll dig into details trying to figure out what is different.
Not directly related to this issue I'll investigate into details the problem that seldomly occurs when turning on/off 3 switches at the same time (through Domoticz and MQTT). Sometimes not all react to change.


Allow acquisition function to be a callable

Should take as input X and a model and should return the acqusition values.

@meshcoder, I am quite new to the project but would like to work on this issue. Could you please explain more how this should be tackled?

pinging @MechCoder (fixing the spelling of the nickname)


How to create an app registration with azure js sdk?

For now I can register an app on azure portal

How can I register an application with this JS sdk? Thanks

I'm not sure that there is an API to automate this process right now. Perhaps @qiaozha can say definitively?

@nickzhums  Looks like we don't have app registration in swagger as a RP ? Can you help to confirm ？ Thanks.

@nickzhums Looks like we don't have app registration in swagger as a RP ? Can you help to confirm ？ Thanks.

Yes, I can't find a specific package under js sdk to do this thing.
But I find a doc about how to register applications with msgraph api, but it seems that the token responded by the azure login is different from the msgraph token.
Do we have a workaround to register applications on code level?

@zidaneymar Right, there is no swagger for this service so we don't have SDKs for it. did you look through Powershell or Azure CLI ?

@zidaneymar Right, there is no swagger for this service so we don't have SDKs for it. did you look through Powershell or Azure CLI ?

I'm trying to do this in a node js service so I need to avoid using Azure CLI or Powershell.
By the way, do we have any operations inside sdk which could deploy the local dotnet service to azure web app service? I found a doc here, but it needs Azure CLI. Does Azure js sdk support this?

@zidaneymar Right, there is no swagger for this service so we don't have SDKs for it. did you look through Powershell or Azure CLI ?

I'm trying to do this in a node js service so I need to avoid using Azure CLI or Powershell.
By the way, do we have any operations inside sdk which could deploy the local dotnet service to azure web app service? I found a doc here, but it needs Azure CLI. Does Azure js sdk support this?

I finally notice that their is a graph client inside this SDK, and it exists a graph api to do this.

@zidaneymar Thanks, let us know if you need any more help


Closed issue #5713 seems to be back

Describe the bug
In a python script, the object returned by GetAnnotationObject('Plot0000'), after creating a plot, seems to be malformed somehow; printing it causes a crash. This appears to be the "same" bug as #5713 at least in terms of steps to reproduce and result.
Helpful additional information

Did VisIt crash: yes
Did you get wrong results: n/a

To Reproduce
See #5713
Expected behavior
See #5713
Desktop

OS and version: toss4 linux
VisIt Version: 3.4.1, but seemed to be the same behavior on several other versions from 3.3.x and 3.2.x.
Server info (if applicable):

@rw-anderson Could you try VisIt 3.4.2 and see if this is still an issue for you. I was unable to reproduce on Windows or Linux with this version of VisIt.


Avoid using RemoteAST for incomplete types.

This allows reading the pure Swift parts of a struct containing Swift
and C members, even if the C module couldn't be imported.

@swift-ci test and merge

@swift-ci test and merge

@swift-ci test and merge

@swift-ci test and merge

@swift-ci test and merge


Check $PATH before appending to it

To address #3.

Sorry, I take it back.
We do need that part... I am going to fix it once this is merged.

Ok, I reverted that $PFILES change back to $1. Shall I merge?

I don't understand... that should be PFILES, not $1... I tested it on my computer and it works only with PFILES. Why did you revert? :-)


Facebook feature url gives 404

https://github.com/AppGyver/steroids/blob/6699fa7e199ccd8bd9882e94ac155960dfb65617/src/steroids/ConfigJsonGenerator.coffee#L42
Where is the correct urls?

That's an XML namespace, not a proper URL. It's not meant to point to a real website, though I can see the confusion. Closing this, in any case.


brew cask install Udeler

you should add udeler as a brew cask formula for easy install

By easy install do you mean building from the source on github for any unreleased version because all the released packages are already in the easiest form, you simply download and install.

when there is a formula by brew cask
then all is done by brew with one line brew cask install udeler
see https://caskroom.github.io/

Ok, I will check it.
Thanks

A cask for Udeler is now available
brew cask install udeler

I was thinking about making a gui for Homebrew Cask. I have used homebrew before, but didn't knew much about cask.
While I was searching, I found your comments on few suggestions that were already raised about this. I think that was in 2013, now Homebrew Cask has grown a lot over time. All the packages available on Homebrew Cask could help in making an app that helps you discover other apps and hence resulting in a unofficial mac app store.


create graph for  kotlin multiplatform modules relations

⚠ Is your feature request related to a problem? Please describe

when I use your module graph in my modular multiplatform app, I can't see my sub-module (common, android, ios) and their relations with other sub-modules and just see parent module relations.
💡 Describe the solution you'd like

I get this graph :
%%{
  init: {
    'theme': 'neutral'
  }
}%%

graph TB
  :composeApp --> :x
  :composeApp --> :y

but I want to create graph like this :
%%{
  init: {
    'theme': 'neutral'
  }
}%%

graph TB
  :android --> :x:android
  :common --> :x:common
  :ios --> :x:ios
  :ios --> :common
  :android --> :common
  subgraph composeApp
   :common
   :android
   :ios
  end
  :x:android --> :x:common
  :x:ios --> :x:common
  subgraph module x
   :x:android
   :x:common
   :x:ios
  end
  :y:android --> :y:common
  :y:ios --> :y:common
  :common --> :y:common
  subgraph module y
   :y:android
   :y:common
   :y:ios
  end

🤚 Do you want to develop this feature yourself?

[ ] Yes
[x] No

Hey there!
So the thing is, those aren't all modules. For instance, in the graph above, ios, android, and common are source sets, not modules. The only module there is common.
That's Gradle terminology. But I get your point, and it's an interesting and useful feature.
There's probably a way to make this work, but I'll need to investigate.
Thanks for the suggestion.

Hi,
thanks for a useful, working and easy to use plugin!
I believe I may have the same problem. We have modularized part of our application stack as separate Gradle sourceSets/configurations, which are then assembled into artifacts to be used as dependencies like this:
implementation project(path: ':commonstuff', configuration: 'security')

where Gradle project commonstuff contains a sourceset/configuration/artifact called security.
Well, it might be that the "supported" Gradle way to modularized would be to split the stuff into separate nested sub-projects of commonstuff, but it would still be awesome if this plugin could support our method.
What do you think, does it complicate the implementation too much to kind of include the referenced configuration with the referenced project?


Open Arguments by default if there are any required arguments

Hi!
Just saw this package on Twitter, great work! This is my first PR so feedback appreciated. I'm trying to figure out Pest to see how I can write tests for this, and will write an unit test without the endpoint soon.
By default, arguments section on the ShowArtisanCommand page is collapsed. This PR aims to toggle it if there's a required argument.

Included tests :)

Awesome, thanks a lot! This was quick :D

Perfect, thank you for your contribution! 🍺

Haha no worries, there's no rush. 😊