Add jammy content images to the build matrix

Modify maybe_install_quarto.sh to only install quarto for 1 version of bionic.
Install latest RC quarto (as of 2023-04-18) for all jammy images.

I did make a PR for this same thing a while ago (it was probably missed since it's still in draft mode): #492
I think we may want to use that one instead and reconcile the two. It has a few more changes to upgrade EoL'd Python and R versions and makes some more updates to make the content images to bring them up to speed with our other images.

@ianpittwood that sounds fine with me, is there anything we can do to help move your PR along? We would like to start using the jammy content images soon

@dbkegley sure, I'm actually looking for some feedback on the R and Python versions used in the matrix.json file: https://github.com/rstudio/rstudio-docker-products/pull/492/files#diff-b96a499343e3b0a7c271edac8badf6d2d87ea589a25c08683536ee0296d165e8
I'm just looking for feedback on whether there's any version combinations we're missing and whether any of the ones removed are crucial in any way. I want to clean up the EoL'd versions to reduce build times and ensure we're using secure versions of R and Python, but I don't want to do that at anyone's expense.
I'll work on getting the maybe_quarto_install.sh script changes implemented in my PR and then once I get the build issues fixed I'll publish it for review.


Unable to set ownership of files properly

Everytime i try to start my existing server, i get following error:
{"name":"wings","hostname":"srv01","pid":29632,"server":"2cb21377-797f-4f86-a704-b2d4793ba128","level":50,"err":{"message":"Unable to set ownership of files properly, exited with code 1 signal null.","name":"Error","stack":"Error: Unable to set ownership of files properly, exited with code 1 signal null.\n    at ChildProcess.Exec.on (/srv/daemon/src/controllers/fs.js:107:29)\n    at emitTwo (events.js:106:13)\n    at ChildProcess.emit (events.js:191:7)\n    at Process.ChildProcess._handle.onexit (internal/child_process.js:219:12)"},"msg":"{ errorId: 'MMA2bxurgCAbK4R9zlui' }","time":"2018-02-14T16:03:32.708Z","v":0}

Version of Panel/Daemon: LATEST (BOTH)
Server's OS: Ubuntu 16.04.3

Does /srv/daemon-data/2cb21377-797f-4f86-a704-b2d4793ba128 exist?

I dont use /srv/daemon-data, i use /srv/server:

i checked the core.json:

Theres something not using the given path ^^

Do userdel pterodactyl and restart the daemon letting it generate a new user.

Worked! thank you

Cool, there was a bit of a hiccup in the user creation stuff in the betas, so this tends to happen occasionally for people when updating. Should be all resolved now, let me know if it occurs again.


Why it required >9.0 deployment target

when I add
pod 'Moya-ObjectMapper/RxSwift', :git => 'https://github.com/ivanbruel/Moya-ObjectMapper' to my Podfile and install it return
Specs satisfying the Moya-ObjectMapper/RxSwift (from https://github.com/ivanbruel/Moya-ObjectMapper) dependency were found, but they required a higher minimum deployment target.
Is there any way I can set the deployment target 8.0 ?

I'm unsure why the podspec actually has over the top minimum deployment targets, let me look into it and get back to you.

Submitted a new version to cocoapods hopefully fixing this issue.
Let me know if that works out for you so I can close this issue.


feat: support for branches and content-types pagination.

This PR resolves CS-38229.

Hey @vkalta , Can you pls update the version ? version": "2.1.2 already released.
https://github.com/Contentstack-Solutions/contentstack-cli-tsgen/blob/master/package.json
cc: @netrajpatel


when reanalyzing, the analysis server should flush info from the .analysis_options file

When a client issues an analysis.reanalyze call, the analysis server should also flush info from the .analysis_options file. @Sfshaza ran into an issue where changing an analysis options file over to strong mode didn't take effect even after a reanalyze call (it was necessary to restart IntelliJ).

or use use the kill analyzer feature (no button anymore though)

Yeah, we would like analysis.reanalyze to work for this.

This turned out to be user error. Server correctly detects changes to the analysis options file and re-analyzes as appropriate.


Cell Outputs aren't actually included in the LLM request.

Here's a notebook visualizing the LLM requests for a lot of examples.
https://gist.github.com/jlewi/b9247059ebb3cb323eee10504ffc9f6c
It doesn't look like the cell output of previous cells is actually part of the LLM request. Rather, what we have is some RunMe metadata.
{
	"type": "stateful.runme/terminal",
	"output": {
		"runme.dev/id": "01J7Y831N6DC58716N7ZM5P2HC",
		"content": "",
		"initialRows": 10,
		"enableShareButton": true,
		"isAutoSaveEnabled": false,
		"isPlatformAuthEnabled": false,
		"isSessionOutputsEnabled": true,
		"backgroundTask": true,
		"nonInteractive": false,
		"interactive": true,
		"fontSize": 16,
		"fontFamily": "Menlo, Monaco, 'Courier New', monospace",
		"rows": 10,
		"cursorStyle": "bar",
		"cursorBlink": true,
		"cursorWidth": 1,
		"smoothScrollDuration": 0,
		"scrollback": 1000,
		"closeOnSuccess": true
	}
}

Here's my Runme doc looking at some sample data. It looks like in the case of a non-interactive cell the stdout is available in one of the cell output items. For interactive cells it doesn't look like the cell output is available in either of the two output items.
Non-interactive cells
It looks like the mime types of the output items are

stateful.runme/output-items

data is a base64 encoded JSON dictionary
This dictionary contains the field output which is itself a dictionary which contains a field "content" which contains the base64 encoded stdout   "type": "stateful.runme/output-items",
   "output": {
           "content": "VGhpcyBjZWxsIGlzIGV4ZWN1dGVkIGluIG5vbiBpbnRlcmFjdGl2ZSBtb2RlIFdlZCBPY3QgIDkgMjA6Mjg6NTUgUERUIDIwMjQNCg==",
           "mime": "text/plain",
           "id": "01J9T7H2DRRW89QJMQNV0SS3YM"
   }

application/vnd.code.notebook.stdout

data is the base64 version of the stdout

Interactive Cells
It looks like the mime types of the output items are

application/vnd.code.notebook.stdout

But there is no actual data

stateful.runme/terminal

Data is a base64 encoded json dictionary that doesn't have stdout or stderr

Questions
@sourishkrout
For non-interactive cells does it matter which output item we use to get the output from? i.e. whether we use the item with mime type application/vnd.code.notebook.stdout or the mime type stateful.runme/output-items? How would stderr end up being encoded?
For interactive cells how difficult would it be to fetch stdout and stderr and include that in the requests that get sent to Foyle?

Output types are defined here
https://github.com/stateful/vscode-runme/blob/3e36b16e3c41ad0fa38f0197f1713135e5edb27b/src/constants.ts#L6
There doesn't appear to be one for stdout.

On the Foyle side; we convert from Cell protos to Block protos here
https://github.com/jlewi/foyle/blob/5eafef9471846d647d6544b7d79a752223ba9c51/app/pkg/runme/converters/cells_to_blocks.go#L102
This should preserve the mime type and data contents.
It looks like if there are multiple output items all of them should be included when the block is converted to markdown.
https://github.com/jlewi/foyle/blob/5eafef9471846d647d6544b7d79a752223ba9c51/app/pkg/docs/converters.go#L34
So my suspicion is that the prompt does get included in non-interactive cells but not interactive cells but most cells are interactive as that appears to be the default.

So I tested it with a non-interactive cell. The command in the cell was
echo "The current time is $(date)."

If you execute it and then generate a completion the markdown that gets sent to the LLM is
echo "The current time is $(date)."
{
	"type": "stateful.runme/output-items",
	"output": {
		"content": "VGhlIGN1cnJlbnQgdGltZSBpcyBUaHUgT2N0IDEwIDE2OjQxOjQ3IFBEVCAyMDI0Lg0K",
		"mime": "text/plain",
		"id": "01J9WCSV2CAQVERX6JT7T8DZ5G"
	}
}
The current time is Thu Oct 10 16:41:47 PDT 2024.

So the output is included. Although it looks like we might want to filter out the mime type stateful.runme/output-items.
What to do about interactive cells?
I suspect there is a bug in how I'm serializing the notebook in the vscode-runme frontend before sending it over the wire to Foyle. The code does this by calling marshalNotebook
https://github.com/stateful/vscode-runme/blob/3e36b16e3c41ad0fa38f0197f1713135e5edb27b/src/extension/ai/generate.ts#L46
https://github.com/stateful/vscode-runme/blob/3e36b16e3c41ad0fa38f0197f1713135e5edb27b/src/extension/ai/ghost.ts#L111
@sourishkrout Am I missing a call to addExecInfo before serializing the notebook? It seems like that function might be synchronizing information from the terminal to the notebookData structure?

This likely relates to how the notebook state is snapshotted in the AI parts of the extension. I'll look into the details next week. We've recently done some work to "centralize" this so consumers other than the actual notebook serialization can use it. However, I have to make sure it supports the use case.
For non-interactive cells, this is also associated with https://github.com/jlewi/foyle/issues/143.

Questions
@sourishkrout
For non-interactive cells does it matter which output item we use to get the output from? i.e. whether we use the item with mime type application/vnd.code.notebook.stdout or the mime type stateful.runme/output-items? How would stderr end up being encoded?
For interactive cells how difficult would it be to fetch stdout and stderr and include that in the requests that get sent to Foyle?

So for non-interactive cells, please use application/vnd.code.notebook.stdout. The reason we have stateful.runme/output-items and some form of duplication is that VS Code's notebook APIs tie web-components tasked with rendering the UI to a media-type and to have the custom UI/UX such as action buttons (Copy, Save, etc) available we are using stateful.runme/output-items.
Re, stderr, I've not been happy with how we're handling stdout vs stderr. I have a refactor on my list that will only send stdout into the notebook (primarily because third-party rich renderers are stdio-unaware) and, unless otherwise configured on the cell or doc, will omit stderr in the notebook-integrated terminal. However, with the same refactor, we'll introduce a multiplexed panel-based terminal that will receive both stdout and stderr. The idea is to enable the UI/UX to highlight the panel-based terminal for non-zero exits, either auto-focusing the terminal or providing a button to call the user's attention to the error.
I am continuing with interactive exec in a separate comment/analysis.

Interactive Cells
It looks like the mime types of the output items are

application/vnd.code.notebook.stdout

But there is no actual data

stateful.runme/terminal

Data is a base64 encoded json dictionary that doesn't have stdout or stderr

Similarly for interactive execution, stateful.runme/terminal renders a web-component with an instance of xterm.js that's configured according to the payload. Studio is bidirectionally streamed directly between kernel <> terminal web-component because the notebook APIs are not capable of terminal emulation, and replacing outputs too frequently causes side-effects that make the notebook unusable (flickering, excessive resource consumption, etc).
So the question here's: why is application/vnd.code.notebook.stdout empty?

Shouldn't this go through serialization before logging it, @jlewi?
https://github.com/stateful/vscode-runme/blob/6eaa8a2d30c29cc7e9c5e84f800b94ad6076758f/src/extension/kernel.ts#L777

@sourishkrout Am I missing a call to addExecInfo before serializing the notebook? It seems like that function might be synchronizing information from the terminal to the notebookData structure?

Update: Yes, you're right. I also totally overlooked that marshalNotebook is called and not serializeNotebook, which includes the logic to fill the terminal buffer. We could attempt moving this business logic up the call chain.

As a work around, I modified Foyle to return all code cells as non-interactive cells.
This works in terms of the output being included in the subsequent prompts.
However, non-interactive cells don't show stderr which make them unusable in many instances.
https://github.com/stateful/runme/issues/684

I belief this is fixed by the linked PRs.


Setup automatic minification and code validation

This could be done using a task runner (Grunt or Gulp)

[x] automatic minification
[ ] code validation

By code validation you mean adding a linter? If yes, then #12 is probably duplicate.

@Nikitaw99 Sure, I think it is a duplicate, the other one was closed.
I'll be working on these issues by the next weekend :)


Add ability to terminate TLS with tailscale cert.

The load balancer should configure HA proxy with certificates obtained from tailscale cert {hostname}

I got this working as a POC with caddy here:  https://github.com/testifysec/tailscale-svc-lb

Personally not a user of Tailscale TLS certificates but I can see the appeal of this + Caddy/HAProxy.
Initially this project used HAProxy but I switched over to iptables DNAT since I managed to make it work for my usecase.
Feel free to PR something once it's ready.

I am doing this by having cert-manager configured with a DNS01 challenge to obtain a letsencrypt certificate


Some lines are disappearing when moving from one screen to another

I don't understand how ti can be possible and how it can be an AtlasMap issue but I'm reporting it to have a confirmation or infirmation:
on first screen

on second screen

the only difference is the "recommended" scale and layout which 125% for first screen and 100% on second

VSCode should fix its HTML rendering

it is working in VS Code. It is in Chrome that it is not working

Still nothing to do from AtlasMap, HTML never changes in that kind of window system events

I'm reproducing by changing the size of the Chrome window


more info on usage

Describe what your pull request addresses

[ ] It removes the incorrect info in the README on importing
[ ] It adds a lot of details about using this component

Test Instructions

[ ] None needed... only updated the README

@RoyalSix I've been trying to understand and document all the properties for this RCL. The PR makes changes to the README to make that info available to everyone. Of course, this may not be the best place to put it.
However, I tagged you because there are still of a lot I didn't have time to figure out. And I need to move on for now to some other work. I thought you might be able to fill in the gaps for us. Thanks for considering it!


Work on optimizing and building DEB packages

When compiled with -Wall it was complaining about type mismatch. It seems it was possible in some extreme circumstances to overflow value of int. Also I added static_cast to make signs match.

Some issues detected. Will reopen when these are fixed.


[feat] Add socat/remote zwave device option

Is your feature request related to a problem? Please describe.
I am slowly moving to docker to make it easier to manage for my relative.
The docker instance in on a virtual environment where i cannot pass usb device directly
Because of this, i use ssl socat server on a raspberry A who's only job is to provide device to VM
Because of https://github.com/docker/for-linux/issues/77, i can't use the "device" option to mount my zwave device, and have to use the "volume" as a work around.
This have the huge drawback that if the connection is restarted between the raspberry and the VM (because the vm migrated, because timed out, because the router cleaned up state) the device in the zwavejs container will "freeze" and report I/O error.
This isn't seen by the /health endpoint, so i cannot restart the container, but to be fair i would prefer not to anyway.
With the actual capability, a lost in network connection break the docker service, and that breakage isn't reported in the /health endpoint so Zwave will be unavailable until a user realize and restart it manually.
Describe the solution you'd like
I would like to be able to specify an IP and a port of a socat server instead of a /dev/XXX device, maybe with a cert and a CA to keep SSL. That way the connection can be restarted when detected closed from inside the docker images, not forcing a full network restart and not forcing user action
Describe alternatives you've considered

a fix docker side don't seam to be there anytime soon.
Trying to work with the /health didn't help, but maybe you could add a regular check that the usb device still reply in the /health/zwave ? it's not ideal because it will force a network restart but at least i can automate it from inside docket.
You can also provide any other way to connect to remote usb device. i know there is a "in kernel" way to do so but i haven't played with it at all (https://www.kernel.org/doc/readme/tools-usb-usbip-README)
Last is not using docker, and will surely be the way for me given to surely low priority of this.

Additional context
i've look into any issue already closed to make sure you didn't already excluded that usecase, but didn't find anything.
I don't know if i am the only one with that use-case, but Home assistant droping OpenZwave (Good !) mean i have to make this project work now and i didn't find an alternative to socat.

@vlycop Did you consider using ser2net ?

@vlycop in the serlport input you can manually write the tcp url in the format tcp://yoururl:5000

Oh, Cool ! i didn't found that in the doc and the last issue did hint about it.
I'll try to see if this work with a socat server, but it should.
The only "missing" bit then, and i will rename that feat if you do think it's interesting, would be to add a cert and CA possibility for that.
for exemple my socat look like this right now

client : socat openssl-connect:ap-rpi3bp-002.gdkprod.com:4435,cert=/home/socat/cert/ap-univm-021.pem,cafile=/home/socat/cert/ap-rpi3bp-002.crt,verify=0 pty,link=/dev/lio_zwave,user=socat,group=dialout,mode=660,nonblock,raw,ignoreof
server: socat openssl-listen:4435,keepalive,reuseaddr,cert=/srv/homeassistant/socat/ap-rpi3bp-002.pem,cafile=/srv/homeassistant/socat/ap-univm-021.crt,verify=0 /dev/serial/by-id/usb-Silicon_Labs_CP2102N_USB_to_UART_Bridge_Controller_92a36bbd7784e911bd509cd6217343c2-if00-port0,raw,echo=0

I believe having a s2 secured zwave network end up silly if all communication go through unencrypted in the network.

I think this is something that would require changes also on zwave-js level
cc @AlCalzone

To keep thing safe for now, i'm double bouncing with ease
rpi --- SSL Socat ---> VM --- Clear local Socat ---> Container and it work :)
Only a single issue that may be easy to fix but i can't find the doc :(
2022-04-04T17:07:16.814Z DRIVER   starting driver...
2022-04-04T17:07:16.825Z DRIVER   opening serial port tcp://<IP_ADDRESS>:5896
2022-04-04T17:10:52.182Z DRIVER   destroying driver instance...
2022-04-04T17:10:52.184Z DRIVER   driver instance destroyed
2022-04-04T17:10:52.197Z DRIVER

The driver don't seam to timeout and retry if the serial port is closed when it start.
Clicking the save button again will trigger a driver restart, and if the port is open, connect without issue
This happens during restart of socat or if the container start before the socat service.

Actually, I have no clue what's necessary to connect to a secure socat instance. The network connection is just a simple TCP socket.
Regarding the automatic reconnection, there seems to be something broken:
https://github.com/zwave-js/node-zwave-js/issues/4174
I hope to investigate this soon.

Thank you for the doc, I'll read it tomorrow.
I will also try to look around in the code.
I can hardly navigate it, but I already found where the connection is made and will dig a bit more tomorrow.
No promises tho.

If this is about the secure connection, packages/serial/src is where the serial/socket code resides.
ZWaveSerialPortBase is the common base class for all serial connections, ZWaveSerialPort is the specific implementation for (local) serial ports, ZWaveSocket for TCP/IPC sockets.
As for the reconnection:

The socket implementation should throw/emit an error when the connection goes down
The driver either catches this in the "error" event handler in the start() method (if already connected), or in the openSerialport method (if not yet connected) and self-destructs

If this is about the secure connection, packages/serial/src is where the serial/socket code resides.
ZWaveSerialPortBase is the common base class for all serial connections, ZWaveSerialPort is the specific implementation for (local) serial ports, ZWaveSocket for TCP/IPC sockets.

Would it be possible to use https://nodejs.org/docs/latest-v17.x/api/tls.html instead of https://nodejs.org/docs/latest-v17.x/api/net.html when cert are provided ?
Having never done any JS, https://riptutorial.com/node-js/example/19326/tls-socket--server-and-client doesn't sound so different.

As for the reconnection:
i am surprised to not see a connectListener here https://github.com/zwave-js/node-zwave-js/blob/966e51fc81eb7a1686f9ac94971a06bab5ed26f1/packages/serial/src/ZWaveSocket.ts#L17 in the connect section, but i don't know much about JS syntax
i only see a serial.on("close", ()) which if i'm not wrong won't catch connection error because it need to listen for error ?
If there is a problem connecting, instead of a 'connect' event, an 'error' event will be emitted with the error passed to the 'error' listener. The last parameter connectListener, if supplied, will be added as a listener for the 'connect' event once.

i am surprised to not see a connectListener here

Its the callback to connect:
https://github.com/zwave-js/node-zwave-js/blob/966e51fc81eb7a1686f9ac94971a06bab5ed26f1/packages/serial/src/ZWaveSocket.ts#L31-L34
The missing error listener might be a problem, at the very least it's handled differently for the serial implementation:
https://github.com/zwave-js/node-zwave-js/blob/966e51fc81eb7a1686f9ac94971a06bab5ed26f1/packages/serial/src/ZWaveSerialPort.ts#L33-L71

@AlCalzone Can I move this to zwavejs?

@AlCalzone Can I move this to zwavejs?

i'll close this and iopen another one if/when new config field are needed for ssl.

@robertsLando out of curiosity, what does the restart on failed ?
Zwave-js or the front end ? Because when trying the source of zwave-js with it's debug script, it does stop both in when it disconnect (but don't restart like in the front end) and if the tcp socket isn't available (while it hang for ever in the front end)
If this is a managed in the front end, then it might be more an issue for here than for AICalzone ?


Fix ENOENT errors when polling on Netlink socket

Summary
When a Netlink response is issued, which a task is currently polling for
on the corresponding AF_NETLINK socket, then the poll call will return
with a ENOENT error.  This is due to the fact that the Netlink response
notifier is automatically torn down after the notification.  Thus, this
change fixes up the ENOENT return value to OK.
Impact
None
Testing
Tested on an STMicroelectronics B-U585I-IOT02A board.

@xiaoxiang781216: I might have misunderstood you.  Did you mean to say to change the generic work_notifier_teardown to return void?

@xiaoxiang781216: I might have misunderstood you. Did you mean to say to change the generic work_notifier_teardown to return void?

Yes, I just create a patch to modify all other place, could you merge into this PR?
https://github.com/xiaoxiang781216/incubator-nuttx/tree/work


Nargo refuses to proceed when Executing / Proving with unfilled return

Description
Aim

Computes an unknown Pedersen hash value.
Expected behavior

Running nargo execute / nargo prove on the following should auto-compute the return value and fills it in Verifier.toml:
main.nr
use dep::std;

fn main(a : Field) -> pub Field {
    std::hash::pedersen([a])[0]
}

Prover.toml
a = "1"
return = ""

Bug

Was prompted with this error instead:
Received parameters not expected by ABI: ["return"]

Environment

[x] Using nargo:

[x] Binary:  nargo 0.1.1 (git:0c16089)
[ ] Compiled from source:

What happens if you remove the return parameter from Prover.toml ?
There was a PR to remove this about two weeks ago

Ah wasn't aware of that being the cause. Thanks Kev.


Vue 3 Support

This PR adds the plugin's Vue 3 support + useShepherd() composable function for the usage in setup hooks.
To make that possible I use vue-demi library, but there are some limitations in terms of building the package.
In package.json's module field  I have changed it to src/entry.js because vue-demi works at build time, so I remove the ESM compilation since it was unnecessary and user can directly use src/entry.js.
The problem is for the SSR and unpkg versions... I think that the solution can be to build two versions of each build type, where:
One will still support Vue 2 (files named vue2-shepherd.min.js & vue2-shepherd.ssr.js)
and the others for Vue 3 (files named vue3-shepherd.min.js & vue3.shepherd.ssr.js).
These changes are not in this commit, if you are ok with this solution I'll make the change.

@fech-dev thank you so much for the PR and sorry for the delay in responding! After thinking about it some more, I think I am okay with completely dropping Vue 2 support, so if you would like to proceed with updating to entirely Vue 3 and composables, I support that 👍

@fech-dev thank you so much for the PR and sorry for the delay in responding! After thinking about it some more, I think I am okay with completely dropping Vue 2 support, so if you would like to proceed with updating to entirely Vue 3 and composables, I support that 👍

Thanks @rwwagner90. Ok, I'll try to update asap.

@fech-dev did you still want to update this PR?

@fech-dev did you still want to update this PR?

Sorry, for the late reply, but I was very busy.
I've made a commit, let me know if it's ok.

@fech-dev do you mind rebasing this please?

@fech-dev thanks for all the hard work! Just released 2.0.0


Proposal: Model our UX after k8s PVs / PVCs

UPDATE: This proposal is now available as a slide deck, complete with diagrams and examples. This is the deck that was presented to SIG Service Catalog on 2016-10-10.
See #212, #213, and #215
I believe this to be a tenable proposal for addressing the issues referenced above as well as the more complex use cases I articulated in #213 which are not currently addressed by any other proposal.
For further reference: http://kubernetes.io/docs/user-guide/persistent-volumes/
In k8s, PersistentVolumes are resources that describe an actual NFS (or similar). PersistantVolumeClaims are requests for the allocation of those resources. To illustrate a similarity to what we're trying to accomplish with Steward, PersistentVolumes can be provisioned in one of two ways:
Dynamic:

When none of the static PVs the administrator created matches a user’s PersistentVolumeClaim, the cluster may try to dynamically provision a volume specially for the PVC. This provisioning is based on StorageClasses: the PVC must request a class and the administrator must have created and configured that class in order for dynamic provisioning to occur.

There is a striking similarity here between a StorageClass and a ServiceCatalogEntry. Neither describes an instance of a volume or service. They describe a kind of volume or service-- a recipe for dynamic instantiation. I'm drawing a parallel here between dynamically provisioned PersistentVolumes and our notion of "dedicated" services (which live and die with a consumer's claim on them).
Static:

A cluster administrator creates a number of PVs. They carry the details of the real storage which is available for use by cluster users. They exist in the Kubernetes API and are available for consumption.

Note the similarity to our notion of "multi-tenant" services. Just as an administrator creates specific, named PersistentVolumes for consumption by apps making PersistentVolumeClaims upon those resources, we are imagining that administrators (operators) pre-define multi-tenant services for consumption by apps making ServicePlanClaims.
If we are appreciative of symmetry and consistency, what Steward currently lacks vis a vis PVs / PVCs is that we don't have a PersistentVolume-like resource type that can represent existing (multi-tenant) services. Were we to introduce a Service type, we'd get that symmetry and consistency and we would address the use cases I brought up in my previous post #213.
Point of clarification: We all know the term "Service" is overloaded. Please forgive my use of Service above (and repeatedly in the following paragraphs). I'm not talking about a k8s Service (load balancer). I am talking about a hypothetical k8s resource type (3PR), instances of which are representations of actual (sorry -->) service  instances. Obviously, we need to come up with a different name for this hypothetical resource type.
What I'm actually proposing and how it solves some problems:
ServiceClass: (formerly ServiceCatalogEntry). Admins / operators create these explicitly to describe a kind of service that can be instantiated. These can reference a service id and plan id (existing concepts) that are found in some Steward's own internal catalog. That Steward can and will remain responsible for managing provisioning.
A couple really great things this accomplishes:

Services or ServiceClaims (formerly ServicePlanClaims) can reference semantically meaningful names chosen by an administrator-- names like "Postgres small." The consumer needs to know nothing of how that is fulfilled (whether it is through RDS or something else is not necessarily evident, and probably irrelevant). The consumer doesn't need to ever deal with long, opaque service ids and plan ids.
The administrator can constrain or restrict any given broker's offerings-- just because the RDS broker offers SQL Server doesn't mean I want to make it available in my cluster. 😉

Services: (again, excuse the overloading). Admins / operators create these explicitly and give them semantically meaningful names (e.g. "zip code database (postgres)"). They reference a ServiceClass. The Steward(s) that can handle provisioning for the service id and plan id referenced by that  ServiceClass will provision such a service when a Service resource is created and deprovision it when it is deleted. (See #212)
ServiceClaims: (formerly ServicePlanClaims). These reference an existing, multi-tentant Service by name (e.g. "zip code database"), or request a dedicated service instance by referencing a ServiceClass by name. In the former case, internally, only binding is necessary. In the latter case, internally, provisioning and binding must both occur. In both cases, these discrete, internal operations are handled by the Steward(s) that can handle service id and plan id referenced by that  ServiceClass.
Note that much of the above is probably further impetus for the creation of a "Steward Manager" (as @slack also has proposed in #215). This would handle the lifecycle of k8s resources discussed above, manage all Stewards, and delegate discrete operations to individual Stewards. (Essentially, then, each individual Steward just becomes a "broker adapter" (of sorts) that can extend the sort of ServiceClasses an operator would be able to define without having to restart / reconfigure the whole system.)
From an operator perspective, this all looks great-- eventually, we (or the community) could offer Helm charts of a Steward pre-configured and bundled up with its backing broker. The chart can also include ServiceClass resources for all of (or a subset of) the backing broker's offerings. Operators can edit the chart to remove ServiceClasses they don't want to expose. Operators can also make their own custom charts that describe their multi-tenant services by referencing ServiceClasses.
From a consumer perspective, this is also very simple: make a ServiceClaim and reference either a multi-tenant Service by name or a ServiceClass by name.
cc @gabrtv @arschles @slack

The SIG has decided on an API that is loosely related to the PV/PVC system, along with other elements. See https://github.com/kubernetes-incubator/service-catalog/pull/31 for details. I'm closing this issue.


[Bug] BetterDiscord Activity Status Won't Properly Update

Before Reporting

[X] I found no existing issues matching my bug
[X] My issue is not caused by a plugin or theme

Describe the Bug

As shown in the above image Persona 5 Royal is shown to be my current game activity however

Persona 5 Royal is not currently open on my PC. Whenever I close a game or open another the activity does not update until I completely restart Discord.
Expected Behavior
With my clean copy of Discord PTB it works normally and the activity is updated as I close and open applications. So I expected BetterDiscord to work the same way after all it seemed to be doing that just fine before.
Steps To Reproduce

Open BetterDiscord
Run any game or activity that is registered.
The activity status should change but it does not.
There was nothing else that I did outside of running plugins. With the only one I can think of that may be a problem being the ShowAllActivites plugin.

Operating System
Windows
Discord Version
Stable
BetterDiscord Version
v1.9.3
Anything else?
I hope I'm not just misunderstanding this section of the FAQ.

If so, I apologize for the wasted time, but if not I hope to receive some assistance on this matter or to know if there is some other external reason as to why it isn't working properly. Either way thank you so much for the response!

Have you tried temporarily removing all plugins and seeing if the issue persists?

I just did Im assuming it was the DoNotTrack plugin which I thought didn't apply to the game tracking. But this worked so thank you!


Added animal population by breed dataset

Animal population by breed dataset

This dataset as reported to the Rural Payments Agency contains a list of animal population by breed

🛠 Fixes Issue
👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

Good one @iamrahul8
Congrats 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow these steps.

Star this repository. ⭐
Follow Clueless on GitHub
SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/) .
Go to the profile section from the top right-hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard!
And don’t forget to share your achievement with your friends.

@nikhil25803  , Here is my profile - https://www.clueless.tech/profile/admin/107087746951592717989


Add K8up permissions to clusterrole

Summary

Updates the ClusterRole template after introduction of the new feature in #69

Checklist

For Helm Chart changes

[x] Categorize the PR by setting a good title and adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog
[x] PR contains the label area:chart
[x] PR contains the chart label, e.g. chart:provider-postgresql
[x] Variables are documented in the values.yaml using the format required by Helm-Docs.
[x] Chart Version bumped if immediate release after merging is planned
[x] I have run make chart-docs
[x] Link this PR to related code release or other issues.

Can you also add pods and PVCs in this PR? Same permissions as Schedules

Can you also add pods and PVCs in this PR? Same permissions as Schedules

That requires the generator to be updated as well, which would be a code change that can't go into this PR.
If you create a PR that adds those permissions, I'm happy to rebase this PR on your PR.
See https://github.com/vshn/appcat-service-postgresql/blob/d3404861b869c27f4970913f6ee4e8720117e78f/operator/standalone/controller.go#L28-L39

Let's not get confusing with this, lets merge it


fix: fix RUnlock timing

Is it mistake?
Currently, deadlock.RLock executes PostUnlock before RUnlock.

This change is

Thanks!

Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Boxing Reddit Streams>> Here's How To Watch Mike Tyson vs Jake Paul LIVE STREAMs FREE At Home 15 NOV 2024

If you haven’t already heard, Mike Tyson and Jake Paul are set to fight in the ultimate eight-round match soon. For months, boxing fans have been counting down to this day. Now that it’s here, some people are wondering if it’s free to stream live. After all, some streaming platforms require subscribers to sign up for premium subscriptions to watch certain movies and TV shows.
Earlier this year, boxing fans were bummed when Most Valuable Promotions pushed the fight back to a later date. The decision came after Tyson suffered an in-flight health scare. He had an ulcer flare-up, and doctors recommended him to participate in light training for a brief time.
“During a follow-up consultation on Thursday with medical professionals on his recent ulcer flare-up, the recommendation is for Mike Tyson to do minimal to light training over the next few weeks and then return to full training with no limitations,” MVP explained in a social media statement, adding that “both Mike and Jake are in agreement that it is only fair to ensure that both athletes have equal training time to prepare for this important match and are able to compete at the highest level.”
Once it came time for them to go toe to toe, Tyson and Paul got into a heated exchange at the weigh-in one day before their match.
Find out whether or not the Tyson vs. Paul fight is free to stream live.
Mike Tyson Slaps Jake Paul at Weigh-in Before Fight: Video
Courtesy of Esther Lin / Most Valuable Promotions
Where to Watch the Mike Tyson vs. Jake Paul Fight
Viewers can watch the entire fight on Netflix. The streamer is exclusively broadcasting the event live.
What Time Does Tyson Fight Paul?
Tyson is scheduled to go head to head with Paul at 8 p.m. ET on November 15.
Is the Fight Free to Stream?
According to multiple outlets, Netflix subscribers will be able to watch the Tyson-Paul fight for free on November 15.
Can You Watch the Tyson vs. Paul Fight for Free Online?
No, there is no way to watch the Tyson-Paul fight anywhere else other than Netflix. Boxing fans will need to get a Netflix subscription in order to view the match.

gj gghj

💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?
💻Best for PC/MAC 1080p/720p 👇
🥊Link 1.🔴📺👉 Boxing: Jake Paul vs Mike Tyson Streams Live Free?
🥊Link 1.🔴📺👉 Boxing: Mike Tyson vs Jake Paul Streams Live Free?


Stutter in animation when dismissing from cancel button

Because the camera overlay view is still running when the CardIOPaymentViewController is dismissed from the cancel button, there is a delay during animation (it moves about the height of the nav bar, pauses and then continues). There is no delay when dismissing from the done button after scanning is complete
The fix I have is to get access to the cardIOView property of cardIOViewController, and call stopSession before dismissing the view controller (category method I am using below).
Obviously using these private methods is not the nicest way of doing things so is there a better alternative to fixing this issue?
- (void)ttn_stopScanningSession
{
    UIViewController *cardIOViewController = self.topViewController;

    if ([cardIOViewController respondsToSelector:@selector(cardIOView)])
    {
        NSObject *cardIOView = [cardIOViewController cardIOView];

        if ([cardIOView respondsToSelector:@selector(stopSession)])
        {
            [cardIOView stopSession];
        }
    }
}

@wattson12 there's a stutter we're aware of that we never found a solution for. If this is that same stutter, and your fix is all we need to solve it, that would be great!
When I get a chance, I'll see how your fix works when integrated into our code. (Or else you can try for yourself in card.io-iOS-source and send us a Pull Request!)
If it works for me, then you'll see a relevant mention in the card.io-iOS-SDK release notes. Until then, you should be safe to continue using your category.

Ah. Different stutter. None of us had previously noticed this one that you're reporting, but now that I look I see what you mean.
I'll try to incorporate your fix in the coming week or so.

Great, thanks for this. Also I only realised there was an open source repo after submitting this, I'll make a proper pull request if I have any new issues.

@wattson12 sounds great!
(Though please don't feel obliged to understand our somewhat convoluted code well enough to actually create a PR whenever you run into a problem.)
Hey -- so if you weren't aware of our open source repo, how did you figure out your fix?

I used the objc runtime to dump a list of class methods, and took a few educated guesses. Thanks for the well named methods :+1:

Well done!

Now officially released.

Awesome, thanks for the quick fix


Toml parsing fails for .workspace keys

First of all, thank you for this!
I was surprised examples and tests do not get cut out automatically on cargo publish. Is there any downside to removing them? iirc someone mentioned package managers which like to run those tests first or some other obscure thing?

might be due to an older toml version parser
workaround: version = { workspace = true } for all dotted keys

Thanks for letting me know!
It looks like the toml parser used here is very old: toml_edit matches 0.2.1;        latest is 0.19.8, and upgrading isn't trivial as the API changed quite a bit.
I will see when I get to it but would appreciate help.

I was surprised examples and tests do not get cut out automatically on cargo publish. Is there any downside to removing them? iirc someone mentioned package managers which like to run those tests first or some other obscure thing?

cargo publish publishes everything in the directory that isn't ignored by git, which can lead to huge packages. This approach seems safe even though it may lead to surprises as well. That's the reason for cargo-diet existing, and I can say that it makes choices and is opinionated. Probably the need for opinion is why cargo tooling can't do it to that extend, but it's a guess.
Thanks for also mentioning the workaround as well, let me repeat it here.
Workaround
Use version = { workspace = true } instead of version.workspace = true


Docker image miss CA certificates for HTTPS RPC

Summary of Bug
When using latest docker image, I get a CA error while trying to use HTTPS RPC endpoint
Following error is found in container log:
INFO ThreadId(01) running Hermes v1.5.1+6583d69
thread 'main' panicked at 'no CA certificates found', /usr/local/cargo/registry/src/index.crates.io-6f17d22bba15001f/hyper-rustls-0.22.1/src/connector.rs:45:13

similar issue is solved here
Version
v.1.5.1 docker image
Steps to Reproduce
Use HTTPS RPS endpoint in the relayer config
Acceptance Criteria
HTTPS endpoint should work

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate milestone (priority) applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Thanks for the report! We'll add the required package and push the updated images tomorrow.


Crash in iOS 13.0 in moya provider

I am facing a crash in moya provider as described in screenshot
I think the issue caused by multithreading because I am using background thread for callback queue

internal let provider = MoyaProvider<MultiTarget>(endpointClosure: MoyaProvider<MultiTarget>.defaultEndpointMapping, requestClosure: MoyaProvider<MultiTarget>.defaultRequestMapping(for:closure:), stubClosure: MoyaProvider<MultiTarget>.neverStub, callbackQueue: .global(qos: .userInteractive), session: JBNetworkManager.defaultAlamofireSession(), plugins: [NetworkLoggerPlugin(configuration: .init(logOptions: .default))], trackInflights: true)

Hey @SherifKamalSalem - sorry for that. Are you able to create a really simple repro code that I could look into?

this is the crash log from crashlytics

here is the trace log
15 elements

0 : "0   ???                                 0x0000000108bb0bac 0x0 +<PHONE_NUMBER>"
1 : "1   ???                                 0x0000000108bb0ce8 0x0 +<PHONE_NUMBER>"
2 : "2   مكتبة جرير                 0x0000000100eea0b4 main + 0"
3 : "3   Moya                                0x0000000106179b68 $s4Moya0A8ProviderC13requestNormal_13callbackQueue8progress10completionAA11Cancellable_px_So17OS_dispatch_queueCSgyAA16ProgressResponseVcSgys6ResultOyAA0N0CAA0A5ErrorOGctFyAPy10Foundation10URLRequestVATGcfU0_yAUcfU_ + 380"
4 : "4   Moya                                0x0000000106181a6c $s4Moya0A8ProviderC20sendAlamofireRequest33_29224DEABDDE279B92464CA1BE771359LL_6target13callbackQueue8progress10completionAA16CancellableTokenCqd___xSo17OS_dispatch_queueCSgyAA16ProgressResponseVcSgys6ResultOyAA0U0CAA0A5ErrorOGct0D00E0CRbd__AA11RequestableRd__lFySo17NSHTTPURLResponseCSg_10Foundation10URLRequestVSgA4_4DataVSgs0W0_pSgtcfU0_ + 1820"
5 : "5   Moya                                0x0000000106181cf0 $s4Moya0A8ProviderC20sendAlamofireRequest33_29224DEABDDE279B92464CA1BE771359LL_6target13callbackQueue8progress10completionAA16CancellableTokenCqd___xSo17OS_dispatch_queueCSgyAA16ProgressResponseVcSgys6ResultOyAA0U0CAA0A5ErrorOGct0D00E0CRbd__AA11RequestableRd__lFySo17NSHTTPURLResponseCSg_10Foundation10URLRequestVSgA4_4DataVSgs0W0_pSgtcfU0_TA + 200"
6 : "6   Moya                                0x000000010616ebb0 $s9Alamofire11DataRequestC4MoyaE8response13callbackQueue17completionHandlerACXDSo17OS_dispatch_queueCSg_ySo17NSHTTPURLResponseCSg_10Foundation10URLRequestVSgAN0B0VSgs5Error_pSgtctFyAA0B8ResponseVyAtA7AFErrorOGcfU_ + 588"
7 : "7   Alamofire                           0x0000000104b33b88 $s9Alamofire11DataRequestC8response5queue17completionHandlerACXDSo012OS_dispatch_E0C_yAA0B8ResponseVy10Foundation0B0VSgAA7AFErrorOGctFyycfU_yycfU_yycfU_yycfU_ + 84"
8 : "8   Alamofire                           0x0000000104a77eb8 $sIeg_IeyB_TR + 52"
9 : "9   libdispatch.dylib                   0x0000000106687b68 _dispatch_call_block_and_release + 32"
10 : "10  libdispatch.dylib                   0x00000001066895f0 _dispatch_client_callout + 20"
11 : "11  libdispatch.dylib                   0x000000010669c4b8 _dispatch_root_queue_drain + 824"
12 : "12  libdispatch.dylib                   0x000000010669cc94 _dispatch_worker_thread2 + 140"
13 : "13  libsystem_pthread.dylib             0x00000001f08008cc _pthread_wqthread + 216"
14 : "14  libsystem_pthread.dylib             0x00000001f080777c start_wqthread + 8"

it's hard to reproduce this crash in demo because it happens randomly

Any update here @sunshinejr

this crash seems to be like #1820

We're experiencing the same issue with using a background thread for callbacks. Though it happens randomly it can be reproduced by spamming the same api every ~500ms.
If you look at the specific code line of
`              if self.trackInflights {
self.inflightRequests[endpoint]?.forEach { $0(result) }
            self.lock.lock()
            self.inflightRequests.removeValue(forKey: endpoint)
            self.lock.unlock()
          }

you notice that theforEach { $0(result) }is not thread safe, hence the data race issue. In our use case the forEach happens at the same time as an insert in happing onMoyaProvider+Internal.swifton line 37:
lock.lock()
var inflightCompletionBlocks = self.inflightRequests[endpoint]
inflightCompletionBlocks?.append(pluginsWithCompletion)
self.inflightRequests[endpoint] = inflightCompletionBlocks
lock.unlock()
`
To resolve this, the forEach should also be happing inside the lock to unsure it is not mutated while getting the date from another thread.

i faced the same issue app getting crash, i have my custom framework, when i call the framework am getting crash like below, could any one suggest how to fix

i faced the same issue app getting crash, i have my custom framework, when i call the framework am getting crash like below, could any one suggest how to fix

any update i am also facing same issue


Linux: Updates are applied while Code is still running

From #45547
This needs to be addressed for both DEB and RPM packages.

@joaomoreno this is less than ideal but is as designed since we I believe it's bad practice to block apt if code is running. Right now users typically get an ENOENT pretty quickly which forces them to restart (a more specific message would be better though.
This will be much better with the snap package which is the future direction https://github.com/Microsoft/vscode/issues/5458

@Tyriar We should still fix it since we're getting bogus error telemetry errors which show processes from different versions interacting with one another.
In Windows, we prevent it using a system-wide app mutex. Can we do the same in Linux?

@Tyriar steps:

launch a vscode window with the terminal hidden
update vscode
toggle terminal

Can we at least send a signal or something such that we at least disable error telemetry... Or perhaps show the user a message that everything is broken...

@bpasero what do you think about watching process.argv[0] with fs.watchFile on the main process and if there's a change notify all renderers to show a message and disable telemetry?

@Tyriar and the intent is to send a command over from another process to VS Code? If so, that should be done by running just code with a command line argument, no?

@bpasero I was thinking leave this completely up to the running version to handle. Relying on a CLI arg could potentially going wrong, for example code --notify-update on any old version would launch the old version.

@Tyriar I am just worried that something generic as watching process.argv[0] could be triggered by other things than us.

@bpasero we could check for something like stats.mtime && size to narrow it to the case we're interested in.

@Tyriar from my experience of using these flags for preventing dirty writes / reads you cannot rely on that information...

Ok, I guess the option is then inside the deb/rpm post install scripts to check if vscode is running (ps axco command | grep code?), check the version inside package.json is compatible and then run something like code --notify-update to inform the windows.

@Tyriar as far as I know we use a mutex on windows to signal that Code is running, can't you do something like that for Linux too?

@bpasero I think blocking install without closing VS Code would not be ideal for Linux. This would cause apt/dnf to fail updating half way through, which may not only be updating VS Code (eg. sudo apt-get upgrade).

@Tyriar we could add a new shell script that you would call that would pass some specific argument to the running Code instance. For older installations it would fail silently because this shell script would not exist. With that we could do it all through a new command line argument.

@bpasero version checking using code --version should be pretty easy and we could leverage the same CLI then?

@Tyriar yeah true that would work as well. After you see that the version is fit for the command, just invoke it again with another CLI argument that you add only for this purpose.

I'll see if I can get to this in April

Duplicate https://github.com/Microsoft/vscode/issues/5908


Some useful functions

While recognizing that deciding what to include and exclude is a difficult balancing act, I was wondering if there was a specific reason that forM/mapM are not included here? (Also, apologies if this is the wrong place to bring this up, I'm still trying to figure out github's interface)

They are included from Data.Traversable.
https://github.com/sdiehl/protolude/blob/master/src/Protolude.hs#L67


"model_name" used for the Albert API calls as project env variable

Until we have PyAlbert as a separate Python module, deployments and redeployements would be more smooth if the API payload var "model_name" would be loaded from a env var of the project, so that we can switch more easily.
Here: https://github.com/etalab-ia/albert-tchapbot/blob/dev/app/pyalbert_utils.py#L49
Let me know your thoughts.

PR: https://github.com/etalab-ia/albert-tchapbot/pull/21


Avoid hard-coded URL

Avoid hard-coded URL so that we can share URL deployed under any domain

Thanks for the contribution.


Broken array, for-loop, and println combo?

or is it because i haven't understood how to use the (damned) language yet?

Loops in general seem to be broken at the moment.

ahahahahaha it appears i have forgotten to serialize arrays!

temporarily fix by removing arrays

Definitely one way to fix it, i guess

the issue is a resolving bug with ranges and for loops i believe


Bundler.settings.without not found in 1.16.pre.2

I use Bundler.settings.without in 1.15 but 1.16.pre.2 does not have this method. Is this the intended incompatibility?
I can use [:without] instead, but this is also incompatible. When '--without ' option isn't specified, 1.15 returns nil, but 1.16.pre.2 returns empty Array.
# in 1.15
Bundler.settings.without # => []
Bundler.settings[:without] # => nil

# in 1.16.pre.2
Bundler.settings.without # => NoMethodError: undefined method `without'
Bundler.settings[:without] # => []

Bundler.settings.without and Bundler.settings.with was considered a private API and we offered no guarantees to it's stability.
It was removed in https://github.com/bundler/bundler/commit/8ccf49c3b4a086f1e0cad2130dc30c04304175b7#diff-75219bd6bc0defc0e7f4ba322d92f950

Same with Bundler.settings[]

"Same with Bundler.settings[]"? How can I get the value of with and without options?

The only supported way to get the with and without settings is by running bundle config

Umm... OK, I will use settings[] in my own risk. Thank you.


🔥 Removed duplicate "Infinity Water Bucket" entry

The link to Infinity Water Bucket shows up twice in the list, this pull request removes the duplicate entry.

Oops, thanks for catching that!


Add LTO to the metadata filename hash.

There are some rare cases where different cargo commands end up building dependencies with different LTO settings. This adds the LTO value to the filename hash so that the cache does not thrash when switching between these commands.
Fixes #8669

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+

:pushpin: Commit 6225143b2ab203ed1cf6bf443261c71a09bae84b has been approved by alexcrichton

:lock: Merge conflict
This pull request and the master branch diverged in a way that cannot be automatically merged. Please rebase on top of the latest master branch, and let the reviewer approve again.
How do I rebase?
Assuming self is your fork and upstream is this repository, you can resolve the conflict following these steps:

git checkout fingerprint-lto (switch to your branch)
git fetch upstream master (retrieve the latest master)
git rebase upstream/master -p (rebase on top of it)
Follow the on-screen instruction to resolve conflicts (check git status if you got lost).
git push self fingerprint-lto --force-with-lease (update this PR)

You may also read Git Rebasing to Resolve Conflicts by Drew Blessing for a short tutorial.
Please avoid the "Resolve conflicts" button on GitHub. It uses git merge instead of git rebase which makes the PR commit history more difficult to read.
Sometimes step 4 will complete without asking for resolution. This is usually due to difference between how Cargo.lock conflict is handled during merge and rebase. This is normal, and you should still perform step 5 to update this PR.

Error message
Auto-merging tests/testsuite/lto.rs
CONFLICT (content): Merge conflict in tests/testsuite/lto.rs
Auto-merging src/cargo/core/compiler/lto.rs
Automatic merge failed; fix conflicts and then commit the result.

:umbrella: The latest upstream changes (presumably #8754) made this pull request unmergeable. Please resolve the merge conflicts.
Note that reviewers usually do not review pull requests until merge conflicts are resolved! Once you resolve the conflicts, you should change the labels applied by bors to indicate that your PR is ready for review. Post this as a comment to change the labels:
@rustbot modify labels: +S-waiting-on-review -S-waiting-on-author

@bors r=alexcrichton

:pushpin: Commit a2e48fdc031b95a979c9677039112853618cd178 has been approved by alexcrichton

:hourglass: Testing commit a2e48fdc031b95a979c9677039112853618cd178 with merge 9d1a4863abd9237dbf9d1b74c78632b6a205f6bb...

:sunny: Test successful - checks-actions
Approved by: alexcrichton
Pushing 9d1a4863abd9237dbf9d1b74c78632b6a205f6bb to master...


Wrong syntax highlight

void test()
{
      auto payload = MY_MAGIC_FUNCTION
      (
         "first", value,
         "another", DateTime(nextMonday, TimeOfDay(12,00)).toRailsString,
         "grayed_out", test_coupon_code,
         "this_too", "3"
      );

      int a = 10;
   }
}

Rewriting code in this way:
immutable blah = DateTime(nextMonday, TimeOfDay(12,00)).toRailsString;
auto payload = MY_MAGIC_FUNCTION
(
   ...
   "another", blah,
   ...
)

restore highlight :)

I need to update the language syntax file with the updates from the language-d plugin.

I've seen other related bugs for example:
struct Hello
{
   void test() { }
}

The last bracket is not highlighted. Removing test() (and other inner code-blocks) it works fine.

Why don't you depend on a different package for syntax highlighting? This package seems more like a "make atom an IDE" style thing, rather than a "make atom a feasible editor for D" thing.That way users get both better workspace-d support (because you guys aren't spending time on syntax highlighting at all, period), and they get a choice of what scheme to use (I personally prefer d-struct's scheme over language-d's).

@ysgard and I are maybe interested in providing syntax highlighting for this package.

if you could make a generic textmate style grammar (cson, json, xml doesn't matter, preferably json) that would be great. It would also be usable in other editors like visual studio code.
You might wanna look at https://rawgit.com/Hackerpilot/DGrammar/master/grammar.html for it too. Maybe make a separate project for it so all extensions can use it, just make sure it's a grammar that represents all features then. I can probably help a bit on it too once you have started it

Actually, we already have a grammar (cson) at ysgard/d-struct.
I was thinking that you'd either:

Remove all of your grammar/syntax highlight material and depend on d-struct
or use a git submodule with the d-struct repo to include it in your project.
I'f you're not interested, that's fine, but I thought I'd offer.

Hm well I would like to support it in other languages like vscode too, which AFAIK needs json or xml. As the project is MIT licensed I suppose the grammar can be used in other projects if the license is included, and of course (but not required) a link to the original repository

I couldn't add it as submodule because atom requires the files to be in grammars/ and a dependency would require a package from npm so instead I just copied the file + license @MggMuggins

@ysgard should be made aware of this as well.

@MggMuggins @WebFreak001 I'm cool with it.  Given the disparity of D integrations for Atom, I think now that d-struct's grammar (really its best point) is in atomize-d the best thing would be to deprecate and unpublish d-struct in favour of this package.

Well... maybe I'll wait until atomize-d is actually published, which I would urge you to do.
Break things, release fast, release often... it's a good methodology that drives results.

well but having multiple plugins and users saying that they would rather use one plugin than the other plugin also drives results :p

Even if you don't update over here very often, ysgard and I would contribute occasionally to fix things. There's no harm in releasing what's here either.


Improve performance for godot_shape_2d.get_xformed_segment_normal

Tested versions
All of them.
System information
All.
Issue description
godot_shape_2d.get_xformed_segment_normal entirely recalculates a normal every single time it's called, it's used excessively in godot_collision_solver_2d_sat inside collision detection from what i can see.
Normals need to be cached here be it a substitute or an improvement to the function itself because normals should not be recalculated every time they are needed, only when there are changes.
Especially in this context because I’ve seen it being used in convex convex collision in the loop that just serves to abort if a projection doesn't overlap.
As it is you can get unneeded lag from shapes that are just close to each other because, for example: each convex to convex failed test right now is always at least 1 sqrt call and if it goes a little bit further, more sqrt calls and a lot of other unnecessary math.

How can you be this lazy in your insults? The info is literally in this issue section.
I had to read the entire SAT code to discover this issue meanwhile you just say whatever is in your head, do you have no shame?

i agree, it's crazy that they can't code. i'm not lazy, and i verified it 100%, guaranteed. this is a vanity fork that has never achieved anything and never will. sorry about that, maybe try reporting the bug to the real project? 🫶🏽 hope they can fix it 💜

oh, the real project is here if you're confused about that. these guys can actually fix your problem. i understand not everyone knows the history here and may be ignorant

yeah, exactly. 3 people who can't make a game engine. and there's nobody attached to the project that can. pretty scary, i agree ☝

oh no…i'm being cancelled because this project is both fake and woke…i thought that's what this fork was designed to avoid

oh no…i'm being cancelled because this project is both fake and woke…i thought that's what this fork was designed to avoid

Sorry, but please refrain from spamming issues.

this @mcdubhghlas can't write code either. what are you pretending to do here?

this @mcdubhghlas can't write code either. what are you pretending to do here?

Is your favourite passtime liking your own comments?

no, i have other hobbies. it's up there, though

no, i have other hobbies. it's up there, though

Very great hobbies if that is up there.

thank you. none of the guys you linked or this guy can write game engine code. you're being scammed.

where are their significant commits on relevant repos? if it's misinformation, this should be easy to provide

Absolutely zero discussion of this over on Godot's github from what I could see. Zero Issues show up when searching godot_shape_2d.get_xformed_segment_normal or the function name alone. Same story for PRs.


feat: create PAT

:x: Build TheIdServer 4.0.1-PAT1-0016 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/db41db5da5 by @)

:white_check_mark: Build TheIdServer 4.0.1-PAT1-0017 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/ec3059f9a4 by @)


fix(MockBuilder): touches kept modules in standalone components #5520

closes #5520

v14.10.0 has been released and contains a fix for the issue. Feel free to reopen the issue or to submit a new one if you meet any problems.


Replace string crate with a handrolled type

Addresses: https://github.com/tychedelia/kafka-protocol-rs/issues/43

The first commit adds a couple of TopicName helpers that make it easier to convert to/from normal string types.

The second commit involves a small bit of unsafe code. If that's a problem for us, there are options to remove it (at some possible runtime cost) which we could discuss. But maybe it's fine? I'm personally a bit split on it.

Also @bkirwi It looks like your first commit got clobbered by the second. I'm not seeing the topic name changes in the diff.

Will you just remove string from our deps?

string should be removed from the deps here: https://github.com/tychedelia/kafka-protocol-rs/pull/44/files#diff-2e9d962a08321605940b5a657135052fbcef87b5e360662bb527c96d9a615542L32
I've checked cargo tree and confirmed that it's no longer present in the transitive deps.

It looks like your first commit got clobbered by the second. I'm not seeing the topic name changes in the diff.

Apologies - this was semi-intentional, and I should have called it out! We get the as_str for free from the Deref implementation. We don't have the From<String> for TopicName, though... StrBytes::from_string(s).into() felt lightweight enough that I didn't bother re-introducing it, but I'd be happy to if y'all prefer.
It looks like I have a merge conflict here... I should be able to address it in the next few days, but feel free to push to the branch if you'd rather push this through sooner!

string should be removed from the deps here

Ope! Somehow missed that. Apologies.

StrBytes::from_string(s).into() felt lightweight enough that I didn't bother re-introducing it, but I'd be happy to if y'all prefer.

It's not too bad at all, but I definitely think "name".into() would be very ergonomic. If you want to, this can be a follow up (in addition to GroupId which is the other StrBytes type). Thanks again for this contribution!

🎉 this is great! It has also been a major ergonomics issue for me as well!


Usage for counters

Hi, folks.
I've decided to use your CHM implemendation. But i have specifical use case. I use
map for counting, actually freqeuency of words. So you guarantee synchronization of each particular
operation. How can i reach consistency when i

read value from map
increment it
write it back to map

Of source i can use my own global lock, but it will totaly destoys idea of map concurrency.
Shoud i take ConcurrentShard and use it internal RWLock?

i've used such code
`	for i := 0; i < len(words); i++ {
shard := wordsMap.GetShard(words[i])
shard.RWMutex.Lock()
count, ok := wordsMap.Get(words[i])
	if ok != true {
		count = 0
	} else {
		count += 1
	}
	wordsMap.Set(words[i], count)
	shard.RWMutex.Unlock()
}`

For int counter, I thinks use Atomic is a good idea


Performance compared to other libraries

Can you provide some benchmarks compared to other libraries like use-supercluster?
Thanks

Here are 2 screenshots from the example using 1000 points (which you can also run yourself and compare)
Supercluster

react-native-clusterer


README.md - mention and link to C++ directory

Signed-off-by: Nadav Har'El<EMAIL_ADDRESS>

I'd commit, but I see that I don't have write rights to the repo (:

I'd commit, but I see that I don't have write rights to the repo (:

@avikivity do you know how come @psarna can't commit to the alternator-load-balancing repository? Don't all the maintainers have write permissions to it? I can't seem to configure the maintainers myself - I don't have permissions to do that...

@avikivity do you know how come @psarna can't commit to the alternator-load-balancing repository? Don't all the maintainers have write permissions to it? I can't seem to configure the maintainers myself - I don't have permissions to do that...

In fact only you had permissions. I adjusted it to all maintainers.

In fact only you had permissions. I adjusted it to all maintainers.

@avikivity thanks.

@avikivity thanks.


Script defaults

In async-profiler I've found that you use ${{GIT_REPO:/tmp/async-profiler}} multiple times. You also declare used variables in a comment - that leads me to a suggestion that the script could be written as:
name: async-profiler
defaults:
  GIT_REPO: /tmp/async-profiler
required:
- ROOT_PASSWORD
- ...
scripts: ...

and qDup would make sure that the required variables are set and non-empty and optionally set defaults on the non-mandatory ones.

I think it is a good idea to stop repetition in scripts.
State variables can currently be defined with a default value and overridden at runtime.
e.g.
states:
  BUILD_PROJECT: false

I think the important change here is to be able to define if the variable is required or not and thrown an exception if the state variable has not been defined either in the script or at runtime.
Perhaps we could have an extra field available when defining a state variable that determines if the variable is required and deprecate the default state values that can currently be defined at the point in the script that a state variable is resolved?
e.g.
states:
  BUILD_PROJECT:
    - value: false
    - required: true
  BUILD_PROJECT_EMPTY:
    - value:
    - required: true

The second definition would cause an exception if BUILD_PROJECT_EMPTY was not passed a value at runtime
@willr3 wdyt?

You don't need to put value/`required into a list.
I agree that while it might be more verbose (repetition of value and required) it's probably better to have single mapping for variables than two mappings (defaults and required as I suggested above). Note that setting value for required state does not make much sense, so you can contract that to
states:
  VAR_WITH_DEFAULT: some-default
  ANOTHER_VAR:
    default: foobar # Just an alternative way to set default
  REQUIRED_VAR:
    required: true # or maybe optional: false?

@johnaohara Btw. yesterday we've discussed with Will that in order to have the states configurable either from bash script (e.g. in CLI) or from qDup script it would be nice to ignore -S FOO=${FOO} when ${FOO} is empty, instead of overriding value in script with an empty value. We could intentionally unset FOO using -S !FOO (though I doubt that will be required much in practice).

states:
  VAR_WITH_DEFAULT: some-default
  ANOTHER_VAR:
    default: foobar # Just an alternative way to set default
  REQUIRED_VAR:
    required: true # or maybe optional: false?

This example is similar to how steps can have multiple key/value mappings with some keys having default values and I think makes the most sense.
If we default optional to false, this would retain the current behaviour

@johnaohara Btw. yesterday we've discussed with Will that in order to have the states configurable either from bash script (e.g. in CLI) or from qDup script it would be nice to ignore -S FOO=${FOO} when ${FOO} is empty, instead of overriding value in script with an empty value. We could intentionally unset FOO using -S !FOO (though I doubt that will be required much in practice).

+1

I created #61 for the change to -S so that we don't override values with an empty string.
We could replace all the ${{GIT_REPO:/tmp/async-profiler}} with a set-state at the start of the script
scripts:
  foo:
  - set-state: repo ${{GIT_REPO:/tmp/async-profiler}}
  - sh: echo ${{repo}} #do stuff with repo instead of GIT_REPO

I probably put too much weight on the copy paste crutch when creating the initial scripts :)
After eb1054e59f3c05effecac38eddb085a863bd14be the run should abort if ${{FOO}} fails to resolve to a state value so we have to use ${{FOO:}} if the script supports an empty value. Please let me know if that isn't happening somewhere :)
- set-state: script_foo ${{FOO:}} #optional value
- set-state: script_bar ${{BAR}} #required value, aborts run if missing

I think runs aborting due to missing state should already provide the desired check on required state so we don't need to add reserved words to the state json. Aborting has the added benefit of catching runtime issues that cannot be detected at compile time. For example set-state: ${{FOO}} ${{BAR}} where FOO or BAR come from a regex could not be detected at compile time.
Is there something we don't get from the current master implementation that we would get from adding default with required or optional?

One note: right now states are declared on a file level, and I am not sure if these are set for the non-main files. I did not properly realize that above.
set-state does the job, though the imperative approach does not stick out in the script structure as declarative one. There's also duplication of the variable name when you write
scripts:
  foo:
  - set-state: FOO ${{FOO:my-default}}
  - sh: ...

and some curly braces that your eyes need to parse. IMO despite a bit more verbosity it's easier to read
scripts:
  foo:
    states:
      FOO:
        default: my-default
    commands:
    - sh: ...

It's true that simple FOO: my-default looks more like overriding that variable than just supplying the default if not set.
About the required vars: I agree with the abort; the thing is to abort as soon as possible. With declarative required variable it would fail before the first command from the script gets executed. Yes, you could have the same check doing just
- sh: echo ${{FOO}} > /dev/null

but again, it's about readability.

I think the benefit of adding required / optional allows for a fail fast path for validating state variables required for a run at startup.

We could replace all the ${{GIT_REPO:/tmp/async-profiler}} with a set-state at the start of the script

If we have multiple scripts running on multiple hosts, there is no guarantee of script ordering. We would need to add a set-state command to the start of every script that used that state variable to check if it was empty if we want to fail fast. Or add the the setup script, but not all qDup files will contain setup scripts.

I think runs aborting due to missing state should already provide the desired check on required state so we don't need to add reserved words to the state json.

I think this is a side effect of set-state and is not explicit behaviour of qDup, I think it would be more difficult for users to find/remember this behaviour compared to a declarative property on a state variable that could be easily documented.

Aborting has the added benefit of catching runtime issues that cannot be detected at compile time. For example set-state: ${{FOO}} ${{BAR}} where FOO or BAR come from a regex could not be detected at compile time.

I agree that in the case of assigning dynamic state based on the result of a cmd, the onus should be on the user to verify that the  state variable contains valid data or abort.

Is there something we don't get from the current master implementation that we would get from adding default with required or optional?

I think we would get a succinct, more robust, fail fast error checking for variables that need to be set for a script

Sorry if I am being dense but what are the goals for this feature change? I'm reluctant to add reserved words but if that proves to be the best way to achieve the goals then we can certainly add them.
If our goal is to mark variables as optional or required then I think we already have that in the syntax.
foo:
- sh: echo ${{OPTIONAL:defaultValue}} #will not fail because a default is provided
- sh: echo ${{REQUIRED}} #will fail because a default is not provided

There is a complaint we have about ${{OPTIONAL:defaultValue}} and having to write the defaultValue each time. I don't think that is necessary, it was just how I wrote the scripts originally. We can either set-state: OPTIONAL ${{OPTIONAL:defaultValue}} or the script can add OPTIONAL: defaultValue to states and rely on the failure if it gets removed from states. A utility yaml can also have a states section which are overridden by the first yaml file in the list of yaml files to qDup.jar. I used the ${{OPTIONAL:defaultValue}}
If our goal is to fail at build time rather than runtime when a required variable is not defined then I would like to take a try at adding that to the static analysis that runs during build time. Right now qDup does the following static analysis:

Calculate expected signal counts for all signals
Ensure there are no remote shell commands in an observable context. (make sure sh isn't called in watch or on-signal or timeout)
Check for script loops
We could check for ${{FOO}} where FOO is not in states and not in set-state: FOO .... This would not deal with set-state: ${{BIZ}} ...' where BIZ=FOObut I suspect that will be uncommon and we either add a flag to disable the static check when we know it fails incorrectly (e.g.--disable statescan) or a flag to ignore specific variable names (e.g. --state-ignore FOO`)

Let me know what you guys think and I might start hacking at the static analysis because that part of the code could use a few minutes of review.

The goal is improved readability and structure, opinionated places to declare input variables for each script. As you've said, the goal can be achieved with set-state/echo.
Static analysis is definitely something extra, out of scope of this issue, and would work even with the imperative syntax.
If you prefer minimal complexity over readability, so be it.

I re-read the thread and I think I missed the initial issue. Did you proposed adding defaults and required as a way to declare the variables for a script? The scripts section right now only uses the key to define the script and then the value is expected to be a sequence of commands in the script. I don't think the current yaml structure would make it possible to provide per-script information about variables. We could use the states section to declare the default values for all variables in the file. The states get loaded based on the yaml file order so the first file would define the current value of states and subsequent files could define the default values.
Another option is to introduce a more verbose way to write the yaml for the scripts section.
scripts:
  - name: async-profiler
  - variables:
    - GIT_REPO: /tmp/async-profiler #provides a default
    - PASSWORD: #no default so user must provide a value

I think I like the idea both of slim and verbose way to write the scripts. Adding variables moves the existing comments into the yaml syntax which makes it easier to read and lets us include the default in the syntax but I doubt it will be more accurate. We will need to scan the script to make sure variables is correct.

Yes, I was thinking about scripts (conceptually thinking about these as functions with parameters), though some symmetry between scripts and the global vars would be welcome.
Your example mixes lists and mappings, what you wanted is
scripts:
- name: async-profiler
  variables:
    GIT_REPO: /tmp/async-profiler #provides a default
    PASSWORD: #no default so user must provide a value
  commands:
  - sh: ...

Name, variable and commands belong to single item, so these all are single mapping in the list of scripts. Also, variable mapping uses unique keys (variable names) so there's no need to put them in as a list.
One downside of the verbose way is that while the name (async-profiler) is unique it is not used as unique key for the scripts; if you did so you'd have to add one more level of indentation:
scripts:
  async-profiler:
    variables:
      GIT_REPO: /tmp/async-profiler #provides a default
      PASSWORD: #no default so user must provide a value
    commands:
    - sh: ...

About the order of loading: to me it seems a bit confusing that you can have multiple files with host definitions etc. but actually only the first is used. I'd prefer the other files to be library-only (so these could contain only scripts - ok, maybe some global vars as well).

Any updates?

The static analysis part is added to qDup.
scripts:
  test:
  - sh: echo "${{does_not_exist}}"

Will not start a run if does_not_exist is not defined
Error: does_not_exist is used before it appears to be set
  role: doit stage: setup script: test
  command: sh: echo "${{does_not_exist}}"

We don't yet have an updated yaml parser that accepts variables and associates a default value. We also need to ensure the default value gets replaced with any other value in state be it in a with, set-state, states, or -S in the command line.

I guess that you're not going to introduce any new syntax for this; superseded by https://github.com/Hyperfoil/qDup/issues/160


Adding a different state at conditional media

Hi Matt,
First of all, I want to thank you for the great work! Your old dashboard has been hanging on the wall for over a year now. I'm following this new project with great interest. Currently, I'm using a "Latest News" feature on the dashboard. This is an image I added using a Genetic Camera. However, I'd like the news title, which is in a different sensor, to be displayed as the title. In summary, it would be great if you could set a different sensor for the title in the conditional media settings.
It also would me nice to change the icon as well.  ;)

Hi, this is not what it's designed to do, but you can make your own template sensor as a workaround.
template:
  - sensor:
      - unique_id: latest_news
        state: >
          {{ states('sensor.news') }}
        attributes:
          friendly_name: Latest news
          icon: mdi:newspaper
          entity_picture: /local/news.png


python311Packages.pyoverkiz: 1.13.4 -> 1.13.6

Diff: https://github.com/iMicknl/python-overkiz-api/compare/refs/tags/v1.13.4...v1.13.6
Changelog: https://github.com/iMicknl/python-overkiz-api/releases/tag/v1.13.6
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

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

3 packages built:

home-assistant-component-tests.overkiz
python311Packages.pyoverkiz
python311Packages.pyoverkiz.dist


Review repo PR checks

DoD

Revisit automatic github PR checks (ie. Linting). Make a decision if need to add additional checks.

I have started working on this.
Few findings:

I have seen that Formatting, Linting, checks are not happening in our repo in a conventional way.
They are happening in the code during the generation of build process.
Though few checks are configured at the repo level (mde repo), they are not getting triggered (e.g. Terraform related checks.)

Proposal/Way forward

Look holistically how we want our repo to have these checks (formatting, linting, other pre-commit checks).
Try these within Fabric E2E sample.
Rest of the code (e.g. databricks, single tech samples)

Current Progress:

I have created a personal repo to demonstrate how this can be done.  (this was demonstrated to Anuj and Lace)
I am currently in process of adding these to MDW Fabric E2E samples.

This is open for review


IoThreadHolder edge case

Okay, so my situation might be a little bit specific, but perhaps can be solved without requiring me to use DISCORD_DISABLE_IO_THREAD. (This is actually the second project I've experienced this issue with, but only now actually figured out exactly what is happening.)
I build the discord-rpc library statically and link it into a dll, which I then dynamically load into a process.
This works great when cleanly shutting down with Discord_Shutdown(), but not on process termination. In this case, ~IoThreadHolder() gets called (because of the static instance of IoThreadHolder), which runs Stop() which causes a deadlock because it's waiting on a non-existing thread that has already been terminated:

All of the threads in the process, except the calling thread, terminate their execution without receiving a DLL_THREAD_DETACH notification.

The MSDN page also states:

If one of the terminated threads in the process holds a lock and the DLL detach code in one of the loaded DLLs attempts to acquire the same lock, then calling ExitProcess results in a deadlock.

The same thing also happens when trying to call Discord_Shutdown() in DllMain when lpvReserved is non-null, but that can be avoided by simply not calling that, as process termination here is essentially not clean.
I know this might be sort of an edge-case for me, but perhaps this can be fixed somehow?

I found a workaround for this that I'll be using for the time being:
    void Stop()
    {
#ifdef _MSC_VER
        // Windows-only hack: don't bother if thread is already killed
        DWORD dwExitCode = 0;
        ::GetExitCodeThread((HANDLE)ioThread.native_handle(), &dwExitCode);
        if (dwExitCode != STILL_ACTIVE) {
            return;
        }
#endif

        keepRunning.exchange(false);
        Notify();
        if (ioThread.joinable()) {
            ioThread.join();
        }
    }

Thank you for letting us know about this and posting a temporary workaround for folks that may also be experiencing this.
@snowyote will take a look and see if there's a way that we can fix this a bit more elegantly.

Honestly, it seems like we just shouldn't call Stop() from ~IoThreadHolder() - if we're at the point that static destructors have been called, pokily going to the trouble of shutting down threads seems pointless as the OS is obviously already doing it for us.
@codecat, does that seem like that would fix your problem without interfering with the Discord_Shutdown() use case?

iirc I added it because there is an abort in std::thread destructor if you didn't join the thread.


Question - Publishing OCI Artifacts - Should the layer be compressed?

I am currently building a new supporting tool and just wondering if this is correct.
$ oras pull ghcr.io/devcontainers/features/git:latest
Downloading ebd18821c6cc devcontainer-feature-git.tgz
Downloaded  ebd18821c6cc devcontainer-feature-git.tgz
Pulled [registry] ghcr.io/devcontainers/features/git:latest
Digest: sha256:f839be8a0203abe12c917b262b1a1330b8286f9576ef21ea4a26ed60c5bc9947
$ file devcontainer-feature-git.tgz
devcontainer-feature-git.tgz: POSIX tar archive

Inspecting the layer manually reveals that the media type is application/vnd.devcontainers.layer.v1+tar. However, the filename from the org.opencontainers.image.title annotation gives a ".tgz" extension. The actual file does not seem to be compressed.
I'm just wondering which is actually correct? A bare tar archive? Or a gzip'd compressed version?

Hi 👋
That's interesting find, thanks for raising this question. Looping in @joshspicer