Feature: support crio+crun container runtime

What is the problem this feature will solve?
support more container runtime.
If you have solution，please describe it
sealos run registry.cn-shenzhen.aliyuncs.com/cnmirror/kubernetes-crio:v1.25.3 --single
sealos run labring/helm:v3.10.1
sealos run labring/calico:v3.24.1

root@ubuntu:~# kubectl get nodes -o wide
NAME     STATUS   ROLES           AGE   VERSION   INTERNAL-IP     EXTERNAL-IP   OS-IMAGE           KERNEL-VERSION      CONTAINER-RUNTIME
ubuntu   Ready    control-plane   17m   v1.25.3   <IP_ADDRESS>   <none>        Ubuntu 22.04 LTS   5.15.0-27-generic   cri-o://1.25.1
root@ubuntu:~#

coredns CrashLoopBackOff
root@ubuntu:~# kubectl get pods -A
NAMESPACE          NAME                                       READY   STATUS             RESTARTS      AGE
calico-apiserver   calico-apiserver-7bddd86d98-dk4cm          1/1     Running            0             10m
calico-apiserver   calico-apiserver-7bddd86d98-q54lx          1/1     Running            0             10m
calico-system      calico-kube-controllers-85666c5b94-jl4cr   1/1     Running            0             12m
calico-system      calico-node-npht4                          1/1     Running            0             12m
calico-system      calico-typha-598b5559d-bhr2b               1/1     Running            0             12m
calico-system      csi-node-driver-sq95f                      2/2     Running            0             11m
kube-system        coredns-565d847f94-7s6gj                   0/1     CrashLoopBackOff   7 (16s ago)   16m
kube-system        coredns-565d847f94-jmcgm                   0/1     CrashLoopBackOff   7 (11s ago)   16m
kube-system        etcd-ubuntu                                1/1     Running            0             17m
kube-system        kube-apiserver-ubuntu                      1/1     Running            0             17m
kube-system        kube-controller-manager-ubuntu             1/1     Running            1 (14m ago)   17m
kube-system        kube-proxy-nz762                           1/1     Running            0             16m
kube-system        kube-scheduler-ubuntu                      1/1     Running            2 (14m ago)   17m
tigera-operator    tigera-operator-6675dc47f4-cpwx5           1/1     Running            0             12m

logs
root@ubuntu:~# kubectl -n kube-system logs -f deploy/coredns
Found 2 pods, using pod/coredns-565d847f94-7s6gj
.:53
[INFO] plugin/reload: Running configuration SHA512 = 591cf328cccc12bc490481273e738df59329c62c0b729d94e8b61db9961c2fa5f046dd37f1cf888b953814040d180f52594972691cd6ff41be96639138a43908
CoreDNS-1.9.3
linux/amd64, go1.18.2, 45b0a11
[FATAL] plugin/loop: Loop (<IP_ADDRESS>:45745 -> :53) detected for zone ".", see https://coredns.io/plugins/loop#troubleshooting. Query: "HINFO<PHONE_NUMBER>827747503.2346149046908645429."
root@ubuntu:~#

/etc/resolv.conf config
root@ubuntu:~# cat /etc/resolv.conf
# This is /run/systemd/resolve/stub-resolv.conf managed by man:systemd-resolved(8).
# Do not edit.
#
# This file might be symlinked as /etc/resolv.conf. If you're looking at
# /etc/resolv.conf and seeing this text, you have followed the symlink.
#
# This is a dynamic resolv.conf file for connecting local clients to the
# internal DNS stub resolver of systemd-resolved. This file lists all
# configured search domains.
#
# Run "resolvectl status" to see details about the uplink DNS servers
# currently in use.
#
# Third party programs should typically not access this file directly, but only
# through the symlink at /etc/resolv.conf. To manage man:resolv.conf(5) in a
# different way, replace this symlink by a static file or a different symlink.
#
# See man:systemd-resolved.service(8) for details about the supported modes of
# operation for /etc/resolv.conf.

nameserver <IP_ADDRESS>
options edns0 trust-ad
search .

i must disable <IP_ADDRESS> for ubuntu default dns
mkdir -p /etc/systemd/resolved.conf.d/
cat >/etc/systemd/resolved.conf.d/98-disable-127-53.conf << EOF
[Resolve]
DNSStubListener=no
EOF

systemctl daemon-reload
systemctl restart systemd-resolved.service

coredns will be Running, but containerd not need do this, why?
root@ubuntu:~# kubectl -n kube-system rollout restart deployment/coredns
deployment.apps/coredns restarted

root@ubuntu:~# kubectl -n kube-system get pods
NAME                             READY   STATUS    RESTARTS      AGE
coredns-6bc9c8896c-4phff         1/1     Running   0             8s
coredns-6bc9c8896c-k5f4l         1/1     Running   0             7s
etcd-ubuntu                      1/1     Running   0             21m
kube-apiserver-ubuntu            1/1     Running   0             21m
kube-controller-manager-ubuntu   1/1     Running   1 (19m ago)   21m
kube-proxy-nz762                 1/1     Running   0             21m
kube-scheduler-ubuntu            1/1     Running   2 (18m ago)   21m
root@ubuntu:~#

What alternatives have you considered?
No response

or delete #loop
root@ubuntu:/etc/cni/net.d# kubectl -n kube-system edit configmap coredns
# Please edit the object below. Lines beginning with a '#' will be ignored,
# and an empty file will abort the edit. If an error occurs while saving this file will be
# reopened with the relevant failures.
#
apiVersion: v1
data:
  Corefile: |
    .:53 {
        errors
        health {
           lameduck 5s
        }
        ready
        kubernetes cluster.local in-addr.arpa ip6.arpa {
           pods insecure
           fallthrough in-addr.arpa ip6.arpa
           ttl 30
        }
        prometheus :9153
        forward . /etc/resolv.conf {
           max_concurrent 1000
        }
        cache 30
        #loop
        reload
        loadbalance
    }
kind: ConfigMap
metadata:
  creationTimestamp: "2022-11-11T03:17:38Z"
  name: coredns
  namespace: kube-system
  resourceVersion: "11307"
  uid: f959dd94-5f51-4374-90ff-c26e8eef17fb

fixed by
KUBELET_EXTRA_ARGS="--resolv-conf /run/systemd/resolve/resolv.conf

maybe this is a  bug ,using sealos install 1.24.0 try it.

https://github.com/labring/sealos/issues/2074

It seems that k8s has been processed, but it doesn't take effect?

The RP: https://github.com/labring/cluster-image/pull/252
have basic test success
root@node1:~# kubectl get nodes -o wide
NAME    STATUS   ROLES           AGE    VERSION   INTERNAL-IP     EXTERNAL-IP   OS-IMAGE           KERNEL-VERSION      CONTAINER-RUNTIME
node1   Ready    control-plane   167m   v1.25.3   <IP_ADDRESS>   <none>        Ubuntu 22.04 LTS   5.15.0-27-generic   cri-o://1.25.1
node2   Ready    <none>          166m   v1.25.3   <IP_ADDRESS>   <none>        Ubuntu 22.04 LTS   5.15.0-27-generic   cri-o://1.25.1
node3   Ready    <none>          166m   v1.25.3   <IP_ADDRESS>   <none>        Ubuntu 22.04 LTS   5.15.0-27-generic   cri-o://1.25.1
root@node1:~# kubectl get pods -A
NAMESPACE          NAME                                       READY   STATUS    RESTARTS       AGE
calico-apiserver   calico-apiserver-5bdbcd4467-gdfvd          1/1     Running   1              165m
calico-apiserver   calico-apiserver-5bdbcd4467-qtv5r          1/1     Running   1              165m
calico-system      calico-kube-controllers-85666c5b94-5zxmw   1/1     Running   1              166m
calico-system      calico-node-gkvsm                          1/1     Running   1              166m
calico-system      calico-node-xrzn8                          1/1     Running   1              166m
calico-system      calico-node-zk6kd                          1/1     Running   1              166m
calico-system      calico-typha-7dd4b4b4cf-c99n2              1/1     Running   1              166m
calico-system      calico-typha-7dd4b4b4cf-krfqj              1/1     Running   1              166m
calico-system      csi-node-driver-bktjd                      2/2     Running   2              165m
calico-system      csi-node-driver-k7hwj                      2/2     Running   2              165m
calico-system      csi-node-driver-mcswq                      2/2     Running   2              165m
kube-system        coredns-565d847f94-6zqd9                   1/1     Running   1              167m
kube-system        coredns-565d847f94-wshz6                   1/1     Running   1              167m
kube-system        etcd-node1                                 1/1     Running   1              167m
kube-system        kube-apiserver-node1                       1/1     Running   1              167m
kube-system        kube-controller-manager-node1              1/1     Running   4 (119m ago)   167m
kube-system        kube-proxy-8hvcp                           1/1     Running   1              167m
kube-system        kube-proxy-985td                           1/1     Running   1              167m
kube-system        kube-proxy-zwvn7                           1/1     Running   1              167m
kube-system        kube-scheduler-node1                       1/1     Running   4 (119m ago)   167m
kube-system        kube-sealos-lvscare-node2                  1/1     Running   1              166m
kube-system        kube-sealos-lvscare-node3                  1/1     Running   1              166m
tigera-operator    tigera-operator-6675dc47f4-f8wmt           1/1     Running   6 (119m ago)   166m

but have two problems
1、crio access private registry use config.json,
[crio.image]
global_auth_file = "/var/lib/kubelet/config.json"

now At present, this piece is dead with YWRtaW46cGFzc3cwcmQ=
root@ubuntu:/data/git/cluster-image/cri-o# cat etc/config.json.tmpl
{
        "auths": {
                "{{ .registryDomain }}:{{ .registryPort }}": {
                        "auth": "YWRtaW46cGFzc3cwcmQ="
                }
        }
}

2、when i run .github/scripts/kubernetes.sh

It use /etc/resolv.conf

Tested with sealos v4.1.3 and v4.1.4 -rc1

v4.1.3 ok with kubernetes v1.24.x v1.25.x
v4.1.4-rc1 failed with kubernetes v1.24.x v1.25.x  (have resolvConf: /etc/resolv.conf in kubeadm-init.yaml )

fixed in v4.1.4-rc2

https://github.com/labring/cluster-image/pull/252


Added Vosk API and updated README

Now, speechrecognition supports offline API called vosk: https://github.com/alphacep/vosk-api
It is completly free and requires no api key

Last, you need to update README and docs accordingly.

@nshmyrev Thank you so much for your help.

Great, thank you. WIll do everything tomorrow
You can keep writing all errors, and i will review them tomorow

Everything is resolved and can be merged
@Uberi

Hi, guys!
@nshmyrev any predictions on when this will be merged?

Hey all, thanks for putting this together and for reviewing! Looks good to me


GAIA sessions not managed properly for multiple users.

Stealthy is using an early version of the Blockstack Android SDK and observed an issue where a user's GAIA session remains scoped to the first ID they log in with after logging out and logging in with a second.
This is a significant issue for users with multiple IDs--the session is so persistent, we've found you literally have to uninstall the App and clear the Android phone's Chrome cache (the version of the SDK we're using is proxying all the Blockstack API calls to a webview running Blockstack.js).
Here are the STR (Steps to Reproduce):

Have an a Blockstack Android App that writes encrypted files to GAIA and reads them back.
Log in with your first.id and write an encrypted file to gaia, e.g. my_encrypted_file.json
Log out
Log in with your second.id and write the same encrypted file to gaia, my_encrypted_file.json
Log out
Log back in as first.id and try reading my_encrypte_file.json

This will fail because the file was over-written in step 4 because the GAIA session is still pointing to first.id's gaia from step 1.

@shreyasthiagaraj Spoke with @jcnelson  about this and am filing it here to document the situation. Not sure if you've addressed this already in Android.

With the upcoming SDK 0.4.0, I couldn't reproduce the problem.

The issue in 0.3.0 is that the user can sign in twice without the item in the browser's localstorage for the gaia url been updated.
@AC-FTW If you can force signout, before logging in again that should fix it for the user with 0.3.0.

I close this issue as it is fixed in 0.4.0


Problems reading files when developing in a localhost env ?

Hello flyimg team !
I'm trying to get working flyimg on two separate docker containers and i have the following error :
Error occurred while trying to read the file Url :

Here's my docker configuration using your documentation and my configuration :
version: '3.8'

services:
 nginx:
    container_name: symbioze-nginx
    image: nginx:latest
    ports:
      - '80:80'
      - '443:443'

  images:
    container_name: symbioze-images
    image: flyimg/flyimg-build
    ports:
      - '81:80'

I did nothing into my nginx configuration because i think it's not needed (?). But when i try to use
http://localhost:81/upload/w_600,h_300,smc_1,q_90/https://www.symbioze.dev/uploads/pages/contents/destiny-2-warmind-622b6192d06bc.jpg

I can't get nothing but the error "can't read the file".
I tried to debug a little, but it seems to be the Line: 93 at /var/www/html/src/Core/Entity/Image/InputImage.php who is throwing the error :
if (!$stream = @fopen($this->sourceImageUlr, 'r', false, $context)) {

Any ideas ? :)
Thanks in advance

Hi @RemyCampanals,
thank you for opening this issue and sorry for the late reply.
is the source of your image accessible already?
I mean https://www.symbioze.dev/uploads/pages/contents/destiny-2-warmind-622b6192d06bc.jpg ?
did you try with another one? like https://raw.githubusercontent.com/flyimg/graphic-assets/main/logo/raster/flyimg-logo-rgb.png

Sorry, it was a ssl problem between the two containers (nginx -> flyimg -> nginx) !
It was totally on my side, you can close the issue ! :)
Thanks by the way !

Sorry, it was a ssl problem between the two containers (nginx -> flyimg -> nginx) !
It was totally on my side, i close the issue ! :)
Thanks by the way !


Added Properties_Restrictions Implementation

Updated the restrictions.rst file with the implementation and working example of the new feature implemented as a part of Exclusion_List, which is the Exclusion of Restricted EntitySet_Properties.
Made changes to the section: EXCLUDE Restrictions, under the subsection named as Exclude_Properties_from_URI_Query_Options with example of the restricted URI and restricted Properties in the Query Options $filter and $orderby.

Codecov Report
Base: 63.14% // Head: 63.14% // No change to project coverage :thumbsup:

Coverage data is based on head (c420da5) compared to base (c3482d5).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #125   +/-   ##
=======================================
  Coverage   63.14%   63.14%
=======================================
  Files          19       19
  Lines        3744     3744
=======================================
  Hits         2364     2364
  Misses       1380     1380

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

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


Mapping conflict when indexing data

I'm getting the following issue when indexing data into Weaviate:
"stacktrace": ["java.lang.IllegalArgumentException: mapper [_meta.interpretation.source.weight] cannot be changed from type [long] to [float]"
This issue is caused by the dynamic mapping functionality  of Elasticsearch.
Take the following example:
1 - First docoument comes in with a weight of 1
2 - ES recognises as a Long mapping
3 - Second doc comes in with weight value of 0.12
4 - Exception for trying to change mapping from long to float
This can be fixed by creating a dynamic index template for all Weaviate properties in ES.

I think your analysis is spot on, while we do explicitly map the user-defined schema-props, the meta props are implicit. So ES might really choose the wrong type based on the first entry.

Closing as obsolete as Elasticsearch is no longer used in 0.23.0 and 1.0.0.

Closing as obsolete as Elasticsearch is no longer used in 0.23.0 and 1.0.0.


[bug] Issues with managing and reporting machines

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
I am having 2 sets of issues with machines:
First, I had created some machines and was working with them. I destroyed the clusters, deleted the machines in Omni, wipe/reset the machines, booted fresh on PXE, and one of the nodes was up and in maintenance mode, but not showing in the UI at all.
However, with some investigating, I found that it is shown on the cluster creation page, and is shown with omnictl. However, I cannot delete it with omnictl, since it says permission denied.
Machine List

Cluster Creation

omnictl output and deletion

Mystery stuck machine is b06c08f0-a27a-24f5-deaf-f02f741ab0a6.
Second, since I decided to start over with the machines, none of the other machines have been showing any metadata in the UI or over CLI, so I can't apply any machine classes based on hardware, and creating a new cluster does not pre-select the installation disk at all. These are on new machines that I've spun up with Proxmox. I have deployed to them with a config patch to select a disk. I've tried wiping them, I've tried creating brand new VMs, I've even tried both PXE booting and ISO booting. I had seen #50, however wiping did nothing, and even on a new machine with no install, there is nothing to wipe.

Expected Behavior
Be able to see all the machines, be able to remove machines, and be able to see machine metadata.
Steps To Reproduce

Create new VM
Wait for it to boot
Machine shows in Omni, but with no metadata

What browsers are you seeing the problem on?
Chrome
Anything else?
No response

It's actually:
omnictl delete link b06c08f0-a27a-24f5-deaf-f02f741ab0a6
To remove the machine from Omni.

@Unix4ever Thanks. Deleting the link cleared up the issue with the stuck machine.

This should be fully resolved (so you cant get in this state) with the recent release of Omni


Failure on vercel integration while the build deployment was success

What version of Remix are you using?
1.14
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce
deploy this repo into vercel:
https://github.com/dellwatson/remix-test-three
Expected Behavior
it should shows error on build or work like in localhost
Actual Behavior
while it's running on local and shows no error on building in the local+ vercel.
the site https://remix-test-three-seven.vercel.app/ still produce error
and shows Unexpected Server Error

Please follow the Vercel docs for deploying Remix to their service. They've got dedicated resources and libraries to make it a great experience:
https://vercel.com/docs/frameworks/remix


59 is not a valid cURL Multi Handle resource

Hi!
I am getting a warning related to Guzzle which is happening randomly and it is being logged millions of times (more than 3M warnings in 2 minutes). These are the two messages:
PHP Warning: curl_multi_exec(): 59 is not a valid cURL Multi Handle resource in /var/www/app/20160715081907/vendor/guzzle/guzzle/src/Guzzle/Http/Curl/CurlMulti.php on line 238
PHP Warning: curl_multi_info_read(): 59 is not a valid cURL Multi Handle resource in /var/www/app/vendor/guzzle/guzzle/src/Guzzle/Http/Curl/CurlMulti.php on line 254

Guzzle 5.2.0 and PHP 5.6

The only thing I can think of is maybe you're running HTTP requests in a destructor somewhere while your application is shutting down?

This issue was automatically classified as stale because it lacks information or cannot be reproduced.
If you think the issue is still valid and/or misclassified, please comment on the issue thread
and provide additional information (if possible).
Also, since the time maintainers can spend on the project is limited, please provide
a possible solution if possible, especially if it involves a platform that we cannot
test ourselves (Windows, very old/specific PHP versions, etc).
Issues marked as stale with no further activity will be marked as rotten
at some point and will automatically be closed within 14 days.
We do this so that we can focus on high priority issues. Thank you for your understanding.


2.0.1 -> 2.1.0

@wsm95  You have changed the property name of an interface, technically, this is a breaking change of the access-control-client package. Should have come as major release.


Could I use atlantic18n DoctrineExtension with a2lix?

Hello,
I used to use atlantic18n DoctrineExtension (stof in symfony) with gedmo with a2lixTranslationFormBundle. I got error missing entityTranslation class now. At least could I use it without Personal Translations but only Entity Translation?
Any you, the best case is I could use it without separate Entity Translation for each entity.
Thanks please :)

This is duplicate of, https://github.com/a2lix/TranslationFormBundle/issues/187
Closing


[FSDE-1381] Changed portal drawer close icon color

https://ctl.atlassian.net/browse/FSDE-1401

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-125/1/. :white_check_mark:

You can check this PRs instance in https://8b60834149119694f51f80c7182adc022dff5063-chi-pr.enterprise.ctl.io

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-125/2/. :white_check_mark:

You can check this PRs instance in https://9f7110cca15ff06f34d2f1e8879585bcc7b7d09c-chi-pr.enterprise.ctl.io

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-125/3/. :white_check_mark:

You can check this PRs instance in https://e86cf072d00c57cde08cdef43ad93b9b82e21fe4-chi-pr.enterprise.ctl.io


the spotlight regex format not clear

I was trying to mark multiple words using thre spotlight regex.
My normal approach would be -s "foo|bar|baz" but that does not seem to work.
It looks like you are using the perl regex format (default in boost) but the boost_regex_replace seems to not allow this in the way you use it.
It been a while since i used the bloated boost lib, but I recall something about adding the following arguments to regex_replace: "boost::match_default | boost::format_all";
This might solve the problem

turns out it was because I was trying to mark the "name" (like bt_stack) which is not passed through the spotlight filter. So this would be a request to make it possible to mark certain apps/processes in the logcat log.

If you want, you can test the latest version. It should include what you asked.

It seems to be done. Re-open if needed.


Streaming link URL not

Describe the bug
Anime episode sources URL is not working
Expected behavior
Normally plays the video
Actual behavior
Takes forever to load the load video,  time line stuck at 00:00
Additional context
No response

Similar issue here!
It seems that the source provider is returning 500 for some reason.
It could be that the source provider is blocking requests if the origin is not correct.

Just checked one  of the streaming URL for cors and cors is responsible for this (try it )[https://cors-test.codehappy.dev/?url=https%3A%2F%2Ffgh5.biananset.net%2F_v7%2F68d599cf606db1687408c36ef1bba53c02bad17fafb3734f86d0b112bba0a69b1c12c285a700d716f8f4b055dd1638bb1bf242c1df96e60b6fc633e3ab50baed78b31d0c5d0e7ea509c4ff6eb6ea94ffd3db37a8ec0e0b51ed2740c9302666d2586b1d06235622c932d25802334bcfdb9298f9c767a7b37571cfaa0a192a523c%2Fmaster.m3u8&origin=https%3A%2F%2Fcors-test.codehappy.dev%2F&method=get]

Yes, seems to work again. ^^

The Streaming URL works now so no more issues


MSKLC link dead

The MSKLC link at https://docs.microsoft.com/en-us/globalization/downloads is not working.
Consider fixing the 82315 and 17223 fwlinks and/or linking directly to
http://www.microsoft.com/en-us/download/details.aspx?id=22339.
(The latter fwlink is from https://support.microsoft.com/en-us/help/823010/the-microsoft-keyboard-layout-creator which is linked from the bottom of https://docs.microsoft.com/en-us/globalization/windows-keyboard-layouts.)

Hello @miloush. I apologize that we have taken so long to respond to you. The site is in the process of changing ownership and unfortunately it has taken us longer than usual to respond.
I believe that this issue has already been addressed. The links to the Keyboard Layout Creator and the Multilingual App Toolkit both now go to valid locations.
Please can you confirm and close this issue if you agree?

@jowilco Is there an ETA? The msklc is an actual jewel and it is dearly missed.

Hi @jmfrank63. Thanks for bringing this to my attention again. I believe that the issue that miloush was seeing and what is happening now are two separate issues. Previously, the link wasn't working, but now the MSKLC is no longer available for download. I'll try to find out who owns this and I'll keep this thread updated.

@jowilco sorry for missing your notice, I guess given the link is now not working again I will keep it open.

Hi @jmfrank63, @miloush - the issue with the MSKLC download is an expired signature. Might it be possible to send you the executable directly if you need it during the time that the team is working on getting the updated signature?

@jowilco that's good news to me. I'm good, I have the installer. Thank you for the offer!

The link that works:  https://microsoft-keyboard-layout-creator.software.informer.com/1.4/
(and click there on "Trusted Download 10.1Mb")
This link should replace non-working link at the bottom of these 2 pages:
https://docs.microsoft.com/en-us/globalization/windows-keyboard-layouts or the same at
https://github.com/MicrosoftDocs/globalization/blob/live/globalization/windows-keyboard-layouts.md

If someone wants to find a 3rd party download that Microsoft doesn't consider safe enough to host, that's up to them, but it shouldn't be linked from the documentation as an official source.

@miloush My bad - I did not look carefully at the link and assumed that it was from microsoft.com

Hi @PaulGor - Thank you for trying to help. I apologize that is taking longer than expected, but we are working on getting the download re-signed and it should be available soon.

When the upload get's fixed, the redirect should probably get updated here too https://github.com/MicrosoftDocs/globalization/blob/master/.openpublishing.redirection.json#L5

@PaulGor @miloush @jmfrank63 The MSKLC is live again. You can access it directly at https://www.microsoft.com/en-us/download/details.aspx?id=102134 and the links within https://docs.microsoft.com/en-us/globalization have been updated.
@nschonni Thank you for catching the redirect - I have updated that too.
I apologize for the delay, and thank you again for bringing it to our attention that MSKLC was no longer available.


[FLINK-13564] [table-planner-blink] throw exception if constant with YEAR TO MONTH resolution was used for group windows

What is the purpose of the change
Checking if a proper resolution was used for group window parameter. Without this check INTERVAL '2-10' YEAR TO MONTH would be a valid window size that would be translated to 34 milliseconds window.
Brief change log

add check in StreamLogicalWindowAggregateRule

Verifying this change
org.apache.flink.table.planner.plan.stream.sql.agg.WindowAggregateTest#testWindowWrongWindowParameter
Does this pull request potentially affect one of the following parts:

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

Documentation

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

my private travis is green https://travis-ci.org/godfreyhe/flink/builds/567259573

thanks @wuchong, i have updated

rebased

Travis passed: https://travis-ci.org/wuchong/flink
Merging..


Fix some type asserts and skip running the linter on test files

golangci-lint has been failing for a minute now - took some time to fix the type asserts I had introduced pre-linter and then got copied.
Also added the config file for golangci-lint to not hold the test files as accountable as the actual source files.

This is cool. I had to do some weird stuff myself because the linter was complaining... But I didn't think of making it shush lol
Having it silenced for tests seems pretty safe, so... Let's go with it!


Finialize SQLite Support

We'd need to change the PeeWee model definitions to include unique=False on all non-unique indexes. SQLite complains without that. This should be an easy one, but my last PR got too big.

Finished by https://github.com/mpope9/nba-sql/pull/46


Added hindi language

Purpose
Added Hindi language to the list of compatible languages.
Note: The flag has a different extension (flag_in.png) from the lang file (menu_hi.lang) because that is what is specified in the link:
List of ISO 639-1 Codes
FamFamFam flag pack ISO 3166-1 alpha-2 country codes
Testing
Unfortunately, the code can't be tested until this Devanagari script is included since Hindi has a couple hundred unique computer characters.
Link: https://www.google.com/get/noto/#sans-deva
After that is done, testing can be done by opening Terasology and going to:
Settings -> Player -> Language -> हिंदी (Hindi)

Can one of the admins please verify this patch?

Nice work! As discussed I'll approve this for GCI early and see if we can figure out what to do about the font situation soonish :-)
@GooeyHub ok to test

Hooray Jenkins reported success with all tests good!

I've added 128 Devanagari glyphs plus all 28 from the Devanagari Extended region. It looks like some of them are used for composition, so it doesn't look quite right yet:

@VaibhavBajaj can you spot the problem here?

As far as I can see, it has not been calibrated properly, so all the upper characters (we call them mathra) are one step ahead of all lower characters. Could I please see the code that has been added?

According to this: There are over 200 charachters. I can confirm that seeing these:
http://indiatyping.com/index.php/typing-tips/hindi-code
Might want to check it out.

Regardless of the problems, it is wonderful to see such a beautiful script in Terasology.

Thanks :P

While reviewing #2737 and #2739 the saga deepened 😕
Dutch is easy - nl & nl. Indonesia came in as id & id. No flag icon working in-game. Huh. Flag has to be "in". Yet India in the flag pack is "in" (and Indonesia "id"), but really it needs to be "hi", and certainly not "id" - that's confusing :D
When we arrive at merging this we might have to rename the flag icon here from "id" to "hi" (looks like the flag is missing in @msteiger's test, even) - but honestly I'd have to wait and see what our internationalization system comes up with when running the game to be sure.

@Cervator You do mean "in" to "hi", don't you? Got a bit confused in all that, :)? Well, I just hope that someday I will see Hindi in Terasology, the same Hindi that I spent 8 sweating hours translating, :P.

Er, yeah. Maybe. I think. I don't know anymore!?!? 😕 😕 😕 😕 😕 😕

Sorry for the delay - here's the commit: https://github.com/MovingBlocks/Terasology/commit/93f85ebf973fdfe50e74f568639a283cad4e0f45
I did not change code, just added the glyphs. The chars are expected to be 16bit only. Is that the case here? The glyphs are rendered one by one as specified in the file. Maybe that commit helps you find out - I don't know what it's supposed to look like, but I'm happy to assist.
What Unicode ranges are you referring to? Is there more than "Devangari" and "Devangari Extended" or does the Noto font not support all glyphs?

Since I am completely new to this, I don't think I have much hope of spotting the error. Sorry, sir.
However, could you humor me so that I may understand this better? I am confused. When I downloaded Noto font, I only saw 40 or so characters. Here I see you have added hundreds. Also, is there a way I can download the .png file or is it self-generated?

I have viewed the hindi characters:

I now only have a few questions:

Is it possible some characters were cut off screen like the | charachter. I cannot spot it anywhere and that may be why some of the characters are misplaced.
Could it be that the characters have different codes in Noto and thus, some characters are messed up. I would love to see more .png files like the one you shared in the topmost post but from what I can see, I would say the EE(ी) character and the AA ( ा) character have been messed up.

About the flags: we might need a (bi-directional) mapping from language to country such as Hindi <-> India.
I'm using BMFont to generate those images. You can download the installer and the config files we use here: https://github.com/MovingBlocks/TeraMisc/tree/master/fonts
For Noto Sans Devanagari, I see the following glyphs:

This seems to be in line with the Unicode standard: https://en.wikipedia.org/wiki/Devanagari_(Unicode_block)
Afaics, the issue is with the dotted circle (a "Spacing Combining Mark"). Maybe we can construct a simple example consisting of 2-3 glyphs? Maybe reversing the order of glyphs already solves the issue.

@VaibhavBajaj as I see it, the translation for "Hindi" has 5 chars. The f-shaped one is the second, but apparently it should be put before the first one. Is that correct? I wonder where the offsets are stored in the font ..

@msteiger Yep That's about correct. Here they are on your BMFont chart (x, y):

ह
ि
Half of न as in न्द (This can also be replaced by a dot on top of the second alphabet)
द
ी
हिन्दी

@msteiger Have you come across any new discoveries. I just wanted to let you know that I was sticking around after GCI as well. So, if I can help in any way, please ping me. 👍

Yes, I tried Hiero, but it even fails to render the word "Hindi" in Devanagari typescript. We need another tool to generate the textures properly.

@msteiger I will try my best to start understanding this code's true working. Then I would be able to explain why the print is failing.
For starters, could you tell me what Hiero is? Also, just so you could experiment, I have added a Devanagari translator that you can fool around with to see exactly how the Devanagari is written.
Link: http://www.lexilogos.com/keyboard/devanagari.htm
Also, here are some guidelines on using it:

The main thing I want you to see is how upper characters are joined to the lower characters.
For starters, type: h i n d I (Cap-sensitive Last char is Cap i).
Next you can try using the letters: a A e E o O etc. to see how words are NORMALLY written when writing Devanagari.

I have come to think that instead of putting the upper character above the one right below it, the current program might be putting it AFTER the current character which is causing issues. Check it out whenever free.
Meanwhile, I will try understanding the exact working of the language system.

That's a great tool. I still haven't figured out completely how the characters are composed. We're using BMFont to create textures from typescript glyphs. This step already fails as it renders the dotted circles as well. The website lists a few alternative tools and one of them is Hiero. Unfortunately, it doesn't work either. Maybe you can find one that does work?

@msteiger I believe rendering is the only thing failing us here. Is there a way that instead of loading the text as, well, text, we could load it instead as an image of some sort?
If not, I guess we will have to keep searching for tools. What is the name of the website you mentioned?

@msteiger I tried using the BMFont tool. Need some help on how to use it. From what it looks to me, I suppose it converts a ttf to bmp, however when I import a ttf file from the "Font Settings" nothing happens.  How did you generate those Devanagari glyphs?
Also, the problem with the dotted circles being rendered can be solved if somehow we can get all character combinations with all the different 'mathras' (i.e ड़ी की are stored separately, even though both have a common ी character). Is this possible?

@nihal111 You mean combining each of the normal characters with each of the mathras and storing it? Well. That is a possibility but it will increase the number of characters to around 600 or so. Probably more. What software would you use? Thanks for the suggestion.
@msteiger Could you please answer my earlier post? Thanks.

No of characters wouldn't be a restriction, would it? :)
I'm wondering if it's possible using BMFont.

Number of characters probably wouldn't be terrible with 3 digits of combinations. 4 or 5 digits ... that gets a little crazier :-)
Also, dunno why this got closed and the branch deleted. We need to figure it out sooner or later, good research here.

Sorry, I didn't see the question in the upper post. The website is this one: http://www.angelcode.com/products/bmfont/
I suppose that we could render all pairs, but this would require special treatment for this typescript (and maybe others). We probably need to incorporate the offsets, but it would be good if it was consistent across all languages somehow.
What I usually do to add a new typescript font is this: create it using BMFont and merge it into the NotoSans-Regular.fnt definition file that is currently used as a default for the menu. You might need to adjust page number and glyph count.

For reference this commit has been merged now :-)
Fixed up the language file and verified its entry shows up in-game, although without symbols due to the known issue. Then I merged #2784 on top to promptly hide it in-game until we resolve said symbol issue.
@VaibhavBajaj thanks again! I look forward to when we find a solution that'll let Hindi show in-game.


Akira xue patch

change  the  pull  of  NavBar

add  NavBar Style change

Can you fix eslint?

Can you explain your motivation ?

???


stat: fix cargo clippy complaint (unnecessary_sort_by)

cargo clippy is complaining about using Vec::sort_by(|a,b| b.cmp(a)) as a complicated closure (see unnecessary_close_by). I'm not sure I agree, but this should be an equivalent construction which avoids the issue. Benchmark comparison on StackOverflow shows no real difference in performance.
Alternatively, I can add a clippy allow exception for the offending line, but, of the two options, I'm inclined to this repair.
Thoughts/concerns?

Sounds good, thanks!


Crash at initialization when compiling with go 1.23

When compiling nctl with go 1.23 I get the following panic at startup:
panic: pattern "GET /debug/pprof/" (registered at net/http/pprof/pprof.go:100) conflicts with pattern "/debug/pprof/delta_heap" (registered at github.com/grafana/pyroscope-go/godeltaprof@v0.1.7/http/pprof/pprof.go:24):
	GET /debug/pprof/ matches fewer methods than /debug/pprof/delta_heap, but has a more general path pattern

https://github.com/grafana/agent/issues/6972 and https://github.com/grafana/alloy/issues/1540 seem to be the same error.
It seems to be fixed in https://github.com/grafana/pyroscope-go/releases/tag/godeltaprof%2Fv0.1.8

I think it was already fixed in https://github.com/ninech/nctl/commit/0c54c5b96690bdc6c8171ae9e2c018fd547874c0 so only a release is necessary.

We just released version 1.74 which fixes this issue. Thanks for reporting!

Thanks, seems to fix it :slightly_smiling_face:


Advice on usage for offline solution

Hey everyone, not actually an issue, would just appreciate your opinion on this.
Kind of in a hurry to implement a solution for an app that does the usual data fetching from a REST like API but also requires an offline mode and persistence on disk (i.e. available between restarts). I'd say there's plenty of data (mostly text, as images are being addressed by SDWebImage), it's a wordpress website.
I'm trying to avoid the Core Data solution and was wondering if TMCache would be ok for it. Assuming that all my model objects conform to NSCoding, on every request i'd get the data from the cache and present it on screen, if available. Then would fetch the latest data from the API and cache it, using the url as key.
Also thought of including a pre-loaded cache in the app bundle.
Do you see any major problems with this?
Thank you for your time

TMCache is a good choice, and with TMCache, you don't need to implement NSCoding protocol by yourself.

Yes, TMCache will work fine for this use case.
Please keep in mind, however, that TMCache is deprecated. It'll work fine for your needs but it might be worth looking for an alternative for long-term, strategic use.

Will do. Thank you


Using nostrconnect URI does work after the first time

After using the fix of #490 with the same example in
https://github.com/rust-nostr/nostr/blob/3d9ebbbe7fdf2fb27990b3ff83353f40c311027e/crates/nostr-sdk/examples/nostr-connect.rs#L28
it seems work at first. But if I cache the generated URI and app key in a json file and on a second run, read the URI from the file, parse it and use it as a Nip46Signer, it times out in the same line as the original issue.
Amber does not show anything in the logs.

I think you have to cache the bunker URI and use that for next connections. You can get it after signer initialization:
let signer = ...;

let bunker_uri = signer.nostr_connect_uri().await;

Yeah, dummy me. This part of the spec is not so intuitive tbh. Thanks!


[STM32] Handle STM32H7 "_C" pins

As discussed in the chat, the split-pxy features are introduced for use with https://github.com/embassy-rs/stm32-data/pull/254
Ideas for improvement:

Verify this actually works with the split functions enabled (the traits are there, but I have no device that actually exposes the PC2/3 pins)
Add a testcase / CI step for compilation with these flags. Any suggestions where that should go? Or maybe a HIL test on an H7 board?

Add a testcase / CI step for compilation with these flags. Any suggestions where that should go?

yes, would be good to add a few test cases of building embassy-stm32 with/without the split-xx features to ci.sh.

Or maybe a HIL test on an H7 board?

perhaps later, since we don't even have ADC hil tests for now...

Won't be able to work on this for the next 2 weeks or so, but I'm happy to receive feedback in the meantime and make a proper implementation later on.

closed in favor of #1988


add unzip dep

Signed-off-by: Dimitris Karnikis<EMAIL_ADDRESS>

Addressed in https://github.com/binpash/pash/pull/467


Fix read

I will release a new version of the library this evening or tomorrow. Thank you for your contribution!


Compile fails on up-to-date Fedora 18 system

What steps will reproduce the problem?
1. Download tarball (or zip file)
2. Expand
3. assert ./configure

What is the expected output?
effective Makefile

What do you see instead?
clean terminal output; error-filled config.log
]$ grep -i error config.log
g++: error: unrecognized command line option '-V'
g++: fatal error: no input files
g++: error: unrecognized command line option '-qversion'
g++: fatal error: no input files
gcc: error: unrecognized command line option '-V'
gcc: fatal error: no input files
gcc: error: unrecognized command line option '-qversion'
gcc: fatal error: no input files
conftest.c:11:28: fatal error: ac_nonexistent.h: No such file or directory
conftest.c:11:28: fatal error: ac_nonexistent.h: No such file or directory
conftest.c:59:23: error: expected expression before ')' token
conftest.c:60:24: error: expected expression before ')' token
conftest.c:61:13: error: '__uint16' undeclared (first use in this function)
conftest.c:61:24: error: expected expression before ')' token
conftest.cpp:30:28: fatal error: ac_nonexistent.h: No such file or directory
conftest.cpp:30:28: fatal error: ac_nonexistent.h: No such file or directory
conftest.cpp:63:37: fatal error: google/malloc_extension.h: No such file or
directory
conftest.cpp:30:37: fatal error: google/malloc_extension.h: No such file or
directory
/usr/lib/gcc/x86_64-redhat-linux/4.7.2/../../../../include/c++/4.7.2/bits/c++0x_
warning.h:32:2: error: #error This file requires compiler and library support
for the ISO C++ 2011 standard. This support is currently experimental, and must
be enabled with the -std=c++11 or -std=gnu++11 compiler options.
conftest.cpp:35:7: error: 'unordered_map' in namespace 'std' does not name a
type
conftest.cpp:36:33: error: 't' was not declared in this scope
/usr/lib/gcc/x86_64-redhat-linux/4.7.2/../../../../include/c++/4.7.2/bits/c++0x_
warning.h:32:2: error: #error This file requires compiler and library support
for the ISO C++ 2011 standard. This support is currently experimental, and must
be enabled with the -std=c++11 or -std=gnu++11 compiler options.
conftest.cpp:35:7: error: 'tr1' in namespace 'std' does not name a type
conftest.cpp:36:33: error: 't' was not declared in this scope
conftest.cpp:35:7: error: 'unordered_map' in namespace 'std' does not name a
type
conftest.cpp:36:33: error: 't' was not declared in this scope
conftest.cpp:41:14: error: 'std::tr1' has not been declared
conftest.cpp:41:24: error: expected primary-expression before 'int'
conftest.cpp:41:24: error: expected ',' or ';' before 'int'

What version of the product are you using? On what operating system?
sparsehash-2.0.2
Fedora 18 - X86-64 PC

Please provide any additional information below.
tarball contains both configure.ac and configure. Autoconf does not modify or
replace configure, so configure.ac seems out of place.
Renaming configure to configure.backup produces a new configure file, with the
following message:
]$ autoconf
aclocal.m4:16: warning: this file was generated for autoconf 2.68.
You have another version of autoconf.  It may work, but is not guaranteed to.
If you have problems, you may need to regenerate the build system entirely.
To do so, use the procedure documented by the package, typically `autoreconf'.
There is no autoreconf.
The new configure file produced the cited error list. The error list produced
by the downloaded configure file was similar.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 8 Apr 2013 at 4:47

There are no libraries as sparsehash is header only.
Running "make" creates a number of binaries on the fly to test your sytem is compatible with sparsehash.


Deserialization failure

Hi did not check deep into it what is wrong, but on livenet some of the dashj (0.15) instances are unable to sync due to the similar errors. Any info regarding this case?
2019-02-27 22:10:38.730 WARN  [NioClientManager] PeerSocketHandler:234 - [<IP_ADDRESS>]:9999 - : org.bitcoinj.core.ProtocolException: Error deserializing message<PHONE_NUMBER>0000000000fd49010100aab00f000100019cfc7ec999e13a421207b26e0d3e0561db4fed681c99773c3100000000000000320000000000000032000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000

        at org.bitcoinj.core.BitcoinSerializer.deserializePayload(BitcoinSerializer.java:210) [dashj-core-0.15.jar:]
        at org.bitcoinj.core.BitcoinSerializer.deserialize(BitcoinSerializer.java:171) [dashj-core-0.15.jar:]
        at org.bitcoinj.core.PeerSocketHandler.receiveBytes(PeerSocketHandler.java:155) [dashj-core-0.15.jar:]
        at org.bitcoinj.net.ConnectionHandler.handleKey(ConnectionHandler.java:223) [dashj-core-0.15.jar:]
        at org.bitcoinj.net.NioClientManager.handleKey(NioClientManager.java:86) [dashj-core-0.15.jar:]
        at org.bitcoinj.net.NioClientManager.run(NioClientManager.java:122) [dashj-core-0.15.jar:]
        at com.google.common.util.concurrent.AbstractExecutionThreadService$1$2.run(AbstractExecutionThreadService.java:60) [guava-18.0.jar:]
        at com.google.common.util.concurrent.Callables$3.run(Callables.java:95) [guava-18.0.jar:]
        at org.bitcoinj.utils.ContextPropagatingThreadFactory$1.run(ContextPropagatingThreadFactory.java:49) [dashj-core-0.15.jar:]
        at java.base/java.lang.Thread.run(Thread.java:834) [java.base:]
Caused by: java.lang.NullPointerException
        at org.bitcoinj.core.Transaction.setExtraPayloadObject(Transaction.java:1546) [dashj-core-0.15.jar:]
        at org.bitcoinj.core.Transaction.parse(Transaction.java:644) [dashj-core-0.15.jar:]
        at org.bitcoinj.core.Message.<init>(Message.java:96) [dashj-core-0.15.jar:]
        at org.bitcoinj.core.Message.<init>(Message.java:128) [dashj-core-0.15.jar:]
        at org.bitcoinj.core.ChildMessage.<init>(ChildMessage.java:57) [dashj-core-0.15.jar:]
        at org.bitcoinj.core.Transaction.<init>(Transaction.java:291) [dashj-core-0.15.jar:]
        at org.bitcoinj.core.BitcoinSerializer.makeTransaction(BitcoinSerializer.java:366) [dashj-core-0.15.jar:]
        at org.bitcoinj.core.BitcoinSerializer.makeMessage(BitcoinSerializer.java:232) [dashj-core-0.15.jar:]
        at org.bitcoinj.core.BitcoinSerializer.deserializePayload(BitcoinSerializer.java:208) [dashj-core-0.15.jar:]
        ... 9 more

Looks like at least this one transaction causes the issue. Not sure how it got broadcasted or accepted to the network (even not all block explorers shows it, or crashes as well). Lacks inputs and outputs. Type is 6. Has extra payload.
1e8bdc29f6593a9a7ec31f90ff2e17d16ab1d98903761b7dfa163077988011a2
https://chainz.cryptoid.info/dash/tx.dws?1e8bdc29f6593a9a7ec31f90ff2e17d16ab1d98903761b7dfa163077988011a2.htm
{
   "txid": "1e8bdc29f6593a9a7ec31f90ff2e17d16ab1d98903761b7dfa163077988011a2",
   "size": 342,
   "version": 3,
   "type": 6,
   "locktime": 0,
   "vin": [],
   "vout": [],
   "extraPayloadSize": 329,
   "extraPayload": "01004ab00f000100017bf8edaf91b2ec0e2881b9e9ed80599ad91179871aafca992000000000000000320000000000000032000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000",
   "qcTx": {
      "version": 1,
      "height": 1028170,
      "commitment": {
         "version": 1,
         "llmqType": 1,
         "quorumHash": "000000000000002099caaf1a877911d99a5980ede9b981280eecb291afedf87b",
         "signersCount": 0,
         "validMembersCount": 0,
         "quorumPublicKey": "000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000"
      }
   },
   "blockhash": "00000000000000316f3f7c6cd53f70e63ee09bf94deea435c36503f5511a3e09",
   "height": 1028170,
   "confirmations": 635,
   "time":<PHONE_NUMBER>,
   "blocktime":<PHONE_NUMBER>,
   "instantlock": false
}

So I believe The Quorum Commitment (QcTx) special transaction support is not implemented in the lib.
https://dash-docs.github.io/en/developer-reference#qctx
Not sure why some of the wallets (instances of dashj) receive such messages, some are not.

See https://github.com/pvyhnal-generalbytes/dashj/commit/4a1264c137056f07394c6245f314eb058c00e54b for a quick fix. The new transaction payload is not parsed but it doesn't crashes blocks downloading.

Sorry about the delay in replying.
We are going to take the fix in the previous comment and add it to our code.
We didn't come across this message in SPV mode.  This is probably because SPV mode uses bloom filters and the filters will not match a transaction without inputs or outputs.

Some transactions in Dash Core 0.13 and above will not have inputs or outputs, but will contain other information that is used by the network.
fix:  https://github.com/dashevo/dashj/pull/30/commits/caa4e4328a36f1b96ae4a088b611187e1942b2a9

After our internal testing, this PR #30 will be merged to master

Our internal testing passed so #30 has been merged.


Bump<EMAIL_ADDRESS>

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (2024ade) 76.43% compared to head (8572a4d) 76.47%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #203      +/-   ##
==========================================
+ Coverage   76.43%   76.47%   +0.04%
==========================================
  Files         194      194
  Lines       16779    16768      -11
==========================================
- Hits        12825    12824       -1
+ Misses       3954     3944      -10

Flag
Coverage Δ

unittests
76.47% <ø> (+0.04%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


go get -u github.com/micro/go-micro

Hi, I have some question  When i go get -u github.com/micro/go-micro
# github.com/micro/go-micro/transport/quic
../../gopath/pkg/mod/github.com/micro/go-micro@v1.18.0/transport/quic/quic.go:54:12: q.s.Close undefined (type quic.Session has no field or method Close)
../../gopath/pkg/mod/github.com/micro/go-micro@v1.18.0/transport/quic/quic.go:121:3: unknown field 'IdleTimeout' in struct literal of type quic.Config

Why？

This will update the dependency of github.com/micro/go-micro.
refer: go help get
The -u flag instructs get to use the network to update the named packages
and their dependencies. By default, get uses the network to check out
missing packages but does not use it to look for updates to existing packages.

Maybe you can use command GO111MODULE="on" go get -v github.com/micro/go-micro

I solve this problem by adding the following content in your go.mod
replace github.com/lucas-clemente/quic-go => github.com/lucas-clemente/quic-go v0.14.1


Help playing local files that aren't statically included in project

My project lets users download mp3 files from the web, I am using "rn-fetch-blob": "^0.10.13" to do this. There are no download issues but when I attempt to require the dynamic files I run into issues because this is a big no-no in JavaScript. So I wanted to see if there were any other ways I can work around this issue by passing in a dynamic path to the local files.
I am porting this app from a native iOS and I was using AVPlayer before and using it was fairly straightforward were I could pass in the dynamic path. But due to the limitations of dynamically sourcing in files this is becoming a major headache.
Any help would be very much appreciated.

Do you have a fork by any chance? I need to include media as an expansion file. And it looks painful. because on iOS I want to leverage existing setup with require, but for Android, I have to do some workarounds.

I didn't fork I just pulled the source from npm. I can fork and make the changes if you are interested. Or I can provide you the files on iOS and Android that were touched that made it work

Fork would be the best. I would then fork it if any additional modifications are needed for my case. Thank you.

One easy way to do it is just passing the URL in the format 'file://' + localPathToFile

Yes, I figured it out. I have to use expansion file and content provider to access it.


[Feature] Add Scoring Support for MCAS Module

Add support in the scoring module to take inputs from the MCAS module

Will use the 'AboveThreholdCount' which will allow customers to set the MCAS score level they care about, prior to assessing a score to this module.


Upgrading to React Native 0.16

@ivanph I'm currently trying to upgrade the app with react native 0.16 under the react-native-0_16 branch, could you confirm if everything is fine with android

Hi @iSimar
The following folders are missing

android/app/src/main/res/drawable-hdpi/
android/app/src/main/res/drawable-mdpi/
android/app/src/main/res/drawable-xhdpi/
android/app/src/main/res/drawable-xxhdpi/
android/app/src/main/res/drawable-xxxhdpi/

They contain the navigation icon for the toolbar in different sizes.
Also package.json is missing the react-native-webintent module that is used to open the posts links.
Maybe you could create a PR and we can discuss the changes there.
Cheers.

I tried upgrading react native to 0.16 with package.json & npm install but it broke a bunch of stuff, so I had to react-native init a new project and copy over the stuff manually.
I have added the missing stuff you mentioned above just now. I did not add react-native-webintent because I think it would be better to upgrade to IntentAndroid. See commit 6e52dcb

I created a branch with the upgrade to version 0.16 and replaced the webintent module with IntentAndroid.
It might be easier if we merge that branch first and then you could add your changes on top.

I'm getting this error when I checked out the branch and installed dependencies

@amilajack I don't seem to be getting  that error after cloning/checking-out the branch. Try pulling the new commits see if you're getting the same error.

Hi,
Tried Android on win10 running node v4.2.1 with npm 3.5.2

If I just try a git clone with repo iSimar or ivanph I cant get npm install to run
I created a sep directory with react-native init HackerNewsReactNative
I then copied package.json, index.android.js and app dir but get "unable to download js bundle error." on my emulator which does show the hacker icon.
John

@johntom I'm unaware of this issue, I will try to look into it.
@ivanph I'm actually recoding most of the app, I don't think that would be necessary

The master has been updated.


New Rule (Attachment): Office file contains OLE relationship to credential phishing page

Yessss this is exactly what I had in mind with this function 💵


Updated grafana image to current tag

Currently i only see the 4.0.2 in the repo:
https://console.cloud.google.com/gcr/images/google-containers/GLOBAL/heapster-grafana-amd64

This change is

updated CLA

I've built gcr.io/google-containers/heapster-grafana-amd64:v4.2.0. Please check the new one.

👍


Can not intercept all the log messages with llama_log_set

NativeApi.llama_log_set allows to intercept log messages from llama.cpp. But even if you set log callback as early as possible in your code, you will loose a few first messages that are produced by a call to the llama_backend_init from here:
https://github.com/SciSharp/LLamaSharp/blob/fa73c8f07c220c42ea26e2d75267564b7fae7250/LLama/Native/NativeApi.Load.cs#L38
For example, this messages aren't handled by the provided callback:
ggml_init_cublas: GGML_CUDA_FORCE_MMQ:   no
ggml_init_cublas: CUDA_USE_TENSOR_CORES: yes
ggml_init_cublas: found 1 CUDA devices:
  Device 0: NVIDIA GeForce <...>

In order for NativeApi.llama_log_set to be executed, static constructor of NativeApi must run. This static constructor will load native library, and also call llama_backend_init. So there is no way to set a callback with NativeApi.llama_log_set before that.
There is also a slightly related problem with logging, but this time with LLamaSharp messages: logs, activated by NativeLibraryConfig.Instance.WithLogs() will always be written to the console and currently there is not way to redirect them elsewhere.
One possible solution is to make an overload for NativeLibraryConfig.WithLogs() that accepts an optional LLamaLogCallback, and pass it to the llama_log_set during the loading before any other function of llama.cpp is called. Also the same callback instance may be used to intercept LLamaSharp initialization log messages, or maybe it should be another dedicated instance, passed along with the one for llama.cpp.

I'll look into this

@dlyz #649 should fix this issue. I'd appreciate your feedback on that PR :)

Hi, I'm afraid this hasn't been fixed yet, on 0.12.0 I redirect the logger before the llama backend initializes and still get this message output to the console.
using ILoggerFactory factory = LoggerFactory.Create(builder => builder.AddDebug());
ILogger debugLogger = factory.CreateLogger("Program");

NativeLibraryConfig.Instance.WithLogCallback(debugLogger);
NativeLibraryConfig.Instance.WithCuda(true);

that output:
ggml_cuda_init: GGML_CUDA_FORCE_MMQ:   no
ggml_cuda_init: CUDA_USE_TENSOR_CORES: yes
ggml_cuda_init: found 1 CUDA devices:
  Device 0: NVIDIA GeForce ...

And all other messages are output to the debug window like:
Program: Information: Detected cuda major version 12.
Program: Information: Loaded './runtimes/win-x64/native/cuda12/llama.dll'
Program: Information: llama_model_loader: loaded meta data with 24 key-value pairs and 195 tensors from ...

Yes, it works.

In that case we'll have new nuget packages out soon which include that fix :)

Those binaries have been released, so I'll close this issue now. If it's still a problem please don't hesitate to re-open or create a new issue!


proposition for crm async action output

We may want to add a stringer for those payloads and use it if --format is not specified


Command clicking should "xor" selection

If you have command key held down and click on something, it should add it to the selection it if it wasn't selected or unselect it if it was already selected.

IMO regular Shift should work like that (so that you don't need to switch keys when you mis-click). At least that's what Figma & Affinity does.

Oh you are right, keynote does that as well!

@giovannigiordano this is actually still on track.


Revert "bugfixclear block for partition values for 2.1 (#39569)"

Reverts apache/doris#39729

run buildall


perf: use graphlib2 as dependency resolution backend

There are unfortunately a lot of changes tangled up in here that come from changing the dependency resolution backend from the custom existing one to basically the graphlib API. So before merging this, the following questions need to be answered:

Is graphlib2 better than graphlib for this application? It is clear that it is asymptotically faster, but it may be slower for very small inputs (benchmarks). But graphlib has no easy way to be deep-copied. Maybe we should just tweak the current implementation?
If we do use graphlib2, is it worth using the node id API over the plain hashable API? Pros are it may be faster (need benchmarks to confirm). Cons are that it is not interoperable with the stdlib graphlib implementation.
This change gets rid of subgraph pruning in favor of simply executing no-ops. This is conceptually murkier since it leaves a lot of open questions like what happens in A -> B -> C if A and C are cached but B is not (previously none would get executed, which is the right thing IMO, now B would get executed), but these are mostly edge-cases that can be resolved easily by users. Pruning subgraphs on every execution is very expensive, even if it is done in Rust extension code. I guess we need to be okay w/ this tradeoff.

Codecov Report

Merging #40 (ce7c306) into main (6044522) will decrease coverage by 0.95%.
The diff coverage is 92.71%.

@@            Coverage Diff             @@
##             main      #40      +/-   ##
==========================================
- Coverage   98.36%   97.40%   -0.96%
==========================================
  Files          61       61
  Lines        2014     2040      +26
  Branches      215      241      +26
==========================================
+ Hits         1981     1987       +6
- Misses         23       36      +13
- Partials       10       17       +7

Impacted Files
Coverage Δ

di/_utils/state.py
100.00% <ø> (ø)

di/executors.py
75.80% <61.53%> (-13.67%)
:arrow_down:

di/_utils/execution_planning.py
100.00% <100.00%> (ø)

di/_utils/task.py
100.00% <100.00%> (ø)

di/api/executor.py
90.90% <100.00%> (-9.10%)
:arrow_down:

di/container.py
99.59% <100.00%> (+0.03%)
:arrow_up:

tests/test_executors.py
100.00% <100.00%> (+2.32%)
:arrow_up:

di/api/container.py
89.47% <0.00%> (-10.53%)
:arrow_down:

docs/src/bind_as_a_dep.py
93.10% <0.00%> (-6.90%)
:arrow_down:

... and 6 more


added new departments to python script that pulls data from fr_tracking

@henhirsch Actually I can create the combined dataset by revising the Python script after you tested/merged your changes. So you don't need to worry about that.

@zhoudanxie can we merge this one now too so that I can run the test?


Remove unnecessary Broadcast()

First of all great job on this plugin!
Broadcast() can be removed in this method because what Broadcast() does is to serialize the node's properties (title, data inputs & their values, triggers, etc.) and send it to the editor (which is essentially a web client for Warudo). Since this method does not change any data input value that is exposed to the editor, removing Broadcast() can improve performance especially when there are a lot of OSC messages.

Thanks for taking a look and thanks for the feedback!


name 'urlparse' is not defined

getting this error no matter what link i input

run the previous cells

i have solve it thanks


Error on Amazon AWS Cloud?

Is there a Tutorial to deploy on amazon cloud.

ubuntu@ip-172-31-25-73:~$ sudo npm install -g torrent-web
npm WARN deprecated<EMAIL_ADDRESS>Browserslist 2 could fail on reading Browserslist >3.0 config used in other tools.
npm WARN deprecated<EMAIL_ADDRESS>Please use the native JSON object instead of JSON 3
npm WARN deprecated<EMAIL_ADDRESS>Critical security issue fixed in 5.1.3. All users should upgrade. (More info: https://github.com/feross/bittorrent-dht/issues/87)
npm WARN deprecated<EMAIL_ADDRESS>Use the parse-torrent package instead
npm WARN deprecated<EMAIL_ADDRESS>I wrote this module a very long time ago; you should use something else.
npm WARN deprecated<EMAIL_ADDRESS>One of your dependencies needs to upgrade to fsevents v2: 1) Proper nodejs v10+ support 2) No more fetching binaries from AWS, smaller package size
/usr/bin/torrent-web -> /usr/lib/node_modules/torrent-web/server.js

<EMAIL_ADDRESS>postinstall /usr/lib/node_modules/torrent-web/node_modules/uglifyjs-webpack-plugin
node lib/post_install.js

<EMAIL_ADDRESS>postinstall /usr/lib/node_modules/torrent-web
webpack

mkdir: cannot create directory ./public: Permission denied
cp: copyFileSync: could not write to dest file (code=EACCES):./public
cp: copyFileSync: could not write to dest file (code=EACCES):./public
Error: EACCES: permission denied, mkdir '/usr/lib/node_modules/torrent-web/public'
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>- 1.7.0 || ^1.7.2 but none is installed. You must install peer dependencies yourself.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/torrent-web/node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>postinstall: webpack
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR!     /home/ubuntu/.npm/_logs/2019-11-15T12_03_19_657Z-debug.log
ubuntu@ip-172-31-25-73:~$ torrent-web
torrent-web: command not found

@mccxiv any details to install it?

@ParveenBhadooOfficial did you find a workaround  ?


move from portaudio to libao

More leightweight, support for nearly all audio subsystems, crossplatform and more recently developed library.
https://xiph.org/ao/

I'm OK with the idea but it would require rust libao bindings.
The two I've found are fairly out of date, and don't build on latest rust
https://github.com/tari/rust-ao
https://github.com/pcwalton/rust-ao
They could be used as a starting point (the first one in particular)

that was an easy fix, https://github.com/tari/rust-ao/pull/2

example c code for libao:
https://www.xiph.org/ao/doc/ao_example.c

Any idea how well libao would compile on Windows? portaudio is quite a hassle, and it's way better to use a pre-compiled DLL and copy it next to the target binary. Would be much nicer to compile the required bits directly into the executable, but only if there are not tons of dependencies.

there seems to be a vcproj fork: https://github.com/manushanga/libao-win32
but nethertheless it should compile fine on windows.
i might setup a mingw compile and try that

Haha, thanks.
I'll look into it.
Since libao is so small, a nice thing to do would be making a ao-sys crate and include the source verbatim, like the ogg-sys crate does as a fallback if not found using pkg-config. That would simplify build, especially on windows and cross compiling

and libao has only optional dependencies for X Window System, ALSA-1.0.28, and PulseAudio-5.0
nothing that prevents compiling when its not available

Meh, I've just realized libao is GPL2. Unfortunately that's going to be a no go


[General] The emoji command should work with names too

The /emoji command says it works with either an emoji name or an emoji link, but it works with a link only. If feasible, make it work with names (based on relevance like /avatar) or change the misleading description.

It's a lot of work to implement the relevance search with potential bad performance, therefore discord.py's built-in emoji lookup is used instead. This means that it matches with full names (eg. "KekWa" won't match "KekWait") as well as it is case-sensitive (eg. "kekwait" won't match "KekWait").


vs code powershell auto responding Y

Issue Type: Bug
when using powershell in vs code terminal, terminal automatically responds Y to Y/N questions without any such setting enabled. also responds Y after conducting an npm install. confirmed regular powershell does not have the same behavior. this was occurring on a separate work computer as well, different windows account, installs, etc. hence i suspect it might have gotten added to a recent update. i would appreciate a workaround, or something i could add to my settings.json.
thank you!
https://i.imgur.com/02TOd6g.png
VS Code version: Code 1.64.0 (5554b12acf27056905806867f251c859323ff7e9, 2022-02-03T04:22:20.678Z)
OS version: Windows_NT x64 10.0.19043
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9700K CPU @ 3.60GHz (8 x 3600)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.94GB (7.38GB free)

Process Argv
--crash-reporter-id 7e6f7f0a-258f-4b0c-89d1-d5708ad6d5fb

Screen Reader
no

VM
0%

Extensions (5)

Extension
Author (truncated)
Version

prettier-vscode
esb
9.2.0

vue
jcb
0.1.5

vetur
oct
0.35.0

synthwave-vscode
Rob
0.1.10

code-spell-checker
str
2.1.5

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
vstes627:30244334
pythonvspyl392:30425749
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
vsc1dst:30433059
pythonvs932:30410667
wslgetstarted:30433507
vs360cf:30404996
vsrem710:30416614

The feature is new and came with the January 2022 release. Automatic Replies v1.64
You can disable it by setting the reply to null.
"terminal.integrated.autoReplies": {
  "Terminate batch job (Y/N)": null
}

thank you for your swift response! i appreciate the workaround.

main issue i would say is: automatically enabling this without user input. this could cause problems for some users.
additionally, having it respond Y after an npm i that does not prompt for [y/n] would not be expected behavior

Since this feature was added I've been seeing tons of  false activations
Y : The term 'Y' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct
and try again.


Update legal link in footer to python.org legal page

As mentioned in #1305, remove the legal page within the app and link to the legal page on python.org.  I'm not sure, based on the comments in the issue, if this change needs to be done, but I figured I would give it a shot just in case it does!

Hey!
Thanks for this PR. I'm currently waiting on @VanL to tell me what exactly we're supposed to do for the legal content, whether it should be a link to the existing page or new content or not.

On second thought, I'm going to go ahead and merge this. If @VanL wants us to do something different bringing it back is trivial.


ES6 import support?

Hey y'all,
Thanks for your work on this lib. Any plans for ES6 import support in the near future? If it's something actively being worked on I'm gonna wait, otherwise I may look into possibly PRing?
EG
import { uuidv1 } from 'node-uuid'

@broofa import uuid from 'uuid/v4'; syntax does not work, at least in a Typescript v3 project running on Node v10.9.0
on the other hand, import {v4 as uuid} from 'uuid'; works as expected

Apologies for the late response on this.  I [still] consider this issue closed, but in light of the "reactions" the above comment has garnered, let me state my position on this...

at least in a Typescript v3 project

... which makes this a Typescript (or TS configuration) issue.  This module conforms with the documented CommonJS and ES6 behaviors.  You can use ES6 syntax either directly in the browser or via webpack w/out issue, so whatever the issue is here, it's specific to TS.

@broofa actually you're right, in my case this was only due to how Typescript is handling ES6 imports
so, just like many other JS libs like this one, it works this way using TS:
import * as uuid from "uuid/v4";

console.log(uuid());

here is a codesandbox to play with: https://codesandbox.io/s/5372omm68n

@Sampath-Lokuge that doesn't work in Typescript.

@jonathanstiansen What was the error? I have used above on Ionic / Typescript app and no issues.

Typescript 3.2.4 in angular 7.2.7
Creating an object from this class
import { UUID } from 'uuid/v4';

export class entity {
  id: string;
  name: string;

  constructor() {
    this.id = UUID();
    this.name = '';
  }
}

results in

ERROR TypeError: "Object(...) is not a function"

Should be reopened and fixed.
Rollup is also giving error, with import uuidv4 from 'uuid/v4'. Plugins need to be used to fix it there and this's a single package I met which requires actions from developer to make imports work.
Current index.js with exports is a mess and should be changed to a correct one so we don't need to import from separate files. Package.json file also doesn't reflect needed files.
Es modules support would be also great since it's actually quite easy to add and maintain using modern tools.
@broofa if you are okay with this, PR's can be made to reflect this changes as this package clearly needs some fresh updates to it 😉

Current index.js with exports is a mess

Agreed.  Hence the deprecation warning about it going away.
FWIW, the problem with hanging all versions off the top-level module is that it it creates a dependency tree that causes packagers to pull in all 350-600 lines of code in this module, even in the very-common case where you only need the 30-50 lines required to implement v4.

Rollup is also giving error

What's the Rollup team have to say about this?

@broofa rollup team is pushing forward es modules, for a very long time. To get rid of such cjs errors, a plugin (rollup-plugin-commonjs) should be used there, but with top imports from index.js file it's not needed.
Top level exports from index.js file shouldn't be a problem because tree shaking is used to get only needed dependencies. Rollup and webpack both have it and should remove unused functions from bundle. So, if you only import v4 function, all other functions should be removed.

tree-shaking only works on ES6 modules (according to webpack).

@broofa yep, this is why I am proposing to add es modules to this package.
Bundlers (webpack, rollup) will use es modules and tree shake unused exports while cjs users will have a separate cjs version. This is how a lot of modern packages work nowadays :)

I recommend using pika pack, it will generate Deno, browser, and Node compatible versions https://next.pikapkg.com/blog/introducing-pika-pack/

Any ETA on this?

@cannjeff:   There's no ETA.  If your packager doesn't support ES6-style import of CommonJS modules like this one, then you need to take that up with the packager project.

Would you accept a PR that created different distribution types?

@arjunyel: Short answer? No, not at this time.
Longer answer: I know this is causing people pain, and I genuinely feel bad about not being responsive to that.  However, to the best of my knowledge this module complies with the CommonJS spec, which makes this an issue with how packagers do/does support CommonJS -> ES6 transpiling.  Adding extra complexity to this project to create packager-specific distributions (which are just going to get repackaged/transpiled a second time anyhow) is not something I'm interested in.
I'm open to continuing this conversation, but only to the extent you or others (or, ideally, the Rollup/Angular/Typescript teams) make a compelling argument for why this problem should be fixed here rather than in the packagers.  To date, I haven't seen that effort being made.

I feel like his may be relevant to the discussion here: https://jasonformat.com/enabling-modern-js-on-npm/ and will hopefully make you reconsider 😊

I came to this issue after going from https://www.pika.dev/search?q=uuid

Package found! However, no "module" entrypoint was found in its package.json manifest. If possible, explore the site for a more web-friendly alternative.

I believe as patterns like this get adopted, ES module is going to be directly used in the browser instead of being repackaged or transpiled away.

We've published a mirror of this repo which is bundled as ESM.
https://github.com/bundled-es-modules/uuid
https://www.npmjs.com/package/@bundled-es-modules/uuid
Cheers

Sorry for taking such a long time but we have finally released<EMAIL_ADDRESS>which features native ESModules-Support for the browser:
npm install<EMAIL_ADDRESS>
The master branch README already reflects the API of v7.0.0.
It would be awesome to get feedback from you if this release works well for you.

To try your different uuid releases using Typescript, you can use this sandbox if you want: https://codesandbox.io/s/gallant-voice-iwy7z
(I personnally didn't try to use uuid without TS, so I don't know how to make sure your ESModule configuration works using vanilla JS)
anyway, both versions (3.4.0 and 7.0.0-beta.0) seem to work fine with Typescript using this syntax:
import uuid from "uuid/v4"

as long as you:

add this dev dependency: "@types/uuid": "3.4.7"
make sure "esModuleInterop": true is set in your tsconfig file

@Oliboy50 thanks for trying it out and posting the link to the sandbox!
I want to highlight that the new recommended way of importing is now:
import { v4 as uuidv4 } from "uuid";

document.getElementById("app")!.innerHTML = `<h1>${uuidv4()}</h1>`;

I quickly tried it out with your sandbox and it seems to work as well! Can you confirm that?

Oh, and @Oliboy50 if I'm not totally mistaken you can drop  "esModuleInterop": false from your tsconfig if you use the new way of importing the library!

We have just released<EMAIL_ADDRESS>which now features native ES6 import support. Please feel free to open a new issue if you encounter any issues with the new version.

I get a syntax error when importing uuid
SyntaxError: The requested module 'uuid' does not provide an export named 'v4'

@dps910 Which version do you see in your node_modules?

@dps910 Which version do you see in your node_modules?

version 7.0.2

Ah, it's node. Only bundlers esm is supported for now.
https://github.com/uuidjs/uuid/pull/402
Can you try this?
import uuid from 'uuid';
uuid.v4

Ah, it's node. Only bundlers esm is supported for now.
#402
Can you try this?
import uuid from 'uuid';
uuid.v4

thanks, it works :smile:

Ah, it's node. Only bundlers esm is supported for now.
#402
Can you try this?
import uuid from 'uuid';
uuid.v4

WARNING in ./src/server/object.js 204:14-18
"export 'default' (imported as 'uuid') was not found in 'uuid'
Yes, it's the latest node trying to use 'import'

@FloodGames I'm not sure what you are trying to say.
Per the docs the recommended way of using uuid as an ES Module is:
import { v4 as uuidv4 } from 'uuid';
uuidv4(); // ⇨ '9b1deb4d-3b7d-4bad-9bdd-2b0d7b3dcb6d'

Does this not work for you?

Oh, I think I lost something in my previous answer. This does indeed currently not work in blank nodeNode.js.
Running Node.js 14.0.0 I can do the following:
import uuid from 'uuid';
console.log(uuid.v4()); // -> 'b80a7443-e722-4877-be96-4f608304ca81'

For everyone in this thread: we have just released<EMAIL_ADDRESS>which comes with native ESM support for Node.js! Let us know if that works for you. See CHANGELOG for more info.
@FloodGames @dps910  this means that the way to go from now on is:
import { v4 as uuidv4 } from 'uuid';
uuidv4();

For everyone in this thread: we have just released<EMAIL_ADDRESS>which comes with native ESM support for Node.js! Let us know if that works for you. See CHANGELOG for more info.
@FloodGames @dps910  this means that the way to go from now on is:
import { v4 as uuidv4 } from 'uuid';
uuidv4();

Yes, it is working thank you :)

Hi, I'm using jest to test my project.
Trying to test a file where I import uuid results with an error:
 ● Test suite failed to run

    SyntaxError: The requested module 'uuid' does not provide an export named 'v1'

          at async Promise.all (index 0)
      at jasmine2 (node_modules/jest-jasmine2/build/index.js:228:5)

I saw @dps910 comment and I guess it's something similar.
I'm using UUID@8, node@14 and Jest@26 (and it's freshly-supported esm modules implementation).
Jest command:
node --experimental-vm-modules .\node_modules\jest\bin\jest.js
The file that I'm testing imports uuid like you suggested:
import { v1 as uuidv1 } from 'uuid';
uuidv1();

It fails only when I'm testing though.

Had the same issue, this worked in fixing the import:
Import correct version (v1-4)
import {v4 as uuid} from "uuid"
use now the function
uuid()

This compiled to typescript doesn't work:
import { v4 } from 'uuid';
v4();

I had to downgrade to v7

const { v4: uuidv4 } = require( 'uuid' );
console.log ( uuidv4( ) )

Angular compilation warning: application.service.ts depends on 'uuid'. CommonJS or AMD dependencies can cause optimization bailouts. For more info see: https://angular.io/guide/build#configuring-commonjs-dependencies

Angular compilation warning: application.service.ts depends on 'uuid'. CommonJS or AMD dependencies can cause optimization bailouts. For more info see: https://angular.io/guide/build#configuring-commonjs-dependencies

Maybe this could be useful to someone.
In a Vue application I call uuid this way: import { v4 as uuid } from "uuid" and receive the following error:
./node_modules/uuid/index.js
Module build failed: Error: ENOENT: no such file or directory, open 'D:\AProject\node_modules\uuid\index.js'

To make it works I have to change the import to: import { v4 as uuid } from "uuid/dist/esm-browser"
Hope it helps
mario

Module build failed: Error: ENOENT: no such file or directory, open 'D:\AProject\node_modules\uuid\index.js'

@crystalfp: This package does not supply a top level index.js.  If you look at our package.json, you'll see that the main and exports sections all point to files within the dist directory.  You'll need to figure out why Vue (or tsc or whatever...) is thinks it's supposed to have one.

Locking this conversation.  Anyone having issues with importing uuid should just file a new issue.


Info

Hi,
I was checking the project, but I don't understand the utility.
What features offers nats.js in a CF environment?
Many thanks in advance.
Juan Antonio

I am sorry that README was not enough.
This project is nats client for the Node-RED
It supports the nats-pub and nats-sub
How to Use
step1: nats-server run
$ nats-server -a <IP_ADDRESS> -p 4222 --user nats --pass nats

step2: install node-red & npm install node-red-contrib-nats
$ git clone https://github.com/node-red/node-red
$ cd node-red
$ npm install
$ npm install node-red-contrib-nats --save
$ node red.js -v

step3: edit flow

step4: deploy flow

step5:
publish test message
$ nats-pub -s nats://nats:nats@localhost:4222/ 'dea.advertise' 'dea message sample'
Published [dea.advertise] : 'dea message sample'

step6:
result

@jabrena
Does that answer your question?

@jabrena
Does that answer your question?

Yes, last week, I was researching a bit about node-red.
It seems interesting.
Are you using it?

Node-RED is very interesting
I'm running on the Cloud Foundry and Raspberry Pi.

Cloud foundry is able to run on RPI2?

Oh. . . I'm sorry it is a another system
my cloud foundry running on bosh-lite(ubuntu14.04)


[Event Request] Table 7311 "Warehouse Journal Line" - OnBeforeOnValidateQtyPhysInventorySerialNo

We are migrating our code from NAV 2018 to BC16.4. In the Table 7311 "Warehouse Journal Line", in the trigger OnValidate of the field 54 "Qty. (Phys. Inventory)", we need to add a condition to the use of the if statement "Serial No." <> ''  at the beginning. To do so, we need a IsHandled flag and an event publisher, like this :

With this integration event :

Do you think it could be possible ?
Thank you for your work.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


fix: (logging): adjusted logging to error instead of warning when min…

…ify failes. This closes #106

👍


Develop a strategy to deal with cases where an HMDB ID could not be found

FEATURE REQUEST
Inspiration
Sometimes no HMDB ID can be found for a compound.  In Michael's recent "water" study, he entered PubChem IDs in the HMDB ID column, which failed validation (because they were too long - they didn't follow the strict HMDB format).  I added the PubChem IDs as compound synonyms, to not lose them, and added FakeHMDB#### IDs, sequentially, given the existing fake IDs.
Description

Make the HMDB ID field in the model optional
Add an optional PubChem ID field
Make one of the 2 IDs required (either within the model definition [if possible] or via the clean method)

Alternatives
Make the construction of fake HMDB IDs automated/assigned sequentially
Dependencies
None
Comment

None

ISSUE OWNER SECTION
Assumptions

List of assumptions that the code will not explicitly address/check
E.g. We will assume input is correct (explaining why there is no validation)

Limitations

A list of things this work will specifically not do
E.g. This feature will only handle the most frequent use case X

Affected Components

change: File path or DB table ...
add: Environment variable or server setting
delete: External executable or cron job

Requirements

[ ] 1. List of numbered conditions to be met for the feature
[ ] 2. E.g. Every column/row must display a value, i.e. cannot be empty
[ ] 3. Numbers for reference & checkboxes for progress tracking

DESIGN
Interface Change description

None provided
Code Change Description

None provided
Tests

[ ] 1. A description of at least one test for each requirement above.
[ ] 2. E.g. Test for req 2 that there's an exception when display value is ''
[ ] 3. Numbers for reference & checkboxes for progress tracking

PubChem is much deeper: it has 120 million compounds whereas HMDB has about 200k.  There are tools to fetch a PubChem ID using HMDB ID.  We could switch to PubChem as the primary identifier, although if the requirement is either/or then I don't know if that matters.

No reason we can't have both, IMO.


Multiple plugins in the Godot Asset Library

There seems to be two identical plugins in the Godot asset library:

I assume the later version is the correct one, but this could still provoke some confusion

Hi @SekoiaTree ,
Thank you for bringing this up.
I'm afraid I'm not able to remove the plugin through the Asset Library.
Initially there were two plugins for two different versions, since the Asset Library was supposed to show a plugin only for the version of Godot you are using.
This feature has not been added, probably because the current Asset Library itself is not been updated.
I'll change the descrition of the latter version.
thanks


Enabled restart policy for all containers

Inconsistent compose file fixed because a reboot started some but not all containers.

I'd suggest to set the restart policy to on-failure for long running process. only the short lifecycle processes like task-check-alerts have always policy

Thanks for the contribution! I'd suggest to set the restart policy to on-failure for long running process. only the short lifecycle processes like task-check-alerts have always policy

Works for me. My motivation to change the default compose file was: When I (accidently) restarted the machine, I noticed hyperdx no longer working. When I ran docker ps there were only a couple of hyperdx processes started, others were not. Then I started digging through logs of those exited containers to understand what was wrong, but did not find anything.
I think this is a bit of inconsistent behaviour, I would expect all being off, or all being on and working.
What is your opinion in such a scenario?

Thanks for the contribution! I'd suggest to set the restart policy to on-failure for long running process. only the short lifecycle processes like task-check-alerts have always policy

Works for me. My motivation to change the default compose file was: When I (accidently) restarted the machine, I noticed hyperdx no longer working. When I ran docker ps there were only a couple of hyperdx processes started, others were not. Then I started digging through logs of those exited containers to understand what was wrong, but did not find anything. I think this is a bit of inconsistent behaviour, I would expect all being off, or all being on and working. What is your opinion in such a scenario?

Thanks for the contribution! I'd suggest to set the restart policy to on-failure for long running process. only the short lifecycle processes like task-check-alerts have always policy

Works for me. My motivation to change the default compose file was: When I (accidently) restarted the machine, I noticed hyperdx no longer working. When I ran docker ps there were only a couple of hyperdx processes started, others were not. Then I started digging through logs of those exited containers to understand what was wrong, but did not find anything. I think this is a bit of inconsistent behaviour, I would expect all being off, or all being on and working. What is your opinion in such a scenario?

Great question. The on-failure should cover the normal exit case (like daemon restart). The reason to stop with non-zero exit code is because the service might be in a bad state that we want to diagnose the issue first. I think its safe to set

db related services to on-failure (db, ch-server, redis)
app + ingestion services to unless-stopped (api, app, miner, go-parser, otel-collector, aggregator, ingestor)
non essential or cron task services to always (task-check-alerts, hostmetrics)

On hold. Currently lack of resources to pursue this.


Spectator View (Preview) README needs updating with new instructions

The instructions direct you to the discontinued ARKit Plugin on the Asset Store for one.
Plus, I feel like we really should bundle the required DLLs with MRTK? It's like that for pretty much everything else in MRTK after all. It's a lot of extra steps for DLLs that are going to usually look identical from one machine to the next.
EDIT: Also the Unity ARKit plugin is provided with the Preview release package

I think there was some question about the legality of redistribution of such dlls

Another suggestion: Recommend a known-to-be-working vcpkg commit. 02839ae4a83d505bae17d5856ce851af29c89c65 seemed to work for me (at least it resolved the issue with tiff I was having)

Seems the main objection to including them was here: https://github.com/Microsoft/MixedRealityToolkit-Unity/pull/1884#discussion_r177601177
@davidkline-ms why...? Policy or not it makes using the plugin considerably more difficult, especially as  using specific versions through vcpkg is pretty damn unintuitive and using master seems to be prone to breaking and/or version updates that mean you have to roll back to a commit or modify the included library files due to OpenCV having versioned DLLs and Libs.

it resolved the issue with tiff I was having

@BenWoodford What issue were you having with tiff? Have you opened a bug on vcpkg?
@StephenHodgson if you wanted to add instructions to recommend 56ea0dca4098754cee6dba7766c9e38d1fe40165, you could do
>git clone -n https://github.com/microsoft/vcpkg
>cd vcpkg
>git checkout 56ea0dca4098754cee6dba7766c9e38d1fe40165

I have updated the issue in question here: https://github.com/Microsoft/vcpkg/issues/4745
Tiff doesn't build on the lastest master.

@chrisfromwork  is this part of the doc update you are doing for spectator view?

So, i have updated the clone instructions to also try master branch if the last known good commit fails. I agree with Ben that we should figure out a better mechanism for distributing the plugin dll's so that developers don't have to obtain opencv/compile native code. Right now its seeming like we may be able to use Nuget packages for this. But it is still TBD.

I'm closing this issue for now based on a lack of activity. If you are still blocked by not being able to obtain spectator view native plugins, please reopen this issue.


refactor: Replace once_cell to std lib

Like with #6514, I think we need to wait to merge this until Rust 1.71 comes out, but at that point I'm interested in merging this too.

This ended up largely happening in the interim years and there's only one dependency on once_cell today for Wasmtime which can't be replaced at this time, so I'm going to close this.


error while creating config via Init command in Windows

Error thrown :-
/d:\codeformer\space-cloud\space-cloud.yaml: The filename, directory name, or volume label syntax is incorrect.

We need to try this on windows @YourTechBud

@YourTechBud This is not a bug. It's working fine for me.
@atharva29 You have a typo on your file address. If you're using the whole drive address, it should start with the drive(volume) letter not with a /. So it should be:
d:\codeformer\space-cloud\space-cloud.yaml
assuming your config file is located there.

Ah! Cool. So it works! Closing the issue now! @akshayjj1997 can you try this once to confirm?

@Inconnu08 the bug exists. The default project name seems to be incorrect! So the init command works only if you change the project name! You think you can fix this?


Let the y axis of flot chart extend to negative numbers.

I'm using riemann to check if some metrics are roughly equal. For this task i'm using riemann-dash inspect the difference between these metrics, which can of course be negative.
The flot chart will not display negative values, it stops at 0.
This pull request makes some small changes to the flot chart so the chart resizes its axis so negative numbers are displayed.
Setting 'min' to 0 gives the old behaviour.

Wonderful idea! Thank you, Chris. :)