Error using Oracle on windows

With ST3 with GoOracle on windows, when I try to issue any command with Oracle, I get the following error:

Running oracle callees command...
'export' is not recognized as an internal or external command,
operable program or batch file.

It looks like the "export" command on line 107 of goOracle.py is not a valid windows command for updating the path.  I believe you need to use the 'set' command for Windows.
(unless I'm missing some configuration for ST3 that is causing Oracle to not work)

Even fixing the above, I get the following error next:

Traceback (most recent call last):
File "goOracle in C:\Users\kyrra\AppData\Roaming\Sublime Text 3\Installed Packages\GoOracle.sublime-package", line 46, in on_done
File "goOracle in C:\Users\kyrra\AppData\Roaming\Sublime Text 3\Installed Packages\GoOracle.sublime-package", line 108, in oracle
TypeError: 'NoneType' object is not subscriptable

I got this fully working for Windows (with breaking linux/osx).  I'll see if I can this working for both and submit a pull request.
In-case I don't the main change is in the 'def oracle' for setting up cmd:
cmd = "set GOPATH=%(go_path)s&&set PATH=%(path)s&&oracle -pos=%(file_path)s:%(pos)s -format=%(output_format)s %(mode)s %(scope)s"  % {

how you set environment variables, separate commands in the same line, and separate PATH variables differ in Windows.  The env variable setup in User.sublime-settings for windows needs to differ a bit as well.  For example:
"env": { "GOPATH": "D:\\go", "PATH": "%PATH%;%GOPATH%\\bin" },


MRAA_ERROR_PLATFORM_ALREADY_INITIALISED

DEBUG: Executing shell function do_compile
| -- The C compiler identification is GNU 4.9.1
| -- Check for working C compiler: /sde/r1.2.0/iot_1.2.0_kernel_3.19.8/build/tmp/sysroots/x86_64-linux/usr/bin/i586-poky-linux/i586-poky-linux-gcc
| -- Check for working C compiler: /sde/r1.2.0/iot_1.2.0_kernel_3.19.8/build/tmp/sysroots/x86_64-linux/usr/bin/i586-poky-linux/i586-poky-linux-gcc -- works
| -- Detecting C compiler ABI info
| -- Detecting C compiler ABI info - done
| -- Detecting C compile features
| -- Detecting C compile features - done
| -- Looking for include file pthread.h
| -- Looking for include file pthread.h - found
| -- Looking for pthread_create
| -- Looking for pthread_create - not found
| -- Looking for pthread_create in pthreads
| -- Looking for pthread_create in pthreads - not found
| -- Looking for pthread_create in pthread
| -- Looking for pthread_create in pthread - found
| -- Found Threads: TRUE
| -- INFO - libmraa Version v0.10.0
| -- INFO - cmake Version 3.3.1
| -- INFO - Target arch is i586
| -- Found PythonInterp: /usr/bin/python2.7 (found suitable version "2.7.8", minimum required is "2.7")
| -- INFO - Package arch is i586
| -- INFO - Adding support for platform
| -- INFO - Adding support for all platforms
| INFO - Adding x86 platforms
| -- The CXX compiler identification is GNU 4.9.1
| -- Check for working CXX compiler: /sde/r1.2.0/iot_1.2.0_kernel_3.19.8/build/tmp/sysroots/x86_64-linux/usr/bin/i586-poky-linux/i586-poky-linux-g++
| -- Check for working CXX compiler: /sde/r1.2.0/iot_1.2.0_kernel_3.19.8/build/tmp/sysroots/x86_64-linux/usr/bin/i586-poky-linux/i586-poky-linux-g++ -- works
| -- Detecting CXX compiler ABI info
| -- Detecting CXX compiler ABI info - done
| -- Detecting CXX compile features
| -- Detecting CXX compile features - done
| -- Found SWIG: /usr/local/bin/swig (found version "3.0.8")
| -- Found PythonLibs: /sde/r1.2.0/iot_1.2.0_kernel_3.19.8/build/tmp/sysroots/quark/usr/lib/python2.7 (found suitable version "2.7.3", minimum required is "2.7")
| -- Found Nodejs: /sde/r1.2.0/iot_1.2.0_kernel_3.19.8/scripts/node
| INFO - Node version is 5.9.0
| INFO - Node using v8 <IP_ADDRESS>
| -- INFO - Padded V8 version to match SWIG format
| -- INFO - swig Version 3.0.8
| -- INFO - CXX compiler Version GNU 4.9.1
| -- INFO - Using V8 version > 3 so requiring C++11 compiler
| -- INFO - install NODE modules to /usr/lib/node_modules/mraa/
| -- Configuring done
| WARNING: Target "_python-mraa" requests linking to directory "/sde/r1.2.0/iot_1.2.0_kernel_3.19.8/build/tmp/sysroots/quark/usr/lib/python2.7".  Targets may link only to libraries.  CMake is dropping the item.
| -- Generating done
| -- Build files have been written to: /sde/r1.2.0/iot_1.2.0_kernel_3.19.8/build/tmp/work/i586-poky-linux/libmraa/0.10.0+gitAUTOINC+52f9c48bbc-r0/git/build
| Scanning dependencies of target mraa
| [  1%] Building C object src/CMakeFiles/mraa.dir/mraa.c.o
| /sde/r1.2.0/iot_1.2.0_kernel_3.19.8/build/tmp/work/i586-poky-linux/libmraa/0.10.0+gitAUTOINC+52f9c48bbc-r0/git/src/mraa.c: In function 'mraa_result_print':
| /sde/r1.2.0/iot_1.2.0_kernel_3.19.8/build/tmp/work/i586-poky-linux/libmraa/0.10.0+gitAUTOINC+52f9c48bbc-r0/git/src/mraa.c:464:9: error: duplicate case value
|          case MRAA_ERROR_PLATFORM_ALREADY_INITIALISED:
|          ^
| /sde/r1.2.0/iot_1.2.0_kernel_3.19.8/build/tmp/work/i586-poky-linux/libmraa/0.10.0+gitAUTOINC+52f9c48bbc-r0/git/src/mraa.c:428:9: error: previously used here
|          case MRAA_SUCCESS:
|          ^
| src/CMakeFiles/mraa.dir/build.make:62: recipe for target 'src/CMakeFiles/mraa.dir/mraa.c.o' failed
| make[2]: *** [src/CMakeFiles/mraa.dir/mraa.c.o] Error 1
| CMakeFiles/Makefile2:93: recipe for target 'src/CMakeFiles/mraa.dir/all' failed
| make[1]: *** [src/CMakeFiles/mraa.dir/all] Error 2
| Makefile:160: recipe for target 'all' failed
| make: *** [all] Error 2
| WARNING: exit code 2 from a shell command.
| ERROR: Function failed: do_compile (log file is located at /sde/r1.2.0/iot_1.2.0_kernel_3.19.8/build/tmp/work/i586-poky-linux/libmraa/0.10.0+gitAUTOINC+52f9c48bbc-r0/temp/log.do_compile.3162)
/**

MRAA return codes
*/
typedef enum {
MRAA_SUCCESS = 0,                             /< Expected response */
MRAA_ERROR_FEATURE_NOT_IMPLEMENTED = 1,       /< Feature TODO */
MRAA_ERROR_FEATURE_NOT_SUPPORTED = 2,         /< Feature not supported by HW */
MRAA_ERROR_INVALID_VERBOSITY_LEVEL = 3,       /< Verbosity level wrong */
MRAA_ERROR_INVALID_PARAMETER = 4,             /< Parameter invalid */
MRAA_ERROR_INVALID_HANDLE = 5,                /< Handle invalid */
MRAA_ERROR_NO_RESOURCES = 6,                  /< No resource of that type avail */
MRAA_ERROR_INVALID_RESOURCE = 7,              /< Resource invalid */
MRAA_ERROR_INVALID_QUEUE_TYPE = 8,            /< Queue type incorrect */
MRAA_ERROR_NO_DATA_AVAILABLE = 9,             /< No data available */
MRAA_ERROR_INVALID_PLATFORM = 10,             /< Platform not recognised */
MRAA_ERROR_PLATFORM_NOT_INITIALISED = 11,     /< Board information not initialised */
MRAA_ERROR_PLATFORM_ALREADY_INITIALISED = 0,  /**< Board is already initialised, same as MRAA_SUCESS */
MRAA_ERROR_UNSPECIFIED = 99 /**< Unknown Error */
} mraa_result_t;

MRAA_SUCCESS and MRAA_ERROR_PLATFORM_ALREADY_INITIALISED have the same value

Yeah this is on purpose so mraa_init() stops replying MRAA_ERROR_PLATFORM_ALREADY_INITIALISED which was a pretty stupid error code.
I'm surrpised this broke your build, it works on travis/arch/debian/fedora/ubuntu/clang :/ And passed static analysis!
I guess if not we can move MRAA_ERROR_PLATFORM_ALREADY_INITIALISED to be a #define.

C99
http://www.open-std.org/jtc1/sc22/wg14/www/docs/n1256.pdf
(The use of enumerators with = may produce enumeration constants with values that duplicate other values in the same enumeration.)  The enumerators of an enumeration are also known as its members.

what about 02ba5f993fef7cb8c23563feaa55adc0fcf968a8?

The problem is not related with enum, it is related with CASE:

this case statement got removed with 51c60a0ac4c52d0d1032fd17d88636642bd0ea85 in the 0.10.0 release though.

| -- Generating done
| -- Build files have been written to: /sde/r1.2.0/iot_1.2.0_kernel_3.19.8/build/tmp/work/i586-poky-linux/libmraa/0.10.0+gitAUTOINC+52f9c48bbc-r0/git/build
| Scanning dependencies of target mraa
| [  1%] Building C object src/CMakeFiles/mraa.dir/mraa.c.o
| [  2%] Building C object src/CMakeFiles/mraa.dir/gpio/gpio.c.o
| [  4%] Building C object src/CMakeFiles/mraa.dir/i2c/i2c.c.o
| [  5%] Building C object src/CMakeFiles/mraa.dir/pwm/pwm.c.o
| [  6%] Building C object src/CMakeFiles/mraa.dir/spi/spi.c.o
| [  8%] Building C object src/CMakeFiles/mraa.dir/aio/aio.c.o
| [  9%] Building C object src/CMakeFiles/mraa.dir/uart/uart.c.o
| [ 10%] Building C object src/CMakeFiles/mraa.dir/iio/iio.c.o
| [ 12%] Building C object src/CMakeFiles/mraa.dir/x86/x86.c.o
| [ 13%] Building C object src/CMakeFiles/mraa.dir/x86/intel_galileo_rev_d.c.o
| [ 15%] Building C object src/CMakeFiles/mraa.dir/x86/intel_galileo_rev_g.c.o
| [ 16%] Building C object src/CMakeFiles/mraa.dir/x86/intel_edison_fab_c.c.o
Also have a problem with mraajsJAVASCRIPT_wrap.cxx.o. Now I will check and open new bug if it is.

"this case statement got removed with 51c60a0 in the 0.10.0 release though."
Sure. But it is downloaded with this case from github. It's strange...

I can't find it in the release tarball from git or my local clone, you sure you've got a clean checkout? 52f9c48bbc seems to be good but I see you're using AUTOINC so maybe something is not quite right? Maybe a bitbake/devtool cleanall mraa is in order?

this is what you should have
brendan@silver ~/git/mraa (git)-[master] % md5sum src/mraa.c
1a4002c73235d86a9b0f4590c48e244d  src/mraa.c

user@user-VirtualBox:/sde/r1.2.0/iot_1.2.0_kernel_3.19.8/build/tmp/work/i586-poky-linux/libmraa/0.10.0+gitAUTOINC+52f9c48bbc-r0/git/src$ md5sum mraa.c
2df3d5f5b6b0e22bf000a9a6725f5b5b  mraa.c
52f9c48bbc is a part of https://github.com/intel-iot-devkit/mraa/commit/52f9c48bbc4c63c6c4ec6285760a4e5c67fad4dd

I have done clean/fetch of mraa several times.

user@user-VirtualBox:/sde/r1.2.0/iot_1.2.0_kernel_3.19.8/build/tmp/work/i586-poky-linux/libmraa/0.10.0+gitAUTOINC+52f9c48bbc-r0/git$ git show
commit 52f9c48bbc4c63c6c4ec6285760a4e5c67fad4dd
Author: Brendan Le Foll<EMAIL_ADDRESS>Date:   Thu Mar 24 17:01:01 2016 +0000
mraa: Update to v0.10.0

Signed-off-by: Brendan Le Foll<EMAIL_ADDRESS>
diff --git a/CMakeLists.txt b/CMakeLists.txt
index b150d2b..6691600 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -25,7 +25,7 @@ include (GetGitRevisionDescription)
git_describe (VERSION "--tags")
if ("x_${VERSION}" STREQUAL "x_GIT-NOTFOUND" OR "x_${VERSION}" STREQUAL "x_HEAD-HASH-NOTFOUND" OR "x_${VERSION}" STREQUAL "x_-128-NOTFOUND")
message (WARNING " - Install git to compile a production libmraa!")

set (VERSION "v0.9.6-dirty")

set (VERSION "v0.10.0-dirty")
endif ()

message (STATUS "INFO - libmraa Version ${VERSION}")
diff --git a/docs/changelog.md b/docs/changelog.md
index 828d8ff..ac014cb 100644
commit 52f9c48bbc4c63c6c4ec6285760a4e5c67fad4dd
Author: Brendan Le Foll<EMAIL_ADDRESS>Date:   Thu Mar 24 17:01:01 2016 +0000
mraa: Update to v0.10.0

Signed-off-by: Brendan Le Foll<EMAIL_ADDRESS>
diff --git a/CMakeLists.txt b/CMakeLists.txt
index b150d2b..6691600 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -25,7 +25,7 @@ include (GetGitRevisionDescription)
git_describe (VERSION "--tags")
if ("x_${VERSION}" STREQUAL "x_GIT-NOTFOUND" OR "x_${VERSION}" STREQUAL "x_HEAD-HASH
-NOTFOUND" OR "x_${VERSION}" STREQUAL "x_-128-NOTFOUND")
message (WARNING " - Install git to compile a production libmraa!")

set (VERSION "v0.9.6-dirty")

set (VERSION "v0.10.0-dirty")
endif ()

if your md5sum doesn't match 1a4002c73235d86a9b0f4590c48e244d then yo don't have the right source files. However I have no idea what's wrong or how that's possible. In the git checkout can you do a git describe --tags and git rev-parse HEAD?

@tingleby any ideas what's possibly up here?

user@user-VirtualBox:/sde/r1.2.0/iot_1.2.0_kernel_3.19.8/build/tmp/work/i586-poky-linux/libmraa/0.10.0+gitAUTOINC+52f9c48bbc-r0/git$ git describe --tags
v0.10.0
user@user-VirtualBox:/sde/r1.2.0/iot_1.2.0_kernel_3.19.8/build/tmp/work/i586-poky-linux/libmraa/0.10.0+gitAUTOINC+52f9c48bbc-r0/git$ git rev-parse HEAD
52f9c48bbc4c63c6c4ec6285760a4e5c67fad4dd

in build/downloads/git2 can you check the right stuff is in there?

are you sure you don't have a patch in SRC_URI?

Ohh.. I have a patch.

phew! I thought I was going crazy &| git/github was breaking!

I confirm that a compilation is correct. That was my fault.

No problem, tbh the C99 thing is still very valid, what do you think of the changes in the branch c99?

May be it is better to add mraa_deinit() (or imraa_deinit - why not?) to imraa_init() and do not use MRAA_ERROR_PLATFORM_ALREADY_INITIALISED at imraa_init(). In this case imraa_xxx performs a real init/deinit, and mraa_xxxx is just a wrapper for speed up. I guess that MRAA_ERROR_PLATFORM_ALREADY_INITIALISED with the same value as MRAA_SUCCESS is useless.And it is better do not use STDC_VERSION for compatibility reasons.

Like this:
  if (plat != NULL) {
        mraa_deinit();
 }

So {imraa,mraa}_init() returning MRAA_ERROR_PLATFORM_ALREADY_INITIALISED was a merge mistake on my part that shouldn't have gotten in. imraa or mraa should return MRAA_SUCCESS reguardless of whether a platform was actually found, it just means structures initialised without error. imraa_init()'s plat != NULL should return an error and mraa_init()'s should return MRAA_SUCCESS.
imraa_init() is meant to be called from imraa only it's not part of mraa's public API, so the check really is redundant but there just to make sure imraa doesn't do something stupid. mraa_init/mraa_deinit are really the ones that could/should be called by users.
So I'm preparing mraa's 1.0, let's completely remove MRAA_ERROR_PLATFORM_ALREADY_INITIALISED in that release. I'll use mraa-ng to stage changes like this.


Completes renaming "basUtils" to "io"

Resolves #126.
In ppafm/cli/plot_results.py, import elements as
from ppafm import io, elements

rather than
from ppafm import basUtils, elements

.

Oh, now I see `plot_results.py' does not work yet. But I should probably start it as a new issue. The error message with which
ppafm-plot-results -k 0.5 -q -0.05 -a 2.0 --df

finishes now is
error:  (<class 'AttributeError'>, AttributeError("module 'ppafm.GridUtils' has no attribute 'load_scal_field'"), <traceback object at 0x1550ab371200>)
cannot load :  Q-0.05K0.50/OutFz.xsf
 ***** ALL DONE *****


[BrokenLinksH2] Fix path in link

Fixing TOC broken links

Docs Build status updates of commit 9e761ee:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sysinternals/bread/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@foxmsft - The C+E Skilling team is fixing broken links on docs.microsoft.com. This will eliminate potential accessibility, security, and usability issues. This PR includes only link fixes/removals and does not change other content. This PR is ready for review and merge. Thank you!

Docs Build status updates of commit d12ef43:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sysinternals/bread/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit a3c6b66:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sysinternals/bread/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit d9d292c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sysinternals/bread/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


[Examples] metadata is missing schema in example datasets

@michael-s-molina Got it, I will work on this issue.

Hi, I've been working on this with @castellanos-ricardo and @ARES-79. Looks like #16041 fixes this issue and this issue is a duplicate of #16051.

Closing in favor of #16051, and hopeful that #16041 fixed this (and maybe that one too!)


Add optional Cloudfront invalidation step at the end

If $AWS_CF_DISTRIBUTION_ID is set, a Cloudfront invalidation request will
be created for paths /*. Useful when hosting websites with S3 + Cloudfront.

Having this in one action will be very handy any idea if / when this can be merged?

@jgarth It would be useful to validate not /*, but objects that were really changed.

That could be a really useful feature, but it could be optional.
Currently we have issues with implementing 100% bulletproof CSP with nonce, and it happens that Cloudfront do not have functionality to inject nonce to all script and style tags in html served, so we are thinking about other solutions that provides desired CSP nonce security. if somebody have non-PHP solutions, please share with me.

In addition to AWS_CF_DISTRIBUTION_ID, what if INVALIDATE_PATHS was also added?  Perhaps 'index.html' could be a suitable default value.
Currently, I'm getting by with https://github.com/chetan/invalidate-cloudfront-action but it sure would be great to have everything under one roof, so to speak.

@jakejarvis Any potential for getting this evaluated/merged?

Any update on this? Could be really helpful!


Cant open from run

Windows Terminal version (or Windows build number)
19042.867
Other Software
No response
Steps to reproduce
Windows+R, wt, ENTER
Expected Behavior
Windows terminal opens up
Actual Behavior
Error message: The application was unable to start correctly (0xc0000022). Click OK to close the application.

There's a pile of issues already tracking this particular error message:

#9452: [MSIX Bucket] Launch issues caused by bad permissions on WindowsApps, and others
#9399: Always crashes with 0xc0000022
#7974: 0xc0000022 error was occured while opening wt.exe after v1.2.2381.0
#7081: 0xc0000022 startup error when opening wt from file explorer or Run window
#6862: Error 0xd0000022 when launching any shell (PS, cmd, git bash, PS7, Azure Cloud Shell) in WT

Please follow up in #9452 if none of the solutions in those threads work for you.
/dup #9452


Matched expressions not showing up in message.match

controller.hears('set volume (\\d+)','direct_message,direct_mention,mention', function(bot, message) {
    console.log(message);
});

@bot: set volume 80

{ type: 'message',
  channel: 'xxx',
  user: 'xxx',
  text: 'set volume 80',
  ts: '1456917563.000012',
  team: 'xxx',
  event: 'direct_mention' }

It matches the expression but there is no match property where I can get the number from.
Is this a bug or am I using it incorrectly? Thanks!

I just tried to reproduce your issue, and got:
{ type: 'message',
  channel: 'x',
  user: 'x',
  text: 'set volume 80',
  ts: '1459047632.000360',
  team: 'x',
  event: 'direct_mention',
  match: [ 'set volume 80', '80', index: 0, input: 'set volume 80' ] }

Can you still reproduce your issue with the latest Botkit code?

Hi! I upgraded to the latest version and it seems to work fine now - thank you :)

Fantastic, thanks for letting me know!  :+1:


Format Exception in WslApi.GetDistributionList() where registry Lxss key contains a subkey without a GUID name

Hello.
Thank you for this library which has proven very useful for a tool I have recently built.  I have spotted a minor issue where this fails with a "Format Exception" on certain user's machines.
It appears that a Microsoft process (I suspect a MS Store installation process) can leave a subkey in the Computer\HKEY_CURRENT_USER\Software\Microsoft\Windows\CurrentVersion\Lxss hierarchy of the registry which has a non-GUID key name.  The key in question is called AppxInstallerCache.

Having had a look at the code in the GetDistributionList routine, I can see that this makes an assumption that all subkeys are in the format of GUIDs.

Please see similar issues in other projects to confirm this is a "known" value in this registry key

https://github.com/DDoSolitary/LxRunOffline/issues/171
https://github.com/emeric-martineau/wsl-gui-tool/pull/6

Hi, @tip2tail,
thanks for the bug report 👍
I have created an alpha release which contains a fix (non-guid usb keys will be skipped) and also added a .net 8 buid.
Please check if https://www.nuget.org/packages/Community.Wsl.Sdk/3.1.0-alpha1 is working for you.
The preview release will be identical to the official 3.1.0 release.

@Gitii
Working great! Thank you!

@tip2tail That are great new! I have created a new release 3.1.0
Thanks again for reporting the issue and feel free to suggest new features or reporting bugs in the future!


fix JWT + API key auth config generation

Right now, if wishing to use JWT auth plus API keys, the OpenAPI
config generator places the API key in a separate security
requirement, which allows JWT auth OR API key. This change adds the
API key to every existing security requirement to change OR to AND. If
no current security requirements exist, a new one is created to solely
allow API key validation.

Codecov Report

Merging #118 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##             master    #118      +/-   ##
===========================================
+ Coverage     79.68%   79.7%   +0.01%
- Complexity     1643    1646       +3
===========================================
  Files           156     156
  Lines          5523    5528       +5
  Branches        706     708       +2
===========================================
+ Hits           4401    4406       +5
  Misses          846     846
  Partials        276     276

Impacted Files
Coverage Δ
Complexity Δ

...oogle/api/server/spi/swagger/SwaggerGenerator.java
77.82% <100%> (+0.47%)
56 <0> (+3)
:arrow_up:

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 058d3ad...5f6c4b4. Read the comment docs.

Sorry for a stupid question but I'm still seeing this problem when using the "endpointsOpenApiDocs" gradle task (from within Eclipse) provided by the endpoints-framework-gradle-plugin
My build.gradle contains:
dependencies {
        classpath 'com.google.cloud.tools:endpoints-framework-gradle-plugin:1.0.2'
        classpath 'com.google.cloud.tools:appengine-gradle-plugin:1.3.4'
    }

dependencies {
    compile 'com.google.endpoints:endpoints-framework:2.0.10'
    compile 'com.google.endpoints:endpoints-management-control-appengine:1.0.6'
    compile 'com.google.endpoints:endpoints-framework-auth:1.0.6'
    compile 'javax.inject:javax.inject:1'
    compile 'javax.servlet:servlet-api:2.5'
}

Will a new version of the endpoints-framework-gradle-plugin be required as well to fix the issue?


Fix sign out after force quit in Android

Closes #373
This seems to address the issue, at the expense of not actually deleting the Realm file on signout, which I think is fine assuming we delete all the data. There are a number of other changes here to logging and the about screen. The logging might be paranoid so I can remove it if it seems excessive. I'd definitely appreciate another set of eyes on this.

Thanks for the review, @jtklein, good suggestions. I think I addressed all of them. I'm going to leave some of the paranoid logging in for now because if this happens again I really want that to be in place. The test issue is, I think, a function of Realm's general flakiness. Tests pass for me locally... most of the time, but I see errors like the one you reported occasionally, in several places. This branch increases reliance on Realm as a place to persist state, so that might have increased our exposure to Realm's flakiness.


validate env: update

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
116

Relevant Lines:
123

💛  - Coveralls


[BUG]Login problem

Compensate with donations ($15 and more) for the time I spent to fix problems or send me your Pull Request with fixes

Tron TRX/USDT: TLprxuzHjXPkHYa7ZfTTt165p7Fphwy1Ee
Bitcoin: bc1qmp7538hktvg2gx994hrpkmsmyjc03q29awe9ra
Ethereum: 0x0E674815d513d4a00F6a7Cd4A990807aB97cC70D
Dogecoin: DSEXzMBYhBTwhNqzPGxVWFsTqVsTCeDejo

Describe the bug
I have followed the instruction to change my proxy for a while. But recently I always meet one error when I tried to login in my account.
The username you entered doesn't appear to belong to an account. Please check your username and try again.
I am sure that my accounts work well.
Traceback
Status 429: Too many requests
Ignore 429: Continue login
Traceback (most recent call last):
File "/Users/yanzhang/miniforge3/envs/myenv/lib/python3.8/site-packages/instagrapi/mixins/private.py", line 298, in _send_private_request
response.raise_for_status()
File "/Users/yanzhang/miniforge3/envs/myenv/lib/python3.8/site-packages/requests/models.py", line 1021, in raise_for_status
raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: https://i.instagram.com/api/v1/accounts/login/
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/yanzhang/Projects/work/ins/ins/setting/loggin.py", line 81, in login
if not self.cl.login(self.user_name, self.passwd, relogin=False):
File "/Users/yanzhang/miniforge3/envs/myenv/lib/python3.8/site-packages/instagrapi/mixins/auth.py", line 391, in login
logged = self.private_request("accounts/login/", data, login=True)
File "/Users/yanzhang/miniforge3/envs/myenv/lib/python3.8/site-packages/instagrapi/mixins/private.py", line 447, in private_request
raise e
File "/Users/yanzhang/miniforge3/envs/myenv/lib/python3.8/site-packages/instagrapi/mixins/private.py", line 434, in private_request
self._send_private_request(endpoint, **kwargs)
File "/Users/yanzhang/miniforge3/envs/myenv/lib/python3.8/site-packages/instagrapi/mixins/private.py", line 354, in _send_private_request
raise UnknownError(**last_json)
instagrapi.exceptions.UnknownError: The username you entered doesn't appear to belong to an account. Please check your username and try again.
Expected behavior
Sometimes I can login successfully. It is very strange the next time will be prevented.

The username you entered doesn't appear to belong to an account. Please check your username and try again.

This error says only one thing - you have bad proxies


Add Package Manager with Version for GitHub Actions

What are you trying to accomplish?
This PR introduces the implementation of a package manager with versioning support for GitHub Actions workflows. The addition ensures that version information can be extracted and managed for dependencies defined in uses keys within GitHub Actions workflow files.
This change is part of improving ecosystem compatibility and dependency management for GitHub Actions workflows.
Anything you want to highlight for special attention from reviewers?

The package_manager method in the file parser extracts the version from uses keys in workflow files.
The default version is set to "0.0.0" if no valid versions are found.
This approach supports the most common use cases for GitHub Actions versioning.

If there are alternate strategies for handling multiple versions or selecting the appropriate version, I welcome feedback.
How will you know you've accomplished your goal?

Versioned package manager initialization will function correctly with GitHub Actions workflows.
Tests have been added to validate that the correct version is extracted from workflows and passed to the package manager.
Successful integration within the Dependabot framework, ensuring accurate dependency resolution for GitHub Actions.

Checklist

[x] I have run the complete test suite to ensure all tests and linters pass.
[x] I have thoroughly tested my code changes to ensure they work as expected, including adding additional tests for new functionality.
[x] I have written clear and descriptive commit messages.
[x] I have provided a detailed description of the changes in the pull request, including the problem it addresses, how it fixes the problem, and any relevant details about the implementation.
[x] I have ensured that the code is well-documented and easy to understand.

I took a quick look, and the places you pointed don't appear (at first glance) to be actually looking for files, but rather providing examples, or places where they need a placeholder file string... in that case, it's fine to just do action.yml because you have to pick something. (although personally I always prefer .yaml over .yml for readability 😄 )
Per https://docs.github.com/en/actions/writing-workflows/workflow-syntax-for-github-actions#about-yaml-syntax-for-workflows, the file name can end with either, so we need to support both... but only where we are actually finding/filtering...

I took a quick look, and the places you pointed don't appear (at first glance) to be actually looking for files, but rather providing examples, or places where they need a placeholder file string... in that case, it's fine to just do action.yml because you have to pick something. (although personally I always prefer .yaml over .yml for readability 😄 )
I haven't looked at the code in depth, but there may be times where we need a placeholder "director/file" and then later we replace that with the actual file that we found. 🤷♂
Per https://docs.github.com/en/actions/writing-workflows/workflow-syntax-for-github-actions#about-yaml-syntax-for-workflows, the file name can end with either .yaml or .yml, so you're completely right that we need to have full support for both... but only where we are actually finding/filtering...
The other reason I suspect this isn't actually a problem is that if it were, we'd be getting issues about it filed here in core, and I don't recall seeing that happening (unless it's a recent regression--I've been too busy the past six months to keep up with Core 😢 ).

Thanks for the explanation. I will check what is required and what is for place holder and come up with a solution.

I want to reiterate that I do see value in adding the "pseudo package manager" for ecosystems that don't already have one. I'm actually kinda surprised this isn't already present in Dependabot Core for GitHub Actions... maybe it is? I didn't check the source... but if it's not, then adding one would let us leverage various features over time.
🤔 although most of the things we use the package manager abstraction for are around versions of package managers... metrics on what versions are being used, then deprecating particular versions... Given that this pseudo package manager wouldn't exist in reality, then if the only benefits of a generic package manager abstraction are around managing versions, then maybe it doesn't actually make sense??
Are there other benefits to a pseudo package manager that don't involve versions?

2. strings-to-constants changes were

Thanks for the points. I will try to come up with a better approach to handle dependencies but if in metrics we want to understand github actions dependencies are only thing we can use since there is no package manager version.
I will create another PR for clean up and keep that PR only focused on package manager.

metrics we want to understand github actions dependencies are only thing we can use since there is no package manager version.

Yes, but those metrics are about package manager versions... we use them for tracking popularity of various versions of a package manager. We don't report versions on the dependencies themselves.
Since GitHub Actions has no package manager, then there's no point in reporting a package manager version for it. If there is a dashboard somewhere showing package manager versions for all ecosystems and it's breaking because there's no value for GitHub Actions, then just hardcode a placeholder version like 1.0.0... do not hoist a dependency's version into the package manager version.


chore(master): release 2.4.0

:robot: I have created a release beep boop
2.4.0 (2024-04-06)
Features

added php 8.3 support (18a078d)
separate workflow for image testing (18a078d)
switch to docker bake action (18a078d)

Bug Fixes

ansible: allow to execute playbook for single image (9847d9b)

Continuous Integration

add build action (42ad54a)
fix failing supervisord tests (d3bc887)
upgrade release-please (fe3613c)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/docker-php-dev/releases/tag/v2.4.0 :sunflower:


Move ruby_parser to a non-dev gem as it's a dep of prism

In dev mode evm:compile_sti_loader was working because this was a dev gem, but in prod, this was failing since it was missing.
@bdunne Please review.
Follow up to #23008

❗ - Line 76, Col 1 - Bundler/OrderedGems - Gems should be sorted in an alphabetical order within their section of the Gemfile. Gem ruby_parser should appear before ruby-progressbar.

I updated the code for rubocop, but:
> %w[ruby_parser ruby-progressbar rubyzip].sort
=> ["ruby-progressbar", "ruby_parser", "rubyzip"]

It looks like it is following whatever the "bash" sorting rule is for files @Fryguy
$ ls -l | grep ruby
-rw-r--r--  1 adam grare    0 May  7 10:17 ruby_parser
-rw-r--r--  1 adam grare    0 May  7 10:17 ruby-progressbar
-rw-r--r--  1 adam grare    0 May  7 10:17 rubyzip

Oh I see - bundler has a setting to ignore symbols or not and it ignores by default...so the check is roughly equivalent to:
%w[ruby_parser ruby-progressbar rubyzip].sort_by { |x| x.gsub(/[^A-Z0-9a-z]/, "") }
=> ["ruby_parser", "ruby-progressbar", "rubyzip"]


Support buildSortQuery in updateOrder

The moveToStart and moveToEnd spatie functions both build the query from the overridable buildSortQuery. The updateOrder function needs to do the same for the behaviour to be consistent with what happens on the other methods.

Seems solid, thanks a ton! Released in version 1.4.9.

Perfect thanks, this means sorting of soft deleted things now works with the following model addition :raised_hands:
    public function buildSortQuery()
    {
        return static::query()->withTrashed();
    }


Review readme, attempt setup and examples.

I've gotten this running happily on lil-bits in a virtual environment.
Can we follow the instructions on the README and make sure they run and any suggestions?
Replacing $ git clone https://github.com/bnl/pub-Maffettone_2020_MLST with $ git clone --branch pub-Maffettone_2020_MLST https://github.com/NSLS-II/bad-seeds until the repo is moved.
It may be specifically worth while to try and do this on a machine without a GPU? (The setup checks for a gpu and installs the appropriate version of tensorflow).
Lastly, it may be worth adding a bash line at the end to turn the output example data into a plot (@tacaswell).

I suggest adding a line to update pip and install wheel to avoid some warnings during installation:
$ python3 -m venv bs_env
$ source bs_env/bin/activate
$ git clone https://github.com/bnl/pub-Maffettone_2020_MLST
$ cd pub-Maffettone_2020_MLST
$ python -m pip --upgrade pip wheel
$ python -m pip install .

I had no trouble with the first demonstration program on my GPU-free laptop.
$ time python bad_seeds/examples/variable_batch_size.py
Episodes: 100%|█| 250/250 [00:26, reward=100.00, ts/ep=48, sec/ep=0.12, ms/ts=2.
Episodes: 100%|█| 250/250 [00:26, reward=0.00, ts/ep=10, sec/ep=0.03, ms/ts=2.6,

real	1m14.584s
user	1m20.924s
sys	0m11.833s

No trouble with the second demonstration program.
$ time python bad_seeds/examples/variable_time_limit.py
Episodes: 100%|█| 250/250 [00:19, reward=100.00, ts/ep=20, sec/ep=0.06, ms/ts=2.
Episodes: 100%|█| 250/250 [00:33, reward=97.50, ts/ep=40, sec/ep=0.11, ms/ts=2.7
Episodes: 100%|█| 250/250 [00:55, reward=100.00, ts/ep=70, sec/ep=0.20, ms/ts=2.

real	2m18.888s
user	2m40.482s
sys	0m22.370s

Excellent, thanks! I've made that update and also added a print statement and the expected results to the readme.


Support for Swift Package Manager?

Would it be possible to add support for Swift Package Manager? Currently this is the only 3rd party library in our app that doesn't support SPM.

I'd love to add SPM support, but to be honest, I don't want to promise anything since I have very little time to dedicate to this project.
If you're willing to work on that, your PR will be more than welcome! Seems like there was some work being done on #105. Maybe you can join forces with Jakub Mazur to tackle this?

Working on that, will submit PR in few days,
@fjcaetano I see you're a contributor in AppSwizzle repo. I just open a PR there:
https://github.com/zixun/AppSwizzle/pull/3
since it has dependency to test target here. You know if this can be merged or I should rather use yours (or mine) fork to link it with test target?

@JakubMazur I'm not a maintainer of the repo, I just had some code merged and am flagged as a contributor. Unfortunately, I don't have the permissions to approve your PR :/

OK, I point AppSwizzle to my fork, detailed explanation in #109


setState is slow

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Reproduction link

Steps to reproduce
hover 到每个节点，严重掉帧

Environment
Info

g6
4.0.3

System
macOS

Browser
Chrome

节点的 setState 过慢，单个节点 setState 耗时高达 0.5ms

setTimeout and clearTimeout in StateController take up a lot of time when updating state
https://github.com/antvis/G6/blob/master/src/graph/controller/state.ts#L100-L108

Why not using immutable to handle state? deepMix is causing a lot of performance problem.

the performance of setState is improved! welcome to try the latest version


⚠ Bofrost.at has degraded performance

In 28fc2fa, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1486 ms

Resolved: Bofrost.at performance has improved in 08295af.


How register some strange holidays

Hello,
I am working on Argentina Holidays, and I have an issue.
In addition to 'traditional holidays', we have some holidays named 'Non-work day' for some religions and 'Bridge holidays' or 'tourist holidays'. The last, is not fixed each year. How could I register this holidays?
Regards!

Well, there should be a way to "predict" those holidays.. or are they just defined at the beginning of each year?
If you can predict them, you should provide a complete algorithm to do so.. otherwise, I think they can't be supported..
Please let me know, thank you very much


演示应用更新到 1.9 后无法保存到手机存储。

如题。

the same

Issue was fixed in commit 2bc1878bebcde5767c227f13d7370fc573522287.
Sorry for my late response and thanks for the feedbacks.


Expose World3D.compositor

this closes https://github.com/godotengine/godot-proposals/issues/9815
I'm positive this just got missed in the Compositor implementation, since i could find no mention of its omission or any reasons for it (and it only makes using World3D resources more cumbersome).

This will be overridden by the one in the first WorldEnvironment currently:
https://github.com/godotengine/godot/blob/be56cab58c056c074d1e02cd0b38641204e39f41/scene/3d/world_environment.cpp#L98-L107

@AThousandShips that is intended+documented, just like the environment and camera_attributes.

The documentation should be updated on the World3D at minimum

@AThousandShips i'm not sure i understand what you mean, i took the documentation of the other properties that work exactly the same way and replaced the names. how should this be documented differently?

(You don't need to tag me I'm right here, just adds annoying pings)
It should match the part with "The default ... if not set on Camera3D." of camera_attributes
The rest is out of scope for this but this is a very important thing to note, or people will be confused when the compositor they assigned is suddenly replaced when a camera or WorldEnvironment enters the scene

the issue there is that the Camera3D logic does actually preserve the default value while tracking the temporary override by "enabled" cameras, while WorldEnvironment just writes all over all of those properties directly. I think I'll add a "this will be overwritten by WorldEnvironment nodes in the world", the "default" implies it stays the default.

I'd say it doesn't imply that and that it should be uniform, so I'd say the documentation should be the same just with Compositor instead


PLease add `.waifutransfer' command

:^)
Have to be able to give your waifu to someone else once you get tired of her, and you feel bad about dumping her back from the ghetto you picked her from.

Added in 2.3.6


BaSTI missing logG for most entries

The included BaSTI isochrones appear to have values of -99 for most of the isochrone logG values (see e.g. isoc_z0.0200.dat). Only the added on remnant populations do not have this value. I was curious because the BaSTI website (old and new) also do not provide logG at all, and since it also does not provide a stellar radius I don't see how you can compute it from the isochrones directly.
Looking through the FSPS code I cannot see an obvious answer to how it works round this limitation when it comes to interpolating spectra onto the isochrone grid. Could somebody perhaps explain or point me in the right direction (or is this actually a bug?)

I should add- logG is derivable from other properties, but I can't see this happening inside of FSPS. I suspect I just haven't found the relevant part of the code, that's all.


Application Insights for Dev is down

Description
The dev instance of application insights stopped receiving data on 5/20/2021
Steps to reproduce

Go to portal.azure.com/.../prime-simple-report-dev-insights
There is no data

Expected behavior
There should be data

This does not appear to be an issue with the insights connection string (at least, the value currently in use in the deployed app matches the value in the console).

Agreed with @benwarfield-usds - are we sure anyone is actually using dev.simplereport.gov?  I just clicked around for a while and I see data now in App Insights:


Implement logging library

Should be same procedure as in the KeyMaster project. I'll get to it when I feel up for it sometime

Will mark this as closed soon!


Please invite me to the GitHub Community Organization

Name
Jose Long
Discord Username (if applicable)
1084785679365586987
Additional Context
Dear Organization Administrator!
I hope this message finds you well. I am writing to express my sincere interest in joining the "Jaidevstudio" organization on GitHub. After carefully reviewing your organization's projects and objectives, I am impressed by the high-quality work and collaborative spirit exhibited by your members.
Allow me to provide some relevant information about myself:
I have been actively involved in the design and development field for the past 10 years. Throughout my professional journey, I have gained extensive experience in web development, working on various projects. My background includes a solid understanding of technical stacks, enabling me to contribute effectively to design and development efforts.
I possess a diverse skill set, which I believe can contribute significantly to the ongoing and future projects undertaken by the "Jaidevstudio" organization.
I am particularly excited about the opportunity to collaborate with like-minded professionals in the "Jaidevstudio" organization. The chance to work alongside talented individuals, share knowledge, and collectively raise the bar in design and code excellence is genuinely inspiring to me. I am eager to contribute my skills and insights to the organization's projects, as well as learn from the collective expertise of its members.
I am confident that my dedication, strong work ethic, and collaborative mindset would make me a valuable asset to the "Jaidevstudio" organization. I am open to any selection or evaluation process you may have in place, and I am willing to provide any additional information or samples of my work that you may require.
Thank you for considering my application to join the "Jaidevstudio" organization. I look forward to the opportunity to contribute and learn from the talented individuals within the organization. Please feel free to reach out to me if you have any further questions or require additional information.
Best regards, Jose Long

It's great having you contribute to this project
If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord Server and our GitHub Organisation - we help and encourage each other to contribute to open source little and often 🤓 . Any questions let us know.

Invitation sent for the GitHub Organisation. Welcome to the community 🎉#
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see, you can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/jaidevstudio/people
Like this

Tips for practising:
Remember contributing to open source is not just about code, its about collaboration, communication and adding value
I hope that helps

Please check your email or organisation page 🙏


*: drop go 1.18 support, add go 1.21 support

Close #2626, close #2627, close #2629 and update all workflows/images.
TODO:

[ ] Use flag.TextVar where possible.
[ ] Update interop deps.

@roman-khimov, I'm still not sure that it will be useful for us integrate flag.TextVar into our repo. Let's take as an example flags.Address and flags.AddressFlag structures and wallet change-password command that uses --address flag:
First of all, we can't completely remove flags.AddressFlag structure because urfave.cli requires a full set of cli.Flag flags to be provided to the command constructor for proper parsing and documentation displaying:
https://github.com/nspcc-dev/neo-go/blob/b284b902d6d6dec3d666e3f140f906e14ff6beb9/cli/wallet/wallet.go#L133-L145
After that, if we're trying to replace flags.Address structure by a single *util.Uint160 value with TextMarshaller interface defined on it, then we can get the following diff:
diff --git a/cli/flags/address.go b/cli/flags/address.go
index f20e9948..52736c10 100644
--- a/cli/flags/address.go
+++ b/cli/flags/address.go
@@ -10,53 +10,20 @@ import (
        "github.com/urfave/cli"
 )

-// Address is a wrapper for a Uint160 with flag.Value methods.
-type Address struct {
-       IsSet bool
-       Value util.Uint160
-}
-
 // AddressFlag is a flag with type string.
 type AddressFlag struct {
        Name  string
        Usage string
-       Value Address
+       Value *util.Uint160
 }

 var (
-       _ flag.Value = (*Address)(nil)
-       _ cli.Flag   = AddressFlag{}
+       _ cli.Flag = AddressFlag{}
 )

-// String implements the fmt.Stringer interface.
-func (a Address) String() string {
-       return address.Uint160ToString(a.Value)
-}
-
-// Set implements the flag.Value interface.
-func (a *Address) Set(s string) error {
-       addr, err := ParseAddress(s)
-       if err != nil {
-               return cli.NewExitError(err, 1)
-       }
-       a.IsSet = true
-       a.Value = addr
-       return nil
-}
-
-// Uint160 casts an address to Uint160.
-func (a *Address) Uint160() (u util.Uint160) {
-       if !a.IsSet {
-               // It is a programmer error to call this method without
-               // checking if the value was provided.
-               panic("address was not set")
-       }
-       return a.Value
-}
-
 // IsSet checks if flag was set to a non-default value.
 func (f AddressFlag) IsSet() bool {
-       return f.Value.IsSet
+       return f.Value != nil
 }

 // String returns a readable representation of this value
@@ -86,7 +53,7 @@ func (f AddressFlag) GetName() string {
 // Ignores errors.
 func (f AddressFlag) Apply(set *flag.FlagSet) {
        eachName(f.Name, func(name string) {
-               set.Var(&f.Value, name, f.Usage)
+               set.TextVar(f.Value, name, f.Value, f.Usage)
        })
 }

It's true that such replacement simplifies our flags package code, but at the same time we have to manually perform some utility conversions in the code:
diff --git a/cli/wallet/wallet.go b/cli/wallet/wallet.go
index f5051efb..700746fe 100644
--- a/cli/wallet/wallet.go
+++ b/cli/wallet/wallet.go
@@ -355,10 +355,10 @@ func changePassword(ctx *cli.Context) error {
        if len(wall.Accounts) == 0 {
                return cli.NewExitError("wallet has no accounts", 1)
        }
-       addrFlag := ctx.Generic("address").(*flags.Address)
-       if addrFlag.IsSet {
+       addrFlag := ctx.Generic("address").(*util.Uint160)
+       if addrFlag != nil {
                // Check for account presence first before asking for password.
-               acc := wall.GetAccount(addrFlag.Uint160())
+               acc := wall.GetAccount(*addrFlag)
                if acc == nil {
                        return cli.NewExitError("account is missing", 1)
                }
@@ -370,7 +370,7 @@ func changePassword(ctx *cli.Context) error {
        }

        for i := range wall.Accounts {
-               if addrFlag.IsSet && wall.Accounts[i].Address != addrFlag.String() {
+               if addrFlag != nil && wall.Accounts[i].Address != address.Uint160ToString(*addrFlag) {
                        continue
                }
                err := wall.Accounts[i].Decrypt(oldPass, wall.Scrypt)

So my question is: am I got the idea right, is it the code that we'd like to change? The same logic is related to the Fixed8 flag.

@AnnaShaleva, if it doesn't fit well --- forget about it. The idea was to drop these flags, if there are valid reasons why we can't do that then OK, we don't do it. We've got #3097 as well for the future.


ResourceUnavailable: Program 'winget.exe' failed to run: An error occurred trying to start process. Access is denied.At line:1 char:1

Brief description of your issue
After upgrading to 22H2 (22621.1344), i am unable to run winget.
Steps to reproduce

Open a Terminal (as admin or normal user)
Execute winget

Expected behavior
Winget should work
Actual behavior
ResourceUnavailable: Program 'winget.exe' failed to run: An error occurred trying to start process 'C:\Users\Ferdinand\AppData\Local\Microsoft\WindowsApps\winget.exe' with working directory 'C:\Users\Ferdinand'. Access is denied.At line:1 char:1
+ winget
+ ~~~~~~.

Environment
Windows 11: 22H1 (22621.1344)
Im unable to run the winget version command, due to the above command

If you install the latest stable release, does it get fixed?
https://aka.ms/getwinget

Yes, a reinstall fixed it

Yes, a reinstall fixed it

Thanks, at least we know we have a workaround until we find a root cause.

After rebooting, the issue returns.
Also im unable to re-install it now:

@denelon:
I got the same error.
Probably some permissions that got messed up, so i would like to try a clean re-install.
Too bad there is no way to unistall winget.

The "for proper functioning" UI is a false negative. That message has no bearing on whether WinGet "should" work or not. You might try doing a downgrade to the earlier version and then do a winget -v to make sure it worked followed by an "upgrade" to the version you want.
I believe we've seen an example where it looked like more than one version of App Installer was "registered" on a system, and we need to figure out what causes that so we can avoid that particular problem as well.

FYI we just released WinGet 1.5-ReleaseCandidate(1) and WinGet 1.6-preview(1) to GitHub.

I installed the previous 1.5 release 2 days ago to fix the issue with winget dirs not beeing accessible.
After that i ran into this permission issue (refering to the main issue here, not Scarjits comment) and reinstalling the 1.5 release gets blocked because "App Installer is already installed".
My winget was unusable for 2 weeks or something now and I don't saw enough benefit in further trying to solve this for me.
So I disabled winget yesterday as far as possible and switched back to using choco, to stay productive.
Thus i wont try the new one for now.
Maybe I'll take another try when winget matured into a state thats usable on a productive system, or at least has a way to do a clean reinstall in such cases.

Yeah so I ran into this same issue today. I'm not sure if it started happening after an update or not as I don't run winget often. Not being able to reinstall winget from the MSIX just like people attempted above is kind of makes this a pain. I could attempt to forcefully uninstall and reinstall Microsoft.DesktopAppInstaller through PowerShell but I'm not quite sure if that's a good idea and if that could make things worse.

If you install the latest stable release, does it get fixed?
https://aka.ms/getwinget

I have the same issue. And reinstall winget fixed the issue.

You should try to use the user path insted of the more general path, in your environment variables
I've changed this:

C:\Program Files\WindowsApps\Microsoft.DesktopAppInstaller_1.20.2201.0_x64__8wekyb3d8bbwe

to

C:\Users<my-user-folder>\AppData\Local\Microsoft\WindowsApps\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe

and it works

Just had same the problem.  Nothing above helped. What fixed it for me was installing 1.7.3481-preview and then downgrading to latest release again 🤷♂


Datetimes on some datapoints are inaccurate

...

Datetimes are the time of the start of the server uptime, meaning the time is a keyword argument set to a static value upon load


DX: Composer's branch-alias leftovers cleanup

Cleanup after https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/4424.

Coverage remained the same at 93.034% when pulling fd83aab524a68bff42522743069bd017c520bfed on werlos:branch_alias_cleanup into f9d15af98faaa90f1eeb3db49cab6def2cb4f40b on FriendsOfPHP:master.


Subscription question

Hey
first of all thanks for the great package. This is not a real issue report more a question. The Mollie API does have something like:
$customer->createSubscription([             "amount" => [                   "currency" => "EUR",                   "value" => "25.00",             ],             "times" => 4,             "interval" => "3 months",             "description" => "Quarterly payment",             "webhookUrl" => "https://webshop.example.org/subscriptions/webhook/",          ]);
which creates a real subscription at mollie with ids like: sub_123456789 as Abonnement reference.
So my question is does this package creates a real subscription inside mollie? If not how the recurring payment works exactly?
best regards

Hi,
Thanks for the ❤.
Cashier Mollie does not use the Mollie Subscriptions API, but the Payments API. For more details I think this documentation answers your question.

PS. We're not using Mollie's subscription API because it cannot support Cashier's more advanced use cases.

Thanks for the quick answer i willl take a look.


Update to conda-build 3 and conda-smithy 3

Hi! This is the friendly automated conda-forge-webservice.
I've rerendered the recipe as instructed in #1.
Here's a checklist to do before merging.

[x] Bump the build number if needed.

Fixes #1

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@CJ-Wright, do we need to do anything extra for the bot to notice this requires a rebuild with the new compilers?

It looks good, let's see if the bot picks it up in 30 mins

Looks like it did. ( https://github.com/conda-forge/chrpath-feedstock/pull/3 ) 😄


[BEAM-8335] Add a TestStreamService Python Implementation

For prior discussion see https://github.com/apache/beam/pull/9720

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Friendly ping

Friendly ping


Animated.createAnimatedComponent(Stop) throws error

Bug
import Svg, { Defs, LinearGradient, Stop, Path } from 'react-native-svg';
const AnimatedStop = Animated.createAnimatedComponent(Stop);

throws
ERROR  Invariant Violation: Unable to locate attached view in the native tree

This error is located at:
    in PathIcon (at DepositWithdraw.tsx:18)

Unexpected behavior
"react-native-svg": "12.2.0",
"react-native": "0.66.4",
Environment info
System:
    OS: macOS 12.2.1
    CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 240.96 MB / 32.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 14.18.1 - /usr/local/bin/node
    Yarn: 1.22.17 - /usr/local/bin/yarn
    npm: 6.14.15 - /usr/local/bin/npm
    Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.11.2 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 21.2, iOS 15.2, macOS 12.1, tvOS 15.2, watchOS 8.3
    Android SDK: Not Found
  IDEs:
    Android Studio: 2021.1 AI-211.7628.21.2111.8092744
    Xcode: 13.2.1/13C100 - /usr/bin/xcodebuild
  Languages:
    Java: 11.0.14 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 17.0.2 => 17.0.2
    react-native: 0.66.4 => 0.66.4
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Same here, did you solve it?

Nope

Stop element does not render anything (https://github.com/react-native-svg/react-native-svg/blob/d859083687fe44f3939b1868f5688306bee35651/src/elements/Stop.tsx#L19), but is used in its parent: https://github.com/react-native-svg/react-native-svg/blob/d859083687fe44f3939b1868f5688306bee35651/src/lib/extract/extractGradient.ts#L64. Therefore, you cannot createAnimatedComponent with it. In order for it to work, you would have to implement some hacky solution with providing the extracted values directly to RNSVGLinearGradient component instead of using Stop component for it. Can I close this issue then?

I can confirm it's possible to animate the native gradient prop directly on LinearGradient. Thanks @WoLewicki for the idea, works great.

I can confirm it's possible to animate the native gradient prop directly on LinearGradient. Thanks @WoLewicki for the idea, works great.

Do you have an example of how to extract the values onto RNSVGLinearGradient? I'm unable to get this to work.


Three missing roms

These games show missing rom, as of sync just now:
Nova 2001
Pac-Man Plus
Magical Pumpkin:Puroland de Daibouken

The rom download code was ported over from update_all. If update_all also can't find these roms, it's likely archive.org doesn't have them. If update_all can find these roms, then that is a bug in ammister.

Ok update_all at least found magical pumpkin, so AMMister is behind here.

update_all has a new rom db it uses. I just asked if it was ok to also use it in ammister. It would be a big improvement.
https://github.com/theypsilon/ArcadeROMsDB_MiSTer/issues/1

Version 0.30.0 now uses theypsilon's rom db. If he doesn't want me to, I'll remove it and implement my own solution (only reason I'm a stickler on that is that db has no license).

Magical Pumpkin: now downloaded
Nova 2001: also missing in update_all, so not pursuing (at least not yet)
Pac Man Plus: this is an hbmame rom, something AMMister does not support. It's now been removed from the catalog.

Nothing against hbmame, just haven't gotten there yet.


Bump org.openrewrite.maven:rewrite-maven-plugin from 5.27.0 to 5.30.0

Bump org.openrewrite.maven:rewrite-maven-plugin from 5.27.0 to 5.30.0
Fixes #16418
Description
Bump org.openrewrite.maven:rewrite-maven-plugin from 5.27.0 to 5.30.0
I have tested integration-tests successfully locally using -Dgroups=query-erro
Upgrade rewrite-maven-plugin
Release note
Upgrade org.openrewrite.maven:rewrite-maven-plugin from 5.27.0 to 5.30.0

Key changed/added classes in this PR

pom.xml

This PR has:

[x] been self-reviewed.
[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)
[ ] added documentation for new or modified features or behaviors.
[ ] a release note entry in the PR description.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[x] been tested in a test Druid cluster.

I'm also not sure about Xmx changes - but https://github.com/apache/druid/pull/16477 have updated this to 5.31.0
so I think this might not be needed anymore


No examples or documentation of what can be filtered / passed in config

Maybe it has been posted somewhere or there is a file that I am missing but I am unable to find information (documentation) on how to use the config file.
I know of the file in the test/ folder:
https://github.com/casewalker/narrator-configs-mod/blob/main/src/test/resources/narratorconfigsmod.json
but this is very rudimentary. I understand they are some kind of object references (chat.type.advancement.) but where can I find a full list of these things?

Hello! This is the wrong mod for this issue (you are referring to the Narrator Configs Mod, so I will close it now.

Yes that makes sense. Sorry about my mistake!

Okay I have written an issue ticket +some comments on the correct git page.
No matter what I do with the config file I can't get anything to work.


Add tests for navigating language menu with keyboard

Is your feature request related to a problem? Please describe.

Thanks a ton for fixing those lint errors - it's looking way better.
A couple of things went a bit wrong, though.
The keyboard navigation is broken - probably because of the changes to doKeyPress, since case matters when it comes to event.key.
This also didn't trip up any tests and I feel like it should have. Doesn't need to be much, but checking that the language selector opens with enter and can be navigated by the arrow keys would be good.

See comment 1

I was going to try and add some tests to this PR - I found that cypress doesn't support pressing the tab key. There's a helper plug-in we can use, but I didn't want to expand this PR that much right now. I say we go with this as it is and I can make a separate PR with some tests.

and comment 2

@moT01  assign me i can work

We don't typically assign issues @Aman123lug - the first PR that fixes the issue will be accepted. Feel free to work on it and create a PR 👍

@moT01  okay thanks

We don't typically assign issues @Aman123lug - the first PR that fixes the issue will be accepted. Feel free to work on it and create a PR 👍

Fine, then I'll work on it too.

Hi @moT01, could you please explain the feature which needs to be implemented or the final goal of this issue?
On official website, I have tried navigating the language menu using the keyboard, it was working fine.
By pressing the enter, the language selector can be opened and can be navigated by pressing key-up and key-down.
What I understand is this issue is only about adding navigation-related tests to header.test.tsx file. No need to change the code inside components folder.
Am I right in my understanding?

this issue is only about adding navigation-related tests

This is correct. The keyboard functionality for the menu and language submenu is fine now, nothing needs to be fixed there. The first part of this issue was explaining that at one time the keyboard functionality was accidentally broken and that we would have some tests in place to test for that so that we don't accidentally break it again and not notice. It also suggests the bare minimum we would need to test.

@moT01 and others, I have proposed PR #47791 for this issue. Please, have a look if that is acceptable or not.

Hi @Sboonny, is this issue available again for contributions? I'd like to work on this.

@Prince-Mendiratta, you have to communicate with @saw-jan, so both of don't get in each other ways while improving the test.
But the issue won't be closed after their PR, there are still tests need to implemented based on this comment
https://github.com/freeCodeCamp/freeCodeCamp/pull/46644#issuecomment-1210022946

I could create a branch based off @saw-jan 's PR and send a new PR with all the new tests so we could have all the tests added in one PR and the majority of relevant discussion being there if @saw-jan is alright with this?

Hey I have written tests as pr #46644 comment

[x] When the main navigation menu is expanded and keyboard focus is on an item in main menu, the Esc key should close the menu and put keyboard focus on the Menu button.

[x] When focus is on the Change Language item in the main menu, the up arrow should open the lang menu and place focus on the last item in the lang menu.

[x] When focus is on the Change Language item in the main menu, the down arrow should open the lang menu and place focus on the first item in the lang menu.

[x] When lang menu is opened and keyboard focus is automatically moved into lang menu, the up and down arrow keys should move the keyboard focus up/down one menu item.

[x] In the lang menu, when focus is on the last item, arrow down should move focus to first item. Similarly, when focus is on the first item, arrow up should move focus to last item.

[ ] When focus is on an item in the language menu, Shift + Tab should close the lang menu and put focus on the item immediately above the Change Language item.

[ ] When focus is on an item in the language menu, Tab should close the lang menu and put focus on item immediately after the Change Language item, or if no such item exists then put focus on the next element that can receive focus on the page (which is the Sign In button at this time).

[x] When focus is on any item in lang menu, the Esc key should close the lang menu and put focus back on the Change Language menu item.

I was unable to write 2 tests as cypress don't support tab key yet. need tab support. This issue is still open.
I also tried to cypress-real-events it didn't workout.
Do let me know how shall I proceed ahead.

I could create a branch based off @saw-jan 's PR and send a new PR with all the new tests so we could have all the tests added in one PR and the majority of relevant discussion being there if @saw-jan is alright with this?

I have added the additional tests in PR https://github.com/freeCodeCamp/freeCodeCamp/pull/47791#issuecomment-1274069643


Please release a new version

So we do not have to pin to "master" for anything building with Xcode 10

Since there's scant active development, the argument can go in either direction: release an arbitrary version and upload a binary for convenience, or don't, because there was virtually no progress.
@mjarvis I don't think it hurts to release 0.6.1 to make life easier for folks using Swift 4.2. Do you have objections?

I don't object to publishing a new release. However, @Ben-G is the only person with permissions for the cocoapods trunk entry for this (see pod trunk info ReSwiftRouter)

I get this error in my terminal please
   ReSwiftRouter (from `https://github.com/ReSwift/ReSwift-Router.git`) was resolved to 0.6.0, which depends on
      ReSwift (~> 4.0.0)

Specs satisfying the `ReSwift, ReSwift (= 4.1.1), ReSwift (~> 4.0.0)` dependency were found, but they required a higher minimum deployment target.

Preparing for this in #117.

Sorry about that!
Can I add either/both of you folks?

On Nov 20, 2018, at 09:45, Malcolm Jarvis<EMAIL_ADDRESS>wrote:
I don't object to publishing a new release. However, @Ben-G is the only person with permissions for the cocoapods trunk entry for this (see pod trunk info ReSwiftRouter)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Yes, seems you've already done so. Thanks, @Ben-G :)


Admins can check Mentor Reviews

Description
Added a component for admins to check reviews that Mentors have given mentees.
Video Link
(https://www.loom.com/share/a1b537a8f55b40068aedeedeec79e75a)
Trello Link
(https://trello.com/c/LcOnE80B/152-as-an-admin-i-want-to-know-how-many-positive-reviews-a-mentor-has-given-a-mentee-so-that-i-evaluate-the-performance)
Type of change

[x ] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have removed unnecessary comments/console logs from my code
[x] My changes generate no new warnings
[x] I have checked my code and corrected any misspellings
[x] No duplicate code left within changed files
[x] Size of pull request kept to a minimum
[x] Pull request description clearly describes changes made & motivations for said changes

This is well scoped and clean. Why not include more information about the mentor when the table is not collapsed? It would be nice to be able to see who all of their mentees are and encourage more reviews for specific mentors. Check the linter warnings. This is a promising feature!

Thanks! The ticket specifically asked for just positive reviews from the mentor of the mentees. I didn't have much backend to work with. I'd like to have the names of each mentee and mentor as it would be way cleaner. As far as the lint error, it has something to do with Node and npm. I reached out about it. Do you have any clue?

Everything looks good, keep up the good work!

Looks great! I can't wait to see what it looks like when backend comes through!


[WFLY-9291] Delete IronjacamarParser and iand related test.

Jira: https://issues.jboss.org/browse/WFLY-9291
EAP Jira: https://issues.jboss.org/browse/JBEAP-12977

EAP PR: https://github.com/jbossas/jboss-eap7/pull/2359


add client for predis

Thank you very much for you PR @flug !  just 2/3 small changes and we're good to go


Zoom on hover or click

Hi hydescarf,
I really like this theme! Thank you for providing it for Obsidian!
Could it be possible to add the functionality to enlarge (pop out) embedded images when clicking or hovering over them?
I tried with the snippet from Minimal Theme but my noob css skills couldn't get it to work.
Khanh

Since this doesn't really fit with the theme, we'll just go with snippet for this.
That being said, the code from where you've linked works fine as it is. Just make sure you don't have other snippets interfering with it.
If you want to hover it instead of clicking(and hold) it, replace the word :active to :hover and it should work as well.
Let me know if you still need any help with that!

You are right. I testet it with an excalidraw embed which does not work but on normal images it works.
I tried to change the selector .image-embed to excalidraw-svg:hover but it just flickers when hovering over an embedded excalidraw drawing.
Selectors:

Selector: .view-content .markdown-preview-view .excalidraw-svg:hover
Selector: .view-content .excalidraw-svg:hover img

Thank you!

I tried out excalidraw, its drawing mode won't work at all as it uses canvas (and not html tags) to draw the components. But its markdown mode works, just as you've tried.
I don't see the flickering part, but there's one thing you'd need to add if you want it to zoom bigger when hover:
.view-content .excalidraw-svg:hover img {
  top:50%;
  transform:translateY(-50%);
  padding:0;
  margin:0 auto;
  width:100%;
  max-width:100%!important;
  max-height:100vh;
  object-fit:contain;
  left:0;
  right:0;
  bottom:0;
  position:absolute;
  opacity:1;
}

I've added max-width:100%!important as excalidraw limits it to 400px max.

Don't really know why it flickers in my vault even when its the only snippet activated. I don't want to further waste your time so I'll just open the pictures in a new tab when needed.
But really appreciated your help! :)

It's alright!
It could also be possible that you have other plugins enabled, or your obsidian app is not up-to-date (there's two type of version, the app and the installer, where you'd need to reinstall manually if the installer version is currently not >1.1.9).
I tried out with a new testing vault and it's working fine on my side. If you have a video showing how it flickers, it might help!
On the other hand, there's an official discord community for obsidian you can check out! There's a section where people ask for styling help and that might help you, too!

I tried it too with a new vault. It works when its in reading view but it flickers if its in live-preview mode.

Haven't thought of asking on discord even though I use it daily xD. Will try it there too. Thanks again!


run Demo.py visulization question

`(openpcdet) thiesy@ubuntu-MS-7C75:~/OpenPCDet/tools$ python demo.py --cfg_file cfgs/kitti_models/pointpillar.yaml --ckpt pointpillar_7728.pth --data_path 000008.bin
2023-10-26 13:46:21,971   INFO  -----------------Quick Demo of OpenPCDet-------------------------
2023-10-26 13:46:21,971   INFO  Total number of samples:        1
/home/thiesy/miniconda3/envs/openpcdet/lib/python3.8/site-packages/torch/functional.py:445: UserWarning: torch.meshgrid: in an upcoming release, it will be required to pass the indexing argument. (Triggered internally at  /opt/conda/conda-bld/pytorch_1639180588308/work/aten/src/ATen/native/TensorShape.cpp:2157.)
return _VF.meshgrid(tensors, **kwargs)  # type: ignore[attr-defined]
2023-10-26 13:46:23，601   INFO  ==> Loading parameters from checkpoint pointpillar_7728.pth to CPU
2023-10-26 13:46:23,613   INFO  ==> Done (loaded 127/127)
2023-10-26 13:46:23,626   INFO  Visualized sample index:        1
No protocol specified
[Open3D WARNING] GLFW Error: X11: Failed to open display :0.0
[Open3D WARNING] Failed to initialize GLFW
Traceback (most recent call last):
File "demo.py", line 112, in
main()
File "demo.py", line 100, in main
V.draw_scenes(
File "/home/thiesy/OpenPCDet/tools/visual_utils/open3d_vis_utils.py", line 49, in draw_scenes
vis.get_render_option().point_size = 1.0
AttributeError: 'NoneType' object has no attribute 'point_size'`

请问你解决了嘛

请问你解决了嘛

一直没有解决啊

Hi @thiesymon,
I am facing the same issue. Could you solve it?
Thanks.

same

大家是不是都是在docker下运行出现的这个问题。


Highlighting breaks on alias before let

The following breaks:
alias less = bat
let foo = $'($env.HOME)'

It's fine when alias is removed.

here's how it looks like

I'm trying to understand if this is in an editor or on the repl. This is what I get on the repl.

and in vscode

I tested it in Neovim. It looks fine for me on the REPL.

ok, thanks. This is what I get in helix with edbb7ddb42205aa040b7b1255b9cf56e387c433f commit.

I don't have tree-sitter-nu installed in nushell.

I don't have tree-sitter-nu installed in nushell.

Ah nor do I. Thanks, I wasn't aware it could be installed for Nushell.

oops, I meant neovim. Sorry.


execute a request and immediately stop it, the request remain in status "sttoping"

Found the source, now we have one more service - pipeline-driver-queue before the job arrived to the pipeline-driver


Instance: Improve exit status and error handling in exec

Improves exit status detection when running lxc exec on a VM:

Detects missing parent directory and returns 127 exit status.
Detects permission denied and returns 126 exit status.

And for both containers and VMs if exit status 126 (command not executable) or 127 (command not found) occurs then an operation error is returned and lxc exec command will now display it, whilst still maintaining the exit status of the command.
Fixes #10534
Signed-off-by: Thomas Parrott<EMAIL_ADDRESS>

There's actually a difference in behaviour in Go's exec PATH logic compared to bash's:
In Bash if a file exists in a PATH but is not executable we get 126 exist status:
touch /usr/bin/foo
foo; echo $?
bash: /usr/bin/foo: Permission denied
126

But in Go's exec.Command.Start() we get a not found error because of the exec.LookPath() function which skips any non-executable files and moves onto the next PATH location, eventually returning a not found error:
https://cs.opensource.google/go/go/+/refs/tags/go1.19.2:src/os/exec/lp_unix.go;l=40-66

The liblxc fix for permission denied errors is here https://github.com/lxc/lxc/pull/4207

Testsuite passed


[WIP] Testing the async API

We want to make sure that responses that don't take the same time to execute still are returned to the proper request

Ok, we finally did a simple reference comparison of the handler before and after going through an elm port to make sure it's still exactly the same, and it is:

This diff results in the message equal? true in the console, which confirms that the handler is the same, and that there's no issues with async requests that way.
Case closed.


fix: COMPASS-4209: Add missing returns on error cases

@mmarcon while I'm not able to reproduce this issue in Compass, this PR adds guards and additional error debugging so we'll have more information aside from callback already called.

Fair enough, LGTM. I just tried the most recent beta and could not reproduce it either.


rotate(0) and rotate(0,false) both manipulate the image and probably shouldn't

Expected Behavior
Console should output:
[252,71]
[252,71]
Current Behavior
Tried this on multiple images, error happens in both:
[252,71]
[254,72]
Failure Information (for bugs)

Steps to Reproduce
let jimp = require('jimp');

let x = jimp.read('myImage.png').then(
  image => {
    console.log([image.bitmap.width, image.bitmap.height]);
    image.rotate(0);
    console.log([image.bitmap.width, image.bitmap.height]);
  }
)

Context

Jimp Version: 0.16.1
Operating System: macOS Mojave Version 10.14.1
Node version: 14.12.0

Failure Logs

:rocket: Issue was released in v1.1.2 :rocket:


Clean data in column

Is there a Sample on How to Clean all String column to remove special characters like , \n etc ?

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

ID: 84a9bf80-52b7-1c0c-74aa-1d624b65ccf2
Version Independent ID: bcbecc73-1cbd-0621-4bf4-6b4f21cb6595
Content: Clean and prepare data for Azure Machine Learning - Team Data Science Process
Content Source: articles/machine-learning/team-data-science-process/prepare-data.md
Service: machine-learning
Sub-service: team-data-science-process
GitHub Login: @marktab
Microsoft Alias: tdsp

@nssidhu If you are using Azure Machine Learning Studio you can use the module apply-sql-transformation to write SQL queries where junk/ascii data can be replaced using REPLACE
A simple example for this sample data i uploaded as dataset in studio:

Junk_Data
Junk_Values

Roh!T
,Microsoft

Mungi
\nAzure

I used the following query in apply-sql-transformation module to replace '!' in the first column.
select replace(t.Junk_Data,'Roh!T','Rohit') new_junk_data,t.* from t1 t;
The new column returned with clean data when I visualized data of apply-sql-transformation.

Similarly you can try to remove the junk data from your dataset by using this module and process it further.

@nssidhu We will now proceed to close this thread. If there are further questions regarding this matter, please tag @RohitMungi-MSFT in your reply. We will gladly continue the discussion and we will reopen the issue.


New UX

:tada: This issue has been resolved in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Code style Firewall Rules

This one cannot be automatically merged, can you please rebase your fork @phil-davis ?

Should be integrated now.

I have a small fix for the change to show selected interfaces on the Floating Rules tab.
When there are no floating rules defined, the box that says this only extends over 10 columns, but now that tab has 11 columns.
The fix is on top of this change, so it will be easy to submit a pull request for it once this pull request is committed.


error log that might have something to do with random stops

`2021-01-30 18:48:37,337:ERROR:Task exception was never retrieved
future: <Task finished name='Task-14' coro=<TradeWorker.check_trade_loop() done, defined at trade_worker.py:46> exception=ConnectTimeout('')>
Traceback (most recent call last):
File "httpx_exceptions.py", line 326, in map_exceptions
File "httpx_client.py", line 1502, in _send_single_request
File "httpcore_async\connection_pool.py", line 218, in arequest
File "httpcore_async\connection.py", line 92, in arequest
File "httpcore_async\connection.py", line 118, in _open_socket
File "httpcore_backends\auto.py", line 44, in open_tcp_stream
File "httpcore_backends\asyncio.py", line 243, in open_tcp_stream
File "contextlib.py", line 135, in exit
File "httpcore_exceptions.py", line 12, in map_exceptions
httpcore.ConnectTimeout
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "trade_worker.py", line 50, in check_trade_loop
File "user.py", line 136, in get_trade_status_info
File "httpx_client.py", line 1548, in get
File "httpx_client.py", line 1371, in request
File "httpx_client.py", line 1406, in send
File "httpx_client.py", line 1444, in _send_handling_auth
File "httpx_client.py", line 1476, in _send_handling_redirects
File "httpx_client.py", line 1502, in _send_single_request
File "contextlib.py", line 135, in exit
File "httpx_exceptions.py", line 343, in map_exceptions
httpx.ConnectTimeout
`

01 30 2021 16 58 20.log

01 30 2021 16 58 20.log


Deprecation warning on locate

In the new typst 0.12.0-rc1, I started getting this warnings (image below)

I think that did not fix, I am still getting this

Whoops! irresponsible maintainer alert

Somehow when I released v0.1.3, it didn't actually include the commits that implemented the fix… so I've had to release them as v0.1.4, see https://github.com/typst/packages/pull/1197 🤡


Missing code highlight in "Update the Delete Razor page" section

The code block shown in this seciton replaces FirstMidName with FullName in the Administrator field. However, its corresponding row (@Html.DisplayFor(model => model.Department.Administrator.FullName)) is not highlighted in yellow/green. Hope this can be fixed so that readers can more easily see which part of the code they need to change.

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

ID: 16a5d321-117b-c37c-6ac9-b56009e7ead7
Version Independent ID: e337d2e1-7447-634a-d11a-511247e0f897
Content: Razor Pages with EF Core in ASP.NET Core - Concurrency - 8 of 8
Content Source: aspnetcore/data/ef-rp/concurrency.md
Product: aspnet-core
Technology: aspnetcore-data
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Thanks, we'll address this in the next update.


[Bug]: Missing "Terms" Section for the Terms Link in the Footer

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
The "Terms" link in the footer is currently not functioning as expected. When users click on the "Terms" link, they are not directed to a specific "Terms and Conditions" section or page. This causes confusion, as users expect a section with relevant legal and policy information.
Expected behavior
Expected Behavior:
Upon clicking the "Terms" link in the footer, users should be directed to a dedicated "Terms and Conditions" page or section, where they can read the legal terms related to the website or service. The page should contain a clear and concise description of the terms governing the use of the website, user rights, and any disclaimers.
Steps to Reproduce:
Go to the footer section of the website.
Click on the "Terms" link.
Observe the absence of a corresponding "Terms" page or section.
Add ScreenShots
No response
On which device are you experiencing this bug?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC-EXT contributor
[X] I'm a HACKTOBERFEST contributor
[X] I have starred the repository

@jeevan10017 ,Please assign me this issue

@jeevan10017 ok


Update the dependencies and manifests to prep for the 1.8 release

Proposed Changes

Add all the necessary changes to release 1.8.

Release Note

/test serving-upgrade-tests

/lgtm


Feature/add where to unique combination of columns

This is a:

[ ] documentation update
[ ] bug fix with no breaking changes
[x] new functionality
[ ] a breaking change

All pull requests from community contributors should target the main branch (default).
Description & motivation

Resolves #610 by adding a where clause to unique_combination_of_columns test
Checklist

[x] I have verified that these changes work locally on the following warehouses (Note: it's okay if you do not have access to all warehouses, this helps us understand what has been covered)

[ ] BigQuery
[x] Postgres
[ ] Redshift
[ ] Snowflake

[ ] I followed guidelines to ensure that my changes will work on "non-core" adapters by:

[ ] dispatching any new macro(s) so non-core adapters can also use them (e.g. the star() source)
[ ] using the limit_zero() macro in place of the literal string: limit 0
[ ] using dbt_utils.type_* macros instead of explicit datatypes (e.g. dbt_utils.type_timestamp() instead of TIMESTAMP

[x] I have updated the README.md (if applicable)
[x] I have added tests & descriptions to my models (and macros if applicable)
[x] I have added an entry to CHANGELOG.md

@dbeatty10 thanks for the feedback. I updated the changelog including correcting [I think] the PR# for 0.8.6

the code itself LGTM! my only comment is that given that all tests get where for free, how does this differ? is it filtering earlier than the native where or something?

@joellabes I think you're right - I didn't realize that config extended beyond the dbt core tests. Should I close the issue?

I think so yeah!


404 Not Found error when accessing images via KV

https://storage12.openstreetcam.org/files/photo/2019/7/26/proc/1595425_86_09fa9_1.jpg
This gives me a 404, although it shows in the viewfinder.
Can somebody please look into this?
Thanks

Looks like this issue is still going on. @foobarna @chip-grab

still not working! @miruna-craciun @zhaitianduo

@brmonaghan  hi i can‘t see the the photo in storage, so the photo may be deleted. let me confirm it next Mon.

ok.

@brmonaghan  hi confrim. this image had deleted.  you can see the lth photos,
I'm really sorry for bringing you trouble, and I will close this issue. You can turn it back on if there's a problem


altool not available error

I receive the "altool is not available" splash when running the app, despite having that tool and Xcode installed.
Self-building the app seems to reveal this error when the xcrun altool --help is run:

2021-10-08 14:44:28.595133+0200 xcrun[5965:4812179] Failed to open macho file at /Applications/Xcode.app/Contents/Developer/usr/bin/altool for reading: Too many levels of symbolic links

However, just below it is the output of the altool --help. So it does indeed seem to execute, but this error or warning seems to potentially throw it off.
Running macOS 11.6 with Xcode 13.0

Thanks for reporting this.
I just made version 1.1.2 available, with Xcode 13 support.
The issue should now be fixed.
Closing for now, please reopen if you still have issues.


Don't resize suggest with no docs

Testing: #27315
I think the suggest widget should not re-size when there are not docs e.g.

I think the widget should re-flow it's layout on a window resize e.g. move below the content in a small window and the back to the right in a wide window.

Will use this issue to track the first of the 2 problems mentioned above.
Logged #27667 for the second one.


specify sheet in form data

Hi
it seems (though im new to google scripting) that you should be able to specify the sheetname in the form (via a hidden input field) and the write the data to that sheet (script would lookfor 'SheetName' or similar and overwrite the default 'Sheet1'.
This would make the system a bit more generic and not mean creating a web app and new url for each googlesheet you wanted to use as a populate.
Ideally be nice to have 1 scripts/web app with a url and supply the name of the destination workbook and sheet in the form but i am not sure scripts work like that - they seem to be attached to a single workbook (and security may be tricky)
Cheers
Rob

thank @lacabra

I  have tried your code @lacabra but it is not working can you please guide me where I am going wrong?
`var sheetName = e.parameter.sheet_name || 'Sheet1';
var scriptProp = PropertiesService.getScriptProperties()
function intialSetup () {
var activeSpreadsheet = SpreadsheetApp.getActiveSpreadsheet()
scriptProp.setProperty('key', activeSpreadsheet.getId())
}
function doPost (e) {
var lock = LockService.getScriptLock()
lock.tryLock(10000)
try {
var doc = SpreadsheetApp.openById(scriptProp.getProperty('key'));
var sheet = doc.getSheetByName(sheetName);

var headers = sheet.getRange(1, 1, 1, sheet.getLastColumn()).getValues()[0]
var nextRow = sheet.getLastRow() + 1

var newRow = headers.map(function(header) {
  return header === 'timestamp' ? new Date() : e.parameter[header]
})

sheet.getRange(nextRow, 1, 1, newRow.length).setValues([newRow])

return ContentService
  .createTextOutput(JSON.stringify({ 'result': 'success', 'row': nextRow }))
  .setMimeType(ContentService.MimeType.JSON)

}
catch (e) {
return ContentService
.createTextOutput(JSON.stringify({ 'result': 'error', 'error': e }))
.setMimeType(ContentService.MimeType.JSON)
}
finally {
lock.releaseLock()
}
}`

@cpaj e hasn't been declared yet, move your first line to above doc

@cpaj e hasn't been declared yet, move your first line to above doc


Flaky Antrea-upgrade tests

Upgrade tests from N-1/N-2 has been pretty flaky in recent PR pre-check runs (https://github.com/vmware-tanzu/antrea/actions?query=workflow%3A"Antrea+upgrade") as well as for post-checkin runs in ToT.
Specifically, the NetworkPolicy test changed by #1586 has been failing at test/e2e/networkpolicy_test.go#L263.
It appears the NetworkPolicy created before upgrading no longer work properly.
It might be related to the following comment: https://github.com/vmware-tanzu/antrea/pull/1586/files#r527796591

/cc @tnqn

@Dyanngg Thanks for filing it. I noticed this flake as well after changing the PodSelector from Everything to MatchExpressions, however it recovered that day after I added 2 seconds of latency and I cannot reproduce it even once on local testbeds with or without the sleep.
I checked logs when it failed on CI, the agent received the policy almost at the same time the controller processed the policy and we never expected 2 seconds delay to realize a policy.
I will investigate more to see the root cause.

I'm running into this issue a lot as well. Seems to happen more than 50% of the time

After digging into the problem, I think the failure was caused by netdev datapath implementation bug:
Some dp flow cache were not flushed after installing new flows that drop new connections. For example:
Before installing drop flows, <IP_ADDRESS> can connect <IP_ADDRESS>
root@k8s-03:~# kubectl exec -it $agent -n kube-system -c antrea-ovs -- ovs-ofctl dump-flows br-int table=100;
 cookie=0x15000000000000, duration=1800.006s, table=100, n_packets=14, n_bytes=1036, priority=200,reg1=0x5 actions=drop
 cookie=0x15000000000000, duration=1800.006s, table=100, n_packets=15, n_bytes=1110, priority=200,reg1=0x7 actions=drop
 cookie=0x15000000000000, duration=1800.354s, table=100, n_packets=11, n_bytes=814, priority=0 actions=resubmit(,101)

root@k8s-03:~# kubectl exec -it $agent -n kube-system -c antrea-ovs -- ovs-appctl dpctl/dump-flows
tunnel(tun_id=0x0,src=<IP_ADDRESS>,dst=<IP_ADDRESS>,flags(-df+csum+key)),recirc_id(0),in_port(1),packet_type(ns=0,id=0),eth_type(0x0800),ipv4(frag=no), packets:3, bytes:198, used:1.571s, flags:F., actions:ct(zone=65520,nat),recirc(0x31)
tunnel(tun_id=0x0,src=<IP_ADDRESS>,dst=<IP_ADDRESS>,flags(-df+csum+key)),ct_state(+new-est-inv+trk),ct_mark(0),ct_label(0/0xffffffff),recirc_id(0x31),in_port(1),packet_type(ns=0,id=0),eth(src=7e:b9:a1:8b:f6:c1,dst=aa:bb:cc:dd:ee:ff),eth_type(0x0800),ipv4(src=<IP_ADDRESS>/<IP_ADDRESS>,dst=<IP_ADDRESS>,proto=6,ttl=63,frag=no),tcp(dst=80), packets:0, bytes:0, used:never, actions:set(eth(src=86:a0:34:80:2d:31,dst=96:d6:69:77:8c:8c)),set(ipv4(ttl=62)),ct(commit,zone=65520),recirc(0x32)
tunnel(tun_id=0x0,src=<IP_ADDRESS>,dst=<IP_ADDRESS>,flags(-df+csum+key)),recirc_id(0x32),in_port(1),packet_type(ns=0,id=0),eth(dst=96:d6:69:77:8c:8c),eth_type(0x0800),ipv4(frag=no), packets:0, bytes:0, used:never, actions:8

After installing drop flows, the corresponding dp flows were not flushed:
root@k8s-03:~# kubectl exec -it $agent -n kube-system -c antrea-ovs -- ovs-ofctl dump-flows br-int table=100;
 cookie=0x15000000000000, duration=1813.653s, table=100, n_packets=14, n_bytes=1036, priority=200,reg1=0x5 actions=drop
 cookie=0x15000000000000, duration=1813.653s, table=100, n_packets=15, n_bytes=1110, priority=200,reg1=0x7 actions=drop
 cookie=0x15000000000000, duration=1.740s, table=100, n_packets=0, n_bytes=0, priority=200,reg1=0x3 actions=drop
 cookie=0x15000000000000, duration=1814.001s, table=100, n_packets=12, n_bytes=888, priority=0 actions=resubmit(,101)

root@k8s-03:~# kubectl exec -it $agent -n kube-system -c antrea-ovs -- ovs-appctl dpctl/dump-flows
flow-dump from the main thread:
tunnel(tun_id=0x0,src=<IP_ADDRESS>,dst=<IP_ADDRESS>,flags(-df+csum+key)),recirc_id(0),in_port(1),packet_type(ns=0,id=0),eth_type(0x0800),ipv4(frag=no), packets:3, bytes:198, used:8.333s, flags:F., actions:ct(zone=65520,nat),recirc(0x31)
tunnel(tun_id=0x0,src=<IP_ADDRESS>,dst=<IP_ADDRESS>,flags(-df+csum+key)),ct_state(+new-est-inv+trk),ct_mark(0),ct_label(0/0xffffffff),recirc_id(0x31),in_port(1),packet_type(ns=0,id=0),eth(src=7e:b9:a1:8b:f6:c1,dst=aa:bb:cc:dd:ee:ff),eth_type(0x0800),ipv4(src=<IP_ADDRESS>/<IP_ADDRESS>,dst=<IP_ADDRESS>,proto=6,ttl=63,frag=no),tcp(dst=80), packets:0, bytes:0, used:never, actions:set(eth(src=86:a0:34:80:2d:31,dst=96:d6:69:77:8c:8c)),set(ipv4(ttl=62)),ct(commit,zone=65520),recirc(0x32)
tunnel(tun_id=0x0,src=<IP_ADDRESS>,dst=<IP_ADDRESS>,flags(-df+csum+key)),recirc_id(0x32),in_port(1),packet_type(ns=0,id=0),eth(dst=96:d6:69:77:8c:8c),eth_type(0x0800),ipv4(frag=no), packets:0, bytes:0, used:never, actions:8

Then <IP_ADDRESS> can still connect <IP_ADDRESS> because of the cache. Note that both stats of openflow and dp flow increased 1 packet, though the openflow's action was drop while the dp flow's action was outputting to port 8.
root@k8s-03:~# kubectl exec -it $agent -n kube-system -c antrea-ovs -- ovs-ofctl dump-flows br-int table=100;
 cookie=0x15000000000000, duration=1820.491s, table=100, n_packets=14, n_bytes=1036, priority=200,reg1=0x5 actions=drop
 cookie=0x15000000000000, duration=1820.491s, table=100, n_packets=15, n_bytes=1110, priority=200,reg1=0x7 actions=drop
 cookie=0x15000000000000, duration=8.578s, table=100, n_packets=1, n_bytes=74, priority=200,reg1=0x3 actions=drop
 cookie=0x15000000000000, duration=1820.839s, table=100, n_packets=12, n_bytes=888, priority=0 actions=resubmit(,101)

root@k8s-03:~# kubectl exec -it $agent -n kube-system -c antrea-ovs -- ovs-appctl dpctl/dump-flows
flow-dump from the main thread:
tunnel(tun_id=0x0,src=<IP_ADDRESS>,dst=<IP_ADDRESS>,flags(-df+csum+key)),recirc_id(0),in_port(1),packet_type(ns=0,id=0),eth_type(0x0800),ipv4(frag=no), packets:7, bytes:470, used:2.652s, flags:SF., actions:ct(zone=65520,nat),recirc(0x31)
tunnel(tun_id=0x0,src=<IP_ADDRESS>,dst=<IP_ADDRESS>,flags(-df+csum+key)),ct_state(+new-est-inv+trk),ct_mark(0),ct_label(0/0xffffffff),recirc_id(0x31),in_port(1),packet_type(ns=0,id=0),eth(src=7e:b9:a1:8b:f6:c1,dst=aa:bb:cc:dd:ee:ff),eth_type(0x0800),ipv4(src=<IP_ADDRESS>/<IP_ADDRESS>,dst=<IP_ADDRESS>,proto=6,ttl=63,frag=no),tcp(dst=80), packets:1, bytes:74, used:2.655s, flags:S, actions:set(eth(src=86:a0:34:80:2d:31,dst=96:d6:69:77:8c:8c)),set(ipv4(ttl=62)),ct(commit,zone=65520),recirc(0x32)
tunnel(tun_id=0x0,src=<IP_ADDRESS>,dst=<IP_ADDRESS>,flags(-df+csum+key)),recirc_id(0x32),in_port(1),packet_type(ns=0,id=0),eth(dst=96:d6:69:77:8c:8c),eth_type(0x0800),ipv4(frag=no), packets:1, bytes:74, used:2.655s, flags:S, actions:8

The bug only happened when the source and destination were on different nodes, that may be why it didn't always fail or succeed. I'm trying to find a way to reproduce it without K8s context so that it can be reproduced easily with some flows.
But for Antrea's upgrade test, I guess we could workaround it by waiting for 10 seconds after pre-checking the traffic so that the cache can expire.

Hi @tnqn, Is this with geneve tunnel? From ./ci/kind/test-upgrade-antrea.sh it seems so.
If the tunnel is geneve, this might be the same issue as https://github.com/vmware-tanzu/antrea/issues/897

@tnqn yes that does seem like a duplicate of #897 and only affects Geneve tunnels. We already reached out to the OVS team a while ago with steps to reproduce it without K8s / Antrea, but they haven't got back to us with a fix. Once #1643 is closed, we can close this issue in favor of #897

Thanks @srikartati and @antoninbas for the information! After looking at the symptom of #897, I think it's same issue.


Bugs/update scanner code structure#1187

Summary

Updated barcode scanner feature code so it follows the correct architecture of the application.
Changelog

[General] [Add] - Scanner provider
[General] [Change] - Scanner integration
Test Plan

Test Scanner feature extensively on iOS and android

The TestFlight build for this does not include the Scandit license key, so scanning changes cannot be tested with TestFlight build 1214


List of TODOs blocked on MSRV

This issue is a repository of TODOs which are blocked on updating to a particular MSRV.
1.65.0

Remove conditional compilation from test_new_box_slice_zeroed_panics_isize_overflow

I would like to work on this as well.


remove cargo bindgen as a dependency

we needed this for wasm3, but that is not longer a dependency, so we don't need bindgen (directly) any more
@Anton-4 is this correct for the nix files, and did I catch all occurences?

Yes, looks good :)
I re-ran the windows test, it may have been skipped on a previous PR when an error was actually present.
I'll approve the PR once we've resolved that or it turns out to be a flake.