time: create ticker with instant first tick - rebooted

I have the same issue as https://github.com/golang/go/issues/17601, but the issue was closed and locked with this reason:

This doesn't seem to rise to the level of new API. You don't have to "copy the section of code underneath the first <-t.C", you just have to put the code in its own function and call that function once at the start and once after each timeout interval. That seems clear enough.

However, "call the function once at the start" does not work when you are not in control of when the function is called.
Consider this clumsy implementation of a rate limiting in rateLimited, vs. the much more concise and elegant one in rateLimitedImmediate:

package main

import (
	"fmt"
	"sync"
	"time"
)

var interval = time.Second

type rateLimited struct {
	rateLimiter <-chan time.Time
	f           func() string
}

func (r *rateLimited) do() string {
	<-r.rateLimiter
	return r.f()
}

type rateLimitedImmediate struct {
	mu          sync.Mutex
	rateLimiter <-chan time.Time
	f           func() string
}

func (r *rateLimitedImmediate) do() string {
	r.mu.Lock()
	defer r.mu.Unlock()
	defer func() { r.rateLimiter = time.After(interval) }()
	<-r.rateLimiter
	return r.f()
}

func main() {
	r := &rateLimited{
		rateLimiter: time.Tick(interval),
		f:           func() string { return "result" },
	}

	fmt.Println(r.do())
	fmt.Println(r.do())
	fmt.Println(r.do())

	r2 := &rateLimitedImmediate{
		rateLimiter: time.After(0), // start immediately
		f:           func() string { return "result" },
	}

	fmt.Println(r2.do())
	fmt.Println(r2.do())
	fmt.Println(r2.do())
}

Just that the nice implementation in rateLimited cannot start immediately.
I am not in control of calling the function myself, the library client does that.
So I second the need for a stdlib solution like proposed in the original issue, and hope the issue can be re-opened (or discussed here).

in this specific case
package main

import (
	"context"
	"fmt"
	"time"

	"golang.org/x/time/rate"
)

var interval = time.Second

type limited struct {
	l *rate.Limiter
	f func() string
}

func (l *limited) do() string {
	l.l.Wait(context.Background())
	return l.f()
}

func main() {
	r3 := limited{
		l: rate.NewLimiter(rate.Every(interval), 1),
		f: func() string { return "result" },
	}
	fmt.Println(r3.do())
	fmt.Println(r3.do())
	fmt.Println(r3.do())
}

It seems to me that the rate.NewLimiter function is going to need to have some way to choose between waiting for an interval before calling the function, or calling the function immediately.  And you can write this without needing any change to the time package.


React-Redux bad usage of peerDependencies

What version of React, ReactDOM/React Native, Redux, and React Redux are you using?

React: 16.14.0
ReactDOM/React Native: 16.14.0
Redux: 4.1.2
React Redux: 7.2.6

What is the current behavior?
while installing react-redux without react-dom make the usage of the UMD impossible.
The UMD needs react-dom aka ReactDOM global setup.
Our build toolchain leverage the peerDependencies field of the package.json of react-redux and because it is badly setup it do not see the dependencies between both.
In the end we have :
Uncaught TypeError: reactDom is undefined

because our toolchain place reactdom script tag way after react-redux.
Looking at the code inside the UMD shows you have to provide react-dom to make it work.
<EMAIL_ADDRESS>  setBatch(reactDom.unstable_batchedUpdates);

there is not conditional on reactDom is here.
peerDependencies
Reading official docs https://docs.npmjs.com/cli/v8/configuring-npm/package-json#peerdependenciesmeta show peerDependenciesMeta field should be used for declared peerDependencies.
<EMAIL_ADDRESS>We can see react-dom is only in peerDependenciesMeta.
What is the expected behavior?
Please add react-dom and react-native as peerDependencies in the package.json 🙏
It would let the user choose between both and our toolchain able to understand react-dom is a peerDependencies.
I will be able to push the PR later on this week I think.
Which browser and OS are affected by this issue?
all
Did this work in previous versions of React Redux?

[ ] Yes

We've had a bunch of discussions on this:

#1261
#1282
#1371
#1366
#1424
#1425
#1559

The main result is we can't put them into peerDeps yet because many people still use older versions of npm (or other installers) that don't support peerDepsMeta and cause lots of warnings for them. This is the best solution for now.

That said, it helps to know that you are using the UMD build. I'd been debating whether to remove that in React-Redux v8.


Allow users to edit their own links only

Hi, I was wondering if there is a way to allow users to edit their own links and only their own links.
Regards,

Out of the box, no. In the future, maybe. Meanwhile, maybe check the plugins.


Rendering of Curves

I'm curious why the decision in how to render a given curve wasn't left to the Form (all curves are passed into the Form as a line group). For example, since bezier drawing is available natively on both canvas and svg it could be optionally used as an alternative rendering choice in those cases.

Agreed. Current implementation is more general to support multiple types of curves (cardinal, b-spline, bezier, etc), but since we should have a special case for bezier since there's native support. Especially important for svg.


feat: resolver rebased

Description of change
This pull request includes several significant changes aimed at updating the resolver to the latest rebased IOTA.
Changes include:

Updating all the dependencies
Refactor for the tests

Links to any relevant issues

Type of change

Bug fix (a non-breaking change which fixes an issue)
[x] Enhancement (a non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Documentation Fix

How the change has been tested

Change checklist

[x] I have followed the contribution guidelines for this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have checked that new and existing unit tests pass locally with my changes
[ ] I have updated the CHANGELOG.md, if my changes are significant enough

@itsyaasir good job! Do you mind waiting a bit before merging this? RN we are reworking a bit DID resolution on Rebased and I feel these changes should account for that as well.


Updated tuner

Update our hyperparameter tuner and added some more documentation.
Also implemented a more naive grid search so we can easily compare the implementations.
The new hyperparameters for OR-tools are chosen according to issue #138. However, for some parameters no upper bound is given by the sat_parameters.proto file... (i.e., for example cp_model_probing_level).

Nice, thanks!


fix podman -v regression

Re-add the shortflag for --version and add e2e tests to avoid regressing
in the future.
Signed-off-by: Valentin Rothberg<EMAIL_ADDRESS>

CI is sick atm, see https://github.com/containers/libpod/pull/3647

I think that we had to remove this because Cobra really, strongly disliked having -podman run -v src:dst:options and podman -v both be possible... Might be misremembering, though

[+2466s] panic: unable to redefine 'v' shorthand in "rm" flagset: it's already used for "volumes" flag
Yep, I think that's our culprit here.

@mheon you are right

Seems like a bug in cobra, but not something we can easily fix.

I read a bit through the docs and found a solution. The issue was that --version was marked as a persistent flag, so it was propagated to all subcommands. Making it a local flag solved the issue.
This comes at the cost of ./bin/podman --version run alpine ls really executing a container and the --version being ignored. Personally I prefer the new behaviour.

All kinds of test unhappiness @vrothberg

All kinds of test unhappiness @vrothberg

That was a brain fart in the tests I added. s/BeFalse/BeTrue will fix it :)

Huh. Nice to know we can work around issues like this.
Code LGTM

LGTM but would like a head nod from @baude

This makes sense to me.
Since podman run -v ... continues to work.
/lgtm


Setting options in react

Hi,
does anyone know how I can set pageOptions when using pace-js with react?

Just add a script tag in your index.html before the script:
paceOptions {
 //your options
}


Feature Request: M5Atom Support

What would it take to get M5Atom/M5Atom Lite support added to platformio?  I'd like to use ESPHome to compile firmware for M5Atoms to use with Home Assistant. ESPHome uses PlatformIO and PlatformIO links back to git-hub repos for the various devices that are supported, so it appears that support has to start here?  Please let me know if I am looking in the wrong place to get M5Atom support for PlatformIO.

For people like me who ended up here looking for how to use platformio with the atom, looking at the M5 docs  you can see that the Arduino IDE board is the same as the M5Stick.
This example config works:
[env:atom]
platform = espressif32
board = m5stick-c
framework = arduino
upload_speed = 1500000
monitor_speed = 115200

lib_deps =
<EMAIL_ADDRESS><EMAIL_ADDRESS>

Hello, we opened a new topic for "wish list" in forum.m5stack.com feel free to ask your wish over there and we will push it forward :)

Hello, we opened a new topic for "wish list" in forum.m5stack.com feel free to ask your wish over there and we will push it forward :)


Dataflow API cleanup (dynamic/configurable channel count)

Update the eth dataflow API to only support the minimum number of fixed ethernet channels (2) and in place of the other 6 channels, provide APIs (host and device side), to support an arbitrary number of channels (up to available L1 space, if so desired).
This will give more flexibility to ops like all-gather to instantiate more, smaller channel buffers with more workers for cases where we need lower latency. It will also let us sweep more combinations for perf characterization.

@aliuTT . Included you here as you are an owner. Feel free to unassign yourself.

no longer relevant as EDM has been updated to support only require a single erisc info channel


what is meant by the comment below in the sample

I'm attempting to build a complex application and the 1st step is to respond to return messages using vuejs controls, but I get the following error when I try to run my vuejs method:
Failed to match method call Microsoft.WebTools.BrowserLink.LivePreview.LivePreviewBrowserUIExtensionFactory.themeChanged
Then I noticed on the documentation on the client side code example there is a message on the
ReturnMessage callback:
We can assign user-supplied strings to an element's textContent because it
is not interpreted as markup. If you're assigning in any other way, you
should be aware of possible script injection concerns.
This is the code I'm using
connection.on("ReceiveMessage", function (user, message) {
        // We can assign user-supplied strings to an element's textContent because it
        // is not interpreted as markup. If you're assigning in any other way, you
        // should be aware of possible script injection concerns.
        //console.log(user + " " + message);
        gamePlay.messageUsers(user + " " + message); //returns error
    });

The line of code: gamePlay.messageUsers(user + " " + message); calls a vuejs method

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

ID: 0dd2991d-c22d-ad0f-80b6-bd1b503db8f3
Version Independent ID: 5e3fbe1d-c97f-9940-4f0c-067d4b2112b1
Content: Get started with ASP.NET Core SignalR
Content Source: aspnetcore/tutorials/signalr.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @bradygaster
Microsoft Alias: bradyg

I resolved it. Coding error


Having a problem installing

Whenever I try to install and run this I get the following error:
pi@raspberrypi:~/gekko $ perl backtest.pl
Can't locate LWP/Protocol/https.pm in @INC (you may need to install the LWP::Protocol::https module) (@INC contains: /etc/perl /usr/local/lib/arm-linux-gnueabihf/perl/5.24.1 /usr/local/share/perl/5.24.1 /usr/lib/arm-linux-gnueabihf/perl5/5.24 /usr/share/perl5 /usr/lib/arm-linux-gnueabihf/perl/5.24 /usr/share/perl/5.24 /usr/local/lib/site_perl /usr/lib/arm-linux-gnueabihf/perl-base) at backtest.pl line 28.
BEGIN failed--compilation aborted at backtest.pl line 28.
I am running this on a raspberry pi. I have already installed all the dependencies as directed.

Same here - have tried for a few days on both Raspberry Pi Zero W (With Raspbian Stretch Lite and DietPi) and Debian Stretch on a Google Cloud Compute VM and cannot get the perl prerequisites installed.  Ultimately, it comes for to a prereq for LWP::Protocol::https being Net:SSLeay, and Net:SSLeay not being installable on any of the OSes I've tried.
Time::Elapsed gave me issues as well under cpanm, but somewhere between using cpanm vs. cpan and installing it via apt, that one got sorted. out.
Just about went crazy trying to get it to work, any insight would be greatly appreciated. @xFFFFF !

I believe you haven't installed the dependencies..
If you haven't done the following please do and let me what happened..
You should run this command in your cmd.exe :
cpan install Parallel::ForkManager Time::Elapsed Getopt::Long List::MoreUtils File::chdir Statistics::Basic DBI DBD::SQLite JSON::XS TOML File::Basename File::Find::Wanted Template LWP::UserAgent LWP::Protocol::https Set::CrossProduct DBD::CSV Text::Table File::Copy

I found the solution by installing it through apt get with the following command "apt-get install libwww-perl"


 rowHeight="fit" crashes page

Bug, feature request, or proposal:
BUG
What is the expected behavior?
I would expect when using rowHeight="fit" for the rowHeight to dynamically change depending on the contents of mat-grid-tile(s)
What is the current behavior?
mat-grid-list crashes
What are the steps to reproduce?
This stackblitz shows the working code, without using rowHeight but as you can see the mat-list is not rendering correctly.
https://stackblitz.com/edit/angular-cccnjp?file=app%2Fsidenav-responsive-example.html
Then if I use the same code and add rowHeight="fit" nothing renders (although the html is generated, if I inspect the element with dev tools)
https://stackblitz.com/edit/angular-cccnjp-ylmzf9?file=app/sidenav-responsive-example.html
What is the use-case or motivation for changing an existing behavior?
N/A
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Currently using Angular5.2.6, Material 5.2.1, Linux & Windows tests TS 2.7.1, Firefox and Chrome tested
Is there anything else we should know?
N/A

From the docs:

Fit: Setting rowHeight to fit This mode automatically divides the available height by the number of rows. Please note the height of the grid-list or its container must be set.

You have to set an exact height.

If you have to set an exact height, how can I create a responsive page? Surely mine cannot be the only use case where you want the container/div/element to scale to the size of the content?

You might be able to stretch it using something like flexbox.

So to get the above combination working correctly, I have had a bit of a hacky solution, see:
https://stackoverflow.com/questions/49234018/angular-material-mat-grid-list-not-expanding?noredirect=1#comment85535332_49234018

@crooksey from Brian's great blog post on using Flex Layout to create a responsive mat-grid-list, you can use Angular Flex Layout's ObservableMedia service and set the rowHeight in the mat-grid-list dynamically as the number of columns change.
example:
<mat-grid-list [cols]="cols | async" gutterSize="16px" [rowHeight]="rowHeight + 'px'">
...
</mat-grid-list>

ngOnInit() {
    const grid = new Map([
      ['xs', 1],
      ['sm', 2],
      ['md', 4],
      ['lg', 4],
      ['xl', 4]
    ]);

    let start: number;

    grid.forEach((cols, mqAlias) => {
      if (this.observableMedia.isActive(mqAlias)) {
        start = cols;
      }
    });

    // on breakpoint, assign cols AND ROW HEIGHT appropriately
    this.cols = this.observableMedia.asObservable()
      .map(change => {

        this.rowHeight = this.heightToCols(grid.get(change.mqAlias));
        console.log(this.rowHeight);

        return grid.get(change.mqAlias);

      })
      .startWith(start);
  }

  heightToCols(cols: number): number {
    if (cols === 1) {
      return 250;
    } else if (cols === 2) {
      return 300;
    } else {
      return 350;
    }
  }

Use Angular flex layout, it will solve all geometry issues. My EVA-L09. On O2.-------- Original Message --------Subject: Re: [angular/material2]  rowHeight="fit" crashes page (#10395)From: Meanstack Consulting<EMAIL_ADDRESS>angular/material2<EMAIL_ADDRESS>Luke Crooks luke@crooksey.org,Mention<EMAIL_ADDRESS>tried to use mat-grid-list within mat-dialog
grid-list would not even show up
it refused to take on the dimensions of the mat-dialog
and with three grid-list one for title,content,actions
not fun
the mat-dialog directives are not blendable with grid-list
grid-list wont take on any styles like
and the rows get munged across different columns
seems we need to roll our own geometry mgt
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or mute the thread.
{"@context":"http://schema.org","@type":"EmailMessage","potentialAction":{"@type":"ViewAction","target":"https://github.com/angular/material2/issues/10395#issuecomment-390529147","url":"https://github.com/angular/material2/issues/10395#issuecomment-390529147","name":"View Issue"},"description":"View this Issue on GitHub","publisher":{"@type":"Organization","name":"GitHub","url":"https://github.com"}}
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/angular/material2","title":"angular/material2","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/angular/material2"}},"updates":{"snippets":[{"icon":"PERSON","message":"@nhhockeyplayer in #10395: I tried to use mat-grid-list within mat-dialog\r\n\r\ngrid-list would not even show up\r\nit refused to take on the dimensions of the mat-dialog\r\nand with three grid-list one for title,content,actions\r\n\r\nnot fun\r\n\r\nthe mat-dialog directives are not blendable with grid-list\r\ngrid-list wont take on any styles like \u003cmat-grid-tile style="vertical-align: middle"\u003e\r\nand the rows get munged across different columns\r\n\r\nseems we need to roll our own geometry mgt"}],"action":{"name":"View Issue","url":"https://github.com/angular/material2/issues/10395#issuecomment-390529147"}}}
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "37567f93-e2a7-4e2a-ad37-a9160fc62647",
"title": "Re: [angular/material2] \u003cmat-grid-list\u003e rowHeight="fit" crashes page (#10395)",
"sections": [
{
"text": "",
"activityTitle": "Meanstack Consulting",
"activityImage": "https://avatars2.githubusercontent.com/u/6886551?s=160\u0026v=4",
"activitySubtitle": "@nhhockeyplayer",
"facts": [
]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n"commandName": "IssueComment",\n"repositoryFullName": "angular/material2",\n"issueId": 10395,\n"IssueComment": "{{IssueComment.value}}"\n}"
}
]
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/angular/material2/issues/10395#issuecomment-390529147"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n"commandName": "MuteNotification",\n"threadId": 312612417\n}"
}
],
"themeColor": "26292E"
}Sent via Migadu.com, world's easiest email hosting>

so this bug is not going to be solved?

@aWeinzierl you need to use angular material with flexbox to get the desired functionality I was initially after.

Angular material says on Grid List Documentation:

Fit: Setting rowHeight to fit This mode automatically divides the available height by the number of rows. Please note the height of the grid-list or its container must be set.

So basically in here:
https://stackblitz.com/edit/angular-cccnjp-ylmzf9?file=app/sidenav-responsive-example.html
Instead of
something like
And it works perfectly

Regrettably, Angular has the worst Google material implementation (in comparison to VueJS/Vuetify or React/MaterialUI) because nothing works really properly. Every solution above does not work as expected.  The only way out is: https://github.com/angular/flex-layout

this problem can be solved using flexbox.
`.container {
display: flex;
flex-wrap: wrap;
}
.item {
flex-grow: 1;
flex-basis: 300px;
margin: 10px;
}`
`  A
B
C
D

`
you can  find more details here :
 https://www.vinta.com.br/blog/2017/responsive-layout-flexbox/


SQL Database Projects fails to build in ADS but succeeded in VS. And then bringing project back into VS is broken

Version: 1.20.0-insider (user setup)
Commit: 710d0462d8d81373aa2bc0f36f88955c902cb173
Date: 2020-06-25T05:31:00.820Z
VS Code: 1.46.0
Electron: 7.3.1
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18363
Steps to Reproduce:
Using Synapse SQL Pool (SQL DW).

Create SSDT Database Project in VS and then run a successful build.

Open the Database Project in ADS and then Build. Observe failure in ADS:

Open VS and attempt to Build the project again. Observe following failure:

does doing Rebuild or Build after Clean in VS fail too? The build failing in VS should've been fixed by #10882

Thanks! After Rebuild, this is resolved. The first issue still occurs though.

The issue is due to the project.assets.json in the obj folder.
I believe a solution is to change the .sqlproj file. data Studio adds this
 <Target Name="AfterClean">
    <Delete Files="$(BaseIntermediateOutputPath)\project.assets.json"/>
  </Target>

if you change it to the following it works for me.
 <Target Name="BeforeBuild">
    <Delete Files="$(BaseIntermediateOutputPath)\project.assets.json"/>
  </Target>

@kevinvngo can you please share the bove info on the property value. thanks for your help.

This need a DacFx nnuget/ SqlToolsService refresh in ADS hence bringing to Sept


NN_DESCENT is slower, almost as slow as CPU

Running with the following line added causes a slowdown:
  index_params.build_algo = raft::neighbors::cagra::graph_build_algo::NN_DESCENT;
https://github.com/SearchScale/lucene-cuvs/blob/main/cuda/src/CudaIndexJni.cu#L55
Indexing time degrades to 16-17seconds, as below:
Mar 17, 2024 3:22:55 PM org.apache.lucene.internal.vectorization.VectorizationProvider lookup
WARNING: Java vector incubator module is not readable. For optimal vector performance, pass '--add-modules jdk.incubator.vector' to enable Vector API.
5000 docs indexed ...
10000 docs indexed ...
15000 docs indexed ...
20000 docs indexed ...
25000 docs indexed ...
Time taken for index building (end to end): 25831
Time taken for copying data from IndexReader to arrays for C++: 76
CUDA devices: 1
Data copying time (CPU to GPU): 744
Cagra Index building time: 16308
Time taken for index building: 17331
Time taken for cagra::search: 1
Time taken for searching (end to end): 3
Found 5 hits.
DocID: 1461, Score: 0.25425205, Title: Atlantic Ocean
DocID: 1472, Score: 0.32853734, Title: Pacific Ocean
DocID: 4668, Score: 0.33573288, Title: List of seas
DocID: 1498, Score: 0.36634398, Title: Southern Ocean
DocID: 1475, Score: 0.37005764, Title: Indian Ocean

Without that line, i.e. when using IVF_PQ, indexing time is much better (4-5 secs):
(base) ishan@4090-workstation:~/lucene-cuvs$ java -jar lucene/target/cuvs-searcher-lucene-0.0.1-SNAPSHOT-jar-with-dependencies.jar
Mar 17, 2024 3:30:06 PM org.apache.lucene.internal.vectorization.VectorizationProvider lookup
WARNING: Java vector incubator module is not readable. For optimal vector performance, pass '--add-modules jdk.incubator.vector' to enable Vector API.
5000 docs indexed ...
10000 docs indexed ...
15000 docs indexed ...
20000 docs indexed ...
25000 docs indexed ...
Time taken for index building (end to end): 25485
Time taken for copying data from IndexReader to arrays for C++: 71
CUDA devices: 1
Data copying time (CPU to GPU): 790
Cagra Index building time: 4312
Time taken for index building: 5465
Time taken for cagra::search: 1
Time taken for searching (end to end): 3
Found 5 hits.
DocID: 1461, Score: 0.25425205, Title: Atlantic Ocean
DocID: 1472, Score: 0.32853734, Title: Pacific Ocean
DocID: 4668, Score: 0.33573288, Title: List of seas
DocID: 1498, Score: 0.36634398, Title: Southern Ocean
DocID: 1475, Score: 0.37005764, Title: Indian Ocean

It seems that the pre-compiled libraries for raft don't perform fast with NN_DESCENT inside JNI layer, but compiling them from scratch works fine. But, this is the case only for RTX 4090 and RTX A6000 devices. The precompiled library performed fine with RTX 2080 Ti on NN_DESCENT, even inside JNI. No idea why and how, but seems like the best thing to do is to invest the first 2 hours in compiling the whole thing (by avoiding miniforge3 altogether). Closing the issue for now, can revisit it if we have more information.


Minor touch-up to finetuning directory

This is GTG from my side, takes care of maintainence for keeping only the working scripts (and also the scripts supported by the latest micro_sam library at the top-level, marking the rest as not relevant (by moving to outdated directories).


resolved 'Iterate array without loops'

Hey @Sayan2908 !
Great work!
You can follow me and star this repository for further updates.
Have fun 🙌🏻


updated retention time for activity logs

you wrote 19 days, but activity logs retention is 90 days

@Ericgre : Thanks for your contribution to the Azure documentation! The author, @unifycloud, has been notified to review your proposed change.

@LizCasey : Thanks for your contribution! The author, @UnifyCloud, has been notified to review your proposed change.

Reviewed suggested change and fixed the merge conflict. Merging this PR. @Ericgre , Thank you so much for your contribution!


Add sliding window

Adds option to utilize a sliding window in place of the standard attention.
Setting default for sliding window to be False.

@777yifei @advancedengineering can you please review.

I am reading sliding window now, I might misunderstand the code but shouldn't the mask for the window be a span of W(hyper parameter, window size) around the diagonal of the matrix?
In my understanding, self.mask is a lower triangular matrix, and after cropping, it will still be a lower triangular matrix like the vanilla attention not sliding window.
mask = self.mask[:, :, -T:, -T:] # crop mask to sequence length

Will try to refactor this after starting a few long-running data-augmentation scripts.  Retracting for the time being.


Disable RN Timers and Animation synchronization to prevent tests from stalling

Is your feature request related to a problem? Please describe.
It's related to a problem we've been facing at Skyscanner. Some of our tests were stalling in Android because of ReactNativeTimersIdlingResource.isIdleNow() and  AnimatedModuleIdlingResource.isIdleNow() never returning true in some cases. I couldn't identify exactly when this happens since some of the times this happened we didn't have any infinite animations running in that particular screen.
Describe the solution you'd like
We already have the disableSynchronization() function but that's only disabling the network synchronization. So my proposal is to add similar methods to disable RN timers synchronization and animation synchronization. You can see an already implemented version of this here: https://github.com/leanmazzu/Detox/commit/efb0dcfd799ffea3d91bd1c8e46ee84df3950dd6. This is what we currently have in place in our Skyscanner fork and it's working just fine for us.
Describe alternatives you've considered
N/A
Additional context
This would be a non-breaking change as it's not modifying the current disableSynchronization() method or anything else.

Hmm disableSynchronization() should disable all synchronization. @rotemmiz Why is this not the case in Android?
@leanmazzu Feel free to submit a PR that implements this.

Cool will do, just one question... the solution I proposed above is only adding the functionality to disable ReactNativeTimersIdlingResource and AnimatedModuleIdlingResource that was enough for us but, do you think it's worth adding the functionality to disable ReactNativeUIModuleIdlingResource as well? Looks like it could help solve this issue https://github.com/wix/Detox/issues/1435

I think a disableSynchronization() option should disable everything, like it does on iOS. Waiting for @rotemmiz

It should, and if it doesn't we need to fix it.
Also, we should probably add per IR disable mechanism (as much as we can)

We don't have that mechanism on iOS either for now. But at least the main disable should disable 😄
@leanmazzu We'd be happy to receive a pull request. Thank you!

Ok cool, I'll do this then:

disableSynchronization() will now disable all the IR.
There will be a disable method for every IR so that they can be disabled/enabled separately.

Let me know if I'm missing something here. Otherwise, I'll send the PR implementing this as soon as it's ready. Thank you!

Sounds good. Thanks!

Sounds about right. Nevertheless @leanmazzu I'm highly intrigued in finding out what your use cases are, I can't help but think whether possibly you're just struggling with an actual bug in one of the idling resources. Either way, 'love the Skyscanner app :)

Usually what we've found on iOS is that people disable sync when they have endless animations or some network. For network, we have a way to disable sync on iOS. Not sure if the same API works for Android (@rotemmiz ?). Regarding animations, we usually recommend mocking the animation calls to only run once or not at all during Detox tests.

Network disable/enable sync is implemented in Android, same API.
https://github.com/wix/Detox/blob/master/detox/test/e2e/14.network.test.js#L29

@rotemmiz I meant this: https://github.com/wix/detox/blob/master/docs/APIRef.DeviceObjectAPI.md#deviceseturlblacklisturls

Thank you all guys! The PR is ready to review (https://github.com/wix/Detox/pull/1521)

Sounds about right. Nevertheless @leanmazzu I'm highly intrigued with what your use cases are, I can't help but think whether possibly you're just struggling with an actual bug in one of the idling resources. Either way, 'love the Skyscanner app :)

@d4vidi in our case we have some screens with complex view hierarchies. We tried mocking all the animations, RN even provides a way of doing this. But that didn't work for us, as some components were not working at all with animations disabled. Anyways, I'm glad you like Skyscanner app :)

That RN feature is completely broken. On iOS, we force it disabled so that apps work. It's not intended for e2e testing.

It seems I am hitting the same problem, I get very randomly below errors after timeouts:
(node:19198) UnhandledPromiseRejectionWarning: Error: Looped for 1978 iterations over 60 SECONDS. The following Idle Conditions failed.
detox[22054] WARN:  [Client.js/PENDING_REQUESTS] App has not responded to the network requests below:   (id = 110) invoke: {"target":{"type":"Class","value":"com.wix.detox.espresso.EspressoDetox"},"method":"perform","args":[{"type":"Invocation","value":{"target":{"type":"Class","value":"androidx.test.espresso.Espresso"},"method":"onView","args":[{"type":"Invocation","value":{"target":{"type":"Class","value":"com.wix.detox.espresso.DetoxMatcher"},"method":"matcherForTestId","args":["back_button"]}}]}},{"type":"Invocation","value":{"target":{"type":"Class","value":"com.wix.detox.espresso.DetoxViewActions"},"method":"click","args":[]}}]} That might be the reason why the test "Teddy scenario Explore" has timed out.
Do you guys think these logs are related? This seems to happen after tapping buttons that trigger an animation
disableSynchronization does nothing at all for me, not even turning off network. I also tried setURLBlacklist or detoxURLBlacklistRegex without any results

I'm hitting a similar issue, UIManagerModule is always busy (because of a looping animation). Was about to implement disable myself but found this! Thanks @leanmazzu for the PR!
@maxime-helen I found it useful to check the instrumentation app logs:
adb logcat *:S Detox:V -T 50
If your app is stuck and those actions are timing out then check the logs for messages like "UIManagerModule is busy"

Will this be merged any time soon? Currently blocked on running android tests as well on RN 0.64. My tests start up by typing is about 1 character per 2 seconds, which causes timeouts. I also see a "UIManagerModule is busy" printed quite frequenty b/w characters appearing

@rotemmiz let's give this high prio, I don't think there's too much left to be done here

heads up: I'm not actively working on this as my team's priorities have shifted a little, especially since the branch in its current state works great for us (thanks @leanmazzu).

@leanmazzu hey, did you open a PR for this in the end? I am experiencing the same issue

@ronilitman he initially has, but it's become obsolete after requiring various changes. If you wanna take it we'd be happy to accept it.

Thank you all guys! The PR is ready to review (#1521)

Sounds about right. Nevertheless @leanmazzu I'm highly intrigued with what your use cases are, I can't help but think whether possibly you're just struggling with an actual bug in one of the idling resources. Either way, 'love the Skyscanner app :)

@d4vidi in our case we have some screens with complex view hierarchies. We tried mocking all the animations, RN even provides a way of doing this. But that didn't work for us, as some components were not working at all with animations disabled. Anyways, I'm glad you like Skyscanner app :)

These RN files are totally broken, they don't preserve animation behaviors (like call a callback or setting final values when they finish), I've made a patch to apply in my project for them, at least with it my tests on android were working again

@Grohden kudos for getting that fixed - even though that in essence, I advise not to disable animations in E2E-level testing. Rather, in our projects, we've force-disabled animations mocking altogether:
https://github.com/facebook/react-native/issues/27010#issuecomment-573645965

I have this issue, my use case is as follows (and I'd happily accept guidance to perhaps mitigate it):

Start a single-screen APP that just refreshes with status in order to run a big queue of tests, no UI interaction is needed
Apparently (? open question) if there is no UI interaction the tests crawl because it appears that Detox requires UI interaction in order to move quickly?
So we send a steady stream of "adb shell input tap" events to the emulator and things move along, but...
If react-native catalyst instance goes down in between an android MotionEvent.ACTION_DOWN and MotionEvent.ACTION_UP, then it only registers the ACTION_UP, and that apparently leaves something on the queue as detected by UIModuleIdlingResource here, so that queue is permanently blocked (this could be a react-native bug as well then, I suppose)
So now the app is stuck and never sends the testee-side login / ready event, so the test never starts

So in my use case, I don't care at all about synchronization, I'd like to disable it completely on Android, exactly as the API disableSynchronization would indicate.
@d4vidi are you still interested in someone picking up the android disableSynchronization PR? It seems at the least, disable should...disable all synch (not just network) ?
Is there some way to make detox pick up events and keep moving, without UI interaction? Perhaps I'm missing some timing config?
Is anyone familiar with react-native MotionEvent handling and why a broken pair of ACTION_DOWN/ACTION_UP events might jam up the UIManager queue?
Cheers

I am having tests that are failing in Android with UI Kitten modals. I think it is an infinite animation when I open modals
Is there any alternative to avoid detox stalling? I tried disableSynchronization() but this only works for iOS...
https://github.com/akveo/react-native-ui-kitten/issues/1250

@napsta32 depending on your needs, you can chop any part of Detox out. I completely cut off the UIManagerIdlingResource from being able to block testing for react-native-firebase because for my case I wasn't driving the UI, I was just running effectively headless tests yet the UIIdlingResource has some bug that causes it to halt anyway.
So you just cut it out as a patch-package patch like so https://github.com/invertase/react-native-firebase/blob/master/tests/patches/detox%2B16.7.2.patch
If you're driving the UI then this won't work, but it's an example of bending Detox to your will for your situation

Thanks @mikehardy
This solution is working for my UI tests (somehow)
I guess if I have any issues I will try to fix the disableSynchronization() UIManagerIdlingResource bug

There was a PR created to fix this a long time ago but for some reason was never merged. We have a similar issue where we have an animation loop (custom input blinking cursor) on certain screens that there is just no good way to test on those screens without hacks. Disable animation synchronization on that screen would be a huge help, works on iOS but not on android, here is the PR that was closed:
https://github.com/wix/Detox/pull/1521

@martintreurnicht , maybe I can look at #1521 next week, but if you feel like you can resurrect and finish the work, you are more than welcome to! 👍

@d4vidi, please advise if a better solution than #1521 would actually be working on debugSynchronization for Android?

Well effectively debugSynchronization != disableSynchronization. We have clear plans regarding the former, but none for the latter. I generally think #1521 is a good approach but we don't have a timeline for that yet.

@martintreurnicht , maybe I can look at #1521 next week, but if you feel like you can resurrect and finish the work, you are more than welcome to! 👍

Sure, i can resurrect it. Doesn't seem like there are any outstanding items there though, seems like it was just abandoned for whatever reason. I'll see if there is any major changes that need to be made once applied to head of master, if it all seems to work ok i'll just open a new PR

@iMaupin that sounds like a (valid) extension of the feature requested here.

are there any updates? thank you.

None so far.

Hi @d4vidi , I see that this thread was closed, was this resolved? And also what happened to #1521 ? I was facing some endless animation sync errors and came across these threads

@pradeipp We have a reference document for that. I hope you'd find that helpful.

Thank you @d4vidi I've already went over that article quite a few times actually. I've been facing issues related to endlessly running Animations and wanted to see if the individual synchronization methods mentioned above were implemented or not.
Saw this line in the docs so I'm guessing there's no way to turn off just animation synchronization through a simple api call as of right now?

If you have an endless looping animation, this may cause Detox to hang. In this case, consider turning off the animation synchronization or remove the endless loop in your E2E build with mocking.

Yes, not at the moment; That statement generally still holds true, although, as noted before -- we could definitely do with help with this from the community.

This is still happening with sync turned off. Makes detox unusable to us.
Is there any way to figure out what the offending animation is? Otherwise the logs give no insight into what is actually hanging.

@owens-ben you can either cut that part out (my updated patch is here https://github.com/invertase/react-native-firebase/tree/main/tests/patches) or use a similar patch to print out exactly what thing is being waited on in order to expect more closely

@mikehardy thanks for responding. I implemented your patch but it doesn't seem to help. Can you explain exactly what it's supposed to accomplish? I see errors in my logcat about UIManagerModule and Catalyst not being available but I'm not sure what the cause is, and I see your patch addresses things related to that.
I'm trying to use --debug-synchronization 500 to figure out what is hanging in my app.
I was thinking maybe it was a request to segment.io so I tried using the blacklist, but that didn't seem to be the issue.
I'm still getting errors like this
13:35:01.051 detox[15794] INFO:  [APP_STATUS] The app is busy with the following tasks:
• UI elements are busy:
  - Reason: UI rendering activity.
• There are enqueued timers.
13:35:01.054 detox[15794] INFO:  [APP_STATUS] The app is busy with the following tasks:
• UI elements are busy:
  - Reason: UI rendering activity.
• There are enqueued timers.
13:35:01.090 detox[15794] INFO:  [APP_STATUS] The app is busy with the following tasks:
• UI elements are busy:
  - Reason: UI rendering activity.
• There are enqueued timers.
13:35:01.557 detox[15794] INFO:  [APP_STATUS] The app seems to be idle
13:35:01.560 detox[15794] INFO:  [APP_STATUS] The app seems to be idle
13:35:01.597 detox[15794] INFO:  [APP_STATUS] The app seems to be idle
13:35:02.067 detox[15794] INFO:  [APP_STATUS] The app is busy with the following tasks:
• "LooperIdlingResource-192-mqt_js" (JS Thread) is executing (JavaScript code).
13:35:02.071 detox[15794] INFO:  [APP_STATUS] The app is busy with the following tasks:
• "LooperIdlingResource-192-mqt_js" (JS Thread) is executing (JavaScript code).
13:35:02.102 detox[15794] INFO:  [APP_STATUS] The app is busy with the following tasks:
• "LooperIdlingResource-193-mqt_native_modules" (Native Modules Thread) is executing (native module calls).
• "LooperIdlingResource-192-mqt_js" (JS Thread) is executing (JavaScript code).
• UI elements are busy:
  - Reason: UI rendering activity.

Do you have any other advice?

It is important to note that on android you need to integrate detox as a "from source" build (not the normal type of detox integration) in order for the patch to matter
My patch just disables one type of synchronization, it may be that the sync you are infinitely waiting on is a different type, I can't say. I don't have any other advice other than to make sure you've de-integrated the pre-built detox and integrated the source build, then to roll sleeves up and see exactly what is being waited on.
The debug stuff looks interesting in that there appears to be lots of stuff going on, native code, JS code and a UI render 🤷

Could this problem have cropped up again? I'm noticing after upgrading to Detox 20.20.0, RN 0.73.6 and Reanimated 3.8.1 that device.disableSynchronization has stopped working in tests that have endless Reanimated animations. As soon as the animation is paused on the UI then the tests proceed.

Could this problem have cropped up again? I'm noticing after upgrading to Detox 20.20.0, RN 0.73.6 and Reanimated 3.8.1 that device.disableSynchronization has stopped working in tests that have endless Reanimated animations running on an Android 14 (API 34) emulator. As soon as the animation is paused on the UI then the tests proceed.
update: Downgrading to an Android 13 (API 33) emulator seems to have helped here. So perhaps there's an idling issue with Android 14?

@mrbrentkelly This issue is quite old. Can you please report that on a new GH issue?


Fix RCS1221: Use pattern matching instead of combination of 'as' operator and null check

PR Summary

Autofix RCS1221
Use is not syntax (like #13323, every change has been manually reviewed)

PR Context

#13287
#13323

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[NA] Experimental feature(s) needed

[NA] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[NA] Documentation needed

[NA] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[NA] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[NA] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

Reverted IDE0019 commits due to high number of issues.

PR is ready for review

@iSazonov Can you review?

@PowerShell/powershell-maintainers reviewed this PR and agreed to accept it for 7.2-preivew. So we should delay merging it until after 7.1-RC1 release branch gets created.

The changes make sense, but it requires line-by-line review to make sure there is no potential incorrect change, so it's better to delay merging it till 7.2 preview to avoid possible regression.

Set Draft till 7.2. Preview1.

@rjmholt Could you please restart tests.

I restarted CI Windows.

@iSazonov Please restart PowerShell-CI-windows another time.

Restarted CI Windows.

@iSazonov It seems Github is still reporting the old PowerShell-CI-windows result. I shall rerun tests by rebasing instead.

@iSazonov Should I fix pre-existing Codacy issues : Add curly braces around the nested statement(s) in this 'if' block.?

@iSazonov Should I fix pre-existing Codacy issues : Add curly braces around the nested statement(s) in this 'if' block.?

In follow PR only.

This PR is ready for maintainer review.

@iSazonov I am considering closing in favour of #13895 because codefix for IDE0019:

automatically uses is not syntax (so no manual changes required)
does not have any problems with stripped comments

Use is not syntax (like #13323, every change has been manually reviewed)

I don't see the change in the PR but we can do this later and I am ready to merge.

Use is not syntax (like #13323, every change has been manually reviewed)

I don't see the change in the PR but we can do this later and I am ready to merge.

I think this change was removed in a rebase to refocus the PR. Further is not changes can be made in follow-up PR. So please merge this PR.

@xtqqczze Do you think about forcing Roslynator rules? We could add new GitHub Check.

@iSazonov I used Roslynator to make these changes as it was the easiest tool for me at the time of the PR. Now codestyle analyzers are included in .NET 5.0 we can use IDE0019 instead.

Now codestyle analyzers are included in .NET 5.0

All?

Now codestyle analyzers are included in .NET 5.0

All?

Many RCSXXXX rules have crossover with IDEXXXX and CAXXXX. If we also add StyleCop analyzers package like in #13963 we can likely cover most scenarios.


[app] Add Integrations for Kubernetes Resources

Add a new integrations feature for kobs. In this first iteration we
implemented the suggested integrations feature from #255 for Kubernetes
Resources.
This integration allows an administrator of kobs to add a set of default
dashboards to Kubernetes Resources. This way we can add for example a
resource usage dashboard to all Pods, without that each Pod must have
the "kobs.io/dashboards" annotation.
The configured dashboards in the integrations and the dashboards from
the "kobs.io/dashboards" annotation are always merged and not
overwritte.

[x] I added a CHANGELOG entry for this change.
[x] I adjusted the corresponding documentation for this change.
[x] I adjusted the values.yaml file and the documentation for all values.

Codecov Report

Merging #401 (13b86b8) into main (4490c9f) will decrease coverage by 0.04%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##             main     #401      +/-   ##
==========================================
- Coverage   45.70%   45.66%   -0.05%
==========================================
  Files         137      137
  Lines       10661    10672      +11
==========================================
  Hits         4873     4873
- Misses       5695     5706      +11
  Partials       93       93

Impacted Files
Coverage Δ

pkg/hub/api/plugins/plugins.go
0.00% <0.00%> (ø)

pkg/hub/api/resources/resources.go
0.00% <0.00%> (ø)

pkg/hub/hub.go
0.00% <0.00%> (ø)

pkg/hub/api/applications/applications.go
100.00% <100.00%> (ø)

pkg/hub/api/clusters/clusters.go
100.00% <100.00%> (ø)

pkg/hub/api/dashboards/dashboards.go
100.00% <100.00%> (ø)

pkg/hub/api/teams/teams.go
100.00% <100.00%> (ø)

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


can this driver support trino with https?

The driver does have a "Use a secure connection (SSL)" checkbox when creating a new Starburst database.


Reason why the color video stream is disabled

Is there a reason why the color stream is disabled (launch file argument useVideo in oakd.launch.py). The preview resolution is pretty low. Likely the reason is computation time, but I wish that there would be a way to get at least single frame with the full resolution.

The video stream publishes the full 1080p video while the preview is just a crop of that full image. The RPi struggles to publish the full 1080p image with its internal WiFi at a reasonable framerate. The intention of the preview image is to assist with teleop.
You can modify the oakd launch file to set the camera parameters to whatever you like. Or you can try some of the other nodes at https://github.com/luxonis/depthai-ros-examples

Thanks for the reply. I understand that streaming the full image over Wi-Fi will not work. In our application we want to implement tag detection on the Raspberry Pi, and that can be done locally on the raspberry pi. For better accuracy we want a higher resolution and accept a lower framerate instead. UseVideo/UsePreview in the launch file can enable that usage. The full resolution gives fps between 1 and 2.

I now learned that the camera has an API to capture a still image at full sensor resolution. However this is not made available in ROS: https://github.com/luxonis/depthai-ros-examples/issues/61


Run bootstrap as a postinstall script

Per https://react-server.slack.com/archives/discussion/p1481135585000084

:rocket:


License of files in tinted-theming/base16-schemes

What is the license of the files in the tinted-theming/base16-schemes project?
I'm asking because I would like to package the files in nixpkgs.

This has always been a bit nebulous - the original base16 was MIT, but many schemes have been added since then. They've been packaged with many template repositories which have been released as MIT, and there haven't been any complaints so far, but I don't know if it's known for sure.
I've been operating under the assumption that if schemes are submitted there's an expectation that they're released as a part of the existing template repos and should be fine to release under MIT (in Chris Kempson's repo, a clarification was added to this effect, but I don't think he actually checked all the existing schemes before adding that).
In the future, I'd like to make the schemes public domain, but I'm not sure that's going to be possible without removing a bunch of schemes.

What is the advantage of public domain vs same a super permission license like MIT?

Public domain would mean template repos could be released under other licenses, not just compatible ones. As an example, I'd love to make the Emacs theme GPL'd similar to most other Emacs packages, but I can't easily because the schemes are sort of licensed under the MIT license and they're included inside the templated files, so I'd need tooling to include the proper MIT copyright file with the relevant schemes.

I have been looking into this, and if I understand correctly, color schemes cannot have a copyright associated with them. They can be trademarked, but that has to be done in a specific context (the example often used is the colors used in the John Deere logo, because they have successfully sued other companies who have mislead consumers by using their colors).
I think this means we could update the license for the base16-schemes without any major issues, but if anyone knows something different, it would be great to hear.

@belak since no one has responded with any reason not to do it, should we change the schemes' license?


HuggingFace(Instruct)Embeddings client kwargs

What does this PR do?
Make it possible to control the HuggingFaceEmbeddings and HuggingFaceInstructEmbeddings client model kwargs. Additionally, the cache folder was added for HuggingFaceInstructEmbedding as the client inherits from SentenceTransformer (client of HuggingFaceEmbeddings).
It can be useful, especially to control the client device, as it will be defaulted to GPU by sentence_transformers if there is any.
Use case:
from langchain.embeddings import HuggingFaceEmbeddings
model_name = "sentence-transformers/all-mpnet-base-v2"
model_kwargs = {'device': 'cpu'}
hf = HuggingFaceEmbeddings(model_name=model_name, model_kwargs=model_kwargs)

Before submitting

[ ] This PR passes tests
[ ] This PR was discussed/approved

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

merged in #3266, thank you @Xmaster6y!


cqlsh is unable to start on slow/loaded clusters

Setup
HEAD: branch-1.6 01b0514098b37c28efe7ef2ce290552e48ed30c2
Description
cqlsh is missing a way to configure a cassandra.cluster.Cluster()'s control_connection_timeout parameter. By default this value is set to 2s, which has proven to be not enough. As a result on slow or loaded Nodes this results in cqlsh failure to start.
$ cqlsh
Connection error: ('Unable to connect to any servers', {'<IP_ADDRESS>': OperationTimedOut('errors=None, last_host=None',)})

This issue is a duplicate of C*'s cqlsh issue: https://issues.apache.org/jira/browse/CASSANDRA-10959
We need to:

Increase a default.
Make sure cqlsh allows to override the default by either/both value in the cqlshrc or/and the command line option.

The issue looks addressed on the master branch.

@vladzcloudius Do you know which commit fixed this issue?

@penberg I don't but the cqlsh in enterprise is very different from the one in the master:

In the enterprise there is only cqlsh.
In the master cqlsh calls the cqlsh.py and the contents on cqlsh.py is very different from the one in the cqlsh in the enterprise.

Another thing - I'm not sure master doesn't have this issue. I didn't test it and we definitely have to check this.

@penberg @slivne @nirmaayan Just to clarify - the customer in question has received the patched version that doesn't have a proper fix but rather hard codes the big timeout values. The proper fix should be to make cqlsh take the user provided parameters into an account like described in the issue.


[Bug] Flinksql job depended jar not changed when i reupload the dependended jar

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
when i reupload the flinksql jar and lauch the app and restart it ,i found the depandency not changed,it show job use the old jar.
StreamX Version
1.2.2
Java Version
1.8
Flink Version
1.14
Scala Version of Flink
2.11
Error Exception
when i reupload the flinksql jar and lauch the app and restart it ,i found the depandency not changed,it show job use the old jar.

Screenshots
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

fixed, please use latest version


Bug: Conflict with Enhancing Export plugin

In the Export to PDF modal the margin configuration part is hidden if you have the Enhancing Export plugin activated.

It should come out like this if you don't have it activated:

The problem is this line in the styles.css file of Enhancing Export plugin:
.setting-item.ex-setting-item{border-top:unset;padding-top:0}*[hidden]{display:none}

The first image should be correct. The custom options will only appear when Margin is set to Custom.


Do not retry NoRefImageErrors

/cc @SevInf @j0tunn

Coverage increased (+0.03%) to 76.558% when pulling 71b293d62f80e377a105f82733e1bb6451ac0214 on fix/no_retry_on_no_ref into 8eaf7284a4bca447fefe555277e2c301ac957bf2 on master.

:ok:

:ok:


fix: typo in use_exclusive_card.yaml and add new exclusive example

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Fix typo
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
Nothing

@JasonHe-WQ

/lgtm

/lgtm
/approve


Finish assignments

This should add functionality to:

[x] Type in a 'finish' command
[ ] Lock the repository after assignment is done
[ ] Generate a 'marks' subfolder for pdfs
[ ] Generate a section in the assignment's hoot.json for the mark.

Created branch for this: https://github.com/rishiosaur/hoot-cli/tree/feature/command/finish


Suggestion box skips item when using keyboard navigation

Hey very cool! I just noticed that the suggestion box will skip every suggestion when navigating with keys.

Hey, what do you mean skip every suggestion? Can you give some screenshots?

Hi, thanks for getting back to me, I can't reproduce the issue anymore, but on my work laptop, the suggestion box would always skip the 2nth item

Ok, I know this bug, but it hard to reproduce indeed.


Daniel Change

My change too

@Daniel-Moffatt-LYIT your pull request should request to pull the changes from your branch into the copy of this repo that you created when forking it (https://github.com/Daniel-Moffatt-LYIT/IntroGitHub) and not this repository. Just create a new PR and remember to select main branch on the correct repository.


Add options for chart legend font style.

Please fix the conflict first~ Thank you

OK, I solved the conflict. -->_--> If you merge my pull request befor sanandrea, there will be no conflict.


Fix broken build gradle as a consequence of missing material dialogs deps

Maven artifact for Material Dialogs has changed version, the version referenced in build.grade was not available anymore and because of it the build was broken. I have updated the version to be in sync with the one in the maven repository

Thanks!


Can the frequency-domain block-based AEC automatically handle single talk and double talk situations?

Hi, I think this is a remarkable job!
I have a question about the frequency-domain block-based AEC. Is the AEC method you used able to handle the doubletalk case automatically?  Or do you have to separate the far-end singletalk and doubletalk cases during pre-processing and manually stop the filter coefficient update during the doubletalk case?

Hi, I think this is a remarkable job! I have a question about the frequency-domain block-based AEC. Is the AEC method you used able to handle the doubletalk case automatically? Or do you have to separate the far-end singletalk and doubletalk cases during pre-processing and manually stop the filter coefficient update during the doubletalk case?

From generate_cache.py， I think linear AEC can automatically deal with single and double talk situations.
Thank you for your project, it’s very useful for me!

Hi, I think this is a remarkable job! I have a question about the frequency-domain block-based AEC. Is the AEC method you used able to handle the doubletalk case automatically? Or do you have to separate the far-end singletalk and doubletalk cases during pre-processing and manually stop the filter coefficient update during the doubletalk case?

hello，I saw that the author used the train-hard folder under the AEC-Challenge dataset in the code, but I did not find this folder in this dataset, did you  see it? could you please give me a reply,thanks very much!

The AEC does not need a VAD, as you can see from the implementation, and the paper being referenced there.

train_hard contains the hardest files from the dataset, which have been selected in terms of SNR.

it's so honored to receive your reply. i will read the code more carefully, thanks ,  best wishes !

train_hard contains the hardest files from the dataset, which have been selected in terms of SNR.

Excuse me,  is this code complete？I didn't see any information about adding data to the train_hard folder (the train_hard folder is empty at first, that is, 0 pieces of data, which brought errors to the operation of the code). If is complete, could you please tell me where is it?   I'm so sorry to disturb you.  And hope that I can get your reply.  Thanks very mush , Best wishes!

Hi, I think this is a remarkable job! I have a question about the frequency-domain block-based AEC. Is the AEC method you used able to handle the doubletalk case automatically? Or do you have to separate the far-end singletalk and doubletalk cases during pre-processing and manually stop the filter coefficient update during the doubletalk case?

From generate_cache.py， I think linear AEC can automatically deal with single and double talk situations. Thank you for your project, it’s very useful for me!

Hello, I'm so sorry to disturb you,from the generate_cache.py,I don't understand how linear AEC automatically deal with single and double talk situations.,can you tell me something about it?I'm looking forward to your reply.


Vault is running but configurer is failing with - unable to get key 'vault-root'

Preflight Checklist

[X] I have searched the issue tracker for an issue that matches the one I want to file, without success.
[X] I am not looking for support or already pursued the available support channels without success.
[X] I agree to follow the Code of Conduct.

Version
1.31.2
Expected Behavior
We expect that the vault configurator apply all configurations correclty when using GCS storage and GCP KMS unseal.
Actual Behavior
The vault is working correctly, storage data is being saved in HA mode to a GCS bucket and all vault pods are auto unsealing correclty, but the vault configurator throws the error:
INFO applying config file: /config/vault-configurer/vault-config.yml
INFO checking if vault is sealed...
INFO vault is unsealed, configuring...
ERROR error configuring vault: unable to get key 'vault-root': error getting data: error getting object for key 'vault-root': storage: object doesn't exist
INFO Failed applying configuration file: /config/vault-configurer/vault-config.yml , sleeping for 32s before trying again

So our configs defined in externalConfig are not being applied.
Steps To Reproduce

Using the latest version of banks-vault (1.31.2), operator (1.22.3), webhook (1.21.3) and hashicorp vault (1.18).
Using the Vault config:

apiVersion: "vault.banzaicloud.com/v1alpha1"
kind: "Vault"
metadata:
  name: "vault"
  namespace: vault
spec:
  size: 3
  image: hashicorp/vault:1.18
  bankVaultsImage: ghcr.io/bank-vaults/bank-vaults:latest

  affinity:
    nodeAffinity:
      preferredDuringSchedulingIgnoredDuringExecution:
      - weight: 1
        preference:
          matchExpressions:
            - key: default
              operator: In
              values:
              - "true"

  serviceAccount: vault

  serviceType: ClusterIP

  ingress:
    annotations:
      external-dns.alpha.kubernetes.io/exclude: "true"
      cert-manager.io/cluster-issuer: ca-internal-issuer
      nginx.ingress.kubernetes.io/backend-protocol: HTTPS
      nginx.ingress.kubernetes.io/ssl-passthrough: 'true'
    spec:
      # prerequisite: Nginx Ingress controller configured for internal access
      ingressClassName: private
      tls:
        - hosts:
          - vault.my.host
          secretName: vault-private-tls
      rules:
        - host: vault.my.host
          http:
            paths:
              - path: /
                pathType: ImplementationSpecific
                backend:
                  service:
                    name: vault
                    port:
                      number: 8200

  unsealConfig:
    google:
      kmsKeyRing: "xxxx"
      kmsCryptoKey: "xxxx"
      kmsLocation: "xxx"
      kmsProject: "xxxx"
      storageBucket: "xxx"

  caNamespaces:
    - "cert-manager"

  # A YAML representation of a final vault config file.
  # See https://developer.hashicorp.com/vault/docs/configuration for more information.
  config:
    disable_mlock: true
    storage:
        gcs:
            bucket: xxxx
            ha_enabled: true
    seal:
        gcpckms:
            project: xxxx
            region: us-central1
            key_ring: xxxx
            crypto_key: xxxx
    listener:
      - tcp:
          address: "[::]:8200"
          cluster_address: "[::]:8201"
          tls_cert_file: /vault/tls/server.crt
          tls_key_file: /vault/tls/server.key
          tls_client_ca_file: /vault/tls/ca.crt
          tls_disable_client_certs: true
    api_addr: https://vault:8200
    cluster_addr: https://vault:8201
    ui: true

  externalConfig:
    policies:
      - name: allow_secrets
        rules: |
          path "secret/*" {
            capabilities = ["read", "list"]
          }
      - name: allow_pki
        rules: |
          path "pki/*" {
            capabilities = ["create", "read", "update", "delete", "list"]
          }
    auth:
      - type: kubernetes
        roles:
          # Allow every pod in the default namespace to use the secret kv store
          - name: default
            bound_service_account_names: ["*"]
            bound_service_account_namespaces: ["*"]
            policies: ["allow_secrets"]
            ttl: 1h
          # Allow the cert-manager pod in the cert-manager namespace to use the PKI engine
          - name: cert-manager
            bound_service_account_names: ["cert-manager"]
            bound_service_account_namespaces: ["cert-manager"]
            policies: ["allow_pki"]
            ttl: 1h
    secrets:
      - path: secret
        type: kv
        description: General secrets.
        options:
          version: 2

      - path: pki
        type: pki
        description: Vault PKI Backend
        config:
          default_lease_ttl: 168h
          max_lease_ttl: 720h
        configuration:
          config:
          - name: urls
            issuing_certificates: https://vault.default:8200/v1/pki/ca
            crl_distribution_points: https://vault.default:8200/v1/pki/crl
          root/generate:
          - name: internal
            common_name: vault.default
          roles:
          - name: default
            allowed_domains: localhost,pod,svc,default
            allow_subdomains: true
            generate_lease: true
            ttl: 1m

Logs
Vault:

Couldn't start vault with IPC_LOCK.  Log data will stream in below:

2024-10-22T11:09:05.124Z [INFO]  core: Initializing version history cache for core
2024-10-22T11:09:05.124Z [INFO]  events: Starting event system
2024-10-22T11:09:05.125Z [INFO]  core: stored unseal keys supported, attempting fetch
2024-10-22T11:09:05.195Z [INFO]  core.cluster-listener.tcp: starting listener: listener_address=[::]:8201
2024-10-22T11:09:05.195Z [INFO]  core.cluster-listener: serving cluster requests: cluster_listen_address=[::]:8201
2024-10-22T11:09:05.195Z [INFO]  core: vault is unsealed
2024-10-22T11:09:05.195Z [INFO]  core: unsealed with stored key
2024-10-22T11:09:05.195Z [INFO]  core: entering standby mode
2024-10-22T11:09:19.577Z [WARN]  core.cluster-listener: no TLS config found for ALPN: ALPN=["req_fw_sb-act_v1"]
2024-10-22T11:09:24.577Z [WARN]  core.cluster-listener: no TLS config found for ALPN: ALPN=["req_fw_sb-act_v1"]
2024-10-22T11:09:25.263Z [INFO]  core: acquired lock, enabling active operation
2024-10-22T11:09:25.552Z [INFO]  core: post-unseal setup starting
2024-10-22T11:09:25.773Z [INFO]  core: loaded wrapping token key
2024-10-22T11:09:25.773Z [INFO]  core: successfully setup plugin runtime catalog
2024-10-22T11:09:25.773Z [INFO]  core: successfully setup plugin catalog: plugin-directory=""
2024-10-22T11:09:26.077Z [INFO]  core: successfully mounted: type=system version="v1.18.0+builtin.vault" path=sys/ namespace="ID: root. Path: "
2024-10-22T11:09:26.107Z [INFO]  core: successfully mounted: type=identity version="v1.18.0+builtin.vault" path=identity/ namespace="ID: root. Path: "
2024-10-22T11:09:26.228Z [INFO]  core: successfully mounted: type=kv version="v0.20.0+builtin" path=secret/ namespace="ID: root. Path: "
2024-10-22T11:09:26.251Z [INFO]  core: successfully mounted: type=cubbyhole version="v1.18.0+builtin.vault" path=cubbyhole/ namespace="ID: root. Path: "
2024-10-22T11:09:26.838Z [INFO]  core: successfully mounted: type=token version="v1.18.0+builtin.vault" path=token/ namespace="ID: root. Path: "
2024-10-22T11:09:26.902Z [INFO]  core: successfully mounted: type=kubernetes version="v0.20.0+builtin" path=kubernetes/ namespace="ID: root. Path: "
2024-10-22T11:09:27.079Z [INFO]  rollback: Starting the rollback manager with 256 workers
2024-10-22T11:09:27.079Z [INFO]  rollback: starting rollback manager
2024-10-22T11:09:27.079Z [INFO]  core: restoring leases
2024-10-22T11:09:27.095Z [INFO]  expiration: lease restore complete
2024-10-22T11:09:27.321Z [INFO]  identity: entities restored
2024-10-22T11:09:27.338Z [INFO]  identity: groups restored
2024-10-22T11:09:27.548Z [INFO]  core: usage gauge collection is disabled
2024-10-22T11:09:28.098Z [INFO]  core: post-unseal setup complete

Configurator:
2024/10/22 11:08:58 INFO vault metrics exporter enabled: :9091/metrics
2024/10/22 11:08:58 INFO applying config file: /config/vault-configurer/vault-config.yml
2024/10/22 11:08:58 INFO checking if vault is sealed...
2024/10/22 11:08:58 INFO watching directory for changes: /config/vault-configurer/
2024/10/22 11:08:58 INFO vault is unsealed, configuring...
2024/10/22 11:08:58 ERROR error configuring vault: unable to get key 'vault-root': error getting data: error getting object for key 'vault-root': storage: object doesn't exist
2024/10/22 11:08:58 INFO Failed applying configuration file: /config/vault-configurer/vault-config.yml , sleeping for 500ms before trying again

Additional Information
No response

Banks-vault had not initialized the vault correctly, the root token was not in the bucket files. So we reinstalled everything and this time the vault was initialized correctly and the root token was generated.


Add BaseRotatingHandler

https://docs.python.org/3/library/logging.handlers.html#baserotatinghandler
Not sure if we need to test this directly or we can test it with the RotatingFileHandler  and others.

Codecov Report

Merging #25 (a38fc86) into main (e1e6b3f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #25   +/-   ##
=======================================
  Coverage   72.80%   72.80%
=======================================
  Files           8        8
  Lines        1059     1059
=======================================
  Hits          771      771
  Misses        288      288

Continue to review full report at Codecov.

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

Related:  https://github.com/python/cpython/issues/94327

Thanks!


Adds size flexibility and scrollability to hud menu

Based off of comments and discoveries from #817

Where's the toggle for the dev hud? I'm not seeing an obvious one in the app.

@dylanesque : It's hidden so it won't be accessed by non-devs. Go to /dev (or /app/dev on the deployed version).

@dylanesque : It's hidden so it won't be accessed by non-devs. Go to /dev (or /app/dev on the deployed version).

Got it. I had been thinking on and off if there was a simple way to ditch the fixed positioning and get the same effect, but all that I'm coming up with at the moment is "make its parent a Grid container", which is like using a battle-ax to kill an ant in this instance.

Thanks for giving it some thought. I also haven't been able to come up with another solution, especially if we want it to be flexible.
The tests look like they're pending in the github page, but when I go to the details, Travis says they've passed, so I'm going to merge.
@ethanbb and @jlangevin : Thanks for the input. If this doesn't match up with what you were thinking, let me know. It should be pretty easy to undo/tweak these changes.

@knod Sorry didn't get to this, but it looks good!


Simpler downloads page

Pinging @bitemyapp and @chrisdone.

The OS-specific instructions are 404's. I pushed a fix.

I like it. The starter file is handy. Works out of the box.


targets: change flash_algorithms from map to array.

This makes it consistent with all other lists of stuff in the
YAMLs, which all are arrays with "name" fields if applicable.

Oh uff, please mr @Dirbaio could you fix the conflict? :)

bors r+


Recycle mv machine

Objects in the recycle get stuck.
The itemwise filter-injector is not working with it.

fixed


stalled installation win7

I am trying to install phy on Window 7, using the instructions provided. I installed POwerShell and I have the version (up to date) of C++ already installed.
The installation in the PowerShell window stops right after I start (image)

How long does it stop for?

I checked after 3 hours and nothing happened, so I closed the session. If
you are coming down to old street for electrophysiology club, you can have
a go yourself.
On Tuesday, 29 September 2015, Max Hunter<EMAIL_ADDRESS>wrote:

How long does it stop for?
—
Reply to this email directly or view it on GitHub
https://github.com/kwikteam/phy/issues/655#issuecomment-144132618.

--
Sent from Mobile

Pressing Ctrl+C solved this issue. Still not sure what/why happened...


Upgrade from 4.0.9 to 4.0.10 then wrangler dev --remote fails

✘ [ERROR] Error on remote worker: ParseError: A request to the Cloudflare API (/accounts/<my-cloudflare-account-id>/workers/scripts/youwave-worker/edge-preview) failed.

      at throwFetchError
  (/home/chientrm/youwave/worker/node_modules/wrangler/wrangler-dist/cli.js:121071:17)
      at fetchResult
  (/home/chientrm/youwave/worker/node_modules/wrangler/wrangler-dist/cli.js:121038:5)
      at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
      at async createPreviewToken
  (/home/chientrm/youwave/worker/node_modules/wrangler/wrangler-dist/cli.js:128483:29)
      at async createWorkerPreview
  (/home/chientrm/youwave/worker/node_modules/wrangler/wrangler-dist/cli.js:128504:17)
      at async start
  (/home/chientrm/youwave/worker/node_modules/wrangler/wrangler-dist/cli.js:147982:34) {
    text: 'A request to the Cloudflare API
  (/accounts/33fb3b5f3143369b69518f1e5f2ae750/workers/scripts/youwave-worker/edge-preview) failed.',
    notes: [
      {
        text: 'Uncaught ReferenceError: exports is not defined\n' +
          '  at index.js:418:5 in node_modules/itty-router/cjs/index.js\n' +
          '  at index.js:15:56 in __init\n' +
          '  at index.js:470:23 in node_modules/itty-router-extras/router/ThrowableRouter.js\n' +
          '  at index.js:18:52 in __require2\n' +
          '  at index.js:484:27 in node_modules/itty-router-extras/router/index.js\n' +
          '  at index.js:18:52 in __require2\n' +
          '  at index.js:513:75 in node_modules/itty-router-extras/index.js\n' +
          '  at index.js:18:52 in __require2\n' +
          '  at index.js:8402:41\n' +
          ' [code: 10021]'
      }
    ],
    location: undefined,
    kind: 'error',
    code: 10021
  }

Everything works fine with 4.0.9

Hey there!  Could I see your package.json (or ideally, the whole repo)?  Also what version of wrangler are you on?  Mine doesn't even have the --remote flag...
Also a note, since I see the stack includes ThrowableRouter and itty-router-extras (which is now deprecated) in general, I'd recommend using the extras now in v4.x (see https://itty.dev/itty-router/migration-guide).

In the meantime, I've hopefully rolled out a reversal in 4.0.11, that for me at least... works in esm/module land, require/cjs land, etc.

@chientrm could you try itty-router@next and let me know if that works?

@chientrm could you try itty-router@next and let me know if that works?

Works as intended 💋

Yes! The change has been merged in and is live as 4.0.12 now :)


Editor: Unify endpoints for saving

We have two endpoints for saving data from the editor frontend, one for metadata and one for all the rest. With the way the editor is set up, two seperate endpoints are wholly unnecessary, so this unifies them into one endpoint instead.
Will break the frontend without respective changes: https://github.com/opencast/opencast-editor/pull/1472
Why is this aimed at legacy?
This constitutes a breaking change. Due to how the project structure of the editor frontend is set up, if this change does not go into legacy, no more frontend releases can go into legacy.

I'm not per se against this PR, but in light of the recently passed proposal, changes to legacy branches have to be explained in order to prevent breaking changes (see https://groups.google.com/a/opencast.org/g/dev/c/VwYqk3vXdd0 and https://github.com/opencast/opencast/pull/6239).

The recent release of Opencast 15.12 is intended to be the final minor release for 15. Please retarget your PR or explain why it warrants another Opencast 15 release.

This PR should target 17.x. Because there will be no editor release for 16.x.

Rebased onto r/17.x


Why can't parse 3857?

Why can't our package parse 3857?
Use this code:
parse.from_epsg_code(3857)

This will return None

from_epsg_code() uses the website spatialreference.org.  spatialreference.org does not have an entry for EPSG:3857 (people have put it on there on their own).  I don't think the website is even being maintained anymore (Last Updated 2013).
Moving to a similar website like EPSG.io would be an improvement.

Switched to using the more up-to-date epsg.io as the source when calling from_epsg_code(). Now available from v1.0.2 on pypi.
pycrs.parse.from_epsg_code(3857) now works.


Temporary fix for TFP version

Can we please fix it properly instead? I've found the issue - see #1473

Closing in favour of actually fixing the issue: #1473 (unit tests pass!)


DIALOGUE implementation completeness?

Hi all,
impressive work!
I am very interested in the python implementation of DIALOGUE.
Do you think this is in a state where I can use it for an important project, or do you recommend me to stick to the original for the time being?
Thanks!

Hi pakiessling,
Thanks so much for your interest in our work! Unfortunately, the pertpy implementation of DIALOGUE is partially incomplete at the moment: you can see that in our tutorial notebook, the extraction of significant genes is not the same as in the original, and we also don't have the PhenoZ score implemented.
However, the extraction of significant genes is considerably faster with the HLM of the original, and I never could get PhenoZ scores to work in R, personally. So, if these latter steps in the pipeline are of less importance to you vs. setting up an R environment, I can say that you can feel confident that the MCP scores assigned in pertpy are extremely close to those in the R implementation. Additionally, we provide an experimental solver that disregards the order in which the cell types are passed, which is not a feature of the original implementation (see https://github.com/livnatje/DIALOGUE/issues/29).
Overall, I would still recommend trying the original implementation, and barring being able to get that to work, switch to the pertpy implementation.
Thanks for your patience!

Thank you for your detailled reply!


Login page not working.

Hi @Vimlesh-Kumar 👋, did you get the latest commit?

Hi @Vimlesh-Kumar 👋, did you get the latest commit?

I have tried to login on this login page but I am unable to login as a test user and human verification also not working properly.

Ok @Vimlesh-Kumar let me take look immediately, and update you on it

hello @Vimlesh-Kumar , i have fixed the problem. You can login now.
It was a CSRF cookie missing issue.  Solution i had to disable CSRF for demo app in production, because of different domain for backend and frontend. Another reason,  cookies in general does not work correctly for PUBLIC PaaS platforms like .herokuapp or .railway ). Hopefully in your own production site, you will have SAME root-domain for frontend and backend.

Thanks for update, Now working fine.


bug: icons of nvim cmp

Did you check docs and existing issues?

[X] I have read all the lazy.nvim docs
[X] I have searched the existing issues of lazy.nvim
[x] I have searched the exsiting issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.8.2
Operating system/version
arch linux
Describe the bug
Then i using lazy instead of packer my nvim cmp icons become brown
Steps To Reproduce
1 download the nvim config from my repo https://github.com/Bogdan-Torkhov/dots
2 type nvim main.lua and try to write something and if cmp works it will show only brown icons
3 try to clone my repo with the latest commit that has packer
Expected Behavior
I expected to see normal lspkind plugin icons
Repro
These is my plugins.lua and you need to add it to init.lua:
local lazypath = vim.fn.stdpath("data") .. "/lazy/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
	vim.fn.system({
		"git",
		"clone",
		"--filter=blob:none",
		"https://github.com/folke/lazy.nvim.git",
		"--branch=stable", -- latest stable release
		lazypath,
	})
end
vim.opt.rtp:prepend(lazypath)
vim.g.mapleader = " "
require("lazy").setup({
	config = function(plugin)
		vim.opt.rtp:append(plugin.dir .. "/custom-rtp")
	end,
	-- dracula theme
	"dracula/vim",
	-- file viewer
	"nvim-tree/nvim-tree.lua",
	"nvim-tree/nvim-web-devicons",
	-- telescope
	"nvim-telescope/telescope.nvim",
	"nvim-lua/plenary.nvim",
	"cljoly/telescope-repo.nvim",
	"nvim-telescope/telescope-file-browser.nvim",
	"nvim-telescope/telescope-project.nvim",
	{
		"nvim-telescope/telescope-fzf-native.nvim",
		build = "cmake -S. -Bbuild -DCMAKE_BUILD_TYPE=Release && cmake --build build --config Release && cmake --install build --prefix build",
	},
	-- lsp plugins
	"neovim/nvim-lspconfig",
	"hrsh7th/nvim-cmp",
	"hrsh7th/cmp-nvim-lsp",
	"hrsh7th/cmp-nvim-lua",
	"hrsh7th/cmp-nvim-lsp-signature-help",
	"hrsh7th/cmp-path",
	"hrsh7th/cmp-buffer",
	"williamboman/mason.nvim",
	"williamboman/mason-lspconfig.nvim",
	"hrsh7th/cmp-vsnip",
	"jose-elias-alvarez/null-ls.nvim",
	"jay-babu/mason-null-ls.nvim",
	"L3MON4D3/LuaSnip",
	"saadparwaiz1/cmp_luasnip",
	"rafamadriz/friendly-snippets",
	{ "nvim-treesitter/nvim-treesitter", build = ":TSUpdate" },
	-- vs code like icons(
	"onsails/lspkind.nvim",
	{
		"iamcco/markdown-preview.nvim",
		build = "cd app && npm install",
		init = function()
			vim.g.mkdp_filetypes = { "markdown" }
		end,
		ft = { "markdown" },
	},
	-- git
	"dinhhuy258/git.nvim",
	"lewis6991/gitsigns.nvim",
	{ "romgrk/barbar.nvim", wants = "nvim-web-devicons" },
	{
		"nvim-lualine/lualine.nvim",
		dependencies = { "nvim-tree/nvim-web-devicons", lazy = true },
	},
	-- auto-pairs and auto-tags
	"windwp/nvim-autopairs",
	"windwp/nvim-ts-autotag",
	-- nvim transparency
	"xiyaowong/nvim-transparent",
	-- wakatime plugin
	"wakatime/vim-wakatime",
	-- terminal emulator inside neovim
	"akinsho/toggleterm.nvim",
	-- colorize colors(hex,rgb)
	"norcalli/nvim-colorizer.lua",
})

in a minimal config you try to get rid of as many plugins as possible
this probably has something to do with the loading order. perhaps try loading lspkind in nvim-cmp manually
it most likely isn't a lazy.nvim issue

How to do these?

And why it's working normally with packer?

-- import nvim-cmp plugin safely
local cmp_status, cmp = pcall(require, "cmp")
if not cmp_status then
return
end
-- import luasnip plugin safely
local luasnip_status, luasnip = pcall(require, "luasnip")
if not luasnip_status then
return
end
-- import lspkind plugin safely
local lspkind_status, lspkind = pcall(require, "lspkind")
if not lspkind_status then
return
end
require("luasnip/loaders/from_vscode").lazy_load()
vim.o.completeopt = "menu,menuone,noselect"
cmp.setup({
snippet = {
expand = function(args)
luasnip.lsp_expand(args.body)
end,
},
mapping = cmp.mapping.preset.insert({
[""] = cmp.mapping.select_next_item(), -- previous suggestion
[""] = cmp.mapping.select_prev_item(), -- next suggestion
[""] = cmp.mapping.scroll_docs(-4),
[""] = cmp.mapping.scroll_docs(4),
[""] = cmp.mapping.complete(), -- show completion suggestions
[""] = cmp.mapping.abort(), -- close completion window
[""] = cmp.mapping.confirm({ select = false }),
}),
-- sources for autocompletion
sources = cmp.config.sources({
{ name = "nvim_lsp" }, -- lsp
{ name = "luasnip" }, -- snippets
{ name = "path" }, -- file system paths
{ name = "buffer" }, -- text within current buffer
{ name = "nvim_lua" }, -- lua
{ name = "nvim_lsp_signature_help" }, -- help for signatures
{ name = "vsnip" }, -- vs code snippets
}),
-- configure vs-code like icons
formatting = {
fields = { "kind", "abbr", "menu" },
format = lspkind.cmp_format({
mode = "symbol",
maxwidth = 50,
ellipsis_char = "...",
}),
},
})
These is my cmp config(in it i load lspkind

And why it's working normally with packer?

because packer and lazy don't behave exactly the same

And what I need to do to back normal colors?

And can we continue talk in telegram?

I think maybe lazy have main color. And it used as color for lspkind.

And if it is exiced where to set it?(didn't finded in docs)

@folke Do you know what to do with these issue?

This is definitely a user error and i can't be expected to debug everyone's issues. the repro (s) you provided are also dat from minimal.
You'll have to figure this out yourself. If you do find this is caused by a bug in lazy, then feel free to open another issue. But i doubt this has anything to do with lazy

I think it's lazy issue because with packer it works normally

@folke I think these real issue. Your plugin installing plugins like they don't need to be installed. Just check how packer installs themes and compare it with your lazy and maybe fix these issue. Because with packer I don't have to fix it, it's just showing normal colored icons for lspkind.

You still haven't told me what the issue is here. Please provide a minimal repro together with the exact steps to reproduce. The template for the minimal init.lua is shown when you submit a new issue

You need a dracula/vim theme and lsp-config,nvim-cmp,lspkind

Can we have a talk? Do you have element or telegram or whats app?

I think I can't fully tell these in text

Where are you???