[Bug] Fix callingMiddleware unit test async running issue

Purpose

Part of the unit tests always run successfully without checking XCTAssert because there is no async await or expectation fulfill
Fix the issue by applying all unit tests in CallingMiddleware with async await

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

is this issue only happening in CallingMiddlewareHandlerTests?

is this issue only happening in CallingMiddlewareHandlerTests?

I haven't looked into other test classes yet. Please bring the issue up when you see similar bugs


Huge blend shape overhead with multiple materials

Godot version
3.2; 3.3.2; 3.4.beta 6a058cb
System information
Windows 10; GLES3; Nvidia RTX 2080 Ti and Linux; GLES3; Nvidia GTX 1050 m
Issue description
Godot 3.x stores vertices and blend shapes per surface. This means that the number of transform feedback passes on a model with blend shapes is multiplied by the number of surfaces.
The blend shape updates are also done per depth and color pass. From looking through renderdoc, it seems to be performing a prepass for some reason, so on a scene with a directional light with shadows, with my model that has 90 blend shapes, that makes 3 * 9 * 90 = 2500 draw calls for a single model.
Here is a GPU utilization graph showing Godot bringing my RTX card to its knees with a single model:

We ran into this issue when debugging performance bottlenecks in a usecase for over 50 blend shapes using real-time face tracking data from iOS.
The original project is available here:
https://github.com/vss-project/vss
In the real-world application, we are not able to get above 20 fps on this model on Godot 3.x.
Here is a screenshot from RenderDoc showing all of the blendshape passes adding up to a total of 60000 events. (This is all for drawing a single animated character):

We also tried GLES2, but the CPU skinning takes half a second per frame, so it's no good for this application.
The "one_material" model is the same model (same number of vertices), but with the materials merged into one surface using an external tool in Unity + UniGLTF.
One fix could be to store vertex buffers and blendshapes contiguously, and then the transform feedback pass can update all surfaces at the same time on each mesh.
Godot 4.0 is fast enough that I am not able to reproduce this issue without a more robust performance benchmark (e.g. 50 characters in the scene at once + FPS counter). But this usecase could come up in an application I am making.
Steps to reproduce
Open the attached project.
For each of the three scenes:
many_active_blendshapes.tscn
many_active_blendshapes_one_material.tscn
no_active_blendshapes.tscn

Open the scene
Click AnimationPlayer in the tree
In the inspector, change [stop] to KeepUpdating
Watch the performance.

Minimal reproduction project
blendshape_performance.zip

From looking through renderdoc, it seems to be performing a prepass for some reason

You can disable the depth prepass in the Project Settings when using GLES3. When using GLES2, the depth prepass is always disabled.


Multiple Protein Sequences

Hi
First of all Great work, just a question :
1- it seems that the Colab version doesn't work because it depends on the requirments.txt file and you just updated this version to work with the environmnet.yml file. I already take care of the Colab so if you want I could push my code to the branch to make it compatible with these new changes.
2- is this possible to pass multiple protein sequences to the file and get the predictions with a list of sequences?

The Colab should be fixed now.
As for 2., the Colab doesn't currently support multiple sequences. The local version does, though.


Configurability

Important changes:

Add configuration via config.ts in the project root
Setup testing for this library
Update version to 0.2.0
Copy assets from within the tooling

@DorianGrey

LGTM.


Cluster did not shutdown when capacity request expired

On 11/1 a capacity request for 20 nodes was entered for the "uw" cluster.  It appears that the request should have expired the next day, but when the expiration came, not action was taken by the provisioning system, and the cluster was left running.  Also, there were no entries in the provisioning tab.

Is it possible that the provisioning system was updated while a capacity request was active, and the expiration of that capacity request was not acted on by the new deployment of the provisioning system?

fixed v3.3.3


MIT License ?

Hello
Is the open source license an MIT license? Couldn't find any specific mention to it.
Could you kindly clarify?
Thanks

Hi @vilav It’s Apache-2 instead, please see:
https://github.com/carbon-design-system/carbon-components/blob/ab384e5a6721c06052bb971ce688b46eff50c9b9/package.json#L143
https://github.com/carbon-design-system/carbon-components/blob/master/LICENSE.md


Increase throttling alarm threshold

Throttling on pods is set to trigger alarm when it is above 25%
This should be much much higher because if the applications are well set up they would consumer efficiently the resources.
I would say this Throttling should be only triggered for CPU usage above 95% to 90%.
It's normal for my clusters to always trigger an alarm for my prometheus-node-exporter pods as they always have a a CPU consumption that is around the 90% requested CPU

CPUThrottling alert threshold is customizable. Threshold value is adjusted with cpuThrottlingPercent variable, which comes from kubernetes-mixin.
Source: https://github.com/kubernetes-monitoring/kubernetes-mixin/blob/master/alerts/resource_alerts.libsonnet

Yes, but the grafana dashboards are pre-generated no?
Unless we find a way to pass the jsonnet code into the helm values config and generate the dashboards in sidecar/init container when instantiating grafana

Yes, but the grafana dashboards are pre-generated no?

No, they are part of https://github.com/kubernetes-monitoring/kubernetes-mixin and changing cpuThrottlingPercent affects them too.

I recon that this is the helms repo

No, helm repository lives in https://github.com/prometheus-community/helm-charts/tree/main/charts/kube-prometheus-stack and this project doesn't use helm.

Closing as this appears to be answered. Please reopen if you think this is still relevant.


Gateway API: Can't  get a GRPCRoute working

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Hi everyone,
I installed Cilium on EKS and I wanted to test the GRPCRoute of the Gateway API but I get an error:
grpcurl -plaintext -proto ./demo.proto boutique-product.cloud.ogenki.io:443 hipstershop.ProductCatalogService/ListProducts
ERROR:
  Code: Unimplemented
  Message:

I installed the experimental CRDs with GAPI enabled (everything else is working like a charm: HTTPRoute, Gateways...). I reproduced the environment of the Lab on advanced GAPI use cases with the Google's microservices demo stack.
When I look at Supported features of the Gatewayclass, GRPC doesn't appear:
kubectl get gatewayclasses -o yaml cilium
apiVersion: gateway.networking.k8s.io/v1
kind: GatewayClass
...
  supportedFeatures:
  - Gateway
  - HTTPRoute
  - HTTPRouteDestinationPortMatching
  - HTTPRouteHostRewrite
  - HTTPRouteMethodMatching
  - HTTPRoutePathRedirect
  - HTTPRoutePathRewrite
  - HTTPRoutePortRedirect
  - HTTPRouteQueryParamMatching
  - HTTPRouteRequestMirror
  - HTTPRouteRequestMultipleMirrors
  - HTTPRouteResponseHeaderModification
  - HTTPRouteSchemeRedirect
  - ReferenceGrant
  - TLSRoute

Here are the manifests that are being declared in my repo
---
apiVersion: gateway.networking.k8s.io/v1beta1
kind: Gateway
metadata:
  name: onlineboutique
  namespace: onlineboutique
  annotations:
    cert-manager.io/cluster-issuer: letsencrypt-prod
spec:
  gatewayClassName: cilium
  infrastructure:
    annotations:
      external-dns.alpha.kubernetes.io/hostname: boutique.${domain_name},boutique-product.${domain_name}
      service.beta.kubernetes.io/aws-load-balancer-scheme: "internet-facing"
      service.beta.kubernetes.io/aws-load-balancer-backend-protocol: tcp
  listeners:
    - name: https
      hostname: "boutique.${domain_name}"
      port: 443
      protocol: HTTPS
      allowedRoutes:
        namespaces:
          from: Same
      tls:
        mode: Terminate
        certificateRefs:
          - name: onlineboutique-tls
    - name: grpc
      hostname: "boutique-product.${domain_name}"
      port: 443
      protocol: HTTPS
      allowedRoutes:
        namespaces:
          from: Same
      tls:
        mode: Terminate
        certificateRefs:
          - name: onlineboutique-product-tls
---
apiVersion: gateway.networking.k8s.io/v1alpha2
kind: GRPCRoute
metadata:
  name: productcatalogservice
  namespace: onlineboutique
spec:
  parentRefs:
    - namespace: onlineboutique
      name: onlineboutique
      sectionName: grpc
  hostnames:
    - "boutique-product.${domain_name}"
  rules:
    - matches:
        - method:
            service: hipstershop.ProductCatalogService
            method: ListProducts
      backendRefs:
        - name: productcatalogservice
          port: 3550

Note: using port-forward the command works
Cilium Version
cilium version
Client: 1.15.3 22dfbc58 2024-03-26T11:45:10+01:00 go version go1.21.8 linux/amd64
Daemon: 1.15.3 22dfbc58 2024-03-26T11:45:10+01:00 go version go1.21.8 linux/amd64

Kernel Version
Amazon EKS Bottlerocket v1.29.1-eks-61c0bbb
Kubernetes Version
v1.29.1-eks-b9c9ed7
Regression
No response
Sysdump
cilium-sysdump-20240409-093535.zip
Relevant log output
No response
Anything else?
No response
Cilium Users Document

[ ] Are you a user of Cilium? Please add yourself to the Users doc

Code of Conduct

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

Thanks for your issue. I have done a quick test with below manifest (no https), but unable to simulate.
Can you share the sysdump for further investigation? Thanks.
$ grpcurl -plaintext -proto ./demo.proto -authority=my-grpc-service.foo.com $lb:80 hipstershop.ProductCatalogService/ListProducts
{
  "products": [
    {
      "id": "OLJCESPC7Z",
      "name": "Sunglasses",
      "description": "Add a modern touch to your outfits with these sleek aviator sunglasses.",
      "picture": "/static/img/products/sunglasses.jpg",
      "priceUsd": {
        "currencyCode": "USD",
        "units": "19",
        "nanos": 990000000
      },
      "categories": [
        "accessories"
      ]
    },
...

apiVersion: gateway.networking.k8s.io/v1beta1
kind: Gateway
metadata:
  name: grpc
spec:
  gatewayClassName: cilium
  listeners:
  - protocol: HTTP
    port: 80
    name: web-gw
    allowedRoutes:
      namespaces:
        from: Same
---
apiVersion: gateway.networking.k8s.io/v1alpha2
kind: GRPCRoute
metadata:
  name: products
spec:
  parentRefs:
    - namespace: default
      name: grpc
  hostnames:
    - my-grpc-service.foo.com
  rules:
    - matches:
        - method:
            service: hipstershop.ProductCatalogService
            method: ListProducts
      backendRefs:
        - name: productcatalogservice
          port: 3550

Hi @sayboras , I provided the sysdump. Thank you

I can confirm that the HTTPS grpc route does not work. I got the same error message that @Smana received. The grpc route using a plain HTTP connection works without an issue. I am facing a similar issue trying to use a grpc route for argocd.

Cilium does not handle ALPN negotiation yet needed for GRPC.  https://github.com/cilium/cilium/issues/30794

Ok if that's a known issue, we should probably close this one.

Interesting. I don't think this is related to the ALPN.
I have a dart/flutter app that communicates with the server via gRPC. I believe dart client is the only one that I managed to get to work with gRPC over TLS without ALPN.
I too have started receiving Unimplemented errors the moment I switched to GRPCRoute from HTTPRoute.
For some reason, routes don't get matched to the method/service.
I tried creating a catch-all GRPCRoute that doesn't specify any matching rules and it received all the grpc requests sent to
the server. But when I dropped the catch-all, cilium/envoy simply couldn't find a matching backend.

I'm encountering the same issue trying to TLS-terminate a GRPC service, also not using any matching rules. Tested with both HTTPRoute and GRPCRoute, go-grpc reports the following errors:

Client TLS
HTTPRoute
GRPCRoute

disabled
rpc error: code = Unimplemented desc =
rpc error: code = Unimplemented desc =

enabled
rpc error: code = Unavailable desc = upstream connect error or disconnect/reset before headers. reset reason: protocol error
rpc error: code = Unavailable desc = upstream connect error or disconnect/reset before headers. reset reason: protocol error

The routes report as accepted, and when the client is using TLS, I can see the initial http-request and http-response flowing into and out of the server pod as the client connects. However, the client never seems to get a proper response, leading to the above errors. Cilium is configured with Gatway API ALPN (and host network) enabled:
$ cilium config view | grep -i gateway
egress-gateway-reconciliation-trigger-interval    1s
enable-gateway-api                                true
enable-gateway-api-alpn                           true
enable-gateway-api-app-protocol                   true
enable-gateway-api-proxy-protocol                 false
enable-gateway-api-secrets-sync                   true
gateway-api-hostnetwork-enabled                   true
gateway-api-hostnetwork-nodelabelselector
gateway-api-secrets-namespace                     cilium-secrets
gateway-api-service-externaltrafficpolicy         Cluster
gateway-api-xff-num-trusted-hops                  0


Timestamp bug for bot messages is one day later

The date.now() returns a date in UTC time as the Github workflow is based in UTC timezone. Update the workflow time where it accounts for the PST time.
For instance, Github actions workflow goes through. The day at 12AM in UTC time is 6/7/2023, which then this converts to closer to 6PM PST time at 6/6/2023

I did update the code earlier today, however, I am keeping this issue open until it actually is fixed tomorrow.

Resolved here


Base window configuration

We have minimize and maximize buttons in the base window. I think it would be better to hide them by default and also provide a possibility to configure it.

I wanna save system decorations. And in some linux like KDE have troubles with hide buttons and save sys decorations. Any ideas?

I see. I checked existing issues of the Avalonia project and I think we need to wait an update with issue fix.


Support for more complex operations on YAML config files, working off YACS

YACS Features:
https://detectron2.readthedocs.io/en/latest/_modules/yacs/config.html

YAML based config objects.
Mutable/Immutable flags.
Supports tree-hierarchy for objects - e.g., config -> config.model and config.training -> config.model.ehr_model.
Basic type checking through type interpretation - e.g., it fails when variables are specified as None.

TODO List:

Does it support config inheritance?
We need to try the yacs._load_cfg_from_file function to see if it can work with python files instead of yaml files for the ease of unit-testing and potentially inheritance.
Check if there are any other alternative libraries that one could use.

Idea for inheritance:
TEMPLATE: ../foo/bar.yml
SYSTEM:
  NUM_GPUS: 2
TRAIN:
  SCALES: (1, 2)

this should allow multiple levels of inheritance

templates could also be added for specific parts of the tree

Allow type checking on fields that are None by default. Could use params library, or do introspection to retrieve the type annotation


/en-us/ hardcord

Please remove /en-us/ for non_en-us local redirection purpose.

@bennage could you please check?

Unfortunately, the URL doesn't forward to a localized version. It has to be hard-coded. 🤒
You can test it here:
Works: https://www.microsoft.com/en-us/trustcenter/cloudservices/azure
Doesn't: https://www.microsoft.com/trustcenter/cloudservices/azure

hm, I checked de-de, fr-fr, ja-jp, it-it, loc version exists, but not es-es, zh-cn etc.
And yes, you are right, seems https://www.microsoft.com/trustcenter/cloudservices/azure doesn't work.
Will look into it further. Thanks!
Close now. @bennage


build: do not create WinSDK symlinks when unneeded

When building on case insensitive filesystems, there is no need to
create the library symlink forest as the paths will be resolved properly
due to the insensitivity.  This avoids a bit of work and spew on
Windows.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - a5d360ba7a2f80fa87e25b709afc66d08beb51cb

@swift-ci please test Linux platform


Add heat index calculation

We have some code to calculate the heat index (i.e. how the human body perceives temperature, which combines relative humidity with air temperature) that we can also add to the list of indices. It uses this formula.

This looks to be fairly simple to implement. Nice suggestion!

I have the code, just haven't pushed it yet :) We were wondering if the right place to put it is this.

I think it would go into _conversion.py. See the humidex index for a closely related index.

@raquel-ucl We are aiming to tag a release of xclim in the coming days. I just wanted to ping you to see if this is something that could be easily integrated before the end of the week. If no, not a problem; We can aim for the following release.
Thanks!

@Zeitsperre Raquel is out of the office for the next couple of weeks, so let's aim for the next release. Thanks for checking!

Hey there @clairbarnes and @raquel-ucl, just a reminder notification!

Hi, we'll be working on this again this week, so hopefully we can get all the remaining indices calculations submitted in the next few weeks and make it to 1.0. Watch this space!

I've looked into humindex and it is quite different from our heat_index calculation in that ours is just looking into what days are considered hot (default is > 20°C). Wouldn't it fit better _threshold.py instead? I've made an initial attempt in #915.


Limit Should Be Inclusive

As a user, I would expect the limit to be inclusive. A limit of 10 seconds should be able to count to 10, not to 9.
In your example you have a limit of 10 seconds and loop = true, but when the clock hits 9 seconds it resets.
The limit should be inclusive, and/or the looping logic should display the limit before reseting. (i.e. loop when greater than limit not equal to).

Will look into this.

https://github.com/slorenzo/react-stopwatch/commit/3916cf2741abee01be16b8323ce18aa3a44f36e4


Dynamic data in table

How would you render dynamic data you get from an external API and therefore changes frequently to render in the table?  In all the examples I see its hardcoded.  I've tried creating an array of objects and passing that into the Row components data prop but it doesn't accept it in that format.

Are you just changing a value in an object, rather than assigning it to the whole object?

can you get example ??


logo changed problm

I'm submitting a ...
[ ] bug report
[ ] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary

Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)

hi sir plz i am try to multiple time but all time failed plz change my logo my project not depend on my token logo
https://github.com/solana-labs/token-list/pull/23797
plz now don't reply merged pls help me for this problem solution,
you are expert in this job sir i know, otherwise i am lose my all investment sir 🙏

@spacemandev-git


Populate name field in exported span with http.target attribute

First of all, thanks for providing this library to support tracing in web apps! I am using it in conjunction with Jaeger for end to end traces.
This works great, but I would have one improvement for the exported spans.

The picture above shows an example trace in the jaeger ui starting from the web app. The ui shows the names of the individual spans
in the Service & Operations pane in the center left of the image.
For the web app, the span name is displayed as GET, while for traces from other (spring-boot) apps, the endpoint url (i.e http get /customers/{customerId}) is used.
Looking at the json response in jaeger reveals that the field is called operationName that determines the name.
Now, when looking at the request of the span exporter, the field that would correspond to this is likely the name field:

Now my question: Is it possible to configure the exporter to use for instance the tags.http.target attribute instead of the http method?
If not, could it be implemented?

Hi, @FieteO
OpenTelemetry's semantic convention for client-type spans is pretty clear on the subject. It should only be of type method, unless we are able to determine the url.template (which is not the case for most calls). The case of Spring Boot is different because it is server-type span.
For more information, I invite you to read this https://github.com/open-telemetry/semantic-conventions/blob/main/docs/http/http-spans.md#name

Ah, I think I see what you mean. The spec is saying: "HTTP span names SHOULD be {method} {target}".
{method} is what is currently exclusively being used by this library (http methods like GET or POST).
While {target} should be
- http.route for HTTP Server spans
- url.template for HTTP Client spans if enabled and available (Experimental)
- Other value MAY be provided through custom hooks at span start time or later.
Here, the client case applies and thus a url.template should be added as a {target} if it is available.
My change is adding a url.path (i.e /customers/mycustomer), while a template would be /customers/{customerId}.
And the latter is (without further changes?) not readily available and thus shouldn't be the default.
Even though this is against the spec, could we perhaps hide this implementation behind a configuration?
Because it would still provide benefit to me and I can imagine that having the proper url.template is quite impossible with the interceptor approach that is taken, isn't it?

Yes, that is indeed the problem, and you have forgotten this sentence: The instrumentation MUST NOT use the URI path as the default {target}.
It is therefore difficult for me to accept this change.
However, I understand your issue and the one with Jaeger, but despite all that, this doesn't seem possible at the moment.


Cache aggregated results

For the nighttime standby analysis, the first step is to compute the minimum consumption over each night for each sensor.  The current script does this every time again starting from the high-resolution data from tmpo.  This is highly inefficient, and probably also causes the RAM issues we have on the droplet.
I want to change this by splitting the functionality
A) get min consumption over night and store in a dataframe with daily index, for a single sensor.  This dataframe gets saved on disk
B) make any analysis you want based on these dataframes (like the current 7-day analysis, but also n evolution of standby power over the last year, etc.)
This approach can serve as a blueprint for other analysis that can be split in an A (getting some aggregated result and caching it) and B (doing an analysis with aggregated data that has been cached).
I start this in a new branch

The cache seems to work well now (already merged and published in release 0.3.4).   I fully agree with the idea of setting up a small db for these daily numbers, but we can already close this issue and discuss that in a separate one.


Regression: Proxy does not work in service mode

Major regression caused by numerous changes within the past two weeks. Didn't realize them during testing because the proxy in the containers (being used for tests) were never updated.
A few issues right now (there may be more):

When proxy calls NewLCAManager() with a serviceName, it calls hashlookup.GetHash(). This in turn calls SendRequest() from hl-common package, which creates a temporary p2pnode to query the lookup service for the hash associated with the service.

The temporary node currently cannot connect to anything. We need some way to pass bootstrap info and PSK to the temporary node

When that temporary p2pnode ends, a further regression issue introduced by Multi-Tier-Cloud/common#12 causes the node to never die.

An issue for this has already been opened in Multi-Tier-Cloud/hash-lookup#14

Issue 1 was primarily addressed in Multi-Tier-Cloud/hash-lookup#15, making the fix here for the proxy a trivial one line change.
Issue 2 was resolved in the Multi-Tier-Cloud/hash-lookup#14


Fix scroll on native when going back to the login page

Details

If the input field is focused when you go back to the login page on native, you are not scrolled to the top of the page. This differs from mweb, where you are scrolled to the top.
Fixed Issues

$ https://github.com/Expensify/App/issues/23718
PROPOSAL: https://github.com/Expensify/App/issues/23718#issuecomment-1665741165
Tests

Tests to make sure login is not broken:

Go to the login page
Tap on the phone/email input field
Enter your phone number or email address
Click on the button labeled Continue
Verify that the magic code input is shown as usual

Tests for fixes on native:
With focus

Go to the login page
Tap on the phone/email input field to open the keyboard
Scroll down a bit
Exit the app (either by clicking on a link or by clicking on the home button)
Reopen the app
Verify that you are scrolled back to the top of the login page

Without focus

Go to the login page
Make sure the phone/email input field is blurred (click outside the input field, the line under it should not be green)
Scroll down a bit
Exit the app (either by clicking on a link or by clicking on the home button)
Reopen the app
Verify that you are not scrolled back to the top of the login page

[X] Verify that no errors appear in the JS console

Offline tests

Same as regular tests
QA Steps

Same as regular tests

[X] Verify that no errors appear in the JS console

PR Author Checklist

[X] I linked the correct issue in the ### Fixed Issues section above
[X] I wrote clear testing steps that cover the changes made in this PR

[X] I added steps for local testing in the Tests section
[X] I added steps for the expected offline behavior in the Offline steps section
[X] I added steps for Staging and/or Production testing in the QA steps section
[X] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[X] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[X] I included screenshots or videos for tests on all platforms
[X] I ran the tests on all platforms & verified they passed on:

[X] Android / native
[X] Android / Chrome
[X] iOS / native
[X] iOS / Safari
[X] MacOS / Chrome / Safari
[X] MacOS / Desktop

[X] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[X] I followed proper code patterns (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[X] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[X] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I followed the guidelines as stated in the Review Guidelines
[X] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] I verified that if a function's arguments changed that all usages have also been updated correctly
[X] If a new component is created I verified that:

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[X] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[X] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[X] All JSX used for rendering exists in the render method
[X] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[X] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

https://github.com/Expensify/App/assets/138504087/39966539-973a-40b4-a8da-53c6d912ab70

Mobile Web - Chrome

https://github.com/Expensify/App/assets/138504087/fbe33e0c-cc12-41c6-84e5-239afea9cc05

Mobile Web - Safari

https://github.com/Expensify/App/assets/138504087/1abd37b8-6cb3-4ec9-8f6a-171c51b4cf35

Desktop

https://github.com/Expensify/App/assets/138504087/9cc450af-02b0-4e8f-9ed4-b24614b2c5cc

iOS

https://github.com/Expensify/App/assets/138504087/ffd342be-29d2-4fdc-a177-c2916491df2d
https://github.com/Expensify/App/assets/138504087/4bd5d807-b450-48cc-aaf9-5718249e0325
https://github.com/Expensify/App/assets/138504087/23019294-6a51-4767-8e3d-65288b5964f2

Android

https://github.com/Expensify/App/assets/138504087/bdfdc7d0-fa36-4849-a8c3-99842ca29bbf
https://github.com/Expensify/App/assets/138504087/d0c82b4b-9448-40d7-a98b-e6aa5aa6f386
https://github.com/Expensify/App/assets/138504087/dddf14fd-0f1d-4a78-88ae-83fca5da9ca0

Regarding the failing unit test check, there is a unit test that fails on main. I've written this on the PR that I think introduced this, and I have also reported it as a bug.

Hey team, do you mind pulling from main to fix the broken test?

Thanks for the quick fix! No unit tests are failing now, ready for review @mollfpr

I'm still struggling to run the android app in my local, and it's always failed to build the CMake. Any idea?

Hmm no I haven't run into that. If you want I can try to build an APK and send it to you. Alternatively there might be someone on Slack who knows how to solve it, seems like there are a few people who've run into CMake issues.

Updated code/responded to comment @mollfpr

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

Friendly ping @mollfpr, would be great if we can get this merged within 9 days of assignment

The code looks good and tests well. @joh42 Could you resolve the conflict? Thanks!

Sorry about that, resolved now @mollfpr

Welp... so while I did resolve the conflict, the commit wasn't signed, and I didn't notice until after I had pushed. I know you're not supposed to force push when a PR review has been started, but reverting won't remove the unsigned commit. Is it okay if I remove the unsigned commit and force push in this scenario @mollfpr?

While that did allow me to sign the commit locally, I'm still unable to push without using force since my local branch differs from the one on Github @mollfpr.
Since there are new conflicts, I think the cleanest solution might be to remove the unsigned conflict resolution commit and instead resolve all conflicts in a signed commit. But again, that would require a force push.

@joh42 Okay, let's open a new PR to get cleaner commit and seems everything looks good.


possible to switch language/locale at runtime?

It's not clear to me what would be required to switch language (locale?) at runtime, e.g. by offering a menu of supported languages in the top nav bar.
It looks to me like the locale is hard-coded into the settings file – how much work would it be to adapt that to an array of supported locales? I also noticed that Telescope/Nova may have used tap-i18n, which might have provided such a solution, but no longer. Is there something similar in the works using react-intl?
Thanks for any guidance you can provide.

The first step would be to re-enable react-intl, which is now disabled by default. After that I think it wouldn't be too hard. The part that controls the active language is the App.jsx component, so you should be able to replace that using replaceComponent.

As for how to control the language in practice, I'd probably recommend handling that through Redux. My Redux is a bit rusty so I generally just follow this model: http://docs.vulcanjs.org/redux.html


More typing fixes

This brings me from 246 errors -> 73 errors in the nerfstudio/ package.
Definitely duplicates some work from #1940, we should really coordinate 🙃

Yes, would definitely recommend collaborating with @jkulhanek on this. Maybe create a single PR fixing typing issues.

I am sorry @brentyi , I didn't know you were working on it. I guess you can finish this PR and I will check if there are any problems left which the other PR fixes.

I am sorry @brentyi , I didn't know you were working on it. I guess you can finish this PR and I will check if there are any problems left which the other PR fixes.

Oh yeah definitely no apology needed! I also only just started working on it this morning on a whim; just glad we have more people who care about the typer errors. 🙂
Overall it seems like the two PRs are pretty complementary, with some changes eerily similar. Hopefully the merge conflicts aren't too bad...


Image Calibration

Hi, I'm having some issues with calibrating my snapshots to generate the perception spreadsheet. I'm following this guide: https://devblogs.nvidia.com/calibration-translate-video-data/ and the one on the deepstream sdk guide.

"The first step in calibration is to get snapshot images from all cameras."
How do I get these dewarped snapshots and what are the resolutions needed?
And with QGIS, there's this line "Make a note of the longitude and latitude of the origin (in this case, the center of the building)."
Does this mean that I have to get the coordinate of the center of the referenced TIF image?
On the nvaisle/nvspot.csv files, what do ROI and the H values represent?
How do I find the gx,gy coordinates? I tried just importing the latitute and longitude but they don't work.

Thank you.

@velushomaz Hi!
Have you managed those issues?

@velushomaz @gisthere I'm also interested in coordinate transformation.Have you managed those issues?

@velushomaz @gisthere I'm also interested in coordinate transformation.Have you managed those issues?

@xhzzc1994 unfortunately not


ARROW-2337: Use Boost shared libraries in Windows release verification script. Parquet fixes

With these changes, the Windows verification script passes cleanly for me with 0.11.0 RC1

+1


Add version: LuaLS.lua-language-server version 3.2.2

Pull request has been created with Komac v1.9.1 :rocket:
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/116126&drop=dogfoodAlpha

@wingetbot waivers Add Validation-Executable-Error


Chart.2

Add chart to display population of a fictional zoo

I have modified the loadChartsApi function to match what I want in my master branch.


Windows Screen Snip screenshot fails to save as picture

Describe the bug
Inserting a picture from the Windows Screen snipping tool fails the Save As Picture feature.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error
Actual behavior
Describe what is actually happening as a result of the repro steps

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS Version : [e.g. Win10 20H1, Win11 21H2, etc.]
App Version [e.g. <IP_ADDRESS>]

Additional context
Add any other context about the problem here.

fixed in <IP_ADDRESS>


Cookiejar handling while crawling the same page after a ban

Hi there,
Thank you very much for such a good extension 👍
There is only one little detail that I miss. It appears that the cookiejar used in the request is not attached anymore to the new request - when the first request is blacklisted.
For example, I'm using a spider that parses 3 urls (one after the other). It needs to keep its cookies set up at the first URL so that data is right at the very last URL.
I got wrong results in my items each time there is a ban during one of the 3 urls parsed.
I'll dig a bit more in your extension - trying to improve it by myself - but I may not be efficient enough yet with scrapy !
What do you think of this?

All right, the rotating proxy is handling cookiejar with copy() request function which is also copying meta values (cookiejar index included).
By enabling the COOKIES_DEBUG setting, I was able to confirm the right cookies were sent.
Sorry for this wrong issue.


halfvec error - ValueError: could not convert string to float: ''

Hi! I keep getting ValueError: could not convert string to float: '' due to the values in my halfvec column being malformed.
This is what my halfvec column looks like:
{"[0.011108398,-0.11376953,-0.044677734,-0.23632812, ...]"}

vector_embedding: Mapped[list[np.array]] = mapped_column(ARRAY(HALFVEC(VECT_DIM)))

I also made sure to convert each vector using np.array before upsert
vector_embedding = [np.array(e) for e in embeddings]
Interestingly, the other field vector_embedding: Mapped[np.array] = mapped_column(HALFVEC(VECT_DIM)) in a separate table works just fine. Am I missing something when using array halfvec?

Hi @frieda-huang, for arrays with SQLAlchemy, you'll need to call register_vector on the underlying adapter. For Psycopg 2, use:
from pgvector.psycopg2 import register_vector

with engine.connect() as connection:
    register_vector(connection.connection.dbapi_connection, globally=True, arrays=True)

Added a test case in the commit above.

Hi @frieda-huang, for arrays with SQLAlchemy, you'll need to call register_vector on the underlying adapter. For Psycopg 2, use:
from pgvector.psycopg2 import register_vector

with engine.connect() as connection:
    register_vector(connection.connection.dbapi_connection, globally=True, arrays=True)

Added a test case in the commit above.

Thank you for the quick reply! I'm still getting the same error despite calling register_vector_async. I'm using psycopg3:
async def add(self, vector_embedding: List[npt.NDArray], page: Page) -> Embedding:
  conn = await psycopg.AsyncConnection.connect(dbname=DBNAME, autocommit=True)
    async with conn:
        await register_vector_async(conn)
        embedding = Embedding(
            vector_embedding=vector_embedding,
            page=page,
            last_modified=get_now(),
            created_at=get_now(),
        )
        self.session.add(embedding)
        return embedding

For Psycopg 3, you'll need to call it on the connection used for the session (rather than a new connection).

The best way to do this would be to use the connect event right after you define the engine. For Psycopg 3:
from pgvector.psycopg import register_vector
from sqlalchemy import engine

@event.listens_for(engine, "connect")
def connect(dbapi_connection, connection_record):
    register_vector(dbapi_connection)

The best way to do this would be to use the connect event right after you define the engine. For Psycopg 3:
from pgvector.psycopg import register_vector
from sqlalchemy import event

@event.listens_for(engine, "connect")
def connect(dbapi_connection, connection_record):
    register_vector(dbapi_connection)

Is there a way we can do it using async? I'm using it along with FastAPI. Having tried multiple approaches including adding the register_vector logic in FastAPI's lifespan, still got the same error :(

If you're using create_async_engine, you'll want to use:
from pgvector.psycopg import register_vector_async

@event.listens_for(engine.sync_engine, "connect")
def connect(dbapi_connection, connection_record):
    dbapi_connection.run_async(register_vector_async)


please reenable hOpenPGP and hopenpgp-tools

Looks like https://github.com/fpco/stackage/commit/0a373104c99522bbc3fbd4e8e8712d6e9d996e76 disabled hOpenPGP and hopenpgp-tools, but ixset-typed is fixed.

safecopy also needs to be fixed:
template-haskell-<IP_ADDRESS> is out of bounds for:

[ ] safecopy-<IP_ADDRESS> (< 2.12). No maintainer. @acid-state. Used by: library, test-suite

time-<IP_ADDRESS> is out of bounds for:

[ ] safecopy-<IP_ADDRESS> (< 1.7). No maintainer. @acid-state. Used by: library, test-suite

Please open a PR when it gets fixed to expedite the process.

safecopy appears to have been fixed


Prune unnecessary OCS resources

Prune unnecessary OCS resources that are not needed by the ManagedMCG
addon, namely:

rook-ceph-operator
ocs-operator
ocs-metrics-exporter

@rexagod Danny mentioned that OB and OBC are dependent on OCS,
Could you please check we are able to create OB and OBC even after downsizing OCS deployments.

If we need OCS, then what's the point of all the Noobaa standalone effort that's happening? I made this PR with that perspective in mind, because if we are including OCS for OB/OBCs, then this PR makes no sense.

Yes @rexagod , That is something we need to discuss, I just wanted to know whether removing OCS related deployment will break our addon or not

// @v-harihar

Closing this since we'll be removing ODF altogether, so no need to remove OCS components individually.
Will reopen if need be.

Reopening, we might still need this (in case abstracting OB/OBCs does not work out).

LGTM


Nativescript 7.x compatibility - Markers not showing

Hello,
I've created a PR which is built around the new nativescript libraries (@nativescript/x).
Actually this has been tested on Android (emulator), a test on iOS would be nice in case the changes will be approved.
PR #420
@dapriett can you check it? thanks :) (Please read the last message in the PR, there's one major problem that must be fixed prior to merging)

Hi @funder7 . Is this plugin been maintained at all? I see lots of open issues with out a response. I noticed the marker is no longer showing also. I just posted one issue a few days ago. I am trying to build ios with xCode 10.1.0 but it fails.
Have you been able to do local builds?
Thanks,
Alexis
Undefined symbols for architecture i386:
"___isPlatformVersionAtLeast", referenced from:
-[GMSCellularInfo SIMCountryCode] in GoogleMaps
-[GMSScreenInfo initWithUIScreen:] in GoogleMaps
-[GMSx_QTMCollectionViewController viewDidLoad] in GoogleMaps
-[GMSx_QTMCollectionViewController viewDidLayoutSubviews] in GoogleMaps
-[GMSx_QTMCollectionViewStyleController contentViewInsetsForCellWithAttribute:] in GoogleMaps
-[GMSx_QTMCollectionViewStyleController setInsetsContentViewsToSafeArea:] in GoogleMaps
-[GMSx_QTMCollectionViewStyleController insetsAtSectionIndex:] in GoogleMaps
...
ld: symbol(s) not found for architecture i386
clang: error: linker command failed with exit code 1 (use -v to see invocation)
note: Using new build systemnote: Planning buildnote: Constructing build description
** BUILD FAILED **
Command xcodebuild failed with exit code 65

Hi, @alexisconsuegra, I don't know what are the author's intentions about this, I'm waiting for a reply too.. About your problem, let's continue the discussion in your issue!

Same problem here, markers not showing. Any workaround?

I have the fix, I'll push it tomorrow

Thank you @funder7, I can't wait to try your fix.

Hi @flocca sorry but I had some delay, I'll push the changes this weekend hopefully.

after upgrading to 7.x map is not showing map at all, it was working in 6.x

Hello @ubaidazad thanks for reporting it. I will definitively check it this weekend to see if I can make it be compatible with the latest {N} version.

any progress here?

apparently not...

Sorry guys I'm doing this in my free time, which is inexistent in the last weeks... Probably I'll meet with other people during the next weekend to complete the update.

thank you. we apreciate. there is a PR that looks like it fixes a lot of compat issues... when you have time...
thanks a lot

after upgrading to 7.x MapView is not showing map at all, it was working in 6.x

On which platform?
Can you try to run on the other one as well and check if is not showing as well? Thanks

thank you. we apreciate. there is a PR that looks like it fixes a lot of compat issues... when you have time...
thanks a lot

No worries, I wish I had the time to close it now as it doesn't look like there's much to end the update.
Which PR are you referring to?

On which platform?
Can you try to run on the other one as well and check if is not showing as well? Thanks

@funder7
platform: Android Device
What other one are you referring to?

Just a note:
in Android with ns 6.8.0 and tns-android 6.5.3 the Map is not displayed at all, while in iOS it seems ok...


fix: 避免regional network-list --zone导致返回结果为空

这个 PR 实现什么功能/修复什么问题:
避免regional network-list --zone导致返回结果为空
是否需要 backport 到之前的 release 分支:

release/3.2

/area region
/cc @zexi

Codecov Report

Merging #6528 into master will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #6528      +/-   ##
==========================================
- Coverage    7.17%    7.17%   -0.01%
==========================================
  Files         647      647
  Lines      108977   108998      +21
==========================================
- Hits         7819     7817       -2
- Misses     100558   100580      +22
- Partials      600      601       +1

Flag
Coverage Δ

#aFlag
7.17% <0.00%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/compute/models/wireresource.go
0.00% <0.00%> (ø)

pkg/appsrv/response.go
66.17% <0.00%> (-2.95%)
:arrow_down:

pkg/cloudcommon/validators/validators_cert.go
39.77% <0.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 daf9db8...8bc4927. Read the comment docs.


Azure Storage Explorer failed to connect to Microsoft Fabric Lakehouse

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Storage Explorer Version
1.30.0
Regression From
No response
Architecture
x64
Storage Explorer Build Number
20230609.2
Platform
Windows
OS Version
Windows 10 pro
Bug Description
AzureStorageExplorerAttemptToConnectToMicrosoftFabricLakehouse.txt
I have followed.instructions : https://learn.microsoft.com/en-us/fabric/onelake/onelake-azure-storage-explorer
url used to connect to gen2 storage:
https://onelake.blob.fabric.microsoft.com/fabricspace/firstLakehouse.Lakehouse/

Steps to Reproduce

Get Latest version of Azure Storage Explorer
Follow instructions as per https://learn.microsoft.com/en-us/fabric/onelake/onelake-azure-storage-explorer
Try to open the storage or try to see properties
Error shown,

Actual Experience

Above attachment is showing error message
AzureStorageExplorerAttemptToConnectToMicrosoftFabricLakehouse.txt
Above attachment is long json which is stored in Error Detailed message, when tried to see properties for this storage connection.
Expected Experience
Storage explorer should be able to show Tables and Files that are on Microsoft Fabric workspace lakehouse.
Additional Context
No response

@proanotiDe for now, can you please reach out to support for this issue? Information on how to get support for Microsoft Fabric can be found here: https://learn.microsoft.com/en-us/power-bi/support/service-support-options#get-started-with-support


Wiimote support?

I have wiimote and i am able to connect them to Yuzu via WiimoteHook. Could you add support to wiimote?
Motion server is on port <IP_ADDRESS>:26760

Duplicate of #21.


⚠ gorzdrav.org has degraded performance

In d11efd8, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 17695 ms

Resolved: gorzdrav.org performance has improved in d27692c after 28 minutes.


:validator=>false ignored on f.select()?

can I have more info?

I'm using these gems to get the i18n patch:
gem 'client_side_validations', '~> 3.2.0.beta.1'
gem 'client_side_validations-mongoid', '~> 2.4.0.beta.2'
mongoid model:
validates :subject, :inclusion => {:in => %w(earth life phys)}
Form helper:
<%= f.select(:subject, [['physical', 'phys'], ['earth'],

['life']], :prompt => act.new?, :validate => false) %>
CSV injection:

Another oddity is that act[status] doesn't block form submission even
though it fails inclusion validation. Must be because I'm using radio
buttons for it. I didn't set :validate=>false but I've verified
:validate=>false works with radio buttons.
Since I need patches for i18n, redundant callbacks, and this, which
version should I get?
On Thu, Jan 26, 2012 at 6:29 PM, Brian Cardarella
<EMAIL_ADDRESS>wrote:

can I have more info?

Reply to this email directly or view it on GitHub:
https://github.com/bcardarella/client_side_validations/issues/264#issuecomment-3680982

Can you try using HEAD instead of the gem?
gem :client_side_validations, :git => 'git://github.com/bcardarella/client_side_validations.git'

For both
client_side_validations
and
client_side_validations-mongoid?
On Thu, Jan 26, 2012 at 6:46 PM, Brian Cardarella
<EMAIL_ADDRESS>wrote:

gem :client_side_validations, :git => 'git://github.com/bcardarella/client_side_validations.git'

no, just client_side_validations
On Thu, Jan 26, 2012 at 9:50 PM, Ken Keller
<EMAIL_ADDRESS>wrote:

For both
client_side_validations
and
client_side_validations-mongoid?
On Thu, Jan 26, 2012 at 6:46 PM, Brian Cardarella
<EMAIL_ADDRESS>wrote:

gem :client_side_validations, :git => 'git://github.com/bcardarella/client_side_validations.git'

Reply to this email directly or view it on GitHub:
https://github.com/bcardarella/client_side_validations/issues/264#issuecomment-3681155

fixed
redundant callbacks
inclusion validator
not fixed
f.select(..., :validate => false)
I noticed some chatter about numericality validation when input field
is blank. mongoid sets the number to nil in this case. It is a
convenient use case because otherwise the model needs another field:
qty_known: Boolean
qty: Integer
and the corresponding form would look odd:
Quantity Known: [unchecked box]
Quantity: [   ]
or the 2nd field would be hidden unless the first was checked.
A purist wouldn't map a persistent model to a form directly
anyway--rather a "UI model" would be introduced. This doesn't appear
to be the Rails Way.
On Thu, Jan 26, 2012 at 6:52 PM, Brian Cardarella
<EMAIL_ADDRESS>wrote:

no, just client_side_validations
On Thu, Jan 26, 2012 at 9:50 PM, Ken Keller
<EMAIL_ADDRESS>wrote:

For both
client_side_validations
and
client_side_validations-mongoid?
On Thu, Jan 26, 2012 at 6:46 PM, Brian Cardarella
<EMAIL_ADDRESS>wrote:

gem :client_side_validations, :git => 'git://github.com/bcardarella/client_side_validations.git'

Reply to this email directly or view it on GitHub:
https://github.com/bcardarella/client_side_validations/issues/264#issuecomment-3681155

Reply to this email directly or view it on GitHub:
https://github.com/bcardarella/client_side_validations/issues/264#issuecomment-3681166

Should I continue w/ this?
gem "client_side_validations", :git => 'git://
github.com/bcardarella/client_side_validations.git'
gem 'client_side_validations-mongoid', '~> 2.4.0.beta.2'
On Thu, Jan 26, 2012 at 9:20 PM, ken keller<EMAIL_ADDRESS>wrote:

fixed
redundant callbacks
inclusion validator
not fixed
f.select(..., :validate => false)
I noticed some chatter about numericality validation when input field
is blank. mongoid sets the number to nil in this case. It is a
convenient use case because otherwise the model needs another field:
qty_known: Boolean
qty: Integer
and the corresponding form would look odd:
Quantity Known: [unchecked box]
Quantity: [   ]
or the 2nd field would be hidden unless the first was checked.
A purist wouldn't map a persistent model to a form directly
anyway--rather a "UI model" would be introduced. This doesn't appear
to be the Rails Way.
On Thu, Jan 26, 2012 at 6:52 PM, Brian Cardarella
<
<EMAIL_ADDRESS>
wrote:

no, just client_side_validations
On Thu, Jan 26, 2012 at 9:50 PM, Ken Keller
<EMAIL_ADDRESS>wrote:

For both
client_side_validations
and
client_side_validations-mongoid?
On Thu, Jan 26, 2012 at 6:46 PM, Brian Cardarella
<EMAIL_ADDRESS>wrote:

gem :client_side_validations, :git => 'git://
github.com/bcardarella/client_side_validations.git'

Reply to this email directly or view it on GitHub:

https://github.com/bcardarella/client_side_validations/issues/264#issuecomment-3681155

Reply to this email directly or view it on GitHub:

https://github.com/bcardarella/client_side_validations/issues/264#issuecomment-3681166


[C++][Parquet] Append mode in parquet-cpp

As said, currently trying to work out a append feature for parquet files in c++.
(been searching through repo etc, can't find example tho..)
Current solution is to (assume no schema changes that is):
Read in metadata
Change metadata based on appended rows+ original rows
Append a new row group (or multiple row group writer)
Write the new rows.
—
The problem is that, is approached this way, the original last row group may not be complete filled. Was wondering if there is a fix or I'm using the api wrong...
Thanks ! : D
Reporter: yugu / @elderRex
Assignee: Wes McKinney](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=wesm) / @wesm
Note: This issue was originally created as PARQUET-1022. Please see the migration documentation for further details.

Uwe Korn / @xhochy:
What do you mean with "the original last rowgroup may not be complete"? Each RowGroup has a number of rows specified in its metadata and exactly this number of rows must be in that RowGroup once the file is written to disk, otherwise we end up with invalid files.
Concerning the general approach, writing a Parquet would initially produce (we take here 2 RowGroups as an example) of the pattern MRRF where M are the 4 magic bytes PAR1, R is a RowGroup each and F is the footer including the metadata.
To make an append mode, you would then read F from the existing file, insert a new RowGroup at the place where currently F is and then write out the new footer with its modified metadata F'.
Still a small note of caution: Normally appends to Parquet datasets are simply done by creating additional Parquet files. Most tools that ingest Parquet are built such that they treat multiple files and a single Parquet file nearly identically. Code-wise this should be much easier for you to handle. But if this append mode is really helpful for you, it would also be really helpful for us to understand why you would need it instead of just using the new file mode!

Wes McKinney / @wesm:
I'm closing as Won't Fix. Trying to modify existing files (overwriting the existing file footer) is a pretty big can of worms, and would add a bunch of complication to the codebase to initialize various classes with a partially written file

Tham / @thamht4190:
I do think so. I'll try merging files as your suggestion. Thanks @xhochy  a lot.

Tham / @thamht4190:
Our application would like to have this feature. Our use case is when the application crashes (it's true we cannot develop a perfect system), so we cannot close the file that is opening at that time. We want to open, close multiple times, then we just need to open when we want to write data, then close the file. With that way, we can reduce the risks to lose data. Any suggestion?

Uwe Korn / @xhochy:
There is no implementation of merging concatenating files in C++ yet but that would be much easier to implement than an Append mode. For merging files you would read the footer of all files, copy the binary content of the RowGroups into the new file and then compute a new footer from the information of existing footer. From what I can think of currently, this should not require decoding any RowGroups thus implementing an explicit merge will be a lot faster then materialising the data first and writing a fully new file.

Wes McKinney / @wesm:
I think theoretically you can write row groups to disk separate from metadata. Not sure if there are many systems out there that implement this

Uwe Korn / @xhochy:
@thamht4190  The solution here is to write more files and combine them afterwards. Reading Parquet files that have no footer is not possible. Even with the possibility of modifying an existing file, you would loose all data in your crash scenario as in the second write, the data would be written over the footer. While no new footer is written, the data in a file cannot be restored.

Nileema Shingte:
@xhochy @wesm - Ability to concat the parquet files is something we've wanted for some time too. When we generate parquet files partitioned by an expression, we often end up with tiny files and would like to add a post-processing step to concat these files together.
Is there a plan to add this ability to the library any time soon?
If not, it would be great if someone can provide a somewhat detailed pseudocode (expanding on what @xhochy) mentioned in the comment above) as a guideline for conditions/scenarios that need to be handled with extra care, so we can contribute this as a PR.

Tham / @thamht4190:
Thanks for your suggestion @xhochy @wesm. I'm thinking between

writing multiple files and merge it into a new single file later (but it seems to take long time to merge)

or write row groups separately from metadata and concat into a single file later.
Is there any feature of merging/concating files on C++?

Tham / @thamht4190:
Uhm, after looking into C++ API, I think it requires a lot of work to either concat multiple files by binary blocks as @xhochy suggested or write data and metadata separately as @wesm suggested. It seems that current implementation is not designed for this purpose and seems tricky to do so.
I also saw that the commit of merge-tool in Java has recently been reverted https://github.com/apache/parquet-mr/commit/ab42fe5180366120336fb3f8b9e6540aadb5da1b :(
Correct me if I'm wrong. Any idea is welcome.


Improve reset attribute error message & docs

The previous exception message was wrong and confusing. This PR improves this message and the corresponding documentation.

Pull Request Test Coverage Report for Build 540237892

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.069%

Totals

Change from base Build 537663247:
0.0%

Covered Lines:
2522

Relevant Lines:
2681

💛  - Coveralls


Affiliate key invalid even though it is valid

I started to get error Invalid Affiliate key when i start the payment.
Now using sdk v3.1.2. and tried on Android 6.0 and 8.1

Hi Tomde,
Thank you for your message and sorry for the late reply. Would you be able to share with us at<EMAIL_ADDRESS>a bit more on your issue and how did you get your affiliate key?
Many thanks,
Jullian

Hi,
I experiment the same issue on the developper account, trying to generate a new  affiliate key, but only the one provided by Android SDK works fine.
Does it mean that  affiliate key generation using the developper account is not valid ?
Thanks in advance,
BR,
Philippe

Does sumup still supports SDK and app integration? Probably not<EMAIL_ADDRESS>does not give support anymore. No reaction.


fix: cached data bug in atom with wrong network

Try out this version of the Hiro Wallet - download extension builds.

Closes #2645
Not entirely sure what was going on here, but we were using onSuccess of react query to populate an atom. As this atom wasn't used by any other atoms, we can remove it entirely, and consume only the hook.  This appears to have solved the problem.
cc/ @radicleart, would you be able to download the build from this PR and ensure it's fixed? This'll help us merge and release this more quickly

I downloaded the extension (refs/pull/2647/merge) and re-tested with this wallet. The bug appears to be fixed as the wallet did not show contract not found. I waited for about 30s and then fired the Tx off - its running on testnet now, as expected.

@fbwoolf to review & merge, could we add this fix to the release PR?


added a new card

Hi @IndraniSom ,
Thank you for your PR. However, I'd like to clarify a few points:

We follow a specific design system and create components internally. We're not accepting new components at the moment.

Sending a PR without raising an issue first goes against our contribution guidelines. Please check the CONTRIBUTING.md for our contribution process and guidelines.

The README contains a link to our Figma design system, which you should reference for design consistency.

We appreciate your enthusiasm, but for new component ideas, please discuss them in the Issues section first. Thanks for understanding.

Ok I actually haven't read the readme file properly from next time before sending the pr I will keep it in mind


LinkedIn redirect url does not match the actual value

Describe the bug
LinkedIn redirect url does not match the actual value. In my case, I the value from the sign in method settings is
https://<subdomain>.auth.us-east-1.nhost.run/v1/signin/provider/linkedin/callback

but the actual uri passed when trying to sign in is
https://<subdomain>.nhost.run/v1/auth/signin/provider/linkedin/callback

fixed with https://github.com/nhost/nhost/issues/1386#issuecomment-1352677065


Logo Implementation TODO

Below is a list of things that still need to be implemented into the parser for the logo language:

[ ] circle > circle radius extend steps
[ ] dot > dot radius
[x] pensize > pensize strokeSize
[ ] pencolor > pencolor color
[ ] fillcolor > fillcolor color note: this is dealing with custom shapes which we currently don't support yet
[ ] fill > fill true/false note: uses fillcolor to fill the shape if true
[ ] begin_fill > start shape
[ ] end_fill > end shape
[ ] write > write text note: this writes to the canvas as text ex "hello world"

Things to implement in turtle to add more features:

[ ] Dynamic stroke size so we can change it with pensize
[ ] Ability to create shapes and ill them with a solid color

--
Note: In Logo color sets both fillcolor and pencolor, this means we don't actually implement color properly yet

Is this up to date?

Yes this is currently up to date, we still have things we need to implement that are in this list
Sent from my iPhone
On Nov 13, 2018, at 6:35 AM, Daniel Shiffman <notifications<EMAIL_ADDRESS>wrote:
Is this up to date?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/CodingTrain/Logo/issues/37#issuecomment-438287401, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABZ1YBvvwHldND2gf5fWga-NRq_hM4Eqks5uutiigaJpZM4YXtnT.

Circle mode center or corner`??

@Toyz Hi for #75  do you know where the circle command is documented?
Looking in the Berkeley Logo Manual I can only find the ARC command:
ARC angle radius

	draws an arc of a circle, with the turtle at the center, with the
	specified radius, starting at the turtle's heading and extending
	clockwise through the specified angle.  The turtle does not move.

Should we implement a circle command because it's intuitive? or just support Berkeley Logo (which has only ARK or both?

For color, should it be rgb, or should it be like this:
command 51 (rgb: 51,51,51)

Circle could he a shortcut to arc

And we could change the color command to support rgb instead of having a colorrgb function

So we should replace the color command with colorRGB or just make it support both hex and RGB?


My Synology Is Overwhelmed

Hoping you can help me with settings to keep from overwhelming my Synology DS920+.  Running crashplan-pro on Docker, two of my disks are almost constantly near 98-99% Drive Utilization and Drive Read Latency is often 2-3 seconds.  Since I have almost 2M files I'm backing up (mostly already backed up and static), I increased the Inotify Watch Limit to around 1.9M.  FYI, Synology blames me running 12G RAM ... but since I stopped the contained running crashplan-pro, Drive Latency has dropped to 50-250 ms.  If there anything I can do to fix this or am I doomed to overload my Synology?  Thanks.

What CrashPlan is doing during that time ?  The initial file scan/sync can use a lot of ressources, but this should be temporary.

I have the same issue. Since 20 days the container has near 98-99% Drive Utilization. I can not login into consol of the  container. Code42 says "no activity since 50 days" :-(

Make sure the container is up to date with the latest pull.  I haven't seen any of these issues with my QNAP NAS docker container with a 4TB backup.

I install the Conainer new befor is start the current test.

It is hard tu access my Synology. It is a DS415+ 2GB RAM and i try to backup 3TB

some additional information. It appears that the container keeps restarting while it is scanning the file information.
In the Log i found this.
02/11/2021 15:09:12 idle keyboard:   turning X autorepeat back on.
[CrashPlanEngine] starting...
02/11/2021 15:36:24 active keyboard: waiting until all keys are up. key_down=37 Control_L.  If the key is inaccessible via keyboard, consider 'x11vnc -R clear_all'
02/11/2021 15:36:26 active keyboard: turning X autorepeat off.
02/11/2021 15:41:25 idle keyboard:   turning X autorepeat back on.
[CrashPlanEngine] starting...
02/11/2021 16:03:10 active keyboard: waiting until all keys are up. key_down=37 Control_L.  If the key is inaccessible via keyboard, consider 'x11vnc -R clear_all'

Alsoin the "log" Folder are many files like "restart.2021-11-02_15.56.30.log" Insite I see
Tue Nov  2 15:56:32 UTC 2021 : /usr/local/crashplan/bin/restartLinux.sh
Tue Nov  2 15:56:32 UTC 2021 : Stopping CrashPlan Engine ...
Tue Nov  2 15:56:54 UTC 2021 : Still running, killing PID=11303 ...
Tue Nov  2 15:56:56 UTC 2021 : OK
Tue Nov  2 15:56:56 UTC 2021 : Waiting for CrashPlan Engine to be restarted automatically ...
Tue Nov  2 15:56:59 UTC 2021 : New CrashPlan Engine process started with PID 12454
Tue Nov  2 15:56:59 UTC 2021 : Exiting restart script


Prepare for property-based testing of Matrix4.inverse

Add utils needed for property-based testing of Matrix4.inverse & co.

Got it.


combinded some ideas within the README file to enhance.  We will need…

… to work together to make it read and flow better.

Sounds like a plan!


Crushing on Meteor <IP_ADDRESS> Error: spawn ENOENT

I've tested both in my project and created new application, the results are as below:
W20150111-14:33:44.461(1)? (STDERR) events.js:72
W20150111-14:33:44.462(1)? (STDERR)         throw er; // Unhandled 'error' event
W20150111-14:33:44.462(1)? (STDERR)               ^
W20150111-14:33:44.463(1)? (STDERR) Error: spawn ENOENT
W20150111-14:33:44.463(1)? (STDERR)     at errnoException (child_process.js:1001:11)
W20150111-14:33:44.463(1)? (STDERR)     at Process.ChildProcess._handle.onexit (child_process.js:792:34)
meteor list
autopublish      1.0.2  Publish the entire database to all clients
insecure         1.0.2  Allow all database writes by default
meteor-platform  1.2.1  Include a standard set of Meteor packages in your app
mike:mocha       0.5.1  Run mocha tests in the browser
velocity:core    0.4.5  Velocity, a Meteor specific test-runner
The code is brand new application created with meteor create app

Same problem here.
Clean install of meteor <IP_ADDRESS>
meteor list
accounts-password  1.0.6  Password support for accounts
insecure           1.0.2  Allow all database writes by default
iron:router        0.9.4* Routing specifically designed for Meteor
less               1.0.12  The dynamic stylesheet language
meteor-platform    1.2.1  Include a standard set of Meteor packages in your app
mike:mocha         0.5.1  Run mocha tests in the browser
Running on Lubuntu 14.10.

I just ran into the same issue. added it to sample todos app. running on Ubuntu 14.04

I'm also using Ubuntu 14.04

I fixed this.  Sorry, away from my notes but google "ENOENT nvm".   I ended up fixing it by installing nvm (node version manager), then running "nvm install stable", "nvm use stable"  and then running meteor. This ensures that you are running the latest stable version of node with meteor.

@taji  nvm works for me, thanks


[BUG] Fix incorrect plugin nightly deployment and release [skip ci]

Signed-off-by: Peixin Li<EMAIL_ADDRESS>follow-up of https://github.com/NVIDIA/spark-rapids/pull/5249
fix https://github.com/NVIDIA/spark-rapids/issues/5259
current setup will deploy empty jar to artifact repo.
I manually deployed the jar w/ classifier for adjusting downstream pipelines setup (compatibility to support 22.06+ plugin only,  and elder versions w/ cudfjni)

also remove a version consistency check error

build

build


Preprocess spectra before adding to index

I did some coding on my flight to Japan and moved the preprocessing code from Casanovo to depthcharge to fix Casanovo#56. The upshot is that invalid spectra are now no longer included in the index on the depthcharge side, rather than later being replaced by a dummy spectrum by Casanovo that will get an incorrect prediction.
This needs some unit tests, but I'm unlikely to be able to add those within the next two weeks while I'm in Japan.
One remaining question is whether we also want to introduce a parameter that specifies the minimum number of peaks (after preprocessing) for spectra to be included?
Linked to Casanovo#181.

I actually had an idea to implement this better. Instead of just moving all of the preprocessing options from Casanovo to depthcharge, maybe it would be cleaner if the preprocessing happens on the depthcharge side, but it is done by giving it a callable with a specific signature (i.e. raw spectrum in, processed spectrum out, ValueError to discard the spectrum). That would make the preprocessing in depthcharge much more general, with each application that can fully define preprocessing based on its preferences. And it still solves the initial problem.
What do you think @wfondrie? The initial coding was done at the tail end of a 14h flight and I thought of this alternative while only sleeping 3 hours last night due to jet lag, so ymmv.

Nice! I think this is a good start. One of the things to consider is making even more flexible. Instead of having a fixed set of transformations, we could execute arbitrary functions, kind of like I setup for the Dataset classes:
https://github.com/wfondrie/depthcharge/blob/25d617f5ad421a5c0dcefd19628449293b414da3/depthcharge/data/datasets.py#L23-L26
If we did do that, I would think we'd actually want to move the preprocessing_fn to the parsing side of things, rather than as something the dataset does. The only reason I made this design choice originally is so that someone could re-use the same index and try out multiple preprocessing functions, obviously with the runtime cost of preprocessing. I'm open to arguments to the latter though!

Indeed, that was my proposal as well. I forgot that there's a preprocessing_fn in the dataloader already. Why did we reimplement basically the same thing in Casanovo then instead of using preprocessing_fn from depthcharge?
I think that moving preprocessing_fn to the parsers is a good solution. I agree that conceptually you might not want to recreate the index every time a simple preprocessing option changes, but in practice I doubt that's going to happen very often. (We don't change it in Casanovo, and for ANN-SoLo I've noticed the same and I'm making a similar change.) Whereas this change would avoid the issue with not being able to skip invalid items in a dataloader. So the upside outweighs the potential negative aspect imo.

Why did we reimplement basically the same thing in Casanovo then instead of using preprocessing_fn from depthcharge?

To be honest, I don't know 🤷♂

I think that moving preprocessing_fn to the parsers is a good solution. I agree that conceptually you might not want to recreate the index every time a simple preprocessing option changes, but in practice I doubt that's going to happen very often. (We don't change it in Casanovo, and for ANN-SoLo I've noticed the same and I'm making a similar change.) Whereas this change would avoid the issue with not being able to skip invalid items in a dataloader. So the upside outweighs the potential negative aspect imo.

I'm fine with this change - I went back and forth debating with my original implementation anyway.

Superseded by #34.


Please publish the ST_AsMVT new signature to npm

I'm trying to use this library with npm but the signature is wrong, can you update the npm package?
thanks.

See https://github.com/Geodan/postgis-mvt-server/commit/2e87d58c3141b1f6ad66acec59670eb9ad8ce695
Now q is the first parameter

@lamuertepeluda yes, I aware thats was fixed, but in npm package its still with the old signature. Im asking to @Stezii publish this new version there.
https://www.npmjs.com/package/tilestrata-postgismvt

For now, I'm installing direct from github.
Heres my package.json
{
 "devDependencies": {
    "tilestrata": "^2.1.2",
    "tilestrata-postgismvt": "github:Stezii/tilestrata-postgismvt"
  }
}