docker hub nginx image error: accept4() failed (24: Too many open files)

Please run microk8s.inspect and attach the generated tarball to this issue.
inspection-report-20181124_142634.tar.gz
We appreciate your feedback. Thank you for using microk8s.
I run docker hub nginx image on NodePort and bump into accept4() failed (24: Too many open files) error. I run the same image on my host and I can curl it without any issue at all. One thing I notice is that the ulimits are different between the container in the cluster and that on my host:
Host:
$ docker run -d -p 8091:80 nginx
0086605b6f152639048a5f8d8f6537609d0e00e225ab60a26389de93373377cb
khteh@khteh-T580:/usr/src/kubernetes/cluster1 2184 $ curl localhost:8091
<!DOCTYPE html>

<title>Welcome to nginx!</title>

Welcome to nginx!

If you see this page, the nginx web server is successfully installed and
working. Further configuration is required.

For online documentation and support please refer to
nginx.org.

Commercial support is available at
nginx.com.

Thank you for using nginx.

microk8s cluster:
$ k get all
NAME                                          READY   STATUS    RESTARTS   AGE
pod/blazegraph-0                              1/1     Running   0          16h
pod/default-http-backend-587b7d64b5-c4rzj     1/1     Running   0          17h
pod/mysql-0                                   1/1     Running   0          16h
pod/nginx-558c57b586-2mdhd                    1/1     Running   0          7s
pod/nginx-ingress-microk8s-controller-b9xcd   1/1     Running   0          17h
pod/web-0                                     1/1     Running   0          16h

NAME                           TYPE        CLUSTER-IP       EXTERNAL-IP   PORT(S)             AGE
service/default-http-backend   ClusterIP   <IP_ADDRESS>    <none>        80/TCP              17h
service/kubernetes             ClusterIP   <IP_ADDRESS>     <none>        443/TCP             19h
service/svc-db                 ClusterIP   None             <none>        3306/TCP,9999/TCP   16h
service/svc-frontend           NodePort    <IP_ADDRESS>   <none>        8090:31034/TCP      7s
service/svc-web                ClusterIP   None             <none>        80/TCP              16h

NAME                                               DESIRED   CURRENT   READY   UP-TO-DATE   AVAILABLE   NODE SELECTOR   AGE
daemonset.apps/nginx-ingress-microk8s-controller   1         1         1       1            1           <none>          17h

NAME                                   DESIRED   CURRENT   UP-TO-DATE   AVAILABLE   AGE
deployment.apps/default-http-backend   1         1         1            1           17h
deployment.apps/nginx                  1         1         1            1           7s

NAME                                              DESIRED   CURRENT   READY   AGE
replicaset.apps/default-http-backend-587b7d64b5   1         1         1       17h
replicaset.apps/nginx-558c57b586                  1         1         1       7s

NAME                          DESIRED   CURRENT   AGE
statefulset.apps/blazegraph   1         1         16h
statefulset.apps/mysql        1         1         16h
statefulset.apps/web          1         1         16h

NAME                                              REFERENCE                TARGETS         MINPODS   MAXPODS   REPLICAS   AGE
horizontalpodautoscaler.autoscaling/istio-pilot   Deployment/istio-pilot   <unknown>/55%   1         1         0          17h
khteh@khteh-T580:/usr/src/kubernetes/cluster1 2188 $ k exec -it nginx-558c57b586-2mdhd /bin/bash
root@nginx-558c57b586-2mdhd:/# cd /var/log/nginx
root@nginx-558c57b586-2mdhd:/var/log/nginx# tail -f app.error_log
2018/11/24 05:50:47 [crit] 9#9: accept4() failed (24: Too many open files)
root@nginx-558c57b586-2mdhd:/etc/nginx# ulimit -Hn
4096
root@nginx-558c57b586-2mdhd:/etc/nginx# ulimit -Sn
4096
root@nginx-558c57b586-2mdhd:/etc/nginx#

nginx-config.yml:
apiVersion: v1
kind: ConfigMap
metadata:
  name: nginx-config
data:
  nginx.config: |
    user nginx;
    worker_processes  3;
    # set open fd limit to 30000
    #worker_rlimit_nofile 10000;
    error_log  /var/log/nginx/error.log;
    events {
      worker_connections  10240;
    }
    http {
      log_format  main
              'remote_addr:$remote_addr\t'
              'time_local:$time_local\t'
              'method:$request_method\t'
              'uri:$request_uri\t'
              'host:$host\t'
              'status:$status\t'
              'bytes_sent:$body_bytes_sent\t'
              'referer:$http_referer\t'
              'useragent:$http_user_agent\t'
              'forwardedfor:$http_x_forwarded_for\t'
              'request_time:$request_time';
      access_log        /var/log/nginx/access.log main;
      server {
          listen       8090;
          server_name  _;
          location / {
              root   html;
              index  index.html index.htm;
          }
      }
      include /etc/nginx/virtualhost/virtualhost.conf;
    }
  virtualhost.config : |
    upstream svc-web {
      server localhost:8090;
      keepalive 1024;
    }
    server {
      listen 8090 default_server;
      root /usr/local/app;
      access_log /var/log/nginx/app.access_log main;
      error_log /var/log/nginx/app.error_log;
      location / {
        proxy_pass http://svc-web/;
        proxy_http_version 1.1;
     }
  }

I made the following change in nginx-config.yml and it seems to fix the error but I am not able to explain why / how:
worker_processes  auto;

maybe you cpu big than 3,when you change to auto,more worker_processes worked ,and then you can open more files.


Job 6 "/dunno" link is broken

Link	/dunno
Location	Lower left author image
Expected result	Anonymous quote?
Routing Strategy	Callback routing
Required	Routing rule which returns a suitable image instead of a web page.
How to Fix It	Surprise! have a callback function that sets a suitable mime-type header and then returns a random SFW image that you have added to the data folder.
Hint	Strictly inside the routing configuration, something like ...
http://comp4711.jlparry.com/show/tutorial/ci-normal01

Why does this "work", even though contrary to what I said in class? Ask me if you can't figure it out!
And, yes, the data folder is outside of public, at the same folder level as application and public!

issued fixed with pull request #12


Show video codec in more human-readable format

We show av01.0.05M.08, and methinks we should add a (AV1) suffix to that.
Same for H.264 etc. They have many different names - let's help our users.

Closed by https://github.com/rerun-io/rerun/pull/7633


TypeScript error in 4.0.0

Hi, since the event name changed from on:change to on:inview_change, TypeScript doesn't seem to recognize it any more and displays an error:
Argument of type '{ class: string; "on:inview_change": ({ detail }: any) => void; }' is not assignable to parameter of type 'Omit<Omit<HTMLAttributes<HTMLDivElement>, "data-sveltekit-keepfocus" | "data-sveltekit-noscroll" | "data-sveltekit-preload-code" | "data-sveltekit-preload-data" | ... 49 more ... | "aria-valuetext"> & EventsWithColon<...>, keyof HTMLAttributes<...>> & HTMLAttributes<...>'.
  Object literal may only specify known properties, and '"on:inview_change"' does not exist in type 'Omit<Omit<HTMLAttributes<HTMLDivElement>, "data-sveltekit-keepfocus" | "data-sveltekit-noscroll" | "data-sveltekit-preload-code" | "data-sveltekit-preload-data" | ... 49 more ... | "aria-valuetext"> & EventsWithColon<...>, keyof HTMLAttributes<...>> & HTMLAttributes<...>'.ts(2345)
No quick fixes available

Is there some way to work around this? I'm using TypeScript 5.0.

Was a bit quick to jump the gun, restarting the Svelte language server in VS Code solved my problem. Thanks for a great package!


Implement optional settings analytics

Add an option for the user to anonymously send their settings to the server, so that analytics about what features / options are most used can be acquired. This will help determine which features should be suggested to new users (#1583) and which features should be prioritized when working on bugs / enhancements.

Will be used for #1503.


How to display date below message?

Hi,
The screenshots show the date can be displayed under that actual message text. I couldn't figure out how? By default its being shown beside the message which wastes a lot of space.

Hi,
The screenshots show the date can be displayed under that actual message text. I couldn't figure out how? By default its being shown beside the message which wastes a lot of space.

You solved that problem?I have same situation... Please, share the answer


DeviceEventEmitter.removeListener issue - undefined is not a function - React Native 0.70+

Hi! 👋
Firstly, thanks for your work on this project! 🙂
Today I used patch-package to patch<EMAIL_ADDRESS>for the project I'm working on.

Here is the diff that solved my problem:
diff --git a/node_modules/react-native-modal-datetime-picker/src/Modal.js b/node_modules/react-native-modal-datetime-picker/src/Modal.js
index 6e5f95d..1466d08 100644
--- a/node_modules/react-native-modal-datetime-picker/src/Modal.js
+++ b/node_modules/react-native-modal-datetime-picker/src/Modal.js
@@ -48,10 +48,12 @@ export class Modal extends Component {
   }

   componentWillUnmount() {
-    DeviceEventEmitter.removeListener(
-      "didUpdateDimensions",
-      this.handleDimensionsUpdate
-    );
+    try {
+      DeviceEventEmitter.removeListener(
+        "didUpdateDimensions",
+        this.handleDimensionsUpdate
+      );
+    } catch (e) {}
     this._isMounted = false;
   }

This issue body was partially generated by patch-package.

Looks like this is happening because DeviceEventEmitter.removeListener has been removed after being deprecated for a while.

@mmazzarolo Hi any updates on this being landed as it is currently crashing devices. Thanks

Hey I'm currently on parental leave and can't work directly on the codebase. If you're willing to update #669 adding support for the updated Dimensions API I can give it a review and try it out.

@mmazzarolo Hey sorry for the delay was busy on new native version for a bit. I've submitted the pull request now so should all be good. Thanks

@fansilva1991 this will be fixed if you update the library, there is no need for this patch, the deprecated API DeviceEventEmitter.removeListener has been already replaced with listener.remove()
@mmazzarolo I don't think we need to do any change here

@khzouroussama not sure I'm following, who replaced DeviceEventEmitter.removeListener with listener.remove()? 👀 It's still here in this lib.

I believe it was changed here https://github.com/mmazzarolo/react-native-modal-datetime-picker/pull/591
I was facing the same issue before because my app was on an older version, so it is not reproducible on the recent versions

Ah, right, I thought this issue was about something different. Good call, thanks!

@mmazzarolo sounds fixed. Want me to close the pull request???

@mmazzarolo sounds fixed. Want me to close the pull request???

Yes please. i still get the some error on ios devices because of this.


getting omni and tomni in regtest mode

Hi there.
I run omnicored in regtest mode.
Here is my ~/.bitcoin/bitcoin.conf file:
server=1
txindex=1
regtest=1
rpcuser=user
rpcpassword=password

I have generated blocks by executing omnicore-cli generate 101.
Then I sent 1 BTC to address moneyqMan7uh8FqdCA2BV5yZ8qVrc9ikLP to get OMNI and TOMNI.
Then I generated 20 more blocks.
I can see result transaction:
{
  "amount": -1.00000000,
  "fee": -0.00003840,
  "confirmations": 20,
  "blockhash": "11973edbd2140de38fb878df939a046b3b5ae480ea176e686c6f7fd481bc93c0",
  "blockindex": 1,
  "blocktime":<PHONE_NUMBER>,
  "txid": "710af8d294685acbe585fa0544e804076241fd157c65b22357644e59d84cb0b0",
  "walletconflicts": [
  ],
  "time":<PHONE_NUMBER>,
  "timereceived":<PHONE_NUMBER>,
  "bip125-replaceable": "no",
  "details": [
    {
      "account": "",
      "address": "moneyqMan7uh8FqdCA2BV5yZ8qVrc9ikLP",
      "category": "send",
      "amount": -1.00000000,
      "vout": 1,
      "fee": -0.00003840,
      "abandoned": false
    }
  ],
  "hex": "010000000126a2fedc8556be21466e91bac2e5ebf6f3bb8cf1a36e552d314a7f25e7fa701d00000000484730440220390dce7a652fb1b497423ee6dc25d37871647a0f8b403deaf6f7d076e3775c1d02207a98ad6e7486fe4f2fa6539e9a16420b2fb0584dbc59698e1558949c07ad02e301feffffff0200021024010000001976a91435c03e1a73c9338a4a34a0d3ceec0bfecce4598188ac00e1f505000000001976a9145ab93563a289b74c355a9b9258b86f12bb84affb88ac65000000"
}

But there's still no OMNI on address:
omni@281fe3c3ded2:~/omnicore$ omnicore-cli getaccountaddress ""
myrUvkXzJdeNZW9fiN2VQzyHdeavXneV8P
omni@281fe3c3ded2:~/omnicore$ omnicore-cli omni_getbalance myrUvkXzJdeNZW9fiN2VQzyHdeavXneV8P 1
{
  "balance": "0.00000000",
  "reserved": "0.00000000",
  "frozen": "0.00000000"
}

What am I doing wrong?

Hi @lo00l :
Try to check the omni_balance of all your address of the account that send bitcoins to "moneyqMan7uh8FqdCA2BV5yZ8qVrc9ikLP" . May be you should create a separate account (not the default). Check the lines  in this test as a example:   https://github.com/OmniLayer/omnicore/blob/master/src/omnicore/test/test_metadexphase3.sh/#L7-L24

Hi @santos177, thank you for reply. I checked all addresses of my account. I still haven't managed to get OMNI and TOMNI.
But I solved my problem by issuing new tokens and using them for tests.

Hey guys,
it's important that the coins used to generate Test Omni are not freshly mined, but sent to an intermediate address. This is because only P2PKH and P2SH inputs are allowed on the Omni Layer, while coinbase coins are not.

FYI, you can't get OMNI or any other token in regtest=1
So you should use testnet or issue your own token (property) with the same parameters as omni has.


fix: add tests to check map_param bug when LANG contains !

Coverage: 90.605% (+0.06%) from 90.54% when pulling 06127fc1ae728be6fa5cafed642602e34ffbbbe9 on fix/map_param_bug into 469edd8b52ee4d5f9a42523be3944e8809af8a67 on master.


DOC: clean up top-level files: THANKS, COPYING, CHANGES

Remove some outdated files that we no longer need.
Move COPYING to LICENSE, is more commonly used and more clear imho.

+1
I don't think any of the info in CHANGES.txt prior to 0.3.0 is in the release notes, though.  Do you think it is worth keeping it around in pre-0.3.0.rst or something like that in the doc/release folder?

Not sure that's still interesting - any release pre 0.2.2 is so old that no one will care.  Not a strong opinion though, I can add it if you prefer.

sounds reasonable.  merging as-is.  thanks


Convert Elasticsearch docs into runbook format.

I dropped the "upgrading" doc as it is not really suitable for a operator user. We can add one when we actually have a version of this operator to upgrade to (which should not be long, since the current one does not work with kudo 0.10.x, I'll file a bug shortly).
This is the first step towards https://github.com/kudobuilder/kudo.dev/issues/153
Once merged, we'll need a PR in the site repo to include these docs too.

Kind ping @meichstedt @realmbgl for review.

Friendly ping.

Looking for more reviewers since this has been waiting for over a week.
cc @nfnt @ANeumann82 @gerred

Thanks @ANeumann82 !


Add json mapper builder

Coverage decreased (-10.5%) to 89.16% when pulling 1916fdc19239aad2a78003f3f8725437782d78c8 on Add-JsonMapperBuilder into c139b338320c2a4d2bb2eca6e9555e12c5e6b430 on develop.


[entropy_src] Improve documentation

This PR contains a series of commits to correct and clarify the documentation of the ENTROPY_SRC hardware block.

thank you!

I am merging this. CI passed before except for the in-depth lint which now passes.


Add !uncall and !callplayers

Add two new queue-related commands:
1.) Add !uncall, which allows a player to remove their call from the list.
2.) Add !callplayers, which lists all of the players who have used !call.

Changes made. Thanks!


Add a flag for controlling the max stack allocation limit allowed on CPU backend

The flag is meant for quick triaging. The default value is expected to
be 32KB.
Issue #9469

clang-format change has been rolled back. Sorry for the trouble

Can someone tickle the bot ? Trying to get this change in to test e2e dynamic models today.

This change itself wont help. I'd suggest using #9478


(fix)delete StackId in request for take repo/refs [EE-6150]

closes #10235
closes #10340
Changes:
Delete StackId Value in 'gitops/repo/refs'

Rejecting this in favor of a more comprehensive fix.


Multi Layout File with different length of records

Hi,
First, thank a lot for this package and the huge work, it's really helpful!
I have a multi layout file and the total length of a record is not always the same.
I tried to simplified my example, the transformation rules are like:
      "transf1": [
                {
                          "type": "ch",
                          "bytes": 2,
                          "offset": 0,
                          "dplaces": 0,
                          "name": "FIELD_1",
                          "part-key": false,
                          "sort-key": false
                },
                .....
                {
                          "type": "ch",
                          "bytes": 10,
                          "offset": 500,
                          "dplaces": 0,
                          "name": "FIELD_2",
                          "part-key": false,
                          "sort-key": false
                }],
      "transf2": [
                {
                          "type": "ch",
                          "bytes": 2,
                          "offset": 0,
                          "dplaces": 0,
                          "name": "FIELD_1",
                          "part-key": false,
                          "sort-key": false
                },
                .....
                {
                          "type": "ch",
                          "bytes": 50,
                          "offset": 500,
                          "dplaces": 0,
                          "name": "FIELD_2",
                          "part-key": false,
                          "sort-key": false
                }]

*-> The bytes for the last field can be 10 or 50
-> The total bytes for one record can be 510 or 550.
-> I can see if I have to use transf1 or transf2 by looking at a specific offset so:
      "transf_rule": [
	                      {
                          "offset": 138,
                          "size": 3,
                          "hex": "404040",
                          "transf": "transf1"
                },
                {
                          "offset": 138,
                          "size": 3,
                          "hex": "E2C4F0",
                          "transf": "transf2"
                }

-> But the issue is that when a record is read, the input_recl parameter used to determine the length is fix and the transf to used is determined after.
-> Do I miss something?

Hello pargass31!
Thank you for your message.
If your multi layout file is also "multi record length" file you've got a Variable Length Blocked (VB) file (https://www.ibm.com/docs/es/epfz/5.3?topic=formats-variable-length-records).
I see 2 alternatives:

FTP your file keeping the RDW (first 4 bytes) (https://bit.listserv.ibm-main.narkive.com/a5Jp3eHb/rdw-with-ftp) and parse it as a VB file (https://github.com/aws-samples/mainframe-data-utilities/blob/main/docs/04-local-single-vb.md)

Convert your file from VB to FB on the mainframe before transferring it out and use the same feature as you are using now.

Please, let me know if that works for you.
thx!

Thanks for you answer.
I extract the file using retrbinary method (sess.retrbinary('RETR %s' % mainframe_file, open(local_file_ebcdic_path, 'wb').write)).
It appears that the RDW is not extracting with this method, because if I specified "input_recfm": "vb" in the json because in the
read function, l = getRDW(input.read(4)) return something not corresponding to the length of the record (but the length of the full file I think).
Do you know how to specify that I want a the RDW in the sess.retrbinary?

I suppose you are using ftplib. Right?
A binary transfer is not enough. You need to tell the server that you want the RDW.
I've never used ftplib by I'd try something like this before retrbinary:
sess.sendcmd('quote site rdw')

yes I'm ussing ftplib, this command is not working but I'm checking from another one

To check if your file contains the RDW investigate using hexadecimal:
head -c 10 your/ebcdic/file | xxd

i'm using Hxd to see the file, I should see the RDW if extracted?

Yes, the RDW must figure in the beginning of each record. I recommend you manually FTP the file first to isolate the issue and understand the data.

You mean with filezila or equivalent? I'm not able to see any file with a simple filezila connection

In fact I just need to sess.sendcmd("site rdw") and I have the rdw extracted.
Thanks!

It's working fine, thanks a lot for the help.

it's working fine, thanks a lot again


Video view time

Coverage remained the same at 100.0% when pulling e37ae5aa821bd61b502d61f03c5a47bd68f84d6d on video_view_time into 9ec0a02dd630f93cca79906eea6e5b78216be9a4 on master.


Remove macro workaround for borrow checker bug

unsafe {&*(r as *const _)} can be used to workaround the borrow checker bug mentioned in the stream, and doesn't require the use of a macro.

Oh, yeah, that's a neat way to do it. Normally I wouldn't use a cast via a *const since it can extend the lifetime to any lifetime, not just to the valid one ('l) which might lead to accidental misuse within the body of protect, but in this case the macro has the same problem, so this seems like a good alternative.
Will wait to merge until the next stream, but :+1:

BTW, see https://github.com/jonhoo/haphazard/issues/10 if you want to take a stab at CI :)


Question: What screws to use with the case?

There's no documentation on which screws to use to fix the protoboard into the case. Could you share which screws you use?

the screw holes are for M2.5 (same as the raspberry pi) since I thought M3 would give clearance issues

Thank you :)


Update config.py

Add EMOTICONS_SOURCE_FILE = 'data/emoticon.json'

Thank you for taking the time to contribute! This is sincerely appreciated. 😇
(Powered by GitMate.io)


How to set a from name?

All the other api's (and the php wrapper according to the mailjet website) allow you to set the FromName param but it doesn't look like the wrapper supports this? is there any way to get that functionality added at all?
If not, and there's no reason you don't support it, I might try adding it in if I get a chance
ta

similarly with html vs text content for the message body

Hi @godspeed20 !
Thank you for using our php wrapper ! :)
Hope this work for you:
$mj = new Mailjet(getenv('MJ_APIKEY_PUBLIC'), getenv('MJ_APIKEY_PRIVATE'));
$params = array(
    "method" => "POST",
    "FromEmail" => "YOUR EMAIL",
    "FromName" => "Mailjet Pilot",
    "Subject" => "Your email flight plan!",
    "Text-part" => "It worked !",
    "Html-part" => "
It worked !!

",
    "Recipients" => array(array("Email" => 'gbadi@mailjet.com'))
);
$result = $mj->send($params);
if ($mj->_response_code == 200){
   echo "success";
   var_dump($result);
} else {
   echo "error - ".$mj->_response_code;
   var_dump($mj->_response);
}

Please let me know !
Best,
Guillaume from Mailjet !

weird, if i do that i get
Class Object ( [ErrorInfo] => [ErrorMessage] => Unknown resource: "send" [StatusCode] => 400 )
If I use sendEmail (as per the homepage of this particular git repo) I get
Warning: strpos() expects parameter 1 to be string, array given in D:\xampp\htdocs\iiuk\Library\Mailjet\php-mailjet-v3-simple.class.php on line 113
and a response 400. Which is odd because that's the file attachment code and I dont have one. At present I can only send a message by calling sendEmail with this
$mailParams = array(
"method" => "POST",
"from" => $fromEmailAddress,
"to" => $emailAddress,
"subject" => $subject,
"text" => $bodyAsText
);
If I set "text" to a html body it just shows up in plain text :/

Please make sure you either clone the updated master branch or download the last release I just published v1.1.0 :)

Perfect, thank you!


[WIP]Add Dockerfile for alpine

Hi there
The Dockerfile has been completed, but the official docker image 500 happened when require "ngx.re", I should wait openresty/docker-openresty/pull/113 merge.
--- 2019-07-12 updated
The official docker image was fixed pcre bug in <IP_ADDRESS>-3 , and I was modified the Dockerfile.

@vkill I am a newbie of alpine, is alpine has package management utility like yum? I want to install the package directly instead of compile in docker, what's do you think about that?

is alpine has package management utility like yum?

It's apk, you can search package name then apk add package-name

@moonming I was completed test. please review it. thx.

@vkill I think we should EXPOSE the ports of APISIX: 9080, 9443

Yes, you are right, it's added.

merged, thx


Fix lint errors

Fix issues found after running npm run lint.

Thanks! Gotta run that with webpack somehow.


[Bug] Website incorrectly states that V11 is in Alpha

Title line template: [Title]: Brief description
Navigating to https://v10.carbondesignsystem.com/whats-happening/v11-release/#release-faqs incorrectly states that Carbon v11 release is currently in alpha release.  See screenshot below.

@aagonzales should we just remove this v11 release page from the v10 site? It's out of date and superseded by the actual v11/current website

Yeah I think its safe to just remove it from the v10 site. We need to then update a link on the v10 "Releases" page to redirect to the v11 site.


fix(travis): upgrade seachest version

upgrade seachest version to Release-19.06.02. This fixes issue of seachest crashing with NVMe devices, in which write cache is supported.
Signed-off-by: Akhil Mohan<EMAIL_ADDRESS>

Codecov Report

Merging #287 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #287   +/-   ##
=======================================
  Coverage   52.15%   52.15%
=======================================
  Files          51       51
  Lines        2385     2385
=======================================
  Hits         1244     1244
  Misses       1042     1042
  Partials       99       99

Continue to review full report at Codecov.

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


DBZ-3692 Oracle incremental snapshot support

https://issues.redhat.com/browse/DBZ-3692

@Naros Generally LGTM. I intend to merge https://github.com/debezium/debezium/pull/2430 first so it is possible few small chnages will be required.
Related to closing connection - I don't think it should be close after each chunk. IMHO the better option would be to add anothe callback method that will be invoked after the snapshot will be completed and that one will be responisble for closing the connection.

Thanks for the feedback @jpechane, I went ahead and added a separate callback that I now use in several spots depending on the context state to trigger the closing of the connection.  Let me know if you'd like to see that changed in any particular way.

I've merged https://github.com/debezium/debezium/pull/2501 so this needs a rebase

MySQL support for read-only incremental snapshotting has been merged. The PR modified/refactored the code to support multiple implementations of incremental snapshotting per connector. When rebased the PR will be safe to apply.

@jpechane Rebased

@Naros And applied! Could you please send a docs PR please? It is probably only adding a docs fragment


New command to create a new configuration from a template

We can add a new command to the tool, to create a configuration file from a template, to reduce the time necessary to create a new configuration.
e.g.: pipconf new <config-name>
The template must contain every parameters listten in the oficial pip documentation.
https://pip.pypa.io/en/stable/topics/configuration/#configuration-files

It must be interesting to follow the same that pylint did. The command pylint --generate-toml-config sends to stdout a content of .toml file with all configuration parameters possible in pylint configuration.
[tool.pylint.main]
# Analyse import fallback blocks. This can be used to support both Python 2 and 3
# compatible code, which means that the block might have code that exists only in
# one or another interpreter, leading to false positives when analysed.
# analyse-fallback-blocks =

# Clear in-memory caches upon conclusion of linting. Useful if running pylint in
# a server-like mode.
# clear-cache-post-run =
...

@sheory I think it would be a nice issue to work now. We can keep in contact or talk through a pull request.

Solved by #12.


Added Team Section for home page

I have added the team section for the home page for both large and small screen devices and I have used Swiper.js to make it
Check to view the Component:- https://www.loom.com/share/60b9d00e6c9a4f9d8a3d433dc7ed000c

@RavindraP04 Looks great
Fix the merge conflicts and I'll merge it

@RavindraP04 Looks great Fix the merge conflicts and I'll merge it

@Debajyoti14 Done

@RavindraP04 Fix the deployment errors


Twinkle Star Prefab

This PR introduces prefab for TwinkleStar.
The prefab has twinkling effect of star with randomization, making them appear to pulsate and change size. It includes properties for controlling the amplitude and frequency of the twinkling animation, as well as methods for enabling and disabling the animation.
Also includes script, StarColor to enable real-time editing of the star color directly within the Unity editor. This feature provides a more intuitive and convenient way to control the visual appearance of stars, allowing for quick and seamless color adjustments without requiring additional scripting or play mode transitions.

Resolving the conflict manually.

One last thing. We have a Prefabs folder. Please move you TwinkleStart prefab from the Prefab to Prefabs.
I would, but that would removed you as the author of the file because Git is silly like that and that's no good.


[QA] Equipping different wearables makes them appear incorrectly

Explorer Alpha build versions:
0.32.0
Issue Description:
Equipping different wearables makes them appear incorrectly
STR:

Open 0.32.0
Equip different wearables at once
Open the backpack
Select a wearable
Validate that Equipping different wearables makes them appear incorrectly

Expected behavior:
User should be able to see different types of wearable when selecting them on the backpack.
Current behavior:
Equipping different wearables makes them appear incorrectly
Evidence:
https://github.com/user-attachments/assets/3d3d3944-8f28-4642-8be2-cd879c900b14
Additional Notes:
Windows 11 PRO
16 GB RAM

I can't reproduce it with current version (0.34.0).


"Connection refused" running docker-osx-dev with docker-machine

docker-osx-dev cannot connect to my docker-machine to be able to rsync files.
I'm on OSX 10.11.2, using docker-machine, not boot2docker.
Relevant output when running docker-osx-dev -l DEBUG
2016-04-10 20:38:30 [DEBUG] Starting sync paths: /Users/snipped/foo
2016-04-10 20:38:30 [DEBUG] rsync --archive --log-format 'Syncing %n: %i' --delete --omit-dir-times --inplace --whole-file -l --rsh="ssh -i /Users/snipped/.docker/machine/machines/default/id_rsa -o IdentitiesOnly=yes -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null" /Users/snipped/foo docker@:/Users/snipped 2>&1 | grep -v "^Warning: Permanently added"
2016-04-10 20:38:30 [INFO] ssh: connect to host  port 22: Connection refused
2016-04-10 20:38:30 [INFO] rsync: connection unexpectedly closed (0 bytes received so far) [sender]
2016-04-10 20:38:30 [INFO] rsync error: error in rsync protocol data stream (code 12) at /BuildRoot/Library/Caches/com.apple.xbs/Sources/rsync/rsync-47/rsync/io.c(453) [sender=2.6.9]
2016-04-10 20:38:30 [INFO] Initial sync done
2016-04-10 20:38:30 [INFO] Watching: /Users/snipped/foo

This part of the rsync command looks like the IP address got truncated: docker@:/Users/snipped
My docker-machine inspect doesn't return an IPAddress. Could be related to docker-machine issue 1041

not all drivers expose an IP Address, and to make matters worse, those that do sometimes spell IPAddress differently.

$ docker-machine inspect
{
    "ConfigVersion": 3,
    "Driver": {
        "IPAddress": "",
        "MachineName": "default",
        "SSHUser": "docker",
        ...

This command works, though:
$ docker-machine ip
<IP_ADDRESS>

Maybe tweaking this line of docker-osx-dev to use docker-machine ip will fix the issue:
DOCKER_HOST_IP=$(inspect_docker_machine "{{.Driver.IPAddress}}" || inspect_docker_machine "{{.Driver.Driver.IPAddress}}")

Thanks for reporting. It looks like Docker driers are pretty inconsistent (and not backwards compatible) with what the configuration they expose. You can see we've already had to support two different ways of looking up the IP, and now, it's broken yet again! I'll check out your PR shortly.


add beer-song

[x] follow https://github.com/exercism/docs/blob/master/contributing-to-language-tracks/porting-an-exercise.md
[x] port tests: https://github.com/exercism/problem-specifications/blob/master/exercises/beer-song/canonical-data.json
[x] implement an example

This was exhausting. I went for a deduplicated example twice, before breaking out of the optimisation habit I got into while working through the earlier exercises myself m-)
@jonmcalder Ready to be reviewed :-) We could use a label for that in this repo's settings, BTW ;-)

About the squashing: I haven't used it before and it's not clear to me whether GH will let me edit the commit message. In master, it should contain beer-song, right? Or it it sufficient if the merge commit contains my branch's name add-beer-song?

Funnily enough neither have I actually - but I think there is a confirmation step so you can edit the commit message beforehand if you need to.

There was :-)


fix lm head overriden issue, move it from checkpoint in-loop loading …

…to out loop

What problem caused by inner loop loading? @sywangyi

r_module.lm_head.weight will be wrongly pointed to word_embeddings buffer since it's in the last sharded checkpoint. acturally in llama70b, there's lm_head weight in the checkpoint. so lm head and work embedding should point to different buffers.

This issue will lead to llama2-70B model generate wrong texts in both CPU and Habana Gaudi.

GPT-J would also have problem when checkpoints are locally sharded, and could be fixed by this PR.

@sywangyi Your PR can fix a correctness bug in Llama2-7b when loaded with from_config, can you rebase to master and resolve conflcit?
Hi @RezaYazdaniAminabadi, could this PR be merged into master?  There is a bug in master branch with Llama2 from_config loading and can be fixed by this PR.  We will open an issue soon.


goresym: 1.5 -> 1.8

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/mandiant/goresym/releases.
meta.description for goresym is: "Go symbol recovery tool"
meta.homepage for goresym is: "https://github.com/mandiant/GoReSym"
meta.changelog for goresym is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.7 with grep in /nix/store/2yqjacrxrgs2yj4n396x0w3hphhm58qa-goresym-1.7
found 1.7 in filename of file in /nix/store/2yqjacrxrgs2yj4n396x0w3hphhm58qa-goresym-1.7

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
goresym

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

goresym

Maintainer pings
cc @theHedgehog0 for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/mandiant/goresym/releases.
meta.description for goresym is: "Go symbol recovery tool"
meta.homepage for goresym is: "https://github.com/mandiant/GoReSym"
meta.changelog for goresym is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.7.1 with grep in /nix/store/4pkncmq9b9j2qx5d8ajhdp6z0r4shw3a-goresym-1.7.1
found 1.7.1 in filename of file in /nix/store/4pkncmq9b9j2qx5d8ajhdp6z0r4shw3a-goresym-1.7.1

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
goresym

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

goresym

Maintainer pings
cc @theHedgehog0 for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/mandiant/goresym/releases.
meta.description for goresym is: "Go symbol recovery tool"
meta.homepage for goresym is: "https://github.com/mandiant/GoReSym"
meta.changelog for goresym is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.8 with grep in /nix/store/wd1mac2q2xpihj787z6n6m754naw8h77-goresym-1.8
found 1.8 in filename of file in /nix/store/wd1mac2q2xpihj787z6n6m754naw8h77-goresym-1.8

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
goresym

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

goresym

Maintainer pings
cc @theHedgehog0 for testing.


Unable to associate IPv6 range with a federation mapping using the API

We're unable to associate an IPv6 range with a federation mapping when posting new associations to the federations API. The subnet fails to validate, which produces the following message (IP range removed):
{
    "alerts": [
        {
            "level": "error",
            "text": "[ xxx/123 ] is not a valid ip address."
        }
    ]
}

I can supply example IPv6 ranges directly to whomever works this issue. Traffic Router can happily consume the range, and I believe that this is due to a previously identified bug in the library being used to validate addresses. I'm opening this to ensure we get a fix regardless of whether it's in the library or if we need to use another validation method to move forward.

This is a dup of #818; closing.


LAMBDA_RUNTIME Failed to get next invocation. No Response from endpoint. Issue only in pnpm sst dev

Original Issue.
https://github.com/aws/aws-lambda-python-runtime-interface-client/issues/142#issue-2295029202
Then I learned that it only happens in pnpm sst dev.
[ERROR] [1715680816939] LAMBDA_RUNTIME Failed to get next invocation. No Response from endpoint
  Traceback (most recent call last):
    File "<frozen runpy>", line 198, in _run_module_as_main
    File "<frozen runpy>", line 88, in _run_code
    File "/var/task/awslambdaric/__main__.py", line 25, in <module>
      main(sys.argv)
    File "/var/task/awslambdaric/__main__.py", line 21, in main
      bootstrap.run(app_root, handler, lambda_runtime_api_addr)
    File "/var/task/awslambdaric/bootstrap.py", line 493, in run
      event_request = lambda_runtime_client.wait_next_invocation()
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    File "/var/task/awslambdaric/lambda_runtime_client.py", line 94, in wait_next_invocation
      response_body, headers = runtime_client.next()
                           ^^^^^^^^^^^^^^^^^^^^^
RuntimeError: Failed to get next

My Dockerfile:
FROM public.ecr.aws/lambda/python@sha256:3df8dcffd27efbaa191051cd2247e07e4a86aac394c6644e717cda3ca407f212 as build
RUN dnf install -y unzip && \
    curl -Lo "/tmp/chromedriver-linux64.zip" "https://storage.googleapis.com/chrome-for-testing-public/124.0.6367.201/linux64/chromedriver-linux64.zip" && \
    curl -Lo "/tmp/chrome-linux64.zip" "https://storage.googleapis.com/chrome-for-testing-public/124.0.6367.201/linux64/chrome-linux64.zip" && \
    unzip /tmp/chromedriver-linux64.zip -d /opt/ && \
    unzip /tmp/chrome-linux64.zip -d /opt/

FROM public.ecr.aws/lambda/python@sha256:3df8dcffd27efbaa191051cd2247e07e4a86aac394c6644e717cda3ca407f212
RUN dnf install -y atk cups-libs gtk3 libXcomposite alsa-lib \
    libXcursor libXdamage libXext libXi libXrandr libXScrnSaver \
    libXtst pango at-spi2-atk libXt xorg-x11-server-Xvfb \
    xorg-x11-xauth dbus-glib dbus-glib-devel nss mesa-libgbm

RUN mkdir -p ~/.aws-lambda-rie && curl -Lo ~/.aws-lambda-rie/aws-lambda-rie https://github.com/aws/aws-lambda-runtime-interface-emulator/releases/latest/download/aws-lambda-rie && chmod +x ~/.aws-lambda-rie/aws-lambda-rie

COPY --from=build /opt/chrome-linux64 /opt/chrome
COPY --from=build /opt/chromedriver-linux64 /opt/

# Move the chromedriver executable to a directory in your PATH
RUN mv /opt/chromedriver /usr/local/bin/
RUN mv /opt/chrome /usr/local/bin

# Make sure the chromedriver executable has executable permissions
RUN chmod +x /usr/local/bin/chromedriver

RUN ls /var/
WORKDIR /var/task

# Copy the current directory contents into the container at /var/task
COPY . /var/task

RUN dnf update -y
RUN dnf install python3 -y
RUN dnf install python3-virtualenv -y

# Create a virtual environment in /venv
RUN python3 -m venv /venv

# Set the virtual environment as the active environment
ENV PATH="/venv/bin:$PATH"

RUN dnf install pip -y

RUN pip install -r requirements.txt --target /var/task
RUN pip install awslambdaric --target /var/task

RUN chmod 755 /usr/local/bin/aws-lambda-rie
RUN chmod 755 /var/task/entry_point.py
RUN chmod 755 /var/task/entry_script.sh
RUN chmod 755 /tmp

ENTRYPOINT ["/var/task/entry_script.sh"]
CMD [ "entry_point.handler" ]

Thing to note. This code worked perfectly two weeks ago.
Not only that, I received the same error on this dockerfile too:
FROM ubuntu:latest

RUN apt update -y
RUN apt install python3 -y
RUN apt install python3-pip -y
RUN mkdir /var/task
RUN pip install awslambdaric --target /var/task

WORKDIR /var/task

# Copy the current directory contents into the container at /var/task
COPY . /var/task

RUN chmod +x /var/task/tester.py
RUN chmod +x /var/task/entry_script.sh

ENTRYPOINT [ "/var/task/entry_script.sh" ]
CMD ["tester.handler"]

Where tester.handler is a function that returns the event from AWS Lambda. The thing is in no case does the lambda ever reach a single line of code in the handlers.
Currently using Ubuntu 24.04.
Has been tested on Ubuntu 22.04 and the same behaviour has been observed (On a different machine).
What seems to be the case is that sst dev is unable to AWS resources with awslambdaric.
I have tested this locally with just docker and it works.
I have tested this in prod/aws resources and it works.
It only does not work when using pnpm sst dev.
Before you say it might be how I setup my routes that's not possible either cause I have had the container work via sst dev before, it seems to be a new development.

I've had this problem on Ubuntu.
I managed to solve this by adding
<IP_ADDRESS> host.docker.internal

to my /etc/hosts

I've had this problem on Ubuntu. I managed to solve this by adding
<IP_ADDRESS> host.docker.internal

to my /etc/hosts

Forgot to reply to you. Thank you very much it indeed worked and that was the correct solution


DateTimeFormatter: Dynamically augment skeleton

Fixes #271
This is rework of #462.
I have implemented another skeleton pattern comparison algorithm, which fixes a lot of issues I have found in #462 implementation.
Also I have added a lot of tests, which were tested manually across Intl.DateTimeFormat and ICU Flex Test.
But not all resulst same from Flex Test and my implementation. This is difference table:

Skeleton
Flex Pattern
Globalize Pattern
Comment

GyMMMEd
EEE, MMM d, y G
E, MMM d, y G
There is explicit available format in cldr-data, which used by globalize

GyMMMEdhms
EEE, MMM d, y G, h:mm:ss a
E, MMM d, y G, h:mm:ss a
This is combined GyMMMEd and hms patterns, which are both available in cldr-data and merged with dateTimeFormat.medium

hhmm
h:mm a
hh:mm a
Globalize expand matched h:mm a to hh:mm a because user requested hh in his pattern

EHmms
EEE HH:mm:ss
E HH:mm:ss
There is explicit entry in cldr-data, which matches EHms to E HH:mm:ss. Because in result pattern mm already present, we don't need to augment any more.

And all other differences are similar. In fact, I am really can not understand, why flexTest augments E to EEE, if you know why please tell me :-)

@Strate, fantastic thanks for your contribution. Please, have you used any external reference (e.g., UTS#35) for implementing your algorithm? If so, could you add internal comments with some references please?

@rxaviers My algorithm is close to http://www.unicode.org/reports/tr35/tr35-dates.html#Matching_Skeletons, but seems with several bugs in it. I do not handle rule "Numeric and text fields are given a larger distance from each other. MMM ≈ MM"
I'll work on it but later (next week I suppose)

Also check/benchmark these ecma-402 polyfil implementations, can we reuse the idea/algo?

https://github.com/ibm-js/ecma402/blob/afe84f2c0cc02ea4722fb54c3e023476cc49f09a/Intl.js#L510
https://github.com/andyearnshaw/Intl.js/blob/master/src/12.datetimeformat.js#L447

Any movement on this? I could really use this fix.

It is stalled. Would you like to take it over? I'd be happy if you could address the above comments. You could create a new pull request on top of this one with additional commits. Just let me know if you have any questions. Thanks


Add TPCDS benchmark test for DuckDB accelerator

Close by: https://github.com/spiceai/spiceai/pull/2853


Add json serialization/deserialization support for c++ generator

Adapted from @hiennguyenle's work in https://github.com/hiennguyenle/finn.
Unlike Hien Nguyen Le's work, this PR does not introduce a JSON type which can be marshalled across language boundaries - in this PR, the JSON serialisation is strictly used to serialise and deserialise from the c++ datatypes.
Because my purpose in creating this PR was primarily to allow human-friendly representation of DJINNI datatypes for e.g. logging, enums are serialized to strings (which correspond to their names in the IDL). In keeping with Hien Ngyuyen Le's work, the deserialisation layer is also maintained - although I'm not as clear on its use case.
This PR does not include the json+extension.hpp required for std::optional support from Hien Nguyen Le's json PR as that would have to be included in the djinni-support-lib.

Awesome! Thanks a lot!!
It will take me some time to look through that but it seems nice on a first glance.
One question I have is, how tight is the coupling to nlohmann/json? I was never a fan of that library since it's slow in every aspect, and the only selling point was a modern interface back then when c++11 was modern.
I think the decision we make now for the serializer lib will stick with us, so I would at least highlight this question since we are going to introduce dependencies.
The other thing is, I wonder if we could get somehow some usage example.
Having test in expected generated text is good and needed.
But we need also to see how that actually works and see how to make functionality available for users. A hello world, because if there is no quick start that shows how to use a feature, it's like not having it

One question I have is, how tight is the coupling to nlohmann/json? I was never a fan of that library since it's slow in every aspect, and the only selling point was a modern interface back then when c++11 was modern.

Ahhhh, yes, well, the coupling is pretty tight. hiennguyenle was using nlohmann's adl_serializer in the nlohmann namespace for the record, so I did the same with the enum - and to be honest, we're using nlohmann json in our project, so this would work out pretty well for us. I'm not sure how best to reduce this coupling while still retaining the benefits of "free" json serialization/deserialization (for consumers who are happy with nlohmann json).
We could pull the serialization/deserialization code out of the nlohmann namespace and put it in the djinni type's namespace, but we're still a bit constrained by the
If we used the more flexible approach of putting the to_json/``from_json` methods in the type's namespace, we could leave the actual json implementation templated - but we'd still be pretty much constrained by how nlohmann json is doing things.

I think the decision we make now for the serializer lib will stick with us, so I would at least highlight this question since we are going to introduce dependencies.

Fair enough.

The other thing is, I wonder if we could get somehow some usage example. Having test in expected generated text is good and needed. But we need also to see how that actually works and see how to make functionality available for users. A hello world, because if there is no quick start that shows how to use a feature, it's like not having it

Yes, I agree - of course I tested the generated code in my own toy project - but I wasn't sure where such a project should be included in the djinni-generator repo.

Hi,
My suggestion is to use quotes instead of "<...>" to follow other headers pattern.
Thanks.

My suggestion is to use quotes instead of "<...>" to follow other headers pattern.

Is this with reference to the <nlohmann/json.hpp> include?

I have a little bit of a high workload this week and not sure how much time  I am able to invest into my favorite spare time projects.
But as a question and thought experiment:
Do you think it would be possible to have an implementations like
--cpp-json-serialization  <None/nlohmann>
and if future we might have , for example
--cpp-json-serialization  <None/nlohmann/rapidjson>
and kind of 'hotsap' the json detail implementation at generation time
We do not need an other implementation right now, but just conceptually preparing this kind of flexibility, so we can control that option in future and don't close the door for users that might use already other json frameworks in their project.

My suggestion is to use quotes instead of "<...>" to follow other headers pattern.

Is this with reference to the <nlohmann/json.hpp> include?

I do not think that should change since this is a 3rd party header and should be threaded as a system header

Do you think it would be possible to have an implementations like
--cpp-json-serialization <None/nlohmann>

This makes sense to me, and isn't too onerous to do now. If more json libraries are added, it seems to me that it will be a bit ugly to continue shove them also into CppGenerator.scala as I have done now with the nlohmann implementation, but I'm not prepared or equipped to refactor CppGenerator now.

Wow! It is nice!

Thanks for adding the flexibility option! Making it more pretty once we might have multiple json lib support is a future problem.
After a first glance, I think this looks good.
I need just find a bit more time to play with it, but I think we will pull that in and have json soon support. Thanks a lot!
It would be an awesome addition to have some doc with some small usage example, or any other way that can show people how to have an easy start into using that feature.

@mutagene , a SwedenCpp user group colleague , @bigdavedev , shared that with me https://godbolt.org/z/1hna1G1zM Do you think that could be a log-friendly and reasonable good solution ?

Ahh, nice - yes, that's quite lovely.

Thoughts?

I was thinking a bit the same when I made the last commit. On the other hand, if we don't provide an implementation at all, then projects using date with json serialization enabled simply won't build.
I guess one solution could be to take @paulo-coutinho's solution and make this the default. If someone wanted to override it with the date.h implementation, they could write their own adl_serializer as outlined in the documentation:
#include <nlohmann/json.hpp>
#include <date/date.h>

namespace nlohmann {
    template <>
    struct adl_serializer<std::chrono::system_clock::time_point>
    {
        static void to_json(json &j, const std::chrono::system_clock::time_point& tp) {
            j = date::format("%F %T %Z", tp);
        }

        static void from_json(const json &j, std::chrono::system_clock::time_point& value) {
            if (j.is_null()) {
                auto dur = std::chrono::milliseconds(0);
                value = std::chrono::time_point<std::chrono::system_clock>(dur);
            } else {
                std::istringstream json_time{j.get<std::string>()};
                std::chrono::system_clock::time_point parsed_time{};
                // Time saved in UTC, so no need to extract time zone
                json_time >> date::parse("%F %T", value);
            }
        }
    };
}

although this would have to be included before json+extension.hpp, which is a bit of a pain 💦

I was thinking a bit the same when I made the last commit.

I am super happy we have the same point of view!

On the other hand, if we don't provide an implementation at all, then projects using date with json serialization enabled simply won't build.

That could be fine , to get a compiler error, but maybe there is one more way

I guess one solution could be to take @paulo-coutinho's solution and make this the default. If someone wanted to override it with the date.h implementation, they could write their own adl_serializer as outlined in the documentation:

Since we are in C++ land, we can always decorate something with defines to remove it 😁
#include <nlohmann/json.hpp>

namespace nlohmann {

#ifndef DJINNI_CUSTOM_JSON_DATE
     template <>
     struct adl_serializer<std::chrono::system_clock::time_point>
     {
         /// .. some default implelemtation
     },
#endif

and then this

-- although this would have to be included before json+extension.hpp, which is a bit of a pain 💦

is not a problem anymore
I will add that as review suggestions.
I think with such adoption we should be done. Great discussion leading to awesome improvements and a real powerful library feature.  Thanks a lot !!


Release v3.10.0

v3.10.0

Very slow compared to previous releases, definitely not ready for release any time soon.


Added edge detection

I don't know if I have the right to do this but I edited an extension that's not mine. Although, I did not add my name anywhere in the code. I just wanted this extension to be able to detect Microsoft Edge.

BTW, all rights goes to @LilyMakesThings.

I thought you added a "touching sprite" extension at first

I thought you added a "touching sprite" extension at first

Ah yes, Microsoft's patented Edge Detection™ software


DependencyLink is not serializable

The change of DependencyLink from case class to standard class (from version 1.21.0) made the class not serializable, which causes issues in the dependencies project (zipkin-dependencies-spark) and probably in the query service too.
Exception in thread "main" org.apache.spark.SparkException: Job aborted due to stage failure: Task 1.0 in stage 3.0 (TID 19) had a not serializable result: com.twitter.zipkin.common.DependencyLink

Considering that the DependencyLink is used almost in the final stage of the Spark job:
    val dependencies: Dependencies = aggregates
      .map { case ((parent: String, child: String), callCount: Long) =>
        Dependencies(0, 0, Seq(DependencyLink(parent = parent, child = child, callCount = callCount)))
      }
      .reduce(_ + _) // merge DLs under one Dependencies object, which overrides +

    saveToCassandra(sc, dependencies)

A simple work around could be to define a local version of it as a normal, serializable case class. This should allow the  final .reduce(_ + _) step to succeed and produce a single in-memory object. Then in saveToCassandra method we can re-map it to the thrift-compatible class.

I'll code a work around for now, but I suppose it would make sense to make the DependencyLink a serializable class, as it forms an interface for various uses.

So, the canonical serialization of zipkin objects are json or thrift, not java serialization. We have tests for the serialized for of json (used in query), for example, but that's also largely because case classes in scala are so hard to deal with.
Scrooge thrifts are marked as Serializable.. what would happen if you stored scrooge thrifts instead?
https://github.com/openzipkin/zipkin/blob/master/zipkin-scrooge/src/main/scala/com/twitter/zipkin/conversions/thrift.scala#L156

Thanks for the clarification, I suppose for Spark it makes sense to use the thrift wrappers. I noticed @yurishkuro faced the same problem with other classes and defined his own case classes, which seems more readable. I'll leave the implementation with his approach, but this can be revisited in the future.

I suggest closing this issue.


Make  Dataset

Hi，thanks a lot for your TrackR-CNN code sharing.
But I'm still confused about making my own MOTS dataset，could you please share your code about
dataset making with us ? We'll appreciate it a lot.
            All the best.

Hi,
first to clarify: with dataset making you mean the semi-automatic annotation procedure, or is it more about the file formats?

Hello，thanks for repyling , I meant the semi-automatic annotation procedure, thanks very much for sharing.

Hi,
unfortunately we're currently quite busy with other things (mainly preparing the MOTS workshop). We can still release code for the annotation procedure, but it will take some time till we get to that (it will require some refactoring and adding explanations from our side).
The key to the annotation procedure is Box2Seg/Refinement-net, you can find a (slightly modified version) of the code here: https://github.com/JonathonLuiten/PReMVOS/tree/master/code/refinement_net

Hi, thanks for your replying, but I still have a question on overlapping, which is ,
1. What if a region proposal simultaneously contains a whole object A and a small part of another object B (A and B don't overlap)? Does this mean this small part which belongs to B will be mistakenly labeled to A ?
2. What if A and B overlap ? Does the overlapped part will only be labeled to a single one ID (A or B) instead of 2 ID (A and B) ?
Thanks very much for replying.

Hi,
does your question refer to the semi-automatic annotation procedure or to TrackR-CNN?

Closing this for now, please reopen if needed

@EddieEduardo where you able to do the annotation of images yourself. Hence, getting the required .png format?


/usr/local/IRFinder/bin/util/irfinder: error while loading shared libraries: libboost_iostreams.so.1.71.0: cannot open shared object file: No such file or directory

Dear author,
After I ran the command, there was only the Logs folder in the folder, and "/usr/local/IRFinder/bin/util/irfinder: error while loading shared libraries: libboost_iostreams.so.1.71.0: cannot" was displayed in irfinder.stderr. open shared object file: No such file or directory", what is the reason?

Dear Chao-Kong,
are you using Docker or Singularity to run the code or are you trying to run it locally? If the latest, you need to install some dependencies ( listed in the Dockerfile ).
Cheers,
Claudio


ModuleNotFoundError: No module named 'numba.misc'

Hi,
I was using numba, and an error appeared. It said,
ModuleNotFoundError: No module named 'numba.misc'
Please help!

Hi, I was using numba, and an error appeared. It said, ModuleNotFoundError: No module named 'numba.misc' Please help!

@zachyboy12 thank you for submitting this. This is most unusual and shouldn't be happening. My recommendation would be to re-install Numba, as the error may hint at a failed installation, or something having gone wrong with the existing installation.
If that doesn't work, please say something about your environment, which operating system you are on, what architecture it has, how you installed Python, how you installed Numba and if there was anything you did before the error started to appear!

@esc I tried reinstalling numba, but it did not work.
I am running on virtualenv, operating system on MacOs, (
forgot how I installed python), I tried installing using pip3 and git clone. pip3 gave me a module not found error, git clone gave me the same issue as described above.

@esc I tried reinstalling numba, but it did not work. I am running on virtualenv, operating system on MacOs, ( forgot how I installed python), I tried installing using pip3 and git clone. pip3 gave me a module not found error, git clone gave me the same issue as described above. I did not do anything before the error started to appear.

OK, has it ever worked for you, or is this a fresh installation? Also, what kind of Mac is it? Is it a new one with the M1 architecture? Can you copy and paste the exact pip3 and git clone commands you used?

This is a fresh installation. Big sur, to be excact. Here are the commands:
pip3 install numba
git clone https://github.com/numba/numba.git

This is a fresh installation. Big sur, to be exact. Here are the commands: pip3 install numba git clone https://github.com/numba/numba.git

OK, can you list what kind of processor this machine has? You can find this information by clicking on the Apple logo in the top right hand corner and then selecting "About this Mac".

Here is the processer: 3.3 GHz 6-Core Intel Core i5

3.3 GHz 6-Core Intel Core i5

OK, that is a x86_64 processor. I am not sure why you are seeing this error. This will be quite difficult to debug. Here are the next things to try:
a) Doing a git clone https://github.com/numba/numba.git will only download the source code for Numba but not compile it for you. I would recommend to delete the clone again. You can can do so by issuing a cd command to the directory in which you originally executed the git clone command and then using rm -rv numba to removed the cloned files.
b) After that, run the command pip3 install numba again and list the complete output including the command itself and any error messages received. That should  yield more hints as to what is going wrong in your case.

Here:
Defaulting to user installation because normal site-packages is not writeable
Collecting numba
Downloading numba-0.55.2-cp38-cp38-macosx_10_14_x86_64.whl (2.3 MB)
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 2.3/2.3 MB 20.1 MB/s eta 0:00:00
Requirement already satisfied: llvmlite<0.39,>=0.38.0rc1 in /Users/zachyboy/Library/Python/3.8/lib/python/site-packages (from numba) (0.38.1)
Requirement already satisfied: numpy<1.23,>=1.18 in /Users/zachyboy/Library/Python/3.8/lib/python/site-packages (from numba) (1.21.6)
Requirement already satisfied: setuptools in /Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/site-packages (from numba) (49.2.1)
Installing collected packages: numba
Successfully installed numba-0.55.2
WARNING: You are using pip version 22.0.4; however, version 22.1.1 is available.
You should consider upgrading via the '/Library/Developer/CommandLineTools/usr/bin/python3 -m pip install --upgrade pip' command.

Here: Defaulting to user installation because normal site-packages is not writeable Collecting numba Downloading numba-0.55.2-cp38-cp38-macosx_10_14_x86_64.whl (2.3 MB) ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 2.3/2.3 MB 20.1 MB/s eta 0:00:00 Requirement already satisfied: llvmlite<0.39,>=0.38.0rc1 in /Users/zachyboy/Library/Python/3.8/lib/python/site-packages (from numba) (0.38.1) Requirement already satisfied: numpy<1.23,>=1.18 in /Users/zachyboy/Library/Python/3.8/lib/python/site-packages (from numba) (1.21.6) Requirement already satisfied: setuptools in /Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/site-packages (from numba) (49.2.1) Installing collected packages: numba Successfully installed numba-0.55.2 WARNING: You are using pip version 22.0.4; however, version 22.1.1 is available. You should consider upgrading via the '/Library/Developer/CommandLineTools/usr/bin/python3 -m pip install --upgrade pip' command.

Thank you for posting this. Looks like the installation was fine. Now what happens when you type python and then import numba?

My command: python3 import numba
Result:
/Library/Developer/CommandLineTools/usr/bin/python3: can't open file 'import': [Errno 2] No such file or directory

My command: python3 import numba Result: /Library/Developer/CommandLineTools/usr/bin/python3: can't open file 'import': [Errno 2] No such file or directory

The correct command would be: `python -c "import numba"

Thank you for your correction.
It worked fine. But when I start coding in the file, it doesn't work.

When I start coding in the text editor, it gives a ModueNotFoundError.

OK; what is the name of the file and what is the contents?

The name is init.py
It is in a directory called rplus
The contents:
from random import randint as __randint
from time import time as __time, strftime as __strftime
from secrets import choice as __choice
import numba.numba
@numba.numba.jit()
def randomnumbergen1(path_to_rplus: str, low: int, high: int, more_random_but_slow=False, return_speed=False):
start_time = __time()
try:
open(path_to_rplus + '/numcache.txt')
except:
open(path_to_rplus + '/numcache.txt', 'w').write('1')
# Checking if user wants randomer numbers
if more_random_but_slow is True:
performance = 4000000  # The con of randomer numbers is that it is very slow
elif more_random_but_slow is False:
performance = 1000000
cache = open(f'{path_to_rplus}/numcache.txt').read()  # Information of earlier calculated numbers
rnums = []  # The random numbers to choose from
for i in range(performance):  # Performance depends if more_random_but_slow is True or is False
rnums.append(__randint(low, high))  # Appending semi-random numbers
rnum = __choice(rnums)  # The random number
if str(rnum) == cache:  # Checking if random number is equal to the previous random number
if high - low == 1:
pass
else:
rnum = __choice(rnums)  # Resetting rnum
open(f'{path_to_rplus}/numcache.txt', 'w').write(str(rnum))  # Write rnum to numcache.txt
end_time = __time()
if return_speed is True:
return end_time - start_time, rnum
elif return_speed is False:
return rnum
def randomnumbergen2(path_to_rplus: str, return_speed=False):
start_time = __time()
rnum = int(__strftime('%S')) + int(__strftime('%I'))
end_time = __time()
if return_speed is True:
return end_time - start_time, rnum
elif return_speed is False:
return rnum
def randompassword(path_to_rplus: str, numberofletters=12, return_speed=False):
start_time = __time()
rpl = []
p = ''
for i in range(numberofletters + 1):
lon = __choice(['l', 'n'])
word = __choice(open('/usr/share/dict/words').read().splitlines())
if lon == 'l':
rpl.append(word[0])
else:
rpl.append(randomnumbergen1(path_to_rplus, 0, 9))
for j in rpl:
p += str(j)
end_time = __time()
if return_speed is True:
return end_time - start_time, p
elif return_speed is False:
return p
def randomwordgen1(numberofletters: int, return_speed=False):
start_time = __time()
for word in open('/usr/share/dict/words').read().splitlines():
if len(word) != numberofletters:
continue
else:
break
end_time = __time()
if return_speed is False:
return word
elif return_speed is True:
return end_time - start_time, word
def randomwordgen2(return_speed=False):
start_time = __time()
word = __choice(open('/usr/share/dict/words').read().splitlines())
end_time = __time()
if return_speed is True:
return end_time - start_time, word
elif return_speed is False:
return word

The name is init.py It is in a directory called rplus
The contents: (Bad indent)
from random import randint as __randint from time import time as __time, strftime as __strftime from secrets import choice as __choice import numba.numba

I don't think that import numba.numba is supposed to work. I think you need to do import numba only.

When I tried doing numba only, it raised an AttributeError

Traceback (most recent call last):
File "/Applications/griffin/asdf.py", line 2, in
@numba.jit()
AttributeError: module 'numba' has no attribute 'jit'

Traceback (most recent call last): File "/Applications/griffin/asdf.py", line 2, in @numba.jit() AttributeError: module 'numba' has no attribute 'jit'

This is all very bizzare. At this point I am not convinced this is a bug in Numba but that there are some kind of installation issues in your case. The Numba issue tracker is meant to track issues/bugs that affect Numba. I am going to close this issue now and kindly recommend to open a new thread on https://numba.discourse.group/ -- with as much information about the issues your encounter. Thank you!


Is it possible to build on platform linux/amd64?

I've tried running ./scripts/docker_build_ros.sh --distro galactic --package ros_base and have a lot of issues.
First, local-gen is not found, so I installed locales through apt, but now it can't find /usr/bin/python.
The pytorch base image (nvcr.io/nvidia/pytorch:22.04-py3) seems to have python installed by conda (/opt/conda/bin/python).  The other base image (nvcr.io/nvidia/l4t-base:r32.5.0) has /usr/bin/python3, but still fails on the update-alternatives step:
Step 15/36 : RUN update-alternatives --install /usr/bin/python python /usr/bin/python3 1
 ---> Running in 5f98927dab86
update-alternatives: error: alternative path /usr/bin/python3 doesn't exist

Thanks so much for any help!

Hi @bluecamel, I haven't tested building the containers from this repo on x86 and I wouldn't expect them to work.   What I would do is just use the upstream ROS containers that already exist for x86:  https://hub.docker.com/_/ros/

Got it, thanks!

Just wanted to leave a comment for future reference that it is possible. Not the easiest thing to do but also not horrible but there is a learning curve in understanding Docker. I used the following references to get it setup: https://medium.com/@Smartcow_ai/building-arm64-based-docker-containers-for-nvidia-jetson-devices-on-an-x86-based-host-d72cfa535786 and https://www.stereolabs.com/docs/docker/building-arm-container-on-x86/
I mainly used the medium article for the most part. I've been able to build docker images for jetson nano using my x86 machine. I was a lot more motivated to get this working since I only had access to Jetson Nano which I once built rtabmap and rtabmap_ros from source and took me about 14 hours. Now I can not only build rtabmap but some of its larger dependencies from source in about 2 to 3 hours which is still a lot but a lot more tolerable if it is ever needed.
There's also this interesting git repo https://github.com/defunctzombie/jetson-nano-image-maker that builds jetson nano images using Docker on GitHub with GitHub Actions. So you might be able to reference this repo as a way to build your own docker images on GitHub instead of your local machine.


weird multithreading performance

I did some practical experiment there and got weird results:
https://github.com/borgbackup/borg/issues/8217#issuecomment-2170637689

Hello, I gave a look to your script, and the following caught my eyes:

The data you generate is not only random in content, but also random in size. I feel like the outputs would be more reliable if you generate the data only once, save it into a global variable, and use the very same data in each case.
The data to be compressed is very small: between 0.5 and 4MiB.
The jobSize is very small as well: 512kiB

Have you tried running the same test with bigger data sizes?
Another thing that way be worth investigating is running the zstd command directly to see if the figures you have are inherent to Zstandard or are more specific to the pyzstd library.


缺乏自定义字段作为text

缺乏自定义字段作为text

缺乏自定义字段作为text

For those of you who don't read Chinese:
"Lack of Custom Fields as Text"

"Lack of Custom Fields as Text"
how to settle this question?

Not enough information


Add sbt-github-actions for basic CI

So that we at least don't accidentally break something.

@pjfanning BTW If you have any general questions/issues regarding sbt-github-actions you can report it upstream at https://github.com/sbt/sbt-github-actions, I am actually a maintainer of the project.

Closing this since there isn't willingless to merge it

@jrudolph Incase you change your mind, there was additional context for this PR (see https://github.com/apache/incubator-pekko-site/issues/13#issuecomment-1617676639). Sorry if it wasn't clear from the beginning.

@mdedetrich I don't object to this PR - I think it is useful to run sbt paradox on the PRs. I just have a fe minor concerns about what the sbt plugin generates.
We should probably even be thinking about a link validator job.

Sure, if @jrudolph changes his mind I will re-open the PR but this is low priority for now


Voiceover when expanding selection could be better

Couple issues:

when expanding the ***** selection, it only ever says 1star star star` selected even when more *'s are shown in the text. This seems to be a macos global bug
The formatting foo unselected, foo bar selected is not great, it hides the most important info. This seems to be a chromium thing, repros in edge, but notes handles it better IMO

@JacksonKearl can you reproduce with VoiceOver and textArea here https://developer.mozilla.org/en-US/docs/Web/HTML/Element/textarea
If yes, can you please file a new issue for VoiceOver here https://bugs.webkit.org/
Then I can share the bug with the VoiceOver authors. Thanks!

It works fine in WebKit, seems to be only Chromium/Electron that's effected.

@JacksonKearl if you can reproduce with Chromium then can you file a new issue in https://bugs.chromium.org/p/chromium/issues/list
Or are you saying you can only reproduce with Electron?

Sure, reported here https://bugs.chromium.org/p/chromium/issues/detail?id=1313990 👍

Is #162152 a duplicate of this bug?

@MarcoZehe yes I believe so, good news is chromium fixed this 12 days ago! https://bugs.chromium.org/p/chromium/issues/detail?id=1172956#c7
Bad news is it may be a while before vscode gets that build.

Thanks! Yes, I saw that it is fixed now, and I also discovered that I was the one who originally filed this bug report in February 2021. I couldn't figure out what version of Chromium this will be in, though, but I suppose it is currently in Canary and has to move through beta and release for the next couple of weeks, then to Electron, until it finally hits VSCode once it moves to that version of Electron that has the fix.

@MarcoZehe thanks for opening the original issue on the Chromium side.
@deepak1556 can we back port the chromium fix so it is included as part of our next Electron update?
@JacksonKearl hey there 👋

Fixed upstream issue: https://bugs.chromium.org/p/chromium/issues/detail?id=1172956
I will backport the fix with Electron 22 update.

Reopening because Electron 22 was reverted for the March milestone.


Ally.com

Tried to paste a dollar amount into a bank transfer screen on secure.ally.com. Nothing happened, so I added & saved the addon rule for the site. Still did not work. Tried pasting random text from URL bar to the field on ally, no result. I have added rules maybe 5-6 times in the last month or so and this is the first time it has failed.

If there is any info I can pull like the source html of the page, let me know.

I can confirm your issue in both Firefox and Chrome but unfortunately this Firefox version of the add-on is just following the upstream Chrome implementation. I am just maintaining the add-on so it works in Firefox. Since this isn't a Firefox specific issue I would recommend that you open up an issue here: https://github.com/jswanner/DontFuckWithPaste/issues.


[BUG] Reset icon doesn't appear in Safari

Describe the bug
The icon used for the reset button does not appear when the game is played in the Safari browser.
Reproduction
Steps to reproduce the behavior:

Open Flippin in Safari
Navigate to main screen
See reset icon is missing

Expected behavior
Reset icon should be visible like other icons are.
Device:

OS: iOS
Browser: Safari

Additional context
This is likely due to the SVG used for that icon not having height and width set. I believe this defaults to auto in other browsers but not in Safari.

This should be solved by adding width and height attributes to the SVG.


Confirmation Email issue with Non-digital downloads

I apparently received a non-digital order fifteen days ago. I didn't receive a notification email. I finally noticed the deposit come through in my financials. I found the order in the admin section, then I clicked the Resend button. I got this crash:
  Rendered /home/deploy/.rvm/gems/ruby-1.9.3-p327/bundler/gems/spree_digital-d652e96cb1b8/app/views/spree/order_mai
ler/confirm_email.text.erb (14.8ms)
Completed 500 Internal Server Error in 69ms

ActionView::Template::Error (undefined method `variant_options' for #<#<Class:0x000000077eae08>:0x000000078c8208>):
    6: Order Summary
    7: ============================================================
    8: <% for item in @order.line_items %>
    9: <%= item.variant.sku %> <%=item.variant.product.name%> <%= variant_options(item.variant) %> (<%=item.quantit
y%>) @ <%= number_to_currency item.price %> = <%= number_to_currency(item.price * item.quantity) %>
    10: <% end %>
    11: ============================================================
    12: Subtotal: <%= number_to_currency @order.item_total %>
  /home/deploy/.rvm/gems/ruby-1.9.3-p327/bundler/gems/spree_digital-d652e96cb1b8/app/views/spree/order_mailer/confi
rm_email.text.erb:9:in `block in _567af581a32791e407057adc78f90ab7'
  activerecord (3.2.13) lib/active_record/associations/collection_proxy.rb:89:in `each'
  activerecord (3.2.13) lib/active_record/associations/collection_proxy.rb:89:in `method_missing'
  /home/deploy/.rvm/gems/ruby-1.9.3-p327/bundler/gems/spree_digital-d652e96cb1b8/app/views/spree/order_mailer/confi
rm_email.text.erb:8:in `_567af581a32791e407057adc78f90ab7'

I'm pretty tired from work this evening, so I might not look at this until tomorrow. Now that I think about it, I'm not sure if I've ever tested a non-digital sales with spree digital installed. Is it compatible with non digital sales?

@createthis, can this issue be closed now?


Update documentation with extra information

Jira ticket URL

Just add the ticket number to the end:

https://dfedigital.atlassian.net/browse/TEVA-2165
Changes in this PR:
Updated the documentation with extra information
Screenshots of UI changes:
Before
After
Next steps:

[ ] Terraform deployment required?

[ ] New development configuration to be shared?

Review app deployed to https://teaching-vacancies-review-pr-3128.london.cloudapps.digital

Closing - it was deemed perhaps the extra info doesnt add value.


Restructures a compound query in rule promotion to be more efficient

to: @ryandeivert or @chunyong-lin
cc: @airbnb/streamalert-maintainers
Background
We have a query in the rule promotion lambda that runs several count queries and UNION ALLs them together.
The problem with UNION ALL type queries is that Presto is will perform the queries independently of one another; overlapping partitions will be redundantly scanned reducing the speed of the query and also increasing the amount of data scanned, making it slower and more costly.
Changes
I modified the UNION ALL query to a GROUP BY query.
Testing
🤷♂ CI?

Coverage increased (+0.001%) to 96.836% when pulling 28fb1046db0f660497385bf1f92d883276b06f8a on dw--optimize into 978868f0815629f16793a4f6bc61930b17ee0977 on master.


fix: allBlogs filter

No longer using a custom function to generate sitemap since we can do it with the in build next 13 sitemap support.


Skip checking resources when --wait=false is specified

What steps did you take:
I currently use Kapp as a CI tool to manage lots of YAML files. I used --wait=false when I deleted the app because sometimes deleting custom resources will take a long time.
What happened:
kapp exits with non-zero code which makes CI fail.
$ kapp delete -a app1 --wait=false -y
Target cluster 'https://<IP_ADDRESS>:6443' (nodes: firgavin)

Changes

Namespace  Name        Kind        Age  Op      Op st.  Wait to  Rs  Ri
default    simple-app  Deployment  22s  delete  -       -        ok  -
^          simple-app  Service     22s  delete  -       -        ok  -

Op:      0 create, 2 delete, 0 update, 0 noop, 0 exists
Wait to: 0 reconcile, 0 delete, 2 noop

11:18:59AM: ---- applying 2 changes [0/2 done] ----
11:18:59AM: delete deployment/simple-app (apps/v1) namespace: default
11:18:59AM: delete service/simple-app (v1) namespace: default
11:18:59AM: ---- waiting on 2 changes [0/2 done] ----
11:18:59AM: ok: noop service/simple-app (v1) namespace: default
11:18:59AM: ok: noop deployment/simple-app (apps/v1) namespace: default
11:18:59AM: ---- applying complete [2/2 done] ----
11:18:59AM: ---- waiting complete [2/2 done] ----

kapp: Error: Expected all resources to be gone, but found: endpointslice/simple-app-vp2dw (discovery.k8s.io/v1) namespace: default, pod/simple-app-64c66864f5-g9sb8 (v1) namespace: default, replicaset/simple-app-64c66864f5 (apps/v1) namespace: default

What did you expect:
Kapp could skip checking resources when --wait=false is specified.
Anything else you would like to add:
I did some research and I found that kapp checks the existence of related resources after applying changes. But resources will be deleted eventually. See https://github.com/vmware-tanzu/carvel-kapp/blob/v0.52.0/pkg/kapp/cmd/app/delete.go#L159.
It would be great if kapp could default to skipping checking resources when --wait=false is specified or add a flag to control this logic. And if that makes sense, I'd like to help implement this ;)
Environment:

kapp version (use kapp --version): v0.52.0
OS (e.g. from /etc/os-release): Ubuntu 20.04.4 LTS
Kubernetes version (use kubectl version): v1.23.6+k3s1

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

Yeah, it seems like setting the wait flag to false would currently lead to an error while deleting recorded apps. So definitely it's a bug.

It would be great if kapp could default to skipping checking resources when --wait=false is specified or add a flag to control this logic.

It does makes sense to allow that behaviour, I am just trying to think of any side effects it could have. One obvious thing that could happen is that one or more resources are not deleted but the app itself (metadata configmap) is deleted.
@cppforlife Any thoughts?

And if that makes sense, I'd like to help implement this ;)

That would be great, we will definitely review  it on priority once we finalize the approach :)

Hey @firgavin good to see your here. Looking forward to your PR for this issue.

One obvious thing that could happen is that one or more resources are not deleted but the app itself

This would be a "known risk" I guess?
We might also lose out on some "returnable cases", where kapp would retry in case of a failed delete due to a returnable error.

I did some research and I found that kapp checks the existence of related resources after applying changes. But resources will be deleted eventually.

i think additional flag would be reasonable to disable this check. may be under dangerous?

i think additional flag would be reasonable to disable this check. may be under dangerous?

This approach makes sense to me

Hi @cppforlife, @100mik, @praveenrewar   - Thanks for your insights! Here's my proposal:
We can add a flag --dangerous-disable-checking-app-deletion to enable or disable the check:

The value is set to false by default, which is compatible with the current behavior.
Once the flag is specified, kapp skips this check, and users might need to manually delete related resources.

Before I work on it, I'd like to discuss the interaction between the two flags. When  --dangerous-disable-checking-app-deletion=false, should we make sure that the value of --wait is overwritten to True? If not, users can still hit the same issue. Of course, we can explain the usage in the docs if we think they should be "orthogonal". Any suggestions?

When --dangerous-disable-checking-app-deletion=false, should we make sure that the value of --wait is overwritten to True?

I think that we should keep the working of these 2 flags independent of each other because a user should be able to use --dangerous-disable-checking-app-deletion irrespective of --wait being enabled or disabled.

If not, users can still hit the same issue. Of course, we can explain the usage in the docs if we think they should be "orthogonal". Any suggestions?

Maybe we can add a hint in the error message?


Extract date mgmt to uucore

extract this to uucore, since it can be used in date as well. We could also make an issue for that after this PR of course.
Originally posted by @tertsdiepraam in https://github.com/uutils/coreutils/issues/4193#issuecomment-1332089757

I think there's now 2 issues for this :) https://github.com/uutils/coreutils/issues/4205.
Closing this one.


Last update broke splats

The latest update fixed the issue with computed object keys #633 but it seems to have broken splats
data.push { doc.data()..., id: doc.id }

Now gives me a coffeescript_error

NOTE: I have tested this code using the latest coffeescript and it works fine.

coffeescript_error occurs when there is a parsing error in coffeescript itself. What does coffee -v return for you? it is possible you the version your environment is using does not support splats.

You misunderstood. CoffeeScript isn’t giving me the error - only coffelint is.

To clarify, the error and the error from #633 you are seeing was generated because coffeelint was unable to parse the coffeescript it was trying to lint, that is what coffeescript_error means. coffeelint can't lint something that's not parseable by the version of coffeescript it's using.
This bit of code seems to run fine for me with the latest version of coffeelint. So either there is some weird override happening, or the coffeescript in the library is being ignored or something didn't get updated.
Knowing what version of coffee and coffeelint your environment is seeing would help clarify what's going on. Do you also have a package-lock.json file?

This may be an issue with my editor. It seems to have downgraded coffeelint to 2.0.7 without my permission.


Type checks on complex props

Clear and concise description of the problem
I am in the process of migrating from BootstrapVue to BootstrapVueNext and I've ended up spending a lot of time debugging JS crashes where prop changes from BV to BVN are significant.
A particularly good example of this is the sortBy prop on BTable. This used to be a string but is now an array of objects. It took a fair bit of debugging to track down where the crash was happening, then cross referencing props to determine what should be required.
Suggested solution
Currently there is no migration guide for BTable, which would have helped, but regardless of this I feel it would be better to type check props in the watch methods when we are dealing with none scalar properties. Issuing a warning there if the type of the property is incorrect.
In this particular example the code was attempting to use the Array.find method on a string, an instant failure, which then prevented the document from loading correctly.
This would help enormously when people are migrating code, as a warning would be seen in dev tools, and the code would still run hopefully to completion. It is far better to ignore an incorrect parameter than to crash and prevent further loading.
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

This is too much overhead and us watching props isn't our responsibility.
You'd be better off assisting with writing the migration guide


ArchetypeFieldsetModel to IPublishedContent?

Recently I've been exploring how to use Ditto with Archetype, (for strongly-typed model mapping).  After various experiments, I ended up figuring out how to map an ArchetypeFieldsetModel to IPublishedContent, (and ArchetypeModel to a "content-set" aka IEnumerable<IPublishedContent>).
So far the code is on my "Ditto Labs" repo, backed up with various unit-tests. I've done some testing with a "real" Umbraco website, (but still needs more testing IMHO).
My question is... would you be interested in having this in Archetype core, and exposed via a ToPublishedContent() extension method?

Lee thats sounds amazing and would definitely welcome it in the core.
I haven't looked at the repo yet but I 100% trust your judgement.
#h5yr
Sent from my Verizon Wireless 4G LTE smartphone
-------- Original message --------
From: Lee Kelleher<EMAIL_ADDRESS>Date: 11/01/2015  11:36 AM  (GMT-05:00)
To: imulus/Archetype<EMAIL_ADDRESS>Subject: [Archetype] ArchetypeFieldsetModel to IPublishedContent? (#302)
Recently I've been exploring how to use Ditto with Archetype, (for strongly-typed model mapping).  After various experiments, I ended up figuring out how to map an ArchetypeFieldsetModel to IPublishedContent, (and ArchetypeModel to a "content-set" aka IEnumerable).
So far the code is on my "Ditto Labs" repo, backed up with various unit-tests. I've done some testing with a "real" Umbraco website, (but still needs more testing IMHO).
My question is... would you be interested in having this in Archetype core, and exposed via a ToPublishedContent() extension method?
—
Reply to this email directly or view it on GitHub.

Excellent, I'll get a PR together!
I thought it would make sense to have it part of Archetype than a separate library.

Closing this ticket, opened PR #303


Add Tokenizer::tokenize_with_location() API

add TokenWithLocation to track Token location
implement Tokenizer::tokenize_with_location() -> Result<Vec<TokenWithLocation>, ..>
refactor line/column tracking to alter current position along with peekable.next() (the old approach with duplicate position tracking was error-prone and contained a bug in multiline word parsing, see the test)
eliminate String copying for Tokenizer, use &'a str instead
minor TokenizerError column fixes, now it shows the missing symbol position and not the expected token start location

Pull Request Test Coverage Report for Build 472789656

140 of 147 (95.24%)  changed or added relevant lines in 1 file are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.05%) to 92.028%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/tokenizer.rs
140
147
95.24%

Files with Coverage Reduction
New Missed Lines
%

src/tokenizer.rs
3
91.68%

Totals

Change from base Build 469532238:
-0.05%

Covered Lines:
4883

Relevant Lines:
5306

💛  - Coveralls

Pull Request Test Coverage Report for Build 472789656

140 of 147 (95.24%)  changed or added relevant lines in 1 file are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.05%) to 92.028%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/tokenizer.rs
140
147
95.24%

Files with Coverage Reduction
New Missed Lines
%

src/tokenizer.rs
3
91.68%

Totals

Change from base Build 469532238:
-0.05%

Covered Lines:
4883

Relevant Lines:
5306

💛  - Coveralls

This looks really cool @segoon . Could you fix the conflicts?

Hi @segoon  -- sorry for the delay in review. I am going to help out now with this repo and we are working to clear the backlog. Is this PR still something you would like to work on to help contribute?

I am closing what look like stale PRs in this repo; I apologize in advance if this is a mistake -- please feel free to reopen if you want to keep working on this issue.


Markisen aktor not supported

The "normal" blinds works you can command them up and down but if you configure in free@home a jalousie actor as awning (German: markise) the element is not imported to homebridge nor exposed to homekit.
Note: This Jalousie actor is the only one my my system which is 2.1 Wireless actor and not a REG component. May this is the diffrent to the other onces.

Is this still relevant?

Yes, I would be interested too!

Hi there! It seems that this issue is still present in ver. 2.0.4. I have a 4 channel REG device with the channels configured as awning actuator. Plugin does not add devices from the free@home system configured as awning actuator ("functionID": "63"). If to change the settings from awning actuator ("functionID": "63") to blind actuator ("functionID": "61") for any channel then it becomes available in Homebridge accessories. It looks like that items with "functionID": "63" from the free@home json configuration are ignored.

Thank for the feedback. Can you explain how exactly an Awning actuator differs from a blind? I only have normal blinds, therefore I can not test this with free@home. What added functionality do you have?

My description is wrong the differences is that it is a wireless actor.

Am 13.11.2022 um 21:16 schrieb Henry Spanka @.***>:

Thank for the feedback. Can you explain how exactly an Awning actuator differs from a blind? I only have normal blinds, therefore I can not test this with @.*** What added functionality do you have?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

Wireless should already work, you are talking about the "Awning" mode that is not working, correct? Can you provide a link that describes the awning functionality or include a video of the free@home app / web ui where an awning actuator is controlled?

Hi there!
Sure I can. I will provide you this data today or tomorrow.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Henry Spanka @.>
Sent: Wednesday, November 23, 2022 11:31:17 PM
To: henry-spanka/homebridge-freeathome @.>
Cc: VadimGolovko @.>; Comment @.>
Subject: Re: [henry-spanka/homebridge-freeathome] Markisen aktor not supported (#55)
Wireless should already work, you are talking about the "Awning" mode that is not working, correct? Can you provide a link that describes the awning functionality or include a video of the @.*** app / web ui where an awning actuator is controlled?
—
Reply to this email directly, view it on GitHubhttps://github.com/henry-spanka/homebridge-freeathome/issues/55#issuecomment-1325625223, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A4EKT22EKFS7PDSRKBIKYEDWJZ5JLANCNFSM5FXVERKQ.
You are receiving this because you commented.Message ID: @.***>


Stack Overflow reading huge old HTML

I had a performance problem displaying huge HTML , then I was looking for huge simple HTML in english.
I found http://www.gutenberg.org/files/1661/1661-h/1661-h.htm and I tried and got StackOverflow error.
I knew this HTML was so old therefore it might be cause error on parsing.
I suggest using SGMLReader for parsing any HTML.
I'm using it for many scraping projects and results are so reasonable.

Thank you for the suggestion.
However it takes time to investigate