Missing some pathing instructions in Angular/AzureCosmos/MongoDB

In the process of creating the Angular UI step, the screenshots show that the user should be working in the /src/client folder with the Angular commands.   The sample Repo also reflects this.
However, the step-by-step text info doesn't acknowledge this context or instruct the user when they should be running commands within that context.  Following the steps as written, the Angular files will be created in the main /src/ folder.

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

ID: 435b849c-e5df-bcdf-44df-ec053cf1368f
Version Independent ID: 918ebbe6-ec72-bca2-f514-8a1d1c60f906
Content: Create the Angular app UI with Azure Cosmos DB's API for MongoDB (Part3)
Content Source: articles/cosmos-db/mongodb/tutorial-develop-nodejs-part-3.md
Service: cosmos-db
Sub-service: mongodb
GitHub Login: @gahl-levy
Microsoft Alias: gahllevy

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

@JeremyVyska, thanks for your feedback.
This tutorial is quite dated and we are going to replace the entire tutorial soon. If you look at the sample code, the last commit was in 2019.
We are going to replace the tutorial wholesale in the coming months to be a little more modern like the tutorial we have for the new Azure Cosmos DB for MongoDB vCore offering: https://learn.microsoft.com/en-us/azure/cosmos-db/mongodb/vcore/tutorial-nodejs-web-app
So, for now I will close this work item as it will be a couple of months until we have the new tutorial ready. I do recommend that you try adapting our vCore tutorial to your Azure Cosmos DB for MongoDB instance.
#please-close

Sounds fair.   I ended up on this one from a Bing chat search, so I'll be happy to move on to a more correct tutorial.
Thanks @seesharprun


Update to v1.9.1

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Looks like a protobuf issue?
Selected implementation cpp is not available
```


Implement Redirects within Docusaurus for outdated pages

https://docusaurus.io/docs/api/plugins/@docusaurus/plugin-client-redirects

e.g. docs.gnosischain.com/bridges/ove should be an automatic redirect to https://docs.gnosischain.com/bridges/
We should continually update the redirects, everytime we move a page in documentation

Redirects from old "run a node" structure to the new one planned here: https://docs.google.com/spreadsheets/d/1ki1NJ3LscDpZHc1VI-zrM5p5tgNPo4MN4p-4wn7jYxg/edit#gid=1700733935
plugin-client-redirects unfortunately are client redirects, we will implement server HTTP permanent redirects.

Redirects implemented on the server.

From an SEO point of view, a server-side redirect should always be preferred over a client-side redirect. The major downside to client-side redirects is the missing information about the reason for the redirect, making it harder for search engines to make a decision on how the redirect should be treated.


ctrl+shift+u is unusable as a shortcut key binding

VSCode Version: 1.45.1
OS Version: Linux Mint 18.3

Steps to Reproduce:

Start VS code
Open Keyboard shortcuts
Attempt to assign ctrl+shift+u to any existing command
Verify that the keys pressed are recognised as ctrl+shift+unknown
Verify that attempting to save the key press sequence fails and that that the shortcut key binding is unusable

Does this issue occur when all extensions are disabled?: Yes

AFAIK, caused by the OS.
Duplicate of #96536
Duplicate of #81955
Duplicate of #73003

It looks like you need a "can't fix / won't fix" label.


Select accent order

In some languages there are some prefered accents, or some that are the most used. If your most used accent is the last one there is realy no point in using this tool. For that reason and in favor of more productivity, there should be a setting to change what is the order in wich the accents are shown.

Thanks @bloodanwar. Functionality issues like this are addressed in the PowerToys product repo -- try taking a look through the issues related to Accents to see if there's once addressing what you've brought up here: https://github.com/microsoft/PowerToys/issues?q=is%3Aissue+accents If not, please feel welcome to open a new issue there. This particular repo is specific to updates needed to the documentation.


kpt fn render results permission problem while running within bitbucket pipeline

Expected behavior
kpt fn render will succeed
Actual behavior
kpt fn render
Package "deployment":
[RUNNING] "[gcr.io/kpt-fn/set-image:v0.1.1](http://gcr.io/kpt-fn/set-image:v0.1.1)"
[FAIL] "[gcr.io/kpt-fn/set-image:v0.1.1](http://gcr.io/kpt-fn/set-image:v0.1.1)" in 0s
  Stderr:
    "docker: Error response from daemon: authorization denied by plugin pipelines: --security-opt is not allowed."
    "See 'docker run --help'."
  Exit code: 125

Information
Env: bitbucket pipeline
Install method: go install
Seems it does't allow docker to using --mount because of security reason
Ref: https://confluence.atlassian.com/bbkb/docker-error-authorization-denied-by-plugin-pipelines-command-not-supported-1142238807.html
Any workaround?

Steps to reproduce the behavior

Interesting thing is kpt is invoking docker with --security-opt=no-new-privileges stating run the docker runtime without granting any additional new privileges. Right thing for bitbucket would have been allowing certain values for --security-opt arg.
Unfortunately, this argument is added by default so no workaround at the moment.

I came across a very nasty temporary solution(very un-recommended, also contradict my intention to try this tool).
Since I just want to update my image version:
- |
  cat << EOF > tmp
  apiVersion: config.kubernetes.io/v1
  kind: ResourceList
  items:
  -
  $(sed 's/^/  /g' *deployment.yaml)
  functionConfig:
    apiVersion: v1
    kind: ConfigMap
    metadata:
      name: labels
    data:
      name: xxxx
      newName: xxxx
      newTag: xxxx
  EOF
- docker run --rm -i gcr.io/kpt-fn/set-image:v0.1.1 < tmp | yq ".items[0]" > $(find . -name "*deployment.yaml")

Actually I construct kpt cli pipeline manually, anyway, hope the community will cover this support.


add custom policy

#16519

Let's align with the other languages on this. Both .NET/C# and Java uses the perCall or perRetry terminology rather than first and last. I have not checked the javascript implementation.

/azp run python - formrecognizer

Fix #16519: Support adding custom policy with per_call_policies, per_call_policies


Improve load times by eliminating dependency on Esprima.

Hey @twada,
Currently AVA is using the babel plugin to do power-assert transforms.
I know you are familiar with the project, but a little additional background: We fork a new process for each test file in an effort to improve performance and prevent tests from contaminating the global environment and having unintended side effects.
Right now we are paying a hefty require time cost, both for the main thread, and each forked thread. See this comment for an overview of the problem. Babel is certainly the biggest problem for us, but escallmatch and power-assert-formatter are currently about 20% or our require time.
There are two areas where I think the power-assert libraries could improve:

Eliminate the Esprima dependency entirely when already running in the Babel environment (esprima accounts for about half your load time). The presence of Babel means babylon is present, so use that. I know the AST's are a little different between those two, so this is no small ask, but I think it will make a huge difference for AVA.
Drop the requirement for parsers entirely during runtime. We intend to eventually only load Babel once in the main process, and have it handle transforming sources for the child processes. Currently power-assert still has a runtime dependency on esprima. We have already discussed embedding the AST, but then that requires something like escodegen, which I would like to see eliminated as well.
Finally, I don't think escallmatch requires a parser at all. See call-signature, it is parsing much the same input with just a regular expression. I just whipped it up, and it still has a few holes, but I still think it could end up being a much smaller, more performant solution.

All that said. Thanks again for the awesome library! It's a big part of the reason why I am so excited about AVA!
// cc: @sindresorhus @vdemedes

Oh, and I hope it goes without saying, but just let me know what I need to do to help!

:+1: This would be very helpful. And I agree with James, power-assert is a big part of what makes AVA awesome.

@jamestalmage @sindresorhus Thanks you for your suggestion.

We intend to eventually only load Babel once in the main process, and have it handle transforming sources for the child processes.

Understood.

escallmatch and power-assert-formatter are currently about 20% or our require time.

Oh that's too huge...I'll fix them ASAP.

@sindresorhus @jamestalmage Let me confirm AVA's current status and future direction. Would this fix be based on Babel6? Babel6 is incompatible with Babe5 so I want to hear from you.

@twada We're in the progress of moving to Babel 6. So you can assume Babel 6.

@sindresorhus Thanks!

@twada FYI, I have successfully gotten a plugin working on both 5 and 6.
That said, I agree with @sindresorhus - we are moving to Babel 6, so that is the best place to focus your efforts.

@jamestalmage I think your call-signature is handy and lightweight alternative to escallmatch and appropriate for power-assert's runtime side.
I made a pull-req to support Identifier as callee, it's a must feature for power-assert. Would you review it?

working on both 5 and 6 sounds great

The latest versions of 5 pass a couple extra undocumented arguments to the visitor methods.  Those give you nearly the same API you end up with in 6 (just passed in a little differently). I am sure it was all in preparation for the big breaking change. The solution I linked to may not work for earlier 5.x releases, but I think it's a decent solution until everybody has moved over to 6. At least that way, people stuck on 5 due to some plugin that is not actively maintained have a chance to see new features.


Package download statistics are not updated

For last three days package download counts on nuget.org are not increased.
Main page is always showing 280,266,060 package downloads and it stayed constant since 2014-11-08. Some packages i use (e.g. Serenity.Web 1.4.0) shows 0 download count.

We are looking to improve the mechanism for stats - adding this for future work

This issue should be fixed now.

It seems count is frozen for my nuget package (https://www.nuget.org/packages/Xunit.Gherkin.Quick/) for the last week.
I'm fine with thinking that there are no new downloads, but I believe there must be.
Has anybody else complained about the same recently? Or is it just me?

I can see similar issue with my package https://www.nuget.org/packages/Guflow/1.0.68.  I did a test and downloaded the package version 1.0.68  from 4-5 different machines today but it did not have any impact on downloaded counts even after 4-5 hours. It is frozen.

This is still happening and has happened several times repeatedly and intermittently.

I am also facing the same issue
(https://www.nuget.org/packages/SimpleAccess.SqlServer)

Same thing for my package: https://www.nuget.org/stats/packages/Z.Blazor.Diagrams
The statistics page says that no stats are currently available, even the NuGet search / Visual Studio package manage shows 0, but when you go into the package, it says it has downloads (77?).


chore: change default aws authenticator command to aws eks get-token instead of aws-iam-authenticator

Description
This PR changes default aws authenticator command to aws eks get-token instead of aws-iam-authenticator since it's already deprecated. Fixes #957.
The next step is to drop aws-iam-authenticator support from the terraform-aws-eks module.
Motivation and Context
Fixes #957.
Breaking Changes
No breaking changes here, just kubconfig file updates.
How Has This Been Tested?
EKS cluster was deployed from the master branch and updated with the suggested changes successfully.
Pre-commit hook was run.

LGTM

this all looks good and makes sense to me, but unfortunately I think its still a breaking change because we have changed the default behavior. are we able to keep the current behavior (aws-iam-authenticator) as the default, but provide users the ability to use the aws eks get-token if they so choose? I think that would make this change very easy to merge in without disruption

I guess it will be a breaking change for those who had customized their iam authenticator args.

At the moment, it's already possible to use aws eks get-token by defining kubeconfig_aws_authenticator_command and kubeconfig_aws_authenticator_command_args input variables.
So it doesn't require any new changes to release that. Something like that:
  kubeconfig_aws_authenticator_command         = var.kubeconfig_aws_authenticator_command
  kubeconfig_aws_authenticator_command_args    = var.kubeconfig_aws_authenticator_command == "aws" ? ["eks", "get-token", "--cluster-name", element(concat(data.aws_eks_cluster_auth.cluster[*].name, [""]), 0)] : []
  kubeconfig_aws_authenticator_additional_args = ["--role-arn", "arn:aws:iam::012345678910:role/EKSDeployerRole"]

But I believe that everyone who keep terraform-aws-eks module up to date and use v.17 have already updated awc cli to the version 1.16.156 or later since it was introduced 3 years ago.
For that users there are two possible cases:

they will see updates in the kubeconfig file if they've used default aws-iam-authenticator, but still can apply the changes and authenticate to the cluster with no additional steps.
If they intentionally want to use use aws-iam-authenticator there is still possible to re-define defaults.

they've already defined specified kubeconfig_aws_authenticator variables to use eks get-token and it will take precedence over our changes. As a result, terraform wouldn't suggest any changes at all.
I believe that is our target audience for the moment.

In case, the user use aws cli version before 1.16.156 it will really be a breaking change. But I'm sure there are no many people who use it. And we shouldn't focus on this since it's too outdated.
So I agree that it can lead to some changes during terraform module minor version upgrade. But I think it's quite enough to leave some details in the CHANGELOG.md to notify everybody about possible changes.
The official AWS documentation proposes to use native aws eks get-token auth instead of aws-iam-authenticator tool, and our default values for aws-iam-authenticator is really confusing.
https://docs.aws.amazon.com/eks/latest/userguide/managing-auth.html
https://docs.aws.amazon.com/eks/latest/userguide/install-aws-iam-authenticator.html
From my side, I'd prefer to introduce this before v18 since there would be too many breaking changes and not everybody are ready to install them.

This issue has been resolved in version 18.0.0 :tada:


When i restart the phone permission to access file is gone

At first when i open my app i granted app permission to access files in the folder but after restarting phone permission is denied again

@Ombenitango , Help me understand your issue is bug or not,
Is there a provision for this functionality by Android?


Use new api of doctrine/inflector 1.4/2.0

The old api is deprecated. The new api is already available in inflector 1.4.

@W0rma Nice work. Thank you!


How to use indexer for creating index that also creates sections of an individual pdf page using azure blob storage

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ X ] feature request
- [ X ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Use indexer to create/update index instead of pushing content via rest apis. Need an document reference to create chunks using indexer.

OS and Version?

Windows 11

azd version?

azd version 1.1.0.

Thanks! We'll be in touch soon.

Did anyone has any idea if an azure indexer can be used for pushing data from blob storage to cognitive search index?
I tried creating indexer and mapped it with azure blob storage and I was able to transfer data(content) to index using indexer but I don't know how I can create multiple sections of a blob (a single page of pdf file) and attach this chunking process with indexer, so indexer can do all these heavy lifting over here [whenever there is a change in blob it automatically creates chunks of that blob and update that in index].
I can create multiple sections (having ~1000 character each) of a page and store those sections into index via rest api but I want to do it using indexer, so please let me know if you have any idea on this.

@sandeeppatidar30 at the moment, indexers don't support chunking. An indexer can only go from one input document, to one entry in your Cognitive Search index. To overcome this current limitation, you could use one indexer that chunks your data and stores this in a blob storage via the knowledge store capability. Another indexer will index this new blob storage and index your chunks.
See https://github.com/Azure/cognitive-search-vector-pr/blob/main/demo-python/code/azure-search-vector-ingestion-python-sample.ipynb.

@iMicknl  - Conceptually, does MS consider Doc Chunking and the experimentation of the sizes therein a feature that needs to be added to Cog Search? Or, is the concept as a whole a current failure of the LLM/context window limit and once mega context limits are available this is an issue that won't need solving anymore..? Speculation on the feature? @pamelafox . And thanks for your contributions, BTW.

Hi @sandeeppatidar30
I know its a late response. I believe what you're looking for can be achieved through integrated vectorization. A (relatively) recent PR has incorporated this feature into the demo. See here.
My understanding is that it achieves extraction, chunking, embedding and indexing of your data in blob storage - so i.e., from storage to index seamlessly.

But still GPT Vision is then not supported afaik


[Feature Request] Project to project replication

A use case would be to have a "dirty" project where both signed and unsigned images are pushed. If an image gets signed in the "dirty" project, it can be promoted (replicated) to a "clean" project where only signed images live.

We are providing a mechanism to make sure only signed images can be pulled under certain projects, does it cover the requirement?

That sounds like a good addition, however the idea of "promoting" an image is a much desired feature.
I'm not sure of the best way to go about it, so I suggested project to project replication. Replicating signed images is one way to go about it. The promotion/replication could instead be based on some kind of notary metadata flag.
Or maybe there could be some sort of promotion mechanism that when triggered, would move/copy the image to another project. The basic idea would be to have a "Prod" project that only contains images that have been promoted from lower projects such as "Dev" and "QA".
I'd be happy to hear other people's thoughts and ideas on a good implementation.

Replicating signed image will break as the "GUN" changes, the image has to be re-signed.

I am not sure how replication works under the hood, but in the theoretical promotion mechanism I described above, I imagine it would work by re-tagging the image from the first project and dumping it in the second project. The image id would be identical; does that mean the GUN would as well?
Otherwise, can the image be re-signed by the user that is configured in the promotion/replication settings?

It is a limitation in notary, as the root keys and target keys are stored offline.


Pings fail in mininet with test-tunneling.p4.

When running the mininet script test_topo.py with the  test_tunneling.p4 loaded, I am getting a Destination Host Unreachable error when attempting to ping h2 from h1.
This error occurs even after insertion of flow rules using the commands sudo ovs-ofctl add-flow s1 in_port=1,actions=prog:1,output:2 and sudo ovs-ofctl add-flow s1 in_port=2,actions=prog:1,output:1 as instructed in the README.
I am getting the same error when using other programs also (such as vlan.c) and my own P4 program that trivially extracts the ethernet header, passes the packet and deparses the ethernet header again. All of these programs show the same behaviour when compiled with either clang-6.0 or clang-10.
I have tested the pings without loading any bpf program and they work just fine. The only way that I can seem to get the pings to work is by editing the C code of each program to trivially return 1 as seen with  vlan.c below where I have commented out most of the entry function:
#include <stdint.h>
#include <inttypes.h>
#include <stdbool.h>
#include <stddef.h>
#include <net/ethernet.h>
#include <netinet/ip.h>

#define ___constant_swab16(x) ((uint16_t)(             \
    (((uint16_t)(x) & (uint16_t)0x00ffU) << 8) |          \
    (((uint16_t)(x) & (uint16_t)0xff00U) >> 8)))

#define ___constant_swab32(x) ((uint32_t)(             \
    (((uint32_t)(x) & (uint32_t)0x000000ffUL) << 24) |        \
    (((uint32_t)(x) & (uint32_t)0x0000ff00UL) <<  8) |        \
    (((uint32_t)(x) & (uint32_t)0x00ff0000UL) >>  8) |        \
    (((uint32_t)(x) & (uint32_t)0xff000000UL) >> 24)))

#define bpf_htons(d) (___constant_swab16((d)))
#define bpf_htonl(d) (___constant_swab32((d)))

#define VLAN_HDR_LEN 4
#define ETH_ADDR_LEN 6

static void (*ubpf_printf)(const char *fmt, ...) = (void *)7;
static void *(*ubpf_packet_data)(const void *) = (void *)9;
static void *(*ubpf_adjust_head)(const void *, uint64_t) = (void *)8;

void* memmove(void* dest, const void* src, size_t len);

struct vlan_eth {
    u_int8_t  ether_dhost[6];   /* destination eth addr */
    u_int8_t  ether_shost[6];   /* source ether addr    */
    uint16_t h_vlan_ethtype;
    uint16_t h_vlan_TCI;
    uint16_t ethtype;
};

/*
 * The example shows VLAN tagging. The BPF program replaces Ethernet header with VLAN-tagged L2 header.
 */
uint64_t entry(void *ctx, uint64_t pkt_len)
{
    bool pass = true;

//    void *pkt = ubpf_packet_data(ctx);
//    if (pkt_len < sizeof(struct ether_header) + sizeof(struct iphdr)) {
//        return 1;
//    }

//    pkt = ubpf_adjust_head(ctx, VLAN_HDR_LEN);
//    struct vlan_eth *veh = (void *) pkt;
//    memmove(veh, (char *)veh + VLAN_HDR_LEN, 2 * ETH_ADDR_LEN);

//    veh->h_vlan_TCI = bpf_htons(20); // set VLAN TCI=20
//    veh->h_vlan_ethtype = bpf_htons(0x8100); // set EtherType to VLAN

    return pass;
}

I realise that this is unlikely to be a bug, but I would appreciate if anyone had some idea as to what I have done wrong, as I believe I have followed all of the installation steps correctly. I you require any more details, I am happy to provide them.

Hi!
There are two problems.
Firstly, you will not be able to make a ping between Mininet hosts using test_tunneling.p4 program, because this program encapsulates packets into MPLS, which is not stripped out by host and packet is not correctly identified. Also, you need to populate table entries by using ovs-ofctl update-bpf-map, did you do that?
Secondly, in case of the vlan.c example there is probably a bug in the compiler related to how the check of packet's length is done. We'll fix it, but to verify that could you modify the C program to have the following piece of code?
if (sizeof(struct ether_header) + sizeof(struct iphdr) < pkt_len) {  // instead of if (pkt_len < sizeof(struct ether_header) + sizeof(struct iphdr)) {
     return 1;
}

With the above modification you need to re-compile program using clang and try to ping again.

Hi,
Thanks for getting back to me so quickly.
I made the change to vlan.c and now the pings are being forwarded properly.
I believe that the issue with my own P4 programs is related to the problem with vlan.c, since any time I return 1 before the following code in the deparser block, the pings are forwarded as normal:
        int outHeaderOffset = BYTES(outHeaderLength) - BYTES(packetOffsetInBits);
        pkt = ubpf_adjust_head(ctx, outHeaderOffset);

With regard to test-tunneling.p4, I had been running tcpdump to check for MPLS packets, but none had been received. I had not populated the table entries as I thought the program forwarded packets by default, but now that I've gone back to try to do that, I am getting a diffferent error. The BPF program seems to be unloading after I try to add the flow rules, giving me the following error: OFPT_ERROR (xid=0x6): OFPBAC_BAD_ARGUMENT once I try to add the second flow rule.

I'll reproduce the issue with the verification of a packet's length locally and provide fix to the p4c-ubpf compiler, if needed.
Regarding OFPT_ERROR (xid=0x6): OFPBAC_BAD_ARGUMENT are you able to provide logs from OVS? By default logs are stored in /var/log/openvswitch/ovs-vswitchd.log. Note also that the uBPF verifier is in the experimental stage and if you don't use --disable-bpf-verifier flag with the ./configure command, you can encounter some unexpected bugs.

The first set of logs below is when both flow rules are added successfully but the pings fail, while the second set of logs below are when the second flow rule is when OFPT_ERROR (xid=0x6): OFPBAC_BAD_ARGUMENT is returned. Both of these logs were produced when running test-tunneling.p4 and with the BPF verifier disabled.
2020-11-18T11:09:43Z|00001|ovs_numa|INFO|Discovered 1 CPU cores on NUMA node 0
2020-11-18T11:09:43Z|00002|ovs_numa|INFO|Discovered 1 NUMA nodes and 1 CPU cores
2020-11-18T11:09:43Z|00003|reconnect|INFO|unix:/usr/local/var/run/openvswitch/db.sock: connecting...
2020-11-18T11:09:43Z|00004|reconnect|INFO|unix:/usr/local/var/run/openvswitch/db.sock: connected
2020-11-18T11:09:43Z|00005|bridge|INFO|ovs-vswitchd (Open vSwitch) 2.13.90
2020-11-18T11:09:45Z|00006|pmd_perf|INFO|Estimated TSC frequency: 3192760 KHz
2020-11-18T11:09:45Z|00007|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports recirculation
2020-11-18T11:09:45Z|00008|ofproto_dpif|INFO|netdev@ovs-netdev: VLAN header stack length probed as 1
2020-11-18T11:09:45Z|00009|ofproto_dpif|INFO|netdev@ovs-netdev: MPLS label stack length probed as 3
2020-11-18T11:09:45Z|00010|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports truncate action
2020-11-18T11:09:45Z|00011|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports unique flow ids
2020-11-18T11:09:45Z|00012|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports clone action
2020-11-18T11:09:45Z|00013|ofproto_dpif|INFO|netdev@ovs-netdev: Max sample nesting level probed as 10
2020-11-18T11:09:45Z|00014|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports eventmask in conntrack action
2020-11-18T11:09:45Z|00015|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports ct_clear action
2020-11-18T11:09:45Z|00016|ofproto_dpif|INFO|netdev@ovs-netdev: Max dp_hash algorithm probed to be 1
2020-11-18T11:09:45Z|00017|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports check_pkt_len action
2020-11-18T11:09:45Z|00018|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports timeout policy in conntrack action
2020-11-18T11:09:45Z|00019|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports ct_state
2020-11-18T11:09:45Z|00020|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports ct_zone
2020-11-18T11:09:45Z|00021|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports ct_mark
2020-11-18T11:09:45Z|00022|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports ct_label
2020-11-18T11:09:45Z|00023|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports ct_state_nat
2020-11-18T11:09:45Z|00024|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports ct_orig_tuple
2020-11-18T11:09:45Z|00025|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports ct_orig_tuple6
2020-11-18T11:09:45Z|00026|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports IPv6 ND Extensions
2020-11-18T11:09:45Z|00027|bridge|INFO|bridge s1: added interface s1-eth2 on port 2
2020-11-18T11:09:45Z|00028|bridge|INFO|bridge s1: added interface s1-eth1 on port 1
2020-11-18T11:09:45Z|00029|bridge|INFO|bridge s1: added interface s1 on port 65534
2020-11-18T11:09:45Z|00030|bridge|INFO|bridge s1: using datapath ID<PHONE_NUMBER>000001
2020-11-18T11:09:45Z|00031|connmgr|INFO|s1: added service controller "punix:/usr/local/var/run/openvswitch/s1.mgmt"
2020-11-18T11:09:53Z|00032|memory|INFO|17280 kB peak resident set size after 10.0 seconds
2020-11-18T11:09:53Z|00033|memory|INFO|handlers:1 ports:3 revalidators:1 rules:4 udpif keys:2
2020-11-18T11:10:04Z|00034|connmgr|INFO|s1<->unix#3: 1 flow_mods in the last 0 s (1 adds)
2020-11-18T11:10:07Z|00035|connmgr|INFO|s1<->unix#6: 1 flow_mods in the last 0 s (1 adds)
Segmentation fault

2020-11-18T11:26:56Z|00045|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports recirculation
2020-11-18T11:26:56Z|00046|ofproto_dpif|INFO|netdev@ovs-netdev: VLAN header stack length probed as 1
2020-11-18T11:26:56Z|00047|ofproto_dpif|INFO|netdev@ovs-netdev: MPLS label stack length probed as 3
2020-11-18T11:26:56Z|00048|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports truncate action
2020-11-18T11:26:56Z|00049|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports unique flow ids
2020-11-18T11:26:56Z|00050|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports clone action
2020-11-18T11:26:56Z|00051|ofproto_dpif|INFO|netdev@ovs-netdev: Max sample nesting level probed as 10
2020-11-18T11:26:56Z|00052|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports eventmask in conntrack action
2020-11-18T11:26:56Z|00053|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports ct_clear action
2020-11-18T11:26:56Z|00054|ofproto_dpif|INFO|netdev@ovs-netdev: Max dp_hash algorithm probed to be 1
2020-11-18T11:26:56Z|00055|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports check_pkt_len action
2020-11-18T11:26:56Z|00056|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports timeout policy in conntrack action
2020-11-18T11:26:56Z|00057|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports ct_state
2020-11-18T11:26:56Z|00058|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports ct_zone
2020-11-18T11:26:56Z|00059|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports ct_mark
2020-11-18T11:26:56Z|00060|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports ct_label
2020-11-18T11:26:56Z|00061|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports ct_state_nat
2020-11-18T11:26:56Z|00062|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports ct_orig_tuple
2020-11-18T11:26:56Z|00063|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports ct_orig_tuple6
2020-11-18T11:26:56Z|00064|ofproto_dpif|INFO|netdev@ovs-netdev: Datapath supports IPv6 ND Extensions
2020-11-18T11:26:56Z|00065|bridge|INFO|bridge s1: added interface s1-eth2 on port 2
2020-11-18T11:26:56Z|00066|bridge|INFO|bridge s1: added interface s1-eth1 on port 1
2020-11-18T11:26:56Z|00067|bridge|INFO|bridge s1: added interface s1 on port 65534
2020-11-18T11:26:56Z|00068|bridge|INFO|bridge s1: using datapath ID<PHONE_NUMBER>000001
2020-11-18T11:26:56Z|00069|connmgr|INFO|s1: added service controller "punix:/usr/local/var/run/openvswitch/s1.mgmt"
2020-11-18T11:26:58Z|00070|connmgr|INFO|s1<->unix#13: 1 flow_mods in the last 0 s (1 adds)
Segmentation fault

I don't have any experience debugging OvS, but when I attached GDB to the ovs-vswitchd process, this was all I got.
(gdb) c
Continuing.
[New Thread 0x7f5dca967a00 (LWP 5492)]
[New Thread 0x7f5dca4e0700 (LWP 5509)]
[New Thread 0x7f5dc9cdf700 (LWP 5510)]
[New Thread 0x7f5dc94de700 (LWP 5511)]
[New Thread 0x7f5dbbfff700 (LWP 5515)]
[New Thread 0x7f5dbb7fe700 (LWP 5516)]

Thread 1 "ovs-vswitchd" received signal SIGSEGV, Segmentation fault.
0x00007f5dca99502b in ?? ()
(gdb) bt
#0  0x00007f5dca99502b in ?? ()
#1  0x000055d6dd0dd058 in ?? ()
#2  0x00007ffc04f65580 in ?? ()
#3  0x0000000000000000 in ?? ()

Hi,
Unfortunately, the logs doesn't say too much. Could you re-compile OVS with: ./configure CFLAGS='-g -O2' --disable-bpf-verifier  and debug the issue with gdb again?

Hi,
Recompling with the debug symbols does not change the output of gdb. The address referred to at #0 is not mapped, while the addresses at #1 and #2 are mapped to the heap and stack respectively. Sorry I couldn't be of more help with this.

No problem. I'll try to find time to re-produce the issue locally. I'll keep you updated.

Okay, thanks!

@blaxie Could you provide update-bpf-map commands, which you used to populate table entries?

I hadn't provided any table entries as I wasn't interested in encapsulating or decapsulating any packets when testing the pings. I have since tried to add a table entry using the command sudo ovs-ofctl update-bpf-map s1 1 0 key 2 0 0 10 value 0 0 0 0. This does not seem to produce any errors.

If you send raw IP packets you should populate tbl_downstream, what means that you should use the following command:
ovs-ofctl update-bpf-map s1 1 1 key 2 0 0 10 value 0 0 0 0
With the above command you should be able to receive packets encapsulated into the MPLS header.

When I use the command ovs-ofctl update-bpf-map s1 1 1 key 2 0 0 10 value 0 0 0 0, I get another error:
OFPT_ERROR (xid=0x4): OFPBRC_EPERM
NXT_BPF_UPDATE_MAP (xid=0x4):

As well as that, the ovs-ofctl show-bpf-prog s1 1 command shows that there actually 4 maps loaded and I have no idea why.
NXT_BPF_SHOW_PROG_REPLY (xid=0x4):
id 1:   loaded_at=2020-11-24 T11:20:48
	map_ids 0,1,2,3

The P4 code is exactly the same as the code you have been discussing, with the tbl_upstream and tbl_downstream tables.

Ok, I think I know why you have this problem. The p4rt-ovs has not been adjusted to the newest version of p4c yet. The latest PR to the uBPF backend  (https://github.com/p4lang/p4c/pull/2572) introduced support for default actions, so that for every P4 table with default action, there will be 2 BPF maps generated. That's why the number of BPF maps equals to 4.
Now.. Taking into account the latest change in p4c the command you should use to populate tbl_downstream is as follows:
ovs-ofctl update-bpf-map s1 1 3 key 2 0 0 10 value 0 0 0 0
Let me check if it works.

The table entry seems to be added without any problems when I use ovs-ofctl update-bpf-map s1 1 2 key 2 0 0 10 value 0 0 0 0, but adding the flow rules after this (e.g. sudo ovs-ofctl add-flow s1 in_port=1,actions=prog:1,output:2) is still causing the same segmentation fault as before.

Could you first add flow and then insert the BPF map's entry ? Unfortunately, P4rt-OVS is not actively developed anymore and it is not free of bugs.

Probably I've found root cause of the segmentation fault.
P4rt-ovs calls uBPF program in this way:
https://github.com/Orange-OpenSource/p4rt-ovs/blob/2511a57e5411aadb824d914281afddbb203fd07c/lib/bpf.h#L39-L45
To the uBPF program packet and its length is passed as parameters.
But recent versions of p4c-ubpf compilers generates following entry point of uBPF program:
uint64_t entry(void *ctx, struct standard_metadata *std_meta){
    void *pkt = ubpf_packet_data(ctx);
    uint32_t pkt_len = std_meta->packet_length;
    struct headers_t hdr = {

So packet and pointer to struct standard_metadata is expected.
In other words, packet's length is treated as a pointer to standard_metadata struct. This must fail, because packet's length isn't a valid pointer. That would explain as well lack of debugging symbols during crash.
Workaround 1
Manually edit auto generated entry point to something like this:
uint64_t entry(void *ctx, uint32_t pkt_len){
    void *pkt = ubpf_packet_data(ctx);
    struct headers_t hdr = {

Workaround 2
Changes connected with standard_metadata were not merged into this repository, so you can use one of these forks:

https://github.com/osinstom/ovs.p4 (@osinstom which branch? I don't remember)
https://github.com/tatry/p4rt-ovs/tree/metadata_ubpf

Probably I've found root cause of the segmentation fault.
P4rt-ovs calls uBPF program in this way:
https://github.com/Orange-OpenSource/p4rt-ovs/blob/2511a57e5411aadb824d914281afddbb203fd07c/lib/bpf.h#L39-L45
To the uBPF program packet and its length is passed as parameters.
But recent versions of p4c-ubpf compilers generates following entry point of uBPF program:
uint64_t entry(void *ctx, struct standard_metadata *std_meta){
    void *pkt = ubpf_packet_data(ctx);
    uint32_t pkt_len = std_meta->packet_length;
    struct headers_t hdr = {

So packet and pointer to struct standard_metadata is expected.
In other words, packet's length is treated as a pointer to standard_metadata struct. This must fail, because packet's length isn't a valid pointer. That would explain as well lack of debugging symbols during crash.
Workaround 1
Manually edit auto generated entry point to something like this:
uint64_t entry(void *ctx, uint32_t pkt_len){
    void *pkt = ubpf_packet_data(ctx);
    struct headers_t hdr = {

Workaround 2
Changes connected with standard_metadata were not merged into this repository, so you can use one of these forks:

https://github.com/osinstom/ovs.p4 (@osinstom which branch? I don't remember)
https://github.com/tatry/p4rt-ovs/tree/metadata_ubpf

Hello,I tried these two methods, but still getting an error when dump-map.what should i do


":" in identifier name

If I understand correctly, according to this http://handlebarsjs.com/expressions.html (search "except for the following" on the page), it should be possible to use : (colon) as part of identifier name. We use colons in our templates, and I faced this issue while migrating our server-side handlebars compiling from V8 to FlavourSaver.
BTW, thanks for the great gem!

This is effectively a duplicate of issue  #32 - resolve in favor of that

Thanks, looks like it works now on the version from the master branch. Could you prepare a release?

@grumbleafrican @jamesotron sorry to bother you guys, but could you make a new release with this and some other fixes? The previous one was more that a year ago. I would like to avoid using gem from master branch in production. Thanks in advance.

Hey. I've added both @flexoid and @grumbleafrican as collaborators for this repo. I'm also happy to add you as gem owners on Rubygems if you let me know your Rubygems email addresses.

@jamesarosen thanks, my profile is https://rubygems.org/profiles/flexoid
I think this issue can be closed.


Cleanup and renaming

Various minor removal and renaming things, each in its own commit.

Codecov Report

Merging #372 (225d3be) into develop-v1.0 (2f465c6) will increase coverage by 1.02%.
The diff coverage is 92.15%.

@@               Coverage Diff                @@
##           develop-v1.0     #372      +/-   ##
================================================
+ Coverage         81.82%   82.85%   +1.02%
================================================
  Files                54       50       -4
  Lines              5382     5149     -233
================================================
- Hits               4404     4266     -138
+ Misses              978      883      -95

Impacted Files
Coverage Δ

coregistration/eolearn/coregistration/utils.py
17.24% <ø> (ø)

mask/eolearn/mask/utils.py
92.10% <ø> (ø)

ml_tools/eolearn/ml_tools/utils.py
45.05% <ø> (ø)

geometry/eolearn/geometry/morphology.py
94.11% <91.30%> (-5.89%)
:arrow_down:

...istration/eolearn/coregistration/coregistration.py
61.58% <100.00%> (ø)

mask/eolearn/mask/cloud_mask.py
96.88% <100.00%> (ø)

mask/eolearn/mask/masking.py
88.23% <100.00%> (+6.87%)
:arrow_up:

mask/eolearn/mask/snow_mask.py
98.01% <100.00%> (ø)

ml_tools/eolearn/ml_tools/classifier.py
66.89% <100.00%> (ø)

Continue to review full report at Codecov.

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


Enhance PropertiesDecoder decoding performance

I'm using Redisson with Spring Boot 2.3.5. When Spring Boot's health actuator of health is enabled and health endpoint is visited, health status of redis will be accessed through org.redisson.spring.data.connection.RedissonReactiveServerCommands#info(), then PropertiesDecoder#decode is used to decode responded string from INFO command.
However if no sections are provided to INFO command, size of responding string can be up to 3.5KB. If a low health check interval, like 2 seconds, is set, decoding the result costs significant CPU time:

To enhance the performance for decode method, splitting result with single char can be used instead of regular expression.
Benchmark from org.redisson.spring.data.connection.PropertiesDecoderTest#benchmark:
Old: 2410ms
New: 960ms

Thanks for contribution!


How to generate two uber jars for one project folder using different filter

Hi,
I Just started to learn scala/sbt and is now trying to build my project in two different jars:
One will only contain the .dll unmanagedResource for windows
The other one will only contain the .so unmanagedResource for Linux
I am now able to set the filter. But I do not know how to generate two jars in one assembly - or some other option without me touching the code of build.sbt for assembly.
Thanks!
Charles

Have figured it out.
Just create two projects pointing to the save folder.
Assign different target and resource to them and then you can apply different filters to create different jars.
.settings(
target := file("...1/scala-2.10/"),
sourceManaged := file("...1/scala-2.10/")
)
.settings(
target := file("...2/scala-2.10/"),
sourceManaged := file("...2/scala-2.10/")
)

@charlessong0 if you create two projects pointing to the same folder, the later one will overwrite the previous one, right?
for example:
lazy val proj1 = (project in file("."))...

lazy val proj2 = (project in file(".")).

now, there is only one project proj2.


iframes

Where is demo and documentation about lazyloading iframes?

there's no specific example, though you can implement lazy loading of iframes in the exact same way as you do for images


使用gradle build -x test编译的时候出错

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':compileJava'.

Compilation failed; see the compiler error output for details.

提供下编译错误的详细信息


No module named selenium.common.exceptions

Has anybody faced similar issues?
Traceback (most recent call last):
File "C:\Users\Roshan Dhamala\Independent Class\Insta\instagram-profilecrawl\crawl_profile.py", line 5, in
from util.datasaver import Datasaver
File "C:\Users\Roshan Dhamala\Independent Class\Insta\instagram-profilecrawl\util\datasaver.py", line 4, in
from util.settings import Settings
File "C:\Users\Roshan Dhamala\Independent Class\Insta\instagram-profilecrawl\util\util.py", line 1, in
from selenium.common.exceptions import NoSuchElementException
ImportError: No module named selenium.common.exceptions

Yes i'm also having the same problem.
is this something because of paths?


ARROW-15297: [C++] The write node options shouldn't require a schema [WIP]

12310

 Quality gate: Passed
 Everything looks good
Details

 Quality gate: Passed
15 Issues

 15 Code issues

Details


sveltekit warning (or error) message

not sure how to reproduce it, but i got the following warning:
Error: Cannot subscribe to 'page' store on the server outside of a Svelte component, as it is bound to the current request via component context. This prevents state from leaking between users.For more information, see https://kit.svelte.dev/docs/state-management#avoid-shared-state-on-the-server
this is the trace: (highlighted the line where superforms appear)
at get_store (/node_modules/@sveltejs/kit/src/runtime/app/stores.js:68:9)
at Object.subscribe (/node_modules/@sveltejs/kit/src/runtime/app/stores.js:26:24)
at subscribe (/node_modules/svelte/internal/index.mjs:57:25)
at Module.get_store_value (/node_modules/svelte/internal/index.mjs:62:5)
at Module.superForm (/node_modules/sveltekit-superforms/dist/client/index.js:59:23)
at +page.svelte:13:60
at Object.$$render (/node_modules/svelte/internal/index.mjs:1892:22)
at Object.default (root.svelte:48:44)
at eval (/node_modules/@sveltejs/kit/src/runtime/components/layout.svelte:8:41)
at Object.$$render (/node_modules/svelte/internal/index.mjs:1892:22)

That's strange, the only subscription to page is done within an if(browser) check. Does it appear in the latest version as well? (0.5.20)
If it does, can you look in the browser debugger for the line where it happens in the superforms file, and paste it here?

never happened again, guess it was some startup error on svelte kit, im going to close the issue


Branch point accuracy

Add tests that verify the accuracy of the functions at the branch point.

The accuracy was correct before this PR, but was not tested.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (cc0ee30) to head (1ee56eb).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #37   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            9         9
  Lines         1865      1872    +7
=========================================
+ Hits          1865      1872    +7

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


possible to validate against transformed image size?

Hey!  Great work on this project!
Is it possible to run the transform plugin first and show THAT filesize and validate against THAT?
Consider someone using an iPhone who takes a picture > 5MB (or > 7MB or 9MB in the future)
It would be great to have the picture already transformed when it gets validated for a max file size of say 3MB.

As transforming the files on the client gives you some control over the output size, I would simply not validate the size. Would that work in your situation?
To give a bit of insight, transforming the file on the client is quite taxing on the CPU so currently the clientside transform is only done right before uploading the file.


couldn't find file 'bootstrap' after bundle install.

I added gem 'bootstrap-sass', '1.4.0' to my Gemfile and did a bundle install. It picked up the gem and installed it.
I added *= require bootstrap to my application.css....
When I hit my page I see:
couldn't find file 'bootstrap'
(in /home/jim/Sites/isr-update/app/assets/stylesheets/application.css:6)
I don't see the files in lib or vendor, should they be there?

Sorry... I had to restart Webrick.


Every field populating: translated: Text

I updated the plugin, but it seems that everytime I try to do a translation it populates all fields with: "translated: Text" that's it. What could be the reason?

If you issue the translate function which makes it fill in translated:Text and then select the language dropdown it shows the other languages (including the source) as empty.

Sorry, had some uncomitted code when publishing the package... :facepalm:
Have pushed a new package 0.3.4

About the second issue that you are reporting, please try with the new version again and open a new issue :)


Potential VPR Parser Bug

Describe the bug
VPR Bug
When implementing the ISCAS '85 benchmark (specifically on the c2670, c5315, and c7552) Verilog circuits, VPR fails to translate the Verilog over to a fabric when using FPGAShell. Using the k4_N4_40nm_standalone_openfpga.xml and k4_N4_tileable_40nm.xml architecture files (w/ yosys_vpr flow) the other benchmark fabrics are generated successfully.

Expected behavior
It should compile each of the benchmarks named above as it does with the others without failing.

Screenshots

Enviornment:

OS:
Ubuntu 18.04
Compiler:
gcc-7
Version:
Current master

The nets are present in the activity file (for example the one posted above) G5196, G5201, G5202... however there is not a net called G5196__G5201__G5202.
Note: When choosing which Verilog files, c2670.v and c5315.v were used (c2670a.v and c5315a.v were also tested with a similar error occurring).
Any help would be greatly appreciated.

@Iripi97 This is a known issue. VPR remove redundant buffers when the option --absorb_buffer_luts is turned on (a default behavior). This causes some mismatch between the activity file and post-implementation netlist.
There are two solutions:

Turn off the option --absorb_buffer_luts in your openfpga shell script. For example: https://github.com/lnis-uofu/OpenFPGA/blob/1603c9b40413a4256c7718b72d0cb40183bfb88b/openfpga_flow/openfpga_shell_scripts/mcnc_example_script.openfpga#L3
Remove the --activity_file option from your openfpga shell script. For example: https://github.com/lnis-uofu/OpenFPGA/blob/master/openfpga_flow/openfpga_shell_scripts/example_without_ace_script.openfpga


isCharging now uses pmset

Pull Request
Changes proposed:

[x] Fix
[ ] Add
[ ] Remove
[ ] Update

Description
If the battery is at 100% but plugged in then isCharging reports false. I don't think this makes sense as it still has a power connection.

@VictorLeach96 : yes you are right. Just fixed it (with a little modification to your pull request). New release 3.37.10 published. Thank you!

Had to correct it one more time ... can you test it on your side? new published version is 3.37.11

So this means, peseta has the following outputs:

discharging
charged
charging
AC connected

for the first two ischarging is false, for the last two it is true, right? I implemented it check now that way. OK for you?

With this configuration ischarging is false when the status is charged which is incorrect I think.
I think discharging is the only state where the power adapter would be unplugged.

OK, then this is a semantic thing ... if ischargingmeans "power adapter plugged" then you are right. But if ischarging should represent if the battery is actually is charging then it should be like this:
|--------------|------------|
| pmset        | ischarging |
|--------------|------------|
| discharging  | false      |
| charged      | false      |
| charging     | true       |
| AC connected | false      |
|--------------|------------|

Yeh I agree it's a semantic thing. Not sure what use cases you'd have for charging itself.
Our use case is we want to check whether the power adapter is plugged in or not.

Then it would make sense to add another property to indicate this - independent from then ischanging. I will add this tomorrow ...

Yes that would be great, thanks a lot!

Added battery.acconnected
Version 3.38.0 published


Provide a flag to ignore exception when template generator encounters…

… null to render in StrictMode. This is particularly useful in our upcoming change where in we want to turn StrictMode on but fear we might encounter a lot of such errors in prod.

I see the flag, but it does not appear to be used to actually do anything.

Sorry, added

I see the flag, but it does not appear to be used to actually do anything.

can see bit now?

Can't you achieve this goal using an InvalidReferenceEventHandler?

InvalidReferenceEventHandler

That would mean a broader check including invalid method names etc, I still want it to behave like strict mode ON except that swallow an error on rendering null value

InvalidReferenceHandler

That would mean a broader check including invalid method names etc,

And? I maintain that it's a typical use case for the InvalidReferenceHandler, and that I'm reluctant to include a new flag (unless it is considered as a backward compatibility flag). The strict mode is, well, strict. If you fear for production errors, just use the strict mode in dev and remove it in prod... or use the suggested handler.


Failing sample

Hello @Nykakin thanks for this wonderful library. Seems like I'm first person opening issue here.
I started using library in my project and find it very useful, but I see one sample I'm having is not parsed correctly. It returns error. I created unit test to reproduce this issue here:
https://github.com/Nykakin/chompjs/compare/master...pawelmhm:failing-sample?expand=1
Running this test in my localhost with Python 3.7.3 I get following:
ERROR: test_incorrect_escape (chompjs.test_parser.TestParser)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/pawel/chompjs/chompjs/test_parser.py", line 121, in test_incorrect_escape
    result = parse_js_object(f.read())
  File "/home/pawel/chompjs/chompjs/chompjs.py", line 31, in parse_js_object
    return json.loads(parsed_json)
  File "/home/pawel/.pyenv/versions/3.7.3/lib/python3.7/json/__init__.py", line 348, in loads
    return _default_decoder.decode(s)
  File "/home/pawel/.pyenv/versions/3.7.3/lib/python3.7/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/home/pawel/.pyenv/versions/3.7.3/lib/python3.7/json/decoder.py", line 353, in raw_decode
    obj, end = self.scan_once(s, idx)
json.decoder.JSONDecodeError: Invalid \uXXXX escape: line 1 column 7206 (char 7205)

----------------------------------------------------------------------
Ran 31 tests in 0.003s

FAILED (errors=1)

I see that other tool for parsing this kind of input works on it and produces valid dictionary.

Should be fixed by #2.


Regression in PGP verification of decrypted data since #1179

After upgrading to 1.72 our PGP code started giving incorrect signature validation errors when decrypting files. The exact same error can be reproduced using the KeyBasedFileProcessor example with the following command:
org.bouncycastle.openpgp.examples.KeyBasedFileProcessor -d encrypted.pgp pgpkey.asc tkhbackup

The expected output is message integrity check passed, but since commit 0132e856dddf759eae90a0e08cc8bca0acaa7705 #1179 , it gives message failed integrity check.
I've attached the private key and encrypted file used for this example. Both are dummy files generated during our integration test and can be used for any purpose you see fit.
example.tar.gz

Thanks, that was a big help. Try https://www.bouncycastle.org/betas 173b04 or later. You just need the PGP jar. Let me know if it checks out and we will do a 172.1 release for PGP. Can't leave something like this floating around.

Okay 172.1 for the BC PG API is now on https://www.bouncycastle.org/latest_releases.html
Give it a whirl, assuming it checks out with everyone I'll publish it to Maven Central

Thanks for the notice. I've tested 1.72.1 and can confirm that the issue is fixed for us.

We had this problem with our application, giving error while validating signature:
[info]   Cause: java.io.IOException: unknown object in stream: -119
[info]   at org.bouncycastle.openpgp.PGPObjectFactory.nextObject(Unknown Source)
[info]   at org.bouncycastle.openpgp.PGPObjectFactory$1.getObject(Unknown Source)
[info]   at org.bouncycastle.openpgp.PGPObjectFactory$1.hasNext(Unknown Source)
[info]   at scala.collection.convert.JavaCollectionWrappers$JIteratorWrapper.hasNext(JavaCollectionWrappers.scala:41)

I can confirm that by changing to use bcpg-jdk18on-173b04.jar this issue is fixed.

fixed in 1.73.


tsc: OOM when using large number of arguments

Version
5.24.2
Describe the bug
Note: Happy to submit a PR for this issue
The utility type RequiredParams, with its use of Permutation, makes our Typescript server crash due to OOM issues.
RequiredParams is fine as long as the number of arguments is OK, but when we've introduced a multiline string with 8 arguments in it, the number of permutations exploded to 8!=40320, and tsc crashed because it ran out of memory.
Here is the diagnostics information from our i18n package, without any external dependency.
tsc --diagnostics:
Files:              1616
Lines:            728468
Identifiers:      790619
Symbols:         5999545
Types:            516404
Instantiations:  2691792
Memory used:    2907860K
I/O read:          0.08s
I/O write:         0.00s
Parse time:        1.38s
Bind time:         0.61s
Check time:        6.05s
Emit time:         0.00s
Total time:        8.04s

tsc --diagnostics, with RequiredParams<...> | string replaced by string:
Files:             1616
Lines:           728468
Identifiers:     790311
Symbols:         447275
Types:             5864
Instantiations:    7532
Memory used:    672460K
I/O read:         0.08s
I/O write:        0.00s
Parse time:       1.47s
Bind time:        0.59s
Check time:       0.12s
Emit time:        0.00s
Total time:       2.18s

One way to solve it is to add a config to allow disabling the use of RequiredParams. The code to switch it on/off is already there, but basically depends on the TS version.
For now we've just patched the generator code to always spit out "string" instead of generateTranslationType(args) and this suits us.
Reproduction
On any sample project:

Start the generator: npm run typesafe-i18n
Run tsc --diagnostics. Write down the Symbols, Types, Instantiations, Memory used
Add translation strings with a large number of arguments (8+). For example, a multi-line string.
Run tsc --diagnostics again. Notice how the 4 values have exploded

Logs
No response
Config
{
  "$schema"<EMAIL_ADDRESS>  "outputPath": "./src/",
  "generateOnlyTypes": true
}

Additional information
No response

Hi @ridem, I know about the problem and was waiting for someone to file an issue to see if it is actually a problem that occurs in practice.
I think the most convenient way to improve this is to output a type without a permutation in cases where nrOfParams > 7.
This needs to be done here: https://github.com/ivanhofer/typesafe-i18n/blob/d26d6c29a65c3f2df913a70842302f53ab816b72/packages/generator/src/files/generate-types/translations.mts#L111
Instead of a primitive string I would go for a type that restricts the parameter names, but allows any arbitrary combination of those:
const message1: T<'a', 'b'>` = 'hello {a} and {b}' // correct
const message2: T<'a', 'b'>` = 'hello {b} and {b}' // correct
const message3: T<'a', 'b'>` = 'hello {a} and {a}' // correct
const message3: T<'a', 'b'>` = 'hello {c} and {d}' // incorrect

This would make sure to offer the best possible typesafety, without running into the OOM issue.
Also this line needs to change to import the correct helper type: https://github.com/ivanhofer/typesafe-i18n/blob/d26d6c29a65c3f2df913a70842302f53ab816b72/packages/generator/src/files/generate-types.mts#L86
It would be great if you could contribute a PR :)

For those who can't wait for this to be fixed in this branch (RIP Ivan, thanks so much for all your contributions), you can use patch-package to work around this.
I didn't care at all about type safety when providing translations in other languages, so I just patched out that whole feature. You may be able to find a more nuanced fix, like what Ivan suggested above. But my patch was simply:
patches/typesafe-i18n+5.17.0.patch
diff --git a/node_modules/typesafe-i18n/types/runtime/src/core.d.mts b/node_modules/typesafe-i18n/types/runtime/src/core.d.mts
index b8c7f07..3128efa 100644
--- a/node_modules/typesafe-i18n/types/runtime/src/core.d.mts
+++ b/node_modules/typesafe-i18n/types/runtime/src/core.d.mts
@@ -48,7 +48,9 @@ export interface ImportLocaleMapping extends LocaleMappingBase {
 type Permutation<T extends string, U extends string = T> = [T] extends [never] ? Array<string> : T extends U ? [T, ...Permutation<Exclude<U, T>>] : [T];
 type WrapParam<P> = P extends string ? `{${P}}` : never;
 type ConstructString<Params extends unknown[]> = Params extends [] ? `${string}` : Params extends [infer Param, ...infer Rest] ? `${string}${WrapParam<Param>}${ConstructString<Rest>}` : Params extends string ? Params : `${string}`;
-export type RequiredParams<Params extends string> = ConstructString<Permutation<Params>>;
+// patch-package START
+export type RequiredParams<Params extends string> = string;  // Was `= ConstructString<Permutation<Params>>;`
+// patch-package END
 export declare const isPluralPart: (part: Part) => part is PluralPart;
 export declare const translate: (textParts: Part[], pluralRules: Intl.PluralRules, formatters: BaseFormatters, args: Arguments) => LocalizedString;
 type GetArg<Arg extends string, Type> = Arg extends '' ? void : Arg extends `${infer OptionalArg}?` ? Partial<Record<OptionalArg, Type | undefined>> : Record<Arg, Type>;

i.e. I just changed RequiredParams to always simply be string. See the link above for patch-package for instructions to make your own patch.


refactor: remove optional_url dead code

Fixes https://github.com/ada-url/ada/issues/152

Merging.


BUG: transposing frame of virtual partitions raises TypeError: 'ray._raylet.ObjectRef' object is not callable

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): macOS Monterey
Modin version (modin.__version__):  cc3bdb7e4bcd2f8305a5faa02d82036af22cc5df
Python version: 3.10.4
Code we can use to reproduce:

import modin.pandas as pd
df_small = pd.DataFrame(list(range(1000)))
df = pd.concat([df_small] * 3)
print(df.T)

Describe the problem
The above snippet works in pandas, but in Modin, I get a wall of errors that includes the following ray task error repeated many times:

Stack trace
RayTaskError(TypeError): ray::_apply_func() (pid=94684, ip=<IP_ADDRESS>)
  At least one of the input arguments for this task could not be computed:
ray.exceptions.RayTaskError: ray::deploy_ray_func() (pid=94684, ip=<IP_ADDRESS>)
  At least one of the input arguments for this task could not be computed:
ray.exceptions.RayTaskError: ray::deploy_ray_func() (pid=94684, ip=<IP_ADDRESS>)
  File "/Users/maheshvashishtha/software_sources/modin/modin/core/execution/ray/implementations/pandas_on_ray/partitioning/virtual_partition.py", line 533, in deploy_ray_func
    result = func(*args, **kwargs)
  File "/Users/maheshvashishtha/software_sources/modin/modin/core/dataframe/pandas/partitioning/axis_partition.py", line 162, in deploy_axis_func
    result = func(dataframe, *kwargs.pop("args", ()), **kwargs)
  File "/Users/maheshvashishtha/software_sources/modin/modin/core/execution/ray/implementations/pandas_on_ray/partitioning/virtual_partition.py", line 419, in drain
    df = func(df, *args, **kwargs)
TypeError: 'ray._raylet.ObjectRef' object is not callable

Solution
This is a bug with virtual partitions. Using non-virtual partitions (no concat followed by rebalance first) works in Modin:
import modin.pandas as pd
values = list(range(1000)) * 3
df = pd.DataFrame(data=values, index=values)
print(df.T)

Works on master.


Tweak styles to be similar to API docs.

Depends on #138.
Adds a few more style tweaks to make the website more similar to the API docs:

Link color is darkened a bit.
Open Sans as font family for regular text.
Source Code Pro for code blocks.
Code and code blocks have a slightly gray background, the red-purple color is removed.

It's beautiful! I appreciate the consistency with the API docs.
Maybe the nav panel needs to be slightly wider in this style? Or the "Lean Community" title slightly smaller. The text is just a bit wider than the max width and forces a line break.

Maybe the nav panel needs to be slightly wider in this style? Or the "Lean Community" title slightly smaller. The text is just a bit wider than the max width and forces a line break.

The line break was actually forced so that we can have a large font size. :smile:  I've reduced the font size now so that it doesn't break;

Merged into #138


Simple tagged dispatch for alternative and external geometry implementation.

This allows alternative or externally defined geometries to be used with the jscad api.
A geometry module should add something like.
const registerImplementation = require('.../registerImplementation')
const jscadTag = 'some-unique-identifier'
registerImplementation(jscadTag, module)
And produce objects which carry { jscadTag: __jscadTag }.
Once that module is imported toImplementation(object) will produce that module, allowing constructions like.
toImplementation(object).eachPoint({}, fun, object)
See design in:
https://docs.google.com/document/d/1nz7dhtErHBqBpuFcu7ooRU91miVTD4UxiJ6od1TbUgs/edit#

hmm I like the concept behind this (as we discussed having multiple implementations is something we want to enable for the future), but I am not sure about the implementation at all

why do you need to 'register anything at all ?

const bspGeom  = require('bspGeom')
const hedgeGeom = require('halfEdgeGeom')

hedgeGeom.eachPoint(object, fun, object)
// OR
bspGeom.eachPoint(object, fun, object)
// use which ever you need, if they provide same api of course

In your example you've tightly coupled the operation to the geometry.
This is reasonable for a function in the geometry interface.
But it is unworkable for functions in the user api.
Consider a translate convenience function, for example.
const translate = (vector, shape) => transform(fromTranslation(vector), shape)
But which 'transform' function do we need?
const translate = (vector, shape) => toImplementation(shape).transform(fromTranslation(vector), shape)
Now, the user apis are decoupled from the geometries and can be extended independently.
e.g., by users.


Fix scaling

Part of issue #41. @jasmainak Please see comments for the other fix needed. Once added to this pull request, the output should checkout out with HNN.

While you're at it, could you add a test? Having a smoke test at the end of this file would be better than nothing !

Merged after some cosmits in https://github.com/jasmainak/mne-neuron/pull/47/commits/9352568846251d786caeca2232c88f210e156663
Thanks @blakecaldwell for the fixes.


Modify RFID hardware key since createDataWedgeProfile

Is possible change the Default value, for the RFID Hardware Key on Datawedge profile?

You should be able to use the StageNow tool (https://www.zebra.com/us/en/support-downloads/software/utilities/stagenow.html) and invoke the KeyMappingManager (https://techdocs.zebra.com/mx/keymappingmgr/) to remap the SCAN or TRIGGER buttons (exact key will depend on your device) to some other key.


introduce generic R function as.ddf

It is the natural complement to as.data.frame
it will have initially one method for data frames, but more are possible for local files, hive tables and such
can be a focal point to progressively refactor all conversion code to these two as.something functions.
the data.frame methods is almost indispensable to write quick test cases.

@piccolbo @binhmop is this still an issue?


[Feat]-CI/CD Pipeline for backend

We need 2 contributors to work on along with @kartikayasijaa

@KiranBaliga Discord channel me project intro dal dijiye or ak google meet kar riye bata dunga kaha karna hai

@Sibam-Paul I guess you mistakenly tagged me. @kartikayasijaa Please see.

@KiranBaliga opps sorry thanks bro

@ChampGupta go to discussion section and mention there

@kartikayasijaa is this issue is still available?

@Ameerjafar yes it is inform in the discussion

please assign me this issue.


Added process.argv support for issue #25

The verify mode works gets the numbers as process.arg[3] and process.arg[4]. The run mode doesn't (It gets passed the name of the submitted file as process.arg[3] and undefined as process.arg[4]).
The description needs a hint that the plugin must treat the input as int, otherwise it's quite tricky: first, I got back { answer: 27 } instead of 9 –looks fine but is the concatenation of "2" + "7".
Could you do the same revisions for problem 6, Extend Client? I got stuck there. It's exactly the same issue with missing input from process.argv.

@iiiw arguments for run are ok, since to run it now you do "node runner.js run program.js 2 3" but once published you'll do "seneca-in-practice run program.js 2 3".
Did you run the 6th step (Extend Client) from the npm version or the repo?
Then yeah i'll update exercise description to suggest to use parseInt.

@iiiw arguments for run are ok, since to run it now you do "node runner.js run program.js 2 3" but once published you'll do "seneca-in-practice run program.js 2 3".

True. That's what I didn't get in exercise 2. I thought they were passed anyway, be it run or verify.

Did you run the 6th step (Extend Client) from the npm version or the repo?

From the npm version.

Then yeah i'll update exercise description to suggest to use parseInt.

Thanks for the work.


Error in output code when using 6to5-loader with coreAliasing

I'm using 6to5-loader with coreAliasing and getting the following error in browser's console:
Uncaught TypeError: Cannot read property 'TypeError' of undefined

It's referencing to TypeError = _core.global.TypeError inside of core-js module. _core object appears to be empty.
Try this project to reproduce error.

Shouldn't this be reported to https://github.com/6to5/6to5-loader/?

Ah, yes, sure.


setImagFromURL is not setting the image at all

In swift 1.2 this code worked and set the images nothing has changed except im using swift 2.0 now.
The images are not displayed.
        let urlString = review.user!.image!
        let url = NSURL(string: urlString)
        vivrcell.userImage.hnk_setImageFromURL(url!)

I had the same problem, but it was due to a NSAppTransportSecurity problem. Check that just in case.

All haneke codes are working.. there might some problems on your code implementation.. the one your showing is just a snippet..

I think I'm seeing the same thing as well.
Example Code ::
        let url = NSURL(string: "https://placeholdit.imgix.net/~text?txtsize=33&txt=Screenshot&w=250&h=250")

        let imageView = UIImageView(frame: CGRect(x: 0, y: 0, width: 250, height: 250))

        imageView.hnk_setImageFromURL(url!, placeholder: nil, format: nil,
            failure: { _ in
            }, success: { _ in
                print("Image loaded") // prints out successfully
                print(imageView.image) // nil
            })

        addSubview(imageView)

@jkosoy  try removing format, also there is a problem with your closure.. read more about handling closures...

I don't think there is a bug here, ping me if I'm wrong I will reopen. thx


SPAPT/dgemv5 error

After removing the extra commas, the annotated dgemv.ancc.src5.c still throws the following exceptions in running.
ERROR: orio.module.loop.submodule.tile.transformation:: tile size (256) must be divisible by the stride value (512) WARNING: orio.module.loop.submodule.tile.transformation: tile size 256 must be greater than the stride value 512; tile index = jjj

That's a huge stride, is it correct? You can't have tile size smaller than it, for sure. Try decreasing the stride perhaps?

Can you include the exact code in the issue (the file dgemv.ancc.src5.c? Thanks!

On Mar 19, 2020, at 15:19, overrainbow<EMAIL_ADDRESS>wrote:
After removing the extra commas, the annotated dgemv.ancc.src5.c still throws the following exceptions in running.
ERROR: orio.module.loop.submodule.tile.transformation:: tile size (256) must be divisible by the stride value (512) WARNING: orio.module.loop.submodule.tile.transformation: tile size 256 must be greater than the stride value 512; tile index = jjj
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://urldefense.com/v3/__https://github.com/brnorris03/Orio/issues/22__;!!C5qS4YX3!TNIbUG83Tyk6LIfNEYS-CM0bwPPNn2SZf6K41AMEPCnJzsAXAiyZm_IOiBkzoFUX_iQ$, or unsubscribe https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/ABRKGGDLHSWDM4GCDL47JGTRIKK6RANCNFSM4LPXH4XA__;!!C5qS4YX3!TNIbUG83Tyk6LIfNEYS-CM0bwPPNn2SZf6K41AMEPCnJzsAXAiyZm_IOiBkzAFu-_5s$.


Add Identity Access Management (IAM) to the Storage API

Adds support for bucket-level IAM (currently in limited alpha). More information
about IAM in Google Cloud Storage can be found at
https://cloud.google.com/storage/docs/access-control/iam

@neozwu could you do a first pass on this PR?

@michaelbausor could you take a look too?

Changes Unknown when pulling 8eabb8543439628146be7be8817077b22c5d9c53 on rybosome:storage-iam into ** on GoogleCloudPlatform:storage-iam**.

@michaelbausor This is ready for another pass.

@michaelbausor Took your suggestion, now ready for review.

Changes Unknown when pulling 9b5ce8cdd80076d0b3acdbf7bc9652a4df70b3a3 on rybosome:storage-iam into ** on GoogleCloudPlatform:storage-iam**.

LGTM


Handle Summarization Failures

Background Issues
Differential Summary Issues
In addition to the below bugs, there is also the problem of unbounded differential summaries. We saw lots of user feedback that perceived performance was bad because there was lots of op replay.

#4676
#4296

To many outstanding ops
Odsp will eventually block a client from sending ops, when the total number of ops passes some threshold (1000). Blocking the main client from sending ops can result in the summarizer not spawning.

#4068

Summarizer Disconnect
The below issue lists several cases where summarization can fall behind in such a way that it is almost impossible to catch up. We should explicitly account for these cases.

#4069

Proposals
Differential summaries were added as a way for the container to continue to make progress in the presence of summary failures. However, the current implementation goes too far in the direction of making progress and doesn't adequately incentivize the container developer to fix the issue. The best solution here would proportionally degrade the user experience, provide a means for a user to unblock themselves, and incentivize developers to fix underlying issues.
Differential summaries are primarily a mitigation for data store summarization problems, but it's also possible for the container as a whole to face similar issues. Currently, we do nothing in this case, and the container will simply carry on until it is blocked, and potentially stuck. Just like differential summaries we should proportionally degrade the user experience, provide a means for a user to unblock themselves, and incentivize developers to fix underlying issues.
Proportionally degrade the user experience
In all these cases I believe the best course of action is to make the affected context readonly until the problem can be resolved. Doing this will block additional changes which prevents any additional data loss, but still provides the user with access to their existing content. While at a high level this approach is easy to understand there is significant complexity in making it work in practice which I will expand on below.
Provide a means for a user to unblock themselves
We are limited in what we can do here in the presence of a code defect in the data store or container which is preventing summaries.

Moving to June since we're hitting higher-pri issues at the moment.


API for POST method request

Before moving forward with the POST request API, I have some questions regarding headers. Specifically, I'm curious if there is a need to establish a process for obtaining header values based on their names. I've already created a pull request that includes this functionality, but it will only be merged if it's necessary. Furthermore, I'd like to verify if there are any outstanding tasks associated with the GET request.

Here's the MDN reference for the POST method: https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods/POST
How do you think we should expand the request API to allow for:

URL encoded form data via application/x-www-form-urlencoded content type
Binary data via multipart/form-data content type

Python requests distinguishes this based on the type of input to the data argument. I.e. if data is a dict, it's a URL form encoded (IIRC). If it's a bytes or file object, it goes as multipart/form-data.
We could take a similar approach, i.e. control the request content type based on the type of the data input argument.
While this is all most useful with POST requests, it applies to all other request methods, e.g. it's possible to send data in the body of a GET request, although normally servers should ignore such data.

Thanks @milancurcic , I also go through the request library. In the requests library, data can be passed to an HTTP request using the data, json, and files parameters.
Here's how each of these parameters works internally:

data: The data parameter is used to pass form-encoded data to an HTTP request. When the data parameter is passed, the Content-Type header is automatically set to application/x-www-form-urlencoded. The data is then encoded as a URL-encoded string and added to the request body.

json: The json parameter is used to pass JSON-encoded data to an HTTP request. When the json parameter is passed, the Content-Type header is automatically set to application/json. The data is then encoded as a JSON string and added to the request body.

files: The files parameter is used to upload one or more files to an HTTP request. When the files parameter is passed, the Content-Type header is automatically set to multipart/form-data. The data is then encoded as a multipart form-data and added to the request body along with the files.

These parameters can be used together in the same HTTP request.

If both the data and json parameters are passed, the json parameter takes precedence and the data is encoded as a JSON string.

If both the data and files parameters are passed, the files parameter takes precedence and the data is encoded as a multipart form-data along with the files.

Sending both json and files in the same request can result in an invalid or ambiguous Content-Type header, which can cause errors or unexpected behavior on the server.

We can adopt a similar approach for our HTTP requests by defining three parameters: json, data, and files. The json parameter would be of type string, while the files parameter would be an array of file_type objects. Additionally, the data parameter would be an array of pair_type objects to hold key-value pairs of data.
To define the file_type object, we can use the following code:
type :: file_type
  character(:), allocatable :: form_field, file_path
end type file_type

This file_type object has two variables: form_field and file_path. form_field would be the name of the form field we want to send the file as, while file_path would store the path of the file we want to send.
To define the pair_type object, we can use the following code:
type :: pair_type
  character(:), allocatable :: key, value
end type pair_type

When using the data parameter in the request, we need to ensure that the key-value pairs are URL-encoded. We can use the curl_easy_escape function in libcurl to encode these strings.
Finally, we can define a request_type object with three variables: json, data, and files. The data variable would be an array of pair_type objects, while json and files would be of type string and an array of file_type objects, respectively. Here's the code:

Based on the discussion from our last meeting, we have agreed to use a request_type object with three variables for sending data: data, form, and file.
The data variable will be a string type that can be used to send various types of content such as JSON, text, binary data, etc. By default, no Content-Type header will be set, so the client must provide the Content-Type header themselves.
The form variable will be an array of pair_type objects used to send form-encoded data. When using the data argument, the Content-Type header will automatically be set to application/x-www-form-urlencoded.
The file variable will be of type pair_type used to send files. When using the files argument, the Content-Type header will be set to multipart/form-data.
Please let me know if there is anything I may have overlooked or misunderstood, or if you have any other ideas that you would like to share.

I have submitted a PR that adds functionality to support sending form data in HTTP requests. Additionally, I have included an example that demonstrates how to send form data using this new functionality

I have tested PUT, PATCH, and DELETE methods and they are functioning properly without the need of any additional setup.


Dicom image can not be displayed

As a front-end developer, I think this open-source project is particularly good, so I embedded it into the developed project. However, I encountered some problems in the process of using it. The problem is: I use the Java version of dcm4che, and the GIT address is https://github.com/dcm4che/dcm4chee-arc-light, The dcm4che version is recent, but the image can not be displayed, and the problem is investigated The main reason is that the "instancenumber" and "SeriesNumber" fields are string type, not numeric type. I hope the author can do some data type compatibility, or provide the highest version supported by Java dcm4chee. Thank you very much!

Could you kindly provide the data if it has been anonymized and you can confirm that there is no patient health information present in any of the headers or embedded within the pixel data?

I am also using v3.7.0 (release) on my v3.7.0 (release) on android mobile.I have 1 problem
The first time the thumbnail was rendered there was no problem, but when I selected another thumbnail it had an error displaying the thumbnail. T hope you can help me. Thanks

It's like this image is encroaching on another image


Fix: cli.js - use var instead of const

Explain the motivation for making this change. What existing problem does the pull request solve?
Fixes https://github.com/facebook/react-native/issues/6203

SyntaxError: Use of const in strict mode.

The rest of the file isn't using es6 anyway and babel-register is run on the file it's included in

Test plan (required)
Shouldn't have issues replacing it since it's used right after one time.

@facebook-github-bot shipit


Add new Connect Endpoints

[X] GET /v1/me/player/devices => SpotifyWebAPI::getMyDevices()
[X] GET /v1/me/player => SpotifyWebAPI::getMyCurrentPlaybackInfo()
[X] GET /v1/me/player/currently-playing => SpotifyWebAPI::getMyCurrentTrack()
[x] PUT /v1/me/player => SpotifyWebAPI::changeMyDevice()
[x] PUT /v1/me/player/play => SpotifyWebAPI::play()
[x] PUT /v1/me/player/pause => SpotifyWebAPI::pause()
[x] POST /v1/me/player/next => SpotifyWebAPI::next()
[x] POST /v1/me/player/previous => SpotifyWebAPI::previous()
[x] PUT /v1/me/player/seek => SpotifyWebAPI::seek()
[x] PUT /v1/me/player/repeat => SpotifyWebAPI::repeat()
[x] PUT /v1/me/player/volume => SpotifyWebAPI::setMyVolume()
[x] PUT /v1/me/player/shuffle => SpotifyWebAPI::shuffle()

Full reference: https://developer.spotify.com/web-api/web-api-connect-endpoint-reference/
Any feedback on names and help with implementing is greatly appreciated! A WIP is available in #74.

Almost done, just one small issue with SpotifyWebAPI::play() and no options. But that endpoint seems to have some issues right now.

Released in 1.10.0! 🎉


Backup admin cert, and regenerate kubeconfig during upgrade

Pre-existing admin certificates (< v1.3.2) are not valid because they are missing the user required for RBAC. For this reason, we backup pre-existing admin certs if we detect that they were generated by us.
If we regenerate the admin cert, we need to regenerate the kubeconfig file because it embeds the admin certificate. The previous kubeconfig file is backed up if it exists, and the user is informed about it.

LGTM


will the training dataset released now?

hello,when the training dataset will be released?thank you

Hi, any updates on this?

we have already release the training data


[BUG] After clearing the userData, then select install guest agent is invalid

Describe the bug

After clearing the userData, then select install guest agent is invalid
To Reproduce
Steps to reproduce the behavior:

go to vm create page
Fill name, cpu, memory, select image
click Advanced Options tab
uncheck install guest agent,

Clear the userData

check install guest agent
userData is still empty

Expected behavior

At this point the userData should contain the data of the quem guest agent
Support bundle

Environment

Harvester ISO version:  v1.0.2
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630):

Additional context
Add any other context about the problem here.

Hi @TachunLin  ,    here is a little enhancement.

If there is no #cloud-config in userData when you check install guest agent, UI need to add it automatically, otherwise it will not take effect.

test step:

go to create vm page, click Advanced Options tab
uncheck inatall guest agent,  clear user data

check install guest agent
we need to add #cloud-config comment automatically at this time

However, if #cloud-config already exists, checking the intall guest agent will not automatically add this comment
e.g:

Verified fixed on master-7b698413-head (10/22). Close this issue.
Result

After unchecking the Install guest agent all user data would be cleaned, then when we check it again, the guest agent user data will be added back

https://user-images.githubusercontent.com/29251855/197442365-5cdc476e-0cd7-47f2-82e4-c3653f277243.mp4

When user data already have #cloud-config, checking the install guest agent will not add the #cloud-config again

https://user-images.githubusercontent.com/29251855/197442505-f4a2489f-2883-4c46-b94e-8f8b3aee938a.mp4
Test Information

Test Environment: 1 node Harvester local kvm machine
Harvester version: master-7b698413-head (10/22)

Verify Steps

Go to create vm page, click Advanced Options tab
Uncheck the inatall guest agent to clear user data
check the install guest agent
Manually add #cloud-config comment to user data
Check the install guest agent again


[Github] AI duplicate issues bot: Issues missing in their comment

Microsoft PowerToys version

Installation method
Other (please specify in "Steps to Reproduce")
Running as admin
None
Area(s) with issue?
General
Steps to reproduce
The bot comments missing open issues that are very old.
For example issue #32600: The issue we deduplicate against is #26. But it is not listed in the bot comment.
I saw this behavior on other issues too.
✔ Expected Behavior
The open issue is listed
❌ Actual Behavior
The open issue we dedup against is not listed.
Other Software
No response

Additionall information: This does not happen for open duplicates for higher numbers.

cc: @craigloewen-msft

cc: @plante-msft


Add missing jumps

#868
Add Jump for DOCT, IARP, IASP, IATU, NROB, SFPF, SFPI, SMIM, SSST, STYL, TOBJ

wow, keeping busy? 👍

perfect, merging, thanks 😄


Fix outputoffset when build dag req

Say we have a query select id, id, it, it from t.
In the past, we construct a DAG req as follows:
table scan
    addCol
        id, id, it, it
        0    1,  2 , 3
DAGReqBuilder
    addOutputOffset
        0 1 2 3

It works in old TiKV, but not in the latest tikv.
The expected table scan is as follow:
table scan
    addCol
        id, it
        0    1
DAGReqBuilder
    addOutputOffset
        0 0 1 1

This PR fixes this.

/run-all-tests

/run-all-tests

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ce6e612). Click here to learn what that means.
The diff coverage is 22.97%.

@@            Coverage Diff            @@
##             master     #615   +/-   ##
=========================================
  Coverage          ?   53.92%
  Complexity        ?      985
=========================================
  Files             ?      136
  Lines             ?     6413
  Branches          ?      761
=========================================
  Hits              ?     3458
  Misses            ?     2628
  Partials          ?      327

Impacted Files
Coverage Δ
Complexity Δ

.../main/java/com/pingcap/tikv/meta/TiDAGRequest.java
41.8% <22.97%> (ø)
48 <3> (?)

Continue to review full report at Codecov.

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

it will be better if bug fixing & refactor are in separated PR

/run-all-tests tikv=release-2.1 tidb=release-2.1 pd=release-2.1

LGTM

/run-all-tests

@marsishandsome PTAL. In this PR, only fix will be merged into master.

/run-all-tests

/run-all-tests

LGTM


Problems running TotalSegmentatorV2

Hi there,
I was checking out V2 and I was able to install it.
Running: TotalSegmentator -i tmp/Example.nii.gz -o tmp/segmentationsv2/ -f
works for me.
If I try to use it without the --fast option, I get the following error:
If you use this tool please cite: https://pubs.rsna.org/doi/10.1148/ryai.230024

Resampling...
  Resampled in 4.53s
Predicting part 1 of 5 ...
100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 18/18 [00:01<00:00, 16.18it/s]
Predicting part 2 of 5 ...
Traceback (most recent call last):
  File "/data/an55321/anaconda3/env/totalsegv2/bin/TotalSegmentator", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/data/an55321/anaconda3/env/totalsegv2/lib/python3.11/site-packages/totalsegmentator/bin/TotalSegmentator.py", line 129, in main
    totalsegmentator(args.input, args.output, args.ml, args.nr_thr_resamp, args.nr_thr_saving,
  File "/data/an55321/anaconda3/env/totalsegv2/lib/python3.11/site-packages/totalsegmentator/python_api.py", line 293, in totalsegmentator
    seg_img = nnUNet_predict_image(input, output, task_id, model=model, folds=folds,
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/data/an55321/anaconda3/env/totalsegv2/lib/python3.11/site-packages/totalsegmentator/nnunet.py", line 372, in nnUNet_predict_image
    nnUNetv2_predict(tmp_dir, tmp_dir, tid, model, folds, trainer, tta,
  File "/data/an55321/anaconda3/env/totalsegv2/lib/python3.11/site-packages/totalsegmentator/nnunet.py", line 205, in nnUNetv2_predict
    predictor.initialize_from_trained_model_folder(
  File "/data/an55321/anaconda3/env/totalsegv2/lib/python3.11/site-packages/nnunetv2/inference/predict_from_raw_data.py", line 86, in initialize_from_trained_model_folder
    checkpoint = torch.load(join(model_training_output_dir, f'fold_{f}', checkpoint_name),
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/data/an55321/anaconda3/env/totalsegv2/lib/python3.11/site-packages/torch/serialization.py", line 797, in load
    with _open_zipfile_reader(opened_file) as opened_zipfile:
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/data/an55321/anaconda3/env/totalsegv2/lib/python3.11/site-packages/torch/serialization.py", line 283, in __init__
    super().__init__(torch._C.PyTorchFileReader(name_or_buffer))
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
RuntimeError: PytorchStreamReader failed reading zip archive: failed finding central directory
Exception ignored in: <totalsegmentator.libs.DummyFile object at 0x7fc0823bdc90>
AttributeError: 'DummyFile' object has no attribute 'flush'

Any suggestions?
Thanks,
Andre

This is strange. Maybe the download of the weights somehow failed and the downloaded weights are corrupted. You can try to remove the directory ~/.totalsegmentator and run again. Then all weights will be newly downloaded.

Hi Jakob. Thanks for the suggestion. That helped with my local installation. Weird. Maybe the download failed yesterday or so.
I need to use TotalSegmentator in a Docker. If I use your provided Dockerfile or alternatively install it via pip inside a docker, I get the following result:
TotalSegmentator -i /in/converted_image.nii.gz -o /app/test

If you use this tool please cite: https://pubs.rsna.org/doi/10.1148/ryai.230024

TotalSegmentator sends anonymous usage statistics. If you want to disable it check the documentation.
Resampling...
  Resampled in 0.81s
Predicting part 1 of 5 ...
100%|█████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 4/4 [00:00<00:00, 18.11it/s]
Bus error (core dumped)
root@7f458668b14e:/workspace# /usr/lib/python3.10/multiprocessing/resource_tracker.py:224: UserWarning: resource_tracker: There appear to be 6 leaked semaphore objects to clean up at shutdown
  warnings.warn('resource_tracker: There appear to be %d '

Any idea about this as well?
Thanks!

This error does not seem familiar to me. I do not really have an idea what the issue might be.

This might be related to the shared memory given to docker/podman you can specify more shared memory using for example:--shm-size 2g where 2g is a larger than default of 64m

That worked. Thanks Ben!

Thanks for pointing this out. I always run my docker container with these options:
docker run --gpus 'device=0' --ipc=host

I think this also fixes the shared memory problem.

Hi Jakob,
I just tried your call and that also works for me. THanks for your help. I'll close this ticket.


[BUG] Script not found

Hello,
After installation can not run any script.
Error looks like: https://ibb.co/4VgYSVv https://ibb.co/3CchdGg
MU version: 0687
Is means that, installation had found error ? The scripts should be loaded to unit ?
Regards

Just restart the unit and run the import scripts button again.

No changes.
When yesterday tried to install toolbox, had a error (can not remember part of text, but there had were two buttons - "again" or "cancel" - something like that : https://github.com/jilleb/mib2-toolbox/issues/27 ). And the update MQB Coding Toolbox 2.7 is empty: https://ibb.co/S3v6SWd. Tried to update it with SD card one more time, but unit could not start it (button was unactived). Today entered to SWDL and the option was active. The "second" MQB Coding Toolbox 2.7 is not empty (POI's are visible - https://ibb.co/CvLTqPs)

Keep on trying. After install, Run the script that imports new scripts.
The copy_phone_customer.sh script is already on the unit, so this error should never occur, unless... your unit has been modified by someone. Some companies like APG remove stuff from the green menu.

Is any way to put that script into The unit ? My unit is already hacked so i looking for The way without software update.
Thanks Jilleb

Yes, check the manual install in the readme. On a hacked unit, I cannot guarantee it's working, because sometimes they just mess things up 😅😅


initial functionality

Part of the approach is hindered by https://github.com/docker/compose/issues/8671

If/when implemented, this would help:  https://github.com/docker/compose/issues/7517

Pull requests:

https://github.com/Senzing/docker-compose-air-gapper/pull/4
https://github.com/Senzing/docker-compose-air-gapper/pull/5


Xcode 15.0.1 install failed with aria2

Precondition:

macOS Sonoma 14.0
aria2 installed (homebrew)

Steps:

run xcodes install 15.0.1

Expected:

Xcode 15.0.1 installed as usual

Actual:

getting an error:

Downloading with aria2 (/opt/homebrew/bin/aria2c)

XcodesKit/Environment.swift:80: Precondition failed: Aria must be called on the main queue
[1]    10494 trace trap  xcodes install 15.0.1 --experimental-unxip

Workarounds:

remove aria2 with brew uninstall aria2 (if installed with brew)
use --no-aria2 options

is there a workaround for this issue?

Upgrading aria2 to 1.37.0 helped for me.


Indent end squiggly heredoc

As a part of updating the Hanami codebase to use Ruby 2.3's squiggly heredoc <<~, I used rubocop's auto-correct feature. It mostly worked perfectly, but it left the end heredoc delimiters where they were. In our case, this was with 0 spaces. See this commit.
I've confirmed that this PR fixes the problem above, but I can't figure out how to write a passing spec. The trouble is that the shared_example calls strip_indent on both the code and thecorrection. If I remove those, this spec passes, but others fail. I'm not sure if I can write a passing spec with the indents being stripped.
Happy to do more work on this, just looking for some direction.
(And I'll clean up this PR/commits to fit with the project, once this is ready to merge)
Here's the Rspec output:
Failures:

  1) RuboCop::Cop::Layout::IndentHeredoc quoted by  EnforcedStyle is `squiggly` autocorrects for first line minus-level indented, with `-`
     Failure/Error: expect(corrected).to eq(correction.strip_indent)

       expected: "puts <<~RUBY2\n  def foo\n    bar\n  end\nRUBY2\n"
            got: "    puts <<~RUBY2\n      def foo\n        bar\n      end\n    RUBY2\n"

       (compared using ==)

       Diff:
       @@ -1,6 +1,6 @@
       -puts <<~RUBY2
       -  def foo
       -    bar
       -  end
       -RUBY2
       +    puts <<~RUBY2
       +      def foo
       +        bar
       +      end
       +    RUBY2

     Shared Example Group: :offense called from ./spec/rubocop/cop/layout/indent_heredoc_spec.rb:272
     Shared Example Group: :all_heredoc_type called from ./spec/rubocop/cop/layout/indent_heredoc_spec.rb:339
     # ./spec/rubocop/cop/layout/indent_heredoc_spec.rb:21:in `block (3 levels) in <top (required)>'

@bbatsov - Are you fine with me opening up a new PR with these changes and the test fix?  I think I could open a PR against this PR, but I believe @cllns would then have to merge my fix.

@garettarrowood I'm not sure if you have the appropriate permissions, but this PR allows commits to my branch from rubocop maintainers with write access. If that doesn't work, I can merge it into my branch (or you could integrate my changes into your branch and open the PR directly against master; I don't feel any need to own this commit.) :)

@garettarrowood I'm fine with a new PR, I think that would be the easiest thing to do.

Guess we can close this one.


Custom css input

Loaded in as a snippet

@theRealPadster It has some merge conflicts with the styling I believe.

Looks like it's just the map file. Should probably merge the latest updates in


7.2.0-preview.7 release changelog

PR Summary
Merge the release branch back to GitHub master.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.

The changes are only to .spelling and change log, and should have nothing to do with the failed Linux CI.