Support for YARP-based reverse proxy to use as ingress

In Tye we could define ingress nodes to route requests to the different services. From this comment, it seems that the idea is to leverage YARP.
It would be cool if we could have something built-in that could let me translate
ingress:
  - name: admin
    tags:
      - admin
    bindings:
      - port: 12000
        protocol: http
        name: http
    rules:
      - service: api
        path: /api/
        preservePath: false
      - service: admin-web
        path: /
        preservePath: true
  - name: app
    tags:
      - app
    bindings:
      - port: 12100
        protocol: http
        name: http
    rules:
      - service: api
        path: /api/
        preservePath: false
      - service: app-web
        path: /
        preservePath: true

into
var database = builder.AddPostgresContainer("database")
    .WithVolumeMount("../../data/database/data", "/var/lib/postgresql/data");

var apiService = builder.AddProject<Projects.API>("api")
    .WithReference(database);

var webAdmin = builder.AddProject<Projects.Admin_Web>("admin")
    .WithReference(apiService);

var webApp = builder.AddProject<Projects.App_Web>("app")
    .WithReference(apiService);

builder.AddIngress("app")
    .WithPath("/api/", apiService, preservePath: false)
    .WithPath("/", webApp, preservePath: true)
    .WithBinding(12100, "http");

builder.AddIngress("admin")
    .WithPath("/api/", apiService, preservePath: false)
    .WithPath("/", webAdmin, preservePath: true)
    .WithBinding(12000, "http");

builder.Build().Run();

I don’t think we want to build ingress into the app model. Given that we have YARP now, just make it one of your projects and configure it there with the service discovery extensibility. Take a look at eshop lite for an example.

I'm not exactly sure  how everything works in Aspire yet, but seems like it would fit nicelly into the AppHost project as, for how we use it at least, it's a local dev thing.  Only the configuration matters for taking to deployment.  If not built in, guessing you can add your own stuff to the AppHost project perhaps?

Where are you deploying to? What extracts this configuration and does something in deployment? The advantage of making it a project with YARP is that it works like any of your other projects and is truly portable and doesn't need to be translated into some other form since it is your proxy project.
You can build an ingress resource that uses YARP under the covers in the apphost project itself (it can host another web server) but then you also need to support that mapping from ingress rules into whatever makes sense for your deployment.
I'll attempt to build a sample that you can use, but we won't be building this into the apphost, at least as of right now.

We have our own specification language (YAML configuration) and we dynamically create the tye.yaml for local dev or terraform / helm (values.yaml) for AKS deployment.  So, it goes to K8 Ingress, we don't need/want our own C# project for an ingress component.  We utilize dapr with communication through the side cars.
I would for sure vote this item up.  To me it makes total sense to be integrated into Aspire, the manifest/tooling/dashboard, etc.  This is potentially something that will keep us from using Aspire anytime soon.  But this could also be due to me not fully understanding Aspire yet...

I understand what you suggest and I understand the complexity of the mapping/rewrite rules.
But now I need to pollute my src/ directory with a project and code that is only to facilitate local development.
Probably the example I posted isn't the best one because you dont really need the ingress in this case.
The other case I use the ingress is in a react+aspnet core app.
We deploy the react bits on a Azure Static Site and let access the backend with relative paths instead of having to sissle around with hostnames to be passed in the react build process.
This deployment solution really works well for us and being able to easily replicate this behavior locally was a huge thumb up for Tye.
On the other hand, being forced to roll out my own mapping project only for development purpose feels a bit of a let down for a "development inner cycle" tool.

The app model is fully extensible C# you can build this yourself, in fact this is a feature. I'd urge you to attempt to build this if it's something you need and give us some feedback about how extensible the app model is for these situations. We're not going to build generic ingress mapping to various proxies.

Hi @davidfowl
I'm trying to giving it a shot here: https://github.com/Kralizek/AspireIngress (see the AspireIngress.Resources project)
I could use some pointers on how to hook into the resource creation after it has been defined. I went through the repo looking at how projects and containers are created but I think I miss something.
Also, I can't find the source generator that takes care of creating the service metadata classes for the referenced projects.

I have a spike of this, I'll be able to share tomorrow hopefully.

This wouldn't be a DCP resource. DCP is the kubernetes API and it supports containers and executables (https://github.com/dotnet/aspire/tree/main/src/Aspire.Hosting/Dcp/Model). The ingress resource would either need to run directly in the app host or be another process that runs as part of DCP orchestration. That's why it's so attractive making it a project.

That's why it's so attractive making it a project.

My idea was to create a "hidden" project. With just the YARP setup extrapolated from the annotations.

@Kralizek where is this hidden project?

Well, hooking into the pipeline of creation of resources/project is what I'm trying to understand :)
That's why I was looking for some pointers on what to look at.

After spending some time with this, I think it would be better to avoid running the proxy code in the hosting process and instead treat it like project or executable installed on the machine. So you can use something like nginx/Yarp/caddy/ etc etc as your proxy (but you need something configurable) and then you need to turn the ingress config into the native config of the proxy being used. YARP requires another project. Nginx works but you have to generate configuration files at runtime (not the end of the world I guess).
The benefit of using a project for YARP is that it's portable and you can deploy it to the environment.

All, I put together a very crude implementation of using YARP with Aspire. This allows you to easily wire-up API project backends for YARP using Aspire semantics within Program.cs of the AppHost project.
My primary motivation in doing this is to be able to wire-up DevTunnels to all my microservice APIs for a mobile app to consume while in development.
Code can be found here:
https://github.com/rjygraham/AspireYarpTest

Sooo there probably isn't anything for us to do here right? Or are we proposing introducing Ngrinx/Caddy resource types with extension methods to drive configuration? (noting that we don't have a YARP container we can just pick up and configure with EnvVars either.

So I've looked at the eshop example and it's linking to source projectsd that look like they'll be nugets. When will these be published?
And It would be really nice to not get the configuration from settings, but to load them from service discovery. I.e. we have most of what we need save the subdirectory for wiring all of the endpoints to Yarp for a single endpoint in the host.
Does it make sense to have the host generate the yarp endpoint and have it just work or do something similar?  It can of course be opt in and should have a project that just takes the WithReferences (change to WithServices that takes the sub path) and does the rest of the work for you.
If the dashboard is split out of host, then host itself could become the yarp endpoint. The only part that would have to be defined is certificate management and then in K8s, how to make the host the ingress so you don't have to use nginx or anything else, just yarp.

I took a stab at this with the latest capabilities of the app host and was able to build something like this https://github.com/davidfowl/AspireYarp.
This is only usable in development, it doesn't attempt to define a generic ingress resource. This is a YARP specific resource with YARP specific configuration.

Beautiful!

Resurrecting this thread as I'm trying to use the work of the repo above on Preview 5.
The only change I needed to get to compile is this line as follows:
// var context = new EnvironmentCallbackContext(options.Value.Publisher!);

var executionContext = new DistributedApplicationExecutionContext(DistributedApplicationOperation.Run);
var context = new EnvironmentCallbackContext(executionContext);

This is how I use it in my AppHost project
builder.AddYarp("login")
    .Route("auth", auth, path: "/");

Now the dashboard shows the resource but doesn't add the link to the resource and doesn't report its running status.

When I navigate to the address from the logs, I get the following error message
info: Aspire.Hosting.DistributedApplication[0]
      Aspire version: 8.0.0-preview.5.24201.12+1b627b7d5d399d4f9118366e7611e11e56de4554
info: Aspire.Hosting.DistributedApplication[0]
      Distributed application starting.
info: Aspire.Hosting.DistributedApplication[0]
      Application host directory is: C:\Development\TD\IdentityManager\App\tools\AppHost
info: Aspire.Hosting.DistributedApplication[0]
      Now listening on: https://localhost:17079
info: Microsoft.Hosting.Lifetime[14]
      Now listening on: http://<IP_ADDRESS>:50751
info: Microsoft.Hosting.Lifetime[0]
      Application started. Press Ctrl+C to shut down.
info: Microsoft.Hosting.Lifetime[0]
      Hosting environment: Development
info: Microsoft.Hosting.Lifetime[0]
      Content root path: C:\Development\TD\IdentityManager\App\tools\AppHost
info: Aspire.Hosting.DistributedApplication[0]
      Distributed application started. Press Ctrl+C to shut down.
info: Yarp.ReverseProxy.Forwarder.HttpForwarder[9]
      Proxying to http://aspire.hosting.applicationmodel.endpointreference:0/ HTTP/2 RequestVersionOrLower
warn: Yarp.ReverseProxy.Forwarder.HttpForwarder[48]
      Request: An error was encountered before receiving a response.
      System.Net.Http.HttpRequestException: No such host is known. (aspire.hosting.applicationmodel.endpointreference:0)
       ---> System.Net.Sockets.SocketException (11001): No such host is known.
         at System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.ThrowException(SocketError error, CancellationToken cancellationToken)
         at System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.System.Threading.Tasks.Sources.IValueTaskSource.GetResult(Int16 token)
         at System.Net.Sockets.Socket.<ConnectAsync>g__WaitForConnectWithCancellation|285_0(AwaitableSocketAsyncEventArgs saea, ValueTask connectTask, CancellationToken cancellationToken)
         at System.Net.Http.HttpConnectionPool.ConnectToTcpHostAsync(String host, Int32 port, HttpRequestMessage initialRequest, Boolean async, CancellationToken cancellationToken)

Apparently, it tries to proxy the request to the address http://aspire.hosting.applicationmodel.endpointreference:0/ but I have no idea if it's correct for the Yarp-based service discovery system.
I guess it originates from this line:
[target.Resource.Name] = new() { Address = $"http://{target.Resource.Name}" }

Finally, the request path transformation isn't working as it throws an exception when starting the AppHost.
fail: Microsoft.Extensions.Hosting.Internal.Host[11]
      Hosting failed to start
      System.InvalidOperationException: Unable to load or apply the proxy configuration.
       ---> System.AggregateException: The proxy config is invalid. (Unknown transform: )
       ---> System.ArgumentException: Unknown transform:
         --- End of inner exception stack trace ---
         at Yarp.ReverseProxy.Management.ProxyConfigManager.ApplyConfigAsync(IReadOnlyList`1 routes, IReadOnlyList`1 clusters)
         at Yarp.ReverseProxy.Management.ProxyConfigManager.InitialLoadAsync()

I'm updating the repo now!

Is the repro updated?

Yes it’s updated to preview 5

It's still giving me issues with the request transformation.

File an issue on the repo with instructions.

I'm confused with how this is working now.
I currently have a Yarp project that uses config file routing setup, and those use ServiceDiscovery. That worked until P5 and now doesn't work because it's not resolving service discovery.
It looks like AddYarp is some sort of extension method now in the host that generates the yarp based on the services instead of having a separate project.
Is there any way to get the service discovery working like it used to with yarp?
The release notes tell you that ServiceDiscovery changed and then that takes you to a github issue that is frankly unintelligable as to what actually changed instead of just telling you what changed and how to fix it.
If not, ok, with putting it in the host, but it really needs to be promoted as requested as a full project for aspire and the aspir8 needs to be updated to translate that into deploy of the yarp ingress etc.

The release notes tell you that ServiceDiscovery changed and then that takes you to a github issue that is frankly unintelligable as to what actually changed instead of just telling you what changed and how to fix it.

I had the same feeling.

I currently have a Yarp project that uses config file routing setup, and those use ServiceDiscovery. That worked until P5 and now doesn't work because it's not resolving service discovery.

Is there any way to get the service discovery working like it used to with yarp?

I'll try to reproduce this normally.

It looks like AddYarp is some sort of extension method now in the host that generates the yarp based on the services instead of having a separate project.

My project is puts YARP in the apphost as a development time only dependency. They are different approaches.

I got 1 endpoint to work and I don't know why. The rest don't.

OK I built a sample and it seems like https isn't working but http is. I created a new preview 6 project and added yarp and was able to forward a simple request to the page. The YARP project had this:
{
"ReverseProxy": {
  "Routes": {
    "fe": {
      "ClusterId": "fe",
      "Match": {
        "Path": "/fe/{**catch-all}"
      },
      "Transforms": [
        { "PathRemovePrefix": "/fe" }
      ]
    }
  },
  "Clusters": {
    "fe": {
      "Destinations": {
        "fe": {
          "Address": "https://webfrontend",
          "Host": "localhost"
        }
      }
    }
  }
}
}

Apphost:
var builder = DistributedApplication.CreateBuilder(args);

var cache = builder.AddRedis("cache");

var apiService = builder.AddProject<Projects.AspireApp47_ApiService>("apiservice");

var fe = builder.AddProject<Projects.AspireApp47_Web>("webfrontend")
    .WithExternalHttpEndpoints()
    .WithReference(cache)
    .WithReference(apiService);

builder.AddProject<Projects.WebApplication1>("gateway")
    .WithReference(apiService)
    .WithReference(fe);

builder.Build().Run();

YARP project:
var builder = WebApplication.CreateBuilder(args);

builder.AddServiceDefaults();

builder.Services.AddReverseProxy()
                .LoadFromConfig(builder.Configuration.GetSection("ReverseProxy"))
                .AddServiceDiscoveryDestinationResolver();

var app = builder.Build();

app.MapReverseProxy();

app.MapGet("/", () => "Hello World!");

app.Run();

I can confirm that YARP works with preview 6. Now we can focus on narrowing to what you might be running into in your existing project.

Why is host required? And how would that change for k8s when we deploy?

There's a bug with https where we set the original host name. The cert mismatch happens when running locally because the ASP.NET dev cert. I don't think this would be required if you use http nor would it matter if you're on k8s.

As for AddYarp, how would one bind that to https (it errors if I try and make it an https port) with the same?

I would have to write more code.

Would the best approach if using AddYarp be to have another microservice for /help that just serves that instead of the proxy or is there a better way?

AddYarp is only useful if you using it for development only. Is that what you're doing?

Would the best approach if using AddYarp be to have another microservice for /help that just serves that instead of the proxy or is there a better way?

AddYarp is only useful if you using it for development only. Is that what you're doing?

My hope would be for Aspire to define the ingress using AdYarp which would dump to the exported data, which could then be used by aspir8 and others to spin up the Yarp ingress with the settings I've defined.
Sort of like it would be nice to be able to define secrets and certificates to use as well and those would dump and then aspir8 would map those to secrets and certificates in k8s or if you were deploying to Azure as an example those would map to your KeyVault equivalents.

My hope would be for Aspire to define the ingress using AdYarp which would dump to the exported data, which could then be used by aspir8 and others to spin up the Yarp ingress with the settings I've defined.

That doesn't exist, but that's what this issue is about. My project does not do this. Ingress is generic and doesn't match to YARP's configuration. It needs to be its own resource type that doesnt' map can map to YARP as an implementation. This isn't the approach that the YARP resource takes.

Sort of like it would be nice to be able to define secrets and certificates to use as well and those would dump and then aspir8 would map those to secrets and certificates in k8s or if you were deploying to Azure as an example those would map to your KeyVault equivalents.

I like it but we're not close to having any end to end out of the box for this. Hopefully, it will be possible to build it from the primitives we offer.

@davidfowl @mitchdenny  - We should revisit the idea of being able to configure routes via the app-model and have the app-model spit out configuration for YARP as a result.
The deployment step would be interesting as it could convert that to a hosted YARP and remap the endpoints based on the deployment target and how service discovery happens in those systems.
https://github.com/microsoft/reverse-proxy/issues/2525 is to drive having a docker image for YARP.

That’s the killer scenario @samsp-msft!

That would be really nice. I currently use the Yarp support from Aspirant and it works really nicely.
But I have few routes that are hidden in the configuration file.
  "ReverseProxy": {
    "Routes": {
      "backend": {
        "ClusterId": "backend",
        "Order": 1,
        "Match": {
          "Path": "/api/v1/{**remainder}"
        },
        "Transforms": [
          { "PathRemovePrefix": "/api/v1" }
        ]
      },
      "legacy-api": {
        "ClusterId": "legacy",
        "Order": 100,
        "Match": {
          "Path": "/api/{**remainder}"
        }
      },
      "legacy-services": {
        "ClusterId": "legacy",
        "Order": 101,
        "Match": {
          "Path": "/services/{**remainder}"
        }
      },
      "swagger": {
        "ClusterId": "backend",
        "Order": 900,
        "Match": {
          "Path": "/swagger/{**remainder}"
        }
      },
      "openapi": {
        "ClusterId": "backend",
        "Order": 901,
        "Match": {
          "Path": "/schema/{**remainder}"
        }
      },
      "frontend": {
        "ClusterId": "frontend",
        "Order": 1000,
        "Match": {
          "Path": "{**catchall}"
        }
      }
    },
    "Clusters": {
      "backend": {
        "Destinations": {
          "backend": {
            "Address": "http://backend"
          }
        }
      },
      "frontend": {
        "Destinations": {
          "frontend": {
            "Address": "http://frontend"
          }
        }
      },
      "legacy": {
        "Destinations": {
          "frontend": {
            "Address": "http://legacy"
          }
        }
      }
    }
  }

From one side, I see the value of not cluttering the AppHost Program file, but I also see the value of keeping these rules in the forefront.

@JohnGalt1717 I don't want to couple these problems with the initial resource. We need to build something that works in local dev and can be deployed for basic scenarios. When we get it working then we need to talk about making local host name routing work and making certs more seamless for local dev, and how to make k8s ingress for YARP work when you deploy in that environment.
We need to sequence features, so we don't boil the ocean.

100%.  Just want to make sure that we don't forget that often, these routes are environment specific.

I don’t think we want to build ingress into the app model. Given that we have YARP now, just make it one of your projects and configure it there with the service discovery extensibility. Take a look at eshop lite for an example.
https://github.com/dotnet/aspire/tree/main/samples/eShopLite/ApiGateway

This link doesn't work....

Related is #6640 to have WithReplica load balancing for ingress.


Add loader

fix #5

Codecov Report

Merging #13 (c814ba5) into master (68570d4) will increase coverage by 0.19%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #13      +/-   ##
==========================================
+ Coverage   82.11%   82.31%   +0.19%
==========================================
  Files          17       18       +1
  Lines         453      458       +5
  Branches       96       96
==========================================
+ Hits          372      377       +5
  Misses         16       16
  Partials       65       65

Impacted Files
Coverage Δ

src/lib.rs
100.00% <ø> (ø)

src/loader.rs
100.00% <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 68570d4...c814ba5. Read the comment docs.


Minor changes when dealing with optional variables

Swift's optional chaining is usually preferable to ternary operators when checking for nil, so it was adopted for the user and pwd variables.  Unnecessary nil assignments were also removed from optional variable declarations.

Hey @collinhundley, thanks for the contribution! Unfortunately, before we can accept any of your PRs you'll need to follow the process in CONTRIBUTING.md.

@dfirsht I've submitted the CLA.  It seems that all PRs are failing the Travis CI build - is this expected?

Ok great! I see it here. Don't worry about the failing CI, there's an issue with it building on the module repo instead of the main one. I'll review your PR and let you know.

Looks great! Thanks for the contribution @collinhundley!


Bypassing client schema validation and setting x-api-project-id heade…

Trying to address issue https://github.com/rancher/rancher/issues/4532
Any suggestions on how to fix the issue without bypassing the schema validation?

Very stale. Closing


[ko] sync safe sysctl

sync https://github.com/kubernetes/website/pull/44170

Due to the release of v1.29, this pr is out of date.


fixed generation of ChiSquared Samples

I noticed a bug in the generation of ChiSquared samples. Probably left from copy-pasting code from Chi.cs

Great catch, thanks for reporting and fixing this!

Yes, this looks like a copy-paste error indeed.


Mount is not supported on freebsd amd64

Describe the bug
As the title says. Trying to run weed mount on FreeBSD fails.
System Setup
OS: Freebsd 13.1
Tested with seaweedfs-3.37_1 from the ports tree. Also tested v3.4.0 from https://github.com/seaweedfs/seaweedfs/releases/tag/3.40. The same not supported message in both cases.
Expected behavior
Expected weed mount to work.
Additional context
I can't find any documentation that states that weed mount is unsupported. The only thing I've seen is this, https://github.com/seaweedfs/seaweedfs/issues/1769#issuecomment-783359358, that indicates it is supported.
So is weed mount supported on FreeBSD? If not, can it be? The objective is to mount SeaweedFS over FUSEFS.

it is supported

Thank you for confirming. I'll see if I can figure out why it's saying it's not. Any ideas welcome.

please report exact errors following the issue template.


Fix bug #61215 : NoSuchMethodException in JMX Proxy (RemoteAddrValve)

OK, just discovered the is="false" attribute in mbean-descriptors.xml.


scalafix: 0.9.0 -> 0.10.0, completions, setup-hook instead of jdk

Description of changes

https://github.com/scalacenter/scalafix/releases/tag/v0.10.0

Shell completions

NOTE: I'm honestly unsure the significance of pinning to a scala version
tag (which I bump from 2.12.7 -> 2.13.8), so someone familiar with this
please take a look :).
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

(grabbed setup-hook fixup from #171067)

Note installCheck doesn't run currently, FWIW, see #166730 for fixing this in packages tree-wide.

Tested on nixos, seems to work fine however I've not used scalafix for a while now.
LGTM.

I have not looked into why it's pinned to java8, leaving alone for now.

Thanks for the review!


python311Packages.rova: 0.3.0 -> 0.4.0

Diff: https://github.com/GidoHakvoort/rova/compare/v0.3.0...v0.4.0
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 290695 run on x86_64-linux 1

4 packages built:

python311Packages.rova
python311Packages.rova.dist
python312Packages.rova
python312Packages.rova.dist


Check whether there are healthy backend available

Currently when there are no backend available in pool Mantrid is
throwing general exception. We want to handle it in more civilized
way so nginx can catch a problem and fallback to other grid
clusters.
This fix is needed for az-separation, as when setting up new cluster
it will not have any backends set in mantrid. As workers connect
only to nginx->mantrid correlated to their grid, we don't want them
to throw errors and bother anyone, so let's just let nginx to
connect to other, healthy grid cluster.

Ładne.
Maybe log which host triggered the exception, if it doesn't already happen?

It was logged in another place. I moved it to exception handler, it's much more appropriate and cleaner. Also we decided to go with custom HTTP response code, as applications happen to return 503 from time to time and nginx retry could do bad things.


fix proxy prepare insert statement with generated key cannot work.

Pull Request Test Coverage Report for Build 7150

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 20.132%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-proxy/sharding-proxy-backend/src/main/java/org/apache/shardingsphere/shardingproxy/backend/communication/jdbc/wrapper/PreparedStatementExecutorWrapper.java
0
1
0.0%

Totals

Change from base Build 7149:
0.0%

Covered Lines:
8187

Relevant Lines:
40667

💛  - Coveralls


Update Simplified Chinese translation

For #14997

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[x] Has a news entry file (remember to thank yourself!).
~[x] Appropriate comments and documentation strings in the code.~
~[x] Has sufficient logging.~
~[x] Has telemetry for enhancements.~
~[x] Unit tests & system/integration tests are added/updated.~
~[x] Test plan is updated as appropriate.~
~[x] package-lock.json has been regenerated by running npm install (if dependencies have changed).~
~[x] The wiki is updated with any design decisions/details.~

Any updates?

Any updates?

@FiftysixTimes7 can you resolve the merge conflicts?

@FiftysixTimes7 can you resolve the merge conflicts?

@brettcannon Conflicts are resolved! Sorry for not marking it as resolved initially.

@brettcannon Conflicts are resolved! Sorry for not marking it as resolved initially.

Codecov Report

Merging #14997 (275bd75) into main (ef3fe4f) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##            main   #14997   +/-   ##
======================================
  Coverage     65%      65%
======================================
  Files        560      560
  Lines      26362    26362
  Branches    3747     3747
======================================
  Hits       17174    17174
  Misses      8494     8494
  Partials     694      694

Codecov Report

Merging #14997 (275bd75) into main (ef3fe4f) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##            main   #14997   +/-   ##
======================================
  Coverage     65%      65%
======================================
  Files        560      560
  Lines      26362    26362
  Branches    3747     3747
======================================
  Hits       17174    17174
  Misses      8494     8494
  Partials     694      694

Thanks!

Thanks!

@FiftysixTimes7 Can you please verify the extension loads correctly when using Chinese?

@FiftysixTimes7 Can you please verify the extension loads correctly when using Chinese?

I can verify it works for me.

I can verify it works for me.

@karrtikr Yeah, it works well :)
But some commands seem unavailable to be translated, such as these 2 commands in the right-click menu:

And everything in the settings panel...
If these can be translated, I'm willing to help :)

@karrtikr Yeah, it works well :)
But some commands seem unavailable to be translated, such as these 2 commands in the right-click menu:

And everything in the settings panel...
If these can be translated, I'm willing to help :)

@FiftysixTimes7 some of those, like "Find All References" come from VS Code and not us. The "Run Current File in Interactive Window" is us and it might be a bug in the code that it isn't being translated. Would be willing to either try and find the bug or at least open an issue for us to fix it?

@FiftysixTimes7 some of those, like "Find All References" come from VS Code and not us. The "Run Current File in Interactive Window" is us and it might be a bug in the code that it isn't being translated. Would be willing to either try and find the bug or at least open an issue for us to fix it?

Actually "Run Current File in Interactive Window" comes from the Jupyter extension, so you can look at https://github.com/microsoft/vscode-jupyter for that text.

Actually "Run Current File in Interactive Window" comes from the Jupyter extension, so you can look at https://github.com/microsoft/vscode-jupyter for that text.

Thanks for tracking down the issue!
Time permitting, I'm ready to translate the Jupyter extension too :)

Thanks for tracking down the issue!
Time permitting, I'm ready to translate the Jupyter extension too :)

I found an issue:

I've translated "StartPage.*", but it seems that the translation isn't displayed.

I found an issue:

I've translated "StartPage.*", but it seems that the translation isn't displayed.

@FiftysixTimes7 yes, please open a new issue.

@FiftysixTimes7 yes, please open a new issue.


feat(nix-direnv): integrate nix-direnv with bare devenv usage [WIP]

I'd really like to first tackle #240 and then see if we can ignore the whole Nix business.
Otherwise this would be a good compromise :)

Hey @thenonameguy, thanks for this work. It took me a while, but I've made a devenv specific direnv integration, based on nix-direnv.
It differs that it can track imports of devenv.yaml and simplifies a lot of code since it's devenv aware.
See https://github.com/cachix/devenv/pull/387

@thenonameguy @fschmtt @janvotava I'm going to close this as #387 will land soon. Thanks again for moving this forward!


ShaderType enum

Types are easier to work with than enums

Why second commit? I think it's quite convenient when enum is transparent.

I would like to avoid exposing OpenGL specific stuff (see my comment on #124 ) so if, one day, another type of context is to be supported, it will not change the interface. I agree that it is nice to know what the enum maps to, but you can still see that in the OpenGL or WebGL specific parts. And in this case it is not really ambiguous.

one day, another type of context is to be supported

What do you mean? Something like Metal?

Yeah, Metal, DirectX, Vulkan or (more likely) WebGPU. It is not the plan right now, but it is likely that it is necessary to support different backends at some point. It will probably be a small subset of the capabilities of modern graphics API's so that it has the same functionality as OpenGL. I don't think it is necessary to plan for it too much, I just don't want to do OpenGL specific stuff where it is not needed.


Export to PDF does not work on MacOS

Hi, I use vscode extension AsciiDoc 0.12.0
I can't install wkhtmltopdf when I do adoc.ExportAsPDF
I think you have to modify download_url in line 87 of ExportAsPDF.ts
now
const download_url = https://github.com/joaompinto/asciidoctor-vscode/raw/master/wkhtmltopdf-bin/wkhtmltopdf-${platform}-${arch}${ext}.gz
new
const download_url = https://github.com/joaompinto/asciidoctor-vscode/wkhtmltopdf-bin/wkhtmltopdf-${platform}-${arch}${ext}.gz
I want to use adoc.ExportAsPDF

What is your OS Version?

It's macOS

The current url at the source is ok, but right now there is no archive for mac because I have no experience with it. Right now do you get an error ? Does it provide the link that is failing to download ?

Now, I get an error

Same problem

Hi. First thank you for the extension. Is a great one...
I think there is two ways we can solve this...

Try with the binary that I extracted from the pkg of the official wkhtmltopdf site.
Instead of downloading on a macOS, find the binary at the path an use it if found. If not instruct to download and install from homebrew or official site. I'm making a homebrew package to install under Cellar and not outside as the current one do. (But this is a discussion for homebrew-core, not here)

I attached the file here for testing. In my machine it worked with it. I cloned the repo put the extracted file in out/src and exported a test file..
wkhtmltopdf-darwin-x64.gz

@fcrespo82 thanks for the help, option 1) is the "standard" approach being used now, I was just missing feedback that it would work on Mac, I have aded wkhtmltopdf-darwin-x64.gz to the git folder so it should now be downloadabed.
@ravipinto The download wkhtmlpdf option that is automatically called by the extension should now work on Mac.
I'll leave the issues until getting a positive feedback about the change.

@joaompinto, I tested the download option and it is working as expected. Thanks for the update!

Closed with the release of v2.0.0


Tomi dev

I've implemented the header view and the horizontal list below the header.
TODO:

Add a radial gradient to the header.
Implement the main navigation menu.
Find a better font for the text?

It looks really good so far, well done.
It definetly need a new font for the text.


Cursor shouldn't animate while scrolling if it stays in the same position

In this video, the cursor is always on line 10, but as I scroll with the mouse wheel, it gets a screen space animation.
https://user-images.githubusercontent.com/2766423/109586119-aaab3580-7ad2-11eb-98bc-5b94b1933fb0.mp4

Ouch, yes, the cursor animation definitely needs some love -


pkl model file for testing MS-Celeb-1M subset pretrained model with classifier.py

I'm trying to test the classifier that was trained on a subset of MS-Celeb-1M. I found the .pb and other files on the Google Drive link, and used the pb as the argument for 'model', but classifier.py also requires an argument for 'classifier_filename' which is a pkl.
How can I create or find this pickle file?

You need to generate it, how to do that you can read that on this wiki page.

Thanks. Is it possible to use the classes from the pretrained model and not supply any data of my own?

Sure that's no problem at all

How would I do that? would I just specify data_dir as an empty directory?


installation broken with react 17

When running npm install react-nice-dates date-fns --save
I get the error:
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer react@"^16" from<EMAIL_ADDRESS>npm ERR! node_modules/react-nice-dates
npm ERR!   react-nice-dates@"*" from the root project

The same for me, and it actually works fine if I'm adding --legacy-peer-deps flag. I see component installed and behave without any errors.
Looks like there is a PR to fix - https://github.com/hernansartorio/react-nice-dates/pull/71

Until the PR #71 get merged, you can try using: npm i @axel-dev/react-nice-dates date-fns --save.


add m68k support

I only tested the version embedded in qtdeclarative-opensource-src, but the patch is trivial.

See commit da11179.

Ah right. Why do you use mc68000 when we use m68k everywhere welse @andreas-schwab  ?


pyEPR load bug

From @zlatko-minev
Information

Qiskit Metal version: all
Python version: all
Operating system: all

What is the current behavior?
Crash happens when importing MetalGUI only if pyEPR is imported first
Steps to reproduce the problem

What is the expected behavior?
No crash
Suggested solutions

Has not occurred recently, also, pyepr will become stand alone and Metal will not use pyepr.


Qobuz integration?

Well, i just want to know if you are thinking in add this option later.
Thanks.

me too!


Banphrase before Linkchecker

Make it so the Banphrase is more important than the Linkchecker.
When bots are spamming a google link,
the bots only get the low timeout and not the permaban from the banphrase.

Fixed in dbf5b8bc457e660cedde273eedd507cf50752ea1


build: disable http3 does not work as specified in docs

I am running bazel build --//source/extensions/wasm_runtime/v8:enabled=false --//bazel:http3=False --define google_grpc=disabled //source/exe/..."
I see this output:
[5,451 / 7,762] Compiling source/common/common/utility.cc; 8s darwin-sandbox ... (16 actions, 15 running)
[5,470 / 7,762] Compiling source/extensions/filters/network/mongo_proxy/bson_impl.cc; 8s darwin-sandbox ... (16 actions, 15 running)
[5,484 / 7,762] Compiling source/extensions/filters/http/grpc_json_transcoder/http_body_utils.cc; 9s darwin-sandbox ... (16 actions, 15 running)
[5,497 / 7,762] Compiling quiche/http2/hpack/varint/hpack_varint_decoder.cc; 6s darwin-sandbox ... (16 actions, 15 running)
[5,541 / 7,762] Compiling quiche/http2/decoder/payload_decoders/settings_payload_decoder.cc; 7s darwin-sandbox ... (16 actions, 15 running)
[5,568 / 7,762] Compiling quiche/http2/hpack/decoder/hpack_entry_decoder.cc; 7s darwin-sandbox ... (16 actions, 15 running)
[5,619 / 7,762] Compiling quiche/http2/core/http2_trace_logging.cc; 7s darwin-sandbox ... (16 actions, 15 running)
[5,652 / 7,762] Compiling source/common/stats/symbol_table.cc; 10s darwin-sandbox ... (16 actions running)
[6,176 / 7,762] Compiling source/common/http/header_map_impl.cc; 14s darwin-sandbox ... (16 actions running)
[6,201 / 7,762] Compiling source/server/config_validation/dns.cc; 10s darwin-sandbox ... (16 actions, 15 running)
[6,239 / 7,762] Compiling quiche/quic/core/quic_alarm.cc; 8s darwin-sandbox ... (16 actions, 15 running)
[6,275 / 7,762] Compiling quiche/quic/core/quic_packets.cc; 12s darwin-sandbox ... (16 actions, 15 running)
[6,319 / 7,762] Compiling quiche/quic/core/crypto/aes_128_gcm_encrypter.cc; 10s darwin-sandbox ... (16 actions, 15 running)
[6,383 / 7,762] Compiling quiche/quic/core/quic_received_packet_manager.cc; 9s darwin-sandbox ... (16 actions, 15 running)
[6,437 / 7,762] Compiling quiche/quic/core/quic_session.cc; 17s darwin-sandbox ... (16 actions, 15 running)
[6,500 / 7,762] Compiling source/extensions/filters/common/local_ratelimit/local_ratelimit_impl.cc; 18s darwin-sandbox ... (16 actions, 15 running)

which contains quiche code that should have been disabled by --//bazel:http3=False, as specified here
Docs don't match bazel behavior.

The --//bazel:http3=false does disable HTTP3 support in HCM, while not disable quiche build. quiche is being pulled by more targets like: https://github.com/envoyproxy/envoy/blob/main/source/common/common/BUILD#L164

Oh I see. In my mind I was thinking of quiche==http3 support, but that is not the case. Also in the future, it looks like the http1 codec will come from quiche, so it probalby won't be feasible to build Envoy without pulling in this dependency.


Simplify fwd.go, and enable WSS (Websockets over TLS)

This is a significant change with the following goals:

Improve websockets implementation with WSS support (there was no way to dial a TLS Websocket port in the previous iteration.)
Improve Streaming Forwarder to be on feature-parity with buffered forwarder.

Consolidation of Forwarders:
After all the changes (described below), it became apparent that the forwarders only differed in
a few small things:

One or two specific fields.
The copyRequest function (which is different for http vs ws)
The ServeHTTP method.

I consolidated all three of them together so the ServeHttp methods of each type benefits from settings configured for httpForwarder. It also avoids a lot of code noise, and redundant functions.
Specific changes of note that led to this:

Remove the "Dialer" for WebSocket forwarder since it didn't make immediate sense. If this is a problem, I'm happy to change that. I want to make Dialer an interface that'll support TLS config.

Added a Rewriter/PassHost fields to stream. This is important because even those connections ultimately might want Forward headers, etc.

This field is added to, but not used in Websockets because I need to debug more. It is used in stream where it works.

Stream uses the same roundtripper for ReverseProxying that buffered forwarder was using. This means it inherits TLS config settings like InsecureSkipVerify.

Websocket logs are more descriptive - especially when something fails.

For the BufferedWriter Response Proxy, implemented interfaces "http.Hijacker" which allows a Websockets upgrade, and more importantly, http.CloseNotifier which allows Serve/ServeTLS to terminate the downstream connection if/when the client disconnects abruptly. This might be a performance improvement when a client creates a lot of connections but disconnects rapidly.

Would appreciate a review on this. WSS was broken before this change, and I'd love for vulcan to be able to pull these in. We've tested this branch in production for over two weeks now. Works reliably. We have a gauntlet of http->https, https->http, wss->ws and ws->wss tests that we run on each build.

I'm going to go ahead and merge this. Again - we've thrown this behind production traffic for about a month. Powering our website. All changes are stable and back-compatible.


💻Playsound Missing .Wav + Extra Characters from Text Splitting🤖

Hi, after re install of everything, there's still these errors about Cannot specify extra characters after a string enclosed in quotation marks, Text splitted to sentences and strange characters? Then the missing .wav file playsound error. I wonder if you found anything more about it?
TEXT SPLIT AND STRANGE CHARACTERS
Listening...
==> answer:  I am an AI language model and do not have the ability to feel emotions or experience physical sensations. However, I am here to assist you with any qu
estions or tasks you may have. How can I help you today?

Text splitted to sentences.
['I am an AI language model and do not have the ability to feel emotions or experience physical sensations.', 'However, I am here to assist you with any questions
or tasks you may have.', 'How can I help you today?']
aɪ æm ən aɪ læŋɡwɪd͡ʒ mɑdəl ænd du nɑt hæv ðə əbɪləti tə fil ɪmoʊʃənz ɔɹ ɪkspɪɹiəns fɪzɪkəl sɛnseɪʃənz. << STRANGE CHARACTERS
[!] Character '͡' not found in the vocabulary. Discarding it.
Processing time: 1.6647748947143555
Real-time factor: 0.10804181312824211

Windows PowerShell
Copyright (C) Microsoft Corporation. All rights reserved.
PS C:\talkGPT4All-main> & c:/talkGPT4All-main/.venv/Scripts/Activate.ps1
(.venv) PS C:\talkGPT4All-main> & c:/talkGPT4All-main/.venv/Scripts/python.exe c:/talkGPT4All-main/src/talkgpt4all/chat.py
==> GPT4All model: mistral-7b-instruct-v0.1.Q4_0.gguf, Whisper model: base

tts_models/en/ljspeech/glow-tts is already downloaded.
vocoder_models/en/ljspeech/multiband-melgan is already downloaded.
Using model: glow_tts
Setting up Audio Processor...
| > sample_rate:22050
| > resample:False
| > num_mels:80
| > log_func:np.log10
| > min_level_db:-100
| > frame_shift_ms:None
| > frame_length_ms:None
| > ref_level_db:0
| > fft_size:1024
| > power:1.1
| > preemphasis:0.0
| > griffin_lim_iters:60
| > signal_norm:False
| > symmetric_norm:True
| > mel_fmin:50.0
| > mel_fmax:7600.0
| > pitch_fmin:1.0
| > pitch_fmax:640.0
| > spec_gain:1.0
| > stft_pad_mode:reflect
| > max_norm:1.0
| > clip_norm:True
| > do_trim_silence:True
| > trim_db:60
| > do_sound_norm:False
| > do_amp_to_db_linear:True
| > do_amp_to_db_mel:True
| > do_rms_norm:False
| > db_level:None
| > stats_path:None
| > base:10
| > hop_length:256
| > win_length:1024
Vocoder Model: multiband_melgan
Setting up Audio Processor...
| > sample_rate:22050
| > resample:False
| > num_mels:80
| > log_func:np.log10
| > min_level_db:-100
| > frame_shift_ms:None
| > frame_length_ms:None
| > ref_level_db:0
| > fft_size:1024
| > power:1.5
| > preemphasis:0.0
| > griffin_lim_iters:60
| > signal_norm:True
| > symmetric_norm:True
| > mel_fmin:50.0
| > mel_fmax:7600.0
| > pitch_fmin:0.0
| > pitch_fmax:640.0
| > spec_gain:1.0
| > stft_pad_mode:reflect
| > max_norm:4.0
| > clip_norm:True
| > do_trim_silence:True
| > trim_db:60
| > do_sound_norm:False
| > do_amp_to_db_linear:True
| > do_amp_to_db_mel:True
| > do_rms_norm:False
| > db_level:None
| > stats_path:C:\Users\S373NTH\AppData\Local\tts\vocoder_models--en--ljspeech--multiband-melgan\scale_stats.npy
| > base:10
| > hop_length:256
| > win_length:1024
Generator Model: multiband_melgan_generator
Discriminator Model: melgan_multiscale_discriminator
Listening...
==> answer:  I am an AI language model and do not have the ability to feel emotions or experience physical sensations. However, I am here to assist you with any qu
estions or tasks you may have. How can I help you today?
Text splitted to sentences.
['I am an AI language model and do not have the ability to feel emotions or experience physical sensations.', 'However, I am here to assist you with any questions
or tasks you may have.', 'How can I help you today?']
aɪ æm ən aɪ læŋɡwɪd͡ʒ mɑdəl ænd du nɑt hæv ðə əbɪləti tə fil ɪmoʊʃənz ɔɹ ɪkspɪɹiəns fɪzɪkəl sɛnseɪʃənz.
[!] Character '͡' not found in the vocabulary. Discarding it.
Processing time: 1.6647748947143555
Real-time factor: 0.10804181312824211

Error 305 for command:
    open "C:\Users\S373NTH\AppData\Local\Temp\talkgpt4all-73qu1lv2.wav"
Cannot specify extra characters after a string enclosed in quotation marks.

Error 305 for command:
    close "C:\Users\S373NTH\AppData\Local\Temp\talkgpt4all-73qu1lv2.wav"
Cannot specify extra characters after a string enclosed in quotation marks.

Failed to close the file: "C:\Users\S373NTH\AppData\Local\Temp\talkgpt4all-73qu1lv2.wav"
Traceback (most recent call last):
File "c:\talkGPT4All-main\src\talkgpt4all\chat.py", line 126, in
chat_bot.run()
File "c:\talkGPT4All-main\src\talkgpt4all\chat.py", line 38, in run
self._text_to_voice(answer)
File "c:\talkGPT4All-main\src\talkgpt4all\chat.py", line 74, in _text_to_voice
playsound(tmp_file.name)
File "C:\talkGPT4All-main.venv\Lib\site-packages\playsound.py", line 72, in _playsoundWin
winCommand(u'open {}'.format(sound))
File "C:\talkGPT4All-main.venv\Lib\site-packages\playsound.py", line 64, in winCommand
raise PlaysoundException(exceptionMessage)
playsound.PlaysoundException:
Error 305 for command:
open "C:\Users\S373NTH\AppData\Local\Temp\talkgpt4all-73qu1lv2.wav"
Cannot specify extra characters after a string enclosed in quotation marks.
(.venv) PS C:\talkGPT4All-main>

The file is being deleted before it's read by playsound?
Downgrade to https://stackoverflow.com/questions/68861878/error-cannot-specify-extra-characters-after-a-string-enclosed-in-quotation-mark
uninstalled playsound-1.3.0
pip install playsound==1.2.2
PS C:\talkGPT4All-main> & c:/talkGPT4All-main/.venv/Scripts/Activate.ps1
(.venv) PS C:\talkGPT4All-main> & c:/talkGPT4All-main/.venv/Scripts/python.exe c:/talkGPT4All-main/src/talkgpt4all/chat.py
==> GPT4All model: mistral-7b-instruct-v0.1.Q4_0.gguf, Whisper model: base

tts_models/en/ljspeech/glow-tts is already downloaded.
vocoder_models/en/ljspeech/multiband-melgan is already downloaded.
Using model: glow_tts
Setting up Audio Processor...
| > sample_rate:22050
| > resample:False
| > num_mels:80
| > log_func:np.log10
| > min_level_db:-100
| > frame_shift_ms:None
| > frame_length_ms:None
| > ref_level_db:0
| > fft_size:1024
| > power:1.1
| > preemphasis:0.0
| > griffin_lim_iters:60
| > signal_norm:False
| > symmetric_norm:True
| > mel_fmin:50.0
| > mel_fmax:7600.0
| > pitch_fmin:1.0
| > pitch_fmax:640.0
| > spec_gain:1.0
| > stft_pad_mode:reflect
| > max_norm:1.0
| > clip_norm:True
| > do_trim_silence:True
| > trim_db:60
| > do_sound_norm:False
| > do_amp_to_db_linear:True
| > do_amp_to_db_mel:True
| > do_rms_norm:False
| > db_level:None
| > stats_path:None
| > base:10
| > hop_length:256
| > win_length:1024
Vocoder Model: multiband_melgan
Setting up Audio Processor...
| > sample_rate:22050
| > resample:False
| > num_mels:80
| > log_func:np.log10
| > min_level_db:-100
| > frame_shift_ms:None
| > frame_length_ms:None
| > ref_level_db:0
| > fft_size:1024
| > power:1.5
| > preemphasis:0.0
| > griffin_lim_iters:60
| > signal_norm:True
| > symmetric_norm:True
| > mel_fmin:50.0
| > mel_fmax:7600.0
| > pitch_fmin:0.0
| > pitch_fmax:640.0
| > spec_gain:1.0
| > stft_pad_mode:reflect
| > max_norm:4.0
| > clip_norm:True
| > do_trim_silence:True
| > trim_db:60
| > do_sound_norm:False
| > do_amp_to_db_linear:True
| > do_amp_to_db_mel:True
| > do_rms_norm:False
| > db_level:None
| > stats_path:C:\Users\S373NTH\AppData\Local\tts\vocoder_models--en--ljspeech--multiband-melgan\scale_stats.npy
| > base:10
| > hop_length:256
| > win_length:1024
Generator Model: multiband_melgan_generator
Discriminator Model: melgan_multiscale_discriminator
Listening...
==> answer:  Hello! How can I assist you today?
Text splitted to sentences.
['Hello!', 'How can I assist you today?']
Processing time: 0.6743762493133545
Real-time factor: 0.19353910216263365
Traceback (most recent call last):
File "c:\talkGPT4All-main\src\talkgpt4all\chat.py", line 126, in
chat_bot.run()
File "c:\talkGPT4All-main\src\talkgpt4all\chat.py", line 38, in run
self._text_to_voice(answer)
File "c:\talkGPT4All-main\src\talkgpt4all\chat.py", line 74, in _text_to_voice
playsound(tmp_file.name)
File "C:\talkGPT4All-main.venv\Lib\site-packages\playsound.py", line 35, in _playsoundWin
winCommand('open "' + sound + '" alias', alias)
File "C:\talkGPT4All-main.venv\Lib\site-packages\playsound.py", line 31, in winCommand
raise PlaysoundException(exceptionMessage)
playsound.PlaysoundException:
Error 275 for command:
open "C:\Users\S373NTH\AppData\Local\Temp\talkgpt4all-vek5skcu.wav" alias playsound_0.5306465774055863
Cannot find the specified file.  Make sure the path and filename are correct.
(.venv) PS C:\talkGPT4All-main>

Notes on Possible Solutions
https://stackoverflow.com/questions/68518657/what-is-problem-in-this-python-code-cannot-specify-extra-characters-after-a-stra
Check if your .wav file is valid (try to test other .wav files). Check whether you use the correct character set, use PlaySoundA ("path\file") or PlaySoundW(L"path\file"), or as comment use PlaySound(TEXT("Path\file")). NOTE to use double backslashes in path. Check if there is a redirected folder in the path.


Package Audit

Fixes following issues|Adds following features:
Runs npm audit fix on our packages to resolve security vulnerabilities.
Resolves: https://jira.corp.adobe.com/browse/MWPW-145743
Steps to test the before vs. after and expectations:

Steps for feature 1
Check out this branch and make sure it runs normally
Visit homepage URL to verify everything works there.
Run npm run test locally to verify package updates did not break testing

Pages to check for regression and performance:

https://package-update-3--express--adobecom.hlx.page/express/?martech=off

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

Project coverage is 41.20%. Comparing base (a5d054d) to head (de96b41).
Report is 138 commits behind head on stage.

Additional details and impacted files
@@            Coverage Diff             @@
##            stage    #1203      +/-   ##
==========================================
+ Coverage   38.69%   41.20%   +2.51%
==========================================
  Files         116       97      -19
  Lines       25143    22903    -2240
==========================================
- Hits         9729     9438     -291
+ Misses      15414    13465    -1949

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


1.12.1: No drops but seed for skeleton crops

MC 1.12.1, Forge <IP_ADDRESS>6, Mob Farm 1.12-2.2.1, Gamemode SSP
The only seed I was interested in and which I tested was skeleton seed as this is necessary for the wither skeleton. TOP said: "Fully grown" as the grow state of the crops.
But when harvesting the fully grown skeleton crops there is only seeds generated. Neither mobs nor drops.

So like, unless you're playing in peaceful, the thing works just fine.  Cannot reproduce.


Only UGUI?

Yo,
Is there any possibility of having support for NGUI?

Sorry, there are no plans to support NGUI in the next few months


Fix #121: fix workspace behavior at boot when persistent workspaces is disabled

@marnas can you test this?

Sorry but no luck, the issue still persist

@marnas should work now

That's great, the issue seems to be resolved now. Thank you


"Advanced Search" placeholder text

Please use placeholder text instead of Toggle to signify the emptiness of a search field.

Added 'blank' tag


Issues following README.md instruction

Expected behaviour:
Following the README.md file instructions should allow me to setup the app without errors/issues
Actual behaviour:

Some files seem to be missing:

$ cp config/locales/*.static.yml.example config/locales/*.static.yml
zsh: no matches found: config/locales/*.static.yml.example

The README.md seems to forget to include this instruction cp config/branding_example.yml config/branding.yml because when I execute rake secret, I got:

rake aborted!
Could not load configuration. No such file - /Users/my-user-name/dev/roadmap/config/branding.yml

I use mysql, and running rake db:schema:load gives

rake aborted!
ActiveRecord::StatementInvalid: Mysql2::Error: Cannot delete or update a parent row: a foreign key constraint fails: DROP TABLE `answers` CASCADE

and
Caused by:
Mysql2::Error: Cannot delete or update a parent row: a foreign key constraint fails

But the workaround is here: https://github.com/DMPRoadmap/roadmap/issues/1569
Basically just remove the default for link

Running rails server and access http://localhost:3000/ will render a page without images. I got this type of error in rails server log:

Started GET "/images/dcc_logo_white.png" for ::1 at 2018-06-06 10:08:21 +0930
ActionController::RoutingError (No route matches [GET] "/images/dcc_logo_white.png"):

I notice the images asset is stored in lib folder instead of app folder. Why is this? Is this done to help us customise our own roadmap?
Steps to reproduce:
Follow instruction on https://github.com/DMPRoadmap/roadmap
Notes: I use mysql

For number 5, the solution is to read /lib/assets/README.md and follow the instruction there. It would be nice if the README.md at https://github.com/DMPRoadmap/roadmap indicate this.

Thanks for reporting the inconsistencies here and the outdated instructions.
https://github.com/DMPRoadmap/roadmap/wiki/Installation
The Installation page on the wiki contains all of the correct installation instructions (as far as I know :) ) including both rails and npm. We should probably just update the Readme to point to the wiki page so that we're only maintaining the one set of instructions.
@Yu-Chieh-Henry-Yang for issue 3, it sounds like you have an existing database with data in it. If this is just seed data from a prior installation attempt, you should run rake db:drop && rake db:create && rake db:schema:load. If this is your production data, you should NOT run schema:load as it will attempt to drop tables. You should instead run rake db:migrate. If you are migrating from the older DMPOnline codebase (non-DMPRoadmap) then you should follow the instructions to migrate your data here: https://github.com/DMPRoadmap/roadmap/wiki/Upgrading-from-DMPonline_v4

hey @briri, can this issue be closed now?

@briri @Bodacious We should double check that the readme is up to date with the release and then close this issue

@briri @xsrust is this ok to close?

yes. we will be updating the readme for Rails5 as part of #2497


Typo "**Changeset **"→"Changeset"

https://docs.microsoft.com/en-us/azure/devops/repos/tfvc/changeset-command?view=azure-devops

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


How to verify read_state request

``When the request and response header parameters are
`
code：200
request_headers:  [('User-Agent', 'python-requests/2.31.0'), ('Accept-Encoding', 'gzip, deflate'), ('Accept', '/'), ('Connection', 'keep-alive'), ('Content-Type', 'application/cbor'), ('Content-Length', '335')]
response_headers:  [('Server', 'nginx/1.21.3'), ('Date', 'Wed, 11 Aug 2023 12:06:57 GMT'), ('Content-Type', 'application/cbor, application/cbor'), ('Content-Length', '761'), ('Connection', 'keep-alive'), ('X-Request-ID', 'b0cac601-1303-24f4-04f1-f62cr51ac111'), ('Access-Control-Allow-Origin', '*'), ('Access-Control-Allow-Methods', 'HEAD, POST'), ('Access-Control-Allow-Headers', 'DNT,User-Agent,X-Requested-With,If-None-Match,If-Modified-Since,Cache-Control,Content-Type,Range,Cookie'), ('Access-Control-Expose-Headers', 'Accept-Ranges,Content-Length,Content-Range'), ('Access-Control-Max-Age', '600'), ('X-Content-Type-Options', 'nosniff')]
`
The verification result is
CertificationResult { passed: false, response: None, verification_version: 1 }

Similar to https://github.com/dfinity/response-verification/issues/192, I'm closing since this is not the intended purpose of this library, https://github.com/dfinity/agent-rs is responsible for validating read_state calls. Please feel free to open a thread on the https://forum.dfinity.org/ if there's further questions.


Name is missing

The extension is working well on a Magento <IP_ADDRESS> install. The only thing missing is the Name on the new order notification. Notification comes in like "A new order has been placed. Order ID: ########, Name: , Amount: ###.## USD"

Hi @PaulM4031, sorry for late answer, was in Holidays. Will check it out in a <IP_ADDRESS> install.

No worries I think it's guest checkout where the name doesn't show.
Sent from my iPad
On May 29, 2015, at 2:53 AM, Marcel Hauri <notifications<EMAIL_ADDRESS>wrote:
Hi @PaulM4031https://github.com/PaulM4031, sorry for late answer, was in Holidays. Will check it out in a <IP_ADDRESS> install.

Reply to this email directly or view it on GitHubhttps://github.com/mhauri/magento-slack/issues/3#issuecomment-106712292.

@PaulM4031 Is this still an issue for you? Are you running the latest release?


convolutional.py create  SpatialActivation2D, add to DenseNetFCN

Compute the spatial softmax or another activation of a convolutional feature map.
Based on the paper Learning visual feature spaces for robotic manipulation with
deep spatial autoencoders. Finn et. al, http://arxiv.org/abs/1509.06113.

Oh also this is based on the implementation of spatial_softmax in master, but that isn't in any release yet https://github.com/tensorflow/tensorflow/blob/master/tensorflow/contrib/layers/python/layers/layers.py#L2619

Hello, I'm a bot! I can make mistakes, notify gabrieldemarmiesse if I made one.
I see that you modified the following file:

keras_contrib/applications/densenet.py

The owner of those file is @titu1994
@titu1994 could you please take a look at it whenever
you have the time and add a review? Thank you in advance for the help.

@ahundt The code seems fine but we have new guidelines for contributions. If you still want this merged, could you please open a new PR for layer and a new PR for the model? With tests and yourself added to the codeowners? Thank you.


fb error with shareit

I am getting the strangest thing, it is not a redirect_uri, but rather a "Something went wrong, whoops" page.
The link shareit creates is:
https://www.facebook.com/dialog/feed?app_id=1032542676825405&link=http%3A%2F%2Flocalhost%3A3000%2FS6CWZe7X7pCeB3wa7&name=bad brajse&description=undefined&redirect_uri=http%3A%2F%2Flocalhost%3A3000%2FS6CWZe7X7pCeB3wa7
I am obviously developing with meteor on 3000.
Can anyone share sample settings in fb for local development since it seems to be a problem on that side and not with shareit (i think).
Thank you.

is anyone still using this software?


chore: temporarily disable konnect roles cleanup

What this PR does / why we need it:
Workaround to unblock CI. https://github.com/Kong/kubernetes-ingress-controller/issues/6253 tracks fixing this.

@pmalek It started passing because I generated a new PAT for a service account (it's no longer possible to generate PATs for regular users). Code responsible for cleaning up roles was relying on /me endpoint that was working only with PATs for regular users. The cleanup workflow passes because there are no control planes and in effect, no roles to clean up as well, therefore we do not get to the point where /me endpoint is called. https://github.com/Kong/kubernetes-ingress-controller/issues/6253 is still required.
2024-06-27T07:03:51.6687806Z 2024-06-27T07:03:51.668Z	INFO	cleanup/konnect_runtime_groups.go:83	Control plane was not created by the tests, skipping	{"name": "default"}
2024-06-27T07:03:51.6689969Z 2024-06-27T07:03:51.668Z	INFO	cleanup/konnect_runtime_groups.go:103	No control planes to clean up
2024-06-27T07:03:51.6691892Z 2024-06-27T07:03:51.668Z	INFO	cleanup/konnect_runtime_groups.go:120	No control planes to clean up, skipping listing roles
2024-06-27T07:03:51.6693786Z 2024-06-27T07:03:51.668Z	INFO	cleanup/konnect_runtime_groups.go:146	No roles to delete


adr: make monitoring forwarding storage independent

Thanks @viktor-f and @Xartos !
I was meaning to take this decision both for Prometheus and Fluentd when used as forwarders. I double-checked and that is the default for Fluentd (and we used it as such):

https://github.com/bitnami/charts/blob/master/bitnami/fluentd/values.yaml#L421
https://github.com/bitnami/charts/blob/master/bitnami/fluentd/templates/forwarder-daemonset.yaml#L123
https://github.com/bitnami/charts/blob/master/bitnami/fluentd/templates/forwarder-daemonset.yaml#L165

And Prometheus (and we do not yet use it as such):

https://github.com/prometheus-community/helm-charts/blob/main/charts/kube-prometheus-stack/values.yaml#L2034
https://github.com/prometheus-operator/prometheus-operator/blob/master/Documentation/user-guides/storage.md#storage

Hence, this ADR really only "creates" one issue: "Set storageSpec of Prometheus on WC to {}".
We are still figuring out how the ADRs should fit in the product pipeline. To me, they should communicate both internally and externally the direction we take and not be a TODO list. They should act as a point of alignment between marketing/product/operations. Sometimes, we will need to take action (documentation, tutorials, code, new OPS routines), sometimes not. Of course, the first ADRs focus on things we need to change, but future ADRs could just as well focus on documenting past decisions without them needing any action. A bit like the IT policy documenting "use full-disk encryption" without me having to move a finger. :smile:
Could I get your feedback on?

Do you agree for this ADR to apply also to fluentd, or should that be further discussed?
Should I document more clearly that the decision applies to both fluentd (when used for forwarding, not aggregation) in addition to Prometheus, as well as the current default?
Do you have any input on how you would like the ADRs to fit in the product pipeline?

First a note that you might have missed: What you said about fluentd is true for the chart that we use in SC but not the chart that we use in WC, there we only have forwarders that send logs directly to elasticsearch. They keep their buffer in a hostpath (https://github.com/kokuwaio/helm-charts/blob/7f45b731d59ebac7f082f7d9f2215be15dffa84b/charts/fluentd-elasticsearch/values.yaml#L130)

Because of my note about SC fluentd I'm not sure if this applies to both fluentd and prometheus.
Yes, I think it should be clarified what types of fluentd this would apply to (forwarder in sc, aggregator in sc, and forwarder in wc)
Not sure if you mean the ADRs by themself or the changes needed to align with new ADRs. I think that writing ADRs can be done out of band from the product pipeline via architecture meetings and other discussions. But if we need to do any tasks (write some code, write documentation except the ADR, update OPS routines, etc), then we should write issues and prioritize them though the normal product pipeline.

@viktor-f @Xartos After having slept over this, I see no obvious reason why fluentd in DaemonSet should not be used with hostPath. Hence, I clarified that this ADR allows that. If we see a problem ... let's cross that bridge when we're at it. :smile:

Not sure if you mean the ADRs by themself or the changes needed to align with new ADRs. I think that writing ADRs can be done out of band from the product pipeline via architecture meetings and other discussions. But if we need to do any tasks (write some code, write documentation except the ADR, update OPS routines, etc), then we should write issues and prioritize them though the normal product pipeline.

Great feedback! We are slowly moving in that direction.


migrator: recommend upgrade.

You’re in a weird middle state if you run brew upgrade but haven’t upgraded a migrated formula.

if you run brew upgrade

-> if you run brew update


Add webhooks for collections create/update/delete

Adds COLLECTION_CREATED, COLLECTION_UPDATED and COLLECTION_DELETED events and webhooks for those.
Impact

[ ] New migrations
[x] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[x] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[x] The changes are tested
[x] GraphQL schema and type definitions are up to date
[x] Changes are mentioned in the changelog

PR docs: https://github.com/saleor/saleor-docs/pull/349


Add backgroundOperation method to the storage

Issue: Link
Short description
Add support for background operations from the storage.
Solution

Extend the Storage protocol.
Add an implementation in the classes that conform the Storage protocol.

GIF

Solves https://github.com/carambalabs/SugarRecord/issues/311

Before moving ahead with tests, what do you think about the interface @nazywamsiepawel & @amayatsky

Hey, I think that the interace is good and would be a good solution for the previous issue #311
The biggest differentiator will be using perform instead of performAndWait which seemed to deadlock quite often. Thanks @pepibumur!

Looks good. :)


Modify Comments: Configuration.StrictMap put(String key, V value) …

When key exists , the exception message already contains value for... is thrown,
which is inappropriate and should be prompted for already contains key for....

tests are dependent on the message there so the tests would need updated as well.  Please make sure to run full build before sending up code as this should have been obvious.  Please make adjustments then this can be merged in.

tests are dependent on the message there so the tests would need updated as well. Please make sure to run full build before sending up code as this should have been obvious. Please make adjustments then this can be merged in.

Ok, I will do this.

The test case has been updated.

The original message looks OK to me.
If we want to make it super explicit, it can be "... already contains value for key 'xxx'", maybe.

The original message looks OK to me.
If we want to make it super explicit, it can be "... already contains value for key 'xxx'", maybe.

The put method of the StrictMap class uses the containsKey method to determine whether the key is contained. If it is contained, it will throw "already contains value for " + key. This is not appropriate to describe the if condition. It should throw "already contains key for " + key, the original code is as follows
    @Override
    @SuppressWarnings("unchecked")
    public V put(String key, V value) {
      if (containsKey(key)) {
        throw new IllegalArgumentException(name + " already contains value for " + key
            + (conflictMessageProducer == null ? "" : conflictMessageProducer.apply(super.get(key), value)));
      }
      if (key.contains(".")) {
        final String shortKey = getShortName(key);
        if (super.get(shortKey) == null) {
          super.put(shortKey, value);
        } else {
          super.put(shortKey, (V) new Ambiguity(shortKey));
        }
      }
      return super.put(key, value);
    }

My English is not great, but "key for 'xxx'" is confusing, at least.
"a map contains key for 'xxx'" implies 'xxx' is a value.
It is either "a map contains value (or entry) for key 'xxx'" or simply "a map contains key 'xxx'".

My English is not great, but "key for 'xxx'" is confusing, at least. "a map contains key for 'xxx'" implies 'xxx' is a value.
It is either "a map contains value (or entry) for key 'xxx'" or simply "a map contains key 'xxx'".

Ok, I think simply throw message "a map contains key 'xxx'" will better good.

The original message looks OK to me. If we want to make it super explicit, it can be "... already contains value for key 'xxx'", maybe.

I thought the same, changes however do reflect the physical check its doing.

Coverage: 87.545% (-0.009%) from 87.555% when pulling e5d74e007b9f3d584ef81021f85325a6bc1a39dd on wayn111:master into 9e43552af312ec0a4f8bee9fe99a83b4865bf4b3 on mybatis:master.


feat(Pattern): add new Pattern component

Overview
added a Pattern component that takes in a width and height and the name of the svg to use in the pattern
for now, there are 3 options: diagonalStripesLoose, diagonalStripesRegular, and diagonalStripesDense

also modified the ProgressBar to give it a stripes prop that makes a new version of the ProgressBar that uses the striped background:

add SVG Pattern component and a striped version of ProgressBar

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.3.13-alpha.268e62.0@codecademy/gamut-labs@7.7.3-alpha.268e62.0@codecademy/gamut@19.9.3-alpha.268e62.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-illustrations@0.3.13-alpha.268e62.0@codecademy/gamut-labs@7.7.3-alpha.268e62.0@codecademy/gamut@19.9.3-alpha.268e62.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5ff8dd84e0e7ce1ef14abb2a--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5ff8dd84e0e7ce1ef14abb2a--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.3.13-alpha.324c48.0@codecademy/gamut-labs@7.7.3-alpha.324c48.0@codecademy/gamut@19.9.3-alpha.324c48.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-illustrations@0.3.13-alpha.324c48.0@codecademy/gamut-labs@7.7.3-alpha.324c48.0@codecademy/gamut@19.9.3-alpha.324c48.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5ff8df93c3e2a81ccdbbf8fc--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5ff8df93c3e2a81ccdbbf8fc--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5ff922eb0b76293dab030bae--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5ff922eb0b76293dab030bae--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.3.13-alpha.4e8058.0@codecademy/gamut-labs@7.7.3-alpha.4e8058.0@codecademy/gamut@19.9.3-alpha.4e8058.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-illustrations@0.3.13-alpha.4e8058.0@codecademy/gamut-labs@7.7.3-alpha.4e8058.0@codecademy/gamut@19.9.3-alpha.4e8058.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-illustrations@0.3.13-alpha.0b848c.0@codecademy/gamut-labs@7.7.3-alpha.0b848c.0@codecademy/gamut@19.9.3-alpha.0b848c.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-illustrations@0.3.13-alpha.0b848c.0@codecademy/gamut-labs@7.7.3-alpha.0b848c.0@codecademy/gamut@19.9.3-alpha.0b848c.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5ffcab7a012c2e00c40a176a--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5ffcab7a012c2e00c40a176a--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.3.13-alpha.76c5e2.0@codecademy/gamut-labs@7.7.3-alpha.76c5e2.0@codecademy/gamut@19.9.3-alpha.76c5e2.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-illustrations@0.3.13-alpha.76c5e2.0@codecademy/gamut-labs@7.7.3-alpha.76c5e2.0@codecademy/gamut@19.9.3-alpha.76c5e2.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5ffdd7b7790ebf409d3b2d12--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5ffdd7b7790ebf409d3b2d12--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.3.13-alpha.5be5d1.0@codecademy/gamut-labs@7.7.3-alpha.5be5d1.0@codecademy/gamut@19.9.3-alpha.5be5d1.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-illustrations@0.3.13-alpha.5be5d1.0@codecademy/gamut-labs@7.7.3-alpha.5be5d1.0@codecademy/gamut@19.9.3-alpha.5be5d1.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5ffddc2859dbe94d0059aad4--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5ffddc2859dbe94d0059aad4--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.1-alpha.6c0de2.0@codecademy/gamut-labs@7.7.4-alpha.6c0de2.0@codecademy/gamut@19.9.4-alpha.6c0de2.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.1-alpha.6c0de2.0@codecademy/gamut-labs@7.7.4-alpha.6c0de2.0@codecademy/gamut@19.9.4-alpha.6c0de2.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5ffde3408355f00098d7b4ff--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5ffde3408355f00098d7b4ff--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.1-alpha.747536.0@codecademy/gamut-labs@7.7.4-alpha.747536.0@codecademy/gamut@19.9.4-alpha.747536.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.1-alpha.747536.0@codecademy/gamut-labs@7.7.4-alpha.747536.0@codecademy/gamut@19.9.4-alpha.747536.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5ffdef39479ce408114d9f21--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5ffdef39479ce408114d9f21--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.1-alpha.e8f921.0@codecademy/gamut-labs@7.7.4-alpha.e8f921.0@codecademy/gamut@19.9.4-alpha.e8f921.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.1-alpha.e8f921.0@codecademy/gamut-labs@7.7.4-alpha.e8f921.0@codecademy/gamut@19.9.4-alpha.e8f921.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5ffe038489b09a0d0225d5cd--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5ffe038489b09a0d0225d5cd--gamut-preview.netlify.appDeploy Logs


Canvas included in LitElement + Test Source Added

Canvas is now included in the html rendered by the LitElement.
LitElement can access shadowroot after first update so the draw function has been moved there.
Example works.

Have added the test source for assistance in development.


Post Install Hooks Should Run Relative to Course.yml Path

This would help make reckoner behavior more consistent for teams that may not consistently be running a reckoner plot from the same path.

💡 Made me think about the concept of a workingpath for the hooks with the default being os.path.dirname(course.yml)


Colormap pulldown for 2D histogram

This PR solves Issue #225.
It creates a colormap dropdown menu in the advanced options section of the Plotter Widget (only visible when HISTOGRAM_2D is selected).
There, the user can choose between the following matplotlib colormaps:

magma (default)
viridis
plasma
PRGn
PuOr
coolwarm

Thank you so much @zoccoler and @haesleinhuepf for your help and guidance through the code!

Awesome @marabuuu ! One idea: Do you think it is possible to replace our custom list of colormaps with the napari one. It took me a moment to find it, but I think napari.utils.colormaps.ALL_COLORMAPS.keys() could do the job. It's a list of colormap names supported by napari:

Big thanks for working on this!

Hi @haesleinhuepf,
Thanks for the hint. I will change it.

It should work now

Hi @marabuuu ,
I tested it and it is working, thanks!
I just added this line to auto-update the plot once the combobox changes, making its behavior similar to the other combobox.
I am just going to check if we can use the same container for both the plotting type and the colormap choice. Then, we could merge for next release!

Big thanks for working on this @marabuuu and @zoccoler !


Use ros2 and run examples but Oak-d indicates that the connection fails

cmd :ros2 launch depthai_examples stereo.launch.py

Hello,
Sorry fro the issue.

Can you check if the stereo works on depthai_demo.py ?
Can you cross-check if the right DepthAI C++ library is installed.

Run python3 depthai-demo.py but apparently it's fine so I don't know what the problem is

In ros installation steps did you install the main branch ? And also when installing the dependencies was there any change ?

I installed it according to the tutorial,command is sudo apt install ros-galactic-desktop
.Dependencies are installed automatically,I wonder what has changed about it.

Can you run the Installation dependencies here followed by cleaning the WS and rebuilding the packages?

Follow the depth-ros tutorial ,i coclon build it and It say that the compilation succeeded,.Run to this step source devel/setup.bash,it  says no such file,so i put it into the ros2 project and compile it again.

devel/setup.bash is for ros 1. install/setup.bash is for ros2.

Thank you for your help
1.It is depthai-ros project,It's just that the name is wrong.
2.depth-ros Dependencies are perfectly installed.
2. According to the tutorial, Dependency installation is based on install_dependencies.sh，Its compile and install prompts are successful.
3. Do I need to try all the steps again including ro2 installation

And you are still facing the issue ?
3. No.

yes,I still don't know what his problem is.I have to put the point cloud data into ROS2，It's an engineering project.

put

Can you share the full logs on the terminal ?

log.txt
This is the log

Ok. In depthai_demo.py can you share the left camera image on the display ?

And logs from depthai demo too.

Is that the choice

No. Once the demo starts you have to change the camera where I have marked to left. And also share the logs printed in the terminal when running the demo

It doesn't seem to have that option

that means like Erik mentioned the connector might be disconnected or you have OAK-1.
I think your connector might be disconnected.

Yes. Or we can get it replaced I think. Do you have an idea when it was purchased ?

I bought it in December 21, I tried to repair it first to see.

thanks you for your help, The problem was solved.


Testing 2022-rc1

Testing the 2022 release candidate.

Switching to 2022 branch, now this has been released.


Improve some poor API design decisions for Talck

There's a few mediocre API decisions in the public API of Talck. These would require a v4 to fix. If there is interest in these changes for their own sake, let me know. Otherwise these will be incorporated along with future breaking changes, should they occur.
These are mostly notes for future me, but feel free to comment or add suggestions.

Talck should hide it's internal structure and provide methods like lock, take, etc. (currently, lock exists as Talck::talc which I personally find unintuitive). Writing TALCK.0.[whatever] looks ugly.
Talck should probably implement the Allocator API directly, rather than through TalckRef - it's not a useful abstraction as opposed to &Talck
I'm of two minds about the current publicity and state of the lock_assume_single_threaded and associated infrastructure. Not sure if it's worth changing, but I avoid drawing attention to it in any of the docs, preferring direct use of new_global for WASM projects (not 100% comfortable with that function name) and otherwise figuring out your own locking solution.

The changes are in with v4


Secure kernel operation

This is a big one, but I think we have a good starting option in SES https://ses-secure-ecmascript.readthedocs.io/en/latest/index.html
SES will allow/require us to restrict access to privileged functionality like file access or network access.
The best short argument for just using SES is because I don't want to tackle the security problems ourselves and the Agoric folks have some solid mojo in this area. It isn't perfect, but I am sure it's better than rolling our own.
The best "it's complicated" argument--even with SES--is that we need a good model of the security threats we care about and a way for authors to specify the privileges needed so users can see and/or approve those privileges manually. (And the UX for such permissioning needs thinking through).
For example, some projects won't need anything but require access to a specific folder (to load the module). Others might need network access (outbound or inbound) or file read/write/execute access.
The BIG goal would be to make these choices feel like BIG, comfortable buttons that just make sense rather than a checklist of mind-numbing capabilities that need to be permissioned. This will require some smart modeling of the kind of privileges we want to enable and prevent, including Full Lock Down and Wild Wild West settings. THEN, we'll need to settle on a mechanism WITHIN the markdown (because that's the contextual boundary, although we could potentially support a manifest .mf file with the same base filename, that gets complicated). People should be able to email or download markdown files and if the right manifest hints are there, it just works.

[ ] Threat model
[ ] Privilege isolation (SES)
[ ] Privilege specification (within markdown)
[ ] Privilege management (within app, as permissions/consent)

/move to satyrnjs/satyrn


Feat/live 2174 dynamic cal

📝 Description
We sometimes need to add erc20 tokens out of a release cycle.
There was some issues since we are using electron with two thread, the renderer and the internal doesn't share the same tokens list so we need to initialize / preload the data on both side.
We also work on a automatic PR to be able to push update from the crypto-assets repository toward an S3 (still WIP waiting for the bucket)
https://github.com/LedgerHQ/ledger-live/actions/runs/3061492352
We also need to work on mobile to be sure everything is working
❓ Context

Impacted projects: live-common live-desktop
Linked resource(s): [LIVE-2174](https://ledgerhq.atlassian.net/browse/LIVE-2174)

✅ Checklist

[ ] Test coverage
[ ] Atomic delivery
[ ] No breaking changes

📸 Demo

🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

It's seems to work fine now :
https://github.com/LedgerHQ/ledger-live/actions/runs/3136154840/jobs/5092709009
So it's now accessible by this CDN :
https://cdn.live.ledger-stg.com/cryptoassets/erc20.json
thanks to infra team !

Im drafting it because I did one revert on new CAL to able QA to test with dynamic CAL


Fix voice msg error

当消息为语音时，rawPayload.content 最近测试已经不是xml string了，会导致计算voiceLength出错，调试发现voiceLength已经是rawPayload的属性了，直接替换即可

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

Wechaty Contributors Nomination
Link to https://github.com/wechaty/PMC/issues/16

Thank you very much for your contribution!
You are welcome to join Wechaty Contributor Program
1. Join Wechaty Organization

You've invited Daniel to Wechaty! They'll be receiving an email shortly. They can also visit https://github.com/wechaty to accept the invitation.

I have invited you to join our Wechaty GitHub Organization. Please accept it by following the above message. (See also: wechaty/PMC#16)
2. Update Your Wechaty Contributor Profile

Please open Contributor Hall of Fame and add yourself to the end of the list, so that other contributors will know you better!
Please add yourself to our Website Contributors by creating a PR and refer to this PR link as well.

3. Join the Wechaty Official Developers Community
On Discord at here: <https://discord.gg/7q8NBZbQzt >
4. Join The Contributor-Only WeChat Room
We also have a WeChat room for contributors who can discuss Wechaty at a deeper level; you are welcome to join if interested.
Please add @lijiarui wechat: ruirui_0914 and send her this PR link. She will invite you to the Wechaty Contributor Room on WeChat.
Cheers!


feat: overhaul

this PR closes https://github.com/catppuccin/rofi/issues/29
Changed/Added:
moved from shipping dotfiles to shipping:

rofi.tera: defines convenient variables in the global scope for the colors in each flavor.
catppuccin-default.rasi: the default theme from Rofi with catppuccin colors.

missing previews.


Not working for pull_request events?

On https://github.com/Neovici/cosmoz-treenode/pull/17/checks?check_run_id=209177055 I got:
2019/09/01 15:10:48 no handler for the received event type "pull_request"

Sorry for the late response, I accidentally disabled notifications for this repo. Since this action does not work with the release version of Github actions anymore, I am going to archive it. I guess the recommended alternative are Problem Matchers now. Here is an example problem matcher for eslint: https://github.com/actions/setup-node/blob/master/.github/eslint-compact.json


ARC 2557 - UI for invalid request ID

What's in this PR?

Updated the UI for invalid/expired request ID.
Refactored the DeferredInstallation component.

Separated out the different flows into distinct components.

Why

Final touches

How has this been tested?

Locally

Screenshot

So much code without tests?... :sadpanda:


#1489 add surface area to volume ratio factor

Description
Add surface area to volume ratio factor
Fixes #1489
Type of change
Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s. If necessary, also add to the list of breaking changes.

[x] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist:

[ ] No style issues: $ flake8
[ ] All tests pass: $ python run-tests.py --unit
[ ] The documentation builds: $ cd docs and then $ make clean; make html

You can run all three at once, using $ python run-tests.py --quick.
Further checks:

[ ] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works

Yes it's dimensionless


Remove Heap snippet

Google Analytics will do everything we need for now.
Let's remove Heap from;

index.html
public/index.html

[ ] https://github.com/OperationCode/operationcode_frontend/blob/master/public/index.html#L24

[ ] https://github.com/OperationCode/operationcode_frontend/blob/master/index.html#L23


putting dynamic headers in proxy requests

Similar to https://github.com/nuxt-community/proxy-module/issues/30 I'm trying to put a  dynamic header in proxy requests.  At the moment I have a piece of middleware that requests data via an API and puts it in the store.
I thought there would be a template for creating this issue, but didn't immediately see one.  Will happily edit request/question if needed.
Many thanks in advance

I found a way round it by having the store api getters access the id set by the middleware.  Maybe the proxy is just not meant to have store access?  I'll do a reproduction if it becomes critical to have this happen in the proxy ...


continueInApp() only works every other time

I'm having issues with continueInApp only working every other time. First time it will open the main app as expected. The next time I share something, the share extension just closes as soon as continueInAppis called, and the main app never opens.
I've worked around it by dismissing the extension a while after calling continueInApp:
        ShareMenuReactView.continueInApp();
        setTimeout(() => {
          ShareMenuReactView.dismissExtension();
        }, 500);

which makes the main app open every time. Am I missing something? (very possible)

@vonstring I've noticed the same thing from my end. I'm seeing the following behavior on iOS (haven't tested Android yet)...
For images, the ShareMenuModuleComponent always loads properly the first time upon clicking my app's icon in the share sheet. However, if I keep my phone's Photos app open and try again, the first time trying to share a second image always fails to bring up the ShareMenuModuleComponent. If I click my app's icon in the share sheet a second time after this first failure, it seems to always work as expected.
The below screenshot is the output from my Metro terminal. The first call to the index.share.js file is unsuccessful (hence there's no log of ShareMenuModuleComponent running) and the second call to index.share.js is successful and all works according to plan. Again, the failures always seem to occur after a successful journey into the app.

Note that if I close the Photos app and start fresh for a second image, it always seems to work. So the issue is when trying a second time without closing the native Photos app.
The above behavior is the exact same for files in the native iOS Files app. However I've noticed that I can't get audio files from Voicemail to ever succeed the second time. I need to navigate out of Voicemail and then come back in for a second audio file upload to work. Again, first time always works, successive tries (without closing the app) always fail.
@caiosba @Gustash please let us know if there's a better way to utilize your APIs (like dismissExtension referenced above) or if there's a native iOS fix to completely wipe the state clean after the first, successful pass through the share extension flow.
I appreciate the support!
Dan

@DanC5 were you able to solve it?

@DanC5 were you able to solve it?

I was not able to solve this, outside of the work around referenced in the original post. A useEffect in my code making the calls looks like this...
useEffect(() => {
    ShareMenuReactView.data()
      .then(({ data }) => {
        const [shareData] = data as unknown as ShareData[];

        if (shareData.data && shareData.mimeType) {
          ShareMenuReactView.continueInApp();
          setTimeout(() => ShareMenuReactView.dismissExtension(), 500);
        }
      })
      .catch(err => {
        console.log(err);
        ShareMenuReactView.dismissExtension();
      });
  }, []);

I see the same issue :|

This is the fix
https://github.com/meedan/react-native-share-menu/pull/156/commits/373a30d9596796011ff31bbacb1f7f70f59e509c


feat(dal,sdf,app): Reimplement variant_defs endpoints

As we are moving away from the concept of SchemaVariantDefinitions, we want to move the metadata that was part of a SchemaVariantDefinition to be part of the SchemaVariant
This will allow us to start bringing back the authoring workflow without the need to have the concept of a SchemaVariantDefinition
This PR also re-implements the get_variant_def endpoint as well as renaming variant_def to be variant in the API routes

bors try

bors try

bors try

Three things:

I was on autopilot and accidentally brought my review style for #3475. I was specifically asked for a nit-heavy, style and convention scoped review there. Apologies to @stack72!
I'm going through my original review here to delete, edit, and/or comment on existing comments.
@stack72 let me know that this isn't done yet too, so I'll keep that in mind when doing the above.

lgtm, executed basic sanity test of product and everything works.

bors try

bors merge


remove PLT_CAR_L4_SHAKER (August 1)

https://github.com/PyLabRobot/pylabrobot/pull/188/

https://github.com/PyLabRobot/pylabrobot/commit/59fdcb78656db320f9ba48fbc37259b80193a4bb
tag @BioCam


Blog section is unresponsive

Describe the bug
The blog section is not responsive at all
Screenshort

I request you to assign me this issue under GSSoC'23

Can i also work on this but on a separate field.