Turning on light via Siri turns on fan as well

I have a Haiku L series fan with a light. I’ve separated the light from the fan in HomeKit, but when I ask Siri to turn on the light, it turns on the light as well. Not sure if bug or I’m doing something wrong

@mathewthomas1721 I'm fairly sure this is a HoneKit issue with the way it parses the name. I had to rename my light without using any indication of "fan" and now it works to turn on solely the light.

@mathewthomas1721, I have also separated the fan and the light. My fan is named Big Fan and my light is named Big Fan Light. When I tell Siri “Hey Siri, turn on the big fan light”, only the light comes on.
What is your fan named? What is is your light named? What exactly are you saying to Siri and can you see what Siri thinks you’re saying?

My setup is as follows :
The fan is named Bedroom Ceiling Fan, the light is named Bedroom Ceiling Light. Bedroom Ceiling Fan is an individual device, while Bedroom Ceiling Light is grouped with all the lights in the room in a group called Bedroom Lights.
Saying “Hey Siri, turn on the Bedroom Lights” turns on all the lights including Bedroom Ceiling Light, but also turns on Bedroom Ceiling Fan.
I’d be happy to check what Siri thinks I’m saying, how can I go about doing that?

It sounds like Siri is correctly interpreting what you're saying.  For what it's worth when I speak to Siri using my iPhone it shows the text of what it hears as I speak.
I've never used groups so I don't know how that might play into this.  What happens if you say "Hey Siri, turn on the Bedroom Ceiling Light"?

So it seems like it only messes up when I group the Ceiling Light with the
other lights. Treating it as an individual device, it seems to work fine.
My theory is, when I say, "Hey Siri, turn on the Bedroom Lights", it's
actually just hearing "Hey Siri, turn on the Bedroom", and turning on
everything in the bedroom.
Thank you for the help, I'll close this issue, clearly this is not a plugin
problem.
On Sat, Jan 14, 2023 at 9:22 PM oogje @.***> wrote:

It sounds like Siri is correctly interpreting what you're saying. For what
it's worth when I speak to Siri using my iPhone it shows the text of what
it hears as I speak.
I've never used groups so I don't know how that might play into this. What
happens if you say "Hey Siri, turn on the Bedroom Ceiling Light"?
—
Reply to this email directly, view it on GitHub
https://github.com/oogje/homebridge-i6-bigAssFans/issues/26#issuecomment-1383029689,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFMJVLXD3EEDAL3XB5IMXXTWSNNNVANCNFSM6AAAAAAT3SFVBI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks for following through.


ISO8601 - Mixed Basic and Extended format

Hi there!
I recently noticed a discrepancy in how Timex formats its ISO-8601 datetime strings.  It looks like its a mixture of both the basic and extended formats.  Basic format (YYYYMMDDThhmmss±hhmm e.g. 19850412T101530+0400) and Extended format (YYYY-MM-DDThh:mm:ss±hh:mm e.g. 1985-04-12T10:15:30+04:00).
This came up as I received an error for google about an invalid datetime I was generating for my sitemap.  I've been using the following to generate my datetime string:
iex> elem(Date.local |> DateFormat.format("{ISO}"), 1)
"2015-11-08T17:44:10.930+0000"

But on closer examination the timezone offset should be colon-separated to be consistent with the extended format (basic + item separators).  According to wikipedia's article on ISO-8601:

Representations can be done in one of two formats – a basic format with a minimal number of separators or an extended format with separators added to enhance human readability.[5] The standard notes that "The basic format should be avoided in plain text."[6] The separator used between date values (year, month, week, and day) is the hyphen, while the colon is used as the separator between time values (hours, minutes, and seconds). For example, the 6th day of the 1st month of the year 2009 may be written as "2009-01-06" in the extended format or simply as "20090106" in the basic format without ambiguity.
https://en.wikipedia.org/wiki/ISO_8601#General_principles

Further:

Either basic or extended formats may be used, but both date and time must use the same format. The date expression may be calendar, week, or ordinal, and must use a complete representation.
https://en.wikipedia.org/wiki/ISO_8601#Combined_date_and_time_representations

I don't know how you want to proceed as this might break backwards compatibility with people currently using the library, but I do believe this is a bug.  A nice improvement beyond this would be an option to pass for generating either the basic or extended format.
I haven't dug too much into the code yet but would be happy to give it a shot and submit a PR w/ tests for this fix if you're interested.

I think this is really a side effect of using {ISO} and {ISOz} as flexible tokens to represent the ISO-8601 spec in it's entirety, when as you've pointed out, the ambiguity causes bugs like this to happen.
I'll fix this bug myself, but if you'd be interested in tackling the addition of four new tokens {ISO:Basic}|{ISO:Basic:Z} and {ISO:Extended}|{ISO:Extended:Z}, I'd be glad to merge that. I can handle making those changes as well if you'd like, but if you're interested in diving in to the code, it's a good place to start :)

@theoretick The bug in question has been fixed in master. I'll leave this open to track the addition of {ISO:Basic} and {ISO:Extended}.

Great, thanks!  That sounds good, I'll take a crack at it when I get some time this week


GitLab VCS Integration

Closes: https://github.com/tramlinehq/site/issues/
Because
Enter the reason for raising this PR...
This addresses
Enter the details of what all this covers...

Just curious, is there any plan to support self-managed instances of Gitlab?

Release with Backmerge + Parallel Working Branch.

Tested with RBM and PWR. It works but there are some glitches.
GitLab does not allow merging right after a PR is created sometimes, it does some async work to do merge checks before allowing merge.
This is somewhat okay for RBM since the finalization of a release can be retried manually. But, for PWR, it can sometimes lead to release kickoff failing as the pre-release PR needs to be merged for the release to start.
I think we can solve it with retries, but it can be done in the second pass as well.
There is one more TODO that I have kept in for the next pass - saving the correct base sha for the PRs from GitLab.


F-droid

Hey, I was wondering if you'd be open to adding this app to https://f-droid.org ?
That would make it accessible to all the people without google app store access. :)

Let's wait first if someone will help maintaining the code. It is opened,
but upgrade for latest owncloud/nextcloud versions needed.
Dne 10. 2. 2017 16:51 napsal uživatel "Marcel Klehr" <
<EMAIL_ADDRESS>
Hey, I was wondering if you'd be open to adding this app to
https://f-droid.org ?
That would make it accessible to all the people without google app store
access. :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Nethar/owncloud-bookmarks/issues/2, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ADurlL4Wz6QjHIAcCFUT6Yuf-O9_l0zEks5rbIeDgaJpZM4L9h3E
.

@marcelklehr I'm part of the f-droid community, so it would be my plan to put it there, but yes before I do that I want to clean, and improve the code. It is a bit messy at the moment, since it's still a webcrawler. So I'm waiting for a proper REST API.

Awesome! :)

So the REST API is there in the bookmarks app, will this app get updated to work with Bookmarks 0.10 at one point in the future?

I'm on it already.


remove resync period

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #597
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

/lgtm
/approve


JSON Schema?

Is your feature request related to a problem?
No
Which solution do you suggest?
Can you please provide a JSON Schema for devspace.yaml? Preferably by contributing it to schemastore.org. This way our IDEs will pick it up automatically and we can get completions and help when authoring this file.
Which alternative solutions exist?
Pillage the docs and create this file by hand.
Additional context
https://json-schema.org/

/kind feature

@mnpenner thanks for opening this issue! Good idea, I will mark this as a feature

@FabianKramm looks like this feature didn't happen.
It would be great for UX if we could do this.
What's your view on this?

Thanks! Was this submitted to SchemaStore.org?


Mount failed after installing openebs on Digitalocean

Description
I installed openebs on Digitalocean with the following steps - aim is to achieve RWX:

One click install Openebs on the Digitalocean marketplace
Installed the provisioner with kubectl apply: https://github.com/openebs/dynamic-nfs-provisioner/blob/develop/deploy/kubectl/openebs-nfs-provisioner.yaml
Created a storageclass as follows:

apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
  name: rwx-storage
  annotations:
    openebs.io/cas-type: nfsrwx
    cas.openebs.io/config: |
      - name: NFSServerType
        value: "kernel"
      - name: BackendStorageClass
        value: "do-block-storage"
provisioner: openebs.io/nfsrwx
reclaimPolicy: Delete

Expected Behavior
When creating a PVC storage, I'd expect the storage to be provisioned through do-block-storage and the NFS server to be running.
Current Behavior
The installation has caused the do-block-storage storage class to stop working for both NFS server (openebs) and without i.e. straight do-block-storage.
Error message
MountVolume.MountDevice failed for volume "pvc-b39a6f9a-4016-4d60-8658-757f4807da3a" : rpc error: code = Internal desc = mounting failed: exit status 255 cmd: 'mount -t ext4 /dev/disk/by-id/scsi-0DO_Volume_pvc-b39a6f9a-4016-4d60-8658-757f4807da3a /var/lib/kubelet/plugins/kubernetes.io/csi/pv/pvc-b39a6f9a-4016-4d60-8658-757f4807da3a/globalmount' output: "mount: mounting /dev/disk/by-id/scsi-0DO_Volume_pvc-b39a6f9a-4016-4d60-8658-757f4807da3a on /var/lib/kubelet/plugins/kubernetes.io/csi/pv/pvc-b39a6f9a-4016-4d60-8658-757f4807da3a/globalmount failed: Invalid argument\n"

I've seen a similar ticket here but with no conclusion? https://github.com/openebs/openebs/issues/3497

This is not do with the NFS provisioner, I tried to install openebs without the NFS provisioner and the same issue happens. The installation of openebs does something that prevents the do-block-storage from mounting.

Digitalocean support managed to replicate this issue and they are busy looking into it.

Digitalocean support managed to replicate this issue and they are busy looking into it.

Any update in this? I have the exact same issue.

Still happening for me as of today

Are you using the nfs provisioner too?

Digitalocean support managed to replicate this issue and they are busy looking into it.

Is there any way we an track DigitalOcean's resolution of this issue? It's been 2 years...

Yes this was using the nfs provisioned too. But even without the nfs provisioner, odd behavior occurs with the default storage class when openebs is in the cluster. Once openebs is removed it fixes itself so it feels like conflict in the way openebs try's to hook into the storage system.


icons or Integration and Streaming landing pages

Updated the overview pages for INtegration and STreaming.

 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.


[RFR][DNM] Support GitHub integration

This PR aims to transform the current Sedy installation (A webhook + a Bot account) to a single GitHub Integration.
Sadly, GitHub integrations are currently in early access and the PR Reviews API too.
We need to wait for Github to fully support these features to do the change.

No ETA for the support. Keep waiting.

Thread about the PR Review API support here: https://platform.github.community/t/api-endpoint-for-pr-reviews/409

Integrations are now in Pre-Release, but the PR Reviews API are still not listed in the integrations available endpoints.

Good news !

Pull Request Reviews API becomes an official part of API v3
https://developer.github.com/changes/2017-05-09-end-black-cat-preview/

I'll try again soon.

Sedy works as a GitHub App!
Back to WIP, I'll finish this PR.

RFR and green!


Typechecking not completing upon build and is preventing intellisense

Describe the bug

Typechecking seems to get stuck upon initial build when opening an f# file and is preventing intellisense. Tried restarting vscode, reinstalling ionide, doublechecking the proj file. Can't seem to find a similar issue like this anywhere. Typechecking just continues to load in the screenshot below
Steps to reproduce

Open Project
Open an .fs or .fsx file
Wait for building to get to typechecking

Screenshots

Machine info

OS: Mac
.NET SDK version: 6.0.16
Ionide version: 7.9.0

Additional context
Add any other context about the problem here.

Experiencing the same, also on Mac. Tried the same workarounds. No luck.

Same here.

I installed an older version (7.5.4) of the extension and it seems to work fine.

I installed an older version (7.5.4) of the extension and it seems to work fine.

Thank you! Didn't think to try that. I've downgraded and seems to be working again. Still haven't got to the bottom of why the most recent version isn't working it but happy with the workaround for now!

I can confirm that this is happening on my machine as well. Win 11 x64, dotnet 6.0.107, and vscode <IP_ADDRESS>.
Has anyone tried to see if this is resolved in 7.12.1?

@pimbrouwers can you also try using a more recent SDK? 7.0.4xx or the 8.0.100-rc1 would be ideal here.

Hey @baronfel!
I updated the referenced sdk in global.json to 7.0.100 and Ionide to 7.12.2.
Happy to report no issues so far. Even with our 35 project monorepo.

Hey, that's great to hear. MS fully supports using more recent SDKs to target older TFMs, so you should feel comfortable doing this - even updating to 8.x!
At the same time, it's mildly concerning that our net6.0 version of FSAC isn't working as expected anymore - we have test coverage but clearly something is going wrong on your system.

@baronfel we do have a few closed issues about the combination of arm macos and net 6.0 having a lot of issues.

Thanks Chet & Jimmy.
@baronfel I usually up the version when a new LTS version is released. But there is no reason I can't update the tooling more frequently.

Updated Ionide extension and a restart and seems to be working again. Not sure if this was fixed or because of the act of updating?

Hey everyone.
So I updated to 8.x rc today and I seem to be getting the same issue.

OS: Windows 11
.NET: 8.0.100-rc.1.23463.5
Ionide: 7.12.5

Seems to be stuck at Typechecking. I tried uninstalling/reinstalling ionide to no avail
Looked at the F# output logs and see:
Unhandled exception. System.AggregateException: One or more errors occurred. (Assembly: netstandard, full path: System.Collections.Generic.IEnumerable1)
---> FSharp.Compiler.DiagnosticsLogger+UnresolvedPathReferenceNoRange: Assembly: netstandard, full path: System.Collections.Generic.IEnumerable1    at FSharp.Compiler.TypedTree.CcuThunk.EnsureDerefable(String[] requiringPath) in D:\a\_work\1\s\src\Compiler\TypedTree\TypedTree.fs:line 5526    at FSharp.Compiler.TypedTree.NonLocalEntityRef.TryDeref(Boolean canError) in D:\a\_work\1\s\src\Compiler\TypedTree\TypedTree.fs:line 3379    at FSharp.Compiler.TypedTree.EntityRef.Resolve(Boolean canError) in D:\a\_work\1\s\src\Compiler\TypedTree\TypedTree.fs:line 3462    at FSharp.Compiler.TypedTree.EntityRef.get_Deref() in D:\a\_work\1\s\src\Compiler\TypedTree\TypedTree.fs:line 3474    at FSharp.Compiler.CodeAnalysis.ItemKeyStoreBuilder.writeType(Boolean isStandalone, TType ty) in D:\a\_work\1\s\src\Compiler\Service\ItemKey.fs:line 235    at FSharp.Compiler.CodeAnalysis.ItemKeyStoreBuilder.writeValRef(ValRef vref) in D:\a\_work\1\s\src\Compiler\Service\ItemKey.fs:line 299    at FSharp.Compiler.CodeAnalysis.ItemKeyStoreBuilder.Write(Range m, Item item) in D:\a\_work\1\s\src\Compiler\Service\ItemKey.fs:line 339    at <StartupCode$FSharp-Compiler-Service>.$IncrementalBuild.clo@343-478.Invoke(CapturedNameResolution cnr) in D:\a\_work\1\s\src\Compiler\Service\IncrementalBuild.fs:line 346    at Microsoft.FSharp.Collections.SeqModule.Iterate[T](FSharpFunc2 action, IEnumerable1 source) in D:\a\_work\1\s\src\FSharp.Core\seq.fs:line 632    at <StartupCode$FSharp-Compiler-Service>.$IncrementalBuild.clo@334-477.Invoke(Tuple5 _arg6) in D:\a_work\1\s\src\Compiler\Service\IncrementalBuild.fs:line 342
at FSharp.Compiler.BuildGraph.Bind@61-1.Invoke(T x) in D:\a_work\1\s\src\Compiler\Facilities\BuildGraph.fs:line 62
at Microsoft.FSharp.Control.AsyncPrimitives.CallThenInvokeNoHijackCheck[a,b](AsyncActivation1 ctxt, b result1, FSharpFunc2 userCode) in D:\a_work\1\s\src\FSharp.Core\async.fs:line 528
at Microsoft.FSharp.Control.Trampoline.Execute(FSharpFunc2 firstAction) in D:\a\_work\1\s\src\FSharp.Core\async.fs:line 112    --- End of inner exception stack trace ---    at Microsoft.FSharp.Control.AsyncPrimitives.Start@1174-1.Invoke(ExceptionDispatchInfo edi) in D:\a\_work\1\s\src\FSharp.Core\async.fs:line 1174    at Microsoft.FSharp.Control.Trampoline.Execute(FSharpFunc2 firstAction) in D:\a_work\1\s\src\FSharp.Core\async.fs:line 112
at <StartupCode$FSharp-Core>.$Async.clo@193-15.Invoke(Object o) in D:\a_work\1\s\src\FSharp.Core\async.fs:line 195
at System.Threading.QueueUserWorkItemCallback.Execute()
at System.Threading.ThreadPoolWorkQueue.Dispatch()
at System.Threading.PortableThreadPool.WorkerThread.WorkerThreadStart()`


Property TPUSpawnPlugin.tpu_global_core_rank may be replaced by global rank property

Proposed refactor
Deprecate tpu_global_core_rank in favor of global_rank.
Pitch
It "looks" like tpu_global_core_rank in the TPUPlugin is just another name for what is known as the global rank. After #10896 we should investigate whether the two are equivalent and revert back to using the unified global_rank property exclusively.

If you enjoy Lightning, check out our other projects! ⚡

Metrics: Machine learning metrics for distributed, scalable PyTorch applications.

Lite: enables pure PyTorch users to scale their existing code on any kind of device while retaining full control over their own loops and optimization logic.

Flash: The fastest way to get a Lightning baseline! A collection of tasks for fast prototyping, baselining, fine-tuning, and solving problems with deep learning.

Bolts: Pretrained SOTA Deep Learning models, callbacks, and more for research and production with PyTorch Lightning and PyTorch.

Lightning Transformers: Flexible interface for high-performance research using SOTA Transformers leveraging Pytorch Lightning, Transformers, and Hydra.

cc @borda @kaushikb11 @rohitgr7 @justusschock @awaelchli @akihironitta @tchaton

Yup, let's go ahead with the deprecation!

Currently blocked by #11330


Google search result summary is strange

We set the appropriate meta tag here I believe, so I don't understand why Google isn't respecting it.
https://github.com/http4s/http4s/blob/cb97abaccd1d7253c6175ca7319b49f8a4ca25d4/index.html#L51

The docs say they might use the description meta tag: https://developers.google.com/search/docs/appearance/snippet

Thanks for that link!

Google sometimes uses  tag content to generate snippets, if we think they give users a more accurate description than can be taken directly from the page content.

So if I'm reading that correctly basically our meta tag is too shitty 😂

I agree with @armanbilge that description is too short. I checked, and it looks like Google is now rendering a new description. Would you like something different still? Id be happy to write something.

Oh, I guess they rotate that huh 😅 still that doesn't really explain what http4s is.
So @katiemarqueling if you want to write something, sure, that would be great! I still need to figure out where exactly it should go, maybe in here.
https://github.com/http4s/http4s/blob/4bcefdd689cce3110b095276ba11187827fa3a87/project/Http4sSitePlugin.scala#L60-L61


Java indent configuration by fileType use JAVA key instead of java

The filetype java is not working anymore with Intellij 2016.2.

I signed it!


Feature Request - Add ability to import exported workspace as a new unique workspace

Currently if you export a workspace and import it into the same installation, it doesn't appear to do anything. My use case is one where I'm working with two different versions of an API and would like to copy over all my requests into a new project. Environments probably take care of this issue too, though, so maybe this use case is too edge to spend time on?

@tthayer currently, if you import a workspace that was exported from the same application, it will overwrite everything that already exists.
A potential solution would be to treat this case like an operating system does and prompt the user with...

Overwrite conflicts
Discard conflicts
Copy all


Add a few telemetry scheduled tasks to disable

		# This task shows various Map related toasts
		"MapsToastTask",

		# Initializes Family Safety monitoring and enforcement
		"FamilySafetyMonitor",

		# Synchronizes the latest settings with the Microsoft family features service
		"FamilySafetyRefreshTask",

		# XblGameSave Standby Task
		"XblGameSaveTask"

Disable-ScheduledTask -TaskName MapsToastTask -TaskPath "\Microsoft\Windows\Maps\"

Disable-ScheduledTask -TaskName FamilySafetyMonitor -TaskPath "\Microsoft\Windows\Shell\"

Disable-ScheduledTask -TaskName FamilySafetyRefreshTask -TaskPath "\Microsoft\Windows\Shell\"

Disable-ScheduledTask -TaskName XblGameSaveTask -TaskPath "\Microsoft\XblGameSave\"

should be able to be done with oo shutup, no?

I was kind of high when I typed that. I meant 'Revo' but mixed it up with 'oo shutup' and I thought you needed help to change it and that you didn't know how to do it manually, so I told you to use Revo Uninstaller. My bad.


Add information about Active Job to README

Since the Active Job support is not documented and it takes precedence
over the other libraries and it's installed as a dependency with later
Rails versions (~> 4.2.1 it seems) some documentation is useful.

@jrgifford maybe take a look? : )

@adamof thanks!


Ability to quickly switch between dashboards

In 1.x Chronograf we had a type-ahead dashboard switching mechanism at the top of every dashboard.
Since we don't have this in 2.0 yet there's an opportunity to offer the same functionality but with a design that better suits our current UI

Since our page headers are a bit more verbose (with the org name) I propose adding the ability to "star" dashboards and have them show up in the nav bar under the dashboards tab


terrascan: 1.2.0 -> 1.3.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/accurics/terrascan/releases.
meta.description for terrascan is: "Detect compliance and security violations across Infrastructure"
meta.homepage for terrascan is: "https://github.com/accurics/terrascan"
meta.changelog for terrascan is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.3.2 with grep in /nix/store/l1r920v31ym1pyhjcp9s7nx9x0h3y30f-terrascan-1.3.2

directory tree listing: https://gist.github.com/77ed5aa6f8b2e2758bf1cfc1bce1e0c8

du listing: https://gist.github.com/0e0b22e0d19f6e226ed4d77edbc13cc0

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
terrascan

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/l1r920v31ym1pyhjcp9s7nx9x0h3y30f-terrascan-1.3.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A terrascan https://github.com/r-ryantm/nixpkgs/archive/e20ac147dda67bc01101369beb5eb2412836e828.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/l1r920v31ym1pyhjcp9s7nx9x0h3y30f-terrascan-1.3.2
ls -la /nix/store/l1r920v31ym1pyhjcp9s7nx9x0h3y30f-terrascan-1.3.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

terrascan

Maintainer pings
cc @fabaff @kalbasit for testing.

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

1 package built:

terrascan


Import hooks multiple handlers

This attempts to fix the issue whereby it was not possible to register multiple distinct import function handlers. All registered XML tags to cache and post-process would have been passed to the same function, even if multiple functions were defined in the settings.
See ticket https://projects.torchbox.com/projects/wordpress-to-wagtail-importer-package/tickets/73

The test have helped me see how to use Mock too.


Proposal: make UserStore.ThrowIfDispose protected

From @sumtec on February 19, 2016 5:51
Hi,
Is it possible to make ThrowIfDispose in the UserStore<> protected? I am trying to make it work for a legacy database and I have to write my own UserStore to enable login. When trying to implementing the UserStore class, I found no way to call the ThrowIfDispose.
I need to query different table to get a wrapped IdentityUser object. So, I can't call the FindByName in the base class directly. Or, any other advice other than implementing my own UserStore class?
Thanks,
Frank
Copied from original issue: aspnet/EntityFramework#4607

472c6f87348c7f8f7a29a22f1fb7fe9cb3b5411e


builder: output to stderr if the cmd fails and reporting is disabled

I run the following command with a script that doesn't exist (asdsda.bash).
builder -build -rev=1a4c984baea846a06e95440b2192eec1bc29917a -v -report=false -cmd="asdsda.bash"  linux-amd64

The output:
# builder -build -rev=1a4c984baea846a06e95440b2192eec1bc29917a -v -report=false -cmd="asdsda.bash"  linux-amd64
2015/01/14 23:11:57 Found old workspace, will use it
2015/01/14 23:11:57 running [git rev-parse 1a4c984baea846a06e95440b2192eec1bc29917a] in /tmp/gobuilder/goroot
2015/01/14 23:11:57 linux-amd64 building 1a4c984baea846a06e95440b2192eec1bc29917a
cd /tmp/gobuilder/goroot
git pull --ff-only
cd .
git clone /tmp/gobuilder/goroot /tmp/gobuilder/linux-amd64-1a4c984baea8/go
cd /tmp/gobuilder/linux-amd64-1a4c984baea8/go
git checkout 1a4c984baea846a06e95440b2192eec1bc29917a
2015/01/14 23:11:59 running [/tmp/gobuilder/linux-amd64-1a4c984baea8/go/src/asdsda.bash] in /tmp/gobuilder/linux-amd64-1a4c984baea8/go/src

https://go-review.googlesource.com/2737


PowerShell fails to launch on Windows ARM64

I did the following:

Installed .NET 5.0 Preview 4 on a Windows AM64 device
Installed the PowerShell global tool
Updated the pwsh.runtimeconfig.json file to reference the MS.NetCore.App framework instead of the Windows Desktop one.
"set DOTNET_ROLL_FORWARD=LatestMajor"

I then see the following:
C:\git\testapps\versioninfo>pwsh
PowerShell 7.0.0
Copyright (c) Microsoft Corporation. All rights reserved.

https://aka.ms/powershell
Type 'help' to get help.

The shell cannot be started. A failure occurred during initialization:
The type initializer for 'System.Management.Automation.LanguagePrimitives' threw an exception.

Updated the pwsh.runtimeconfig.json file to reference the MS.NetCore.App framework instead of the Windows Desktop one.

Why do you change the default? PowerShell 7.0 depends on Windows Desktop.

I’m concerned that this bug report is “I modified powershell’s runtime environment and that made it break.” Isn’t that sorta you-break-it-you-bought-it?
Does the same thing happen if you try to retarget the GA version to .NET 5 on AMD64? Does it work if you don’t retarget it, and use it with .NET Core 3 as it was produced for GA?


Running multiple test should only be done when number of tests > 0

In blitzgsea/init.py in line 374,  multiple testing correction should only be carried out when number of test > 0. Its possible to have number of test = 0 if none of the genes get a match in any of the geneset databases.

The issue was fixed. Thank you for pointing this out.


Test case and fix for #121

This is a simple test case and fix for #121
In my solution, I've taken the simplest possible approach: instead of inventing a new image or docker parameter that goes into options and would need special treatment, I'm simply checking for the presence of options.exec and use what's provided there.
This comes at the risk of having people override the values we infer for our own exec object, but I'd assume people using the exec parameter know what they are doing.

Codecov Report

Merging #122 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #122      +/-   ##
==========================================
+ Coverage   99.21%   99.21%   +<.01%
==========================================
  Files          14       14
  Lines         381      383       +2
==========================================
+ Hits          378      380       +2
  Misses          3        3

Impacted Files
Coverage Δ

lib/actions.js
100% <100%> (ø)
:arrow_up:

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 31d729f...b087b45. Read the comment docs.

@trieloff Thanks for raising this issue and the PR to resolve. I like the approach used here with minimal external API changes needed.

@trieloff the build is failing due to the following mirror issue.
standard: Use JavaScript Standard Style (https://standardjs.com)
standard: Run `standard --fix` to automatically fix some problems.
  /home/travis/build/apache/incubator-openwhisk-client-js/test/unit/actions.test.js:479:1: More than 1 blank line not allowed.
  /home/travis/build/apache/incubator-openwhisk-client-js/test/unit/actions.test.js:495:3: Newline required at end of file but not found.

can you fix the lint error and re-push?

@jthomas I took the liberty of fixing the white space issue, please squash all the commits.

Will do a new release with this in shortly.


[Proposal] Seed for generate_random_map() in FrozenLake?

Proposal
Could be useful to be able to pass a seed to the generate_random_map() function in FrozenLake.
Motivation
Being able to get reproducible generated maps.
Pitch
Being able to call generate_random_map(size=8, seed=123).
Alternatives
No response
Additional context
No response
Checklist

[X] I have checked that there is no similar issue in the repo

This seems like a critical issue, could you make a PR that solves this.
Not sure, if we want the __init__ to contain a map_seed parameter or for the map data to be set in reset and use the seed value


other languages error

When trying to add links with foreign letters (like the lovely ø or æ for intance) the log gives error
addDocumentToNamespace PineconeClient: Error calling upsert: PineconeError: not supported value type Failed to vectorize website
and english articles go through as they should

noted

This has been fixed since the merge of the new and improved document collector!
719521c307d81c430d7d90872768d6de556c38d7


How can I change DUXBetaFPVWidget.fpvView.backgroundColor

I want to change fpvWidget.fpvView.backgroundColor,
but the fpvView is fileprivate.
How can I change the fpvView.backgroundColor?


Bump smoke test timeout to 30 minutes

The 15 minute timeout does not appear to be sufficient anymore, though I'm not sure why. In any case, I'm not sure there's a good reason the timeout needs to be so low, these tests only run once a night and do not take up much time in the agent pool.

/azp run java - smoke-tests

Seems like it's a functional issue, not a timeout issue. Closing for now.


Blue/green strategy for applications with port on 443

Thanks for the reference example. The post provides a Blue/green strategy of using different ports and listeners to control the traffic between blue and green. In enterprises like ours, we use products for authentication and they can work only in certain ports like 443. What could be the blue/green strategy in such cases?
We have an apache container running on 443 linked to a springboot container that is running on 8080. Only port 443 can be used since the tool (Cleartrust) used for authentication require that port.

@geethaRam  did you ever find an answer to this?  Just making guesses, was it to use two load balancers perhaps, or two DNS names with ALB rules?


Add iops and against device flag.

@stfeng , PTAL

Minor comments mostly, thanks for great work!

Tested locally on fio and block_storage_workload.


Alternative serial connexion for serial PMS5003

This is based on the on the code from the learn guide for https://www.adafruit.com/product/3686
The learn guide is here: https://learn.adafruit.com/pm25-air-quality-sensor/python-and-circuitpython
The code can be found here: https://github.com/adafruit/Adafruit_Learning_System_Guides/blob/693f8f250edef32fbd0ed44932bf9fca19277d17/PMS5003_Air_Quality_Sensor/PMS5003_CircuitPython/PMS5003_example.py

Looks good at first glance! Thanks for updating it. I will be testing this in the next couple of days and will get it merged once verified.


Twilio live speech recognition

Hi, I'm trying to implement live speech recognition in a twilio phone call. I iteratively get data in json, where audio data is placed in payload.
{'event': 'media', 'sequenceNumber': '5', 'media': {'track': 'inbound', 'chunk': '4', 'timestamp': '270', 'payload': '9e/x8u3s9Pn19vv9fnl0cHh5cW5vc29yeHRydXp++vf5/P/8+Pb0+/31+Pz18/X3+fXz+fz'}, 'streamSid': 'MZ24b09f59eda9c32fd074ac85a9fdf81d'}
What should I change in this code to be able run recognition on stream, not from a file.
Thanks a lot.

def push_stream_writer(stream):
    # The number of bytes to push per buffer
    n_bytes = 3200
    wav_fh = wave.open("example.wav")
    # start pushing data until all data has been read from the file
    try:
        while True:
            frames = wav_fh.readframes(n_bytes // 2)
            print('read {} bytes'.format(len(frames)))
            if not frames:
                break
            stream.write(frames)
            time.sleep(.1)
    finally:
        wav_fh.close()
        stream.close()  # must be done to signal the end of stream

def speech_recognition_with_push_stream():
    """gives an example how to use a push audio stream to recognize speech from a custom audio
    source"""
    speech_config = speechsdk.SpeechConfig(subscription="", region="")

    # setup the audio stream
    stream = speechsdk.audio.PushAudioInputStream()
    audio_config = speechsdk.audio.AudioConfig(stream=stream)

    # instantiate the speech recognizer with push stream input
    speech_recognizer = speechsdk.SpeechRecognizer(speech_config=speech_config, audio_config=audio_config)
    recognition_done = threading.Event()

    # Connect callbacks to the events fired by the speech recognizer
    def session_stopped_cb(evt):
        """callback that signals to stop continuous recognition upon receiving an event `evt`"""
        print('SESSION STOPPED: {}'.format(evt))
        recognition_done.set()

    speech_recognizer.recognizing.connect(lambda evt: print('RECOGNIZING: {}'.format(evt)))
    speech_recognizer.recognized.connect(lambda evt: print('RECOGNIZED: {}'.format(evt)))
    speech_recognizer.session_started.connect(lambda evt: print('SESSION STARTED: {}'.format(evt)))
    speech_recognizer.session_stopped.connect(session_stopped_cb)
    speech_recognizer.canceled.connect(lambda evt: print('CANCELED {}'.format(evt)))

    # start push stream writer thread
    push_stream_writer_thread = threading.Thread(target=push_stream_writer, args=[stream])
    push_stream_writer_thread.start()

    # start continuous speech recognition
    speech_recognizer.start_continuous_recognition()

    # wait until all input processed
    recognition_done.wait()

    # stop recognition and clean up
    speech_recognizer.stop_continuous_recognition()
    push_stream_writer_thread.join()

@atgorvi Could you confirm that Twilio speech input is in MULAW format? Based on earlier issues reported that seems to be the case.
In case of MULAW stream, you need to create push (or pull) audio stream using following audio configuration.
audio_format = speechsdk.audio.AudioStreamFormat(samples_per_second=8000, bits_per_sample=16,
channels=1, wave_stream_format=speechsdk.AudioStreamWaveFormat.MULAW)
stream = speechsdk.audio.PushAudioInputStream(stream_format=audio_format)
audio_config = speechsdk.audio.AudioConfig(stream=stream)
and then write the bytes from the Twilio input to stream instead of using file input as in your example.
Is your question specifically how to take the audio payload from Twilio JSON blob and convert it to bytes for push stream input?

@atgorvi Please see below an idea level example implementation (not tested at all and created with the help of chat GPT), how you could possible proceed further. Let me know if that helps you. Note, I am not 100% sure how to use Twilio data and where once we get this done, I would appreciate that you will share the working example because we have had multiple customer issues using Twilio. Below it is assumed that audio is base64 encoded and it is decoded before pushing to SDK.
import base64
import threading
import speechsdk

class TwilioAudioProcessor:
    def __init__(self, speech_key, service_region):
        self.speech_key = speech_key
        self.service_region = service_region
        audio_format = speechsdk.audio.AudioStreamFormat(samples_per_second=8000, bits_per_sample=16,
                                                         channels=1, wave_stream_format=speechsdk.AudioStreamWaveFormat.MULAW)
        self.stream = speechsdk.audio.PushAudioInputStream(stream_format=audio_format)
        self.audio_config = speechsdk.audio.AudioConfig(stream=self.stream)
        self.recognition_done = threading.Event()

        self.speech_config = speechsdk.SpeechConfig(subscription=self.speech_key, region=self.service_region)
        self.speech_recognizer = speechsdk.SpeechRecognizer(speech_config=self.speech_config, audio_config=self.audio_config)

        def session_stopped_cb(evt):
            print('SESSION STOPPED: {}'.format(evt))
            self.recognition_done.set()

        def canceled_cb(evt):
            print('CANCELED: {}'.format(evt.reason))
            self.recognition_done.set()

        self.speech_recognizer.session_stopped.connect(session_stopped_cb)
        self.speech_recognizer.canceled.connect(canceled_cb)

        self.recognize_thread = threading.Thread(target=self.recognize_audio)
        self.recognize_thread.start()

    def push_audio(self, audio_data):
        audio_bytes = base64.b64decode(audio_data.get('data', '').encode())
        self.stream.write(audio_bytes)

    def recognize_audio(self):
        self.speech_recognizer.start_continuous_recognition()
        self.recognition_done.wait()
        self.speech_recognizer.stop_continuous_recognition()

    def process_twilio_audio(self, twilio_audio_json):
        audio_data = json.loads(twilio_audio_json)
        self.push_audio(audio_data)

# Example usage
twilio_audio_json_blob = '{"data": "base64_encoded_audio_data_here"}'
speech_key = "YOUR_AZURE_SPEECH_KEY"
service_region = "YOUR_AZURE_SERVICE_REGION"

twilio_audio_processor = TwilioAudioProcessor(speech_key, service_region)

# Change this to be called from your Twilio audio hook every 10ms
twilio_audio_processor.process_twilio_audio(twilio_audio_json_blob)

Thanks a lot, it works. I'll write an article on Medium with a full tutorial and share the link when it's ready.

@atgorvi Great to hear, thank you for writing the article, I am sure there are other customers which would benefit on your project findings,

https://medium.com/@dghulay41/twilio-voice-chat-assistant-using-azure-speech-recognition-and-python-efaf6319b583

@atgorvi Great, thank you! This will be very helpful for other customers using Twilio. I am closing this issue.


Seamless downloading of unencrypted files

Sometimes we will store a mix of encrypted and unencrypted files in the same bucket. This is common as we migrate workflows to use crossing, but there are some legacy applications that use the same bucket. It would be nice if crossing could download objects that were not encrypted, and just pass them through.
A warning that it was downloaded as plaintext would be helpful as well, just in case I think that a file should be encrypted but isn't.

#16


Paypal sandbox not reflecting transactions

Hi @ianfleeton,
Could please help me,
On https://developer.paypal.com/developer/accounts/ I have 2 sandbox accounts:

[business]<EMAIL_ADDRESS>[personal]<EMAIL_ADDRESS>
Then in the sandbox paypal store I login successfully with the buyer's account and click the continue button representing payment. I'm successfully redirected back to my success_url and receive what look like some successful params:
  /orders/success/6 token=SAME_TOKEN_FROM_PAYPAL_AS_THE_ONE_ABOVE&PayerID=THE_PAYER_ID_OF_SANDBOX_BUYER_ACCOUNT

Ok so cool the transaction is finished but when I log into the sandbox paypal site with either the buyer or seller's login details, no transactions are listed. When I'm logged into my main paypal developer account, I also cannot see any transactions.

@SachinDev108 my guess is that the transaction is not complete. Did you call DoExpressCheckoutPayment with the token and payer ID in the success URL as described in https://github.com/ianfleeton/paypal-express/wiki/Instant-Payment?

Thanks @ianfleeton  for reply
I did only below changes
paypal_options = {
no_shipping: true, # if you want to disable shipping information
allow_note: false, # if you want to disable notes
pay_on_paypal: true # if you don't plan on showing your own confirmation step
}
request = Paypal::Express::Request.new(
:username   => SET_YOUR_OWN,
:password   => SET_YOUR_OWN,
:signature  => SET_YOUR_OWN
)
payment_request = Paypal::Payment::Request.new(
:currency_code => :JPY,   # if nil, PayPal use USD as default
:description   => FOO,    # item description
:quantity      => 1,      # item quantity
:amount        => 1.00,   # item value
:custom_fields => {
CARTBORDERCOLOR: "C00000",
LOGOIMG: "https://example.com/logo.png"
}
)
response = request.setup(
payment_request,
YOUR_SUCCESS_CALLBACK_URL,
YOUR_CANCEL_CALLBACK_URL,
paypal_options  # Optional
)
response.redirect_uri
Could you please correct me what I missed

Got your point @ianfleeton and thank you so much :)
I need to right this code in my success path
Is am right ?
response = request.checkout!(
token,
payer_id,
payment_request
)
inspect this attribute for more details
response.payment_info

Yes, that's it. :)


Comments being converted from double-slash to star-slash

Just installed v2.0.0 and noticed that double-slash comments are being converted to star-slash comments in sass/scss files by this extension.
Before:
// My comment

After:
/* My comment*/

This doesn't happen when using the Autoprefixer demo here: https://autoprefixer.github.io/. Perhaps updating the version of autoprefixer used by this extension from 6.5.0 (according the the package.json file) to the latest (7.1.2) will address the issue? If not, is there a setting that can be added to prevent this from happening?
Thanks!

Hello, @jhildenbiddle,
For first time you can install autoprefixer@7 and use autoprefixer.findExternalAutoprefixer from setting.
I'll look into this issue soon.

Should be resolved by 2.1.0.

Thanks for the quick turnaroud, @mrmlnc .
Unfortunately, this does not appear to be resolved.

I have disabled all VSCode extensions except for Autoprefixer
I have upgraded the Autoprefixer extension to 2.1.1
I have tried using an external version of autoprefixer by installing it via NPM and setting autoprefixer.findExternalAutoprefixer to true in VSCode
I have verified that the issue is not related to autoprefixer by comparing the output of the extension and postcss-cli + autoprefixer

SCSS code:
// Top level comment

body {
    display: flex;
    // color: #000;
}

Processed using postcss-cli and autoprefixer:
// Top level comment

body {
    display: -webkit-box;
    display: -ms-flexbox;
    display: flex;
    // color: #000;
}

Processed using VSCode wtih Autoprefixer plugin (2.0.0 & 2.1.1):
/* Top level comment*/

body {
    display: -webkit-box;
    display: -ms-flexbox;
    display: flex;
    /* color: #000;*/
}

Thanks again for taking a look at this!


Question about simulation environment

Dear authors,
Thanks for your excellent work. I'd like to ask are there any referred code or repo that can achieve the real-time mapping of real Go2 in simulation environment?

I wonder if you mean sim-to-sim projects. You can cross-verify the neural network in Webots, Mujoco or Gazebo.

Just use Matplotlib to visualize robot states.


Running OpenSpiel in Windows

Have had a first stab at running OpenSpiel within the Windows Subsystem for Linux (WSL) version 1 on Windows 10, and its been partially successful - 54% of tests passed. Not suggesting this as a long term solution for Windows support, but may hep some in the short term.
First of all installed WSL as per the instructions here: https://docs.microsoft.com/en-us/windows/wsl/install-win10
The highest version of SUSE support was 18.04, so had to upgrade cmake via the following commands:
wget http://www.cmake.org/files/v3.12/cmake-3.12.4.tar.gz
tar -xvzf cmake-3.12.4.tar.gz
cd cmake-3.12.4/
./configure
make
sudo make install
sudo update-alternatives --install /usr/bin/cmake cmake /usr/local/bin/cmake 1 --force
After that, the install instructions seemed to work fine. Looks like the tests failing are the Python ones - anyone got any suggestions on resolving this?
54% tests passed, 43 tests failed out of 94
Total Test time (real) = 130.80 sec
The following tests FAILED:
49 - python_api_test (Failed)
50 - python_playthrough_test (Failed)
51 - python_action_value_vs_best_response_test (Failed)
52 - python_best_response_test (Failed)
53 - python_cfr_test (Failed)
54 - python_deep_cfr_test (Failed)
55 - python_dqn_test (Failed)
56 - python_eva_test (Failed)
57 - python_evaluate_bots_test (Failed)
58 - python_expected_game_score_test (Failed)
59 - python_exploitability_descent_test (Failed)
60 - python_exploitability_test (Failed)
61 - python_fictitious_play_test (Failed)
62 - python_generate_playthrough_test (Failed)
64 - python_rl_losses_test (Failed)
65 - python_lp_solver_test (Failed)
66 - python_mcts_test (Failed)
68 - python_nfsp_test (Failed)
69 - python_outcome_sampling_mccfr_test (Failed)
70 - python_policy_gradient_test (Failed)
71 - python_projected_replicator_dynamics_test (Failed)
72 - python_generalized_psro_test (Failed)
73 - python_rectified_nash_response_test (Failed)
74 - python_random_agent_test (Failed)
75 - python_rcfr_test (Failed)
76 - python_sequence_form_lp_test (Failed)
78 - python_bluechip_bridge_wrapper_test (Failed)
79 - python_uniform_random_test (Failed)
80 - python_alpharank_test (Failed)
81 - python_alpharank_visualizer_test (Failed)
82 - python_dynamics_test (Failed)
83 - python_heuristic_payoff_table_test (Failed)
84 - python_utils_test (Failed)
85 - python_visualization_test (Failed)
86 - python_catch_test (Failed)
87 - python_cliff_walking_test (Failed)
88 - python_data_test (Failed)
89 - python_bot_test (Failed)
90 - python_games_sim_test (Failed)
91 - python_matrix_game_utils_test (Failed)
92 - python_policy_test (Failed)
93 - python_pyspiel_test (Failed)
94 - python_rl_environment_test (Failed)
Errors while running CTest
jfadmin@QJF-SURFACEBOOK:~/open_spiel/build$

Nice! Looks like the C++ tests are passing, but the Python ones all fail.
On Tue, 17 Sep 2019, 14:55 jamesdfrost<EMAIL_ADDRESS>wrote:

Have had a first stab at running OpenSpiel within the Windows Subsystem
for Linux (WSL) version 1 on Windows 10, and its been partially successful

54% of tests passed. Not suggesting this as a long term solution for
Windows support, but may hep some in the short term.

First of all installed WSL as per the instructions here:
https://docs.microsoft.com/en-us/windows/wsl/install-win10
The highest version of SUSE support was 18.04, so had to upgrade cmake via
the following commands:
wget http://www.cmake.org/files/v3.12/cmake-3.12.4.tar.gz
tar -xvzf cmake-3.12.4.tar.gz
cd cmake-3.12.4/
./configure
make
sudo make install
sudo update-alternatives --install /usr/bin/cmake cmake
/usr/local/bin/cmake 1 --force
After that, the install instructions seemed to work fine. Looks like the
tests failing are the Python ones - anyone got any suggestions on resolving
this?
54% tests passed, 43 tests failed out of 94
Total Test time (real) = 130.80 sec
The following tests FAILED:
49 - python_api_test (Failed)
50 - python_playthrough_test (Failed)
51 - python_action_value_vs_best_response_test (Failed)
52 - python_best_response_test (Failed)
53 - python_cfr_test (Failed)
54 - python_deep_cfr_test (Failed)
55 - python_dqn_test (Failed)
56 - python_eva_test (Failed)
57 - python_evaluate_bots_test (Failed)
58 - python_expected_game_score_test (Failed)
59 - python_exploitability_descent_test (Failed)
60 - python_exploitability_test (Failed)
61 - python_fictitious_play_test (Failed)
62 - python_generate_playthrough_test (Failed)
64 - python_rl_losses_test (Failed)
65 - python_lp_solver_test (Failed)
66 - python_mcts_test (Failed)
68 - python_nfsp_test (Failed)
69 - python_outcome_sampling_mccfr_test (Failed)
70 - python_policy_gradient_test (Failed)
71 - python_projected_replicator_dynamics_test (Failed)
72 - python_generalized_psro_test (Failed)
73 - python_rectified_nash_response_test (Failed)
74 - python_random_agent_test (Failed)
75 - python_rcfr_test (Failed)
76 - python_sequence_form_lp_test (Failed)
78 - python_bluechip_bridge_wrapper_test (Failed)
79 - python_uniform_random_test (Failed)
80 - python_alpharank_test (Failed)
81 - python_alpharank_visualizer_test (Failed)
82 - python_dynamics_test (Failed)
83 - python_heuristic_payoff_table_test (Failed)
84 - python_utils_test (Failed)
85 - python_visualization_test (Failed)
86 - python_catch_test (Failed)
87 - python_cliff_walking_test (Failed)
88 - python_data_test (Failed)
89 - python_bot_test (Failed)
90 - python_games_sim_test (Failed)
91 - python_matrix_game_utils_test (Failed)
92 - python_policy_test (Failed)
93 - python_pyspiel_test (Failed)
94 - python_rl_environment_test (Failed)
Errors while running CTest
jfadmin@QJF-SURFACEBOOK:~/open_spiel/build$
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/deepmind/open_spiel/issues/59?email_source=notifications&email_token=AHAF7TC73ZHEXREYBWMC3E3QKDOWHA5CNFSM4IXQKTAKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HL3TQWQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHAF7TFJLM5LTB4YJOUUUBDQKDOWHANCNFSM4IXQKTAA
.

That's correct - I've followed the same process on a clean SUSE 18.04 Azure VM and all of these tests pass. Have also confirmed that Python is installed (3.6.8) and working within the WSL environment.
Suspect a missing dependency - any suggestions on troubleshooting this welcome.

Nice.. consider me partly blown away!!
To build the python extensions you need python3-dev. It's possible that pyspiel (i.e. pybind11 that provides the python API) is failing to build without it. Or it could be that PYTHONPATH is not setup properly once it is.
Or it could be because you don't have pip3 and the python dependencies.
Could you try the manual Python instructions here, export the PYTHONPATH manually in the shell, and then run the python example via python3 python/examples/example --game=tic_tac_toe ?

Another idea: can you run python3 and import pyspiel on the command-line?

Thanks Lanctot - after following the first set of instructions you posted above, and with the Python environment activated: 100% tests passed, 0 tests failed out of 94
So it works under WSL with a bit of tweaking - the main disadvantage with this approach is this currently won't support CUDA for training.
Something doesn't quite make sense to me in what part of this fixed the issue, so I'm going to reset the environment and re-run through the process. Will update findings above.

Wow, @jamesdfrost -- I'm very impressed! And WSL is quite a lot more than I thought.
Would you be willing to put together a windows.md markdown write-up that we can link from the main page that documents the steps?
Again: this is really amazing, thanks for extending the reach of OpenSpiel.

So why no cuda, btw-- is WSL an emulator? Would it not use a CUDA-enabled Tensorflow through WSL?

Lanctot - happy to try and document this properly, do I do a pull request for that or just send you the .md file?
WSL is getting quite impressive these days. Under the hood WSL uses Hyper-V, and currently on Win10 this doesn't allow GPU passthrough (although it does on Win2016). It's fairly frustrating as this also applies to Linux Docker images running under Windows.
WSL2, which should be coming in the September update to Win10 introduces a number of performance improvements, and although GPU passthrough isn't going to be one of them initially there is a hint that a range of "hardware improvements" will be coming in a future release.
Most of the DL experimenting I've done has been Windows based, where I can get TF / Cuda / Python working in a pure Microsoft environment under Anaconda, but I suspect getting OpenSpiel running in native Windows is going to be difficult.

Yes, a PR would be great (you can put it in the docs directory, and we'll link it either from the main page or installation doc). That way others can take a look too, and we can comment on it if necessary.
Originally I thought it would be difficult to get native support as well, but you're turning me into a believer. The C++11 is failry simple (another benefit of the first "Keep it Simple" design philosophy!) so I am now thinking it might not actually be too hard to compile on Windows compilers.
I thought the biggest obstacle might be pybind11, but I just checked and it works under Visual Studio 2015 or newer. So I imagine it might be possible!

Have an MD written, but on running through it I'm getting a fail with one test -
50 -python_playthrough_test (Failed)
I've run through the process twice and still get this. This test was definitely working yesterday, has something else changed that would cause this?

@jamesdfrost, that was my fault -- apologies. If you pull and merge, that should be fixed.

I take it back; might not have been my fault. :) But yes there were changes, but none should have affected you. Hmm, maybe still try a pull though, if you are on a branch could be why?

Have re-run the whole process and all good now. I was starting to worry about the stability of WSL, but I think it's all good. Will try and do the PR tonight.

Most of the DL experimenting I've done has been Windows based, where I can get TF / Cuda / Python working in a pure Microsoft environment under Anaconda, but I suspect getting OpenSpiel running in native Windows is going to be difficult.

Actually a while ago I did get as far as you did in your initial comment here -- all C++ tests succeeding, all python tests failing -- in native Windows.
That did involve quite a bit more trickery already than your solution does though... including a couple of small changes in OpenSpiel's C++ code, and forcing Visual Studio to ignore an unresolved-external-symbol-error and continue building anyway. And then there was still the issue where python bindings didn't seem to work, and at that time I decided it was easier for me to just reboot to Linux than it was to try to figure out how to make that work.
But if you also got that last part to actually work... I guess I might give it another try again when I get bored some time, on native Windows.

I have imported your doc, @jamesdfrost , should go in at the next update. I will leave this issue open because eventually it will be great if we have full/native support, and people can use it to coordinate efforts.

Closing this as it's no longer an issue. We still want better Windows support, but people will be able to find it this thread and I'd like to keep the Issues open for actual issues.


Add socok8s package repo in Airship deployer

Add socok8s package repo on the Airship deployer node. Refactored the
repo list from openstack mechanism to common-vars so we can start
sharing the data between different deploy mechanisms.

Why do we have this PR twice? https://github.com/SUSE-Cloud/socok8s/pull/408


rclone config should be provided as a Secret

An rclone config file contains data that should be kept secret, such as API keys for storage backends. Currently, the supported way to set the rclone config file contents is to put these contents into the values for a release. The contents will then be stored in a ConfigMap, which is not a secure place to put secrets.
It's currently possible to override this behaviour with a safer approach, if you set up the volume mounts using extraVolumes and create the secret manually. Here's what I'm using:
    extraVolumes:
      - volumeMounts:
        - name: real-rclone-config
          mountPath: /config/rclone
        volumes
        - name: real-rclone-config
          secret:
            secretName: my-rclone-config-secret

However, it's probably not a good idea for the chart/documentation to be encouraging users to use an unsafe configuration. The ConfigMap-based approach should be deprecated or transparently replaced with a Secret-based approach. This chart should also allow users to specify an existing secret, similarly to how the RCON password is handled; this is especially useful for GitOps-style workflows (e.g. Flux) where secrets have to be created separately from Helm release inputs.

Excellent point @wlritchi. It would be great if you (or @merkinbob perhaps?) could contribute this change. The backup sidecar support is new enough that a breaking change from configmap to secret seems acceptable, if that helps.

sure.  Can someone post me a password redacted rclone file for testing.
When I setup the mcbackup I knew nothing about rclone, as I use a Cephfs mount for my backup drive, and tar is good enough for my purposes.

you can assign this to me

Here's an rclone config file with one backend/bucket named b2-wlr configured (with fake credentials). Ideally, this would just be stored as an opaque secret, probably under the rclone.conf key.
[b2-wlr]
type = b2
account = e0777be97401fd2d39cc
key = K001aW7bs4b9KlP3P0N7LvMJ4Q

Also, a correction about my workaround above. While it was working briefly for me, k8s is now rejecting it due to the duplicate mount point. I would have expected it to fail anyway, and I'm not sure what changed. The upshot is that binding the ConfigMap volume is mutually exclusive with binding the Secret, so we'll either need to remove the ConfigMap altogether (my preferred route) or add some conditional logic to the template to prevent trying to bind both.

sometimes at work I use ENV substitution for secrets mounted via config maps, something like this
[b2-wlr]
type = b2
account = $ACCESS_KEY
key = $SECRET_KEY

then you can add extraEnv to mount a secret to the ENVs.
not sure if rclone has the ability to substitute ENVs from the configs

Config files are going to be pretty variable I think; rclone supports a lot of backends and they look pretty different. Here's an example from their docs:
[remote]
type = google cloud storage
client_id =
client_secret =
token = {"AccessToken":"xxxx.xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx-xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx","RefreshToken":"x/xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx_xxxxxxxxx","Expiry":"2014-07-17T20:49:14.929208288+01:00","Extra":null}
project_number = 12345678
object_acl = private
bucket_acl = private

I don't think it's worth trying to template it; anyone who's using rclone already has a working config file and the Kubernetes documentation has a decent article on creating Secrets from a file. It should be as easy as kubectl create secret generic my-rclone-config --from-file=~/.config/rclone/rclone.conf.

I did debate between myself and I between configmap and secret when I setup the rclone, I didn't read much about rclone except it had a config file.  I can change it to a secret no problem.

I will keep the ability to inject the config file via the values files, but I will also add an exisitingSecret so that we can support the kubectl from-file option, which is something I have done a number of times.


fix(package): add babel-runtime for commonjs builds

Fixes #1260
This PR adds babel-runtime as a package dep.
I've opted for a regular dependency over a peer dependency here as I don't want users to have to install more modules to get up and running.  Also, our package only requires the necessary modules from runtime, so it is cherry picked, similar to how we handle lodash.

Codecov Report

Merging #1306 into master will not change coverage.

@@           Coverage Diff           @@
##           master    #1306   +/-   ##
=======================================
  Coverage   99.74%   99.74%
=======================================
  Files         140      140
  Lines        2394     2394
=======================================
  Hits         2388     2388
  Misses          6        6

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 070178e...9bcf7f0. Read the comment docs.

Released in<EMAIL_ADDRESS>


Consider passing an env with the current execution context to start/set_xxx methods

Sometimes an operation will need to do different things depending on which execution context the prior operation completes on. However there isn't currently an easy way to determine which context you completed on.
The get_completion_scheduler() query only lets you report a single scheduler which doesn't help if an operation could potentially complete on one of several different schedulers. e.g. when_all() could complete on any of the contexts of the input senders.
One avenue that may be worth exploring is having execution contexts explicitly pass information about the current context to start() and also to set_value, etc. methods. This way these operations can know on what context they are starting/completing by querying the environment for the scheduler.
The beginnings of a prototype exploring this idea can be found here:
https://godbolt.org/z/537dcerjo
There are a number of outstanding problems a potential design here would need to solve, however.

how to pass the context through stop-requests
how to pass through type-erasing layers - may require including the completion-context environment types in the completion-signatures.

I think this is too big a change to be making at this stage, even if we had a mature design ready and proven out (which we don't).
Closing as wont-fix.


Bump version to 1.0.2

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
17

Relevant Lines:
17

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
17

Relevant Lines:
17

💛  - Coveralls


Consider breaking azure-core down to smaller components

Look into how we can decompose azure-core into smaller components like moving all HTTP-related types into azure-core-http.

@srnagar Can you write a proposal on a possible new structure, so that we can more concretely discuss the pro's and con's of this change?


method_7672 of class_1735 has protected access but is accessed in a public way by class_1703

line 444 (procyon) of class_1703 net/minecraft/screen/ScreenHandler calls protected method method_7672 onTake of class_1735 net/minecraft/screen/slot/Slot. ScreenHandler doesnt have access to the method (onTake) as it it protected. the method needs to be updated to be public.

See https://github.com/FabricMC/yarn/issues/2057#issuecomment-808949368
The yarn mappings intentionally ignores a few of such access issues (for the better organization of classes into packages). If you are running in dev, these are always patched public by the fabric loader fyi. In intermediary and usual installed games, they are just organized similar to in obf, so the issues are not there either.
The list of all such clashes are still reported for referential purposes. See: https://github.com/FabricMC/yarn/runs/3219557940#step:6:197


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Links to websites not working

Hi there, first of all thank you for developing such a great bit of kit.
I'm trying to adapt the interactive business card example, however I cannot get the links to open either on your live demo or on my own project. I'm using an iPhone 7 if that's any insight.
I found some suggestions to move the icons/buttons away from the middle of the screen so I put them under the portfolio arrows, still not avail.
Any help or thoughts would be greatly appreciated.
Cheers,
M

I realise now that the icons update the bar at the bottom. My bad, however, do you have any reccomendations for where to look regarding turning them into 'open link in new tab' events

You can try window.open(XXX, "_blank")

Thanks a lot this worked


Add the anchor to the url link when available.

Remember, this is only a light fork following upstream and add assembly signing. So you should send your pr upstream too.

Remember, this is only a light fork following upstream and add assembly signing. So you should send your pr upstream too.

here is the current official repo

here is the current official repo

Project is no longer maintained


Feature request: Ring bell when the countdown finishes

Hello IDK if that is possible for a plugin in logseq to use sound. But it would be great if the pomodoro plugin could ring a bell everytime the countdown finishes, so when the user is multitasking in another app, they can be notified by the end of the pomodoro, or rest time.
Thanks a lot for this plugin, I use it everyday.

This plugin already uses the default notifications from your OS.


[DateRangePicker] cannot select a date after tab focus.

Finding: after tab into the picker, the control considers neither start nor end as focused element, which is obviously wrong.


使用security.oauth2.resource.user-info-uri时无法校验resourceId

正常逻辑每次请求会在org.springframework.security.oauth2.provider.authentication.OAuth2AuthenticationManager#authenticate 校验resourceIds，
不包含时抛出throw new OAuth2AccessDeniedException("Invalid token does not contain resource id (" + resourceId + ")");
但是如果配置的是security.oauth2.resource.user-info-uri默认会使用org.springframework.boot.autoconfigure.security.oauth2.resource.UserInfoTokenServices
这个类里面resourceIds一直是null，这样导致资源服务器就算设置了resourceId也不会去校验

@synchronizeds would you be able to explain your problem in English? Or, could you post to GitHub a minimal sample with a test that demonstrates the issue you are experiencing?

The normal logic will verify the resourceids at org.springframework.security.oauth2.provider.authentication.oauth2authenticationmanager every time it requests,
Throw throw new oauth2accessdeniedexception ("invalid token does not contain resource ID (" + resourceid + ")");
But if security.oauth2.resource.user-info-uri is configured, org.springframework.boot.autoconfigure.security.oauth2.resource.userinfotoken services will be used by default
The resourceids in this class are always null, which causes the resource server not to verify even if the resourceid is set

Thanks for the report, @5u5u. I think it makes sense to add a UserInfoTokenServices#setResourceId method.
Would you be interested in submitting a PR?

Due to the lack of feedback on this ticket, I'm going to close it. Feel free to reopen if you feel like there is more to discuss.


Eventsub websocket update

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Problem/Feature
These changes rewrite the EventSub WebSocket mock server implementation.
Description of Changes:

Brand new EventSub WebSocket implementation

Checklist

[X] My code follows the Contribution Guide
[X] I have self-reviewed the changes being requested
[X] I have made comments on pieces of code that may be difficult to understand for other editors
[X] I have updated the documentation (if applicable)

Solves #212


Added file type to image info

Fixes
Missing file type in image info panel.
Fixes #[895] by @[DantrazTrev]
Description
Added file type in image info panel.
Technical details
The image object has no type property, a computed fileType was added and it's based on the file extension of the URL.
Tests

Screenshots

Checklist

[ x] My pull request has a descriptive title (not a vague title like Update index.md).
[ ] My pull request targets the master branch of the repository.
[ x] My commit messages follow best practices.
[ x] My code follows the established code style of the repository.
[ ] I added tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@dos077 I left comments on the ticket (https://github.com/creativecommons/cccatalog-frontend/issues/895) regarding conflicts with a new design that @brenoferreira is currently implementing.


Remember Rooms in Redis

We are trying to figure out how to get Lita to remember the rooms it has joined after a reboot. I'm thinking this may be pretty helpful for other people as well.
We were thinking Lita would have a list of rooms in redis so that when lita restarted it could look at that list and redis and rejoin the rooms it was in before it got shut down. On the join and part commands this list would be updated so the redis list stays up to date.
Does this sound like a helpful feature? I could try and do this right and open a PR if other people think this is helpful.

This has been requested a few times, so I think it's a feature that needs to be introduced in one way or another. The open question for me is what will become of the configuration in lita_config.rb that currently handles this. Will it become a no-op? Will it populate this list in Redis initially but have no effect if the list already exists? I don't want to confuse users with multiple ways to do the same thing that are subtly different.
Imagine the situation where you have rooms A, B, and C in the config file. Then you launch Lita, and send it the part command to leave room C. This is then persisted to Redis. What happens the next time you start Lita? The config file says it should join room C, but you've asked it not to at runtime since then. What is the source of truth?

I'm honestly not sure about how we should handle that. I can describe what my first stab at it looked like and how it works, then we can discuss how it should work.
module Lita
  # A namespace to hold all subclasses of {Handler}.
  module Handlers
    # Allows administrators to make Lita join and part from rooms.
    # @since 3.0.0
    class Room < Handler

      on :connected, :join_rooms

      route(/^join\s+(.+)$/i, :join, command: true, help: {
        t("help.join_key") => t("help.join_value")
      })

      route(/^part\s+(.+)$/i, :part, command: true, help: {
        t("help.part_key") => t("help.part_value")
      })

      # Joins the room with the specified ID.
      # @param response [Lita::Response] The response object.
      # @return [void]
      def join(response)
        Lita.redis.sadd('room_list', response.args[0])
        robot.join(response.args[0])
      end

      # Parts from the room with the specified ID.
      # @param response [Lita::Response] The response object.
      # @return [void]
      def part(response)
        Lita.redis.srem('room_list', response.args[0])
        robot.part(response.args[0])
      end

      def join_rooms(_payload)
        rooms = Lita.redis.smembers('room_list')
        rooms.each do |room|
          begin
            robot.join(room)
          rescue
          end
        end
      end

    end
  end
end

This was my first implementation. So with this implementation it would load all the rooms in the config and then all the rooms that lita was connected to when it was shut down. However in my implementation we don't use the list in lita_config because we invite the bot to a room once and then it will stay there over restarts. So we can worry about how that list will work but I suspect once we implement this most people will just invite lita into whatever rooms they want and then stop using the list in lita_config.

I think you're right that there isn't a reason to have this in the config file once it's being managed with runtime commands, but that's kind of backwards incompatible. The other tricky thing is that these configuration attributes are actually handled by each adapter, not Lita itself, so there isn't an easy way to deprecate it and provide an upgrade path in one fell swoop.

Agreed, I don't really know a great way to migrate forward. I wouldn't be upset if whenever Lita reboots we do an or with the list in redis and the list in the lita_config. I know that would work for my use case but obviously I can't speak for anyone.

What I'm probably going to do is add this as a core feature in the near future, and provide it as an official way to do what the existing adapters have each been implementing themselves. Then each adapter is free to use it or not, with the intent being that they probably should, and provide their own upgrade paths for doing this via the config file.

Sounds good, let me know if I can help.


Add fmod to pkgs.fmod

Issue description
When writting a derivation that requires fmod as a dependecy you can't just add fmod to the derivations inputs on the first line. That's becuase who ever added fmod to nixpkgs didn't do it correctly.
Here is the fmod derivation that was added:
https://github.com/NixOS/nixpkgs/blob/1cb3831f1c64ad9b64db1565572b9c737bae0707/pkgs/games/zandronum/fmod.nix
However, this person didn't add fmod to pkgs.fmod, and instead does an explicit callPackage to add to scope it like this:
https://github.com/NixOS/nixpkgs/blob/1cb3831f1c64ad9b64db1565572b9c737bae0707/pkgs/games/zandronum/default.nix#L8
In my opinion this is an oversight that should be fixed up.

I think we can close this.  This seems to have been. Deliberate choice to remove it from the public package set.


Removed commented File.Delete line

which prevented DLLs from being cleaned on disk in generative type
providers. Fixes #97

Thanks!


Start Contributing: Add best practices to 'Review docs pull requests' section

Add best practices to Review docs pull requests section. This is based on a Slack thread with @sftim and @zacharysarah
Note: it looks like my markdown editor (Atom), cleaned up unnecessary whitespace in other sections of the document when I saved.
Preview: https://deploy-preview-16770--kubernetes-io-master-staging.netlify.com/docs/contribute/start/#review-docs-pull-requests
Signed-off-by: Aimee Ukasick<EMAIL_ADDRESS>

/approve


possibility to export account media accounts

https://github.com/yii2tech/csv-grid
https://github.com/kartik-v/yii2-export


functional dark mode switch and non overlapping google translate and …

##Fixes #961
Overlapping of the Dark Mode Switch and the google translate
I resolved the problem of overlapping between the Google Translate widget and the dark mode switch by relocating the Google Translate widget to the footer of the webpage.
I ensured that the dark mode switch functionality now works seamlessly, even for users who haven't logged in or signed up to the webpage.
I made improvements to the layout by fixing the position of the dark mode switch and adjusting the positioning of the sun and moon images within the button.
I also fixed the size of the navigation bar in the dark mode as initially it was overlapping the text below it.

These changes enhance the user experience and improve the overall usability of your website's homepage

Sir @anuragverma108 please review my pull request for #961

please recheck . i have removed the merge conflicts @anuragverma108

Conflicting files
assets/css/style.css
Solve the conflicitng issues & raise PR again


Is there any functions to support DIVISION?

I only see these functions:
evaluator.add()
evaluator.sub()
evaluator.multiply()
……

Is there any functions to support DIVISION?

If the elements are on a field, division is also treated as multiplication?
On Mon, Nov 11, 2019, 19:52 Yuanhang Luo<EMAIL_ADDRESS>wrote:

I only see these functions:
evaluator.add()
evaluator.sub()
evaluator.multiply()
……
Is there any functions to support DIVISION?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/SEAL/issues/74?email_source=notifications&email_token=ABAV52BVOHAAZ2PSBUXLS63QTID6PA5CNFSM4JL5CXU2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HYSEBOQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABAV52H2YREU3E33KBZ6CJTQTID6PANCNFSM4JL5CXUQ
.

It depends on what you want to do. There is no real way to compute encrypted-encrypted division; for example, you can't know if the encrypted denominator is zero or not. If you know the plaintext then this is easy: just multiply_plain with the inverse. It's also possible to do approximate division in CKKS, but you'll have to assume that the inputs are in a specific range.

@kimlaine I see. Thx.


MUI components lose default styling inside an Iframe.

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
MUI components don't have default styling inside an iframe.
Expected behavior 🤔
MUI components look the same inside an iframe as they look outside an iframe
Steps to reproduce 🕹
codesandbox
Context 🔦
I am trying to utilize mui components inside and iframe using a React portal to achieve DOM isolation from the rest of the application.
Your environment 🌎
System:
OS: Linux 5.11 Ubuntu 20.04.3 LTS (Focal Fossa)
Binaries:
Node: 16.13.1 - /usr/bin/node
Yarn: 1.22.15 - /usr/bin/yarn
npm: 8.3.0 - /usr/bin/npm
Browsers:
Chrome: 96.0.4664.110
Firefox: 95.0.1
npmPackages:
@emotion/react: ^11.7.1 => 11.7.1
@emotion/styled: ^11.6.0 => 11.6.0
@mui/base:  5.0.0-alpha.60
@mui/material: ^5.2.4 => 5.2.4
@mui/private-theming:  5.2.3
@mui/styled-engine:  5.2.4
@mui/system:  5.2.4
@mui/types:  7.1.0
@mui/utils:  5.2.3
@types/react: 17.0.37 => 17.0.37
react: 17.0.2 => 17.0.2
react-dom: 17.0.2 => 17.0.2
typescript: ^4.5.2 => 4.5.2

Currently, I believe that MUI does not provide out-of-the-box support for iframe. I think it is worth having a section explaining how to use MUI inside an iframe.

I would really appreciate it if you could point me to anything that helps me in getting MUI v5 working inside iframes or shadow DOM in the meanwhile.

You can take a look at our docs implementation. https://github.com/mui-org/material-ui/blob/f80abbb5f942064c9c270b2ac5bb2a2318876754/docs/src/modules/components/DemoSandboxed.js?_pjax=%23js-repo-pjax-container%2C div[itemtype%3D"http%3A%2F%2Fschema.org%2FSoftwareSourceCode"] main%2C [data-pjax-container]#L18
Note: neglect the jss if you are using MUI v5

Note: neglect the jss if you are using MUI v5

Do you mean in the DemoSandboxed function we can omit the StylesProvider and ThemeProvider? Thank you very much in advance. I'm looking everywhere for this and couldn't find any precise answer/documentation on how to use MUI v5 with iframe

Hi, has anyone managed to set up this on v5? I need some guidance on this, please 🙏

Hey, I got this working using the emotion cacheProvider. Like:
const muiCache = createCache({
    key: 'mui',
    container,
  });

  export function Provider(props: PropsWithChildren) {
    return (
      <CacheProvider value={muiCache}>
        <ThemeProvider theme={theme} {...props} />
      </CacheProvider>
    );
  };

Where container, is the parent node when emotion will inject the styles. In my case, inside a iframe.head

@erickjth  CacheProvider managed to import only certain default styles. Any idea of how to import styles that were provided individually to MuiComponents by classes and classNames through css modules.  ?


More rules out of stub author's control

Ruff 0.8.0 ignores ambiguous-variable-name (E741) automatically in pyi files. This prompted me to look at other rules I disable specifically for .pyi files and found the following which I think should apply in all cases:

PLR0904 https://docs.astral.sh/ruff/rules/too-many-public-methods/
PLR0913 https://docs.astral.sh/ruff/rules/too-many-arguments/
PLR0917 https://docs.astral.sh/ruff/rules/too-many-positional-arguments/
PLW3201 https://docs.astral.sh/ruff/rules/bad-dunder-method-name/
SLOT https://docs.astral.sh/ruff/rules/#flake8-slots-slot
FBT https://docs.astral.sh/ruff/rules/#flake8-boolean-trap-fbt
Possibly some of A https://docs.astral.sh/ruff/rules/#flake8-builtins-a
Maybe PLC2701 https://docs.astral.sh/ruff/rules/import-private-name/

I haven't gone back and validated if these still trigger in stub files since I added them to our shared config, sorry if I tagged a rule that's already ignored.

Thanks! Your analysis all seems correct to me. I'd happily accept PRs disapplying these rules to stub files.
For E741 we were quite cautious, and made the change preview-only for several minor releases before stabilising the change. I don't think we need to be as cautious this time around. Our versioning policy states that it's fine to apply changes in patch releases if they reduce scope (and these would be reductions in scope); we only need to wait for a minor release if a change increases the scope of a rule.

The linked PR is for all but the last two bullets since those were suggested with less oomph 😄

I leave that to @Samuel-Therrien-Beslogic to decide - as I said, I did not do anything regarding the last two bullet points. So it depends on whether there is interest in continuing to discuss those items.

Thanks @dylwil3 !
For PLC2701, I think that some users may still want a warning about using private names that are subject to change and suddenly break a stub, as per:

Further, as private imports are not considered part of the public API, they are prone to unexpected changes, especially outside of semantic versioning.
Instead, consider using the public API of the module.

But also

This rule ignores private name imports that are exclusively used in type annotations. Ideally, types would be public; however, this is not always possible when using third-party libraries.

(that last sentence is why I've been ignoring it, since more often then not there's no alternatives other than re-writing Aliases, TypeVars and Protocols)
So I'm not 100% convinced either way whether this should be disabled for everyone.

flake8-builtins (A)
The following I'm pretty sure should be disabled in stubs:

A002 https://docs.astral.sh/ruff/rules/builtin-argument-shadowing/
A003 https://docs.astral.sh/ruff/rules/builtin-attribute-shadowing/
A005 https://docs.astral.sh/ruff/rules/builtin-module-shadowing/

As for the rest:

A001 https://docs.astral.sh/ruff/rules/builtin-variable-shadowing/
This could happen with aliases and other top-level variables. non-runtime type aliases should be private anyway. devs shouldn't try to import builtins from a module that happens to import said builtin anyway. And finally this rule mostly tries to protect against using the shadowed variable. In a typing-only context, you should mostly be guarded by static tests anyway.
A004 https://docs.astral.sh/ruff/rules/builtin-import-shadowing/
Very similar as A001 but can actually be worked around with import aliases and re-export aliases.
A006 https://docs.astral.sh/ruff/rules/builtin-lambda-argument-shadowing/
I don't think lambdas apply to type stubs anyway. Probably doesn't need to be ignored.

Just wanted to chime in and say that most if not all of the pep8-naming (N) rules should probably not be applied to stub files either.

I did an experiment where I enabled nearly all rules in typeshed, and here's what I found typeshed would disable in PYI files (that isn't just an opinionated matter): https://github.com/Avasam/typeshed/pull/39/files#diff-50c86b7ed8ac2cf95bd48334961bf0530cdc77b5a56f852c5c61b89d735fd711

pep8-naming (N): As @harahu just mentioned, most pep8-naming rules don't apply for third-party stubs

But relevant rules could be made to apply to names decorated with @type_check_only, since that's a clear signal the stub author is very likely in control of that name.
N811, N812, N813, N814, N817 are probably fine in stubs, as long as the name isn't re-exported (in __all__, or var = var)

A001, A002, A004 and PLC2701 as previously mentioned in https://github.com/astral-sh/ruff/issues/14535#issuecomment-2496160152
PIE796 non-unique-enums: I already raised https://github.com/astral-sh/ruff/issues/15132 for different reasons
FURB189 subclass-builtin: Stub author should follow runtime superclass when possible
PLW1641 eq-without-hash: If runtime doesn't implement __hash__, stubs shouldn't show it as such
flake8-bandit (S)

S105, S106, S107: Lots of false-positive on regex literals assigned to Final. I don't feel like this applies to stubs anyway. Debatable.
S4: no need to warn about importing insecure and vulnerable libraries in stubs. Those are not runtime files. Nothing gets executed.

As for opinionated rules: RUF022, RUF023, F403, F405 (caught by type-checkers), FURB180 (often breaks mypy/stubtest) are disabled for reasons specific to typeshed and can be kept out of this conversation.


Migration infra

This PR contains migration infra containing logger file.
Output tables for sapHana, sapNetweaver and unsupported provider type
Code is added for converting ARM id of AMSv1 to extract subscriptionId, resource grp and resource name
Code added to fetch secrets from keyVault
Customer is need to input ARM id of AMSv1 and providerType they wish to migrate. If providerType flag is not provided then automatically all supported providers are migrated
This PR has entry point from Migration.ps1
Some other parameters are added for testing. We can remove that later
The PR also contains a sample MigrationLog file

Suhani consider restructuring the code a bit more to make the directory structure a bit more cleaner.

Suhani consider restructuring the code a bit more to make the directory structure a bit more cleaner.

Yes agreed. I've restructured the code. Let me know if any further modularization required


Specify that imported crates should build for HOST

This is specifically for "build tools" - basically, if you're cross-compiling, you still want your build tools to build for the host architecture. Similarly to cargo build scripts, this will probably mean just not using the --target flag, which generally gives the correct result for the host architecture - should be the same as the toolchain architecture.
Considerations: Potentially not allowing you to call corrosion_link_libraries on these imported crates? Or only for libraries that aren't CMake targets, perhaps? I don't know, need to consider this more.

Corrosion supports marking a target as ´hostbuild´, so I think this issue is already resolved.


Fix extension validator to handle filenames with extra periods

Sorry. I need to test more before submitting this pull request.


Strongly Type Configuration with GetSection not working

Hi all,
Is anyone have a problem like me when configure the option using GetSection like the documentation show?
ex (try to implement the sample code but return error):
services.Configure<MySubOptions>(Configuration.GetSection("subsection"));
It's work when I configure the option 1 by 1
services.Configure<MyOptions>(myOptions =>
    {
        myOptions.Option1 = "value1_from_action";
    });

I was having the exact same problem and scratching my head for a while.
You'll need to add a dependency to your project.json file for the ConfigurationExtensions
"Microsoft.Extensions.Options.ConfigurationExtensions": "1.0.0"

@BurtonR Microsoft.Extensions.Options.ConfigurationExtensions is not support with full .net framework

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


Implement new JsonValueParser to parse JSON into org.embulk.spi.json.JsonValue

This new JsonValueParser for JsonValue will eventually replace the existing JsonParser for MessagePack Value. (Every plugin has to replace it by themselves.)
JsonValueParser will add support for "capturing" multiple JsonValues at once by CapturingJsonPointerList added in #25 and #26. However, this pull request is just simply reading value-by-value.

Thanks!


sqlite: 3.30.1 -> 3.31.0 (security)

Motivation for this change
Related to #77944
Related to #77271
Related to #76033
Related to #76032
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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 nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

@disassembler @worldofpeace we may want this in 20.03, but there are always some regressions when updating sqlite.


Changed the content of line #664

Poor usage of capitalization. Try again.

Also, it's just a poor heading overall.  Read what other hospitals have on their site.  Do not pull stuff out of the top of your head.
If you are changing English > English, then you must be very picky on what you change it to & think it through well.


refactor(coral): Use API_PATHS map instead of hardcoded paths in domain

About this change - What it does
In domain, we write functions to interact with the Klaw API in coral. The way we used to write such functions  was like this:
api
  .get<KlawApiResponse<"getAclRequests">>(
    `/getAclRequests?${new URLSearchParams(filteredParams)}`
  )

api
  .post<KlawApiResponse<"deleteRequest">, DeleteAclRequestPayload>(
    `/request/delete`,
    { requestEntityType: "ACL", reqIds }
  );

This relied entirely on typing the correct string as the path, which is uncomfortable and allows for easily avoidable errors.
This PR introduces an API_PATHS map, allowing to write such functions in this way, which is more ergonomic and more secure:
api
  .get<KlawApiResponse<"getAclRequests">>(
     // no hardcoded path, autocomplete available
     API_PATHS.getAclRequests,
     // no need for string interpolation of params, passed as second argument
     new URLSearchParams(filteredParams).
  )

api
  .post<KlawApiResponse<"deleteRequest">, DeleteAclRequestPayload>(
    // no hardcoded path, autocomplete available
    API_PATHS.deleteRequest,
    // no change
    { requestEntityType: "ACL", reqIds }.
  );

Resolves: #743

// no need for string interpolation of params, passed as second argument

What do you mean by that? 🤔
(We still need to transform e.g. a number into a string (SADLY) because of the way URLSearchParams is typed but I'm not sure if you're referring to that)

What do you mean by that? 🤔

@programmiri I meant that the string interpolation only happens once, here: https://github.com/aiven/klaw/pull/1036/files#diff-62c7c1817a5cd349a58c932b705a9bbf578cd02f0b1102fe3aa3ec72f6024d25R469
Instead of every time we add one of these functions ^^


Empty target_site_features.csv file

Hello,
I've been running the test data set for DeepEdit, when I completed the 1.Reads_extract.sh script. I realized that my target_site_reads.txt is different than the file located here in GitHub (DeepEdit/Getting_Started/target_site_reads.txt)
Here is my code:

Here is my output:

And here is the file posted in Github:

Even though, both look different I went ahead and run 2.feature_extract.py by using this code:

And when it finished, the code generated an empty target_site_features.csv file.
Please, I was wondering if you could help me to troubleshoot it, I want to run DeepEdit first with your testdata set and then try it with my own data to detect A-to-I RNA editing sites in my directRNA sequencing reads.
I'm looking forward to hearing from you.
Best,
Camila

Hi Camila,
I guess this may due to that I compressed the nanopore_reads folder into a nanopore_reads.tar.gz file when I uploaded them on Github.
I have re-uploaded the related files, you may re-clone the files and try again.
Best,
Longxian Chen

Hello,
I re-cloned the DeepEdit directory, and by using this code:

I got the same result, my target_site file still looks the same:

I test the code on another server again and it works well.
So, you may debug on this following two aspects:

Do you have installed samtools in your server ?
Does the nanopore.sam file you used in your code is the one that I provided ?

Hello,
Thank you, I didn't load samtools before. So now I loaded it and I was able to generate the same target_sites.txt file.
However, when running the 2.feature_extract.py code.
I got the same target_site_features.csv, but with this variation.

I have 6 columns filled with nan values. I checked, and I'm using python 3.8.3.
I uploaded my outputs in this link: https://1drv.ms/f/s!AokqkR3muxL0jvohT7RWoCTBOV3Ncg?e=w25uhy

Hi,
Please take a look at the parameter options for tombo re-squiggle in our other repository:

https://github.com/weir12/DENA/blob/2c584c9a22f2903a1c44abe9a734fef5e9d158c8/README.md?plain=1#L116

In summary, you need to include the --include-event-stdev option in the tombo re-squiggle stage to calculate the standard deviation for each base.
Best regards,
Liang Ou

It worked. I was able to run your test dataset.
Thank you so much for your help.
Now I'm going to try it with my data.

Hello,
I'm currently trying with my own data.
I'm performing the basecalling. However, I have a question regarding this step:
guppy_basecaller -i DeepEdit/Getting_Started/nanopore_reads/ -s DeepEdit/Getting_Started/nanopore_reads/ --flowcell FLO-MIN106 --kit SQK-RNA001 --cpu_threads_per_caller 1 --qscore_filtering --fast5_out
What version of guppy did you use for your basecalling? Cause when I submitted my script to perform the basecalling I got this error.

It looks like qscore filtering is not longer a parameter for the guppy I have, and when I deleted it from my code, I was able to run it. However, I'm worried that this parameter is essential for further steps.

Hi,
Guppy (version 4.0.15)
That is clearly described in the Methods section of the paper.
https://genomebiology.biomedcentral.com/articles/10.1186/s13059-023-02921-0
Best regards,
Liang Ou

Perfect, thank you.


Version of vstest.console.exe for TPv2 > TPv1

Description
Version of vstest.console.exe for TPv2 > TPv1
There are components in our task which depends on version on the vstest.console.exe version including minor version and hence this should be more than what TPv1 was releasing

Please reactivate if this is still relevant.