Unsuspension failure

Managed kyma reports cases when unsuspension fails with the following error
"Operation dos not contain Provisioner Operation ID"

In the reported cases, the preceding suspensions for these cases failed as well. Additional symptoms are that the RuntimeID is the same before suspension as well as after. In all cases are log entries similar to these:
Failure in unsuspending happening on 2022-08-08T02:18:14Z

unsuspend log payload
{
  "insertId": "kijpt7l3i1ua2wxq",
  "jsonPayload": {
    "operation": "0002a0d9-0d19-4378-93c8-ef13b6f90f99",
    "provisioning": "manager",
  --"instanceID": "0B83FC4F-731A-46D9-BECC-528FAD307DCB",
  --"runtimeID": "a23c1dd9-c231-4e6d-8760-87f0edcff56d",
    "stage": "create_runtime",
    "step": "Check_Runtime",
    "planID": "7d55d31d-35ae-4438-bf13-6ffdfa107d9f",
    "msg": "Operation dos not contain Provisioner Operation ID",
    "level": "error"
  },
  "resource": {
    "type": "k8s_container",
    "labels": {
      "cluster_name": "kyma-mps-prod",
      "pod_name": "kcp-kyma-environment-broker-7d895c54cc-6n57w",
      "project_id": "sap-ti-dx-kyma-mps-prod",
      "location": "europe-west1",
      "namespace_name": "kcp-system",
      "container_name": "kyma-environment-broker"
    }
  },
--"timestamp": "2022-08-08T02:18:14Z",
  "severity": "ERROR",
  "labels": {
    "k8s-pod/service_istio_io/canonical-name": "kyma-environment-broker",
    "k8s-pod/security_istio_io/tlsMode": "istio",
    "k8s-pod/app_kubernetes_io/name": "kyma-environment-broker",
    "k8s-pod/pod-template-hash": "7d895c54cc",
    "compute.googleapis.com/resource_name": "gke-kyma-mps-prod-kyma-mps-prod-app-f-8e63c4fd-qhon",
    "k8s-pod/service_istio_io/canonical-revision": "latest",
    "k8s-pod/app_kubernetes_io/instance": "kcp"
  },
  "logName": "projects/sap-ti-dx-kyma-mps-prod/logs/stderr",
  "receiveTimestamp": "2022-08-08T02:18:18.465560576Z"
}

Upgrade cluster happening on 2022-07-26T11:57:23Z, days before unsuspension with same InstanceID as well as RuntimeID

upgrade log payload
{
  "insertId": "dv5xl1y9m3qb6ra2",
  "jsonPayload": {
    "level": "info",
  --"instanceID": "0B83FC4F-731A-46D9-BECC-528FAD307DCB",
    "operation": "750244e6-a733-48a0-9a5f-6464523abad3",
    "upgradeCluster": "manager",
    "msg": "call to provisioner returned Succeeded status",
  --"runtimeID": "a23c1dd9-c231-4e6d-8760-87f0edcff56d",
    "step": "Upgrade_Kyma_Initialisation"
  },
  "resource": {
    "type": "k8s_container",
    "labels": {
      "container_name": "kyma-environment-broker",
      "location": "europe-west1",
      "namespace_name": "kcp-system",
      "pod_name": "kcp-kyma-environment-broker-5686c4cb6-wglq2",
      "cluster_name": "kyma-mps-prod",
      "project_id": "sap-ti-dx-kyma-mps-prod"
    }
  },
--"timestamp": "2022-07-26T11:57:23Z",
  "severity": "INFO",
  "labels": {
    "k8s-pod/service_istio_io/canonical-name": "kyma-environment-broker",
    "k8s-pod/security_istio_io/tlsMode": "istio",
    "k8s-pod/app_kubernetes_io/name": "kyma-environment-broker",
    "compute.googleapis.com/resource_name": "gke-kyma-mps-prod-kyma-mps-prod-app-f-ab76fd88-w4bz",
    "k8s-pod/pod-template-hash": "5686c4cb6",
    "k8s-pod/app_kubernetes_io/instance": "kcp",
    "k8s-pod/service_istio_io/canonical-revision": "latest"
  },
  "logName": "projects/sap-ti-dx-kyma-mps-prod/logs/stderr",
  "receiveTimestamp": "2022-07-26T11:57:25.638408720Z"
}

This happens only when suspension fails and RuntimeID is not cleaned because KEB can't reliably guarantee the cluster has been deprovisioned and should prevent resource leaks. The unsuspension skips provisioning of new cluster because it still thinks the old cluster is usable (which KEB can't determine)
https://github.com/kyma-project/control-plane/blob/192bc81f941bcd0db754a86c2c6f3e1659f494de/components/kyma-environment-broker/internal/process/provisioning/create_runtime_without_kyma_step.go#L47-L50
And because no new operation got created in provisioner, KEB fails the operation on provisioner check
https://github.com/kyma-project/control-plane/blob/192bc81f941bcd0db754a86c2c6f3e1659f494de/components/kyma-environment-broker/internal/process/provisioning/check_runtime_step.go#L53-L57

also worth mentioning in this case KEB can't just clean the RuntimeID and proceed as if suspension was ok. The cluster still may exist and could be in a broken state, provisioner / gardener failed to deprovision it in a rather long timeout and there is a requirement that unsuspended clusters use the same gardener shoot as before the suspension (but new unique RuntimeID for some reason).


Fixing storage test

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug

There are 9 failed test cases in storage in a recent test run. Can you have a look? Thanks!
Latest link: https://clitestresultstac.blob.core.windows.net/latest/index.html
To Reproduce

Expected behavior

Environment summary

Additional context

storage

I don't think it is time to close it. Feature-request label can avoid SLA check.

@Juliehzl please take a look.

According test results in https://github.com/Azure/azure-cli/issues/16656, it could be closed now.

According test results in https://github.com/Azure/azure-cli/issues/16656, it could be closed now.


Use Inject annotation from 'annotations' [run-systemtest]

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

System test failures in https://factory.vespa.oath.cloud/testrun/45750959/test/CheckVespaLogTest::test_start_stop_no_unexpected_log_entries probably caused by this:
1654800515.835831	517d3d78.systemtest.vespa-vespa-factory.gq2.ows.oath.cloud	21893/1	configserver	Container.com.yahoo.container.di.componentgraph.core.ComponentNode	warning	Multiple public constructors found in class com.yahoo.vespa.model.VespaModelFactory, there should only be one. If more than one public constructor is needed, the primary one must be annotated with @Inject.

Seems to be fixed in https://github.com/vespa-engine/vespa/pull/23037


Feature: Silent mode?

Right now when I do "CTRL+C" it would run final eval and print accuracy. But now this gets scrolled away with several screens of logs from wandb (things like wandb: Synced https://app.wandb.ai/yaroslavvb/transformer-xl/runs/yq4r2ru7), it would be useful to have this go to some logfile instead

Is the output getting jumbled (i.e. mixed with your eval metrics) or you just want to eliminate all wandb: output in your terminal?

Eliminate all output, I'm trying to keep stdout for things that are of immediate interest to the user (ie, someone running my training script)

0.8.2 now supports an environment variable WANDB_SILENT=true


data must be immutable in a vertex

At the moment it's possible to call put() multiple times at the same vertex. Also, it's possible to call data() even if put() was never called for a given vertex. Let's fix this by introducing the following rules:

Until put() is called for a vertex, all data() calls return Err
It's only possible to call put() once

@UARTman please, help here

@yegor256 is this one free?

@l3r8yJ yes, you can take it


ACTION not handled properly & Feature Request: customizable action formatting

Please answer the following questions.
Which version of matterbridge are you using?
run matterbridge -version
version: 0.13.1-dev
Please describe the expected behavior.
irc: /me does something
-> discord:  {RemoteNickFormat} _{Text}_
-> slack: /me user: does something
Please describe the actual behavior.
actions are not recognized as special and just transmitted like any other messages
i believe actions ( and /me ) is integrated into almost every platform reasonably well
and can be detected with relative ease (fallbck would be to not display them)
i think discord is the most ambiguous of the platforms i know well, /me just wraps the message in '_'

i just tried it again and apparently its actually incorrectly handled
a action from irc is sent as /me action test
and getting received as
bot: <username> username action test

Yes, that was explicit done like this, so that you know it's an action from irc, because the nick/username is added. So you know that user is performing the "action".
I'm not sure if it's worth all the extra code for something like this, mattermost handles actions with *, discord with _, no idea what telegram etc does.

setting a prefix, like you can already
and a postfix,
to allow wrapping the message in formatting characters,
setting it to "" by default is possible though, depending on the bridge

There is also an other problem with ACTION commands:
Some users send actions like this "does something" and the client then will sand it over as bold text for example. The problem now is that not every network supports text formatting, so you then cant see any differences between normal and ACTION text, which can get really confusing.

at least on irc it is defined as ACTION
on discord a regex ^_(.*)_$ should match all actions send with /me
and i think it is similar for slack and matrix, cannot tell if mattermost adapted it the same way .. never used it

@NikkyAI  I've added support for this for slack,mattermost,irc,gitter,matrix,xmpp and discord
You can try a binary from https://bintray.com/42wim/nightly/Matterbridge/_latestVersion (or build it yourself from master)
It's not customizable, it should do the action conversion correctly for the listed protocols.

i tested on irc and discord, it works
although i would tweak it on platforms where its possible to not wrap the entire message in cursive
but only the message part
so instead of: <Nikky> message
you get: <Nikky> message
on discord
apart from that it works very well

Ok, changed in master.

@42wim This doesn't look like it's handled correctly right now between irc and discord when using webhooks:


Feature: is it possible to show summary stats in the 4-tile home window?

It would be nice to see summary stats (not too frequently updated) in the 4-tile home window (see attached picture). I’m not aware if the app is capable of writing data to this home tile.

After the battery temperature thats the single most wanted feature 🤣
Sadly there is no documentation on how this is archived. Maybe I'll try to play with media controls some more in the future.

Maybe I'll try to play with media controls some more in the future.

@Ixam97 Do you have a branch with your previous trials? I've also been frustrated by the lack of documentation and was thinking of trying things with media controls and MediaSession
If someone manages to dig up an apk of the bluetooth app or the radio app or something, I'm willing to spend a few days staring at decompiled code to try to figure out how they are doing it.

@jjtt I have tried it with media sessions in the past, but without anny success. That does not mean it does not work tough, since that was quite a while back at the beginning of this endeavour. So I just might have done it wrong. Also, mixing media functions with a non-media app might clash with Googles guidelines and hinder a public release.
For now I have put it back as a possible feature without a high priority. I also have mantioned this while talking to my contacts at Polestar. I didn't get clear answer on how and if it is possible for third party apps, but what they said sounded very much like its prittey much a first-party thing. But maybe I can get more information frm them in the future.

Don't know if it will be helpful but Audiowagon media player can post text and pictures to the main screen tile in the lower right, although I suspect this is what you were refering to as media sessions, anyway it is open source at the following address:
https://github.com/MoleMan1024/audiowagon/

@spike3312
I am aware of how media apps do it. It would probably be possible to mimic a media app. Though I beleive there would be side effects that mess with media playback in the system.
Essentially, the app would have to present itself as a media app to the system, providing a media service containing stuff like Title, Author, Playlist, Cover Art etc. as well as the specified actions for the buttons. Those could be defined in such a way that it shows consumption data. But the system will think it is a media app. I have not tried it, because it is quite a bit of work to implement this media service. But I guess whenever CSV would be in focus, you would loose media playback, because its media service is being selcted as the media source. Likewise, CSV would have to be in focus at least once to populate it's widget, making it neccessary to open CSV, close it, and start your prefered media again.
The widgets are a Polestar-Specific thing anyways. I have no idea, how different OEMs systems would react to this kind of workaround.
If I am really bored at some point, I might give it a try. But I don't think this would be a good solution for the long run.


register code block processors after setting modification

If you modify the prefixes list, in settings, the document is updated without having to restart obsidian.
the try is to avoid error if the code block processor already existed for a prefix.

It looks good. Thanks for fixing this problem!

I added a new version without any try catch because it was not the best to do


Update CSI sepc to support expand volume

Since, current spec is 1.0.0 and does not support  expand volume. I  would to update the spec to 1.1.0 to support it.

@msau42 I need help

/assign @msau42

Hi @cwdsuzhou, which driver are you interested in adding resizing support for?  We are deprecating this repo and moving each driver to its own separate repository.

Hi @cwdsuzhou, which driver are you interested in adding resizing support for? We are deprecating this repo and moving each driver to its own separate repository.

Thank you! I wonder if you will deprecate the csi-common or move it to a separate repository?
Actually, we develop our own csi driver and it is depended on csi-common.

Which part if csi-common do you find useful? We were not planning to update csi-common but we can consider pulling out pieces that you find useful

Which part if csi-common do you find useful? We were not planning to update csi-common but we can consider pulling out pieces that you find useful

We extended the csi-common overall. Would you like to mv it to a separate repository as a common lib like provisioner-lib?  And I found other drievrs like alibaba csiplugin、ceph csi also depend on csi-common.


Test clock

We are basking in the warm glow of Test Clocks. Life is good, maybe even great. Basically feeling like a Timelord.
I doubt we want to merge this PR, because all of the code will be obsoleted when Stripe updates the openapi spec to add test-clocks. I'm opening and closing to have something to link to in discussions. If we hear from anyone else that wants a test clocks sneak peak, b0b84ebc2be351126d5e21d88fbab5aec171a347 would be a good commit to cherry-pick.

Hi, just so I don't lose oversight of this, was this added to the openapi?

Hey @imalsogreg looks like the openapi just got test clocks which we will merge asap. (see #176)
Let me know if we need any additional changes to support these (ie test_clock_ext)


Unable to use $id with $ref in schema validation

As explained in json schema spec, it should be possible to reference schema definitions via $id: https://json-schema.org/understanding-json-schema/structuring.html#using-id-with-ref
I have tested this with monaco editor, but I'm getting an error:
$ref 'version' in 'http://myserver/foo-schema.json' can not be resolved.

monaco-editor version: 0.23.0
Browser: Chrome
OS: Mac OS
Playground code that reproduces the issue:
// Configures two JSON schemas, with references.

var jsonCode = [
    '{',
    '    "p1": "v3",',
    '    "p2": false',
    "}"
].join('\n');
var modelUri = monaco.Uri.parse("a://b/foo.json"); // a made up unique URI for our model
var model = monaco.editor.createModel(jsonCode, "json", modelUri);

// configure the JSON language support with schemas and schema associations
monaco.languages.json.jsonDefaults.setDiagnosticsOptions({
    validate: true,
    schemas: [{
        uri: "http://myserver/foo-schema.json", // id of the first schema
        fileMatch: [modelUri.toString()], // associate with our model
        schema: {
            definitions: {
                version: {
                    $id: "#version",
                    enum: ["v1", "v2"]
                }
            },
            type: "object",
            properties: {
                p1: {
                    $ref: "#version"
                }
            }
        }
    }]
});

monaco.editor.create(document.getElementById("container"), {
    model: model
});

Right side is
{
    "p1": "v3"
}

Same here, this basically breaks 99% of all JSON Schemas out there. It does work fine in VS Code, so this is a monaco issue.
@aeschli @alexdima The issue is reproducible in the playground with a minimal schema as posted above. Any chance you could have a look at this?

@letmaik Can you explain what is breaking? Are you saying 99% of JSON schemas depend on $id with $ref ?
Last month we added support for '$id with $ref' to the JSON language service (4.2.0-next.x) which is now part of VS Code.
But it's not yet consumed by the monaco editor.

I guess it doesn't "break" anything since it never worked. I think I have to retract my 99% statement as it seems it's actually more common to use JSON pointers instead of '$id'. The following schema works in current monaco:
        schema: {
            definitions: {
                version: {
                    enum: ["v1", "v2"]
                }
            },
            type: "object",
            properties: {
                p1: {
                    $ref: "#/definitions/version"
                }
            }
        }

Great to see that the language service gets support for $id though. I thought that monaco is always released in lockstep with VS Code since it's just extracted from it.

The extension model is different with Monaco, so there's some manual adoption and testing needed.

I verified in the playground that the example works now.


Upgrade times out waiting for condition.

Output of helm version: v2.14.2
Output of kubectl version: v1.13.5
Cloud Provider/Platform : AKS
I'm trying in deploy my helm chart using azure devops ci-cd. The initial install is successfull but every subsequent deployments fail with
Error: timed out waiting for the condition

My k8s cluster uses istio and because of that any namespace has to have a label istio-injection=enabled
As suggested I created a Job that labels the namespace with the required value.
That job has the following annotations
helm.sh/hook: pre-install
helm.sh/hook-delete-policy: before-hook-creation

I've also tried the success and failed flag as delete policies.
Any ideas how I can fix the issue so that my builds don't time out when I upgrade ?
Deleting the namespace (clean install) works fine it's only upgrades that fail.
If I don't put the delete policy in the pipeline fails with the job already exists
Kind regards

On first run I can see this in the tiller log
deleting pre-install hook label-ns for release 0.1.88 due to "hook-succeeded" policy

On my upgrade release it looks like it's timing out because it can't find label-ns job, because it's already deleted. it looks like it goes into a loop because the hook failed
2019-07-27T19:56:35.970843921Z [tiller] 2019/07/27 19:56:35 deleting pre-install hook label-ns for release 0.1.89 due to "before-hook-creation" policy
2019-07-27T19:56:35.976732265Z [kube] 2019/07/27 19:56:35 Starting delete for "label-ns" Job
2019-07-27T19:56:35.984233621Z [kube] 2019/07/27 19:56:35 jobs.batch "label-ns" not found
2019-07-27T19:56:35.984244721Z [kube] 2019/07/27 19:56:35 Waiting for 60 seconds for delete to be completed
2019-07-27T19:56:38.003710534Z [kube] 2019/07/27 19:56:38 building resources from manifest
2019-07-27T19:56:38.007010258Z [kube] 2019/07/27 19:56:38 creating 1 resource(s)
2019-07-27T19:56:38.036412178Z [kube] 2019/07/27 19:56:38 Watching for changes to Job label-ns with timeout of 5m0s
2019-07-27T19:56:38.036733581Z [kube] 2019/07/27 19:56:38 Add/Modify event for label-ns: ADDED
2019-07-27T19:56:38.037148884Z [kube] 2019/07/27 19:56:38 label-ns: Jobs active: 0, jobs failed: 0, jobs succeeded: 0
2019-07-27T19:56:38.148374316Z [kube] 2019/07/27 19:56:38 Add/Modify event for label-ns: MODIFIED
2019-07-27T19:56:38.148699918Z [kube] 2019/07/27 19:56:38 label-ns: Jobs active: 1, jobs failed: 0, jobs succeeded: 0
2019-07-27T19:56:42.635203765Z [kube] 2019/07/27 19:56:42 Add/Modify event for label-ns: MODIFIED
2019-07-27T19:56:42.635226565Z [kube] 2019/07/27 19:56:42 label-ns: Jobs active: 1, jobs failed: 1, jobs succeeded: 0
2019-07-27T19:56:52.810132711Z [kube] 2019/07/27 19:56:52 Add/Modify event for label-ns: MODIFIED
2019-07-27T19:56:52.810435714Z [kube] 2019/07/27 19:56:52 label-ns: Jobs active: 1, jobs failed: 2, jobs succeeded: 0
2019-07-27T19:57:13.031088402Z [kube] 2019/07/27 19:57:13 Add/Modify event for label-ns: MODIFIED
2019-07-27T19:57:13.031348804Z [kube] 2019/07/27 19:57:13 label-ns: Jobs active: 1, jobs failed: 3, jobs succeeded: 0
2019-07-27T19:57:53.172990617Z [kube] 2019/07/27 19:57:53 Add/Modify event for label-ns: MODIFIED
2019-07-27T19:57:53.17336422Z [kube] 2019/07/27 19:57:53 label-ns: Jobs active: 1, jobs failed: 4, jobs succeeded: 0
2019-07-27T19:59:13.252379155Z [kube] 2019/07/27 19:59:13 Add/Modify event for label-ns: MODIFIED
2019-07-27T19:59:13.252979259Z [kube] 2019/07/27 19:59:13 label-ns: Jobs active: 1, jobs failed: 5, jobs succeeded: 0
2019-07-27T20:01:38.036923514Z [tiller] 2019/07/27 20:01:38 warning: Release 0.1.89 pre-install core-web/templates/label-ns.yaml could not complete: timed out waiting for the condition
2019-07-27T20:01:38.036965215Z [tiller] 2019/07/27 20:01:38 deleting pre-install hook label-ns for release 0.1.89 due to "hook-failed" policy
2019-07-27T20:01:39.285983842Z [kube] 2019/07/27 20:01:39 Starting delete for "label-ns" Job
2019-07-27T20:01:39.300140542Z [kube] 2019/07/27 20:01:39 Waiting for 60 seconds for delete to be completed
2019-07-27T20:01:41.318561704Z [tiller] 2019/07/27 20:01:41 failed install perform step: timed out waiting for the condition

I managed to resolve the issues by ensuring that my ci-cd pipeline uses a consistent name for the Release. for example (namespace name)
this resolved my issues


Fix common status bar disappering bug for status bar based appearance

I’m curious to know what this common status bar disappearing bug is. Do you have some sample code that demonstrates the problem?
The MPMoviePlayerViewController class (XCDYouTubeVideoPlayerViewController’s superclass) purposely overrides the prefersStatusBarHidden to always return YES, so I wonder why you need to revert that behavior.

This could be because i have UITabBarController + UINavigationController, but as you see when you presenting MPMoviePlayerController status bar is always visible even on full screen. I have problem similar to this: http://stackoverflow.com/questions/22196565/mpmovieplayerviewcontroller-hides-statusbar-and-destroys-frame-after-watching, http://stackoverflow.com/questions/7176211/disappearing-status-bar-at-the-top-after-mpmovieplayercontroller-is-closed, http://stackoverflow.com/questions/26167038/mpmovieplayerviewcontroller-status-bar-issue-ios-8-specific

It would be really nice if you could provide a sample project with a clear description of the problem and how to reproduce it. It will be much more efficient. I’d prefer not to have to go through all these Stack Overflow questions and try to build a project that exhibit a bug I have not yet fully understood.


Add feature to keep track of 'active' pdfurls

There is the possible edge case that a pdfurl for a manual entry changes. It's most likely the case that Propublica has replaced it with an auto version. However, in our current system, hypothetically if a manual entry is replaced by a link for another manual entry, the manual data that was gathered for the initial entry would be lost. In an ideal scenario, the user would have an opportunity to copy the manual entry from the old sheet into the new manual entry.
In order to facilitate this, we would need to keep track of all of the manual pdfurls that come back from API calls and see if there are any left over from get_existing_data that weren't used. At the end of the script, these entries would be put in the bottom of the spreadsheet and the user would be warned about their presence.
Since I think this situation is not terribly likely, this is definitely not a high priority.

Oh lit, I didn't even see that you already added this. Correct me if I'm wrong though, the script in its current from removes the data for the outdated pdfurls, right? Should we have some way of preserving those? It might be worth piping them to another file like outdated.csv or something so that the user can copy and paste the old data. Or would that not be as helpful? Your thoughts @jdhe1120 ?

yeah i did! so the script pops out everything that's used which means anything that remains are the outdated pdfurls + the manual #s inputted for those pdfs, we could indeed pipe it to an outdated.csv!

done! to test the feature change one of the manualpdfurls in the input file to anything random


Update documentation for Format.bnetDNF2list

Format.bnetDNF2list requires the input to be in disjunctive normal form, but this is not made explicit in the documentation.

Also, it doesn't take the full rule, just the expression (i.e.., the part after the comma).

I've updated the docstrings and manual.


Support for macOS Swift?

MediaPipe Solution (you are using)
Face Detection (would like to use)
Programming language
No response
Are you willing to contribute it
None
Describe the feature and the current behaviour/state
macOS is not listed in the supported platform
Will this change the current API? How?
No response
Who will benefit with this feature?
No response
Please specify the use cases for this feature
macOS Swift application development
Any Other info
No response

Hi @yiyinglai,
At this time, we are unable to provide Swift Package Manager support for Vision, Text, Audio, and Generative AI tasks. We currently offer iOS development through CocoaPods, which includes support for Face Detection. The corresponding documentation for implementing this in your iOS application is available on our documentation here https://ai.google.dev/edge/mediapipe/solutions/vision/face_detector/ios. Moving forward, we will maintain iOS development exclusively through CocoaPods and not other package managers as of now.
Thank you!!

Hi @kuaashish , thanks for your prompt reply. I am also wondering whether I can use MediaPipe for C++/rust projects on MacBook M2.

Hi @yiyinglai,
Yes, you can use the example in C++. However, the examples provided are for legacy solutions, which you can find here: https://ai.google.dev/edge/mediapipe/framework/getting_started/cpp. Unfortunately, the new Task API is still in the testing phase and not yet available. We will inform you as soon as it is publicly released.
Thank you!!

Hi @kuaashish , thanks for your reply! BTW, is MediaPipe slack community open of all? It seems like I couldn't join it for some reasons.

Hi @yiyinglai,
Yes, the Slack community is open to everyone. If you have trouble joining, you can use this invitation link: https://mediapipe.slack.com/ssb/redirect.
Thank you!!

Hi @calbot,
Could you please confirm if you have signed up for Slack? If not, kindly create an account first and then try to join the invitation. I have reviewed the issue again and was able to join the Slack Mediapipe community using the same invitation.
Thank you!!

Hi @yiyinglai,
Yes, the Slack community is open to everyone. If you have trouble joining, you can use this invitation link: https://mediapipe.slack.com/ssb/redirect.
Thank you!!

I can't sign in to slack... it says<EMAIL_ADDRESS>doesn’t have an account on this workspace. When I try to log in with my Google Account using the link

@calbot Same here. I'm writing an email to Slack team for help. cc @kuaashish

Hi @kuaashish , below is the reply from Slack team:

No, the link above is not the invitation link. Our Shared Invitation links have /join or /shared-invite in them as well as a long unique code that is unique to the Shared Invitation link.  The link you provided does not have that.  Please reach back out to your contact at Media Pipe and request an invitation to join their workspace.

Can you double check the invitation link? Thanks!

Hi @yiyinglai, @calbot,
Could you please try once using this Slack invitation link: https://join.slack.com/t/mediapipe/shared_invite/zt-1jqmrzg34-waJBaqdLt7_228BUOgxPwg? It worked for me, so I am not sure what issue you are experiencing. Also, it seems the Slack community is less active compared to GitHub.
Thank you!!

Hi @kuaashish , the page directly by this link says my current country is not supported any more. I thought Slack might have some casual discussion going on. Don't worry about it. Thanks for your help~

Hi @yiyinglai,
That might be issue with your case, Could you please let us know If we would like to proceed with marking it as resolved and closing the issue?
Thank you!!

Hi @kuaashish , I am closing this issue. Thanks!


Use org.apache.xml

I integrated maven-jaxb2-plugin into Debian recently and had to add these two patches for it. Hope they are useful.

Arrays.<Dependency> is OK, can be merged.

The internal version gave: "package com.sun.org.apache.xml.internal.resolver does not exist". Should we add org.apache.xml to the classpath, or what would you suggest? We can also leave this as a Debian only patch, if you would be ok with that.

The internal version gave: "package com.sun.org.apache.xml.internal.resolver does not exist".

This is weird. mvn dependency:tree on maven-jaxb2-plugin-core gives:
org.jvnet.jaxb2.maven2:maven-jaxb2-plugin-core:jar:0.14.1-SNAPSHOT
+- javax.xml.bind:jaxb-api:jar:2.3.0:provided
+- org.slf4j:slf4j-api:jar:1.7.25:compile
+- org.apache.commons:commons-lang3:jar:3.2.1:compile
+- com.sun.org.apache.xml.internal:resolver:jar:20050927:compile
+- org.glassfish.jaxb:txw2:jar:2.3.0:compile
+- org.apache.maven:maven-plugin-api:jar:2.0.9:compile
+- org.sonatype.plexus:plexus-build-api:jar:0.0.7:compile
+- org.codehaus.plexus:plexus-utils:jar:1.5.15:compile
+- org.apache.maven:maven-project:jar:2.0.9:compile
|  +- org.apache.maven:maven-settings:jar:2.0.9:compile
|  +- org.apache.maven:maven-profile:jar:2.0.9:compile
|  +- org.apache.maven:maven-model:jar:2.0.9:compile
|  +- org.apache.maven:maven-artifact-manager:jar:2.0.9:compile
|  |  +- org.apache.maven:maven-repository-metadata:jar:2.0.9:compile
|  |  \- org.apache.maven.wagon:wagon-provider-api:jar:1.0-beta-2:compile
|  +- org.apache.maven:maven-plugin-registry:jar:2.0.9:compile
|  +- org.apache.maven:maven-artifact:jar:2.0.9:compile
|  \- org.codehaus.plexus:plexus-container-default:jar:1.0-alpha-9-stable-1:compile
|     \- classworlds:classworlds:jar:1.1-alpha-2:compile
+- org.apache.maven.plugin-tools:maven-plugin-annotations:jar:3.5:compile
+- org.apache.maven.plugin-testing:maven-plugin-testing-harness:jar:2.1:test
|  \- org.codehaus.plexus:plexus-archiver:jar:2.2:test
|     \- org.codehaus.plexus:plexus-io:jar:2.0.4:test
\- junit:junit:jar:4.8.1:test

The artifact com.sun.org.apache.xml.internal:resolver:jar:20050927 contains the com.sun.org.apache.xml.internal.resolver.tools.CatalogResolver class.

Should we add org.apache.xml to the classpath, or what would you suggest? We can also leave this as a Debian only patch, if you would be ok with that.

I would like to understand, what's happening first of all.

com.sun.org.apache.xml.internal:resolver:jar:20050927:compile is not packaged in Debian (afaik), which is a requirement.

com.sun.org.apache.xml.internal:resolver is effectively a dependency of jaxb-xjc. I don't think we can just throw out this dependency and hope for good...
Another question: why is there a need in a special packaging for Debian? I don't know this story.

Well, I don't trow it out but replace it with the apache version, which should be the same (apart from the package name).
Debian requires to be self contained, i.e. all sources need to be in Debian as well.

I've pushed a patch to add the apache version to the classpath to make this working.

com.sun.tools.xjc.CatalogUtil is using the apache version as well in Debian, if I'm not wrong:
https://salsa.debian.org/java-team/jaxb/blob/master/debian/patches/04-xml-resolver-compatibility.patch

So I would propose to keep that patch in Debian and update this PR to only contain the Arrays fix, would you agree?

I think we should keep that patch in Debian. Accepting it into the master will most definitely break things here.
As for the Arrays fix, I don't quite understand, what the problem is. Arrays.asList is generic so the type parameter for the resulting list should be happily derived. I don't see any warning there - am I missing something?

I updated the PR to only contain the Arrays patch. The error I get is:
maven-jaxb2-plugin/plugin-core/src/main/java/org/jvnet/jaxb2/maven2/AbstractXJC2Mojo.java:[1303,70] incompatible types: java.util.List<org.jvnet.jaxb2.maven2.DependencyResource> cannot be converted to java.util.List<org.apache.maven.model.Dependency>

I'm not getting this error, but OK, it's a minor change.


'Permission denied' when running :prepareSandbox a second time

I'm using the following in the build file:
...
intellij {
    version '2018.3.2'
    plugins 'com.google.idea.bazel.ijwb:2<IP_ADDRESS>.2'
}

This adds Bazel plugin support in the sandboxed IntelliJ.
It works great the first time, the plugin zip is downloaded, and extracted to the sandbox directory.
Unfortunately, after making some change, trying to run the plugin again causes the :prepareSandbox step to fail with the following:
* What went wrong:
Execution failed for task ':prepareSandbox'.
> Could not copy file '/Users/<usr>/.gradle/caches/modules-2/files-2.1/com.jetbrains.intellij.idea/unzipped.com.jetbrains.plugins/com.google.idea.bazel.ijwb-2<IP_ADDRESS>.2/ijwb/lib/ijwb_bazel.jar' to '/Users/<usr>/code/permissions-bug/build/idea-sandbox/plugins/ijwb/lib/ijwb_bazel.jar'.

* Try:
Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Exception is:
org.gradle.api.tasks.TaskExecutionException: Execution failed for task ':prepareSandbox'.
	at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeActions(ExecuteActionsTaskExecuter.java:110)
	at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.execute(ExecuteActionsTaskExecuter.java:77)
	at org.gradle.api.internal.tasks.execution.OutputDirectoryCreatingTaskExecuter.execute(OutputDirectoryCreatingTaskExecuter.java:51)
	at org.gradle.api.internal.tasks.execution.SkipUpToDateTaskExecuter.execute(SkipUpToDateTaskExecuter.java:59)
        ...
	at org.gradle.internal.concurrent.ExecutorPolicy$CatchAndRecordFailures.onExecute(ExecutorPolicy.java:63)
	at org.gradle.internal.concurrent.ManagedExecutorImpl$1.run(ManagedExecutorImpl.java:46)
	at org.gradle.internal.concurrent.ThreadFactoryImpl$ManagedThreadRunnable.run(ThreadFactoryImpl.java:55)
Caused by: org.gradle.api.GradleException: Could not copy file '/Users/<usr>/.gradle/caches/modules-2/files-2.1/com.jetbrains.intellij.idea/unzipped.com.jetbrains.plugins/com.google.idea.bazel.ijwb-2<IP_ADDRESS>.2/ijwb/lib/ijwb_bazel.jar' to '/Users/<usr>/code/permissions-bug/build/idea-sandbox/plugins/ijwb/lib/ijwb_bazel.jar'.
	at org.gradle.api.internal.file.AbstractFileTreeElement.copyTo(AbstractFileTreeElement.java:79)
	at org.gradle.api.internal.file.copy.DefaultFileCopyDetails.copyTo(DefaultFileCopyDetails.java:111)
	at org.gradle.api.internal.file.copy.FileCopyAction$FileCopyDetailsInternalAction.processFile(FileCopyAction.java:44)
        ...
	at org.gradle.internal.operations.DelegatingBuildOperationExecutor.run(DelegatingBuildOperationExecutor.java:31)
	at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeAction(ExecuteActionsTaskExecuter.java:120)
	at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeActions(ExecuteActionsTaskExecuter.java:99)
	... 31 more
Caused by: java.io.FileNotFoundException: /Users/<usr>/code/permissions-bug/build/idea-sandbox/plugins/ijwb/lib/ijwb_bazel.jar (Permission denied)
	at org.gradle.api.internal.file.AbstractFileTreeElement.copyFile(AbstractFileTreeElement.java:91)
	at org.gradle.api.internal.file.AbstractFileTreeElement.copyTo(AbstractFileTreeElement.java:74)
	... 79 more

Running ls -al on the target directory shows me:
-r-xr-xr-x  1 <usr>  MYDOMAIN\Domain Users   7.5M Dec 24 10:22 ijwb_bazel.jar

Compared to other plugins, this seems to be missing the w permission on all files.

Compared to other plugins, this seems to be missing the w permission on all files.

It's worth to ask Bazel team why they do this.
What do you suggest to change in gradle-intellij-plugin?

I'm not sure they're aware of any problem, to be honest...
I wasn't (yet) able to debug the gradle-intellij-plugin to understand how :prepareSandbox worked, I imagine it tries to unzip the downloaded plugins to the sandbox directory. I wonder if there could be an additional step performed for (re)setting file permissions.
I currently "worked around" the problem by creating a "Before launch" task, calling rm -rf on the sandboxed ijwb directory.
Is there an easy way I could step through the task in the debugger? Is there an equivalent of :runIde for the gradle plugin? Or manually attach sources to an existing installation?

I'm not sure they're aware of any problem, to be honest...

You can report it to them. I think it might lead to problems on the production as well. Probably, IDE won't be able to upgrade or delete the plugin.

Is there an easy way I could step through the task in the debugger?

Clone the repo, choose any of the project in examples/ directory, change it as you need and debug its :runIde task. Debugger will be able to stop at breakpoints in gradle-intellij-plugin sources.

I'm suddenly not sure it has anything to do with write access, but I'll try to repro this and see if I can pintpoint the problem. Thanks for the debugging instructions!

closing due to lack of feedback. feel free to add details and I'll reopen the issue

Hi, @zolotov I am seeing this issue, what information can I gather to help you?


feat!: upgrade to flame 1.9.1

Upgrades to flame version 1.8.2.
Refactors code for the new flame version and applies some standards from effective Dart.

Thanks for the PR! looks great on a quick glance I'll follow up and merge as soon as I can

hey @kurtome! just applied your feedback to the PR, let me know if it looks good to you

@kurtome I'll apply the changes in a bit so we can merge all of this together 👍

@kurtome changes are up!

@kurtome I think it should be fixed now

Thanks for this!


Log the settings dump on debug 3

The settings should be dumped to the log file on debug3
logger.debug3(TerraJSONEncoder.serializableSettings(settings))

The question is where in terra should this be slipped in, on logger initialization? It would be easy to slip it into _SetupTerraLogger.configure_logger
@drewgilliam

Note you do appear to log settings in _SetupTerraLogger.configure_logger, but only to the file handler:
https://github.com/VisionSystemsInc/terra/blob/31694d83726e0e8162f5acf217d09787aa3fe80b/terra/logger.py#L260-L267

Ah, that makes sense.
A nice compromise would be to, on a per service basis:

Have settings track which elements get accessed. That should be easy enough, just has to be recursive.
Have a way to get that list, loop through the list, and print those settings at the end of a service, or on crash
Reset the list and start again for the next service

This way, only the settings used are listed, and you don't have to keep the list up to date.

The logger "logs" individual settings files to disk, to take the place of this.


Concepts doc minor fixes and updates

Makes minor improvements and updates to the concepts doc.
/assign @bowei
/cc @robscott

Thanks!
/lgtm

/lgtm
/approve


Dnx sdk failed to install due to quote in my username

When opening a beta6 based new asp.net web project a dialog appears that tells me that the dnx sdk failed to install and I should review the dnvm output.
dnvm reports the following among others:
At line:1 char:158

... hread.CurrentUICulture = '';$CmdPathFile='C:\Users\Carl in 't Veld.d ...

                                                            ~

Unexpected token 't' in expression or statement.

So it surely looks like the quote in my username does not get escaped correctly.

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain spinthemoose.com has exceeded the max emails per hour (219/200 (109%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from o6.sgmail.github.com ([<IP_ADDRESS>]:7333)
by cp1164.blacksun.ca with esmtps (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128)
(Exim 4.89_1)
(envelope-from<EMAIL_ADDRESS>id 1eVkH9-0001JD-Sg
for<EMAIL_ADDRESS>Sun, 31 Dec 2017 14:32:08 -0600
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=github.com;
h=from:reply-to:to:cc:in-reply-to:references:subject:mime-version:content-type:content-transfer-encoding:list-id:list-archive:list-post:list-unsubscribe;
s=s20150108; bh=9biYWi7HU6PzzvK2xqrHLmZc414=; b=XZOf96bcIsVEPGYk
8JbLFVqFH7bfzI1uCVJqQPbJuZblR/qfK3CwAdVzIp7B6hLuoOmLgAxzwIYsMrnk
h6w6dSYcdwXUbZ1dM3a4V7meNjSRiQwyDiPjkNH/r5DlVWzKZfVe6TIEK93FqwPr
zvbqdg7J31kh1x2a4AdpVB5ivHU=
Received: by filter0501p1las1.sendgrid.net with SMTP id filter0501p1las1-18538-5A494931-C
2017-12-31 20:31:45.995885175 +0000 UTC
Received: from github-smtp2b-ext-cp1-prd.iad.github.net (github-smtp2b-ext-cp1-prd.iad.github.net [<IP_ADDRESS>])
by ismtpd0019p1iad2.sendgrid.net (SG) with ESMTP id PzwyHR14QseQuiol7BFVCg
for<EMAIL_ADDRESS>Sun, 31 Dec 2017 20:31:45.972 +0000 (UTC)
Date: Sun, 31 Dec 2017 20:31:46 +0000 (UTC)
From: "ASP.NET Hello Bot"<EMAIL_ADDRESS>Reply-To: aspnet/Home<EMAIL_ADDRESS>To: aspnet/Home<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [aspnet/Home] Dnx sdk failed to install due to quote in my
username (#845)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_5a494931cb3e5_1e6a3ff717e04f2c11550ec";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: aspnet-hello
X-GitHub-Recipient: davidalpert
X-GitHub-Reason: subscribed
List-ID: aspnet/Home <Home.aspnet.github.com>
List-Archive: https://github.com/aspnet/Home
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AACpRL-SiTFbuzXWTGZwUYcPnE85CjfNks5tF-8xgaJpZM4FtC8t
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-SG-EID: rlj3t9ahcX0BZUzAOlajQ3VmBXo43xFJc4M1ImIDJhGhSL61aE28Rau5SaPtnp8xFDrMCiQsaR9BbB
oLYH4OnCbaYBJzdssEbARebb/KMlkK8R8BaMdl0oFVt7NdiuVJ7ip9Q79v7PbNJ1wGP8ZciOYd7SsU
MnUTpnz3+lY2YR1fwwXAuqmjYxN2gAd0rgtLbVPrO+1+SVAZB3cX/dQIWvdNOBpsrVfK01HHkgIaAj
Y=
----==_mimepart_5a494931cb3e5_1e6a3ff717e04f2c11550ec
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #845.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/aspnet/Home/issues/845#event-1405408571
----==_mimepart_5a494931cb3e5_1e6a3ff717e04f2c11550ec
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #845.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

----==_mimepart_5a494931cb3e5_1e6a3ff717e04f2c11550ec--

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain spinthemoose.com has exceeded the max emails per hour (218/200 (109%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from o8.sgmail.github.com ([<IP_ADDRESS>]:8763)
by cp1164.blacksun.ca with esmtps (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128)
(Exim 4.89_1)
(envelope-from<EMAIL_ADDRESS>id 1eVkH9-0001JH-PQ
for<EMAIL_ADDRESS>Sun, 31 Dec 2017 14:32:07 -0600
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=github.com;
h=from:reply-to:to:cc:in-reply-to:references:subject:mime-version:content-type:content-transfer-encoding:list-id:list-archive:list-post:list-unsubscribe;
s=s20150108; bh=K6BbohgqebiyfpIEicKNxrsP7Qc=; b=Uap0PbrFUSXG9WaD
scOjA2OJIu+jiLyDyg9rGdwPJtQFViXtBBeFkE1wwABqKyyUCN4uWrdf/agd+7YF
6r72bbXZRKAqwaCFS6TaQZJPWfh2nq/TwqR3vX6qrHNi/sGHTWGCKhW2eBdYNGZl
dvZRV9hiLyzZrsD67RbEFQSMa78=
Received: by filter0808p1mdw1.sendgrid.net with SMTP id filter0808p1mdw1-2484-5A494932-F
2017-12-31 20:31:46.282619529 +0000 UTC
Received: from github-smtp2b-ext-cp1-prd.iad.github.net (github-smtp2b-ext-cp1-prd.iad.github.net [<IP_ADDRESS>])
by ismtpd0017p1iad2.sendgrid.net (SG) with ESMTP id mV32DnerSHu_LYpBf652hw
for<EMAIL_ADDRESS>Sun, 31 Dec 2017 20:31:46.298 +0000 (UTC)
Date: Sun, 31 Dec 2017 20:31:46 +0000 (UTC)
From: "ASP.NET Hello Bot"<EMAIL_ADDRESS>Reply-To: aspnet/Home<EMAIL_ADDRESS>To: aspnet/Home<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [aspnet/Home] Dnx sdk failed to install due to quote in my
username (#845)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_5a494932176b7_278d3fbb505e8f301108684";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: aspnet-hello
X-GitHub-Recipient: davidalpert
X-GitHub-Reason: subscribed
List-ID: aspnet/Home <Home.aspnet.github.com>
List-Archive: https://github.com/aspnet/Home
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AACpRBASmujTaKWyk56-p9D4mAb3TjAwks5tF-8ygaJpZM4FtC8t
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-SG-EID: rlj3t9ahcX0BZUzAOlajQ3VmBXo43xFJc4M1ImIDJhH//mvzhQvBm99ckqZHkmqZTCeNQWYinzaC3e
+B+Ksuh2Xl0Pmdlyi3mjJi7ou7TOWDD/DHXIL3BR+SBYcLY+UsrGuEx4bNh0wJmXXN1yAgvjZgUDhq
jl6f2Bhl791aESsZaj88Xw+jhvJtxoz4YBIajK3/g1bzFp7KnpLfmj8jpsV9PVRBRuBt3vNlFkPd/j
8=
----==_mimepart_5a494932176b7_278d3fbb505e8f301108684
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/aspnet/Home/issues/845#issuecomment-354622571
----==_mimepart_5a494932176b7_278d3fbb505e8f301108684
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: quoted-printable
This issue is being closed because it has not been updated in 3 months.<=
/p>
We apologize if this causes any inconvenience. We ask that if you are st=
ill encountering this issue, please log a new issue with updated informatio=
n and we will investigate.
&mda=
sh;You are receiving this because you are subscribed to this thread.<=
br />Reply to this email directly, view it on GitHub, or mute the thread.

=
----==_mimepart_5a494932176b7_278d3fbb505e8f301108684--


Copyable text

Is there a way to make part of a text (or a full markdown text) selectable and copyable?

It's not implemented. I'm not sure whether it even is possible to implement with the current egui version.

As of egui 0.26 the text has become selectable and copyable by default so I'm closing this.


Can't find extension settings (and can't type the uppercase letter 'T')

Hello!
Great updates in the last few days. :+1:
But I've just come across a major issue.
I updated the extension to v5.0.1 but then hastily closed the settings pop-up that popped up after restarting Brackets. Now I just can't seem to find it! I don't know if I'm missing something but I just can't find it under any menu.
Now the reason why I want to find the settings is I can't seem to type the uppercase letter 'T' into any editor. I remember seeing Shift + T used as a key binding in the settings pop-up.
Thanks!
[Brackets Release 1.5 build 1.5.0-16538 (release cf9cf4698)]

And I also cannot use CTRL to move through words, rather than tabs... if it would have been Alt+Ctrl+arros, it would have been way simpler and create less erros like this...

Yep, I use Ctrl + Direction keys to move through words too, but I'm hoping this key binding can be disabled or changed in the settings pop-up.

Looked through the code, but didn't found anything useful... I would manually delete the bindings, if I would be able to find them...

Same here. Can't type uppercase letter 'T'!

@DH3ALEJANDRO  Could you make the shortcuts overwriteable?
On OSX I use the Command-Left Command-Right to go at the beginning end of a line, with the new version I switch on the left-right tab: is a bit of a nightmare...

please update to version 5.0.2 and then report if solved.
I apologize for any inconvenience and thank you for using Custom Work for Brackets!

Seems solved for me. Thank you for the fast update.

Works fine now. Thank you

Thanks for the awesome update!

:+1:

Seems solved after the update to 5.0.2. Thanks!
One thing though - I've observed that an extension's preferences menu item is usually placed in the Edit menu and not the Navigate menu. This is nitpicking, but I thought I'd let you know.
Cheers!


Configure CT01 to use traccar server

Do anyone have configuration commands to configure ct01 to use traccar server on my vps? As i understood this tracker uses meiligao, i've tried to send serveral sms messages but still not working

Closing as stale.


Sponsorship

DO NOT MERGE THIS YET
This branch (stacked upon the tickets branch) introduces the following two changes:

A new "Sponsorship" page (/sponsorship) based upon old wiki content, Pycon content and Steve's PDF brochure.
CSS changes for the above page.

THIS PAGE NEEDS TO BE LINKED TO IN SOME WAY! (Ideas in the comments please)

OK... all fixed as requested. Please do a final check! ;-)
Thanks!

LGTM.


Add Mobile NeRF sample

Description

Adds the Mobile NeRF sample, which is a modified version of the WebGL version developed by Google (https://mobile-nerf.github.io).
This version runs much faster on Vulkan and is a very good demonstration of a new rendering technique that is based on neural radiance fields.
Tested on Android and Windows.
This is probably not in the right sample directory, but I could not find a better location for it.
Depends on the assets added by this PR: https://github.com/KhronosGroup/Vulkan-Samples-Assets/pulls
General Checklist:
Please ensure the following points are checked:

[x] My code follows the coding style
[x] I have reviewed file licenses
[x] I have commented any added functions (in line with Doxygen)
[x] I have commented any code that could be hard to understand
[x] My changes do not add any new compiler warnings
[x] My changes do not add any new validation layer errors or warnings
[x] I have used existing framework/helper functions where possible
[x] My changes do not add any regressions
[x] I have tested every sample to ensure everything runs correctly
[x] This PR describes the scope and expected impact of the changes I am making

Note: The Samples CI runs a number of checks including:

[x] I have updated the header Copyright to reflect the current year (CI build will fail if Copyright is out of date)
[x] My changes build on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions

Sample Checklist
If your PR contains a new or modified sample, these further checks must be carried out in addition to the General Checklist:

[x] I have tested the sample on at least one compliant Vulkan implementation
[x] If the sample is vendor-specific, I have tagged it appropriately
[x] I have stated on what implementation the sample has been tested so that others can test on different implementations and platforms
[ ] Any dependent assets have been merged and published in downstream modules
[x] For new samples, I have added a paragraph with a summary to the appropriate chapter in the samples readme
[x] For new samples, I have added a tutorial README.md file to guide users through what they need to know to implement code using this feature. For example, see conditional_rendering
[x] For new samples, I have added a link to the Antora navigation so that the sample will be listed at the Vulkan documentation site

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

There is no need to include stb_image.h. It's already included in our repo via the stb submodule in https://github.com/KhronosGroup/Vulkan-Samples/tree/main/third_party

And any reason this is in the extensions folder? And why is it called mobile_nerf when it's running fine on desktop?
Maybe we need a new category for samples like these that showcase specific functionality not directly related to Vulkan?

Gave it a first test on Windows 11 + Nvidia RTX 4070 and the sample works, but there are some oddities:

I can't rotate the camera, only zoom in and out
The sample doesn't properly handle resize. If I e.g. make the window larger, the viewport isn't adjusted.

@SaschaWillems just addressing some of your questions:

Thanks for pointing out, I will update the branch to use the repo stb instead
Yeah, makes sense to just call it NeRF or NeRF Rendering since it (should) generally works well on any device that supports Vulkan
For the camera/viewport issue, I will also be fixing these on the next couple days, the initial iteration had a static rotating camera around the loaded object, but a free camera will probably work much better

@SaschaWillems and @JoseEmilio-ARM

Sample renamed to just NeRF (including supporting files)
Created the "general" category and moved the sample there
You should be able to rotate the camera now
Resizing code added, it should work as expected on windows

Thanks for reviewing!

Unfortunately the latest version is crashing for me on Android. I tried a couple of devices (Pixel 6, Vivo X90):
E/VulkanSamples: [error] [samples/general/nerf/nerf.cpp:1247] Failed to open mlp data
I/VulkanSamples: [info] Parsing mlp data /storage/emulated/0/Android/data/com.khronos.vulkan_samples/files/assets/scenes/morpheus_team/lego_ball_phone/mlp.json
E/VulkanSamples: [error] [framework/platform/platform.cpp:169] Error Message: [json.exception.parse_error.101] parse error at line 1, column 1: syntax error while parsing value - unexpected end of input; expected '[', '{', or a literal
E/VulkanSamples: [error] [framework/platform/platform.cpp:170] Failed when running application nerf

Unfortunately the latest version is crashing for me on Android. I tried a couple of devices (Pixel 6, Vivo X90):
E/VulkanSamples: [error] [samples/general/nerf/nerf.cpp:1247] Failed to open mlp data
I/VulkanSamples: [info] Parsing mlp data /storage/emulated/0/Android/data/com.khronos.vulkan_samples/files/assets/scenes/morpheus_team/lego_ball_phone/mlp.json
E/VulkanSamples: [error] [framework/platform/platform.cpp:169] Error Message: [json.exception.parse_error.101] parse error at line 1, column 1: syntax error while parsing value - unexpected end of input; expected '[', '{', or a literal
E/VulkanSamples: [error] [framework/platform/platform.cpp:170] Failed when running application nerf

Samples works fine for me on Win11 with an RTX 4070.
There are some minor nitpicks mostly not directly related to the samples, but I don't want to defer the release of it and they can be fixed afterwards. Will probably open an issue for those after the sample has been merged.

How is the main (NeRF) render-pass output synchronised with the UI rendering? I see the NeRF render-pass has an 1 -> external sub-pass dependency, but this is using dstStage = BOTTOM, so doesn't sync with the UI pass. Similarly, the UI pass (looking at the swapchain re-creation code) only appears to have an incoming dependency on COLOR -> COLOR, but COLOR happens after FRAGMENT where the NeRF output would presumably be read.
Good chance I'm missing something, so to ask :) Has anyone run with the GPU validation layer to check if there are any complains about sync issues?

Given we've resolved the issues we discussed on Monday, I'll go ahead and merge this.  @RodrigoHolztrattner-QuIC, please see if you can reply to @cforfang's comment above. Thanks!
I'll merge the assets too..

How is the main (NeRF) render-pass output synchronised with the UI rendering? I see the NeRF render-pass has an 1 -> external sub-pass dependency, but this is using dstStage = BOTTOM, so doesn't sync with the UI pass. Similarly, the UI pass (looking at the swapchain re-creation code) only appears to have an incoming dependency on COLOR -> COLOR, but COLOR happens after FRAGMENT where the NeRF output would presumably be read.
Good chance I'm missing something, so to ask :) Has anyone run with the GPU validation layer to check if there are any complains about sync issues?

I could be wrong on this, but my view is that dstStage = BOTTOM basically makes no assumption about the next dependency (as it's the same as <dstStage = NONE, dstAccess = NONE>) but the render pass we use to draw the UI waits for all previous commands, since it uses srcStage = BOTTOM (which should be the same as <srcStage = ALL_COMMANDS, srcAccess = NONE>).
Here is where we setup the begin render pass for the UI: https://github.com/RodrigoHolztrattner-QuIC/Vulkan-Samples/blob/0bd1509453b28fffcdc37b9af13ca2bd77ddb1b4/samples/general/nerf/nerf.cpp#L762
And here is where the UI dependencies are set: https://github.com/RodrigoHolztrattner-QuIC/Vulkan-Samples/blob/0bd1509453b28fffcdc37b9af13ca2bd77ddb1b4/framework/api_vulkan_sample.cpp#L758)
I believe some other examples are doing something similar, where they have dstStage = BOTTOM at their last dependency before the UI pass. I didn't see any synchronization validation error on the platforms I tested (tried debug builds with validation active), but please let me know in case someone sees any.

the render pass we use to draw the UI waits for all previous commands, since it uses srcStage = BOTTOM (which should be the same as <srcStage = ALL_COMMANDS, srcAccess = NONE>)

Ah, thank you I missed there is probably a transitive dependency going through BOTTOM here! I'd be a bit concerned about the access-masks being generally None, as if these are not correct there is only an execution dependency not a memory dependency (meaning ordering would be correct, but the right caches might not be flushed/invalidated to make sure the data is written/read correctly) -- but if the GPU sync validation layer is happy, then I'm happy :)


Correct Transit API docs to reflect changes

The Transit documentation needs to be updated to reflect changes in the API.

Oh also if you rebase this on top of current trunk, the CI tests should start passing. It was an unrelated problem that I just fixed in a different PR.. obviously a docs change should not be able to cause unit tests to fail :).

I think that should do it. Let me know if there’s anything else you’d like me to do on this.

I think you accidentally checked in some vscode setting file.


release version 0.5.0

Change log:
1. Fix some time the global timeout will exceed
2. Add new API to support TcpStream::connect_timeout
3. Add new APIs to support getting command exit status
4. pub LocalChannel & ChannelBroker as some of their methods are necessary

This PR closes #90 and #92
Examples are updated.


Fixed broken link to Postnord Api

Fixed broken link to Postnord Api.

[x] My submission is formatted according to the guidelines in the contributing guide
[x] My addition is ordered alphabetically
[x] My submission has a useful description
[x] The description does not have more than 100 characters
[ ] The description does not end with punctuation
[x] Each table column is padded with one space on either side
[x] I have searched the repository for any relevant issues or pull requests
[x] Any category I am creating has the minimum requirement of 3 items
[x] All changes have been squashed into a single commit

HugoW5:patch-1


Updated UI of the app

Imparted a UI to the app.
Fixes(#3 )
also added corresponding screenshots.
Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.
If there are any UI change, please include the screenshots also.
Fixes #(issue)
Type of change
Just put an x in the [] which are valid.

[ ] Bug fix (non-breaking change which fixes an issue)
[ x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ x] My code follows the style guidelines of this project
[ x] I have performed a self-review of my own code
[ x] I have commented my code, particularly in hard-to-understand areas
[ x

] I have made corresponding changes to the documentation

[x ] My changes generate no new warnings

Hey @chanmol1999,
Thanks for designing this layout. Now the issues #3 required only mockups from participants and not the actual code. The reason behind this was to first reach a consensus on how the app should look like before the coding of the UI can begin.
Moreover, participant must express their interest before taking up any issue. I understand a lot of your effort and time was invested in to this, but I'll have to close this.


Support length for various fields

Ready for another round of review now.

Mind pulling in the latest changes from https://github.com/google/python-spanner-orm/pull/190 ? I'm hoping that will get github to run the tests on this so we can merge it.

Just rebased. It seems that it requires you to approve running workflows.


EAS Build not passing environment variables to expo prebuild

Build/Submit details page URL
https://expo.dev/accounts/nzapponi/projects/dctr-sandbox/builds/f968445a-12df-4fe0-a4ce-cbbf9373a9f3
Summary
I have a project with an app.config.js which is dynamic based on an environment variable called APP_VARIANT.
I set APP_VARIANT inside eas.json, in the env dictionary.
I also added my local .env and ios and android folders to .gitignore.
I can see the env variable is being picked up correctly during the remote build, as I can see the app config has been generated correctly.
However, something goes wrong during the Prebuild step, as the target isn't found right after:
Configuring Xcode project
Assigning provisioning profile '*[expo] XXXXXXXX AppStore 2024-06-14T23:09:24.981Z' (Apple Team ID: XXXXXXXX) to target 'XxxxSandbox'
Could not find target 'XxxxSandbox' in project.pbxproj

The only way around it is to remove ios from .gitignore, prebuild locally, and then build in EAS. In that case, it's all smooth.
Managed or bare?
Managed
Environment
expo-env-info 1.2.0 environment info:
System:
OS: macOS 14.5
Shell: 5.9 - /bin/zsh
Binaries:
Node: 20.8.0 - ~/.nvm/versions/node/v20.8.0/bin/node
npm: 10.2.0 - ~/.nvm/versions/node/v20.8.0/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.15.2 - /opt/homebrew/lib/ruby/gems/3.0.0/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 23.5, iOS 17.5, macOS 14.5, tvOS 17.5, visionOS 1.2, watchOS 10.5
IDEs:
Android Studio: 2023.3 AI-233.14808.21.2331.11842104
Xcode: 15.4/15F31d - /usr/bin/xcodebuild
Expo Workflow: managed
✔ Check Expo config for common issues
✔ Check package.json for common issues
✔ Check native tooling versions
✔ Check dependencies for packages that should not be installed directly
✔ Check for common project setup issues
✔ Check npm/ yarn versions
✔ Check for issues with metro config
✔ Check Expo config (app.json/ app.config.js) schema
✔ Check that packages match versions required by installed Expo SDK
✔ Check for legacy global CLI installed locally
✔ Check that native modules do not use incompatible support packages
✔ Check that native modules use compatible support package versions for installed Expo SDK
Error output
No response
Reproducible demo or steps to reproduce from a blank project
See above

Never mind. I had a postinstall script calling prebuild without any env variables, it must have confused it


compass/sass config not accessible?

Expected behavior and actual behavior
When in config.rb:
compass_config do |config|
  config.output_style = :expanded
  config.line_comments = true
end
activate :sprockets

I would expect to see output of css in development to be "expanded" and to see "line comments", but no. I am seeing css output as :nested. I also tried:
set :sass, output_style: "expanded"

And then when visiting http://<IP_ADDRESS>:4567/__middleman/config I see the :output_style has changed from the config's point of view.. but no change in the actual output in the css.
When I comment out sprockets, the expected behavior is good.. but then out goes the coffee2js compilation so... any way to ensure compass/sass settings in config are picked up and so sprockets does what I'd expect?
Steps to reproduce the problem (from a clean middleman installation) or example repo
Gemfile:
ruby '2.3.1'

# Middleman Gems
gem 'middleman', '>= 4.1.8'
gem "middleman-sprockets", github: 'middleman/middleman-sprockets'
gem 'middleman-livereload'
gem 'middleman-compass', '>= 4.0.1'

# See: https://github.com/Compass/compass/pull/2088
# --
# The following github branch should be replaced when Compass catches up and merges
# the new interpolation near operator syntax fixes and middleman-compass is updated.
git 'https://github.com/ably-forks/compass', branch: 'sass-deprecation-warning-fix' do
  gem 'compass-core'
end

gem 'middleman-gh-pages'
gem 'susy', '~> 2.2.11'
gem 'breakpoint', '~> 2.5.0'
gem 'middleman-syntax', '~> 2.1.0'
gem 'middleman-jquery'
gem "faker", "~> 1.6.3"
gem 'rack-contrib'
gem 'puma'

Additional information

Ruby version: 2.3.1
Middleman version: 4.1.8
Middleman Sprockets version: latest master as of today 11/30/2016
OS version: Mac OS 10.11.6

Ah interesting, I hadn't ever thought about it as I'd always used the default in dev and depended on the css compressor extension for production. It looks like actually configuring Sass is something that hasn't landed in a "stable" Sprockets yet (but will come with 4.x). That should definitely be something we support.
That wouldn't really help you though if you do need anything special from Compass. Middleman::Sprockets & Middleman::Compass aren't really compatible since Middleman::Compass configures Tilt templates that would never get used w/ Sprockets.
For you, I think if you don't need any of Sprockets specific features in your sass, the best thing would be to disable it for those files and only run it on your js.
activate :sprockets do |c|
  c.supported_output_extensions = [".js"]
end

Hope that is helpful, thanks for the great report.


Examining wishlist bdd bug

IBM Cloud toolchain: Delivery Pipeline deployed shopcarts to prod, including commit 0249c56b51c27c5ba040147711120d01510b7a09

IBM Cloud toolchain: Delivery Pipeline deployed shopcarts to dev, including commit 0249c56b51c27c5ba040147711120d01510b7a09


vpptrace: Allow to specify multiple input interfaces.

With this pull request you can use vpptrace to capture packets on mutliple input VPP nodes:
./vpptrace.sh -i tap -i dpdk
tap+dpdk is now also the default, so the "-i" option is no longer mandatory, i.e. the above is equivalent to: ./vpptrace.sh

Coverage decreased (-0.09%) to 76.358% when pulling 93fc781effadeabe043811aac8d002f8a97902b2 on milanlenco:master into 25075db2d3273d1fdaf5e2127846510b7af840ff on contiv:master.


Re-enable CircleCI webhook event type

This will enable "CircleCI (webhook)" as a Source option on the Tokens page.
Before we can stop using the circleci-manual event type, we need to
verify that the CircleCI webhook now works in production, and then
switch over all audited repos to use the official webhook.
Discussion about why the manual CircleCI webhooks exist: https://jira.fundingcircle.com/browse/ENG-86

LGTM

as agreed: we might roll-back on this if the CircleCi webhook messages are still being blocked by our production systems.


feat: allowing walStorage to be changed on existing cluster

#675
Signed-off-by: John Long<EMAIL_ADDRESS>

/test

/test

/test limit=local

/test limit=local

the other commit given that is too large and has a proper meaning has been moved to: https://github.com/cloudnative-pg/cloudnative-pg/pull/1319

/test limit=local

/test limit=local

/test

/ok-to-merge


Netty fails to build on ppc64le platform

OS: Ubuntu 14.04
Arch: Power (ppc64le)
Command to build:
mvn clean install -DskipTests
Failure message:
[INFO]
[INFO] --- maven-enforcer-plugin:1.2:enforce (enforce-tools) @ netty-parent ---
[WARNING] Rule 2: org.apache.maven.plugins.enforcer.RequireProperty failed with message:
x86_64 JDK must be used.
Any inputs on above will be greatly appreciated.
Thanks in advance,
Anup Halarnkar

@anuphalarnkar does this work ?:
mvn clean install -DskipTests=true -Denforcer.skip=true

@anuphalarnkar let me close this ... if it still not works let me know


Series fetching info from TMDB and not TVDB

Description
All media currently prefers to fetch metadata from TMDB.
This causes issues as Sonarr uses TVDB and you get season discrepancies.
Version
1.6.0
Steps to Reproduce

Search for Jujutsu Kaisen

Observe that it shows up as a single season

Search it up on TVDB

Observe that it is split into several seasons

Request Jujutsu Kaisen

Check Sonarr

Only the first season out of two is requested

Screenshots

Logs
Do not think this is relevant, but can provide if necessery

Platform
desktop
Device
Any
Operating System
Docker on Ubuntu
Browser
Any
Additional Context
No response
Code of Conduct

[X] I agree to follow Overseerr's Code of Conduct

Series is fetching from tmdb because tvdb api is not free

Did some searching, and it seems like Sonarr uses TheXEM which is a community driven database. Any chance of using that?

Dont they use skyhook


Travis Fortran tests

This PR:

enables the Fortran unittests with pFUnit (#46)
disables all macOS builds (commented out)

launch jenkins


List records to moderate

Need to cherry pick last commit off into new branch - one of the previous commits that will be merged in other PR's has changed.

did in another pull request


Adding warning boxes in target

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes
Fixes #6876
by showing the warnings available through Prometheus API response in form of error boxes in Target UI
Verification
They are tested through unit tests! It is not possible to get fake warnings in the Prometheus API response, as warnings are only included in the response if there were warnings while executing the request. So programmatic tests are the only option!

Hey! @GiedriusS
I tried to find a way to make a query so that the Prometheus API's response has warning in it, but unfortunately I am unable to do so, thus the PR is untested!! Sorry.

Have you seen https://github.com/thanos-io/thanos/pull/7036/files? Maybe you want to collaborate with them on tests?

We collaborated on the API code instead of the tests (I wrote the tests myself). Can you review please @GiedriusS

But the API code was totally mine though 🤔

But the API code was totally mine though 🤔

Absolutely! I wrote it in the message! Now I have also tagged you in the comment to give you sincere credit of your work! Thanks

Closed the PR as I was not able to manually test the UI.


Lazysizes not showing images on iOS Safari 11.0 & 12.0

First of all thanks for creating Lazysizes. It performs very well.
Although I have a problem where users of Safari on iOS 11.0 & 12.0 are not seeing any images loaded via data-src. Is this a known problem? Or are these versions too old to understand the way Lazysizes is working. Couldn't find an issue in the issues list directly related to this.
Thanks
Brian

I just tested with 12.2 and it works perfect. Can you give me a testcase that shows your problem.
It is a bug. But I need to be able to reproduce it.

Thanks for the quick reply.
Yeah it works perfect on newer versions. The problem is appearing on https://www.maxilia.nl
Maybe you can check?

I now additionally tested your site using 11.4 and I still can not reproduce. I will try 12 soon. but what I can tell is that you are using a fairly unusual and unnecessary markup:
<picture class="fixed-ratio" style="padding-bottom:61%;">

</picture>

Please change this to the follwing:

Any updates to this? I'm also experiencing this issue when using a <picture> tag... It works fine on Chrome and Firefox, but not on Safari.

I had the same issue and found this thread. The issue for me wasn't with lazysizes but Safari and image URLs not including an extension. We're using Amazon's static image resizing solutions and the urls do not contain the .webp extension. Safari attempts to use that type despite not supporting it and the image fails to load. The solution is to include the image MIME type on the source tag for unsupported formats, this let's Safari know to use the next compatible image.
`

`

I'm sharing my experience.
I faced an issue with images on Iphone, using Lazysizes.
I thought the library was the reason.. Actually I was serving WebP formatted images.. which is not supported by Safari.

Can anyone see anything wrong with the following? I'm experiencing the same weirdness in Safari (13.0.5) on macOS but where I have been careful to avoid WebP.
`

`

Also sharing my experience using React & NextJS:
    "next": "^9.5.4",
    "next-on-netlify": "^2.3.2",
    "react": "^16.13.1",

The lazyload class remains lazyload regardless of scrolling to the element in Safari. I have no clue why:
https://www.loom.com/share/0844e68699774fbab41b22a06bc0ec0c
Link for testing:
https://beta.daysetter.com/content/california-us/inyo-county-ca/itinerary/a-daytinerary-to-death-valley-from-vegas
I'm using a blur placeholder, so I'm thinking of just falling back to a normal image (non blurred).


Feedback on specifications, submitted by Rachel Lawson

Rachel Lawson (cmxhd3NvbkBkaWdpdGFsaW1wYWN0YWxsaWFuY2Uub3Jn) submitted the following feedback: Looking at page: https://govstack.gitbook.io/bb-workflow/5-cross-cutting-requirementsJust another check - I need to insert the page we are interested in talking about.

okay to close test issue


#[wasm_bindgen(skip)] for impls functions

Motivation
Add way to skip pub functions in #[wasm_bindgen] impls when generating JS glue.
Proposed Solution
#[wasm_bindgen]
impl Foo {
    #[wasm_bindgen(skip)]
    pub fn bar(&self) -> i32 {
        7
    }
}

Alternatives
Skip pub (crate/self/anything) functions.

You can accomplish this by simply not using #[wasm_bindgen]:
impl Foo {
    pub fn bar(&self) -> i32 {
        7
    }
}

#[wasm_bindgen]
impl Foo {
    // functions that you want to send to JS
}

This works because you can have many impl blocks for a single type.

@Pauan ,
Thanks! Closing then.


Multijson and schema validation

This PR includes a few changes we found useful:

Use multi_json to handle swappable JSON backends with a robust implementation rather than the homegrown thing here.
Add a JSON::Validator.fully_validate_schema method that makes it easy to get all the errors in a given schema.

Let me know if you have any questions.

Thanks for the commits. I had originally looked into multi_json when looking to solve the issue of json backends, but did not want to add a dependency to the library. I've pulled multi_json directly into the library and updated a few of the changes for backwards compatibility.

Thanks for merging this so promptly :)!


Update README.md for graph transform

Do not recommend to useremove_nodes(op=Identity, op=CheckNumerics) in the README. It breaks the control flow based operations.

Can one of the admins verify this patch?

Thanks for sending the PR @raingo. @petewarden mind taking a look? Thanks!

Thanks for the PR! Can you give some more background on the specific issues you're hitting? Control flow is generally only needed for Variable assignment, which generally doesn't happen during inference, so it would be good to know more about what's going wrong in your case.

@petewarden For example, for object detection, there is a tf.image.non_max_suppression step per each object class. I used tf.map_fnto iterate through each class. This iteration is necessary in the deploy time.

@petewarden any modifications we would need here?

Sorry for the slow response on this. Can we address this by leaving the code example but adding documentation along the lines of "this may cause problems if you have a model with control-flow based operations"? I'd hate people to miss out on the optimizations in other cases.
I think the real solution would be to warn or otherwise avoid removing ops that are significant for control flows, but I'm not sure how to do that yet.

ping for @raingo !

@petewarden PTAL. Just added a note about the control flow ops.


Fix: Magic link login not working properly when booted from docker

Problem
When using docker-compose to deploy to any remote environment, the same problem as #186 occurs.
This is caused by the following factors happening at the same time:

the Secure attribute exception does not work because the domain is not localhost (or browsers that respect the Secure attribute even on localhost, such as safari).
NODE_ENV is fixed to production and Secure: NODE_ENV === "production" is always true.

It can be reproduced in the local environment by replacing http://localhost with http://<IP_ADDRESS>.nip.io.
This pull request solves factor 2. of the above factors.
Therefore, it is assumed that #186 will be resolved in many cases.
Detailed cause of occurrence
When booted from a docker image, entrypoint calls /docker/scripts/entrypoint.sh,
Inside it, exec dumb-init pnpm run start is executed.
where cross-env NODE_ENV=production node --max-old-space-size=8192 . /build/server.js will be executed.
At this time, since NODE_ENV is fixed to production, even if environment variables are specified in env_file etc., they are overwritten and the above problem occurs.
Solution
Add start-from-docker command to package.json and call it when starting from docker container.
The command is exactly the same as start except that NODE_ENV is not overwritten.
This fix will ensure that the NODE_ENV specified in the .env file is respected and the above bug is repaired.
/claim #186

The command name was changed to start:local.
Please let me know if you have additional suggestions or comments.

Thank you so much for the contribution! I will review this tomorrow 👍

It has been two weeks. still planning to merge this PR?


I cannot get custom plugins and running workflows at the same time

Hi,
I'm playing with this Github action. It deploys my blog, but the somehow I can't get it to deploy the state of the gh-pages branch.
My repository with the blog is here: https://github.com/list4c/list4c.github.io/tree/gh-pages
In pictures:

This is how the figures look when my publishing source is main (set in Repository - > Settings -> Pages)
If I change it to gh-pages, then the website content gets updated and the timeago plugin works, showing yesterday - see here:

So I was thinking that I should change it to gh-pages, but when I do it, the workflows stop running. So that must be the wrong option. What could I be doing wrong?
So I would like to: have workflows running as defined + have the plugin correctly working on the deployed site, on a continuous basis. I guess something might be set up incorrectly with my deployments.
I'll greatly appreciate your expertise on this matter

Hi!
Ok, I found out what was going on :)
I deactivated one of the workflows, thinking that Github Actions somehow 'remembered' another github action that I tried before yours. But in fact the during deployment there are two workflows running - one that builds the site and pushes the output to gh-pages branch

And then the Github-owned workflow which builds and deploys the page from the selected source branch (gh-pages here), right?

So, after activating back that workflow and switching the source to gh-pages branch, everything works like a charm.

Hi @list4c
Sorry for replying your issue late, have you solved this issue, I'm happy to help you. : )
Thanks & Regards

Yes, thanks for asking!


query servers install via env var only but docs state about [query_servers]

Hello, I've just found out that in 2.3 the only way to configure a query server is via env vars, which is neat, but documentation and distributed local.ini are misleading in sense that they let persons think there is still an inifile way.
I'd like to suggest you to update documentation so it's more clear otherwise users can't relax :P

Thou shalt read the release notes: https://docs.couchdb.org/en/stable/whatsnew/2.3.html#upgrade-notes
:)
B.
--
Robert Samuel Newson
<EMAIL_ADDRESS>On Tue, 5 Mar 2019, at 16:09, Alessandro Ogier wrote:

Description

Hello, I'm trying to install a query server on a local instance but so far no luck.

I've dropped an inifile in local.d:

[query_servers]
python = /usr/bin/python3 /usr/local/bin/couchpy
and running couchdb seems to give me a positive feedback about this:

curl -sH 'Content-Type: application/json'
http://localhost:9999/_node/nonode@nohost/_config
| jq .query_servers

{
"python": "/usr/bin/python3 /usr/local/bin/couchpy"
}

but when I hit a view whose ddoc is:

{
"_id": "_design/dd",
"_rev": "4-46dba025cd653009291b774ae5926444",
"views": {
"vv": {
"map": "def map(doc):\n    yield(doc['_id'], 'pp')"
}
},
"language": "python"
}
server logs this:

`[info] <0.16030.0> -------- Starting index update for db: shards/40000000-5fffffff/ciao.1551798662 idx: _design/dd
[error] emulator -------- Error in process <0.16035.0> with exit value: {{nocatch,{unknown_query_language,<<"python">>}},[{couch_query_servers,get_os_process,1,[{file,"src/couch_query_servers.erl"},{line,575}]},{couch_query_servers,start_doc_map,3,[{file,"src/couch_query_servers.erl"},{line,55}]},{couch_mrview_updater,start_query_server,1,[{file,"src/couch_mrview_updater.erl"},{line,235}]},{couch_mrview_updater,map_docs,2,[{file,"src/couch_mrview_updater.erl"},{line,188}]}]}
[error] <0.15987.0> 5cfdafc411 rexi_server: from: nonode@nohost(<0.15593.0>) mfa: fabric_rpc:map_view/5 throw:{unknown_query_language,<<"python">>}[{couch_mrview_util,get_view_index_state,5,[{file,"src/couch_mrview_util.erl"},{line,137}]},{couch_mrview_util,get_view,4,[{file,"src/couch_mrview_util.erl"},{line,81}]},{couch_mrview,query_view,6,[{file,"src/couch_mrview.erl"},{line,247}]},{rexi_server,init_p,3,[{file,"src/rexi_server.erl"},{line,140}]}]
[error] <0.15593.0> 5cfdafc411 req_err(4089041121) unknown_query_language : python
[<<"couch_mrview_util:get_view_index_state/5 L137">>,<<"couch_mrview_util:get_view/4 L81">>,<<"couch_mrview:query_view/6 L247">>,<<"rexi_server:init_p/3 L140">>]
[notice] <0.15593.0> 5cfdafc411 localhost:9999 <IP_ADDRESS> undefined GET /ciao/_design/dd/_view/vv?reduce=false&skip=0&limit=21 500 ok 6

`

Can't get any clue from documentation (which is very little, in fact I've copied inifile snippet from vanilla local.ini) nor online, but that unknown_query_language let me think something is still wrong.

Can you please give me a pointer I can't find by myself?

Your Environment

couchdb 2.3.0 dockerhub's image

curl -sH 'Content-Type: application/json'
http://localhost:9999/
| jq

{
"couchdb": "Welcome",
"version": "2.3.0",
"git_sha": "07ea0c7",
"uuid": "36f4f3fa844df0434a60b28a6fd3704d",
"features": [
"pluggable-storage-engines",
"scheduler"
],
"vendor": {
"name": "The Apache Software Foundation"
}
}

entire configuration: https://gist.github.com/aogier/72b7fc522f5df03a94a30e6d67b74f36

Thank you for everything, regards

ale

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/apache/couchdb/issues/1966, or mute the thread https://github.com/notifications/unsubscribe-auth/AAC4dxC8y3lEyroFnh1ws9ZkElPyhaVGks5vTpQtgaJpZM4be7RJ.

you're right, my bad, I started with this one so I just read the manual. Sorry for the noise, ciao!

@aogier We're very much accepting pull requests for the docs over at https://github.com/apache/couchdb-documentation if you'd like to help out! :)


Ajnart/rework docs

Thank you for contributing to Homarr! So that your Pull Request can be handled effectively, please populate the following fields (delete sections that are not applicable)
Category

One of: Bugfix / Feature / Code style update / Refactoring Only / Build related changes /  Documentation / Other (Please specify!)

Overview

Briefly outline your new changes...

Issue Number (if applicable)

Related issue: #00

New Vars (if applicable)

If you've added any new build scripts, environmental variables, config file options, dependency please outline here.

Screenshot (if applicable)

If you've introduced any significant UI changes, please include a screenshot.

This is not ready yet and should be reviewed


incompatible dep with php 7.3/dev

upgrading to latest php 7.3 dev (beta3)
	php -v
		PHP 7.3.0-dev (cli) (built: Aug  7 2018 07:23:47) ( NTS )
		Copyright (c) 1997-2018 The PHP Group
		Zend Engine v3.3.0-dev, Copyright (c) 1998-2018 Zend Technologies
		    with Zend OPcache v7.3.0-dev, Copyright (c) 1999-2018, by Zend Technologies

now,

	./vendor/bin/php-cs-fixer -V
		PHP needs to be a minimum version of PHP 5.6.0 and maximum version of PHP 7.2.*.

	composer show  | grep php/php-cs-fixer
		friendsofphp/php-cs-fixer                v2.12.2            A tool to automatically fi...

checking,
	composer update
		Loading composer repositories with package information
		Updating dependencies (including require-dev)
		Your requirements could not be resolved to an installable set of packages.

		  Problem 1
		    - friendsofphp/php-cs-fixer v2.12.2 requires php ^5.6 || >=7.0 <7.3 -> your PHP version (7.3.0-dev) does not satisfy that requirement.
		    - friendsofphp/php-cs-fixer v2.12.1 requires php ^5.6 || >=7.0 <7.3 -> your PHP version (7.3.0-dev) does not satisfy that requirement.
		    - friendsofphp/php-cs-fixer v2.12.0 requires php ^5.6 || >=7.0 <7.3 -> your PHP version (7.3.0-dev) does not satisfy that requirement.
		    - friendsofphp/php-cs-fixer v2.11.2 requires php ^5.6 || >=7.0 <7.3 -> your PHP version (7.3.0-dev) does not satisfy that requirement.
		    - friendsofphp/php-cs-fixer v2.11.1 requires php ^5.6 || >=7.0 <7.3 -> your PHP version (7.3.0-dev) does not satisfy that requirement.
		    - friendsofphp/php-cs-fixer v2.11.0 requires php ^5.6 || >=7.0 <7.3 -> your PHP version (7.3.0-dev) does not satisfy that requirement.
		    - friendsofphp/php-cs-fixer v2.10.5 requires php ^5.6 || >=7.0 <7.3 -> your PHP version (7.3.0-dev) does not satisfy that requirement.
		    - friendsofphp/php-cs-fixer v2.10.4 requires php ^5.6 || >=7.0 <7.3 -> your PHP version (7.3.0-dev) does not satisfy that requirement.
		    - friendsofphp/php-cs-fixer v2.10.3 requires php ^5.6 || >=7.0 <7.3 -> your PHP version (7.3.0-dev) does not satisfy that requirement.
		    - friendsofphp/php-cs-fixer v2.10.2 requires php ^5.6 || >=7.0 <7.3 -> your PHP version (7.3.0-dev) does not satisfy that requirement.
		    - friendsofphp/php-cs-fixer v2.10.1 requires php ^5.6 || >=7.0 <7.3 -> your PHP version (7.3.0-dev) does not satisfy that requirement.
		    - friendsofphp/php-cs-fixer v2.10.0 requires php ^5.6 || >=7.0 <7.3 -> your PHP version (7.3.0-dev) does not satisfy that requirement.
		    - friendsofphp/php-cs-fixer v2.12.2 requires php ^5.6 || >=7.0 <7.3 -> your PHP version (7.3.0-dev) does not satisfy that requirement.
		    - Installation request for friendsofphp/php-cs-fixer ^2.10 -> satisfiable by friendsofphp/php-cs-fixer[v2.10.0, v2.10.1, v2.10.2, v2.10.3, v2.10.4, v2.10.5, v2.11.0, v2.11.1, v2.11.2, v2.12.0, v2.12.1, v2.12.2].

currently, all of the available branches,
2.12
3.0
master

have
	"require": {
	  "php": "^5.6 || >=7.0 <7.3",

Indeed, PHP CS Fixer is not yet compatible with PHP 7.3,
as each minor release of PHP introduces changes in syntax, for regular users it's a new feature, yet for us, who operates on low-level of tokenization, this is BC breaker, and without adjusting the project it will crash on trying to fix 7.3 code.
If you want to help, please visit https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/3697
I'm also closing this one as duplicate of it.

You can still enforce composer to install PHP CS Fixer on PHP 7.3 anyway with --ignore-platform-reqs.
Then, you can set up PHP_CS_FIXER_IGNORE_ENV env var to enforce PHP CS Fixer to run under PHP 7.3, but you may experience unexpected issues.

Then, you can set up PHP_CS_FIXER_IGNORE_ENV env var
How?

please see https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/v3.3.2/doc/usage.rst#environment-options @mnogokotin


Break up Cookie into Request and Response

This PR adds a RequestCookie and ResponseCookie per the discussion in https://gitter.im/http4s/http4s?at=5a90d82b0202dc012e79241a.
I opened this PR due to the following motivation:

The only reason I ask is that, at work, I, having not used Cookie's in a real app, wrongly wrote code that checked on an HTTP Request's Cookie's cookie.isSecure && cookie.isHttpOnly. If an HTTP Request's Cookie only contained name=value pairs, then that would eliminate that type of error

per https://gitter.im/http4s/http4s?at=5a90d82b0202dc012e79241a.
It uses the abstract class approach that @jmcardon mentioned in https://gitter.im/http4s/http4s?at=5a90d8cd0202dc012e79266d.

Please excuse the git push -f. I almost always avoid re-basing on a PR with comments.
Anyway, I was able to run sbt "; project tests; test successfully on this branch.

Putting back the WIP since I see sbt ci had a few failures. Sorry for the premature removal of WIP!

Could you please take a look, @rossabaker or @jmcardon?
Jose - I know that I didn't use the trait Cookie approach. Please let me you know if you disagree with the PR. Thanks!

Thank you very much for your contribution in the project

Thank you and the maintainers for this excellent library!

Please let me know if you're waiting on me to do anything more before merging! Since this is a breaking change, would you expect it to be part of v19 or 20?
Thanks!

@kevinmeredith nope! All good, thank you!
Just waiting on someone to hit the green button is all. You got three :+1: from maintainers, so it's all good.


SAML response signature verification

Hi!
I'd like to use samael to create a SAML2 service provider. In this role, it's vital to verify the digital signature(s) of the response provided by the identity provider, as they'd be pretty easy to fake otherwise.
I'm afraid that the current code flow as implemented (first deserializing the XML document into Rust structs, then attempting validation on the deserialized form) is not really feasible, as the XML canonicalization process (which determines how exactly the document is turned into the bytes that are then digitally signed) is guided by many options that get lost during Rust deserialization (and rightfully so, as they lose their relevance once the signature has been verified).
Moreover, thanks to the flexibility of the SAML2 standard, a signature can be attached to the whole response or just a single assertion. This has led to a few security vulnerabilities where an unsigned assertion overrides or expands upon a signed one. On Breaking SAML: Be Whoever You Want to Be has a pretty good overview of this type of vulnerability, an overview of existing SAML2 implementations and whether they were previously susceptible to it, and a few proposals for countermeasures. One of these countermeasures is See-what-is-signed, which ensures that only those elements that have been digitally signed are processed further (by marking the signed element, its subtree, and its chain of ancestors with an attribute).
I have implemented a proof-of-concept of a similar process, which would be placed before the Rust XML deserialization in ServiceProvider::parse_xml_response.

Parse the SAML response using libxml.

Collect all ID attributes in the SAML response and tell libxml about them. (This is necessary for xmlsec to resolve the references during signature checking.)

Find all <dsig:Signature> nodes in the SAML response and verifies them using xmlsec. If the verification of any signature fails, return an error.

Remove all occurrences of a custom signature verified attribute from the document to evade attempts at tricking us by integrating such attributes into the original response. (We can only do this after the signatures have been verified, as it might otherwise invalidate one or more of them.)

Place the custom signature verified attribute on all elements that are signed, their descendants and their chain of ancestors.

Remove all elements that do not have this signature verified attribute.

Remove the signature verified attribute from all remaining elements, as it is no longer needed.

Serialize the modified response back to XML.

Continue processing as currently implemented (Rust XML deserialization, validate_destination, etc.) on the re-serialized response.

Since this is a rather involved process, I thought I'd better discuss its design (and whether it is a good match for samael) before opening a pull request. (For example, as an alternative to the attribute, I could collect all xmlNodePtr values of the document, then remove those in relation to a signed element (itself, descendants and ancestors) from the collection, then delete the ones remaining in the collection.) Comments are welcome!

So you're planning on keeping the existing rust definition as it is currently, and adding this as an additional layer prior to the rust deserialization?
Ex:
fn parse_xml_response<AsStr: AsRef<str> + Debug>(
        &self,
        response_xml: &str,
        possible_request_ids: &[AsStr],
    ) -> Result<Assertion, Error> {
        // Steps 1-8
        // Step 9 (but operating on some arbitrary result returned from step 8)
        let response: Response = response_xml
            .parse()
            .map_err(|_e| Error::FailedToParseSamlResponse)?;
        self.validate_destination(&response)?;

Yup:
fn parse_xml_response<AsStr: AsRef<str> + Debug>(
    &self,
    response_xml: &str,
    possible_request_ids: &[AsStr],
) -> Result<Assertion, Error> {
    let reduced_xml = if let Some(sign_certs) = self.idp_signing_certs()? {
        reduce_xml_to_signed(
            response_xml,
            &sign_certs,
        ).map_err(|_e| Error::FailedToValidateSignature)?
    } else {
        String::from(response_xml)
    };
    let response: Response = reduced_xml // <- this changed
        .parse()
        .map_err(|_e| Error::FailedToParseSamlResponse)?;
    // ...

Totally fine with this proposal. Though we might want to keep existing functionality under a name like parse_xml_response_without_validating_signatures. Something long and scary 😆

The current implementation has the old behavior gated behind the xmlsec feature, but if you're fine with the general design, I'll open a pull request and we can continue the discussion in a more concrete context. :-)


multiprocessing.pool Error

Any idea why I would be getting this issue: Running python 3.8.5 and have spotdl 3.1
(myenv) C:\Users\bigja>spotdl https://open.spotify.com/playlist/4yEeu0tZt5DRiExmNaSNSo?si=FrfF3EL0Q1W66Ouf5k3ITA
Fetching Playlist...
0%|                                                                                |ETA: ?, ~min/songTraceback (most recent call last):
File "c:\users\bigja\anaconda3\envs\myenv\lib\runpy.py", line 194, in _run_module_as_main
return run_code(code, main_globals, None,
File "c:\users\bigja\anaconda3\envs\myenv\lib\runpy.py", line 87, in run_code
exec(code, run_globals)
File "C:\Users\bigja\anaconda3\envs\myenv\Scripts\spotdl.exe_main.py", line 7, in
File "c:\users\bigja\anaconda3\envs\myenv\lib\site-packages\spotdl_main.py", line 116, in console_entry_point
downloader.download_multiple_songs(songObjList)
File "c:\users\bigja\anaconda3\envs\myenv\lib\site-packages\spotdl\download\downloader.py", line 293, in download_multiple_songs
self.workerPool.starmap(
File "c:\users\bigja\anaconda3\envs\myenv\lib\multiprocessing\pool.py", line 372, in starmap
return self._map_async(func, iterable, starmapstar, chunksize).get()
File "c:\users\bigja\anaconda3\envs\myenv\lib\multiprocessing\pool.py", line 771, in get
raise self._value
multiprocessing.pool.MaybeEncodingError: Error sending result: '<multiprocessing.pool.ExceptionWithTraceback object at 0x0000018AC5DD2D60>'. Reason: 'TypeError("cannot pickle '_io.BufferedReader' object")'
0%|

I think its either myenv or anaconda, most likely its myenv, it interferes with the default implementation of the multiprocessing library.

It's probably multiprocessing. We're Channing that to multiprocess #927