API error when (manually) updating risk status between midnight and ~3 am.

Avoid duplicates

[x] Bug is not mentioned in the FAQ
[x] Bug is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
[x] Bug is not already reported in another issue

Describe the bug
When I open the app between roughly midnight and 3 am, risk status is shown as 'unknown' (grey). If I open 'risk status' it doesn't get updated, and when I return to the main menu the app throws an error. Risk status turns green, but with the last timestamp from successful risk status update. The app does start again afterwards, but with the same erroneous behavior. If I open the app after ~3 am, updating seems to work, risk status timestamp gets updated. Everything looks good. Error returns the next night.
Expected behaviour
No error.
Steps to reproduce the issue

Screenshots still disabled 😡!

Technical details

Mobile device: Samsung Galaxy NoteFE
Android version: Android 9.0
CWA version: 1.0.5

Additional context
I don't know the exact time window. It always happened after midnight. Earliest successful return to normal functionality was at 3:02 am. I couldn't make a screenshot and didn't have the option to write down the error message yet, but I remember it was something about the API. I will try to write it down when this error occurs again.
I have been using the app since the first day of release. It also seems as if automatic updating in the background doesn't work. I have to open the app manually. I made sure to exclude it from battery optimization and to prevent the phone from sending it to sleep. It still needs to be opened manually to update risk status.

Dear @treysis,
the errors which you are describing are probably related to already existing issues. Please check the error code which the app is telling you and use the search for open issues here to check if for example #774 applies to you as well. For the background update error please see #756.
In case you really can't find an already existing issue with this error code/text, please open another issue with the detailed description.
Mit freundlichen Grüßen/Best regards,
SW
Corona Warn-App Open Source Team


Allow argument :if-exists nil for  retrieve-file

Hi,
this small change allows to write (retrieve-file name :if-exists nil) to silently ignore files which exist locally.
This wasn't possible previously, as retrieve-file would subsequently try to write to the stream NIL.
Cheers
Kambiz

Thanks!


Can I run this sample on a Raspberry Pi?

I am sorry to ask this question here, but I am afraid I haven't received any answer yet in the other forum.
Is a Cognitive Services sample I wanted to run on a RPI running Windows IOT Core.
This is my question
Whenever I try to deploy the code to the remote device, it asks me for a Windows Phone or to use an Emulator.
I think the Error Code was DEP3310
Thanks in advance!

Closing as you have an answer at your prior post.


Movement requirements

This is a quality-of-life change that would let us change in-area exits that check for open doors to be simple movements (and thus get times added automatically).
[ ]: Accept req in movement items and create a table of requirements.
[ ]: Evaluate these requirements in the condenser.
[ ]: Check these requirements when checking movements.

This kind of conflicts with movement_states which are cached versions of movement requirements (that affect time as well).

Alternate idea: add an exit parameter e.g. movement: base which calculates the exit time as if it were a local movement. For non-base movements we'll have to decide whether to require the movement reqs are included explicitly or added implicitly.
If we wanted to allow multiple movements in this field, it would have to create multiple exits as we don't currently have variable times.


theme get command fails with: x509: certificate signed by unknown authority

I tried following the Getting Started documentation but when I run the theme get command it fails with the error message: x509: certificate signed by unknown authority and appears to be referencing the meta.json file
It's worth noting that this is an existing app that I'm taking over from another developer.
My Environment
OS : MacOS High Sierra
Themekit version: v.0.8.0

If you go to the shops domain in your browser like this: https://[shop-name].myshopify.com/meta.json does that succeed? Or do you get a certificate warning?

Yes it succeeds

Hmm okay, it may have something to do with your keychain access settings. I don't know too much about it so I will get back to you when I can investigate it.

By any chance are you using a proxy in your config or in general?

Sorry I'm not sure what config you're referring to?
This is a shop that is halfway through development, and I've taken over. And I've never worked on a Shopify app before. I want to be able to work on it from my local. I saw that they named the app Theme Kit so I figured that's what they were using.

Oh you need to make sure they are using themekit or Slate they handle your projects in different ways. If your theme looks like this: https://github.com/Shopify/skeleton-theme they were probably using Slate.
If however you are using themekit, then there should be a config.yml in the root directory of your project.

Ok, I don't see any config file. It also doesn't look like it was done with Slate.
What's interesting to note is I don't seem to have any issues when getting it on my Windows machine...

Okay did you inherit a repo or just the shopify files? If just from shopify, you are getting files from shopify using theme get -t=[theme_id] -s=[store] -p=[password]?

When I run the same command on my Windows PC everything works fine.

Okay, that indicate that there is something silly with your KeyChain Access. Okay I will have too look into that. Thank you for you help.

Ok no problem. Look if it's a big mission for you to help me then don't worry about it. I can work from my Windows machine... it's not ideal but it will do fine.

Okay it seems like this can happen when there are duplicate and untrusted keys in your keychain access.
You can actually see this play out in an issue with Go https://github.com/golang/go/issues/24147
From that issue, I don't have many ways forward with this. I would recommend opening your keychain access and removing any invalid system and root keys and then trying again.

I have also just migrated to Go 1.11 which it seems the go developers have tried to mitigate issues like yours. Please try it out but running theme update --verison=v.0.8.1-prerelease

I temporarily had this problem which broke all commands that made HTTP requests. I fixed it by opening up a new terminal. I have no idea why that worked.

@IOIIOOIO Did your problem disappear after a restart as has been reported? Because as far as my research has gotten me, it looks like this would most likely be an operating system issue.

@tanema FWIW, the issue is still happening for me, I am on latest Mojave. Restarting does not help.

Cloning the repository and compiling from source fixed the panic, but still see the x509 cert error.
make test seems to succeed, though I don't see an entry for download in the test list?
❯❯❯ make test
ok  	github.com/Shopify/themekit/cmd	(cached)	coverage: 86.5% of statements
?   	github.com/Shopify/themekit/cmd/static	[no test files]
?   	github.com/Shopify/themekit/cmd/theme	[no test files]
?   	github.com/Shopify/themekit/cmd/tkrelease	[no test files]
ok  	github.com/Shopify/themekit/src/atom	(cached)	coverage: 100.0% of statements
ok  	github.com/Shopify/themekit/src/cmdutil	(cached)	coverage: 90.1% of statements
?   	github.com/Shopify/themekit/src/colors	[no test files]
ok  	github.com/Shopify/themekit/src/env	(cached)	coverage: 84.8% of statements
ok  	github.com/Shopify/themekit/src/file	2.482s	coverage: 94.5% of statements
ok  	github.com/Shopify/themekit/src/httpify	(cached)	coverage: 93.9% of statements
ok  	github.com/Shopify/themekit/src/ratelimiter	(cached)	coverage: 100.0% of statements
ok  	github.com/Shopify/themekit/src/release	(cached)	coverage: 89.5% of statements
ok  	github.com/Shopify/themekit/src/shopify	(cached)	coverage: 90.1% of statements
ok  	github.com/Shopify/themekit/src/static	(cached)	coverage: 77.4% of statements
?   	github.com/Shopify/themekit/src/static/cmd	[no test files]

Poking around the repositories, I am pretty sure the issue is https://github.com/golang/go/issues/24652 (resolved in go 1.12) / https://github.com/golang/go/issues/26039 (backport to 1.11, still not released)
As homebrew go version is 1.11, not 1.12, the issue is still present for me...
For now I was able to work around it by doing the following steps:
0. run openssl s_client -connect my-store.myshopify.com:443 and see what certs are used. Check Keychain for presence of the certificates listed. DigiCert SHA2 High Assurance Server CA seems to be missing...

download the DigiCert SHA2 High Assurance Server CA from DigiCert site
Open Keychain Access and drag the downloaded certificate into login keychain
right-click the certificate, expand "Trust" section, and mark it as "Always Trust"

After that, everything seems to work so far...

Hey @Aeon thanks for the deep dive!
Regarding the panic, that should be handled better (and my development path in the trace is just a characteristic of Go) but the reason that was happening to you is that you were trying to update to v.0.8.2-prerelease the problem is that you have an extra period between the v and 0. This caused a panic because that is an invalid semver version. I can add an appropriate error for that.
Regarding the certificate issue, thank you for posting a solution that worked for you, thank you for doing the research, I was aware of those issues while looking into this issue but couldn't say for sure if that was the issue. I don't think I know enough about SSL to understand why you would be missing a certificate on your machine. I will try to talk some people will the experience to figure that out, but identifying that you were actually missing a cert could be a critical bit of information, so thank you so much!
If someone else has this issue as well, please try out a version without CGO by running theme update --version=v1.0.0-issue544 and check to see if it was fixed.

Unfortunately @Aeon's solution didn't work for me, even after rebooting (macOS Mojava 10.14.3)

Okay I have updated to a fixed version of Go as well as disabled CGO in themekit. This will be fixed in the v1.0.1 release

This should be fixed in 1.0.1 If you still experience it happening please reopen

I have this same issue
theme get --list -p=[password] -s=groven.myshopify.com
request failed after 5 retries with error: Get https://groven.myshopify.com/meta.json: x509: certificate signed by unknown authority
openssl s_client -showcerts -connect groven.myshopify.com:443
Verify return code: 0 (ok)
theme version
ThemeKit 1.1.4 darwin/amd64
MacOS Big Sur
Version 11.1

I have this same issue
theme get --list -p=[password] -s=groven.myshopify.com
request failed after 5 retries with error: Get https://groven.myshopify.com/meta.json: x509: certificate signed by unknown authority
openssl s_client -showcerts -connect groven.myshopify.com:443
Verify return code: 0 (ok)
theme version
ThemeKit 1.1.4 darwin/amd64
MacOS Big Sur
Version 11.1

@MartijnOost, did you fix the above issue?


feat: Use meta tags to control how your website appears in search results

Is your feature request related to a problem? Please describe.
Feature
Describe the solution you'd like
I want to add og meta tags similar to vitess website to improve how website looks when shared
Additional context
Add any other context or screenshots about the feature request here.
P.S - I want to work on it.

Hello @Lucif3r-in thank you for opening this issue. This sounds interesting, feel free to open a Pull Request with the proposed fix and @Camillemtd and I will review it.


Segmentation fault when taking proc dump

Expected behavior
Get the proc dump snapshot
Actual behavior
gnuhpc@gnuhpc-pc:~/app/ProcDump-for-Linux/bin$ sudo ./procdump -p 3151
ProcDump v1.0.1 - Sysinternals process dump utility
Copyright (C) 2017 Microsoft Corporation. All rights reserved. Licensed under the MIT license.
Mark Russinovich, Mario Hewardt, John Salem, Javid Habibi
Monitors a process and writes a dump file when the process exceeds the
specified criteria.
[13:58:02 - ERROR]: Error getting process name.
Process:                (null) (3151)
CPU Threshold:          n/a
Commit Threshold:       n/a
Threshold Seconds:      10
Number of Dumps:        1
Press Ctrl-C to end monitoring without terminating the process.
Segmentation fault
Steps to reproduce the behavior

Start a redis server (empty for test purpose only) and obtain the pid of the server and redis-cli client:

gnuhpc@gnuhpc-pc:~/app/redis-5.0.4/src$ ps -ef |grep redis
gnuhpc    2947  2839  0 13:54 pts/11   00:00:00 redis-cli
gnuhpc    3151  2714  0 13:57 pts/10   00:00:00 ./redis-server <IP_ADDRESS>:6379

proc dump for the server and client

gnuhpc@gnuhpc-pc:~/app/ProcDump-for-Linux/bin$ sudo ./procdump -p 3151
ProcDump v1.0.1 - Sysinternals process dump utility
Copyright (C) 2017 Microsoft Corporation. All rights reserved. Licensed under the MIT license.
Mark Russinovich, Mario Hewardt, John Salem, Javid Habibi
Monitors a process and writes a dump file when the process exceeds the
specified criteria.
[13:58:02 - ERROR]: Error getting process name.
Process:                (null) (3151)
CPU Threshold:          n/a
Commit Threshold:       n/a
Threshold Seconds:      10
Number of Dumps:        1
Press Ctrl-C to end monitoring without terminating the process.
Segmentation fault
gnuhpc@gnuhpc-pc:~/app/ProcDump-for-Linux/bin$ sudo ./procdump -p 2947
ProcDump v1.0.1 - Sysinternals process dump utility
Copyright (C) 2017 Microsoft Corporation. All rights reserved. Licensed under the MIT license.
Mark Russinovich, Mario Hewardt, John Salem, Javid Habibi
Monitors a process and writes a dump file when the process exceeds the
specified criteria.
Process:                redis-cli (2947)
CPU Threshold:          n/a
Commit Threshold:       n/a
Threshold Seconds:      10
Number of Dumps:        1
Press Ctrl-C to end monitoring without terminating the process.
14:01:12 - INFO: Core dump 1 generated: redis_cli_time_2019-04-25_14:01:12.2947

System information (e.g., distro, kernel version, etc.)
Linux distro:

gnuhpc@gnuhpc-pc:~/app/ProcDump-for-Linux/bin$ cat /etc/release
DISTRIB_ID=LinuxMint
DISTRIB_RELEASE=19
DISTRIB_CODENAME=tara
DISTRIB_DESCRIPTION="Linux Mint 19 Tara"
NAME="Linux Mint"
VERSION="19 (Tara)"
ID=linuxmint
ID_LIKE=ubuntu
PRETTY_NAME="Linux Mint 19"
VERSION_ID="19"
HOME_URL="https://www.linuxmint.com/"
SUPPORT_URL="https://forums.ubuntu.com/"
BUG_REPORT_URL="http://linuxmint-troubleshooting-guide.readthedocs.io/en/latest/"
PRIVACY_POLICY_URL="https://www.linuxmint.com/"
VERSION_CODENAME=tara
UBUNTU_CODENAME=bionic

ProcDump for Linux:
using the master branch from the github and make the source code by myself
Kernel version:

gnuhpc@gnuhpc-pc:~/app/ProcDump-for-Linux/bin$ uname -a
Linux gnuhpc-pc 4.15.0-47-generic #50-Ubuntu SMP Wed Mar 13 10:44:52 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Try ProcDumping ProcDump.

Try ProcDumping ProcDump.

@JL2210 What do you mean by saying this?

Expected behavior
Get the proc dump snapshot
Actual behavior
gnuhpc@gnuhpc-pc:~/app/ProcDump-for-Linux/bin$ sudo ./procdump -p 3151
ProcDump v1.0.1 - Sysinternals process dump utility
Copyright (C) 2017 Microsoft Corporation. All rights reserved. Licensed under the MIT license.
Mark Russinovich, Mario Hewardt, John Salem, Javid Habibi
Monitors a process and writes a dump file when the process exceeds the
specified criteria.
[13:58:02 - ERROR]: Error getting process name.
Process:                (null) (3151)
CPU Threshold:          n/a
Commit Threshold:       n/a
Threshold Seconds:      10
Number of Dumps:        1
Press Ctrl-C to end monitoring without terminating the process.
Segmentation fault
Steps to reproduce the behavior
Start a redis server (empty for test purpose only) and obtain the pid of the server and redis-cli client:
gnuhpc@gnuhpc-pc:~/app/redis-5.0.4/src$ ps -ef |grep redis
gnuhpc    2947  2839  0 13:54 pts/11   00:00:00 redis-cli
gnuhpc    3151  2714  0 13:57 pts/10   00:00:00 ./redis-server <IP_ADDRESS>:6379
proc dump for the server and client
gnuhpc@gnuhpc-pc:~/app/ProcDump-for-Linux/bin$ sudo ./procdump -p 3151
ProcDump v1.0.1 - Sysinternals process dump utility
Copyright (C) 2017 Microsoft Corporation. All rights reserved. Licensed under the MIT license.
Mark Russinovich, Mario Hewardt, John Salem, Javid Habibi
Monitors a process and writes a dump file when the process exceeds the
specified criteria.
[13:58:02 - ERROR]: Error getting process name.
Process:                (null) (3151)
CPU Threshold:          n/a
Commit Threshold:       n/a
Threshold Seconds:      10
Number of Dumps:        1
Press Ctrl-C to end monitoring without terminating the process.
Segmentation fault
gnuhpc@gnuhpc-pc:~/app/ProcDump-for-Linux/bin$ sudo ./procdump -p 2947
ProcDump v1.0.1 - Sysinternals process dump utility
Copyright (C) 2017 Microsoft Corporation. All rights reserved. Licensed under the MIT license.
Mark Russinovich, Mario Hewardt, John Salem, Javid Habibi
Monitors a process and writes a dump file when the process exceeds the
specified criteria.
Process:                redis-cli (2947)
CPU Threshold:          n/a
Commit Threshold:       n/a
Threshold Seconds:      10
Number of Dumps:        1
Press Ctrl-C to end monitoring without terminating the process.
14:01:12 - INFO: Core dump 1 generated: redis_cli_time_2019-04-25_14:01:12.2947
System information (e.g., distro, kernel version, etc.)
Linux distro:
gnuhpc@gnuhpc-pc:~/app/ProcDump-for-Linux/bin$ cat /etc/release
DISTRIB_ID=LinuxMint
DISTRIB_RELEASE=19
DISTRIB_CODENAME=tara
DISTRIB_DESCRIPTION="Linux Mint 19 Tara"
NAME="Linux Mint"
VERSION="19 (Tara)"
ID=linuxmint
ID_LIKE=ubuntu
PRETTY_NAME="Linux Mint 19"
VERSION_ID="19"
HOME_URL="https://www.linuxmint.com/"
SUPPORT_URL="https://forums.ubuntu.com/"
BUG_REPORT_URL="http://linuxmint-troubleshooting-guide.readthedocs.io/en/latest/"
PRIVACY_POLICY_URL="https://www.linuxmint.com/"
VERSION_CODENAME=tara
UBUNTU_CODENAME=bionic
ProcDump for Linux:
using the master branch from the github and make the source code by myself
Kernel version:
gnuhpc@gnuhpc-pc:~/app/ProcDump-for-Linux/bin$ uname -a
Linux gnuhpc-pc 4.15.0-47-generic #50-Ubuntu SMP Wed Mar 13 10:44:52 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Hi @gnuhpc , thanks for reporting this issue. I took a look at it and it is actually two-fold. Firstly, the code does lack an error-check against a null process name.  Secondly, the code fetching the process name will be reviewed as well. Because I didn't get a null process name when I launched redis-server by systemctl. However, I did reproduce this issue when I launched redis-server directly from my bash session as a plain daemon.  I'm working on this and I might separate this Open issue to 2 issues for better tracking.

Expected behavior
Get the proc dump snapshot
Actual behavior
gnuhpc@gnuhpc-pc:~/app/ProcDump-for-Linux/bin$ sudo ./procdump -p 3151
ProcDump v1.0.1 - Sysinternals process dump utility
Copyright (C) 2017 Microsoft Corporation. All rights reserved. Licensed under the MIT license.
Mark Russinovich, Mario Hewardt, John Salem, Javid Habibi
Monitors a process and writes a dump file when the process exceeds the
specified criteria.
[13:58:02 - ERROR]: Error getting process name.
Process:                (null) (3151)
CPU Threshold:          n/a
Commit Threshold:       n/a
Threshold Seconds:      10
Number of Dumps:        1
Press Ctrl-C to end monitoring without terminating the process.
Segmentation fault
Steps to reproduce the behavior
Start a redis server (empty for test purpose only) and obtain the pid of the server and redis-cli client:
gnuhpc@gnuhpc-pc:~/app/redis-5.0.4/src$ ps -ef |grep redis
gnuhpc    2947  2839  0 13:54 pts/11   00:00:00 redis-cli
gnuhpc    3151  2714  0 13:57 pts/10   00:00:00 ./redis-server <IP_ADDRESS>:6379
proc dump for the server and client
gnuhpc@gnuhpc-pc:~/app/ProcDump-for-Linux/bin$ sudo ./procdump -p 3151
ProcDump v1.0.1 - Sysinternals process dump utility
Copyright (C) 2017 Microsoft Corporation. All rights reserved. Licensed under the MIT license.
Mark Russinovich, Mario Hewardt, John Salem, Javid Habibi
Monitors a process and writes a dump file when the process exceeds the
specified criteria.
[13:58:02 - ERROR]: Error getting process name.
Process:                (null) (3151)
CPU Threshold:          n/a
Commit Threshold:       n/a
Threshold Seconds:      10
Number of Dumps:        1
Press Ctrl-C to end monitoring without terminating the process.
Segmentation fault
gnuhpc@gnuhpc-pc:~/app/ProcDump-for-Linux/bin$ sudo ./procdump -p 2947
ProcDump v1.0.1 - Sysinternals process dump utility
Copyright (C) 2017 Microsoft Corporation. All rights reserved. Licensed under the MIT license.
Mark Russinovich, Mario Hewardt, John Salem, Javid Habibi
Monitors a process and writes a dump file when the process exceeds the
specified criteria.
Process:                redis-cli (2947)
CPU Threshold:          n/a
Commit Threshold:       n/a
Threshold Seconds:      10
Number of Dumps:        1
Press Ctrl-C to end monitoring without terminating the process.
14:01:12 - INFO: Core dump 1 generated: redis_cli_time_2019-04-25_14:01:12.2947
System information (e.g., distro, kernel version, etc.)
Linux distro:
gnuhpc@gnuhpc-pc:~/app/ProcDump-for-Linux/bin$ cat /etc/release
DISTRIB_ID=LinuxMint
DISTRIB_RELEASE=19
DISTRIB_CODENAME=tara
DISTRIB_DESCRIPTION="Linux Mint 19 Tara"
NAME="Linux Mint"
VERSION="19 (Tara)"
ID=linuxmint
ID_LIKE=ubuntu
PRETTY_NAME="Linux Mint 19"
VERSION_ID="19"
HOME_URL="https://www.linuxmint.com/"
SUPPORT_URL="https://forums.ubuntu.com/"
BUG_REPORT_URL="http://linuxmint-troubleshooting-guide.readthedocs.io/en/latest/"
PRIVACY_POLICY_URL="https://www.linuxmint.com/"
VERSION_CODENAME=tara
UBUNTU_CODENAME=bionic
ProcDump for Linux:
using the master branch from the github and make the source code by myself
Kernel version:
gnuhpc@gnuhpc-pc:~/app/ProcDump-for-Linux/bin$ uname -a
Linux gnuhpc-pc 4.15.0-47-generic #50-Ubuntu SMP Wed Mar 13 10:44:52 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

@gnuhpc Can you check if you have the file of /proc/{YOUR-REDIS-SERVER-PID}/cmdline when your redis-server is running?

I tried and found the cmd in my redis server pid directory of proc.
I think this issue (maybe not a problem) could be reproduced since the redis server can be easily compiled and started. Could you give it a try in your env?
Thanks!

15 10:13 cwd -> /home/gnuhpc/app/redis-5.0.4/src

@gnuhpc Actually I have only been able to reproduce part of your issue. I could reproduce the crash by forcing a null process name and this is easy to fix. But I haven't been able to reproduce the absence of the process name. Can you run procdump again in the same env but with the -d option for diagnostic logs and share the logs here?

I tried but failed to obtain any further logs for this issue.

gnuhpc@gnuhpc-pc:~/app/ProcDump-for-Linux/bin$ ps -ef |grep redis-server
gnuhpc   27679 10050  0 09:55 pts/0    00:00:00 ./redis-server *:6379
gnuhpc   27880 27367  0 09:57 pts/1    00:00:00 grep --color=auto redis-server

gnuhpc@gnuhpc-pc:~/app/ProcDump-for-Linux/bin$ sudo ./procdump -p 27679 -d
ProcDump v1.0.1 - Sysinternals process dump utility
Copyright (C) 2017 Microsoft Corporation. All rights reserved. Licensed under the MIT license.
Mark Russinovich, Mario Hewardt, John Salem, Javid Habibi
Monitors a process and writes a dump file when the process exceeds the
specified criteria.
[09:57:41 - ERROR]: Error getting process name.
Process:		(null) (27679)
CPU Threshold:		n/a
Commit Threshold:	n/a
Threshold Seconds:	10
Number of Dumps:	1
Press Ctrl-C to end monitoring without terminating the process.
Segmentation fault
gnuhpc@gnuhpc-pc:~/app/ProcDump-for-Linux/bin$ sudo ./procdump -d -p 27679
ProcDump v1.0.1 - Sysinternals process dump utility
Copyright (C) 2017 Microsoft Corporation. All rights reserved. Licensed under the MIT license.
Mark Russinovich, Mario Hewardt, John Salem, Javid Habibi
Monitors a process and writes a dump file when the process exceeds the
specified criteria.
[09:57:46 - ERROR]: Error getting process name.
Process:		(null) (27679)
CPU Threshold:		n/a
Commit Threshold:	n/a
Threshold Seconds:	10
Number of Dumps:	1
Press Ctrl-C to end monitoring without terminating the process.
Segmentation fault

@gnuhpc Thanks for your effort and collaborations so far. Can you also share any logs in syslog? I’m asking this because the function of GetProcessName() logs few errors in the level of “debug” which are sent to syslog instead of stdout. Also according to the code, if you don’t have such errors in syslog, it is possible you don’t have the “cmdline” file under your redis-server’s folder under the /proc filesystem, and if you do have errors from GetProcessName() in syslog, it will be helpful to get more insights of your env. I’ve been running procdump with redis-server on Ubuntu 1804 but I haven’t had null process name. I installed redis via apt-get. I haven’t tried compiling redis.

It seems that the procdump couldn't obtain the cmdline of the redis server pid.

Aug 21 09:20:02 gnuhpc-pc ProcDump[18814]: [09:20:02 - DEBUG]: Failed to extract process name from /proc/PID/cmdline
Aug 21 09:20:02 gnuhpc-pc ProcDump[18814]: [09:20:02 - ERROR]: Error getting process name.
Aug 21 09:20:02 gnuhpc-pc ProcDump[18814]: [09:20:02 - DEBUG]: GetOpts and initial Configuration finished in src/ProcDumpConfiguration.c, at line 293
Aug 21 09:20:02 gnuhpc-pc ProcDump[18814]: [09:20:02 - DEBUG]: TimerThread: Starting Trigger Thread in src/TriggerThreadProcs.c, at line 128
Aug 21 09:20:02 gnuhpc-pc ProcDump[18814]: [09:20:02 - INFO]: Timed:
Aug 21 09:20:02 gnuhpc-pc kernel: [297268.298775] procdump[18815]: segfault at 0 ip 00007fb5868865a1 sp 00007fb5866f6178 error 4 in libc-2.27.so[7fb5866f8000+1e7000]
Aug 21 09:20:35 gnuhpc-pc ProcDump[18958]: [09:20:35 - DEBUG]: Failed to extract process name from /proc/PID/cmdline
Aug 21 09:20:35 gnuhpc-pc ProcDump[18958]: [09:20:35 - ERROR]: Error getting process name.
Aug 21 09:20:35 gnuhpc-pc ProcDump[18958]: [09:20:35 - DEBUG]: GetOpts and initial Configuration finished in src/ProcDumpConfiguration.c, at line 293
Aug 21 09:20:35 gnuhpc-pc ProcDump[18958]: [09:20:35 - DEBUG]: TimerThread: Starting Trigger Thread in src/TriggerThreadProcs.c, at line 128
Aug 21 09:20:35 gnuhpc-pc ProcDump[18958]: [09:20:35 - INFO]: Timed:
Aug 21 09:20:35 gnuhpc-pc kernel: [297300.772416] procdump[18959]: segfault at 0 ip 00007f54496725a1 sp 00007f54494e2178 error 4

However, when I checked the proc directory, the cmdline did exist ( and also could be showed using cat command):

gnuhpc@gnuhpc-pc:/proc/27679$ ls -l
total 0
dr-xr-xr-x 2 gnuhpc gnuhpc 0 8月  21 09:22 attr
-rw-r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 autogroup
-r-------- 1 gnuhpc gnuhpc 0 8月  21 09:22 auxv
-r--r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 cgroup
--w------- 1 gnuhpc gnuhpc 0 8月  21 09:22 clear_refs
-r--r--r-- 1 gnuhpc gnuhpc 0 8月  18 09:56 cmdline
-rw-r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 comm
-rw-r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 coredump_filter
-r--r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 cpuset
lrwxrwxrwx 1 gnuhpc gnuhpc 0 8月  21 09:22 cwd -> /home/gnuhpc/app/redis-5.0.4/src
-r-------- 1 gnuhpc gnuhpc 0 8月  21 09:22 environ
lrwxrwxrwx 1 gnuhpc gnuhpc 0 8月  18 09:55 exe -> /home/gnuhpc/app/redis-5.0.4/src/redis-server
dr-x------ 2 gnuhpc gnuhpc 0 8月  21 09:22 fd
dr-x------ 2 gnuhpc gnuhpc 0 8月  21 09:22 fdinfo
-rw-r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 gid_map
-r-------- 1 gnuhpc gnuhpc 0 8月  21 09:22 io
-r--r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 limits
-rw-r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 loginuid
dr-x------ 2 gnuhpc gnuhpc 0 8月  21 09:22 map_files
-r--r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 maps
-rw------- 1 gnuhpc gnuhpc 0 8月  21 09:22 mem
-r--r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 mountinfo
-r--r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 mounts
-r-------- 1 gnuhpc gnuhpc 0 8月  21 09:22 mountstats
dr-xr-xr-x 6 gnuhpc gnuhpc 0 8月  21 09:22 net
dr-x--x--x 2 gnuhpc gnuhpc 0 8月  21 09:22 ns
-r--r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 numa_maps
-rw-r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 oom_adj
-r--r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 oom_score
-rw-r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 oom_score_adj
-r-------- 1 gnuhpc gnuhpc 0 8月  21 09:22 pagemap
-r-------- 1 gnuhpc gnuhpc 0 8月  21 09:22 patch_state
-r-------- 1 gnuhpc gnuhpc 0 8月  21 09:22 personality
-rw-r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 projid_map
lrwxrwxrwx 1 gnuhpc gnuhpc 0 8月  21 09:22 root -> /
-rw-r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 sched
-r--r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 schedstat
-r--r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 sessionid
-rw-r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 setgroups
-r--r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 smaps
-r--r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 smaps_rollup
-r-------- 1 gnuhpc gnuhpc 0 8月  21 09:22 stack
-r--r--r-- 1 gnuhpc gnuhpc 0 8月  18 09:55 stat
-r--r--r-- 1 gnuhpc gnuhpc 0 8月  18 09:56 statm
-r--r--r-- 1 gnuhpc gnuhpc 0 8月  18 09:56 status
-r-------- 1 gnuhpc gnuhpc 0 8月  21 09:22 syscall
dr-xr-xr-x 6 gnuhpc gnuhpc 0 8月  21 09:22 task
-r--r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 timers
-rw-rw-rw- 1 gnuhpc gnuhpc 0 8月  21 09:22 timerslack_ns
-rw-r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 uid_map
-r--r--r-- 1 gnuhpc gnuhpc 0 8月  21 09:22 wchan
gnuhpc@gnuhpc-pc:/proc/27679$ cat cmdline
./redis-server *:6379gnuhpc@gnuhpc-pc:/proc/27679$

would you try to use the compiled redis-server to reproduce the issue, pls?

@gnuhpc The new logs from you actually help a lot. Can you please do a "hexdump -C cmdline" and share the outputs?

gnuhpc@gnuhpc-pc:/proc/27679$ hexdump -C cmdline
00000000  2e 2f 72 65 64 69 73 2d  73 65 72 76 65 72 20 2a  |./redis-server *|
00000010  3a 36 33 37 39                                    |:6379|
0000001

@gnuhpc Frome the hexdump output, the content in your cmdline is not null-terminated. I'm working on this to make sure ProcDump always append a null-terminator to the fetched content from cmdline, and once I finish my work I'll ask you to try my branch in your env. Another thing is that I compiled and tried the version 4.0 of redis-server, and I didn't get the null process name. I didn't try 5.0 because it doesn't appear to be stable since it didn't pass all tests from make test.

@nikkitan I haven't given redis 4.x a try due to the fact that the version 5.x is the latest stable version of Redis according to the page: https://redis.io/download.
Actually, I skipped the make test before starting the redis server.
Anyway, keep up your good work and also keep in touch if needed.

@gnuhpc Please try this branch in your env and keep us updated here: https://github.com/nikkitan/ProcDump-for-Linux/tree/nullify_cmdline_buffer

You did fix the null name isue while dump trigger.... Please check the dump file at :https://wetransfer.com/downloads/8338655d5d6ccd54941ad5c98e77aa1820190822143756/3d3290dcda15342e5808df1ca3ba69eb20190822143756/75167d

gnuhpc@gnuhpc-pc:~/app/ProcDump-null/ProcDump-for-Linux/bin$ sudo ./procdump -p 27679
ProcDump v1.0.1 - Sysinternals process dump utility
Copyright (C) 2017 Microsoft Corporation. All rights reserved. Licensed under the MIT license.
Mark Russinovich, Mario Hewardt, John Salem, Javid Habibi
Monitors a process and writes a dump file when the process exceeds the
specified criteria.
Process:		redis-server *:6379 (27679)
CPU Threshold:		n/a
Commit Threshold:	n/a
Threshold Seconds:	10
Number of Dumps:	1
Press Ctrl-C to end monitoring without terminating the process.
[22:35:34 - INFO]: Core dump 1 generated: redis_server___6379_time_2019-08-22_22:35:33.27679
gnuhpc@gnuhpc-pc:~/app/ProcDump-null/ProcDump-for-Linux/bin$ ls -alrt
total 35740
drwxrwxr-x 10 gnuhpc gnuhpc     4096 8月  22 22:35 ..
-rwxrwxr-x  1 gnuhpc gnuhpc   107232 8月  22 22:35 procdump
-rwxrwxr-x  1 gnuhpc gnuhpc    11144 8月  22 22:35 ProcDumpTestApplication
drwxrwxr-x  2 gnuhpc gnuhpc     4096 8月  22 22:35 .
-rw-r--r--  1 root   root   36463368 8月  22 22:35 redis_server___6379_time_2019-08-22_22:35:33.27679

@gnuhpc Thanks for the confirmation and I'm glad I can help. I will integrate the fix into master and we will release it. Please don't forget to close this issue whenever you are fully unblocked from this issue by the new release.

Thanks for your tracking! You could close this issue as problem resoloved after the merge
to master action.


allow filtering on all event types

It would be useful if this could filter based on changes to HEAD. I've ran into patterns where I want to send dispatches to start other workflows for syncing translations or uploading changed assets. But those jobs cannot use this because it gets the error: This action can be triggered only by pull_request or push event
Thanks for the work!

I'm not exactly sure what do you mean by changes to HEAD.
Anyway I could make it work for any event the same way as it currently works for push events:

For feature branches changes would be detected against the merge-base with base branch
For change detection since previous push you could pass the "before" commit SHA using workflow_dispatch inputs and use it as a base parameter for this action.
As a fallback if there is no base branch or SHA of previously pushed commit, it would consider only changes in the last commit

Would this work for you?

yea. that'd be awesome. and all those sound really useful as well (especially the before commit one).
for more context on what I'm trying to do. I'm starting to split out workflows because the jobs are getting really large. I'm sending a couple repository dispatch events every time a merge happens to master. So, I really just want to check for changes on the last commit on the current branch that is checked out. sorry, it's been a long day so I hope I'm still making sense :)

Ok, so the idea is clear now. Implementing the changes should be quite easy. I plan to do it later today.

Should work now. I tested it with example bellow.
Please reopen this issue if you will still run into some issues.

Thanks for the work!

Consider starring the repo :)
Example:
First workflow which triggers the second one:
name: "Dispatch"
on:
  push:
    branches:
      - master

jobs:
  build:
    runs-on: ubuntu-latest
    steps:
    - name: Invoke workflow with inputs
      uses: benc-uk/workflow-dispatch@v1
      with:
        workflow: PostBuild
        token: ${{ secrets.PERSONAL_TOKEN }}
        inputs: '{ "before": "${{ github.event.before }}" }'

Follow up workflow:
name: "PostBuild"
on:
  workflow_dispatch:
    inputs:
      before:
        description: 'SHA of last previously pushed commit'
        required: true

jobs:
  post-build:
    runs-on: ubuntu-latest
    steps:
    - uses: actions/checkout@v2
    - uses: dorny/paths-filter@v2
      id: filter
      with:
        base: ${{ github.event.inputs.before }}
        filters: ...


error when running truffle test

C:\azure-asset-transfer-box>truffle test
Error: ENOENT: no such file or directory, stat 'C:\azure-asset-transfer-box\test'

?

Looking into this now.

Thanks for the feedback here, I am adding tests for each of these azure boxes.  Since the file was not there, the folder doesn't show up and truffle will show this error.  I will get these checked into the repos asap to resolve this.


Optimize the QpMap processing and add a compute filter to the encoder.

Optimize the QpMap to use inlined command buffer transfer operations or direct handling of the images.
Add a compute filter as a preprocessor for the encoder to do last row and last column replication, color conversion, and scalling.

Reviewed and verified by Raju Konda.


Why does updateFrame only fetch at most one byte per invocation?

Thank you for providing this repo. I have noticed that updateFrame only fetches, at most, one data byte from the sensor per call when, potentially, there could be more than one. I tried changing line 41 of Planter_PMS7003.cpp from:
if(serial->available()) {
to:
while (serial->available() && bufferIndex < PMS7003_DATA_SIZE) {
It all works well in my project but I was wondering if I have missed something and there is a reason why it is thus limited?

Your recommendation looks good to me. If you've tested it, go ahead and submit a pull request.

Thank you, I've created a PR for this.

Merged - thanks!


Add GetInputs() and GetOutputs() functions

Thanks so much for creating this repo! It's worked really well for running my ONNX models from Go.
I was wondering if convenience functions could be added to make it easier to know what the input/output tensor shapes of a model are. With the way things are now, I build a new Go program for every ONNX model I have, since the NewSession function has to have the input and output tensors from the beginning.
I'm not sure if it would be easier to create something like GetInputs(modelPath) and 'quickly' create a session just to get the input tensor's name/shape/type, or maybe there is a reason you wouldn't want to include a function like this in the module.

Thank you, and I'm happy this project has helped!
Unless I'm misunderstanding your question, I already have a function like that in the module: GetInputOutputInfo, which returns two slices: one containing information for each input and one containing information for each output.  Each input or output's name, size, and type is provided in a corresponding InputOutputInfo struct.  I added this relatively recently, in response to issue #31.
For example:
inputs, outputs, err := onnxruntime_go.GetInputOutputInfo("path/to/file.onnx")
if err != nil {
    // ...
}

for i, input := range inputs {
    fmt.Printf("Input %d: name=%s, shape=%s, type=%s\n", i, input.Name, input.Dimensions, input.DataType)
}

// And outputs can be checked similarly.

I hope this is what you're looking for; let me know if I'm not understanding!

Awesome! Thanks, I didn't see it in the documentation here, so didn't know it was available https://pkg.go.dev/github.com/yalue/onnxruntime#section-readme
It was exactly what I was looking for


Dynamic Step Component Example

Do you have any example of a Dynamic Step component, which brings data from REST API and injects  them into the react native chat bot?
Thanks!

@LucasBassetti any update on this?

please give some ideas how to use the rest API in my chat bot and waiting for your reply


Sort out task formatting

#776  👈
main

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

Join @alanvardy and the rest of your teammates on  Graphite


[🚀esl-forms]: (Draft) base form / forms element

As an ESL architect, I want to have a base form component + item to have extended control over native HTML Forms
Features:

async form controller (to create ajax based forms)
built-in extended validators base

Details: TBD
Priority: Minor (for now)

@exadel-inc/esl-core-team, @andreybelous feel free to add an additional details for this epic


tour: Slice defaults

Context: http://<IP_ADDRESS>:3999/moretypes/10
func main() {
s := []int{2, 3, 5, 7, 11, 13}
s = s[1:4]
fmt.Println(s)

s = s[0:3]
fmt.Println(s)

RESULT IS GIVING
[3 5 7]
[3 5 7]
WAT ???
SOULD BE
3 5 6
2 3 5
WOOOOOOOOOO
FORGET THATTTTT
THAT GUY JUST UPDATED THE S
WOOOOOOOOOOO
When you lose  the last ones is not LOST
but when you lose the first ones.. its lost..
s := []int{2, 3, 5, 7, 11, 13}
s = s[0:1]
fmt.Println(s)

s = s[1:2]
fmt.Println(s)

s = s[0:5]
fmt.Println(s)
	s = s[:]
fmt.Println(s)

RunFormatReset
[2]
[3]
[3 5 7 11 13]
[3 5 7 11 13]
ow.. an THIS is how its supposed to work.. ..ok..

Unclear if you are reporting a tour bug... but reading the issue it seems like you solved your doubts yourself so I'm closing here.


showprintbutton does not seem to work in Firefox 38

I can't easily roll back to Firefox 37 on my Archlinux system, so I am not 100% sure.
Though http://build.webconverger.org/webc-0fa21c1.iso aka https://github.com/Webconverger/webc/commit/0fa21c157325c9cb7ad1a34b6e90495596166dee I can confirm is missing the icon.

https://github.com/Webconverger/webconverger-addon/pull/55


from binary (elf file) to LLVM bitcode

Hi,
I would like to know if I can use this tool to generate the llvm bitcode from a binary file?if yes could you please tell me how.
thanks.

Yes, it does code lifting to do it's job, but you'll have to figure it out...


Logout must not delete user

I recently implemented deletion of users during logout, but that breaks name lookup from login IDs.
A proper way (which may be overkill in the end?) would be to

remove users only if they are not part of any tables anymore
mark users as logged out otherwise
remove tables if all users are logged out (or have left the table)

This issue is mostly tackled, with the introduction of reference-counted users. Deleting tables seems to be a different issue.


HTMLElement.shadowRoot is deprecated.

The contents of atom-text-editor elements are no longer encapsulated
within a shadow DOM boundary. Please, stop using shadowRoot and access
the editor contents directly instead.
HTMLElement.shadowRoot (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor-element.js:13:10)
MarkdownScrlSync.getVisTopHgtBot (/Users/jiayiguan/.atom/packages/markdown-scroll-sync/lib/utils.coffee:20:33)
MarkdownScrlSync.setMap (/Users/jiayiguan/.atom/packages/markdown-scroll-sync/lib/map.coffee:25:14)
MarkdownScrlSync.startTracking (/Users/jiayiguan/.atom/packages/markdown-scroll-sync/lib/main.coffee:75:12)
<unknown> (/Users/jiayiguan/.atom/packages/markdown-scroll-sync/lib/main.coffee:58:23)
Function.module.exports.Emitter.simpleDispatch (<embedded>:64158:20)

duplicate of #94


Related from two different source_code

From our chat on discord - I'm trying to create an error that has a label+span pointing at one source file, and then a separate label+span pointing into a custom source_code.
Here's how I'm trying to do it:
Defining the error (ShellError is Nushell's built-in error type):
    #[error("{0}")]
    #[diagnostic()]
    SpannedLabeledErrorRelated(
        String,
        String,
        #[label("{1}")] Span,
        #[related] Vec<ShellError>,
    ),

How the error gets built:
            return Err(ShellError::SpannedLabeledErrorRelated(
                "error when loading".into(),
                "could not load file".into(),
                head,
                vec![ShellError::OutsideSpannedLabeledError(
                    string_input,
                    "error when loading".into(),
                    err.to_string(),
                    err.span(),
                )],
            ));

Only seeing part of the error, without any related errors:
Error:
  × error when loading
   ╭─[entry #2:1:1]
 1 │ "{x: 100" | from nuon
   ·             ────┬────
   ·                 ╰── could not load file
   ╰────

User error: I wasn't fully implementing miette::Diagnostic so related was coming back with None


Send confirmation screen - text fixes

Change "." to "," between "Regular, permanent"
Current implementation (testnet)

Design

checked

checked


[doc] Update dependency document

Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

Codecov Report
Base: 72.08% // Head: 71.22% // Decreases project coverage by -0.86% :warning:

Coverage data is based on head (eb70c2c) compared to base (bb5073f).
Patch coverage: 71.46% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2134      +/-   ##
============================================
- Coverage     72.08%   71.22%   -0.87%
- Complexity     5126     6276    +1150
============================================
  Files           473      624     +151
  Lines         21970    27847    +5877
  Branches       2351     3004     +653
============================================
+ Hits          15838    19835    +3997
- Misses         4925     6550    +1625
- Partials       1207     1462     +255

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

...rc/main/java/ai/djl/modality/cv/output/Joints.java
71.42% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...main/java/ai/djl/modality/cv/output/Rectangle.java
72.41% <0.00%> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

.../modality/cv/translator/ImageFeatureExtractor.java
0.00% <0.00%> (ø)

.../ai/djl/modality/cv/translator/YoloTranslator.java
27.77% <0.00%> (+18.95%)
:arrow_up:

...modality/cv/translator/wrapper/FileTranslator.java
44.44% <ø> (ø)

...y/cv/translator/wrapper/InputStreamTranslator.java
44.44% <ø> (ø)

... and 559 more

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

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


Support for Carthage

It would be cool if you could add support for Carthage.
https://github.com/Carthage/Carthage#supporting-carthage-for-your-framework

Ok... I don't see a pull request. You can make the appropriate changes in a fork and I will be happy to add them.

https://github.com/mattstevens/RoutingHTTPServer/pull/21


Remove outdated nsprc

Fixes #681

:squint: Hard to see how those travis failures would be related to this change.
Is this based on an up-to-date master?

@gigabo I merged up from master, rebased, and force pushed. 🍀


partitioned write support

Todo

[x] support partitioned append()

[x] support append with identity transform
[x] fix scenario when arrow table schema not aligned with iceberg schema (finished by others)
[x] add integration test for null column partitioning after issue#348 is closed
[x] avoid sorting input arrow table when it is already sorted

[x] support partitition field in manifest file see PartitionKey
[x] apply transform for partitioning algorithm efficiency analysis when transform involved
[x] support partitioned static overwrite()

[x] overwrite entire table
[x] overwrite with expression or filter string (specified partition)
[ ] overwrite filter validatoin (https://github.com/jqin61/iceberg-python/pull/4)

[x] extend summary for partitioned stats (https://github.com/apache/iceberg-python/pull/521)
[x] support partitioned dynamic overwrite()

As discussed in the monthly meeting, this will be broken down into 4 prs of:

Partitioned append with identity transform
Dynamic overwrite using delete + append, 2 snapshots in one commit
Hidden partitioning support (for slicing the arrow table, manifest file entry.partition, data file path)
Static overwrite using delete + append, 2 snapshots in one commit


Cleanup to remove code causing "more than one element found for locat…

…or" warnings - also removed unnecessary function from page object

Can one of the admins verify this patch?

@ldimaggi add to whitelist

[test]

[test]

merged


Swift 2 Support and major refactoring

TODO

[ ] Add skeletons for other Keychain classes that might be needed in future (Passwords)
[ ] Get rid of Keychain.swift
[ ] Complete KeychainCertificate
[ ] Complete KeychainIdentity
[ ] Add README that describes the design and usage
[ ] Add License
[ ] Rename Repository to final name before publishing
[ ] Test on an actual phone
[ ] Remove old commented code that currently serve as documentation

This is a major rewrite because the previous attempts didn't work. I used Swift 2, protocol extensions and exceptions (throws) throughout which made the implementation much saner.
This PR is in a state where I think it can be reviewed.
General Design
The purpose of this Library is to add the infrastructure for Client Side authentication into IOS Apps. This means that you need to deal with the IOS Keychain. Apple's APIs are severely limited and do not provide support for generating certificate signing requests (CSRs) at all. To do this, the library uses OpenSSL.
The library provides Object Oriented wrappers for the IOS Keychain Objects and hide the complexities of dealing with the Keychain API in IOS.
The classes have been designed so that it should be relatively straightforward to extend the Library to also provide OO wrappers for OpenSSL objects that implement the same protocols as the Keychain Objects in this library.
For security purposes, we decided not to use OpenSSL's functions for generating Key Pairs, because it uses a software Pseudo Random Generator, whereas Apple's APIs uses hardware security. For this reason KeychainKeyPair provides mechanism to export raw keys from the Keychain so that they can be used in OpenSSL. This is potentially something that should be improved, because the actual cryptographic data is in the clear in memory in these objects. The correct way to do it is to generate a storage key when the Library is instantiated (using SecKeyGenerate) and then immediately encrypt the output of SecItemCopyMatching(...) under this key and then store the Key Data encrypted in Memory. The encrypted keys should be passed to the OpenSSL wrapper and only be decrypted right before they are needed. This will prevent anyone from inspecting the memory and extracting the sensitive data. (The Storage Key will be in the IOS Keychain, which is a secure container on the iPhone and iPad).
KeychainItem
The basic design is this. The Apple keychain uses Dictionaries for the input and output of the four SecXYZ functions. It can store 5 types of items (Keys, two kinds of Passwords, Identities and Certificates).
Apple's secItemCopyMatching(...) API returns the details of things in the IOS keychain. These things are instances of KeychainItem:

KeychainKey
KeychainKeyPair
KeychainCertificate
KeychainIdentity
These classes represent actual items in the IOS Keychain.

KeychainItem has a Security Class and a dictionary of Keychain Attributes.
KeychainDescriptor
KeychainDescriptor is the base class for the data needed for IOS Keychain queries. This class also has a Security Class and a Dictionary of Keychain Attributes.
Transport Classes

TransportKeyPair
TransportPublicKey
TransportPrivateKey
TransportCertificate
TransportIdentity

Importing an Identity
In order to add something (like a PKCS12 Identity from a .p12 file), you first have to import it. This you do with a call to SecPKCS12Import(...). This returns a temporary reference to an Identity, but it is not stored in the keychain yet. In order to add it, the code uses objects named TransportXYZ to store these temporary values. In the case of an identity, this class is called TransportKeyPair. TransportKeyPair supports the KeychainAddable protocol, which means it has a method addToKeychain() -> KeychainIdentity. The Transport classes derive from KeychainDescriptor.
OpenSSL
This Library uses a precompiled OpenSSL library to speed up compile time. This is not a good thing, because someone can replace the library with hacked code and we will not have any way to know that the code has been compromised.
The code uses OpenSSL to do the following:

Generating a Certificate Signing Request (CSR) with a Key Pair stored in the IOS Keychain:

public class func generateCSRWithKeyPair(keyPair: IOSKeychain.OpenSSLKeyPair, csrData: [NSObject : AnyObject]) throws -> NSData

Importing an RSA Key Pair from a passphrase protected PEM file:

public class func keyPairFromPEMData(pemData: NSData, encryptedWithPassword passphrase: String) throws -> IOSKeychain.OpenSSLKeyPair

Convert a certificate received from a certificate authority into a PKCS12 Identity that can be added to the IOS Keychain:

public class func pkcs12IdentityWithKeyPair(keyPair: IOSKeychain.OpenSSLKeyPair, certificate: IOSKeychain.OpenSSLCertificate, protectedWithPassphrase passphrase: String) throws -> IOSKeychain.OpenSSLIdentity

Keychain Protocols
KeyChainAttributeStorage
public protocol KeyChainAttributeStorage {
    var attributes : [String : AnyObject] { get }
    subscript(attribute: String) -> AnyObject? { get }
}

KeychainMatchable
Indicates that items in the IOS keychain of securityClass can matched with keychainMatchPropertyValues()
Returns the properties to pass to SecItemCopyMatching() to match IOS keychain items wit
public protocol KeychainMatchable {
    func keychainMatchPropertyValues() -> [ String: AnyObject ]
    var securityClass: SecurityClass { get }
}

KeychainFindable
Marks that the item provides a findInKeychain() method that returns matching keychain items
public protocol KeychainFindable {
    typealias QueryType : KeychainMatchable
    typealias ResultType : KeychainItem
    static func findInKeychain(matchingProperties: QueryType) throws -> ResultType?
}

KeychainAddable
Indicates that the item can be added to the IOS keychain
public protocol KeychainAddable {
    typealias KeychainClassType : KeychainItem, KeychainFindable
    func addToKeychain() throws -> KeychainClassType?
}

SecItemAddable
Generic Protocol to mark that the item can be added to the IOS Keychain
public protocol SecItemAddable {
    func secItemAdd() throws -> AnyObject?
}

Obsolete.


Proposal #43947 was accepted and implemented for this instead.

Proposal #43947 was accepted and implemented for this instead.
Originally posted by @bcmills in https://github.com/golang/go/issues/38736#issuecomment-1179032113

I'm not sure how you're producing the issues like this one and https://github.com/golang/groupcache/issues/160, but please stop. This is not an appropriate way to communicate on an issue tracker.

@bcmills It's on the 3-dot menu of a comment, then "reference in new issue".


Create a new branches

please create the 2 parent branches and feature branches as well
dev
dev-backend
dev-backend-feature/wallet
dev-backend-feature/keysharding
@cb7chaitanya

Created

Created, closing as completed


ParquetSampler slow for large files

Seems to be a design choice in InternalParquetRecordReader so hard to solve unless we fork that logic.
https://github.com/apache/parquet-mr/blob/master/parquet-hadoop/src/main/java/org/apache/parquet/hadoop/InternalParquetRecordReader.java#L190


[RCx] After update from Beta11 to RCx, i can't run my app on mobile

From @rodriguesgm on November 3, 2016 22:51
Hi all
So, I spend the last days working on the upgrade to RC0, RC1 and RC2. I had to change all my .js files to .ts files, and some others problems, but finally I could run ionic serve and test my app on a browser.
But I tried to run ionic run android and it didn't work. This is what I'm getting on the console:

To avoid this problem, on ./platform/android/build.gradle, I added the following line
...
android {
...
defaultConfig {
multiDexEnabled true
...
}
}
This made the ionic run android to work, but when I try to launch the app on my mobile it just doesn't start and show the message "The app stop working".
Does anyone have an idea what it could be?
Copied from original issue: driftyco/ionic#9017

From @apparition47 on November 4, 2016 18:33
It's failing because you need to increase your heapsize in your gradle file.
Create a /platforms/android/build-extras.gradle, then add:
android {
    dexOptions {
        jumboMode = true
        javaMaxHeapSize "4g"
    }
}

From @apparition47 on November 5, 2016 18:0
If all those plugins worked before your port from beta.11, I don't think that's the issue. Did you try clearing your /platforms/android/build/ folder before re-building the apk? If that doesn't work, try asking the folks on the Slack #ionic-v2 channel.

From @rodriguesgm on November 4, 2016 21:16
@apparition47 thanks for the help.
The problem is that even adding the config that I said or adding your config, after build succeed and I try to launch the app on my mobile I get the error "The app stop working". And I have no way to see what happened. My app just doesn't open.
could it be because of some plugin that is not compatible with Ionic2 RCx? Because my app is working fine with beta11.
Here is the list of plugins I'm using:
cordova-plugin-console 1.0.3
cordova-plugin-whitelist 1.2.2
cordova-plugin-splashscreen 3.2.2
cordova-plugin-statusbar 2.1.3
ionic-plugin-keyboard 2.2.1
cordova-plugin-facebook4 1.7.1
cordova-plugin-camera 2.2.0
cordova-plugin-file 4.2.0
cordova-plugin-file-transfer 1.5.1
cordova-plugin-filepath 1.0.2
cordova-plugin-apprate 1.1.12
phonegap-plugin-push 1.7.4
cordova-plugin-network-information 1.2.1
cordova-plugin-datepicker 0.9.3
cordova-plugin-google-analytics 1.5.6
cordova-plugin-intercom 3.0.8
cordova-plugin-geolocation 2.2.0

From @rodriguesgm on November 6, 2016 1:18
@apparition47 yep, I tried that. I tried remove the android platform, clean all plugins and install them again too.
thanks again

ok, the problem is some plugin... I removed all of then and the ionic run android worked.... now I'll add one by one to see where is the problem

after remove all plugins and add one by one with the latest version of each one, the problem was mysteriously solved..
@jgw96 I think we can close this issue


skim: 0.5.0 -> 0.5.1

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/skim/versions.
meta.description for skim is: '"Fuzzy Finder in rust!"'.
Release on GitHub
Compare changes on GitHub

Checks done (click to expand)

built on NixOS
/nix/store/yfiwj1h379igkrc163ylk0d3cr62iqww-skim-0.5.1/bin/sk-share passed the binary check.
/nix/store/yfiwj1h379igkrc163ylk0d3cr62iqww-skim-0.5.1/bin/sk passed the binary check.
/nix/store/yfiwj1h379igkrc163ylk0d3cr62iqww-skim-0.5.1/bin/sk-tmux passed the binary check.
3 of 3 passed binary check by having a zero exit code.
2 of 3 passed binary check by having the new version present in output.
found 0.5.1 with grep in /nix/store/yfiwj1h379igkrc163ylk0d3cr62iqww-skim-0.5.1
directory tree listing: https://gist.github.com/0a88bc5d50e279349923b0faab4eeecc
du listing: https://gist.github.com/3046484f8cd48b1f59a7ab5c53b20f9a

Test this build with Cachix (click to expand)

One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout 0b098a79bc8d4939046485383fb0968314803664 && nix-shell --pure -I nixpkgs=. -p skim

cc @dywedir for testing.

Success on x86_64-linux (full log)
Attempted: skim
Partial log (click to expand)
strip is /nix/store/1hi76hr87bd1y1q1qjk0lv8nmcjip1c8-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/fyv7la1nycf726yb08wk6pp78rgjaicl-skim-0.5.1/bin
patching script interpreter paths in /nix/store/fyv7la1nycf726yb08wk6pp78rgjaicl-skim-0.5.1
/nix/store/fyv7la1nycf726yb08wk6pp78rgjaicl-skim-0.5.1/bin/sk-tmux: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/dkh7l9a4sx7zqh8riqbj3z21sz25p8xy-bash-4.4-p23/bin/bash"
checking for references to /build in /nix/store/fyv7la1nycf726yb08wk6pp78rgjaicl-skim-0.5.1...
shrinking RPATHs of ELF executables and libraries in /nix/store/c6s2ygxzk4afd15f1dj45hlkz4d09jsr-skim-0.5.1-vim
strip is /nix/store/1hi76hr87bd1y1q1qjk0lv8nmcjip1c8-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/c6s2ygxzk4afd15f1dj45hlkz4d09jsr-skim-0.5.1-vim
checking for references to /build in /nix/store/c6s2ygxzk4afd15f1dj45hlkz4d09jsr-skim-0.5.1-vim...
/nix/store/fyv7la1nycf726yb08wk6pp78rgjaicl-skim-0.5.1

Failure on aarch64-linux (full log)
Attempted: skim
Partial log (click to expand)
'+fp' is not a recognized feature for this target (ignoring feature)
'+fp' is not a recognized feature for this target (ignoring feature)
LLVM ERROR: IO failure on output stream: No space left on device
error: Could not compile `bootstrap`.

To learn more, run the command again with --verbose.
failed to run: /nix/store/sc1kbb2p9dcd9h5ba5j7p7rrw8afr7q8-rustc-bootstrap-1.26.2/bin/cargo build --manifest-path /build/rustc-1.27.0-src/src/bootstrap/Cargo.toml --frozen
Build completed unsuccessfully in 0:02:23
make: *** [Makefile:28: all] Error 1
error: unexpected end-of-file

Success on x86_64-darwin (full log)
Attempted: skim
Partial log (click to expand)

installing
post-installation fixup
strip is /nix/store/qfxcr8c4fg7lkybrny9n2sb77bmippdx-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/6ksq9r34j1535y4r1ja2w68azak9xsfi-skim-0.5.1/bin
patching script interpreter paths in /nix/store/6ksq9r34j1535y4r1ja2w68azak9xsfi-skim-0.5.1
/nix/store/6ksq9r34j1535y4r1ja2w68azak9xsfi-skim-0.5.1/bin/sk-tmux: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/qb76s7s3p6pnsi7082sjfc7rb64f3s6a-bash-4.4-p23/bin/bash"
strip is /nix/store/qfxcr8c4fg7lkybrny9n2sb77bmippdx-cctools-binutils-darwin/bin/strip
patching script interpreter paths in /nix/store/v16x04nd6fdmfhqqyjs88ij1yv0gcg5f-skim-0.5.1-vim
/nix/store/6ksq9r34j1535y4r1ja2w68azak9xsfi-skim-0.5.1

Success on x86_64-linux (full log)
Attempted: skim
Partial log (click to expand)
strip is /nix/store/gpc2wld1s0c6qzx9326cwn1wcx29xzsj-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/ns6v3wkahla57jk81fx5a48vnxsa9am7-skim-0.5.1/bin
patching script interpreter paths in /nix/store/ns6v3wkahla57jk81fx5a48vnxsa9am7-skim-0.5.1
/nix/store/ns6v3wkahla57jk81fx5a48vnxsa9am7-skim-0.5.1/bin/sk-tmux: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/56nrxy58wbhvs2sy3rir1jqa68p0kkm5-bash-4.4-p23/bin/bash"
checking for references to /build in /nix/store/ns6v3wkahla57jk81fx5a48vnxsa9am7-skim-0.5.1...
shrinking RPATHs of ELF executables and libraries in /nix/store/a9i64y17i4zxmmhjibfd3qq24s36pljp-skim-0.5.1-vim
strip is /nix/store/gpc2wld1s0c6qzx9326cwn1wcx29xzsj-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/a9i64y17i4zxmmhjibfd3qq24s36pljp-skim-0.5.1-vim
checking for references to /build in /nix/store/a9i64y17i4zxmmhjibfd3qq24s36pljp-skim-0.5.1-vim...
/nix/store/ns6v3wkahla57jk81fx5a48vnxsa9am7-skim-0.5.1

Success on aarch64-linux (full log)
Attempted: skim
Partial log (click to expand)
strip is /nix/store/ah0va6j4cnwj9nx4j6rwcfc8nh785jwm-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/plvs5jb6mldpj4nwplbf1pw77dj7ns16-skim-0.5.1/bin
patching script interpreter paths in /nix/store/plvs5jb6mldpj4nwplbf1pw77dj7ns16-skim-0.5.1
/nix/store/plvs5jb6mldpj4nwplbf1pw77dj7ns16-skim-0.5.1/bin/sk-tmux: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/7q3ayirslrcva28wava6zpjcflcz1h3b-bash-4.4-p23/bin/bash"
checking for references to /build in /nix/store/plvs5jb6mldpj4nwplbf1pw77dj7ns16-skim-0.5.1...
shrinking RPATHs of ELF executables and libraries in /nix/store/v99p7hnv1zyqc7fh4v517x1w9cnk5pf8-skim-0.5.1-vim
strip is /nix/store/ah0va6j4cnwj9nx4j6rwcfc8nh785jwm-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/v99p7hnv1zyqc7fh4v517x1w9cnk5pf8-skim-0.5.1-vim
checking for references to /build in /nix/store/v99p7hnv1zyqc7fh4v517x1w9cnk5pf8-skim-0.5.1-vim...
/nix/store/plvs5jb6mldpj4nwplbf1pw77dj7ns16-skim-0.5.1


Variable function argument count

suppose that I want to have a function which grabs one named parameter but is using rest (unknown / unlisted args in method getArgumentNames) to build query string e.g
{{ route(action='action1', params={'a': 'b', 'c': 'd'}) }} -> '/my/action1?a=b&c=d'
but I want to achieve syntax simplification:
{{ route(action='action2', a='b', c='d') }}
I failed while implementing it. Is it even possible? I noticed error "'a' is not a parameter' or sth like that.

Named varargs in functions is not supported, and AFAIK is not on the planned feature list. If you want a function with a variable number of arguments, only positional access is allowed. The recommended way is the same that you got working (using maps).

@pun-ky
Did you find a solution? I have the same request for Pippo.

I don't know why, but first variant (comment) doesn't work and the second variable works.
What I want to do in my RouteFunction is to capture the arguments and to call Router.uriFor(String routeName, Map<String, Object>parameters).
{#
{{ stock.symbol }}
#}
{{ stock.symbol }}

My RouteFunction looks like:
class RouteFunction implements Function {

    private Router router;

    public RouteFunction(Router router) {
        this.router = router;
    }

    @Override
    public List<String> getArgumentNames() {
        List<String> names = new ArrayList<>();
        names.add("routeName");
        names.add("parameters");

        return names;
    }

    @Override
    public Object execute(Map<String, Object> args, PebbleTemplate self, EvaluationContext context, int lineNumber) {
        String routeName = (String) args.get("routeName");
        Map<String, Object> parameters = (Map<String, Object>) args.get("parameters");

        return router.uriFor(routeName , parameters);
    }

}

Am I doing something wrong?

Remove {{ }} in your call to route function around stock symbol

@ebussieres
Thanks!
It's 1:30 AM in Bucharest and maybe it's to late :)
Do you think that the approach is good, and the route function must work with multiple parameters (for example route('quote', {'symbol': stock.symbol, ''date': date, ... }))?

@ebussieres
Thanks!
It was 1:30 AM in Bucharest and maybe was to late :)
Do you think that the approach is good, and the route function must work with multiple parameters (for example route('quote', {'symbol': stock.symbol, ''date': date, ... }))?


[MLIR][Lite] Update emit_quant_adaptor in PostQuantizePass

Make separate quant-adaptors for inputs and outputs of model in PostQuantizePass.

@Vooblin Can you please address reviewer comments and keep us posted? Thanks!

It has been 14 days with no activity and the awaiting response label was assigned. Is this PR still valid? Assigning the stalled label. Please comment to reassure me that this is still being worked on.

I'm going to go ahead and close this PR, because it seems to have stalled. If you're still interested in pursing this (and responding to my comments), please feel free to reopen!


chore: update docker use of extensions

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)

Thank you, Mrugesh. I have added that to my TODO list. I have many things to do for the repos I recently transferred, in terms of config and CI.
For the most part, I have pushed that down my TODO priority, until these have been beta tested, which is finishing tomorrow 🙂


Group getProjects() allways return null

Group getProjects() function allways return null instead of group project list.

This has been fixed and verified.


Worker exits gracefully on an empty queue

This change allows the Worker to exit gracefully after a configurable number of calls to the Server result in an empty result. This is useful in the case of a Worker running on a cloud instance, so that the instance can automatically self-terminate (or take other actions) when there are no jobs to render. This reduces compute costs.

Looks like a great improvement @backbeatmedia!
Will take a closer look and merge it soon


Fix defaultProps with React 16

Fix invalid property getDefaultProps() with React 16:
Warning: Setting defaultProps as an instance property on HighchartsChart is not supported and will be ignored. Instead, define defaultProps as a static property on HighchartsChart.

Thank you!


HOW TO INSTALL

I installed the code as zip and transferred them to plugin folder but there is no toggle in the plugin menu
is there something that needs to be done

in same way as any other powercord plugin, go to https://discord.gg/powercord and ask for support


No copyright notice exists for the source code being copied and used

I am tired of pointing out license violations as I have done before, but there is no copyright notice for code copied from my public library (published under the MIT License). This is not in accordance with the MIT License.
I consider the statement "inspired by" to be quite different from a copyright notice. In fact, it is just a copy.

hello. I'll be happy to put the terms you find necessary..... I never had the intention of violating anything... if you look at the code, there is a reference to the origin, it is not a copy, but an evolution. there is a huge difference.... that's why it's an inspiration... anyway what do you want to be cited and where?

You write that you were inspired by the README, but in fact the implementation and testing of EastAsianWidth was simply copied from my original, so why don't you state that and the license in the README?

The EastAsianWidth and base-control was inspired by the work of Sharprompt

Don't forget that you have a history of using my library code almost verbatim and changing only the name to your own work for reasons of of evolution and inspiration.

Hi @shibayan, I'd like to address this since, assumably, it shouldn't be a lot of work, and I don't believe there is negative intent here. People should be credited for their hard work as is needed.
Since I'm not super familiar with the ins and outs of software licensing, what kind of PR would satisfy the conditions here? Do you have some pointers for me?
Particularly you mention "a history of using library code"; should more be attributed than just EastAsianWidth from Sharprompt?

@FRACerqueira I notice that FIGlet.Net uses the GNU Affero license, which requires that any works using it must also use the same GNU Affero license as well. May I assume you have used FIGlet.Net code (and perhaps expanded on it) rather than just "being inspired"? In that case that is also a licensing issue, as PromptPlus would have to be a GNU Affero project.

Hi @ividyon. I have released the source code of the library under the MIT License, so I see no problem at all in using it.
However, I believe that the use of the word "Inspired" and the absence of a license notice in the README shows disrespect for the original library and its license.
In essence, you are asking that the README state the license and that you are using the code. The words Inspired or Evolution would be inappropriate in that context.
As I mentioned in one previous comment, I don't feel any respect for the original library. That would be the fundamental issue.

@FRACerqueira I notice that FIGlet.Net uses the GNU Affero license, which requires that any works using it must also use the same GNU Affero license as well. May I assume you have used FIGlet.Net code (and perhaps expanded on it) rather than just "being inspired"? In that case that is also a licensing issue, as PromptPlus would have to be a GNU Affero project.
You may re-implement the necessary code yourself so that it's a true inspiration, to avoid this.

I notice that library provides fairly minor benefits to PromptPlus, and is also a rewrite of a previous library which is under MIT:
https://github.com/auriou/FIGlet
Might be prudent to use that one instead.

I made an according PR: #56

A commit to branch 4.0.5 was performed on this issue.
@shibayan , I hope this is in line with the licensing wishes that have been requested.
If you still have another wish on this subject, or find some code that does not have the copywrite mention, just let us know and we will correct it. Thank you for collaborating on improvements!

I think this can safely be closed.


Allow user to change selection

User should be able to move around the code and highlight code blocks, and when they resume typing, continue from the previous selection point

Fixed


fix: use addPostFrameCallback to delay initializion or updates of observer

Fixes 41y08h/fquery#9

Wow seems like it's solved, though for the sake of testing and making sure it stays with the style of the project, and that'll be useful to make it a standard style to go with, I'll write suggestions, tips, and set up tests to make sure people's valuable contribution can be instantly merged.
I very much value your active participation and initiative in this project and I'll be merging the changes soon. Thanks.

Thanks for your PR. Your contribution is well appreciated. 👍

For taking the initiative to fix bugs in code, work on new ideas, and being patient, @all-contributors please add @giva9712


Add devise gem?

We need to add support for users to update their passwords and for password resets, etc. Adding Devise will probably be the easiest way to achieve this, but if someone wants to do this by hand that works too.
This would partially address #28 as well, Devise includes some validations out of the box.

Is there any more work needed on the add-devise branch?


consumer_token saved with user_id=nil

Hi,
I'm using OAuth-plugin and devise for Linkedin API.
I (should) have set up correctly the devise aliases in oauth_consumers_controller.rb : almost everything is working fine.
When I grant the rights to my linkedin account on this page : http://localhost:2002/oauth_consumers/linkedin , the consumer_tokens is saved with user_id=nil
In all the other request, the current_user is correctly retrieved, but not in the insert.
I believe this is a kind of stupid error, but I can't figure where to correct this.
Here is the log from the server :
Started GET "/oauth_consumers/linkedin/callback?&oauth_token=7XXXXXXXXX735&oauth_verifier=17860" for <IP_ADDRESS> at 2011-09-20 15:27:39 +1200
Processing by OauthConsumersController#callback as HTML
Parameters: {"oauth_token"=>"7XXXXXXXXXXXXXXa735", "oauth_verifier"=>"17860", "id"=>"linkedin"}
User Load (0.1ms)  SELECT "users".* FROM "users" WHERE "users"."id" = 1 LIMIT 1
LinkedinToken Load (0.2ms)  SELECT "consumer_tokens".* FROM "consumer_tokens" WHERE "consumer_tokens"."type" IN ('LinkedinToken') AND "consumer_tokens"."user_id" = 1 LIMIT 1
CALLBACK
LinkedinToken Load (0.3ms)  SELECT "consumer_tokens".* FROM "consumer_tokens" WHERE "consumer_tokens"."type" IN ('LinkedinToken') AND "consumer_tokens"."user_id" = 1 AND "consumer_tokens"."token" = '699XXXXXXXXXXXX3b0' LIMIT 1
CACHE (0.0ms)  SELECT "users".* FROM "users" WHERE "users"."id" = 1 LIMIT 1
SQL (13.1ms)  INSERT INTO "consumer_tokens" ("created_at", "secret", "token", "type", "updated_at", "user_id") VALUES (?, ?, ?, ?, ?, ?)  [["created_at", Tue, 20 Sep 2011 03:27:40 UTC +00:00], ["secret", "5f27XXXXXXXXXXX6c40"], ["token", "699XXXXXXXXXXXXdb3b0"], ["type", "LinkedinToken"], ["updated_at", Tue, 20 Sep 2011 03:27:40 UTC +00:00], ["user_id", nil]]
Redirected to http://localhost:2002/

Found the solution here : http://unhandledexpression.com/2011/06/28/rails-and-oauth-plugin-part-2-the-consumer/#comment-205
You should add this to your ApplicationController:
def current_user=(user)
current_user = user
end


Added interactive tag to shell-type event property

Added interactive indicator to ShellType sent for telemetry

Codecov Report

Merging #6566 into dev will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##           dev   #6566   +/-   ##
===================================
  Coverage    0%      0%
===================================
  Files       11      11
  Lines      133     133
  Branches     8       9    +1
===================================
  Misses     133     133

Continue to review full report at Codecov.

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

Let's wait for @JasonRShaver 's feedback


[RIP-10] Add test case for ConsumeMessageOrderlyService

What is the purpose of the change
Add test case for ConsumeMessageOrderlyService
Brief changelog
ConsumeMessageOrderlyService
Verifying this change
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage increased (+0.3%) to 48.052% when pulling 4527b84ff3fad2eaec7ce5d671a370660e48286d on tandanzZ:TD-develop into 6eff504877e2924f76cd0b48f8daf7bfff4621c5 on apache:develop.


phenotypes in input json

Hi,
What is phenotypes in input json? It is not documented.
Thanks

In the family.wdl input documentation:
| String? | phenotypes | Comma-delimited list of HPO terms. | [Human Phenotype Ontology (HPO) phenotypes](https://hpo.jax.org/app/) associated with the cohort. If omitted, tertiary analysis will be skipped. |


[testing] Test with NVFuser enabled

Stack from ghstack:

-> #5959

@datumbox  thanks! We're tracking these issues here https://github.com/pytorch/pytorch/issues/76791 and there should be a fix landing today

to summarize failures:

many are the same failures as were seen in other PRs opened around the same time
https://github.com/pytorch/pytorch/issues/76791 these were also observed, and a fix was landed.


sample_multi_transcode fails with 10-bit -> 8-bit conversion

Setup:

KBL GT2
Ubuntu 18.04
Latest drm-tip kernel
git versions of media-driver/MediaSDK/etc

Test-case:

Transcode 10-bit 4K HEVC content to 8-bit FullHD HEVC:
sample_multi_transcode -hw -dc::nv12 -i::h265 Netflix_FoodMarket_4096x2160_10bit_420_100mbs_600.h265 -w 1920 -h 1080 -b 10000 -ec::nv12 -o::h265 output.h265

Output:
[ERROR], sts=MFX_ERR_UNSUPPORTED(-3), EncodePreInit, m_pmfxENC->Query failed at /opt/builder/source1/media-sdk/samples/sample_multi_transcode/src/pipeline_transcode.cpp:537

[ERROR], sts=MFX_ERR_UNSUPPORTED(-3), Init, EncodePreInit failed at /opt/builder/source1/media-sdk/samples/sample_multi_transcode/src/pipeline_transcode.cpp:3778

[ERROR], sts=MFX_ERR_UNSUPPORTED(-3), Init, pThreadPipeline->pPipeline->Init failed at /opt/builder/source1/media-sdk/samples/sample_multi_transcode/src/sample_multi_transcode.cpp:305

[ERROR], sts=MFX_ERR_UNSUPPORTED(-3), main, transcode.Init failed at /opt/builder/source1/media-sdk/samples/sample_multi_transcode/src/sample_multi_transcode.cpp:908

This happens regardless of whether I specify the transcode result format for decode (-dc::nv12) or encode (-ec::nv12).  Only removing both gets the command working, but then there's no output conversion.  None of the other documented output formats worked either.
This isn't related to regression #1344, it was broken already before.

According to my understanding, call only with -ec::nv12  (without -dc::nv12) is correct, as we set a parameter for encoder.

Thanks, I can verify that by using just the -ec option the command works now, and the resulting output looks OK.
Btw. What the encode & decode side conversion actually means, isn't the conversion actually done in-between them (decode with FF HW and compute based encode in KBL case), using the  SFC (scale & format conversion) HW block?


AKS: Remove preview flag for Virtual Node

Since AKS Virual Node has been graduated GA, should we have to remove the old PREVIEW tag?
https://azure.microsoft.com/en-us/updates/azure-kubernetes-service-aks-virtual-node-is-now-available/

This checklist is used to make sure that common guidelines for a pull request are followed.

[X] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[X] I adhere to the Command Guidelines.

Oh or it looks like it's GA for Linux but still in Preview for Windows, that's maybe why?

Let's ask @cpuguy83 before we merge this.

Ping @VDuleb

Please reopen a new PR to remove the is_preview=True kwarg if you need to accomplish this. This PR is now stale.


FastReport.OpenSource.SQLite broken all netcore pkgs when call CreateAllTables method is used

.GetSchema() is not a method available in Microsoft.Data.Sqlite which is used when compiled under netstandard/ core builds and that method is needed when CreateAllTables() is invoked from report.
Error -> System.MissingMethodException: 'Method not found: 'System.String SQLitePCL.raw.sqlite3_errstr(Int32)'.'
the issue that the method GetSchema() is not implemented in library that you are using for netcore, but is available if you build for framework 4.7 (issue I believe is that on non core you are using system.data.sqlite and for core you are microsoft.data.sqlite which doesn't support GetSchema() for sqlite).
Example
Creation console application (.net core console 3.1)
my program.cs  contains ->
using System;
namespace FastReportDemo
{
class Program
{
static void Main(string[] args)
{
Console.WriteLine("Hello World!");
SQLitePCL.Batteries_V2.Init();
FastReport.Data.SQLiteDataConnection conn = new FastReport.Data.SQLiteDataConnection();
conn.ConnectionString = @"data source = .\SQL_DB\chinook.db";
conn.CreateAllTables(); // <--- this will fail under netstandard or netcore build/ but works with framework
Console.ReadLine();
}
}

}
my FastReportDemo.csproj contents -->

Exe
netcoreapp3.1

Hello!
I was able to reproduce this problem
I informed the developers
Best regards
Marat


[FR]: PowerToys Run: Hide blue background behind UWP apps

Description of the new feature / enhancement
the blue background of UWP oder PWAs should be hidden as in windows search.
Currently app icons are smaller und it doesn't fit well with the modern Windows 11 System.

Scenario when this would be used?
better design and consistence
Supporting information
No response

I found the relevant code is in src\modules\launcher\Plugins\Microsoft.Plugin.Program\Programs\UWPApplication.cs. But just want to know why does it in the first place is given that blue background and is just removing that or making it transparent the solution?


Japanese Localization

Work with community localization assets to get VRCBCE playable by a native Japanese speaker who doesn't speak any English.

Which part do you want to change to Japanese?
The short English in the current UI appears to be understandable by most Japanese.

Ideally, the entire prefab should be localized to a few different languages. Forcing everyone to know a little English is a little unfair.
Japanese is the obvious first language to localize to as the VRC Japanese community is quite large, but the idea would be to start with Japanese but add more languages as time goes on (e.g. French/Spanish/Portuguese/Italian/German/Korean)


Enabled aufs-over-btrfs support back

AUFS author supports this configuration (there are still bugs found from time to time like fixed http://sourceforge.net/p/aufs/aufs3-standalone/ci/af8aee7bb0ee38a8f4e565956470555d434f728d/, but in general it's usable with latest aufs version).
Usage of this configuration may be reasonable for user, if he/she wants to get advantages of inode vfs cache sharing, because btrfs subvols don't support such option and unlikely to do this in future due to linux pagecache architecture.

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "bugfix/enable-aufs-over-btrfs" git@github.com:iavael/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

This will update the existing PR, so you do not need to open a new one.

The AUFS author may support this, but we don't support it for a very good reason: the AUFS author doesn't have control over the kernels found in distributions and btrfs isn't particularly stable on some distributions with AUFS patches applied to the kernel, nor can the author tell users why they've lost data when combining AUFS with btrfs.
Btrfs still has problems. I wouldn't advise mixing btrfs' problems together with AUFS' problems to risk running into even weirder problems.

@unclejack +1 do you think we should close? or find a way to disallow it over btrfs like we do for overlay

the AUFS author doesn't have control over the kernels found in distributions

Same for BTRFS devs. Who known what patches distro maintainers would apply. :)

I wouldn't advise mixing btrfs' problems together with AUFS' problems to risk running into even weirder problems.

Well, if user wants to combine tricky out-of-tree overlay fs with unstable and complex COW fs, it's up to his/her responsibility. There are countless possible risky configurations, but it doesn't mean to explicitly prohibit all of them in code, does it?

They don't support the 3.13 and 3.16 kernels from Ubuntu 14.04.x - as in they can't go and apply all the AUFS patch changes to 3.13 or make sure that btrfs patches are applied to fix AUFS+btrfs problems.

That's why there are maintainers in every distro including ubuntu (and it is not the only one distribution in the world to depend on it, if docker is not ubuntu-specific, of course)! If maintainers commited to support a specific kernel version for a long term and build it with out-of-tree patches, it's obvious that they would have to backport them anyway, if patch developer is not very cooperative, it's unreasonable to discuss a specific bugfix in aufs here. More important is that aufs developer actually supports configuration with btrfs and fixes problms.

Making this change would increase the likelihood of setting up Docker in a way that leads to data loss through the use of AUFS on top of btrfs.

If it's likely that users would use docker with aufs over not-ext4-fs, it may be reasonable write a warning to the log if a non-ext4 fs is detected. Could this be an acceptable solution?

AUFS is much faster on top of ext4 and there isn't much of an advantage in having it on btrfs. Using the btrfs graph driver is recommended instead.

Max disk performance is not necessarily something that I as a user want to achieve. My target may be minimal disk and memory usage, especially if we talk about, e.g. about laptops.

@iavael This doesn't seem like the right way to approach the problem. Some users want to be able to use AUFS on the file system they choose. However, there's a problem with allowing it to work on btrfs by default: there are distributions out there which don't support this and there's absolutely no guarantee we wouldn't break fully functional working Docker setups for some people. This means that those distributions with AUFS patched kernels: a) don't support btrfs, b) don't support aufs on top of btrfs, c) don't support aufs, nor aufs on top of btrfs
Support means they're not going to provide technical support for that.
The problem with allowing aufs to work on btrfs like this is that it's not opt in, it'd be a default with no way to change the setting.
The fact that something is supported by someone doesn't mean everyone will choose to support it. Right now, supporting AUFS on btrfs doesn't make any sense at all.
Allowing AUFS on btrfs by default would also interfere with automatic fallback to btrfs for all those who rely on the current behavior of using the btrfs graph driver on an AUFS patched kernel.
Instead of having this PR, we should have a storage driver option like disable-aufs-safety-checks which has to be passed explicitly to allow AUFS to work on unsupported file systems. This option wouldn't be supported (no aufs related bug reports from this would get investigated).
@jfrazelle This PR can be closed.

@unclejack Thanks for detailed explanation, looks quite reasonable.


Allow marist-1 build box to build openj9/jdk12

The job https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk12u/job/jdk12u-linux-s390x-openj9/ (and maybe others) is locked to the https://ci.adoptopenjdk.net/computer/build-marist-rhel74-s390x-2/ machine and therefore won't run on https://ci.adoptopenjdk.net/computer/build-marist-rhel74-s390x-1/ - this is not ideal for redundancy and we need to verify why and remove the lock to that machine.

@sxa Since both the marist machine and JDK12 job have been removed/disabled (unless they have been renamed to build-marist-rhel77-s390x-1, build-marist-rhel77-s390x-2?), can this be closed?

Disabled as all the machines were recereated on new hosting hardware in December, so yeah might as well close this.


Doesn't cover dedicated vCPUs

Doc also needs to cover dedicated vCPUs, which are a separate quota for dedicated hosts.

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

ID: fe04a05a-f753-ee1f-5171-fd2a81a030f9
Version Independent ID: 1507a7ac-11cc-81a7-e932-9e8ef9644e2b
Content: Azure Resource Manager vCPU quota increase requests
Content Source: articles/azure-portal/supportability/resource-manager-core-quotas-request.md
Service: azure-supportability
GitHub Login: @sowmyavenkat86
Microsoft Alias: svenkat

@jtuliani Thank you for bringing this to our notice. I have assigned this issue to our document author to further evaluate the feedback and update the document accordingly.

I apologize that this issue has remained open so long. The quota documentation has been refreshed several times since this issue was filed. I'm going to close it now, but please open a new issue if you see further discrepancies.
#please-close


NO RESULTS FOUND....when searched "html to figma"

NO RESULTS FOUND When search for "html to figma" in the search box of figma.

What can i do to fix it ?

hey @SubrataJZ - you sure you have the plugin installed? the menu is not something we have control over - if the plugin is installed then figma will have it in the menu, and if not theres not a ton we can do
if you definitely have it installed but it won't show in the menu, perhaps contact figma support?

Go to this link below and install:
https://www.figma.com/community/search?model_type=public_plugins&q=html to figma


Help, dictionary keys changed during iteration error

Select your issue type: (check at least one)

[x] Bug
[x] Question
[ ] Suggestion
[ ] Other (please describe):

I haven' been able to get it started once even when i restart or reload tor i get this error:

What did you do for - -create-configuration because that didn’t work for me. If you can help me with that I might be able to help you with your problem.

What did you do for - -create-configuration because that didn’t work for me. If you can help me with that I might be able to help you with your problem.

instagram-py --create-configuration --default-configuration
Just the default

What did you do for cd /root because for me it says permission denied?

What did you do for /root$ ls?

What did you do for /root$ ls?

Not sure what you mean by that, maybe attach a picture
If you cant access /root run as su
e.g. sudo cd /root

Go to 3:09 in the video. He does /root$ ls. I tried it and it said no such file or directory.

Go to 3:09 in the video. He does /root$ ls. I tried it and it said no such file or directory.

What video?

Go to 3:09 in the video. He does /root$ ls. I tried it and it said no such file or directory.

/root is a directory
ls is the command to list files in that directory

https://youtu.be/KZEcFn3Z5L0

The white text is the user input

I typed in sudo nano /etc/tor/torcc and it went to the screen but there’s no text.

I typed in sudo nano /etc/tor/torcc and it went to the screen but there’s no text.

Yeah copy everything from torrc.orgin and paste in torrc
.... Then uncomment the Control Port(that is remove the # before it)

Sorry for people who have tried to help with this issue, turns out it was my router setting denying the tor services from working properly.


Update screeps-game-api to Rust 2018.

Ran cargo fix --edition and added the edition flag in Cargo.toml.

A lot could be done to get rid of the extern crate and to properly scope the macros. But when I saw the 500 errors, I decided it wasn't an emergency.

Sounds reasonable! I'm good with merging this without all the edition-idiom fixes.
Travis seems to be complaining about one more :: instead of crate:: in the reference_wrappers! macro - I had to use --all-features to get the error to show up.

Sounds reasonable! I'm good with merging this without all the stuff like extern crate. Not sure if you were using cargo fix --edition-idioms or doing it manually, but if you were trying manually, cargo-fix might help? It can also be imperfect, though.

I missed that in my cargo fix --help. I'll give it a shot!

Travis seems to be complaining about one more :: instead of crate:: in the reference_wrappers! macro. I had to use --all-features flag to get the error to show up locally, so cargo-fix might not have picked it up.

Will have a look. I saw travis' notification too late yesterday.


Additional settings endpoints

Adding additional settings endpoints from issue #101

This issue was migrated from an old repository: https://github.com/CiscoDevNet/catalystwan/pull/105


Use label-checker PR with set-output fix

Closing as this PR was never intended to be merged.  It was created only to test that https://github.com/agilepathway/label-checker/pull/315 works properly, which it does.


add Geocentric and Local Cartesian projectors

The UTM projector creates shifts in projected maps relative to actual world positions.
Added Geocentric (ECEF) and LocalCartesian projectors from geographic lib as available projectors.
These projectors have the advantage over the UTM projector in that they properly use lat/lon/alt to compute XYZ using WGS84 ellipsoid, whereas the UTM projector only uses lat/lon to convert into X/Y/Z on the ellipsoid and just adds elevation, which is incorrect.

In the image above, the point P' is the shifted point created by the UTM projector, compared to the correct point P.

@mantkiew thanks! Briefly browsing through: would you please add tests?

@m-naumann I added the tests and mentioned the new projectors in the README and CHANGELOG as well.
BTW. We are using these extensions/fixes from my PRs in this project:
https://uwaterloo.ca/waterloo-intelligent-systems-engineering-lab/news/avin-lanelet2-mapping-project-completed
Additionally, the WISE ADS now also depends on these projectors
https://uwaterloo.ca/waterloo-intelligent-systems-engineering-lab/projects/wise-automated-driving-system
We previously used Lanelet1 but we're now migrating to use Lanelet2.

@m-naumann I added the tests and mentioned the new projectors in the README and CHANGELOG as well.
BTW. We are using these extensions/fixes from my PRs in this project:
https://uwaterloo.ca/waterloo-intelligent-systems-engineering-lab/news/avin-lanelet2-mapping-project-completed
Additionally, the WISE ADS now also depends on these projectors
https://uwaterloo.ca/waterloo-intelligent-systems-engineering-lab/projects/wise-automated-driving-system
We previously used Lanelet1 but we're now migrating to use Lanelet2.

Thanks a lot! And nice to see the usage. You'll certainly not regret the update to lanelet2 ;) And at least I personally can back your call for correct elevation consideration.
Could you also add tests for the python bindings (even though we didn't provide them initially, we should try to increase as much covered code as possible).
And please bear with us for a bit, we need to get a public CI pipeline up and running again, such that static code analysis is available on github and there is no longer a need to report issues from the internal pipeline.

@m-naumann  I have added the tests. Note that the values coming back from the projectors have to be rounded for comparison. The maximum number of decimals that work is 8. The tests pass, of course.
I believe that is ready to be merged.

@m-naumann on the second thought: maybe we should round the values to the eight decimal ourselves in the API? That would take away the burden from the user.

And please bear with us for a bit, we need to get a public CI pipeline up and running again, such that static code analysis is available on github and there is no longer a need to report issues from the internal pipeline.

This is now done, @mantkiew would you please merge master into your branch to have the pipeline running for this PR?

Wonderful! Glad to finish this up. On it....

@m-naumann All GitHub actions have successfully completed in my repository. You have to allow them here.

@m-naumann the test job on noetic fails with some nonsense Error: sha is not defined. Could you please restart it? All jobs passed on my repository (see https://github.com/wiselabuw/Lanelet2/actions/runs/3281889493)

@m-naumann the test job on noetic fails with some nonsense Error: sha is not defined. Could you please restart it? All jobs passed on my repository (see https://github.com/wiselabuw/Lanelet2/actions/runs/3281889493)

This is a bug in the new CI, once #259 is merged it should also pass here.

@poggenhans ok, this is now ready to go.

@poggenhans hi, anything preventing this to be merged?


Do not guard initializeFlipper for bridgeless for RN Tester

Summary:
We don't need this if-than-else because the initializeFlipper already checks if we're on bridgeless or not
Changelog:
[Internal] [Changed] - Do not guard initializeFlipper for bridgeless for RN Tester
Reviewed By: NickGerleman
Differential Revision: D49881903

Merged by b93bd797797


refactor(connector): [Stax] Currency Unit Conversion

Type of Change

[x] Refactoring

Description

Closes https://github.com/juspay/hyperswitch/issues/2246
cargo clippy throws below warning on Rust version v1.73.0:

Motivation and Context

Hacktoberfest
How did you test it?

Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed the submitted code

Closing this.