How can I use other Apache modules?

So, Apache (or httpd) is very modular which I consider one of its biggest strength. But how can I use 3rd party modules like mod_xsendfile?

You can create a derived Docker image which installs mod_xsendfile, but the image doesn't use the system Apache packages as they do not support HTTP/2. This means you cannot simply install any mod_xsendfile packaged by the Linux distribution. You would instead need to down mod_xsendfile source yourself and build it using the Apache version which has been installed.
That said, there are few issues.
The first is whether I should consider including mod_xsendfile myself since you aren't likely to be the latest to ask.
The second is though that depending on your requirements, mod_wsgi in daemon mode (as it runs with this Docker image), has something similar to mod_xsendfile. That os, you can return an empty 200 response with a Location header with a URL path to a subsequent resource to be served up.
So it isn't a file system path and as a result the URL must be able to be mapped by Apache, but it can map to static files. If those static files need to be private and not directly accessible, there is a little bit of extra config required to protect them. The 200/Location mechanism may not be supported by extensions for frameworks for doing this sort of delegation of file serving.
The third is that mod_wsgi-docker is heading towards being marked as deprecated. I have a much better version of the image which does things a lot better and is much more flexible. Depending on how you are using the current image, you may need to tweak some things, but otherwise shouldn't be that hard to move to it when it is marked as ready for use.
So quickest thing initially can do is see whether can use the 200/Location mechanism instead. What WSGI framework are you using and are you using some extension for that which tries to wrap up this functionality, or are you setting headers directly?

Thanks for replying.
I wasn't using anything before, so my question came up while evaluating options, in the end I went with httpd's image and installed mod_wsgi and mod_xsendfile using apxs.
I think it's crucial though that an image based on httpd supports other modules.


Update zigbee2mqtt alpine

Please verify that you have read and understood the guidelines.
yes
A clear and concise description of the issue.
There is a newer version of zigbee2mqtt on alpine:
https://pkgs.alpinelinux.org/packages?name=zigbee2mqtt&branch=edge&repo=&arch=x86_64&origin=&flagged=&maintainer=

but if it run your update script i am not getting 1.40.2

What settings are you currently utilizing?
Default Settings
Which Linux distribution are you employing?
Alpine
If relevant, including screenshots or a code block can be helpful in clarifying the issue.
No response
Please provide detailed steps to reproduce the issue.
No response

The Alpine package of zigbee2mqtt has indeed fallen behind the current release, and is missing support for a number of devices.
The reason the script does not install the version you are wanting, is because it installs from the stable repository.
So you could either wait for the package to make its way to the next stable release, or upgrade your existing container to the edge:
https://wiki.alpinelinux.org/wiki/Include:Upgrading_to_Edge
Upgrading to edge is pretty straightforward, but its essentially a testing version of alpine so it might have bugs. If you want to do an upgrade and have issues, I can try to help.

The Alpine package of zigbee2mqtt has indeed fallen behind the current release, and is missing support for a number of devices.
The reason the script does not install the version you are wanting, is because it installs from the stable repository.
So you could either wait for the package to make its way to the next stable release, or upgrade your existing container to the edge:
https://wiki.alpinelinux.org/wiki/Include:Upgrading_to_Edge
Upgrading to edge is pretty straightforward, but its essentially a testing version of alpine so it might have bugs. If you want to do an upgrade and have issues, I can try to help.

thx i will test this

I was in the same place a while back and in the end exported the zigbee config and imported into a debian rig.  It uses a few more resources, but not too excessive


Running the custom kernel example - error when starting kernel from JupyterLab

When going through the custom kernel example everything starts correctly in k8s, but when starting the kernel from the notebook it produced a 500 HTTP error:

Seems like the error is some version incompatibility with enterprise gateway. Here are the logs from the gateway:
Error: TypeError: __init__() got an unexpected keyword argument 'kernel_id'
 HTTPServerRequest(protocol='http', host='jupytergateway-elastic-with-custom-kernels.default:8888', method='POST', uri='/api/kernels', version='HTTP/1.1', remote_ip='<IP_ADDRESS>')
Traceback (most recent call last):
File "/opt/conda/lib/python3.9/site-packages/tornado/web.py", line 1704, in _execute
result = await result
File "/opt/conda/lib/python3.9/site-packages/enterprise_gateway/services/kernels/handlers.py", line 90, in post
await super(MainKernelHandler, self).post()
File "/opt/conda/lib/python3.9/site-packages/jupyter_server/services/kernels/handlers.py", line 47, in post
kernel_id = await km.start_kernel(kernel_name=model["name"], path=model.get("path"))
File "/opt/conda/lib/python3.9/site-packages/enterprise_gateway/services/kernels/remotemanager.py", line 156, in start_kernel
kernel_id = await super(RemoteMappingKernelManager, self).start_kernel(*args, **kwargs)
File "/opt/conda/lib/python3.9/site-packages/jupyter_server/services/kernels/kernelmanager.py", line 211, in start_kernel
kernel_id = await ensure_async(self.pinned_superclass.start_kernel(self, **kwargs))
File "/opt/conda/lib/python3.9/site-packages/jupyter_server/utils.py", line 189, in ensure_async
result = await obj
File "/opt/conda/lib/python3.9/site-packages/jupyter_client/multikernelmanager.py", line 194, in _async_start_kernel
await fut
File "/opt/conda/lib/python3.9/site-packages/jupyter_client/multikernelmanager.py", line 165, in _add_kernel_when_ready
await kernel_awaitable
File "/opt/conda/lib/python3.9/site-packages/jupyter_client/utils.py", line 34, in ensure_async
return await obj
File "/opt/conda/lib/python3.9/site-packages/enterprise_gateway/services/kernels/remotemanager.py", line 365, in start_kernel
await super(RemoteKernelManager, self).start_kernel(**kwargs)
File "/opt/conda/lib/python3.9/site-packages/jupyter_client/manager.py", line 362, in _async_start_kernel
raise e
File "/opt/conda/lib/python3.9/site-packages/jupyter_client/manager.py", line 351, in _async_start_kernel
await ensure_async(self._launch_kernel(kernel_cmd, **kw))
File "/opt/conda/lib/python3.9/site-packages/jupyter_client/utils.py", line 34, in ensure_async
return await obj
File "/opt/conda/lib/python3.9/site-packages/enterprise_gateway/services/kernels/remotemanager.py", line 425, in _launch_kernel
proxy = await self.process_proxy.launch_process(kernel_cmd, **kwargs)
File "/opt/conda/lib/python3.9/site-packages/enterprise_gateway/services/processproxies/kubeflow.py", line 49, in launch_process
await super(KubeflowProcessProxy, self).launch_process(kernel_cmd, **kwargs)
File "/opt/conda/lib/python3.9/site-packages/enterprise_gateway/services/processproxies/container.py", line 81, in launch_process
self.local_proc = self.launch_kernel(kernel_cmd, **kwargs)
File "/opt/conda/lib/python3.9/site-packages/enterprise_gateway/services/processproxies/processproxy.py", line 456, in launch_kernel
return launch_kernel(cmd, **kwargs)
File "/opt/conda/lib/python3.9/site-packages/jupyter_client/launcher.py", line 169, in launch_kernel
raise ex
File "/opt/conda/lib/python3.9/site-packages/jupyter_client/launcher.py", line 157, in launch_kernel
proc = Popen(cmd, **kwargs)
TypeError: __init__() got an unexpected keyword argument 'kernel_id'
[E 220201 17:06:22 web:2239] 500 POST /api/kernels (<IP_ADDRESS>) 23.14ms

This seems related to this issue: https://github.com/jupyter/enterprise_gateway/issues/1024
Seems like it's using jupyter-client 7.1.0, which might be the problem? I'll try to confirm

Can you please show the image tag of the gateway and kernel?

Hello, could I know whether you solve this problem? I meet the same issue, I check the version of jupyter_client is 7.1.0. Could changing the version of jupyter_client work. But I try to deploy with enterprise-gateway alone, that works. I 'll appreciate it if you could help.

Can you please show the image tag of the gateway and kernel?

Hello, Mr. Gao. I meet the same question. I try the demo as quick-start but it doesn't work. The tags of my gateway and kernel are both 2.6.0. I want to know whether some version conflict exists. I have tried deploying only with enterprise-gateway and it works. I am interested in what kind of differences between your images and theirs. I am sorry I cannot access your image information and I do not obtain the answer in your code. I'll appreciate it if you could help.

OK I will verify it today.

@ycsong1212 Did you follow the guide here? https://github.com/skai-x/elastic-jupyter-operator/tree/master/docs#remote-kernels
Remote kernel or Remote kernels with custom configuration

Or this https://github.com/skai-x/elastic-jupyter-operator/blob/master/docs/quick-start.md

@ycsong1212 Did you follow the guide here? https://github.com/skai-x/elastic-jupyter-operator/tree/master/docs#remote-kernels
Remote kernel or Remote kernels with custom configuration

I think I do. First I apply crds by prepare.yaml and then make deploy. I tried elastic-with-custom-kernels demo. I apply four crds under examples/elastic-with-custom-kernels directory.  Then I port-forward my notebook pod. When I launch kernel, kernel error occurs. Such error is reported in gateway container.

I have same error， how can i fix it?

TypeError: init() got an unexpected keyword argument 'kernel_id'

It looks like a version problem related to enterprise gateway.

@gaocegege Let's try to fix the issue

This repo is not maintained?

This repo is not maintained?

The org and repository belong to Tencent Cloud. And I am not sure if there is actively maintained.

When going through the custom kernel example everything starts correctly in k8s, but when starting the kernel from the notebook it produced a 500 HTTP error:

Seems like the error is some version incompatibility with enterprise gateway. Here are the logs from the gateway:
Error: TypeError: __init__() got an unexpected keyword argument 'kernel_id'
 HTTPServerRequest(protocol='http', host='jupytergateway-elastic-with-custom-kernels.default:8888', method='POST', uri='/api/kernels', version='HTTP/1.1', remote_ip='<IP_ADDRESS>')
Traceback (most recent call last):
File "/opt/conda/lib/python3.9/site-packages/tornado/web.py", line 1704, in _execute
result = await result
File "/opt/conda/lib/python3.9/site-packages/enterprise_gateway/services/kernels/handlers.py", line 90, in post
await super(MainKernelHandler, self).post()
File "/opt/conda/lib/python3.9/site-packages/jupyter_server/services/kernels/handlers.py", line 47, in post
kernel_id = await km.start_kernel(kernel_name=model["name"], path=model.get("path"))
File "/opt/conda/lib/python3.9/site-packages/enterprise_gateway/services/kernels/remotemanager.py", line 156, in start_kernel
kernel_id = await super(RemoteMappingKernelManager, self).start_kernel(*args, **kwargs)
File "/opt/conda/lib/python3.9/site-packages/jupyter_server/services/kernels/kernelmanager.py", line 211, in start_kernel
kernel_id = await ensure_async(self.pinned_superclass.start_kernel(self, **kwargs))
File "/opt/conda/lib/python3.9/site-packages/jupyter_server/utils.py", line 189, in ensure_async
result = await obj
File "/opt/conda/lib/python3.9/site-packages/jupyter_client/multikernelmanager.py", line 194, in _async_start_kernel
await fut
File "/opt/conda/lib/python3.9/site-packages/jupyter_client/multikernelmanager.py", line 165, in _add_kernel_when_ready
await kernel_awaitable
File "/opt/conda/lib/python3.9/site-packages/jupyter_client/utils.py", line 34, in ensure_async
return await obj
File "/opt/conda/lib/python3.9/site-packages/enterprise_gateway/services/kernels/remotemanager.py", line 365, in start_kernel
await super(RemoteKernelManager, self).start_kernel(**kwargs)
File "/opt/conda/lib/python3.9/site-packages/jupyter_client/manager.py", line 362, in _async_start_kernel
raise e
File "/opt/conda/lib/python3.9/site-packages/jupyter_client/manager.py", line 351, in _async_start_kernel
await ensure_async(self._launch_kernel(kernel_cmd, **kw))
File "/opt/conda/lib/python3.9/site-packages/jupyter_client/utils.py", line 34, in ensure_async
return await obj
File "/opt/conda/lib/python3.9/site-packages/enterprise_gateway/services/kernels/remotemanager.py", line 425, in _launch_kernel
proxy = await self.process_proxy.launch_process(kernel_cmd, **kwargs)
File "/opt/conda/lib/python3.9/site-packages/enterprise_gateway/services/processproxies/kubeflow.py", line 49, in launch_process
await super(KubeflowProcessProxy, self).launch_process(kernel_cmd, **kwargs)
File "/opt/conda/lib/python3.9/site-packages/enterprise_gateway/services/processproxies/container.py", line 81, in launch_process
self.local_proc = self.launch_kernel(kernel_cmd, **kwargs)
File "/opt/conda/lib/python3.9/site-packages/enterprise_gateway/services/processproxies/processproxy.py", line 456, in launch_kernel
return launch_kernel(cmd, **kwargs)
File "/opt/conda/lib/python3.9/site-packages/jupyter_client/launcher.py", line 169, in launch_kernel
raise ex
File "/opt/conda/lib/python3.9/site-packages/jupyter_client/launcher.py", line 157, in launch_kernel
proc = Popen(cmd, **kwargs)
TypeError: __init__() got an unexpected keyword argument 'kernel_id'
[E 220201 17:06:22 web:2239] 500 POST /api/kernels (<IP_ADDRESS>) 23.14ms

This seems related to this issue: jupyter-server/enterprise_gateway#1024
Seems like it's using jupyter-client 7.1.0, which might be the problem? I'll try to confirm

I encountered the same issue, have you resolved it?


auth-client: login onSuccess async

Is your feature request related to a problem? Please describe.
According README the function login should be use as follow:
authClient.login({
  onSuccess: async () => {
    // authClient now has an identity
  },
});

To the contrary, currently implemented, the function is async and the callback is void. Therefore the proper example would be:
await authClient.login({
  onSuccess: () => {
    // authClient now has an identity
  },
});

Nevertheless, it would probably be for the best if both would support async, respectively:
await authClient.login({
  onSuccess: await () => {
    // authClient now has an identity
  },
});

Describe the solution you'd like
A description which match the implementation and I suggest also to have both function and callback support async.

login already returns an Promise, why do we need the additional onSuccess and onError callbacks?
I'm wrapping the call up like this now, but it would be easier if I could just use the returned promise from login.
  await new Promise((resolve, reject) => {
    authClient.login({
      onSuccess: resolve,
      onError: reject
    })
  })

Would you consider simplifying the login API so that the authClient.login just returns a Promise<Identity> and then we can just await the result. And for errors, we can wrap the login call in a try-catch.
Thoughts?
@krpeacock


Retry submariner connectivity check

Try to fix the connectivity failures in submariner test with retries. This may be enough if the issue is temporary, and if it is not not, this will provide more info to submariner developers.
The issue was not reproduce with submariner 0.17.0.
Tested:

200 runs with submariner.yaml - 100% passed
100 runs with regional-dr.yaml - 100% passed

Average time for connectivity check in 100 runs of regional-dr.yaml:
$ grep 'Access completed' e2e-rdr/out.0{3,4}/*.log | awk '{sum+=$8} END {print sum/NR}'
0.206505

Based on #1274 for testing.

Example log showing the fix in action:
$ grep -A10 'Failed to connect' out.06/*.log
out.06/046.log:2024-04-05 01:24:03,127 DEBUG   [hub/1] Failed to connect: Command failed:
out.06/046.log-2024-04-05 01:24:03,127 DEBUG   [hub/1]    command: ('kubectl', 'exec', '--context', 'dr2', 'test', '--namespace=ns1', '--', 'curl', '--no-progress-meter', 'nginx.ns1.svc.clusterset.local')
out.06/046.log-2024-04-05 01:24:03,127 DEBUG   [hub/1]    exitcode: 6
out.06/046.log-2024-04-05 01:24:03,127 DEBUG   [hub/1]    error:
out.06/046.log-2024-04-05 01:24:03,127 DEBUG   [hub/1]       curl: (6) Could not resolve host: nginx.ns1.svc.clusterset.local
out.06/046.log-2024-04-05 01:24:03,127 DEBUG   [hub/1]       command terminated with exit code 6
out.06/046.log-2024-04-05 01:24:03,127 DEBUG   [hub/1]
out.06/046.log-2024-04-05 01:24:03,127 DEBUG   [hub/1] Retrying in 1 seconds
out.06/046.log-2024-04-05 01:24:04,127 DEBUG   [hub/1] Accessing 'nginx.ns1.svc.clusterset.local' in cluster 'dr2'
out.06/046.log:2024-04-05 01:24:04,305 DEBUG   [hub/1] Failed to connect: Command failed:
out.06/046.log-2024-04-05 01:24:04,305 DEBUG   [hub/1]    command: ('kubectl', 'exec', '--context', 'dr2', 'test', '--namespace=ns1', '--', 'curl', '--no-progress-meter', 'nginx.ns1.svc.clusterset.local')
out.06/046.log-2024-04-05 01:24:04,305 DEBUG   [hub/1]    exitcode: 6
out.06/046.log-2024-04-05 01:24:04,305 DEBUG   [hub/1]    error:
out.06/046.log-2024-04-05 01:24:04,305 DEBUG   [hub/1]       curl: (6) Could not resolve host: nginx.ns1.svc.clusterset.local
out.06/046.log-2024-04-05 01:24:04,305 DEBUG   [hub/1]       command terminated with exit code 6
out.06/046.log-2024-04-05 01:24:04,305 DEBUG   [hub/1]
out.06/046.log-2024-04-05 01:24:04,305 DEBUG   [hub/1] Retrying in 2 seconds
out.06/046.log-2024-04-05 01:24:06,308 DEBUG   [hub/1] Accessing 'nginx.ns1.svc.clusterset.local' in cluster 'dr2'
out.06/046.log:2024-04-05 01:24:06,462 DEBUG   [hub/1] Failed to connect: Command failed:
out.06/046.log-2024-04-05 01:24:06,462 DEBUG   [hub/1]    command: ('kubectl', 'exec', '--context', 'dr2', 'test', '--namespace=ns1', '--', 'curl', '--no-progress-meter', 'nginx.ns1.svc.clusterset.local')
out.06/046.log-2024-04-05 01:24:06,462 DEBUG   [hub/1]    exitcode: 6
out.06/046.log-2024-04-05 01:24:06,462 DEBUG   [hub/1]    error:
out.06/046.log-2024-04-05 01:24:06,462 DEBUG   [hub/1]       curl: (6) Could not resolve host: nginx.ns1.svc.clusterset.local
out.06/046.log-2024-04-05 01:24:06,462 DEBUG   [hub/1]       command terminated with exit code 6
out.06/046.log-2024-04-05 01:24:06,462 DEBUG   [hub/1]
out.06/046.log-2024-04-05 01:24:06,462 DEBUG   [hub/1] Retrying in 4 seconds
out.06/046.log-2024-04-05 01:24:10,462 DEBUG   [hub/1] Accessing 'nginx.ns1.svc.clusterset.local' in cluster 'dr2'
out.06/046.log:2024-04-05 01:24:10,644 DEBUG   [hub/1] Failed to connect: Command failed:
out.06/046.log-2024-04-05 01:24:10,645 DEBUG   [hub/1]    command: ('kubectl', 'exec', '--context', 'dr2', 'test', '--namespace=ns1', '--', 'curl', '--no-progress-meter', 'nginx.ns1.svc.clusterset.local')
out.06/046.log-2024-04-05 01:24:10,645 DEBUG   [hub/1]    exitcode: 6
out.06/046.log-2024-04-05 01:24:10,645 DEBUG   [hub/1]    error:
out.06/046.log-2024-04-05 01:24:10,645 DEBUG   [hub/1]       curl: (6) Could not resolve host: nginx.ns1.svc.clusterset.local
out.06/046.log-2024-04-05 01:24:10,645 DEBUG   [hub/1]       command terminated with exit code 6
out.06/046.log-2024-04-05 01:24:10,645 DEBUG   [hub/1]
out.06/046.log-2024-04-05 01:24:10,645 DEBUG   [hub/1] Retrying in 8 seconds
out.06/046.log-2024-04-05 01:24:18,645 DEBUG   [hub/1] Accessing 'nginx.ns1.svc.clusterset.local' in cluster 'dr2'
out.06/046.log-2024-04-05 01:24:18,803 DEBUG   [hub/1] Access completed in 15.838 seconds


feat: ability to have the rc file as json or common js module

I want to put comments in the rc files, but the json does not allow it
filenames added there are wide open for discussion
Tests are currently failing, will get to that, just wanted to open a discussion on the topic.

yeah i think json or js makes sense.
would just go for .nodegenrc.js and .nodegenrc.json though, and as a back compat just check for .nodegenrc if neither of those are found.
It's a bit more difficult for the reader, but if you have either .nodegenrc.json (json) or .nodegenrc.js (cjs) and your code does
require('.nodegenrc') then it will import and parse whichever is present

Again, it's a bit magic, but would let the user do either, however if only .nodegenrc exists and it's json, the require will fail


fix: auto fill should delete dv rules

close https://github.com/dream-num/univer-pro/issues/2432
close https://github.com/dream-num/univer-pro/issues/3016

Pull Request Checklist

[ ] Related tickets or issues have been linked in the PR description (or missing issue).
[ ] Naming convention is followed (do please check it especially when you created new plugins, commands and resources).
[ ] Unit tests have been added for the changes (if applicable).
[ ] Breaking changes have been documented (or no breaking changes introduced in this PR).

@lumixraku 这个下拉填充未修复，粘贴后数据验证也有问题 https://github.com/dream-num/univer-pro/issues/2432
2024_11_21 11_58_56 video.webm
预期：粘贴和填充后选区被拆分为多个

Google sheet 自动填充可以复制现有的数据验证配置


Error loading profile

When installing the module using
Install-Module WslInterop

adding
Import-WslCommand "awk", "emacs", "grep", "head", "less", "ls", "man", "sed", "seq", "ssh", "tail", "vim", "bash"
$WslDefaultParameterValues["grep"] = "-E"
$WslDefaultParameterValues["less"] = "-i"
$WslDefaultParameterValues["ls"] = "-AFh --group-directories-first"
$WslDefaultParameterValues["bash"] = "--login --init-file ~/.nix-profile/etc/profile.d/nix.sh"

and the reloading profile I get a bunch of:

Cannot index into a null array.
At ...\Documents\PowerShell\Microsoft.PowerShell_profile.ps1:8 char:1

$WslDefaultParameterValues["grep"] = "-E"

CategoryInfo          : InvalidOperation: (:) [], RuntimeException
FullyQualifiedErrorId : NullArray

My bad, just saw that), you also mentioned it.

Thx @mikebattista for the great module it is outstanding in easing my work, can i. e. use bash -c 'nix-shell --pure -p pandoc -p texlive.combined.scheme-small' instead of tedious custom functions in Profile.ps1 or even make an alias for that now.

Glad you find this project helpful.
For the nix-shell command question, could you clarify what it is you want to be able to do? What is it that you want to type into PowerShell and what do you want that to do?

I. e. when I wanted a shell with pandoc and pdflatex available I typed that command (I even could wrap it further into a pwsh function to just have to type pandoc -o cheatsheet.pdf cheatsheet.tex directly without the extra prompt but to show that function definition would surely bloat the scope of a demo) to get a prompt where I typed:
pandoc -o cheatsheet.pdf cheatsheet.tex

Thus I can convert a tex file from the internet to a pdf using tex and everything just under the hood, basically staying in my powershell session.
Does that sound weird or complicated ?

Sounds like if you have the pandoc package installed within WSL, you should be able to add Import-WslCommand "pandoc" to your PowerShell profile to replace your custom function. Then you should be able to call it as shown in all the demos without needing to leave your PowerShell session.
Would that work for you?

Thanks, it's fine. Just works.


Norwegian translation issue

Norwegian translation issue, that breaks the ui in March.
"mar." should be "mars"

bump @chillu any thoughts on this?

Given that the upstream jQuery UI lib has changed from "mar" to "mars", I'm happy to change this in our repo as well. Even though it temporarily "forks" the lib, once we update jQuery UI the change will be part of upstream - so low maintenance/risk.


BEATS models links are down

All models in https://github.com/microsoft/unilm/tree/master/beats are unreachable
e.g. https://valle.blob.core.windows.net/share/BEATs/BEATs_iter3_plus_AS2M_finetuned_on_AS2M_cpt2.pt?sv=2020-08-04&st=2023-03-01T07%3A51%3A05Z&se=2033-03-02T07%3A51%3A00Z&sr=c&sp=rl&sig=QJXmSJG9DbMKf48UDIU1MfzIro8HQOf3sqlNXiflY1I%3D

closing as it is a duplicate of: https://github.com/microsoft/unilm/issues/1492


Fix: fix frontend dependencies

add typescript as devDependency for frontend (worked while testing, since it was still present in my node_modules dir)
resolve issue with duplicate react router dependency
prevent react-scripts from erroring on eslint issues
use react18's method createRoot instead of ReactDOM.render

Thanks !
So this means we go from React v17 to React v18, I hope it will be OK :-)

So this means we go from React v17 to React v18, I hope it will be OK :-)
React 18 was installed already, I just changed the way, it is initialised (previously it gave out a deprecation warning and said that it was behaving the previous version's way) so I thought this way react and its dependencies might behave more predictable.


Bullet and Numbering

Can i use bullet and numbering ?
Thank you.

You can use both bullets and numbering in the editor.

Hello,
Sorry but how to create like that i tried but to search the option but i cant find it. or i need to use embed code ? Thank you sir.

Nope, all you need to do is use the - (dash) or 1 (number) in the editor. Hitting Enter to a new line takes the list and continues it as either ordered or unordered.


fix: created correct raw-callsite for hooks

[closes #7089]
Purpose
actionId for hooks in raw-tests has a prefix, that is why it is impossible to parse it as Integer and impossible to create a callsite. It needs to use actionId without parsing.
Approach

Add test for error in hooks
Fix using actionId

References
https://github.com/DevExpress/testcafe/issues/7089
Pre-Merge TODO

[X] Write tests for your proposed changes
[x] Make sure that existing tests do not fail

please include it in 2.0.1 minor


Add row actions to App & Policy search results

Related bug: https://issues.redhat.com/browse/ACM-7075

/test unit-tests-sonarcloud


Document mvn cq:await-release -Dcq.version=$VERSION

I can confirm that at least aws2-sqs is ran using localstack by default (and completes successfully).


updated generate docs

Adding information about the flags for generate.

Pull Request Test Coverage Report for Build 542

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

Totals

Change from base Build 535:
0.0%

Covered Lines:
684

Relevant Lines:
1759

💛  - Coveralls


Fix shared model conditional compilation

Signed-off-by: Victor Drobny<EMAIL_ADDRESS>

Description of the Change
Fixes for conditional compilation shared_model

SonarQube analysis reported 9 issues

 blob.hpp#L34: Undocumented API: toBinaryString
 blob.hpp#L41: Undocumented API: Bytes
 blob.hpp#L55: Undocumented API: Blob
 blob.hpp#L125: Complete the task associated to this TODO comment.
 blob.hpp#L130: Undocumented API: toBinaryString
 keypair.hpp#L45: Undocumented API: Keypair
 keypair.hpp#L65: Undocumented API: privateKey
 keypair.hpp#L91: Undocumented API: public_key_
 keypair.hpp#L92: Undocumented API: private_key_

SonarQube analysis reported 9 issues

 blob.hpp#L34: Undocumented API: toBinaryString
 blob.hpp#L41: Undocumented API: Bytes
 blob.hpp#L55: Undocumented API: Blob
 blob.hpp#L125: Complete the task associated to this TODO comment.
 blob.hpp#L130: Undocumented API: toBinaryString
 keypair.hpp#L45: Undocumented API: Keypair
 keypair.hpp#L65: Undocumented API: privateKey
 keypair.hpp#L91: Undocumented API: public_key_
 keypair.hpp#L92: Undocumented API: private_key_


Reduce Gap issue on Center Scroll Layout Manager.

I want to reduce the gap size b/w items using center scroll layout manager because of that i have only 5 items in recyclerview and first one and the last one is just empty view. I also want that recyclerview not move to the empty views.

I have solved my problem while implementing this part of the code.I also write a wrapper on top of CarouselLayoutManager for simplifying my solution to this problem.
layoutManager.setPostLayoutListener(new PostLayoutListener() {

            @Override
            public ItemTransformation transformChild(@NonNull final View child, @NonNull final RecyclerView.ViewHolder currentItem,
                                                     @NonNull final RecyclerView.ViewHolder previousItem,
                                                     final LayoutOrder layoutOrder, final int orientation) {
                // because scaling will make view smaller in its center, then we should move this item to the top or bottom to make it visible
                float translateX;
                float itemPositionToCenterDiff = layoutOrder.mItemPositionDiff;

                translateX = child.getWidth() * itemPositionToCenterDiff * .1f;

                return new ItemTransformation(1.0f, 1.0f, translateX, 0.0f);

            }
        });


chore(main): release 0.0.2

:robot: I have created a release beep boop
0.0.2 (2024-05-30)
Full Changelog: https://github.com/QuivrHQ/MegaParse/compare/v0.0.1...v0.0.2

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/QuivrHQ/MegaParse/releases/tag/v0.0.2 :sunflower:


Hardcode smee.io

Now that we have smee.io set up, I think it makes sense to simplify setting the channel by hardcoding https://smee.io as the webhook proxy URL, then providing a SMEE_CHANNEL environment variable.
For ease of testing, I've kept the url property passed to createWebhookProxy - this can also be used to enable another environment variable to set a custom webhook proxy host. I haven't included that yet; I don't want to add more env vars to the system unless necessary. Let me know what y'all think about that.
I'd like to write docs that instruct on how to do all of this, but I'll wait until this PR is resolved since it will change the instructions.

👍 once smee.io is solid and we're recommending it in production for folks. Until then, I think it's helpful to provide a full URL (I've already done it a few times to test changes to smee).
What do you think about letting this set for a week or so until we drop localtunnel in place of smee?

Yep that's cool with me. Just wanted to open this up to an eventual merge.

Any thoughts on automatically creating the URL? I can go directly to https://smee.io/whatever-i-want and get my own unique channel, so there's no reason we have to make users go to smee.io and create a random channel. One thought I had was to use their github username + the WEBHOOK_SECRET variable. So the default could be https://smee.io/tcbyrd-development or we could generate a hash.

Ooh @tcbyrd that's a great idea. The only issue I can see is conflicting channels (two people might want to listen on /probot, which would be a commonly used channel I bet), so we'd want to include some kind of way to inform users on how many things are listening on the channel.
I like the GitHub username. I don't think the WEBHOOK_SECRET is totally necessary but for the purposes of namespacing I get it.

two people might want to listen on /probot
My point was, if we just make the unique channel for them, they probably won't even go make their own channel. Whatever routes they listen for on the deployed app don't really matter for smee, right?

My point was, if we just make the unique channel for them, they probably won't even go make their own channel. Whatever routes they listen for on the deployed app don't really matter for smee, right?

Yep! That sounds about right. Any kind of namespacing to help make it unique is a good thing

My only issue with encouraging people to specify their URL manually is that it's easily guessable, and you have no clue if someone is eavesdropping on your channel. With localtunnel, you at least get an error when you try to connect if someone else is using the channel name.
I opened an issue on https://github.com/probot/webhooks/issues/27 to discuss secure channels.

For now I think it's been working fine to use the full URL. Closing this for now, but feel free to re-open if you want to get this up to date with master and make it backward compatible so people using URLs continues to work.

@bkeepers how about implementing this as a progressive enhancement. People who are using smee.io can provide SMEE_CHANNEL (if they know about this) to get a working smee url, autoconfigured for listening to github webhooks otherwise they can keep on using WEBHOOK_PROXY_URL.

I have not personally felt any pain with using the full url, and I’m not sure the benefit of supporting both options. If other people feel strongly, feel free to share your thoughts.

After working with the full URL for a while, I've also felt that this change isn't really necessary.
Especially as we dive into authenticated channels in Smee, it helps to be more declarative and clear with whats going on. It also leaves Probot open to other, future webhook proxy services that aren't Smee, should they ever exist.


Presence of gemfile does not necessarily imply that bundler is being used

Many ruby projects nowadays support or make use of bundler in some fashion. I don't use bundler and don't have it installed. Usually this is does not present a problem: after putting the bundler require in a begin-rescue block, I found that tests can still be run and packages created. Today however I discovered that yajl-ruby cannot be tested without bundler, due to rspec performing a bundle exec: https://gist.github.com/dbbac9db30cce38f5016
Presence of a gemfile does not mean the user running the code is using bundler, or has it at all.

@indirect, @wycats, @carllerche - any reason rspec-core should not depend on BUNDLE_GEMFILE as proposed above?

@indirect, @wycats, @carllerche - I'm planning to release this in an rc, but am willing to roll back if you guys convince me that's the right way to go.

If BUNDLE_GEMFILE is already set by Bundler, then it's superfluous to invoke bundle exec. Anytime Bundler has set up the ENV, it has also set up RUBYOPT, and therefore any ruby process will automatically be set up to use the bundled gems. So while BUNDLE_GEMFILE is a great way to tell if you're already inside a bundled environment, but if it's there you already don't need to do anything.
The only exception I can think of is some of the more esoteric use-cases that involve setting BUNDLE_GEMFILE manually before Bundler has been invoked, to ensure that Bundler will find the correct Gemfile when it eventually loads.
On Oct 9, 2011, at 6:31 AM, David<EMAIL_ADDRESS>wrote:

@indirect, @wycats, @carllerche - I'm planning to release this in an rc, but am willing to roll back if you guys convince me that's the right way to go.
--
Reply to this email directly or view it on GitHub:
https://github.com/rspec/rspec-core/issues/454#issuecomment-2337216

@indirect - just so I'm clear: if rake is invoked with bundle exec, or if Bundler.setup appears in the Rakefile, when the rspec rake task shells out using rspec, it will operate as though the command was bundle exec rspec. Correct?

@indirect re: the edge case you mention, that should still work the same way, no? i.e. if I run BUNDLE_GEMFILE=special.gemfile rake spec, the subshell will get BUNDLE_GEMFILE, and bundler will also have set RUBYOPT so the subshell runs using bundler with the gemfile prescribed in the parent shell. Right?

Yes on both counts. Here's an example of how it works (and why rspec can, afaict, just default to skip_bundler =true at all times): https://gist.github.com/8f59bedcd961c3da1c28
As that example hopefully shows, subshells inherit the bundle, so rspec doesn't need to include bundle exec. As an upside, that will speed up each rspec invocation, too. :)


Forms with unicode file names fail upload

Test file: tést.xlsx.zip
Central reports Something went wrong: there was no request.
Here's what I know so far...

Conversion with latest xlsform.getodk.org works, but Enketo preview fails with "Bad Request. Server URL is not a valid URL."
Conversion with curl to latest pyxform-http works

curl --request POST --data-binary @/path/to/tést.xlsx http://<IP_ADDRESS>:5000/api/v1/convert

Conversion with xls2xform works
Uploading the XML to Central works and so does the Enketo preview
Nothing obvious in Central logs

My gut says this is some interaction with Central and pyxform-http.

When I look at the browser console, I see the error message

Failed to execute 'setRequestHeader' on 'XMLHttpRequest': String contains non ISO-8859-1 code point.

Reading about it, I think the issue has to do with the X-XlsForm-FormId-Fallback header containing a non-ASCII character.
Is a form ID allowed to contain a non-ASCII character? If so, then I think we should encode the filename before specifying it in the header. Probably percent-encoding would be easiest? Frontend would need to encode the header, and pyxform-http would need to decode it, but I'm not sure that Backend would need to change.
We generally encode form IDs throughout Central, and Enketo links in Central use tokens separate from the form ID, so hopefully this would be a one-off change.

When I look at the browser console, I see the error message

Failed to execute 'setRequestHeader' on 'XMLHttpRequest': String contains non ISO-8859-1 code point.

Reading about it, I think the issue has to do with the X-XlsForm-FormId-Fallback header containing a non-ASCII character.
Is a form ID allowed to contain a non-ASCII character? If so, then I think we should encode the filename before specifying it in the header. Probably percent-encoding would be easiest? Frontend would need to encode the header, and pyxform-http would need to decode it, but I'm not sure that Backend would need to change.
We generally encode form IDs throughout Central, and Enketo links in Central use tokens separate from the form ID, so hopefully this would be a one-off change.

Is a form ID allowed to contain a non-ASCII character

Yes, it is.
Your analysis seems right to me and percent-encoding between frontend and pyxform-http seems like the right approach.

Is a form ID allowed to contain a non-ASCII character

Yes, it is.
Your analysis seems right to me and percent-encoding between frontend and pyxform-http seems like the right approach.

Sounds good! @yanokwa, are you able to add percent-decoding to pyxform-http? If so, I'll plan to add percent-encoding to Frontend.

Sounds good! @yanokwa, are you able to add percent-decoding to pyxform-http? If so, I'll plan to add percent-encoding to Frontend.

As an update, as part of v1.2, we will have a new header, X-Action-Notes, that will need to be percent-encoded. Given that, I do think it makes for Frontend to percent-encode X-XlsForm-FormId-Fallback, as long as it's not hard for pyxform-http to decode it.

Related: https://github.com/getodk/collect/issues/4554


Sonar fixes

Changes from dependabot

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


Fix Fizz query

Fix Fizz query after a backwards-incompatible change happened in the IR dataflow library.
See https://discuss.lgtm.com/t/the-fizz-example-doesnt-work/2384
The backwards-incompatible change happened here: https://github.com/Semmle/ql/pull/1888

LGTM


@apply() does not seem to work within custom-style

In my element:
  <style>
    :host {
      @apply(--mflo-spinning-numbers-theme);
    }
  </style

In my document:
  <style is="custom-style">
    :host {
      --mflo-spinning-numbers-theme {
        background-color: red;
        color: blue;
        display: inline-block;
        font-size: 32px;
        min-width: 280px;
      };
    }
  </style>

None of the styles are applied. Am I coding it incorrectly?

<style is="custom-style"> is a custom component, so it can't be applied in the main document like a style tag.
Have a look at Polymer Starter Kit for an example implementation:
https://github.com/PolymerElements/polymer-starter-kit/blob/ba9ac0113c5a9f88bc5570e155beca91d122d5eb/app/elements/elements.html#L38
https://github.com/PolymerElements/polymer-starter-kit/blob/master/app/styles/app-theme.html
More info here:
https://www.polymer-project.org/1.0/docs/devguide/styling.html#custom-style

Many thanks for your reply but I must be misunderstanding it as I am successfully using custom-style in the main document to style my element in exactly the way the docs you reference describe.
But I can only do that by creating specific custom properties in my element for each property I want to style, as in:
:host {
  background-color: var(--mflo-spinning-numbers-background-color);
  color: var(--mflo-spinning-numbers-color);
  etc
}

As the Polymer docs point out, it's really impossible to enumerate all the properties so I want to use the mixin feature -- where I can define a grab bag and @apply() it as a whole.
I've been digging in the custom-style.html code and its dependencies but I haven't yet found out where @apply() might be handled.
I'll keep digging but also take any help I can get :)

The :host selector refers to the custom element itself and properties set there apply to the element's local dom. A custom-style has no local dom so it doesn't make sense to use :host. Instead if you'd like to set properties that apply to the entire document tree, you can use the :root selector. For example, try:
<style is="custom-style">
    :root {
      --mflo-spinning-numbers-theme {
        /* ... */
      };
    }
  </style>

Many thanks. Sadly, I still can't get @apply in my element to work using a theme set in the local DOM by custom-style -- but custom properties that my element uses via var() work just fine.
As the issue is closed I'll just assume it is me -- but I'll dig in the code more to try to see why.

When you have more information, please feel free to re-open the issue or open a new one if necessary.

@mflorence99 I've run into the same problem. Did you ever figure out what was wrong?

Here's an example in Plunker: http://plnkr.co/edit/rmu87kemXD1D8jE6Rfs0?p=preview
What am I doing wrong?

Hey -- it has been a long time since I looked at that! But I must've, as my
code is working at: http://mflo-spinning-numbers.surge.sh/demo.html
So I looked back at what I did, and this works:
In my element:

In my document:

If you compare that to what I wrote in the issue, I got the :host selector
wrong in the custom-style. Good luck!
On Fri, Dec 18, 2015 at 3:52 PM Justin<EMAIL_ADDRESS>wrote:

@mflorence99 https://github.com/mflorence99 I've run into the same
problem. Did you ever figure out what was wrong?
—
Reply to this email directly or view it on GitHub
https://github.com/Polymer/polymer/issues/2011#issuecomment-165895154.

@mflorence99 that wasn't my problem but your working example did help me. The problem I had, and your original code shares, is a missing : between the mixin name and the { when defining the mixins value. Thanks!

One word: doh!


[sql] Add support for iOS (SQLite)

Hello! 👋 I'm currently evaluating Tauri for a cross platform Android, iOS and Windows project.
However there is currently no iOS (SQLite) support for the sql plugin. Is this on the roadmap? If not, is there community interest in contributing toward this effort?

+1; came here looking for this exact information for the same reasons
I am potentially willing to lend development efforts to make this happen, if a maintainer can help identify the scope of the work required

More context here: https://github.com/tauri-apps/plugins-workspace/issues/1867

Thanks for the context @FabianLars. What’s the process for moving forward with adding iOS support?
If interest exists to get this done, is a member of the working group assigned to coordinate this effort or is it simply a matter of pushing a draft PR etc for discussion?

What’s the process for moving forward with adding iOS support?

Well again, i don't think anyone even tested the plugin on iOS yet, so that would be good a first step 🙃 Without that it's hard for me to suggest how to move forward.
So far i only know about an issue in the low level sqlite bindings when building for the ios simulator rust target aarch64-apple-ios-sim https://github.com/rusqlite/rusqlite/issues/1579

If interest exists to get this done, is a member of the working group assigned to coordinate this effort or is it simply a matter of pushing a draft PR etc for discussion?

At the moment I don't have any member available (including myself) to whom i could assign this to. So the more initiative there is from the community, the better.

@velocitysystems @FabianLars It seems to work on iOS. I've added the plugin by following the steps from the docs (https://v2.tauri.app/plugin/sql/):
npm run tauri add sql
cd src-tauri
cargo add tauri-plugin-sql --features sqlite

Then I added these two lines to capabilities/default.json:
"sql:default",
"sql:allow-execute",

After that, I was able to open and read a SQLite DB from TypeScript, running both in the iOS Simulator and on device:
import Database from '@tauri-apps/plugin-sql';

...

const db = await Database.load('sqlite:customers/customers.db');
const result = await db.select<Customer[]>(`SELECT ...`);

I'd venture to say that the docs can be updated to reflect that it works on iOS.

Thank you for testing it (and telling us about the result...)! I updated the docs just now (waiting for CI to get it live)


[entropy_src/doc] Checklist updates in preparation for V2 review

Signed-off-by: Martin Lueker-Boden<EMAIL_ADDRESS>

This PR closes #11315 .

Ah, what I forgot to mention earlier: this PR should also update the file hw/ip/entropy_src/data/entropy_src.prj/hjson to change "V1" -> "V2".

Ah, what I forgot to mention earlier: this PR should also update the file hw/ip/entropy_src/data/entropy_src.prj/hjson to change "V1" -> "V2".

Fair enough.  Though my hope is that this will also move to V2S immediately after!

Ah, what I forgot to mention earlier: this PR should also update the file hw/ip/entropy_src/data/entropy_src.prj/hjson to change "V1" -> "V2".

Fair enough. Though my hope is that this will also move to V2S immediately after!

Yes that is my hope, too. And it looks like we're going to make it :-)

Thanks for the review everyone, are there any other inputs before we merge this and call it V2?

Thanks for the review everyone, are there any other inputs before we merge this and call it V2?

Nothing from my side, I think we can move forward with this one :-D

Though, any idea why does the Cadence report show toggle coverage at ~82% even after the exclusion of the XHT interface?

The tool also takes toggles at internal modules (e.g., prim_counts) into account, even though it shouldn't. We'll try to resolve this with a new UNR run or, if that doesn't do the trick, with explicit exclusions. (If I understood correctly, our scripts/configs should automatically exclude such internal modules from toggle coverage, but we don't have time to debug that at this point.) We agreed to fix this as part of the V2S review action items (https://github.com/lowRISC/opentitan/issues/16330).

LGTM.
Though, any idea why does the Cadence report show toggle coverage at ~82% even after the exclusion of the XHT interface?
https://reports.opentitan.org/hw/ip/entropy_src/dv/latest/cov_report/index.html

I looked into this.  There are a number of 32-bit prim counters in entropy_src_core (example, u_entropy_src_core.u_entropy_src_cntr_reg_repcnts) that are not part of the health tests, rather they are there for statistics gathering.  They have no role in generating alerts, they are only for logging the overall prevalence of a particular type of health test failure.   It's the 32-bit outputs that are often not toggling (presumably in the higher bits).   Also these particular instances (not included in the health test modules) seem to not be exercised by the functional_errors test.

LGTM.
Though, any idea why does the Cadence report show toggle coverage at ~82% even after the exclusion of the XHT interface?
https://reports.opentitan.org/hw/ip/entropy_src/dv/latest/cov_report/index.html

I looked into this. There are a number of 32-bit prim counters in entropy_src_core (example, u_entropy_src_core.u_entropy_src_cntr_reg_repcnts) that are not part of the health tests, rather they are there for statistics gathering. They have no role in generating alerts, they are only for logging the overall prevalence of a particular type of health test failure. It's the 32-bit outputs that are often not toggling (presumably in the higher bits). Also these particular instances (not included in the health test modules) seem to not be exercised by the functional_errors test.

I'm ready to rerun UNR now, but I don't think UNR will exclude these, as there is no reason these prim_count outputs should be unreachable.  It might be reasonable to trigger errors in these also, but pushing for full coverage on the 32-bit outputs is not reasonable.
Also, for many of the counters internal to the HT, the MSB's of those 16-bit counters are not being fully exercised, which is fairly expectable.

Though, any idea why does the Cadence report show toggle coverage at ~82% even after the exclusion of the XHT interface?

The tool also takes toggles at internal modules (e.g., prim_counts) into account, even though it shouldn't. We'll try to resolve this with a new UNR run or, if that doesn't do the trick, with explicit exclusions. (If I understood correctly, our scripts/configs should automatically exclude such internal modules from toggle coverage, but we don't have time to debug that at this point.) We agreed to fix this as part of the V2S review action items (#16330).

Yes that is correct. The "common" Xcelium infrastructure for all IPs should have handled this. But this also means it will show up again on other IP blocks signed off with Xcelium.


MeshAdaptPUMI: predictive resolution for interface and combined approach

This PR contains a number of developments for the MeshAdaptPUMI module including:

predictive resolution for interface based adapt
interface based adaptive trigger
combined approach of interface-based adapt coupled with error estimator using size-field intersection
corrected solution transfer for the system of rans2p, vof, ls, rd, mcorr using Backward Euler
crude way of getting restart/hotstart with adaptivity and almost-exact reconstruction of the data structures.

This PR is actually continuation of #788 (around August 2018) but has duplicate commits (likely from an unnecessary rebase when forming the previous PR).

Yes it was a memory leak detected by

I've added a 2D dambreak Colagrossi case that can use the MeshAdapt workflow which uses the SpatialTools setup in https://github.com/erdc/air-water-vv/tree/scorec/meshAdaptAniso_examples/MeshAdapt_PredictiveControl/dambreak_Colagrossi_SpatialTools


Tensordot ad op sd op interface

I am aware that TestMul now fails, but I can't see why: Do you in general agree with the changes to Mul?

Checking

Coverage increased (+0.4%) to 66.279% when pulling 4039dec72d300b6575412ed97b94caf86ffd7cf7 on siquus:tensordot_ADOp_SDOp_interface into 73fe8c2d5729abf7afb745f23a269728bb48595f on chewxy:master.

 As a result, I've ported over your changes to gorgonia/tensor and I'm awaiting your greenlight for being OK to merge

@chewxy  Should be fine to merge.


CONTENT-91 - add path param for put/patch

this doesn't actually fix the problem yet, it just verifies the validator will catch the problem

https://issues.redhat.com/browse/14696935

⚠ This task currently requires qe-approval, but this PR does not fully resolve the task.  Please contact QE to determine appropriate testing required.


Add picture-in-picture example in sample app

I got Picture in Picture to work on my app!

Well done :)


er info

Description

Steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Additional information you deem important (e.g. issue happens only occasionally):
Output of docker version:
(paste your output here)

Output of docker info:
(paste your output here)

Additional environment details (AWS, VirtualBox, physical, etc.):

Looks like this was opened accidentally


Fix patches

Copied and pasted code from mostly the revert strategy.

Resolved by adding "a/" and "b/" as prefixes to the diff


Using django-filers drag & drop for images not possible

Hi,
i'm using django-filer for images as plugin in feincms3, which gives users the ability to drag & drop images into its file upload field.
This does not work with django-content-editor since the content-editor js handles the drop event and errors since there is no row which is marked as dragged:
https://github.com/matthiask/django-content-editor/blob/dd90e4e01a3cea2ae8039983833af134a911fb03/content_editor/static/content_editor/content_editor.js#L92
This event is not fired, so there is no row defined in:
https://github.com/matthiask/django-content-editor/blob/dd90e4e01a3cea2ae8039983833af134a911fb03/content_editor/static/content_editor/content_editor.js#L278
In a quick try I made the event listeners for dragover and drop dependent from the existence of window.__fs_dragging  and it seems to work. But i'm not really in such complex javascripts, so this is more of a question than an issue, wether you would see such a patch as a viable solution for me in my projects or a way to incorporate a solution for this in django-content-editor itself?

@gradel That is definitely something which I would like to integrate into django-content-editor. If you have a thing which works for you please submit a pull request! We can iterate on it.

Thanks, this should be fixed now and will soon be released. I'm thinking about waiting for a resolution of #15.

Thanks for this and your great work in general!


How to use CppUtest's API

I have a question about the CppUtest's API, for example, I want to know how to use 'withParameterOfType'  and Why the third parameter of 'withParameterOfType' must be 'void* object = (void*)1'??  So, I hope I can find some documents to scan the function of the CppUtest's API. Thank you!!

Hi there,
There is a nice example on cpputest.github.io.
You obviously looked at the test cases (which is a very good idea), but the test cases tend to be a bit minimalistic in places, rather than worked examples :)
So, yes, the third argument must be cast to (void*), because the address of the custom object is passed as a void* since it could be anything (even a pointer with the value 0x0000001), but it certainly doesn't have to be 1. In reality, it will be your the address of your custom object.
Another point worth noting is that the type of the object is matched via its name (the first string), not it's C++ type.
Finally, to use withParameterOfType() sensibly, you need a comparator for your custom object, that gets installed using the same name.
I have a worked example at https://github.com/arstrube/MiscellaneousTests/tree/master/WeirdMockTests -- but beware of The Bug (#650)!

The example I linked was originally written to solve one very specific problem, not for public consumption. I think you will still find it useful, though. Remember to check out the commit history as well. It will show you, among other things, how it should be (but doesn't work) and various alternatives, both working and not-working.

Play around with the tests, too, to see them fail.

Fixed, I assume. Closing. Please re-open when incorrect.


Example code doesn't extract text properly

Hi,
I am following the example code in README.md for url='http://fox13now.com/2013/12/30/new-year-new-laws-obamacare-pot-guns-and-drones/' and everything works fine except that the extracted text is not correct. In your example you show that when getting the article text this is the output:
>>> article.text
'Washington (CNN) -- Not everyone subscribes to a New Year's resolution...'

But when I am testing it I am getting this output:
>>> article.text
By Leigh Ann Caldwell\n\nWASHINGTON (CNN) — Not everyone subscribes to a New Year’s resolution...

I tried the same example in the online demo and it shows the correct output, without the author.
I tried cloning the repository and importing the dependencies directly but I get the same result. Am I doing something wrong here?

Hey,
I seemed to have solved this, or at least, found a temporary fix.  See my bug thread:
https://github.com/codelucas/newspaper/issues/485
I have resorted to using newspaper=0.0.8 for python 2 in order to get proper text extraction.
pip install --upgrade newspaper=0.0.8
It matches the output you expect from the online demo.  (It's actually the same version the demo uses.)
Cheers.


Less explicit PyFuncArgs is better.

implemented a From trait for PyFuncArgs, which helps in less verbose function calling.

Me too, slowly I start to understand how rust works.


typings install  not working

i am not able to install any type definitions. if i do a typings install it says (No dependencies) if i do typings install "package name" it gives me this error.
typings ERR! message Unable to find "angular-protractor" ("npm") in the registry. Did you want to try searching another source? Also, if you want contribute these typings, please help us: https://github.com/typings/registry
typings ERR! caused by https://api.typings.org/entries/npm/angular-protractor/versions/latest responded with 404, expected it to equal 200
I am stuck here. please help

same here, it happens on v1.0.4, and v0.8.1 has no problem.

@BhaskaranR If you're talking about when it prints the name of the module and it says that module has no dependencies, then it worked. Anything else, you should read the README.


⚠ Bofrost.nl has degraded performance

In d3c96ef, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1415 ms

Resolved: Bofrost.nl performance has improved in 8826019.


LCD flickering

Hi Sir, I test a WT32-SC01. I have a problem that every  updating the LCD screen will cause the screen to flickering.

I'm not sure about what kind of "flickering" you have seen.Can you tell me more details about where the flicker come from such as backlight voltage or the transmission of screen data or at least show me some photos about it?


meter~ -100

Hi,
what do you think about adding a - 100 to the meter~ object.
So 0db is the peak of a fullscale signal.
Thanks for the great library.
It is very useful.
Best
Tim

sorry, I don't understand

At the moment when I am sending an audio signal that ranges between -1 and 1 to the meter~ object of the pd-else library, the gui-meter completely clips. I expected that the gui-meter shows me something near 0.  But right now it shows me something near 0 when I send a silent audio signal.
Or am I doing something wrong?

Or am I doing something wrong?

I belive so :) just check the help file of [meter~] and you'll see it works right

just check the help file of [meter~] and you'll see it works right

if not, reopen this issue and show me how it ain't working

So the help patch works like this for me:
no signal:

fullranged signal:

This is not what I would expect. The meter~ uses only 12 steps of the whole range..
Or is this the way it should work? Thanks a lot.

well, I need more info now, what OS are you on? What version of Pd and else?

OS: Linux Ubuntu 19.10
PD:  0.5
else: v1.0beta24(Linux-amd64-32)

all should be fixed by now and a new version will come out next week

new release just out!

Thanks again for changing!


[AGORA-1975, AGORA-1973, AGORA-1968, AGORA-1979]🔐🪙 JWT generation, validation

PR-Codex overview
This PR updates authentication middleware paths and constants, refactors field names in AuthInfo, and adds a new script to generate JWT.
Detailed summary

Moved authenticateApiUser to serverAuth path
Renamed and updated constants in constants.ts
Refactored field names in AuthInfo
Added generate-jwt script for JWT generation

The following files were skipped due to too many changes: src/middleware.ts, src/app/api/v1/retrofunding/rounds/[roundId]/ballots/[ballotCasterAddressOrEns]/impactMetrics/route.ts, src/scripts/generateJwt.ts, src/app/lib/auth/serverAuth.ts, src/app/lib/auth/edgeAuth.ts, yarn.lock

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

It might be better to split the verification logic into 2 flows (maybe more in the future) depending on the endpoint being called.
Maybe I'm missing something here but it's difficult to see how we'd manage different permissions this way

Can you explain a little more about what you mean by splitting them into two separate flows? When I look at how authentication is performed now, I do see two separate flows in the sense that we do not issue a JWT to API Key users. I'm up for refactoring to enforce more modularity at the code level, if that's your concern.
whatever validation we do for an arbitrary request in the edge middleware needs to be unified simply by virtue of how the edge runtime works (i.e. it's a single exported function from middleware.ts)
for managing different permissions, I don't think I see the issue you're mentioning, since the scope/permissions for a given user will either be stored in the JWT itself, or associated with their API Key. The flows for resolving the permissions for the supplied JWT or API Key are currently still separate. But, once we have the permissions, we can authorize access to requested resources as appropriate.
for instance, here's how I see it working right now, where the only routes we need to permission are related to ballots:

user validates identity with SIWE
user is issued JWT containing permissions for the address and chain authenticated with SIWE
user supplies JWT to access /ballot endpoints
authentication layer validates that the JWT is valid, signed, not expired, and the requested route (i.e. /ballots/{address}) matches the permissions in the supplied token

For now, API Key users are only able to access "default public" information like delegate statements, votes, proposals, basically the things that anyone can see just be using our frontend.  In the future, as our routes become more complex, we can associate more permissions with a given API key by storing them alongside the user record.

user issued API Key with particular permissions
user supplies API Key to access /ballot endpoints
authentication layer on the server retrieves permissions associated with API Key, and checks authorization based on the requested resource and the permissions associated with the API Key

N.B. that I haven't yet disallowed access to /ballots for API Key users, so it might become more clear if that's included in this PR too. LMK what you think @stepandel

this is a great explanation, thank you. The part that I was a bit confused about is how you're planing on managing different permissions. I can see the validateScopeAgainstRoute function now and encoding scope into JWT is a great approach


runtime: TestMemStats is flaky

TestMemStats is currently flaking on several builders:
2020-04-27T20:20:53-a7e9e84/linux-386-longtest
2020-04-27T20:20:53-a7e9e84/linux-ppc64le-power9osu
2020-04-27T20:06:57-d2f5e4e/linux-amd64-nocgo
2020-04-27T19:49:49-a637ee1/linux-amd64-clang
2020-04-27T19:49:42-d646c03/linux-amd64-clang
2020-04-27T19:35:01-6a4441d/darwin-amd64-10_12

The failure is
--- FAIL: TestMemStats (0.00s)
    malloc_test.go:98: Bad sys value: ... [blob of text]

Given the timing, the most likely cause is the new mcentral implementation somehow. It's not clear to me yet how it's related, if that's the case.

What's particularly strange is the "bad sys value" since the way sys is set is very clear. In the darwin-amd64-10_12 builder, Sys is 256 KiB smaller than the sum of the values it's supposed to match. Maybe there's now an update to one of the values that goes into sys after sys is calculated but before the rest of the values are copied?

I looked at a few more: all of them seem to be off by 256 KiB.

Still happening, and even 32-bit builders are off by exactly 256 KiB.

What this means for certain is that between the first few lines of updatememstats() which set memstats.sys and the memmove of memstats into MemStats one of the values that goes into memstats.sys increases. It's unlikely that this is multiple increases and decreases because the value difference is so consistent. Operating on the assumption that this is the result of exactly one stat only increasing, these are the following stats that could have been increased:
heap_sys
stacks_inuse
buckhash_sys
gc_sys
other_sys

All the other stats are too small (<256 KiB) to be culprits operating under this assumption.
Given all the evidence (the timing implicating mcentral work, the value of 256 KiB) I'm starting to think this is persistentalloc's update of other_sys and gc_sys not getting read properly by ReadMemStats (i.e. it's reading a stale value because it's read non-atomically). Notably, persistentalloc allocates chunks of memory (for allocations < 64 KiB) in 256 KiB blocks. I always figured a STW event would have so many memory barriers in it that this couldn't be a possibility, but maybe? The way this would relate to the mcentral changes is that maybe we're hitting persistentalloc regularly (note that the span set blocks are < 64 KiB in size) due to span set blocks leaking (fixed in: golang.org/cl/230497; if this is the case, landing that might also "fix" this, though not the underlying issue).

Ah but hold up, this is happening on amd64/386 where regular loads are the same as atomic loads (in the runtime) because of TSO (and the increments are atomic).

Figured it out.
Consider the following chain of events:

updatememstats sets sys
It purges the mcaches and calls mcentral.uncacheSpan on them.
mcentral.uncacheSpan does a push into a span set, causing a persistentalloc.
The persistentalloc needs a new chunk.
memstats.gc_sys and memstats.other_sys are increased in total by 256 KiB.
We do the rest of the MemStats stuff and copy the memstats struct into the MemStats struct.
We now have a stale Sys value.

The fix is to purge the mcaches before doing anything else in updatememstats.

@mknyszek when I discussed #24686 with @aclements ages ago he suggested it could let us eliminate mcache. If you encounter other mcache troubles, it might be worth exploring this other route.


Release Google.Cloud.GkeHub.V1Beta1 version 1.0.0-beta03

Changes in this release:

Commit 1d19cbd:

feat: Add request_id under DeleteMembershipRequest and UpdateMembershipRequest
feat: Add OnPremCluster and MultiCloudCluster as GkeCluster equivalent field
fix!: Move GkeCluster under oneof

Commit ac367e2: feat: Regenerate all APIs to support self-signed JWTs

Created release for Google.Cloud.GkeHub.V1Beta1-1.0.0-beta03


Enable chunk-wise application in ctapipe-apply-models

Please describe the use case that requires this feature.
The TableLoader can read data in chunks, we should add the option to apply models chunkwise to reduce memory consumption.
Describe the solution you'd like
Add a --chunksize option that applies models in chunks of array events.

This was done in #2133


Climate component default min/max temp assumes Celcius regardless of configured unit system

Home Assistant release with the issue:
0.98.5
Last working Home Assistant release (if known):
unknown
Operating environment (Hass.io/Docker/Windows/etc.):
Docker
Component/platform:
https://www.home-assistant.io/components/climate.mqtt/
Description of problem:
My HA instance is configured to use the Imperial unit system. Some preliminary code searching suggests that due to this, HA should expect to use Fahrenheit units. I also configured an MQTT climate entity, initially without configuring a default min and max temp. I noticed that in the state of the entity, the min_temp is set to 7 and max_temp is set to 35. This appears to be the default min/max assuming that the temperature units are in Celsius.
I have since overridden the min and max temp to more sensible values, but I am unclear on why this is occurring. To me this appears to be a bug related to setting the default min/max which should be based on the temperature unit which is based on the unit system configured.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
homeassistant:
  name: Home
  latitude: !secret 'lat_home'
  longitude: !secret 'long_home'
  elevation: !secret 'elevation_home'
  unit_system: imperial
  time_zone: America/New_York
  customize: !include 'customize.yaml'
  auth_providers:
  - type: homeassistant
climate:
- platform: mqtt
  name: Thermostat
  unique_id: thermostat
  current_temperature_topic: smartthings/Thermostat/temperature
  mode_command_topic: smartthings/Thermostat/thermostatMode
  mode_state_topic: smartthings/Thermostat/thermostatMode
  modes:
  - 'off'
  - heat
  - cool
  - auto
  fan_mode_command_topic: smartthings/Thermostat/thermostatFanMode
  fan_mode_state_topic: smartthings/Thermostat/thermostatFanMode
  fan_modes:
  - auto
  - 'on'
  temperature_low_command_topic: smartthings/Thermostat/heatingSetpoint
  temperature_low_state_topic: smartthings/Thermostat/heatingSetpoint
  temperature_high_command_topic: smartthings/Thermostat/coolingSetpoint
  temperature_high_state_topic: smartthings/Thermostat/coolingSetpoint
  temp_step: 0.5

The default temperature has fixed unit of measurement.
See: https://github.com/home-assistant/home-assistant/blob/7e7ec498cac6ad34ef6a2a6315bd1c6480cdc8c0/homeassistant/components/mqtt/climate.py#L210-L213

Understood, but if I am understanding the code in the base component correctly, there is handling that will convert the default min/max temps, which are in Celsius, to the user configured temperature unit:
https://github.com/home-assistant/home-assistant/blob/7e7ec498cac6ad34ef6a2a6315bd1c6480cdc8c0/homeassistant/components/climate/__init__.py#L479-L491
I am suggesting that the MQTT Climate component should be doing the same

I am suggesting that the MQTT Climate component should be doing the same

The climate component cannot convert the min/max or initial temperature to the user's chosen temperature unit since they are used inside of the PLATFORM_SCHEMA. In this part it is impossible to check user's chosen temperature unit and to convert the min/max or initial temperature because the hass object is undefined and the chosen temperature unit cannot be polled.
@raman325 therefore please close this issue, thank you :)


OpenAI Whisper large-v3-turbo model does not translate text

I have tried this model with a sample audio file (language was urdu). The response from the asr endpoint when task=translate is still in urdu.

Can you provide more info about this? Was the same file working with whisper in the CLI?

audio.mp3.zip
Please see the attached zip file which contains one audio.mp3 file. This audio is in urdu. If you use the faster-whisper model (I ran it through docker + nvidia gpu) the translations/transcriptions work perfectly. However, if you use the openai large-v3-turbo model (again running through docker + nvidia gpu) and the API action is translate, it will only return transcribed text and will not do the translation.
I have not tried running it in the CLI.

Have you validated that translation works with other models when using openai_whisper?

#259


Allow empty associations

The spec for mimetypes.list doesn't seem to forbid the value of an association/default application to be the empty string.
Either way, I think handlr-regex should be able to tolerate a mimetypes.list that has such lines. For example, with a mimetypes.list file like this:
[Default Applications]
text/plain=

launching the program results in a panic:
$ handlr
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Error { kind: ParseApps(Error { variant: ParsingError { positives: [value_char], negatives: [] }, location: Pos(34), line_col: Pos((2, 12)), path: None, line: "text/plain=␊", continued_line: None }) }', handlr-regex/src/apps/user.rs:16:65
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

So in this PR, we relax the parser to not require a value to be non-empty, which was the source of the error that was unwrapped with a panic. Then, when each line of the file is iterated over, the empty string will result in an empty handlers collection on https://github.com/Anomalocaridid/handlr-regex/blob/d06e76c675c2dbeb9d8bb7fdafe6f867f7b04549/handlr-regex/src/apps/user.rs#L173
This will mean the fact that the association value was the empty string is silently ignored.

Thanks for your pull request!
I think you are right that empty associations are allowed in mimeapps.list. If I use the original xdg-open to open a file with an empty association, it picks a default program as if there was no such association, just like your branch does when I run it.


Multiple errors in sample transfer/streaming/streaming-02-kafka-to-http

Bug Report
Describe the Bug
Sample
transfer/streaming/streaming-02-kafka-to-http
Action
Start negotiate contract
curl -H 'Content-Type: application/json' -d @transfer/streaming/streaming-02-kafka-to-http/5-negotiate-contract.json  -X POST "http://localhost:28181/management/v3/contractnegotiations" -s | jq [
Explanation
when trying to start negotiate contract with 5-negotiate-contract.json the returned json states that the multiple fields are missing or blank.
Expected Behavior
contract-negotiation-id in json will be returned
Observed Behavior
[
  {
    "message": "https://w3id.org/edc/v0.0.1/ns/policy/@type was expected to be http://www.w3.org/ns/odrl/2/Offer but it was not",
    "type": "ValidationFailure",
    "path": "https://w3id.org/edc/v0.0.1/ns/policy/@type",
    "invalidValue": [
      "https://w3id.org/edc/v0.0.1/ns/Offer"
    ]
  },
  {
    "message": "https://w3id.org/edc/v0.0.1/ns/policy/http://www.w3.org/ns/odrl/2/assigner/@id cannot be null or blank",
    "type": "ValidationFailure",
    "path": "https://w3id.org/edc/v0.0.1/ns/policy/http://www.w3.org/ns/odrl/2/assigner/@id",
    "invalidValue": null
  },
  {
    "message": "https://w3id.org/edc/v0.0.1/ns/policy/http://www.w3.org/ns/odrl/2/target/@id cannot be null or blank",
    "type": "ValidationFailure",
    "path": "https://w3id.org/edc/v0.0.1/ns/policy/http://www.w3.org/ns/odrl/2/target/@id",
    "invalidValue": null
  }

Steps to Reproduce
Steps to reproduce the behavior:
Follow
(https://github.com/eclipse-edc/Samples/tree/main/transfer/streaming/streaming-02-kafka-to-http)
Context Information
Used commit: https://github.com/eclipse-edc/Samples/commit/1d55b7c2ef109a7b94a9ae2ea124e79ea6fbeb2f
OS: Ubuntu 24.04
Detailed Description
Same in https://github.com/eclipse-edc/Samples/tree/main/transfer/streaming/streaming-03-kafka-broker
Possible Implementation

5-negotiate-contract.json

{
  "@context": {
    "@vocab": "https://w3id.org/edc/v0.0.1/ns/",
    "odrl": "http://www.w3.org/ns/odrl/2/"
  },
  "@type": "ContractRequest",
  "counterPartyAddress": "http://localhost:18182/protocol",
  "protocol": "dataspace-protocol-http",
  "policy": {
    "@context": "http://www.w3.org/ns/odrl.jsonld",
    "@id": "{{offerId}}",
    "@type": "Offer",
    "permission": [],
    "prohibition": [],
    "obligation": [],
    "odrl:assigner": {
      "@id": "provider"
    },
    "odrl:target": {
      "@id": "kafka-stream-asset"
    }
  }
}

6-transfer.json

  "@context": {
    "@vocab": "https://w3id.org/edc/v0.0.1/ns/"
  },
  "@type": "TransferRequest",
  "dataDestination": {
    "type": "HttpData",
    "baseUrl": "http://localhost:4000"
  },
  "protocol": "dataspace-protocol-http",
  "assetId": "stream-asset",
  "contractId": "{{contract-agreement-id}}",
  "connectorId": "provider",
  "counterPartyAddress": "http://localhost:18182/protocol",
  "transferType": "HttpData-PUSH"
}

this is somewhat related to what happened here: https://github.com/eclipse-edc/Samples/issues/334
because the test covering the samples are not using the same payloads as the samples themselves.
@arekusutarakasiku would you feel comfortable in contribute with a PR? (some inspiration can be drawn from here: https://github.com/eclipse-edc/Samples/pull/335)

I've changed 5-negotiate-contract.json and 6-transfer.json(s. Possible implementation).
But  on  stepconsume events-step, but nothing appeared in n the console of the http-request-logger.
@ndr-brt , do you have any idea why?

I've changed 5-negotiate-contract.json and 6-transfer.json(s. Possible implementation). But on stepconsume events-step, but nothing appeared in n the console of the http-request-logger. @ndr-brt , do you have any idea why?

no clue, without knowing what you changed and if there's any information on the console output it's hard to tell.


[FLINK-13540] DDL do not support properties key that contains number …

What is the purpose of the change
This patch changes the table option keys format to string literal, they are as sql identifiers before.
Brief change log

Change the table option key to string literal

Verifying this change
See tests in FlinkSqlParserImplTest.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving):  no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@wuchong  Thanks for the review, i would rather to keep the SqlProperty, because it is a common pattern, we may still need to use this structure for future grammars, i.g. CLUSTER BY .

+1 to this from an API perspective.


Downloader in CentOS is missing some libraries

I'm trying to use the downloader on a CentOS system. However, it looks like is missing some dependencies
// karma.conf.js
const ChromiumRevision = require('puppeteer/package.json').puppeteer.chromium_revision
const Downloader = require('puppeteer/utils/ChromiumDownloader')
const revisionInfo = Downloader.revisionInfo(Downloader.currentPlatform(), ChromiumRevision)

process.env.CHROME_BIN = revisionInfo.executablePath

module.exports = function(config) {
  config.set({
    browsers: ['ChromeHeadless']
  })
}

When I run CHROME_BIN I get
error while loading shared libraries: libgtk-3.so.0: cannot open shared object file: No such file or directory

Check out CentOS section in troubleshooting.md.


Scrolls upwards whenever new message is received

Scrolls upwards when new msg is recieved as well as when chat windows is opened.

Possibly related to an issue I logged awhile ago. See https://github.com/Slyce-Inc/SlyceMessaging/issues/28 I have a fix suggestion there.


Default parameters file name fix

@nicolasdanelon thank for this fix

@nogo your walcome, anytime.
please can you add a password input? ;)

@nicolasdanelon You mean issue #76 ? You want to have the ability to change the password?

Yes sr!


Feature http xa

add http xa pattern #68

Codecov Report

Merging #69 (dcd3f4b) into main (10272be) will decrease coverage by 5.29%.
The diff coverage is 15.52%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main      #69      +/-   ##
==========================================
- Coverage   76.28%   70.99%   -5.29%
==========================================
  Files          41       44       +3
  Lines        1754     1903     +149
==========================================
+ Hits         1338     1351      +13
- Misses        416      552     +136

Impacted Files
Coverage Δ

src/Dtmcli/Xa/Xa.cs
0.00% <0.00%> (ø)

src/Dtmcli/Xa/XaGlobalTransaction.cs
0.00% <0.00%> (ø)

src/Dtmcli/Xa/XaLocalTransaction.cs
0.00% <0.00%> (ø)

src/DtmCommon/Constant.cs
100.00% <100.00%> (ø)

src/DtmCommon/Imp/DbSpecialDelegate.cs
100.00% <100.00%> (ø)

src/Dtmcli/DtmClient.cs
87.35% <100.00%> (-0.56%)
:arrow_down:

src/Dtmcli/ServiceCollectionExtensions.cs
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

For mysql version less than 8.0, connection should be closed after xa end, or else xa commit in another connection will not succeed. It is a bug of mysql, and is fixed in mysql 8

For mysql version less than 8.0, connection should be closed after xa end, or else xa commit in another connection will not succeed. It is a bug of mysql, and is fixed in mysql 8

There is indeed this issue in versions below MySQL 8, and even after closing the connection after xa end or xa commit, the problem still persists. However, when I add pooling=false to the connection string, it seems to work fine. I suspect that the MysqlConnector connection pool cannot support this scenario.

So for XA transaction, you may disable pool.
In the document of dtm, the performance of XA transaction are not high, so disable pooling should be OK

Can we merge this PR and release a new version now?

Can we merge this PR and release a new version now?

Of course you can.


Add enable/disable user-functionality rest api

Depends on https://github.com/wso2/identity-api-user/pull/108

Move to https://github.com/wso2/identity-rest-dispatcher/pull/238


[Feature request] Reload database ingestion status dynamically

Is your feature request related to a problem? Please describe.
The recent run status within the database ingestion tab does not seem to update automatically. You need to do a full page reload to see the latest status.
Describe the solution you'd like
Dynamic page reload in order to show the most recent status without forcing a full page refresh/ reload.
Describe alternatives you've considered
Manually reloading page.
Additional context
none

@mohityadav766 lets look at if we can use websockets to do this?


help_menu: Fix help_menu message formatting overlay not opening.

This commit fixes the behaviour of message formatting overlay in help menu not opening due to assertion error in rendered markdown after changes from #28418.

Fixes: bug from #28418

Screenshots and screen captures:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Added a block comment explaining some of the background here and marked this to merge once CI passes, thanks @pratik-pc.


decoder uses encodeP instead of decodeP for unknown type P

My reproduction:
GenSrc.hs:
{-# LANGUAGE DeriveAnyClass #-}
{-# LANGUAGE DeriveGeneric #-}
{-# LANGUAGE DerivingStrategies #-}
module GenSrc where

import GHC.Generics (Generic)

import qualified Elminator as Elm

data P = PC
  deriving stock (Generic)

data T = TC P
  deriving stock (Generic)
  deriving anyclass (Elm.ToHType)

Main.hs:
{-# LANGUAGE OverloadedStrings #-}
{-# LANGUAGE TemplateHaskell #-}
{-# LANGUAGE TypeApplications #-}
module Main where

import qualified Data.Aeson as Aeson
import Data.Proxy (Proxy (Proxy))
import Data.Text (Text)
import qualified Data.Text.IO as Text
import qualified Elminator as Elm

import GenSrc

main :: IO ()
main = Text.putStrLn generatedCode

generatedCode :: Text
generatedCode =
  $(Elm.generateFor Elm.Elm0p19 Aeson.defaultOptions "Gen" (Just "Gen.elm") $ do
    Elm.include (Proxy @T) (Elm.Everything Elm.Poly)
  )

produces the following Gen.elm:
module Gen exposing (..)

import Json.Encode as E
import Json.Decode as D

seqApp : D.Decoder (a1 -> v) -> D.Decoder a1 -> D.Decoder v
seqApp inDec oDec =
  let
    mapFn v = D.map (\x -> x v) inDec
  in D.andThen mapFn oDec

elminatorEncodeList0p19 : (a -> E.Value)-> List a -> E.Value
elminatorEncodeList0p19 fn ls = E.list fn ls

encodeMaybe : (a -> E.Value)-> Maybe a -> E.Value
encodeMaybe fn ma = case ma of
  Just a -> fn a
  Nothing -> E.null

type T = TC P

encodeT : T  -> E.Value
encodeT a =
 case a of
  TC a1 -> encodeP (a1)

decodeT : D.Decoder T
decodeT  =
 D.oneOf ([ D.map (TC) (encodeP)])

In particular, note that both encodeT and decodeT use encodeChar.
This issue goes away if I derive ToHType for P, but in my actual code P is Char so I can't derive ToHType for it. It'd be nice to support Char natively, but in the absence of that, I'd like to at least be able to supply my own encodeChar and decodeChar functions to make the generated code work, but since they're both encodeChar, I can't. (I'd also need the ability to bring them into scope here, but my current solution to that is just manually adding imports after code generation.)

(In fact this is even visible in the example in the README example)

@bmillwood  Thanks for reporting the issue.


MQTT support

Can somebody help us with Instagram MQTT implementation?
I started to develop a draft which based on another draft for legacy version of this library.
You can find it here:
https://gist.github.com/realinstadude/4b9af1ccfa5299cb6de61cfc3f74ea46
Some usefull links:

https://github.com/mqttjs/MQTT.js
https://github.com/mgp25/Instagram-API/blob/master/src/Realtime/ (PHP implementation)

I'm working on it currently, though I mostly do it inside of my app that uses this lib. Were you able to connect successfully to insta mqtt server?

@lomegg you can check a gist from the first message.

Any progress?

I have a working implementation for mqtt here. The updated client with support for custom mqtt packets is on this branch.
I'm currently stuck on fbns, any help would be appreciated.

My implementation works. (i hope so)
It isnt final and there are a lot of design decisions to be made, but the general concept is here.

RealtimeClient example
FBNS/Push notification example
You can try it and report back.

@Nerixyz really work, thanks.


Tuple causes too much slow down

I'm playing with some toy Monte Carlo and realized the tuple interface makes it extremely slow, the best I can do for now is:
const AC = full_deck()
function one_round(N=7)
    _deck = Deck(copy(AC))
    shuffle!(_deck)
    _table = pop!(_deck, 5)
    my_rank = hand_rank(CompactHandEval(pop!(_deck,2), _table))
end

julia> @btime one_round()
  56.815 ms (85673 allocations: 5.09 MiB)

the copy(AC) makes it 10x faster than invoking ordered_deck, as expected, but I can't figure out a way to make hand_rand(CHE) faster because Tuples are immutable thus memory reuse pattern fails.

I think that this slow down is from allocations for the deck (and maybe cards, too). Here is an example that uses the tuple interface with fewer allocations and better performance:
julia> using BenchmarkTools, Combinatorics, PlayingCards, PokerHandEvaluator

julia> const all_possible_hands = Tuple.(collect(combinations(full_deck(), 5)));

julia> sample_hand() = all_possible_hands[rand(1:2598960)]
sample_hand (generic function with 1 method)

julia> function one_round(N=7)
           my_rank = hand_rank(CompactHandEval(sample_hand()))
       end
one_round (generic function with 2 methods)

julia> @btime one_round()
  1.989 μs (8 allocations: 149 bytes)
6462

That said, this example is perhaps a bit awkward, and I think the original post is a bit more logical. It may be that we just need a lazy way to deal cards / get a set of N unique cards from the deck. I'm inclined to leave this issue open until we have a nice interface to work with.

Actually, my example is not equivalent (the OP uses the 7 card interface), I’ll try to fix it up a bit later

ideally, (without knowing how exactly does evaluate_7 works), one should be able to do something like:
seq = sample(1:52, 7; replace=false)
AC = full_deck()
hand_rank(@view AC[seq])

this should have minimal allocation as we always view into the deck. of course, it is entirely possible that by spending 10x more memory I/O it can actually be faster, but i personally feel that's unlikely.

See this comment


feat: setup kind cluster

Another environment setup based on kind utility https://kind.sigs.k8s.io/. IDK how it should be duplicated across guides, so just one file to start a discussion.

✊


"Key not found: Escape", and other newbie questions. Need better getting started docs

The title says it all. I installed lem 10 minutes ago and so far not too much joy.
On macOS terminal and ITerm2, the Escape key is not assigned as the meta key. How to set this up.
It took several minutes of wild experimentation to discover that the escape key has to be hit at the exact same time as another key, for example Escape + x. This would require 1 short paragraph in the README file. I would like to know best practices for work flow, etc. The 1 minute YouTube video did not have any narration - that would have been a good place to add some newbie tips.
I would submit a pull request for the README file, but as a complete lem novice, it would be better if someone else much more experienced did it.
I don't intend to whine, lem is a cool project.

updating helped with the Escape key problem


Readme clarification?

Under the Usage section of the README it says at the end of the index function: return $this->render('home');.
I presume that index() is part of a controller? But what does return $this->render('home'); do? That doesn't work in my default L5.2 installation.

Fixed https://github.com/DraperStudio/Laravel-Chartist/commit/88492e3de7ceaa556ed9df1df1680529792e6d1b


OutMap optionals and OutMap options

Changes

Naming fixes. For example (OutMap):

func from(_ int: Int) -> Self?

became
func fromInt(_ int: Int) -> Self?

Name changed due to clashes with generic from<T>(_ value: T)

It is now possible to set optionals in OutMappable, and OutMap will handle that nil appropriately:

struct NilTest1: BasicOutMappable {

    let int: Int?
    let string: String?

    func outMap<Destination : OutMap>(mapper: inout BasicOutMapper<Destination>) throws {
        try mapper.map(self.int, to: "int")
        try mapper.map(self.string, to: "string")
    }

}

Inspired by real-life issues: introducing OutMapWithOptions protocol. It allows to «outmap» in different ways. Use-case: handle nils. In some cases it is appropriate not to ignore nil, but to represent it as JSON’s null or as an empty string (for example, in PUT requests). So one can write:

let dict: JSON = try instance.map(withOptions: [.nilAsJSONnull])

Options are defined by OutMap.
Alternatives considered
Different ways of out-mappings can be achieved by creating «wrappers» around existing OutMap types, for example, JSON.NilAsNull or something like that. So the relevance of OutMapWithOptions can be questioned.

Codecov Report

Merging #11 into master will increase coverage by 1.66%.
The diff coverage is 59.3%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
+ Coverage   48.08%   49.75%   +1.66%
==========================================
  Files          12       12
  Lines        1281     1413     +132
==========================================
+ Hits          616      703      +87
- Misses        665      710      +45

Impacted Files
Coverage Δ

Tests/MapperTests/Readme.swift
13.26% <42.1%> (+13.26%)
:arrow_up:

Sources/Mapper/OutMapper/OutMapper.swift
55.3% <49.12%> (-10.39%)
:arrow_down:

Sources/Mapper/OutMapper/OutMap.swift
60% <62.5%> (+60%)
:arrow_up:

Sources/Mapper/OutMapper/OutMappable.swift
71.42% <70.58%> (-28.58%)
:arrow_down:

Tests/MapperTests/Map.swift
25.83% <71.42%> (+1.19%)
:arrow_up:

Tests/MapperTests/OutMappingTests.swift
91.66% <90.47%> (-0.21%)
:arrow_down:

Continue to review full report at Codecov.

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


Bug: E5108: Error executing lua when commenting kitty.conf

Hi numToStr! I'm running into a small bug with commenting on kitty terminal config files. I'm trying to use gcip on a paragraph and I get:
E5108: Error executing lua Vim(lua):E5108: Error executing lua ...ite/pack/packer/start/Comment.nvim/lua/Comment/utils.lua:179: Vim:no parser for 'kitty' language, see :help treesitter-parsers
stack traceback:
        [C]: in function '__index'
        ...ite/pack/packer/start/Comment.nvim/lua/Comment/utils.lua:179: in function 'parse_cstr'
        ...te/pack/packer/start/Comment.nvim/lua/Comment/opfunc.lua:53: in function 'opfunc'
        .../site/pack/packer/start/Comment.nvim/lua/Comment/api.lua:43: in function 'linewise'
        [string ":lua"]:1: in main chunk
        [C]: at 0x010437c3a0
stack traceback:
        [C]: at 0x010437c3a0

I've looked through my config to see if I enabled some extra treesitter magic somewhere but couldn't seem to find anything. The line its pointing to suggests it might be a lack of vim.bo.commentstring but that shows as:
:set commentstring
  commentstring=# %s

(which is correct)
The commenting still actually works correctly, but it just also throws this error message.
I'd be happy to send an nvim config if needed!

I am pretty sure that I handled the missing treesitter parser here https://github.com/numToStr/Comment.nvim/blob/ad7ffa8ed2279f1c8a90212c7d3851f9b783a3d6/lua/Comment/ft.lua#L233
And I just checked and no error for me with kittty filetype using gcip.

I am pretty sure that I handled the missing treesitter parser here

That does look like it would handle it! I just removed all of my plugins besides treesitter and Comment and it works - so it's something in my config!
Thanks for the help!


Fix collector stalling on Mac OS

Fix #415.

Tests a passing locally. Possible pipeline bug.  Possibly obviated by deprecation of these compare tests in 2.3.


bound_emission emissions equal to 0 dissapear in Output gdx

Hey, I am just doing some model runs where I limit emissions in a certain period to 0. The emission bound is considered in some way (however, the emissions are fully not reduced to 0) but also the bound parameter itself is not portrayed in the output gdx (I guess 0 are not passed through?). This is not a big issue, however it is somewhat inconsistent, if the solution is calculated considering the bound but the bound itself is not to be found in the output gdx.

@ClaraLuisa, the output GDX doesn't show values of zero, but these values are taken into account in calculations. Can you elaborate the other issue that the bound was zero but still there were some emissions? Is this only small decimals?


fix: incorrect version for wrangler types --x-include-runtime inclusion

Summary
It was pointed out that we had the incorrect version for the inclusion of wrangler types --experimental-include-runtime (see here for correct release). This PR is the fix.
Documentation checklist

[x] The documentation style guide has been adhered to.

Thanks @chalkygames123 for pointing this out!


Add a simple contribution experience

Users looking to add simple contributions or to contribute a curation for something that has not been harvested need a simple way to input the data they have. Editing on the curation page is a bit more advanced and requires that the component is already harvested.
Here I'm thinking of some simple but well structured form that exposes all the right data bits with some field validation.

I dug up some stuff from my previous work, and can contribute some thoughts. This is the wireframe we came up with for our curation pane in the elog.io project: https://github.com/commonsmachinery/catalog/blob/master/doc/wireframes/export/edit_a_work_metadata_editor.png
I'm not saying it's the best, but it is structured. There are two aspects of this which I think would benefit ClearlyDefined:

For curated data, it would be nice to have some abstract understanding of confidence in the data. It doesn't need to be (and maybe shouldn't be, for legal reasons) a percentage indicator, but if we have 4-5 data sources which all indicate that file X is under Y license, then that should be highlighted -- or perhaps hidden -- because we're already quite sure the data is accurate. Instead, the data which is missing, or where there are discrepancies between the data sources, could be highlighted, as those would need manual intervention and work. Simplify by hiding data and showing only that which is most relevant for the curation work.
Not all curation need to happen through the same interface. It should be possible to break out the curation bits, especially where there are discrepancies in the data. Could even do some gamification/crowdsourcing of contributions :-) You have five minutes to spend on helping ClearlyDefined, how can you do that best? One way could be to click a link on the front page, which starts asking you questions. Granted, it's a bit out there and could be mostly marketing rather than collecting reliable data, but some food for thought :-)

Look at the source code file to the right, does this file include a copyright notice?
In this copyright notice, what would be the name of the copyright holder?

@jeffmcaffer I think this issue is likely either closable or superseded by the other issues we've opened recently. Can you confirm?

yup. we have completely revisited the contribution experience.
Having said that, @jonasob second suggestion around gamification and making it really easy to get partial results would be really interesting to preserve somewhere. I like the idea of asking super simple, focused questions that build consensus.
That will be something we do in the future but don't want to forget about it.
I'll close this issue. @fossygirl if you can think of a good place for that design idea in a UI design doc or some such, please do include.


Fix loading issue raw conll bug

Account for the different encodings, split the "raw" CoNLL files in the right way, and DataParallel for GPUs -> fix the memory problem seen in another issue as well.

Separate subclass of BioDataset specific for CoNLL.
DataParallel in a different PR.


Support electron ^1.6.0

It's throwing an error for electron version 1.6.0.
Why it's important to support for me? Because this version support native async|await and few other critical features. Thanks.
`✖ Rebuild Failed
An unhandled error occurred inside electron-rebuild
Could not detect abi for version 1.6.0 and runtime electron
Error: Could not detect abi for version 1.6.0 and runtime electron`

node-abi has been updated to support Electron 1.6.0
If you run npm update you should get the new version of node-abi.
If that doesn't work then simply
npm uninstall electron-rebuild
npm install electron-rebuild

And that will re-resolve the dependency tree, sorry for the hassle 👍

FYI submitted lgeiger/node-abi/pull/13 to make the error message it outputs slightly more informative