[Extensions] BUG NPE handling and malformed Extension.yml

… for this scenario.
Signed-off-by: varuntumbe<EMAIL_ADDRESS>
Description
NPE and OpenSearch crash on malformed Extensions.yml
Issues Resolved
https://github.com/opensearch-project/opensearch-sdk-java/issues/583
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Hi @dbwiddis ,
I have done the basic changes required.

open search has stopped crashing
code which used to error out has been moved to try block and throwing parsingException
if one extension does not have required field, parsingException will be thrown and will be catched there and will be logged a warning message.
added a testcase for this scenario

Things I think might need some work on,

Exception message might need some work
I am printing extension name in the exception message ( what if name itself is missing in .yml file ? )
etc.

Thanks and Regards
Varun

Codecov Report

Merging #6758 (4bc17c3) into main (5f81930) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main    #6758      +/-   ##
============================================
- Coverage     70.74%   70.73%   -0.02%
- Complexity    59203    59222      +19
============================================
  Files          4810     4810
  Lines        283487   283495       +8
  Branches      40877    40879       +2
============================================
- Hits         200543   200518      -25
- Misses        66465    66590     +125
+ Partials      16479    16387      -92

Impacted Files
Coverage Δ

...a/org/opensearch/extensions/ExtensionsManager.java
46.99% <100.00%> (+1.54%)
:arrow_up:

... and 485 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Hi @dbwiddis,
I have pushed the new changes.
Thanks and regards
Varun

@varuntumbe the changes are good, do you want to flip the PR to open?

Hey @saratvemulapalli ,
I will flip it today.
Thanks

Code looks good. Please X-off the boxes in the initial comment (and delete the one for change log since this is under experimental feature) to signal to the maintainers that you've done all those things!

Code looks good. Please X-off the boxes in the initial comment (and delete the one for change log since this is under experimental feature) to signal to the maintainers that you've done all those things!

Hi @dbwiddis ,
sorry I did not get what you meant here ( delete the one for change log since this is under experimental feature )

Code looks good. Please X-off the boxes in the initial comment (and delete the one for change log since this is under experimental feature) to signal to the maintainers that you've done all those things!

Ohh understood ! You want me to remove the skip- change log ?

Hi @ryanbogan,
Could you please review this ? ( I will change skip-changelog )
Thanks and regards
Varun.

Ohh understood ! You want me to remove the skip- change log ?

Sorry I missed this question, @varuntumbe .
This change is behind an experimental feature branch. Since the feature has not been released, the change log for it hasn't been either, so it is not necessary for this bug fix to be in the change log.
What I meant was to add skip-changelog tag (which it already is) but remove the last line of check boxes (or use strikethrough font on it with ~).

Ohh understood ! You want me to remove the skip- change log ?

Sorry I missed this question, @varuntumbe .
This change is behind an experimental feature branch. Since the feature has not been released, the change log for it hasn't been either, so it is not necessary for this bug fix to be in the change log.
What I meant was to add skip-changelog tag (which it already is) but remove the last line of check boxes (or use strikethrough font on it with ~).

Ohh yes removed the last line.

LGTM - @varuntumbe thanks for the contribution!

My pleasure. Happy to learn more.
Thanks.


Wrong results from running "openssl speed -evp aes-256-cbc" in Emscripten

When it is run in the native system (Ubuntu 20.40), the first line of the result looks like this:
openssl speed -evp aes-256-cbc
Doing aes-256-cbc for 3s on 16 size blocks: 26616414 aes-256-cbc's in 1.46s

But when it is run in my Web Assembly build, there are two problems: 1) it is very slow (waiting for minutes), 2) the result seems to be wrong:
node openssl speed -evp aes-256-cbc
Doing AES-256-CBC for 3s on 16 size blocks:<PHONE_NUMBER> AES-256-CBC's in 0.00s

What does the "0.00s" mean?
Here is how I built the OpenSSL (the latest from the GitHub) in Web Assembly:
emconfigure ./Configure linux-generic64 -no-asm -no-threads -static -no-afalgeng -DOPENSSL_SYS_NETWARE -DSIG_DFL=0 -DSIG_IGN=0 -DHAVE_FORK=0 -DNO_SYSLOG
sed -i 's|^CROSS_COMPILE.*$|CROSS_COMPILE=|g' Makefile
emmake make

Most probably there is some problem with timers in your environment where the command is run. I do not think there is a bug in the command itself.

@t8m
Could you please tell me how to run an OS command or something to verify if the timers are running correctly?

Unfortunately I do not know of any easy way how to verify that apart from writing some test programs.

Marking as inactive, to be closed when 3.4 dev cycle ends, barring further input


Installation issues

Hello,
I am trying to install on Ubuntu 16.04.02 and having several issues.

I am using docker option since I could not succeed in any other method.
with the following commands I can make the onlyoffice work in community however no mail server is possible

Document server
sudo docker run --net onlyoffice -i -t -d --restart=always --name onlyoffice-document-server -v /app/onlyoffice/DocumentServer/data:/var/www/onlyoffice/Data
-v /app/onlyoffice/DocumentServer/logs:/var/log/onlyoffice onlyoffice/documentserver
OK
Mail server
sudo docker run --net onlyoffice --privileged -i -t -d --restart=always --name onlyoffice-mail-server -p 25:25 -p 143:143 -p 587:587
-v /app/onlyoffice/MailServer/data:/var/vmail -v /app/onlyoffice/MailServer/data/certs:/etc/pki/tls/mailserver -v /app/onlyoffice/MailServer/logs:/var/log
-v /app/onlyoffice/MailServer/mysql:/var/lib/mysql -h my_private_ip onlyoffice/mailserver
OK
sudo docker run --net onlyoffice -i -t -d --restart=always --name onlyoffice-community-server
-p 80:80 -p 5222:5222 -p 443:443 -v /app/onlyoffice/CommunityServer/data:/var/www/onlyoffice/Data
-v /app/onlyoffice/CommunityServer/mysql:/var/lib/mysql
-v /app/onlyoffice/CommunityServer/logs:/var/log/onlyoffice
-v /app/onlyoffice/DocumentServer/data:/var/www/onlyoffice/DocumentServerData
-e DOCUMENT_SERVER_PORT_80_TCP_ADDR=onlyoffice-document-server
-e MAIL_SERVER_DB_HOST=onlyoffice-mail-server onlyoffice/communityserver
Not OK .. impossible to access the community server
sudo docker run --net onlyoffice -i -t -d --restart=always --name onlyoffice-community-server
-p 80:80 -p 5222:5222 -p 443:443 -v /app/onlyoffice/CommunityServer/data:/var/www/onlyoffice/Data
-v /app/onlyoffice/CommunityServer/mysql:/var/lib/mysql
-v /app/onlyoffice/CommunityServer/logs:/var/log/onlyoffice
-v /app/onlyoffice/DocumentServer/data:/var/www/onlyoffice/DocumentServerData
-e DOCUMENT_SERVER_PORT_80_TCP_ADDR=onlyoffice-document-server
onlyoffice/communityserver
OK however I cannot see the possibility to create own domain name, basically the mail functionality will be limited to client mail not a mail server.

Oneclick install does not work form me at all. Although I enabled the SSH port 22 through my firewall , I can see request coming but later I get an error and does not proceed.

Could someone help me on the mail issue (point 1)

Hello @tedd77 !
Please run the following commands
docker logs onlyoffice-community-server
docker logs onlyoffice-mail-server

Then send to us resulting logs

Hello @JohnPerkin
Thank you for your reply, I assume that I need to run the command
sudo docker run --net onlyoffice -i -t -d --restart=always --name onlyoffice-community-server
-p 80:80 -p 5222:5222 -p 443:443 -v /app/onlyoffice/CommunityServer/data:/var/www/onlyoffice/Data
-v /app/onlyoffice/CommunityServer/mysql:/var/lib/mysql
-v /app/onlyoffice/CommunityServer/logs:/var/log/onlyoffice
-v /app/onlyoffice/DocumentServer/data:/var/www/onlyoffice/DocumentServerData
-e DOCUMENT_SERVER_PORT_80_TCP_ADDR=onlyoffice-document-server
-e MAIL_SERVER_DB_HOST=onlyoffice-mail-server onlyoffice/communityserver
then the following
docker logs onlyoffice-community-server
docker logs onlyoffice-mail-server

Before using commands docker logs you need to make sure that the corresponding containers are running. You may use docker ps.
If community server and mail server have statuses 'UP' there is no need to use docker run

Hello @JohnPerkin
~# docker logs onlyoffice-community-server

Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 12485: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty
Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 51: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty
Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 51: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty
Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 50: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty
Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 50: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty
Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 50: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty
Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 50: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty
Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 50: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty
Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 50: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty
Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 52: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty
Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 50: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty
Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 50: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty
~# docker logs onlyoffice-mail-server
[ INFO ] Checking configuration file: /usr/src/iRedMail/config ... [FOUND]
[ INFO ] Import installation process status from file: /usr/src/iRedMail/runtime/install.status.

Start iRedMail Configurations

[ INFO ] Create required system accounts: vmail, iredapd.
[ INFO ] Configure Apache web server and PHP.
[ INFO ] Configure PHP.
[ INFO ] Configure MySQL database server.
170522 13:17:46 mysqld_safe Logging to '/var/log/mysqld.log'.
170522 13:17:46 mysqld_safe Starting mysqld daemon with databases from /var/lib/mysql
170522 13:17:51 mysqld_safe mysqld from pid file /var/run/mysqld/mysqld.pid ended
[ INFO ] Configure Postfix (Message Transfer Agent).
[ INFO ] Configure Cluebringer (postfix policy server).
[ INFO ] Configure Dovecot (pop3/imap/managesieve server).
[ INFO ] Configure ClamAV (anti-virus toolkit).
[ INFO ] Configure Amavisd-new (interface between MTA and content checkers).
[ INFO ] Configure SpamAssassin (content-based spam filter).
[ INFO ] Configure OpenDkim.
[ INFO ] Configure Fail2ban (authentication failure monitor).
[ INFO ] Configure Server API.
[ INFO ] Configure SpamTrainer.
mkdir: cannot create directory `/var/log/spamtrainer': File exists
[ INFO ] Configure Awstats (logfile analyzer for mail and web server).

iRedMail-0.9.2 installation and configuration complete.

[ INFO ] Disable SELinux in /etc/selinux/config.
[ INFO ] Removing package(s): sendmail
Loaded plugins: fastestmirror
Setting up Remove Process
Resolving Dependencies
--> Running transaction check
---> Package sendmail.x86_64 0:8.14.4-9.el6_8.1 will be erased
--> Processing Dependency: sendmail = 8.14.4-9.el6_8.1 for package: sendmail-devel-8.14.4-9.el6_8.1.x86_64
--> Running transaction check
---> Package sendmail-devel.x86_64 0:8.14.4-9.el6_8.1 will be erased
--> Finished Dependency Resolution
Dependencies Resolved
================================================================================
Package               Arch          Version                 Repository    Size
Removing:
sendmail              x86_64        8.14.4-9.el6_8.1        @base        1.5 M
Removing for dependencies:
sendmail-devel        x86_64        8.14.4-9.el6_8.1        @base        254 k
Transaction Summary
Remove        2 Package(s)
Installed size: 1.8 M
Downloading Packages:
Running rpm_check_debug
Running Transaction Test
Transaction Test Succeeded
Running Transaction
Warning: RPMDB altered outside of yum.
Erasing    : sendmail-devel-8.14.4-9.el6_8.1.x86_64                       1/2
Erasing    : sendmail-8.14.4-9.el6_8.1.x86_64                             2/2
warning:    erase unlink of /var/log/mail/statistics failed: No such file or directory
warning:    erase unlink of /var/log/mail failed: No such file or directory
Verifying  : sendmail-devel-8.14.4-9.el6_8.1.x86_64                       1/2
Verifying  : sendmail-8.14.4-9.el6_8.1.x86_64                             2/2
Removed:
sendmail.x86_64 0:8.14.4-9.el6_8.1
Dependency Removed:
sendmail-devel.x86_64 0:8.14.4-9.el6_8.1
Complete!
grep: /etc/ssh/sshd_config: No such file or directory
[ INFO ] Copy firewall sample rules: /etc/sysconfig/iptables.
[ INFO ] Restarting firewall ...
[ INFO ] Copy MySQL sample file: /etc/my.cnf.
[ INFO ] Enable SSL support for MySQL server.

URLs of installed web applications:

Webmail:

Username<EMAIL_ADDRESS>Password: oxx61vxxx15

Web admin panel (iRedAdmin): httpS://192.168.xx.xx/iredadmin/

You can login to above links with same credential:

o Username<EMAIL_ADDRESS>o Password: oxx61vxxx15

Congratulations, mail server setup completed successfully. Please
read below file for more information:

/usr/src/iRedMail/iRedMail.tips

And it's sent to your mail account<EMAIL_ADDRESS>

********************* WARNING **************************************
*

Rebooting your system is required to enable mail services.

mkdir: cannot create directory /var/log/httpd': File exists rm: cannot remove /var/run/syslogd.pid': No such file or directory
rm: cannot remove /var/run/cbpolicyd.pid': No such file or directory rm: cannot remove /var/run/opendkim/opendkim.pid': No such file or directory
Starting mysqld:                                           [  OK  ]
/usr/src/iRedMail/functions/mysql.sh: line 33: ECHO_INFO: command not found
/usr/src/iRedMail/functions/mysql.sh: line 36: ECHO_DEBUG: command not found
Starting crond:                                            [  OK  ]
Starting Dovecot Imap:                                     [  OK  ]
Starting system logger:                                    [  OK  ]
Starting amavisd:                                          [  OK  ]
Starting postfix:                                          [FAILED]
Starting cbpolicyd:                                        [  OK  ]
Starting Clam AntiVirus Daemon: LibClamAV Warning: **************************************************
LibClamAV Warning: ***  The virus database is older than 7 days!  ***
LibClamAV Warning: ***   Please update it as soon as possible.    ***
LibClamAV Warning: **************************************************
[  OK  ]
Starting clamd.amavisd: LibClamAV Warning: **************************************************
LibClamAV Warning: ***  The virus database is older than 7 days!  ***
LibClamAV Warning: ***   Please update it as soon as possible.    ***
LibClamAV Warning: **************************************************
[  OK  ]
Starting httpd: httpd: Could not reliably determine the server's fully qualified domain name, using 192.168.xx.xx for ServerName
[  OK  ]
Generating default DKIM keys:                              [  OK  ]
Default DKIM keys for 168.xx.xx created in /usr/local/etc/opendkim/keys.
Starting OpenDKIM Milter:                                  [  OK  ]
Starting spamd:                                            [  OK  ]
Starting fail2ban:                                         [  OK  ]
Starting server

Hello @JohnPerkin ~# docker logs onlyoffice-community-server

Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 12485: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty

Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 51: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty

Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 51: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty

Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 50: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty

Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 50: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty

Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 50: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty

Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 50: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty

Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 50: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty

Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 50: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty

Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 52: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty

Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 50: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty

Restarting nginx nginx                                                       start-stop-daemon: warning: failed to kill 50: No such process
[ OK ]
MAIL_SERVER_API_HOST is empty
~# docker logs onlyoffice-mail-server
[ INFO ] Checking configuration file: /usr/src/iRedMail/config ... [FOUND]
[ INFO ] Import installation process status from file: /usr/src/iRedMail/runtime/install.status.

Start iRedMail Configurations

[ INFO ] Create required system accounts: vmail, iredapd. [ INFO ] Configure Apache web server and PHP. [ INFO ] Configure PHP. [ INFO ] Configure MySQL database server. 170522 13:17:46 mysqld_safe Logging to '/var/log/mysqld.log'. 170522 13:17:46 mysqld_safe Starting mysqld daemon with databases from /var/lib/mysql 170522 13:17:51 mysqld_safe mysqld from pid file /var/run/mysqld/mysqld.pid ended [ INFO ] Configure Postfix (Message Transfer Agent). [ INFO ] Configure Cluebringer (postfix policy server). [ INFO ] Configure Dovecot (pop3/imap/managesieve server). [ INFO ] Configure ClamAV (anti-virus toolkit). [ INFO ] Configure Amavisd-new (interface between MTA and content checkers). [ INFO ] Configure SpamAssassin (content-based spam filter). [ INFO ] Configure OpenDkim. [ INFO ] Configure Fail2ban (authentication failure monitor). [ INFO ] Configure Server API. [ INFO ] Configure SpamTrainer. mkdir: cannot create directory `/var/log/spamtrainer': File exists [ INFO ] Configure Awstats (logfile analyzer for mail and web server).

iRedMail-0.9.2 installation and configuration complete.

[ INFO ] Disable SELinux in /etc/selinux/config. [ INFO ] Removing package(s): sendmail Loaded plugins: fastestmirror Setting up Remove Process Resolving Dependencies --> Running transaction check ---> Package sendmail.x86_64 0:8.14.4-9.el6_8.1 will be erased --> Processing Dependency: sendmail = 8.14.4-9.el6_8.1 for package: sendmail-devel-8.14.4-9.el6_8.1.x86_64 --> Running transaction check ---> Package sendmail-devel.x86_64 0:8.14.4-9.el6_8.1 will be erased --> Finished Dependency Resolution
Dependencies Resolved
================================================================================
Package Arch Version Repository Size
Removing: sendmail x86_64 8.14.4-9.el6_8.1 @base 1.5 M Removing for dependencies: sendmail-devel x86_64 8.14.4-9.el6_8.1 @base 254 k
Transaction Summary
Remove 2 Package(s)
Installed size: 1.8 M Downloading Packages: Running rpm_check_debug Running Transaction Test Transaction Test Succeeded Running Transaction Warning: RPMDB altered outside of yum. Erasing : sendmail-devel-8.14.4-9.el6_8.1.x86_64 1/2 Erasing : sendmail-8.14.4-9.el6_8.1.x86_64 2/2 warning: erase unlink of /var/log/mail/statistics failed: No such file or directory warning: erase unlink of /var/log/mail failed: No such file or directory Verifying : sendmail-devel-8.14.4-9.el6_8.1.x86_64 1/2 Verifying : sendmail-8.14.4-9.el6_8.1.x86_64 2/2
Removed: sendmail.x86_64 0:8.14.4-9.el6_8.1
Dependency Removed: sendmail-devel.x86_64 0:8.14.4-9.el6_8.1
Complete! grep: /etc/ssh/sshd_config: No such file or directory [ INFO ] Copy firewall sample rules: /etc/sysconfig/iptables. [ INFO ] Restarting firewall ... [ INFO ] Copy MySQL sample file: /etc/my.cnf. [ INFO ] Enable SSL support for MySQL server.

URLs of installed web applications:

Webmail:

Username<EMAIL_ADDRESS>Password: oxx61vxxx15

Web admin panel (iRedAdmin): httpS://192.168.xx.xx/iredadmin/

You can login to above links with same credential:

o Username<EMAIL_ADDRESS>

o Password: oxx61vxxx15

Congratulations, mail server setup completed successfully. Please

read below file for more information:

/usr/src/iRedMail/iRedMail.tips

And it's sent to your mail account<EMAIL_ADDRESS>

********************* WARNING ************************************** *

Rebooting your system is required to enable mail services.

mkdir: cannot create directory /var/log/httpd': File exists rm: cannot remove /var/run/syslogd.pid': No such file or directory rm: cannot remove /var/run/cbpolicyd.pid': No such file or directory rm: cannot remove /var/run/opendkim/opendkim.pid': No such file or directory Starting mysqld: [ OK ] /usr/src/iRedMail/functions/mysql.sh: line 33: ECHO_INFO: command not found /usr/src/iRedMail/functions/mysql.sh: line 36: ECHO_DEBUG: command not found Starting crond: [ OK ] Starting Dovecot Imap: [ OK ] Starting system logger: [ OK ] Starting amavisd: [ OK ]
Starting postfix: [FAILED] Starting cbpolicyd: [ OK ] Starting Clam AntiVirus Daemon: LibClamAV Warning: ************************************************** LibClamAV Warning: *** The virus database is older than 7 days! *** LibClamAV Warning: *** Please update it as soon as possible. *** LibClamAV Warning: ************************************************** [ OK ] Starting clamd.amavisd: LibClamAV Warning: ************************************************** LibClamAV Warning: *** The virus database is older than 7 days! *** LibClamAV Warning: *** Please update it as soon as possible. *** LibClamAV Warning: ************************************************** [ OK ] Starting httpd: httpd: Could not reliably determine the server's fully qualified domain name, using 192.168.xx.xx for ServerName [ OK ] Generating default DKIM keys: [ OK ] Default DKIM keys for 168.xx.xx created in /usr/local/etc/opendkim/keys. Starting OpenDKIM Milter: [ OK ] Starting spamd: [ OK ] Starting fail2ban: [ OK ] Starting server

Was there ever a fix for this?

Hello @Dvalin21, can you describe your problem in more detail:

How did you install Onlyoffice Workspace?
Did you have the same problem when running a container with a mail server?

Hello @Dvalin21, can you describe your problem in more detail:

How did you install Onlyoffice Workspace?
Did you have the same problem when running a container with a mail server?

version: "3.3"
services:
  communityserver:
    networks:
      - onlyoffice
    stdin_open: true
    tty: true
    restart: always
    container_name: onlyoffice-community-server
    ports:
      - 8500:80
      - 5222:5222
      - 9443:443
    volumes:
      - /volume1/docker/onlyoffice/communityserver/data:/var/www/onlyoffice/Data
      - /volume1/docker/onlyoffice/communityserver/mysql:/var/lib/mysql
      - /volume1/docker/onlyoffice/communityserver/logs:/var/log/onlyoffice
      - /volume1/docker/onlyoffice/documentserver/data:/var/www/onlyoffice/DocumentServerData
    environment:
      - DOCUMENT_SERVER_PORT_80_TCP_ADDR=onlyoffice-document-server
      - MAIL_SERVER_DB_HOST=onlyoffice-mail-server
    image: onlyoffice/communityserver
  documentserver:
    networks:
      - onlyoffice
    container_name: onlyoffice-document-server
    volumes:
      - /volume1/docker/onlyoffice/documentserver/logs:/var/log/onlyoffice
      - /volume1/docker/onlyoffice/documentserver/data:/var/www/onlyoffice/Data
    environment:
      - JWT_ENABLED=true
      - JWT_SECRET=my secret
      - JWT_HEADER=Authorization
      - JWT_IN_BODY=true
    restart: always
    image: onlyoffice/documentserver
  onlyoffice-mail-server:
    networks:
      - onlyoffice
    privileged: true
    stdin_open: true
    tty: true
    restart: always
    container_name: onlyoffice-mail-server
    ports:
      - 25:25
      - 143:143
      - 587:587
    volumes:
      - /volume1/docker/onlyoffice/mailserver/data:/var/vmail
      - /volume1/docker/onlyoffice/mailserver/data/certs:/etc/pki/tls/mailserver
      - /volume1/docker/onlyoffice/mailserver/logs:/var/log
      - /volume1/docker/onlyoffice/mailserver/mysql:/var/lib/mysql
    hostname: houseofmanns.com
    image: onlyoffice/mailserver
networks:
  onlyoffice:
    external: true


Add PackageReferences to nuget packages

I think we are currently simply missing all PackageReferences?
There's no corresponding NuGetPackageId on a PackageReference, so I've gone with simply obtaining the Include value.
The whitespace changes earlier in the file are the result of dotnet fantomas ..

Gonna go with the other PR which is cleaner.


v5: Variable for button hover contrast

https://github.com/twbs/bootstrap/blob/a1708ec2a38d331c4c9eacac08ce9c9839649476/scss/mixins/_buttons.scss#L10
Howdy! Would it be possible to either darken or accept a custom variable for the button hover contrast?
For the meantime, we can override the "button-variant" mixin, but it seemed like an element others might find useful to adjust as well.
Thanks!

But this is the variable as a mixin argument, or do you mean to call button-variant with global variable $btn-hover-background ?

Closing for #32290 since that one's more specific.


springboot3.1.7 version can`t consum message

Before Creating the Bug Report

[X] I found a bug, not just asking a question, which should be created in GitHub Discussions.

[X] I have searched the GitHub Issues and GitHub Discussions  of this repository and believe that this is not a duplicate.

[X] I have confirmed that this bug belongs to the current repository, not other repositories of RocketMQ.

Runtime platform environment
windows 10
RocketMQ version
release 5.2.0
JDK Version
graalvm 17
Describe the Bug
messages can be sent through the producer normally, but the consumer cannot consume, the console can see the producer information and queue, can not see the available consumer, and no error is reported at startup, when the springboot version 2.5.7 can be sent and consumed normally.
Steps to Reproduce
Create a springboo project using springboot 3.1.7 and add         <dependency>             <groupId>org.apache.rocketmq</groupId>             <artifactId>rocketmq-spring-boot-starter</artifactId>             <version>2.2.2</version>         </dependency>, add a profile, create a consumer, and run the code to reproduce
What Did You Expect to See?
consumers can spend normally
What Did You See Instead?
consumers cant spend normally
Additional Context
No response

You can start a new issue in project https://github.com/apache/rocketmq-spring/issues


Including flutterCompute

Are there any plans to include the new flutterCompute functionality?
I'm having issues using the google_sign_in plugin in the isolates so I'm hoping to use the new flutterCompute and a Completer to make a thread.join() type of functionality in the main isolate.

We rely on another package and I'm not yet sure which path that package will take.


DO NOT SUBMIT

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1122  👈
#1113
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @matan-starkware and the rest of your teammates on  Graphite


Altitude support

Logic is done, but I guess I am sending altitude with wrong parameter
let userInfo: [AnyHashable: Any] = [
        "simulateLocationLatitude": location.coordinate.latitude,
        "simulateLocationLongitude": location.coordinate.longitude,
        "simulateLocationAltitude": location.altitude, // THIS line is I guess wrong
        "simulateLocationDevices": simulators,
    ]

https://github.com/RobertSasak/set-simulator-location/blob/master/sources/notification.swift#L10

As discussed here this still isn't supported https://github.com/lyft/set-simulator-location/issues/21
I double checked Xcode 12 beta 1 to make sure. Feel free to re-open if they ever introduce support for this!


Conversion from NSURLConnection to NSURLSession

Finished #119
Disclaimer, I've only tested with the Demo App, which uses only simple .GET and .POST requests.
Someone should probably test out Streaming and Media (Data) Uploads before merging this.

This is amazing! Thanks a million for this :clap: :clap:
I'll test the streaming enpoints this evening (9am here).
@mattdonnelly please add @meteochu to the repo collaborators: he's way more worthy than me.
I don't mind if you even remove me from the collaborators list: I can still participate like I did in the past, what matters is that @meteochu becomes a collaborator :smile:

Haha, good call. I'll add him as one now :smile:

Also @meteochu, thank you for all the work you've been putting in, this looks great! . I can try testing it with the streaming and media APIs this evening.

Thank you @mattdonnelly , I'm glad to hear that! (both the testing and Andy addition among the collaborators :+1: )

Streaming doesn't seem to work. Returns the following error:
An SSL error has occurred and a secure connection to the server cannot be made.```

Never mind, just had to enable NSAllowsArbitraryLoads and it worked :+1:

Never mind, enabled NSAllowsArbitraryLoads and it works now :+1:

And media uploads worked too! I'm happy to merge this in now. Nice one @meteochu, thanks for the contribution :100:

Thanks @meteochu! :clap:


Commit associated with tag cardano-ledger-conway-<IP_ADDRESS> is gone

:'(

Commit associated with a git tag cannot disappear. The fact that release was made from a branch that diverged from master branch is totally normal for a patch release.
Here is the tag: https://github.com/IntersectMBO/cardano-ledger/tree/cardano-ledger-conway-<IP_ADDRESS>
Feel free to reopen the ticket if I misunderstood you concern.

It is no longer possible to checkout this commit or pull the tag altogether because the associate git object no longer exist.

If this was the case our release process would break couple years ago.
I suspect you doing something wrong. Could you please submit instructions on how to reproduce this issue.
Here is my attempt just now:
lehins@localhost:~/tmp> git clone git@github.com:IntersectMBO/cardano-ledger.git
Cloning into 'cardano-ledger'...
remote: Enumerating objects: 178560, done.
remote: Counting objects: 100% (11535/11535), done.
remote: Compressing objects: 100% (4696/4696), done.
remote: Total 178560 (delta 6551), reused 8853 (delta 4727), pack-reused 167025 (from 1)
Receiving objects: 100% (178560/178560), 114.56 MiB | 1.66 MiB/s, done.
Resolving deltas: 100% (100210/100210), done.
lehins@localhost:~/tmp> cd cardano-ledger/
lehins@localhost:~/tmp/cardano-ledger> git checkout cardano-ledger-conway-<IP_ADDRESS>
Note: switching to 'cardano-ledger-conway-<IP_ADDRESS>'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by switching back to a branch.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -c with the switch command. Example:

  git switch -c <new-branch-name>

Or undo this operation with:

  git switch -

Turn off this advice by setting config variable advice.detachedHead to false

HEAD is now at 0268e812b Bump up the version to `cardano-ledger-conway-<IP_ADDRESS>`
lehins@localhost:~/tmp/cardano-ledger>

Indeed, that's curious. Cloning the repository anew works fine. Somehow, I cannot fetch this tag from my existing repository, which I've been doing for more than 4 years now 🙃.
Go figure...

@KtorZ You don't have to do a fresh clone the repo to get the tag or a commit, you can always fetch them this way:
$ git fetch origin cardano-ledger-conway-<IP_ADDRESS>
From github.com:IntersectMBO/cardano-ledger
 * tag                   cardano-ledger-conway-<IP_ADDRESS> -> FETCH_HEAD

FYI, this approach I believe even works for some commits that do not belong to any branches or are unassociated with any tags, just because github might need to keep them around forever anyways, so changes are visible in UI (eg. closed PRs etc). However, don't quote me on that.

you can always fetch them this way:

Well, no. Not in this case, as this is exactly what I typically do. I honestly don't know what happened but it looks like my local git got corrupt somehow. I am a fairly well seasoned git user 🤓, so I usually manage to solve my problems alone. And here, it looked like a case of a tag being pushed without the associated commit (a tag being merely just a pointer / reference to an object; but the said object must be in the database for the reference to be valid). I can fetch <IP_ADDRESS> just fine, but somehow, <IP_ADDRESS> is completely missing.
I did recall seing (and ignoring) a warning when I fetched about some remote revision test that was no fetchable, but it looked quite unrelated. So 🤷.


Resolved the bug in instagram submodule

Description

The changes made in this pull request should be briefly described.

Previously the Instagram module used beautiful soup to scrape the user data from the meta tag of the website. It's been observed that the meta tag has some discrepancies due to which accurate information was not able to be retrieved. I made the whole submodule (more like the user.py) from scratch using Selenium. I went with selenium because it was not possible with beautiful soup. I created a single function instead of 3 separate functions. This function gets all the data i.e. followers, following, and posts. This should work since Im directly getting the data from the span tag.
Resolves: [#271]
Checklist

Before submitting this pull request, kindly verify that the ensuing checkpoints have been reached.

[x] Have you adhered to the repository's defined coding convention rules?
[x] Have you updated the 'documentation.md' file with the method/function documentation?
[x] Have you sent a message along with the result or response?
[x] Have you used the try-catch technique?
[x] Has the method/class been added to the documentation (md file)?

Screenshots

Uploading a screenshot illustrating the approach you developed for validation is required.

Additional Notes/Comments

Any additional remarks or suggestions concerning this pull request are welcome.

If it is showing "page not available" please try to clear the cache and then try running it.

I certify that I have carried out the relevant checks and provided the requisite screenshot for validation by submitting this pull request.
I appreciate your contribution.

Hey @nikhil25803 can you add the gssoc label to this repo?

Hey @nikhil25803 Can I know how a level is fixed for a pr? I had to make this thing work from scratch and had to spend a lot of time on this. Can you reconsider the level if possible?

Its upon me @karthikbhandary2

oh ok @nikhil25803 understood thanks for letting me know

so its going to be level2?

Yes @karthikbhandary2


Don't throw an error if the symlink already exists

I'm running Unicorn in development with two worker processes, which is causing the symlink creation to be attempted twice.

Still getting this error after your change to remove the rescue, and even when I bump the number of unicorn workers down to 1. Probably a race condition between the unicorn master and forked worker.
If you don't want to use a rescue maybe wrap it in a mutex?

 If it still blows up given that if condition, mutex won't help. It definitely needs a rescue.

Thanks for reporting that. Again. :)

Should be fixed in 0.0.15

Thank you! :smile:


Switch all dt-tools references from master to main

These are all of the bits and bobs here that need to change when we change the branch name.

It should be! But, happy to wait to make sure.

I just haven't used it before; if you're satisfied with its ability to not disrupt open PRs, we might as well ship this.

This is what we'll have to do locally:
git branch -m master main
git fetch origin
git branch -u origin/main main
git remote set-head origin -a

Okay, rename happened. Closing and reopening to make sure it works.

To fix everything locally, go to https://github.com/jakebailey/DefinitelyTyped-tools/settings and rename master to main (for your own fork, not mine).
Then locally run:
git branch -m master main
git fetch origin
git branch -u origin/main main
get remote set-head origin -a

👍 confirmed - I can go through and update any PRs on my end.

No need, they're already updated. Mainly this is just to not have goofy branches on your fork.


Grasping CAMISIM

The setup files rhimgCAMI2_setup.tar.gz and the genomes rhimgCAMI2_genomes.tar.gz used for the A. thaliana rhizosphere mock data have been downloaded.

wget -P /path/ https://frl.publisso.de/data/frl:6425521/plant_associated/short_read/rhimgCAMI2_setup.tar.gz
wget -P /path/ https://frl.publisso.de/data/frl:6425521/plant_associated/rhimgCAMI2_genomes.tar.gz

Read through and understand how these files can be edited to meet my needs.

CAMISIM GitHub
Fritz*, Hofmann*, et al. (2019). CAMISIM: Simulating metagenomes and microbial communities. Microbiome, 2019, 7:17. doi:10.1186/s40168-019-0633-6

NOTES FROM THE PAPER
CMISIM allows customization of many properties:

The overall number of genomes (community complexity).
Strain diversity
The community genome abundance distribtuions
Sample sizes
number of replicates
NGS or 3GS technologies

These settings are determined by the configuration file config.ini.
CAMISIM works in three stages:

Design of the community, which includes the selection of the community members and their genomes, and assigning them relative abundances.
Metagenome sequencing data simulation.
Postprocessing, where the binning and assembly gold standards are produced.

1. Designing community

replicates mode generates metagenome data sets with multiple samples that have similar genome abundance.

Example of pileup file from mpileup
#CHROM  POS     REF     COV     READS   QUALS
chr1    1       A       2       AG      FE
chr1    2       C       3       CCA     FFA
chr1    3       G       1       T       E

The headings in this pileup file are:

#CHROM: the chromosome or contig name or genome.
POS: the position in the chromosome or contig
REF: the reference base at that position
COV: the read coverage at that position
READS: the read bases that map to that position
QUALS: the base qualities for the reads that map to that position

NOTES FROM THE GITHUB MANUAL

genome_to_ids.tsv: TSV file without header that holds genome_ID and path to reference genome.
metadata.tsv: TSV file with header that holds genome_ID, operational taxonomic unit assignment OTU, taxid NCBI_ID and novelty_category.

genome_ID	OTU	NCBI_ID	novelty_category
Otu522.1	41294	374	new_species
Otu522.0	41294	85413	new_species
Otu984	        987045	987045	known_strain
Otu981	        323097	323097	known_strain
Otu980	        521096	521096	known_strain

Configuration File
MAIN

seed: Sets seed to bee consistent with RNG.
phase: Full run (0), Only community design (1), Read Simulation (2)
max_processors: Number of cores to run on
dataset_id: ID for created sample data
output_directory: Set path to output directory. Will create if it does not exist.
temp_directory: Set path to store tmp files.
gsa: True/Yes or False/No to create gold standard assembly for each sample.
pooled_gsa: True/Yes or False/No to greater GSA for all samples combined.
anonymous: Set to No/False as it is not relevant for this project.
compress: Set value between 0-9, 0 is No compression, 9 is max. Recommended to set at least 1.

ReadSimulator

readsim: ART comes with CAMISIM, set path to path/tools/art_illumina-2.3.6/art_illumina
error_profiles: Path to the error profiles, set path/tools/art_illumina-2.3.6/profiles
samtools: SAMtools v1.3 comes with CAMISIM, set path to path/tools/samtools-1.3/samtools
profile: Select error profile, recommended mbarc.
size: Size of each sample in Gigabasepairs (Gbp). One Gbp equaled one GB in size for the rhizosphere mock data
type: set to art
fragments_size_mean: Mean size of the fragments. Set 270.
fragments_size_standard_deviation: SD of fragments. Set 27.

CommunityDesign

distribution_file_paths: Path to each abundanceX.tsv file as a list. ['path/abundance0.tsv', 'path/abundance1.tsv', 'path/abundance2.tsv'
ncbi_taxdump: database containing the hierarchical classification of all known organisms. Set to  path/tools/ncbi-taxonomy_20170222.tar.gz
strain_simulation_template: Path to a template.tree for the sgEvolver from the mauve suite. scripts/StrainSimulationWrapper/sgEvolver/simulation_dir
number_of_samples: Must match the number of abundance profiles.

community0

metadata: Path to the metadata.tsv
id_to*genome_file: Path to the genome_to_ids.tsv
id_to_gff_file: Optional, set it to nothing.
genomes_total: Total number of simulated genomes.
genomes_real: Number of genomes used as input.
max_strains_per_otu: Max number of strains drawn from genomes belonging to single OTU.
ratio: Ratio between different communities. Set to 1.
mode: Either replicates or differential should be used for the project.
log_mu=1
log_sigma=2
gauss_mu=1: Used with timeseries, so set to 1.
gauss_sigma=1: Used with timeseries, so set to 1.
view: Set to False as we don't need to see the distribution of genomes.


Fix currency formatter

All tests passing again!

This fixes #56. Please review.

Ping.

Thanks!


[Feature] Hive datasource support authorization

When hive authorization enabled, sql like:"set role admin" is needed before executing "add jar ..." to prevent these errors:
Error while processing statement: Permission denied: Principal [name=xxx, type=USER] does not have following privileges for operation ADD [[ADMIN PRIVILEGE] on Object [type=COMMAND_PARAMS, name=[jar, hdfs://xxx/escheduler/tenant-dev/udfs/xxx.jar]]]

For example:
https://github.com/apache/incubator-dolphinscheduler/blob/1.1.0/escheduler-server/src/main/java/cn/escheduler/server/utils/UDFUtils.java#L72
private static void buildJarSql(List<String> sqls, Set<String> resources, String uploadPath) {
        String defaultFS = HadoopUtils.getInstance().getConfiguration().get(Constants.FS_DEFAULTFS);
        if (!uploadPath.startsWith("hdfs:")) {
            uploadPath = defaultFS + uploadPath;
        }

        for (String resource : resources) {
            sqls.add(String.format("add jar %s/%s", uploadPath, resource));
        }
    }

change to :
private static void buildJarSql(List<String> sqls, Set<String> resources, String uploadPath) {
        String defaultFS = HadoopUtils.getInstance().getConfiguration().get(Constants.FS_DEFAULTFS);
        if (!uploadPath.startsWith("hdfs:")) {
            uploadPath = defaultFS + uploadPath;
        }

        sqls.add("set role admin");

        for (String resource : resources) {
            sqls.add(String.format("add jar %s/%s", uploadPath, resource));
        }
    }

In dolphin scheduler 1.1.0, preStatementsBinds is executed after funcStmt, so pre sql can't solve this issue. (https://github.com/apache/incubator-dolphinscheduler/blob/1.1.0/escheduler-server/src/main/java/cn/escheduler/server/worker/task/sql/SqlTask.java#L271)
Suggested that it could be configured on udf or somewhere.

good idea!
could you submit a pr to solve this?


Management for non-graph/embedded types

It would be useful to have schema management for non-graph types in orient db.
A usecase I am contemplating is having a class with an embedded map from string to objects of a particular class. Having a representation for the value class automatically imported (and being able to create in-memory instances of that would be very useful

you mean define a schema for a class and automatically have that class as results of queries instead of the generic record?

Not quite.
the latest addition let us build OGM schema from the database. But that includes models only for classes that derive from vertices and edges. I was trying to experiment with embedded maps from string to an object, where an object is not itself derived from a vertex. And the current class import skips those.
But it looks like orientdb itself does not quite support those https://github.com/orientechnologies/orientdb/issues/3511


WebSocket connection should timeout and report the timeout when system is overloaded

To reproduce, open this and click "expand macros". The result after a bit of waiting is this:

I would expect either some sort of error message if the server is overloaded or the invocation timed out, or the macro-expanded output.

For me the entire site is broken right now, running and building does nothing

It may not seem like it, but this report is music to my ears!
Here's the child container and process graph of the playground around the time you made this report:

I'm pretty sure that prior to #1090, this would have simply killed the entire machine, so this is useful forward progress.

Ah yes, getting no error is certainly better than killing the machine :-)


web3api.build.yaml Shouldn't Require config property

Currently, the config property is required, but it shouldn't be. In the case where a developer wants to write a custom Dockerfile, the config isn't needed.

I've been looking at this and I don't think the config property is required. I can build just fine without it.
However, an exception is thrown suggesting the config property is required when you try to build without it after first building with it. So let's say you first run w3 build with config set. This generates the .w3 folder used to build the API. If you then try building from a customer Dockerfile, an exception will be thrown saying the config property is required (fails jsonschema validation). But if you delete the .w3 folder, you can once again build just fine from a custom Dockerfile.
I am currently trying to make sure the presence of a .w3 folder doesn't affect a user's ability to build from a custom Dockerfile.


Syntax highlighting for #ifdef seems borked

Running off Wiremod's version: 'Git 84dcff6'.
(Garry's Mod version: 2018.10.10 - current public branch)
Here is a screenshot of how it looks like (given that, all of the builtin extensions are disabled/off):

See, effect() is highlighted red as there is no such function, but what's up with ranger(number)?!
There is no ranger(n) function in E2Helper, of course because ranger extension is disabled. But, why is it colored differently then?
Also, you can observe how it picks up the color of UDF. It just makes confusion. #ifdef doesn't even work for UDFs, because it is not even intended to. So, why does it color them?
Syntax highlighter seems borked, and should be cleaned up.

Yeah, #ifdef is part of the preprocessor, so it cannot catch UDFs, which are only available at parsing time (or was it run time?)
Anyway, since it doesn't make sense to ifdef a UDF anyway, the tiny mis-highlight doesn't really affect anyone.
If you can fix it and guarantee there are no side-effects, go for it.
Otherwise, better to leave the mis-highlight in.

Fixing the highlighting just would just a matter of changing how the highlighter works.

And that is precisely why I worry :)
yeah, probably you need to split udf highlighting from regular function highlighting and only allow regular function highlighting after an #ifdef

Note, this is not just about UDF mishighlighting. See how ranger(number) function is highlighted like it exists, but it doesn't because ranger extension is disabled. It should be red, just like it is for effect; which is the only one correctly highlighted in this case.

Did it myself. Needs testing.

They should all be highlighted like they exist, at least that's how I think it should be.

yeah the last thing you described is actually correct behaviour

They should all be highlighted like they exist, at least that's how I think it should be.

Not a good idea.

This is how Visual Studio highlights undefined symbols after after an #ifdef:

I suggest we follow that well-established example.


#30: No longer returning undefined when colWidth === defaultWidth

This is the second of two possible solutions to this issue. I don't see any reason why we should return undefined from getWidth when the col width specified by the consumer of Col is 12 (which is what we specify when we want it to span 100%).
I think this is the preferred solution.

This solution could work, but requires some more changes. Currently, all the default widths of Col are 12, so by removing this line of code, the intended inheritance would also not work anymore. Probably this can be fixed by making the default widths all null.

You mean the default widths of the screenClass widths of Col? Or defaultWidth in the getWidth function?

The default widths of the Col defaultProps I was talking about. The defaultWidth parameter of the getWidth function can probably disappear.

It's fixed in the latest version :smile:


disable time unit name field

Because time unit names cannot currently be edited (and making them editable is a big task), the field is now disabled.

true, will do

Updated the test to check for invalid up_bound/low_bound combination


MAINTAINERS.md: soften the responsiveness language

I tried to soften a few other parts too: please have a look and see if it looks okay/makes sense.

Yes, even better.


AADSTS50064 - CredentialAuthenticationError description could be more useful

The current description for this error is "Credential validation has failed.". It's unclear which credentials might be the cause or what kind of validation failed. There's insufficient information to diagnose the problem further.

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

ID: a826ee6f-f457-7109-9f77-67c57977d4e7
Version Independent ID: 11766f27-492e-a579-dced-8a42528cbb47
Content: Azure Active Directory authentication and authorization error codes
Content Source: articles/active-directory/develop/reference-aadsts-error-codes.md
Service: active-directory
GitHub Login: @CelesteDG
Microsoft Alias: celested

@Tragedian Thanks for your feedback! We will investigate and update as appropriate.

@Tragedian I have assigned this issue to content author to investigate and update the document as appropriate.


[CT-2187] [Feature] dbt docs stop

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
I am running dbt on a remote server and would like a user to have the ability to stop the docs server using a GUI. However, as of now there is only a command to start, not stop the server.
I would like to stop the webserver from the CLI using a command like dbt docs stop
This SO Thread could be a starting point.
Describe alternatives you've considered
No response
Who will this benefit?
I think it would be great to be able to stop the server when it is only needed rarely.
Are you interested in contributing this feature?
I can ultimately try to submit something, but would appreciate some support.
Anything else?
No response

@rainermensing Thanks for opening!
dbt docs serve does little more than start a simple HTTP server to view the static documentation site. This is just about all the code defining the operation:
https://github.com/dbt-labs/dbt-core/blob/0ef9931d197c996ccd4e3caf641729176eceda99/core/dbt/task/serve.py#L23-L28
Nowadays, if I'm honest, this doesn't make much sense as functionality that lives right in dbt-core. It's not really specific to dbt at all. So, this isn't something we'd be likely to prioritize; I hesitate to even expand its capabilities.
So, two questions:

Is there a reason why you'd want a separate command, versus stopping the remote server via a simple interrupt (ctrl^C)?
At the point where you're running your own docs server remotely, wouldn't it make more sense for you to fully "own" the code for spinning up & down that server? That feels both reasonable, given how simple the existing implementation is, and desirable, gvien the amount of customization I expect you may want/need.

I'm going to close this issue as a wontfix in the meantime, but I'm happy to hear more about your use case, and to hear if other folks have the same need.


combine dhcp-proxy and netavark into one binary

The proxy will now be called as netavark dhcp-proxy .... The new binary is about 18 MB big, this safes about 10 MB in space compared to both binaries together.
Fixes #608

LGTM

LGTM
On Sun, Mar 12, 2023 at 15:38 Brent Baude @.***> wrote:

LGTM
—
Reply to this email directly, view it on GitHub
https://github.com/containers/netavark/pull/619#issuecomment-1465281870,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB3AOCGVDVXKHS5OMZVL2BTW3YX3HANCNFSM6AAAAAAVWRUP7M
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

/lgtm
/approved


fix: convert all eslint warnings to errors

Rather than having to configure each rule to be an error rather than a
warning, we just fail the linting if there are any issues at all.

Curiously, I cannot get ESLint to show up in the editor (VSCode) - no red squiggles

Also, I do not see this doing anything, because we have no rules defined to warn - either off or error. Am I missing something?

Yup, I was also expecting the CI to fail because of the change and subsequent "fixes"

I think the 0 should be replace with 1. I tested it with 1 in Gitpod and got an error when the command was run.

Yup, I was also expecting the CI to fail because of the change and subsequent "fixes" ad additional commits.

There aren't any errors in CI because there aren't any warnings to set it off.

I think the 0 should be replace with 1. I tested it with 1 in Gitpod and got an error when the command was run.

Thing is, that would mean that the CI would pass with 1 warning.  We could do that, but I'd rather have rules either produce errors or be disabled entirely.
@ShaunSHamilton GitPod?

@ShaunSHamilton GitPod?

Quite. Regardless, I still do not understand the point of this change, as we have no config set to warn.

Quite.

Seems to be a coincidence.  main has the same issue, too.

Regardless, I still do not understand the point of this change, as we have no config set to warn.

typescript-eslint has warnings as part of the recommended config.  We don't see any errors because none of the TS files in main have triggered them.


general category listing, cleanup code, tlc

This pull request replaces #14 and several branch based changes (code in api_client.py is too interwinded for small PRs). A.o. it fixes #10. Partly using snippets/code from LE. Areas:

in _request also flatten the params
improve file_upload signature and code
generalise category listing
setup folder/category mockup for tests
delete legacy functions
improve docstrings

Thanks a lot! Merging ...


Ctrl + c while uploading does not remove temporary file

Expected Behavior
Whenever a large file is being uploaded it might be possible that the user cancels the upload, in which case pressing ctrl+c should allow for gracefull shutdown of the upload (and deleting the temporary file)

The temporary file could be created, opened for reading/writing then immediately deleted from the file system. The file handle in the program itself will then be the only reference and the file will be automatically deleted by the OS when the program exits or closes the handle. It doesn't matter if the program gracefully exits or crashes.

@Overv  thats cool! Didn't know that was possible, i'm gonna use that as an approach


Enhanced the blog

Description
Added the images and formatted the blog
This PR fixes #1651
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@Satakshijain done?

Nope, I need a mockup of meshery-design for light theme

Nope, I need a mockup of meshery-design for light theme

you don't need a mockup, just had to grab a light and dark mode screenshot of same and apply some existing logic which just change the images based on theme here, just that


Type variables can be inferred as autoType, bypassing various type safety

Edit: This issue affects not only evolving arrays but all control-flow evolving variables.

TypeScript Version:  master (60986ad)

Search Terms: autoType control flow evolving array push any infer return noImplicitAny
Code (compile with --noImplicitAny)
// The type of `arg` blocks inference but simplifies to T.
function logLength<T extends string, U extends string>(arg: {[K in U]: T}[U]): T {
    console.log(arg.length);
    return arg;
}
logLength(42);  // error
let z;
z = logLength(42);  // no error; T is inferred as `any`

function logFirstLength<T extends string[], U extends string>(arg: {[K in U]: T}[U]): T {
    console.log(arg[0].length);
    return arg;
}
logFirstLength([42]);  // error
let zz = [];
zz.push(logLength(42));  // no error; T is inferred as `any`
zz = logFirstLength([42]);  // no error; T is inferred as `any[]`

Expected behavior: An error on each line marked "no error".
Actual behavior: As marked.
Playground Link: link (remember to enable --noImplicitAny)
Related Issues: Didn't find any.

maybe we should be more conservative with any in return type inference.


Improve zebra-state logging and metrics

This change makes the state logging and metrics consistent with downloads and verifies.
It can help us work out where blocks are getting stuck.

@hdevalence #989 and #990 could help us work out where all that RAM is being used.


Unexpected benchmark module get installed with arrow

Describe the bug, including details regarding any error messages, version, and platform.

Uninstalling benchmarks-0.1.0:
Would remove:
/Users/Nok_Lam_Chan/miniconda3/envs/kedro/lib/python3.10/site-packages/benchmarks-0.1.0.dist-info/*
/Users/Nok_Lam_Chan/miniconda3/envs/kedro/lib/python3.10/site-packages/benchmarks/*
Would not remove (might be manually added):
/Users/Nok_Lam_Chan/miniconda3/envs/kedro/lib/python3.10/site-packages/benchmarks/array_ops.py
/Users/Nok_Lam_Chan/miniconda3/envs/kedro/lib/python3.10/site-packages/benchmarks/common.py
/Users/Nok_Lam_Chan/miniconda3/envs/kedro/lib/python3.10/site-packages/benchmarks/convert_builtins.py
/Users/Nok_Lam_Chan/miniconda3/envs/kedro/lib/python3.10/site-packages/benchmarks/convert_pandas.py
/Users/Nok_Lam_Chan/miniconda3/envs/kedro/lib/python3.10/site-packages/benchmarks/io.py
/Users/Nok_Lam_Chan/miniconda3/envs/kedro/lib/python3.10/site-packages/benchmarks/microbenchmarks.py
/Users/Nok_Lam_Chan/miniconda3/envs/kedro/lib/python3.10/site-packages/benchmarks/parquet.py
/Users/Nok_Lam_Chan/miniconda3/envs/kedro/lib/python3.10/site-packages/benchmarks/streaming.py

Context:
I am trying to use asv for benchmarking and seems that the PyArrow project also use this: https://github.com/apache/arrow/tree/main/python/benchmarks
More evidence:
I've downloaded the latest package from Pypi: https://pypi.org/project/pyarrow/#files, and unzip the folder locally.

I found this document discuss the benchmarks folder. It makes sense to have this in the source code, but it doesn't make sense to me to include this into the python package. Since the way that it works you need to run asv run so user are expected to run this with source code.
Suggestion
Is this necessary to be packaged into the pyarrow library? If this is unexpected, I can raise a PR to fix this. I think the change would be excluding the benchmarks from packaging the pyarrow library.
Component(s)
Benchmarking, Developer Tools, Python

Sorry for this issue, it has been fixed as of PyArrow 18.0.0.

Amazing, thanks for fixing this quickly!


change car=* to motorcar=* on charging_station

Following a discussion on the tagging mailing list and on the wiki, this changes the tag car=yes/no on amenity=charging_station to motorcar=yes/no to be consistent with the tag used for access.

I don't know if this is the intended feature, but car != yes is turned to motorcar=yes

Following a discussion on the tagging mailing list and on the wiki, this changes the tag car=yes/no on amenity=charging_station to motorcar=yes/no to be consistent with the tag used for access.

This seems ok to me - can you link to the discussion?

I don't know if this is the intended feature, but car != yes is turned to motorcar=yes

Seems wrong 😆

https://lists.openstreetmap.org/pipermail/tagging/2020-January/050486.html & https://wiki.openstreetmap.org/wiki/Talk:Tag:amenity%3Dcharging_station#Vehicles

I don't know if this is the intended feature, but car != yes is turned to motorcar=yes

Seems wrong laughing

Yes, my bad, I fixed it

any update on this PR ? It would be really cool to have it in the next release, we are planning a Project of the Month on charging_station in the French community for March.

Looks great, @nlehuby! Thanks for your patience.
Be aware we're not planning an iD release soon, so I'd recommend running a mass edit if you need these cleaned up in a timely manner.

Ok, thanks :+1:


Makefile robustness

Update make test trigger
test/agda2hs was rebuilt upon changes in src/*.hs only, but these are not all the source files (in fact, they're the less frequently used ones). Extend this to all files.
We can't use a Makefile wildcard here because the nest goes deeper.
Update test make dependencies
for efficient and correct multiprocessing

Thank you! This was indeed bothering me for a while already.


Crappy results

what are these results

Show me what you saw.

In the first try it just deepfried the photo and in the second try it blured it. It does nothing else than ruining photos

In the first try it just deepfried the photo and in the second try it blured it. It does nothing else than ruining photos

Well, can you show your results and give more details as to how you arrived at that instead of just lambasting at the results? Doesn't help anyone.

Show me what you saw.

Hey, I am training the full model on 4 x 2080 Ti's and the dataset is and everything else is same as yours with a minor difference 4k trainA, 4k train B, 100 testA and 100 testB. For how many epochs will I have to train?

@tafseerahmed

For how many epochs will I have to train?
I recommend a 50 or 100 epoch
the Generator loss is way higher than discriminator is that normal?
Yes, because cam_weight is high.


addChain does not support stakeCurrency which is required for Sei and Keplr

When trying to connect to the Sei testnet (atlantic-2) using Keplr, an error is thrown of:

"stakeCurrency" is required

In the Keplr Chain Registry the param stakeCurrency can be found.
However, in the Cosmos Chain Registry this param can't be found. That includes its case-respective alternative, stake_currency.
If the Sei network is added to Keplr by connecting it through an external source, such as their faucet, it will connect without an issue.
In the chain blob Keplr presents for approval you can see the following:
"stakeCurrency": {
  "coinDenom": "SEI",
  "coinMinimalDenom": "usei",
  "coinDecimals": 6
},

Now that Sei is included in Keplr, connecting to a cosmos-kit app will work, since we added it with the stakeCurrency param.
Please add the ability to include this param.

@lvn-skate-dragon We're using import { chainRegistryChainToKeplr } from '@chain-registry/keplr'; to convert Cosmos Chain Registry to Keplr Chain Registry. stakeCurrency is converted as follows.
https://github.com/cosmology-tech/chain-registry/blob/51dac38c0a630c1ca49d55de9b4a9c71665a6775/packages/keplr/src/index.ts#L99-L101
So "stakeCurrency" should be obtained if you correctly provide the Sei testnet (atlantic-2)  chain and assetList info.
Did you add them in ChainProvider?

@liujun93 thanks, you pointed me in the right direction.
It seems like seitestnet is mapped to 3 different AssetLists instead of just the one relevant for v1.
Since I'm using seitestnet2 I would expect the chain_name to have this key. Look at item 164 in my screenshot.

@lvn-skate-dragon Glad it helps.
About the 3 items on screenshot, I don't see difference. Maybe you can choose any one?
As for chain_name, feel free to modify it before it's added to ChainProvider props (both chains and assetLists).

@liujun93 sorry if it wasn't clear. The difference with those three objects is the chain_name.
Each of them have the value seitestnet, but really they should have the values seidevnet3, seitestnet, and seitestnet2.
The issue then comes when trying to map them. So if I know my chain is seitestnet2 and I loop through the assets list for a chain_name with this value, none will return.
It seems like this issue however is not with you but with chain-registry. If that's correct, then this can be closed.

Opened a PR. This can be closed.


Performance issues for Scrobble History - Everyone

As the group size gets larger, the time it takes for the query to execute increases. Any way to optimize?

https://stackoverflow.com/questions/52387410/sql-query-using-in-with-a-list-very-slow


Nifi 1495 - AWS Kinesis Firehose

This pull request combines Http proxy enhancement for aws processors (nifi-1489) and aws kinesis firehose processor (nifi-1495)

@mans2singh Would you be able to rebase and squash these commits now that the proxy support has been included?

@apiri I've rebased the forked branch.  Thanks

Just wanted to check if there is any other feedback/recommendation on Kinesis Firehose put processor.

@apiri - Do you have any other comments/suggestions for me ?

Hi Mans,
Been a bit busy, will certainly get to the full review this week.  Thanks for rebasing.

@mans2singh Did an initial look over the code.  The approach seems good but would like to understand more about the batching process and how we can perform this in a safe manner that does not exhaust heap so readily.
Also, would you please be able to edit the PR name to something with NIFI-1495 in it so that hopefully the JIRA integration will link and include those items on the issue.  It is unneeded to keep the reference to NIFI-1489 as those changes have been merged and PR closed in #209
Thanks!

Corrected title of pull request.  Please let me know if there is anything else required.

@apiri Your recommendation for using buffer size is great. So, should we add another property - total batch bytes, and then send them in batches of that size ? If you have any other suggestion/pointers on how this can be implemented, please let me know and I will work on it.  Thanks again for your time.

@mans2singh Yep.  I think we can probably replace the number of items in a batch size (or make this secondary to the buffer size).  The idea would be to continuously grab FlowFiles until we've reached this threshold and then when the next item arrives that would put us over that limit, we transfer it back to the incoming queue session.transfer(flowFile) note the lack of relationship.  We can then create the batch much the same way you have done now.  Does that make sense?
Thanks for your work on this (and all the other great AWS stuff, very popular extensions), and apologies for the lag on following up again on this issue.

@apiri - Just wanted to clarify - We will grab a batch of flow files and check that the flow files combined size threshold is reached.  Then we send the batch to the successful relation, and if not we send the flow files back to the work queue with the transfer method you mentioned.  This might cause additional delays if the threshold set by the user is not reached for some time because the flow files are too small or the rate at which flow files are being input is low.
No worries on the time - I know you guys are busy and I really appreciate your time/advice in helping me work on this project.

@mans2singh I think your logic for collecting the flowfiles should probably just continuously grab individual files at a time.  This continues until one of a couple scenarios is reached:  max buffer size is met or exceeded (based on the the size of each flowfile that has been collected), there are no more files coming into the processor (one or more flowfiles have been collected but we have not yet eclipsed the max buffer size).  In either case, we would close out that collected batch of files and send them on their way much as you had before.
In terms of the 250MB by default, this is from the default batch size of 250MB and, the worst case scenario, each file is 1MB in size. While each of these files is converted to a byte array for sending, they are continuously sitting on the heap.  In the event multiple instances of this processor are running, we could quickly consume some big chunks of the heap.  What I am proposing is to either get rid of the batch size (that property no longer exists) or make this a secondary consideration where we try to receive a certain batch size but first ensure we do not exceed the configured buffer and second, do not exceed the batch size, with similar semantics to the above scenarios in terms of when those batches are sent on their way plus, when a certain batch size has been reached.
Does that clarify a bit?

@apiri  - To make things simple we get rid of the batch size and we just get one flow file at a time on each invocation of the onTrigger.  We store the flow files in memory until the max size limit or no more are coming in and then send the batch held in memory to kinesis firehose.  Do we have to do anything to "close out" the collected batch ?

Nope, just send it off to Kinesis and then transfer the files to the appropriate relationship afterward.

Hey @mans2singh,
Just wanted to see how the progress was going.  We are coming up on release time for 0.6.0 and need to start wrapping things up.  If this is something we can bring to a close in the next day or, great, if not, can we push this off for 0.7.0?
Thanks and let me know!

@aldrin - I've added max buffer size limit check while getting flow files from the session as you had recommended.  I've also kept the batch size limit in place. I've added integration tests to check that the messages are sent once the max buffer size is exceeded. Can you please review the code and let me know if it meets your design ?
Thanks

@mans2singh Overall, the changes look good and appreciate the inclusion of some additional tests.  Going to do some functional testing but think we can get this merged in.  There are, unfortunately, a lot of conflicts, so for some of the minor points mentioned in the review, would you also be able to please get this rebased so we are good to go for a merge?
Thanks!

Looking good going to AWS endpoints and services.  Good stuff.

@aldrin - I've moved the check for buffer size and rebased as you had recommended.  I could not find the data size validator that you have mentioned though. Please let me know if you have any other recommendation.  Thanks

@mans2singh I think Aldrin was referring to DATA_SIZE_VALIDATOR in org.apache.nifi.processor.util.StandardValidators. If you need custom min/max size (in bytes), you can use StandardValidators.createDataSizeBoundsValidator(minBytesInclusive, maxBytesInclusive)

@mattyb149 has it.
@mans2singh the updates look great.  Concerning the validator, how does 6bed3bcaad61c516eacf268400cdd03c0cd58dae look to you?
Otherwise, I think we can send this one on its way.

@aldrin @mattyb149 Thanks for your pointers and samples for using data validator.  I've made the changes and updated test cases.  Please let me know if there is anything else required.  Thanks again.

@mans2singh Looks great, thanks for getting those changes together and the contribution.  I'll be merging this into master shortly.


add colorRanges

It's a reasonable addition, thanks! Just be aware the API for it will probably change relatively soon.

Did this change work for you? Aframe doesn't know how to parse the type: 'colorRanges' in the schema.

It did work, e.g if I use jxrhighlight() in https://fabien.benetou.fr/pub/home/future_of_text_demo/engine/ (sorry I often switch branches so might not be there when someone else checks, working on a solution for that).
Here is how I use it https://git.benetou.fr/utopiah/text-code-xr-engine/src/branch/syntax-highlighting/index.html#L2319 namely setting el.setAttribute("troika-text", {colorRanges: foundRanges}) on an <a-troika-text> entity. Maybe the problem comes from setting the troika-text on an <a-entity>? It should work still

Thanks, the problem was only when defining it using markup, it didn't know how to parse the string attribute. I added parsing instructions to the schema (just JSON.parse()) and it works now.
I've published v 0.10.0 with this change, let me know if you have any problems with it.

Unfortunately I'm getting
    Aframe 6
    jxrhighlight https://fabien.benetou.fr/pub/home/future_of_text_demo/engine/:2330
    jxrhighlight https://fabien.benetou.fr/pub/home/future_of_text_demo/engine/:2325
    <anonymous> debugger eval code:1
debugger eval code:45199:742

now so should using e.g el.setAttribute("troika-text", {colorRanges: {0: 0x0099ff, 3: 0xffffff}}). Could you please provide a working example?

Ah, I see, after my parsing change it now expects a JSON string. You could JSON-encode it:
el.setAttribute("troika-text", {colorRanges: JSON.stringify({0: 0x0099ff, 3: 0xffffff})})
But I've published 0.10.1 which allows the object.

Yeay it works as-is now, thanks!


Issue Importing strax

I am suddenly facing issues in importing strax now (was working fine till yesterday). This is how I installed it:
git clone https://github.com/XENONnT/straxen
pip install -e ./straxen
import strax

And the error is:
ModuleNotFoundError: No module named 'strax'

Hi Nikita,
Perhaps you want to checkout a tutorial on making your own conda and installing it there?
A good place to start is the following:
# <<First install conda>>
conda create -n strax python=3.7 # create a conda environment
which python
cd ~/software # or wherever you want to store software
pip install straxen # this should work
git clone https://github.com/XENONnT/WFSim #WFsim is not pip installable
pip install -e WFSim # install
git clone https://github.com/XENONnT/utilix # same here
pip install -e utilix# install
python -c "import straxen; print('Yes, straxen works!')" # test and hopefully success

Perhaps it's good to reach out through slack, the github issues are more for code-specific bugs and this seems to be related to your personal installation.
I just reinstalled straxen to double check, this should work.

Hi, I am actually using Google Colab to simulate events. It doesn't have provisions for creating conda environments. I'm sorry but could something else work out?
Yes, I am an undergrad in the nT experiment. I don't have access to the Slack community or the analysis guide (the site asks for credentials) yet. That's why I had to put up an issue.


Recycle bin - fixed position on iPhone

microbit.org-ticket: 1251
One of the customers who reported an issue that led to
https://github.com/Microsoft/pxt/issues/1828
Has got back in contact to confirm the fix in that ticket worked.  Thanks :)
He's now reported the following:
The rubbish bin (trash can) icon that shows up in place of the same block categories list when you drag blocks back to it (to delete them) had a position that is fixed in the block category list, as opposed to the screen position (or alternatively, does not take the list scroll/start relative position into account), meaning when using blocks from categories further down the newly scrolled list, the rubbish bin cannot be seen as it's off the top of the screen.
 I would suggest the bin icon should always appear on the same fixed location relative to the top and bottom of the VISIBLE part of the block categories list.

Please check makecode.microbit.org/beta to very the issue has been fixed.
This will make it out to the public on the next release.


Label causing import error

I am importing a large (500K+ messages) mbox to Gmail. The process started fine, imported a few thousand messages and got interrupted with the following message:
Creating label Muted
400: Invalid label name - invalidArgument

When I re-run the command, it exits with the same message. I guess "Muted" causes a conflict. I'd be okay skipping importing the messages with the label "Muted" but I don't think that's an option. I would appreciate the groups input on how I can resolve this issue.

Can you try running the restore with the additional argument of:
--extra-reserved-labels muted

this tells GYB to treat Muted as a reserved label it should not try to create. Instead it will be created as _Muted.

Thanks. I ended up doing --strip-labels. I may redo the whole thing and try your suggestion.


Strange step behavior

Using @vueform/slider version 2.1.9.
Step is :step="1" and :max="2000" --- when users drag with their mouse, the step increases by 2.
If I set the max to 1000 or less, the value will correctly step by 1.
Anything above 1000 seems to cause the step to increase by 2, sometimes 3.
What might be happening here?

<script setup>
  import { reactive } from "vue"

  const vueformSliderState = reactive({fees: 0})

  const formatCurrency = (value, maxFractionDigits=0) => {
    return new Intl.NumberFormat('en-US', {
      style: 'currency',
      currency: 'USD',
      currencyDisplay: 'symbol',
      currencySign: 'standard',
      maximumFractionDigits: maxFractionDigits
    }).format(value)
  }

  const sliderFormat = (value) => {
    return formatCurrency(value)
  }
</script>

<style lang="scss">
  @import<EMAIL_ADDRESS>  .slider {
    --slider-height: 18px;
    --slider-handle-width: 32px;
    --slider-handle-height: 32px;
  }
</style>

<template>
  <VueformSlider
    v-model="vueformSliderState.fees"
    :min="0"
    :max="2000"
    :step="1"
    :lazy="false"
    showTooltip="always"
    :format="sliderFormat"
    />
</template>

I've realized that this is likely a limitation of screen real-estate. In my case, the parent container of the slider is 1000px wide, which roughly explains why the slider jumps by 2 when dragging it -- as there are not enough pixels to represent 2000 ticks on a slider. The problem is compounded in a mobile view, whereby it appears to jump something like 15-20 per tick, when sliding using your finger.


Validator::allowedValues() doesn't respect Validator::ifPresent()

When using ifPresent() followed by allowedValues(), a ValidationException will always be thrown if the environment variable is missing.
The following example assumes no .env file.
Example
<?php

use Dotenv\Dotenv;

require __DIR__ . '/vendor/autoload.php';

$dotenv = Dotenv::createMutable( __DIR__ );
$dotenv->safeLoad();

$dotenv->ifPresent( 'FOO' )
       ->allowedValues( [ 'bar', 'baz' ] );

Actual Output
PHP Fatal error:  Uncaught Dotenv\Exception\ValidationException: One or more environment variables failed assertions: FOO is not one of [bar, baz]. in [...]/phpdotenv/vendor/vlucas/phpdotenv/src/Validator.php:181

Expected Output
Nothing.
Version
4.1.2

Thanks for getting in touch, but we already have a test that confirms this is not the case, and no exception will be thrown: https://github.com/vlucas/phpdotenv/blob/master/tests/Dotenv/ValidatorTest.php#L36-L42.
Note that there is a distinction between being not present, and being the empty string. The empty string is considered to be present, and set to the empty string, which is different from being unset.

Interesting thanks. Related failing test: https://github.com/vlucas/phpdotenv/pull/419.

This will be fixed in the upcoming v3.6.2 and v4.1.3 releases, which should be released within the next few hours.


About schema

Description of issue
It's unclear on documentation that if you setting up the schema you'll need to write each field explicitly, even if some field is not required.

Usage example

@HigoRibeiro Can you provide the documentation for that?

Closing this issue since it was done on #92.


check supports_interface best practices for new Cairo and create a sensible interface ID for Kakarot EOA interface

We currently have:
In https://github.com/kkrt-labs/kakarot/blob/0c6e7a62a38e973ad6ceebfbf648841749ffe0ee/src/kakarot/accounts/eoa/externally_owned_account.cairo#L105
// @dev Return true if the interface_id is supported
// @dev TODO: check what interfaces the contract should support and maybe create one for a kakarot account
// @param interface_id The interface Id to verify if supported
@view
func supports_interface{syscall_ptr: felt*, pedersen_ptr: HashBuiltin*, range_check_ptr}(
    interface_id: felt
) -> (success: felt) {
    if (interface_id == ExternallyOwnedAccount.INTERFACE_ID) {
        return (success=1);
    }
    return (success=0);
}

### Tasks
- [ ] Check if `supports_interface` is still an interesting and idiomatic method
- [ ] If so, check how ID is officially determined (what is the source EIP? SNIP?), then choose an ID for our EOA smart account
- [ ] Implement supports_interface

Important update: we are gathering some bugs in the Kakarot v0 codebase, we need to make sure each issue and each PR in Kakarot-ssj is aware of the lists of known bugs. Look at this link everytime you take an issue and check your issue isn't targeted by a known bug.

Important update: we are gathering some bugs in the Kakarot v0 codebase, we need to make sure each issue and each PR in Kakarot-ssj is aware of the lists of known bugs. Look at this tracking issue everytime you take an issue and check your issue isn't targeted by a known bug. Will add this reminder in many places to make sure we keep track of known bugs.

The SNIP-5 (https://github.com/starknet-io/SNIPs/blob/main/SNIPS/snip-5.md) is related to this issue.
You can even find the way an id should be calculated in it, the python scripts is given.
So, admitting a kakarot EOA is defined by theses functions :
fn supports_interface(felt252) -> bool;
fn is_valid_signature(felt252, Array<felt252>) -> bool;
fn __execute__(Array<Call>) -> Array<Span<felt252>>;
fn __validate__(Array<Call>) -> felt252;
fn __validate_declare__(felt252) -> felt252;

The id should be : 0xda6e5bc089be1ce54d429c85fdabf042e660021917106fe636eb4f9c040cca.
A discussion (https://community.starknet.io/t/starknet-standard-interface-detection/92664/30) can be found discussing the adoption of SRC5 and, 1 month ago, it seemed many projects already adopted this practice.
An example of implementation can be found on the OpenZep ERC721 (https://github.com/OpenZeppelin/cairo-contracts/blob/a1b559c46eaa48d855b049134fd294ce43ee44e7/src/token/erc721/erc721.cairo#L93)
I am available to enlight any confusion.
And if Kakarot adopts this methods, I'd be glad to try to implement it ! :)


CASSANDRA-19030: Fix vector quickstart documentation example CQL

CASSANDRA-19030 - Fix CQL examples for Vectors not working properly.
Just some small fixes to some incorrect CQL examples. Was trying to play with Cassandra 5 Alpha 2 and I came across this.
The below CQL seems to work now after the tweaks.

Thanks @polandll! Let me know when that's done and I'll close this PR off without a merge. Appreciate the fast replies on this!

committed with https://github.com/apache/cassandra/commit/f347c58efae4faf0e294bc1d4a086cb174597068


Add custom component

I'd like to add new component such as videos, and other stuff like google forms. How can i achieve that?

If you want to contribute to the project please follow the following instructions.
Pull requests
Good pull requests - patches, improvements, new features - are a fantastic
help. They should remain focused in scope and avoid containing unrelated
commits.
Please ask first before embarking on any significant pull request (e.g.
implementing features, refactoring code, porting to a different language),
otherwise you risk spending a lot of time working on something that the
project's developers might not want to merge into the project.
Please adhere to the coding conventions used throughout a project (indentation,
accurate comments, etc.) and any other requirements (such as test coverage).
Adhering to the following process is the best way to get your work
included in the project:

Fork the project, clone your
fork, and configure the remotes:
# Clone your fork of the repo into the current directory
git clone https://github.com/<your-username>/ng-surveys.git
# Navigate to the newly cloned directory
# Assign the original repo to a remote called "upstream"
git remote add upstream https://github.com/alexitaylor/ng-surveys.git

If you cloned a while ago, get the latest changes from upstream:
git checkout master
git pull upstream master

Create a new topic branch (off the main project development branch) to
contain your feature, change, or fix:
git checkout -b <topic-branch-name>

Commit your changes in logical chunks. Please adhere to these git commit
message guidelines
or your code is unlikely be merged into the main project. Use Git's
interactive rebase
feature to tidy up your commits before making them public.

Locally merge (or rebase) the upstream development branch into your topic branch:
git pull [--rebase] upstream master

Push your topic branch up to your fork:
git push origin <topic-branch-name>

Open a Pull Request
with a clear title and description.

IMPORTANT: By submitting a patch, you agree to allow the project
owners to license your work under the terms of the MIT License.

i'd like to contribute but i can't make a new component for this, how can i create custom component?

If you want to add more question types, you add them here:
https://github.com/alexitaylor/ng-surveys/tree/master/projects/ng-surveys/src/lib/builder/question-type
If you want to add a new feature fill out the prompts below then you can assign yourself the issue.
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.


AvoidBlockGoal shouldnt make tps go brrr, please test it serverside swan

Will test on server first before merging
also pr #100 nice


About the event that is not started by immediate execution

In the immediate execution of batch processing, there is a rare event that the processing is not started without being executed.
This event does not seem to occur when it is executed at a specified time or when it is executed periodically (every minute (hours)).
The frequency of occurrence is not every time, but it occurs frequently, and after it occurs, if the same process is executed immediately again, it will be executed normally.
When this event occurs, no error or other error is output to the log, making it difficult to identify the cause.
We are using the following quartz assets, but would you please let us know if there are any possible factors or solutions,
such as the past time being referred to even at the time of immediate startup and not being executed?

quartz-1.8.3.jar
quartz-oracle-1.8.3.jar
quartz-weblogic-1.8.3.jar

The environmental information in question is described below, but it has also occurred in other environments.

This event has not been resolved yet.
We will provide any other necessary information.
Even small things are fine, so if you have any information or countermeasures that will help you identify the cause, please let us know.


Add license to pom.xml

Cherry-picked and applied with 352115ce862dffb2e4da2628a5743c9d4f1801ff which worked https://clojars.org/io.github.nextjournal/clerk/versions/0.16.1016. Thank you!


How to handle segmentations and labeling that requires manual input?

Related to #20
As it stands, the segmentation for some subjects and RF shim cases has to be corrected manually. How should this be handled? Would the user be asked to manually correct the segmentations, as they are currently asked to manually help with the labeling, or should the manually corrected segmentations be uploaded as part of the derivaties folder?

upload in derivatives— i’ll take care of it once the analysis pipeline is cleaned up #6

Fixed in https://github.com/shimming-toolbox/rf-shimming-7t/pull/6


add action create --copy

Is your feature request related to a problem? Please describe.
Sometimes, it is convenient to create an action by copying an other. The wsk cli supports wsk action create newAction --copy oldAction as an example.
Describe the solution you'd like
Allow action create with --copy as convenience to copying an existing action to a new name.
Describe alternatives you've considered
The alternative is to manually get the action, and save the code, params and annotations to files then recreate the action.

aio runtime:action:create --copy ACTIONNAME [ACTIONPATH]


perfectly renders single recipe view

single recipe view now uses the real data when clicked on single recipe from the all recipes view

please avoid the error of travis if it's related to timeout issue regarding neo4js !


Add link to MSAL overview from Java migration doc

This is to help readers learn more about MSAL overview and details before proceeding to follow migration guide.

@navyasric : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Publishing in this repository is suspended until Tuesday, May 25. Merging will resume Tuesday, and we'll merge this approved pull request at that time. Thanks.


Add MUL operator to relay tflite frontend

MUL operator is used in TFLite SSD Resnet 50 model (ssd_resnet_50_fpn_coco).
Model graph: ssd_resnet_50_fpn_coco.tflite.pdf
### ssd_resnet_50_fpn_coco summary:
# op_id: op_name - count
 0: ADD - 58
 2: CONCATENATION - 2
 3: CONV_2D - 110
14: LOGISTIC - 1
17: MAX_POOL_2D - 4
18: MUL - 42
22: RESHAPE - 14
32: CUSTOM - 1
34: PAD - 1

This PR transforms existing convert_add method to generic _convert_elemwise method (similar approach is used in tensorflow frontend code)
Existing convert_add and new convert_mul methods use _convert_elemwise method now.
I also created _test_elemwise method based on existing _test_add method.
Both _test_add and _test_mul use _test_elemwise method now

@srkreddy1238 @jroesch @FrozenGene Can you have a look?

@kevinthesun please manage the PR


Cannot provide externally obtained NOC and externally managed key pair to Darwin framework on controller init

Problem
When managing your own fabric for multi-fabric use-cases on iOS, there is currently no API that allows providing an NOC for a controller, and associated key pair protocol implementation.
This has the following side-effects:

Clients have to provide the root CA key pair (including private key) and let the framework decide of NOC issuance for the local controller, which is impossible if the key pair resides outside the app calling the framework (e.g. a cloud-based CA).
There is no way to provide a Keychain-based or other secured non-native version of the implementation of the operational key used during CASE, to safeguard access to the private key bits, which increases attack surface to operational keys that may underpins administrator controller applications in a fabric.

Proposed Solution

Allow providing an optional already-generated NOC and associated keypair if the issuance of controller NOCs is delegated to outside the framework
Allow the P256Keypair* in the fabric table to be backed by a bridge to externally provided crypto implementation with the following minimal API which suffices for both initiator and responder roles in CASE:

GetPublicKey()
SignMessage()
ECDHDeriveSecret()

ECDHDeriveSecret()

As discussed today, this is not needed, because secrets are derived from ephemeral keys during CASE; the operational key is only used to sign things.
The rest can be done once https://github.com/project-chip/connectedhomeip/pull/18482 merges.


run-vcpkg cache

Hi Lukka,
I've been using your github action successfully to build packages via vcpkg and cache them.
Today I tried building a different package via vcpkg and for some reason it's crashing.
The package itself builds successfully but there seems to be a crash in run-vcpkg:
Please advise.
I'm using the following command:

name: Restore from cache and install vcpkg
uses: lukka/run-vcpkg@v6
with:
vcpkgGitCommitId: 595777db2332a3442b73f9af9f656355f207aec9
vcpkgTriplet: x64-windows
vcpkgArguments: shaderc
cleanAfterBuild: false

Here's the log output
-- Building x64-windows-rel
-- Installing: ...
-- Performing post-build validation
-- Performing post-build validation done
Stored binary cache: C:\Users\runneradmin\AppData\Local\vcpkg\archives\7f\7f0726ce6b0955530160c72a91edc69418d014b7.zip
Building package spirv-tools[core]:x64-windows... done
Installing package spirv-tools[core]:x64-windows...
Installing package spirv-tools[core]:x64-windows... done
Elapsed time for package spirv-tools:x64-windows: 18.47 min
Starting package 4/4: shaderc:x64-windows
Building package shaderc[core]:x64-windows...
Error: Last command execution failed with error code '3221225477'.
:stopwatch: elapsed: 1458.747 seconds
Error: Last command execution failed with error code '3221225477'.
at BaseUtilLib.throwIfErrorCode (D:\a_actions\lukka\run-vcpkg\v6\dist\index.js:4930:19)
at VcpkgRunner. (D:\a_actions\lukka\run-vcpkg\v6\dist\index.js:5625:28)
at Generator.next ()
at fulfilled (D:\a_actions\lukka\run-vcpkg\v6\dist\index.js:5440:58)
at processTicksAndRejections (internal/process/task_queues.js:93:5)
Error: run-vcpkg action execution failed: Error: Last command execution failed with error code '3221225477'.

@jmoguillansky-gpsw looks like some tool executed is failing with exit code<PHONE_NUMBER>. Do you have a link to the failed run?
Also, it may be needed to have a run where all the output is verbosely collected in the logs.

Hi Lukka,
Here's a link to the run.  There's some extra code for debugging in the workflow file.
https://github.com/gopro/gopro-lib-node.gl/runs/1876803404?check_suite_focus=true
Thanks,
Jeff
Get Outlook for Androidhttps://aka.ms/ghei36

From: Luca<EMAIL_ADDRESS>Sent: Wednesday, February 10, 2021 10:46:18 PM
To: lukka/run-vcpkg<EMAIL_ADDRESS>Cc: Jeffrey Moguillansky<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [lukka/run-vcpkg] run-vcpkg cache (#76)
@jmoguillansky-gpswhttps://github.com/jmoguillansky-gpsw looks like some tool executed is failing with exit code<PHONE_NUMBER>. Do you have a link to the failed run?
Also, it may be needed to have a run where all the output is verbosely collectedhttps://docs.github.com/en/actions/managing-workflow-runs/enabling-debug-logging in the logs.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/lukka/run-vcpkg/issues/76#issuecomment-777235558, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AO3BUE64ENIURMFZE7VSWD3S6N4LVANCNFSM4XOGPA6A.

@jmoguillansky-gpsw the log shows that the command vcpkg install --recurse shaderc --triplet x64-windows is being executed, and during its execution it is failing and returning an exit code other than 0. Does not look like there is anything that we could possibly do on the run-vcpkg side in this case.

Let me know if there is otherwise anything that can be done on run-vcpkg side to address this problem


Add utility to pipe all events to an EventBus

Right now if you want to pipe all events on an EventBus you do bus.when(stack, "rule", () => true).pipe(...);. It would be nice to be able to do bus.whenAny().pipe(...); or bus.all().pipe(...) instead

Duplicate https://github.com/sam-goodwin/functionless/issues/89


Allow Zabbix API v3.4 access (Closes: #81)

At least on our systems this version is now running in production and performing well.

Need to update zabbixapi/spec/server.rb and it's high time we should drop support for 2.4 and 3.0 per the README (https://github.com/express42/zabbixapi#version-policy). I'll see what I can do.
it 'should be 2.4.x or 3.0.x or 3.2.x' do
  expect(zbx.server.version).to match(/(2\.4|3\.[02])\.\d+/)
end

Looking over https://www.zabbix.com/life_cycle_and_release_policy and we may also want to consider changing the version policy for the gem to include the last two Zabbix versions along with the LTS release (ie. 3.2, 3.4, and 3.0 LTS).

From my experience working with Python library I can see what new API rarely have breaking changes: 3.2 library works with 3.4 and 4.0 as well without any changes. If your code raising proper errors in case API call returned an error, it seems safe not to check the version and assume the user will handle the problem.
It's a good practice not to hardcode supported versions list in my humble opinion.

Let's keep the current policy of official support for last 2 versions, so 3.4 and 4.0 for now.