amplify push overwrites manual changes to userPoolGroups/template.js

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
No response
If applicable, what version of Node.js are you using?
No response
Amplify CLI Version
4.52.0
What operating system are you using?
Mac
Amplify Categories
auth
Amplify Commands
push
Describe the bug
I need to make some changes to my user pool groups template. More specifically I am adding another action to the AssumeRolePolicyDocument section under the section that defines one of my user pool groups.
   "orgAdminGroupRole": {
            "Type": "AWS::IAM::Role",
            "Properties": {
                "RoleName":{
                    "Fn::Join": [
                        "",
                        [
                          {
                            "Ref": "authcognitoUserPoolId"
                          },
                          "-orgAdminGroupRole"
                        ]
                    ]
                },
                "AssumeRolePolicyDocument": {
                    "Version": "2012-10-17",
                    "Statement": [
                        {
                            "Sid": "",
                            "Effect": "Allow",
                            "Principal": {
                                "Federated": "cognito-identity.amazonaws.com"
                            },
                            "Action": [
                                "sts:AssumeRoleWithWebIdentity",
                                "sts:TagSession"
                            ],
                            "Condition": {
                              "StringEquals": {"cognito-identity.amazonaws.com:aud": {"Ref":"authcognitoIdentityPoolId"}},
                              "ForAnyValue:StringLike": {"cognito-identity.amazonaws.com:amr": "authenticated"}
                            }
                        }
                    ]
                }
            }
        },

After amplify push, the Action section gets overwritten to
"Action": "sts:AssumeRoleWithWebIdentity"
Expected behavior
Amplify push should not override the file but should just deployed.
Reproduction steps
Make changes to the template
Run amplify push
GraphQL schema(s)

# Put schemas below this line

Log output

# Put your logs below this line

Additional information
No response

@krikork Please take a look at how to add custom policies to your group IAM roles out here with the "user-pool-group-precedence.json" file  in the last section  - https://docs.amplify.aws/cli/auth/groups#group-access-controls

Hey @kaustavghosh06, I am trying to add sts:TagSession in the IAM group role’s trust policy as shown here: https://docs.amazonaws.cn/en_us/cognito/latest/developerguide/using-attributes-for-access-control-policy-example.html
Do you have any example of how to do that in the "user-pool-group-precedence.json" file?
Also, can you clarify why amplify overwrites the local changes on amplify push? The behavior seems inconsistent with other resources, it also makes it far harder to achieve what the documentation link above achieves with 1 line.

@krikork Right now we do not support modifying or adding the sts:TagSession to the IAM roles, but we're currently working on a feature to make this sort of customizability possible in a generic way. cc @akshbhu

Similar issue here but with the user-pool-group-precedence.json file, i'm trying to add some custom policies to give access to geolocations resources to my default users group but the amplify push command overrides the file...

Closing in favor of customizing via overrides (mentioned in https://github.com/aws-amplify/amplify-cli/issues/7573#issuecomment-894435621)


CI for Node

Initializes with two players (4 ms)
Starts the game with a random player (2 ms)
Sets the current player to be whoever it is not (1 ms)


cmd/link: fingerprint mismatch: after adding default.pgo

What version of Go are you using (go version)?

$ go version go1.21rc3 linux/amd64

Does this issue reproduce with the latest release?
yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=''
GOARCH='amd64'
GOBIN=''
GOCACHE='/home/qiu-lai-dong-feng/.cache/go-build'
GOENV='/home/qiu-lai-dong-feng/.config/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='amd64'
GOHOSTOS='linux'
GOINSECURE=''
GOMODCACHE='/home/qiu-lai-dong-feng/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='linux'
GOPATH='/home/qiu-lai-dong-feng/go/'
GOPRIVATE=''
GOPROXY='https://goproxy.cn,https://proxy.golang.org,direct'
GOROOT='/home/qiu-lai-dong-feng/.go/current/'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/home/qiu-lai-dong-feng/.go/current/pkg/tool/linux_amd64'
GOVCS=''
GOVERSION='go1.21rc3'
GCCGO='/usr/bin/gccgo'
GOAMD64='v1'
AR='ar'
CC='clang'
CXX='g++'
CGO_ENABLED='1'
GOMOD='/home/qiu-lai-dong-feng/go/test/pkg/go.mod'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -Wl,--no-gc-sections -fmessage-length=0 -ffile-prefix-map=/tmp/go-build2593822558=/tmp/go-build -gno-record-gcc-switches'

What did you do?

git clone https://gitee.com/u-language/u-language.git test
cd test
cd pkg
su
go test ./...
What did you expect to see?
Normal operation test.
What did you see instead?
output

# gitee.com/u-language/u-language/pkg.test
/home/qiu-lai-dong-feng/.go/current/pkg/tool/linux_amd64/link: fingerprint mismatch: os has d62759c2b920f031, import from gitee.com/u-language/u-language/pkg expecting 18ac6ddfd6f53c08
?       gitee.com/u-language/u-language/pkg/astdata     [no test files]
FAIL    gitee.com/u-language/u-language/pkg [build failed]
ok      gitee.com/u-language/u-language/pkg/ast 0.037s
ok      gitee.com/u-language/u-language/pkg/cast        0.006s
?       gitee.com/u-language/u-language/pkg/config      [no test files]
ok      gitee.com/u-language/u-language/pkg/check       0.014s
?       gitee.com/u-language/u-language/pkg/enum        [no test files]
?       gitee.com/u-language/u-language/pkg/errcode     [no test files]
ok      gitee.com/u-language/u-language/pkg/data        0.029s
?       gitee.com/u-language/u-language/pkg/format      [no test files]
?       gitee.com/u-language/u-language/pkg/internal/debug      [no test files]
?       gitee.com/u-language/u-language/pkg/internal/errutil    [no test files]
ok      gitee.com/u-language/u-language/pkg/fasm/amd64  0.007s
?       gitee.com/u-language/u-language/pkg/internal/parser     [no test files]
?       gitee.com/u-language/u-language/pkg/internal/teco       [no test files]
?       gitee.com/u-language/u-language/pkg/internal/time       [no test files]
?       gitee.com/u-language/u-language/pkg/internal/utils      [no test files]
ok      gitee.com/u-language/u-language/pkg/internal/math/big   0.011s [no tests to run]
ok      gitee.com/u-language/u-language/pkg/ir  0.008s
?       gitee.com/u-language/u-language/pkg/parser      [no test files]
ok      gitee.com/u-language/u-language/pkg/lex 0.013s

I tried replicating this following the provided steps and it happens for me on tip as well. The mismatch happens for pkg.a -> _go.o -> main

Okay I was able to bisect this testing with -pgo=auto and found it started failing after this commit 74502e9bb4.

cc @cherrymui

cc @golang/compiler


[Config Request] Disable piles of Dirt/Sand/Gravel

Hey devs,
The new dirt piles seem like a good option for hardcore packs.  I'd appreciate if this could be disabled for packs that aren't looking to modify vanilla mechanics though.  Could a config be added?
If one already exists, my apologies.  I compared my old config file to the new and only saw a new line to disable enchanting*, so there doesn't appear to be an option currently.
*Side note: You might want to invert the language on this one.  All the "Hardcore X" features are defaulted to true, but this is written as "Disable X" and defaulted to false.  A bit inconsistent.

It was put in on the 1.11.2 branch, I'll just backport that real quick and see if there's anything else to do before uploading a new build.


Added .idea directory to base names skipped by Builder's Run() method

Fixes #56 by adding .idea directory to skipped directories for Builder.

Thanks!


Clone the frontend to nextjs

We need to clone the frontend of https://github.com/pbdsce/landing-page to nextjs. This would be the starting point after which further improvements to the UI will be made

@Akash-Singh04 Assign me


Add optimize option to Image Save

Adds a optimize image Boolean to the Image Save node. Fix for #265

Thank you.


Change parsedData from a mostly pass-by-value type to a pass-by-pointer

This more easily paves the way for using the metadata proto, which will be pass-by-pointer, in place of the parsedData type.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

51 of 58 (87.93%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 57.024%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tools/addrundata/list_json.go
0
1
0.0%

tools/addrundata/suite.go
4
6
66.67%

tools/addrundata/rundata.go
12
16
75.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.3%

Covered Lines:
1230

Relevant Lines:
2157

💛  - Coveralls


Make Namespace Behavior Public

Changes introduced with this PR
Export namespace building behavior so that it is available to other packages within the arcaflow engine.

By contributing to this repository, I agree to the contribution guidelines.

How is this intended to be used externally? Should this be added to an interface?


fix: set type Downshift.stateChangeTypes.clickItem on item click

What: Item click handler does not set type: Downshift.stateChangeTypes.clickItem. The bug was introduced from #265.

Why: It breaks stateReducer in case Downshift.stateChangeTypes.clickItem is being used

How: I added back the missing code.

Checklist:

[ ] Documentation
[ ] Tests
[x] Ready to be merged

[ ] Added myself to contributors table

Codecov Report

Merging #355 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #355   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           4      4
  Lines         330    330
  Branches       84     84
=====================================
  Hits          330    330

Impacted Files
Coverage Δ

src/downshift.js
100% <100%> (ø)
: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 b54d8f6...4548e85. Read the comment docs.


Can't change settings/base.py with new funfactory script

The new funfactory script uses embedded patch files to change the project name to the package you specify when running the script. A side effect of this is that making changes to the playdoh settings file requires updating the patch file as well, to avoid errors during the project setup.
Is there are different way to do this replacement that doesn't involve re-generating patch files whenever the settings change?

Uuuugly. Are you sure it throws errors (rather than "applying patch with fuzz")?
Maybe @kumar303 has some insights, but I think we might just want to regex-replace (with sed or Python, up to you) the relevant part of the file instead of applying a patch. That still has the potential to break, but at least it won't die just because you changed anything about the settings file.


plugin.Open("plugin/krakend-client-example"): plugin was built with a different version of package runtime

Describe what are you trying to do
I have a gateway with 1 sample plugin which mentioned in krakend documentation(https://www.krakend.io/docs/extending/http-client-plugins/). so, when I tried to run krakend using docker file. Docker container doesn't able to include plugin.
Your configuration file
The content of your krakend.json. When using the flexible configuration option, the computed file can be generated using FC_OUT=out.json
{ 	"version": 3, 	"name": "my-gateway", 	"port": 8080, 	"plugin": { 		"pattern": ".so", 		"folder": "./plugin/" 	}, 	"cache_ttl": "3600s", 	"timeout": "30s", 	"host": [], 	"extra_config": { 		"security/cors": { 			"allow_headers": [ 				"Origin", 				"Authorization", 				"Content-Type", 				"Content-Length", 				"Cookie" 			], 			"allow_methods": [ 				"POST", 				"GET" 			], 			"allow_origins": [], 			"expose_headers": [ 				"Content-Length" 			], 			"max_age": "12h" 		}, 		"telemetry/logging": { 			"level": "DEBUG", 			"prefix": "[KRAKEND]", 			"stdout": true, 			"syslog": false 		}, 		"telemetry/metrics": { 			"backend_disabled": false, 			"collection_time": "60s", 			"endpoint_disabled": false, 			"listen_address": "8090", 			"proxy_disabled": false, 			"router_disabled": false 		} 	}, 	"endpoints": [ 		{ 			"endpoint": "/test/{id}", 			"backend": [ 				{ 					"host": [ 						"http://localhost:8080" 					], 					"url_pattern": "/__debug/{id}", 					"extra_config": { 						"plugin/http-client": { 							"name": "krakend-client-example", 							"krakend-client-example": { 								"path": "/__debug/hijack-me" 							} 						} 					} 				} 			] 		} 	] }
go.mod
`module krakend-client-example
go 1.17
**Command Used** DockerFileFROM golang:1.17-alpine3.15 as goplugin
WORKDIR /app
COPY /plugins/krakend-client-example/go.mod .
COPY /plugins/krakend-client-example/*.go  plugin/
RUN apk add make gcc musl-dev
RUN go build -buildmode=plugin -o ./plugin/krakend-client-example.so ./plugin/main.go
FROM devopsfaith/krakend:2.0.5 as builder
COPY krakend.json .
COPY config .
RUN FC_ENABLE=1
FC_OUT=/tmp/krakend.json
FC_PARTIALS="/etc/krakend/partials"
FC_SETTINGS="/etc/krakend/settings"
FC_TEMPLATES="/etc/krakend/templates"
krakend check -d -t -c krakend.json
RUN krakend check -c /tmp/krakend.json
FROM devopsfaith/krakend:2.0.5
USER root
COPY --from=builder --chown=root /tmp/krakend.json /etc/krakend
COPY --from=goplugin /app/plugin  /etc/krakend/plugin/
WORKDIR /etc/krakend
CMD [ "run", "-dc", "/etc/krakend/krakend.json" ]
**Logs** Docker logs Parsing configuration file: /etc/krakend/krakend.json
[KRAKEND] 2022/09/19 - 10:09:24.966 ▶ DEBUG [SERVICE: Plugin Loader] Starting loading process
[KRAKEND] 2022/09/19 - 10:09:24.973 ▶ DEBUG [SERVICE: Executor Plugin] plugin #0 (plugin/krakend-client-example.so): plugin.Open("plugin/krakend-client-example"): plugin was built with a different version of package runtime
[KRAKEND] 2022/09/19 - 10:09:24.974 ▶ DEBUG [SERVICE: Handler Plugin] plugin #0 (plugin/krakend-client-example.so): plugin.Open("plugin/krakend-client-example.so"): plugin was built with a different version of package runtime (previous failure)
[KRAKEND] 2022/09/19 - 10:09:24.974 ▶ DEBUG [SERVICE: Modifier Plugin] plugin #0 (plugin/krakend-client-example.so): plugin.Open("plugin/krakend-client-example.so"): plugin was built with a different version of package runtime (previous failure)
[KRAKEND] 2022/09/19 - 10:09:24.974 ▶ DEBUG [SERVICE: Plugin Loader] Loading process completed
[KRAKEND] 2022/09/19 - 10:09:24.974 ▶ DEBUG [SERVICE: Stats] The endpoint /__stats is now available on 8090
[KRAKEND] 2022/09/19 - 10:09:24.974 ▶ DEBUG [SERVICE: Gin] Debug enabled
[KRAKEND] 2022/09/19 - 10:09:24.974 ▶ INFO Starting the KrakenD instance
[KRAKEND] 2022/09/19 - 10:09:24.975 ▶ ERROR [SERVICE: Stats] listen tcp: address 8090: missing port in address
`
Please let me know what I am doing wrong in this part.

Hi @deven1009,
I think the problem is that you are using the latest available golang version (golang:1.17-alpine3.15) which is different from the one used to build KrakenD.
You can check the required Go and Alpine versions here:
https://github.com/krakendio/krakend-ce/blob/v2.0.5/Makefile#L22
You can also use the KrakenD command check-plugin, that will tell you which versions do you need for that specific binary.
In the next version we are going to include a command (krakend version) that dumps the Golang and Glibc versions directly.

Thanks @taik0 for quick response.
It is working fine with golang:1.17.11-alpine3.15
Yeah it is very useful if we can check compatible version of Golang and Glibc directly from krakend version command.

Hi @taik0,
I have one more doubt in custom plugin is there any provision to include go plugin which is triggered on the endpoint level not on the backend. Like I want to implement Role based access for krakend endpoint. So that only authorized user access krakend API.  I have one external service API which check is user authorized for endpoint according to their email and scope parameter.
Can It be integrated with this external API with krakend API which trigger after JWT validation? if the response is true then its only allow to execute backend calls else return forbidden response.
Please refer this usec

ase:

Hi @deven1009
Let's move this conversation to our slack channel so the issue comments are only related to the issue title.
Thank you!


Fix broken "issue12394" test-case

This test-case is currently broken, with the reference image being completely empty, since it uses the old "annotationStorage" format in the manifest.

/botio makeref

/botio makeref


Feat(web): Introduce CheckboxField component (refs #DS-136)

[x] Remove unused scss
[x] Style input color/shape
[x] Message margin/padding

Ready to review.

We should discuss with designers if the component should be named just Checkbox (that's what they use now) or Checkbox Field (what you use and what I would prefer).

Well, discuss this naming according to the TextField component. I think we should be consistent in every control. So using TextField and just Checkbox makes no sense to me. Maybe we can use the suffix Control.

Please check the design in more detail, the construction is not quite the same yet.

Yes, I know. I have already discussed it with @crishpeen today. Sorry, my eyes are not so pixel-perfect and also I do not know some Figma-fu.


Make Ctrl+Shift+Enter work with query arguments as well

OS: MacOS Monterey
Platform: Desktop app
Version: 4.1.0

Hey! For it to work, you need to place the cursor between the braces and hit Ctrl+Shift+Enter before entering any character. I tested locally to verify. If you type "e" and then use the shortcut, it wouldn't do anything

@imolorhe im afraid not, it doesn't work on nested arguments

Oh it does not work for arguments. Only query fields

Closing this as I believe the question is answered.

@imolorhe no way its closed, is there a way to make it work?

There should be a way to make it work, but it does not at the moment. It only works for query fields. I can reopen the issue to track adding the ability for it to work in arguments but it wouldn't likely be done soon. But it can be kept open until then.

As @sooxt98 mentioned, it doesn't seem to work at all for nested fields. Any updates on this?
https://user-images.githubusercontent.com/15722561/184702543-457d9ea4-f426-44e4-a400-dbd475dc5891.mov
https://github.com/altair-graphql/altair/issues/605

@ryancraigmartin I think yours might be a different thing. Did you increase the addQueryDepthLimit value in your settings to a greater value?


[feature request] App/Window Focus Safety

Description of the new feature / enhancement
Focus safety locks the mouse and keyboard when focus is changed.
It stops windows/applications that pop-up and steal focus from accidentally being interacted with.
When a user is using a window/application, if another application steals focus or appears, the mouse and  keyboard are temporarily disabled to stop the user from accidentally interacting with the new window!

User is writing messages in vscode
A new window appears in the foreground, stealing focus
The keyboard and mouse are instantly disabled completely
A small message appears " has appeared"
User can wait 3 seconds or press a shortcut that deactivates focus guard

Scenario when this would be used?

User is typing in a text file using vscode
User is about to start a new line by pressing the ENTER key (hand raised, not physically pressed)
A new window suddnely opens from a different application

New window has the message "Are you stinky?"
There is a button in the center of the new window with the text "Yes"
if the user hits the ENTER key, the button will be activated

The new window steals focus and is in the foreground of windows now

User is unable to react fast enough and presses the enter key

The user did not even get a chance to react to the new window

User doesn't know what happened.

User is about to click on an icon on the desktop with the mouse

A pop-up about updating windows appears

The pop-up appears on foreground

User was unable to react fast enough

User clicks an option about updating windows, that they did not intend on clicking

Supporting information
This is a very common issue and happens a lot, sometimes even clicking on something, a new window is opened in foreground and is clicked accidnetally.
Currently (27Dec2023) powertoys offers always on top, however I do not beleive it guarantees safety from such situations.
A feature that stops pop-ups and windows coming to foreground from being
accidentally interacted with is necessary.

/dup #65


Do Not Use AIM Proxy In EurekaClient

Fixes #3174

Codecov Report

Merging #3240 into 1.4.x will increase coverage by 0.01%.
The diff coverage is 71.42%.

@@             Coverage Diff              @@
##              1.4.x    #3240      +/-   ##
============================================
+ Coverage     69.24%   69.26%   +0.01%
- Complexity     1934     1935       +1
============================================
  Files           240      240
  Lines          8561     8566       +5
  Branches       1074     1075       +1
============================================
+ Hits           5928     5933       +5
  Misses         2220     2220
  Partials        413      413

Impacted Files
Coverage Δ
Complexity Δ

.../netflix/eureka/EurekaClientAutoConfiguration.java
90.8% <71.42%> (-1.88%)
18 <0> (ø)

...ework/cloud/netflix/archaius/ArchaiusEndpoint.java
91.3% <0%> (+8.69%)
8% <0%> (+1%)
: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 01ac98d...fbcd878. Read the comment docs.

@brenuart if you have some time to try this that would be appreciated

@ryanjbaxter I don't have much time for the moment - maybe next week - sorry about that.
But from your commits, I can tell this is exactly the workaround we made in our codebase waiting for an official solution: we simply "unproxy" the ApplicationInfoManager instance before passing it to the CloudEurekaClient.
However, as described in #3174, the problem takes its roots far deeper in how the RefreshScope is destroyed (order not preserved).

@brenuart yes we realize it is not the optimal solution, but for now it gets us passed the exception


Any example of how one would apply multiple labels using the Fluent Extension?

Greetings -
Hoping someone has done this before. I'm trying to assign 2 labels to a node using fluent configuration. The following is an example that references a custom type called Person. This shows up as 1 label. I'd like to also apply the label of Person's base class but wasn't able to find any obvious way to do so. Any help is greatly appreciated.
Thanks very much
Rajiv
e.g.
FluentConfig.Config(ctx)                         .With<Person>()                         .Match(p => p.Id)                         .Merge(p => p.Id)                         .MergeOnMatchOrCreate(p => p.Id)                         .MergeOnMatchOrCreate(p => p.Name)

Is this from Neo4jClient.Extensions?

Yes indeed Chris. Specifically, FluentConfig is coming from the Neo4jClient.Extensions.Cypher namespace.
        {
            var ctx = new CypherExtensionContext();
            ctx.JsonContractResolver = new Newtonsoft.Json.Serialization.DefaultContractResolver();
            FluentConfig.Config(ctx)
                        .With<Person>()
                        .Match(p => p.Id)
                        .Merge(p => p.Id)

I think you're going to need to ask the guys who wrote it, Neo4jClient.Extension is an addon I'm afraid! https://github.com/simonpinn/Neo4jClient.Extension

Oh right - I keep forgetting that its Simon who wrote that. Thanks Chris!

No worries! I'll close this if that's ok with you?


chore(message): improve debug log for send and received msg with attr…

print all attributes and value to debug log in send and received msg
update message_type to the expected type and not the value
add self object to debug log
add _format_attribute func to format all the bytes/bytearray attributes.
add  get_key_from_value in MessageType

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 25.00000% with 9 lines in your changes missing coverage. Please review.

Project coverage is 40.09%. Comparing base (e0e6370) to head (adb1a6c).
Report is 3 commits behind head on main.

Files with missing lines
Patch %
Lines

midealocal/message.py
25.00%
9 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #294      +/-   ##
==========================================
+ Coverage   37.79%   40.09%   +2.30%
==========================================
  Files          84       84
  Lines        7477     7489      +12
==========================================
+ Hits         2826     3003     +177
+ Misses       4651     4486     -165

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


CSS

[x] Use flexbox?
[x] Specific names for classes used

#41 addresses both of these but the issue is not yet closed because there is a bug that does not allow narrowing the 3d-brain div. It'll flex-grow but never wants to flex-shrink.

Addressed by #61


New version: PortSwigger.BurpSuite.Professional version 2024.8.5

Created by 🥟 Dumplings in workflow run #8801.

Log
Updated: 2024.8.4 → 2024.8.5
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

[Policy] Reset Feedback


Stake system tokens while genesis import

To give imported accounts access to resources in Cyberway we need to convert GESTS to system tokens and stake them.

PR #520


minor tweaks to framework pages

The frameworks landing page should also link to the Vue resources page.
Any page that links to both the webpack plugin and esri-loader should probably link to this geonet article as well: https://community.esri.com/people/TWayson-esristaff/blog/2018/05/10/arcgiswebpack-plugin-vs-esri-loader
And we still need to do #124 for at least React, Vue, an Ember.

closing in favor of #194


Disable Config Vars Not Working

The disable config variables aren't working for me. Here's what I have in my config/general.php file:
return array(
	// ALL ENVIRONMENTS
	'*' => array(
		'sendPoweredByHeader' => false,
		'siteUrl' => $baseUrl,
		'disableDevmodeMinifying' => true
	),

	// LOCAL
	'local' => array(
		'env'	 => 'local',
		'devMode'  => true,
		'enableTemplateCaching' => false
	)
)

It also doesn't work if I try disableTemplateMinifying instead. I have my main layout template wrapped in {% minify %} tags.

They are actually environmentalVariables, so your config file should look like this:
return array(
    // ALL ENVIRONMENTS
    '*' => array(
        'sendPoweredByHeader' => false,
        'siteUrl' => $baseUrl,
        'environmentVariables' => array(
	        'disableDevmodeMinifying' => true
        ),
    ),

    // LOCAL
    'local' => array(
        'env'    => 'local',
        'devMode'  => true,
        'enableTemplateCaching' => false
    )
)

Okay, I updated to 1.0.4, which moves the settings directly into craft.config so your original setup should work now.

Excellent! Thanks.


Cache secrets in AWS secrets using Lambda extension

This extension demonstrates how to host an HTTP server which cache secrets required for the lambda function. The extension does the following:

Hosts a local HTTP server, which will be invoked by lambda function to retrieve the secret instead of directly going to AWS Secret manager
The extension using config.yaml file inside the Lambda to get the list of secrets that needs to get cached in memory
Secrets cache is refreshed based on the value (in minutes) set in the environment variable "CACHE_TIMEOUT". If no value is specified its defaulted to 10

It would be great to get this merged and I am happy to take any feedback

Hi,
Would love the pull request to be merged part of aws-sample, happy to incorporate any code review comments or feedback. Please let me know your thoughts.
Thanks,
Hari


[FR] "Submit and create next part" option on Create Part dialogue

Problem
Usually I add 10-20 new parts in a row. It would be great in those cases to go from completed and submitted "Create Part" dialogue to a fresh one, for the next part.
Suggested solution
Something like "Submit and create next" button on a bottom at the "Submit".
Just a small thing to speed up the workflow

Anything to save some clicks! I really like this idea.


🚧 Produce bundles with Rollup

fix #41

Codecov Report

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

@@          Coverage Diff          @@
##           master   #163   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          79     79
  Lines         315    315
=====================================
  Hits          315    315

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 92f08e6...fdac7ff. Read the comment docs.


Response from Glance's /v2/images may return null disk_format, container_format

...and Exosphere needs to handle this. JSON decoder should accommodate null values, and model should probably represent this as a maybe.

#94 fixes this.


Only send Content-Encoding when request isn't from Cloudflare Workers

Cloudflare will always inspect the Content-Encoding header and automatically decompress if its set to gzip. The Cloudflare Worker for kurl-sh is being updated to override the user-agent for terminal requests so we can conditionally not set this header and prevent automatic decompression when sending terminal requests through Cloudflare.
https://github.com/replicatedhq/cloudflare-workers/pull/129

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


CSS conflict when loading mulitple svg files?

I have some Adobe illustrator svg files. When I tried to load them, the colors are all messed up. Then I found out this is due to the style written in each svg files conflicts with each other.

No matter what method I try, loading multiple svg files will be messed up.

If you put them together without Snap do they still mess up ? Maybe it would be an idea to put it on a jsfiddle

@ibrierley Nope, just with snap

I would put it on a fiddle then as its hard for anyone to look at it otherwise :).

@ibrierley I can't use fiddle because I'm not supposed to expose the assets to the web.
However I'm able to pin where the problem is.
Because the svgs are generated by illustrator, so the class name used to describe svg style are generated and written in a <style>...</style> at every beginning of the file.  When snapsvg load those files, it got really confused about those <style>  and colors( or fill to be exact) are messed up.
So, Is this an issue with Snap.svg? I can not use fiddle to show the messed up effect or give a screen shot, but if needed, I can post the code I use to load files

Its unlikely to be Snap, but not impossible.
What I would do in your situation is recreate it in a jsfiddle still, you don't need the exact assets, all you need is any 2 SVGs with a couple of lines in there, not the whole lot, just a circle or rect in each. Add a couple of similar style entries and 2 Snap.load commands. This will also help you isolate the most minimal code needed.


CI: Upgrade clang-format 10 -> 11

What's the reason for this switch? Presumably this collides with existing formatting in this repo, no? :thinking:

Isn't the formatting defined in .clang-format?
I would expect upgrading clang-format to not lead to changes?
I initially set this up trying to fix the build, but I was able to fix it with upgrading the used GitHub action, that was sufficient. I left this open planning to (maybe/probably) upgrade the clang-format version expecting no difference in the actual formatting verification, only an upgrade in tooling.

Unfortunately yes, different clang-format versions can lead to a different output due to bug fixes and overall changes.

Yeah, clang-format only produces equal outputs for equal versions. That's a big shame, but that's how it is.

Shall I discard this then?
The action has versions up to 18, which I would have tried.
If the pr checks do not report any issue then the upgrade should be fine even without code changes!?
At least the current PR state upgrading from 10 to 11 doesn't report and changes!?
In the log I don't even see a success message. So I assume it only reports issues? https://github.com/mumble-voip/mumble-json-bridge/actions/runs/8253456912/job/22575337407?pr=10

ok, v 18 fails :D
I guess I'll close instead of finding max version, nobody's using this project anyway

nobody's using this project anyway

I thought I was in mumblekit…


Broken link to Josephine in PROJECTS.md

The link to Josephine listed under Commercial Products Live in PROJECTS.md is broken.

fixed
thanks!

Glad to help!


Cannot get GitHub social link in footer to show

I'm not sure what I've done wrong, I've added my GitHub username to my _config.yml, with github_username:  "johnramsden" but it just won't show. I've tried it locally as well as remotely and this is what I end up with.
I've done exactly as I've done on other sites where it's worked, so the only thing I can think of is that I've done something strange in the configuration.

Looks like you're using the theme, but then you have imported the files  of entire repo (or gem), as well.  And the copy of the repo (or gem) that you imported is from before #190 was merged.  So you have the social_sites key prefixing everything in social.html
https://github.com/johnramsden/flatpakrepo/blob/6331f84/docs/_includes/social.html

@ashawley Brilliant, thanks! I deleted social.html and everything's working. I needed to modify some the files which is why I imported the _includes. I guess I just need to delete all files I'm not trying to override.


feat(assists): raw string <-> usual string manipulation

Fixes #1730

uh, I don't know what I've done with git Oo


Introduces "entityClass" for Services.

This PR introduces  the "entityClass" property for all Services.
An entity in a service can now be an entity of the correct subclass if necessary. To achiev this the service hast to work with "E" instead of the concrete class the service is made for. See ImageFileService in this PR.
We allready have this in controllers and daos.

One minor note. Otherwise this looks good to me. Thanks and please merge.


Wrong hint paths for referenced assemblies

I created a project using ProjectScaffolding and added a NuGet reference to it using
mono .paket/paket.exe add nuget JetBrains.ReSharper.SDK project SDKTest.
When I try to build, there's a lot of errors like
2) FS0078: FSC(0,0): Unable to find the file '..\..\packages\AsyncBridge.JetBrains\lib\net40-Client\AsyncBridge.dll' in any of /usr/local/Cellar/mono/<IP_ADDRESS>/lib/mono/4.0-api /Users/Eugene/Developer/ReSharperSDKTestFs/src/SDKTest /Users/Eugene/Developer/ReSharperSDKTestFs/packages/FSharp.Core/lib/net40
3) FS0078: FSC(0,0): Unable to find the file '..\..\packages\AsyncIO\lib\net40\AsyncIO.dll' in any of /usr/local/Cellar/mono/<IP_ADDRESS>/lib/mono/4.0-api /Users/Eugene/Developer/ReSharperSDKTestFs/src/SDKTest /Users/Eugene/Developer/ReSharperSDKTestFs/packages/FSharp.Core/lib/net40
4) FS0078: FSC(0,0): Unable to find the file '..\..\packages\DotNetZip.Reduced\lib\net20\Ionic.Zip.Reduced.dll' in any of /usr/local/Cellar/mono/<IP_ADDRESS>/lib/mono/4.0-api /Users/Eugene/Developer/ReSharperSDKTestFs/src/SDKTest /Users/Eugene/Developer/ReSharperSDKTestFs/packages/FSharp.Core/lib/net40
5) FS0078: FSC(0,0): Unable to find the file '..\..\packages\FSharp.Core\lib\net40\FSharp.Core.dll' in any of /usr/local/Cellar/mono/<IP_ADDRESS>/lib/mono/4.0-api /Users/Eugene/Developer/ReSharperSDKTestFs/src/SDKTest /Users/Eugene/Developer/ReSharperSDKTestFs/packages/FSharp.Core/lib/net40

Can you please put a repro sample in a zip or something and put it here? Thx
Does the csproj look correct?

Sure, here it is. ReSharperSDKTestFs.zip I removed packages directory from there. fsproj looks correct apart from assemblies hints paths.

If you run paket update -f - would that fix it for you?
Am 29.10.2016 11:05 vorm. schrieb "Eugene Auduchinok" <
<EMAIL_ADDRESS>
Sure, here it is. ReSharperSDKTestFs.zip
https://github.com/fsprojects/Paket/files/559852/ReSharperSDKTestFs.zip
I removed packages directory from there. fsproj looks correct apart from
assemblies hints paths.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/1990#issuecomment-257080514,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNKkpOhu6v7WS7KzeVdaYxxMpS594ks5q4ww8gaJpZM4KkD0C
.

Nope, it doesn't fix the issue.

When using VS 2015, NuGet installs everything fine and adds correct hints.
There's just another issue with build in F# 4.0 which is fixed in upcoming F# 4.1.

@auduchinok can't see any issue with the zip. what's the concrete problem?

Path hints in the last zip were added using NuGet in VS and are correct. There're wrong paths in first attached zip.

ok can you please show me exactly which path is incorrect and where?


Fix crash from impossible Object::cast_to

A Vector<>-variant can't be used in an Object::cast_to, because Vector doesn't inherit from Object and this cast always returns a nullptr.
resolve #68377
followup to #65325
This patch replaces the Object::cast_to and accesses the contained Vector directly.
Here is one example:
https://github.com/godotengine/godot/blob/60e25835a5d19dee9382052549d7f7a6523a6b97/scene/resources/animation.cpp#L5922-L5925
Here arr_a->size() crashes, because arr_a  == nullptr.

Thanks!


In step 5. renamedColumnsDF.write ...

should be renamedColumnsDf.write as it's case sensitive and throws an error

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

ID: d863ab39-0b6a-eb81-9dde-5eb25b46cc33
Version Independent ID: 655abadf-5f9e-a65b-0109-c5166f956c4d
Content: Tutorial: Perform ETL operations using Azure Databricks
Content Source: articles/azure-databricks/databricks-extract-load-sql-data-warehouse.md
Service: azure-databricks
GitHub Login: @nitinme
Microsoft Alias: nitinme

@creationsoft Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.


get-def consumes all the CPU on Windows

Feature or bug?

Bug.

Description
Every time I try to jump to a definition, multiple node processes spawn and freeze my PC:

I also reported this here https://github.com/steelbrain/flow-ide/issues/103.
Is something I can do to give you more information? Like a debug report?
Tech details

OS: Windows 10.
IDE: Atom 1.19.0-beta4.
Atom plugin: flow-ide: https://github.com/steelbrain/flow-ide/

Thank you for your work!

The flow binary will show up in windows as flow.exe. This is not a flow issue, it seems as though it is an issue with the plugin you're using.

@billschaller - What's the proper way to discover the location of flow.exe? It sits in a version specific directory and there's no shortcut created by the npm installer.
Invoking ./bin/flow.cmd doesn't execute flow.exe, it calls node.exe with arguments leading to a much slower execution and the problem @lgraziani2712 is facing.


Validate test labels using the test runner

Description:
$title.
We need to

validate that the labels are correct, and have no typos
identify extra labels we have used but haven't yet added in the label list

Fixed with https://github.com/ballerina-platform/ballerina-lang/pull/35308


Publish magpiego as a multi-arch image

Area for Improvement
Functional testing
Observed behavior
magpiego is an image we use for functional testing. Right now this is only published as an amd64 image.
For users on MacOS running the functional tests locally will fail with exec format error unless they have Rosetta installed.
Desired behavior
Functional testing just works.
Proposed Fix
Publish magpiego as a multi-architecture image.
rad Version
aaaa
Operating system
MacOS + M1 chip
Additional context
Found while debugging with @superbeeny
Would you like to support us?

[ ] Yes, I would like to support you

AB#12849

just change 2 docker file definitions

Some proof of what's missing:

(supports multiple architectures) https://github.com/radius-project/radius/pkgs/container/ucpd
(single architecture) https://github.com/radius-project/radius/pkgs/container/magpiego

Good

Bad
(notice the lack of architectures tab)

I don't think this is fixed yet. Check the link to the magpiego image.
PR #7745 updated the dockerfiles, but I think we need makefile changes too.


The Vim-Ti acc on Imagente-1k is 76.1%

Why you repost 73.1???? Where do you get this result???

@FanqingM ,
Please remain composed. During the development of our paper, the accuracy of the model reported in both the GitHub repository and the paper was 73.1%. Subsequent to our implementation, the authors made significant modifications to the model architecture and training strategy, as documented in this commit: https://github.com/hustvl/Vim/commit/4567f9d64e7dd40e5c914be0fa9d59c15199fea7. These changes improved the model's accuracy to 76.1%.

Our implementation, however, was based on the earlier version (v1 as published on arXiv (link)), and did not incorporate the advanced modifications found in v2. Thus, we reported the accuracy as 73.1%, which reflects the state of the model at that time.
Below are screenshots from the historical README and the v1 arXiv paper that corroborate the accuracy figures:

(origin)

(origin)
To clarify the version of the model compared in our study, we have added a note to our README (https://github.com/hunto/LocalMamba/commit/b0ed2433fcc1daf0da541c9a418e5b9e346e2826). Furthermore, we plan to update our model to the newest version, v2, in the near future.
Thank you for your understanding.

It seems that your paper is released at arxiv in [Submitted on 14 Mar 2024]
But vim v2 is released at Feb 10.
There is a month gap within it....  And the number of papers with mamba is increased quickly.
I think you must fix it in arxiv version

See the latest commit.

😂😂


As an author, I want to be able to make posts that are unlisted, that are publicly shareable by URI alone (or for embedding images)

By publicly shareable, does it mean anyone with the link can access it, regardless of the visibility option selected?

It makes more sense for it to require the same permission levels
It's like when you share a Google doc with a link but limit it to "only the company" (i.e. intuit or UofA) so that just anyone can't get the link and see the thing you've shared


bump consent-management-platform to v13.7.1 for babel traverse fix

What does this change?
Bumps the version of consent-management-platform to 13.7.1.
https://github.com/guardian/consent-management-platform/releases/tag/v13.7.1
Why?
Includes updates to the vendor.ts file (update Permutive, add Magnite) and fix babel/traverse vulnerability.

Is it fair to say that this should have been a major change?

the @babel/traverse change should not have caused a version bump, as it’s a development dependency
the update of peer dependencies to a narrower set should have been a major bump


Cannot import Messages from '@cucumber/messages';

👓 What did you see?
When trying to import Messages from '@cucumber/messages';
I got the error
file:///Users/t/Desktop/projects/CodeceptJS/lib/interfaces/gherkin.js:3
import Messages from '@cucumber/messages';
       ^^^^^^^^
SyntaxError: The requested module '@cucumber/messages' does not provide an export named 'default'
    at ModuleJob._instantiate (node:internal/modules/esm/module_job:124:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:190:5)

📦 Which tool/library version are you using?
"@cucumber/messages": "24.0.1"

The error message looks accurate, why do think this a problem?

Ah, I'm not to deep into JS but the messages module doesn't have a default export. So you either have to import the individual classes or the whole namespace.
So I believe that  one of these should work:
import * as messages from "@cucumber/messages"

import { Background, Feature, GherkinDocument, Rule, Scenario, Step } from '@cucumber/messages'

They both show up in other cucumber projects and match with the MDN docs. https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/import


Redesign landing page

Draft is ready!

If you want to build it yourself, see the readme.

Footer text half the width of the page.
Version above attributions


Log file view is word wrapped making it very hard to look through log files.

I love the new authentic theme except for one big issue, log file view has wordwrap which makes them almost un-readable.  I modified the css to fix this, but though i would share/suggest the change to the css
CSS FILE:
/usr/libexec/webmin/theme/authentic-theme/unauthenticated/css/default.min.css
after deminifying & beautifying the file at about line 2335 you have
pre {
font-family: monospace;
font-size: 12px;
margin: 0;
white-space: pre-wrap;
border: 0
}
comment out the white-space: pre-wrap;
pre {
font-family: monospace;
font-size: 12px;
margin: 0;
/* white-space: pre-wrap; */
border: 0
}
and the output for viewing log files is back to readable.

Yes, thank you! I have already fixed it in my dev built, as it was reported already, few days ago. It will be alright in version 10.30, which is released in few days.


Modify vcl::utils::Pos by implementing cast conversion to cv::Point_

Will help implementing graphical items classes.

completed.


Created new section for computational geometry

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Added a new section for algorithms related to computational geometry.

Should I create a separate section - Computational Geometry?

yes you can
Originally posted by @shraddhavp in https://github.com/TesseractCoding/NeoAlgo/issues/1492#issuecomment-745934371
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#1492

Thanks!


Add Boost on the include and link path.

It was only done when the auto-build was triggered, resulting in build errors
when Boost is available on the system but not on the standard paths.

Thanks!


What is the computational cost of training?

How many GPUs (V100 or A100) are required?

For the first stage training it requires 32 A100 GPUs. For the second instructional tuning, a single A100 is sufficient.

We provide both first and second stage checkpoints for convience.

For the first stage training it requires 32 A100 GPUs. For the second instructional tuning, a single A100 is sufficient.

A100 40GB? or  80GB?

40G is sufficient. But 80Gb is better. See #35

40G is sufficient. But 80Gb is better. See #35

In the paper, batch size = 256 for instruction tuning, in this setup, how many GPU is required?

You can use the gradient accumulation technique to enlarge your batch size without increasing the number of GPUs.


Library not downloadable

Arduino IDE 2.2.1
Adafruit GFX 1.11.9
BusIO 1.15
When installing pre-requisites first, this library fails to install on the Arduino IDE.
Tried 2 windows machines with the same result.
Tried on MacOs with IDE 1.8.19, still and error on download due to dependencies however you can force the download. Compilation of FPS example gives errors.
Platform IO does appear to complile.
While this might be an Arduino IDE problem i'd still like to validate against a known setup - can you confirm what IDE and library versions you have proven this to work with?

Hi laytll, did you ever figure this out?

I did, it was a while ago - but from memory it was important the order of including dependencies. I used PlatformIO
Platformio.ini
lib_deps =
adafruit/Adafruit<EMAIL_ADDRESS>adafruit/Adafruit GFX<EMAIL_ADDRESS><EMAIL_ADDRESS>In sketch
#include <Arduino.h>
#include <Wire.h>
#include <JDI_MIP_Display.h>

Thanks for answering! Good to know about the order of import.
I ended up skipping the library download step and just downloaded the repo straight to “Arduino/Libraries” and that did the trick. I think the repo is more up to date than the uploaded library.


Revert "[Windows] Update OpenSSL version to 3.x (#6837)"

This reverts commit 9753e7301e19e29b89b0622b811bbb9b3891d02e.
Description
Openssl-3.x breaks some azdo tasks which use openssl directly so we have to revert it.
Related issue:
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run windows2019,windows2022


[Source Request] NovelOwl

Source name
NovelOwl
Source link
https://novelowl.com/
Language
English
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a title with source name.
[X] I have checked that the extension does not already exist by searching the GitHub repository and verified it does not appear in the code base.
[X] I will fill out all of the requested information in this form.

NovelOwl already exists in the app.


Remove internal listeners from user listeners

@krmahadevan Feel free to update the branch. I think the test you added in the other branch should be added at least.

@krmahadevan Feel free to update the branch. I think the test you added in the other branch should be added at least.

I went with ensuring that this changeset does not break any tests. Now I will rebase my branch off of master and see what I should be removing


Images stay blurred on first load

I added gatsby-plugin-image to the plugins in gatsby.config.js.
Looks like this fixed the bug.


Keplr Mobile connection issues on Android

Trying to build on top of a template created with create-cosmos-app and having connection issues with the latest version v0.13.3.
Deploying the vanilla template and trying to connect with Keplr Mobile on an Android phone, the wallet connection modal reads "Oops!  This does not happen on iOS and did not use to happen on Android with version v0.13.0.

QR code or deep links?
FYI Wallet connect v1 will be sunset across all wallets — this means keplr, cosmostation, etc etc... so the best thing I can say is that, we're rushing (starting next sprint) to work on Wallet Connect v2.
I think a lot of issues will go away with the latest wallet connect version.

Deep links it is.


[Webview][QML] Native settings menu

This feature adds a scrolling bug. Bug that is not present on the jolla-email with this patch

Asked on the next community-meeting. Hopefully, we'll get an answer.

I agree. I enable a check to disable the pulley menu when the user is in the settings menu. I'll add another check.
You have added some pages for the pulley menu, on which branch are they?

You have added some pages for the pulley menu, on which branch are they?

Oh, already merged. Nice!
I need to find out a way to only disable Hydrogen settings. I may have an idea.

Alternatively, we coikd have the pulkey menu only on the settings page and if we detect a "modal" dkalog, to present a Back option, just in case

Closing this one in favor of https://github.com/hydrogen-sailfishos/sfos-hydrogen/issues/42


x11docker dependencies

I an making a gentoo ebuild for x11docker. For all optional features, gentoo have a policy of USE flags. With them, the users can choose if they want (USE="myflag") or not (USE="-myflag") to install the myflag support. Which imply it will be a lot of them into that ebuild, and they will be used to trigger the installation of the needed/wanted dependencies. I also want to be able to make my own docker images.
When I read

This recommended base allows advanced usage of x11docker and an improved X security setup. Some of them are probably already installed on your system.
Desired commands: nxagent xinit xauth xclip xhost xrandr xdpyinfo catatonit jq

does that mean that I will need all of these commands? And does that mean I need them in the host?
And with

Recommended additional base for --gpu
Desired commands: weston Xwayland xdotool xpra

Does that mean that I need both weston and Xwayland with wayland, or both of xdotool and xpra with xorg? And does I need them only in the images, or also in the host?

Making an image, I begin to understand how the whole thing is working. So  we can close that bug.

Great that you found out yourself. :-)
Is your gentoo ebuild intended to be for public use / in some official repository?
If you have further questions, please ask. Unfortunately I am only rarely at this project nowadays.

Yes, I will publish it into an overlay. When done I will also make a bug report on gentoo asking for a new package.
I have 2 public docker images on the way, one based on Debian 10 for the dkbuilder (toolkit for generation and building of guitarix and lv2 plugins from electronic schematics), the other one for x11docker based on Debian unstable and running fvwm-crystal from git.

Yes, I will publish it into an overlay. When done I will also make a bug report on gentoo asking for a new package.

Great, thank you!
You might find these hints useful:
https://github.com/mviereck/x11docker/wiki/Packaging-x11docker

I  saw than on gentoo, the X tools dependencies need x11-apps/setxkbmap too.

I have done a gentoo bug for a new package: https://bugs.gentoo.org/941144

I published it here: https://github.com/domichel/GenCool
It's not an official gentoo overlay, but it is referenced here, which is an official gentoo overlay: https://github.com/domichel/proaudio-gentoo


--quiet flag prints info logs

Problem
I'm programmatically parsing the output of cargo locate-project --quiet but finding that 'info' logs are being printed to stderr when I would expect them to not be shown.
Steps

rustup toolchain uninstall stable-aarch64-apple-darwin
cargo locate-project --quiet

Possible Solution(s)
Redirecting stderr is a workaround I'm using:
cargo locate-project 2>/dev/null

But ideally the --quiet flag would do as it describes and not display 'info' logs.
Notes
No response
Version
cargo 1.67.0 (8ecd4f20a 2023-01-10)
release: 1.67.0
commit-hash: 8ecd4f20a9efb626975ac18a016d480dc7183d9b
commit-date: 2023-01-10
host: aarch64-apple-darwin
libgit2: 1.5.0 (sys:0.16.0 vendored)
libcurl: 7.84.0 (sys:0.4.59+curl-7.86.0 system ssl:(SecureTransport) LibreSSL/3.3.6)
os: Mac OS 13.0.1 [64-bit]

What kind of 'info' log did you see? It would be great if you can provide some log output.
Also, could you check the value of the environment variable CARGO_LOG? Setting CARGO_LOG  will turn on debug logging regardless of --quiet being set.

Hi @weihanglo
So the CARGO_LOG environment variable is not set, and the output I'm seeing is as follows...
$ cargo locate-project --quiet
info: syncing channel updates for 'stable-aarch64-apple-darwin'
info: latest update on 2023-01-26, rust version 1.67.0 (fc594f156 2023-01-24)
info: downloading component 'cargo'
info: downloading component 'clippy'
info: downloading component 'rust-docs'
info: downloading component 'rust-std'
info: downloading component 'rustc'
info: downloading component 'rustfmt'
info: installing component 'cargo'
info: installing component 'clippy'
info: installing component 'rust-docs'
info: installing component 'rust-std'
info: installing component 'rustc'
info: installing component 'rustfmt'
{"root":"/Users/integralist/Code/test-projects/testing-fastly-cli/Cargo.toml"}

NOTE: I even tried --offline, thinking that it wouldn't attempt to sync and download but that didn't work either (albeit that flag has a different purpose than --quiet but I was just trying things out).

If you install Rust via rustup, cargo command is very likely a proxy provided by rustup. I believe the implicit download behavior is from rustup. See https://github.com/rust-lang/rustup/issues/1397. People interested in this issue could discuss over there.
Thank you for the report anyway!


TIMESTAMP value anomaly

In the test of TestBinlogReplicationForAllTypes, the case failed because TIMESTAMP value unmatched:

Maybe this is due to the unmatch of time zone between primary MySQL(In UTC) and the replica MyDuck Server(In +08:00).

I have never observed this failure. How was it produced? Are your running MyDuck Server in non-default TimeZone setting?

I have never observed this failure. How was it produced? Are you running MyDuck Server in non-default TimeZone setting?

No. The MyDuck Server is launched by the test case and I think maybe it use the local time zone on my Mac which is Asia/Shanghai, aka +08:00. But the source data is from the MySQL instance which is lauched in a docker container and the time zone is UTC, aka +00:00. So the time value on MyDuck Server is 8 hours ahead comparing to the one in MySQL.


Madscientist314

Ok, I updated the BCM svg logos, which should now be in the proper format.  Sorry for the confusion.

Thanks @MADscientist314, but I already did this for you in https://github.com/nf-core/nf-co.re/pull/310
Seems to be working well on the site I think: https://nf-co.re/community

Also, these SVGs are still embedding images 😄 You can see it in the code: xlink:href="data:image/png;base64,
This tells me that the SVG has a base64 encoded PNG image embedded in the file.
If you're going from a PNG or other raster image, it's impossible to create a true vector SVG file. You have to start with a vector image. This is what I did in the above PR - I did a bit of googling and found a vector SVG file on a website. I then edited this to make a white version.
Let me know if you have any questions or concerns!
Phil

@ewels looks good to me, that for closing the pull request.  If you're happy I'm happy!


Added separate actions for tests & code coverage, PMD and checkstyle.

Description
Closes #5.
Retained the end to end builds and added separate actions for tests & code coverage, PMD and checkstyle.
This is done so that it is easier to figure out the check that caused the failure in the end to end build as well
as to collect metrics on the checks after some time.
Author's Checklist

[x] These changes don't break existing dependants. If they do, and you suggest to proceed,
please explain why.

Reviewer's Checklist

[ ] All classes have documentation comments.
[ ] All methods have documentation comments.
[ ] No parameters are hardcoded. Should be loaded as properties instead. If there needs to be hardcoded properties, explain why.
[ ] All information to be logged/displayed to the user are internationalized. If not, explain why.
[ ] README.md has been updated if needed.
[ ] pom.xml version is set to the latest date If not, explain why.

Assignee's Checklist

[ ] All GitHub action checks have passed.
[ ] All reviewers have approved.
[ ] Relevant documentation has been updated if needed.
[ ] pom.xml version is set to the latest date If not, explain why.

I've retained the original end to end build as in the past problems have occurred when running different maven goals one after the other. And running the end to end builds help catch that as well.


[NT-2110] Update User Account Mutation

📲 What
As part of the switch from our GQL DSL to Apollo, we have to update some mutations that pertain to the user's account information.

changing a password
creating a password
changing an email

🤔 Why
In order to remove dependency on our GQL DSL, which is the current ongoing body of work on iOS.
🛠 How
Pretty straight forward, nothing in the view models changed. The only changes were made behind the ServiceType protocol.

added a UpdateUserAccount.graphql mutation file, which added the relevant mutation in the GraphAPI.swift
instead of returning GraphMutationEmptyResponseEnvelope and GraphError from the 3 calls:

changeEmail
changePassword
createPassword
created a UpdateAccountEnvelope and used the existing ErrorEnvelope to capture the return values from the network. This even further reduces our dependency on the GraphMutation files that were specifically used in conjunction with the DSL. Also it seems like better practice to just handle the native Envelope types in the view models, instead of ever having to deal with a Graph object there. Small change, but adds cleanliness.

created adapter for changing over the Mutation.Data into a UpdateAccountEnvelope.
created adapter for changing over the GraphMutationInput into GraphAPI.Input used by Apollo.

👀 See
No diff for the three operations above. Change password and change email can be found in the account section.
Note: Difficult to test the creation of the password even on production with a valid facebook account. Will keep working on this, but I don't think it's a blocker because these requests are all pretty much carbon copies of their previous implementation, just with a different type of input. And 2/3 are successfully passing and failing as expected. Also the tests on all the view models are passing.
✅ Acceptance criteria

[x] Change a password should succeed with valid current password.
[x] Change a password should fail if the current password is incorrect.
[x] Change an email should succeed with valid current password.
[x] Change an email should fail if the current password is incorrect.
[x] Create a password should succeed with valid password parameters (unable to test, due to facebook account issues on prod and staging)
[x] Create a password should fail with invalid password parameters (unable to test, due to facebook account issues on prod and staging)

SwiftFormat found issues:

File
Rules

KsApi/MockService.swift:547:1
warning: (blankLinesAtStartOfScope) Remove leading blank line at the start of a scope.

KsApi/MockService.swift:575:1
warning: (blankLinesAtStartOfScope) Remove leading blank line at the start of a scope.

KsApi/MockService.swift:592:1
warning: (blankLinesAtStartOfScope) Remove leading blank line at the start of a scope.

KsApi/models/graphql/adapters/GraphAPI.UpdateUserAccountInput+UpdateUserAccountInput.swift:6:1
warning: (indent) Indent code in accordance with the scope level.

KsApi/models/graphql/adapters/GraphAPI.UpdateUserAccountInput+UpdateUserAccountInput.swift:16:1
warning: (indent) Indent code in accordance with the scope level.

KsApi/models/graphql/adapters/GraphAPI.UpdateUserAccountInput+UpdateUserAccountInput.swift:27:1
warning: (indent) Indent code in accordance with the scope level.

KsApi/models/graphql/adapters/GraphAPI.UpdateUserAccountInput+UpdateUserAccountInputTests.swift:6:1
warning: (wrapArguments) Align wrapped function arguments or collection elements.

KsApi/models/graphql/adapters/GraphAPI.UpdateUserAccountInput+UpdateUserAccountInputTests.swift:7:1
warning: (wrapArguments) Align wrapped function arguments or collection elements.

KsApi/models/graphql/adapters/GraphAPI.UpdateUserAccountInput+UpdateUserAccountInputTests.swift:14:1
warning: (indent) Indent code in accordance with the scope level.

KsApi/models/graphql/adapters/GraphAPI.UpdateUserAccountInput+UpdateUserAccountInputTests.swift:16:1
warning: (wrapArguments) Align wrapped function arguments or collection elements.

KsApi/models/graphql/adapters/GraphAPI.UpdateUserAccountInput+UpdateUserAccountInputTests.swift:17:1
warning: (wrapArguments) Align wrapped function arguments or collection elements.

KsApi/models/graphql/adapters/GraphAPI.UpdateUserAccountInput+UpdateUserAccountInputTests.swift:18:1
warning: (wrapArguments) Align wrapped function arguments or collection elements.

KsApi/models/graphql/adapters/GraphAPI.UpdateUserAccountInput+UpdateUserAccountInputTests.swift:26:1
warning: (indent) Indent code in accordance with the scope level.

KsApi/models/graphql/adapters/GraphAPI.UpdateUserAccountInput+UpdateUserAccountInputTests.swift:28:1
warning: (wrapArguments) Align wrapped function arguments or collection elements.

KsApi/models/graphql/adapters/GraphAPI.UpdateUserAccountInput+UpdateUserAccountInputTests.swift:29:1
warning: (wrapArguments) Align wrapped function arguments or collection elements.

KsApi/models/graphql/adapters/UpdateAccountEnvelope+UpdateAccountEnvelope.DataTests.swift:11:1
warning: (indent) Indent code in accordance with the scope level.

KsApi/models/graphql/adapters/UpdateAccountEnvelope+UpdateAccountEnvelope.DataTests.swift:13:1
warning: (indent) Indent code in accordance with the scope level.

KsApi/models/graphql/adapters/UpdateAccountEnvelope+UpdateAccountEnvelope.DataTests.swift:15:1
warning: (indent) Indent code in accordance with the scope level.

KsApi/models/graphql/adapters/UpdateAccountEnvelope+UpdateAccountEnvelope.DataTests.swift:19:1
warning: (indent) Indent code in accordance with the scope level.

KsApi/models/graphql/adapters/UpdateAccountEnvelope+UpdateAccountEnvelope.DataTests.swift:24:1
warning: (indent) Indent code in accordance with the scope level.

KsApi/models/graphql/adapters/UpdateAccountEnvelope+UpdateAccountEnvelope.DataTests.swift:26:1
warning: (indent) Indent code in accordance with the scope level.

KsApi/models/graphql/adapters/UpdateAccountEnvelope+UpdateAccountEnvelope.DataTests.swift:28:1
warning: (indent) Indent code in accordance with the scope level.

Library/ViewModels/ChangeEmailViewModelTests.swift:88:1
warning: (wrapArguments) Align wrapped function arguments or collection elements.

Library/ViewModels/ChangeEmailViewModelTests.swift:89:1
warning: (wrapArguments) Align wrapped function arguments or collection elements.

Library/ViewModels/ChangeEmailViewModelTests.swift:90:1
warning: (wrapArguments) Align wrapped function arguments or collection elements.

Library/ViewModels/ChangeEmailViewModelTests.swift:91:1
warning: (wrapArguments) Align wrapped function arguments or collection elements.

Library/ViewModels/ChangeEmailViewModelTests.swift:315:1
warning: (indent) Indent code in accordance with the scope level.

Library/ViewModels/ChangeEmailViewModelTests.swift:317:1
warning: (indent) Indent code in accordance with the scope level.

Library/ViewModels/ChangeEmailViewModelTests.swift:321:1
warning: (indent) Indent code in accordance with the scope level.

Library/ViewModels/ChangeEmailViewModelTests.swift:323:1
warning: (indent) Indent code in accordance with the scope level.

Library/ViewModels/ChangeEmailViewModelTests.swift:325:1
warning: (indent) Indent code in accordance with the scope level.

Library/ViewModels/ChangePasswordViewModelTests.swift:306:1
warning: (wrap) Wrap lines that exceed the specified maximum width.

Library/ViewModels/ChangePasswordViewModelTests.swift:307:1
warning: (wrapArguments) Align wrapped function arguments or collection elements.

Library/ViewModels/ChangePasswordViewModelTests.swift:308:1
warning: (wrapArguments) Align wrapped function arguments or collection elements.

Library/ViewModels/ChangePasswordViewModelTests.swift:309:1
warning: (wrapArguments) Align wrapped function arguments or collection elements.

Library/ViewModels/CreatePasswordViewModelTests.swift:10:1
warning: (wrap) Wrap lines that exceed the specified maximum width.

Library/ViewModels/CreatePasswordViewModelTests.swift:11:1
warning: (wrapArguments) Align wrapped function arguments or collection elements.

Library/ViewModels/CreatePasswordViewModelTests.swift:12:1
warning: (wrapArguments) Align wrapped function arguments or collection elements.

Library/ViewModels/CreatePasswordViewModelTests.swift:13:1
warning: (wrapArguments) Align wrapped function arguments or collection elements.

Library/ViewModels/CreatePasswordViewModelTests.swift:14:1
warning: (wrap) Wrap lines that exceed the specified maximum width.

Library/ViewModels/CreatePasswordViewModelTests.swift:15:1
warning: (indent) Indent code in accordance with the scope level.

Library/ViewModels/CreatePasswordViewModelTests.swift:56:1
warning: (redundantSelf) Insert/remove explicit self where applicable.

Library/ViewModels/CreatePasswordViewModelTests.swift:80:1
warning: (redundantSelf) Insert/remove explicit self where applicable.

  Generated by :no_entry_sign: Danger

Strange that the coverage dropped, I just ran the formatting and pushed, if it's still 0% I'll have to do some digging.
I suspect it might be a one-off, the coverage report shows inaccurate results rarely.

          1 Warning

:warning:
Big PR

  Generated by :no_entry_sign: Danger

Spent some time investigating this.
It's not immediately clear to me where the 3.14% drop is coming from.
According to the report given, all the file deltas show at something like 0.23% drop in coverage.
And the same report overview graph on this PR, doesn't show any drops in coverage on the dashboard.
Finally, the hits and misses here are for 290+ files, when I've only changed about 30 in this PR.
It's worth noting that codecov does say this feature is being improved at the moment.
Merging for now. Comparing this branch (which includes the parent) against main shows a tiny increase in coverage on codecov.


Make marshal.MustFoo not go via defer-recover

The panic-recover thing makes the stack trace harder to read.

2640880 LGTM


Add ignore block to match expessions

Fixes #48

Codecov Report

Merging #81 (c30ab1e) into main (41d2178) will increase coverage by 0.04%.
The diff coverage is 76.19%.

@@            Coverage Diff             @@
##             main      #81      +/-   ##
==========================================
+ Coverage   75.48%   75.53%   +0.04%
==========================================
  Files          59       59
  Lines        3423     3442      +19
==========================================
+ Hits         2584     2600      +16
- Misses        731      733       +2
- Partials      108      109       +1

Impacted Files
Coverage Δ

internal/config/config.go
44.93% <40.00%> (+1.43%)
:arrow_up:

internal/config/rule.go
58.60% <81.08%> (+1.60%)
: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 41d2178...c30ab1e. Read the comment docs.


Zcoin <IP_ADDRESS> make check fails on ubuntu

make check fails on Ubuntu 16.04.5 LTS.
===========================================
   Zcoin Core 0.13.7: src/test-suite.log
===========================================

# TOTAL: 1
# PASS:  0
# SKIP:  0
# XFAIL: 0
# FAIL:  1
# XPASS: 0
# ERROR: 0

.. contents:: :depth: 2

FAIL: test/test_bitcoin
=======================

Running 396 test cases...
Balance before 0
Balance after 200 blocks:<PHONE_NUMBER>00
Testing denomination 1
Testing denomination 10
Testing denomination 25
Testing denomination 50
Testing denomination 100
Balance before 0
Balance after 200 blocks:<PHONE_NUMBER>00
Testing denominations 1 and 10
Testing denominations 10 and 25
Testing denominations 25 and 50
Testing denominations 50 and 100
Testing denominations 1 and 1
Testing denominations 10 and 10
Testing denominations 25 and 25
Testing denominations 50 and 50
Testing denominations 100 and 100
Balance before 0
Balance after 200 blocks:<PHONE_NUMBER>00
Testing denominations 50 and 25
Balance before 0
Balance after 200 blocks:<PHONE_NUMBER>00
Testing number of inputs for denomination 1Balance before 0
Balance after 109 blocks:<PHONE_NUMBER>0
Testing denomination 1
Creating 5 mints at height 110
Creating 6 mints at height 111
Creating 10 mints and one spend at height 117
Creating 20 blocks with 1 spend each starting at height 118
Creating 19 mints at height 138
Creating 19 blocks with 1 spend each starting at height 144
Testing denomination 10
Creating 5 mints at height 163
Creating 6 mints at height 164
Creating 10 mints and one spend at height 170
Creating 20 blocks with 1 spend each starting at height 171
Creating 19 mints at height 191
Creating 19 blocks with 1 spend each starting at height 197
Balance before 0
Balance after 200 blocks:<PHONE_NUMBER>00
Testing denomination 1
Testing denomination 10
Testing denomination 25
Testing denomination 50
Testing denomination 100
Balance before 0
Balance after 150 blocks:<PHONE_NUMBER>00
Balance before 0
Balance after 150 blocks:<PHONE_NUMBER>00
unknown location(0): fatal error in "mtp_transition": memory access violation at address: 0x00000000: no mapping at fault address
test/mtp_trans_tests.cpp(123): last checkpoint

*** 1 failure detected in test suite "Bitcoin Test Suite"
test_bitcoin: /usr/include/boost/thread/pthread/condition_variable_fwd.hpp:81: boost::condition_variable::~condition_variable(): Assertion `!ret' failed.
FAIL test/test_bitcoin (exit status: 134)

Can't repro. @soerenmartius how much memory is on your Ubuntu?

@psolstice 4gb

that's not enough to run this test. If it's a VM please try to reserve at least 6-8GB. This test involves MTP mining and has high memory requirements


feat(docs): search input fixed on top of the page

Resolves #360
https://github.com/user-attachments/assets/e4a84df8-3fe9-4401-9f5f-4272c3decfd6

@stanlee974 Could you please review it whenever you are free and let me know if there are any changes req!!

Hi @vr-varad !
Sorry for the delay. It's good for me. Thank you again for your help 💯


member: zjhmale

@zjhmale Can you help to optimize starcoin-ide together?

@zjhmale Are you still going?

@zjhmale I read your PRs, but none of them have been merged. You need to merge 3 PRs to graduate.

sure, kinda busy these days, will continue on the PRs shortly!

@zjhmale Can you help to optimize starcoin-ide together?
I am working on this: starcoinorg/starcoin-ide#11 move-language/move#218

sure, will keep an eye on this, great work!


WIP: blog: generic ephemeral volumes + storage capacity tracking

These are new alpha features coming in Kubernetes 1.19.
The introduction of these two features is meant to be a summary of similar text from the Kubernetes "concepts" documentation, with less details to keep the blog post short.
The example is new content that is more suitable for the blog post rather than some official documentation.

Until Kubernetes 1.19 gets released, a lot of the links are not valid yet. The same applies to external-provisioner. Work that has to be completed before the blog post can go live:

[ ] merge https://github.com/kubernetes-csi/external-provisioner/pull/450
[ ] release external-provisioner 2.0.0 images or at least some pre-release images
[ ] use those images in the PMEM-CSI example

I used a locally compiled external-provisioner in the example, but that's too complicated for readers and would imply adding more instructions.

/cc @mkhaas

/hold until scheduled the feature blogs for the week after k8s 1.19 release

Until Kubernetes 1.19 gets released, a lot of the links are not valid yet. The same applies to external-provisioner. Work that has to be completed before the blog post can go live:
* [ ]  merge [kubernetes-csi/external-provisioner#450](https://github.com/kubernetes-csi/external-provisioner/pull/450)

* [ ]  release external-provisioner 2.0.0 images or at least some pre-release images

* [ ]  use those images in the PMEM-CSI example

I used a locally compiled external-provisioner in the example, but that's too complicated for readers and would imply adding more instructions.

Seems you can finish this off, too :)

Careful, non-native English speaker opinion incoming :)

Thanks, that all looked reasonable to me (also non-native English speaker) except for one suggestion that I fixed differently. New version pushed.
The example is also ready, so this PR is no longer WIP.

thank you @pohly it looks good!
@sftim @jimangel @mrbobbytables anything to add? :)
Would be happy to get it soon ready to publish it at 01.09. at around 5pm CET

/lgtm
/approve
/hold cancel


feat: migrate crvUSDN and crvUSDP to v2

Found address 0x881b06da56BB5675c54E4Ed311c21E54C5025298 in files app/containers/Vaults/customOrder.json. This address is from vault yLINK
Found address 0x1B5eb1173D2Bf770e50F10410C9a96F7a8eB6e75 in files app/containers/Vaults/customOrder.json, app/containers/Vaults/migrationWhitelist.json. This address is from vault yvusdp3CRV
Found address 0xFe39Ce91437C76178665D64d7a2694B0f6f17fE3 in files app/containers/Vaults/customOrder.json, app/containers/Vaults/migrationWhitelist.json. This address is from vault yvusdn3CRV
Found address 0x7047F90229a057C13BF847C0744D646CFb6c9E1A in files app/containers/Vaults/customOrder.json. This address is from vault yvCurve-renBTC
Found address 0xC4dAf3b5e2A9e93861c3FBDd25f1e943B8D87417 in files app/containers/Vaults/customOrder.json, app/containers/Vaults/migrationWhitelist.json. This address is from vault yvCurve-USDP
Found address 0x3B96d491f067912D18563d56858Ba7d6EC67a6fa in files app/containers/Vaults/customOrder.json, app/containers/Vaults/migrationWhitelist.json. This address is from vault yvCurve-USDN
Found address 0x7Eb40E450b9655f4B3cC4259BCC731c63ff55ae6 in files app/containers/Vaults/migrationWhitelist.json. This address is from token usdp3CRV
Found address 0x4f3E8F405CF5aFC05D68142F3783bDfE13811522 in files app/containers/Vaults/migrationWhitelist.json. This address is from token usdn3CRV

I updated the custom order to remove duplicates from v1, and complete revamped ordering of the stablecoin vaults in v2


Investigate exceptions thrown under options page

I was working on options page and noticed two exceptions are thrown. I think they were introduced with library extractor.

will. look into it when possible

Fixed it. Basically the error was happening in attachIfPossible() for extractors using third-party libs which aren't imported in the options.html. Because there's nothing to attach on options page, I added an if statement to only run attachIfPossible() outside options.html. Not the fanciest solution, but better that importing unnecessary libs and running code without any need.


non-latin urls

I encountered this bug while using the latest sprint code from Flask, but I assume it's Werkzeug-level.
If I register a URL that has non-latin characters (particularly, cyrillic), it doesn't work. The request seems to invariably go for ÑÐµÑÑ, whatever that is (not cyrillic), and therefore result in a 404 or something of the sort.
I've been staring at what I think might be relevant pieces of code, but I can't nail it down to where the issue might be.
In Python 2, if there is a route of the form /someurl/<name> where 'name' is cyrillic, it fetches the correct page. In Python 3, it gives me a 404.
I've also checked and found out that registering non-latin routes at top level (/некий-урл) crashes the Python 2 instance of Werkzeug completely, whereas the Python 3 instance doesn't register non-latin view functions at all (but doesn't crash).

Hi all!
I've an issue similar to this one, with non-latin URLs, which I have posted on Odoo's Help Forum.
Can you give me a hint on why non-latin URLs are failing on the werkzeug level?
FIX?
As far as I could debug it, it seems that wsgi_decoding_dance() method should look similar to wsgi_encoding_dance() method and should have this check at the beginning of it:
        if isinstance(s, unicode):
            return s


Any method to destroy the webview? It seem to has memory leak problem if there is a list of mapview in screen


Node webkit app error not valid win32 application

Hello,
I have created a sample application using this module. When I run it as node app it lists all usb. Now when I wrap it with Node-webkit same application is crashing, As per the issue comments I have complied and build the usb module using following command:
node-pre-gyp configure --runtime=node-webkit --target=0.12.3
node-pre-gyp build --runtime=node-webkit --target=0.12.3
But I am getting following error:
"%1 is not a valid Win32 application.
↵D:\Projects\…er\node_modules\usb\src\binding\usb_bindings.node"}

maybe you did not rebuild the source.the files you down recently are soure code,only can bu used after build

changuish: Thanks that worked!!!

How do you rebuild the source?

Hi, Actually it was long back. Probably its nw-gyp rebuild --target=0.5.0 command.


PYTHONFRAMEWORK mismatch

Hi,
I upgraded today from d7e201c3308c51eb1766701cbe615f38e4762274 to 56409629cf1fec75c7cf1a6209076594d3aa2290 (master ATM).
When building kitty using the newly built python, I faced this exception.
After digging, I realized that the value of PYTHONFRAMEWORK is different between the two commits.
Empty for the former and "" for the latter.
The command to reproduce is simply:
python -c 'import sysconfig;print(sysconfig.get_config_var("PYTHONFRAMEWORK"))'

Thank you :)
(and truly thank you for this project, cpyton's autoconf is a nightmare.)

Another issue, probably related (sysconfig still), using the latter commit, I get this warning from a recent pip:
  WARNING: Value for scheme.platlib does not match. Please report this to <https://github.com/pypa/pip/issues/10151>
  distutils: /home/build/.local/lib/python3.6/site-packages
  sysconfig: /home/build/.local/lib/python/site-packages
  WARNING: Value for scheme.purelib does not match. Please report this to <https://github.com/pypa/pip/issues/10151>
  distutils: /home/build/.local/lib/python3.6/site-packages
  sysconfig: /home/build/.local/lib/python/site-packages
  WARNING: Additional context:
  user = True
  home = None
  root = None
  prefix = None

Reproducible bug: before & after.

Thanks so much for the detailed report :pray:
This explains the following warning:
/home/build/Python-3.7.12/Programs/python.c:15:12: warning: implicit declaration of function '_Py_UnixMain' [-Wimplicit-function-declaration]
   15 |     return _Py_UnixMain(argc, argv);
      |

Another issue is that Python fails to compile when LDFLAGS contains -Wl,-z,defs as you can see in this CI. Tons of undefined reference.

And finally, the bug that took me LITERALLY a day to properly reproduce it in a CI, even if I’m yet able to identify why. This bug emerged when I tried to compile gobject-introspection.
When using a combination of ThinLTO and -march=x86-64-v3 -fstack-protector-strong. I get:
samu: job terminated due to signal 11: /home/build/build/tools/g-ir-scanner --output=gir/GLib-2.0.gir [...]

My hypothesis is that the linker segfaults when handling -lpython3.6m.
This issue emerged between d7e201c3308c51eb1766701cbe615f38e4762274 and 56409629cf1fec75c7cf1a6209076594d3aa2290 too.
Here are the logs: before it worked, but not after.
The only difference between the two builds is the commit hash of python-cmake-buildsystem.


Added badge to README.md

@chentsulin Sorry it took me so long to get around to this. I finally added tests, so I'm merging this now. Thanks!


[chip-tool] Replace 'ReportAttribute' by 'ReportEvent' when using rea…

…d-event otherwise it does returns attributes
Problem
I have made a typo when landing wildcards. read-event is calling ReportAttribute instead of ReadEvent...

fast track: trivial delta, typo correction. It makes sense that class ReadEvent sends ReportEvent instead of ReportCommand


Collinearity Analysis among #func,#unit,#proptery_based,#fuzz_target

Add collinearity analysis part of 4 predictors in our regression model

don't commit the .png files, please remove

OK
Yifeng He @.***> 于2024年2月21日周三 19:23写道：

@.**** requested changes on this pull request.
In Analysis/collinearity_analysis.py
https://github.com/EYH0602/TestEval/pull/3#discussion_r1498565604:

@@ -0,0 +1,54 @@
+import pandas as pd

also, please just put this file in src/
—
Reply to this email directly, view it on GitHub
https://github.com/EYH0602/TestEval/pull/3#pullrequestreview-1894787484,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANOIUJ7N3QICKKBJYP4DSG3YU223LAVCNFSM6AAAAABDUBGKTKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQOJUG44DONBYGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

OK
On Thu, Feb 22, 2024, 9:21 PM Yifeng He @.***> wrote:

@.**** requested changes on this pull request.
In src/collinearity_analysis.py
https://github.com/EYH0602/TestEval/pull/3#discussion_r1500220327:

@@ -0,0 +1,63 @@
+"""
+This script make collinearity analysis based on 4 predictors for our linear regression
+"""

+import pandas as pd
+from scipy.stats import pearsonr
+import matplotlib.pyplot as plt
+import seaborn as sns
+import fire
+
+def collinearity_analysis(dataset_path:str,barchart_name:str,heatmap_name:str):
please format using black formatter
https://marketplace.visualstudio.com/items?itemName=ms-python.black-formatter.
You can do this by installing this vscode extension and right click then
choosing "format document".
—
Reply to this email directly, view it on GitHub
https://github.com/EYH0602/TestEval/pull/3#pullrequestreview-1897390546,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANOIUJZUKAD7TL2K4ND7IG3YVARMZAVCNFSM6AAAAABDUBGKTKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQOJXGM4TANJUGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

OK I agree!
On Thu, Feb 22, 2024, 9:24 PM Yifeng He @.***> wrote:

@.**** requested changes on this pull request.
In src/collinearity_analysis.py
https://github.com/EYH0602/TestEval/pull/3#discussion_r1500221753:

sns.set_theme(style="whitegrid")

plt.figure(figsize=(10, 8))
sns.heatmap(correlation_matrix, annot=True, cmap='coolwarm', linewidths=.5)
plt.savefig(f"./src/{heatmap_name}.png",dpi=500)
plt.show()

+def main():

dataset_path = './data/static_analysis_data/dataset.csv'
barchart_name = "correlation_coefficient"
heatmap_name = "collinearity_heatmap"
collinearity_analysis(dataset_path,barchart_name,heatmap_name)

+if name == "main":

fire.Fire(main)

fire.Fire is for automatically generating command line interfaces (CLIs)
from absolutely any Python function/object.
Suggest change:
def collinearity_analysis(
dataset_path: str = "./data/static_analysis_data/dataset.csv",
barchart_name: str = "correlation_coefficient",
heatmap_name: str = "collinearity_heatmap",
):
...
if name == "main":
fire.Fire(collinearity_analysis)
—
Reply to this email directly, view it on GitHub
https://github.com/EYH0602/TestEval/pull/3#pullrequestreview-1897392572,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANOIUJ7OFYSY45F3B44UFRTYVARYRAVCNFSM6AAAAABDUBGKTKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQOJXGM4TENJXGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Ok, I will do it.
On Fri, Feb 23, 2024, 1:42 PM Yifeng He @.***> wrote:

@.**** requested changes on this pull request.
In src/collinearity_analysis.py
https://github.com/EYH0602/TestEval/pull/3#discussion_r1501193356:

   'size': 16,

   }

plt.xlabel('Predictors',fontdict=font)
plt.ylabel('Pearson Correlation Coefficient',fontdict=font)
for i, corr in enumerate(correlations):

   plt.text(i, corr if corr > 0 else corr - 0.014, f'{corr:.4f}', ha = 'center', va = 'bottom')

plt.savefig(f"./src/{barchart_name}.png",dpi=500)
plt.show()

columns_of_interest = ['#funcs', '#unit', '#proptery_based', '#fuzz_target']
correlation_matrix = dataset[columns_of_interest].corr()

sns.set_theme(style="whitegrid")
plt.figure(figsize=(10, 8))
sns.heatmap(correlation_matrix, annot=True, cmap='coolwarm', linewidths=.5)
plt.savefig(f"./src/{heatmap_name}.png",dpi=500)

don't save images to src/ which stands for "source code", I suggest
making output directory also a cli input
In src/collinearity_analysis.py
https://github.com/EYH0602/TestEval/pull/3#discussion_r1501200756:

   'size': 16,

   }

plt.xlabel('Predictors',fontdict=font)
plt.ylabel('Pearson Correlation Coefficient',fontdict=font)
for i, corr in enumerate(correlations):

   plt.text(i, corr if corr > 0 else corr - 0.014, f'{corr:.4f}', ha = 'center', va = 'bottom')

plt.savefig(f"./src/{barchart_name}.png",dpi=500)
plt.show()

columns_of_interest = ['#funcs', '#unit', '#proptery_based', '#fuzz_target']
correlation_matrix = dataset[columns_of_interest].corr()

sns.set_theme(style="whitegrid")
plt.figure(figsize=(10, 8))
sns.heatmap(correlation_matrix, annot=True, cmap='coolwarm', linewidths=.5)
plt.savefig(f"./src/{heatmap_name}.png",dpi=500)

save figures to PDF format and reduce the white surrounding white space,
use format="pdf", bbox_inches="tight"
—
Reply to this email directly, view it on GitHub
https://github.com/EYH0602/TestEval/pull/3#pullrequestreview-1898979161,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANOIUJZIFFXA3AKSANTXLXDYVEELHAVCNFSM6AAAAABDUBGKTKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQOJYHE3TSMJWGE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Port to v1 config format

The config file format has changed, and the old "v1beta" format is not supported. This changes them to the new format. Tested on Google Container Engine on 2015/10/17.

Thank you!

:smile:


Indent size not preserved

I have this .editorconfig file in top-level directory:
root = true

[*]
charset = utf-8
end_of_line = lf
insert_final_newline = true
indent_style = space
indent_size = 4

[*.{json,yml,html,js}]
indent_size = 2

[{Makefile,*.java,*.sh}]
indent_style = tab

[*.{c,cpp,h,hpp}]
indent_style = tab
indent_size = 2

Now all files in subdirecotries that fall into last rule have indent_size of 4 instead of 2, yet indent_style is properly set to tab. There are no .editorconfig files in these subdirectories and Preferences->Editor->Tab Type is set to "auto".
 :)
Since your problem affects subdirectories i want to suggesr trying to address
[**.{c,cpp,h,hpp}]

Since the single asterisk matches" any string of characters, except path separators (/)", this should be the reason why your subdirectories aren't affected. However, the editorconfig-parsing is done by https://github.com/editorconfig/editorconfig-core-js, so if you identified a bug it will likely belong into that repository.
If this issue isn't resolved for you please come back, it then would be helpful to have a full file-tree of the given project (tree -a .). 🍻

Thanks, I am almost sure it worked until last week or so. ;)

@zgoda -- you are welcome.. 😄  You would save me some time if you verify it.

Thanks. One thing seems at least strange that indent_style for eg. *.cpp files is set properly yet indent_size is not. The parser did not change in last months, your plugin code for weeks and the only thing that changed in my instance is Atom update from 1.11.0 to 1.11.1 - changelog shows this update was related to intendation bug and this may be some hint.

Hmm.. interesting.. I don't see that this change is affecting the work of atom-editorconfig, since we're resetting the tabType based on the configuration, which then should be recognized properly.
It takes me a bit of time to reread the whole implementation of the tabs to see if there were any other changes introduced.
I would like to ask you to set the tab-type of the last setting to spaces and take a look if the rule gets applied  in general (or if the tab-setting comes from elsewhere).

I ran Atom in safe mode for a moment to see if I perhaps messed whitespace in my files. After that right intendation was back with the same .editorconfig I had. I guess this issue was somehow related to Atom update (perhaps something was reset in configuration?) and the setting for open file was preserved. Now that I reopened all files right settings parsed from .editorconfig have been set.
I guess it is safe to close this issue, thank you for your support.

@zgoda - thank you for clarification. I am happy it's no bug on our side! 😅
Thanks for getting in touch! 💝


Use Hamcrest assertions instead of JUnit in examples/integrations/neo4j/neo4j-mp - backport 2.x (#1749)

Part of #1749
Original PR

/trigger


Leading not taken into account

When an emoji is rendered, the leading for the current text box isn't subtracted from the y coordinate for the image so it ends up rendering above the baseline for a line of text with leading set.
I looked into trying to fix this, but I wasn't able to find a way to access the current leading.
Example:
@document.text '…', leading: 8

@xtian It seems to me that the leading is properly applied.

However it seems that ends up rendering a little above the baseline even if the leading is set 0. So i fix  like the below:
def draw_emoji(text, text_options)
  left_text, emoji_unicode, remaining_text =<EMAIL_ADDRESS>
  return text if emoji_unicode.empty?

  emoji_image = Emoji::Image.new(emoji_unicode)

  emoji_x, emoji_y = text_options[:at]
  emoji_x += @document.width_of(left_text, text_options)
- emoji_y += @document.font_size
+ emoji_y += @document.font_size - 1

  draw_emoji_image emoji_image, at: [emoji_x, emoji_y], width: @document.font_size
  ...
 end

@xtian Sorry, I've changed my mind. I won't fix, because it is the same position in comparison with the emoji in the original text.

@hidakatsuya Ok, I understand. Thank you for taking the time to investigate.


Remove unreachable status

What's changed?
bugfix: Remove unreachable status
Checklist

[ ]  I have read the Contributing Guide
[ ]  I have written the necessary doc or comment.
[ ]  I have added the necessary unit tests and all cases have passed.

@all-contributors please add @YutingNie for code


gsc_egg.lua gives erroneous values for some species

After running the gsc_egg.lua script rather successfully for various pokemon, I ran into a rather odd issue. Some species of pokemon, namely Drowzee, Hypno, and Skarmory amongst a few others more than likely but these are the three I ran into. These pokemon when using the script, it returns back the same DV values regardless, though the specific DV values differ between the bad species. I did check the pokemon, and they do have proper DV values, but the script never reported them. Due to this, it can never properly find a shiny egg for these bad species. Any help would be appreciated, thank you!

I'm years late to this so I don't know if you can check or remember your Drowzee/Hypno/Skarmory, but I suspect there are two possible reasons that happened:

The "parents" couldn't actually breed with each other. I was trying to breed my shiny Typhlosion/Quilava/Cyndaquil with a shiny Ditto and had the same issue where the DVs never changed... but I completely forgot about Gen 2's anti-inbreeding check and how it affects shinies (or even just two random Pokemon that coincidentally have similar DVs). So I genned up a non-shiny Typhlosion for testing, and I had slightly more success.

Typhlosion's cry is so long that it takes up the entire time between savestate loads, and I think this might also prevent a proper check since the Drowzee line also has long cries. I tried loading the script partway through the cry, which kind of works and gets a brief period where the DVs change, but the script gradually desyncs and eventually saves and loads after the daycare worker's dialogue is finished. I devolved the test mon into Quilava and Cyndaquil and the script worked perfectly, though. I was also able to get Typhlosion to work by putting it in first and Ditto second - and just to be thorough I tested it again with the shiny Typhlosion and once again the DVs wouldn't change.

To summarize my findings, in order of first + second = result
Typhlosion + Shiny Ditto = Changing DVs
Shiny Ditto + Typhlosion = Frozen DVs (with imperfect workaround)
Shiny Ditto + Quilava/Cyndaquil = Changing DVs
Shiny Typhlosion + Shiny Ditto = Frozen DVs
If anyone else is having the same issue, try switching the order in which the parents are deposited, or go talk to them in the yard to see if they're incompatible ("no interest" or "brimming with energy").


Added Soln For Edit Distance

Hi @TecHAyusH6476 I have added the solution for edit distance problem. I request you to pls merge my PR.

@adit2005 Done

Hi @TecHAyusH6476 Should i add soln again bcuz its showing this repo i excluded from hacktober fest.

I think u added the hacktoberfest label tag later maybe bcuz of that it didn,t got accepted.