regexp: compiling incorrectly

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

$ go version
go version go1.20.2 darwin/amd64

Does this issue reproduce with the latest release?
yes
and with 1.19 on the Go Playground
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/andrew/Library/Caches/go-build"
GOENV="/Users/andrew/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/andrew/Go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/andrew/Go:/Users/andrew/Dropbox/suneido_tests"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.20.2"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/andrew/Dropbox/gsuneido/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 -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/dc/0y3gfqnn56g6pd8d3_b5wz300000gn/T/go-build1311023722=/tmp/go-build -gno-record-gcc-switches -fno-common"
GOROOT/bin/go version: go version go1.20.2 darwin/amd64
GOROOT/bin/go tool compile -V: compile version go1.20.2
uname -v: Darwin Kernel Version 22.3.0: Mon Jan 30 20:42:11 PST 2023; root:xnu-8792.81.3~2/RELEASE_X86_64
ProductName:		macOS
ProductVersion:		13.2.1
BuildVersion:		22D68
lldb --version: lldb-14<IP_ADDRESS>
Apple Swift version 5.7.2 (swiftlang-<IP_ADDRESS>.5 clang-14<IP_ADDRESS>)

What did you do?
pat := regexp.MustCompile(`0A|0[aA]`)
fmt.Println(pat.MatchString("0a"))

(https://go.dev/play/p/jCLPp8AXnJB)
What did you expect to see?
true
What did you see instead?
false
I tested on regex101 to confirm that this should match.
(Strangely, it shows golang matching)
It appears to compile incorrectly. Simplify doesn't seem to matter.
syntax.Parse("0A|0[aA]", 0)
fmt.Println(syntax.Compile(re))
  0	fail
  1*	rune1 "0" -> 2
  2	rune1 "A" -> 3
  3	nop -> 4
  4	match

syntax.Parse("0a|0[aA]", 0)
fmt.Println(syntax.Compile(re))
  0	fail
  1*	rune1 "0" -> 2
  2	rune "AAaa" -> 3
  3	match

cc @rsc

The regexp 0A|0[aA] means 0(A|0)[aA].  You seem to be looking for (0A)|(0[aA]).

From https://github.com/google/re2/wiki/Syntax , it says

The operator precedence, from weakest to strongest binding, is first alternation, then concatenation, and finally the repetition operators... Some examples: ab|cd is equivalent to (ab)|(cd)

That makes me think the OP's regexp is (0A)|(0[aA]), not 0(A|0)[aA].

The regexp 0A|0[aA] means 0(A|0)[aA]. You seem to be looking for (0A)|(0[aA]).

No, re2 flavor regex don't bind that way, see https://github.com/google/re2/wiki/Syntax

Some examples: ab|cd is equivalent to (ab)|(cd);

https://go.dev/play/p/EQ6nkEEHJCT

For re `aB|a[cd]` parsed as `a[Bc-d]`
For re `0A|0[aA]` parsed as `0A(?:)`
For re `0a|0[aA]` parsed as `0[Aa]`

Notice the second one parsed as the string 0A followed by an empty non-capturing group, which is just wrong.  It should be similar to the third item where the alternation collapsed and the second character is casefolded a (either [aA] or (?i:a)).
An prefix followed by a capital letter with a character class with capital and lower case seems broken, larger prefixes are still broken (see the play link above)

I think that syntax.Regexp Equal() is broken between literals and caseFolded literals.
justA, _ := syntax.Parse(`A`, syntax.Perl)
foldA, _ := syntax.Parse(`(?i:A)`, syntax.Perl)
fmt.Println(justA.Equal(foldA)) // should not be true

Given the odd pattern as above (0A|0[aA]).
In src/regexp/syntax/parse.go once factor() pulls out the 0 prefix it then tries to factor() between A and (?i:A) (what [aA] is parsed as) and is able to find a prefix of A as Equal() says true.  Once the code goes wrong it then does some sillyness with alternation between two empty non-capturing groups, which makes no sense as one would expect down a wrong branch.
In src/regexp/syntax/regexp.go Equal() under the case OpLiteral does not have any logic for the FoldCase flag, which I think is the error.
At first glance the logic in re2/regexp.cc TopEqual() for Literal does care about FoldCase which lets me be more sure this is what needs fixing.

The regexp 0A|0[aA] means 0(A|0)[aA]. You seem to be looking for (0A)|(0[aA]).

Even if that was the case, there would still be a problem because 0A|0[aA] is compiling and working very differently from 0a|0[aA] (as I showed)

Apologies for being confused.

$ perl -E 'say ("0a" =~ /0A|0[Aa]/)'
1
$ go run github.com/dolmen-go/goeval@latest -i=regexp -i=fmt 'fmt.Println(regexp.MustCompile("0A|0[Aa]").MatchString("0a"))'
false

By the way, (?:0A)|(?:0[Aa]) (which uses non-capturing groups) doesn't work either (reported in #64668).
$ go run github.com/dolmen-go/goeval@latest -i=regexp -i=fmt 'fmt.Println(regexp.MustCompile("(?:0A)|(?:0[Aa])").MatchString("0a"))'
false

@apmckinlay You must use the regexp/syntax.Perl flag with regexp/syntax.Parse to match regexp.Compile behaviour.
Here is your example fixed: https://go.dev/play/p/OrLoZxhMwMN
But the issue is still there.

RE2 has a similar bug, FWIW, but it's caused by this assumption. I haven't looked into what the Go regexp package is doing, but I suspect that @crisman is correct or, at the very least, on the right track.


How to make GridEditorBase.ts to work with [Required] attribute?

I have successfully implemented Serenity.IReadOnly to work with [ReadOnly] attribute.
Will you please give some hints to solve the title question?
Regards.

I don't think that's possible simply, as grid editor is not an input. You may override validateBeforeSave, raise an error and return false if it is not filled in.


iOS SMS, Email - Broken in Xamarin.Forms 2.3.4

With the latest Xamarin Forms release <IP_ADDRESS>, on SMS, only keyboard shows. Nothing happens on Email. Phone works fine. This is for the iOS version in both device and simulator.

@eopeter I've added a forms sample to simulate your errors, but I'm not seeing the same issues. It works fine on the device. Please note that you cannot test the plugin functionality on the simulator as you cannot test phone, sms and email on the simulator itself.

I have the same problem!
I have a test project in Xamarin forms and updated to 4.0 version of the message plugin.
Xamarin forms was on <IP_ADDRESS>. Everything is working fine.
When i update my Xamarin Forms project to Xamarin forms <IP_ADDRESS> ( and the same Xamarin.forms.maps) the email function does stop working. No email screen is popping up.
Making a call is still working
Going back to previous package of message plugin doesn't matter so it looks like it has to do with Xamarin.forms <IP_ADDRESS>

+1 email no longer works in <IP_ADDRESS>.

this should be reopened

This is fixed in Xam Forms 2.3.5 betas. I believe this had to do with calling it from an actionsheet.


Cannot install EWS via NuGet, Ubuntu 16.04 and 14.04

From @jwcrawley on October 4, 2016 19:18
Steps to reproduce
install Powershell for Linux (stock Ubuntu 14.04 or 16.04 - tested with both)
execute powershell in terminal
PS :> Install-Package Microsoft.Exchange.WebServices   (as listed https://www.nuget.org/packages/Microsoft.Exchange.WebServices/ )
Actual behavior
Install-Package : No match was found for the specified search criteria and
package name 'Microsoft.Exchange.WebServices'. Try Get-PackageSource to see
all available registered package sources.
At line:1 char:1

Install-Package Microsoft.Exchange.WebServices

 + CategoryInfo          : ObjectNotFound: (Microsoft.Power....InstallPacka
ge:InstallPackage) [Install-Package], Exception
 + FullyQualifiedErrorId : NoMatchFoundForCriteria,Microsoft.PowerShell.Pac
kageManagement.Cmdlets.InstallPackage

Expected behavior
Should successfully install required module, so EWS can be utilized in Linux Powershell scripts. Tested on Windows machines, and completes/installs successfully.
Environment data
Name                           Value

PSVersion                      6.0.0-alpha
PSEdition                      Core
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   <IP_ADDRESS>
GitCommitId                    v6.0.0-alpha.10
CLRVersion
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
Copied from original issue: PowerShell/PowerShell#2424

From @andschwa on October 12, 2016 18:27
Looks to me like package sources aren't setup. How did you add NuGet.org as a package source?

This issue was moved to OneGet/oneget#229


Missing "live" version of Business Data Google My Business Info

This client is missing the Google My Business Info Live types and endpoints.
It has endpoints for Task POST, Task GET, and Tasks Ready - but not the Live version.
Therefore we cannot do something like this, as per the docs of this client in the README:
import * as client from "dataforseo-client";

const businessDataApi = new client.BusinessDataApi(
  "https://api.dataforseo.com",
  {
    fetch: authFetch,
  },
);

let task = new client.BusinessDataGoogleMyBusinessInfoLiveTaskInfo(); // < --- THIS IS NOT FOUND
task.location_name = "New York,New York,United States";
task.language_code = "en";
task.keyword = "RustyBrick, Inc.";

const resp = await businessDataApi.googleMyBusinessInfoLive(task);  // < --- THIS IS NOT FOUND

thanks for noticing the issue. that endpoint added in new latest version (1.0.19)
code example:
`
const authFetch = createAuthenticatedFetch(username, password);
let serpApi = new client.BusinessDataApi("https://api.dataforseo.com", { fetch: authFetch });
let task = new client.BusinessDataTaskRequestInfo();
task.location_name = "New York,New York,United States";
task.language_code = "en";
task.keyword = "RustyBrick, Inc.";

let resp = await serpApi.googleMyBusinessInfoLive([task]);

`


font or text size

I am using the example code that prints hello world with white border.
Is there a way to increase the text size?

yes pass in size kwarg
https://github.com/adafruit/Adafruit_CircuitPython_framebuf/blob/master/adafruit_framebuf.py#L335


Empty Status' for EAS dashboard

#582
https://chefio.atlassian.net/browse/A2-772

Im taking over this work!


Juz tab

Title of change
Fixed UX of juz tab
Checklist

[ ] Unit tests written
[x] Manually tested
[x] Prettier & ESLint were run
[ ] New dependencies are included in package-lock.json

Screenshot
insert screenshot if needed

Deployed to: http://staging.quran.com:32825

Deployed to: http://staging.quran.com:32826

Deployed to: http://staging.quran.com:32827

rebuild

Deployed to: http://staging.quran.com:32828

Deployed to: http://staging.quran.com:32829


Should lock_api::RawMutex::unlock (and other unlock methods) be unsafe?

For most Mutex implementations, calling RawMutex::unlock without calling RawMutex::lock first is likely undefined behavior. The same goes for all other unlock_* methods. Therefore, I think that these trait methods should be unsafe.

That's a fair point, do you want to write up a PR for this?

Since you agree, I can look into it.

Fixed by #243


Add module path support to Javadoc task

Fixes #12433

@bot-gradle test ReadyForMerge plz

OK, I've already triggered ReadyForMerge build for you.


GDScript: Cannot use the class_name to call new() in the same file

Godot version: 3.2.1
OS/device including version: Linux
Issue description: If I define a class_name, I cannot use this name to instantiate the class from within the class from itself: fails with Identifier not found: Foo.
Steps to reproduce: Please see below.
Minimal reproduction project:
I have this "driver" code that I run using a headless build with godot-headless --script main.gd:
# main.gd
extends SceneTree

var Foo = preload("res://foo.gd")

func _init():
    var f = Foo.make_a_foo()
    quit()

And here's foo.gd, where the problem happens:
# foo.gd
extends Reference
class_name Foo

static func make_a_foo() -> Foo: # <--- no problem here
    return Foo.new() # <--- error here

When I run this project I get this error: SCRIPT ERROR: GDScript::reload: Compile Error: Identifier not found: Foo.

Duplicate of #21461.


Switch to curl

Blocked by https://github.com/alexcrichton/curl-rust/issues/282 and https://github.com/alexcrichton/curl-rust/issues/286


Microsoft.Insights/scheduledQueryRules@2023-12-01 : missing

Resource Type
Microsoft.Insights/scheduledQueryRules
Api Version
2023-12-01
Issue Type
Type is unavailable
Other Notes
No
     | registered resource provider found for location 'westeurope' and API
     | version '2023-12-01' for type 'scheduledqueryrules'. The supported
     | api-versions are '2024-01-01-preview, 2023-03-15-preview,
     | 2022-08-01-preview, 2022-06-15, 2021-08-01, 2021-02-01-preview,
     | 2020-05-01-preview, 2018-04-16, 2017-09-01-preview'. The supported
     | locations are 'westcentralus, westeurope, northeurope, norwayeast,
     | norwaywest, germanywestcentral, germanynorth, swedencentral,
     | francesouth, francecentral, westus, westus2, westus3, southcentralus,
     | polandcentral, switzerlandnorth, switzerlandwest, eastasia,
     | southeastasia, uaenorth, uaecentral, brazilsoutheast, brazilsouth,
     | qatarcentral, australiaeast, australiasoutheast, australiacentral,
     | australiacentral2, koreasouth, koreacentral, jioindiawest,
     | jioindiacentral, canadaeast, canadacentral, centralus, eastus, eastus2,
     | japaneast, japanwest, southafricanorth, southafricawest, uksouth,
     | ukwest, southindia, centralindia, northcentralus, italynorth,
     | israelcentral, spaincentral'.

Bicep Repro
n/a as the type and API version is just not available just.
Confirm

[X] I have read the troubleshooting guide and looked for duplicates.

I've encountered the same issue. This time it was related to renovate bot tool that we use for automatic detection and upgrade of dependencies. For bicep/ARM it uses this repository as the source for available api versions.
I met this issue for Microsoft.Insights/scheduledQueryRules with 2023-12-01 as @slavizh but also the same error with Microsoft.Consumption/budgets proposed to be updated to 2023-11-01 while the azure learn docs show the latest version is 2021-10-01.

We're encountering the same issue. Is there an ETA on a fix?

I got the same issue when I tried updating the resource from an older version.

2023-03-15-preview still works, but it throws a warning - it can be used as a temporary solution.

The saga continues:
Message: No registered resource provider found for location 'westeurope' and API version '2023-12-01' for type 'scheduledqueryrules'. The supported api-versions are '2024-01-01-preview, 2023-03-15-preview, 2022-08-01-preview, 2022-06-15, 2021-08-01, 2021-02-01-preview, 2020-05-01-preview, 2018-04-16, 2017-09-01-preview'. The supported locations are 'westcentralus, westeurope, northeurope, norwayeast, norwaywest, germanywestcentral, germanynorth, swedencentral, francesouth, francecentral, westus, westus2, westus3, southcentralus, polandcentral, switzerlandnorth, switzerlandwest, eastasia, southeastasia, uaenorth, uaecentral, brazilsoutheast, brazilsouth, qatarcentral, australiaeast, australiasoutheast, australiacentral, australiacentral2, koreasouth, koreacentral, jioindiawest, jioindiacentral, canadaeast, canadacentral, centralus, eastus, eastus2, japaneast, japanwest, southafricanorth, southafricawest, uksouth, ukwest, southindia, centralindia, northcentralus, italynorth, israelcentral, spaincentral'. (Code:NoRegisteredProviderFound)

@alex-frankel is there someone from the Azure Monitor you can reach to fix this? as you can see below versions that are not available are constantly pushed and higher versions that are available are not pushed.

Any update? Seems like 2024-01-01-preview is also available in the resource provider:

@faarbaek

Any updates? I'm trying to use Bicep to create LogAlert Rules, but the whole template and the issues with deploying them is so complex I've already spent hours trying to find the right combination.


Feature/a1 sldt 489@getting started guide

This PR introduces a docker-compose file based on pre build images.
The images are built currently by me locally:
EDC images are based on EDC milestone-3.1.
API Wrapper / Proxy / Backend Service is based on the "merge-aas-setup" branch.
Digital Twin Registry is the latest version from the INT environment.
As @florianrusch-zf  suggested, i pushed the images to ghcr.io/catenax-ng/catenax-at-home/* they are publicly available.
I had build issues because the gradle-wrapper.jar were not available, I added them as well.
Please don't merge this yet, as it's still work in progress.

[...] I had build issues because the gradle-wrapper.jar were not available, I added them as well.

if you use the gradlew.sh|.bat file it should download the jar file...or I'm wrong?

[...] I had build issues because the gradle-wrapper.jar were not available, I added them as well.

if you use the gradlew.sh|.bat file it should download the jar file...or I'm wrong?

The gradle-wrapper.jar is used by the gradlew.sh to download gradle. So it's required to be there.

The gradle-wrapper.jar is used by the gradlew.sh to download gradle. So it's required to be there.

 Thanks!

@fmisir can you then also add the maven-wrapper.jar files? I also suggest a change of the .gitignore


codecov improvements

Pull Request type
Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[ ] No

Other information

This change is

Current dependencies on/for this PR:

main

PR #705   👈

This comment was auto-generated by Graphite.


Update Product json

#200 maybe
#201 maybe

So VSCode is able to do this because their app is always most updated, and these proposed API may have breaking changes with different versions of VSCode (like we have Code OSS). So potential danger from this. Let's just make sure we have Code OSS as updated as possible.

s/o @Himanshu-Singh-Chauhan

s/o @Himanshu-Singh-Chauhan


Dashboard app logout session management

When Dashboard app is the originator of the logout request, it will not destroy its session before forwarding the request to its IdP (the WSO2 server) .
Currently dashboard destroys its session after receiving a logout response from the IS.
So, for some reason, dashboard did not receive a logout response, it will not try to call the IS for re-authentication since dashboard has the session still.
Affects IS 5.6.0

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.


Add support for square brackets

With Slim, using CSS-style square brackets is useful to disambiguate between content and empty attributes.
This is extremely useful in some situations such as using web components (e.g with Polymer).
Example in Slim ported from the Polymer tutorial:
body[unresolved]
  core-header-panel

  core-toolbar
    paper-tabs[id="tabs" selected="all" self-end]
      paper-tab[name="all"] All
      paper-tab[name="favorites"] Favorites

  .container[layout vertical center]

This code in Ace currently generates e.g acontainer[layout tag and center] as content.
Current, ugly workaround is to use empty equals to force Ace to recognise the attributes as such instead of outputting layout vertical center as content:
body unresolved=""
  core-header-panel

  core-toolbar
    paper-tabs id="tabs" selected="all" self-end=""
      paper-tab name="all" All
      paper-tab name="favorites" Favorites

  .container layout="" vertical="" center=""

@lloeki amber (https://github.com/eknkc/amber) does square brackets, maybe that would suit you more?

@matejkramny amber looks interesting. But the square brackets seem mandatory: am I right? Not putting it down, btw. But I'd like to know.
@lloeki I've encountered the same problem u have with empty attributes. You dont actually need to specify the "" at the end. So the .container, for instance, can just simply be
.container layout= vertical= center=
Would be nice to have them be recognized as attributes still without the equal (tough request, yes I know!), but definitely still better than what you're doing now!

@jf yes correct, square brackets are necessary for attributes.
Looking at it now, it may not be optimal since square brackets are valid xml characters. If you need an attribute that needs square brackets, it sort of gets confusing rather quickly. eg: a[href="yeah" [ng-click="doYeah()"]]


Is this release has the same configuration options as in morris.js?

for example the option to set resize(true/false), grid, etc. Because I have not found it in the documentation

not at all, PR´s are welcome.


Added support for java.jdt.ls.java.home vs-code configuration

@ravihara could you run ./gradlew format before committing since the CI reports some linting issues.

Hi @CsCherrYY I have done the required changes as mentioned in the above conversation. Please review the changes and take it forward. Thanks.
I have run ./gradlew format and ./gradlew build to check things.

@ravihara the changes LGTM. There are merge conflicts since we have updated all the deps in #1177. Could you just rebase and resolve all the conflicts? BTW could you exclude the changes in npm-package/package-lock.json(which is not used now), then I think we can merge this PR.

@ravihara please note that we have moved our default branch to develop and I have changed your pr's target branch to develop, thus, when rebasing, the target branch should be develop.

@ravihara please note that we have moved our default branch to develop and I have changed your pr's target branch to develop, thus, when rebasing, the target branch should be develop.

Hi, sorry for the delay! Got tied up with the work. I have done the necessary changes. Please review it again and take it forward.

thank you @ravihara !


virtctl command not working on Arm Mac

/kind bug
What happened:
I followed the documentation to install virtctl.
It suggested to use the following code:
VERSION=$(kubectl get kubevirt.kubevirt.io/kubevirt -n kubevirt -o=jsonpath="{.status.observedKubeVirtVersion}")
ARCH=$(uname -s | tr A-Z a-z)-$(uname -m | sed 's/x86_64/amd64/') || windows-amd64.exe
echo ${ARCH}
curl -L -o virtctl https://github.com/kubevirt/kubevirt/releases/download/${VERSION}/virtctl-${VERSION}-${ARCH}
chmod +x virtctl
sudo install virtctl /usr/local/bin

My arch is darwin-arm64 because I use a MacBook M1. This architecture is not supported and it downloaded a file containing Not Found instead of returning an error.
What you expected to happen:

An error message like plugin "virt" does not offer installation for this platform when you use krew.
Support for the darwin-arm64 architecture.

How to reproduce it (as minimally and precisely as possible):
VERSION=$(kubectl get kubevirt.kubevirt.io/kubevirt -n kubevirt -o=jsonpath="{.status.observedKubeVirtVersion}")
ARCH=unicorn-arm2048
curl -L -o virtctl https://github.com/kubevirt/kubevirt/releases/download/${VERSION}/virtctl-${VERSION}-${ARCH}
file ./virtctl

Anything else we need to know?:
The following code is a workaround:
VERSION=$(kubectl get kubevirt.kubevirt.io/kubevirt -n kubevirt -o=jsonpath="{.status.observedKubeVirtVersion}")
# Force AMD64 build and use the emulation
ARCH=darwin-amd64
curl -L -o virtctl https://github.com/kubevirt/kubevirt/releases/download/${VERSION}/virtctl-${VERSION}-${ARCH}
chmod +x virtctl
sudo install virtctl /usr/local/bin

Environment:

KubeVirt version (use virtctl version): NA
Kubernetes version (use kubectl version): NA
VM or VMI specifications: NA
Cloud provider or hardware configuration: NA
OS (e.g. from /etc/os-release): MacOs ARM
Kernel (e.g. uname -a): Darwin fungiboletus.local 21.3.0 Darwin Kernel Version 21.3.0: Wed Jan  5 21:37:58 PST 2022; root:xnu-8019.80.24~20/RELEASE_ARM64_T8101 arm64
Install tools: NA
Others: NA

I'm afraid I don't know your build system and adding support to this architecture seems like a big task for someone outside of the project. It's not about adding a line in a github action yaml file.
However I could probably do a pull request to update the documentation, so the example script downloads the darwin-amd64 instead of a 404 error file.

The next NixOS release (22.05) will have virtctl packaged for all platforms.

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale

/close
Release 1.0 has the support, see https://github.com/kubevirt/kubevirt/releases/download/v1.0.0/virtctl-v1.0.0-darwin-arm64

Great to hear! Thanks everyone.


Plugin : Sudo - Add support of sudoedit

If $EDITOR is the first word of the line, change it to sudoedit when Esc-Esc

Hey there -- I really appreciate the contribution.
Cheers,
Robby


Update sap-deploy-solution.md

Fix phrasing (as versions > 7.50 may also require notes)

approved - thanks @YoavDaniely!
#sign-off


Best practice when use with React

I wonder what's the best practice to use immutable.js with React :
Few confusions :
Where should I use the immutable object, should I use convert all my data to immutable object ?
such as :

get data from API then convert JSON to an immutable object
update immutable object with update, setIn function, and never call toJS() for best performance
In components : obj.getIn( ['foo', 'bar'] )

Is this the best practice? or any other suggestions?

but if the value is List or Map?

@ishanyang sorry, what you mean?

It sounds like you're doing reasonable things so I'll close this issue (trying to use GH Issues to only track bugs)


AlwaysOnTop.exe - High CPU Usage

Microsoft PowerToys version
0.53.1
Running as admin

[X] Yes

Area(s) with issue?
Always on Top
Steps to reproduce
AlwaysOnTop option causing heavy CPU usage

✔ Expected Behavior
Low CPU Usage. There is a shortcut to activate, so it should be working only if any window is pinned.
❌ Actual Behavior
High CPU Usage.
Other Software
No response

Duplicate of #15362

/dup #15362


feat: Deployment Scripts for Modular Smart Account Architecture

Refactor deploy.ts to reduce code duplication.
Add Deployment functions for:

ECDSA Registry Module
Multichain Validator Module
Passkey Module
Session Key Manager Module
ERC20 Session Validation Module
Smart Contract Ownership Registry Module

Tested on Polygon Testnet with throwaway salts:
{
  "SmartAccount": "0x9777a082B23C09f81cB23C2635cCb93603D1AF42",
  "SmartAccountFactory": "0xD8E65814d5F528fa573eF9bb5Aa22817DEE3E1bf",
  "GasEstimator": "0x7706236FD4212A48FC0eB5856f956E8441bF4956",
  "Decoder": "0x2246d8899F0175095Dc236161e5945F11BDE346D",
  "MultiSend": "0xf06115ef5431369208c4084407816242D562A688",
  "MultiSendCallOnly": "0x24f4b5E148DC0fc5893BD8111831e3086107A0E1",
  "VerifyingPaymaster": "0x9a7240D70972D9c3a693525a341D171e7d738a82",
  "EcdsaOwnershipRegistryModule": "0xF993fc8Dc0EE7aece7abf0d6B6939f9d67875dBa",
  "MultichainValidatorModule": "0xCCCC01Bef3F9a28814b88aC36a819e96eec47E15",
  "PasskeyModule": "0x920Bc4f3E861258Dd0eB23272e6D36c5c9F5CF25",
  "SessionKeyManagerModule": "0x10E21264E056F68ff0279c08e6edE2ea35a06770",
  "ERC20SessionValidationModule": "0xbfB744D8b2804D4C5727D24abe87AE895A0eFe5f",
  "SmartContractOwnershipRegistryModule": "0x7DAF145b8dd02Bd8fD47db39b8712202070905Fe"
}

which salts and scripts are used to deploy SmartWallet and SmartWalletFactory?

which salts and scripts are used to deploy SmartWallet and SmartWalletFactory?

The salts were random strings which I didn't save, I will start a fresh deployment with standardized salts.

I'm manually merging the changes from #105 for the fixes done in VerifyingSingletonPaymaster. I will resolve the comments as I deploy the contracts on each chain.

I'm manually merging the changes from #105 for the fixes done in VerifyingSingletonPaymaster. I will resolve the comments as I deploy the contracts on each chain.

yes ok.

which salts and scripts are used to deploy SmartWallet and SmartWalletFactory?

The salts were random strings which I didn't save, I will start a fresh deployment with standardized salts.

ok I am testing multichain module and I need same address for mumbai and bnb mainnet/testnet. at least two chains

which salts and scripts are used to deploy SmartWallet and SmartWalletFactory?

The salts were random strings which I didn't save, I will start a fresh deployment with standardized salts.

ok I am testing multichain module and I need same address for mumbai and bnb mainnet/testnet. at least two chains

I blindly tried to deploy on bnb mainnet but script failed for me.
`
npx hardhat run --network bnb_mainnet scripts/deploy.ts
const deployerInstance = await getPredeployedDeployerContractInstance();
// await deployEntryPointContract(deployerInstance);
console.log("=========================================");
await deployBaseWalletImpContract(deployerInstance);
console.log("=========================================");
await deployWalletFactoryContract(deployerInstance);
console.log("=========================================");
//  await deployGasEstimatorContract(deployerInstance);
console.log("=========================================");
// await deployDecoderContract(deployerInstance);
console.log("=========================================");
// await deployMultiSendContract(deployerInstance);
console.log("=========================================");
// await deployMultiSendCallOnlyContract(deployerInstance);
console.log("=========================================");
// await deployVerifySingeltonPaymaster(deployerInstance);
console.log("=========================================");
await deployEcdsaOwnershipRegistryModule(deployerInstance);
console.log("=========================================");
await deployMultichainValidatorModule(deployerInstance);
console.log("=========================================");
// await deployPasskeyModule(deployerInstance);
console.log("=========================================");
// await deploySessionKeyManagerModule(deployerInstance);
console.log("=========================================");
// await deployErc20SessionValidationModule(deployerInstance);
console.log("=========================================");
// await deploySmartContractOwnershipRegistryModule(deployerInstance);
console.log("=========================================");
Generating typings for: 6 artifacts in dir: typechain for target: ethers-v5
Successfully generated 15 typings!
Compiled 6 Solidity files successfully
Deploying with EOA 0x2cf491602ad22944D9047282aBC00D3e52F56B37 through Deployer Contract 0xD3f89753278E419c8bda1eFe1366206B3D30C44f
SmartAccount Computed Address: 0x0000cc4a2BC176001E72A353f7660A7F59D78628
Submitted transaction 0xa79bdbd697299641cd4a776a05ae846f83837aa2e643f76d57a085cf6aa4b486 for deployment
Transaction 0xa79bdbd697299641cd4a776a05ae846f83837aa2e643f76d57a085cf6aa4b486 is included in block 31026504
DEVX_WALLET_IMP_V0_11042023_AwPKF0R Deployed Successfully
Nothing to compile
No need to generate any newer typings.
Warning: Return value of low-level calls not used.
--> contracts/smart-account/BaseSmartAccount.sol:89:13:
|
89 |             payable(msg.sender).call{
|             ^ (Relevant source part starts here and spans across multiple lines).
Successfully submitted source code for contract
contracts/smart-account/SmartAccount.sol:SmartAccount at 0x0000cc4a2BC176001E72A353f7660A7F59D78628
for verification on the block explorer. Waiting for verification result...
Successfully verified contract SmartAccount on Etherscan.
https://bscscan.com/address/0x0000cc4a2BC176001E72A353f7660A7F59D78628#code
Error: invalid address (argument="address", value="", code=INVALID_ARGUMENT, version=address/5.7.0) (argument=null, value="", code=INVALID_ARGUMENT, version=abi/5.7.0)
at Logger.makeError (/Users/chirag/work/biconomy/scw-playground/scw-contracts/node_modules/@ethersproject/logger/src.ts/index.ts:269:28)
at Logger.throwError (/Users/chirag/work/biconomy/scw-playground/scw-contracts/node_modules/@ethersproject/logger/src.ts/index.ts:281:20)
at Logger.throwArgumentError (/Users/chirag/work/biconomy/scw-playground/scw-contracts/node_modules/@ethersproject/logger/src.ts/index.ts:285:21)
at AddressCoder.Coder._throwError (/Users/chirag/work/biconomy/scw-playground/scw-contracts/node_modules/@ethersproject/abi/src.ts/coders/abstract-coder.ts:68:16)
at AddressCoder.encode (/Users/chirag/work/biconomy/scw-playground/scw-contracts/node_modules/@ethersproject/abi/src.ts/coders/address.ts:22:18)
at /Users/chirag/work/biconomy/scw-playground/scw-contracts/node_modules/@ethersproject/abi/src.ts/coders/array.ts:71:19
at Array.forEach ()
at pack (/Users/chirag/work/biconomy/scw-playground/scw-contracts/node_modules/@ethersproject/abi/src.ts/coders/array.ts:54:12)
at TupleCoder.encode (/Users/chirag/work/biconomy/scw-playground/scw-contracts/node_modules/@ethersproject/abi/src.ts/coders/tuple.ts:54:20)
at AbiCoder.encode (/Users/chirag/work/biconomy/scw-playground/scw-contracts/node_modules/@ethersproject/abi/src.ts/abi-coder.ts:111:15) {
reason: 'invalid address (argument="address", value="", code=INVALID_ARGUMENT, version=address/5.7.0)',
code: 'INVALID_ARGUMENT',
argument: null,
value: ''
}`

I blindly tried to deploy on bnb mainnet but script failed for me.

Fixed


COM mode datetime option

I need the datetime option to be added to df2sd method when the mode is COM.
Here is the current code in sasiocom.py:
LINE 633 - 634:
DATETIME_NAME = 'DATETIME26.6'
DATETIME_FMT = '%Y-%m-%dT%H:%M:%S.%f'
LINE 682-683:
definition = "'{}'n num informat={} format={}".format(name, DATETIME_NAME, DATETIME_NAME)
formats[name] = lambda x: "'{:{}}'DT".format(x, DATETIME_FMT) if pd.isnull(x) is False else 'NULL'
My proposed changes are adding the following format constants:
DATE_NAME = 'DATE9.'
DATE_FMT = '%d%b%Y'
and updating the code in lines 682-683 to:
if datetimes.get(name, '').lower() == 'date':
definition = "'{}'n num informat={} format={}".format(name, DATE_NAME, DATE_NAME)
formats[name] = lambda x: "'{:{}}'D".format(x, DATE_FMT) if pd.isnull(x) is False else 'NULL'
else:
definition = "'{}'n num informat={} format={}".format(name, DATETIME_NAME, DATETIME_NAME)
formats[name] = lambda x: "'{:{}}'DT".format(x, DATETIME_FMT) if pd.isnull(x) is False else 'NULL'
Thanks for your attention

Hey @HamidAbshahi , would you mind opening this on the SASPy GitHub site? This is an example notebook repo. If you could use https://github.com/sassoftware/saspy/issues instead, it will be where it needs to be. My first question would be why not use the IOM access method since it supports everything and performs significantly better than COM. COM is missing many features and functionality that the other 3 access methods have, not just his one. Hence the note in the doc: https://sassoftware.github.io/saspy/configuration.html#com
Thanks,
Tom


Compute full Kodaira type of singular loci

Add functionality to singular_loci to compute orders of vanishing of (f, g, Delta) at each singular locus.

Codecov Report
Base: 95.41% // Head: 90.87% // Decreases project coverage by -4.54% :warning:

Coverage data is based on head (9d662b6) compared to base (ac26fd0).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
- Coverage   95.41%   90.87%   -4.55%
==========================================
  Files           8        8
  Lines         218      241      +23
==========================================
+ Hits          208      219      +11
- Misses         10       22      +12

Impacted Files
Coverage Δ

src/TateModels/constructors.jl
82.22% <ø> (-17.78%)
:arrow_down:

src/WeierstrassModels/constructors.jl
80.95% <ø> (-19.05%)
:arrow_down:

src/TateModels/attributes.jl
100.00% <100.00%> (+2.43%)
:arrow_up:

src/WeierstrassModels/attributes.jl
100.00% <100.00%> (+4.54%)
:arrow_up:

src/auxiliary.jl
90.32% <0.00%> (+3.22%)
:arrow_up:

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

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

Thank you @HereAround for the comments. They should all be addressed in e57209a.


do you use different bounding box regression?

because the feature will be changed in the net, so do you have to use different bounding box when detection?

I dont quite get it, could you expand on this?

when changing the pooling layers or the conv layers, the perception field may be affected, so there may be some effects in the bounding box regression operation. are there any changes in this step?

Why the features will be changed and by whom will it be changed? If the offset of final deform_psroi_pool were changed, then the bounding box would no doubt be changed.


Add conversion documentation

This adds a document regarding all the values which are converted from
Docker Compose to Kubernetes / OpenShift. Adding both notes as well as
the value that docker-compose is mapped to in Kubernetes / OpenShift.

I should also have link to this document in readme

@kadel I agree. I'll add an "index" to the README eventually.
I've updated the PR based on your comments as well as converted it to a "compact" table instead of having all the spacing everywhere.

@kadel Updated again, I shortened the unsupported column to Y/N as well as labeled the separate sections by boldness (volumes, services, networks, etc.). Looks better now :+1:


City environment issues

Problem description
1) City environment steering problem
2) Deep Learning Learning Outcomes Optimization Problem
Problem details
_1) Run x360e.exe in .. \ AD_Cookbook_AirSim \ City \ CityEnviron \ Binaries \ Win64. Then, the steering is freely input in the game, and it is not operated. How can I solve this problem?

It runs well in a neighborhood environment. After learning the deep learning, the car moves very zigzag. Is there any way to optimize it for normal driving?_

Exepriment/Environment details

Tutorial used: AirSimE2EDeepLearning
Environment used: *1) city, neighborhood *
Versions of artifacts used (if applicable): Python 3.6, Keras 2.1.2, Cuda9.0, Xbox One controller, GTX960

I don't understand the issue in (1). Is the car not responding to the xbox controller? Is the car responding, but doing the wrong thing?
Also, I think the latest builds produced on the airsim repo support controllers natively, so you may have luck trying that build.

What is the plot that you shared? There are ways to change the behavior of the model - change the architecture, change the preprocessing steps. If the car is swerving too much, think about which of our preprocessing steps might address that and try changing the parameters. If you're still stuck, feel free to email me at<EMAIL_ADDRESS>and we can continue the discussion - I don't want to post too many analysis answers for fear that others will just read the answer instead of thinking about and understanding the solution.

답변 주셔서 감사합니다.
1번 문제는 새로운 빌드 버전으로 시도해보겠다!
위의 질문들을 상세히 적어서, 메일로 따로 드리겠습니다!

Closing due to inactivity.


No pre-commit file?

Errors out if there is no .git/hooks/pre-commit
Steps to Reproduce

Using an existing git project, pull down latest
Install lefthook: npm -i —save-dev @arkweid/lefthook
Copy over existing config

commit-msg:
  commands:
    lint-msg:
      run: npx commitlint

pre-commit:
  commands:
    lint-html:
      glob: "*.{html,tpl}"
      run: npx stylelint {staged_files}
    lint-md:
      glob: "*.{md,markdown}"
      run: npx mdl {staged_files}
    lint-css:
      glob: "*.{css,scss}"
      run: npx stylelint {staged_files}
    lint-js:
      glob: "*.{js,jsx,ts,tsx}"
      run: npx eslint {staged_files}
    test:docs:
      glob: “*.{md, markdown}”
      run: alex
    test-js:
       run: npx jest

Run following command to install:

#603: $ ./node_modules/.bin/lefthook install
SYNCING lefthook.yml
SERVED HOOKS: pre-commit, commit-msg, prepare-commit-msg
open /home/newton/code/.git/hooks/pre-commit: no such file or directory

Does the directory .git/hooks exist? I noticed the same an when I mkdir .git/hooks it worked.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #412 (c352a69) into main (18dce90) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #412   +/-   ##
=======================================
  Coverage   95.69%   95.69%
=======================================
  Files           1        1
  Lines         395      395
  Branches       42       42
=======================================
  Hits          378      378
  Misses         17       17

Help us with your feedback. Take ten seconds to tell us how you rate us.


Corrects React Native version specifier in Mocking

[x] This is a small change
[ ] This change has been discussed in issue #<?> and the solution has been agreed upon with maintainers.

Description:
The two scenarios for the different React Native version ranges misses 0.59. This corrects the latter range.

Thanks!


FIX: Remove restriction on upload data directory structure

Change Summary
Overview

Our APIs don't require the full path anymore, they can use just the filename and infer the directory structure. This is helpful for the upload to be able to upload a file from the current working directory.
imap-data-access -v upload imap_mag_l1a_burst-magi_20240314_v007.cdf
INFO:imap_data_access.io:Uploading file /Users/grlu5547/code/imap/imap-data-access/data/imap/mag/l1a/2024/03/imap_mag_l1a_burst-magi_20240314_v007.cdf to https://api.dev.imap-mission.com/upload/imap_mag_l1a_burst-magi_20240314_v007.cdf
HTTP Error: 409 - Conflict
Server Message: "imap/mag/l1a/2024/03/imap_mag_l1a_burst-magi_20240314_v007.cdf already exists."

closes #41

I was trying to find doc strings to update which reflects this change. Looks like there are few places to update. I can help pin down which ones needs update.

I was trying to find doc strings to update which reflects this change. Looks like there are few places to update. I can help pin down which ones needs update.

I didn't find any that needed to be updated, so if you have ones you think should be changed then yes please let me know. I thought all of the other locations were valid still. For example: https://github.com/IMAP-Science-Operations-Center/imap-data-access/blob/82e9bbb1b51c30a41c10af6d6892747ded89e0de/imap_data_access/cli.py#L169-L170
is still correct because you must give the path to the file, we don't infer it anywhere. Maybe changing it to "This can be a relative or absolute path." would help though?

@tech3371 I didn't actually remove the default directory data/ requirement on anything other than uploading. So, all path validation and setting of the IMAP_DATA_ACCESS global variables are still valid I think. The only thing this PR does is remove the check that we are within that structure. It lets you upload relative paths. imap-data-access path/to/file.txt is equivalent to imap-data-access $PWD/path/to/file.txt now and doesn't care that IMAP_DATA_ACCESS directory is pointing to /my/other/location.

I was trying to find doc strings to update which reflects this change. Looks like there are few places to update. I can help pin down which ones needs update.

I didn't find any that needed to be updated, so if you have ones you think should be changed then yes please let me know. I thought all of the other locations were valid still. For example:
https://github.com/IMAP-Science-Operations-Center/imap-data-access/blob/82e9bbb1b51c30a41c10af6d6892747ded89e0de/imap_data_access/cli.py#L169-L170
is still correct because you must give the path to the file, we don't infer it anywhere. Maybe changing it to "This can be a relative or absolute path." would help though?

The README has the following statement regarding the data directory. Does this only apply to downloading files and not to uploading files?
If the ``IMAP_DATA_DIR`` variable is not set, the program defaults to the user's current working directory + ``data/``.

The README has the following statement regarding the data directory. Does this only apply to downloading files and not to uploading files?

If the IMAP_DATA_DIR variable is not set, the program defaults to the user's current working directory + data/.

Yes, that is correct with this PR. Would it help to add something like this sentence?

The IMAP_DATA_DIR is only used for downloads.

Yes, that is correct with this PR. Would it help to add something like this sentence?

The IMAP_DATA_DIR is only used for downloads.

Okay thanks for the clarification. Yeah, I think a short note about that would be helpful

Merging now, feel free to open another PR or leave suggestions here and I'm happy to follow-up if there are any other docs suggestions.


Check if IntegrationTest and Debug RPC modules are enabled by default

If not, enable them.
Ref： https://github.com/nervosnetwork/ckb/blob/4218e7dfd5b42c24fbea2f920a087c3804425e77/rpc/src/module/test.rs#L16-L33

Have to enable them explicitly.


changed eps

Hi, I was getting some NANs in the entropy loss term when training with fp16. I verified that this was fixed when increasing eps a little bit.

nice! thank you!
how is LFQ holding up in your experiments otherwise?

lfq is great
I don't know if entropy loss makes a big difference. But even without any entropy loss, with properly normalized inputs, lfq gets really good perplexity/codebook usage.
I haven't played around with whether commitment loss makes a difference.
I'm using LFQ for transformers, to try and train an image autoencoder over DCT features your-vae-sucks

@theAdamColton great to hear you are seeing positive results! i haven't witnessed the improvements first hand but will get back into autoencoders soon

@theAdamColton LFQ is super interesting to me as the binary codes are perfect for genetic algorithms

@theAdamColton if you ever hit a roadblock or a negative result, do let me know. it will save all of us a bunch of time and compute


Improve our scroll physics on iOS

Changes in this patch:

iOS now uses a different scrollDrag constant than Android.

ScrollConfigurationDelegate now knows about target platforms.
ScrollBehaviors now know about target platforms.
RawInputLine now has to be told what platform it's targetting.
PageableList now has a concept of target platform.

make debugPrintStack filter its stack.

move debugPrintStack to assertions.dart.

add support for limiting the number of frames to debugPrintStack.
make defaultTargetPlatform default to android in test environments.
remove OverscrollStyle and MaterialApp's overscrollStyle argument. You
can now control the overscroll style using Theme.platform.
the default scroll configuration is now private to avoid people
relying on the defaultTargetPlatform getter in their subclasses (since
they really should use Theme.of(context).platform).
fix some typos I noticed in some tests.
added a test for flinging scrollables, that checks that the behavior
differs on the two target platforms.
made flingFrom and fling in the test API pump the frames.
added more docs to the test API.
made the TestAsyncUtils.guard() method report uncaught errors to help
debug errors when using that API.

I also added a skipped test that shows a bug in LazyBlock flinging.
See https://github.com/flutter/flutter/issues/5339

This fixed https://github.com/flutter/flutter/issues/4737


bug: clicking through global search activates VISUAL mode

Check the following:

[X] I have checked the behavior after setting "vscode-neovim.neovimClean" in VSCode settings.
[X] I have read the vscode-neovim docs.
[X] I have searched existing issues.

Neovim version (nvim -v)
NVIM v0.9.4
Operating system/version
Windows 11
Describe the bug
When jumping from global search, VISUAL mode is automatically activated.

Steps To Reproduce

click global search results
hit enter key to jump to text
visual mode is activated

Expected Behavior
Cursor should jump in the editor in NORMAL mode.

Duplicate of https://github.com/vscode-neovim/vscode-neovim/issues/1738


Make Error on Ubuntu

Hey! I have an error when running the make command in the build folder. I get an error that I should use C++ 11, But i'm unsure how to do this, I read you do it in the Makefile, but how?
from /home/thybe/Desktop/gainput/lib/source/gainput/GainputAllocator.cpp:2:
/usr/include/c++/7/bits/c++0x_warning.h:32:2: error: #error This file requires compiler and library support for the ISO C++ 2011 standard. This support must be enabled with the -std=c++11 or -std=gnu++11 compiler options.
 #error This file requires compiler and library support \

Do you build it using make or cmake?

I started by cmake .. and followed up with make, Like the README says

make sure you have a recent version of gcc and clang installed

Ubuntu Version: 17.10
GCC Version: 7.2.0
CLang Version: 4.0.1-6
I'm pretty sure all of these are up to date.

Hi,
I'll admit that I find this error message a bit confusing. It complains that gainput.h includes cstdint on line 8. But this doesn't seem to be the case, at least not in the last version.
Can you confirm that this also the case for you?
Cheers,
Johannes

Hey!
I checked out my version of gainput.h. Turns out it actually does include .
I then looked at the code on GitHub, and it's not there. So as you said i'm using an outdated version.
I will now clone the latest version and see how it goes.
Thanks in advance!

Using the latest version of the repository fixed the problem. FYI i was using the 1.0.0 release before.
Maybe you can make another release with the current version, as people usually resort the the Releases tab. Thanks alot! :+1:

Awesome, that's good to hear.
You're right. I should do another release. Thank you!


Remove configuration from init & add style

Fixes CLI-25
The current init experience is overwhelming and broken. This simplified experience heavily encourages the user to visit rivet.gg/learn in both the init output and the default config.
The engine-specific onboarding will be replaced by an in-engine experience.
This will require a lot of updates to the default tutorials on the site.

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

Current dependencies on/for this PR:

main

PR #125

PR #126

PR #127

PR #128

PR #129

PR #130   👈

This stack of pull requests is managed by Graphite.

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

Current dependencies on/for this PR:

main

PR #125

PR #126

PR #127

PR #128

PR #129

PR #130   👈

This stack of pull requests is managed by Graphite.

Merge activity

Dec 20, 4:20 AM: @@NathanFlurry started a stack merge that includes this pull request via Graphite.

Merge activity

Dec 20, 4:26 AM: @@NathanFlurry merged this pull request with Graphite.


Implemented auto-kicking members if no introduction provided in 24 hours of joining the group

Fixes #12

@daemon1024 Please close and reopen the PR to run the GH Workflow tests.

@sidntrivedi012 I guess that should do :)

@daemon1024 Since its a quite dangerous feature, thus it will be great if you or anyone can write tests for it too. Not necessarily in this PR. But, writing tests will help to validate this feature in a better way.


About endpoint

Hey! Thanks for your work on this project. I've been using roux a little bit on a side project, and was missing a user specific endpoint, thus this PR.
Adds /{user}/about endpoint.
Example API call: https://www.reddit.com/user/beneater/about/.json
There's one wrinkle, AboutData has a SubredditData field which looks like just about the content of SubredditData, but cargo test fails if I add it back in. Currently it's commented out.
I'd not feel comfortable adding an almost copy of SubredditData, unless you wish it be added as such

Awesome, thanks for the pr!
I've looked at the subreddit issue and it seems like the issue is that there's a single field: public_description_html that isn't an option type. I'll fix that and uncomment subreddit in the About response.
Thanks!


use slugs in URLs

Feature Request
Is your feature request related to a problem or unsupported use case? Please describe.
Not a fix
Describe the solution you'd like
use slugs instead of uuids to reach a team page. i.e.: teams/my-team instead of teams/5393482e-7ee4-45f2-a5fa-0ee4753e007f
advantages :

easier to remember / type by hand for direct access

drawbacks :

as slugs can change, we'd have to be careful to still reference team by their ids

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Discovery, Documentation, Adoption, Migration Strategy
If you can, explain how users will be able to use this and possibly write out a version the docs (if applicable).
Maybe a screenshot or design?
Do you want to work on it through a Pull Request?
yas !

This was done in #36 - and undone in #480. (Sometimes that's the price of learning!)


The "next-server" process consumes too much memory.

The "next-server" process consumes too much memory. VS Code and Cursor both launched an instance separately.
It may be a bug of the next js.

@davila7


query string parameters on GET

hi, i just read that it's no supported by node-fetch. i'm trying to consume an API that only works with GET so y cannot use the POST body. Any ideas of how to solve this?

You might want to explain it with a bit of code, there are really 3 ways to append data: as url, as headers and as body.

On similar lines how do I attach query paramets to a get url?
is this correct?
    return fetch('http://localhost:' + index.port + '/api/v1/todos/users', {
        method: 'GET',
        headers: {'Content-Type': 'application/json'},
        qs: {access_token: PAGE_ACCESS_TOKEN},
    })

I get this error
{ message: 'An active access token must be used to query information about the current user.',
2016-10-14T11:47:44.862805+00:00 app[web.1]:      type: 'OAuthException',
2016-10-14T11:47:44.862805+00:00 app[web.1]:      code: 2500,
2016-10-14T11:47:44.862806+00:00 app[web.1]:      fbtrace_id: 'D3AXnal4LBE' } }

im having this exact problem
fetch('shittyApi/', {         method: 'GET',         headers: {'Content-Type': 'application/json'},         qs: {param1: "string"},     })
and the qs just is not pased. y also tried pasing to string and atach it in the end of the url

Huh, where do you guys find the tutorial that it says Fetch accepts qs as an option? It doesn't, Fetch is a low-level request module like request.
If you have to do this by hand, I am assume you need to send a header named "access_token" (maybe named differently, depending on oauth version and provider), but find that oauth header name, and set your token as a value.
But you probably shouldn't do oauth with fetch anyway. There are many oauth modules that help you with it. For example, grant.

I just have to pass access_token: PAGE_ACCESS_TOKEN as query parameter. How do i do it?

You append it to the url

when appended to url get this err:
ERROR TypeError: Cannot read property '1' of undefined     at /Users/democles/remux/build/webpack:/data/queries/studies.js:59:28     at Array.map (native)     at /Users/democles/remux/build/webpack:/data/queries/studies.js:53:24     at runCallback (timers.js:574:20)     at tryOnImmediate (timers.js:554:5)     at processImmediate [as _immediateCallback] (timers.js:533:5) From previous event:     at Promise.then (/Users/democles/remux/node_modules/sequelize/lib/promise.js:21:17)     at resolve (/Users/democles/remux/build/webpack:/data/queries/studies.js:52:10)     at resolveOrError (/Users/democles/remux/node_modules/graphql/execution/execute.js:454:12)     at resolveField (/Users/democles/remux/node_modules/graphql/execution/execute.js:445:16)     at /Users/democles/remux/node_modules/graphql/execution/execute.js:252:18     at Array.reduce (native)     at executeFields (/Users/democles/remux/node_modules/graphql/execution/execute.js:249:42)     at executeOperation (/Users/democles/remux/node_modules/graphql/execution/execute.js:190:10)     at /Users/democles/remux/node_modules/graphql/execution/execute.js:116:13     at execute (/Users/democles/remux/node_modules/graphql/execution/execute.js:115:10)     at /Users/democles/remux/node_modules/express-graphql/dist/index.js:169:37     at process._tickCallback (internal/process/next_tick.js:103:7)

Your call stack doesn't contain node-fetch, this is not our error.

url string will look messy :|. but okay works for me.

solved, i ended passing the params in the url. the previous error was an stupid promise in other function that somehoy affected the fetch promise.

Was also looking for this as it's possible in the official fetch impl:
https://fetch.spec.whatwg.org/#fetch-blob-example
var url = new URL("https://geo.example.org/api"),
    params = {lat:35.696233, long:139.570431}
Object.keys(params).forEach(key => url.searchParams.append(key, params[key]))
fetch(url).then(/* … */)

Which would save me from doing a cheap version of my own :)

@intellix, the URL class referenced in the spec is defined in the WHATWG URL Standard, which in Node.js is implemented as whatwg-url module, and as an experimental feature in Node.js 7.0.0. However, currently the latter does not yet implement the full specification (pending PR in nodejs/node#9484), so you'll have to use whatwg-url in order to achieve that currently.

Have a look at the built in node module querystring to easily create a query string from an object.

I was also looking for a way to specify GET query string parameters as an object and ended up creating a convenience wrapper to do it: node-fetch-json
The example above would look something like:
const url = 'https://graph.facebook.com/v2.6/me/messages';
const params = { access_token: PAGE_ACCESS_TOKEN };
fetchJson.get(url, params).then(console.log);

const url = 'https://graph.facebook.com/v2.6/me/messages?';
const params = new URLSearchParams({ access_token: 'PAGE_ACCESS_TOKEN' })
fetch(url + params).then(console.log);


Continue cannot be enabled in VSCode after installation

Before submitting your bug report

[X] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[X] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- OS:
- Continue:
- IDE:

Description
After installation, there is no button to enable the Continue extension in the active bar on the left side of the Vscode interface. Could you tell me what other way can I enable Continue?

To reproduce
No response
Log output
No response

@Iamgood01 It may be possible that Continue has been installed before or that for some other reason it is in the right side bar. To open the Continue panel, you can use the shortcut ctrl+L, or search the command palette (ctrl+shift+p) for Continue.
If neither of these work, there may be an error preventing Continue from loading. If this is the case, there will be more details in the dev tools console (cmd+shift+p, "Toggle Developer Tools") if you wouldn't mind sharing these

@sestinj You are right! The interface appears after I type  ctrl+L. Thank you! The startup icon is missing on either the left or the right side bar, but I noticed a prompt that popped up when I opened VSCode, and I wonder if it has something to do with the icon not coming out:


feat(runtime-core): unwrap nested refs on the component

The commit 6b10f0cd1da942c1d96746672b5f595df7d125b5 broke the auto-unwrap when the reference is in the middle of the object, added tests to cover some more cases.
  it('should expose return values to template render context', () => {
    const Comp = defineComponent({
      setup() {
        return {
          // ref should auto-unwrap
          ref: ref('foo'),
          // object exposed as-is
          object: reactive({ msg: 'bar' }),
          // primitive value exposed as-is
          value: 'baz',
          // nested ref at the end should auto-unwrap
          nested: { ref: ref('qux') },
          // ref with nested should auto-unwrap
          refNested: ref({
            nested: {
              ref: ref('foo')
            }
          }),
          // nested ref in the middle should auto-unwrap
          refMiddleNested: {
            nested: ref({
              msg: 'bar'
            })
          }
        }
      },
      render() {
        return `${this.ref} ${this.object.msg} ${this.value} ${
          this.nested.ref
        } ${this.refNested.nested.ref} ${this.refMiddleNested.nested.msg}`
      }
    })
    expect(renderToString(h(Comp))).toMatch(`foo bar baz qux foo bar`)
  })

@yyx990803 Just read the commit message from the commit I mentioned... Making this PR invalid, but also making the typescript typings (unwrapped ) invalid also.
On the example above the correct access to the properties is:
`${this.ref} ${this.object.msg} ${this.value} ${this.nested.ref.value} ${this.refNested.nested.ref} ${this.refMiddleNested.nested.value.msg}`

But the current recursive unwrap, makes the typescript throwing an error on that. The documentation is also misleading, it should be updated to reflect the new behaviour.

FYI this behavior will be reverted in the next release (back to implicit deep unwrap)


Docker image missing metafinder

Describe the bug
I followed the instructions from the readme to run reconftw's docker image:
sudo docker pull six2dez/reconftw:main
docker run -it --rm -v "${PWD}/OutputFolder/":'/reconftw/Recon/' six2dez/reconftw:main -d mydomain.com -r
The process stopped/failed at the following step:

Scanning metadata in public files
metafinder command failed

I checked the OutputFolder/mydomain.com/.log/ file. It shows:

Start 2023-10-20 00:43:14
[ 2023-10-20 00:43:14 ] Start function : domain_info
[ 2023-10-20 00:43:53 ] End function : domain_info
[ 2023-10-20 00:43:53 ] Start function : emails
./reconftw.sh: line 251: emailfinder: command not found
[ 2023-10-20 00:43:53 ] End function : emails
[ 2023-10-20 00:45:22 ] End function : google_dorks
[ 2023-10-20 00:45:22 ] Start function : github_dorks
[ 2023-10-20 01:18:35 ] End function : github_dorks
[ 2023-10-20 01:18:35 ] Start function : github_repos
[ 2023-10-20 01:18:35 ] End function : github_repos
[ 2023-10-20 01:18:35 ] Start function : metadata
./reconftw.sh: line 207: metafinder: command not found

To Reproduce
Steps to reproduce the behavior:

Run sudo docker pull six2dez/reconftw:main
Run docker run -it --rm -v "${PWD}/OutputFolder/":'/reconftw/Recon/' six2dez/reconftw:main -d mydomain.com -r
See error

Expected behavior
I would expect the docker image to contain all the toolings required to run the sample test script provided in the readme file, which uses -d and -r parameters.
Screenshots

Additional context

Docker Image hash: "sha256:faf781ba336689480700fbfb0b10679fb276b10578a4c140dbbddbd3a1cef5ea"
RepoDigest: "six2dez/reconftw@sha256:be27ccd805b513a281a93879358afb816142b226483e9aada5ce8dbad0d8145f"

Metafinder on Docker image is installed by running the installer, this could be some kind of temporal issue, could you rebuild the container and tell me the results please?

Same here...Container was rebuilt several times...

same issue using build via source non docker as well:

Managed to solve it by installing the missing metafinder command: pip3 install metafinder

I investigated a bit further and it seems the
pip3 install -I -r requirements.txt
fails due to
ERROR: Could not find a version that satisfies the requirement postleaksNeg (from versions: none) ERROR: No matching distribution found for postleaksNeg
This dependency was added recently - but it cannot be resolved.
I assume this results in many other python-tools not being installed as well.

Fixed requirements


Add a simple registry/hook/plugin mechanism to override templates for customization

Maybe implement a custom django template backend, its purpose is more or less exactly that: https://docs.djangoproject.com/en/3.1/howto/custom-template-backend/

No, this is a bad idea, too complicated, too much code for little to no gain. Functions which take a request parameter will be enough.

No, this is a bad idea, too complicated, too much code for little to no gain. Functions which take a request parameter will be enough.


Show deployments in Time Travel

Resolves https://github.com/weaveworks/service-ui/issues/2614 (affects only Scope in Weave Cloud).
Reuses a lot of code from <TimeTravelWrapper /> in Service UI - @foot and I had a discussion about converging those two, but that's a separate issue.
Other changes

Bump ui-components v0.4.82 -> v0.4.95
Use z-index: $layer-front on the whole header

It doesn't filter by namespace...

Related to https://github.com/weaveworks/service-ui/issues/1358.

...and the deployment ticks don't match the color of the deployment they represent

Not sure how to address that at the moment, but it should probably go into a separate issue :)


Text directly under the number?

Is there a way to have the text show up directly under the percentage amount inside the circle?  I tried textBelow, but it shows up outside the circle.

See first circle https://jsfiddle.net/9dajqcr1/28/


Assign color of the legend dynamically in js or css?

Hi,
How can I assign the color of the legend according to the color of the pie chart series? Right now its defined in CSS and hard coded, but what if the number of values increases then how I will know at which level i have to define the background-color in CSS?
I have searched it a lot, I have not even found the solution on any link.
Thanks.
Regards,
Nosheen Javed.

Data series are colored by CSS in Chartist. I am pretty sure there is no clean way to read those values in JS.
That means, your issue would be best solved by using custom CSS that applies to both series and corresponding legends. You'd probably end up implementing something like this.
http://codepen.io/chimmer/pen/aJvBgv

Mi 2 cents to this topic:
you pass the ID of the div section(where the graph goes painted), the chart object(put a return on the graph paint function) and an array of colors( in hex and as string '#000000').
function f_graph_custom_colors(nId, chart, Fcc) {
    var flagstone = true;
    var abc = ['a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z'];
    var estilo = '<style>';
    chart.on('draw', function (context) {
        if (context.type === 'bar' && flagstone ) {
            flagstone = false; // only enters this If once
            for (var i = 0; i < Fcc.length; i++) {
                estilo = estilo + nId + ' .ct-series.ct-series-' + abc[i] + ' line {stroke: ' + Fcc[i] + ';} ';
            }
            estilo = estilo + '</style>';
            $('head').append(estilo);
        }
    });
}

I think styling is not the concern of the plugin itself, hence closing this.


Suggestions for Colouring Tags

Hello, I love this MentionsTextInput component!
Now, I am facing the following challenge : I need to give a different font colour to the mention that the user has selected (like mentions on Facebook). For this I will have to split (or embed) the TextInput component. This seems quite difficult/complex.
Has anyone already given some thought on how to approach this?
Actually, what I need is Facebook's Draft.js capabilities on React Native. So far, this module is the thing that brought me the closest. Any suggestion on which direction to move forward, is highly appreciated!
Thank you.

@droegier any progress on this? Draft.js for react native would be fantastic.

This is not possible to do at the moment since you cant nest other components inside a textInput. Love to implement something like that though. Any suggestions are welcome.

Is it possibly use a regular text component to render the input's text?

Kinda feel like it would not be the ideal solution since we will have to manually calculate cursor positions.( eg: text tag will take more space than a text )

@harshq Not necessary since it's just the background color, take a look:

https://gist.github.com/scf4/012e9f615f6b43a1712a083b162afd94
The method for splitting words could be done better I'm sure.

Actually this becomes problematic when you want to scroll in the input. But perhaps there's a solution?

Actually this becomes problematic when you want to scroll in the input. But perhaps there's a solution?

Has this been addressed? There seems to be something off when trying to scroll the text box from the Android platform. The keyboard glitches and disappears all the time. Even if the '@' has been entered or not.


Allowing symfony/yaml 3.0.

Fixes #121

@juliangut Can you take a look?

@bshaffer Can you also take a look?

Looks good to me

make that change to all symfony repos, not only yaml one.
set one travis job to test 2.x line, second to test 3.x line.
Done

set one travis job to test 2.x line, second to test 3.x line. Or you will again have problems with deps that on lower deps your code won't work.

This is already done thanks to you! PHP 5.3.3 build is using 2.x line and PHP 7.0 build is using 3.x line.
Please take another look.
BTW, @keradus, your expertise adds great value on this project. I'd like to propose that you become one of the collaborators of this project, if you're interested. What do you think?

LGTM

@bshaffer Thanks. I'm going to merge this. @keradus Let me know if you have any concerns though.

@keradus Let me know if you have any concerns though.

Please wait for merging if you are asking for feedback. Feedback, in general, is good thing. No need to merge PRs in rush.

BTW, @keradus, your expertise adds great value on this project. I'd like to propose that you become one of the collaborators of this project, if you're interested. What do you think?

I have nothing against that. You know my mail, feel free to contact ;)

@keradus

Please wait for merging if you are asking for feedback. Feedback, in general, is good thing. No need to merge PRs in rush.

Gotcha.

I have nothing against that. You know my mail, feel free to contact ;)

Good, I'll ask @satooshi about it.

@keradus Added you to collaborators. Thanks :+1:

Thanks for mail contact to discuss it first ;|


How to add new page linked by the navigation bar

Is there a way to add new pages such as full pub list or blogs and be linked by the navigation bar at the top?

You can easily add something in about.md, just like "Blogs" and then add a new title "Blogs" and its url in the format of existing others in _data/navigation

You can add something new in about.md like blogs, and then add a new title "Blogs" and its url in _data/navigation in the format of existing others.


Unable to find file<EMAIL_ADDRESS>

I have not changed anything in my config and suddently installing, it says:
[InvalidArgumentException]
Unable to find file<EMAIL_ADDRESS>
March 12th, I deployed and it installed perfectly. I just do a deploy today and it breaks.
It's a recurrent problem with this bundle that it just "breaks" from time to time probably because a lack of "locking" the versions.
the required version in my composer.json is
"avanzu/admin-theme-bundle": "^1.3.12",

In the lock, I see this:
"version": "1.3.x-dev",

Log
php bin/console avanzu:admin:build-assets --env=prod
22:28:34 ERROR     [console] Error thrown while running command "avanzu:admin:build-assets --env=prod". Message: "Unable to find file<EMAIL_ADDRESS>["error" => InvalidArgumentException { …},"command" => "avanzu:admin:build-assets --env=prod","message" => "Unable to find file<EMAIL_ADDRESS>[]

  [InvalidArgumentException]
  Unable to find file<EMAIL_ADDRESS>
avanzu:admin:build-assets [-c|--compress] [-m|--mangle] [--uglifyjs-bin [UGLIFYJS-BIN]] [--uglifycss-bin [UGLIFYCSS-BIN]] [-h|--help] [-q|--quiet] [-v|vv|vvv|--verbose] [-V|--version] [--ansi] [--no-ansi] [-n|--no-interaction] [-e|--env ENV] [--no-debug] [--] <command>

Debug info

Component
Version

Symfony version
Symfony 3.3.10

AdminThemeBundle
^1.3.12 (composer.json), 1.3.x-dev (composer.lock)

Operating System
Linux xxxx 4.4.0-89-generic #112-Ubuntu SMP Mon Jul 31 19:38:41 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

The problem is a major-bump change in the versioning of fullcalendar.
in my-nice-project/vendor/avanzu/admin-theme-bundle/Resources/public/vendor/fullcalendar/CHANGELOG.md I can see that the properly working version starts with this:
v3.10.0 (2019-01-10)

while the failing installation downloaded a version whose CHANGELOG.md starts with:
v4.0.1 (2019-03-18)

So we need to lock-down the installed version to 3.10.0 to ensure that things do not randomly break.
 It is a pitty that this amazing AdminThemeBundle is failing every each other week because the installation is not deterministic.
If we correct package by package as they break we will not be preveting this to happen again, if package A does not fail it will fail package B. We should find a way to ensure that installation is always sure. Otherwise it completely destroys the confidence in the next deploy even if you just changed a single line in your code.
Who should decide on this?

@xmontero thanks for your report. Unfortunally the old 1.3.x series has in bower.json a lot versions with "*" and with the time it gets rusty. In new series releases it will be more adjusted for avoid BC. Closing as fixed with 1.3.13

I'm going to test now, but taking advantage we are on this, and both online, I'll try to discover also the proper version for bootbox. I had to tune my deployer to make symlinks to make it work also due to versioning...

Ah well... taking this "Feel free to make a PR against 1.3.x branch if you know every locking dependency stable" invitation, I'll do it tomorrow (2am here in Spain and tomorrow I'll have to be at a client's office soon).
Thanks for the current correction. I'll try to lock all them and do a PR. Thanks for that.

PR at #252 and tested locally that it perfectly installs.
To do so, I've done an installation from the v1.3.13 and checked all the versions that were installed.
Then I replaced all the versions to te specific numbers. Even for those that previously they were'nt * but ranges, I opted to lock them down to specific patch version so in theory the risk of breaking an installation is nearly zero.
The only risk is if any of the locked versions in turn depend on other packages and also they did their dependencies badly. We'll assume this does not happen or if so, there are large communities there taking care of all those javascript packages, we'll care only of our loved Avanzu bundle.
I also locked down the AdminLte itself to a concrete version. This is on purpose.

From now on, (if merged) this 1.3.x branch will always have to update those files if the main developers require a specific new feature released in future versions.


Add to calendar functionality update

Add to calendar should have the additional functionality to have the date read by screenreader – like “Add  to calendar” “Add  to calendar”. “Add <date/title> to calendar”. Please suggest the best way to solve it.
Issue flagged in Siteimprove that we have Links in the same context with the same text alternative. https://my2.siteimprove.com/Accessibility/550893/NextGen/Issue/1?ruleId=81&pageSegments=&issueKind=1&exceptTags=1,2&conformance=&siteTarget.conformanceLevels=0,1,3

@meols  Added a textsnippet under "Events" for the aria label which is used on the button.

@meols  This is applicable to 'Add to calendar' button on the event page and event promotion cards. When tabbing through this button, screen reader reads the button as 'Add [event title] to calendar' .
Sample links:
https://web-global-development-equinor-web-sites-dev.c2.radix.equinor.com/test/padms
https://web-global-development-equinor-web-sites-dev.c2.radix.equinor.com/event/q3-2022
aria label is added to the button
Private Zenhub Image

@meols Removed quotes from the label

@padms - shouldn't the marked code read Add This is an event with 60 characters to calendar?
If {eventTitle} is read out load by a screen reader, it doesn't identify the event.
Private Zenhub Image

@meols We just need to update the text snippet. It shows correct text now.

@meols Yes, This is how it looks after removing quotes from the text snippet. Private Zenhub Image
Can be testes on the same page now..

Thanks @padms - not sure how to test this, but I trust you - so this is approved :smile:


Php7.2

Bad pull request but php7.2 is working !

The 7.2 fix works great for me!
The other changes at the search function i didn't checked


Update docs > Environment Variables for Now

Tip on using environment variables when deploying to Now. Discussed in #96.

Thanks @bearcanrun!


backend: Add e2e test for OIDC auth mode

This patch adds end to end test for OIDC auth mode.
Signed-off-by: Santhosh Nagaraj S<EMAIL_ADDRESS>

I think the CI is failing because of golang version mismatch,Ill fix this. I have raised an issue for the same https://github.com/kinvolk/nebraska/pull/559.

No, there's nothing listening on 6000. I'll try and debug it.

Ok, quite a bit of messing about, and found a few things but not any fixes yet...

Seems we now have some sort of build issue with docker images not being rebuilt when things change. After pruning my containers and images I got things working in the last commit on proof-of-concept3 branch. I see this in the proof-of-concept3 branch, but haven't gone back to see when the issue was introduced yet.
However, even after pruning, I can't get this branch to pass tests (with make check or make check-backend-with-container).

Make check needs the skipping via env added.
For testing with the container, seems a server is staying open after a test... which causes the next test to fail because the port is open(I think). Maybe a bit more error checking/logging and deferring the shutdown so it runs on a test failure would help here.

Oh got the reason for the docker images not building. I think we should add --build tag. I'll look into the the make check and the server not shutting down properly.

@yolossn , once the final touches are done, and since you've mentioned it now also works for you, I think the commits should be squashed together onto your original one and merged.


Refactor the User listing in "Admin" menu with respect to the latest roles implementation

when I changed Maggie in the Admin menu to superuser, she (who is a viewer in keycloak) completely lost access to the system

@Connor1892 I tried exactly the same in dev environment and I could see maggie has now SuperUser  role assigned. We should sit together to test this.


Top Widgets: the main focus is the number (median speed) rather than the chart

@smithkelseyj is this still an issue? The top widgets reflect the wireframes directly now (I think the updates may not have been merged when this issue was opened) so I'd like to close it unless there's changes to be made!

I can check but I don't remember how to log in to MLBN. Could you send me those details?

Yup! (http://viz.measuringbroadband.org:3000/)[http://viz.measuringbroadband.org:3000/admin]
I'll message you the user and pw on slack

As far as I can remember, looks good!

Thanks! Closing then


Clean up APIs for people, stream_data, etc.

This may be helpful for PR #13914 in particular.

Looks great, merged, thanks @showell!


Translations for autofill's new auth flow

Task/Issue URL: https://app.asana.com/0/488551667048375/1203508506791934/f
Description
Translations for autofill empty list placeholder text
Steps to test this PR

[ ] Visit credential management screen when there are no saved logins
[ ] Verify text is correct

Current dependencies on/for this PR:

develop

PR #2608

PR #2612   👈

This comment was auto-generated by Graphite.

Closing in favor of https://github.com/duckduckgo/Android/pull/2632


The list view makes queries that are in the form function

Package
filament/filament
Package Version
2.16.33
Laravel Version
9.35.1
Livewire Version
No response
PHP Version
8.1.11
Problem description
When viewing the list page of a resource, debugbar shows queries that aren't in the table function, but on the form function.
For example, if the form has a Select and a query is made to define the select options, the query is performed in the list view
Expected behavior
No form queries in the list view
Steps to reproduce
Install and seed the repository.
Seed will create an admin user<EMAIL_ADDRESS>with pass secret
Navigate to activities list -> /admin/activities
Check the queries in debugbar
Backtrace points to /vendor/filament/filament/src/Resources/Pages/ListRecords.php line 257
Reproduction repository
https://github.com/buzkall/double
Relevant log output
No response

Retrieving select options from the database should be wrapped in a closure to prevent it from being called multiple times internally:
Select::make('tags')
    ->options(fn (): array => Tag::getOptions(TagTypeEnum::content_type))

And this has been fixed in v3, so you don't need to use the callback then 👍


devcam: better error when in wrong pwd

This isn't helpful:
$ devcam
2015/03/30 18:21:26 open dev/devcam: no such file or directory

It should say it only works in the src/camlistore.org directory.

https://camlistore-review.googlesource.com/4966


Cheetah-Software install issue

I tried to install cheetah-software.
LCM, QT5.12.0, eigen3 were all successfully installed.
when i tried build, there was some issue.
first, when i tried cmake ..
contain a literal, direct call to the project() command.  Add a line of
code such as
project(ProjectName)

near the top of the file, but after cmake_minimum_required().
CMake is pretending there is a "project(Project)" command on the first
line.
This warning is for project developers.  Use -Wno-dev to suppress it.
But I thought this warning was negligible.
and then when i tried ./../scripts/make_types.sh
another problem was occurred
rm: cannot remove '/.java': No such file or directory
rm: cannot remove '/.hpp': No such file or directory
rm: cannot remove '/.class': No such file or directory
rm: cannot remove '/.py': No such file or directory
rm: cannot remove '/.pyc': No such file or directory
Notice: enclosing LCM types without package into java namespace 'lcmtypes'.
cp: cannot stat '/usr/local/share/java/lcm.jar': No such file or directory
./../scripts/make_types.sh: line 21: javac: command not found
./../scripts/make_types.sh: line 22: jar: command not found
mv: cannot stat 'my_types.jar': No such file or directory
mv: cannot stat 'lcm.jar': No such file or directory
ls: cannot access '/.class': No such file or directory
but in this guide https://www.programmersought.com/article/75147245766/
He says there is no problem with this error.
Finally when i tried make -j4
Every time I tried this command again, new errors occurred.

first trial
/usr/include/x86_64-linux-gnu/bits/string_fortified.h:106:34: error: ‘char* __builtin_strncpy(char*, const char*, long unsigned int)’ specified bound 2056 equals destination size [-Werror=stringop-truncation]
106 |   return __builtin___strncpy_chk (__dest, __src, __len, __bos (__dest));
|          ^~~~~~~~~~~~~~

[ 70%] Built target test-common
make: *** [Makefile:130: all] Error 2

second trial (Run command again without changing anything)
/home/nao-uncle/Cheetah-Software/robot/src/rt/rt_serial.cpp:34:3: error: ‘ioctl’ was not declared in this scope
34 |   ioctl(fd, TCGETS2, &tty);
|   ^~~~~

[ 80%] Linking CXX shared library libWBC_Ctrl.so
[ 80%] Built target WBC_Ctrl
make: *** [Makefile:130: all] Error 2

The above two errors continue to be printed.

how can i fix it?
PS) i tried to fix ioctl problem, so i put include in /robot/src/rt/ rt_serial.cpp
when i put  include<ioctl.h>
fatal error: ioctl.h: No such file or directory
14 | #include <ioctl.h>
when i put include<sys/ioctl.h>
from /home/nao-uncle/Cheetah-Software/robot/src/rt/rt_serial.cpp:17:
/usr/include/asm-generic/termios.h:15:8: error: redefinition of ‘struct winsize’
15 | struct winsize {
|        ^~~~~~~
In file included from /usr/include/x86_64-linux-gnu/sys/ioctl.h:29,
from /home/nao-uncle/Cheetah-Software/robot/src/rt/rt_serial.cpp:14:
/usr/include/x86_64-linux-gnu/bits/ioctl-types.h:27:8: note: previous definition of ‘struct winsize’
27 | struct winsize
|        ^~~~~~~
In file included from /usr/include/x86_64-linux-gnu/asm/termios.h:1,
from /home/nao-uncle/Cheetah-Software/robot/src/rt/rt_serial.cpp:17:
/usr/include/asm-generic/termios.h:23:8: error: redefinition of ‘struct termio’
23 | struct termio {
|        ^~~~~~
In file included from /usr/include/x86_64-linux-gnu/sys/ioctl.h:29,
from /home/nao-uncle/Cheetah-Software/robot/src/rt/rt_serial.cpp:14:
/usr/include/x86_64-linux-gnu/bits/ioctl-types.h:36:8: note: previous definition of ‘struct termio’
36 | struct termio
|        ^~~~~~
tada~ a new issue T_T

I had a lot of similar problems as well. When I followed the instructions closely and built on 18.04 LTS it worked. Building on any other version did not work for me and gave similar errors.
Another user in the SpotMicroAI discord helped me out with instructions he used on his Jetson:
---follow normal instruction stuff:
get qt5 to ~/Qt
get qtgamepad
compile qt5 (not enough memory to multithread compile so do single thread and come back to it later)
sudo apt install libeigen3-dev liblcm-dev liblcm1 liblcm-java
clone cheetah-software
set path to QT in cheetah cmakelists
-- weird jetson stuff:
*** hack: turn off sim and rctest in cheetah cmakelists - i couldn't get qtgamepad working right, this should be figured out and removed. -
modify common/cmakelists and robot/cmakelists and user/cmakelists to correct usr/include local path for lcm and eigen3
add https://raw.githubusercontent.com/jratcliff63367/sse2neon/master/SSE2NEON.h as 'immintrin.h' in /usr/include for ARM NEON intrinsics
edit immintrin.h:1678 (very end) to remove the name 'p' from parameter list to avoid the warning
-- end weird jetson stuff
-- continue normal cheetah instructions
make build folder in cheetah
../scripts/make_types.sh
cmake ..
make
don't use -j4, it runs out of memory
Hope that helps you

I also had the same error and figured out the problem was on Java compiler that is used for LCM. When I installed LCM, I used “openjdk-8-jdk” instead of  "default jdk." The following websites helps you to use "openjdk-8-jdk."
https://docs.datastax.com/en/jdk-install/doc/jdk-install/installOpenJdkDeb.html
https://github.com/lcm-proj/lcm/issues/241


Update 003-hilbert-curve.md

Community Contribution
Another Hilbert curve for the collection.
The page and the source
Sharing

[x] I would be happy to have my project shared on The Coding Train's social media!

Great user interface! Thanks for sharing


fix: Avoid hitting etcd limit for workflow pod with large parameters

Fixes https://github.com/argoproj/argo-workflows/issues/8790.
Signed-off-by: Yuan Tang<EMAIL_ADDRESS>

🙏

added a comment to the original issue on how we're relying on the ARGO_TEMPLATE inputs

@terrytangyuan could we have an env flag to enable omitting this?

would line 325 break as it expects inputs?
pod, err = substitutePodParams(pod, woc.globalParams, tmpl)

There are certainly things breaking because of this change. The tests were not passing. If someone wants to pick this up, feel free to do so.


Fixes Tooltips Showing Outside Minimum Time Bounds [redo]

A redo of PR #5287 to add updates from master. This fixes #5228 by checking if x is outside the minimum border of the graph then allowing no tooltips to be created if so.

@simonbrunel @MPierre9 a test would be great. Not sure how easy one is to implement

@MPierre9 can you add a test so that we can merge this change?

@MPierre9 would you be able to add a test? thanks!

@MPierre9 I have marked this PR as stale since we haven't gotten any response from you in quite awhile. Please let us know if you plan to add a test. If you don't respond this PR will be closed as inactive

I would be interested in writing this test case. Could you point me in the direction of documentation on how to write such a test case?

There's some documentation here: https://www.chartjs.org/docs/latest/developers/contributing.html#building-and-testing
There are some existing tests in test/specs/plugin.tooltip.tests.js


Support async calls in QosIoExceptionAwareCall

My only thing with this stuff is that I think the place you're meant to do this kind of retry logic is in an interceptor - the effect is that you e.g. don't need to write custom code for async. What was the reason for this?

My understanding of the OkHttp flow is that in the context of an interceptor I'm trapped in the thread that's currently executing the request. For async calls, this means that N calls that are waiting for a later retry are consuming all N OkHttp threads. Hence the circumnavigation. I don't particularly like it and would love to hear about better alternatives, but I currently don't know one.

Moved tests around a bit and added end-to-end tests for Retrofit (sync and async) and Feign clients, verifying that 503 get retried.

Rob, I don't quite get it. With this implementation, aren't you blocking two threads?

Ah, I figured it out :) It's because if you're using a ScheduledExecutorService you're not just doing the equivalent of like, Thread.sleep.

Yep. The construction is a bit crazy ("scheduled recursion"?), bit I think it should be OK? Would love to do something simpler if you have a suggestion...
Sent from Email+ secured by MobileIron
-------- Original Message --------
From: James Baker <notifications<EMAIL_ADDRESS>Date: Wed, Sep 6, 2017, 1:38 AM
To: palantir/http-remoting <http-remoting<EMAIL_ADDRESS>CC: Robert Fink <rfink@palantir.commailto:rfink@palantir.com>,State change <state_change<EMAIL_ADDRESS>Subject: Re: [palantir/http-remoting] Support async calls in QosIoExceptionAwareCall (#533)
Ah, I figured it out :) It's because if you're using a ScheduledExecutorService you're not just doing the equivalent of like, Thread.sleep.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub[github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_palantir_http-2Dremoting_pull_533-23issuecomment-2D327332831&d=DwMCaQ&c=izlc9mHr637UR4lpLEZLFFS3Vn2UXBrZ4tFb6oOnmz8&r=UfcWCaR4ui50AFap-gezrx5XYtPtH-9JpazU7tbRW-4&m=oSi6ByUU-lp8nbGGhioMOUbXGYPHNMcK-qpQcEUFfTA&s=-YlxWnEdBXzT7zmvrGRzQCZJuipYQF5J_XSsEn_zhO4&e=, or mute the thread[github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AGOdwcTYrN9yz3T26PZOW1hrYNe-2D6EsHks5sfdtVgaJpZM4PLQyT&d=DwMCaQ&c=izlc9mHr637UR4lpLEZLFFS3Vn2UXBrZ4tFb6oOnmz8&r=UfcWCaR4ui50AFap-gezrx5XYtPtH-9JpazU7tbRW-4&m=oSi6ByUU-lp8nbGGhioMOUbXGYPHNMcK-qpQcEUFfTA&s=mRI6XN21-dr8_DZDOIY4rhlJwdBlAANivTplE5rVDxQ&e=.

Scheduled callback to the same function makes sense to me FWIW

Though, as I mentioned before, it seems weird to have a thread sleeping + doing nothing

Sorry, which thread exactly?
Sent from Email+ secured by MobileIron
-------- Original Message --------
From: Alice Purcell <notifications<EMAIL_ADDRESS>Date: Wed, Sep 6, 2017, 12:48 PM
To: palantir/http-remoting <http-remoting<EMAIL_ADDRESS>CC: Robert Fink <rfink@palantir.commailto:rfink@palantir.com>,State change <state_change<EMAIL_ADDRESS>Subject: Re: [palantir/http-remoting] Support async calls in QosIoExceptionAwareCall (#533)
Though, as I mentioned before, it seems weird to have a thread sleeping + doing nothing
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub[github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_palantir_http-2Dremoting_pull_533-23issuecomment-2D327459098&d=DwMCaQ&c=izlc9mHr637UR4lpLEZLFFS3Vn2UXBrZ4tFb6oOnmz8&r=UfcWCaR4ui50AFap-gezrx5XYtPtH-9JpazU7tbRW-4&m=zXqwVGQSexjYIjxDrB9WL04Vggq_hjbUu9S9YVvwFQg&s=crIBlpx8N5_4h3SYe-7MQU2OYtYdQHp3AzWDOlcA34s&e=, or mute the thread[github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AGOdwcZEFhqpT3Xl3tGFrRgbzXZjFFaTks5sfoahgaJpZM4PLQyT&d=DwMCaQ&c=izlc9mHr637UR4lpLEZLFFS3Vn2UXBrZ4tFb6oOnmz8&r=UfcWCaR4ui50AFap-gezrx5XYtPtH-9JpazU7tbRW-4&m=zXqwVGQSexjYIjxDrB9WL04Vggq_hjbUu9S9YVvwFQg&s=iTKgSs4OcFkWSX2H4FWR476E2xLAx-mTdZbdPFcgGEc&e=.

https://github.com/palantir/http-remoting/blob/9bba3ac5d8e24ec20b2687a70beee08bbc980c40/okhttp-clients/src/main/java/com/palantir/remoting3/okhttp/QosIoExceptionAwareCall.java#L45

But this is unavoidable -- it's a synchronous call that's coming in, we'll have to block a thread somewhere, no?

Sure, but that thread could handle all the sleeping + retrying etc rather than a separate executor pool.


Error: Send SMH No handler registered for W_M_SEND_TX

Describe the bug
per title
Environment

Smapp version: latest
Node version: latest
OS: Windows > 10 | macOS High Sierra > (10.13) | Ubuntu > 20.04

Steps to reproduce

u have some SMH on your wallet
create a transaction
click SEND on preview screen
see error

Example

I reproduced this error.
Environment:
Smapp version: v0.2.10-pr.1040
Node version: v0.2.19-beta.0
OS: Windows 10 Home
Steps to reproduce the same:

u have some SMH on your wallet
Account already spawned
create a transaction (the very first SEND transaction from this account)
click SEND on preview screen
see error

Tried all the possible combinations of the transaction criteria.
The receiver was loosing the connection to the node every now and then, syncing while trying to execute the transaction.
Same error after the receiver was all synced.
Same error while trying to send SMH to myself.
Didn't find anything in logs.
After restart of the application the transaction was successful at the firt attempt, but it was doubled (!)
There are two identical records in the transaction log, the wallet was charged twice and the receiver got 2 identical transactions with doubled amount of SMHs.

Tried to reproduce on:
Environment:
Smapp version: v0.2.10-pr.1040
Node version: v0.2.19-beta.0
OS: macOS Ventura Version 13.0.1 (22A400)
The transaction successfully sent at the first attempt , no issues whatsoever. No error messages, no problems with the amouts etc.

The second SEND transaction from Windows was correct. Nothing got doubled, it was executed at the first attemt, nice and smooth.

@monikasmolarek pls, if you won't be able to reproduce this bug on the new Smapp version — close the issue until we'll catch it again

Closing as recommended. The error did not show up in the latest release; I also checked Sentry and Discord, no reports for this specific issue.

@monikasmolarek , @brusherru , last one 53 min ago https://sentry.io/organizations/spacemesh/issues/3255922679/?project=6324919&query=is%3Aunresolved&referrer=issue-stream
I will create and issue on this because all our not registered handlers are based on 1 problem with not updated the Smeshing / Node screen and registered handlers for our Electron app, which should be in the preload file for the Electron. I did research on it and will create 2 issues for our handlers and issues with them.


feat(sca): add empty scan commands

Codecov Report

Merging #571 (0614d57) into main-sca (e69b742) will decrease coverage by 0.02%.
The diff coverage is 92.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff              @@
##           main-sca     #571      +/-   ##
============================================
- Coverage     93.81%   93.80%   -0.02%
============================================
  Files            84       86       +2
  Lines          3883     3908      +25
============================================
+ Hits           3643     3666      +23
- Misses          240      242       +2

Flag
Coverage Δ

unittests
93.80% <92.00%> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

ggshield/cmd/sca/scan.py
88.23% <88.23%> (ø)

ggshield/cmd/main.py
91.17% <100.00%> (+0.13%)
:arrow_up:

ggshield/cmd/sca/__init__.py
100.00% <100.00%> (ø)


Delete Branch: sphinx-documentation ?

@Drake81 Is this branch still active ?

https://github.com/DLR-SC/prov-db-connector/tree/sphinx-documentation

If not, we should delete the branch ;)

Branch deleted


open: ignore case for name globbing

If I have for example this
[opener]
browser = [
	{ exec = 'chromium "$@"', display_name = "Open in Chromium" },
]

[open]
rules = [
	{ name = "*.html", use = ["browser"] },
]

and want to open a file called test.HTML it won't work. But it will work for test.html. Is there an easy way around that?
Edit: it seems to also apply to the theme icons.

*.[hH][tT][mM][lL] looks silly but works... maybe I can add case insensitivity if needed.

Hi, I just noticed that *.@(html|HTML) works for your case.
I wanted to know if you're still interested in adding a case-insensitive option. If so, I will add it now.

Sorry, ignore what I said below. The Rust glob library doesn't support the @() syntax. I will add it now :)

*.@(html|HTML)

Doesn't seem to work for me.

I wanted to know if you're still interested in adding a case-insensitive option. If so, I will add it now.

Yes, I'm still interested. But I don't see a good way to add it for the theme icons.