[SDK BUG] Can't have transparent widget while Game Bar is open

Describe the bug
As of the late October update, one can't have a fully transparent widget while the Game Bar is open.
To Reproduce
Launch any widget (WidgetAdvSampleCS for example) and try to make it transparent while the Game Bar is open. You can't do it. RequestedOpacity doesn't change unless you close the Game bar and have the widget pinned.
Expected behavior
If you make the Background property of the page (MainPage.xaml) and the AppManifest's VisualElements.BackgroundColor property equal to "transparent", it should be transparent.
Desktop (please complete the following information):

OS version: Windows 10 20H2
Game Bar version 5.420.11102.0
SDK version: 5.3.200605002

Additional context
This used to work up until the Game Bar update in late October.

Hi jurkog,
Thank you for reporting. We'll investigate and see what's happening.
Thanks,
Ben

Hi jurkog,
Thank you for reporting. We'll investigate and see what's happening.
Thanks,
Ben

Hi Ben,
Thank you for looking into this, let me know if you need any additional information, I'd even be happy to hop on a call with you and give you a walk through of how this issue affects our product on the Game Bar. My business email is<EMAIL_ADDRESS>Best,
Jurko

Hi Ben,
Thank you for looking into this, let me know if you need any additional information, I'd even be happy to hop on a call with you and give you a walk through of how this issue affects our product on the Game Bar. My business email is<EMAIL_ADDRESS>Best,
Jurko

Hi Jurko,
I circled back with the team and we discussed this.  We performed a lot of user experience adjustments in the 2010 release last year, and one of those was ensuring all widgets (including our inbox ones) had a opaque backplate while in foreground (with the exception that some widgets will show transparency during settings preview modes).  However, based on your feedback and understanding the value of positioning your widget with transparency, we're going to look at allowing this again in 2104 (april release).  I did a quick test just turning off our backplate in WinGForeground mode, but it will regress some of the 3rd party widgets in the store (they rely on that solid backplate in foreground).  Due to this, we'll likely introduce a manifest flag you can specify that says "let me control opacity in foreground"... or something to that affect.
Sorry for this regression to your scenario and we'll try to have a solution as soon as possible.   We unfortunately cannot fix for 2102 (Feb) as we're already in final testing for that release.

Hi Jurko,
I circled back with the team and we discussed this.  We performed a lot of user experience adjustments in the 2010 release last year, and one of those was ensuring all widgets (including our inbox ones) had a opaque backplate while in foreground (with the exception that some widgets will show transparency during settings preview modes).  However, based on your feedback and understanding the value of positioning your widget with transparency, we're going to look at allowing this again in 2104 (april release).  I did a quick test just turning off our backplate in WinGForeground mode, but it will regress some of the 3rd party widgets in the store (they rely on that solid backplate in foreground).  Due to this, we'll likely introduce a manifest flag you can specify that says "let me control opacity in foreground"... or something to that affect.
Sorry for this regression to your scenario and we'll try to have a solution as soon as possible.   We unfortunately cannot fix for 2102 (Feb) as we're already in final testing for that release.

Hi Ben,
Thank you so much for the update! This is great news for us as we (and our users) really miss the previous functionality. Really forward to the April release. Let me know if there's anything you need from our end.
Thanks again,
Jurko

Hi Ben,
Thank you so much for the update! This is great news for us as we (and our users) really miss the previous functionality. Really forward to the April release. Let me know if there's anything you need from our end.
Thanks again,
Jurko

VSO: 31425937

Hi Ben,
Any update on this? We've been eagerly waiting for the release this April and any update on this would be much appreciated.
Best Regards,
Jurko

Hi Jurko,
Yes this is implemented, along with a center widget API.  You can see the code for both feature usage in the samples branch "prerelease". However, we haven't pushed a game bar build that supports these features or the corresponding NuGet as we discovered a MSVC compiler bug in our early flights (yes surprising) that was causing significant crashes.  MSVC team is investigating and we're trying to get unblocked ASAP for our April release.   Sorry for the delay.
Thanks,
Ben

Hi Ben,
I really appreciate your swift response and update. No problem, we're really looking forward to the update!
Thanks again,
Jurko

2104 version of Game Bar and SDK that supports centering API and transparency while in foreground.  please note the pkg-manifest doc page for how to enable transparency in foreground
SDK on NuGet.org
https://www.nuget.org/packages/Microsoft.Gaming.XboxGameBar/5.6.210414002
Docs updates here
https://docs.microsoft.com/en-us/gaming/game-bar/api/xgb-widget
https://docs.microsoft.com/en-us/gaming/game-bar/guide/pkg-manifest
Please note when you build against the new SDK and release it, in the rare case users acquire your new widget without acquiring the 2104 game bar, they will be instructed to upgrade game bar in order to launch your widget.  The link to upgrade game bar will be shown directly in your widget window.  After updating Game Bar, your widget will load.  Again this should be a rare occurrence for users.

Hi Ben,
Words cannot express the joy we felt seeing the transparency and centering features work on our end. Thank you and the Game bar team for this and all of your extended support.
Best regards,
Jurko & Sandra from CenterPoint Gaming


chore(main): release 0.8.18

:robot: I have created a release beep boop
0.8.18 (2023-12-08)
Bug Fixes

change goreleaser archive (e7c371c)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/kevinah95/hacienda/releases/tag/v0.8.18 :sunflower:


swarm init segmentation fault

Description
While running "docker swarm init" one of the processes of docker daemon gets SIGSEGV.

Steps to reproduce the issue:

Run docker daemon with gdb: "gdb --args dockerd -D -s devicemapper".
(gdb) run
In other terminal session do: "docker swarm init --advertise-addr $IP"

Describe the results you received:
Docker daemon gets segfault:
DEBU[0406] Assigning addresses for endpoint ingress-endpoint's interface on network ingress
DEBU[0406] RequestAddress(LocalDefault/<IP_ADDRESS>/16, <IP_ADDRESS>, map[])
DEBU[0406] Assigning addresses for endpoint ingress-endpoint's interface on network ingress
Detaching after fork from child process 11405.

Thread 9 "dockerd" received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0x7ffff1a02700 (LWP 11055)]
0x000000000066bf12 in net.networkNumberAndMask ()

Describe the results you expected:
Work fine without segfault.
Output of docker version:
docker version
Client:
 Version:      1.12.2
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   bb80604
 Built:
 OS/Arch:      linux/amd64

Server:
 Version:      1.12.2
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   bb80604
 Built:
 OS/Arch:      linux/amd64

Output of docker info:
Backing Filesystem: xfs
 Data file: /dev/loop0
 Metadata file: /dev/loop1
 Data Space Used: 16.91 MB
 Data Space Total: 107.4 GB
 Data Space Available: 95 GB
 Metadata Space Used: 585.7 kB
 Metadata Space Total: 2.147 GB
 Metadata Space Available: 2.147 GB
 Thin Pool Minimum Free Space: 10.74 GB
 Udev Sync Supported: true
 Deferred Removal Enabled: false
 Deferred Deletion Enabled: false
 Deferred Deleted Device Count: 0
 Data loop file: /var/lib/docker/devicemapper/devicemapper/data
 Metadata loop file: /var/lib/docker/devicemapper/devicemapper/metadata
 Library Version: 1.02.122 (2016-04-09)
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: overlay bridge null host
Swarm: active
 NodeID: 29ql6fig3b3xu21k8ysckisyg
 Is Manager: true
 ClusterID: 06j5qk4aln8uqo6c11gnt6pzb
 Managers: 1
 Nodes: 1
 Orchestration:
  Task History Retention Limit: 5
 Raft:
  Snapshot Interval: 10000
  Heartbeat Tick: 1
  Election Tick: 3
 Dispatcher:
  Heartbeat Period: 5 seconds
 CA Configuration:
  Expiry Duration: 3 months
 Node Address: <IP_ADDRESS>
Runtimes: runc
Default Runtime: runc
Security Options: seccomp
Kernel Version: 4.7.7-200.snorch.fc24.x86_64
Operating System: Fedora 24 (Workstation Edition)
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 15.54 GiB
Name: dhcp-10-30-29-163.sw.ru
ID: FOBI:53SZ:3XT7:5E4Q:JJND:QPOY:PQAT:GPCK:IVQ6:L2AO:IZPY:QWKU
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
 File Descriptors: 34
 Goroutines: 119
 System Time: 2016-10-21T15:16:15.941817613+03:00
 EventsListeners: 0
Registry: https://index.docker.io/v1/
Insecure Registries:
 <IP_ADDRESS>/8

Additional environment details (AWS, VirtualBox, physical, etc.):
Fedora 24, docker-1.12.2
gdb with symbols:
#0  0x000000000066dcb2 in net.networkNumberAndMask (n=0x0, ip=..., m=...) at /usr/local/go/src/net/ip.go:433
#1  0x000000000066e002 in net.(*IPNet).String (n=0x0, ~r0="") at /usr/local/go/src/net/ip.go:483
#2  0x00000000004b3e1a in fmt.(*pp).handleMethods (p=0xc8204369c0, verb=118, depth=0, handled=true) at /usr/local/go/src/fmt/print.go:730
#3  0x00000000004b43a9 in fmt.(*pp).printArg (p=0xc8204369c0, arg=..., verb=118, depth=0, wasString=false)
    at /usr/local/go/src/fmt/print.go:806
#4  0x00000000004bb70d in fmt.(*pp).doPrintf (p= []uint8, format="error setting interface %q IPv6 to %v", a= []interface {} = {...})
    at /usr/local/go/src/fmt/print.go:1238
#5  0x00000000004aeeef in fmt.Sprintf (format="error setting interface %q IPv6 to %v", a= []interface {} = {...}, ~r2="")
    at /usr/local/go/src/fmt/print.go:203
#6  0x0000000000d50a52 in github.com/docker/libnetwork/osl.configureInterface (nlh=0xc820b4d8a0, iface=..., i=0xc820436d00, ~r3=...)
    at /go/src/github.com/docker/docker/vendor/src/github.com/docker/libnetwork/osl/interface_linux.go:333
#7  0x0000000000d4f25a in github.com/docker/libnetwork/osl.(*networkNamespace).AddInterface (n=0xc8200f7080, srcName="ov-000100-eko1d",
    dstPrefix="br", options= []github.com/docker/libnetwork/osl.IfaceOption = {...}, ~r3=...)
    at /go/src/github.com/docker/docker/vendor/src/github.com/docker/libnetwork/osl/interface_linux.go:297
#8  0x0000000000d82538 in github.com/docker/libnetwork/drivers/overlay.(*network).setupSubnetSandbox (n=0xc820158820, s=0xc820148a00,
    brName="ov-000100-eko1d", vxlanName="vx-000100-eko1d", ~r3=...)
    at /go/src/github.com/docker/docker/vendor/src/github.com/docker/libnetwork/drivers/overlay/ov_network.go:473
#9  0x0000000000d82e5c in github.com/docker/libnetwork/drivers/overlay.(*network).initSubnetSandbox (n=0xc820158820, s=0xc820148a00,
    restore=false, ~r2=...) at /go/src/github.com/docker/docker/vendor/src/github.com/docker/libnetwork/drivers/overlay/ov_network.go:507
#10 0x0000000000d99756 in github.com/docker/libnetwork/drivers/overlay.(*network).joinSubnetSandbox.func1 ()
    at /go/src/github.com/docker/docker/vendor/src/github.com/docker/libnetwork/drivers/overlay/ov_network.go:229
#11 0x000000000070d584 in sync.(*Once).Do (o=0xc8209139e0, f={void (void)} 0xc820b45658) at /usr/local/go/src/sync/once.go:44
#12 0x0000000000d7fd0a in github.com/docker/libnetwork/drivers/overlay.(*network).joinSubnetSandbox (n=0xc820158820, s=0xc820148a00,

cc @aboch PTAL

The issue is not related to swarm init. Any operation which triggers the IP address assigment of an interface inside the sandbox would cause this. For example, this will cause it for me: docker run busybox.


Clappr not working with HLS in chrome or firefox

Added player script and it only works in safari and nothing else.

Is there any error showing in the console?

@djevo1 since we're using hls.js (a js hls implementation) you might be facing CORS problems, anyway is there any error on your console as @hounvs asked you?

Not sure how to view the console, hosting it on a wordpress site. Should i just ssh into it?
---Sent from Boxer | http://getboxer.com
On January 19, 2016 at 9:00:20 AM EST, Leandro Moreira<EMAIL_ADDRESS>wrote:@djevo1 since we're using hls.js (a js hls implementation) you might be facing CORS problems, anyway is there any error on your console as @hounvs asked you?  —Reply to this email directly or view it on GitHub.

@djevo1, when you view the page in Chrome and Firefox, you should be able to pull up the Javascript console (Windows: Ctrl+Shift+i, Mac: CMD+Shift+i)
If you are seeing any error that says "Access-Control-Allow-Origin", it's a CORS issue and can be fixed by adjusting the headers on your streaming server. Firefox and Chrome have CORS enabled by default as a security measure whereas other browsers don't.
I see that you're using Wowza like I am. This is what I used to fix the problem for me: https://streamroot.readme.io/docs/wowza-configuration
Let me know if that was the issue


Add support for macOS?

Since it works in Linux Desktop, maybe getting it to work on macOS probably only needs a recompilation with clang (if the original binary was built with clang for macOS) and maybe some more minor tweaks of some constants?

Simple recompilation may not work, since machine code matching is compiler dependent and must be reworked for different platforms. I'm happy to see someone else could do it...
The key reason for lack of macOS version is that I have no mac devices. As you know, they are so expensive for the people lived in China that nearly all players here are playing game on Windows.

Yes, I can see most of your use base do not require macOS support, but Apple products are quite prevalent in some other countries. I am not well versed enough to do manual machine code matching myself but if you are willing to write up a detailed instructions for the steps required, I'd give it a try.
If it is any useful info., I am running Debian testing in an old MacBook Pro, and it works. So x86-64 platform and the compiler is what we need match up against? Here is the result of objdump -all-headers dontstarve_steam: https://termbin.com/orqc though not sure if it contains anything useful.

Yes, I can see most of your use base do not require macOS support, but Apple products are quite prevalent in some other countries. I am not well versed enough to do manual machine code matching myself but if you are willing to write up a detailed instructions for the steps required, I'd give it a try.
If it is any useful info., I am running Debian testing in an old MacBook Pro, and it works. So x86-64 platform and the compiler is what we need match up against? Here is the result of objdump -all-headers dontstarve_steam: https://termbin.com/orqc though not sure if it contains anything useful.

This is a hard and boring work. But you may try it with following steps (maybe interesting at the first time if you have never done it before):

Compile linux version of the mod. Startup with linux build world saves much of your time.
Run the game with the mod and it crashed. Capture the log it prints out.
Fix the function mismatch (marked by "Missing") one by one via assembly code comparison (totally almost 100+ functions starts with lua_ or luaL or luaopen) with:
a) modify comparsion algorithm.
b) do some modification on lua51 project so it can generate matching machine code.

As you know, the 3.b) step requires various experiences on compiler, you must use the same compiler as Klei and find out the modifications by Klei via disassembled machine code. After reimplementing theses changes to lua51 project, you can check the newly generated machine code by running game again. If it's matched, then move to next missing function and repeat the instructions.

objdump

objdump shows that it exports no lua functions. Machine code matching is the only approach.

Is it possible to automate some part of step 3, maybe in a loop, run the game, grep first missing function, prompt to select from a few matching rules and append that rule to a file. Once done, we can then refine that file into the lua51 project and build it... I have no idea if what I am saying here is even feasible as I've never done stuff like that...

Is it possible to automate some part of step 3, maybe in a loop, run the game, grep first missing function, prompt to select from a few matching rules and append that rule to a file. Once done, we can then refine that file into the lua51 project and build it... I have no idea if what I am saying here is even feasible as I've never done stuff like that...

Unfortunetly it's impossible to select serveral candidates automatically. Once matching failed, it turns out that klei modified some functions, for example:
int _cdecl lua_XXXXX(lua_State* L) {
// luaC_checkGC(L); // One call to checkGC was removed
}
It would generate totally different machine code at first serveral bytes and invalidates all candidates you find out...
When I was developing windows version, I searched them by some reverse engineering techniques (such as tracing back from resolved ones) and finally get them all resolved.

Is it possible to automate some part of step 3, maybe in a loop, run the game, grep first missing function, prompt to select from a few matching rules and append that rule to a file. Once done, we can then refine that file into the lua51 project and build it... I have no idea if what I am saying here is even feasible as I've never done stuff like that...

In fact I'm a not experienced in reverse engineering so I've no better approaches but do it with brute force. My interest is in rendering optimization but it cannot be done due to lack of source.


Vite Plugin - Watch Workspace

Adds a plugin that will watch local workspace dependencies for changes, rebuilding when they change.

[x] watches all files in all packages in all workspaces
[x] excludes current package where vite.config.ts is located (these are already watched)
[x] has optional excludes via ignore globs
[x] works for JSX

@forgetso ready to approve after discussed changes to the switch case + defaults


🐛 Bug Report: Refused to execute script from '' because its MIME type ('text/html') is not executable, and strict MIME type checking is enabled and manifest file 404 with blank screen

📜 Description
i have set up the application and then exposed it to ingress path based, in google cloud
app:
title: Backstage App
baseUrl: https://qa.leap.net/backstage
organization:
name: Spotify
backend:
baseUrl: https://qa.leap.net/backstage
listen:
port: 7007
csp:
connect-src: ['*']
cors:
origin: https://qa.leap.net/backstage
methods: [GET, HEAD, PATCH, POST, PUT, DELETE]
credentials: true
database:
client: better-sqlite3
connection: ':memory:'
integrations:
github:
- host: github.com
token: ${GITHUB_TOKEN}
proxy: {}
techdocs:
builder: 'local'
generator:
runIn: 'docker'
publisher:
type: 'local'
auth:
providers:
guest:
dangerouslyAllowOutsideDevelopment: true
scaffolder: {}
catalog:
import:
entityFilename: catalog-info.yaml
pullRequestBranchName: backstage-integration
rules:
- allow: [Component, System, API, Resource, Location]
locations:
- type: file
target: ../../examples/entities.yaml
- type: file
target: ../../examples/template/template.yaml
rules:
- allow: [Template]
- type: file
target: ../../examples/org.yaml
rules:
- allow: [User, Group]
Consolidated Kubernetes configuration
kubernetes:
serviceLocatorMethod:
type: 'multiTenant'
clusterLocatorMethods:
- type: 'config'
clusters:
- url:
name:
authProvider: 'google'
skipTLSVerify: false
skipMetricsLookup: true
permission:
enabled: true
ingress file
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
annotations:
cert-manager.io/cluster-issuer: letsencrypt-prod
nginx.ingress.kubernetes.io/configuration-snippet: |
rewrite ^/backstage/static/(.*)$ /static/$1 break;
add_header X-Content-Type-Options nosniff;
add_header X-Frame-Options SAMEORIGIN;
add_header X-XSS-Protection "1; mode=block";
add_header Content-Security "default-src 'self' 'unsafe-inline' 'unsafe-eval'; script-src 'self' 'unsafe-inline' 'unsafe-eval'; connect-src 'self' *; img-src 'self' data:; style-src 'self' 'unsafe-inline';";
nginx.ingress.kubernetes.io/proxy-redirect-from: "off"
nginx.ingress.kubernetes.io/rewrite-target: /
nginx.ingress.kubernetes.io/service-upstream: "true"
nginx.ingress.kubernetes.io/upstream-vhost: backstage.backstage.svc.cluster.local
nginx.ingress.kubernetes.io/use-regex: "true"
creationTimestamp: "2024-12-17T06:07:01Z"
generation: 9
name: dummy
namespace: backstage
resourceVersion: "921390731"
uid: 78127ce5-b2ed-421a-b7ff-2faafc19e230
spec:
ingressClassName: nginx
rules:

host: qa.leap.net
http:
paths:

backend:
service:
name: backstage
port:
number: 80
path: /backstage/(.*)
pathType: Prefix
backend:
service:
name: backstage
port:
number: 80
path: /static/*
pathType: Prefix

👍 Expected behavior
I am getting application is running inside the pod but 404 error for manifest.json file
👎 Actual Behavior with Screenshots
I am getting errors:
Refused to execute script from '' because its MIME type ('text/html') is not executable, and strict MIME type checking is enabled.
and then getting 404 manifest.json file not found
so https:qa.leap.ai is coming black screen
👟 Reproduction steps
i was editing the app.config.yaml and ingress files to resolve,
📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

Hey :wave: I replied in the other issue that you commented on. Going to close this, and keep the conversation in one place https://github.com/backstage/backstage/issues/25662


Please consider un-deprecating the _ReadBarrier intrinsic

It appears to be the best way to guarantee instructions order with modern optimizing C++ compilers, VS2022 in that case.
See this stackoverflow question for a use case and a demo.
That’s neither inter-thread communication nor hardware access. Other workarounds do exist like volatile keyword or C++ atomics, but they all come at cost, they cause values to be evicted from registers into stack. The _ReadBarrier compiles into nothing i.e. free performance-wise, but it does the job by preventing instructions reordering.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ee4a0e5b-e0a4-b337-c981-3ef47e1d78f7
Version Independent ID: 5f89a8cd-b8c6-25ee-1519-1c2dd1f18ff7
Content: _ReadBarrier
Content Source: docs/intrinsics/readbarrier.md
Product: visual-cpp
Technology: cpp-tools
GitHub Login: @corob-msft
Microsoft Alias: corob

@Const-me
I appreciate your suggestion, but you're making it in the wrong place. The documentation team doesn't have much influence on the product itself. If you'd like the development team to take your idea up, then you should let them know directly by creating an issue in the C++ Developer Community forum. Check first to see if someone else has made a similar suggestion, which you can upvote. The more upvotes, the more likely the product team is to consider a change. Best of luck.

@corob-msft Hi. But the product works just fine now.
The only thing which needs to be changed is the documentation, to remove the deprecation warning.
Does it still count as influence on the product?

@Const-me
The documentation team doesn't decide what's deprecated. We added the deprecation note because the product team told us to. Those intrinsics aren't deprecated because they're going away, they're deprecated in favor of using platform-independent code. I suspect it's because the compiler now has ARM targets that don't have exactly the same kinds of memory ordering facilities, and emulation of these intrinsics in x64 code running on ARM has performance penalties. So, if you want the deprecation warning removed, talk to the product team about why it's deprecated, and they will let us know if we should change the docs.

@corob-msft Got it, thanks.


query_arrow is unusable if results contain strings and server cannot be configured by the user

Describe the bug
Whether I can run query_arrow at all depends on whether I can configure server side settings. It doesn't seem to matter whether I pass use_strings=True or use_strings=False
This is a suboptimal user experience. It's not clear why the client must be able to configure the server to request results in a different format.
Steps to reproduce

Run query_arrow(...) against the clickhouse playground.

Expected behaviour
A successfully executed query
Code example
In [15]: import clickhouse_connect as cc

In [16]: con = cc.create_client(
    ...:     host="play.clickhouse.com",
    ...:     port=443,
    ...:     user="play",
    ...:     secure=True,
    ...:     password="clickhouse",
    ...: )

In [17]: con.query_arrow("show tables")
...
ProgrammingError: Setting output_format_arrow_string_as_string is unknown or readonly

clickhouse-connect and/or ClickHouse server logs
I'm not sure how to access either of those logs.
Configuration
Environment

clickhouse-connect version: 0.5.22
Python version: 3.10.11
Operating system: Linux

ClickHouse server

ClickHouse Server version: <IP_ADDRESS>1
ClickHouse Server non-default settings, if any: No idea, I don't manage the server.
CREATE TABLE statements for tables involved: None

Thanks for the detailed report!  This should be fixed in the 0.5.23 version which will be released shortly.


Adjust egeria community bridge insights link

The another its invalid, showing blank dash.

You will need to sign the DCO to make the change
( git commit --amend -s should work )

Many thanks for the proposed contribution. However without the DCO we cannot merge this so are closing and replacing.
Replaced by #3161


Improve Vulkan command buffer thread safety

This addresses part of #54530
The goal is to protect the command buffers from simultaneous modification on multiple threads.
Because I'm far from an expert with Vulkan, it's more than likely I've made a few mistakes. I'm hoping for a thorough review so this doesn't bork anything.

I'm not sure if the command buffers being used are always for the current frame, or if previous frame command buffers might be touched. If there's a mix, then each frame would need its own mutex.

After staring at the vulkan renderer for quite a bit longer, I've realized this is not correct.


LOOP-2534 Refactor onboarding and support manager plugins

https://tidepool.atlassian.net/browse/LOOP-2534
https://tidepool.atlassian.net/browse/LOOP-1550
https://tidepool.atlassian.net/browse/LOOP-2955
Refactor onboarding and support manager functionality into separate plugin types
Allow onboarding to provide view controller
Remove providesOnboarding service functionality
Add LoopUIColorPalette to collect up all relevant UI colors for plugins
Refactor setup/settingsViewController to use LoopUIColorPalette
Add isOnboarded property to DeviceManager and Service protocols
Separate setup notifications into create and onboard notifications
Move service delete notification into ServiceDelegate from ServiceNotifying
Add explicit delete to Service
Add pump manager, CGM manager, and service Provider protocol for onboarding purposes
Add UIResult enum to capture success, failure, or user interaction required
Rename AvailableDevice to CGMManagerDescriptor and PumpManagerDescriptor
Rename AvailableService to ServiceDescriptor
Remove deprecated LoopUIPlugin
Update Mocks to support latest protocols
Allow plugin extensions to provide additional functionality
Fix warnings

Draft PR for LW for build: https://github.com/tidepool-org/LoopWorkspace/pull/409

Draft PR for LW for build: https://github.com/tidepool-org/LoopWorkspace/pull/409

Wouldn’t that fit into the TidepoolService plugin? How is the event going to be submitted? Via the Tidepool service, no?

Wouldn’t that fit into the TidepoolService plugin? How is the event going to be submitted? Via the Tidepool service, no?

I think an enum shared between them is fine, but the error state doesn’t belong,

I think an enum shared between them is fine, but the error state doesn’t belong,

@ps2 @nhamming @rickpasetto Re-requesting review on the changes since your last review. These changes were made to address various PR comments.


downloadas can not be set correctly

Thanks for your great project, but i catch an exception that  "downloadas" can not be set correctly. Here is my code.
const jswget = require('jswget');
let option = {
    url: "http://dldir1.qq.com/qqfile/qq/QQ8.9.1/20437/QQ8.9.1.exe",
    downloadmode: true,
    // if not given will try to get name from last path, otherwise force to use this name
    downloadas: "qq.exe",
    // if not given, the default path is where the program run
    downloadpath: "./",
    method: "GET",
    onsend: function(req, options){
        console.log("Start",options)
    },
    onhead: function(fstat, req, res){
        console.log(res.headers)
    },
    ondata: function(chunk, req, res){
        console.log("Get: " + chunk.length)
    },
    onsuccess: function(resp, req, res){
        console.log("Done")
    },
    onerror: function(err, req){
        console.log("Error", err);
    }
};
jswget(option);

After run these code, i find a file named undefined in my project directory.

Then, i debug source code and find a bug in line 143 of jswget.js.

When i edit  line 143 as follows, it works well. So i hope you have time to check the code here. Thank you again!

sorry, i find same problem in issue #5


Resizing like in chunkwm

Hi, yabai is really cool, thanks for all the hard work that went into it!
I’m wondering how to achieve the same resizing with keys I had working in chunkwm like so:
shift + alt - a : chunkc tiling::window --use-temporary-ratio 0.05 --adjust-window-edge west; chunkc tiling::window --use-temporary-ratio -0.05 --adjust-window-edge east
shift + alt - s : chunkc tiling::window --use-temporary-ratio 0.05 --adjust-window-edge south; chunkc tiling::window --use-temporary-ratio -0.05 --adjust-window-edge north
shift + alt - w : chunkc tiling::window --use-temporary-ratio 0.05 --adjust-window-edge north; chunkc tiling::window --use-temporary-ratio -0.05 --adjust-window-edge south
shift + alt - d : chunkc tiling::window --use-temporary-ratio 0.05 --adjust-window-edge east; chunkc tiling::window --use-temporary-ratio -0.05 --adjust-window-edge west

Thanks for your help, I tried something like
shift + alt - w : yabai -m window --resize top:0:-10; yabai -m window --resize bottom:0:10
shift + alt - s : yabai -m window --resize top:0:10; yabai -m window --resize bottom:0:-10

but that does not work as expected with all windows wherever they are located on the display.

This is a tough problem to solve. Assuming you have three windows side by side, which window should be resized?
The below came up in a previous thread about this and behaves just like i3 and similar to chunkwm, which is basically what you were suggesting.
ctrl + cmd - h : yabai -m window --resize left:-20:0 ; yabai -m window --resize right:-20:0
ctrl + cmd - j : yabai -m window --resize bottom:0:20 ; yabai -m window --resize top:0:20
ctrl + cmd - k : yabai -m window --resize top:0:-20 ; yabai -m window --resize bottom:0:-20
ctrl + cmd - l : yabai -m window --resize right:20:0 ; yabai -m window --resize left:20:0

Personally I use the below and find it to be a lot more convenient. E.g. when moving to the left using ctrl + cmd + h, I try to shrink the westwards windows right border, and if that fails because no westward window exists, I instead resize the focused windows right border).
This still does not cover all situations, but I do think it feels a lot more natural.
ctrl + cmd - h : yabai -m window west --resize right:-20:0 2> /dev/null || yabai -m window --resize right:-20:0
ctrl + cmd - j : yabai -m window north --resize bottom:0:20 2> /dev/null || yabai -m window --resize bottom:0:20
ctrl + cmd - k : yabai -m window south --resize top:0:-20 2> /dev/null || yabai -m window --resize top:0:-20
ctrl + cmd - l : yabai -m window east --resize left:20:0 2> /dev/null || yabai -m window --resize left:20:0

Also you can just use the mouse to resize splits with yabai: just resize the window and the splits will be adapted.

Thanks! I knew about the mouse feature, but your second suggestion works fine for me when I use the keyboard!

@dominiklohmann I wanted to try your suggested commands:
ctrl + cmd - h : yabai -m window west --resize right:-20:0 2> /dev/null || yabai -m window --resize right:-20:0
ctrl + cmd - j : yabai -m window north --resize bottom:0:20 2> /dev/null || yabai -m window --resize bottom:0:20
ctrl + cmd - k : yabai -m window south --resize top:0:-20 2> /dev/null || yabai -m window --resize top:0:-20
ctrl + cmd - l : yabai -m window east --resize left:20:0 2> /dev/null || yabai -m window --resize left:20:0

If I have three windows with equal size side-by-side, if I use ctrl + cmd - h on the middle window, the left window shrinks as expected, and the middle window expands to the left as expected, however, the right-most window also slightly increases in size towards the left, and the middle window slightly moves its right-most border to the left.
Is this expected behaviour?

Update to the above issue:
This does not happen if I resize the middle window to the right:

This is the intended behavior. You need to think of it in terms of the underlying tree structure.
You'll likely get better results if you just resize windows using the mouse. All other windows will be adjusted accordingly.

Take a look at these keybindings, it's more intuitive, take less cost and mute noise.
shift + alt - h : yabai -m window --resize right:-20:0 2> /dev/null || yabai -m window --resize left:-20:0 2> /dev/null
shift + alt - j : yabai -m window --resize bottom:0:20 2> /dev/null || yabai -m window --resize top:0:20 2> /dev/null
shift + alt - k : yabai -m window --resize bottom:0:-20 2> /dev/null || yabai -m window --resize top:0:-20 2> /dev/null
shift + alt - l : yabai -m window --resize right:20:0 2> /dev/null || yabai -m window --resize left:20:0 2> /dev/null

@qizidog Thanks a ton.
I was looking for something like that.
It works like a clock.


Adding a number

Hello!
I'm quite new to whatsapp, so it could be simple ignorance on my part, but I did not found a way to add a contact with a phone number.
Please help me, or fix it if it is a bug!
Thank you for this app!

I have researched this a bit, and I can't really fathom why it is like that, but it seems there is no way to add a contact from the web interface. (Maybe they want to get your precious contact list from the app, as even the app doesn't allow to add a contact IF you deny access to your contacts list.)
BUT!
Check out this:
https://faq.whatsapp.com/5913398998672934
So with a specially crafted link, it is possible to add a contact:
https://wa.me/1XXXXXXXXXX
This could be an extra option in the drawer, or something.

Hi, will also be fixed with #87


Feature Request: Dummy Window Switch

Already present in the old plugin, now unfortunately missing 😅
The window switch as a dummy switch under the central switch is needed for contact sensor based automations to be set as a condition.
Through these conditions the fresh air automations can be switched off at any time.
Would be great if you would implement this again. Thank you!
Ps. Further details I have written to you in the forum.

Added in v6.0.9
https://github.com/SeydX/homebridge-tado-platform/blob/master/README.md#dummy-switch

There is still a problem with the dummy switch.
It does not save its status after a restart of the HB.
Can you still implement this?

Fixed in v6.0.10

👌 Thx


Can Zwift run in maximized windows ?

I use Fedora with two HD monitors. Zwift runs on one monitor in the window.
However, this window is not as big as the screen. I still have to make it a "fullscreen window". Can I tell Docker to run Zwift in a maximized window?

Once you're logged in, you can go to the settings and pick the fullscreen option.
Click on your avatar (in the top bar) > My settings > Sound & Display tab > Screen Mode

Thanks a lot! When I do this: Zwift opens as fullscreen; but when I use the second monitor the fullscreen-zwift is close to the taskbar. I can open it again, but it is not the I want. I think I need a maximized windows, no fullscreen windows for zwift.

Sounds like something your window manager should be handling.
Maybe look up docs for your desktop environment.

Closing as this is definitely a window manager issue.


smv_released_pattern = r'^tags/.*$' does not work with tag "2.0"

smv_released_pattern = r'^tags/.*$' does not work with tag like 2.0
{
    "2.0": {
    "name": "2.0",
    "version": "",
    "release": "",
    "is_released": false,
    "source": "tags",
    "creatordate": "2021-12-27 04:14:55 +0000",
    "basedir": "/tmp/tmpcs03xejr/4b135877bb1966e13d50e3a14e70722e921883ff",
    "sourcedir": "/tmp/tmpcs03xejr/4b135877bb1966e13d50e3a14e70722e921883ff/tdd/source",
    "outputdir": "/home/minhgdd/sphinx-example/tdd/build/2.0",
    "confdir": "/tmp/tmpcs03xejr/4b135877bb1966e13d50e3a14e70722e921883ff/tdd/source",
    "docnames": [
    ]
  }
}

is_released is still false

Found solution in #66


refactor: create one initRule helper and use in all rule name prompting

Description
Replaces each init<plugin>rule helper function with one initRule function that uses the new validation rule interface. This function calls the methods defined on that interface, which each plugin now implements.

@mattwelke please resolve conflicts

Sorry, this should have been marked as a draft. It's still a WIP. I need to fix it once the MAAS release is fixed, resolve any remaining conflicts, and test it.


Video freeze for sometimes and start it again

I am using this for video playing in my app. But it seems to be freeze for some time and start it again. When I add bufferConfig in Video component ,it works fine for iOS ,but still issue is facing on android
            <Video
                key={videos[index].videoId}
                style={styles.backgroundVideo}
                ref={(ref) => {
                    setPlayer(ref);
                }}
                ignoreSilentSwitch={"ignore"}
                posterResizeMode={'contain'}
                resizeMode={'contain'}
                source={{
                    uri: videoUrl,
                    type: 'mp4',
                }}

                    bufferConfig={{
                        minBufferMs: Platform.OS === 'ios' ? 1500 : 1500,
                        maxBufferMs: Platform.OS === 'ios' ? 5000 : 50000,
                        bufferForPlaybackMs: 5000,
                        bufferForPlaybackAfterRebufferMs: 5000,
                    }}
         />

react-native-video version - 5.1.1
react-native-version - 0.63.4
Please help me

+1

Any updates?


Missing support for Shared OpenGL Context on Windows?

Just getting my feet wet with shared contexts on Windows and I maybe I'm wrong here, but it seems that the sharedGLContext is ignored in PlatformWGL.cpp and it should be passed into wglCreateContextAttribs(). Or am I wrong here?
WGL_ARB_create_context

@bejado

I think you're right, good catch.

What's strange wglCreateContextAttribs is a exclusive Windows function (starts with wgl) but you can't find this function in the Microsoft docs.


[Bug] Database\Eloquent: Should syncOriginal method be called before firing created?

If I do some post creation process on the record I just created in the created event and want to save the updated information I get an error :
Undefined index: id
file: vendor/laravel/framework/src/Illuminate/Database/Eloquent/Model.php
line:1317
This is because the new created item has its attribut exists set to true, so it tries to update but no id attribut is filled in original array because syncOriginal() has not been called yet.
I have to manually call that method in my created event before changing any attribut value.
public static function boot()
{
    parent::boot();
    MyModel::created(function($object)
    {
    	$object->syncOriginal(); // will trigger an error if not called
    	$object->position = MyModel::where('user_id', $object->user_id)->count();
    	$object->save();
    });
}

This is still not fixed in v4.2.17.


Replace the no_std feature with the std feature

Features should be additive as pointed out in issue #29. The feature is called std instead of the proposed use_std, because it's preferred to call the features same as the crates they enable. (Features and dependencies share a namespace.)
Closes: #29

Have you been waiting for me to merge this?
It didn't hit me until now that I can.


[Question] How do I combine colmap dataparser parameters with splatfacto training?

Hi, I just have a quick question. I am currently training a model using splatfacto, but the orientation is messed up. I wanted to use
--orientation_method=none --center_method=none --assume_colmap_world_coordinate_convention=False  to attempt at fixing it, but unfortunately those works only when using the colmap subcommand of splatfacto.
Is there any way I could use the parameters of ns-train splatfacto colmap while also using the pipeline parameters of ns-train splatfacto?

you can try to convert your colmap project to nerfstudio format using ns-process-data (without rerunning the colmap) and load it into ns-train.

I was already processing the images using ns-process-data, but those parameters where missing, they were only availble using ns-train. I used comlmap2nerf script in the past but I was hoping for a more compact solution. Gonna use that then, thanks anyway!

What's the actual purpose of ns-train splatfacto colmap tought? Cause the processing is done through ns-process-data and the train with ns-train splatfacto --data etc. I can't understand how does it fell in between

@Filocava99 yeah if you have a colmap project already and doesn’t want to convert it to nerfstudio format, you can still quick start training the model using colmap parser.

Ok but here's the thing: ns-process-data creates a colmap project and converts it to nerfstudio format. It lacks the orientation method parameter, resulting in the final model being slighty tilted.


feat(visibility): change visibility for some fields

make chunksCount public
make isTouched private settable

I guess isTouched was settable before, since it was var, but nvm, thanks for contribution, making chunksCount public makes sense tho

About isTouched, I think it should be done via a Listener.
And add the property isTouchable? I would like to disable the "touchable" feature

@Gounlaf Thought about it as well, alright, I'll add it if you haven't done this yet 😉

I think I'll issue a new release with isTouchable and your changes together, is that okay for you?

@Gounlaf Thought about it as well, alright, I'll add it if you haven't done this yet 😉
I think I'll issue a new release with isTouchable and your changes together, is that okay for you?

I was writing an issue before make a PR xD
I haven't started it yet, but I can do it within the hour

@Gounlaf I did it already, nvm

@Gounlaf I did it already, nvm
Well, I will wait for your updates @alxrm ;) Thanks!

@Gounlaf, I drafted a new release including your changes


values_list return string insted of HASHID object

Model.objects.values_list("id", flat=True)
returns list of HashID objects that require additional conversion to string. Is this possible to return flat string instead ?

For future reference (anyone that finds this issue), the Hashid object can be turned off by either setting  HASHID_FIELD_ENABLE_HASHID_OBJECT = False in the Django settings, or passing enable_hashid_object=False in the Hashid*Field constructor. See README for more info.


Feature/fix geodetic conversion

Tickets addressed: bsk-353
Review: By commit
Merge strategy: Merge (no squash)

Description
In geodeticConversion.cpp/h the methods PCI2PCPF() and PCPF2PCI() use the transpose of the required DCM
Verification
The methods are being called and yielding the correct coordinate frame transformations.
Documentation
None
Future work
None

@patkenneally would have to explain to me how to use this new testing framework.  I did glance at it this weekend and I was left with several questions just looking at the prior commits.

added unit tests using the new google test framework.

Thanks for the feedback @patkenneally  on @spiggottCO addition to my branch.  I'll find time to go over review comments and apply or respond.


Support non-public clouds for Azure Storage Blob

Proposal
Support non-public clouds for Azure Storage Blob
Use-Case
No response
Anything else?
Relates to #1285

Done through https://github.com/kedacore/keda/pull/1863


Feature request: RSS feed for posts

Would be great to be able to subscribe to your blog in an RSS reader.

Sorry for the delay in replying,it has been a bit busy days :).
Definitely,thanks for the idea,I will definitely add this feature.

Thank you! I've subscribed now and it's working well.


ERROR FIXED: Trying to save file in a location that doesn't exist.

I gather those XMLs are for LVC fleet, however with this being a public release, we don't all share the same filesystem. This commit resolves the error by creating and saving to a localappdata folder!
Folder Location: %localappdata%\SirenSharp

This was my mistake. I left testing files in the code. Thank you for fixing this!


add video call feature in code room

implement video call with agora RTC SDK
to use it, either sign up for an account at https://console.agora.io/ or use the following values in .env.local:
NEXT_PUBLIC_AGORA_APP_ID="07a3b6e024c94596ae016c0bb2646e38"
NEXT_PUBLIC_AGORA_APP_CERTIFICATE="201827a36cb44e3c9086019cca1eaa48"

@fannyjian hmmm i could not replicate the permission error, on my side when i deny permission it will simply not show up... but lemme add some error boundary for that
for the second error i'll be pushing a fix for that!


Event bus ADD_CARD support index

fixes #471
Allow support of passing index to ADD_CARD event.

@dapi Is there something that's holding up getting this PR merged?


fix: possible serialization error due to non-transient variable inside serializable class

In file: Holder.java, the serializable class Holder contains a non-serializable field. In this case, serialization of the class can lead to a NotSerializableException. iCR suggested to make the fields serializable or transient. To learn more check CWE 594  and also more explanation and its side effects are briefly explained here: What Happens When A Serializable Object Contains a Non-Serializable Field?
Sponsorship and Support:
This work is done by the security researchers from OpenRefactory and is supported by the Open Source Security Foundation (OpenSSF): Project Alpha-Omega. Alpha-Omega is a project partnering with open source software project maintainers to systematically find new, as-yet-undiscovered vulnerabilities in open source code - and get them fixed – to improve global software supply chain security.
The bug is found by running the Intelligent Code Repair (iCR) tool by OpenRefactory and then manually triaging the results.

@lukasj this change seems concerning.  Holder will never hold anything if it is serialized.  It will always have a null value when deserialized now.  I am not sure if this was the right change to make for this issue.

A possible other solution for the problem described is doing:
    public Holder(T value) {
        if (value != null && !value instanceof Serializable) {
            throw new IllegalArgumentException("Value is not serializable");
        }
        this.value = value;
    }

The risk here is that it will never by serialized so it doesn't matter if it is serialize-able or not.  If that is the case that Holder is never serialized, then it doesn't matter if it is transient or not.


perf(prow-jobs): update trigger condition for jobs of pingcap/tiflow

Reverts PingCAP-QE/ci#2239

/approve


Build failure

I've attempted to build the project, as I'd like put some effort into fixing #53. However, I'm receiving the following error:
><EMAIL_ADDRESS>build:js /Users/nicholas/git/github/ngrx/effects
> ngc -p tsconfig.dist.json

/Users/nicholas/git/github/ngrx/effects/node_modules/@angular/compiler-cli/src/private_import_compiler.js:10
exports.AssetUrl = compiler_1.__compiler_private__.AssetUrl;
                                                  ^

TypeError: Cannot read property 'AssetUrl' of undefined
    at Object.<anonymous> (/Users/nicholas/git/github/ngrx/effects/node_modules/@angular/compiler-cli/src/private_import_compiler.js:10:51)
    at Module._compile (module.js:573:32)
    at Object.Module._extensions..js (module.js:582:10)
    at Module.load (module.js:490:32)
    at tryModuleLoad (module.js:449:12)
    at Function.Module._load (module.js:441:3)
    at Module.require (module.js:500:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/nicholas/git/github/ngrx/effects/node_modules/@angular/compiler-cli/src/reflector_host.js:13:33)
    at Module._compile (module.js:573:32)

I don't particularly want to go down a rabbit hole, investigating this. Any suggestions as to what needs to be done? I've noticed that the CI builds appear to be failing, too - although, much earlier in the build process.

Please check this against NgRx v4, and if it’s still an issue, please reopen on https://github.com/ngrx/platform. Thanks!


Unable to create cluster if Service-Linked Role already exists

My account already has another Elasticsearch cluster. When applying my project with your module, Terraform reports:
module.elasticsearch.aws_iam_service_linked_role.default: 1 error(s) occurred:
aws_iam_service_linked_role.default: Error creating service-linked role with name es.amazonaws.com: InvalidInput: Service role name AWSServiceRoleForAmazonElasticsearchService has been taken in this account, please try a different suffix.

@johnowennixon thanks for testing!
we'll look how to fix it

@johnowennixon
In the latest release
https://github.com/cloudposse/terraform-aws-elasticsearch/pull/6
we added
variable "create_iam_service_linked_role" {
  type        = "string"
  default     = "true"
  description = "Whether to create `AWSServiceRoleForAmazonElasticsearchService` service-linked role. Set it to `false` if you already have an ElasticSearch cluster created in the AWS account and AWSServiceRoleForAmazonElasticsearchService already exists. See https://github.com/terraform-providers/terraform-provider-aws/issues/5218 for more info"
}

which could be set to "false" to disable the service-linked role creation.
It's probably the easiest way to do it instead of trying to check if the role already exists using TF.
Let us know if it works for you.

Unless I've got the wrong end of the stick this breaks the TF model in that I the TF author have to decide true or false for this variable depending on the state of the target account, whereas the promise of TF is that it will make these decisions at runtime, no?

yeah so for example if you have the service role already created in your dev environment but not in your stg environment.
you need to set it to true or false depending on the env.

Another option would be providing custom_suffix, as stated in the official terraform docs - https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/iam_service_linked_role#argument-reference
If we were able to provide iam_service_linked_role_custom_suffix that would solve a lot of problems. I'll create a PR, and hopefully it will be merged

Does setting "create_iam_service_linked_role" variable solve this issue? I tried doing this and it did not work and eventually had to set the count to 0 manually depending upon the env i am in.
Do the 9 downvotes on the solution denote that it is not working or that is not a good solution?

There is NO way to determine in Terraform if the Service Linked Role is already created on AWS in the account.
So yes, you have to check it manually, and if it does exist, set create_iam_service_linked_role to false.
Otherwise, set it to true.
We have not found a better way of doing this.
If somebody has a better option please chime in.

Thanks forasking for options. I have an idea :)

There is NO way to determine in Terraform if the Service Linked Role is already created on AWS in the account.

The name and ARN of the role are always the same. This could be used to determine if the role exists.

@aknysh Thanks for asking for options. I have an idea :)

There is NO way to determine in Terraform if the Service Linked Role is already created on AWS in the account.

The name and ARN of the role are always the same. This could be used to determine if the role exists.

@aknysh Thanks for asking for options. I have an idea :)

There is NO way to determine in Terraform if the Service Linked Role is already created on AWS in the account.

The name and ARN of the role are always the same. This could be used to determine if the role exists.

Has anyone done this yet ? Would like to do this directly in cloudformation seems to be much nicer then adding a boolean value to make the creation optional.

@simonlewandowski how do you define depends_on = [...] between aws_elasticsearch_domain and aws_iam_service_linked_role in your case?

It's probably best to keep the new variable false and manage the service role creation from its own root terraform module (component)
See https://github.com/cloudposse/terraform-aws-components/tree/main/modules/iam-service-linked-roles

@nitrocode I like the idea of removing the dependency on the service linked role from OpenSearch deployment by using the module, however when doing a quick test using just the resource it fails if the service linked role has already been created.  Perhaps I've misunderstood what you were suggesting?
Error:
creating IAM Service Linked Role (opensearchservice.amazonaws.com): InvalidInput: Service role name AWSServiceRoleForAmazonOpenSearchService has been taken in this account, please try a different suffix.

NOTE: Opensearch service linked role doesn't support custom suffix.
The tactical fix I've used is:
#main.tf
module "opensearch" {
...
...
local.create_opensearch_service_linked_role[var.environment]

#locals.tf
locals {
  create_opensearch_service_linked_role = {
    prod    = "true" #AWS prod account
    qa      = "true" #AWS non-prod account
    uat     = "false" #AWS non-prod account
    test    = "false" #AWS non-prod account
    dev2    = "false" #AWS non-prod account
    dev     = "false" #AWS non-prod account
  }
}

This isn't ideal.  Do you know if its possible to raise a request with the AWS team to enable custom suffix on the OpenSearch service linked role and if its something they would consider fixing?  Thanks.

The service role has to be created once per account.
If you instantiate this module more than once and it tries to create the same service role, it will fail. For this reason, is why I suggest setting create_iam_service_linked_role = false and creating the service linked role separately.
Contrived example of separating the
# components/terraform/service-linked-roles
# In a separate root dir or component
resource "aws_iam_service_linked_role" "default" {
  aws_service_name = "es.amazonaws.com"
  description      = "${local.service_linked_role_name} Service-Linked Role"
}

# components/terraform/elasticsearch
# In a separate root dir or component
module "elasticsearch" {
  source = "cloudposse/elasticsearch/aws"
  # Cloud Posse recommends pinning every module to a specific version
  # version     = "x.x.x"

  for_each = toset([
    "es1",
    "es2",
    "es3",
  ])

  name = each.key

  # set to false since this is already created outside of the module
  create_iam_service_linked_role = false

  # ...
}


Is it possible to show weekly data, depending on daily data ?

Good day,
I have chart with daily data.
Is it possible to make it display weekly data depending on daily data i have ?
Thanks for your time!

To achieve this, you will need to handle the data aggregation on your end. The Lightweight Charts library (LWC) is designed to display the data as provided and does not perform any internal data aggregation, such as converting daily data into weekly data. You will need to preprocess your daily data into the desired weekly format before passing it to the chart.

thanks for the clear answer, have a good day!


add lsp4s

sequel to #740, which added all of lsp4s's dependencies
this is now blocked waiting for @olafurpg to upgrade lsp4s itself to upgrade to monix 3 (
https://github.com/scalameta/lsp4s/issues/13)

@olafurpg you should wait for monix 3.0.0-RC2 (or, if it comes to that, 3.0 final); see https://github.com/scala/community-builds/pull/787#issuecomment-421393741

I'm working on it, sorry for the delay.

FYI @SethTisue I plan to use lsp4j instead of lsp4s in metals and it seems intellij-scala will do the same. The only remaining user of lsp4s that I'm aware of is bloop, I leave it to you to decide if you want to keep the module in the community build.

FYI @SethTisue I plan to use lsp4j instead of lsp4s in metals and it seems intellij-scala will do the same. The only remaining user of lsp4s that I'm aware of is bloop, I leave it to you to decide if you want to keep the module in the community build.

@jvican opinion?

I don’t plan on doing maintenance of lsp4s that doesn’t affect bloop, and bloop is still stuck on monix 2.x and will be for a long time... I think the best solution is to remove lsp4s from the community build.


"Medical crops"

Sativa

Columbian
Sour Diesel
Nepalese
Thai
Super Silver Haze
Oaxacan

Indica

Acapulco Gold
Purple Kush
Afghan Kush
Bubba Kush
La Confidential
Northern Lights

Hybrid

Durban
Hindu Kush
Blue Dream
Maui
Golden Goat
White Widow
Pineapple Express

Ruderalis

Skunk
Do-Si-Dos
Membrana Hiper Autoflowering
Auto Mazar

nah. wont do this...


Ram savings

@anttiylitokola @teetak01 : Can you run some corner cases on this PR ? Especially Device object related ones, which we fixed last week.

rebuild

Device Object, Observation and some other basic connector-cases at least work fine.

@anttiylitokola : I just started a e2e-job for this but could you sanity-check the two new commits too.


DropBox Unicode configuration section

This PR adds a short section to the docs advising on how to customise the PYthon configuration to deal with Unicode charactewrs in path and file names.
See: https://trello.com/c/lFY8DI8Z/20-unicode-in-upgrade

Staged at http://www.openmicroscopy.org/site/support/omero5.2-staging/sysadmins/dropbox.html#unicode-path-and-file-names.

The text and HTML page look fine and good to merge.
The only question I have here is why this isn't the default behaviour?  It seems like having it able to process Unicode filenames is an expected default, not a special-case customisation.

@rleigh-dundee It isn't the default because @joshmoore and I battled long and hard trying various solutions without success. This local customisation was the only thing that worked without breaking other things. An eventual move to Python 3 would remove the problem but for now this seemed to be the best fix without major chnages to the code.

@joshmoore are you okay with this being merged?

Happy for it to be merged. Thinking back, I already can't remember the various conditions under which it happens/happened. If you remember something like "when locale=X,..." that may be useful to include so that users don't assume they always have to make this setting.

@joshmoore I've been unable to find anything obvious in the various forum threads and cards that might help here.

Fair enough.


Refactor ComplexMethod

Now considers

&&, || and ?: everywhere
break, continue (throw is explicitly not counted like in ndepend)
functions used to replace if or for are now configurable
threshold increased to 15 like ndepend does - also linked their page

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@584fe66). Click here to learn what that means.
The diff coverage is 94.11%.

@@            Coverage Diff            @@
##             master    #2090   +/-   ##
=========================================
  Coverage          ?   80.97%
  Complexity        ?     2020
=========================================
  Files             ?      336
  Lines             ?     5813
  Branches          ?     1058
=========================================
  Hits              ?     4707
  Misses            ?      546
  Partials          ?      560

Impacted Files
Coverage Δ
Complexity Δ

...tekt/core/processors/ProjectComplexityProcessor.kt
100% <100%> (ø)
3 <0> (?)

...rturbosch/detekt/rules/complexity/ComplexMethod.kt
94.44% <92.59%> (ø)
16 <10> (?)

...rbosch/detekt/api/internal/CyclomaticComplexity.kt
94.54% <94.54%> (ø)
23 <23> (?)

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 584fe66...ad97b59. Read the comment docs.

I particularly like the config implementation for the CyclomaticComplexity class.
TIL that you can now use Kotlin functions inside Spec's context() functions. Cool!
As I commented, the ComplexMethod case file is very hard to understand and maintain.
Hopefully, we can also approach at some point in the future.
For now, this PR looks very fine!

Thanks! I've refactored some test cases and removed that ComplexMethod case file :)


Kubernetes debugging to other containers in different namespaces in same pod

I have 1 pod running with 2 namespaces. I want to debug container 1 which calls container 2. I to a post like this, but the request times out. Is this a limitation of kubernetes debugging or do I need to use something else? ip's or cluster ip do also time out.
http://.<namspace container 2>.svc.cluster.local:8080
[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: baf294f2-e31d-f580-e16f-64a26221dfcf
Version Independent ID: 3030e5bf-42be-82ae-2826-1869d7cd7ef6
Content: How Bridge to Kubernetes works - Visual Studio
Content Source: docs/containers/overview-bridge-to-kubernetes.md
Product: visual-studio-windows
Technology: vs-azure
GitHub Login: @ghogen
Microsoft Alias: ghogen

@V4A001 -- thank you for your question.  Please consider these resources, and let me know if they help answer your question:

Visual Studio on Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

Not really. I think it is not possible, but documentation is not very clear: "Bridge to Kubernetes lets you work in isolation from colleagues using the same cluster and namespace by leveraging our new routing technology."
https://devblogs.microsoft.com/visualstudio/bridge-to-kubernetes-ga/


With dgraph

I wonder if this helps when working with dgraph ?

I'm not sure, what are you picturing?
I am thinking about schema stitching, which might let you attach parts of a dgraph (or grpc or twirp) to parts of your local app data.

I see your point.
Funnily enough the query engine fro dgraph is called "CQL" like your
library is named.
Its really an awesome project.
I will try it out with dgraph !! They have a none standard graphql
implemenation:
https://github.com/dgraph-io/dgraph/issues/933
will let you knwo..
On Mon, 12 Feb 2018 at 23:44 Adam Scarr<EMAIL_ADDRESS>wrote:

I'm not sure, what are you picturing?
I am thinking about schema stitching, which might let you attach parts of
a dgraph (or grpc or twirp) to parts of your local app data.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vektah/gqlgen/issues/3#issuecomment-365088978, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ATuCwtM095FNTO401RbakAGq7c27aVd7ks5tUL7cgaJpZM4SCYss
.


Make git diff prettier

Involves making a custom difftool

Depends on #46

Took a first cut at this. Here's a sample for some feedback:

@craffel I think you said this depends on #46 because we want the diff tool to give some idea of what intermediate changes were applied between two versions of a checkpoint (e.g., between commit A and commit B there were 2 sparse updates and one dense update). As per my current understanding this will not be possible with the information that git provides external diff tools. The interface git enforces for diff tools is that they take the following parameters: diff-tool path old-file old-hash old-mode new-file new-hash new-mode. Since there's no information about what commit the two versions come from, we won't be able to display the intermediate changes applied.

I think with the way I setup the parameter updates on the FS we can actually figure out the updates applied.
Basically each update to a parameter is stored in a sub-dir that is named with the hash of the parameters for that update. So if we follow to metadata backpointers from .../params/updates/${new-file["${param_name}"]["hash"]} (or the parameter metadata file which I think has dup information rn) until we hit old-file["${param_name}"]["hash"] we should be able to tell what updates were applied.
I think this is dependent on the diff direction through? Like we need the on-disk .git_theta/... dir to be the one for the most recent model. If the on-disk value is older commit we can't do this.
A demo FS dir can be found here https://github.com/r-three/git-theta/blob/3e8566a83b6ef1e5cc5a24dc41ad7e0d73b8a0d3/git_theta/updates/base.py


Update README.md

General formatting and text updates to overall document.
Verified and updated links to remote jobs.
Removed companies that are no longer around or have been acquired.

See above


[#25] Adding GainNode

I added a minimal working solution for a gainNode. Tested on Google Chrome 56.0.2924.87 (64-bit) / Ubuntu 16.10. See #25

[ ] Make gainNode configurable
[ ] Provide a test
[ ] Test on more browsers

You probably need some getter to access to that gainNode and make some settings? Or not?

Close in favor of #32


404 on Nova Routes after integration

Hi, i followed the doecumentation on integration with nova, but after that, the nova routes return 404, am i doing something wrong?
thanks
Denis

Are you accessing Nova on a central domain or a tenant domain?

i acces them on the central domain, i added my domain to the tenancy.exempt_domains config.
i am working on laragon, could be this the issue?

So do you want to use Nova to manage tenants or to manage resources inside tenant databases?
The Nova integration guide in the docs states that it's for using Nova inside the tenant app.

oh ok, i misunderstood this..
my app uses nova as a backend for tenants.. so i need it for both tenant and also to manage tenants.. i should use universal route right?

You can't use Nova to manage tenants in v2 -- you can do that in v3 which is coming soon.
And I don't think you can use Nova inside the tenant app and to manage tenants at the same time. Or you could, but it would be terribly hacky. Different resources.

sorry, than i misunderstood the whole concept, haha.. have to read about it again, not a native speaker so i have many language barriers..
my usecase was:
i have exisiting app made with nova as backend which i want to offer as saas.
so i need problably a clean installation of laravel with whatever backend to manage the tenants separately with all the subscription forms etc. to work as a central app.
from this central app i create the tenants with configured migrations.. sorry for the stupid question as i am not experienced, but do i have to copy the models, controllers etc from my existing app to the new tenant folders or how does it work?
or maybe i will wait for the v3 to have a look into the  example SaaS application :S

i have exisiting app made with nova as backend which i want to offer as saas.
so i need problably a clean installation of laravel with whatever backend to manage the tenants separately with all the subscription forms etc. to work as a central app.

That's correct. Follow the Nova integration guide, but use something else for managing tenants.

but do i have to copy the models, controllers etc from my existing app to the new tenant folders

No, you don't have to move anything

or how does it work?

Like this: https://tenancyforlaravel.com/docs/v2/getting-started/#how-does-it-work

ok, thank you!


Create example project

Because the documentation of this project doesn't use the curriculum vitae theme, there must be an example project, showing what can be done with the theme. This example project's link must be here in the documentation.

Done in https://github.com/Paul-Riviere/mkdocs-curriculum-vitae/issues/33


fix(client): remove multiple redirects after account deletion

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #39508

Nice work @Prathamesh010, on fixing this long pending issue - I will wait on someone from @freeCodeCamp/dev-team to confirm and merge this.

Thank you for your contribution to the page! 👍
We are happy to accept these changes and look forward to future contributions. 🎉


solder ball height bug fix

@SMoraisAnsys

@SMoraisAnsys

@ring630 I suppose you pinged me to help me track the changes in EDB ? Thanks if that's the case ! Btw, good to see that github actions added the right label 👍


Poem w/ Tonic doesn't return gRPC status header

Expected Behavior
As I am exposing a grpc API through Poem, I should be able to call it correctly with a tool like BloomRPC and make a quick test of the API, the tonic example is good as a reference for this ticket.
Actual Behavior
BloomRPC doesn't know the status of the response, and can't establish if the request has been properly handled, when debugging with the client code from poem examples folder, I see that the Poem's version of the API doesn't return a  grpc-status  header that should returned.
Steps to Reproduce the Problem

Clone the tonic example.
Launch the server with cargo run --bin example-tonic
Launch the client with cargo run --bin example-tonic-client
Launch BloomRPC, add the proto from the example folder, update the address to the localhost:3000 that has been set in the server config.

Specifications

Version: Any version
Platform: Ubuntu 20.04
Subsystem: /

Really fixed this time. 😁

Works perfectly! Thank you very much! :clap:


Unknown error: BAD_SESSION / Failure in verify PDPKA5

have issue login.
useing email getting Unknown error: BAD_SESSION / Failure in verify PDPKA5
useing username it fail login just open my profile
pfp is blanked out.
still have accese veiew and download files on my mac but nothing else works.
all devices says revoked while never revoked them.

Same here, I assume keybase services are dying and no one is monitoring it anymore.


Order Change not working properly

Autoskill_List[3][1] = "Saber Rider Embers" Autoskill_List[3][2] = "hi5,#,x21cg1def4,#,5"
When I tried to order change, it failed because the cursor wasn't able to click the ok button in the confirmation window (despite having this variable set: Skill_Confirmation = 1). Other master skills work fine, but the order change just won't work. Any ideas why?

I think that order change doesn't consider the situation that Skill_confirmation = 1.
Maybe you can try Skill_Confirmation = 0 for order change.

The problem is that if I set Skill_Confirmation = 0, my Servant skills would then not be working properly. Not sure if Order Change would work since I set it at Turn 2, as the script already fails at Turn 1 without activating my Servant skills (with Skill_Confirmation = 0).
Well, even if Order Change does work with Skill_Confirmation = 0, I think it just creates another problem, which is not activating my Servants' skills properly.

You haven't disable skill confirm in game option. The option is inside "munu".
When you disable it, the Skill Use dialog won't pop-up.

The problem is that if I set Skill_Confirmation = 0, my Servant skills would then not be working properly. Not sure if Order Change would work since I set it at Turn 2, as the script already fails at Turn 1 without activating my Servant skills (with Skill_Confirmation = 0).
Well, even if Order Change does work with Skill_Confirmation = 0, I think it just creates another problem, which is not activating my Servants' skills properly.

I am not sure what is "skill confirm" called in EN server, but you can find similar option in "menu".

Which file is this "menu" located at? I cannot seem to find the option in FGO_EN_REGULAR.lua, and I can't seem to find a filed named "menu.lua" either.

Its not a file.
It is an option of FGO game.
There is a "MENU" button and a "SKILL" button  in the right side of battle screen.
You can see them in your link..

Yep I found it. And it works now with Skill_Confirmation = 0. I think there should be a disclaimer in the README that Order Change only works with Skill_Confirmation = 0 to avoid confusion. Thanks a lot by the way!

You can manually fix this by editing the autoskill.lua file in the modules folder.
Change this:
local function BeginOrderChange()
	return function()
		OpenMasterSkillMenu()
		click(game.BATTLE_MASTER_SKILL_3_CLICK)
		wait(0.3)

Into this:
local function BeginOrderChange()
	return function()
		OpenMasterSkillMenu()
		click(game.BATTLE_MASTER_SKILL_3_CLICK)
		if Skill_Confirmation == 1 then
			click(game.BATTLE_SKILL_OK_CLICK)
		end
		wait(0.3)

(Preferrably on a computer, cuz formatting is a nightmare on mobile)


Add vendor config support for Burt analytics

This PR adds support for Burt analytics as a vendor. The purpose of this PR is to have an internal discussion before sending it upstream.
There are currently two types of requests: pageviews and pagepings. They are not identical to Agency's page and pageping fragments as I will explain.
Usage
Using Burt as a provider is straight forward:
<amp-analytics type="burt">
  <script type="application/json">
    {
      "vars": {
        "entity": "B31VT4QH1BFR", // Required
        "category": "sports",
        "subCategory": "golf"
      }
    }
  </script>
</amp-analytics>

The entity variable
The variable entity is available for configuration. However, the customer might not specify this variable (by mistake). The way I've solved this here is to use ignore as default value, hence the requests will be sent to ignore.c.richmetrics.com.
QUESTION: Would it be possible to configure this so that all requests sent to this host will be ignored?
If not possible, we could solve this using a host variable that the customer must specify. Unless specified, the requests will fail and never leave the browser.
Parameters
These parameters are sent on each fragment.
e - Entity ID
pi - Pageview ID
av - Agent version (will always have value amp)
ct - Client timestamp
sn - Sequence/Serial number
These parameters are sent on the pageview fragment:
type - Fragment type (will always have value page)
ca - Category (if specified)
sc - Sub-category (if specified)
ln - Language
lr - Last referrer
eu - Embed URL
tz - Timezone
ui - User ID
pd - Page Dimensions
sd - Screen Dimensions
wd - Window Dimensions
ws - Window Scroll
These parameters are sent on the pageping fragment:
type - Fragment type (will always have value pageping)
The definitions of the parameters in AMP are not always identical to those in Agency with the same name. This is because we don't have the same control here, hence cannot be changed.
1. pi - Pageview ID
The format of the AMP pageview id is obviously not the same (example 398). The definition differs as well.
"likely to be unique per URL, user and day"
We would somehow have to make a new id in the backend, which is a combination of the pageview id, url, user and date.
2. ui - User ID
The format of the user id is quite different from Agency (example U6XEpUs3yaeQyR2DKATQH1pTZ6kg140fvuLbtl5nynbUWtIodJxP5TEIYBic4qcV).
The definition is a bit different as well.
"The client id will be the same for the same user if they visit again within one year"
Agency does not set an expires option, hence it might be around for longer than a year.
3. ct - Client Timestamp
In Agency, the ct parameter is the number of milliseconds that has gone since the Agent started. All we have available here is the timestamp (eg.<PHONE_NUMBER>278)
4. ln - Language
AMP downcases the ln parameter value, for example en-US => en-us.
5. wd - Window Dimension
The wd parameter in Agency will not include width/height for object such as the Chrome developer tool, but AMP ignores this. That means, if the visitor has a dev-panel, extension, etc... that takes up screen-space, the window dimensions will be larger in AMP than what they are in Agency.
Pagepings
The definition of a pageping is much more complex in Agency. The pagepings here will be sent every five seconds and will continue to ping for 1200 seconds (both these are options and can be changed to whatever we find appropriate).
Another quirk is that a pageping will be sent directly on page load and not after five seconds. We can unfortunately not control this.
There might be other things that are different that I have yet to realize, but the ones above should be the most important.

Please note that this conversation will be public since this repo is public (forks of public repos can't be private).


support the enrichment of ConfigMaps to include text files in the data entries

here's an example of where we embed an application.properties file inside a ConfigMap. We may want to maintain the contents separately - so its easier to test the file for example (e.g. parse it or use it in a junit test case).
So maybe we could replace this with something like:
metadata:
  annotations:
    file-import.maven.fabric8.io/application.properties: ${basedir}/target/test-classes/application.properties

which would load the given key application.properties from the given local file system file and import it into the generated YAML like this https://github.com/fabric8-quickstarts/spring-boot-webmvc/blob/master/src/main/fabric8/configmap.yml#L4-L7

Agree that this makes much sense to prepare configmaps and secrets (see also #975 ) from files. Instead of using annotations for fragments though (which would have no meaning during runtime), I'd prefer to add an XML configuration for preparing such cm / secrets. It would make the definition of such imports more concise:
<configuration>
   <resources>
      <configmaps>
          <cm>
               <!-- Plain entry -->
               <entry>
                    <name>myKey</name>
                     <value>myValue</value>
               </entry>
              <!-- Import with name -->
              <entry>
                    <name>config</name>
                    <file>${basedir}/target/test-classes/application.properties</file>
               </entry>
               <!-- Would import with the filename as key. Or just use an <entry> without name -->
              <import>${basedir}/target/test-classes/application.properties</import>
               .....
          <cm>
      <configmaps>
   </resources>
</configuration>

If we would go for support for yaml annotation, I would like then to remove the annotation befire adding it to kubernetes.yml (as it exposed build details which are of no use and maybe even unwanted during deployment time)

Am happy with the XML and/or annotation approach. I guess the XML approach may be easier to tool one day (IDEs might be smart enough to grok  is-a file etc).
Agreed that if we did go with annotations we should remove the file-import.maven.fabric8.io/* annotation during enrichment.

Agree that the XML configuration is convenient.  But yaml configuration is the standard way to use kubernetes. If the maven tool moves the configuration way to XML from yaml, and builds the yaml fragment when compiling, it is difficult for user migrating to this maven plugin with former kubernetes configuration or background knowledge.
Is there any way to replace the yaml annotation from maven settings file?
BTW, I notice the issue #11 , which requests for replacing the yaml annotation with environment variables, is held on more than one year. What is the concern for holding this issue? I ask this question since I think replace yaml annotations with docker secret is similar with environment.

We can use maven properties in the YAML file fragments which we can then replace (e.g. using the maven resource filtering); so that we can refer to docker image versions or whatnot using maven properties.
When it comes to Secrets I think those should mostly be maintained out of band from applications. i.e. the aim of the fabric8-maven-plugin is to generate an application YAML which can be versioned, shared in public maven repositories or used in Helm chart repositories and then installed on any cluster or namespace; and so have no environment specific data such as Secrets or PVs inside which are highly specific to a particular cluster / namespace / environment installation.
I can see folks wanting to use a tool like fabric8-maven-plugin to help generate, enrich, combine, process resources to make YAML - but I think Secrets should be handled completely different to the current way we handle resources in fabric8-maven-plugin. e.g. they should always just be generated into, say, target/fabric8/secrets and never included in the jars or published to maven repos or included in Helm charts.

Agree that the XML configuration is convenient. But yaml configuration is the standard way to use kubernetes. If the maven tool moves the configuration way to XML from yaml, and builds the yaml fragment when compiling, it is difficult for user migrating to this maven plugin with former kubernetes configuration or background knowledge.

Well, the extension proposed is by no means a k8s standard (although sharing the same format), so it might make even more difficult to understand what's k8s and what's fmp specific if intermixing features freely.  And btw, you can use both together, the XML config syntax + the Yaml fragmetns. They get simply merged together.
Said that, I think it makes sense to support both.

@jstrachan I agree that Secrets are a runtime aspect much like ImageStreams and must not be distributed within the artefact jar (and via Maven repositories)
wrt to secrets, I think using ~/.m2/settings.xml (see #975)  and applying them on the fly for a fabric8:apply or even with a fabric8:secrets (without generating it into the kubernetes.yaml) might help, if fmp is used as a deployment tool (e.g. for CI / CD) ?

@yuwzho wrt #11 : This issue pre dates the fragment support and was replaced by fragments with enrichers as we have now. tbh I think that #11 can be closed.
As @jstrachan says, you already can use Maven properties within the fragments, which can be also environment variables with the syntax ${env.MY_ENV}. This will be resolved when you create the descriptors, so not sure whether this is of much use for the use case at hand.

@rhuss agreed - we may want to generate and use Secrets in a separate folder (e.g. target/fabric8/secrets and use them when using mvn fabric8:deploy)

@rhuss @jstrachan I get your point. So for the secret things, we can do the following steps. Is that correct?

User put the yaml fragment into src/main/fabric8/secrets folder as this:

data:
    .dockerjsonconfig: {docker.serverid}
metadata:
    name: dockerregistrykey

Has a goal mvn fabric8:secret to generate a yaml secret fragment in target/fabric8/secrets

When call mvn fabric8:apply, also load the yaml file in target/fabric8/secrets folder

Something like this or also the XML configuartion as sketched out in https://github.com/fabric8io/fabric8-maven-plugin/issues/977#issuecomment-314037889.
For the servier id a regular Maven property like ${fabric8.server-config.myServerId} could be used and all such properties are either created and set brute force from settings.xml before the frament is processed or we might require an intermediate configuration step in pom.xml to declare only the properties which need to be pre-populated.

@rhuss For the XML configuration, of course user can refer the secret in XML, and then read in the maven parameters can easily generate the yaml file.
But where is the metadata thing come from? User should specific the name property and whatever properties they like in the metadata.
Need user put these things like XML configuration or just a yaml fragment, and the code merges yaml and XML. If do the merge thing, how to specific which XML segment merge with which yaml fragment?

I want to split the topic in this thread since we already talked about config-map, secret.generic and secret.docker-registry things. And all of them we talk about XML configuration and yaml fragment
But the XML configuration for secret thing is still need to be discussed as I mentioned in last comment.

Config Map
Secret generic
Secret docker registry

Yaml
Write the value into property file and refer the file in yaml annotation. As this issue

similar with docker registry, but the yaml fragment as

data:
    password: ${secret.password}
metadata:
    name: secretkey

But which section in maven setting file to save these custom secret key-value pair.

Write the secret things into maven settings file, and compile it in target place. As this comment

XML
Configure with pom.xml and property file. As this comment
TBD
TBD

Hi @jstrachan and @rhuss , since we have a project is so urgent need the docker-registry secret. We would like to contribute the docker-registry secret feature with both Yaml and XML implementation (The last column in the above table).

@yuwzho for the secret could we do it similar like for the ConfigMap with an annotation ? E.g. use a custom annotation instead of a 'magic' property (which would not play nicely with properties replaced by the resource maven plugin) ? This annotation would then be processed, removed and add the proper secret from the registry.
The implementation for the evaluation of annotations in the fragments and the custom processing could probably already easily done by a dedicated Enricher (who has access to the settings.xml, I believe), so that this feature can be easily removed/added or put into profiles like the other enrichers we are using.
wdyt?
we are totally open to pull request and would integrate them (said that, I'm at the moment a quite disconnected from f-m-p development as I'm working in a totally different project right now). But its not only me ;-)

I've seen that this support for external data for config maps and secrets has been added via #1013, so I think we should do something similar for ConfigMaps, too.


Update cosmos_client.py

Fixing request_timeout comment, from miliseconds to seconds.

/azp run python - cosmos - ci

Hi @Rodrigossz , it seems this is an old PR with conflicts that are important :(. I'm clsoing feel free to re-open a new PR once you have solved all conflicts. Thanks.


include profile in signIn / signOut / fetch response

If I want to get my profile data I have to do two calls
hoodie.account.signIn({username: 'foo', password: 'bar'})
.then(function () {
  return hoodie.account.profile.fetch()
})
.then(function () {
  // hoodie.account.profile.get() now returns locally cached profile data
})

Our JSON API supports the ?include parameter for all requests, so we could do two things
I would suggest to  include option to signIn / signOut methods (we already do this for some of the admin API methods). The fetch method would need to send the ?include=account.profile param if it was called with something like account.fetch('profile.settings')
With that change all profile properties would be cached offline after signIn, so we can do hoodie.account.profile.get() later any time, even without an internet connection.
hoodie.account.signIn({username: 'foo', password: 'bar', include: 'profile'})
.then(function () {
  // hoodie.account.profile.get() now returns locally cached profile data
})

As suggested in #111 I rather tend to always include the .profile property in all the things for sake of simplicity.


fix various deprecation warnings

A couple I'm seeing right now:
DEPRECATION WARNING: Static attributes will be removed in FactoryBot 5.0. Please use dynamic
attributes instead by wrapping the attribute value in a block:

password { "password_is_12345" }

To automatically update from static attributes to dynamic ones,
install rubocop-rspec and run:

rubocop \
  --require rubocop-rspec \
  --only FactoryBot/AttributeDefinedStatically \
  --auto-correct

 (called from block (2 levels) in <top (required)> at /home/ubuntu/newspaper_works/spec/factories/user.rb:5)

and
DEPRECATION WARNING: human_readable_type= is deprecated and will be removed from a future release (human_readable_type is deprecated. Set the i18n key for activefedora.models.#{model_name.i18n_key} instead. This will be removed in Hyrax 3). (called from <class:NewspaperTitle> at /home/ubuntu/newspaper_works/app/models/newspaper_title.rb:46) (1 times); e.g.:
    /home/ubuntu/newspaper_works/app/models/newspaper_title.rb:46:in `<class:NewspaperTitle>'
    /home/ubuntu/newspaper_works/app/models/newspaper_title.rb:2:in `<top (required)>'
    /var/lib/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:292:in `require'
    /var/lib/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:292:in `block in require'
    /var/lib/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:258:in `load_dependency'

Add more as comments on this issue as they pop up.

Fixed via #127


How to use with proxy?

Hello. I don't understand, will this software work through a proxy if a proxy is installed in my browser or do I need to modify it so that it works through a proxy? This means if you install this software on my local computer.

Please use VPN or a global proxy.


Set fixed_bottom_bar to true

The experiment was positive, probably because otherwise you can miss notifications. Let's hardcode for simplicity and to avoid having to think about different layouts.
Test Plan
Verify the bottom bar shows up on iOS, web, Android.

I think I found a small issue caused by this. O a Pack page the last item will be cut.
Eg: https://go.bsky.app/PMWf5Zw

Yeah, I believe this is already fixed on main — thanks.


Handle single string as path in find_file()

As title.
Fixes #3831

Looks good, pending successful CUDA smoketest

Pass test on build farm.  Ready to merge.


[labs/ssr] Access to slots during SSR

Description
Some components such as Shoelace's <sl-button> component differentiate rendering based on which slots are assigned.
When Lit renders to HTML it does not call the slotchange, which causes the <sl-button> to incorrectly render.
I think this might be a limitation of Lit's SSR implementation, which wishes to not implement a lot of DOM. Could there perhaps be an alternative API available only during SSR that a component could use to know which slots are assigned?
cc @claviska
Steps to Reproduce
See this bug: https://github.com/shoelace-style/shoelace/issues/476
Live Reproduction Link
The stackblitz does not do SSR.
Expected Results
Properly render.
Actual Results
Missing required attributes due to slotchange not firing.
Browsers Affected
Server rendering issue only.

The use of "slot detection" is not unique to Shoelace nor Lit components. I've seen this pattern in the wild a number of times, including in FAST components (code example).
Using attributes such as has-default-slot is a stopgap, albeit a messy one. As a library author, making this "just work" is the ideal solution since it would likely work for all WCs, but I'd also be OK with opting in to some kind of Lit-provided utility if necessary.

We're discussing this. The issue is that we want to avoid requiring a slow DOM emulation and/or preventing streaming. This looks like a pretty complicated situation.
I think there are few things to consider:

I looked at a few elements that do this and mainly they seem to set classes based on the existed (and type 😬) of slotted elements. If we look at rendering as UI = f(state), this is a gap in the declarative rendering that requires imperative code to derive state from slotted children.
We can try to allow the existing imperative code, or build a semi-declarative API that can be implemented in SSR code as well. The declarative form would not deal in DOM events and nodes, but would probably just provide info on the directly slotted elements tag names and static attributes.
We can test and document how to run lit-ssr with full DOM emulation like JSDOM and make a renderer that uses that to run a more complete element lifecycle. This would be much slower than the string renderer, so we'd want to be able to interleave string and DOM renderers to only run the DOM renderer for elements that need it. This would let us group elements into "fast mode" and "full DOM (slow) mode".
I do wonder if many of the cases that set class names can be ported to work purely in CSS. Could slots and fallback content by styled so that the absence of the fallback content triggers the correct styles? Could ::slotted() be used with positional selectors to add needed spacing to first and last elements? I think some cases would require :has() or a new pseudo-class for slots with slotted content, so I filed: https://github.com/WICG/webcomponents/issues/936

@justinfagnani Thanks! I think a CSS solution for the use-case would be best, but I don't know enough about it to say if it would be possible.
Otherwise I think a declarative solution here would be nice and a reasonable fallback, rather than adding a separate DOM requirement just for certain elements.

It would be extremely difficult for us to materialize slots during SSR, both because of the additional parsing and bookkeeping we'd have to do and because we stream out shadow root contents before the light DOM that would fill the slots.
Luckily, the :has-slotted pseudo-class proposal was approved and is being implemented as we speak!
Closing this issue on our end now.


How to turn off bold?

Great package! Usability question:
Say I've got:
print(style.white('•', style.bold.white(' [{}] {}'.format(name, style.dim.white('{}'.format(msg))))))

I want to replace 'dim' with just the normal, unbolded text... but I can't figure out how to do that. i.e., I want to turn off bold after using it in the same line.

Hi @crunchex, actually you ran into two issues with my package at the same time:

I have not added support for nesting styles using % or .format() yet.
There is no way to reset only the modifier style. A possible workaround would be to use style.default or style.reset which will reset (if nested) all styles of the outer style .

For now the best solution for your problem would be:
print(style.white('•', style.bold('  ['), name, style.bold('] '), msg, sep=''))

or simply
print('•', style.bold('  ['), name, style.bold('] '), msg, sep='')

if white is your terminals default.


Remove _dsfid fields from _attribute_source tables

These columns were added as a part of the initial feature source tracking but as of 03152022 updates they are no longer used. The datasource to featureid link is stored in the _feature_source table instead. We didn't remove these at the time but they should be removed eventually to prevent duplication and confusion.
See the end of cabd/cabd.updates.03152022.datasources.sql script for examples.

I can take care of this today before the DB migration - any concerns with that?

No concerns on my side. There is no rush to get it done either so if it makes sense for you to wait that's fine - I just didn't want it forgotten about.

Ok! This is all finished up - it's actually a great time to do this as we are working through some tasks from the backlog before the migration.


Replace QtWebEngine with QtWebEngineQuick

Description
Replace QtWebEngine with QtWebEngineQuick and fix up a few other misc. issues.
Type of change

[x] Bug fix
[ ] New sample implementation
[ ] Sample viewer enhancement
[ ] Other enhancement

Platforms tested on:

[x] Windows
[ ] Android
[ ] Linux
[ ] macOS
[ ] iOS

@JamesMBallard Thanks for reviewing. All those other changes are intentional, body of the PR mentions "fix up a few other misc. issues." but I noticed the PR title might be misleading. I will update that as well. Please review/approve if everything looks good


Large version only generate noise

Hi, i am trying the model out and using the inference code like exactly on huggingface. For the small version of the model, everything work fine. However, for the large one, it only generate noise. All my installed pacakages are latest

from another issue, these worked for me
model = ParlerTTSForConditionalGeneration.from_pretrained("parler-tts/parler-tts-large-v1", revision= "refs/pr/9").to(device)

Thank you for the comment. Let me try that revision and update

Thank you for the comment. Let me try that revision and update

Hi there,
Did you managed to make it work. I noticed that after I use the revision, but after 20 second, I noticed that the audio kept breaking a leaking into another language.