Use Page.DisposeAsync and Browser.DisposeAsync hang forever

Before you file a bug, have you:

Tried upgrading to newest version of PuppeteerSharp, to see if your issue has already been resolved and released?
Yes, I am using the latest version.

Checked existing open and closed issues, to see if the issue has already been reported?
Yes, I do search quite a lots and also try to find solution from the original puppeteer repo also.

Tried reproducing your problem in a new isolated project?
Not yet, I might try next time but since it's a production issue, so I need to fix it quickly.

Considered if this is a general question and not a bug?. For general questions please use StackOverflow.
Not sure but after I change to use Dispose() seems these chrome process close correctly.

Description
I am running puppeteer-sharp with docker, I found there are quite a lot of zombie chrome process never be kill. I was try to use tini as entry point (tips from here tips) also no luck. I was check the logs, I found that some DisposeAsync sometime never complete. So I change to use Dispose instead, looks good so far.
Complete minimal example reproducing the issue
There is not complete code right now, I might update later. I just simply to call DisposeAsync.
E.g.
var options = new LaunchOptions
{
    Headless = Configurations.Puppeteer.Headless,
    Args = new string[]
    {
        "--no-sandbox",
        "--disable-dev-shm-usage",
        "--incognito"
    }
};
Browser = await Puppeteer.LaunchAsync(options, loggerFactory);
var browserPages = await Browser.PagesAsync();
if (browserPages.Length > 0) {
    Page = browserPages[0];
    await Task.WhenAll(browserPages.Skip(1).Select(x => x.CloseAsync()));
} else {
    Page = await Browser.NewPageAsync();
}

// some other steps

logger.LogInformation("browser is disposing");
await Page.DisposeAsync();
await Browser.DisposeAsync();
// this log sometime never print out
logger.LogInformation("browser is disposed");

Expected behavior:
Browser dispose correctly, and no zombie process
Actual behavior:
I found some zombie process never be kill
Versions

Which version of PuppeteerSharp are you using?

Version: 2.0.3

Which .NET runtime and version are you targeting? E.g. .NET framework 4.6.1 or .NET Core 2.0.

netcoreapp3.1, aspnet core mvc

Additional Information
Running puppeteer-sharp in docker, dockerfile similar with the one you provided, I just added tini as entry-point. (BTW, I think the docker example should also add this and run with args --disable-dev-shm-usage).

@clement128 could you try using ConfigureAwait(false) across the board?

Hi @kblok , I am running with ASP.NET Core, base on my understanding I don't need to use ConfigureAwait(false) since there is not any SynchronizationContext by default? Correct me if I am wrong.

That's true @clement128.

@kblok any updates on this issue.

Sorry @clement128 @pmdevers I will take a look tomorrow.

@clement128  @pmdevers could you help me break this project https://github.com/kblok/PuppeteerSharpOnAspNetCoreDemo?
That project is working for me.

Issue of zombie processes is resolved when switching to netcore 3.1
Issue was on net core 2.2

Hi @kblok, are you run it on docker?
@pmdevers  hmm interesting, I was running on netcore 3.1 also. with Orleans.

Upgraded the project with the issue to 3.1 and still having zombie processes.
It looks like the project is locking the chrome processess.

https://github.com/puppeteer/puppeteer/issues/1825
Have found that the following arguments resolved the zombie processes
Args = new[] {
"--headless",
"--no-sandbox",
"--disable-gpu",
"--single-process",
"--no-zygote"
}

Just a note to say before I attempt the workarounds people have listed that this is definitely still an issue "out of the box" with PuppeteerSharp 2.0.4 running in a .net core webapi server on Docker / Alpine linux.
I have a report printing setup and each time I run a report 3 new zombie chrome processes are added and never go away but rather build up over and over even with proper disposal inside a using() block.  Doesn't appear to be an issue on Windows at all.

I'm running into a problem with zombie processes too - on Windows.
I cannot reproduce it as most of the time it seems to work ok.
After hanging for a few hours I killed them off from task manager and got these log messages:
2020-09-22 18:04:27.688 +02:00 [WRN]
PuppeteerSharp.TargetClosedException: Protocol error(Runtime.callFunctionOn): Target closed. (The remote party closed the WebSocket connection without completing the close handshake.)
   at PuppeteerSharp.CDPSession.SendAsync(String method, Object args, Boolean waitForCallback)
   at PuppeteerSharp.CDPSession.SendAsync[T](String method, Object args)
   at PuppeteerSharp.ExecutionContext.ExecuteEvaluationAsync(String method, Object args)
   at PuppeteerSharp.ExecutionContext.EvaluateFunctionHandleAsync(String script, Object[] args)
   at PuppeteerSharp.WaitTask.Rerun()
   at PdfPrinter.PrintPdf(String html, String hostUrl)
   at ReportController.CreateReport(Int32 id)

I'm not sure what this means, but I'm guessing PuppeteerSharp is waiting for some response from Chromium which it never gets?

Any news on this? I'm also seeing 3-4 processes hanging on Linux

Any news on this? I'm also seeing 3-4 processes hanging on Linux

i seemingly got this working by ditching alpine and using the args mentioned above. i'm not sure how to integrate the dumb-init bit so not doing that currently.
i'm also doing a flavor of this: https://github.com/hardkoded/puppeteer-sharp/issues/996#issuecomment-471521462
except instead i'm grabbing the process early and holding on to it, then doing a refresh before checking if i need to kill it. doing it that way because the pid lookup throws an exception if it's already been terminated on linux.
still testing things, but fingers crossed that things keep behaving from here...

@pmdevers I thought I was in the clear but still having the issue. Did you solve for this? I looked at your pdf repo but don't see you using dumb init in there. Was trying to figure out how to integrate that and not coming up with something that felt like it made sense

@mrtristan no after some investigation i did not need the dumb-init this is the snipped code.
`
         browserFetcher = new BrowserFetcher();
        _options = new LaunchOptions
        {
            Headless = true,
            ExecutablePath = "/usr/bin/google-chrome",

            Args = new[] {
                "--headless",
                "--no-sandbox",
                "--disable-gpu",
                "--single-process",
                "--no-zygote"
            }
        };
        await _browserFetcher.DownloadAsync(BrowserFetcher.DefaultRevision);
        using (var browser = await Puppeteer.LaunchAsync(_options))
        using (var page = await browser.NewPageAsync())
        {
            await page.SetRequestInterceptionAsync(true);

            page.Request += async (sender, e) =>
            {
                var header = e.Request.Headers;
                header.Add("Authorization", _userService.GetToken());
                var payload = new Payload()
                {
                    Url = e.Request.Url,
                    Method = e.Request.Method,
                    Headers = header,
                };
                await e.Request.ContinueAsync(payload);
            };

            await page.GoToAsync($"{url}/{query.DossierId}/{query.Version}",
                new NavigationOptions {WaitUntil = new[] {WaitUntilNavigation.Networkidle0}});

            var data = await page.PdfDataAsync(new PdfOptions {Format = PaperFormat.A4});

`
This what i used in the Docker file to install google chrome
RUN apt-get update && apt-get install -y \  	apt-transport-https \  	curl \  	gnupg \  	--no-install-recommends \  	&& curl -sSL https://dl.google.com/linux/linux_signing_key.pub | apt-key add - \  	&& echo "deb [arch=amd64] https://dl.google.com/linux/chrome/deb/ stable main" > /etc/apt/sources.list.d/google.list \  	&& apt-get update && apt-get install -y \  	google-chrome-stable \  	--no-install-recommends \  	&& apt-get purge --auto-remove -y curl \  	&& rm -rf /var/lib/apt/lists/*
And because we use kubernetes we use de following resource limits
resources:
limits:
memory: "1.5Gi"
cpu: "500m"
This is running in production for over 6 months without any issues

@pmdevers interesting. thanks for the quick response. i seem very closely aligned to your approach but still choking. will dig in further. i'm running things on AWS ECS fargate with mostly same resource limits, just an even 1G on memory. i wonder if the environmental differences are having some bearing on things.
one thing worth calling out is that it seems like you're preinstalling chrome but you're still calling await _browserFetcher.DownloadAsync(BrowserFetcher.DefaultRevision); which is likely duplicative. i wonder if that's relevant to things working for you in some way.

@mrtristan the browserfetcher is for local development as we set the ExecutablePath to _browserFetcher.GetExecutablePath(BrowserFetcher.DefaultRevision) and add remove "--single-process" argument if in debug mode

@pmdevers gotcha.
I think i'm down to my issue being environmental. I'm able to debug and run it locally on windows, it also functions fine in my local (linux) docker container. once i push to amazon and try and run it up there in Fargate is when i seemingly randomly get it to work or not, with the scales tipped towards usually not. extremely frustrating.
seems like it's largely due to running in a constrained environment. i have to figure out how to impose those limitations locally to see if i can break things i guess.

@pmdevers if you're able, can you see if that still works on a fresh docker build? i was fine until today when a new chrome version hit the stable channel (appreciate your help getting to this point). getting an immediate failure now. curious if it's me.
i pinned the installed version to 88.0.4324.182-1 (the last stable release from earlier last month) and now it's back to working.
i've also tracked the new version's issue down to being with the single-process flag.
 ---> System.Net.WebSockets.WebSocketException (0x80004005): Unable to connect to the remote server
 ---> System.Net.Http.HttpRequestException: Connection refused
 ---> System.Net.Sockets.SocketException (111): Connection refused
   at System.Net.Http.ConnectHelper.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)
   --- End of inner exception stack trace ---
   at System.Net.Http.ConnectHelper.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)
   at System.Net.Http.HttpConnectionPool.ConnectAsync(HttpRequestMessage request, Boolean allowHttp2, CancellationToken cancellationToken)
   at System.Net.Http.HttpConnectionPool.CreateHttp11ConnectionAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at System.Net.Http.HttpConnectionPool.GetHttpConnectionAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at System.Net.Http.HttpConnectionPool.SendWithRetryAsync(HttpRequestMessage request, Boolean doRequestAuth, CancellationToken cancellationToken)
   at System.Net.Http.RedirectHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at System.Net.WebSockets.WebSocketHandle.ConnectAsyncCore(Uri uri, CancellationToken cancellationToken, ClientWebSocketOptions options)
   at System.Net.WebSockets.WebSocketHandle.ConnectAsyncCore(Uri uri, CancellationToken cancellationToken, ClientWebSocketOptions options)
   at System.Net.WebSockets.ClientWebSocket.ConnectAsyncCore(Uri uri, CancellationToken cancellationToken)
   at PuppeteerSharp.Transport.WebSocketTransport.CreateDefaultWebSocket(Uri url, IConnectionOptions options, CancellationToken cancellationToken)
   at PuppeteerSharp.Transport.WebSocketTransport.CreateDefaultTransport(Uri url, IConnectionOptions connectionOptions, CancellationToken cancellationToken)
   at PuppeteerSharp.Connection.Create(String url, IConnectionOptions connectionOptions, ILoggerFactory loggerFactory, CancellationToken cancellationToken)
   at PuppeteerSharp.Launcher.LaunchAsync(LaunchOptions options, Product product)
   --- End of inner exception stack trace ---
   at PuppeteerSharp.Launcher.LaunchAsync(LaunchOptions options, Product product)
   at PuppeteerSharp.Launcher.LaunchAsync(LaunchOptions options, Product product)
   at xxx.PuppeteerServiceTemp.ExecuteWithPageAsync[T](Func`2 callback, Nullable`1 timeoutMilliseconds) in /app/PuppeteerServiceTemp.cs:line 26
   at xxx.Program.Main(String[] args) in /app/Program.cs:line 128```

Closed due to inactivity. Feel free to reopen it if needed.

still exist this case in new version in puppeteer 8


Update server-side-apply.md

Deleted the duplicate section of the issue(Server-Side Apply for custom resources ) #43806

/assign @tengqm

/approve
Thanks.

/lgtm


canonifyurls should append baseurl to items in srcset

To support responsive images it'd be great if the canonifyurls parameter in the configuration file could also append the baseurl to each of the image sources in srcset.  This could be tough if you have to parse srcset as a long string, but would be a great addition.
Current output:

It should be doable.
See

http://www.w3.org/html/wg/drafts/html/master/semantics.html#attr-img-srcset
https://github.com/spf13/hugo/blob/master/transform/absurlreplacer.go

@rhewitt22 I have admittedly tested this mostly regarding non-srcset sources (it was vital that I could do this without losing speed ...), but it passes all the tests I could come up with. Shout if any trouble!


Support network mode for container creation in job

Fixes #492

Looks like this needs rebasing.

:+1:


Having problems with the installation

When I type "./muda.rb -h", I get this:
"read 1810030 –directory /media/Backup/Imagens/Spurdo
/usr/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in require': cannot load such file – mechanize (LoadError) from /usr/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in require'
from ./muda.rb:2:in `'"
Help, please!

Hi Eric,
I noticed your ruby version is 1.9.1 - this will not work for Muda. It must be at least 2.1.5.
Simplest thing you could do here unless you want to mess around with your OS's packages (not recommended) is to install Rbenv and its associated ruby-build plugin (it's simple, just a couple of git clones and two lines in your bash_profile). Once that's done, you should be able to do something similar to this:
rbenv install 2.1.5
cd muda
rbenv local 2.1.5
gem install bundler
bundle install
muda -h

If any of that fails, make sure your distro's software development tools are installed. (Debian calls this build-essential, Rhellikes call it the "Development Tools" group)

When i type rbenv install 2.1.5 I just get this: eric@Eric:~$ rbenv install 2.1.5
ruby-build: definition not found: 2.1.5

Also,
eric@Eric:~/.rbenv$ cd
eric@Eric:~$ rbenv install 2.1.5
ruby-build: definition not found: 2.1.5
eric@Eric:~$ gem install bundler
ERROR:  While executing gem ... (Gem::FilePermissionError)
You don't have write permissions into the /var/lib/gems/1.9.1 directory.
eric@Eric:~$ sudo gem install bundler
Successfully installed bundler-1.9.4
1 gem installed
Installing ri documentation for bundler-1.9.4...
unable to convert U+2015 from UTF-8 to US-ASCII for lib/bundler/friendly_errors.rb, skipping
Installing RDoc documentation for bundler-1.9.4...
unable to convert U+2015 from UTF-8 to US-ASCII for lib/bundler/friendly_errors.rb, skipping
eric@Eric:~$ bundle install
Could not locate Gemfile or .bundle/ directory

Now when I try to update, i get this:
eric@Eric:~$ rbenv install 2.1.5
rbenv: no such command `install'

Trying with sudo:
eric@Eric:~$ sudo rbenv install 2.1.5
[sudo] password for eric:
ruby-build: definition not found: 2.1.5
eric@Eric:~$

You shouldn't have to use sudo for any of this, something is very wrong. When you did the rbenv and ruby-build git clones, did you do it using your account in your home directory?

Yes. OK.

Hi again,
Could you try the latest release if you happen to have Java kicking around? You should find it a bit easier to use :)

Going to close this since I haven't heard from you in a while - please feel free to reopen if you're still having difficulties with the new Java version :)

I just gave up. I'm using #!++ and still can't update the ruby. Thanks for the support.
Enviado do Yahoo Mail no Android
De:"Michael Parks"<EMAIL_ADDRESS>Data:14:37 seg, 8 de jun de PM
Assunto:Re: [muda] Having problems with the installation (#8)
Going to close this since I haven't heard from you in a while - please feel free to reopen if you're still having difficulties with the new Java version :)
—
Reply to this email directly or view it on GitHub.￼

Did you try the new Java release? The rewrite was, in part, so the messing
around with Ruby was no longer necessary.
On Mon, Jun 8, 2015 at 11:50 AM, ericfariase<EMAIL_ADDRESS>wrote:

I just gave up. I'm using #!++ and still can't update the ruby. Thanks for
the support.
Enviado do Yahoo Mail no Android
De:"Michael Parks"<EMAIL_ADDRESS>Data:14:37 seg, 8 de jun de PM
Assunto:Re: [muda] Having problems with the installation (#8)
Going to close this since I haven't heard from you in a while - please
feel free to reopen if you're still having difficulties with the new Java
version :)
—
Reply to this email directly or view it on GitHub.￼
—
Reply to this email directly or view it on GitHub
https://github.com/Karunamon/muda/issues/8#issuecomment-110090752.

I didn't know that there was a new release. Where can I get it?
Enviado do Yahoo Mail no Android
De:"Michael Parks"<EMAIL_ADDRESS>Data:15:14 seg, 8 de jun de PM
Assunto:Re: [muda] Having problems with the installation (#8)
Did you try the new Java release? The rewrite was, in part, so the messing
around with Ruby was no longer necessary.
On Mon, Jun 8, 2015 at 11:50 AM, ericfariase<EMAIL_ADDRESS>wrote:

I just gave up. I'm using #!++ and still can't update the ruby. Thanks for
the support.
Enviado do Yahoo Mail no Android
De:"Michael Parks"<EMAIL_ADDRESS>Data:14:37 seg, 8 de jun de PM
Assunto:Re: [muda] Having problems with the installation (#8)
Going to close this since I haven't heard from you in a while - please
feel free to reopen if you're still having difficulties with the new Java
version :)
—
Reply to this email directly or view it on GitHub.￼
—
Reply to this email directly or view it on GitHub
https://github.com/Karunamon/muda/issues/8#issuecomment-110090752.

—
Reply to this email directly or view it on GitHub.￼


Add scheme to endpoint value of s3 secret

The value of the s3 endpoint in tekton-results-s3 secret is missing the
URL scheme (host copied as it is from Vault) this makes connections from
tekton-results fail. This patch adds the "https://" scheme

/lgtm

/lgtm

@Mo3m3n please update the commit message and pull request description describing what this change does and why the change is needed.

Done

@Michkov Can you move this PR forward?


Invite list not responsive in mobile view

Describe the bug
Invite list is not responsive in mobile view
Expected behavior
It should responsive in mobile view also
Screenshots
If applicable, add screenshots to help explain your problem.


Once rejected an new evidence item can not be recovered

I accidentally hit the reject button on this evidence item:
https://civic.genome.wustl.edu/admin/evidence_items/749
Is it possible to create path at least in the admin interface to change the status back to pending or accepted?

We should retain the status history in these cases.

To summarize, in the admin interface, admins should be able to manually change the status of an evidence item.
In the front end, if a revision is proposed for a rejected evidence item, the status of that evidence item will be returned to pending/submitted.  This can effectively be a mechanism for appeal of a rejected a evidence item.

Admins are now presented with status change buttons in the admin interface. An accepted item can be rejected or "resubmitted" (maybe need a better name for that - basically just puts it back in "submitted" state) and a rejected item can be accepted or resubmitted.
If a revision is suggested to a rejected evidence item, it will now automatically change back to "pending".

When I accepted a revision on the evidence item again, it seemed to work but the status was still rejected. Refreshing page did not help.
Using the resubmit option in the admin interface did work though.

In this case - proposing the revision is what will move the evidence item back to Pending, if the revision was pre-existing, accepting it won't change the status.
Both cases should probably trigger the status change though - that's an easy fix!

Ah, now I understand.  Yeah, I agree that both cases should trigger the change.

Update pushed to master - both suggesting a change to a rejected evidence item and accepting a change to a rejected evidence item will cause the evidence item to go back into a submitted state!


Maintenance: Reviewed and improved JavaDoc, increased API contract en…

…forcement and corresponding test cases.

Coverage increased (+0.02%) to 91.966% when pulling d0ad0998b8a27c1c211e83e2e0537cb1a68b3317 on maintenance/javadoc-and-api-contracts into 4bbb52334d10a1bb20c300d9057de13b85c67507 on develop.


reachabilitySerialQueue assignment type

I am using swift and when building reachability using the cocoa pods swift branch, I get an error on line 44 of the main:
@property (nonatomic, strong) dispatch_queue_t          reachabilitySerialQueue;

to fix the error i change it to:
@property (nonatomic, assign) dispatch_queue_t          reachabilitySerialQueue;

I cloned the git just now on the commandline and get the compile error:
...SuperSolitaire/Classes/Reachability.m:44:1: Property with 'retain (or strong)' attribute must be of object type
(dispatch_queue_t is not an NSObject and can not receive retain messages)
@property (nonatomic, strong) dispatch_queue_t          reachabilitySerialQueue;

Surely this line should be
@property (nonatomic) dispatch_queue_t reachabilitySerialQueue;

No it should be strong, its valid for all versions above iOS6
we don't support iOS5 (if you read the docs they tell you to use an older tag that still supports iOS5). for reference dispatch_queue_t became a retainable object as of iOS6

ah, spotted.


CIF-1447 - The Core CIF Components must use a service user to access configurations

Description
Add the service user mapping to the content package.
Implement an adapter factory that adapts a resource to a specific configuration object using a service resolver. I chose this approach because we cannot adapt directly to a ValueMap.
The configuration object is a wrapper around the ValueMap and it's in the public space, so HTL scripts can use them if needed.
The MagentoGraphqlClient has been updated to use this new method.
Related Issue
CIF-1447
Motivation and Context
Align our implementation with the security requirements that state that all access to /conf should be done via service users, not directly manipulating the ACLs
How Has This Been Tested?
Functional testing, unit tests
Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes and the overall coverage did not decrease.
[ ] All unit tests pass on CircleCi.
[x] I ran all tests locally and they pass.

Following the last comment from @LSantha , I realised you probably also need to update StoreConfigExporterImpl to make everything consistent, no?


Can this be run on Debian and alternate board to Raspberry Pi?

I am having trouble getting Raspberry Pi, they sell alternate boards that can run Debian do you think that will work for this software or it has to be Rasbian OS?

Hi @philmurp, our software is targeted for the Raspberry Pi OS only, it is possible to modify it to support other Debian based OS. But we currently don’t have the capacity to work on this. You are welcome to contribute to the project and extend it to support other OS.


Updated link class to find

updated class to find in get_host_link.
from text-info cursor-pointer to is link-info cursor-pointer

Hello,
Can i ask you?
in which file I change?
from text-info cursor-pointer to is link-info cursor-pointer
Have great day
Thank you

you can find details on commit https://github.com/loblab/noip-renew/pull/40/commits/99d80ab7fd501787951de8c2612ff5062ed4884c


[ALLUXIO-2182]  Add tests for block worker metrics

https://alluxio.atlassian.net/browse/ALLUXIO-2182?jql=text ~ "metrics"
Increase the test coverage for metrics

@aaudiber  if you're okay with this pattern, I can add it for the rest of metrics.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19270/Build result: FAILURE[...truncated 2645 lines...][JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/client/fs/pom.xml to org.alluxio/alluxio-core-client-fs/1.8.0-SNAPSHOT/alluxio-core-client-fs-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/client/fs/target/alluxio-core-client-fs-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-core-client-fs/1.8.0-SNAPSHOT/alluxio-core-client-fs-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/client/fs/target/alluxio-core-client-fs-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-core-client-fs/1.8.0-SNAPSHOT/alluxio-core-client-fs-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/client/fs/target/alluxio-core-client-fs-1.8.0-SNAPSHOT-tests.jar to org.alluxio/alluxio-core-client-fs/1.8.0-SNAPSHOT/alluxio-core-client-fs-1.8.0-SNAPSHOT-tests.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/common/pom.xml to org.alluxio/alluxio-core-common/1.8.0-SNAPSHOT/alluxio-core-common-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/common/target/alluxio-core-common-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-core-common/1.8.0-SNAPSHOT/alluxio-core-common-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/common/target/alluxio-core-common-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-core-common/1.8.0-SNAPSHOT/alluxio-core-common-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/common/target/alluxio-core-common-1.8.0-SNAPSHOT-tests.jar to org.alluxio/alluxio-core-common/1.8.0-SNAPSHOT/alluxio-core-common-1.8.0-SNAPSHOT-tests.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/worker/pom.xml to org.alluxio/alluxio-core-server-worker/1.8.0-SNAPSHOT/alluxio-core-server-worker-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/logserver/pom.xml to org.alluxio/alluxio-logserver/1.8.0-SNAPSHOT/alluxio-logserver-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/client/pom.xml to org.alluxio/alluxio-core-client/1.8.0-SNAPSHOT/alluxio-core-client-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/pom.xml to org.alluxio/alluxio-examples/1.8.0-SNAPSHOT/alluxio-examples-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/target/alluxio-examples-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-examples/1.8.0-SNAPSHOT/alluxio-examples-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/target/alluxio-examples-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-examples/1.8.0-SNAPSHOT/alluxio-examples-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/pom.xml to org.alluxio/alluxio-core/1.8.0-SNAPSHOT/alluxio-core-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/proxy/pom.xml to org.alluxio/alluxio-core-server-proxy/1.8.0-SNAPSHOT/alluxio-core-server-proxy-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/proxy/target/alluxio-core-server-proxy-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-core-server-proxy/1.8.0-SNAPSHOT/alluxio-core-server-proxy-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/proxy/target/alluxio-core-server-proxy-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-core-server-proxy/1.8.0-SNAPSHOT/alluxio-core-server-proxy-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/shell/pom.xml to org.alluxio/alluxio-shell/1.8.0-SNAPSHOT/alluxio-shell-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/shell/target/alluxio-shell-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-shell/1.8.0-SNAPSHOT/alluxio-shell-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/shell/target/alluxio-shell-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-shell/1.8.0-SNAPSHOT/alluxio-shell-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/tests/pom.xml to org.alluxio/alluxio-tests/1.8.0-SNAPSHOT/alluxio-tests-1.8.0-SNAPSHOT.pomchannel stoppedArchiving artifacts
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19271/
Test PASSed.

@yupeng9 This style looks good to me, except that I'd prefer to verify return values instead of method invocations
Do you have ideas on how we could write integration tests to test the end-to-end metrics system behavior?

I think our integration tests already serve metrics in json format from the web server - we would just need to read them and parse the json. We could also read metrics directly out of the metrics system.
How would we test that the "bytes written to UFS" metric with a unit test of this style? If we mock the worker, we can't check that the accounting is done correctly within the worker.
Similar for these gauges, we check that we call the right methods from the gauge, but we don't test that those methods are implemented correctly, or that the underlying data they use is computed correctly.
Unit tests are a nice to have, but don't give much confidence that the metrics system will actually work correctly in production. We need something end-to-end to get that confidence

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19276/
Test PASSed.

Merged build finished. Test PASSed.

The common case is for metric calculation to be very trivial (no calculation needed, just update a counter at the right times). What's important is that the counter actually does get updated at the right times. We need a testing methodology which will detect if our metrics counters are not being updated correctly

hmm, but this is about metrics library internals? The counters are updated each time the sink pulls the source, but such invocation is outside our system?

Also, the integration test is outside the scope of this PR

Agreed, no reason to block this PR.


Copy custom attributes from origin to injected type

Delegate types require custom attributes to function properly on some occasions.

:white_check_mark: Build ConfuserEx 889 completed (commit https://github.com/mkaring/ConfuserEx/commit/9d96bb52cb by @oSumAtrIX)

Closing the PR. Seems abandoned.


proselint: disable tests and put in correct scope

Didn't build before due to tests referencing the home directory. Also, it was in
the wrong scope.

[X] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] Linux

[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

cc @alibabzo


[iOS] App rejected while App Review

Hello, my app is getting rejected on App Review on Apple Store.
It looks like getEtitlements() is never resolved while review. It works fine on TestFlight and installed directly.
I have read that this library is production ready...So could someone help me?

Hello, my app is getting rejected on App Review on Apple Store.
It looks like getEtitlements() is never resolved while review. It works fine on TestFlight and installed directly.
I have read that this library is production ready...So could someone help me?

Can you show me the full message you're getting

I'm not getting any message as it is happening while review and I'm unable to reproduce this. I saw the screenshot from reviewer and it looks like getEtitlements() is never resolved while review
This is part of Apple Review:
Issue Description
The app exhibited one or more bugs that would negatively impact App Store users.
Bug description: The IAP would not load.

We've released this in the app store without issues. So it might not be the library


Add --rm to docker cmds for auto-cleanup

Resolves the requested need in https://github.com/bats3c/shad0w/issues/61 in a better way than introducing a new command.

Nice work make, thanks

Nice work make, thanks


Sqlvm Add Azure CLI for Sqlvm Azure AD authentication feature

Related command
az sql vm enable-azure-ad-auth
az sql vm validate-azure-ad-auth
Description
This PR is to add Azure CLI commands to enable and validate Azure AD authentication for SqlVM.
The Azure AD authentication is a new feature for SqlVM as described in the public doc .
Here is the design specification of the Azure CLI work for the Azure AD authentication feature.
Testing Guide
The automatic test "test_sqlvm_aad_auth_negative" covers all different scenario of validation errors. The positive tests are tested manually because of the requirement of high Azure AD privileges from the account running the test. All the examples in the help page have been tested.
History Notes
[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Sqlvm

/azp run


No Option To Map To Stored Procedures

There is no option to map Idenetiy to stored procedures. I cannot run my code without it and don't know how to do a workaround.
In MVC 5 I simply added:
modelBuilder.Entity().MapToStoredProcedures();
overriding onmodelcreating in my Identity model but I can't seem to do that now.
Anyone help?

@rowanmiller what is the bug number tracking this in EF? @divega mentioned it is in the backlog

Found it.
The Identity stores we offer out of the box are for EF Core, and EF Core will not support stored procedure mapping in the initial release (we use the following issue to track the feature in our backlog: https://github.com/aspnet/EntityFramework/issues/245).
In the meanwhile it should be possible to produce your own Identity store implementations (i.e. UserStore and RoleStore) that use stored procedures for persistence based on some other means.


Error when dynamically adding new marker on open cluster

I am using this package for clustering the markers on a map. The markers are updated dynamically and they can be added or removed at any time. When I see a cluster with X markers, and a marker is added in this cluster, then everything works fine and the cluster label is changed from X to X+1. The problem is when I have already zoomed in the cluster and then the new marker (which belongs in the same cluster) is added, the app crushes and I get the error below:

TypeError: undefined is not an object (evaluating 'clusterChildren[_i + start].properties')

My code looks like this:
import { Marker } from 'react-native-maps';
import MapView from "react-native-map-clustering";

<MapView
    ref={mapRef}
    ...
>
    {this.state.markers.map((marker, index) => {
        return (<Marker ... />)
    })}
</MapView>

Updating the markers looks like this:
this.setState({ markers: updated_markers})

Could this be fixed in a newer version?
If you need me to clarify anything feel free to ask.

Hey @Thanasis1101 any update on this, please?

@SirPhemmiey I haven't tested it with any newer version, probably it is now fixed. I see that a check has been added at this line. As the error suggested, there was no clusterChildren[i + start], so it couldn't get its properties. But with the new check, this is no longer the case, after this  commit.
When did you test it last time? If it was before May 2020 then can you update the package, test it again and report the results? Maybe I will check it too, and if we see it is now fixed I can mark this issue as solved.


Bloxstrap won't launch.

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[ ] My answer given in the checkbox above is a lie.

What problem did you encounter?
2024-08-09T09:39:30Z [App::OnStartup] Starting Bloxstrap v2.7.0
2024-08-09T09:39:30Z [App::OnStartup] Compiled Tuesday, 9 July 2024 at 8:03:19 PM from commit a7d2a2c3d5af7137804416ba4e5fbdd27bd10f1e (tag/v2.7.0)
2024-08-09T09:39:30Z [App::OnStartup] Loaded from %UserProfile%\AppData\Local\Bloxstrap\Bloxstrap.exe
2024-08-09T09:39:30Z [LaunchSettings::Parse] Parsing launch arguments
2024-08-09T09:39:30Z [JsonManager::Load] Loading from %UserProfile%\AppData\Local\Bloxstrap\Settings.json...
2024-08-09T09:39:30Z [JsonManager::Load] Loaded successfully!
2024-08-09T09:39:30Z [JsonManager::Load] Loading from %UserProfile%\AppData\Local\Bloxstrap\State.json...
2024-08-09T09:39:30Z [JsonManager::Load] Loaded successfully!
2024-08-09T09:39:30Z [JsonManager<Dictionary2>::Load] Loading from %UserProfile%\AppData\Local\Bloxstrap\Modifications\ClientSettings\ClientAppSettings.json... 2024-08-09T09:39:30Z [JsonManager<Dictionary2>::Load] Loaded successfully!
2024-08-09T09:39:30Z [FastFlagManager::SetValue] Changing of 'FFlagHandleAltEnterFullscreenManually' from 'False' to 'False' is pending
2024-08-09T09:39:30Z [JsonManager::Save] Save request ignored
2024-08-09T09:39:30Z [RobloxDeployment::TestConnection.https://setup.rbxcdn.com] Connecting...
2024-08-09T09:39:30Z [HttpClientLoggingHandler::ProcessRequest] GET https://setup.rbxcdn.com/version
2024-08-09T09:39:30Z [RobloxDeployment::InitializeConnectivity] Testing connectivity...
2024-08-09T09:39:32Z [RobloxDeployment::TestConnection.https://roblox-setup.cachefly.net] Connecting...
2024-08-09T09:39:32Z [HttpClientLoggingHandler::ProcessRequest] GET https://roblox-setup.cachefly.net/version
2024-08-09T09:39:32Z [RobloxDeployment::TestConnection.https://setup-ak.rbxcdn.com] Connecting...
2024-08-09T09:39:32Z [HttpClientLoggingHandler::ProcessRequest] GET https://setup-ak.rbxcdn.com/version
2024-08-09T09:39:34Z [RobloxDeployment::TestConnection.https://s3.amazonaws.com/setup.roblox.com] Connecting...
2024-08-09T09:39:34Z [HttpClientLoggingHandler::ProcessRequest] GET https://s3.amazonaws.com/setup.roblox.com/version
2024-08-09T09:40:00Z [RobloxDeployment::TestConnection.https://setup.rbxcdn.com] System.Threading.Tasks.TaskCanceledException: The request was canceled due to the configured HttpClient.Timeout of 30 seconds elapsing.
---> System.TimeoutException: A task was canceled.
---> System.Threading.Tasks.TaskCanceledException: A task was canceled.
at System.Net.Http.HttpClient.g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts, CancellationTokenSource pendingRequestsCts, CancellationToken originalCancellationToken)
--- End of inner exception stack trace ---
--- End of inner exception stack trace ---
at System.Net.Http.HttpClient.HandleFailure(Exception e, Boolean telemetryStarted, HttpResponseMessage response, CancellationTokenSource cts, CancellationToken cancellationToken, CancellationTokenSource pendingRequestsCts)
at System.Net.Http.HttpClient.g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts, CancellationTokenSource pendingRequestsCts, CancellationToken originalCancellationToken)
at Bloxstrap.RobloxDeployment.TestConnection(String url, Int32 priority)
2024-08-09T09:40:00Z [App::GlobalExceptionHandler] An exception occurred
2024-08-09T09:40:00Z [App::FinalizeExceptionHandling] System.Threading.Tasks.TaskCanceledException: The request was canceled due to the configured HttpClient.Timeout of 30 seconds elapsing.
---> System.TimeoutException: A task was canceled.
---> System.Threading.Tasks.TaskCanceledException: A task was canceled.
at System.Net.Http.HttpClient.g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts, CancellationTokenSource pendingRequestsCts, CancellationToken originalCancellationToken)
--- End of inner exception stack trace ---
--- End of inner exception stack trace ---
at System.Net.Http.HttpClient.HandleFailure(Exception e, Boolean telemetryStarted, HttpResponseMessage response, CancellationTokenSource cts, CancellationToken cancellationToken, CancellationTokenSource pendingRequestsCts)
at System.Net.Http.HttpClient.g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts, CancellationTokenSource pendingRequestsCts, CancellationToken originalCancellationToken)
at Bloxstrap.RobloxDeployment.TestConnection(String url, Int32 priority)
at Bloxstrap.RobloxDeployment.InitializeConnectivity()
at Bloxstrap.App.OnStartup(StartupEventArgs e)
at System.Threading.Tasks.Task.<>c.b__128_0(Object state)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
2024-08-09T09:40:02Z [RobloxDeployment::TestConnection.https://roblox-setup.cachefly.net] System.Threading.Tasks.TaskCanceledException: The request was canceled due to the configured HttpClient.Timeout of 30 seconds elapsing.
---> System.TimeoutException: A task was canceled.
---> System.Threading.Tasks.TaskCanceledException: A task was canceled.
at System.Net.Http.HttpClient.g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts, CancellationTokenSource pendingRequestsCts, CancellationToken originalCancellationToken)
--- End of inner exception stack trace ---
--- End of inner exception stack trace ---
at System.Net.Http.HttpClient.HandleFailure(Exception e, Boolean telemetryStarted, HttpResponseMessage response, CancellationTokenSource cts, CancellationToken cancellationToken, CancellationTokenSource pendingRequestsCts)
at System.Net.Http.HttpClient.g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts, CancellationTokenSource pendingRequestsCts, CancellationToken originalCancellationToken)
at Bloxstrap.RobloxDeployment.TestConnection(String url, Int32 priority)
2024-08-09T09:40:02Z [RobloxDeployment::TestConnection.https://setup-ak.rbxcdn.com] System.Threading.Tasks.TaskCanceledException: The request was canceled due to the configured HttpClient.Timeout of 30 seconds elapsing.
---> System.TimeoutException: A task was canceled.
---> System.Threading.Tasks.TaskCanceledException: A task was canceled.
at System.Net.Http.HttpClient.g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts, CancellationTokenSource pendingRequestsCts, CancellationToken originalCancellationToken)
--- End of inner exception stack trace ---
--- End of inner exception stack trace ---
at System.Net.Http.HttpClient.HandleFailure(Exception e, Boolean telemetryStarted, HttpResponseMessage response, CancellationTokenSource cts, CancellationToken cancellationToken, CancellationTokenSource pendingRequestsCts)
at System.Net.Http.HttpClient.g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts, CancellationTokenSource pendingRequestsCts, CancellationToken originalCancellationToken)
at Bloxstrap.RobloxDeployment.TestConnection(String url, Int32 priority)
2024-08-09T09:40:04Z [RobloxDeployment::TestConnection.https://s3.amazonaws.com/setup.roblox.com] System.Threading.Tasks.TaskCanceledException: The request was canceled due to the configured HttpClient.Timeout of 30 seconds elapsing.
---> System.TimeoutException: A task was canceled.
---> System.Threading.Tasks.TaskCanceledException: A task was canceled.
at System.Net.Http.HttpClient.g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts, CancellationTokenSource pendingRequestsCts, CancellationToken originalCancellationToken)
--- End of inner exception stack trace ---
--- End of inner exception stack trace ---
at System.Net.Http.HttpClient.HandleFailure(Exception e, Boolean telemetryStarted, HttpResponseMessage response, CancellationTokenSource cts, CancellationToken cancellationToken, CancellationTokenSource pendingRequestsCts)
at System.Net.Http.HttpClient.g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts, CancellationTokenSource pendingRequestsCts, CancellationToken originalCancellationToken)
at Bloxstrap.RobloxDeployment.TestConnection(String url, Int32 priority)

Read the Wiki


Add support for HuggingChat custom integration

Can you add support for HuggingChat conversation?
https://github.com/PhoenixR49/hass-huggingchat-conversation

I am not sure I understand.
The repo you linked is an integration that uses HuggingChat. There doesn't seem to be an official HuggingChat API.
More importantly however, huggingchat itself is essentially a user interface to interact with models. It is not meant to serve as a free provider of GPU power...
I am not sure what you mean by "support for HuggingChat conversation" because the integration you linked to is not a provider.


[Snyk] Security upgrade webpack-cli from 2.1.4 to 3.0.0

Snyk has created this PR to fix one or more vulnerable packages in the `yarn` dependencies of this project.

Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
yarn.lock

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

479/1000  Why? Has a fix available, CVSS 5.3
Regular Expression Denial of Service (ReDoS) SNYK-JS-GLOBPARENT-1016905
Yes
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Pull Request Test Coverage Report for Build 183

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 90.367%

Totals

Change from base Build 61:
0.0%

Covered Lines:
135

Relevant Lines:
149

💛  - Coveralls


Added Image option for articles

Article writers can now add image to their mdx articles and the SEO component will use the image as the OG:Image.
The [slug].js for articles will now display an image if provided.
Example Article with Image

This is good but I am not liking how the image is displayed. For one, we should not display the logo image AND the feature image. This looks odd. Maybe if there is a feature image use that, if not use the logo? Or, is there a way to have them together and look nice? I am not sure. Do you have any ideas? Lastly, can we use NextImage to display the image? It is better for seo, load time, layout shift etc.

Yeah I agree its weird. I'll see if there's a way to convert the logo image to an OG:Image that includes the title similar to how Github generates for issues and pull requests.

Could you elaborate on what GitHub does? I'm not familiar with it.

That is actually really cool! The blog post mentions Vercel ogimage - https://github.com/vercel/og-image Maybe this could work?

@Umayarz18 I just merged your other pull request (the guide to oss article). As for this PR, maybe we want to merge now and then create a new PR for the Open Graph image using https://github.com/vercel/og-image? I think we can actually go ahead and delete the feature images for now.


Ability to disable new message notifications

It would be nice to be able to disable the new message notifications for Views. I enjoy having a [zeekdageek] tab so I can easily click through to it and be able to have /twitch #zeekdageek prepend every message but having it so it glows red all of the time at the bottom is slightly annoying.

Wait, the "Show New Messages" option should already do this (at least for when the chat is closed).

Yeah the reason the bug report exists is because it doesn't work though. <;


editor.cursorSurroundingLines interacts unexpectedly with mouse clicks

If I click within the 'surrounding lines' region the editor scrolls begins scrolling unexpectedly. It thinks I'm trying to select text and everytime it moves the cursor within the desired range it selects some more text, moves, the cursor to the end of the selection, and repeats the process.
I can't click in the 'surrounding lines' region without highlighting several lines of text and scrolling further than I intended.
Expected:

When clicking within the 'surrounding lines' region it should not be treated as a selection event unless I drag my mouse while the left button is down.

When selecting text with the mouse the editor.cursorSurroundingLines should not be active. Let selection go to the edge of the screen. Then when the mouse button is released do the extra scrolling based on this setting. Selecting text with the keyboard does not need to be changed.

That's definitely a very annoying bug. There is another issue on this matter: https://github.com/microsoft/vscode/issues/79935

/duplicate of #79934 most likely, assuming @akbyrd isn't using the Insider build where that is already fixed.

@gjsjohnmurray Not an insiders build. 1.38 release.

@akbyrd in which case there's a good chance your issue is already fixed. I suggest you retest when 1.39 is out (see #80591).


OSD-17456 Add alert handling for hypershift SilenceAlerts

What?
Implement OSD-17456.
CAD can now handle SilenceAlerts (hypershift inhibition alerts). It will resolve all old alerts when a SilenceAlert is triggered.
The next steps would be to add orchestration rules to all hypershift pd services that route only the SilenceAlerts to CAD.
Why?
We use alert inhibition as silencing method for hypershift. The issue with this is that inhibiting alerts does not result in active alerts being resolved. This is very frustrating e.g. when starting to work on an alert but the cluster ends up being mid uninstallation.

/hold

/unhold

/hold

/unhold
Checklist:

some feedback on this - done
look into how many new pipelines would get routed through CAD - done
discuss this with the team - done
test on live alerts - done

/lgtm


Is round-robin (weighted or not) possible for docker provider to route traffic into several different services?

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
I think what I'm asking pertains to apps that have multiple versions of the same services OR has their own "clustering" (such as a splunk search head cluster).
In my docker-compose file,  I have 3  services which are 3 splunk search head clustering nodes, and no they can't be "scaled" like some other applications, so they have to be defined as separate services.
I need to know whether it is possible to use the same traefik services for different docker-compose services.
I know this is possible for file or ingress-route for k8s.  My question is specific to docker.
What did you see instead?
It doesn't work.
What version of Traefik are you using?
2.3
What is your environment & configuration?
# (paste your configuration here)

Add more configuration information here.
If applicable, please paste the log output in DEBUG level
sh1:
    networks:
      splunknet:
        aliases:
          - sh1
    image: ${SPLUNK_IMAGE:-splunk/splunk:latest}
    command: start
    hostname: sh1
    container_name: sh1
    depends_on:
      - shd
    environment:
      - SPLUNK_START_ARGS=--accept-license
      - SPLUNK_INDEXER_URL=idx1,idx2,idx3 #,idx4,idx5,idx6,idx7,idx8,idx9,idx10,idx11,idx12,idx13,idx14,idx15
      - SPLUNK_SEARCH_HEAD_URL=sh2,sh3
      - SPLUNK_SEARCH_HEAD_CAPTAIN_URL=sh1
      - SPLUNK_PREFERRED_CAPTAINCY=true
      - SPLUNK_CLUSTER_MASTER_URL=cm
      - SPLUNK_ROLE=splunk_search_head_captain
      - SPLUNK_DEPLOYER_URL=shd
      - SPLUNK_DEPLOYMENT_SERVER=ds
      - SPLUNK_LICENSE_URI
      - DEBUG=true
    ports:
      - 8000
      - 8089
    volumes:
      - ./defaults/sh:/tmp/defaults
    labels:
       - traefik.http.routers.www-router.entrypoints=web
       - traefik.http.routers.www-router.service=www-service
       - traefik.http.routers.www-router.rule=Host(`shc.bayes-dev.lan`)
       - traefik.http.services.www-service.loadbalancer.server.port=8000

  sh2:
    networks:
      splunknet:
        aliases:
          - sh2
    image: ${SPLUNK_IMAGE:-splunk/splunk:latest}
    command: start
    hostname: sh2
    container_name: sh2
    depends_on:
      - shd
      - sh1
      - cm
      - ds
    environment:
      - SPLUNK_START_ARGS=--accept-license
      - SPLUNK_INDEXER_URL=idx1,idx2,idx3 #,idx4,idx5,idx6,idx7,idx8,idx9,idx10,idx11,idx12,idx13,idx14,idx15
      - SPLUNK_SEARCH_HEAD_URL=sh2,sh3
      - SPLUNK_SEARCH_HEAD_CAPTAIN_URL=sh1
      - SPLUNK_CLUSTER_MASTER_URL=cm
      - SPLUNK_ROLE=splunk_search_head
      - SPLUNK_DEPLOYER_URL=shd
      - SPLUNK_DEPLOYMENT_SERVER=ds
      - SPLUNK_LICENSE_URI
      - DEBUG=true
    ports:
      - 8000
      - 8089
    volumes:
      - ./defaults/sh:/tmp/defaults
    labels:
       - traefik.http.routers.www-router.entrypoints=web
       - traefik.http.routers.www-router.service=www-service
       - traefik.http.routers.www-router.rule=Host(`shc.bayes-dev.lan`)
       - traefik.http.services.www-service.loadbalancer.server.port=8000

Hi! I'm Træfiker :robot: the bot in charge of communication regulation.
Thanks for your interest in Traefik!
We dedicate the issue tracker to bug reports and feature requests only. My advanced AI has spotted that your issue might be a configuration problem or relates to something that doesn't look like a bug.
To confirm this, please join our Community Forum and reach out to us on the Traefik section.
In case I'm wrong (well, that would be embarrassing :sweat_smile:), my developers will re-open the issue and fix me!
In the meantime, you can double check Traefik's documentation.

please reopen this!


Export relations to biblatex Crossref?

It would be great if we could use the "related" info to export to the crossref field in biblatex. I realize that biblatex relations are 1:1, while zotero uses 1:M, but it could be a nice feature. For now, I'm stuck hand-coding crossrefs in the extras field.

I've tried this before, but the data models don't match. Zotero
cross-reference have no direction, it just connects two references without
saying how they're connected. Bib(la)tex xref/crossref are parent-child
relationships, and zotero has no information on which is the child and
which is the parent.
If you have an idea on how to get this to work conceptually, if be happy to
take another stab at it.

It's a partial solution, but that should work -- I'll give that a stab. Can you try and find out what other hierarchical relations there could be between Zotero types? What would you expect to happen if one @incollection is related to multiple @books?

I suppose at that point, one would have to default to some kind of arbitrary criteria; first related, alphabetic, etc., which isn't ideal. I'll look into possible hierarchical relationships available in Zotero types.

So, I looked into this more closely; I'm not sure how you feel about biblatex specific features, but biblatex has—in addition to crossref—xref, which establishes "relationships" without applying inheritance. I'm not sure if it allows multiple relationships (like xdata), but it might be good to look into.

BibLaTeX-specific is not a problem. The bundled BibTeX and BibLaTeX share code but are distinct exporters, and there's already a lot that's different between the two.

But the problem remains: xref indeed says "no inheritance", but the biblatex manual states that

xref differs from crossref in that the child entry will not inherit any data from the parent entry specified in the xref field

(emphasis mine). The only difference is "no inheritance", but it's still a strict parent-child relationship, and Zotero doesn't do those. In those cases that

There is a list of obvious parent-child relationships (which I don't currently have), and
A given reference of a "child-type" is only related to a "parent-type", and not to another "child-type" or an other "non-hierarchical" type reference

I can put the xref in place safely. Otherwise, I'd rather not. If you look at the BibLaTeX manual, you'll see that xref and crossref can pull references into your bibliography even when you haven't cited them, I have to be really sure before I put something in place that could cause that.

Any more ideas in the interim?

So I'm new to the discussion, and have to admit that I'm a little unclear about the difficulty here: almost all of the cases that I can think of have two items that have a relatively clear, implicit, parent-child relationship: "Book section" to "Book" type entries, for example. (It seems to me that you might also use specified page numbers to check the relationship: a parent probably shouldn't have any page numbers specified, right?)
I can think a few edge cases where there might be ambiguities, where a single child might have multiple parents: an essay that was originally printed in one book but has then been anthologized. In these edge cases, wouldn't it be simplest to stipulate in the documentation that such edge cases will not be provided with cross- or xrefs?
One final question: would there be any way to specify precisely which Bibtex or Biblatex type a parent and child are exported as? My biblatex-chicago files with crossrefs mostly have @Incollection items, but there are a few @Inproceedings and @Inbook items in there, too (though my impression is that many of these types are simply aliased to one another).

For those cases where there's a simple and unambiguous relationship it's fairly easy (if someone could give me a list at least of the known parent child relationships). There's a few niggles still:

Where they are ambiguous, I'd have to choose a behavior. Skipping is an option, but since I don't use it, I wanted like to hear some opinions
If the parent doesn't happen to be in the export, I can't pull it in. If you're counting on crossref inheritance, it won't work in that case.

If by the latter part of your question you mean whether the reference output can vary by the kind of relationship(s) it has... in principle, yes, in practice, no. The caching system counts on the output per reference being static as long as the reference itself doesn't change. It'd mean either foregoing the cache (and you don't want that; the cold-cache scenario can only have gotten worse as the BibTeX generation got more intricate), or adding cache variants, which is a level of complexity I'd rather not add.

[ ] currently I invalidate the cache for a reference when it changes. With crossref, I have to invalidate the cache for any related items.

OK, if someone can get me a list of relations, I have some basic wiring in place.

Alright, here's a list of relations I've pulled from the biblatex-chicago documentation:
@book is a parent to:

@inbook
@bookinbook (which I've never used, but is in the spec)

@collection is a parent to:

@incollection

@proceedings is a parent to:

@inproceedings

@reference is aliased to @collection in standard biblatex, but has some special features in biblatex-chicago, and is a parent to:

@inreference

Either is OK, although Zotero relations would be easier.
Can you right-click a couple of references that are so related as above and select "report better bibtex error"? If you post the red ID here I'll have test cases.

(I currently have tests running on the relations you mentioned. If those are OK, no need for Zotery-type relations)

For an inreference item: 2KEVB9KV
For an incollection item: QU4QWSWK
For an inproceedings item: QSJ3W6DQ
At the moment, I don't have any inbook items with crossrefs.

Do those include the parent refs? You can multi-select references and send them all at once BTW.

Oh, no-- here they are:
The inproceedings items (w/parent): 723QD3IE
The incollections items (ditto): 3PZGK4FR
And the inreference items: CFD6UM5S

Why do you want to have graham_basmala_2001 crossref to McAuliffe2001? Both are journal articles. And you have a forced crossref from Burnett2013 (book section) to Lindberg2013 (Journal article)
You also had Bourbou2013 (conference paper -> inproceedings) crossref'd to Voutsaki2013 (Book, and Zotero doesn't know a proceedings-equivalent, nor is there a booktype field)
So one of the problems I'm hitting is how to generate things like proceedings. I could of course look at the relations and decide that for a book, if it has only conference paper relations, it was an inconference, and for an conference paper, if it has a single relation to a book, it'd just assume that that was OK to crossref to. But this is so far all rather messy business.

As a matter of fact, in Zotero, book is the only "collection-ish" reference type available.

Which brings us back to the difficulty here: Zotero's model for relations between references and reference types diverges in important ways from how BibLaTeX thinks about them, and the mismatch shows in cases like this. It takes heuristics and guesswork to even generate an proceedings entry because Zotero really doesn't have enough information to do it reliably. The Zotero book type doesn't allow for explicit differentiation.

You can override the reference type being exported (e.g. by including something like bibtex[referencetype=collection] but that reference type isn't available to the child references, which can only see the Zotero reference type (and inferring a zotero type).
I do have a version that infers collection from the fact that a reference only has relations to incollection items, but that also means if you remove that relation, or you add a non-incollection item, it will fall back to being a book. I really don't like that sort of variability, but I suppose it's better than nothing.
I have something close to working, so I'll have a test version soon. In the interim, could you try the test version I put up for #549? Those fixes are useful to have in place before we do much more work on this, and I like to get confirmation before I merge into the main line.

Test version at https://github.com/retorquere/zotero-better-bibtex/releases/download/builds/zotero-better-bibtex-1.6.72-circle-2801.xpi .
child references
It will add a crossref field to any reference that maps to Bib(La)TeX type inbook, bookinbook, incollection, inproceedings or inreference if it has a single relationship that links it to a Zotero book.
parent references
Currently, it maps a parent reference from book to proceedings if it has one or more relations that are exclusively of Zotero type conferencePaper.
It will map book to reference  if it has one or more relations that are exclusively of Zotero type encyclopediaArticle or dictionaryEntry, but only for BibLaTeX.
It leaves all other book references as-is.
I don't know what to do for the other cases. Zotero bookSection references currently export to incollection rather than inbook, so I could use the same rule for proceedings to map a book to collection, but there'd be no way to generate an book entry with child crossrefs without manually specifying the reference type on the book.

This looks great! Thanks so much for all of your work on this; I've been messing around with the updated plugin and crossref export function, and it seems to be working with barely a hitch.
The only (minor) hiccups I've noticed are two: the first is that the Biblatex export does not seem to, in fact map book to reference where a book has an encyclopediaArticle relation. (I've submitted an error report under 9XKSXCUT.) The second is that manually specifying the item type seems to work as you describe above; including bibtex[referencetype=book] in the Extra field will change the item type of that item from collection to book. However, it does not change the item types of the items that are related to it; instead of being inbook items, they are still incollection items (I'm assuming this is unintentional). For this, I've submitted a but report under UDI5N7CM.)

There's no debug info in 9XKSXCUT -- you must either turn on extended debugging and do it again, or right-click and submit the relevant references. Or both, which gives me the most info.
The second part is more problematic. The change you make by adding bibtex[referencetype=book] is not available when the child entries are generated. The caching system (and trust me, you want the caching system, BBT is horrendously slow otherwise) just skips over entries it "knows" are up to date, so there's no structured info available on those. It'd take some major work to work around that.

For the debugging info, try VFKN98A5 instead.

As far as the "adjusting the related item type" problem, perhaps it's best to resign oneself to using collection and incollection items (or to setting lots and lots of bibtex[referencetype=] entries by hand, I suppose). Probably not worth expending a ton of energy on before the Zotero 5 switchover.

Apologies for abandoning this thread—I'm in the middle of studying for my doctoral exams. Looks like you have some great stuff happening and it all seems quite reasonable from my perspective.

Interested in picking this back up?

Closing for inactivity -- feel free to suggest to reopen.


[dl-chrome-extension] [GoPlay] add download buttons to list entry + full season

see https://github.com/BelgianNoise/dl-utils/commit/63b2055593e2fdb5a5bfc8d2f887595275ba8cb5


Containerized workflows on HPC

Workflow steps are currently Python environments. To integrate with the infrastructure, they must be converted into containers, orchestrated by, e.g., Apache Airflow.
Goal: execute DT workflows on a Kubernetes cluster where each step is deployed as an independent container. Orchestrations can be achieved by means of some "advanced" orchestrator (e.g., Airflow), or by executing a DT workflow step-by-step in a very similar way as it is currently done by run-workflow.py now. The only difference is that the command is executed in a container, rather that in a Python virtual environment.

[x] Containers for itwinai distributed ML
[x] tutorial on how to use containers

Solved by https://github.com/interTwin-eu/itwinai/pull/197


HLS in Android

HLS works fine in iOS - normal stream and even adaptive.
HSL does not work in Android.
Error:
com.google.android.exoplayer2.source.UnrecognizedInputFormatException: None of the available extractors (MatroskaExtractor, FragmentedMp4Extractor, Mp4Extractor, Mp3Extractor, AdtsExtractor, Ac3Extractor, TsExtractor, FlvExtractor, OggExtractor, PsExtractor, WavExtractor) could read the stream.
Streams:
Adaptive: http://player.vimeo.com/external/231992762.m3u8?s=ad80951022693de245a532b4c01bde2808e01636
HLS 640x386: https://skyfire.vimeocdn.com/1508921023-0x8bda84f05b98b5781572ad893f98684382851cd1/231992762/video/821174298/playlist.m3u8
According to ExoPlayer Documentation (https://google.github.io/ExoPlayer/guide.html#preparing-the-player): "The ExoPlayer library provides MediaSource implementations for DASH (DashMediaSource), SmoothStreaming (SsMediaSource), HLS (HlsMediaSource) and regular media files (ExtractorMediaSource)"
Seems to me that Viro uses ExtractorMediaSource, so we can't use HLS right now.
Could you please give us a hint, how could we get HLS stream to play now in Android?

Hi @newbot23, thanks for alerting us to this issue! Currently there isn't a way to do this on android. That being said we've added this issue in our backlog and will update this thread when we fix it which should be in our next release.

Hi @VikAdvani, thanks for your quick reply! Would you please tell us whether we should expect the fix this month, next month or probably this year? It's very important feature for us, so we can't launch our product (more information via pm) without that. So we need to make a decision whether we can wait till your next release.

Hi @newbot23, we looked into this and we'll be able to get the fix into our next release which will be in mid november.  Hopefully that works for you :)

Hi @VikAdvani, thanks for your feedback and time, mid November is OK for us, so we will wait till that:)

Hi @VikAdvani, thanks for the update! HLS does work now, so we are more than happy:)


raidboss: Eliminiate a race condition during raidboss initialization.

If we attach the overlay listeners before gPopupText has been set, we risk triggering a "Cannot read property '...' of undefined" error. If we attach them in the constructor instead, we can be sure that the object exists.
Since only one instance of PopupText is ever created, we don't have to worry about detaching the overlay listeners.
This should fix the issue reported in https://github.com/quisquous/cactbot/issues/660#issuecomment-542819459.
I've also eliminated a duplicated call to cactbotReadDataFiles which was part of the modified code.

Thanks!


Question: Is it safe to delete these directories?

System: Debian 9
Updating to DBeaver 5.3
Preparing to unpack .../dbeaver-ce_5.3.0~ubuntu16.04_amd64.deb ...
Unpacking dbeaver-ce (5.3.0~ubuntu16.04) over (5.2.5~ubuntu16.04) ...
dpkg: warning: unable to delete old directory '/usr/share/dbeaver/plugins/org.eclipse.equinox.launcher.gtk.linux.x86_64_1.1.800.v20180827-1352': Directory not empty
dpkg: warning: unable to delete old directory '/usr/share/dbeaver/plugins/org.eclipse.equinox.security.linux.x86_64_1.1.100.v20180827-1235': Directory not empty

Is it safe to delete the mentioned directories?
tia,
d

Yes, sure. Directory /usr/share/dbeaver/ doesn't contain any user data and can be deleted.
All your database configurations and scripts are in user home in .dbeaver4 folder.

OK, so all the jar files in /usr/share/dbeaver/plugins/ are not needed?

They are needed to run DBeaver. But if you are upgrading to a newer version then you can easily remove everything from /user/share/dbeaver

Thanks, I just removed the folders which the installer complained it was unable to.
Everything seems fine.
Many thanks again for a great application!
d


feat: single annotation download dialog step 1

#435

fixes some copy bugs with file format dropdown
file format dropdown now auto-initializes to the first value in its list
query params can be changed while overwriting the previous history
'via AWS S3' -> 'via AWS CLI'
single annotation download dialog query params, state, hooked up

Demo
https://github.com/chanzuckerberg/cryoet-data-portal/assets/29165011/16c010ed-e7bf-4ac4-b28e-29607642b2e6

live demo: https://dev-kira-1.cryoet.dev.si.czi.technology/

When i look at the file format tooltip for segmentation mask, the styling seems to be broken?

that's odd. I'm not encountering it on the dev site/locally, could you provide steps to reproduce?

Is .ndjson not format option for segmentation mask annotations? I dont see it in the demo so I want to confirm if this was a change/bug/incorrect in the design (cc @manasaV3 ?)

.ndjson will only appear in the list of options if the file exists in the backend

The "next" button should be active because the field always has a value (please ignore if this is just because its not hooked up to step 2 dialog)

yeah it'll be fixed when step 2 is implemented


Emit Module reconciler metrics

Module reconciler should emit Prometheus metrics about reconciliations it is processing. The reconciler should emit metrics about:

number of reconciliations as a counter
reconciliation duration as a histogram
failed reconciliations as a counter

Describe the solution you'd like
The reconciler should accept a Monitor in its constructor, which will need to implement new metrics and methods to accept new metrics.

@quest-bot embark


Enable full 10.3min ionospere simulations

Enable a simulate of full 10.3min observation.
Input:

same as normal

How:

Choose a gridding setup that can manage the simulation within 1TB ram.
If we build row by row this means we need to store in total (Na*Nt*Nd)^2 values in matrix form.
Cholesky only needs the lower triangular, so we can use compute just for that (for sampling that's all we need).
We can fit this in memory with the 16x16 directions, 60 second time resolution (11 time grid points), 128 points in aperture plane.

if __name__ == '__main__':
    import numpy as np
    from scipy.linalg import cholesky

    # Function to create a symmetric positive-definite matrix
    def create_spd_matrix(n):
        A = np.random.rand(n, n)
        A = np.dot(A, A.transpose())  # Make it symmetric
        A += n * np.eye(n)  # Make it positive-definite
        return A

    # Main demonstration
    n = 5  # Dimension of the matrix
    A = create_spd_matrix(n)  # Create SPD matrix
    A_lower = np.tril(A)  # Extract lower triangular part

    # Compute Cholesky decomposition using only the lower triangular part
    L = cholesky(A_lower, lower=True)

    # Verify the decomposition
    A_reconstructed = L @ L.T
    # Since we're working with floating point numbers, use np.allclose for comparison
    is_correct = np.allclose(A_lower, np.tril(A_reconstructed))

    print(f"Original Lower Triangular Part:\n{A_lower}")
    print(f"Reconstructed Lower Triangular Part from Cholesky Decomposition:\n{np.tril(A_reconstructed)}")
    print(f"Decomposition Correct: {is_correct}")

    import jax.numpy as jnp
    from jax import random
    from jax.scipy.linalg import cholesky

    # Function to create a symmetric positive-definite matrix using JAX
    def create_spd_matrix(n, key):
        A = random.normal(key, (n, n))
        A = jnp.dot(A, A.T)  # Make it symmetric
        A += n * jnp.eye(n)  # Make it positive-definite
        return A

    # Main demonstration
    n = 5  # Dimension of the matrix
    key = random.PRNGKey(0)  # Random key for JAX
    A = create_spd_matrix(n, key)  # Create SPD matrix
    A_lower = jnp.tril(A)  # Extract lower triangular part

    # Compute Cholesky decomposition using only the lower triangular part
    L = cholesky(A_lower, lower=True)

    # Verify the decomposition
    A_reconstructed = L @ L.T
    # Since we're working with floating point numbers, use jnp.allclose for comparison
    is_correct = jnp.allclose(A_lower, jnp.tril(A_reconstructed))

    print(f"Original Lower Triangular Part:\n{A_lower}")
    print(f"Reconstructed Lower Triangular Part from Cholesky Decomposition:\n{jnp.tril(A_reconstructed)}")
    print(f"Decomposition Correct: {is_correct}")


fix(curriculum): Fixed the typo of Cat photo app step 14

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #56999

Hi @avneets0419. Congrats on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We are happy to accept these changes and look forward to future contributions. 📝


Add a Help button (with icon and label text) and modal blurb to WebVR Template's on-page UI

Clicking the Help button should set the page anchor to #help (i.e., `window.location.hash = '#help') and make a modal box appear.
(See https://fonts.google.com/ for inspiration.)

Parent issue: #165
Related issues: #164 for documentation/messaging

Note: this needs to have an opt-out mechanism in order to be considered completed.

Postponing the need for this for now.


feat(mempool): add simple back-pressure mechanism on peers

Currently, when attempting to add a tx received from a peer, if the mempool is full, the transaction will just be discarded.
This PR implements a simple backpressure mechanism on the mempool by adding a new channel MempoolControlChannel and a new control message MempoolIsFull:

If node A, with a full mempool, receives a tx from B, it will reply MempoolIsFull.
Node B on receiving MempoolIsFull from A will stop sending transactions to A during a predefined time.

 A node that doesn't have MempoolControlChannel will just ignore messages received on this channel. And a node will not send messages on MempoolControlChannel if it knows that the peer doesn't have the corresponding channel.
cc @sergio-mena @ValarDragon

PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments
[ ] Title follows the Conventional Commits spec

So I think it might be smart to put an integer in the mempool full message and basically the idea would be an application specific priority value and it would indicate that " I don't want to receive messages lower than priority n where n is the integer in the message".
I think that priority value could be integrated into the mempool in a. number of ways.

If we are adding an additional channel for control, my approach would be more proactive than reactive.
Think about TCP. We don't say "stop sending us stuff" because I can't consume data fast enough. We instead send acks (implicitly or not) informing what we already have processed and thus allowing the sender to adapt its sending rate.
In other words, we say to the other side: I can afford to handle X messages or bytes. And if it does not comply with my affirmation, it is a bad peer, and I'll probably drop his messages.
Not to mention here that bad/attackers peers will completely ignore this message. Also, once we have space in our mempool we will have peers not sending us anything because they are waiting for an arbitrary time to be defined. Also, we cannot punish anyone for sending stuff to us during this arbitrary time, because we just don't known how long it takes to this "stop sending me stuff" message to arrive.
Again, all this discussion should occur on an issue.

instead of "MempoolIsFull" do we want to generalize this to some sort of "Stop sending me stuff" message

Closing for now as currently we don't have the capacity to work on this. The brainstorming should continue in an issue or discussion.


docs: Error Reporting not as instructive as it could be

The Error Reporting section has two issues:

It seems that we need to manually enable the Error Reporting API to get it to work. This should be added to the instructions in the docs.
This demonstration would be more powerful if the user were able to manufacture some errors and see them reported on the UI. The docs could have some "how to break it" instructions, e.g. stopping nodes through the GCP console or a particular pattern using the load generator.

We should not be manually enabling Error Reporting API. It should be done by the provisioning script. I'll open a separate issue on this.
For your second point, please open up a separate issue - we should add instructions on how to break things so folks will see errors in Error Reporting. @gzhao408

Closing in favor of #53


⚠ Bofrost.be has degraded performance

In 46d3324, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 865 ms

Resolved: Bofrost.be performance has improved in 86db88c after 7 minutes.


Develop

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@396f19c). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #11   +/-   ##
=========================================
  Coverage          ?   85.59%
=========================================
  Files             ?       30
  Lines             ?      361
  Branches          ?       95
=========================================
  Hits              ?      309
  Misses            ?        3
  Partials          ?       49

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 396f19c...8beb252. Read the comment docs.


Correct yarn install command on Getting Started

What kind of change does this pull request introduce?
Minor documentation update (code syntax and markdown formatting).
What is the current behavior?
Yarn installation instructions are incorrect and formatted incorrectly.
What is the new behavior?
Formatting and command syntax is fixed.
What steps did you take to test this? This is required before we can merge, make sure to test the flow you've updated.
N/A
Checklist

[x] Documentation;
[ ] Storybook (if applicable); N/A
[ ] Tests; N/A
[x] Ready to be merged;

npm instructions are formatted like a full code block:

but yarn instructions are formatted like an inline code block, and yarn i should be yarn add:

ohhh thank you so much!
I'm going to fix it on another PR because I found more cases.


It opens login page on clicking on theme

🐛 Bug Report
login form open on clicking theme

I am interested may you assigned me.

Please assign this task to me @royalleads15 .

/assign


SUGGESTION: Remove language from release name

Currently, a release is tagged like en0.10.0-v0.4.5. However, it should be language agnostic, right?
I'm using the Github API to determine the latest release and add the language like lang + getLatestC11y()

Makes sense, let's start that with change proposed in #28. Closing here.


Tag a relase

Can you tag a release as well? 0.2 for example?
Don't forget to update the branch alias in composer.json to 0.3

Yeah I was just doing that :)

Awesome ;)


SUSE support: file SUSE Vulnerability data under "CVE-yyyy-id" vs "SUSE-SU-yyyy:id-v"

In the trivi.db "vulnerabity" bucket, the CVE-yyyy-id is collecting "VendorSeverity", vendor specific "CVSS" scores and url "References" data from redhat, oracle, ubuntu, ... but not from SUSE.
For SUSE, this data is instead stored only into a specific SUSE-SU-... or òpenSUSE-SU-...` entry.
The corresponding fields are available in the testing data pkg/vulnsrc/suse-cvrf/testdata.  The SUSE specific CVSSScoreSets are empty here, but they are populated in more recent files, like cvrf/suse/opensuse/2015/openSUSE-SU-2015-0225-1.json
Is this a feature or a bug?
Should the SUSE VendorSeverity, CVSS and References from the be added to the CVE from the SUSE-*.json data?  instead of creating a SUSE-SU item?  or in addition?

I guess I'd have to move the VulnerabilityDetail construction all the way to  storing of the severity into the severity scan loop above and use the cvuln.CVE instead of cvrf.Tracking.ID?


ttl for RpcClient

Ensure that TTL from CallOptions are passed in the UAttributes and then the futures are completed exceptionally when the message times out (we did not get a reply)

OK. Now I got it. The timeout in CallOptions and TTL in UAttributes are the same thing.
I thought TTL meant the hop number limitation before. I'm misled by the router concept.


Add pricing page link to footer

Closes #1064

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/d8e5e03b1537321af58bab17677e5d4af5f8bc91/


Add Elements Docs

Coverage remained the same at 100.0% when pulling 1e2cc0046dcc998d8f4b6719cbdb64e15fc289a6 on documentation-elements into 45d564590144ae7505a744f086a4016f5af244a3 on dev.


Exception when height and width are not provided in constructor

Version of flutter_calendar_carousel
v1.2.3
Expected behavior
Showing the calendar or raising an error
Actual behavior
Showing nothing and raising a ConstaintBox exception on Flutter
flutter doctor result (run flutter doctor in terminal)
I/flutter (18178): startWeekDay: 5
I/flutter (18178): endWeekDay: 1
I/flutter (18178): ══╡ EXCEPTION CAUGHT BY RENDERING LIBRARY ╞═════════════════════════════════════════════════════════
I/flutter (18178): The following assertion was thrown during performLayout():
I/flutter (18178): BoxConstraints forces an infinite height.

Steps to reproduce the behavior
  @override
  Widget build(BuildContext context) {
    return Container(
      child: Column(
        children: [
          CalendarCarousel()
        ],
      ),
    );
  }

Flutter itself tends to produce this kind of exception when a height of the root is not specified. In your example, the root Container that CalendarCarousel is referencing has no height which confuses the calculation of widget tree. I think this is expected behavior. Please reopen when you feel this is still a problem.


”Go back to last viewed article” is missing

Switching from NN4 to 5 this is the most severe bug: this feature, to hop along the ”thread” of read articles, is missing.
In NN4 I often browsed through articles/items, and after hopping to the next one my brain clicked ”wait, the last one indeed sounded interesting". In NN4, I pressed ⌘[ and hopped back. In NN5 I can’t find that feature.
This worked across origins, so "cursor up" in the item list view is not a replacement.

Duplicate of #44


ref #16: fix name of image crop preset after field was made multilingual

Q
A

Branch
master

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Tests pass?
yes

Fixed issues
chameleon-system/chameleon-shop-theme-bundle#16

License
MIT

chameleon-system/chameleon-base#263 needs to be merged before this.

Thanks @fabianbarth-esono


How to mute 'Warning: An unexpected error occurred...'?

Warning: An unexpected error occurred. Something may be wrong with WordPress.org or this server’s configuration.

This warning is raised directly by WordPress core (in several places) and isn't anything to do with the HTTP API.
Annoying isn't it?

I personally think that the Airplane way of disabling HTTP traffic needs refining.
Please steal code from my MU plugin: https://github.com/szepeviktor/wordpress-plugin-construction/blob/master/mu-disable-updates/disable-updates.php#L94-L111

Airplane Mode disables much more than just the update checks.

I think the update checks complain: Warning: An unexpected error occurred...

with WordPress.org

Solved by https://github.com/norcross/airplane-mode/pull/42

correct, this is solved.


rootfs-builder: ARM64 support

Install the right version of golang depending of the host architecture
fixes #60
Signed-off-by: Julio Montes<EMAIL_ADDRESS>

@bergwolf @jbryce @jcvenegas  ping

lgtm

@jodh-intel  I prefer to have this map, instead of have go as dependency just to to know the version. What do you think?

@jcvenegas - yes, now I understand the reason for this, wfm too ;)
lgtm


Unify the Approach to Field Setting in Feature Files

Request is to standardize ability to set fields in feature files
e.g.
Given "" creates data for "registerlistener"
And sets "entity.name" to "body.access"
And sets "resource.name" to "door"
And sets "resource.instance" to "front_left"
And sets "resource.message" to "Door"
When sends "registerlistener" request
And user waits "3" second
Then the status received with "code" is "OK"
should use source and sink like
And sets "attributes.source.entity.name" to "body.access"
And sets "attributes.source.resource.name" to "door"
And sets "attributes.source.resource.instance" to "front_left"
And sets "attributes.source.resource.message" to "Door"
When sends "registerlistener" request
And user waits "3" second
Then the status received with "code" is "OK"

@agosh01 The "register listener" method of uTransport takes a UURI input, consisting of authority, entity, and resource message. When setting the proto fields from the feature file, for the "register listener" method, the key would be "entity.name". However, for the "send" method, the key would be either "message.attributes.source.entity.name" or "message.attributes.sink.entity.name".

HI @neelam-kushwah This is more towards standardization of umsg creating in .feature file to improve readability for user. i.e. User will make UMessage format in .feature and for register listener wrapper we use sink/source uri from UMSG.


jenkins update

New jenkins setup that enables to run two jobs -- parallel and non-parallel.

It have worked out.
I'm expecting non-parallel job to run at 11pm and parallel at 5am

Merging this since it's working.


Doc needs to clarify whether Tenant ID, Directory ID, Power Apps Tenant are one and the same?

Doc needs to clarify whether the tenant sub-domain created at the time of signup
<Name>.OnMicrosoftOnline.com
is mapped to Tenant ID, Directory ID, Power Apps Tenant are one and the same?
https://docs.microsoft.com/en-us/onedrive/find-your-office-365-tenant-id

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

ID: 46a11050-b442-7a48-0d37-9bce8908fc14
Version Independent ID: 5456e564-2629-cfc2-dfc9-2b7934fcf43a
Content: Domains FAQ - Microsoft 365 admin
Content Source: microsoft-365/admin/setup/domains-faq.md
Service: o365-administration
GitHub Login: @pebaum
Microsoft Alias: pebaum

@sing-ideas Tenant ID and Directory ID is the same thing when talking about Azure.
I could not find any reference to Power Apps in the articles you mentioned, so i don't know the context, but a Power Apps tenant would be the Azure AD tenant on top of which you are buildng your Power Apps platform.

@sing-ideas - Thank you for submitting feedback.
From our understanding, the issue you raised has been answered by VLG17 so we will close this issue.
Thank you for your contribution to make the docs better! Much appreciated!


[New Feature]: Music ON OFF Button

🎮 feature Request
feature logic and basic description
Adding Music ON OFF button While playing the game so that player can choose to on the music or off the music
Point down the features
feature points
-User can OFF the sound if he dont like
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

/assign

@ayush-t02 Please assign me the issue so that i can start working


java.lang.NoClassDefFoundError: com/pneumaticraft/commandhandler/PermissionsInterface

I'm getting that error when I try to run the latest Multiverse-Core. Server log: http://colonelthirtytwo.net/server.log

Did you compile it yourself?

Yea, the latest git verison I should have said.

What command did you enter?

In the server console? Just "stop."

No, how did you compile it? Did you do it as described in the wiki?

Oh forgot to do the submodules. Whoops


RPC isn't used in healthchecks

RPC is used for plan and apply. It doesn't work for healthchecks, and apparently it needs a little bit of thought about how to do that.

Fixed in #289


Rules of engagement

Define rules of engagement for quick sprints - seed into new READMEs from vizSkeleton

completed with https://github.com/USGS-VIZLAB/vizlab/wiki/Sprint-rules and https://github.com/USGS-VIZLAB/vizlab/wiki/Style-guide


Destination only choose SAPCPMobileServices in "Use Mobile Services to Create UI"

Tutorial URL: https://developers.sap.com/tutorials/cp-apim-sales-tracker-create-ui.html
Please specify the step you are referring to: I created API_Portal_Trial, but I can only choose destination SAPCPMobileServices.

Dear Colleague,
If you have created a destination and saved it with the same name we will get it in mobile service. Are you in trial?
Regards
Shilpa


[New game]: CodePen Simulator

🎮 Game Request
This game simulates the CodePen environment where you have to type in the keyboard to create pens. They will gain stats in HTML, CSS, JS & Design based on your skill level and if they are good enough, they will be featured.
Point down the features

Interactive
Addictive

Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @Sriparno08 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @Sriparno08 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Finding Internal Api Calls

Hello!
I am developing an achievement tracker app. I also want it to fetch the best achievements guides. This is a page I am accessing.
https://steamcommunity.com/app/1942280/guides/?searchText=&browsefilter=toprated&browsesort=creationorder&requiredtags%5B%5D=Achievements&requiredtags%5B%5D=English
However, when going to this website, in network tab I cannot see any API requests that response with JSON. How do you guys reverse-engineer it? I also tried mitmproxy but just can't find it
I know that at the end I can web scrape but since you are finding all of those, I believe that it is possible also there!
Likewise, I really appreciate any help!

I don't think it exists

Yea, I got more understanding of the steam client server flow and found out that the first request returns all the guides in HTML already rendered on the server. It is not the case with some other things where we can see the exact request being made on the client side to fetch additional data

Yea, I got more understanding of the steam client server flow and found out that the first request returns all the guides in HTML already rendered on the server. It is not the case with some other things where we can see the exact request being made on the client side to fetch additional data

you need to use the documented web api.
https://api.steampowered.com/IPublishedFileService/QueryFiles/v1/?key=&format=json&query_type=0&&filetype=10&page=1&numperpage=100&appid=1942280&match_all_tags=0&include_recent_votes_only=0&totalonly=0&return_vote_data=0&return_tags=0&return_kv_tags=0&return_previews=0&return_children=0&return_short_description=0&return_for_sale_data=0&return_metadata=1