New version: CUDA v2.5.0 (registered via LocalRegistry.jl)

New version: CUDA v2.5.0
UUID: 052768ef-5323-5732-b1bb-66c8b64840ba
Tree: 2ed30f30c8905cea954b0904a98211745b3818f5

This pull request uses LocalRegistry.jl to register CUDA.jl version 2.5.0 from the master branch of CUDA.jl.

For posterity, here is the Bash script that I ran to generate this pull request. I used the latest Julia master.
rm -rf ~/.julia
rm -rf temp-localregistry
mkdir temp-localregistry
cd temp-localregistry
git clone git@github.com:JuliaRegistries/General.git General
cd General
git checkout master
git checkout -B dpa/register-cuda-2.5.0
cd ..
julia -e 'import Pkg; Pkg.Registry.add(Pkg.Registry.RegistrySpec(path = joinpath(pwd(), "General")))'
rm -rf General
git clone https://github.com/JuliaGPU/CUDA.jl.git CUDA.jl
cd CUDA.jl
git checkout master
cd ..
julia -e 'import Pkg; Pkg.develop(Pkg.PackageSpec(path = joinpath(pwd(), "CUDA.jl")))'
julia -e 'import Pkg; Pkg.add(Pkg.PackageSpec(name = "LocalRegistry", uuid = "89398ba2-070a-4b16-a995-9893c55d93cf", version = "0.3.1"))'
julia -e 'import InteractiveUtils; InteractiveUtils.versioninfo()'
julia -e 'import Pkg; Pkg.status(; mode = Pkg.PKGMODE_PROJECT)'
julia -e 'import Pkg; Pkg.status(; mode = Pkg.PKGMODE_MANIFEST)'
julia -e 'import LocalRegistry; LocalRegistry.register("CUDA", "General")'
cd ..
rm -rf temp-localregistry
cd ~/.julia/registries/General
git push -f origin dpa/register-cuda-2.5.0
cd ~
rm -rf ~/.julia

And here is the output: (click to expand)
$ rm -rf ~/.julia
$ rm -rf temp-localregistry
$ mkdir temp-localregistry
$ cd temp-localregistry
$ git clone git@github.com:JuliaRegistries/General.git General
Cloning into 'General'...
remote: Enumerating objects: 96, done.
remote: Counting objects: 100% (96/96), done.
remote: Compressing objects: 100% (91/91), done.
remote: Total 386704 (delta 24), reused 21 (delta 5), pack-reused 386608
Receiving objects: 100% (386704/386704), 114.02 MiB | 10.98 MiB/s, done.
Resolving deltas: 100% (213476/213476), done.
Updating files: 100% (19408/19408), done.
$ cd General
$ git checkout master
Already on 'master'
Your branch is up to date with 'origin/master'.
$ git checkout -B dpa/register-cuda-2.5.0
Switched to a new branch 'dpa/register-cuda-2.5.0'
$ cd ..
$ julia -e 'import Pkg; Pkg.Registry.add(Pkg.Registry.RegistrySpec(path = joinpath(pwd(), "General")))'
     Copying registry from `~/Downloads/temp-localregistry/General`
       Added registry `General` to `~/.julia/registries/General`
$ rm -rf General
$ git clone https://github.com/JuliaGPU/CUDA.jl.git CUDA.jl
Cloning into 'CUDA.jl'...
remote: Enumerating objects: 718, done.
remote: Counting objects: 100% (718/718), done.
remote: Compressing objects: 100% (269/269), done.
remote: Total 35164 (delta 338), reused 567 (delta 216), pack-reused 34446
Receiving objects: 100% (35164/35164), 8.00 MiB | 10.53 MiB/s, done.
Resolving deltas: 100% (23416/23416), done.
$ cd CUDA.jl
$ git checkout master
Already on 'master'
Your branch is up to date with 'origin/master'.
$ cd ..
$ julia -e 'import Pkg; Pkg.develop(Pkg.PackageSpec(path = joinpath(pwd(), "CUDA.jl")))'
Path `/Users/dilum/Downloads/temp-localregistry/CUDA.jl` exists and looks like the correct package. Using existing path.
   Resolving package versions...
   Installed Reexport ─────────── v0.2.0
   Installed MuladdMacro ──────── v0.2.2
   Installed AbstractFFTs ─────── v0.5.0
   Installed ExprTools ────────── v0.1.3
   Installed OrderedCollections ─ v1.3.2
   Installed NNlib ────────────── v0.7.11
   Installed GPUArrays ────────── v6.2.0
   Installed DataStructures ───── v0.18.8
   Installed Adapt ────────────── v3.0.0
   Installed TimerOutputs ─────── v0.5.7
   Installed Scratch ──────────── v1.0.3
   Installed MacroTools ───────── v0.5.6
   Installed Compat ───────────── v3.25.0
   Installed Requires ─────────── v1.1.2
   Installed GPUCompiler ──────── v0.9.1
   Installed LLVM ─────────────── v3.5.2
   Installed CEnum ────────────── v0.4.1
   Installed BFloat16s ────────── v0.1.0
   Installed ChainRulesCore ───── v0.9.24
Updating `~/.julia/environments/v1.7/Project.toml`
  [052768ef] + CUDA v2.5.0 `~/Downloads/temp-localregistry/CUDA.jl`
Updating `~/.julia/environments/v1.7/Manifest.toml`
  [621f4979] + AbstractFFTs v0.5.0
  [79e6a3ab] + Adapt v3.0.0
  [ab4f0b2a] + BFloat16s v0.1.0
  [fa961155] + CEnum v0.4.1
  [052768ef] + CUDA v2.5.0 `~/Downloads/temp-localregistry/CUDA.jl`
  [d360d2e6] + ChainRulesCore v0.9.24
  [34da2185] + Compat v3.25.0
  [864edb3b] + DataStructures v0.18.8
  [e2ba6199] + ExprTools v0.1.3
  [0c68f7d7] + GPUArrays v6.2.0
  [61eb1bfa] + GPUCompiler v0.9.1
  [929cbde3] + LLVM v3.5.2
  [1914dd2f] + MacroTools v0.5.6
  [46d2c3a1] + MuladdMacro v0.2.2
  [872c559c] + NNlib v0.7.11
  [bac558e1] + OrderedCollections v1.3.2
  [189a3867] + Reexport v0.2.0
  [ae029012] + Requires v1.1.2
  [6c6a2e73] + Scratch v1.0.3
  [a759f4b9] + TimerOutputs v0.5.7
  [0dad84c5] + ArgTools
  [56f22d72] + Artifacts
  [2a0f44e3] + Base64
  [ade2ca70] + Dates
  [8bb1440f] + DelimitedFiles
  [8ba89e20] + Distributed
  [f43a241f] + Downloads
  [b77e0a4c] + InteractiveUtils
  [4af54fe1] + LazyArtifacts
  [b27032c2] + LibCURL
  [76f85450] + LibGit2
  [8f399da3] + Libdl
  [37e2e46d] + LinearAlgebra
  [56ddb016] + Logging
  [d6f4376e] + Markdown
  [a63ad114] + Mmap
  [ca575930] + NetworkOptions
  [44cfe95a] + Pkg
  [de0858da] + Printf
  [3fa0cd96] + REPL
  [9a3f8284] + Random
  [ea8e919c] + SHA
  [9e88b42a] + Serialization
  [1a1011a3] + SharedArrays
  [6462fe0b] + Sockets
  [2f01184e] + SparseArrays
  [10745b16] + Statistics
  [fa267f1f] + TOML
  [a4e569a6] + Tar
  [8dfed614] + Test
  [cf7118a7] + UUIDs
  [4ec0a83e] + Unicode
  [e66e0078] + CompilerSupportLibraries_jll
  [deac9b47] + LibCURL_jll
  [29816b5a] + LibSSH2_jll
  [c8ffd9c3] + MbedTLS_jll
  [14a3606d] + MozillaCACerts_jll
  [83775a58] + Zlib_jll
  [8e850ede] + nghttp2_jll
$ julia -e 'import Pkg; Pkg.add(Pkg.PackageSpec(name = "LocalRegistry", uuid = "89398ba2-070a-4b16-a995-9893c55d93cf", version = "0.3.1"))'
    Updating registry at `~/.julia/registries/General`
    Updating git-repo `git@github.com:JuliaRegistries/General.git`
┌ Warning: Some registries failed to update:
│     — /Users/dilum/.julia/registries/General — branch origin/dpa/register-cuda-2.5.0 not found
└ @ Pkg.Types ~/dev/repos/JuliaLang/julia/usr/share/julia/stdlib/v1.7/Pkg/src/Types.jl:1183
   Resolving package versions...

Counting objects: 100% (13/13), done.
Delta compression using up to 4 threads
Compressing objects: 100% (7/7), done.
Writing objects: 100% (7/7), 1.50 KiB | 1.50 MiB/s, done.
Total 7 (delta 5), reused 0 (delta 0), pack-reused 0
remote: Resolving deltas: 100% (5/5), completed with 5 local objects.
remote:
remote: Create a pull request for 'dpa/register-cuda-2.5.0' on GitHub by visiting:
remote:      https://github.com/JuliaRegistries/General/pull/new/dpa/register-cuda-2.5.0
remote:
To github.com:JuliaRegistries/General.git
 * [new branch]            dpa/register-cuda-2.5.0 -> dpa/register-cuda-2.5.0
$ cd ~
$ rm -rf ~/.julia

After running the Bash script, I went to the GitHub website and created a new pull request to merge the dpa/register-cuda-2.5.0 branch into master.

CI is expected to fail on this pull request because CI runs with Julia 1.5, and CUDA v2.5.0 requires Julia 1.6.

CI is expected to fail on this pull request because CI runs with Julia 1.5, and CUDA v2.5.0 requires Julia 1.6.

@maleadt This is ready to review.

@maleadt This is ready to review.

I think this has caused bad things to happen in packages that depend on CUDA.jl when you try to run those packages on Julia < 1.6.
The reason is that the UUID 4af54fe1-eca0-43a8-85a7-787d91b784e3 doesn't match any registered package, and in Julia < 1.6 it doesn't match any standard library, so the resolver breaks.
Can we register a LazyArtifacts.jl package with this UUID in the General registry? So that packages that depend on CUDA.jl don't break on Julia < 1.6?
cc: @staticfloat @giordano @maleadt
See e.g. https://github.com/JuliaGPU/KernelAbstractions.jl/runs/1680372624 for a broken build

I think this has caused bad things to happen in packages that depend on CUDA.jl when you try to run those packages on Julia < 1.6.
The reason is that the UUID 4af54fe1-eca0-43a8-85a7-787d91b784e3 doesn't match any registered package, and in Julia < 1.6 it doesn't match any standard library, so the resolver breaks.
Can we register a LazyArtifacts.jl package with this UUID in the General registry? So that packages that depend on CUDA.jl don't break on Julia < 1.6?
cc: @staticfloat @giordano @maleadt
See e.g. https://github.com/JuliaGPU/KernelAbstractions.jl/runs/1680372624 for a broken build

We could probably even have a LazyArtifacts "package" that has zero registered versions. So it can't actually be installed, but at least the resolver on Julia < 1.6 can map the UUID 4af54fe1-eca0-43a8-85a7-787d91b784e3 to an actual package.

We could probably even have a LazyArtifacts "package" that has zero registered versions. So it can't actually be installed, but at least the resolver on Julia < 1.6 can map the UUID 4af54fe1-eca0-43a8-85a7-787d91b784e3 to an actual package.

https://github.com/JuliaPackaging/LazyArtifacts.jl

https://github.com/JuliaPackaging/LazyArtifacts.jl

Register please?
And since the name LazyArtifacts is a stdlib in 1.6, I think it's fine to merge the registration of https://github.com/JuliaPackaging/LazyArtifacts.jl immediately.

Register please?
And since the name LazyArtifacts is a stdlib in 1.6, I think it's fine to merge the registration of https://github.com/JuliaPackaging/LazyArtifacts.jl immediately.

Oh boy merging this PR broke the entire Julia ecosystem 😬
At least it got reverted pretty quickly.

Oh boy merging this PR broke the entire Julia ecosystem 😬
At least it got reverted pretty quickly.

Register please?

I don't have the time to do it right now, so please go ahead if you can do it. It'd also be good to test it actually works across all julia versions (well, v1.3+ I guess?), I never got around that

Register please?

I don't have the time to do it right now, so please go ahead if you can do it. It'd also be good to test it actually works across all julia versions (well, v1.3+ I guess?), I never got around that

The fact is that the package is likely completely useless in v1.3-v1.5. I added the same exports as in the standard library (at least those actually available in v1.5), for consistency, but perhaps an empty package is good enough. Someone needs to decide what to do in practice

The fact is that the package is likely completely useless in v1.3-v1.5. I added the same exports as in the standard library (at least those actually available in v1.5), for consistency, but perhaps an empty package is good enough. Someone needs to decide what to do in practice

For the use case here (being able to register CUDA 2.5.0 in the General registry without breaking the General registry for everyone on Julia < 2.5), I think it's sufficient to register LazyArtifacts.jl without any versions.
This is done here: https://github.com/JuliaRegistries/General/pull/27753
It would be good to get a sanity check to confirm that it's safe to register a package without any versions.

For the use case here (being able to register CUDA 2.5.0 in the General registry without breaking the General registry for everyone on Julia < 2.5), I think it's sufficient to register LazyArtifacts.jl without any versions.
This is done here: https://github.com/JuliaRegistries/General/pull/27753
It would be good to get a sanity check to confirm that it's safe to register a package without any versions.


Unable to add devices to HomeKit even they appear in HomebridgeUI

I am unable to add my two airconditions to HomeKit even they appear properly in Homebridge. I followed the description but the process is timing out in HomeKit with the message “Devise doesn’t respond”. I was running the plug-in from zephyus/ Homebridge-MElcloud2 and there is no need for this manual step in HomeKit to add the devices. As soon the Homebridge is connected to HomeKit my two units appear in HomeKit. Maybe you can contact him how he pulled this off in his plugin.
https://github.com/zephyrus/homebridge-melcloud2
Thanks for your effort

Hi,
This plugin is a dynamic external accessory and use other way to add it to HomeKit.
Are You realised this steps:
https://github.com/grzegorz914/homebridge-melcloud-control/wiki#adding-to-homekit https://github.com/grzegorz914/homebridge-melcloud-control/wiki#troubleshooting
Note!!!
You can only use one plugin at same time, firs remove other MELCloud plugin and its cache from homebridge before add this plugin.

Yes I was following these steps, removed the other plugin from Homebridge, deleted the bridge ad it’s devices from HomeKit. Restarted Homebridge and installed the new plugin. Added the Bridge again to HomeKit and tried to add my two devices who are shown after choosing manually but the last step fail’s.

Tried it yesterday and again today but no success. Added now the older plugin again and they MEL devices appear right in HomeKit without any additional action needed.

Can You enable debug mode in plugin config advanced section and get the complete log durig this proces.

I send the log files per direct email as they contain personal information.

The log and all data is correct, I think the response time from device is too long and pairing fail.
I have 5 A.C devices and all paired correct without any problem, also other users do not report any Probleme with pairing.
What version of Homebridge You use? May be try pairing with other device, reboot iDevice, reboot network, etc..

I use HomeBridge v1.6.0 with node.js v16.16.0 on a Synology NAS in Switzerland. The connection from MELCloud to the actual AirCon is via poor German mobile network. I switched now back to the plugin from Kristof and it paired immediately.
I will send you the complete log per email as I have at least learned now how to get it from Homebridge. I am an old men but still learning. I have tried pairing on iPhone, iPad and Mac. Internet speed from my NAS to MELCloud should not be an issue, the only slow connection is the 3rd world German mobile connection from the Aircon to MELCloud.


[Feature request] Allow updating speak config dynamically

Scenario
My application gets user config on load to get user's default locale and supported locales. I have a default speakConfig like below
{
     defaultLocale: { lang: 'en-US' },
    supportedLocales: [{ lang: 'en-US' }],
    assets: ['common'],
}

I tried 2 attempts:

Creating a wrapper component around QwikSpeakProvider and pass a reactive store data into my wrapper component.  The reactive store data is set based on API response. However, this way doesn't work.
Manually set value via useSpeakContext. For example speakContext.config.supportedLocales = [{ lang: 'en-US' }, { lang: 'nl-NL' }]. This way doesn't work either

Request
A new feature to allow config to be dynamic configured or if new feature is not possible, maybe some documentation updates?

Hi @ahnpnl,
I blocked with Object.freeze(config) editing speck config dynamically for two reasons:

making it reactive and updatable consumes unnecessary resources, because Qwik creates and serializes subscriptions
in my opinion a configuration must be available before the app is rendered and shouldn't be changed at runtime (also for security reasons... even if it's just a translation library, ok!)

So about you first attempt: if you create a component to update a state in the parent component, then Qwik (yet for the moment) doesn't make the state immediately available, but only within the child components (note the usage of the Speak component in this library).
I think you should use useTask$ with isServer (ex useServerMount$), i.e.:
root.tsx
export default component$(() => {
  const urlEnv = useServerData<string>('url');
  const origin = isServer && urlEnv ? (new URL(urlEnv)).origin : '';

  let config: SpeakConfig = {
    defaultLocale: { lang: '' },
    supportedLocales: [],
    assets: []
  };

  useTask$(async () => {
    if (isServer) {
      const response = await fetch(`${origin}/config.json`);  // just an example
      const result: SpeakConfig = await response.json();
      config.defaultLocale = result.defaultLocale;
      config.supportedLocales = result.supportedLocales;
      config.assets = result.assets
    }
  });

  return (
    /**
     * Init Qwik Speak (only available in child components)
     */
    <QwikSpeakProvider config={config} translationFn={translationFn}>
      ...
    </QwikSpeakProvider>
  );
});

Another way could be to get the config server side (adapter), for example in node, and pass it as an option to the root component (at least in production) and use a local config in dev mode.
However, all this reasoning, I don't think it has to do with the library, but with how Qwik works. I don't think they should be in the doc, also because Qwik is constantly changing on these things and they would never be updated...

Thanks, I think using useTask$ with isServer would work since it will block rendering until the async api call is resolved. My 1st attempt actually tried to optimize by not blocking rendering but do streaming.
Thank you for your response, I think I would have to take the downside about blocking rendering.

I tried to allow config dynamically configurable, just remove this line: https://github.com/robisim74/qwik-speak/blob/434c2ebf9e49751edf8bdb58199518780a9928bb/src/library/qwik-speak-component.tsx#L80,  but I get this error from Qwik:
Error: State mutation inside render function. Move mutation to useWatch(), useBrowserVisibleTask() or useServerMount()
The message is not updated (useWatch() + useServerMount() = useTask$), but in the end Qwik doesn't seem to allow the change once the context is initialized during rendering.
If you have any idea, we could work on it.


Update runtime for ROCm6.0 (SWDEV-434155)

This PR enables CuPy to use the updated API's in hip/cupy_hip_runtime.h for ROCm6.0.

Fixes issues related to https://ontrack-internal.amd.com/browse/SWDEV-434155:

Addresses changes  for the removal of "memoryType" in hipPointerAttribute_t
Added support for using updated "gcnArchName"
Updates GitHub runner ci/cd

Passing log:
https://amdcloud-my.sharepoint.com/:u:/g/personal/csrikris_amd_com/EU4ofnSXhulHkls1k-G6geUBFOiHy2nVh3swfyglRS62_w?e=ucYqhn


Microsoft Copilot

I would like Microsoft Copilot to be integrated into VS Code as more people might use it, such as myself, because of the easy access and easily understandable responses that are well detailed 90% of the time. This would be extremely helpful (At least to me) who use AI to give feedback on already created works and search for issues to correct problems in their code.

Thank you for filing this issue, we do have an extension for this,
Please try using https://marketplace.visualstudio.com/items?itemName=GitHub.copilot


Accept multiple function declarations on a single line.

See #100

@dherman I remember you saying that the asm.js spec is not meant to be a grammar. Is the change proposed in this pull request and issue #100 needed, or do you interpret the existing asm.js spec to cover code like this?


Instances for MonadThrow and MonadCatch typeclasses

From exceptions package

@starper test-fixture won't support this out of the box. You'll have to write any related instances yourself. It's not that opinionated. Otherwise, you have two options.

monad-mock, a latteral improvement over test-fixture. Alexis wrote an example how to use it.
fixie, a fork with an incrementally different opinion using ExceptT underneath the testing monad transformer. Currently, it has a MonadError instance, but MonadThrow/MonadCatch could be easily added.

@jxv, I think that @starper is simply talking about lifting instances for MonadThrow and MonadCatch on TestFixtureT, which are quite reasonable, and ought to be added.

@starper , do you have an example use case? It could be really that simple as she mentioned.

@jxv @lexi-lambda yes, i'm talking about instances for TestFixtureT like the one for MonadError in this issue. BTW, do you recommed to use monad-mock instead of test-fixture?

do you recommed to use monad-mock instead of test-fixture?

That’s largely up to you. They have slightly different philosophies. We’ve mostly stopped using test-fixture and have switched to monad-mock + hand-written fake implementations instead, but test-fixture is not inherently bad, it’s just a different style of testing. You might also be interested in my mtl-style-example repo, which tests some mtl-style Haskell code without any mocking at all (which is really the ideal if you can help it).

@lexi-lambda Thanks


Feature/descw 547 subcontractors form view and edit improvements

Pull Request Considerations
Although some of these things can only be done after merged, please ensure all items have been considered when doing pull requests.

Updated README.md(s) for new functionality or changes.
Did you update .env / create a new environment variable?

Did you update/create an explanation in the appropriate README.md file for it?
Did you make/update an Openshift configmap/secret template for it?

Have you changed any Openshift configurations/templates?

Did you execute the appropriate oc process command on the template to put the configmap/secret in openshift?

Did changes trigger Openshift image rebuilds?

Check Openshift instance of application to ensure your changes are working.

Did you update the changelog CHANGELOG.md.

used to keep track of development changes including (env changes, docker changes, functionality changes)
help developers keep track of new/updating changes in code.

Cat hit enter key on pull request. -_-


cant use zero-offload

assert not args.offload, "zero-offload is not currently supported but coming soon!"

Hi @yanqiangmiffy , we are working on this.

Thanks！Looking forward to your work

@yaozhewei 可以了吗？急用

@yaozhewei Why can't we use zero-offload? Can you tell me the specific reason? I can successfully enable zero-offload。

Is this ok？

I am also curious as to why the current version does not support the zero-offload feature. This seems to be a basic functionality powered by the deepspeed library, and it could be easily integrated into the codebase. Are there any conflicts with other components that prevent its implementation?

This is now supported: https://github.com/microsoft/DeepSpeedExamples/pull/553


[AutoPR portal/resource-manager] Add Swagger and Examples for shared dashboards to Azure

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5811

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/4721


Add Touch Bar / Touch ID support

The Macbook Pro with Touch Bar has a physical Touch ID button that can be clicked. This key also functions as the notebook's power key.
This PR allows PowerKey to work with the Touch ID button. Single-tap the Touch ID button to input Forward Delete or any of your favorite PowerKey scripts or commands.
Many thanks to JD Lien (@jdlien) for his work in this branch! I made a few small changes, mainly to fix Xcode warnings. The initial PR was here:
https://github.com/pkamb/PowerKey/pull/54
Apple support site for the Touch ID (power button) key:
https://support.apple.com/en-us/HT207054

@jdlien I removed the handler for triple clicks on the Touch ID button. Do you find yourself using that? I think I'd prefer to leave the triple-click accessibility handler in place, and only intercept single taps of the key.

I didn't really use it, but I mainly put it there because the response to clicking the delete key is slow... so you could easily find yourself triple-clicking it if you wanted to delete three things quickly, and then you'd get the accessibility menu.
So if you're removing that, users need to be cognizant of the fact that they have to click the button -slowly- if they are clicking multiple times. I'm sure this is one of those things that users would get used to... unless, of course, you're able to find a way to eliminate that delay in the click. I have no idea how you'd do that... the old Eject key had a similar problem, but I think this is implemented differently.

Good point. Re-added after testing a bit... tapping it fast now either hits the single or triple click, and activates PowerKey in either case. Feels more natural.
May need a setting for this at a later date if people are needing the accessibility shortcut as well. But it may be that you'll want to use one or the other, in which case this isn't a problem.

Good idea. I think this would be a good option for people that might need that shortcut. Or possibly they could assign it to something else.

Touch Bar support has been released in version 1.6!
https://github.com/pkamb/PowerKey/releases


can't pull private image from docker hub despite credentials entered

Bug description
I can't pull a private image from the docker hub althought login details were entered on the registry screen.
Expected behavior
After entering hub.docker.com credentials into the form on the Registry screen I can pull my private images from the docker hub.
Steps to reproduce the issue:
Steps to reproduce the behavior:

Go to the registry screen
enter your docker.com credentials into the box that says "Docker Hub"
Update and wait for the green confirmation
GO to the image screen and try to pull a private image

Technical details:

Portainer version: 1.19.1
Docker version (managed by Portainer): 18.06.1-ce
Platform (windows/linux): Linux
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer):

docker run -d -p 9000:9000 -e "VIRTUAL_HOST=my.domain.com" -e "LETSENCRYPT_HOST=my.domain.com" -e<EMAIL_ADDRESS>-v /var/run/docker.sock:/var/run/docker.sock -v portainer_data:/data portainer/portainer

Browser: Chrome 69

Additional context
n/a

Could it be related to https://github.com/portainer/portainer/issues/1900#issuecomment-421908480 ?

I was unable to reproduce this on Portainer 1.23.2. @Helmi can you try and reproduce this on version 1.23.2 of Portainer and without your reverse proxy? @jtbrough I tried your suggestion with the special characters and had no issue


Constrain CenterOwner window to screen.

When a window is opened with WindowStartupLocation.CenterOwner it can be opened partially offscreen. This can make it particularly tricky if the title bar is shown offscreen as the user can no longer move the window fully on-screen.
Constrain the window to the current screen when showing in this mode.
This logic is also present in WPF: https://github.com/dotnet/wpf/blob/3880169c020787b7e3383acb3679e5e4d538c602/src/Microsoft.DotNet.Wpf/src/PresentationFramework/System/Windows/Window.cs#L3707-L3711
Fixed issues
Fixes #2578

Ah, talk about bad timing! I'm removing it from the queue due to the potential crash added.


Streamer mode

would it be possible to add a feature to make the radar invisible to streams such as on obs and discord

would it be possible to add a feature to make the radar invisible to streams such as on obs and discord

Does this plugin work for you? Just not visible in Dalamud

it does not

Just don't monitor capture, if you are using OBS it doesn't need to inject to the game. If you game capture then you won't capture any overlays.


Unexpected module loading through NuGet

Steps to reproduce
Get VS 2017 RTW.
Install an “experimental” VSIX of NuGet like the one here by double clicking on it:
\nuget\nuget\Share\Drops\CI\NuGet.Client\dev\2381\artifacts\VS15\NuGet.Tools.vsix
Open PackageManagerConsole…
Expected behavior
correct version of  'NuGet.PackageManagement.PowerShellCmdlets.dll' is loaded
Actual behavior
the wrong NuGet.PackageManagement.PowershellCmdlets.dll is pulled in
Environment data
PSVersion                      5.1.14393.693
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14393.693
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Seems like this is an issue with VS2017 which hosts PowerShell and has their own PackageManagement module?

NuGet Package Manager Console (PMC) hosts PowerShell for running NuGet cmdlets and executing install/uninstall/init package scripts. It imports <extension root>\Modules\NuGet\NuGet.psd1 module which in its turn depends on a nested module NuGet.PackageManagement.PowerShellCmdlets.dll.
The last one is loaded from wrong location when working with "experimental" VS instance.

Any updates here?
ETA?

Close via #5161


Fix Typo Preventing Usage of Deprecated "update_route_table" Variable

Fix a typo that prevented the deprecated "update_route_table" variable from being utilized

Good finding @SpaicyGaming. Thanks for the PR. LGTM.


Why there is no Chinese in MMS?

zh-cmn

does cantoness not included?

@lucasjinreal For Chinese Mandarin(cmn) we support ASR but not TTS. Check this table

oh，so pity


Add a windows testing step

This adds a step that runs the tests on a Windows 2016 w/ Containers instance.
To get our tests passing under windows, I replaced the glob package with z-glob which comes with some nice fastwalk optimizations for linux.
For the actual infrastructure, I created a hand-rolled m4.large windows instance running Windows 2016 Server. Will replace with something more scalable when we need it.

👏🏼


Add Trivial Networking Library and WireLib.Notify

WireLib.Net is a namespace for WireLib networking utilities. Currently included is only a shared trivial networking functionality that is extremely basic.
WireLib.Notify and WireLib.NotifyAll are similarly simple functions to print colored messages in the player's chat to notify them of something, like warnings.

I don't really know how to style WireLib.Notify better. I mean, I know it looks bad, but I also don't really care that much as long as it's not unreadable. Changing colors is easy anyway. Nothing but E2 compiler warnings have been changed to use this.
There are 4 levels of WireLib.NotifySeverity, None, Info, Warning, and Error, demonstrated below. Additionally, you can optionally change the color of the text body. This does not mirror the entire capability of chat.AddText and instead is limited in this way for efficiency.

Add WireLib.Net (wire/sh_wirenet.lua)

WireLib.Net.Start trivial mirror for net.Start
WireLib.Net.Receive trivial mirror for net.Receive

Add WireLib.Notify to notify a single player
Add WireLib.NotifyAll to notify all players
Add enum WireLib.NotifySeverity for use with WireLib.Notify
Change warning ErrorNoHalt in E2 compiler to use WireLib.NotifyAll

Feel like theres a better way to do this. At the moment this is just a basic wrapper as to not use AddNetworkString. Okay. But now you're adding the overhead of sending this static string to every single net message. I'd be more accepting of a library that would just network a byte or two, that would be significantly smaller and support a ton of messages. It's also kinda wrong to call this a library when it just brings this small functionality.
Additionally I don't think the users should be notified in their chat, just in the console

Oh yeah, and that typed networking library I wanted I finally got around to making. So I guess that'd have to go after this PR.
Then we can rewrite all the net.WriteTable use to use typed structs instead.

Sorry, was a bad idea to start with sh_wirenet. Don't know why I thought it would need to be like that.

So now it notifies the server in MP using only MsgC and the player in SP using a Gmod notification. I think that's the best of both worlds.


Routing issues when upgrading to Fluxible 0.5

I saw some inconsistent routing behavior when upgrading to Fluxible 0.5, where a route would render in an incomplete state. After some digging, my issue was resolved after I specified this in Application.jsx (which I saw being used in fluxible.io's github repo):
    shouldComponentUpdate(nextProps, nextState) {
        return nextProps.isNavigateComplete;
    }

Is this something that the Application components in the flux examples should have as well?

This was a decision that we made to make navigation be optimistic, meaning it will show changes immediately rather than waiting for the navigateAction to complete. This improves user-perceived performance, but does mean that you should handle it correctly in your application. You may want to set a loading flag in your state somewhere to show a loading spinner, but I wouldn't recommend blocking any re-renders until the action has completed or the user may not know that anything is happening.


Custom icons don't work with CDN configuration

Gitea version (or commit ref): 1.9.5
Git version: 2.17.1
Operating system: Ubuntu Server 18.04
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist: https://gist.github.com/RalphORama/8971298a0734ac3724b9765e5652f222

Description
I have gitea set up with a CDN as specified in the docs.  I'm trying to change the default user avatar (also specified in the docs) by placing my custom image in /var/lib/gitea/custom/public/img/avatar_default.png, but the default avatar doesn't change.
I've tried:

Restarting gitea (duh)
Clearing my browser cache
Clearing my browser cache & purging all files on CloudFlare

only my CDN is under CloudFlare)

Unfortunately, none of these attempts fixed the issue.

I think v1.9.5 haven't support that feature. refs: #7911

Sounds good.  I suppose I could create a custom folder on my CDN and modify templates in custom to point at custom assets.

Update: Not sure what was going on with file caching, but I loaded the site on a different computer and the change took effect.  Probably some CloudFlare weirdness.


Missing 'CudaDnnDll.8.dll' from nuget

Running the project, I get this error for a file that hasn't been installed. Other versions of the CudaDnnDll are installed so I guess it just wasn't included in the Nuget?
 Failed to load 'C:\Program Files\vvvv\vvvv_gamma_2019.1.0-0975-gbe94390fc5\CudaDnnDll.8.dll'!    at MyCaffe.common.CudaDnn1..ctor(Int32 nDeviceID, DEVINIT flags, Nullable1 lSeed, String strPath, Boolean bResetFirst, Boolean bEnableMemoryTrace)    at MyCaffe.MyCaffeControl1.LoadLite(Phase phase, String strSolver, String strModel, Byte[] rgWeights, Boolean bResetFirst, String strStage, Boolean bEnableMemTrace)
at Main.ExploringmyCaffe2Application.UpdateDEBUG(PatchTracer tracer) in C:\Users\Igor\Documents\TFProjects_In-House\VLMyCaffe\ExploringmyCaffe2.vl:line 185248`

Hi again. Should CudaDnnDll.8.dll have been packaged with the Nuget, or should it be unnecessary?

My apologies, I've looked further into it and discovered there is a problem with the setup of my new IDE, It couldn't identify the placement of the native dlls from the Nuget.


Leaflet-Geoman does not work with WebView after update Java

I have verified Leaflet-Geoman (OS Windows 10) with WebView of the javaFX versions 15 ,16, 17,17.0.1,18-ea+5; JDK1.8.0_301.
In all these cases mouse Events as dragging, Edit Circle,Edit Polygon
as well as osm-map dragging do not work.
Is it possible to adapt some Leaflet Codes as Draw.Circle, Edit.Circle,
Drag.Layer to Java 1.8.0_301? Any help would be greatly appreciated.
Thank you

Would it be possible to set up a demo project, we never used Geoman with Java so we would need a start point

I would like to transfer the jar-File. How I can it do?

Create a zip and then upload it here. Or try for example: https://www.transfernow.net/

LeafletGeoMan-WebView.zip
Start the jar-file LeafletGeoMan-WebView.jar from console:
java -jar LeafletGeoMan-WebView.jar or by doppel-click
All features of Leaflet-geoman under jre 1.8.0_xxx (xxx < 291) work fine however them do not work under xxx>=291.

src.zip

Please write to my mail<EMAIL_ADDRESS>Would it be possible to have a web meeting in the evening? We can speak German.

After a meeting we found out, that it is a JavaFX but also Leaflet related problem. After switching to Leaflet 0.7.7 the dragging does work again

Hallo Florian,
ich bedanke mich vielmals bei Dir für die Hilfe. OSM-Karte bewegt sich!
Ich teste leaflet weiter.
Schöne Grüße
Liudmila
Falke Design @.***> schrieb am Mo., 29. Nov. 2021,
07:59:

After a meeting we found out, that it is a JavaFX but also Leaflet related
problem. After switching to Leaflet 0.7.7 the dragging does work again
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/geoman-io/leaflet-geoman/issues/1041#issuecomment-981339997,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE5TSUP5VFCMYLEOZB5JZOLUOMQDZANCNFSM5IQZFAEQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Resolve endless loop that blocks homebridge

this.disableWait doesn't exist anymore so getStatus kept on recursing on itself.
This commit removes this.disableWait and makes getStatus a bit morereadable.
Also added a comment to enableAutoRefresh because it was a bit unclear (to me) how it actually handles the auto-refresh
Fixes https://github.com/iRayanKhan/homebridge-roomba2/issues/10

Should be merged in a few hours. Thank you for this!
Just need to run it when I get home, and it will be on NPM.

Thank you!
NPM: 1.2.0


[sui-execution/v1] Hardcode banning entry init

Description
...and without this, the verifier implementation no longer needs
access to the protocol config.
Test Plan
Existing tests
Stack

#12740
#12741
#12742
#12774
#12776

Landed via #12785


test/release

Fix: sync
Feat: changelog

:tada: This PR is included in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


.query doesn't trigger events

Any queries made with the knex query builder don't trigger events
return bookshelf.model('User').forge().query().where({ name: 'Foo' }).update({ name: 'Bar' })

I'd expect this to trigger the "updated"/"updating" events but it doesn't. So 2 questions:

Is this expected? It's not a raw knex query so I'd think it would still trigger bookshelf events
If this is expected, is triggerThen the best way manually trigger the events?

Seems a bit messy to have to do something like this to get the event triggered
return bookshelf.model('User').forge().query().where({ name: 'Foo' }).update({ name: 'Bar' }, 'id')
.map(function (id) {
   return bookshelf.model('User').forge({ id: id }).fetch()
   .call('set', 'name', 'Bar')
   .then(function (model) {
      return model.triggerThen('updating', model)
   })
})

From the docs:

[Model#query] Will return this model or, if called with no arguments, the underlying query builder.

So the only events you'll get from calling methods on the QueryBuilder are Knex events. It's not part of Bookshelf.
If you want to use Bookshelf events you must use the Bookshelf API:
var User = bookshelf.model('User');

// Triggers "saving", "updating", "saved" and "updated".
// (your second code example - much terser btw.
return User.where('name', 'Foo').save({ name: 'Bar' }, { patch: true }).then(function (users) {
  ...
});

// Or something fancier using `query`: triggers "fetching", "fetched".
User.query('whereRaw', 'age % 2 = 0').fetch().then(function (usersWithEvenAge) {
  ...
});

Read #882 for more explanation. Possibly solved by #892.


Windows System Command does not work（e.g. 'shutdown','hibernate")

Microsoft PowerToys version
0.41.3
Running as admin

[X] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce
When I upgrade from v37.2 to v41.3 , System Command does not work,has this feature been removed?

I have tried to turn on this feature：

PowerToysReport_2021-06-30-00-14-23.zip
✔ Expected Behavior
No response
❌ Actual Behavior
it doesn't work!!!
Other Software
No response

Hi @realLiamTurner
System Commands have been localized in v0.41.3.
Can you try to search the command in your Windows language?

Hi @realLiamTurner
System Commands have been localized in v0.41.3.
Can you try to search the command in your Windows language?

Hi! @davidegiacometti
Thank you so much for your answer! It does work!

It would be great if there was a language choice! Because I prefer to type ‘hiber’ and press enter instead of typing ‘休眠’(hiberbate in chinese)‘.
anyway,thank u   XD


[#59, #48] Add hack for snoozed emails issue

described in #59 comments.

created against wrong repo.


563 With Context as Decorator

This is the best I was able to get for a rough morning's worth of work. I think I actually kinda like the direction it is heading. Still need to figure out what to do with staticmethods :(

I still haven't fully figured out how I would deal with the staticmethods, but iirc, we were already considering making both create_model and create_ensemble non-static for better default paths anyway.
I know we may be scrapping this implementation in favor of the queued logging and client side filtering. That said, if we need to be able to inject context into the Experiment methods, this is how I think I would do it to avoid re-settings fn attrs on all Experiment instances.


Fix Address of Korea Community

fix address to "https://electron-kr.github.io/electron-kr"
I changed it to '@ragingwind' request.
You can refer to #11078 in electron/electron

Thanks!


WINDOWS - Missing emoji's

Issue:
-On Windows OS there are a few emoji's missing within the menu
OS:
-Windows
STR:
-Open and scroll through the emoji's menu
Screen recording:
https://user-images.githubusercontent.com/93608357/233515655-ac955fda-0b83-4967-a461-d440aea8100f.mp4

What's the expect fix on this one? Like, should we limit the emoji set on windows to only ones they can see? Or limit the set of all emojis to just ones that are universal?

I think the consensus was to just hide the emojis that don't show up on windows. @Jekrimo  . . . you also mentioned that it was random which ones were not showing up. Did you resolve that?

I did resolve that! I was unawares of some stuff I had to do for the extension

@Jekrimo this seems to be an issue . . . are you working on this one?


Federico: Watch oldwoodward.com

Federico requests the watch of the watch_keyword oldwoodward.com. See the Metasmoke search here and the Stack Exchange search here.

!!/approve


[AIRFLOW-2261] Check config/env for remote base log folder

Make sure you have checked all steps below.
JIRA

[ ] My PR addresses the following Airflow JIRA issues and references them in the PR title.

https://issues.apache.org/jira/browse/AIRFLOW-2261

Description

[ ] Here are some details about my PR, including screenshots of any UI changes:
With this change, one should be able to enable remote log storage via REMOTE_BASE_LOG_FOLDER, REMOTE_LOGGING, and REMOTE_LOG_CONN_ID in either the config file or environment variables.

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:
The change is fairly minor.  Plus, I'm not sure how automated testing is set up for this (e.g. mock remote destinations)--update me and I'll add one.

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

[ ] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Codecov Report

Merging #3166 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3166      +/-   ##
==========================================
- Coverage   73.85%   73.84%   -0.02%
==========================================
  Files         191      191
  Lines       14349    14349
==========================================
- Hits        10598    10596       -2
- Misses       3751     3753       +2

Impacted Files
Coverage Δ

airflow/config_templates/airflow_local_settings.py
84.21% <100%> (ø)
:arrow_up:

airflow/jobs.py
79.91% <0%> (-0.18%)
:arrow_down:

Continue to review full report at Codecov.

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


Update Readme.md

Readme.md updated

DONE @meerhamzadev

Changes done @meerhamzadev

@meerhamzadev on my hacktoberfest profile its showing excluded. Can you pls resolve the issue.


[docs] Fix accessibility in Post to comment button

Resolves #36037

[x] I have followed (at least) the PR section of the contributing guide.

@ZeeshanTamboli
Thanks for the approval, and about span, I tried few different ways, however did not get success.
I am open for suggestions


tsp, loadtesting, update client.tsp to fix the path change issue for action operation

Overview
This is just client-side change and have no impact for service side. In typespec reusing 'post' action operation by myNewAction is someInterface.myOldAction would change the action suffix and see linter and issue.
So in client.tsp stopTestProfileRun is LoadTestRun.stop; would change the path to /test-runs/{testRunId}:stopTestProfileRun which actually is  /test-runs/{testRunId}:stop. If we want to change the operation name the recommanded way is to leverage the @clientName to do that.
Data Plane API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

API Info: The Basics
Most of the information about your service should be captured in the issue that serves as your API Spec engagement record.

Link to API Spec engagement record issue:

Is this review for (select one):

[ ] a private preview
[ ] a public preview
[ ] GA release

Change Scope
This section will help us focus on the specific parts of your API that are new or have been modified. Please share a link to the design document for the new APIs, a link to the previous API Spec document (if applicable), and the root paths that have been updated.

Design Document:
Previous API Spec Doc:
Updated paths:

Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.
Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.
❔Got questions? Need additional info?? We are here to help!

 Contact us!
The Azure API Review Board is dedicated to helping you create amazing APIs. You can read about our mission and learn more about our process on our wiki.

💬 Teams Channel
💌 email

Click here for links to tools, specs, guidelines & other good stuff
Tooling

Open API validation tools were run on this PR. Go here to see how to fix errors
Spectral Linting

Guidelines & Specifications

Azure REST API Guidelines
OpenAPI Style Guidelines
Azure Breaking Change Policy

Helpful Links

Schedule a data plane REST API spec review

Getting help

First, please carefully read through this PR description, from top to bottom.
If you don't have permissions to remove or add labels to the PR, request write access per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

Looks good for python


feat: add site favicon

Closes #62
I whipped up a quick favicon.svg for the site based off what was existing. Let me know if you're looking for something different and I can swap it out!

Hey @hay-kot, thanks for this PR! The SVG looks and works great, but unfortunately it blends in too much when the tab itself is also white or light gray. Is there anything we could do about this?

Just pushed a gray version that looks good

Looks excellent, thank you!


checkov: 2.0.563 -> 2.0.566

Motivation for this change
Update to latest upstream release 2.0.566
Change log: https://github.com/bridgecrewio/checkov/releases/tag/2.0.566
Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

1 package built:

checkov


ensure link to coc is accessible

This repository uses all lowercase for the markdown files but when you click the link GitHub displays a 404 because it is looking for the uppercase version of the file.

@skylerkatz Thanks!!


Testing SWAP 2755 - no  request body for /api/v3/datasets in swagger

@nitrosx : As you suggested I use the SWAP-2755 for internal testing.
I saw that in the current version there is no Request body for '/api/v3/datasets' in swagger anymore. Therefore one can no longer use swagger to push a new dataset. I guess this is some sort of open-api issue.

compared to Sample where things are still ok:

looking a bit into this again I saw that in the openAPI-json the 'requestBody' section is missing for the 'datasets' endpoint in the SWAP-2755 branch.

compared to

This was reported (in less detail) in #122

Done and tested with v4.2.1


Doesn't work in IE 8.0.7601.17514 / Win 7

I've just tried your polyfill in the above version and it doesn't work. It fails here: https://github.com/eligrey/classList.js/blob/master/classList.js#L174
Turns out the exception thrown isn't what you were expecting. The exception has no number property. It is a TypeError stating that "getters & setters can not be defined on this javascript engine".
Removing the condition if (ex.number === -0x7FF5EC54) { ... fixes the problem.

Hmm, odd. I will need to investigate further before removing the condition, but thanks for informing me. Years ago when I first made classList.js that condition was working fine in IE8, but maybe a security patch changed something.

Agreed, having a condition there is probably a good idea. There may be some other kind of duck typing you can check on the error.

I can also confirm this issue. ex.number results to undefined in IE11's compatibility mode for IE8.
Removing the condition seems to solve it.

:+1: Tested IE8 in XP VM and this is still a problem. The solution proposed works great.

How can you replicate the problem? I have the same IE8 version in a Windows XP VM and I don't have this issue.

Windows XP VM from Microsoft running in VirtualBox on a host Macbook.

I'm running my Windows XP VM (32 bits) in a Virtual Box hosted on a Windows 7 VM (64 bits).
First time I hear about transpiling. I simply load classList.js in my  and it has been working well so far.

What you had hold of initially was an ActiveX exception (the number is an HRESULT). Clearly IE 11 is no longer using ActiveX objects behind the scenes.
Might as well remove the check as it simply causes all other exceptions to fail silently. There's really no way to deal with the exception anyway. What's the point of having it enumerable in some browsers and not in others?
Realize this will never be a perfect imitation of classList. It's simply an impossible task; for one, the array-like object returned is not live. Then there's the unsolvable IE 8- issue at hand.
A "polyfill" that doesn't actually work as expected only serves to deceive other scripts. It fools their feature detection into thinking that the feature is there, but it's really just an incomplete imposter.
In general, DOM "polyfills" are highly ill-advised and this one is a good example of why. Better to write wrappers (e.g. hasClass, addClass, etc.)
Also note that transpiling is just a red herring here.
HTH

Sorry, I'm super busy with my proprietary stuff and missed this a year ago. I merged #43 just now.
I'm sorry about this issue affecting anyone.

@eligrey Could you please update the classList.js on the CloudFlare CDN and post the updated URL's in the readme.md?

I use this commit github.com/beck/classlist-polyfill/commit/d94a623c25bc69caf09f7089c0066fd65e760e82 but it seems ie11 wont support multiple args. So I have to write:
e.classList.add(a);
e.classList.add(b);


Dependabot php composer strange/wrong behaviour

I have created a test repo to reproduce the problem: https://github.com/b1rdex/dependabot-composer-bug
As you can see in b1rdex/dependabot-composer-bug#2 (and other opened pull requests), dependabot tries to rearrange the order of some records and replaces emojis with ?.
But when I run composer update package-name locally with composer 1.9.0 there are no such changes.
Such rearrange/emojis replacement breaks "auto-mergeability" in my setup because each opened pull request tries to change the same lines and that produces conflicts with local updates.
ps. I run dependabot on my self hosted Gitlab, but luckily issue is reproducible on GitHub too,

AFAIK, composer.lock contains content-hash key, so it cannot be "auto-merged" anyway. See https://github.com/composer/composer/issues/6929

I get the same re-orderings and emoji removal when running locally with Composer 1.9.0 (running composer update beberlei/assert --no-autoloader --no-scripts). Perhaps these are changes that come in with Composer 1.9.0, rather than an issue with Dependabot?

Looks like this is indeed an issue with Composer / Packagist: https://github.com/composer/composer/issues/8268

AFAIK, composer.lock contains content-hash key, so it cannot be "auto-merged" anyway. See composer/composer#6929

@fmasa content-hash field changes only on composer.json changes. If dependabot simply updates with no constraint change, then almost all merge requests are mergeable.
@greysteil thank you for the composer issue info.

Picking this back up here after the discussion in https://github.com/composer/composer/issues/8268.
I'm not sure what to do here. https://github.com/b1rdex/dependabot-composer-bug/pull/2/files#diff-1da2c7edc898c70e5a79a9997c98ceccR9274 is definitely wrong, but I'm guessing these lines are in the Dockerfile for a reason. Further, since emoji aren't supported for me locally Dependabot would cause flapping for many people if it didn't convert them to question marks.
It's not ideal, but I think we're best off leaving things how they are here, unless there's a fix  possible in Composer.


Cameras load, but then constantly reload while viewing

Using version 3.1.0
All configs are similar to thistype: custom:webrtc-camera
entity: camera.front_door
background: true
muted: true

All 10 of my Amcrest cameras load (not having to disable backchannel anymore)
While viewing the camera feeds in HA cards with WebRTC, they continually reload every 3-10 seconds (shows loading2 while reloading). This is while just watching the screen and no interaction. Sometimes they don't load back after reloading and are just black and loading with MSE in corner. The ones that don't reload will sometimes reload again a few seconds later and work again. This all happens over and over again.
This doesn't happen every time, but most of the time it does.
While camera is not loaded (or reloaded), it does not show up in the go2rtc UI list. Once it loads back, it shows back in the go2rtc UI list
I didn't have this issue prior to upgrading to v3

Do you have all 10 cameras on same page? What technology do you see on top right corner for all of them?

Do you have all 10 cameras on same page? What technology do you see on top right corner for all of them?

I have a page with all 10 on one and other pages split up with 5 on each. It happens on all of them.
They have RTC in to top right after successfully loading. Sometimes they don't load back after doing a reload and are just black with a loading spinner with MSE in corner.

If you send me your email, I can try to do a screen recording of it to send to you. I don't really want to post my streams publicly. Not sure if that would be helpful or not.

Are you sure v2 uses RTC? It is also can work over MSE.

I just reverted back to v2.3.1 and all cameras are working fine again with RTC.

Yes, I am in the local network on my desktop computer.

If your stream has AAC audio - v3 will prefer MSE technology

Hi i'm having the same issue i have all the streams in my home assistant they are always reloading around every 10 seconds but it seems to be random

fixed it by adding this in home assistant
mode: mse

Hi @AlexxIT  - I have this same issue where the camera feed constantly reloads. I had mode at RTC, but changed to to MSE per this thread. It doesn't seem to change anything

These are my settings:
type: custom:webrtc-camera
url: rtsp://<IP_ADDRESS>:7447/{redacted}
style: ".fullscreen {display: none}"
mode: mse
media: video

Hi @AlexxIT - I have this same issue where the camera feed constantly reloads. I had mode at webrtc, but changed to to mse per this thread. It doesn't seem to change anything. These are camera feeds from UniFi Protect, I have a good network setup and the device they are streaming on has a solid connection as well.

These are my settings:
type: custom:webrtc-camera
url: rtsp://<IP_ADDRESS>:7447/{redacted}
style: ".fullscreen {display: none}"
mode: mse
media: video

Are you accessing home asssistant trough you local IP or going trought the internet? Also i fixed this issue cant remember exactly how ,but i will try to reply back if i find it


Improve error message with invalid configuration

Description

Closes #2655

The integration expects an object for its configuration and we assume that this user input is always an object and we unpack it into variables from the integration parameters.
This can lead to confusing error messages when the user inputs an invalid configuration or even no configuration at all, e.g. after running npx astro add starlight.
This PR adds an extra check to ensure that the configuration is an object and throws a more helpful error message if it is not.

size-limit report 📦

Path
Size

/index.html
6.15 KB (0%)

/_astro/*.js
22.41 KB (0%)

/_astro/*.css
13.73 KB (0%)


Cleanup

Codecov Report
Base: 82.05% // Head: 80.88% // Decreases project coverage by -1.16% :warning:

Coverage data is based on head (732754a) compared to base (fc2e244).
Patch coverage: 75.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
- Coverage   82.05%   80.88%   -1.17%
==========================================
  Files           4        4
  Lines         273      272       -1
  Branches       41       39       -2
==========================================
- Hits          224      220       -4
- Misses         37       39       +2
- Partials       12       13       +1

Impacted Files
Coverage Δ

jupyter_server_fileid/extension.py
47.61% <0.00%> (ø)

jupyter_server_fileid/manager.py
87.96% <85.71%> (-1.44%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Order of reports-in-progress is confusing

Steps to reproduce:

Start ladybug-test-webapp using frank-runner/specials/ladybug, without FF!.
Run the shown links on http://localhost in the shown order:

Open report-in-progress #2 as shown:

The nodes are "Waiting for thread to start". Expected behavior is "Waiting for message to be captured".

This is a backend issue, checking the network tab I can see that report in progress with storageId=1 is requested, while report in progress with storageId=2 is returned.


[VS Code] LiveShare does recognize a file which has # in filename as binary even file itself is not

Product and Version [VS/VSCode]: VSCode
OS Version [macOS/Windows]: Windows 10 (tested and confirmed on both Build 1709 AND 1809)
Live Share Extension Version: 0.3.1151
Target Platform or Language [e.g. Node.js]:
Steps to Reproduce / Scenario:

Open a file contains '#' in filename

Remote vscode client shows an error message saying

The file is not displayed in the editor because it is binary or uses an unsupported text encoding.

even the local vscode (liveshare host) opens the file without any encoding errors. if I click Do you want to open it anyway? to continue anyway. the vscode shows only
Unable to open this file -- only text documents are supported

Extras
Switching the sides (host and client) did NOT help at all. causing only same errors.

Screenshots

@Alex4386 Thanks for reporting this! We hadn't seen this bug before, so we'll use this issue to track resolving it 👍

Adding myself since I think this is related to a bug on my plate.

This should be fixed now.

In the newest version 1.39.0 on Windows it still pops up the same error also when there's "#" in the folder's name

Same still happening here.


Add progress bar to quint test and quint run

Closes: #1457
test

run

[ ] Tests added for any new code
[ ] Documentation added for any new functionality
[ ] Entries added to the respective CHANGELOG.md for any new functionality
[ ] Feature table on README.md updated for any listed functionality

@bugarela What do you think of the result? Shall we add a flag to enable/disable the progress bar? Perhaps we should also only enable it if we are running with a TTY, in order to avoid polluting GitHub actions logs, etc.


Typed data array operations are slow, read/write one byte at a time

Optimized code for `_Float64List.[]`
 (func $_Float64List.\5b\5d (type $type$63) (param $0 (ref $type$17)) (param $1 i64) (result (ref null $type$56))
  (local $2 (ref $type$4))
  (local $3 (ref $type$66))
  (local $4 (ref $type$65))
  (local $5 i32)
  (local.set $3
   (ref.cast $type$66
    (local.get $0)))
  (block $label$1
   (if
    (i64.ge_s
     (local.get $1)
     (i64.const 0))
    (br_if $label$1
     (i64.gt_s
      (call $_TypedListBase.length
       (local.get $3))
      (local.get $1))))
   (struct.set $type$65 0
    (local.tee $4
     (struct.new_default $type$65))
    (i32.const 271))
   (call $IndexError.withLength
    (local.get $4)
    (local.get $1)
    (call $_TypedListBase.length
     (local.get $3))
    (global.get $global$54))
   (call $Error._throw
    (local.get $4)
    (call $StackTrace.current))
   (unreachable))
  (struct.new $type$9
   (i32.const 11)
   (f64.reinterpret_i64
    (i64.or
     (i64.or
      (i64.or
       (i64.or
        (i64.or
         (i64.or
          (i64.or
           (i64.extend_i32_u
            (array.get_u $type$4
             (local.tee $2
              (struct.get $type$66 3
               (local.get $3)))
             (local.tee $5
              (i32.wrap_i64
               (i64.shl
                (local.get $1)
                (i64.const 3))))))
           (i64.shl
            (i64.extend_i32_u
             (array.get_u $type$4
              (local.get $2)
              (i32.add
               (local.get $5)
               (i32.const 1))))
            (i64.const 8)))
          (i64.shl
           (i64.extend_i32_u
            (array.get_u $type$4
             (local.get $2)
             (i32.add
              (local.get $5)
              (i32.const 2))))
           (i64.const 16)))
         (i64.shl
          (i64.extend_i32_u
           (array.get_u $type$4
            (local.get $2)
            (i32.add
             (local.get $5)
             (i32.const 3))))
          (i64.const 24)))
        (i64.shl
         (i64.extend_i32_u
          (array.get_u $type$4
           (local.get $2)
           (i32.add
            (local.get $5)
            (i32.const 4))))
         (i64.const 32)))
       (i64.shl
        (i64.extend_i32_u
         (array.get_u $type$4
          (local.get $2)
          (i32.add
           (local.get $5)
           (i32.const 5))))
        (i64.const 40)))
      (i64.shl
       (i64.extend_i32_u
        (array.get_u $type$4
         (local.get $2)
         (i32.add
          (local.get $5)
          (i32.const 6))))
       (i64.const 48)))
     (i64.shl
      (i64.extend_i32_u
       (array.get_u $type$4
        (local.get $2)
        (i32.add
         (local.get $5)
         (i32.const 7))))
      (i64.const 56))))))

Currently typed data lists use a i8 array as the buffer and read/write one byte at a time. I can't tell from the code why we use a byte array in typed data lists, but I suspect it has to do with JS interop or getting different views of a lists without copying (which would break sharing).
Opening this issue to document the requirements and alternatives.
It's a bit difficult to find where the typed list constructors and accessors are generated, so here are a few links:

Allocating typed data lists: https://github.com/dart-lang/sdk/blob/82b78916fd747e3069c6cc0ab3bb1559952e3019/pkg/dart2wasm/lib/intrinsics.dart#L1375
[] and []=: https://github.com/dart-lang/sdk/blob/82b78916fd747e3069c6cc0ab3bb1559952e3019/pkg/dart2wasm/lib/intrinsics.dart#L216
For the class definitions we mostly use VM's patch: https://github.com/dart-lang/sdk/blob/main/sdk/lib/_internal/vm/lib/typed_data_patch.dart

We override _TypedList._getFloat32, _TypedList._setInt64 etc.

We discussed this with @askeksa-google. The main issue here is the views. Example:
import 'dart:typed_data';

void main() {
  Float64List f64s = Float64List.fromList([1.2, 3.4, 5.6]);
  print(f64s);

  // Uses the same storage with f64s
  Float32List f32View = Float32List.sublistView(f64s);
  print(f32View);

  // Updates f64s
  f32View[2] = 7.8;
  print(f64s);
}

When creating a view we can't copy the original list's array as an update to one of the original list or views need to update all the others.
The solution is to use a byte array as the backing storage which is shared with all of the views. Each view class can then do reads and writes according to their element types.
The problem is, unlike linear memory, Wasm GC arrays don't support reading elements other than the array's declared element type. So when we use an i8 array as the storage type we have to read and write one byte at a time.
It seems to me like this can potentially be considered as an omission in the MVP GC spec, and maybe we can consider adding a "byte array" type that allows reading/writing different sizes in single read/write, similar to linear memory instructions i32.load, i64.load etc. (I think @eyebrowsoffire has some ideas on this)
However this will have to be turned into a proposal and in the best case it will take months before we can use this new array type.
An alternative that doesn't rely on any new platform features is: we optimize the case when a typed array is used directly (or with a view with the same element type) by allocating an array with the right type. The views will have to do the conversions if necessary. For this we also have two options:

We create a view for every conversion. E.g. a view type that reads F64 from I32 array, a view for reading F32 from I32 array and so on.
This will have smaller overhead in views when reading elements of different types, but there will be N^2 view classes. (where N is the number of typed array element types)

We create a single view class for each element type, and add a "read one byte" method to typed arrays. So if I have a F64 array and a F32 view, the view will use the byte read/write methods of the original array.

Fix in https://dart-review.googlesource.com/c/sdk/+/312907.


A related issue?
https://github.com/flutter/flutter/issues/2976#issuecomment-305027045

A workaround:
$ ulimit -S -n 2048

This command increases the maximum number of files the system can open at the same time. 2048 may changes depending on machines.


Reference: https://www.digitalocean.com/community/tutorials/how-to-set-up-an-nfs-mount-on-ubuntu-18-04

Thanks @nickshek. We don't yet deploy on Ubuntu with Ansible, though we're looking to have all our roles support it moving forward. I'm going to merge this as is and release 1.1.1 considering:

the when clause will prevent it from affecting centos/rhel systems
testing this role in travis has always been something of an issue


fix: typos in comments, tests, and func names

The PR fixes typos.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

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

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/internal/tests/support.go
0
2
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
263

Relevant Lines:
663

💛  - Coveralls

Good Catch. Thanks!


Created QuantityViewHolderFactory to handle the Quantity Input type

Please add a test file similar to this: https://github.com/google/android-fhir/blob/master/datacapture/src/androidTest/java/com/google/android/fhir/datacapture/views/QuestionnaireItemEditTextIntegerViewHolderFactoryInstrumentedTest.kt
Once done please re-request review. Thanks!


Update README.md branch1

Task name:
Description:
Documentation changes required: (Y/N)
Added unit tests: (Y/N)
Attached related issue: (Y/N)
Checklist:

[ ] Task version was bumped - please check instruction how to do it
[ ] Checked that applied changes work as expected

All good


Feature Request: Sections

In the directions it would be great to another formatting option (other than list of direction). Being able to specify sections. For instance I'm adding a recipe for fried chicken and it would be great to call out different sections like: "The Night Before", "Friend Chicken", "Spicy Oil".
I used to use Pepper plate and you could format the sections like so.
[The Night Before]
Cut up and debone chicken
Season chicken generously with salt
[The Chicken]
In one bowl whisk together eggs, buttermilk and hot sauce
In another bowl whisk the flour, 2 pinches of salt
Place dutch oven over medium heat.
Add oil to a depth of 1.5 inches
Heat oil to 360F
Dredge the chicken (dry, wet, dry)
Cook chicken in the oil for about 12 minutes (9 for smaller pieces, 16 for larger pieces)
Let cool on a wire rack set over a baking sheet
[Spicy Oil]
Combine the cayenne pepper, brown sugar, paprika, cumin, and galric in a small bowl.
Add 0.25 cups of oil.
Brush the chicken with the spicy oil.

Having sections in the ingredients would also be great.

If you add a colon (:) at the end of the line with the section name, it gets rendered in bold face and not as a list item. Maybe that is already sufficient for your use case.

Oh yeah, that's exactly what I need. Thanks so much.


build: Update Vulkan-Headers to 1.3.259

CI Vulkan-ExtensionLayer build queued with queue ID 11480.

CI Vulkan-ExtensionLayer build # 634 running.

CI Vulkan-ExtensionLayer build # 634 passed.


Dynamic code generation is not supported on this platform

PleasantUI.Example    PublishTrimmed = true

Hi! I apologize for the late reply.
For some reason unknown to me, publishing via IDE is not possible if NativeAOT is activated in the project. Please use this command in the project folder:
dotnet publish -r win-x64 -c Release

Since there hasn't been a response, I'm closing this issue as resolved. If anything comes up, feel free to reach out and I'll reopen it.


Discovery page - Add "load more results" button to the end of all search pages

When you search for apps (via text, categories, type, or agency), the backend returns the first 24 results in addition to a next url (if necessary) to get the next set of results (a common pagination strategy). Center currently has no 'load more' button on the search results page that allows the next set of apps to be retrieved.

Passed -- tested on STG
Verified fixed by searching on a word like and which should bring up every app.  The Load More button appeared at the bottom several times.


use ensure vs on_failure and use cache directories

Shouldn't the backup in the container always be cleaned up (on failure and on success)? The ensure task item runs in both scenarios. Also, using cached folders avoids copying the backup from one container task to another. If you hijack the containers you can see it is indeed copied.
I can create a pull request for this: https://github.com/pivotal-cf/bbr-pcf-pipeline-tasks/compare/master...patrickhuber:master but never received customer feedback that it was working. My local tests showed that it worked as expected.

@patrick-huber-pivotal I've heard of this issue before. Can you please elaborate on why not cleaning up the backup artifacts in the Concourse jobs is a problem?
cc @jama-pivotal for context

I'll close this for now due to lack of activity. Feel free to reopen :)

@glestaris sorry for the late response. Not cleaning up the artifacts is an issue for most customers because they take up disk space on the concourse worker and can easily blow out the available disk space. In the end, the pipeline is just a vehicle for moving the backups from bosh deployments to an external store like S3 or NFS, so there is little value in caching them in the pipeline container volumes.


Delete button not visible when Title text is huge

@DhritiShikhar if this is the description, let's put a max length on the description field, say 256 characters and then implement a "Show more/less" link. Is this possible?

@Mgranfie this is the workitem title. A "show more/less link" makes sense. How about a vertical scroll bar?

I was talking about the Work Item description... was that example the WI
Title?!!
Is there any character limit on the Work Item title? If not it would either
make sense to impose one and or truncate the title and have a hover...
We are trying to stay away from scrolls and only scroll the entire work
item. Otherwise you can end up with columns of abutting scrolls and the
experience degrades.. with so many scroll bars..
On Fri, Feb 17, 2017 at 7:56 AM, Dhriti Shikhar<EMAIL_ADDRESS>wrote:

@Mgranfie https://github.com/Mgranfie this is the workitem title. A
"show more/less link" makes sense. How about a vertical scroll bar?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fabric8io/fabric8-planner/issues/966#issuecomment-280641809,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARTwqxsd39FIvPF53dPP8qcTcZl364wlks5rdZj_gaJpZM4MD7FQ
.

@Mgranfie there is no character limit on work item title yet.

Hi Dhriti...
I don't know if Unicode strings have a limit, I would cap it out at 256
characters to be reasonable. Let me know if this is too many...
Thanks,
Monica
On Wed, Mar 1, 2017 at 1:16 AM, Dhriti Shikhar<EMAIL_ADDRESS>wrote:

@Mgranfie https://github.com/Mgranfie there is no character limit on
work item title yet.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fabric8io/fabric8-planner/issues/966#issuecomment-283254946,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARTwq-gk4h8RIsxMXfRKB8ARzLixAt2Aks5rhQ1IgaJpZM4MD7FQ
.

@Mgranfie I think 256 characters is fine but please confirm it with some one else also.

@DhritiShikhar I am not able to find Todd on this repo... not sure if he has joined this. I have asked him in MatterMost... I will ask the platform team on MM too

@Mgranfie sure. Thanks!

@DhritiShikhar Let's go with 356 and truncate at the end of the allotted visual space that we have, not expanding it, but showing the full name in a hover.
Let me know if this makes sense...

@Mgranfie sounds good!


Straks hashrate is not accurate

One of my users reported that his hashrate was being marked as half what his miner was reporting. I've doubled the hashrate of the pool, pool 24 h history and miner stat on the ui for the time being.

I am looking at this at well, has anyone else started working on this item?


Cannot read property 'isCascadeDelete' of undefined

When having Account, Contact, Work Order, Work Order Line Item, Product, Product Consumed, Product Item and Order Product Objects selected and adding the Order-Object this error comes up:

This page has an error. You might just need to refresh it. Action failed: gvf2:DiagramOutput$controller$onDiagramUpdated [Cannot read property 'isCascadeDelete' of undefined] Failing descriptor: {gvf2:DiagramOutput$controller$onDiagramUpdated}

The error is not gone with a page refresh and even after fresh login to salesforce the error comes up when opening the stored diagram.
When adding Order-Object as the first or second object I was not able to reproduce the issue.

Duplicate of #58 - Steve was faster to create the issue.

Thanks Sascha!


[Question] Why do you exclude some airplay software

Thanks for making this nice and useful piece of software.
When experimenting how to make a workaround to spotify connect temporarily not working with spotty in LMS
I noticed this line
glExcludedModels[STR_LEN] = "aircast,airupnp,shairtunes2,airesp32,";
I  managed to get it working with the shairtunes2 plugin and LMS by removing it from the blacklist.
Since all 4 of the blacklisted programs are your own programs I wonder why you blacklisted them.
Sincerely Tycho

Everything that is AirPlay can be used directly by Spotify client.

Ok , I can indeed use the airplay audio directly by the spotify clients from apple devices  or from linux with pipewire and it's  raop module, but the advantage of using connect is to play make the stream independent of the hardware you are running the client on

To be more precise

airupnp: use spotupnp instead of airupnp + spotraop
aircast: AFAIK, all Spotify controller support chromecast on all OS
airesp32: Spotify is built-in squeezeesp32
shairtunes2: this is LMS ecosystem, better use Spotty rather convoluted shairtunes2 + spotraop

Thank you, makes sense , the convoluted spotraop+ shairtunes2 was intended only as stopgap/workarround when the spotify connect  option in Spotty does not work