Make the "Generated dependency info XXXX.dmk" not have absolute paths

Currently when a dependency file is created it outputs the following message;
Generated dependency info /home/travis/build/mithro/symbiflow-arch-defs/.deps//vpr/ff/vpr_ff.model.xml.dmk.
It would be good if these messages didn't have the absolute path in them.

Example image;

@mithro I'm not seeing this. Did you already fix it with #56 ?


[SPARK-8567] [SQL] Add logs to record the progress of HiveSparkSubmitSuite.

Merged build triggered.

Merged build started.

Test build #35756 has started for   PR 7009 at commit b22cf7d.

Merged build triggered.

Merged build started.

NAVER - http://www.naver.com/
<EMAIL_ADDRESS>님께 보내신 메일 <Re: [spark] [SPARK-8567] [SQL] Add logs to record the progress of HiveSparkSubmitSuite. (#7009)> 이 다음과 같은 이유로 전송 실패했습니다.

받는 사람이 회원님의 메일을 수신차단 하였습니다.

Test build #35765 has started for   PR 7009 at commit 62fb1f9.

Test build #35756 has finished for   PR 7009 at commit b22cf7d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test build #35765 has finished for   PR 7009 at commit 62fb1f9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

I am merging it to master. If it fixes the flaky test, I will merge it to 1.4 branch. Then, we can also add sc.stop to other test (SPARK-8489: MissingRequirementError during reflection and SparkSubmitSuite) .


Incorrect Assignment of ReloadlySDKClientId and ReloadlySDKSecretId in ReloadlyAuthentication.shared.configure()

The Authentication configuration function ReloadlyAuthentication.shared.configure() incorrectly assigns the ReloadlySDKClientId and ReloadlySDKSecretId
Describe the bug
In the ReloadlyAuthentication.shared.configure() the following function is found in the ReloadlyAuthentication class under configure function:
self.clientId = clientId ?? (dictionary?["ReloadlySDKSecretId"] as? String) ?? ""
self.clientSecret = clientSecret ?? (dictionary?["ReloadlySDKClientId"] as? String) ?? ""

As shown, the ReloadlySDKSecretId value fetched from the info.plist is assigned to the clientId instead of the clientSecret while the ReloadlySDKClientId value fetched from the info.plist is assigned to the clientSecret.
Expected Behavior
This part of the code should be written in this way, where the correct assignments are done:
self.clientId = clientId ?? (dictionary?["ReloadlySDKClientId"] as? String) ?? ""
self.clientSecret = clientSecret ?? (dictionary?["ReloadlySDKSecretId"] as? String) ?? ""

File Reference
https://github.com/Reloadly/reloadly-sdk-ios/blob/2c7577452e879b21f890f92474829e975c889535/ReloadlySDK/Classes/ReloadlyAuthentication/ReloadlyAuthentication.swift#L61
Context
SDK authentication call fails unless this is reversed.

@mhlangagc Thank you for reporting this, we are working to address it quickly.


Godot 3.1.1 ignores invalid command line options and continues with execution

Godot version:
v3.1.1.stable.official
OS/device including version:
Mac OS X
Issue description:

Godot does not validate command line options. Instead continues executing.
Command line option -v does not work. Use --version instead.

Steps to reproduce:
E.g. godot -xyz will run godot anyway.
Also godot -v is ignored. However godot --version works
Expected behavior
Should produce an error message when option is invalid.

will be fixed by https://github.com/godotengine/godot/pull/26213

Duplicate of #20000.

Also godot -v is ignored. However godot --version works

This is because -v is short for --verbose :slightly_smiling_face: We could have -V as a shorthand for version.


Executors depending on pretrained models can't pass the tests because no possible working default value.

Problem
As in PR #448 , #372  and some others, these executors cannot be passed because they do not have the chance to have a valid default value.

Solved by jina-ai/jina#1065 and applied to #385 #372 and others


S3 request path is trimmed at the end incorrectly

Sending signed request using AWS SDK with request path containing space(s) at the end leads to an exception with error SignatureDoesNotMatch
Expected Behavior

GetObject request do not trim versionId (or another query params) at the end
Current Behavior

Sending request without spaces in versionId gives me following HTTP request:
GET /bucket/object?versionId=AGdtSBttoV2r97kIKQlvwSKt17M0PiS HTTP/1.1
User-Agent: aws-sdk-dotnet-coreclr/<IP_ADDRESS> aws-sdk-dotnet-core/<IP_ADDRESS> .NET_Core/4.6.27129.04 OS/Microsoft_Windows_10.0.17763 ClientAsync
Host: XXX
X-Amz-Date: 20190128T044823Z
X-Amz-Content-SHA256: e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855
Authorization: AWS4-HMAC-SHA256 Credential=XXXXXXXXXXXXXXXXXXXX/20190128/us-east-1/s3/aws4_request, SignedHeaders=host;user-agent;x-amz-content-sha256;x-amz-date, Signature=5ba54605c53b8f449af690d0289ff9abf2894e640fd815f41c6df5eb7694cb60

With space in the end of versionId (signature differs from request without spaces, but query parameter does not have space at the end - %20):
GET /bucket/object?versionId=AGdtSBttoV2r97kIKQlvwSKt17M0PiS HTTP/1.1
User-Agent: aws-sdk-dotnet-coreclr/<IP_ADDRESS> aws-sdk-dotnet-core/<IP_ADDRESS> .NET_Core/4.6.27129.04 OS/Microsoft_Windows_10.0.17763 ClientAsync
Host: XXX
X-Amz-Date: 20190128T044816Z
X-Amz-Content-SHA256: e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855
Authorization: AWS4-HMAC-SHA256 Credential=XXXXXXXXXXXXXXXXXXXX/20190128/us-east-1/s3/aws4_request, SignedHeaders=host;user-agent;x-amz-content-sha256;x-amz-date, Signature=43d551e322a7325d57824549cece0a13f1b9a150563c383df0e7bfc9c65e167f

This request leads to an exception:
Amazon.S3.AmazonS3Exception: Error making request with Error Code SignatureDoesNotMatch and Http Status Code Forbidden. No further error information was returned by the service.

With space in the beginning of versionId (space is present in query params):
GET /bucket/object?versionId=%20AGdtSBttoV2r97kIKQlvwSKt17M0PiS HTTP/1.1
User-Agent: aws-sdk-dotnet-coreclr/<IP_ADDRESS> aws-sdk-dotnet-core/<IP_ADDRESS> .NET_Core/4.6.27129.04 OS/Microsoft_Windows_10.0.17763 ClientAsync
Host: XXX
X-Amz-Date: 20190128T044944Z
X-Amz-Content-SHA256: e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855
Authorization: AWS4-HMAC-SHA256 Credential=XXXXXXXXXXXXXXXXXXXX/20190128/us-east-1/s3/aws4_request, SignedHeaders=host;user-agent;x-amz-content-sha256;x-amz-date, Signature=7644030ebf58475c867e7bbc0dd424c946dfd2b60ba3801bf44e448217292473

Possible Solution

Do not trim request path at the end?
Steps to Reproduce (for bugs)

var client = new AmazonS3Client(bucketRegion);
using (var response = await client.GetObjectAsync("bucket", "key", "versionWithSpaceAtTheEnd "))
{
}

or
var client = new AmazonS3Client(bucketRegion);
var request = new PutObjectRequest
                {
                    Key = "test ",
                    BucketName = "bucket",
                    ContentBody = "test"
                };

using (var response = await client.PutObjectAsync(request))
{
}

Context

Your Environment

AWSSDK.Core version used: <IP_ADDRESS>
Service assembly and version used:
Operating System and version: Windows 10 Enterprise 10.0.17763 Build 17763
Visual Studio version: 15.9.6
Targeted .NET platform:

.NET Core Info

.NET Core version used for development: 2.2.1
.NET Core version installed in the environment where application runs: 2.2.1
Output of dotnet --info:

.NET Core SDK (reflecting any global.json):
 Version:   2.1.503
 Commit:    4c506e0f35

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.17763
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\2.1.503\

Host (useful for support):
  Version: 2.2.1
  Commit:  878dd11e62

Contents of project.json/project.csproj:

I'm going to close this out because I'm not sure there are any valid inputs where this could cause an issue.  If you fine an example, please feel free to reopen or comment.

@xakep139, where is the space in your request?  As far as I can tell, this is all theoretical.  Are you only seeing this error if you are purposely appending a space to a value that shouldn't have it, or are you using normal inputs?

This error can be established if someone trying to use fuzzing requests to server.

@klaytaybai, simply create new file with vi "test " and try to send PutObjectRequest with key equals to this filename. Do you consider such case as a valid input?

Hi @xakep139, you can refer to this S3 naming documentation for valid key names.  A file named "test " should be fine.  I confirmed when this issue was first created that posting a file using a key with a space at the end works.

I've updated the issue summary, so you can reproduce use-case with PutObjectRequest. It leads me to
Amazon.S3.AmazonS3Exception: Error making request with Error Code SignatureDoesNotMatch and Http Status Code Forbidden

even with latest AWSSDK.S3

@klaytaybai so... maybe reopen this issue?


[ISSUE #7305] Fix metrics and transactional module not shutdown while broker offline cause coredump

Which Issue(s) This PR Fixes

Fixes #7305
Brief Description

How Did You Test This Change?

Codecov Report

Merging #7307 (73e16e4) into develop (d000ef9) will increase coverage by 0.02%.
Report is 3 commits behind head on develop.
The diff coverage is 70.17%.

@@              Coverage Diff              @@
##             develop    #7307      +/-   ##
=============================================
+ Coverage      42.63%   42.65%   +0.02%
- Complexity      9453     9468      +15
=============================================
  Files           1151     1151
  Lines          82669    82705      +36
  Branches       10771    10778       +7
=============================================
+ Hits           35246    35280      +34
+ Misses         43018    43017       -1
- Partials        4405     4408       +3

Files Changed
Coverage Δ

...ache/rocketmq/broker/topic/TopicConfigManager.java
56.68% <0.00%> (-0.34%)
:arrow_down:

...a/org/apache/rocketmq/broker/BrokerController.java
43.96% <50.00%> (+0.02%)
:arrow_up:

...saction/queue/TransactionalMessageServiceImpl.java
52.68% <50.00%> (+1.78%)
:arrow_up:

...mq/client/impl/producer/DefaultMQProducerImpl.java
44.58% <71.42%> (+0.58%)
:arrow_up:

...ocketmq/broker/processor/AdminBrokerProcessor.java
24.37% <92.30%> (+0.30%)
:arrow_up:

... and 20 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Gitops basic tutorial update

What does this PR do?
Update gitops basic guide including:

kubectl gs gitops commands
gitops-template repo structure
sops encryption

What does it look like?
Apart from the first part of the doc that explains concepts, all the practical part has been replaced.
Any background context you can provide?
Part of this issue:
https://github.com/giantswarm/roadmap/issues/1624
What is needed from the reviewers?
Ensure that the doc has no errors and that the process is explained so users can follow
Have you maintained the front matter?
Yes, updated review date.
(Please bump the last_review_date in case this qualifies as a review for an entire page. Provide user_questions which should be answered in the page. Provide a meaningful description.)

I'd like to review this one as soon as suggestions are applied. :)

Some comments were not yet replied and I added some other suggestions.
It is hard to see how it will look after all these changes. I have had to use View File option. I think the steps are explained in the right order with the right commands but I miss a bit of context on why the user is doing the action.  My suggestion would be to introduce the purpose "In GitOps, secrets are usually stored together with the rest of the configuration. For security implications, even if our git repository is private, the secrets must be encrypted. For that reason we use GPG .... " WDYT?

Some comments were not yet replied and I added some other suggestions.
It is hard to see how it will look after all these changes. I have had to use View File option. I think the steps are explained in the right order with the right commands but I miss a bit of context on why the user is doing the action.  My suggestion would be to introduce the purpose "In GitOps, secrets are usually stored together with the rest of the configuration. For security implications, even if our git repository is private, the secrets must be encrypted. For that reason we use GPG .... " WDYT?

As this article is pretty outdated I would rather merge an imperfect version and address this issue in another iteration than block the merge.


Implicit annotations support for ngDialog module

Hello!
Thank you for this great library!
However, it would be great if you will be able to add implicit annotations support for ngDialog module (it's quite popular). It uses resolve and controller properties to specify resolves and controllers for dialogue windows.
Thanks!

https://github.com/olov/ng-annotate/issues/245


install.sh : compressed cloning + automatic path (revised + working)

I revised my previous code with various installs and I came up with a solution. Instead of installing the entire project, moving the 'sudo' files one by one to ~/bin, and then deleting the project, now all we have to do is grab all the files from the new 's' folder and clone them into ~/bin. I also added a new 'PATH' code line using the wildcard feature :
PATH=$PATH:/c/Users/*/win-sudo
The '*' in this line represents any folder in 'Users' with 'win-sudo' inside of it, which should be just installed. And the best part is, it can be run directly from script, meaning only 1 line of code is needed to install and setup win-sudo.
For the install I excluded the .md and .gif files because they weren't necessary for installation, and also install.sh because that was already installed using the cloning command.
I named the folder 's' for two reasons : 1. It is the first letter of every file that needs to be moved to ~/bin 2. It's name is unique from every file in the folder so it won't be confusing
**Important Note : **
I have tried this .sh file when installing win-sudo and the 'git clone' command is proper syntax. It should work fine as long as there is connection to the network. The new PATH command I added also works on my machine.
Extra Changes :

Removed the 'else' section because another error is displayed after the 'echo' if the directory exists, so it wasn't really necessary

Please take a look at this and any feedback would be appreciated.

Hm... I can't make it work. I just check it and git clone https://github.com/imachug/win-sudo.git/s line doesn't work.

P.S if something needs to be done with .git it can just be moved with another command

So, what about that git clone https://github.com/imachug/win-sudo.git/s ~/bin/win-sudo line that doesn't work?

Check my fork at DemonixCraft/win-sudo for reference

I am getting fatal: repository 'https://github.com/DemonixCraft/win-sudo.git/s/' not found when I replace imachug with DemonixCraft everywhere.

Can you screenshot the code that you used?

I just changed my install.sh to grab from DemonixCraft's repo for the moment

Sorry, I meant when you entered the 'git clone' command into your Git Bash window, you should take a screenshot of the error message. Sorry again for the misunderstanding.

C:\Users\Ivanq\Documents>git clone https://github.com/DemonixCraft/win-sudo.git/s ~/bin/win-sudo
Cloning into '~/bin/win-sudo'...
remote: Not Found
fatal: repository 'https://github.com/DemonixCraft/win-sudo.git/s/' not found

By the way, what is your Git version?

maybe there has to be a '/' after the 's'?

I'll take a look into revisions and do some more testing. Keep this pull open until I can figure out the issue, if that's okay with you.

Can you please make a screenshot of your result?

I'm actually not at my own computer right now, I'll try it on my Git in about an hour.

Okay, I'll keep this PR open then.

Okay so I did some more testing and every line of code except for 'git clone' works perfectly fine. Apparently, git can only grab the entire directory with git clone, but somehow I'm managing to work around the problem. It's an unfinished one-liner that will perform the following :

Clone a copy of my fork to ~/bin/win-sudo
Navigate to ~/bin/win-sudo
Remove the 'img' folder and it's contents, install.sh, and README.md
Once done successfully it will set the path to the User's win-sudo.
I can remove install.sh and README.md, but I'm still trying to work on img/screenshot.gif

Also I got the same output you did.

Okay imachug, this line is nearly complete but I need your help :
I'm currently using the line : git clone https://github.com/DemonixCraft/win-sudo.git ~/bin/win-sudo | git rm -rq ~/bin/win-sudo/.It looked like everything installed fine but it complained about '/c' : Bradley@WINDOWS-CTR6ND1 MINGW64 ~ (master)
$ curl -s https://raw.githubusercontent.com/DemonixCraft/win-sudo/master/install.sh | sh
Cloning into 'C:/Users/Bradley/bin/win-sudo'...
fatal: Invalid path '/c': No such file or directory
remote: Counting objects: 402, done.
remote: Compressing objects: 100% (125/125), done.
remote: Total 402 (delta 39), reused 0 (delta 0), pack-reused 277
Receiving objects: 100% (402/402), 3.34 MiB | 572.00 KiB/s, done.
Resolving deltas: 100% (166/166), done.
failed to install, sorry :'(`
Do you have any idea what happened here?

Okay I got the 'git clone' command to work but it's two lines and I'm still revising PATH

Maybe you should try so-called sparse checkout?

I've been looking at that but idk how it works. It seems like a solution though so I'll look into it.

When you manage to get it working, please tell me.


Album.type field is null unexpectedly

The following code did not work:
private fun candidatePhotos(): List<Photo>
    {
        val albums = mediaOperations.albums
        val profilePictures = albums.find { it.type == Album.Type.PROFILE }
        val coverPhotos = albums.find { it.type == Album.Type.COVER }

        val results = ArrayList<Photo>()
        if (profilePictures != null)
        {
            results.addAll(mediaOperations.getPhotos(profilePictures.id))
        }
        if (coverPhotos != null)
        {
            results.addAll(mediaOperations.getPhotos(coverPhotos.id))
        }
        return results.toList().take(5)
    }

I had to change instead to:
val profilePictures = albums.find { it.name == "Profile Pictures" }
val coverPhotos = albums.find { it.name == "Cover Photos" }

http://stackoverflow.com/questions/34040210/spring-social-facebook-getting-null-photo-information-urls-only-id-is-not-nul

getAlbum() needs to be updated to specify the fields that we want in the response.

This is still an issue

I've made a pull request. I guess it will never be merged as the other requests but let's give it a try again.


Add secondary-sid support

Add support for secondary-sid option in mpv via dlang field. Field is functionally identical to slang field except that it is optional.

Sorry for the delayed response, I was on holiday. If I have not reviewed this PR in a weeks time please ping this thread to remind me.

Sorry for the delayed response, I was on holiday. If I have not reviewed this PR in a weeks time please ping this thread to remind me.

ping

I have a number of problems with the current form of this PR, but before I go in to them there is a specific question that needs to be answered: how does the secondary subtitle selection and the normal subtitle selection influence each other.
The normal subtitle selection code is based on the currently selected audio. If you have audio-mode enabled, then the selection of both audio and subtitles depend on each other; a pair of audio and subtitle tracks must pass a match for them both to be selected.
How does the secondary subtitle option fit into this? Is it completely separate from the normal subtitles where it gets selected separately? Should the primary subtitles be selected first and the secondary subtitles on be selected afterwards based on that selection? Would both the primary and secondary subtitles need to match to pass a specific preference? This question needs to be answered and justified before this PR can be properly reviewed.


Are we able to pass url query parameters?

Are we able to pass url query parameters?
ie. example://test/2?abc=2&param2=3

Hey, nope not at the moment. Would be open to a PR for it, though


Add secret parameter to the webhooks.

Hi, I've added the ability to add a secret "token" to the webhooks so that they can't just be called by anyone on the Internet.
This addresses #11, although not in the way he requested.  I think adding a token like this is sufficient though and much easier to use in a webhook.

Thank you, great idea! I'll have a look at this.


New package: DiffUtils v0.1.0

Registering package: DiffUtils
Repository: https://github.com/simeonschaub/DiffUtils.jl
Created by: @simeonschaub
Version: v0.1.0
Commit: b83985d9a2308a3bed6f8f3bc55a9fc639209d97
Reviewed by: @simeonschaub
Reference: https://github.com/simeonschaub/DiffUtils.jl/commit/b83985d9a2308a3bed6f8f3bc55a9fc639209d97#commitcomment-46055233

@simeonschaub can you please retrigger registration to fix the merge conflict?
[noblock]

@simeonschaub can you please retrigger registration to fix the merge conflict?
[noblock]

Sure, will do!
[noblock]

Sure, will do!
[noblock]


Problem with composite paths

I'm currently integrating NanoVG to ZGameEditor as an external library (Windows DLL and Android .so) and everything works fine, except of one problem - composite paths do not work. For example, the following code:
nvg_BeginPath();
nvg_Rect(0, 0, 2, 2);
nvg_Circle(2, 2, 1);
nvg_FillColor(0.8, 0.1, 0.1, 1.0);
nvg_Fill();

nvg_BeginPath();
nvg_Rect(0, 0, 2, 2);
nvg_StrokeWidth(0.05);
nvg_StrokeColor(0.1, 0.8, 0.1, 1.0);
nvg_Stroke();

nvg_BeginPath();
nvg_Circle(2, 2, 1);
nvg_Stroke();

(note I'm using nvg_ prefix instead of nvg, colors are specified as <0-1> 4 floats, and positions are scaled) results in this:

Independently on usage or omitting nvg_PathWinding() the result of composing paths is always the same: bounding box comprising all the included paths.
When I compile your C++ examples, path composition works fine, but when I try to call it from ZGameEditor the problem occurs. It maybe/probably uses some settings or handling of OpenGL which is not compatible to NanaoVG's expectations.
I can provide you my example(s), binaries, VS projects for building the library, etc. to reproduce the problem. But before that, you maybe already know or suspect what's the problem and how to solve it. Thanks for your help!

Looks to me like you may need to enable stencil buffer on your OpenGL context and/or framebuffer.

Thanks Ryan. You are right. ZGameEditor did not use stencil buffer in created OpenGL context. I discussed this with the author of the tool and this feature will be added soon. After it's done,  NanoVG will be used as an extension of another game engine.


More styles.

Especially for check boxes.
Now, themes are more accurate and easier to use.
Rice inputs with the right format and Jamie's text suggestions fixed.

Here you go, @mbarrenechea
I also fixed the "toggleClass" think we talked about.


Vulnerability roundup 107: jitsi-meet-1.0.5307: 1 advisory [6.1]

search, files

[ ] CVE-2021-39205 CVSSv3=6.1 (nixos-21.05)

CVE details
CVE-2021-39205
Jitsi Meet is an open source video conferencing application. Versions prior to 2.0.6173 are vulnerable to client-side cross-site scripting via injecting properties into JSON objects that were not properly escaped. There are no known incidents related to this vulnerability being exploited in the wild. This issue is fixed in Jitsi Meet version 2.0.6173. There are no known workarounds aside from upgrading.

Scanned versions: nixos-21.05: 3b1789322fc.
Cc @mmilata
Cc @petabyteboy
Cc @ryantm

Problem like in #138676, it's for a different package. We are not affected as our version is newer.


fix: non-spec emphasis (fix #1806 #2708)

https://github.com/nhn/tui.editor/pull/2572 introduced a dirty fix for a very rare non-spec emphasis and lead to many regular emphasis being replaced with html tags, which is no good (#1806 #2708).
This PR fixes that by making sure only non-compliant emphasis will be replaced with html tags.
Please check if the PR fulfills these requirements

[x] It's the right issue type on the title
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes/features)
[x] Docs have been added/updated (for bug fixes/features)
[x] It does not introduce a breaking change or has a description of the breaking change

@jajugoguma check, please.
@kshnurov Thank you for your contribution. It will take a long time to check this because we are working on other projects. we'll check this PR by mid-October.

After doing some testing I've found some very common cases that are completely broken either in master:
    Expected: "test ***test another* test**"
    Received: "test ***test another* test"

    Expected: "test **test [some](link)** test"
    Received: "test test [some](link)** test"

or in this PR:
    Expected: "test **test [some](link).** test"
    Received: "test **test [some](link). test"

Looking at how this editor works with marks and opening/closing tags I don't think it possible to fix that without a major rework.
I'm suggesting to revert #2572 completely and ignore the rare cases it was introduced for, and here's why:

This is a WYSIWYG editor, it's output will be used on the web, which means it will be converted to HTML and that HTML will be valid. There's much more broken things with #2572 in place than with having non-spec markdown in some very rare cases (which will be rendered correctly anyway).
I've checked how CKEditor works with such non-spec emphasis - it ignores them, i.e. produces non-spec markdown but valid HTML.

@jwlee1108 @jajugoguma what do you think? If you agree, please revert #2572 and release a new version.

@jwlee1108 @jajugoguma 2 months. There's 200+ issues labeled as bug. Shouldn't this repo be marked as unmaintained, so people won't rely on it in production?

I think @kshnurov is right, #2572 should be reverted.
I think having HTML in markdown output is an issue, or at least it would be great to have a strict markdown mode without any HTML in it.

Resolved in #2754


[Model] Optimize image preprocess for vision model

This PR improves the performance of image preprocessing for vision model. Generally, image storage formats include NCHW and NHWC, with NCHW being more suitable for GPU-based computations. Additionally, performance is enhanced by binding the for loop to GPU hardware threads.

@mengshyu Let us know if it is ready for review :-)

Hi @MasterJH5574 , It's ready to review, could you take a look, thanks.

Looks good, thanks!


Add Prettier to keep code consistent.

Hoping this a welcome addition. I added prettier and setup a config file that closely matches the existing code style that already existed. I'm also only checking index.ts for now which seems like one of our only TypeScript source files.
I also setup the repo's first Github Action to ensure code is properly formatted from people posting PRs.

Wow, Awesome contribution @tbezman ! We definitely need to add GitHub actions here.
However, we are already using eslint across our projects. We have lint and fix commands in Borsh JS package.json. eslint is a linter, so it's more powerful and can look for potential errors in code. But I can agree that Prettier is doing a better job in code formatting. So I can see two ways how we can stay consistent and make this project better.

(Easy) Use eslint instead of Prettier in added GitHub Action.
(Hard) Use both eslint and Prettier. They can be in conflict, so we will need to read this article first. Maybe we will need to have two separate actions for them.

I would rather use default parameters for Prettier and not override them with a separate config file. More chances that it will not be in conflict with text editors and IDEs.
Bonus points for adding Unit tests GitHub Action (just run yarn test)

Thanks for getting back to me. I'll apply all of this feedback ASAP.

@volovyk-s Yeah there is a config we can use to disable all eslint rules around code formatting so we can use prettier for formatting, and eslint for code conventions. I think I'll save the ESLint configuration for an upcoming PR since we have a bunch of warnings atm. I'll do the eslint stuff as a followup.

Okay @volovyk-s, I updated the PR. I removed the prettier config (now we're using prettier defaults), and added unit tests to the github action workflow. I think you'll have to make the action a required check on PRs after this merges.

@volovyk-s On this branch, when I run yarn pretty, none of my files actually change but I do get some output in the console about which files prettier ran against. Are you seeing the same behavior or is something borked?

@tbezman I have double-checked. All good, thank you!

Hm, it's getting changed after yarn test, that is weird :/

Test and pretty (with small fix) GitHub actions works!
https://github.com/near/borsh-js/pull/35


Bump Jax version?

Hello, I'm unable to install the project on an a mac as jax-metal 0.0.7 depends on jax>=0.4.22, and somap is pinned to 0.4.20. would it be possible to bump the minimum required version of jax or set a lower limit?

Error Log
Added somap>=0.1.0 as optional (jax) dependency
Reusing already existing virtualenv
Generating production lockfile: /Users/srivarra/Davis Lab/NBL/nbl/requirements.lock
× No solution found when resolving dependencies:
╰─▶ Because jax-metal==0.0.6 depends on jax>=0.4.22 and only the following versions of jax-metal are available:
        jax-metal<=0.0.6
        jax-metal==0.0.7
    we can conclude that jax-metal>=0.0.6,<0.0.7 depends on jax>=0.4.22.
    And because jax-metal==0.0.7 depends on jax>=0.4.26 and somap==0.1.0 depends on jax==0.4.20, we can conclude that jax-metal>=0.0.6 and somap==0.1.0
    are incompatible.
    And because only somap<=0.1.0 is available and nbl[jax]==0.0.1 depends on jax-metal>=0.0.6, we can conclude that nbl[jax]==0.0.1 and somap>=0.1.0
    are incompatible.
    And because nbl[jax]==0.0.1 depends on somap>=0.1.0 and you require nbl[jax]==0.0.1, we can conclude that the requirements are unsatisfiable.
error: could not write production lockfile for project

You're right. New version somap v0.1.1 released that includes a constraint relax on jax dependency

@mthiboust Thank you!


How to debug or run tests in v.1.9.2

I am having a problem installing into Laravel 5.6 on my local mac environment. I get an "invalid upload request" error when uploading an image. The file structure is created, but no image is uploaded.
Earlier posts referring to this same problem mention a debug script php ./vendor/unisharp/laravel-filemanager/bin/debug, but it doesn't exist in this version (1.9.2) of laravel-filemanager. There is a tests folder in the /vendor/unisharp/laravel-filemanager/ folder.
Is there a way to do debug or run tests in this version?

@docmojoman error exception logs should be in storage/logs/laravel.log. Please check it out. You can also check if any extension is missing from requirement section in installation document.

@g0110280 thanks!


Adds :onhoverlost and separates out event based actions on eventbox widget (See #251 for more info).

Description
Deprecates :timeout, :onscroll, :onhover and :cursor from top of widget hierarchy.
Moves these to a new widget (eventbox ...).
Adds :onhoverlost property to (eventbox ...).
Fix #250 and fix #251 (see them for more details).
Usage
(eventbox :timeout <timeout>
          :onscroll "<cmd to run on scroll with {} being replaced by up or down>"
          :onhover "<cmd to run on hover with {} being replaced by 'x y' coordinate>"
          :onhoverlost "<cmd to run on hover loss wih {} being replaced by 'x y' coordinate>"
          :cursor "<type of cursor>"
    "Content")

Showcase
https://user-images.githubusercontent.com/26714676/131639040-89781f4e-68af-4079-be2b-0c6ac4ff6d97.mp4
Edit: I should have probably add revealer within the box contained in the eventbox widget, so that hover applies to both revealer and the date here to remain it opened. Ignore it please 😅😅
Additional Notes
Widget must contain exactly one child (restriction from GTK).
Checklist
Please make sure you can check all the boxes that apply to this PR.

[x] All widgets I've added are correctly documented.
[x] The documentation in the docs/content/main directory has been adjusted to reflect my changes.
[x] I used cargo fmt to automatically format all code before committing

Applied fix (donot discriminate nested elements for hover events):
https://user-images.githubusercontent.com/26714676/131795837-d9ef5d3b-a8ab-46cf-aefe-e842e47e6316.mp4
I guess PR is ready now, ping me if there's any problem remaining!

I would replace "hoverlost" by leave also moving those to a different widget is a big change. Is there any reason why there actually need to be a new widget?

@snakedye See #251 GTK does not sent events to most of the widgets, its inconsistent behavior.

If using eventbox is genuinely the only way to support these events across all widgets, then I fear this is the best way to go about it. auto-wrapping widgets that use any of these properties would make debugging more confusing and definitely be complex to implement,...
The deprecation explanation definitely should give a bit more details as to what eventbox does, why it's necessary, and optimally link to the related issue. An explanation such as

Using the attribute :cursor here has been removed, as GTK does not support it consistently. Instead, use eventbox to wrap this widget and set the attribute there. See https://github.com/elkowar/eww/issues/251 for more details

would make this a lot cleaner.

auto-wrapping widgets that use any of these properties would make debugging more confusing and definitely be complex to implement

I thought of auto-wrapping, but changed decision to just let user manually wrap for the same reason of easier debugging from end user, also it looks clean and easier to know which positions events are going to be captured in debugger as well as config file via this...
I also couldn't figure out a better way to establish a consistent behavior among all widgets which changes their support with runtime versions of gtk...

would make this a lot cleaner.

👍

Also what do you prefer (is better option to have) :onhoverlost or :onleave?

given that the name onhover is better matched by a onhoverlost, i'd prefer that

Otherwise, some conflicts, but otherwise this looks good!

For some reason gtk 0.14 update was creating a lot of conflict in other files as well, gimme some time to resolve them 😅😅

looks good now! thanks a lot!


Reduce requests made

Heyo! 👋
I noticed that the app fetches new questions every time the user presses submit. However, it does not have to: the first obtained data can be used for all questions - since the app fetches 10 of them

thanks for contributing


Google Colab demo fails at the setup steps...

Dear Jeff,
First of all thank you so much for sharing this implementation. It looks really impressive.
I just want to report that the google colab notebook does not seem to work. Each time I start it from scratch, fail at:
# conda environment setup
!conda create -n hybrik python=3.7
!source activate hybrik && pip install ipython

Google Colab session crashes. At pip install ipython.
Can you help me solve that?

Hi @bsense-rius, the Colab notebook will automatically restart and you can directly run the next block.

Hi!
You are right, Sorry! It might be useful to add a text cell commenting that out, so that dumber people like me, that are unaware of this ipython installation technique overriding colab's default know that is perfectly right to proceed.
Thanks again

Thank you very much for your suggestion!


Fix tsdb/index/inmem benchmark tests

Fixes broken benchmarks.

During a run of megacheck the following issues were discovered:


Fix infinite loop in DNS parsing

The primary goal of this PR is to fix an infinite loop that was seen in the wild when parsing DNS entries from a sample with private data logging enabled. This is due to a lack of verification of the IP type (4 vs 6) with no corresponding else block in the subsequent parsing loop. I also took the opportunity to do some minor cleanup, reducing some of the parsing down to one-liners with nom.

thanks for the fix!


CHD8

CHD8
https://www.nature.com/articles/s41398-022-02189-1

We have added this to the project, the notebook is at gpsea-cs!


Add quick reference of overlapping states

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 83.411%

Files with Coverage Reduction
New Missed Lines
%

src/overlap.rs
1
78.95%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1954

Relevant Lines:
1994

💛  - Coveralls


[SPARK-23318][ML] FP-growth: WARN FPGrowth: Input data is not cached

What changes were proposed in this pull request?
Cache the RDD of items in ml.FPGrowth before passing it to mllib.FPGrowth. Cache only when the user did not cache the input dataset of transactions. This fixes the warning about uncached data emerging from mllib.FPGrowth.
How was this patch tested?
Manually:

Run ml.FPGrowthExample - warning is there
Apply the fix
Run ml.FPGrowthExample again - no warning anymore

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Test build #4092 has started for PR 20578 at commit 374a49c.

Test build #4092 has finished for PR 20578 at commit 374a49c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.


Enable Elm code highlighting

... by adding something like hljs.registerLanguage("elm",function(e){...}) to https://github.com/elm-lang/package.elm-lang.org/blob/master/assets/highlight/highlight.pack.js. As a first approximative hack (if an Elm version of that functionality is not yet available), could simply copy the existing hljs.registerLanguage("haskell",function(e){...}) part and replace "haskell" by "elm" in there. (Best to keep the part with "haskell" intact, instead of replacing it, since there are Elm packages out there whose README.md contains code blocks marked with haskell.)

Further thoughts on rendering of doc comments (rather than explicit "elm" blocks) on the mailing list.

Also relevant: https://github.com/evancz/elm-markdown/issues/4. Then, you could even try to rely, here in package.elm-lang.org, on highlight.js automatically choosing Elm style highlighting for the rendering of doc comments (though short snippets of Elm code might end up looking indistinguishable from Haskell code for highlight.js, so it is not certain how good the accuracy would be; but then, Elm code that is syntactically indistinguishable from Haskell code probably wouldn't suffer much from being highlighted as Haskell code).

Alright, this is in, with Elm as the default syntax. Thank you for your work on this, it's much better now!


[HOLD for payment 2024-12-12] Hybrid - Android - Settings- General section on settings disappears when navigating back from profile

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.69-1
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Open the Expensify app.
Tap on "Settings" on the bottom of the screen.
Tap on "Profile"
Tap on the search icon on the top right corner and select any chat from search results.
Tap on compose box to open the keyboard.
Tap on the arrow on the top left corner.
When landing on "Profile" again, tap fast on the arrow on the top left corner again.
Verify you land on "Settings" tab and all the sections are correctly displayed.

Expected Result:
When navigating back to settings from profile, all the tab should be completely visible and no section should disappear.
Actual Result:
When navigating back fast to "Settings" from profile, after opening a chat from search results, the "General" section disappears.
Workaround:
Unknown
Platforms:

[ ] Android: Standalone
[x] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/e802b74e-4c79-407e-9ee5-4a0a3c86e40d

View all open jobs on GitHub
Issue OwnerCurrent Issue Owner: @slafortune

Hi 👋 I tried to go through steps on the newest main branches and can't reproduce the issue
https://github.com/user-attachments/assets/5891dc41-ee0f-4474-b82f-5d2dc94c3dbf

Our testers reported that this issue is not consistently reproducible. It's reproduced 1 out of 5 times. 3 Testers were able to reproduce the issue. Here's a video of another attempt:
https://github.com/user-attachments/assets/abad604e-d2f1-423d-8ca8-bf81a10dc5d4

Thanks! I am adding it to the quality project and marking as critical, but given this does not block the user and its not reliably reproducible i dont think it has to be a blocker.
@kirillzyusko any ideas why the keyboard might be causing this? Probably long shot as you dont have access to hybrid

I found out that if we set
shouldEnableKeyboardAvoidingView={false}

in ScreenWrapper props the issue is gone.
Based on that, the bug is somewhere there: https://github.com/Expensify/App/blob/cb0613c083baa987c63a5bdd724a729c5a1795b8/src/components/KeyboardAvoidingView/index.android.tsx

@war-in @mountiny Interesting - looks like KeyboardAvoidingView can not be resized properly 🤔 But very curious why it happens only in Hybrid app?..

After some KeyboardAvoidingView investigation I can tell that the issue comes from the wrong bottom value which is computed here
I think the problem is with the progress value of the keyboard const
keyboard.progress.get()

It returns 1 but I guess it should return 0 when there is no keyboard 🤔

What do you think? Any ideas on what could be the root cause of it?

@war-in First of all I would log values from onStart/onMove/onEnd handlers to understand which handler is broken. Once we understnad that we can go to the native code to see how it's getting calculated so that we can fix it.
But you are right - even when the problem gets fixed on a single page, there is no guarantee that the same problem will not re-appear on other pages. And you are also right - if keyboard is closed then progress should be 0 🙂
Feel free to drop me any messages in Slack - I'm responding much faster there 👀

BugZero Checklist:

[ ] [Contributor] Classify the bug:

Bug classification

Source of bug:

[ ] 1a. Result of the original design (eg. a case wasn't considered)
[ ] 1b. Mistake during implementation
[ ] 1c. Backend bug
[ ] 1z. Other:

Where bug was reported:

[ ] 2a. Reported on production (eg. bug slipped through the normal regression and PR testing process on staging)
[ ] 2b. Reported on staging (eg. found during regression or PR testing)
[ ] 2d. Reported on a PR
[ ] 2z. Other:

Who reported the bug:

[ ] 3a. Expensify user
[ ] 3b. Expensify employee
[ ] 3c. Contributor
[ ] 3d. QA
[ ] 3z. Other:

[ ] [Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
Link to comment:

[ ] [Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
Link to discussion:

[ ] [Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.

Regression Test Proposal Template

[ ] [BugZero Assignee] Create a GH issue for creating/updating the regression test once above steps have been agreed upon.
Link to issue:

Regression Test Proposal
Precondition:

Test:

Do we agree 👍 or 👎

Closing - https://github.com/Expensify/App/issues/53380#issuecomment-2517163504


feat: add FunctionDef::body

Description
Problem
Part of #5668
Summary
Also allows quote { ... }.as_expr() to work with statements and L-values. Then adds an example that injects a _context: Context parameter to functions annotated with #[aztec], and to calls inside those functions (any call for now). Also adds an incomplete Expr::map to help doing this.
Oh, and allows unquoting Value::Expr, which wasn't possible before.
Additional Context
A lot is remaining, but I thought this was a good cutting point. Next I might work on #5828 because debugging comptime code without it is tricky.
I didn't add docs for Expr::map yet as it's still not fully functional. I'll add them once it works for in all cases.
Documentation
Check one:

[ ] No documentation needed.
[ ] Documentation included in this PR.
[x] [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

map -> mutate

Can you rename it modify instead of mutate since we're not mutating the expression?

@michaeljklein


allocator: Fix kvstore identity leak [alternative]

Alternative to https://github.com/cilium/cilium/pull/34893

Please ensure your pull request adheres to the following guidelines:

[ ] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[ ] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[ ] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Are you a user of Cilium? Please add yourself to the Users doc
[ ] Thanks for contributing!

Fixes: #issue-number
<!-- Enter the release note text here if needed or remove this section! -->

Closing after discussion in https://github.com/cilium/cilium/pull/34893/files#diff-d3d4d49f98b76b99ed5652e68f4087224f26abe656791e203d1900859922c977


Display a success message after the member is added

https://discord.com/channels/943612659163602974/1139581328908628078/1141520705658290277

Could be done using Chakra Toast with the Bandada branding and colors: https://chakra-ui.com/docs/components/toast


Fix HTML tables in output cells

Codecov Report

Merging #51 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #51   +/-   ##
=======================================
  Coverage   98.47%   98.48%
=======================================
  Files           5        5
  Lines         328      330    +2
=======================================
+ Hits          323      325    +2
  Misses          5        5

Impacted Files
Coverage Δ

jupyter_docx_bundler/tests/test_converters.py
100.00% <100.00%> (ø)

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 efad335...22e0dc2. Read the comment docs.

Codecov Report

Merging #51 into master will decrease coverage by 0.92%.
The diff coverage is 92.85%.

@@            Coverage Diff             @@
##           master      #51      +/-   ##
==========================================
- Coverage   98.47%   97.54%   -0.93%
==========================================
  Files           5        5
  Lines         328      367      +39
==========================================
+ Hits          323      358      +35
- Misses          5        9       +4

Impacted Files
Coverage Δ

jupyter_docx_bundler/converters.py
92.30% <71.42%> (-3.44%)
:arrow_down:

jupyter_docx_bundler/tests/conftest.py
99.28% <100.00%> (+0.01%)
:arrow_up:

jupyter_docx_bundler/tests/test_converters.py
100.00% <100.00%> (ø)

jupyter_docx_bundler/tests/test_bundler.py
100.00% <0.00%> (ø)

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 efad335...ab29780. Read the comment docs.


node-sass/vendor/linux-x64-57/binding.node: failed to map segment from shared object: Operation not permitted

I create docker-compose.yml configuration and dockerfile for frontend development.
FROM node:8

RUN usermod -u 1000 node
RUN groupmod -g 1000 node

WORKDIR /var/www

USER node

Node image is a debian jessie, SELinux doest not exists
When i up all containers, i got errors on frontend service, example:
[0] ./node_modules/css-loader!./node_modules/sass-loader/lib/loader.js!./application/modules/bonuses/styles.scss 1.27 kB {0} [built] [failed] [1 error]

    ERROR in ./node_modules/css-loader!./node_modules/sass-loader/lib/loader.js!./application/modules/bonuses/styles.scss
    Module build failed: Error: /var/www/node_modules/node-sass/vendor/linux-x64-57/binding.node: failed to map segment from shared object: Operation not permitted
        at Object.Module._extensions..node (module.js:681:18)
        at Module.load (module.js:565:32)
        at tryModuleLoad (module.js:505:12)
        at Function.Module._load (module.js:497:3)
        at Module.require (module.js:596:17)
        at require (internal/module.js:11:18)
        at module.exports (/var/www/node_modules/node-sass/lib/binding.js:19:10)
        at Object.<anonymous> (/var/www/node_modules/node-sass/lib/index.js:14:35)
        at Module._compile (module.js:652:30)
        at Object.Module._extensions..js (module.js:663:10)
        at Module.load (module.js:565:32)
        at tryModuleLoad (module.js:505:12)
        at Function.Module._load (module.js:497:3)
        at Module.require (module.js:596:17)
        at require (internal/module.js:11:18)
        at Object.<anonymous> (/var/www/node_modules/sass-loader/lib/loader.js:3:14)
        at Module._compile (module.js:652:30)
        at Object.Module._extensions..js (module.js:663:10)
        at Module.load (module.js:565:32)
        at tryModuleLoad (module.js:505:12)
        at Function.Module._load (module.js:497:3)
        at Module.require (module.js:596:17)

I try to use any this ways, not working

try to change TMPDIR
try change node version
npm rebuild node-sass

--

NPM version 5.6.0:
Node version v8.11.3:
Node Process

{ http_parser: '2.8.0',
  node: '8.11.3',
  v8: '6.2.414.54',
  uv: '1.19.1',
  zlib: '1.2.11',
  ares: '1.10.1-DEV',
  modules: '57',
  nghttp2: '1.32.0',
  napi: '3',
  openssl: '1.0.2o',
  icu: '60.1',
  unicode: '10.0',
  cldr: '32.0',
  tz: '2017c' }

Node Platform linux:
Node architecture x64:
node-sass version

node@603519e433f6:/var/www$ node -p "require('node-sass').info"
module.js:681
  return process.dlopen(module, path._makeLong(filename));
                 ^

Error: /var/www/node_modules/node-sass/vendor/linux-x64-57/binding.node: failed to map segment from shared object: Operation not permitted
    at Object.Module._extensions..node (module.js:681:18)
    at Module.load (module.js:565:32)
    at tryModuleLoad (module.js:505:12)
    at Function.Module._load (module.js:497:3)
    at Module.require (module.js:596:17)
    at require (internal/module.js:11:18)
    at module.exports (/var/www/node_modules/node-sass/lib/binding.js:19:10)
    at Object.<anonymous> (/var/www/node_modules/node-sass/lib/index.js:14:35)
    at Module._compile (module.js:652:30)
    at Object.Module._extensions..js (module.js:663:10)

npm node-sass versions --<EMAIL_ADDRESS>

Run npm rebuild node-sass inside the container
On Fri., 10 Aug. 2018, 6:16 pm Andrey<EMAIL_ADDRESS>wrote:

I create docker-compose.yml configuration and dockerfile for frontend
development.
FROM node:8
RUN usermod -u 1000 node
RUN groupmod -g 1000 node
WORKDIR /var/www
USER node
When i up all containers, i got errors on frontend service, example:
[0] ./node_modules/css-loader!./node_modules/sass-loader/lib/loader.js!./application/modules/bonuses/styles.scss 1.27 kB {0} [built] [failed] [1 error]
ERROR in ./node_modules/css-loader!./node_modules/sass-loader/lib/loader.js!./application/modules/bonuses/styles.scss
Module build failed: Error: /var/www/node_modules/node-sass/vendor/linux-x64-57/binding.node: failed to map segment from shared object: Operation not permitted
    at Object.Module._extensions..node (module.js:681:18)
    at Module.load (module.js:565:32)
    at tryModuleLoad (module.js:505:12)
    at Function.Module._load (module.js:497:3)
    at Module.require (module.js:596:17)
    at require (internal/module.js:11:18)
    at module.exports (/var/www/node_modules/node-sass/lib/binding.js:19:10)
    at Object.<anonymous> (/var/www/node_modules/node-sass/lib/index.js:14:35)
    at Module._compile (module.js:652:30)
    at Object.Module._extensions..js (module.js:663:10)
    at Module.load (module.js:565:32)
    at tryModuleLoad (module.js:505:12)
    at Function.Module._load (module.js:497:3)
    at Module.require (module.js:596:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/var/www/node_modules/sass-loader/lib/loader.js:3:14)
    at Module._compile (module.js:652:30)
    at Object.Module._extensions..js (module.js:663:10)
    at Module.load (module.js:565:32)
    at tryModuleLoad (module.js:505:12)
    at Function.Module._load (module.js:497:3)
    at Module.require (module.js:596:17)

I try to use any this ways, not working

try to change TMPDIR
try change node version
npm rebuild node-sass

--

NPM version 5.6.0:
Node version v8.11.3:
Node Process

{ http_parser: '2.8.0',
node: '8.11.3',
v8: '6.2.414.54',
uv: '1.19.1',
zlib: '1.2.11',
ares: '1.10.1-DEV',
modules: '57',
nghttp2: '1.32.0',
napi: '3',
openssl: '1.0.2o',
icu: '60.1',
unicode: '10.0',
cldr: '32.0',
tz: '2017c' }

Node Platform linux:
Node architecture x64:
node-sass version

node@603519e433f6:/var/www$ node -p "require('node-sass').info"
module.js:681
return process.dlopen(module, path._makeLong(filename));
^
Error: /var/www/node_modules/node-sass/vendor/linux-x64-57/binding.node: failed to map segment from shared object: Operation not permitted
at Object.Module._extensions..node (module.js:681:18)
at Module.load (module.js:565:32)
at tryModuleLoad (module.js:505:12)
at Function.Module._load (module.js:497:3)
at Module.require (module.js:596:17)
at require (internal/module.js:11:18)
at module.exports (/var/www/node_modules/node-sass/lib/binding.js:19:10)
at Object. (/var/www/node_modules/node-sass/lib/index.js:14:35)
at Module._compile (module.js:652:30)
at Object.Module._extensions..js (module.js:663:10)

npm node-sass versions --<EMAIL_ADDRESS>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sass/node-sass/issues/2474, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjZWKPvgf9hrS_kTWPmWhXnxjeHiS82ks5uPUFXgaJpZM4V3xuy
.

@xzyfer not working

You didn't post the output of a failure.

@saper https://pastebin.com/9i1PsBSi - output of npm rebuild node-sass inside container

Sounds similar to this https://github.com/lovell/sharp/issues/704#issuecomment-278354475 this is a problem with the mounting
Also https://stackoverflow.com/questions/45077967/error-with-webpack-vue-cli-sass-failed-to-map-segment-from-shared-object

I try to install global node-sass and i created link to locally project, its work, but it bullshit ( sorry for bad word )
i need to right way to solve this problem (

Yes, thisi a problem with a filesystem. Probably set to noexec. Probably your image is doing this for you, the Stackoverflow post about changing /etc/fstab quoted by @nschonni is most probably the right answer. It is not related to node-sass at all.

Linking may work because things are on different filesystems probably. Please research your environment more carefully.

I think I'm running into this on a Chromebook and I don't think there is an easy way to get around it?
><EMAIL_ADDRESS>postinstall /home/chronos/u-6264041317e5864e8f0891065e7585fa25e8958c/Downloads/projects/vue-starter/node_modules/node-sass
> node scripts/build.js

Binary found at /home/chronos/u-6264041317e5864e8f0891065e7585fa25e8958c/Downloads/projects/vue-starter/node_modules/node-sass/vendor/linux-x64-57/binding.node
Testing binary
Binary has a problem: Error: /home/chronos/u-6264041317e5864e8f0891065e7585fa25e8958c/Downloads/projects/vue-starter/node_modules/node-sass/vendor/linux-x64-57/binding.node: failed to map segment from shared object
    at Object.Module._extensions..node (module.js:682:18)
    at Module.load (module.js:566:32)
    at tryModuleLoad (module.js:506:12)
    at Function.Module._load (module.js:498:3)
    at Module.require (module.js:597:17)
    at require (internal/module.js:11:18)
    at module.exports (/home/chronos/u-6264041317e5864e8f0891065e7585fa25e8958c/Downloads/projects/vue-starter/node_modules/node-sass/lib/binding.js:19:10)
    at Object.<anonymous> (/home/chronos/u-6264041317e5864e8f0891065e7585fa25e8958c/Downloads/projects/vue-starter/node_modules/node-sass/lib/index.js:14:35)
    at Module._compile (module.js:653:30)
    at Object.Module._extensions..js (module.js:664:10)

The mount output for this dir:
/dev/mmcblk0p1 on /home/user/6264041317e5864e8f0891065e7585fa25e8958c type ext4 (rw,nosuid,nodev,noexec,noatime,commit=600,data=ordered)
Not convinced this is easily solvable for me?

I would say not. Chromebooks must have a non-standard libc
On Sat., 13 Oct. 2018, 12:52 am delanthear<EMAIL_ADDRESS>wrote:

I think I'm running into this on a Chromebook and I don't think there is
an easy way to get around it?

<EMAIL_ADDRESS>postinstall /home/chronos/u-6264041317e5864e8f0891065e7585fa25e8958c/Downloads/projects/vue-starter/node_modules/node-sass
node scripts/build.js

Binary found at /home/chronos/u-6264041317e5864e8f0891065e7585fa25e8958c/Downloads/projects/vue-starter/node_modules/node-sass/vendor/linux-x64-57/binding.node
Testing binary
Binary has a problem: Error: /home/chronos/u-6264041317e5864e8f0891065e7585fa25e8958c/Downloads/projects/vue-starter/node_modules/node-sass/vendor/linux-x64-57/binding.node: failed to map segment from shared object
at Object.Module._extensions..node (module.js:682:18)
at Module.load (module.js:566:32)
at tryModuleLoad (module.js:506:12)
at Function.Module._load (module.js:498:3)
at Module.require (module.js:597:17)
at require (internal/module.js:11:18)
at module.exports (/home/chronos/u-6264041317e5864e8f0891065e7585fa25e8958c/Downloads/projects/vue-starter/node_modules/node-sass/lib/binding.js:19:10)
at Object. (/home/chronos/u-6264041317e5864e8f0891065e7585fa25e8958c/Downloads/projects/vue-starter/node_modules/node-sass/lib/index.js:14:35)
at Module._compile (module.js:653:30)
at Object.Module._extensions..js (module.js:664:10)
The mount output for this dir:
/dev/mmcblk0p1 on /home/user/6264041317e5864e8f0891065e7585fa25e8958c type
ext4 (rw,nosuid,nodev,noexec,noatime,commit=600,data=ordered)
Not convinced this is easily solvable for me?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sass/node-sass/issues/2474#issuecomment-429332885,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjZWCQtBo2kHt4KIVKlKh5TJfGHEfzNks5ukJ6JgaJpZM4V3xuy
.

Did the comment you responded to got deleted? I think the failed to map segment from shared object can be caused by having the filesystem mounted as noexec.

Apologies I deleted the comment as I got around it.  I didn't expect it to be picked up and responded to :)
I managed to install node-sass using the global option and while I was in a different directory (no idea if this affected it) and it installed fine!  (no idea!)

Yes, this was confusing - I got an email notification and then I could not find it on Github. Yes, it is quite possible that your global installation directory is on another file system. You can say "mount" and see all your file systems. There is probably "noexec" word in the line that begins with "/home" and there is probably nothing like that in "/usr" or wherever your global install directory might be.

+1 to @saper . The default (at least on ubuntu) for a mount on fstab is "noexec". Adding the "exec" to the fstab options fixed this. Someone should fix this error message.

@surak The message comes from the kernel and the C library. Sorry for that, but it cannot be fixed here.

Thanks @saper , I mean that since the error is known, a useful message would be more helpful than a simple stack trace, don't you think?

It is not known that filesystem settings are the cause of this. This is only one of the possible explanations what happened.
This error is not generated by node-sass, it comes from the way node itself uses system-specific way to load a shared library.


repeated efi entry

Im currently dual booting with windows, with the clean install i had just 1 macos entry and just 1 BOOTcamp entry, after updating the hac-mini from 2.2 to 2.3.3, Im getting BOOTcamp  2 times and mojave...
Did clear the nvram and no change.. any guide on how to delete de duplicated entry?

Seems like an OC issue. 2.2 was using OC 0.0.4 and 2.3 uses OC 0.5.1.

It's also possible you installed the EFI files to both your Windows partition (NTFS) and the ESP partition (where OC resides). Anyways this is not a hac mini issue, so closing.


Add Nong Song:

Song Name
Bling Bang Bang Born
Artist Name
Creppy Nuts
Source
Direct file download
Youtube Link
No response
Direct File Link
https://nahieluniversal.7m.pl/nongs/102030.mp3
Song ID
102030
Start Offset [ms]
No response
Extra
Used in Bling Bang Bang Born by ph4lip ID:102986289

A


Animation Recorder: Recording an "Animator" controller generates 80 errors in the console

Trying to record the Animator component will generates a set of 80 errors once we try to show the asset in the inspector.

Record a session on Animator
Stop the record session
Browse to the generated anim clip
Once we select the asset, errors are printed in the console

Then any subsequent record on an animator will generate the same errors when we start recording.

Issue Tracker link: https://issuetracker.unity3d.com/issues/animation-recorder-error-in-console-when-recording-custom-or-third-party-component

unity issue. closing it here.


Update cluster definition.md by adding location and kubernetesbaseimage

@wangtt03,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


chore: 🤖 bump node

so we can use nullish coalescing and optional chaining

Looks like there is a required build(12.x) from master which is just pending..bug? It won't succeed anyways. I wonder what is the best practice to do this update when we have actions set up like this.

@momentiris it had to do with the branch protection rules, I updated them now and I will merge it right away so other branches can make use of this! 👍


Add dask.dataframe as dependency

Closes #537

I'll merge this now, and fix up the failing CI later today (seems to primarily array_function fallout).


download button added

To download glb or other model files loaded in the visualizer, otherwise we have to copy file from server's output folder.
https://github.com/user-attachments/assets/fbfcac31-9b36-48f7-9ea0-e480d7d076c3
@MrForExample If this PR looks ok then can you please test it once locally. I tested it on my local machine with some workarounds in place (because macos 😞 )

Thank you for your contribution 👍

@MrForExample it already downloads with correct extension, checked with .glb and .obj files

Thank you for your contribution!!!!!!
When my comfyui is not deployed locally, it will not be displayed normally.
I have added client IP in configs/system.conf file.
Do you have this problem? Thanks 🙇

To download glb or other model files loaded in the visualizer, otherwise we have to copy file from server's output folder.
Screen.Recording.2024-07-13.at.9.45.00.PM.mov
@MrForExample If this PR looks ok then can you please test it once locally. I tested it on my local machine with some workarounds in place (because macos 😞 )

Hello, thank you for your great contribution. I wonder how to right-click the node, and then download in the menu, then how should I implement it?
Specifically, it is in the folder--"web/scripts/app.js"，and code --"node.prototype.getExtraMenuOptions = function (_, options)".

Thank you for your contribution!!!!!! When my comfyui is not deployed locally, it will not be displayed normally. I have added client IP in configs/system.conf file. Do you have this problem? Thanks 🙇

Hello @QianquanChina I never had to configure any IPs in the conf file. I have always hosted ComfyUI on a normal vm instance behind an nginx server.

To download glb or other model files loaded in the visualizer, otherwise we have to copy file from server's output folder.
Screen.Recording.2024-07-13.at.9.45.00.PM.mov
@MrForExample If this PR looks ok then can you please test it once locally. I tested it on my local machine with some workarounds in place (because macos 😞 )

Hello, thank you for your great contribution. I wonder how to right-click the node, and then download in the menu, then how should I implement it? Specifically, it is in the folder--"web/scripts/app.js"，and code --"node.prototype.getExtraMenuOptions = function (_, options)".

@WorkSync I never though of doing it like that but I think that would be better than just adding a button. Did you figure out any way to implement it? I haven't looked at it yet.


fix(core): reset model on each change

Otherwise there's the issue that form fields don't get properly
recreated (i.e. form array parts)
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Fixes #768
Please check if the PR fulfills these requirements

[x] The commit messages follow our guidelines: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format
[ ] A unit test has been written for this change.
[x] Running npm run build produced a successful build. (Unit testing can be done by running npm test;)
[x] My code has been linted. (npm run lint to do this. npm run build will fail if there are files not linted.)

This change is

@aitboudad applied the change, however it still doesn't work apparently.

then debugging :)
   private resetForm(fields: FormlyFieldConfig[], newModel: any, modelToUpdate: any) {
     fields.forEach(field => {
-      if (field.fieldGroup && field.fieldGroup.length > 0) {
+      if ((field.fieldGroup && field.fieldGroup.length > 0) || field.fieldArray) {
         let newFieldModel = this.fieldModel(field, newModel),
           fieldModel = this.fieldModel(field, modelToUpdate);
+
         if (field.fieldArray) {
           field.fieldGroup.length = 0;
-          fieldModel.length = 0;
+          if (fieldModel !== newFieldModel) {
+            fieldModel.length = 0;
+          }
+
           const formControl = <FormArray>field.formControl;
           while (formControl.length !== 0) {
             formControl.removeAt(0);

just apply the above patch and it should work

Ya sorry, had to leave work & couldn't continue to debug it. Will fix tomorrow, thx man 🙂

Codecov Report

Merging #770 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master     #770   +/-   ##
=======================================
  Coverage   94.45%   94.45%
=======================================
  Files          18       18
  Lines         793      793
  Branches      227      227
=======================================
  Hits          749      749
  Misses         44       44

Impacted Files
Coverage Δ

src/core/src/components/formly.form.ts
95.94% <0%> (ø)
: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 b19c593...87586a1. Read the comment docs.

Thank you!


Remove minimap

Hi, I am trying to remove the minimap that's in the lower part of the screen during the game.
Does anybody know where I should look in the game code?
Thanks

Hi, You can disable minimap by not constructing Gui2Radar object here:
https://github.com/google-research/football/blob/master/third_party/gfootball_engine/src/onthepitch/match.cpp#L237
and not referencing it later on.
We can consider adding a flag to control this behavior, but we would like to better understand the use case first (maybe score board should be hidden as well?).
Can you tell us what is your use case?
Thanks.

Thank you, that worked!
I am working on an action recognition system so I needed the game video without extra elements for training


The reply address does not match

Hi,
I've followed the instructions but can not get it to work correctly.
I keep getting the error : "The reply address 'http://localhost:10800/Home/Authorize' does not match the reply addresses configured for the application"
I get the same result with :
Microsoft.Office365.OutlookServices-V2.0 version="<IP_ADDRESS>"
Microsoft.Office365.OutlookServices-V2.0 version="<IP_ADDRESS>"
Microsoft.Experimental.IdentityModel.Clients.ActiveDirectory version="4.0.209160138-alpha"
Microsoft.Experimental.IdentityModel.Clients.ActiveDirectory version="4.0.208020147-alpha"
I've tried various settings like changing case and toggling the "Allow Implicit Flow" option.
Any ideas what it might be?

The Azure auth platform is case-sensitive when it compares the redirect URIs. So you have it in all lowercase in your app registration, but the app is using capitals. Try recreating your registration with a redirect URI that uses the same case as the app.

You are correct, I was changing it in the Authorize action but not the SignIn action. Changed it to lowercase and it worked.
In the apps.dev.microsoft.com portal any URL I entered was converted to lowercase.
Thanks for your help.

Guys? i know i'm late to the party, but we in azure could i adjust this? i.e the reply address?

I'm getting this but every solution is for Azure AD when I don't have Azure AD. The company I'm with has regular AD that they provide themselves. All the documentation I find has to do with Azure, where is the docs for doing this on regular AD that's not hosted on Azure?


Warning calling find_package(g3log) in newer CMake

Describe the bug
g3log is built and installed, and I'm using find_package() like below to include it in a different CMake project.
find_package(g3log PATHS <actual paths> REQUIRED)

This uses config mode.  With CMake 3.19.8, the following warning is issued:
CMake Warning (dev) at /opt/local/share/cmake-3.19/Modules/FindPackageHandleStandardArgs.cmake:426 (message):
  The package name passed to `find_package_handle_standard_args` (G3LOG) does
  not match the name of the calling package (g3log).  This can lead to
  problems in calling code that expects `find_package` result variables
  (e.g., `_FOUND`) to follow a certain pattern.
Call Stack (most recent call first):
  /Users/greg/Software/Installed/gems-tpls/lib/cmake/g3log/g3logConfig.cmake:74 (find_package_handle_standard_args)
  CMakeLists.txt:498 (find_package)
This warning is for project developers.  Use -Wno-dev to suppress it.

To Reproduce
I'm running on macOS Big Sur, but after some searching it seems this warning was added to CMake in version 3.17 and people have encountered this on other OSes including ubuntu.  It's probably reproducible in any environment as long as the CMake version is 3.17 or higher.
Additional context
This is just a warning, and I can suppress it, but it would be nice if the issue were fixed.  Thanks for the work on this software!  It's a great package for logging.

Yup it’s benign, still a good warning.  Thanks for calling it out
G3log is community driven so unless there’s a severe bug I am mostly helping out with code reviews and feature discussions
I would suggest that you create a PR to resolve this issue and I’ll review it promptly

Reopening (to later close it hopefully) with the pending PR.

thanks to @zjeffer for addressing this in https://github.com/KjellKod/g3log/pull/457 and for @gmackey  who reported it in https://github.com/KjellKod/g3log/issues/413
Update in g3sinks will happen shortly as well: https://github.com/KjellKod/g3sinks/pull/125


Add Algorithms in Java | As many as you can!

Assign me this issue I have some java data structures and algorithms

Hey @Kanishq10! You can start working on it! 🔥

Assign me this issue, I have lot of Java data structures and algorithms problems

Assign me this issue, I will add some Java data structures and algorithms

Hey, I have some experience in Java and Competitive Programming. Looking to help!

Hi @nishant-ai ,
could you please assign the issue to me. I could share some techniques/algorithms which can help people in CP.
Would be happy to share my knowledge with the help of your repo.
Cheers!

I would like to contribute. Please assign the issue to me.

@nishant-ai  I would like to contribute. Please assign the issue to me.

Added PR #154
Array & Matrices solutions. Please review and merge.

Assign me this issue, I have to add some Java data structures and algorithms


Thumbnails on main page not showing up?

Hello,
This is a great theme and I'm loving it. Thank you for sharing. I noticed in your preview example that in your "Recent Posts" module, there are small thumbnails next to your article post names. Mine are not showing up for whatever reason (see attached image below). How are you achieving this?

Best,
M

Hi, @mvxt
You can refer to my hexo source:  https://github.com/iTimeTraveler/hexo-theme-hiero/tree/site-source
this is a header of example post markdown file:
---
title: Hillary Clinton Calls for Unity in Concession Speech
date: 2016-11-10 09:55:37
comments: true
categories:
	- LifeStyle
photos:
	- https://hillaryclintonspeeches.files.wordpress.com/2016/11/10clinton2-master768.jpg
---

the photos field is the thumbnail that will be shown on Recent Posts widget.

Thanks, works perfectly!

Looking to remove the photo from the main post, but keep the thumbnail.
Removing this line from article.ejs, failed to produce the desired functionality.

    <a href="<%- url_for(post.path) %>"><img src="<%= post.photos[0] %>" class="attachment-thumb-featured size-thumb-featured wp-post-image" alt="">

Any ideas?

@kiarafrobles I actually did the same thing to my articles. Remove line 3 from /layout/_partial/article.ejs.
3    <%- partial('post/gallery') %>


Add listener on close.

Hello, i want to do something when the user close the notification, is this possible?
Thanks!

Yes absolutely. You can specify a onclose callback with the options.
Refer Docs


when using dom renderer, command execution stalls and cursor can be in the wrong place

Testing #161851
with gpuAcceleration:off EDIT or on
https://user-images.githubusercontent.com/29464607/192535337-bb6bab6e-8fb8-4315-8d2c-c781124aa284.mov

on bash the stall is less, but nothing gets printed

stalls in bash and puts the cursor in the wrong place when I echo though 🤔

also happens with gpuAcceleration:on

@jrieken did some change happen around trusted types? The dynamically loaded xterm.js script needs to set inline attributes/innerHTML

@jrieken did some change happen around trusted types?

No, nothing that I am aware of. Tho, the charElement. innerHTML = 'Enbsp;": & assignment (https://github.com/microsoft/vscode/issues/161970#issuecomment-1259488459) should have never worked with trusted types enabled because it's a bare string, not TrustedHTML
If you have to use innerHTML assignment (which should be the exception) you need to use a trusted types policy and create trusted html through that. This is a sample:
https://github.com/microsoft/vscode/blob/1643a42b2510f99d524135e270e218c5220731ef/src/vs/editor/contrib/stickyScroll/browser/stickyScrollWidget.ts#L218.
However, you should try to avoid that and keep things more simple, e.g use innerText and/or appendChild

Ah you're right, this was a coding error introduced with underlines: https://github.com/xtermjs/xterm.js/commit/72d18f7d5e


Log format issues with VCOMBINED format

Command line args: -log-format '%v:%^ %h %^[%d:%t %^] "%r" %s %b "%R" "%u"' --time-format '%H:%M:%S' --date-format '%d/%b/%y'
Sample log:
subdomain.example.com <IP_ADDRESS> - [16/Apr/2017:10:00:39 -0400] "GET /path/to/page HTTP/2.0" 200 12598 "http://referrer.example.com/" "Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:52.0) Gecko/20100101 Firefox/52.0"

Error: Token '-0400]' doesn't match specifier '%h'
I have tried many different format strings, but it never seems to work.
Thanks in advance!

Please try this:
goaccess access.log --log-format='%v %h %^[%d:%t %^] "%r" %s %b "%R" "%u"' --date-format=%d/%b/%Y --time-format=%T

Thanks so much! It worked. I think it would be a good idea to add documentation for the format somewhere. Also, another questionL what does %^ mean?


Correct CertificateRequest documentation

Signed-off-by: Katie<EMAIL_ADDRESS>I believe this doc said controller on line 52 when it meant to say certificate signing request, or perhaps just certificate?

/assign @joshvanl

Indeed, thanks for that @katiekeel  :)
/lgtm
/approve
/ok-to-test


Added /whoami endpoint

What
Added a /whoami endpoint to the auth handlers.
Why
Lets the frontend check that someone is actually logged in to detect Lighthouse restarting and other auth failures.
Endpoints
GET /whoami HTTP/1.1

Examples
Someone is logged in:
HTTP/1.1 200 OK
<EMAIL_ADDRESS>
Someone is not logged in:
HTTP/1.1 404 Not Found

@ngmiller whill this work for you for the auth stuff we've been working on?

I think this will take care of all the auth edge cases we've encountered. I'll give it a shot within the next couple of hours and let you know!


Strange behavior when zooming Mermaid

Since some versions of hugo-theme-relearn, zooming Mermaid does not work as expected.
See for example:
https://grafioschtrader.github.io/gt-user-manual/de/admindata/taskdatachangemonitor/index.html
or also
https://grafioschtrader.github.io/gt-user-manual/de/watchlistinstrument/index.html
The text/code can be found at gt-user-manual

Hi, thanks for reporting.

zooming Mermaid does not work as expected

What do you mean by that? Using Pointer events by using two fingers on a track pad of my Windows notebook works as expected with your example.
The last fix was in version 5.19.0 which is used in your examnple.

You are right. It works flawlessly on almost all platforms. The problem relates to Google Chrome (not Firefox and Edge) in a Windows 11 environment.  Unfortunately, I don't have a second Windows 11 where I can verify this behavior. The scaling step is huge (for WheelEvent, the deltaY is: 2350) when using the mouse scroll wheel. When this is done, the chart gets lost and disappears completely. Surprisingly, zooming works in Chromium on Linux or Chrome on Windows 10. You are probably not responsible for this and can close this issue.

I couldn't reproduce this even with a mouse (Logitech MX Master 3). I am also using Windows 11 and the latest Chrome - so the environment looks pretty much the same. Maybe you've reconfigured the step amount for the mouse wheel in your system settings?
Zooming is done by d3.js and I doubt that this is an issue on their side. Because of its wide popularity this should have been experienced by other users, too - which isn't as far as I know.

@hugograf As I couldn't reproduce it in the way you described it, I saw a different issue with the Mermaid zoom and fixed it.
Maybe you want to give that fix a second try and report back?


erroneous #if __cplusplus

in  rosidl_typesupport_microxrcedds/rosidl_typesupport_microxrcedds_c/resource/rosidl_typesupport_microxrcedds_c__visibility_control.h.in, the statement #if __cplusplus is used.
This causes errors when not compiling with C++. It should read #ifdef __cplusplus instead. The dependent statements in the same file need to be adjusted as well.

Solved in https://github.com/micro-ROS/rosidl_typesupport_microxrcedds/pull/42


[BUG] Migrating from EventSender to PartitionSender - how to tell if sender is closed?

I am trying to migrate code that used the old EventHubSender from the Microsoft.ServiceBus (.Net Framework) to the new PartitionSender in the Microsoft.Azure.EventHubs package. In the old package I could run OnClosed on the sender and recreate it if needed. However, I can't find any way to do it here.
I have a "sender pool" that I am maintaining so I want to make sure that no one closed the sender before I pass it to some other call.
Any direction in this matter will be greatly appreciated, thanks!

Unfortunately clients don't have a way to tell whether close called or not on them. There is a flag for this hidden for internal use only. I will send a PR to make it available publicly.

PR here https://github.com/Azure/azure-sdk-for-net/pull/7365

Fix is merged to master.


feat: support init SDK with custom configuration

Description

support init SDK with custom configuration
this PR makes breaking changes to the update configuration api

General Checklist

[x] Added new tests to cover change, if needed
[x] Build succeeds using Swift Package Manager
[x] All unit tests pass
[ ] Security oriented best practices and standards are followed (e.g. using input sanitization, principle of least privilege, etc)
[x] Documentation update for the change if required
[x] PR title conforms to conventional commit style
[x] If breaking change, documentation/changelog update with migration instructions

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

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

Project coverage is 96.66%. Comparing base (0837645) to head (59b4b2f).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #59      +/-   ##
==========================================
+ Coverage   95.66%   96.66%   +0.99%
==========================================
  Files          35       35
  Lines        1985     2127     +142
==========================================
+ Hits         1899     2056     +157
+ Misses         86       71      -15

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


IBM User Build - ${buildFile.workspaceDir} return a relative path including also Workspace name (not relative to Workspace)

Development environment used

Z Open Editor version: 3.0.0
Editor Platform

[X] Visual Studio Code
[ ] Red Hat CodeReady Workspaces
[ ] Eclipse Che
[ ] Standalone Theia

Editor Platform Version: 1.72.2
Operating System: Windows 10
Java Version: 11
[ ] Related to RSE API?

RSE API Plugin version:
Zowe CLI version:
Node.js version:

Logs attached: yes

setup and run IBM user build.txt
Problem Description
Detailed steps for reproducing the problem:

Define the dbb profile as this:

profiles:
  - name: dbb-userbuild
    type: dbb
    settings:
      application: CEAB3.MVP2
      command: "$DBB_HOME/bin/groovyz"
      buildScriptPath: "/var/DBB/PROD/SCRIPTS/buildBPCE.groovy"
      buildScriptArgs:
        - "--userBuild"
        - "--workspace ${zopeneditor.userbuild.userSettings.dbbWorkspace}"
        - "--application ${application}"
        - "--hlq ${zopeneditor.userbuild.userSettings.dbbHlq}.${application}"
        - "--outDir ${zopeneditor.userbuild.userSettings.dbbLogDir}"
        - "--dependencyFile ${dependencyFile}"
      additionalDependencies:
        - application-conf
        - .build
        - "${buildFile.workspaceDir}\\${buildFile}.build"     <== problem here
      logFilePatterns:
        - "${buildFile.basename}.log"
        - "BuildReport.*"

create a src\COBOL\S9TL1A.cbl and a src\COBOL\S9TL1A.cbl.build

run "setup for IBM User Build" from src\COBOL\S9TL1A.cbl

src\COBOL\S9TL1A.cbl is relative to the Workspace.
Observed behavior
Identified the local root workspace path as c:\Trv\DepotsGIT\ceab3-environn-outinfr-central-mvp2
Running setup to build c:\Trv\DepotsGIT\ceab3-environn-outinfr-central-mvp2\src\COBOL\S9TL1A.cbl program.
[...]
Uploading additional dependencies matched inside workspace of the build file ...
Matching files for application-conf
c:/Trv/DepotsGIT/ceab3-environn-outinfr-central-mvp2/application-conf
Matching files for .build
c:/Trv/DepotsGIT/ceab3-environn-outinfr-central-mvp2/.build
Matching files for ceab3-environn-outinfr-central-mvp2\src\COBOL\S9TL1A.cbl.build
No matched files found

The ${buildFile.workspaceDir} variable does not return the path relative to the Workspace folder, but returns a path including the name of the Workspace folder.
${buildFile.workspaceDir}\\${buildFile}.build is translated as ceab3-environn-outinfr-central-mvp2\src\COBOL\S9TL1A.cbl.build.
ceab3-environn-outinfr-central-mvp2 is the name and folder of the Workspace itself.
This is not consistent with the path to the file to build relative to the Workspace.
This is also not in agreement with the documentation: section Substitution variables of https://www.ibm.com/docs/en/cloud-paks/z-modernization-stack/2022.4?topic=code-setting-up-user-build:

As-is not usable in IBM User Build setup for a Git repository opened in VS Code.
Expected behavior
Expected translation for ${buildFile.workspaceDir}\\${buildFile}.build from  src\COBOL\S9TL1A.cbl is src\COBOL\S9TL1A.cbl.build or to a full path as c:\Trv\DepotsGIT\ceab3-environn-outinfr-central-mvp2\src\COBOL\S9TL1A.cbl.
Workaround
profiles:
  - name: dbb-userbuild
    type: dbb
    settings:
      application: CEAB3.MVP2
      command: "$DBB_HOME/bin/groovyz"
      buildScriptPath: "/var/DBB/PROD/SCRIPTS/buildBPCE.groovy"
      buildScriptArgs:
        - "--userBuild"
        - "--workspace ${zopeneditor.userbuild.userSettings.dbbWorkspace}"
        - "--application ${application}"
        - "--hlq ${zopeneditor.userbuild.userSettings.dbbHlq}.${application}"
        - "--outDir ${zopeneditor.userbuild.userSettings.dbbLogDir}"
        - "--dependencyFile ${dependencyFile}"
      additionalDependencies:
        - application-conf
        - .build
        - "src\\**\\${buildFile}.build"     <== workaround
      logFilePatterns:
        - "${buildFile.basename}.log"
        - "BuildReport.*"

Matching files for src\**\S9TL1A.cbl.build
c:/Trv/DepotsGIT/ceab3-environn-outinfr-central-mvp2/src/COBOL/S9TL1A.cbl.build

The problem has been identified...
The people in charge of implementing DBB on our site have modified the access paths for storing logs to meet constraints related to our context:
Original version (zAppbuil 1.0.3)

build.groovy

if (props.userBuild) {	// do not create a subfolder for user builds
  props.buildOutDir = "${props.outDir}" as String }
else {// validate createBuildOutputSubfolder build property
  props.buildOutDir = ((props.createBuildOutputSubfolder && props.createBuildOutputSubfolder.toBoolean()) ? "${props.outDir}/${props.applicationBuildLabel}" : "${props.outDir}") as String
}

Cobol.groovy

File logFile = new File(props.userBuild ? "${props.buildOutDir}/${member}.log": "${props.buildOutDir}/${member}.cobol.log")`

Modified version

build.groovy

if (props.userBuild) {	// do not create a subfolder for user builds
  //props.buildOutDir = "${props.outDir}" as String
  props.buildOutDir = ((props.createBuildOutputSubfolder && props.createBuildOutputSubfolder.toBoolean()) ? "${props.outDir}/${props.applicationBuildLabel}" : "${props.outDir}") as String
}
else {// validate createBuildOutputSubfolder build property
  props.buildOutDir = ((props.createBuildOutputSubfolder && props.createBuildOutputSubfolder.toBoolean()) ? "${props.outDir}/${props.applicationBuildLabel}" : "${props.outDir}") as String
}

Cobol.groovy

File logFile = new File( props.userBuild ? "${props.buildOutDir}/logs/$systemName/${member}.log": "${props.buildOutDir}/logs/$systemName/${member}.cobol .log")

This demonstrates that IBM User Build of ZOE is in strong adherence with zAppbuild and that any modification made to zAppbuild breaks IBM User Build...
There should be more independence between IBM User Build and zAppbuild.

This issue has been fixed.


feat(gatsby-plugin-fastify): add customizable headers with sensible defaults for security

Description
@moonmeister I think I've got this working as expected now. Please take a look and let me know.

options.headers plugin configuration added for applying custom headers to specific files/paths
default security headers added

Documentation
Headers section at the bottom of the README
Related Issues
closes #98
fixes #59
@moonmeister test Gatsby Node API › Should Build Config is failing - I'm not sure why. Can you take a look?

@moonmeister what's the best way to test this out in a real site pre-release? For example, can we do an experimental release?

Great question, It is possible, but I've never done it with Changesets, I'll try to make time to do some reading...though if you want to look into the docs and see what's possible that'd be helpful.

@moonmeister what's the best way to test this out in a real site pre-release? For example, can we do an experimental release?

Great question, It is possible, but I've never done it with Changesets, I'll try to make time to do some reading...though if you want to look into the docs and see what's possible that'd be helpful.

I think I'll need to manually do a pre-release. LEt's get this a little further along then I can do that.

Thanks @moonmeister - getting back to it today.
FYI:

@moonmeister test Gatsby Node API › Should Build Config is failing - I'm not sure why. Can you take a look?

I think I figured out why this is failing, looking at a fix. Having to do with trying to read the webpack.stats.json from the __tests__ files which is now used in onPostBuild but wasn't previously

@moonmeister I've refactored the code with a new headers plugin that takes care of adding all of the necessary headers for all modules in one place via hooks/decorators. It now also supports the rest of the modules as well. I've also moved tests for headers out of each modules test suite into a new headers specific test suite and added test coverage for header-builder.ts. I think it should resolve all of the previous comments. Please take a look and let me know!

Already looking better, but I realized I need to do this on a computer. Will try and get to it soon. Thanks @tsdexter !!!!

Thanks again @tsdexter ...we're getting closer...step by step. Lots in that review...sorry. I can make some time to write code soon so if you are tight on time, do what you can and let me know specific things I can help with...but do as much as you want. I'm happy to keep providing feedback.

Okay, so I think we need to document some of these complex interactions of what does and doesn't recieve various headers. This will help us. future maintainers, and users hopefully. PLEASE VERIFY EVERYTHING HERE. Things with question marks mean I didn't know...Answers with an additional question mark mean I was unsure. Once we verify all this and finalize all answers we can merge. Please add any rows/columns I may habve missed.

Default Security Headers
Y?
Y?
Y?
Y?
Y?
Y?
Y?
Y?
N
Y?
Y?
Y?

Y
??

The only thing not reflected here is which override which...I think that can just be explained.

@moonmeister  nothing to review tonight, but I made good progress today...will have a push ready sometime tomorrow which I think will definitely get us in a ready to release state. At least ready for a pre-release and more extensive testing if you want to do that first - our site is ~7000 pages and uses most features so I can do some testing there.

@moonmeister  nothing to review tonight, but I made good progress today...will have a push ready sometime tomorrow which I think will definitely get us in a ready to release state. At least ready for a pre-release and more extensive testing if you want to do that first - our site is ~7000 pages and uses most features so I can do some testing there.

@moonmeister just finishing up the rest of the tests against your feature table and new benchmarks...gotta head out, but I'll try to get back to finishing it up and committing later tonight

@moonmeister the test site with the updated version should build on gatsby cloud from the latest commit and I'll also get the plugin-fastify test site built and pushed to WPE Atlas platform running fastify on Monday so we can compare the two.

[ ] TODO: update docs to match refactor

@moonmeister didn't get much time to work on this today...thoughts on https://github.com/gatsby-uc/plugins/pull/392#issuecomment-1474398906 before I get back to it tomorrow?

Definitely seems like we're on the right track.

RE: Benchmarks - Glad we did some testing to validate this is the correct path forward.
RE: Table - Thanks for validating and giving feedback. I agree with your assessments. Only one question, re: Functions and Security headers, What does Gatsby Cloud do?
RE: Refactor - Thanks, I'll do a code review and provide feedback there

@moonmeister haven't forgot about this, just been having some other production issues and deadlines. Will hopefully get back to it later this week or next.

@tsdexter You're not waiting on me are you?

Nope, no worries. Just had to get some other stuff done in the meantime. See above.

Gatsby officially has an Adapter spec on the way 🥳 . It specifically notes the inclusion of headers in the spec to eliminate our very need to do this crazy implementation per adapter. 🥳
https://github.com/gatsbyjs/gatsby/discussions/38231

Gatsby officially has an Adapter spec on the way 🥳 . It specifically notes the inclusion of headers in the spec to eliminate our very need to do this crazy implementation per adapter. 🥳
gatsbyjs/gatsby#38231

And I had it on the schedule to finally get back to this after my vacation in July. I wonder how long until it's stable - looks great so far, been playing around with the demo repo.

Gatsby officially has an Adapter spec on the way 🥳 . It specifically notes the inclusion of headers in the spec to eliminate our very need to do this crazy implementation per adapter. 🥳
gatsbyjs/gatsby#38231

And I had it on the schedule to finally get back to this after my vacation in July. I wonder how long until it's stable - looks great so far, been playing around with the demo repo.

Yeah but also this is going to make life so easy. We can close every issue ever and it'll just work...hopefully.

With the announcement of the Gatsby Adapters spec, we'll be discontinuing any work on this plugin. Should the need for a performant node server still be needed, this work can continue under the official spec as gatsby-adapter-fastify.

@moonmeister this makes sense. Just one comment. I think there is still a bit of uncertainty around adapters, but to me it seems they are intended to be more platform specific.
I think there is probably still a path for something generic like gatsby-adapter-fastify or gatsby-adapter-express etc but from the initial post in the RFC I think it may be intended to be less generic and more specific to the platform.
For example, gatsby-adapter-atlas or gatsby-adapter-google-cloud either of which may still use fastify or express under the hood, but also takes care of the specific requirements for deployment on their respective platforms.
Thoughts?

@tsdexter I just opened #440 , let's have this discussion there...thanks for the feedback, I'll respond there.


feat: merge package credentials back from oras-credentials-go

Add package credentials (migrated from oras-credentials-go) and its corresponding description
Add new type auth.CredentialFunc
Migrate an example from oras.land
Change auth.DefaultCache to auth.NewCache() in examples

Resolves: #584

Codecov Report

Merging #589 (deb17e4) into main (92eb05c) will increase coverage by 0.16%.
Report is 1 commits behind head on main.
The diff coverage is 76.14%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #589      +/-   ##
==========================================
+ Coverage   74.21%   74.38%   +0.16%
==========================================
  Files          51       58       +7
  Lines        4832     5246     +414
==========================================
+ Hits         3586     3902     +316
- Misses        914      990      +76
- Partials      332      354      +22

Files Changed
Coverage Δ

registry/remote/credentials/trace/trace.go
0.00% <0.00%> (ø)

registry/remote/credentials/native_store_linux.go
80.00% <80.00%> (ø)

...istry/remote/credentials/internal/config/config.go
80.36% <80.36%> (ø)

registry/remote/credentials/store.go
80.95% <80.95%> (ø)

registry/remote/credentials/registry.go
81.25% <81.25%> (ø)

registry/remote/credentials/native_store.go
83.33% <83.33%> (ø)

registry/remote/auth/client.go
73.12% <100.00%> (ø)

registry/remote/credentials/file_store.go
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes


Support for transforming tables (changing column types, dropping columns, changing order)

This will be a significant UI design challenge, but unlocking the functionality added in https://github.com/simonw/sqlite-utils/issues/114 will be extremely useful.

It's starting to come together:


Add a util which was very useful today

This emerged out of testing a bunch of cameras on a Pi and wanting to be sure that:

they worked with our image capture behaviour, and
they could successfully capture multiple images with needing some form of manual reset shenanigans

@prophile I've put this in utils/ and tidied it up a bit, though I'm not sure if that's the best place for it. I see there's already the start of a CLI for the calibration stuff, though I'm not sure what you had in mind for a CLI of the main library.
I think it would be useful to have a CLI which supports the following, though I think most of it would be beyond the scope of this PR:

plain image capture,
image capture with April Tags detection (maybe with a visual overlay), and
just April Tags detection from a saved image

I kinda think this would be better placed in the script directory?