Helm upgrade with conditional dependencies

requirements.yaml:
 dependencies:
 - name: "mysql"
   version: "5.7"
   repository: "helm-repo.local"
   condition: "mysql.enabled"

values.yaml:
"mysql": {"enabled": false}

With helm install chart --name examplerelease --values values.json the mysql subchart is not installed.
Then helm upgrade examplerelease chart --reuse-values ignores the mysql.enabled=false value and installs the mysql subchart which is not the expected behavior.
In contrast, with helm upgrade examplerelease chart --reuse-values --set mysql.enabled=false mysql subchart is skipped, but in this case I am forced to set again the mysql.enabled value.
Output of helm version:

Client: &version.Version{SemVer:"v2.14.3", GitCommit:"0e7f3b6637f7af8fcfddb3d2941fcc7cbebb0085", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.14.3", GitCommit:"0e7f3b6637f7af8fcfddb3d2941fcc7cbebb0085", GitTreeState:"clean"}

Output of kubectl version:

Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.2", GitCommit:"81753b10df112992bf51bbc2c2f85208aad78335", GitTreeState:"clean", BuildDate:"2018-04-27T09:22:21Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"11+", GitVersion:"v1.11.0+d4cacc0", GitCommit:"d4cacc0", GitTreeState:"clean", BuildDate:"2019-03-29T12:30:44Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):

OpenShift Origin 3.11

did you make sure to run helm dependency build or helm dependency update before installing? The chart needs to be available in the charts subdirectory prior to install for it to be installed. Helm will not resolve dependencies prior to install unless you provide the --dep-up flag.

The subchart is in the charts directory.
I attached a tiny example. It contains a main chart with a configmap (configmap-main) and a subchart (with condition: "subchart.enabled") also with a configmap (configmap-sub). The subchart.enabled is true by default in the chart's values.yaml file, but false in the used install-values.yaml:
subchart:
  enabled: false

I Tried the following:
Package the helm chart:
$> helm package main --version 0.0.1
Successfully packaged chart and saved it to: /home/csupi/example/main-0.0.1.tgz
Install on OpenShift:
$> helm install main --values install-values.yaml
NAME:   piquant-clownfish
LAST DEPLOYED: Mon Sep  9 08:13:39 2019
NAMESPACE: helm-issue
STATUS: DEPLOYED
RESOURCES:
==> v1/ConfigMap
NAME            DATA  AGE
configmap-main  1     0s
Upgrade without any changes:
$> helm upgrade piquant-clownfish main --reuse-values
Release "piquant-clownfish" has been upgraded.
LAST DEPLOYED: Mon Sep  9 08:13:53 2019
NAMESPACE: helm-issue
STATUS: DEPLOYED
RESOURCES:
==> v1/ConfigMap
NAME                DATA  AGE
configmap-main      1     14s
configmap-subchart  1     0s
As you can see the upgrade installed the subchart despite it is disabled in the values file and release values:
$> helm get values piquant-clownfish
subchart:
  enabled: false

main.zip
Did I miss something?

 This clearly looks like a bug to me. Will have to check what's causing this behavior

This happens when using --reuse-values but it works fine when explicitly using --values during the upgrade. We'll check on this 😄

Similar to what I noticed in helm v3 https://github.com/helm/helm/issues/6530#issuecomment-536318742 I'm having suspicion on the processing of chart dependencies before sending to tiller in helm v2 too. And only in tiller existing values is known. Here too I have to confirm the suspicion. This is just a hunch 😅

The hunch for #6530 was correct and I raised a PR for it here #6556
In v2, since there's a client and a server (tiller), I noticed that on the client side we do processing of conditions over here
https://github.com/helm/helm/blob/9fb9850aca4afab0d38bcd73218098a145b4e7ac/pkg/helm/client.go#L185-L192
I think this has to be done on the server side after getting the values - since the reuse values has been enabled - we get values only in server and then we can do the processing. And I think processing of conditions can be always be done on server side and not just when reuse values is enabled . Have to see how to write a test case for this and then try to fix it
What do you think about this @bacongobbler ? Let me know. As I don't want to put lots of effort (like this #6391) on a wrong / non viable solution and then have the PR closed 🙈 😅 for an alternative PR

Hi there, I've got the same issue when trying to run helm upgrade --reuse-values: the conditional dependencies seem like ignored. Apart from migrating to helm 3, is there any workaround? I'm using helm/tiller v2.15.2.

@utay nothing that I know of. If you are interested to work on a PR for this, we can help you with it 😄

@karuppiah7890 Erf, I'm afraid that it's worth it for us to migrate to helm 3 then... 😛But thank you for the quick response and the proposal!

Haha. And migrating to Helm 3 comes with lots of perks, so it's worth it !! 😄

I am still seeing this bug in
verrsion.BuildInfo{Version:"v3.2.1", GitCommit:"fe51cd1e31e6a202cba7dead9552a6d418ded79a", GitTreeState:"clean", GoVersion:"go1.13.10"}

@csupi As mentioned above in this issue, we recommend moving to Helm v3, as v2 is being deprecated: https://helm.sh/blog/helm-v2-deprecation-timeline/
Please let us know if you experience problems with v3. Thanks!


Add packages required by Sphinx.

Since pip install is passed --no-deps, we need to explicitly list
Sphinx's dependencies in requirements.txt in order for docs generation to
work.

Automated message from Dropbox CLA bot
@christi3k, it looks like you've already signed the Dropbox CLA. Thanks!

Can you use hardcoded (fixed) versions, rather than version ranges, and also add comments of the form # dependency of sphinx for each of these?  The latest versions available today are pretty reasonable.  We want to make sure everything is 100% reproducible in our environment, and that means fixing the versions of all of our dependencies.

@timabbott Do you recommend that I fix the version to the lower end of the range in each case? Or to  which versions are actually installed in my working instance? I took these directly from what's indicated in the sphinx pip package, which uses ranges, so I'm not sure how to identify which are the ideal versions to specify.

I'd use the versions that were installed in a working instance (usually that's the latest versions available).

@christi3k I just noticed you'd pushed a new version; does that indicate you're done with changes to this?  A (un)fun fact about GitHub is that it doesn't send any sort of notification when you push a new version, so you need to post a comment to get the reviewer's attention again (I just now noticed you pushed something).
Also, can you rebase past the merge conflicts?

I went ahead and did the rebase and merged this, thanks @christi3k!


can i import css file in app/dist/index.html ?

Eh, what do you mean by import css file ? app/dist/index.html is just a regular html file.

when i run 'npm run app'
this file reset to the original version

is my mistake ,sorry


GUI.py error log ( Linux )

/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/setuptools/sandbox.py:13: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
import pkg_resources
/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/pkg_resources/init.py:2871: DeprecationWarning: Deprecated call to pkg_resources.declare_namespace('mpl_toolkits').
Implementing implicit namespace packages (as specified in PEP 420) is preferred to pkg_resources.declare_namespace. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
declare_namespace(pkg)
/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/pkg_resources/init.py:2871: DeprecationWarning: Deprecated call to pkg_resources.declare_namespace('google').
Implementing implicit namespace packages (as specified in PEP 420) is preferred to pkg_resources.declare_namespace. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
declare_namespace(pkg)
/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/torchvision/transforms/_functional_pil.py:242: DeprecationWarning: BILINEAR is deprecated and will be removed in Pillow 10 (2023-07-01). Use Resampling.BILINEAR instead.
interpolation: int = Image.BILINEAR,
/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/torchvision/transforms/_functional_pil.py:288: DeprecationWarning: NEAREST is deprecated and will be removed in Pillow 10 (2023-07-01). Use Resampling.NEAREST or Dither.NONE instead.
interpolation: int = Image.NEAREST,
/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/torchvision/transforms/_functional_pil.py:304: DeprecationWarning: NEAREST is deprecated and will be removed in Pillow 10 (2023-07-01). Use Resampling.NEAREST or Dither.NONE instead.
interpolation: int = Image.NEAREST,
/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/torchvision/transforms/_functional_pil.py:321: DeprecationWarning: BICUBIC is deprecated and will be removed in Pillow 10 (2023-07-01). Use Resampling.BICUBIC instead.
interpolation: int = Image.BICUBIC,
Running Verify Fluid Program ...
W0126 19:30:46.957288  2808 gpu_resources.cc:61] Please NOTE: device: 0, GPU Compute Capability: 7.5, Driver API Version: 12.3, Runtime API Version: 11.7
W0126 19:30:46.957516  2808 dynamic_loader.cc:307] The third-party dynamic library (libcudnn.so) that Paddle depends on is not configured correctly. (error code is /usr/local/cuda/lib64/libcudnn.so: cannot open shared object file: No such file or directory)
Suggestions:

Check if the third-party dynamic library (e.g. CUDA, CUDNN) is installed correctly and its version is matched with paddlepaddle you installed.
Configure third-party dynamic library environment variables as follows:

Linux: set LD_LIBRARY_PATH by export LD_LIBRARY_PATH=...

Windows: set PATH by `set PATH=XXX;
[<class 'RuntimeError'>] In user code:
File "gui.py", line 394, in
subtitleRemoverGUI = SubtitleRemoverGUI()
File "gui.py", line 26, in init
fluid.install_check.run_check()
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/install_check.py", line 142, in run_check
test_simple_exe()
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/install_check.py", line 130, in test_simple_exe
simple_layer0 = SimpleLayer(input_size=2)
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/install_check.py", line 37, in init
self._linear1 = nn.Linear(
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/dygraph/nn.py", line 993, in init
self.weight = self.create_parameter(shape=[input_dim, output_dim],
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/dygraph/layers.py", line 471, in create_parameter
return self._helper.create_parameter(
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/layer_helper_base.py", line 381, in create_parameter
self.startup_program.global_block().create_parameter(
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/framework.py", line 3988, in create_parameter
initializer(param, self)
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/initializer.py", line 56, in call
return self.forward(param, block)
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/initializer.py", line 184, in forward
op = block.append_op(type="fill_constant",
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/framework.py", line 4040, in append_op
op = Operator(
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/framework.py", line 2879, in init
for frame in traceback.extract_stack():
PreconditionNotMetError: Cannot load cudnn shared library. Cannot invoke method cudnnGetVersion.
[Hint: cudnn_dso_handle should not be null.] (at /paddle/paddle/phi/backends/dynload/cudnn.cc:60)
[operator < fill_constant > error]
Traceback (most recent call last):
File "gui.py", line 394, in
subtitleRemoverGUI = SubtitleRemoverGUI()
File "gui.py", line 26, in init
fluid.install_check.run_check()
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/install_check.py", line 142, in run_check
test_simple_exe()
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/install_check.py", line 138, in test_simple_exe
exe0.run(startup_prog)
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/executor.py", line 1463, in run
six.reraise(*sys.exc_info())
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/six.py", line 719, in reraise
raise value
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/executor.py", line 1450, in run
res = self._run_impl(program=program,
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/executor.py", line 1661, in _run_impl
return new_exe.run(scope, list(feed.keys()), fetch_list,
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/executor.py", line 631, in run
tensors = self._new_exe.run(scope, feed_names,
RuntimeError: In user code:
File "gui.py", line 394, in
subtitleRemoverGUI = SubtitleRemoverGUI()
File "gui.py", line 26, in init
fluid.install_check.run_check()
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/install_check.py", line 142, in run_check
test_simple_exe()
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/install_check.py", line 130, in test_simple_exe
simple_layer0 = SimpleLayer(input_size=2)
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/install_check.py", line 37, in init
self._linear1 = nn.Linear(
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/dygraph/nn.py", line 993, in init
self.weight = self.create_parameter(shape=[input_dim, output_dim],
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/dygraph/layers.py", line 471, in create_parameter
return self._helper.create_parameter(
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/layer_helper_base.py", line 381, in create_parameter
self.startup_program.global_block().create_parameter(
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/framework.py", line 3988, in create_parameter
initializer(param, self)
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/initializer.py", line 56, in call
return self.forward(param, block)
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/initializer.py", line 184, in forward
op = block.append_op(type="fill_constant",
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/framework.py", line 4040, in append_op
op = Operator(
File "/home/user/miniconda3/envs/videoEnv/lib/python3.8/site-packages/paddle/fluid/framework.py", line 2879, in init
for frame in traceback.extract_stack():
PreconditionNotMetError: Cannot load cudnn shared library. Cannot invoke method cudnnGetVersion.
[Hint: cudnn_dso_handle should not be null.] (at /paddle/paddle/phi/backends/dynload/cudnn.cc:60)
[operator < fill_constant > error]

cudnn 环境没装好


Paste images from your clipboard in android

Fixes #231
Description
This PR makes use of a method channel to connect the native Android code with Flutter, enabling the retrieval of images from the clipboard. The key changes include:

Method Channel Implementation:
A method channel named clipboard_image_channel is created in the native Android code for communication with Flutter. This channel handles method calls to check the clipboard for images.

Base64 Encoding:
The image retrieved from the clipboard is converted to a Base64 string before being sent to the Flutter code. This allows for easier handling of the image data in Dart.

Image Storage:
The decoded Base64 image string is stored in the same list that is used to display images selected from the gallery, while maintaining the total count of allowed images, which is capped at 5. This ensures a friendly UX of handling images from both the clipboard and the gallery.

No image in clipboard handling:
Shows a snackbar(with a theme used for snackbar before).
New Dependencies in android folder

The following dependencies are required for this change:

androidx.core:core-ktx:1.10.1
androidx.appcompat:appcompat:1.6.1
androidx.activity:activity-ktx:1.7.2
io.flutter:flutter_embedding_debug:1.0.0

Key changes in android folder

Gradle version in use now - 7.5
android.tools.build:gradle - 7.4.2
replaced deprecated jcenter() to mavenCentral() under repositories config

Attached video demo of the fix
https://github.com/user-attachments/assets/a33b956b-5b34-4824-8d81-4a320e0e0934

I'll update the deprecated upload-artifact from .github/workflows/build-flutter.yaml for the checks, and since the flutter automatically made changes sentry plugin , I'll see to it , if i dont find a plausible cause , i'll revert the changes in the windows folder.


fix(backend): Replace deprecated http2 directive

Fixes #4060
This PR replaces the listen ... http2 directive with the new and not deprecated directive http2 on. This prevents warnings in the log output.

Closing in favour of #4106 that addresses these comments


fix(telemetry): improvised the telemetry event

Improved the broadcasting event for telemetry.
Added a single event name for all the analytics tracking user events.

@invincibleJai please review this.

:tada: This PR is included in version 1.0.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Will omnisharp.json still be supported?

I'm developing a container image where a directory is specified in the omnisharp.json to host code refactoring dlls, so they can be discovered by omnisharp. Will it still work in the new extension? Will it only work if I set dotnet.server.useOmnisharp to true?

The omnisharp.json file itself is associated with the O# server.  It will only work in the new version of the extension if you're using O# (setting dotnet.server.useOmnisharp to true as you mention).  omnisharp.json will not be supported in the new LSP server.
However - in the new LSP server we should still support loading additional code refactorings via nuget package references in your project, but we have not yet decided if we're going to support directly loading additional dlls directly in the extension.
Is supporting code refactorings loaded via nuget packages good enough for you, or is there a specific scenario where you can't reference them via nuget package and directly need to load them in the extension?

Since I was under the impression that deploying code refactorings with NuGet packages was not possibile, I was adding them through omnisharp.json. Although https://github.com/dotnet/roslyn/pull/35685 seems to suggest the contrary, I can't find a simple example of this.
Yes, NuGet package support is in fact enough, can't get it to work though - and I apologize since this is not the correct repo to ask this - but can you throw me a bone?
Packaging them to a NuGet package's analyzers/dotnet/cs does not seems to do the trick, even though analyzers .dlls in the very same directory get discovered just fine.

This is an area I'm not as familiar with to be honest, so I'll tag in @sharwell to help answer how to deploy refactorings in a nuget package.

They are deployed identically to code fixes in analyzer packages.

They are deployed identically to code fixes in analyzer packages. If you have an example package which is causing problems I could take a look and see if anything stands out.

Made up an ugly example, but easily reproducible: replacing the code refactoring from the "Code Refactoring (.NET Standard)" template to the code fix in the template "Analyzer with Code Fix (.NET Standard)".
The latter would normally deploy fine both to VSIX and NuGet. With the code refactoring in place of the code fix, it only deploys successfully to VSIX.
As NuGet package, it is placed under the analyzers section as it is packaged to "analyzers/dotnet/cs", but the UI prompting the refactor won't show up. Am I missing something?

I use the omnisharp.json file to define the location of an RSP file to load an assembly, in a scripting project context (csx):
/r:"C:/Program Files/MySoft/MyLibrary.dll"
How can I do this without using an O# server?
Thanks.

I have a scripting scenario as well.. Would be nice if we could specify additional references in an array in some .vscode .json file.. That way we could use environment variables instead of hardcoding absolute paths which change per machine.

we have not yet decided if we're going to support directly loading additional dlls directly in the extension

@dibarbet This may be the only way to get analyzer nuget packages to work in a Unity project. There is a way to set up these DLLs as Unity assets (https://docs.unity3d.com/2023.3/Documentation/Manual/roslyn-analyzers.html) but I am not sure that it would result in those analyzers being reported in VSCode.

@dibarbet This may be the only way to get analyzer nuget packages to work in a Unity project. There is a way to set up these DLLs as Unity assets (https://docs.unity3d.com/2023.3/Documentation/Manual/roslyn-analyzers.html) but I am not sure that it would result in those analyzers being reported in VSCode.

Hmm, I don't know how that works exactly - @jbevain do you know if analyzers added in unity by the above method get added to the generated csproj as well (maybe as  nodes?)

I wasn't able to get analyzer DLLs to work with Unity so gave up entirely. Wish I could have given better feedback but just couldn't get through various errors even after following and retracing the steps in their docs.

@marcospgp I use Unity 2021.3, Visual Studio Code 1.86.2 and Microsoft Unity Analyzers 1.15.0. Here is my current setup:
In the settings of Visual Studio Code Dotnet > Server: Use Omnisharp is set to true (because I still use Omnisharp and not C# DevKit).
Next to the *.sln file of the Unity project there is a omnisharp.json file with the following content:
{
  "RoslynExtensionsOptions": {
    "EnableAnalyzersSupport": true,
    "LocationPaths": ["./NuGet/Microsoft.Unity.Analyzers"]
  }
}

In the path referenced by LocationPaths in the omnisharp.json file there is the file Microsoft.Unity.Analyzers.dll.
To get that dll file go to https://www.nuget.org/packages/Microsoft.Unity.Analyzers/ and on the right side click "Download package". You will get a file named microsoft.unity.analyzers.*.*.*.nupkg. Open that that file with your favorite archive application, for example 7-Zip. Within the archive in the folder analyzers\dotnet\cs\ there is the file Microsoft.Unity.Analyzers.dll. Extract it.
Both files, omnisharp.json and /NuGet/Microsoft.Unity.Analyzers/Microsoft.Unity.Analyzers.dll, are added to my repository (git).

I use the omnisharp.json file to define the location of an RSP file to load an assembly, in a scripting project context (csx):

csx support is still an open issue, tracked here - https://github.com/dotnet/vscode-csharp/issues/6411
For the moment I would recommend sticking to O# for that.

@dibarbet This may be the only way to get analyzer nuget packages to work in a Unity project. There is a way to set up these DLLs as Unity assets (https://docs.unity3d.com/2023.3/Documentation/Manual/roslyn-analyzers.html) but I am not sure that it would result in those analyzers being reported in VSCode.

Supporting analyzers / code actions through direct dll references is something we will likely tackle at some point.  That can be tracked here - https://github.com/dotnet/vscode-csharp/issues/6790
Going to close this issue - if there are other specific features that omnisharp.json supported that you would like to see, please file a new issue specifically for that feature.  Its easier for us to track that way.


chore: update cmdlet names

Summary

Breaking Change: Renamed Set-SddcCertificateAuthority to Set-VCFCertificateAuthority
Breaking Change: Renamed Request-SddcCertificate to Request-VCFSignedCertificate
Updated docs as per above changes.

Type

[ ] Bugfix
[x] Enhancement or Feature
[ ] Code Style or Formatting
[ ] Documentation
[ ] Refactoring
[ ] Chore
[ ] Other
Please describe:

Breaking Changes?

[x] Yes, there are breaking changes.
[ ] No, there are no breaking changes.

Test and Documentation

[x] Tests have been completed.
[x] Documentation has been added or updated.

Issue References

Closes #73
Additional Information

Applied suggested changes.


Updated project dependencies and added debug module

Restored project's git repository URL
Replaced console.log usage to debug module (conflicts with central logger instance & performance improvement)
Fixed tests to run on local environments
Updated lib directory from build

Coverage increased (+0.05%) to 93.671% when pulling 777a28108ae7ed861e6605842625f5bb8ba5ce66 on LushaTeam:replace-console-log into 43928bdc6148552ee8947a26a8bb177a04cbff82 on idealley:master.


Fix Firefox/Android, using new webm file

This PR contains a:

[x] bugfix
[ ] new feature
[ ] code refactor
[ ] test update
[ ] typo fix
[ ] metadata update

Motivation / Use-Case
Fixes https://github.com/richtr/NoSleep.js/issues/63 by doing exactly what Phyks did (reconvert the mp4 to webm, replacing the existing webm). I haven't dug into the detail here, but it seems to work again on Firefox/Android now.

Breaking Changes

Additional Info

Hey,
tried this and still didn't work for me in firefox android. I'm importing it like this: import * as NoSleep from 'nosleep.js/dist/NoSleep' and using it in VueJS

Any updates? Yes its not working for me too @abbasnake

Any updates? Yes its not working for me too @abbasnake

No updates, I didn't dig deeper yet though


measure polymer-interop overhead

We need some benchmarks to know what's the impact of using a polymer.js element from Dart vs using a port of the same element.
There is probably value in having some things implemented in Dart, but there are probably many cases where it doesn't matter that a custom-element is running in JS.

This issue has been moved to dart-lang/polymer-dart#223.


adapt to rio-tiler 1.0.0

see https://github.com/mapbox/rio-tiler/pull/6

There are some weird things happening !
What

I've been  noticing random black tiles when playing with the demo at viewer.remotepixel.ca. Those tiles seemed to appear when doing a lot of panning.
Then I noticed something really strange 👇

This is what append when you call the same tile twice in a row (using hot lambda container). So basically, when calling for a tile ta first it will work but then some caching mechanism will return black tiles for all latest calls.
something is happening here:
https://github.com/mapbox/landsat-tiler/blob/65072ecae93aab153fd820dc0ca4d605777a409c/app/landsat.py#L91-L100

when removing the rescaling part (L94-98)  the black tile do not happen.
this is only for the tiles endpoint (not for the processing one).

👇 this is a log for the 2 consecutive request where I'm logging the min and max of the tile variable
01:23:43 START RequestId: 420ad5b2-daed-11e7-82a7-2f2cc7d38d3b Version: $LATEST
01:23:44 [landsat-tiler] - [DEBUG] - 380 - 1263 #before rescaling
01:23:44 [landsat-tiler] - [DEBUG] - 22 - 157  #after rescaling
01:23:44 END RequestId: 420ad5b2-daed-11e7-82a7-2f2cc7d38d3b

01:23:46 START RequestId: 4577e8af-daed-11e7-9995-190fb543c25f Version: $LATEST
01:23:46 [landsat-tiler] - [DEBUG] - 22 - 157 #before rescaling
01:23:46 [landsat-tiler] - [DEBUG] - 1 - 1 #after rescaling
01:23:46 END RequestId: 4577e8af-daed-11e7-9995-190fb543c25f

Basically the variable tile is cached and get reused and rescaled again 🎉


release: highlights of v0.4.0

/changelog

/changelog

/changelog

/release

/shipit

⚠ Failed rebasing release PR
Required status check "golangci-lint" is expected.

/shipit

🎇 🎉 🎆 New version has been shipped! 🚀🚀🚀


Fix: Undefined classes

This PR

[x] fixes the root cause of my favourite IDE™ complaining about undefined classes, which is, missing imports or weird aliases
[x] fixes a few invalid property-level comments
[x] fixes a few method-level comments with invalid @return annotations

Coverage remained the same at 81.02% when pulling 24dc60726d1bbf9ea7bd82c7c9d0886c6b346e04 on localheinz:fix/undefined-classes into 8ac20f17414229e5e8964e9e343d1fcfe3669192 on EvanDotPro:master.


The generated JS overwrites a valid export with an invalid one

I'm submitting a bug report
Webpack version:
2.1.0-beta.21
Please tell us about your environment:
Windows 10
Current behavior:
In the reproduction (see below), instead of just the (re-)export of the ViewEncapsulation member from metadata, webpack also generates an entirely invalid export with the same property name a from di.
Expected/desired behavior:
There shouldn't be a breaking, overwriting property definition.

What is the expected behavior?
sigh

What is the motivation / use case for changing the behavior?
So there isn't a bug.

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Language: [ES6]

Repro: https://github.com/Tragetaschen/angular-core-export-overwrite
Steps to reproduce:
$ npm install
$ webpack
$ node dist/main.js

Expected: An output of 0.
Actual:
C:\Users\kairu\Source\Repos\angular-core-webpack-export-overwrite\dist\main.js:41
/******/                Object.defineProperty(exports, name, {
                               ^

TypeError: Cannot redefine property: a
    at Function.defineProperty (native)
    at Function.__webpack_require__.d (C:\Users\kairu\Source\Repos\angular-core-webpack-export-overwrite\dist\main.js:41:19)
    at Object.<anonymous> (C:\Users\kairu\Source\Repos\angular-core-webpack-export-overwrite\dist\main.js:12855:54)
    at __webpack_require__ (C:\Users\kairu\Source\Repos\angular-core-webpack-export-overwrite\dist\main.js:20:30)
    at Object.AnimationKeyframe.AnimationKeyframe.styles.offset (C:\Users\kairu\Source\Repos\angular-core-webpack-export-overwrite\dist\main.js:12351:68)
    at __webpack_require__ (C:\Users\kairu\Source\Repos\angular-core-webpack-export-overwrite\dist\main.js:20:30)
    at Object.<anonymous> (C:\Users\kairu\Source\Repos\angular-core-webpack-export-overwrite\dist\main.js:15334:72)
    at __webpack_require__ (C:\Users\kairu\Source\Repos\angular-core-webpack-export-overwrite\dist\main.js:20:30)
    at exports.s (C:\Users\kairu\Source\Repos\angular-core-webpack-export-overwrite\dist\main.js:64:18)
    at Object.<anonymous> (C:\Users\kairu\Source\Repos\angular-core-webpack-export-overwrite\dist\main.js:67:10)

Reason:
12853: /* harmony namespace reexport */ __webpack_require__.d(exports, "a", function() { return __WEBPACK_IMPORTED_MODULE_0__metadata__["b"]; });
12854: /* unused harmony namespace reexport */
12855: /* harmony namespace reexport */ __webpack_require__.d(exports, "a", function() { return __WEBPACK_IMPORTED_MODULE_2__di__["j"]; });

Instead of just generating the export for ViewEncapsulation in line 12853, for some reason webpack also creates line 12855. The property j doesn't even exist in the di export.

Fixed in beta.22 https://github.com/webpack/webpack/commit/6d1f3e3fa3904f68844cf0e7633864e55a33e625


Remove compiled resources to allow overriding assets

Remove inadvertently/wrongly commited files into the resources folder
fixes #51

Close this PR as this fix is already in dev


Density matrix elements must be floats

SCF's get_j only seems to take in density matrices whose elements are floats. Integer elements result in entirely zero J matrices.

Could you please provide a small example to reproduce the error?

Fixed in PR #1266


Application icon and Rebranding atom to my own app name

Hi
So far this module helped me a lot for easy building. When can we expect Application icon and rebranding atom be available. Thanks in advance.

Judging from when you posted this question I'm going to assume never. Is this repo abandoned?

It's been a challenge carving out time this summer. I can give it a look in
the next few days.
On Aug 1, 2015 1:41 PM, "Kris Siegel"<EMAIL_ADDRESS>wrote:

Judging from when you posted this question I'm going to assume never. Is
this repo abandoned?
—
Reply to this email directly or view it on GitHub
https://github.com/entropi/grunt-electron-app-builder/issues/20#issuecomment-126940272
.


💡[FEATURE]: Dynamic Rebalancing Strategies for ETFs

Is your feature request related to a problem? Please describe.
Optimize the rebalancing strategy for exchange-traded funds (ETFs).
Describe the solution you'd like
Steps

Define the parameters for optimization (e.g., rebalancing frequency, asset allocation).
Initialize a population of rebalancing strategies.
Evaluate the fitness of each strategy based on historical ETF data.
Select the best-performing strategies.
Apply crossover and mutation to generate new strategies.
Repeat the process for a defined number of generations.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@Akshat111111 pls assign it to me I'm Gssoc'24 contributor


Missing Double Quotes in the Creating a Sawtooth Network Instructions

Double Quotes are missing in the "Creating a Sawtooth Network", step 1, #6.
This line:
sawtooth.poet.report_public_key_pem=$(cat /etc/sawtooth/simulator_rk_pub.pem)
Should be:
sawtooth.poet.report_public_key_pem="$(cat /etc/sawtooth/simulator_rk_pub.pem)"
Otherwise, the following error occurs:

sawset:error : unrecognised argument sawset proposal create -k

Signed-off-by: danintel<EMAIL_ADDRESS>

In case it's not obvious, the double quotes are needed because the output of
$(cat /etc/sawtooth/ias_rk_pub.pem) has embedded spaces, so double quotes are needed to indicate the PEM file contents are not separate parameters.
The  method of using doublequotes "$(cat ...)" is (correctly) used elsewhere in the docs:
https://sawtooth.hyperledger.org/docs/core/releases/latest/sysadmin_guide/configure_sgx.html
This bug fixes this documentation problem reported today on https://chat.hyperledger.org/channel/sawtooth :

May 22, 2018 3:54 AM
Hi, can some one please help me ! after entering this command into the terminal as a root , I am getting an error :
sawset:error : unrecognised argumentsawset proposal create -k /etc/sawtooth/keys/validator.priv -o config.batch sawset.consensus.algorithm = poet \ sawtooth.poet.report_public_key_pem=$(cat /etc/sawtooth/simulator_rk_pub.pem) \ sawtooth.poet.valid_enclave_measurements=$(poet enclave measurement) \ sawtooth.poet.valid_enclave_basenames=$(poet enclave basename)

@danintel - Are double-quotes also needed for the other two settings in this command?
sawtooth.poet.valid_enclave_measurements=$(poet enclave measurement) \ sawtooth.poet.valid_enclave_basenames=$(poet enclave basename)

@chenette No, double quotes are not needed.  They don't hurt, but they are not needed because there is no embedded whitespace for the other settings.


add mobile availability grid

Adds a mobile availability grid.  Right now this only detects browser width on initial page load, so dragging and resizing your screen won't do anything.

:+1: apt from comments


Curly bracket notation should be used by the "Extract the string resource" functionality.

Regarding the notes: https://github.com/long1eu/flutter_i18n#notes
It would be helpful if ExtractStringResourceDart.kt could use this when extracting strings. Why not always use the ${variable} notation?

@long1eu what do you think? I don't like enforcing this...

Maybe it could be configurable then? In some languages the space between words does not seem to make sense. If this was the default translaters would not have to know that they should modify the syntax in this case.

I got your point, but I don't see why we should always use the bracket notation. That is an exception from the rule, not the rule.


More control over file path and URL for remote assets

Hi team!
I'm working with remote assets, and this feature looks very promising. However, there's one feature that I'm missing: the ability to specify the URL to the file.
Describe the Feature
This feature would allow for more control over the URL of remote assets. Currently, the folder structure in the build folder generated for remote assets and the URL represents the folder structure in the repository. I would like to have more control over the URL. Therefore, I'm proposing to add an optional field to the remote object called filePath. If used, this field will control both the generated folder structure and the URL used to fetch the asset. It can be used as follows:
const filePath = getFilePath(...);
const fileHash = generateHashForFile(...);
remote: {
  enabled: isRemoteAsset,
  filePath: isRemoteAsset ? path.join(filePath, fileHash) : undefined,
  publicPath: getRemoteAssetsPublicPath(platform),
},

For example, assuming filePath is my/path/to/file and fileHash is abc123, the generated URL should be ${publicPath}/my/path/to/file/abc123/image.jpg, and this will also be the folder structure in the output folder.
Motivation
The goal is to address my use case where I need to inject a hash into the asset filename or place the asset in a folder named with the hash. I tried using the Output plugin to rename selected assets, but it did not affect the URL injected into the component pointing to the asset. In theory, I could just use publicPath for this and generate a different public path for every asset, but this would create a bit of a mess on the CDN. I created a small proof of concept to test if this is the right direction, although it is not ready to be merged. You can find more details here: https://github.com/adammruk/repack/pull/1.
Feel free to make any additional adjustments or let me know if I can solve my issue in a different way!

Hi @adammruk,
I love the proposal, this is definitely a functionality that should be there! I'm not sure about the final API, hear me out:
Remote assets functionality is something that very much resembles asset/resource from Webpack, so I was thinking of aligning the API to utilise that. Take a look here: https://webpack.js.org/guides/asset-modules/#custom-output-filename and also here: https://webpack.js.org/configuration/module/#rulegeneratorfilename since these options are related. This also allows use to re-use the templating functionality from Webpack.
WDYT? Since this is a breaking change, it would require a new major, we could also implement an experimental API using something similar (or exactly like that) to what you've provided.

Hi @jbroma, nice to talk with you again :D
Ok that looks super promising, I love the idea of reusing webpack templating! It crossed my mind while working on that if it would be possible. I'm not insisting on the API, the example was more about explaining the problem.
So, if I understand it correctly, we could have something like this:
remote: {
  enabled: isRemoteAsset,
  assetModuleFilename: "[path]/[name]-[hash].[ext]"
  publicPath: getRemoteAssetsPublicPath(platform),
},

So the "assetModuleFilename" field will be applied only to remote assets.
Why do you think this is a breaking change, since it will be a new field? It should be backwards compatibile, cause not specifying this field will just work the same as now. Od did I just misunderstood something 😅

I was thinking even more than that: we could reuse the existing fields in webpack config without duplicating them - they only get applied to asset modules of type asset or asset/resource anyways, and these are not used atm on platforms like android and ios with Re.Pack.
We could go even a step further and override the asset/resource for android/ios platforms through createGenerator hook from Webpack (docs), so essentially running the that hook in RepackTargetPlugin.

Hi @jbroma!
I have been playing around with it a bit, but I think I’m stuck. I tried to utilize assetModuleFilename in the webpack config, within the output field, but I think I’m missing a piece here. I think the main issue is that I don’t understand when and where the template passed to the assetModuleFilename field is actually transformed into the final path.
I feel like the logic should be inside Repack’s AssetsLoader because we are calling convertToRemoteAssets from there, so at that point, the final path should be available. Could you give me a few more hints on how to continue with that?

Hi @adammruk, sorry for the the delayed response on my part this time!
there is no way to utilize assetModuleFilename right now, the assetLoader would need to become a replacement for asset generator (specified using type: asset/resource as an example, instead of use: '@callstack/repack/assetLoader'). Perhaps I went a little bit too far ahead in the concepts here 😅
Lets go back to the beginning, I think we should proceed with your original idea and just mark the API as unstable to leave some room for improvements and denote that explicitly. Perhaps a callback with some asset info like assetPath or assetData could prove useful for generating the desired filePath?

That's fine:D
Ok, so it is indeed not that easy 😅 But anyway sounds like a nice final solution.
So if it's fine for you, I can move forward with the  initial idea. I have been experimenting with the callback function, but the problem is that we have a validation here: https://github.com/callstack/repack/blob/main/packages/repack/src/webpack/loaders/assetsLoader/options.ts
and I cannot pass the function as a type for a field. I can't remember the original error but it was complaining that function type is not compatible with JSON Schema.
I'm thinking about the possibility to use webpack templating here, so we could pass something like this:
assetModuleFilename: "[path]/[name]-[hash].[ext]"

to have at least a little bit of aligned API with webpack standards, but I'm not sure how to transform this template in the Repack's AssetsLoader to the "real" filename. WDYT?

Ok I was able to find a solution, still testing it but initial checks looks promising, I updated my PR: https://github.com/adammruk/repack/pull/1/files
What are your thoughts about it?

Hi @adammruk
For the function type, I think we can use instanceOf: 'Function' instead of type like in packages/repack/src/webpack/plugins/CodeSigningPlugin/config.ts (there it's used for RegExp).
I'm not sure about using interpolateName from loader-utils since we've recently removed loader-utils as a dependency of Re.Pack in #628 - loader-utils might not be compatible with rspack in V5. I took a quick look and it should be fine but that might change in the future.
The rest of that looks good to me 🚀, I guess we should open a PR a discuss implementation details there 👍

Ok great, thanks for the hints! 🚀  I'm sitting on 3.4.0 in my project that's why I was tinkering with that version, was a bit easier to test for me. I'll switch now to some example project:)
{ instanceOf: 'Function'} is not throwing the error, so I'll try go with function approach. PR soon :D

Hi @jbroma ! PR here: https://github.com/callstack/repack/pull/651 - let me know what do you think and what can be improved :)

Hi @lstoyanoff ! I'm happy that I'm not the only one having this case 😅
I agree, there are different usecases, technically different scale outputs a different content hash, but the general issue is to identify each asses, not necessary each scale. That's why I'm opting for more flexible approach, where you can decide it for yourself.
@jbroma any updates about that change?

Hi @lstoyanoff ! I'm happy that I'm not the only one having this case 😅 I agree, there are different usecases, technically different scale outputs a different content hash, but the general issue is to identify each asses, not necessary each scale. That's why I'm opting for more flexible approach, where you can decide it for yourself.
@jbroma any updates about that change?

feels like we should go with a flexible approach for now, we can always address things later if needed.
@adammruk I've reviewed the PR 👍


Enable vertex pulling (storage buffers + vertex id)

Now that GLES2/WebGL1 has been dropped it's time to look into the topic of "rendering without vertex buffers" again:

don't require a vertex layout specification when creating a pipeline object
d3d11: _sg_validate_shader_desc(): don't require a semantic name in the first vertex attribute
don't require a vertex buffer to be bound for rendering
add storage buffer type (e.g. indexable items)
check that gl_VertexID is properly supported by sokol-shdc

PS: WebGL2 doesn't have storage buffer support either. It still should support rendering without vertex buffers though (e.g. pulling the data from a texture or uniform)

WIP vertex pulling example:

C: https://github.com/floooh/sokol-samples/blob/storage-buffers/sapp/vertexpull-sapp.c
GLSL: https://github.com/floooh/sokol-samples/blob/storage-buffers/sapp/vertexpull-sapp.glsl

'bufferless rendering' example:

C: https://github.com/floooh/sokol-samples/blob/storage-buffers/sapp/triangle-bufferless-sapp.c
GLSL: https://github.com/floooh/sokol-samples/blob/storage-buffers/sapp/triangle-bufferless-sapp.glsl


cone twist joint not updating on viewport

Godot version:

3.0.2
OS/device including version:

linux mint
750ti
Issue description:

I am playing with the joints and cone twist joint doesn't show when added to scene

Steps to reproduce:
add a cone twist joint
Minimal reproduction project:

In order to show it you should connect it to a rigid / static body

With more testing, I think the issue is that the joint doesn't update.

Here I placed the joint originally between the two object and a straight line is drawn. I translate it and it is still a straight line. It only update when I toggle the visibility or switch scenes and show two cone.

Yes the problem is the gizmo update, that should be called when the spatial position changes.
Doesn't exists a signal to attach to know when it changes, so a solution could be add a cycle that iterate only when the editor is on, and checks if the attached rigid body position changes.
@reduz what do you think about?

Is this still reproducible in the latest 3.2 builds?


Added parameter to Objective-C CHIPDeviceController to accept PAA cer…

…tificates from Objective-C client
Fixes #14126
Change overview
Added a new parameter to CHIPDeviceController that is an array of NSData PAA certificates.
Created an Objective-C bridge for the AttestationTrustStore class. The PAA certificates provided
by the client are passed to the default device attestation verifier. If the client passes a nil
value for the parameter then the testing root store is used.
Testing
Used an Objective-C client to pass valid certificates to ensure device attestation procedure
succeeded with an m5stack device. Passed in nil to ensure the device attestation procedure used
the test PAA certificates.

 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.

/rebase

Need to apply the diff from https://github.com/project-chip/connectedhomeip/pull/17079/commits/c52d6ee21a0f7432640172fe0a554ca94ce04444

Need to apply the diff from c52d6ee

The files changes section shows the diff is already applied. Is there something else I am missing?

/rebase


order status set wrong on bets matched immediately on placing

Bug with order status when placing bets - happens live when replacing orders and suspect same might happen when initially placing an order if the price is outside current best price (eg back at 1.01 just takes current best price). The 3 log entries below show what happens with a  replace bet that has its price changed to one that will be automatically matched with best execution - ends up as EXECUTABLE although fully matched.

an existing EXECUTABLE bet that has it's price changed to a price outside the current prices using market.replace_order() is matched immediately (due to best price execution) and the returned status is EXECUTION_COMPLETE.

{"asctime": "2024-02-28 19:31:30,382", "levelname": "INFO", "message": "execute_replace", "trading_function": "replace", "elapsed_time": 0.31018781661987305, "response": {"customerRef": "99c1a3806bc14d62920763d3587774ea", "status": "SUCCESS", "marketId": "1.225432455", "instructionReports": [{"status": "SUCCESS", "cancelInstructionReport": {"status": "SUCCESS", "instruction": {"betId": "339845419341"}, "sizeCancelled": 0.01, "cancelledDate": "2024-02-28T19:31:30.000Z"}, "placeInstructionReport": {"status": "SUCCESS", "instruction": {"selectionId": 47705983, "limitOrder": {"size": 0.01, "price": 16.5, "persistenceType": "LAPSE"}, "orderType": "LIMIT", "side": "LAY"}, "betId": "339845613475", "placedDate": "2024-02-28T19:31:30.000Z", "averagePriceMatched": 14.0, "sizeMatched": 0.01, "orderStatus": "EXECUTION_COMPLETE"}}]}, "order_package": {"id": "99c1a380-6bc1-4d62-9207-63d3587774ea", "client": "<flumine.clients.betfairclient.BetfairClient object at 0x0000028F7B014CA0>", "market_id": "1.225432455", "orders": ["139284414004067119"], "order_count": 1, "package_type": "Replace", "customer_strategy_ref": "livestrat", "bet_delay": 0, "market_version": null, "retry": true, "retry_count": 0, "async": false}}

flumine raises a Pending state for this bet 8ms later

{"asctime": "2024-02-28 19:31:30,390", "levelname": "INFO", "message": "Order status update: Pending", "market_id": "1.225432455", "selection_id": 47705983, "handicap": 0, "id": "139284414903893751", "customer_order_ref": "stratXXxxxxxx-139284414903893751", "bet_id": "339845613475", "date_time_created": "2024-02-28 19:31:30.072237", "publish_time": "2024-02-28 19:31:30.362000", "market_version": null, "async": false, "trade": {"id": "7b0ef6b5-a29b-40a7-8855-9de1be2ccb8f", "strategy": "stratXX", "place_reset_seconds": 0.0, "reset_seconds": 0.0, "orders": ["139284413990493287", "139284414004067119", "139284414903893751"], "offset_orders": [], "notes": "", "market_notes": "12,12.5,12.5", "status": "Pending", "status_log": "Pending, Live, Pending, Live, Pending, Live, Pending"}, "order_type": {"order_type": "Limit", "price": 16.5, "size": 0.01, "persistence_type": "LAPSE", "time_in_force": null, "min_fill_size": null, "bet_target_type": null, "bet_target_size": null, "price_ladder_definition": "CLASSIC"}, "info": {"side": "LAY", "size_matched": 0.01, "size_remaining": 0.0, "size_cancelled": 0.0, "size_lapsed": 0.0, "size_voided": 0.0, "average_price_matched": 14.0}, "responses": {"date_time_placed": "2024-02-28 19:31:30.390370", "elapsed_seconds_executable": null}, "runner_status": null, "line_range_result": null, "status": "Pending", "status_log": "Pending", "violation_msg": null, "simulated": {"profit": 0.0, "piq": 0.0, "matched": []}, "notes": "5,12.483517615336229,11.5,10.912637127536383,0.0,0.0,0.92", "market_notes": null, "client": "CLIENTCODE"}

2ms later things are going wrong somewhere and the order status is changed back to EXECUTABLE

{"asctime": "2024-02-28 19:31:30,392", "levelname": "INFO", "message": "Order status update: Executable", "market_id": "1.225432455", "selection_id": 47705983, "handicap": 0, "id": "139284414903893751", "customer_order_ref": "stratXXxxxxxx-139284414903893751", "bet_id": "339845613475", "date_time_created": "2024-02-28 19:31:30.072237", "publish_time": "2024-02-28 19:31:30.362000", "market_version": null, "async": false, "trade": {"id": "7b0ef6b5-a29b-40a7-8855-9de1be2ccb8f", "strategy": "stratXX", "place_reset_seconds": 0.0, "reset_seconds": 0.0, "orders": ["139284413990493287", "139284414004067119", "139284414903893751"], "offset_orders": [], "notes": "", "market_notes": "12,12.5,12.5", "status": "Pending", "status_log": "Pending, Live, Pending, Live, Pending, Live, Pending"}, "order_type": {"order_type": "Limit", "price": 16.5, "size": 0.01, "persistence_type": "LAPSE", "time_in_force": null, "min_fill_size": null, "bet_target_type": null, "bet_target_size": null, "price_ladder_definition": "CLASSIC"}, "info": {"side": "LAY", "size_matched": 0.01, "size_remaining": 0.0, "size_cancelled": 0.0, "size_lapsed": 0.0, "size_voided": 0.0, "average_price_matched": 14.0}, "responses": {"date_time_placed": "2024-02-28 19:31:30.390370", "elapsed_seconds_executable": null}, "runner_status": null, "line_range_result": null, "status": "Executable", "status_log": "Pending, Executable", "violation_msg": null, "simulated": {"profit": 0.0, "piq": 0.0, "matched": []}, "notes": "5,12.483517615336229,11.5,10.912637127536383,0.0,0.0,0.92", "market_notes": "13.5,14,14", "client": "CLIENTCODE"}

Initial post assumed issue was down to sync socket mode. Testing live reveals the same issue in async mode having set the active flumine lib config.py with async_place_orders = True  Presuming that the config file entry dictates the mode means that the issue is not down to just sync/async as initially thought.


Integrating Phone-Based lang (Lexicon ) into Zipformer Model

I'm seeking guidance on how to incorporate a Phone-Based Language Lexicon (in icefall/egs/librispeech/ASR/prepare.sh in Step 6) into the latest Zipformer Model, a state-of-the-art solution in speech recognition.
I'm unsure about which parameters need adjustment in the Zipformer Model Architecture to optimize performance specifically for phone-level recognition, rather than sub-word or sentence-piece levels, which are typical in Byte Pair Encoding (BPE) models.
Description:
I understand the benefits of open vocabulary systems like BPE, which eliminate the need for prior knowledge of word pronunciation, I'm unsure how BPE handles variations in word pronunciation found in training materials or using words in text training material without normalization them to all lower or upper characters. Additionally, during decoding, there's a possibility of encountering words with multiple variants or specific terminology (such as legal or medical terms or special foreign words) that may contain some tokens do not be in BPE model or not in the token list (tokens.txt)!
-How does BPE handle variations in word pronunciation during training and decoding !, What strategies can I use to address the limitations of BPE models when encountering specialized terminology or words with multiple variants during decoding! This might be the drawback of using BEP based lexicon system.
I have few questions:
1-How can I effectively use a Phone-Based Language Lexicon into the Zipformer Model ? And which Zipformer model or recipe shall be used ?
2- Which parameters in the Zipformer Model Architecture (that layers run at different speeds) should be adjusted or tuned to be able to work well with phone level not sub-word level or sentence piece level Byte Pair Encoding (BPE) as this model designed for ?
Also, It would be good for me to compare the old technology using the TDNN Model in original Kaldi to the Zipformer Model in Next-gen Kaldi icefall using Phone-Based lexicon with the same dataset and also in different languages.
Any advice on these questions would be greatly appreciated. Thanks in advance.

You may refer to egs/librispeech/ASR/tiny_transducer_ctc on how to incorporate the phone lexicon. Basically you use a UniqLexicon object to convert texts to phone tokens. Note that it doesn't handle multiple pronunciations.
Based on my experience, BPE models have better WER than phone models. I'm looking forward to your results.

Thanks,  @wangtiance, I tried with small set of Librispeech 360 hrs. It seems not good comparing to old Kaldi. And also from my customer dataset.
I would appreciate it if anyone can help me :) @danpovey, @wangtiance, @csukuangfj, @pzelasko, @kasidis-kanwat :)
I would like to move from the old Kaldi to the Icefall. However, I have noticed that the available models with multiple pronunciation lexicons are weaker than the TDNN models in the old Kaldi, such as "librispeech/ASR/tiny_transducer_ctc" and "egs/librispeech/ASR/tdnn_lstm_ctc".
I have to use the multiple pronunciations in the lexicon which generated from G2P sequitur-g2p.
a) The egs/librispeech/ASR/tiny_transducer_ctc model handles only unique transcriptions (Tested).
b) The egs/librispeech/ASR/tdnn_lstm_ctc model supports words with multiple pronunciations but does not include the attention-decoder loss because the supervision sequence for the attention part handles only one transcription per word.
As a result, we either lose the attention loss part or the ability to handle multiple transcriptions per word.(Not Tested by me).
How can i use the phone-based in the newest Zipformer model (use only CTC loss, which is not recommended) by "deactivate the attention loss (https://github.com/k2-fsa/icefall/blob/master/egs/librispeech/ASR/zipformer/train.py)
--use-transducer False --use-ctc True "?
Thank you!

I think what you want to do is to start from zipformer CTC recipe, and then replace the BPE code with the phone Lexicon and CTC graph compiler code found in the TDNN LSTM CTC recipe.
In order to use other losses than CTC you'd have to try working around the multiple pronunciation thing; for example, sample one of available pronunciations on each training step, etc.


wip: changes for latest rolldown

Description
Wait for https://github.com/rolldown/rolldown/pull/3009, otherwise the tests fail

cherrypicked


Add more than 6 disciplines

Today the insertion of disciplines are limited.

done in version 1.0.3


CS2030 Practice Assessment #3  Aggregrate Class (ungraded) Level 4

Hi for level 4 of PA#3 (ungraded), i am stuck on how to proceed or go about designing the method for flatmap. Can someone explain what this level is talking about or the thought process of how flatmap works for this level. Thanks!

Hi im going to take a gander at this!
Dissecting flatmap, it seeks to aggregate the seed (similar to map) in addition to aggregating the final object. From the test cases we can see that flatmap takes in a lambda which results in the formation of a new Aggregate object using the static of function. Inside the of() we can see that it takes in another lambda function, converting an object of type S into a Pair of Type S and T which are taken in to the Aggregate.of() in the main mapper input.

I am very confused how the static Aggregate.of() method takes in a Function and somehow produces an Aggregate.
The only way I can think of to reproduce this behaviour is for Aggregate.of() to return a wrapped Function in some AggFunction class, which when used by flatMap produces an Aggregate, and if not used by flatMap return a String?


Multiselect with dynamic values - ignores data from pivot table while reading

I'm trying to implemet "Multiselect with dynamic values"
It seems all working fine. except reading data from pivot table, when building form.
Checking the fields and saving item - works, I see data in pivot table http://prntscr.com/srpmk8
But I when I render form (or just refreshing page) - all fields is unchecked.
I've a project on older Twill version and here all working fine
What I can miss ? And can you plz check this on actual version?

Hi @ResetButton, sorry we have not been responding on this for so long. If you're still dealing with this issue, feel free to ping me, but since this hasn't been reported or confirmed since then, I'm going to close this issue.

I'm also having this issue, @ifox shame that you'd ignore the original issue for 2 years and then just close it at the first chance you get.

I'm not sure why you feel the need to say it's a shame. This issue was created 4 years ago and had no further activity. If you are facing this issue, you are free to comment here, create a new issue, open a discussion, or even join our Discord to discuss it. Blaming me for closing it is not going to get you the support you're looking for. Please provide a reproduction.

It had no further activity because you didn't respond :/  Then you closed the issue because it had no "further activity", what kind of weird logic is that?
The issue was created 4 years ago, you didn't respond for 2 years. Does that mean you should assume that it is no longer an issue? Don't you see the irony here?

What's your point? Are you trying to give me feedback? There are more appropriate ways to do that.
Thank you for opening a new issue. You could have simply referenced the closed issue with a comment saying you believe it hasn't been addressed. Blaming maintainers for closing an issue when hundreds of issues have been addressed and hundreds of community contributions have been merged for many years with a positive tone is just not necessary.

I simply pointed out how it is a shame that an issue was closed out without the OP even getting any response for 2 years and the reason for closing it was inactivity.
 The OP did his part by posting the issue, from then onwards if you had any follow up questions or needed more information then that's on you. I find it a bit ridiculous that you'd close the issue based on the following:

since this hasn't been reported or confirmed since then, I'm going to close this issue

But it is reported already? this exact issue is the report. Are you saying you need multiple reports of the same issue for you to take a look at it any further?


new(driver): update exit events PPME_SOCKET_SOCKET_X with enter params

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area driver-kmod
/area driver-bpf
/area driver-modern-bpf
/area libscap-engine-gvisor
/area libscap
/area libsinsp
/area tests
Does this PR require a change in the driver versions?
/version driver-SCHEMA-version-major
What this PR does / why we need it:
This PR is part of https://github.com/falcosecurity/libs/pull/2068.
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 3b4dac7335c93921a803038038d4db051a8e8844

LGTM label has been added.  Git tree hash: 7fb0eb1e129e9f087beb614a564c95a4fc967239

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

New changes are detected. LGTM label has been removed.

LGTM label has been added.  Git tree hash: 756534102c210cd5c58f3d8afd385a9eef9ccdab

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Promise.{all,race} static method's iterable definitions incorrectly omit other definitions by overloading (regression)

es2015.iterable.d.ts must not omit es2015.promise.d.ts definitions by overloading. However, npm package distributions work correctly.
 I need to find a repro.

No, this can be repro.

I can't repro this. Is your TS version for that project set to 2.6?:

If it's still 2.3, you'll end up using the 2.3 lib, which has this bug.

I'm using only .csproj file: https://github.com/falsandtru/pjax-api/blob/master/pjax-api.csproj

I don't seem to have an issue with your project:

@paulvanbrenk Is there possibly some kind of install issue going on here?

Oh, wait, you said VS 2015, and I've been looking at 2017. That's probably important.

Confirmed in vs2015 only:

The files are still ordered correctly in the library files, so I can only assume this is because the vs2015 extension is loading the TS library files in some other order than what is specified. Either that, or.... this is the only extension where we still use chakra, so we may be relying on object insertion ordering somewhere? Continuing to look into this.

Thanks for your investigation.


Multi-GPU for neon

Hey Anil,
Is there a flag I can set in --backend mgpu or something to get the project to use both of my GPUs? It uses one ok.  I tried setting -m flag, but I'm wonderig if
'max number of GPUs (only used with mgpu backend')
is some sort of reserved feature for your product vs open source release. can you comment?

You're right. Multi-GPU is a reserved feature for now.


Make results bookmarkable.

A major problem is that search results are not book-markable. They should be.
We already have the csv download which is persistent for a search result, so we should be able to make everything else persistent. I'm not sure how angular does this but we must be able to update the URL on every search so that user and bookmark and come back later.

Kind of buggy, this is doing something weird to legends for maps and trees. I'll try and fix it this weekend.

@stefelisabeth - how's this going?

@SimonGreenhill Ahhhh, I think I fixed this, but leave it open and I'll check/close/update this weekend.

This has been done (and I did fix the legend issue), so I'll close for now.


Icon request: Fusang

URL of the site or app (required): https://www.fusang.co/
Logo resources for the icon (optional, but recommended): No official page, but the official color is the yellow they use above.

Here you go!
https://github.com/aegis-icons/aegis-icons/blob/master/SVG/Bitpay.svg
❓ How to add icon(s) to Aegis?


Better matching for titles with roman numerals

We currently attempt to swap out digits in replaceDigits()/word_form() when fuzzy matching if we didn't get a good match without it (e.g., five and 5), but roman numerals are very common in movie titles too.
In particular, if all that distinguishes one movie from another is the roman numerals, we're likely to either match the wrong movie or none at all.
Take for instance the user requesting: "Alexa, ask Kodi to watch movie back to the future two"
And in the user's library, we have these titles:
Back to the Future
Back to the Future II
Back to the Future III

The fuzzy match currently will stop on and select "Back to the Future" (the first movie).

And this is even more difficult when we're supporting multiple languages...

This is done and pushed up, but as noted, word form number conversions are only supported for English.
Closing the issue as the language issue is arguably separate.


[Plugin] managing feature flags / experimentation for entities

Summary
Implement a Feature Flags management interface per service.
The motivation of this plugin is to either:

Integrate with currently existing Feature flags management services such as Cloudbees Feature Management, LaunchDarkly or other services; OR
Implement a Feature flags pluggable interface that can trigger any custom implementation of Feature Flags in case the service has a custom mechanism (such as a database or REST service to manage them)

Project website (if applicable)

Cloudbees Feature Management
LaunchDarkly

Context
The focus of this plugin is not to implement Feature flags for Backstage itself, but for the projects/services/repos that are managed by Backstage

Great idea!  Maybe add in support for Split.io also !  I'd think a feature like this would also require enhanced role based permissioning, as we wouldn't want an entire population of Backstage users to be able to flip feature flags across an ecosystems. So maybe just VIEWING the flags as a start?

Unleash is another vendor that could leverage this interface.

Is there any open-source/hosted Feature Flags vendor or any RFC/Standard for Feature Flags?

Is there any standard protocol or RFC for managing Feature Flags? Is it something we should look at before implementing it?

Hi! I am feeling hesitant against the idea of a general interface for this. We try to invent as few general wrappers as possible, and instead try to prioritize making it easy to make + install distinct plugins. Additionally, my gut tells me that the interfaces to these various systems probably have significant, hard-to-bridge-over differences.
That being said, individual plugins for the systems mentioned in here definitely sound like great things!

Shall we focus in a single one then? Like Unleash or any other vendor? I am thinking in Unleash as it is Open Source and could be installed in a private infrastructure.

I'll let the community choose what it feels brings the most value.
As we do this, I reckon that we'll learn things as we go. Maybe we'll discover some commonalities and implement some "least common denominator" pluggable interface in the end. But usually it's best to start specific and only then go generic.

👍 for LaunchDarkly plugin

Any traction on this feature request? +1 for Launchdarkly plugin

Is there any standard protocol or RFC for managing Feature Flags? Is it something we should look at before implementing it?

https://openfeature.dev/ might be of interest, however to be clear this is in early stages, and no implementation exists to date

:wave: Hello, I'm one of the maintainers at OpenFeature. When @dackroyd made the above comment, we were indeed in the very early stages!
Since then we've been accepted as a CNCF sandbox project, and we've released stable SDKs (including TypeScript) along with providers for a few solutions, including those mentioned above. If you'd like more info, please reach out to us on our CNCF slack channel.

Hi all, I am the author of Unleash.
I think it make sense to view the current status of the feature toggles relevant for the project/team, and maybe simple operations (turn on/off), but for mode advanced use cases it might be easier to enter the tool itself, to avoid re-implementing everything,
Let me know if we can help out in any way with the integration of Unleash, which does have Open API Specifications.

I reworded the title a bit to ensure that it's clear that the main goal, is to write Backstage plugins that interface with third party experimentation products, to manage (typically) non-Backstage systems that are being experimented on.
Of course we're happy to accept contributions of plugins that give quick and easy access to basic functionality of Unleash, OpenFeature or others!

Open feature is the common abstraction layer for feature flag vendors. This would allow your plugin authors to speak open feature, and then the people who set up backstage can pick a flags backend that they like (many mentioned on the thread above).


Errors writing parameters to Seat vehicle

Hi,  app is reading perfectly all values, I have an electric car and I want to modify charging timers using some logic, but even manually when I try to change a parameter I get an error, picture attached.

I need more complete log than this to troubleshoot. Please include a few rows prior as well. There should at least be one warning row before the traceback.

I need more complete log than this to troubleshoot. Please include a few rows prior as well. There should at least be one warning row before the traceback.

Logger: seatconnect.vehicle
Source: /usr/local/lib/python3.9/site-packages/seatconnect/vehicle.py:527
First occurred: 14:42:56 (1 occurrences)
Last logged: 14:42:56
Failed to execute departure timer request - getDeparturetimer() missing 1 required positional argument: 'baseurl'

Please test latest updated release and report back. I checked the code and this error should not appear, not in latest code at least.

You are right, now it works!
Thanks 👍

Great! Closing issue.


Add About section

If you open the app, there isn't any About section, where to show information like link to Github, version of the app, license, acknowledgement, ...
If you want to report something you have to find yourself the link to GitHub

I would like to work on this. @airon90 @mrcsxsiq


Package shouldn't remove the ImageSharp.Web PhysicalFileSystemProvider

When this package is installed the default ImageSharp.Web PhysicalFileSystemProvider is removed so you can't use ImageSharp.Web outside of the /media path.
e.g. /img/logo.png?width=100 works before the package is installed but doesn't after

I can confirm that using AddAzureBlobMediaFileSystem() does indeed change this behavior 😢
The default PhysicalFileSystemProvider is registered first, matches all requests and therefore will always be the provider used by the ImageSharpMiddleware. This is why it's replaced by using builder.Services.AddUnique<IImageProvider, AzureBlobFileSystemImageProvider>(), which only matches the path that's mapped to the blob container.
For now, you can manually re-add the PhysicalFileSystemProvider after the AddAzureBlobMediaFileSystem() call, so it will not be the first provider and therefore be used for all physical files outside of the configured Azure Blob Storage container media path by using:
builder.Services.AddSingleton<IImageProvider, PhysicalFileSystemProvider>();

Note that any physical files inside the physical media folder will still be served by the static file middleware, but will not be processed by ImageSharp. I think this should be fixed in ImageSharp, so it first filters all providers using Match and then uses the first provider that returns a non-null IImageResolver.

I think this should be fixed in ImageSharp, so it first filters all providers using Match and then uses the first provider that returns a non-null IImageResolver.

Disagree strongly here. The default provider is specifically designed to be catch all. You should register the providers in the order you want them to run. Adding extra filtering hurts performance.

@JimBobSquarePants Then what's the reason for allowing multiple IImageProviders to be registered? And yes, doing more lookups/filtering might hurt performance, but that shouldn't prevent adding a bit more flexibility to ImageSharp: https://github.com/SixLabors/ImageSharp.Web/pull/185.
ImageSharp already does double filtering, first by using Match(HttpContext context) and then IsValidRequest(HttpContext context): if only a single provider is picked by the first, why do you need to check if it's a valid request too?

ImageSharp.Web (Note name) is flexible enough thank you. It works fine for everyone else including other CMS systems who proactively engaged with me and I have worked closely with. If something doesn’t work your way it doesn’t mean that it doesn’t work.


any way to disable getting component-report?

We're running into an issue at the end when we run this plugin on our jenkins instance, because our jenkins instance doesn't have connectivity to the outside internet (we;re using modified locations for cveUrls). since this is when all the dependencies have already been scanned, is there a way to skip this step?
[DEBUG] Connecting to: https://ossindex.sonatype.org/api/v3/component-report
[DEBUG] Error requesting component reports
java.net.SocketTimeoutException: connect timed out
org.owasp.dependencycheck.analyzer.exception.AnalysisException: Failed to request component-reports
at org.owasp.dependencycheck.analyzer.OssIndexAnalyzer.analyzeDependency(OssIndexAnalyzer.java:164)
at org.owasp.dependencycheck.analyzer.AbstractAnalyzer.analyze(AbstractAnalyzer.java:131)
at org.owasp.dependencycheck.AnalysisTask.call(AnalysisTask.java:88)
at org.owasp.dependencycheck.AnalysisTask.call(AnalysisTask.java:37)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
at java.lang.Thread.run(Thread.java:748)
Caused by: java.net.SocketTimeoutException: connect timed out

If running in an offline mode you would need to add the following configuration to disable to OSS Index Analyzer.  Note - this analyzer will report vulnerabilities that are not found in the NVD - but it requires sending the package URL (e.g. GAV) for your dependencies to Sonatypes OSS Index service and a list of known vulnerable components will be returned to dependency-check.
<configuration>
   <ossindexAnalyzerEnabled>false</ossindexAnalyzerEnabled>
</configuration>

thank you


add: libglibutil-deb

As part of #6454 requests, the libglibutil-deb are the firsts important dependence should be added first before libgbinder-deb, python3-gbinder-deb, and then waydroid-deb
Here's merge map should do for merging my commit which based on requests #6454
add:libglibutil-deb --> add:libgbinder-deb --> add:python3-gbinder-deb --> add:waydroid-deb
Please review and merge immediately and follow the merge maps I made for merging it.

Hold, cause debian:stable tests are failed cause the dependencies needed are missing. I'll convert it as draft and update the pacscript.

Until libglibutil-deb are ready to review, it will NOT creating new commit requests until the problem are done

Because libglibutil-deb are nearly good to merge and being one of three dependencies needed by requests #6454, I'll make a commit for second importance dependencies.


Unicode + SMS in Laravel

Migrated from #8
cc @asxer

@mheap Hello! I'm trying to send emoji in sms. I upgraded this package and nexmo/laravel package to 1.0.0 version. I try to call example above
$nexmo = app(\Nexmo\Client::class);

$nexmo->message()->send([
    'type' => 'unicode',
    'to'   => $phone,
    'from' => config('nexmo.from'),
    'text' => '👍'
]);

As result i got many ?? symbols instead of emoji and non-latin symbols. Could you please explain how can i fix it?
php version 5.6.30-0+deb8u1

Hi @Asxer,
I've just run the code that you've provided on a clean Laravel installation and it seems to be working fine. I receive the emoji on my phone.
PHP 7.1.8 (cli) (built: Aug  7 2017 15:02:45) ( NTS )
The PHP version shouldn't have an impact, but that's the only difference I can find. Could you try a newer version of PHP?

Hi @Asxer,
I'm going to close this issue as it's over two weeks since we heard from you. If you're still having issues, leave a comment and we can re-open the issue


Some constructs are impossible to properly express with nobind17

Let's consider this code:
// create an AudioSamples object
const samples = AudioSamples.create(buffer, format, 48000, ffmpeg.AV_CH_LAYOUT_STEREO, 48000);
// get its timebase -> returns a reference to the `av::Rational` in `samples`
const timeBase = samples.timeBase();
// last use of samples -> after the await, samples is potentially garbage-collected
const packet = await this.encoder.encodeAsync(samples);
// crash
console.log(timeBase.toString());

Calling samples.timeBase() returns a av::Rational object that is a reference to the internal field of the samples packet. In this extract, samples can get garbage-collected before the .toString() method.

This is a variation of https://github.com/mmomtchev/nobind/issues/1


Fix README reference to formtastic.rb config

Very small change to README.
Currently reads:
Customize the default class used for hints on each attribute or globally in the config/formtastic.rb
Should read:
Customize the default class used for hints on each attribute or globally in the config/initializers/formtastic.rb

Pulled in, thanks!


Exception in thread "main" java.lang.NoSuchMethodError ???

I'm trying build simple desktop application compose and in my normal kotlin code try something like:
fun main() = Window {
    Text("kotlin compose!!")
}

So when I make run to code, I got this Error :
Exception in thread "main" java.lang.NoSuchMethodError: 'void androidx.compose.desktop.AppWindow_desktopKt.Window-BIDTD-Q$default(java.lang.String, long, long, boolean, java.awt.image.BufferedImage, androidx.compose.ui.window.v1.MenuBar, boolean, boolean, androidx.compose.desktop.WindowEvents, kotlin.jvm.functions.Function0, kotlin.jvm.functions.Function0, int, java.lang.Object)'

To be honest code like this work fine when I use Default desktop compose project,
But I trying in normal kotlin project in this time.
OS: Ubuntu 20.10
INV: Intellij Idea 2021.1

Hi! Which Compose MP build do you use?

I used desktop compose 0.4.0 and that's one works fine like I said, The Error showing when only I run the compose in normal kotlin project.

Could you try to use latest build and new Window API?
plugins {
kotlin("jvm") version "1.5.10"
id("org.jetbrains.compose") version "0.5.0-build235"
}
@OptIn(ExperimentalComposeUiApi::class)
fun main() {
application {
Window(::exitApplication) {
Text("Hello!")
}
}
}

Could you try to use latest build and new Window API?
plugins {
kotlin("jvm") version "1.5.10"
id("org.jetbrains.compose") version "0.5.0-build235"
}
@OptIn(ExperimentalComposeUiApi::class)
fun main() {
application {
Window(::exitApplication) {
Text("Hello!")
}

Thanks a loot, And yes this solve my issue compose and everything works fine.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Upgrade JUnit to version5

JUnit 5 has been out for a while, let's upgrade all tests to use it.
It will allow usage of lambdas in few cases, like asserting thrown exception - we should get rid of ExpectedExceptions in the code during the upgrade process and upgrade surefire and failsafe to the latest versions available (there were issues relatetd to JUnit 5 there).

@paulodamaso/z please, pay attention to this issue

Manual assignment of issues is discouraged, see §19: -5 point(s) just awarded to @paulodamaso/z

Bug was reported, see §29: +15 point(s) just awarded to @krzyk/z

@krzyk the puzzle #895 is still not solved.

@oridan almost, but junit-platform-surefire-provider and the other dependency in surefire/failsafe is not needed since surefire/failsafe version 2.22.0 AFAIR

@0crat wait for PR #896 be merged.

@0crat wait for PR #896 be merged. (here)

@oridan The impediment for #877 was registered successfully by @oridan/z

@krzyk please close this issue.

@oridan thanks

@ypshenychka/z please review this job completed by @oridan/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

The job #877 is now out of scope

@oridan According to our QA Rules:

All commit messages have links to issues

Please add link to this issue into all related commits.

@ypshenychka ok, just added.

@oridan Did you? I don't see links in most of the commits here and here

@oridan Valid point. Then let's leave them as they are now. But please confirm you'll try to follow this rule next time.

@ypshenychka I'll follow this rule next time.

@oridan thanks

@0crat quality acceptable

@0crat quality acceptable  (here)

@ypshenychka The project doesn't have enough funds, can't make a payment

Order was finished, quality is "acceptable": +30 point(s) just awarded to @oridan/z

Quality review completed: +8 point(s) just awarded to @ypshenychka/z

@krzyk the only puzzle #895 is solved here.


feat(deps): update to sws2apps/react-sw-helper beta version

:tada: This PR is included in version 1.0.0-alpha.114 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


pdns-recursor: 4.0.6 -> 4.0.8 (fixes CVE-2017-15120)

Motivation for this change
For more details see [1].
[1] http://www.openwall.com/lists/oss-security/2017/12/11/1
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

This should probably be backported to 17.09. The stable release still carries 4.0.4.
While pdns-resolver 4.1.0 was already released I would defer bumping it to the next major version until some tests have been written and/or someone has verified that our existing module still works.

Was also about to patch this. Thanks!


Can i use vim-polyglot instead of treesitter?

Neovim is heavily integrated-with, and generally works best with, treesitter. Using the treesitter plugin is probably the easiest way to install and set it up for the different langs you need. Currently, most of the syntax highlighting provided by this plugin is done so via treesitter, although there is support for legacy highlight groups as well. So it should still work in theory.
Why do you need vim-polyglot?


Create an example of logging with file rotation

I've also added a small snippet at the bottom to generate log message. I can remove this if it is not desirable

Was there something more I needed to do on this? Sorry I don't remember.

@c-git Can you please rebase and add your example to the Cargo.toml

I didn't mean to request reviews I just clicked update. Haven't quite figured out how to also do the edit from my phone. If I don't figure it out I'll do it after the holidays when I'm back on my computer.

No worries, just happened to take a look. If you could also fix the magic numbers that'd be great

I moved the configurable parts of the demo to the top of the file (I hope those are the numbers you were referring to).
I don't know which features are required to be included in the Cargo.toml. I tried none and it worked when I ran cargo run --example log_to_file_with_rolling so I don't know if I'm testing it wrong. But I checked the docs and guessed which would be needed.

Your testing was correct, because all of the features you're using are part of the default list you technically do not need to list any features. However, it's following convention to list them out so it looks good.

Let me know if there is anything else I need to do?

No actions from me. I'll see what I can do

@c-git Please resolve the conflicts, then we are good to go.

I think I fixed it but did it from the GitHub UI so not quite sure but it doesn't show the conflict anymore

Sorry about that, I'll try to see if I can open up more context next time. Seems like an obvious mistake in hindsight.

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

Comparison is base (b0ad8ba) 61.77% compared to head (accc0a5) 61.77%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #295   +/-   ##
=======================================
  Coverage   61.77%   61.77%
=======================================
  Files          23       23
  Lines        1431     1431
=======================================
  Hits          884      884
  Misses        547      547

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

@c-git Just need you to fix that CI job

Ok I'll try to check it. Will probably be tomorrow night.

Ok I think I fixed it I ran cargo +nightly fmt and committed those changes

@c-git, hey the MSRV needs to get bumped to support the toml crate. Here is a commit making the same change.

@c-git, hey the MSRV needs to get bumped to support the toml crate. Here is a commit making the same change.

Hey @bconn98 I'm a bit lost. I don't follow the connection between the example in this PR and the toml crate. What do you need me to do?

The CI/CD pipeline will fail. There was a non major revision change to one of the toml libraries that increased the MSRV to 1.69. When that tries to build in our pipelines with 1.67 it will fail. Therefore, nothing right now can be merged down to main without the fix.


Set spread mode from query string

Note that no such parameter exists in https://www.adobe.com/content/dam/acom/en/devnet/acrobat/pdfs/pdf_open_parameters.pdf, and adding non-standard URL parameters is something that has previously always been avoided.

Closing since I agree with the comment above. We only aim to offer the parameters from the specification, and to make an exception there must be a good use case. I don't see why it would be beneficial to be able to change this via the URL parameters instead of just using the viewer toggles or the preferences (spreadModeOnLoad).


PyPi docs: Add web deface details

Status
READY
Description
PyPi docs: Add web deface details.
Related PRs
List related PRs against other branches:

branch
PR

other_pr_production
link

other_pr_master
link

Todos

[ ] Tests
[x] Documentation

Deploy Notes
Notes regarding deployment the contained body of work.  These should note any
db migrations, etc.
Steps to Test or Reproduce
Outline the steps to test or reproduce the PR here.
git pull --prune
git checkout <feature-branch>

Impacted Areas in Application
List general components of the application that this PR will affect:

user_guide_pypi.md

@all-contributors please add @abhisharma404 @rejahrehim @ananthus @adeyosemanputra

@all-contributors please add @abhisharma404 @rejahrehim @ananthus @adeyosemanputra code

@all-contributors add @abhisharma404 for code, doc
@all-contributors add @ananthus for code
@all-contributors add @adeyosemanputra for code, doc
@all-contributors add @rejahrehim for doc

@all-contributors add @abhisharma404 for code and doc


Keyboard Language typing mistake

Description of the new feature / enhancement
When we have multiple keyboard layouts installed, and we type by mistake in an unintended language; allow to select and convert the mistaken text into the intended language
Scenario when this would be used?
When typing with the wrong keyboard layout selected
Supporting information
No response

This proposal does not call for a translator program at all.
My 2 cents would be:

If there is text selected anywhere
Intercept the keyboard change shortcut/call (Widows key + space,  or  ALT+SHFT) after a new layout is selected
If a different keyboard layout is selected, then convert the selected text from the current keyboard layout to the new selected keyboard layout
Tada

Maybe AutoHokey could do this

Good solution!
On Thursday, August 10, 2023 at 12:01:25 AM GMT+3, Jay @.***> wrote:
Maybe AutoHokey could do this
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Configure webpack build to generate es5 module.

This project is amazing. However, it is annoying to rely on webpack to build it together with my application. This changes generate a ES 5 modules which can be imported in my project.
Until this is merged, I published a npm package on top of the fork. There's a disclaimer in the bottom of the README regarding the fork's version.

Using this fork i can't seem to import the reducer, i'm getting the following error:
warning.js:14No reducer provided for key "growler"
module installed as expected and import is present as:
import { GrowlerReducer as growler } from 'react-redux-flash-notification';

I'm seeing the same issue as @tommyalvarez 🤔

@cmkshaw use the npm package toastr in the meanwhile. I found it excellent, it uses redux and is es6 compatible. Works out of the box with the react-starter-kit project


FSUP-10060: framework uplift

Added target frameworks net6.0 & net8.0
Newtonsoft.Json downgraded to 12.0.1
Other dependencies upgraded

I think this is not a breaking update, so we don't have to update first number, so version number should be 2.1.0


add dashboard to view all answers & questions

Option 1:

Add button to show modal
Add modal to show all questions
Create js code to show all questions
Collapse answers on click

Option 2:

Add bootstrap
Add navbar
Add navigation between 2 pages
Add new page to show all questions
Create js code to show all questions
Add responsive design to navbar
Add responsive design to new page

Prefer option 2

I would also use Bootstrap but I would say that we use a custom compiled bootstrap repo. We just need to fork and adjust the bootstrap repo and keep our current Design.


Add badges to README.md

Most open source repositories have badges to the contributor covenant in the README.md.

If this is a valid issue, I would like to work on it.

I'm definitely happy to add a badge for the CoC but please point it at ours since we might change it soon.


Update base.py to allow securityDefinitions

Currently there's no way to specify the securityDefinitions.
This is a super simple change to allow add it on app initializacion.
app.config['SWAGGER'] = {
    "swagger_version": "2.0",
    "basePath": "/v1",
    "securityDefinitions": {
        "petstore_auth": {
            "type": "oauth2",
            "authorizationUrl": "/oauth/dialog",
            "flow": "implicit",
            "scopes": {
                "write:pets": "modify pets in your account",
                "read:pets": "read your pets"
            }
        },
        "api_key": {
            "type": "apiKey",
            "name": "api_key",
            "in": "header"
        }
    }

Sorry, I didn't realized that. It should be ok now.

It would be wonderful if you make a new release of the pip package with this improvement!

@microstudi will fix some other issues then I'll release soon


The slack badge looks small in the readme

Current Behavior

The Slack badge does not match the allignment with the other badges in the ReadMe file
Expected Behavior

The Slack badge should match the allignment with the other badges in the ReadMe file
Screenshots/Logs

Environment

Host OS: Mac Linux Windows
Browser: Chrome Safari Firefox

Done !