[nr-k8s-otel-collector]  Updates on README and chart values

Is this a new chart
What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

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

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: mangulonr:x: dbudziwojskiNRYou have signed the CLA already but the status is still pending? Let us recheck it.


Spelling fix

Post-merge review for #92.
Couldn't think of a place to say "All ingesters are considered not ready if one ingester is considered not ready. This is so Kubernetes will only roll out one ingester at a time, and more importantly, only shut ingesters down one at a time, as our ring consistency depends on graceful shutdown."

This change is

I'd stick it on the Ready in ingester.go
Note it doesn't behave like that right now, as multiple ingesters are added / removed concurrently.  We want it to do that, so I'd class this as a bug.  But its low priority as the consistency scheme copes with this adequately.

LGTM otherwise


git clone on Windows doesn't create symlinks, causes Recompiling templates, PLEASE FIX!!

git clone on Windows doesn't create the symlink at zulip/node_modules/.bin/handlebars and causes the error below.
git clone https://github.com/zulip/zulip.git -c core.autocrlf=false
vagrant up
# Now inside the container
cd /srv/zulip
source /srv/zulip-venv/bin/activate
./tools/run-dev.py --interface=''

Recompiling templates

/srv/zulip/node_modules/.bin/handlebars:1
(function (exports, require, module, __filename, __dirname) { ../handlebars/bi
                                                              ^
SyntaxError: Unexpected token .
    at Module._compile (module.js:439:25)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Function.Module.runMain (module.js:497:10)
    at startup (node.js:119:16)
    at node.js:902:3

PLEASE FIX!!

Fix it by creating a symlink.
cd /srv/zulip/node_modules/.bin/
mv handlebars handlebars.txt
ln -s ../handlebars/bin/handlebars handlebars

That traceback is not familiar, but it could be related to Windows line-endings (#37).

OK looks like you did find the solution in #26 and it worked for you.  So I think this ticket should be documenting the Windows hack needed here:
git clone https://github.com/zulip/zulip.git -c core.autocrlf=false

@ydsdsa @wwarlock can you check out the documentation I just added to #259 and confirm it's correct?   (https://github.com/timabbott/zulip/blob/tabbott-centos/README.dev.md).
I don't have a windows environment to test on ...

I'm really sorry, but for now I haven't Windows too.
The text looks good

@timabbott this ticket isn't related to the line endings ticket. I had this symlink issue after fixing the line endings. You have to add to the Windows documentation that after vagrant up, you need to create a symlink. I didn't need cygwin on Windows. I just installed git cloned and installed Vagrant. I had VirtualBox installed prior.
cd /srv/zulip/node_modules/.bin/
mv handlebars handlebars.txt
ln -s ../handlebars/bin/handlebars handlebars

OK.  I think that is actually going to be solved when we merge #204 since that will eliminate the vendored handlebars code entirely...

OK merged the relevant piece of #204 , so I think that handlebars workaround shouldn't be required anymore.


Add matrix for testing multiple python versions

If this is a library package then we might consider adding a matrix for testing different python versions in addition to testing on different platforms

Fixed in https://github.com/jorgensd/reproducibility/pull/33


Switch to Jupytext

Solves for #727 by switching *.ipynb files to *.md ones with the help of Jupytext.

Although tests are passing locally with tox -e docs and documentation is built up without problem it seems that Read The Docs complains about the build.

/home/docs/checkouts/readthedocs.org/user_builds/poliastro/checkouts/978/docs/source/examples/Analyzing the Parker Solar Probe flybys.md:"nbsphinx-thumbnail": No outputs in cell 57

:thinking:
Can you try
export READTHEDOCS=1
tox -e docs

?

export READTHEDOCS=1
tox -e docs

Previous commands were not successful... :cry:
I did not modify the docs/source/jupyter.rst file which contains a toctree referencing the name of each notebook but with *.ipynb extension. It is strange that Sphinx does not complain about the fact those files do not exist anymore. Let me check also for that :+1:

export READTHEDOCS="True"
tox -e docs

If changing to "True" I am finally able to recreate the errors from RTD. Therefore, should we force nbsphinx_execute = True no matter the environment?

@jorgepiloto is this still a draft? Or is it ready for review?

I still need to deal with Binder notebooks but not sure how to do that :thinking:

True, opening the .md example in Binder now displays a text file. Doing just %pip install jupytext didn't fix it. But perhaps adding it to requirements.txt as done in this demo will do the trick?

(Notice that in this case you would need to add it to requirements.in and then pip-compile to generate requirements.txt - ping me if you need help)

Relevant: https://github.com/mwouts/jupytext/#jupytexts-contents-manager

(Notice that in this case you would need to add it to requirements.in and then pip-compile to generate requirements.txt - ping me if you need help)

I ran echo "jupytext" >> requirements.in && pip-compile and although it was added to the requirements.txt a lot of other deps were upgraded, is this normal? I thought required packages were defined in the pyproject.toml...

Yes, the dependencies for users that do pip install poliastro are defined in pyproject.toml. But Binder does not know how to read that, so we need to repeat some of them in requirements.in. So, yeah, it's a duplication :/

YAYYYYYYY it worked! :heart_eyes:

Awesome work @jorgepiloto! :rocket:


status controller: allow setting RelatedObjects dynamically

Some operators (e.g. cno) don't know the set of related objects at startup. So, allow that to be dynamically generated.

@deads2k can I get a review please? this would help us migrate CNO to library-go tooling.

@mfojtik updated, thanks!

/lgtm
@sttts second look/approve?

@sttts updated, thanks for the review. PTAL.

/lgtm
/approve


Android c++ interface

Thank you very much for the open source of the mediapipe series of algorithms by the Google team. I wonder if BlazePose will provide the Android ndk c++ interface, similar to the calling method of the desktop version of the python interface. The coordinate information can be obtained by calling the interface. For example, so libraries and header files.

What does that all mean

Hi @Source666, We have very clear documentation of MediaPipe solutions with Android platform . Have a look at here and let us know if your use case is different.


When will the Gem be published?

Hi!
Thanks for the great Gem. I am glad because I was in the process of creating a similar Gem.
When will the Gem be published on rubygems.org?
I have tried using the Gem, and it appears to work well.
I can't wait to be able to run gem install rspec-inline-snapshot.

Hey! We're so glad you found it right after we published it to Github 😄 , we just hadn't got a chance to publish to rubygems.org yet. I just pushed it now, so it should be available 🎉  https://rubygems.org/gems/rspec-inline-snapshot
Hope you continue to find it useful!

Thank you for your swift response!


Unable to add description to the last SD.

https://github.com/bithyve/bitcoin-keeper/assets/62699947/02a587f9-ef15-4ed9-9f32-89cb0e20c8ac

Unable to add description to the last SD and when I went to click for the second time,it didn't click.
Happening on android device: One plus 9R

https://github.com/bithyve/bitcoin-keeper/assets/62699947/fb45988f-5568-41d9-99b5-8136f59fcbca

Unable to add description to the last Sd.
Tested on dev app v 1.1.7(260)
Device : Android one plus 9R

Still having the same issue in dev app v 1.1.7(262)
Tested on android device one plus 9R

Uploading Record_2023-11-10-15-05-01.mp4…

Can't reproduce. Check with different devices and point to more details please @cakesoft-swati


Revise step desc in image-volumes

Have a try to revise the step text, to tell why it is required to run those commands.

/lgtm


Target process crashes while running under the debugger on macOS

Issue Description
dotnet quits unexpectedly when running under debug.
Steps to Reproduce

clean and build project
start debugging
open project in browser.

Expected Behavior

Project should run normally. Web page should load and allow navigating to page that communicates with the client-middleware (C#) code.
This works as expected when debugging is not started.

Actual Behavior

A network error is thrown and reported by the browser.
Apple throws a Problem report for dotnet

Logs
OmniSharp log
Starting OmniSharp server at 8/28/2019, 9:56:01 AM
    Target: /Users/ddumaresq/EducationPlannerBC/Dockerize/EPAS/sources/develop-branch/epbc-ui/EPBC.PSAS.sln
OmniSharp server started.
Path: /Users/ddumaresq/.vscode/extensions/ms-vscode.csharp-1.21.1/.omnisharp/1.34.2/run
PID: 60235
    Starting OmniSharp on MacOS 10.14.6 (x64)

    DotNetPath set to dotnet

    Located 1 MSBuild instance(s)
        1: StandAlone 16.0 - "/Users/ddumaresq/.vscode/extensions/ms-vscode.csharp-1.21.1/.omnisharp/1.34.2/omnisharp/.msbuild/Current/Bin"

    MSBUILD_EXE_PATH environment variable set to '/Users/ddumaresq/.vscode/extensions/ms-vscode.csharp-1.21.1/.omnisharp/1.34.2/omnisharp/.msbuild/Current/Bin/MSBuild.dll'

    Registered MSBuild instance: StandAlone 16.0 - "/Users/ddumaresq/.vscode/extensions/ms-vscode.csharp-1.21.1/.omnisharp/1.34.2/omnisharp/.msbuild/Current/Bin"
        MSBuildExtensionsPath = /Library/Frameworks/Mono.framework/Versions/5.4.1/lib/mono/xbuild
        BypassFrameworkInstallChecks = true
        CscToolPath = /Users/ddumaresq/.vscode/extensions/ms-vscode.csharp-1.21.1/.omnisharp/1.34.2/omnisharp/.msbuild/Current/Bin/Roslyn
        CscToolExe = csc.exe
        MSBuildToolsPath = /Users/ddumaresq/.vscode/extensions/ms-vscode.csharp-1.21.1/.omnisharp/1.34.2/omnisharp/.msbuild/Current/Bin
        TargetFrameworkRootPath = /Library/Frameworks/Mono.framework/Versions/5.4.1/lib/mono/xbuild-frameworks

    Detecting Cake files in '/Users/ddumaresq/EducationPlannerBC/Dockerize/EPAS/sources/develop-branch/epbc-ui'.

    Could not find any Cake files

    Detecting projects in '/Users/ddumaresq/EducationPlannerBC/Dockerize/EPAS/sources/develop-branch/epbc-ui/EPBC.PSAS.sln'.

    Queue project update for '/Users/ddumaresq/EducationPlannerBC/Dockerize/EPAS/sources/develop-branch/epbc-ui/EPBC.PSAS.csproj'

    Queue project update for '/Users/ddumaresq/EducationPlannerBC/Dockerize/EPAS/sources/develop-branch/epbc-middleware-client/EPBC.BusinessMiddleware.Client/EPBC.BusinessMiddleware.Client.csproj'

    Detecting CSX files in '/Users/ddumaresq/EducationPlannerBC/Dockerize/EPAS/sources/develop-branch/epbc-ui'.

    Could not find any CSX files

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.CSharpFormattingWorkspaceOptionsProvider, Order: 0

    Loading project: /Users/ddumaresq/EducationPlannerBC/Dockerize/EPAS/sources/develop-branch/epbc-ui/EPBC.PSAS.csproj

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.RenameWorkspaceOptionsProvider, Order: 100

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.EditorConfigWorkspaceOptionsProvider, Order: 200

    Configuration finished.

    Omnisharp server running using Stdio at location '/Users/ddumaresq/EducationPlannerBC/Dockerize/EPAS/sources/develop-branch/epbc-ui' on host 60046.

    "/Users/ddumaresq/.vscode/extensions/ms-vscode.csharp-1.21.1/.omnisharp/1.34.2/omnisharp/.msbuild/Current/Bin/NuGet.targets" cannot be imported again. It was already imported at "/Users/ddumaresq/.vscode/extensions/ms-vscode.csharp-1.21.1/.omnisharp/1.34.2/omnisharp/.msbuild/Current/Bin/Microsoft.Common.CurrentVersion.targets (6060,3)". This is most likely a build authoring error. This subsequent import will be ignored. [/Users/ddumaresq/EducationPlannerBC/Dockerize/EPAS/sources/develop-branch/epbc-middleware-client/EPBC.BusinessMiddleware.Client/EPBC.BusinessMiddleware.Client.csproj]

    Successfully loaded project file '/Users/ddumaresq/EducationPlannerBC/Dockerize/EPAS/sources/develop-branch/epbc-ui/EPBC.PSAS.csproj'.

/Users/ddumaresq/EducationPlannerBC/Dockerize/EPAS/sources/develop-branch/epbc-ui/EPBC.PSAS.csproj
/Library/Frameworks/Mono.framework/Versions/5.4.1/lib/mono/xbuild/15.0/Microsoft.Common.targets/ImportAfter/Microsoft.NuGet.ImportAfter.targets(21,3): Error: "/Users/ddumaresq/.vscode/extensions/ms-vscode.csharp-1.21.1/.omnisharp/1.34.2/omnisharp/.msbuild/Current/Bin/NuGet.targets" cannot be imported again. It was already imported at "/Users/ddumaresq/.vscode/extensions/ms-vscode.csharp-1.21.1/.omnisharp/1.34.2/omnisharp/.msbuild/Current/Bin/Microsoft.Common.CurrentVersion.targets (6060,3)". This is most likely a build authoring error. This subsequent import will be ignored. [/Users/ddumaresq/EducationPlannerBC/Dockerize/EPAS/sources/develop-branch/epbc-middleware-client/EPBC.BusinessMiddleware.Client/EPBC.BusinessMiddleware.Client.csproj]
    Adding project '/Users/ddumaresq/EducationPlannerBC/Dockerize/EPAS/sources/develop-branch/epbc-ui/EPBC.PSAS.csproj'

    Loading project: /Users/ddumaresq/EducationPlannerBC/Dockerize/EPAS/sources/develop-branch/epbc-middleware-client/EPBC.BusinessMiddleware.Client/EPBC.BusinessMiddleware.Client.csproj

    Successfully loaded project file '/Users/ddumaresq/EducationPlannerBC/Dockerize/EPAS/sources/develop-branch/epbc-middleware-client/EPBC.BusinessMiddleware.Client/EPBC.BusinessMiddleware.Client.csproj'.

     Did this suddenly start happening? Does it happen for all projects for you or just one?

I have

removed the ~/.vscode/extension folder
re-installed the C# extension
and the problem has not reappeared.

Okay, thanks. I will close the issue for now, but let us know if it comes back.

will do. thanks!


Heads up: Discord might start to ban accounts (i lost mine)

Acknowledgements

[X] I have checked that there is no other issue describing the same or
similar problem that I currently have, regardless if it has been
closed or open.

[ ] This bug affects Discord website.

[X] This issue is confirmed to be reproducible when WebCord is packaged
on at least all three latest supported Electron major releases.

[X] This issue is reproducible in Chrome, Chromium or any
Chromium-based browser, e.g Brave or Edge (please write in
Additional Context which browser you have used if it is neither
Chrome nor unmodified Chromium).

[X] There are no fixes done to master which resolves this issue.

[ ] My issue describes one of the unstable and/or not fully implemented
features.

[ ] I have found a workaround to mitigate or temporarily fix this issue
in affected releases (please write it in Additional context section
below).

Operating System / Platform
🐧 Linux
Operating system architecture
x64 (64-bit Intel/AMD)
Electron version
31.1.2
Application version
4.10.2
Bug description
My acc was banned while adding someone as friend, beware
Additional context
idk if that's related to webcord

I suppose that means this should be closed.
Also discussions are the place where something like this should probably be discussed tbh, regardless of the urgency.


Section 3 failures are failures

Tests 3.1.1-A through 3.1.1-C report as a failure, but in fact they aren't as long as an implementation of Fedora doesn't allow you to use ldp:contains as a membership predicate.
I was confused by this as well and opened an issue on the spec and they are proposing some wording changes. https://github.com/fcrepo/fcrepo-specification/issues/332
But essentially (as I understand it).
If an implementation can keep containment uses of ldp:contains separate from membership uses then tests 3.1.1-A through C must pass.
But if you try to make a direct or indirect container using ldp:contains as the membership predicate and get a 409 Conflict error, then tests 3.1.1-A through C can be skipped.

The following modifications for tests in section 3.1.1 LDP Containers will be done to solve the issue an meet the new API Specification.
https://fcrepo.github.io/fcrepo-specification/#ldpc
3.1.1-A Test if API permits the creation of client-managed containment triples.
"A server that allows client-managed triples with the ldp:contains predicate must be able to keep those client-managed triples separate from server-managed containment triples."
3.1.1-B If the creation of client-managed triples is not permitted it will test for the correct response status code and for the "constraints document" header. It will test for both Direct and Indirect Containers.
"If a server cannot keep these two uses of ldp:contains separate then any request to use ldp:contains as the membership predicate ([LDP] 2) for Direct and Indirect Containers, or as a client-managed property, must be rejected with a 409 (Conflict) and an accompanying constraints document."
3.1.1-C For tests using the prefer header:

Create a container with client managed ldp:contains. GET the previously created container with prefer "PreferContainment" header. Check for correct response headers.
Create a container with client managed ldp:contains. GET the previously created container with prefer "PreferMembership" header. Check for correct response headers.
Create a container with client managed ldp:contains. GET the previously created container with prefer "PreferMinimalContainer" header. Check for correct response headers.

"Implementations that can distinguish containment triples may permit ldp:contains as a membership predicate or client-managed property; such implementations should support Prefer: header values allowing clients to distinguish this data in the representation of an LDPR as defined in [LDP]: http://www.w3.org/ns/ldp#PreferContainment, http://www.w3.org/ns/ldp#PreferMembership, and http://www.w3.org/ns/ldp#PreferMinimalContainer. ([LDP] <IP_ADDRESS> expansion)"
For test 3.1.1-C:  Can someone provide an example that shows a request to create (POST) a container with the triples corresponding to each of the three types of preferred "ldp:contains" relationships?  I'd also like to see what a correct response should look like, headers and body content, for each of the preferred header types (PreferContainment, PreferMembership, PreferMinimalContainer). That way, I can verify that the tests are sending the correct requests and verifying the correct responses.

@whikloj the section 3.1.1 tests are passing for me at this point, can this issue be closed?

@bbpennel if you are good with the resolution then I'm fine.


OpenAPI of DM in engine is not human readable

Is your feature request related to a problem?
$ curl -v "http://<IP_ADDRESS>:10245/api/v1/jobs/664a9425-e48d-4bab-bfcd-7ca1e1e11924/status"
*   Trying <IP_ADDRESS>:10245...
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 10245 (#0)
> GET /api/v1/jobs/664a9425-e48d-4bab-bfcd-7ca1e1e11924/status HTTP/1.1
> Host: <IP_ADDRESS>:10245
> User-Agent: curl/7.84.0
> Accept: */*
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 200 OK
< Content-Length: 461
< Content-Type: application/json; charset=utf-8
< Date: Thu, 04 Aug 2022 07:08:23 GMT
<
{
    "JobMasterID": "664a9425-e48d-4bab-bfcd-7ca1e1e11924",
    "WorkerID": "664a9425-e48d-4bab-bfcd-7ca1e1e11924",
    "TaskStatus": {
        "mysql-01": {
            "ExpectedStage": 2,
            "WorkerID": "a8d95481-950a-44c7-a1e9-7619b77907f7",
            "Status": {
                "ErrorMsg": "",
                "Unit": 11,
                "Stage": 4,
                "Result": null,
                "Status": null
            }
        }
    }
* Connection #0 to host <IP_ADDRESS> left intact

Describe the feature you'd like
should we change the number to string?
Describe alternatives you've considered
No response
Teachability, Documentation, Adoption, Migration Strategy
No response

Seems JobMasterID equals WorkerID always, maybe we can remove it.

Also, the keys should be lowercase


Bump current release wgpu's version with wgpu = "23"

Current release 0.29.1 is on fe368bacc4a15697e347607e73a56c0299b3d42d wgpu is on 22.1.0
Current commit, have it on  version = "23.0.0" (latest version is 23.0.1)
please update version on crates

im here from https://github.com/EmbarkStudios/puffin/pull/234#issuecomment-2543362177 MR
It would be awesome if puffin_egui will be updated also

just run cargo update -p wgpu in your own project


Flutter: Avoid using context in asynchronous code after await

Consider the example:
Widget build(BuildContext context) {
  return TextButton(
    onPressed: () async {
      Navigator.of(context).pop(); // OK
      await something;
      final obj = Provider.of<Something>(context); // Fails as context is defunct.
    },
    // ...
  );
}

I had spent some time tracing these in the wild.
The rule could be to discourage the uses of BuildContext in async functions after await statement. Before await it is alright because the context cannot get defunct in the build method synchronously.
This is a Flutter lint, while this is dart-lang org. Still I was brought here by the link at https://github.com/flutter/packages/tree/main/packages/flutter_lints
If Flutter lints are not appropriate here, you may want to update that link.

It sounds like you're asking for https://dart-lang.github.io/linter/lints/use_build_context_synchronously.html. Please let us know if that's not the case.


Integration for Lazy Minting Video Data as NFTs on IPFS with meToken Pricing

We would like to integrate a new feature where each video upload has the option to lazy mint the video data from IPFS as an NFT. The claim price for the video NFT can be set via the uploader's chosen meToken price available on Optimism or Base.
Feature Details:

[ ] Lazy Minting: Enable lazy minting of video data on IPFS as NFTs.

[ ] Users should have the option to mint their video data as NFTs during the upload process.

[ ] Pricing Mechanism: Allow uploaders to set the claim price for their video NFTs using their chosen meToken.

[ ] Support for meToken pricing should be available on both Optimism and Base.

Technical Requirements:

IPFS Integration: Ensure seamless integration with IPFS for storing video data.

NFT Minting: Implement lazy minting functionality for video NFTs.

meToken Support: Support meToken pricing for NFTs on Optimism and Base.

Blockchain Interaction: Ensure interactions with Optimism and Base blockchains for setting and claiming NFT prices.

Additional Context:
This feature will enhance the platform by providing users with the ability to monetize their video content through NFT minting and pricing flexibility.

Am going to attempt this.

Some questions please;

Do I need a membership NFT for development?
Is the upload function available?

Some questions please;

Do I need a membership NFT for development?

Yes. Sorry about that, I'll send that over to you now. How did you realize you needed a membership NFT?

Some questions please;

Do I need a membership NFT for development?

Yes. Sorry about that, I'll send that over to you now. How did you realize you needed a membership NFT?

Well, I was on the profile page and saw the message.

I'm not sure if this overlaps but I implemented the Thirdweb ERC1155 TS SDK to the upload process to lazy mint NFTs so that I could move forward with testing the video token gating webhook I setup. See my recent comment on issue #77 for more information.
I didn't realize this might overlap so I apologize if that is the case. I didn't do anything for the pricing but I did add a checkbox to the CreateInfo component to indicate that the user wants to token gate their new video.
Let me know what you think.

I don't think it would. Carry on.


bad sdp: invalid value vzinfo

[sdp.pcap.zip](https://github.com/bluenviron/gortsplib/files/13671001/sdp.pcap.zip)

vzinfo is not one of the media types supported (or even defined in the SDP spec). vnd.onvif.metada should have media type of application according to the ONVIF spec.

Hello, just provide the name and manufacturer of the camera and we will adjust the library in order to make it compatible with this SDP.


X509: fix self-signature check for self-issued certs

This fixes #19095 by providing the most accurate certificate self-signature check for self-issued certs that do not have an SKID.
In this rare case, which can only happen for EE certs and for CA certs not conforming to RFC 5280, we can afford doing the actual self-signature verification on the cert when its data is cached, which is happens once in x509v3_cache_extensions().
Update: The effectiveness if this fix is demonstrated by a new test case in  25-test_req.t using self-issued_v1_EE_no_KIDs_signed_by_CA.pem.
In a separate commit this PR also fixes a subtle bug in the internal ossl_x509_likely_issued() function that was revealed after the above change by a failing new test case:
apps/openssl verify -partial_chain  \
   -trusted test-runs/test_req/self-issued_v1_CA_no_KIDs.pem \
            test-runs/test_req/self-issued_v1_CA_no_KIDs.pem

This is the corner case that its issuer and subject arguments are (semantically) the same cert and this cert is self-issued and appears self-signed (also according to any included AKID and SKID and signature/key algorithm) but in fact is not self-signed.
In this case, in combination with the new - and now actually correct - judgement that the cert is not self-signed, the chain building is mislead to form the chain [cert, cert] which then cannot be verified, which leads to incorrect verification failure.
The fix is to use in this case the ex_flags & EXFLAG_SS information for the cert(s) in question.
Update:
Adding the test case involving self-issued_v1_CA_no_KIDs.pem, I discovered a further bug: when the req, x509, or ca apps produce a cert with no extensions and with SKID and/or AKID being explicitly disabled (i.e., with the specifications subjectKeyIdentifier = none and/or authorityKeyIdentifier = none), the cert version was set to 3 because at first it looked like the cert had extensions, but these were just the intermediate indications that SKID and/or AKID have been disabled.
Moreover, analyzing and fixing thees issues, I noticed that the documentation of X509_check_issued() and X509_check_purpose() was incomplete.
I fixed this as well, and on this occasion also added the missing documentation of X509_check_akid().
Checklist

[x] documentation is added or updated
[x] tests are added or updated

This PR is in a state where it requires action by @openssl/otc but the last update was 30 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 30 days ago

Also we do not have a testcase.

This is questionable if it is a bug fix or a feature.
Definitely not something for 1.1.1 and I am hesitant to approve it for 3.0 too.

I'm fine with not backporting this, as it can also be seen as an enhancement (to fix #19095).

Also we do not have a testcase.

Good point; I'm going to add one.

Adding test cases, I found a further bug, now mentioned in the updated OP above.
Added a fix for it as well, and fixes for incomplete documentation of two functions involved.
Meanwhile I am convinced again that this PR does constitute bug fixes and thus should be backported.

This PR is in a state where it requires action by @openssl/otc but the last update was 30 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 61 days ago

Rebased to fix merge conflicts

This PR is in a state where it requires action by @openssl/otc but the last update was 30 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 61 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 92 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 123 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 154 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 185 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 216 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 247 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 30 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 61 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 92 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 123 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 154 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 185 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 216 days ago

Given that this PR has been inactive for a year and requires a rebase, I'm closing for inactivity.  Please reopen if you would like to pick this back up.


MxModelReflection -> Mx Model Reflection

Thanks for doing this Luyao.
I thought it was my job, but you had already merged the improvements before I got as far as doing my review.
So, obviously, I didn't see that there were problems.


Correct bidi types for remaining Arabic chars

As pointed out in mozilla/pdf.js#7844 by @timvandermeij. Maybe we should pull this into a shared module?

Merged. Now that we have a build system, relying on NPM modules would be workable, so a shared module could work.


gdi+ status 18 on image.loadFromFile()

When trying to run app in Windows there is error message like in screenshot (https://drive.google.com/file/d/1sjqSQYcinqL93HpZA1Z1iZB18_7A0iui/view?usp=share_link).
From Microsoft page this error means:
Indicates that the GDI+API is not in an initialized state. To function, all GDI+ objects require that GDI+ be in an initialized state. Initialize GDI+ by calling
GdiplusStartup.
Im not sure where could I make mistake.
Additionaly I have tried to use gtk3 on windows with forceGTK.
Unfortunately this also doesn't work for me.
I installed gtk3 and demo app (which is not bundle with gtk) and it runs without problems.
But my app give error not libgtk-3-0-dll (https://drive.google.com/file/d/16PbfZEDea9q9j3rFcfQS9XkstXOx96Uc/view?usp=share_link).
Any help or suggestions how to resolve my problems would be appreciated :)

For first issue I have found solution.
loadFromFile cannot be call before app init.
I thought of image.loadFromFile as constat to put on beggining of file :<
Problem solved, but thanks for awesome tool for building simple apps ! :)


VS not found by installer

Does this extension also works in the Community Edition? The installer doesn't even find the installation on my machine. I have the latest VS version 15.7.4 installed. What means product "Global Location"? Might AtmelStudio interfere?
04.07.2018 15:08:43 - Microsoft VSIX Installer
04.07.2018 15:08:43 - -------------------------------------------
04.07.2018 15:08:44 - Initializing Install...
04.07.2018 15:08:44 - Extension Details...
04.07.2018 15:08:44 - 	Identifier         : Microsoft.VisualStudio.LiveShare
04.07.2018 15:08:44 - 	Name               : VS Live Share - Preview
04.07.2018 15:08:44 - 	Author             : Microsoft
04.07.2018 15:08:44 - 	Version            : 0.3.371.21992
04.07.2018 15:08:44 - 	Description        : Real-time collaborative development from the comfort of your favorite tools.
04.07.2018 15:08:44 - 	Locale             : en-US
04.07.2018 15:08:44 - 	MoreInfoURL        :
04.07.2018 15:08:44 - 	InstalledByMSI     : False
04.07.2018 15:08:44 - 	SupportedFrameworkVersionRange : [4.5,)
04.07.2018 15:08:44 -
04.07.2018 15:08:46 - 	SignatureState     : ValidSignature
04.07.2018 15:08:46 - 	SignedBy           : Microsoft Corporation
04.07.2018 15:08:46 - 	Certificate Info   :
04.07.2018 15:08:46 - 		-------------------------------------------------------
04.07.2018 15:08:46 - 		[Subject]       : CN=Microsoft Corporation, OU=OPC, O=Microsoft Corporation, L=Redmond, S=Washington, C=US
04.07.2018 15:08:46 - 		[Issuer]        : CN=Microsoft Code Signing PCA 2010, O=Microsoft Corporation, L=Redmond, S=Washington, C=US
04.07.2018 15:08:46 - 		[Serial Number] : 33000001FC5DB8191027A666C10000000001FC
04.07.2018 15:08:46 - 		[Not Before]    : 05.10.2017 20:26:08
04.07.2018 15:08:46 - 		[Not After]     : 05.10.2018 20:26:08
04.07.2018 15:08:46 - 		[Thumbprint]    : 5E940B33DEAE5945F506ADAB946A997867A40FEF
04.07.2018 15:08:46 -
04.07.2018 15:08:46 - 	Supported Products :
04.07.2018 15:08:46 - 		Microsoft.VisualStudio.IntegratedShell
04.07.2018 15:08:46 - 			Version : [15.0.27220,16.0)
04.07.2018 15:08:46 -
04.07.2018 15:08:46 - 	References         :
04.07.2018 15:08:46 - Signature Details...
04.07.2018 15:08:46 - 	Extension is signed with a valid signature.
04.07.2018 15:08:46 -
04.07.2018 15:08:46 - Searching for applicable products...
04.07.2018 15:08:48 - Found installed product - Global Location
04.07.2018 15:08:48 - Found installed product - AtmelStudio
04.07.2018 15:08:48 - Found installed product - ssms
04.07.2018 15:08:48 - VSIXInstaller.NoApplicableSKUsException: This extension is not installable on any currently installed products.
bei VSIXInstaller.App.InitializeInstall(Boolean isRepairSupported)
bei VSIXInstaller.App.InitializeInstall()
bei System.Threading.Tasks.Task.InnerInvoke()
bei System.Threading.Tasks.Task.Execute()

I removed Atmel Studio and the installer found my Community Edition.

@needfulthing Thanks for reporting this! Even though you unblocked yourself, we’ll use this issue to track the general issue of our installer getting “confused” when a machine has other products installed (we saw this same issue with Team Explorer as well).

This issue should have been fixed now with the latest release of Live Share extension. Closing it. Please re-activate if you still see it reproduce.


Curtains switch

Hello,
Do you think you will be to add the ablitlity to manage curtains switch thru homebridge plugin ?
Homebridge last version
On a raspberry pi

Duplicate of #18


CancelationToken registration fix

This PR adds proper cleanup of CancellationTokenRegistations by wrapping CancellationToken.Register() into a using.

checkout 415c7bb and add a breakpoint at Android UserDialogsImpl.cs:344
compile and tap on new example (Specific cases -> Two alerts with one Cancellation Token Source)
tap Ok
wait until second dialog gets dismissed
count breakpoint hits (you will count 2 hits,but it should only be one, because the first dialog has been closed)
checkout 5b42c4d
do 2-5. (you will count 1 hit)

Because of the missing deregistration, you could end up with some memory leaks because of the captured closure in all CancellationToken.Register().
see: MSDN: How to: Register Callbacks for Cancellation Requests

Thanks for the PR


WinApi Run linux?

It's possible to compile winapi project on linux?

Compile? Sure. I see no reason why you shouldn't be able to. Run? That's a whole different story - Perhaps, through Wine, as long as the API surface is supported by Wine.


The mqtt-client adapter seems not to work with a mqtt v3 broker

Describe the bug
For me it is not possible to connect the mqtt-client adapter to my current mosquitto broker. It is a version 3 broker so I select the mqtt version 3 in the adapter settings.
# mosquitto -h
mosquitto is an MQTT v3.1 broker.

IP, port and firewall are not an issue. I did some tests with nc on the iobroker instance and it told me the port is open. Other applications are also very fine with the setup.
To Reproduce
Steps to reproduce the behavior:

Install the mqtt-client adapter v1.6.3
Go to the adapter settings and insert IP /  Port (1883) and select mqtt version 3
Save and restart the adapter
The adapter will be yellow, not green

Expected behavior
The adapter should switch to green state.
Screenshots & Logfiles

Versions:

Adapter version: v1.6.3
JS-Controller version: 4.0.23
Node version: v14.20.1
Operating system: Synology DSM 6 --> Docker (Docker version 18.09.8, build bfed4f5) --> Debian 11 Image with iobroker

Damn... solved :)

Can you implement that? Maybe the possibility to switch between protocol version 4 and 3 is not enough. Maybe a selector for the mqtt server version?


Module not found: Can't resolve 'click-to-react-component' in '/Users/mike/Documents/work/bix-b2b/src'

my app was created by Create-react-app, when I use this component, I get this error;
Module not found: Can't resolve 'click-to-react-component' in '/Users/mike/Documents/work/bix-b2b/src';

React: 16.8.6
webpack: 4.x

Are you able to reproduce with https://github.com/ericclemmons/click-to-component/tree/main/apps/cra?  It seems to be working there...


Getting an integer return from REDIS::publish()

https://github.com/fermyon/spin-rust-sdk/blob/1e9cc89ee7309e477d20bcfe0af730e687934695/wit/redis.wit#L19
Currently .publish returns Result<(), Error>.
But if this can return an integer value instead of () would be better like redis-cli.
redis-cli returns an integer how many nodes get the published msg.
This is an enhancement.

Sounds good to me.  As a workaround, you should be able to use the general-purpose execute method instead of publish and get the result that way.


Using c3-angular with require.js - problem!!

Hello, I am introducing require.js modularisation in my app, which uses c3-angular.
I get an error c3 is not defined which puzzles me a lot, because I have specified that c3-angular depends on c3.
Here are a couple of my requirejs shims for c3-angular. Have you got any experience with this sort issues?
'c3-angular' : { deps: ['angular','c3'] },
'c3' : { deps: ['d3'], exports: 'c3' },

One of the major reasons of trouble in this case is that all c3-angular code is minified.
An uncompressed version would help me enormously with analyzing this problem, and I am opening another issue just for that.

Hm, sorry, I have o experience with requirejs. If I can find the time I'll try to investigate. At the moment I ignore the non minified version. I do not really see a problem to include it. Will do that in the next release. The non minified version is in the examples folder by the way. So you can check it. Also in browsers the .map file is used to make the minified version readable again.

+1

The latest release does contain the non minified version. Since I have no experience with RequireJS I am not sure a fix will come soon from my hand. Any help would be nice


impl back/fwd navigation

it'd be useful if we could go backward and forwards like the browser back/fwd button when GS nagivated somewhere... currently .[ is broken but it only worked in a very limited way so it needs a little thought

+1 for this

Hi,
It still seems broken. Just wanted to check if there is any plan to fix this ?

(Assuming it works properly) Sublime Text has this feature: Menu > Goto > Jump {Forward/Backward}

Yay, works !


Explicit experience source patch

Before submitting

[X] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[X] Did you read the contributor guideline, Pull Request section?
[X] Did you make sure to update the docs?
[X] Did you write any new necessary tests?

What does this PR do?
This patch removes the experience_source code from the RL examples that were based on the Ptan library. Instead, the logic for generating the batch and stepping through the environment is provided explicitly by in the models train_batch function.
This makes the RL examples more transparent and easier to build upon by PL Bolts users
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Hello @djbyrne! Thanks for opening this PR.

In the file pl_bolts/models/rl/vanilla_policy_gradient_model.py:

Line 130:29: E126 continuation line over-indented for hanging indent
Line 131:25: E126 continuation line over-indented for hanging indent

Do see the Hitchhiker's guide to code style

@Borda , are tests frozen?

@djbyrne it seems that something went wrong with the CI configs that they were not found/recognized...

@williamFalcon resolved and prevented in future...


PE file header "Load_Configuration_Directory_Table" isn't fully parsed , missing CFG structures

When analyzing PE files compiled with CFG,  Binary Ninja doesn't recognize and parse the full CFG structure within the  Load_Configuration_Directory_Table
https://docs.microsoft.com/en-us/windows/win32/debug/pe-format#the-load-configuration-structure-image-only
It misses information about the CFG function table which is a list of RVA offsets to all the functions within the binary that are marked as valid CFG calls.
BN could automatically mark all these offsets as valid cross refs to the corresponding functions
Incomplete Load_Configuration_Directory_Table

CFG RVA offset table

It seems that BN does recognize the CFG structures in certain PE files (example below) like jscript.dll but for some other files the analyzer does not parse it.

Apologies if you already shared it on slack and I've forgotten, but do you have an example you can share where it fails to parse?

@psifertex here is the example dll file-
jscript9.dll.zip

according to the header, 0x10008f38 should be the CFG table but BN identifies the region around it as code and creates a function as can be seen below-

Thanks! That's helpful. Yeah, if the original parsing isn't correct the linear sweep will try to fill in the gaps (sometimes erroneously). Appreciate the test case.

Fixed in 2.1.2321-dev.


Freezing, Reboot,  Resprings after update of ldid/u0 ressources freezed and b38 was installed

I tried to update the mandatory packages i.e. ldid, unc0ver jb resources etc, but the installation froze and my phone rebooted.
Then I installed b38 over b34 and there the real problems began.
First "failed to extract bootstrap" for 5 times.
I checked reinstall Cydia, delete Cydia cache, got the same error again, but was able to open Cydia. So I updated the mandatory tweaks again and got into jb mode after respring.
Then my phone started lagging after opening some apps - to completely freeze afterwards, trying to respring after some time, but failing and rebooting.
Then in nojb mode the jailbreak worked on first try. But the freezing persists. Can't even use my phone for more than 5 minutes without freezing and rebooting
edit: downgraded to b36 and seems to run fine now. Some users on reddit reported the same issue.
edit2: nope. Still the same, but it runs for a longer time
iPhone 8 12.1.1 b38
edit: crashlogs+unc0ver Diagnostics Data (b36): CrashReporter.zip (count: 55+1)
most prominent: stacks+backboardd and panic-full
Some users on reddit reported the same issue.
https://www.reddit.com/r/jailbreak/comments/aw6m5a/question_random_crashes_on_b38/
https://www.reddit.com/r/jailbreak/comments/aw8x3z/help_springboard_keeps_restarting/
https://www.reddit.com/r/jailbreak/comments/aw8rhd/help_last_update_on_cydia_freeze_my_device_after/
https://www.reddit.com/r/jailbreak/comments/aw9u58/help_having_issues_reinstalling_jailbreak/

@13XxM1CHA3lxX37 please post logs and screenshots stated in the guidelines: #627 .

Downgraded to b36, but still the same issue. I'm trying to get the crashlogs, but kinda difficult if the device keeps rebooting after a short time

@13XxM1CHA3lxX37 read the guidelines, it doesn't just ask for crash logs.

This are all crashlogs generated today + diagnostics data from unc0ver. Some may be redundant, but I can't differentiate.
CrashReporter.zip

#669
#658

culprit found: cr4shed.
The freezing occured with a libbulletin update that was updated along with ldid and u0 ressources.


GameChip Tile() method should return object

There should be a tile class with all of a tile's properties.

This has been fixed.


ENH: Integrating OpenBLAS Meson build

Is your feature request related to a problem? Please describe.
This issue is meant to discuss the feature and integration approach of Meson based OpenBLAS build.
There have been discussions around adding Meson based build to OpenBLAS (e.g., https://github.com/scipy/scipy/issues/12633#issuecomment-1078681133). In https://github.com/HaoZeke/OpenBLAS/tree/mesonBasic there's already a Meson port of OpenBLAS that supports x86_64 (four subarchitectures) and arm64 (only armv8). NumPy's and SciPy's test suites pass when this port is used.
Describe the solution you'd like.
The initial PR is here: https://github.com/scipy/scipy/pull/21380. Rather than git submodule, it should be incorporated as a Meson submodule, and build from source with I think meson submodule foreach ... commands (?).
Other topics to discuss taken from https://github.com/scipy/scipy/pull/21380#issuecomment-2288784487:

the desired behavior of the feature itself (I think this is a last fallback, after we've failed to locate not only another OpenBLAS but any other library like MKL etc.)
the name of a dev.py flag
what the CI job should look like (I think it has to use pip install ., not dev.py)

Describe alternatives you've considered.
No response
Additional context (e.g. screenshots, GIFs)
No response

What does the meson submodule mean? Making everyone git clone/checkout openblas is quite heavy. Does a meson submodule only git clone or download openblas if it's asked to?
W.r.t a CI job, how long does it take to build?
My concern with the dev.py flag is that if we call it something like --with-meson-openblas people are going to get confused and think that's the default way to build with openblas.

The docs for meson subprojects are at https://mesonbuild.com/Subprojects.html

W.r.t a CI job, how long does it take to build?

Assuming it were built from source, I think around 10-40 minutes is the right ballpark.
There are some SciPy maintainers who maintain a pip package which contains binaries for OpenBLAS, and looking at their Actions logs, this takes about 10 to 30 minutes to build and test. https://github.com/MacPython/openblas-libs/actions
(There are workflows that take longer, but they're either build matrices that are building lots of different options, or doing something like emulating a different CPU architecture.)
@mattip might be a better person to ask, though.

What is the desired behaviour of the CI entry? Given that the build demands will be larger, it's important to justify that. Building OpenBLAS is already checked by the openblas-libs/build of the wheel, and most of the CI entries use OpenBLAS to run the scipy test suite. This means we've already checked most of that kind of functionality.
If the point of the test is to check that the dev.py flag works, then it may be less important to have such a test. Alternatively, we could hijack one of the existing CI entries to check that the flag works, but I think we'd definitely want to be using ccache (fcache?) to minimise the OpenBLAS build time.

What is the desired behaviour of the CI entry? Given that the build demands will be larger, it's important to justify that. Building OpenBLAS is already checked by the openblas-libs/build of the wheel, and most of the CI entries use OpenBLAS to run the scipy test suite. This means we've already checked most of that kind of functionality.

As I understand the CI job is meant to check that OpenBLAS can be built with meson as a fallback option when no existing version (including the wheel build, which doesn't use the meson build system) works. The build seems to complete with tests in around 25 minutes on CI so it shouldn't be too onerous. From a SciPy perspective then, is the wheel a better option than a meson subproject? I'd have thought not, since the wheel builds are "fatter" since they build symbols for a dynamic list of architectures instead of just the build system (currently the meson build doesn't support DYNAMIC_ARCH and uses the makefile native getarch system to determine the architecture and sub-architecture (e.g. HASWELL, NEHALEM etc).

If the point of the test is to check that the dev.py flag works, then it may be less important to have such a test. Alternatively, we could hijack one of the existing CI entries to check that the flag works, but I think we'd definitely want to be using ccache (fcache?) to minimise the OpenBLAS build time.

+1

What does the meson submodule mean? Making everyone git clone/checkout openblas is quite heavy. Does a meson submodule only git clone or download openblas if it's asked to?

Yes, as per the command line options in the document @lucascolley linked to. Rephrasing the relevant bits, the subproject can be configured as an opt-in "absolute fallback" if all other detection fails. No major additional clones take place before that (the .wrap file defining the meson subproject is miniscule, but will need to be part of scipy).

W.r.t a CI job, how long does it take to build?
My concern with the dev.py flag is that if we call it something like --with-meson-openblas people are going to get confused and think that's the default way to build with openblas.

Maybe not even have a dev.py flag for it explicitly? With the subproject approach, dev.py can default to --wrap-mode=nodownload we only need to document that a user can, if the build fails due to not having openblas run meson subprojects download to get the source or meson subprojects update to update previously downloaded sources.

I am still stuck on the motivation of adding this to SciPy. I don't understand why SciPy should be building OpenBLAS from source.

As I understand the CI job is meant to check that OpenBLAS can be built with meson as a fallback option when no existing version (including the wheel build, which doesn't use the meson build system) works.

Doesn't the scipy wheel build use meson? I see meson is the build system in pyproject.toml.
Maybe I am missing something basic here.

I think the rationale is that some people have issues setting up BLAS. So this flag will take care of building + installing BLAS so it can be used as a fallback, and they don't need to deal with PKG_CONFIG_PATH, etc.
Whilst I can see the benefit of that, I don't think it deserves a special CI item, unless it's used as part of something else

I think the rationale is that some people have issues setting up BLAS.

Thanks, that makes sense. Is there an issue open for that use case, or some other way to engage with such users? One of the justifications for the effort that went into making the scipy-openblas wheels was exactly this scenario. Perhaps these people are not aware of the scipy-openblas wheels or we can guide them through building the wheels in a way that will solve their use case, rather than offloading that problem from MacPython/openblas-libs onto SciPy.

FWIW, I agree with Matti on this one. It's really not clear what is the benefit w.r.t either using a scipy-openblas wheel, or (if for some reason that does not cut it?) building OpenBLAS manually just before starting to build scipy and linking to this fresh build.

Hey all, sorry for the lack of input here so far (this issue and work were my idea, but I got ill just when this discussion kicked off).
The tl;dr of my opinion is:

We should do this, it can be a very useful tool for both end users and for debugging.
The constraint is of course that it should be robust and performant enough that it solves a lot more problems than it creates. That speaks for itself - and doing this is reversible, so there isn't much risk here.
It should be opt-in only so users know what is happening, via a message saying "Your build failed because a BLAS library could not be located. Please see http://scipy.github.io/devdocs/building/ for how to install one. Alternatively, you can try again with pip install scipy -Csetup-args=-Dblas=openblas-src, which will build OpenBLAS from source as part of the SciPy wheel build." (and we show the second half of that message only on platforms that we know should work)

Now to address the comments above on build time, difference with scipy-openblas, etc.
Build time: OpenBLAS builds in just under 50% of the time that SciPy itself builds (and with much lower memory requirements, because no C++). The reason that the scipy-openblas builds are so slow is that they have to be built with DYNAMIC_ARCH=1, meaning the same sources get rebuilt 5 times with different sets of SIMD flags. When one builds locally, the build can just target whatever the CPU supports, which is way faster (and slightly more optimal).
Binary size: for the same reason as the build time being faster, the binary size also ends up being smaller. It looks like they'll be about 20% - 25% smaller for x86-64.
Build complexity: copy-pasting a build command like pip install scipy -Csetup-args=-Dblas=openblas-src from an error message and it "just working" is way easier than having to install scipy-openblas, messing with pkg-config, adding a pre-loading snippet to _distributor_init.py, and then stringing everything together. Really one must then use auditwheel on it as well, because there is also the risk that pip will cache the built wheel, which is then going to break the next time the user installs scipy into a new venv (cause scipy-openblas32 won't be present, nor is the dependency declared). The average maintainer still has trouble with this, and for users not familiar with SciPy it is much harder still - this recent thread on Discourse illustrates that.
What about scipy-openblas? The scipy-openblas machinery and wheels are useful for nightlies and very much necessary for our own wheel builds, but it's not really usable as a debugging tool. The cycle of going through a large rebuild there (which so far needs Matt's shepherding), then uploading wheels to PyPI or anaconda.org, then downloading them locally and figuring out how to link against them has so far proven to enough overhead that it's not being used in practice. It is also very expensive CI time/jobs wise to rebuild and publish wheels for all platforms when you really just need a single OpenBLAS build that works on your machine.
So potential benefits:

Helps users that now struggle with installing a BLAS library or messing with delvewheel & co. (especially on Windows this is a mess).

Can make it a lot easier for ourselves to test changes in OpenBLAS (just build a wheel against a different commit). See this comment for an example of how tricky it is right now to link in a different OpenBLAS build when debugging. Even experienced SciPy and scikit-learn maintainers tend to have a hard time here (we can probably count the number of people who can figure it out in less than 30 minutes on one hand).

The ability of figuring out easily whether a change in OpenBLAS affects some linear algebra related problem we're seeing is, in my opinion, in itself already valuable enough to implement this "build from source" option.

(much more speculative, thanks to @mattip for the suggestion) could potentially be useful if we ever intend to unify thread pools. Right now we have a bunch of separate ones, with more opportunities for over-subscription.

A bit of context for how this can actually work. This was actually significantly more difficult to fully understand and do than I had anticipated. After writing the docs in https://github.com/mesonbuild/meson-python/pull/700 I have a pretty good idea about all the options for folding a shared library into a wheel build. The obvious alternative, building a static libopenblas.a and folding that into a regular shared library within the SciPy build config, unfortunately did not work. That static library approach is usually the most straightforward one, we use it for HiGHS and for vendored Fortran code for example, but OpenBLAS has internal machinery that seems to make this difficult. I seem to remember some summary about that being written somewhere, but can't find it anymore.
So, the way that seems to be working is this:

Users opts in and kicks off a wheel build with: pip install scipy -Csetup-args=-Dblas=openblas-src (or uv, or python -m build, etc.)
We have a small shim in subprojects/openblas that retrieves OpenBLAS sources and build config from the Meson WrapDB service
A regular libopenblas.so gets built and marked for installation (we can name it as libscipy_openblas.so or whatever, to avoid clashes). It is built without DYNAMIC_ARCH, so each source file only gets compiled once, optimized for the CPU on the user's machine.
The rest of the SciPy build completes.
meson-python knows how to automatically fold in libopenblas.so into the wheel at scipy.mesonpy.libs and rewrite RPATHs with patchelf (on Windows slightly differently of course, since there are no rpaths).


Feature: add a 'error' event to ImageOverlay

As per https://stackoverflow.com/questions/41841381/leaflet-imagelayer-seturl-error-handling , it should be useful to have a error event in L.ImageOverlay, akin to the tileerror event in L.TileLayer.

Looking into this issue.
The problem is in
https://github.com/Leaflet/Leaflet/blob/master/src/layer/ImageOverlay.js
I suppose ... ?

Yeah. See:
https://github.com/Leaflet/Leaflet/tree/ad75456fa54aee4d38e92dd7654128819cf8e31a/src/layer/ImageOverlay.js#L167
and also:
https://github.com/Leaflet/Leaflet/tree/ad75456fa54aee4d38e92dd7654128819cf8e31a/src/layer/tile/TileLayer.js#L144

@AP-whitehat yes, exactly. From a quick look, an error handler should probably be added in _initImage.

What would be better to use as a dummy placeholder  for error links
http://www.birthdaydirect.com/popup_image.php?pID=16068
or http://hdimagesnew.com/wp-content/uploads/2016/09/404.png
or something else .
I'll have to include the png in dist/images/

Issue not yet Fixed ,
am I going in the right direction ?
please point out my mistakes 😛

@AP-whitehat Please open up a pull request, so we can comment on specific lines of your code :-)

@IvanSanchez
Please Review my PR

A pull request related to this issue was merged, can we close this one?

Closed by #5307!


[TF 2.0 API Docs] tf.image.central_crop

URL(s) with the issue:
https://www.tensorflow.org/versions/master/api_docs/python/tf/image/central_crop
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/ops/image_ops_impl.py
Description of issue (what needs changing):
Usage example
No usage example provided
Submit a pull request?
Yes

Closing the issue since PR is merged to master. Thanks!


[idea] computed properties with S.js

Feature request
We can use a getter inside of updated to have semi-computed properties, for example https://github.com/skatejs/skatejs/issues/1334#issuecomment-408751758.
But this doesn't handle caching values and the glue code associated with that.
If using a render function, we don't need much glue code, we just pass vslues into the markup (f.e. using renderer-preact with JSX.
However, for imperative code, glue code has to be written (f.e. above example).
Maybe we can use S.js in a mixin called withComputed or similar to make caching automatic, and so that when a user's updated method is called they can use the variables without worrying about checking if props have changed.
Use case(s)
Writing less code

The core, functional concept of Skate revolves around computing values in render() (or a possible future getDerivedStateFromProps(), or similar. I'd prefer if this were an independent mixin.

Let's keep in mind the cost of diffing and how we can avoid it: DOM diffing was brought to the spotlight by React, but only because it was better than alternatives (f.e. handlebars with innerHTML was popular with Backbone, but slow). Direct notifications (f.e. Surplus) will always be faster. Outside of DOM, I've solved performance problems by moving data away from Redux and using light-weight EventEmitter (direct notification, but not batched).

Functional data flow doesn't equate to eventual diffing. Things like LitHTML present the ability to reach into the DOM and update specific spots without the baggage of diffing and virtual DOM.


Integration with 'personal' keymaps

Meaning, my efforts at integrating legendary with my mappings-ng.lua file.
I've achieved this by using the :require('legendary').bind_whichkey(wk_tbls, wk_opts) (thanks to mat) command from the vim comandline. All good. But when I tried to integrate this into my config (by using the above 'require' in place of my 'require('legendary').setup()' line everything blew up (nvim init failed).
My current (working) config is attached for reference.
Seems to me this solution is rather 'inelegant'. Looking for a cleaner solution.
/tom
paste.md

pasting the snippet for convenience:
-- nvim/lua/configure/legendary.lua

local path

local paths = require('paths')
if vim.fn.isdirectory(paths.join(paths.home, 'Projects/legendary.nvim')) > 0 then
  path = '~/Projects/legendary.nvim'
else
  path = 'mrjones2014/legendary.nvim'
end

return {
  path,
  config = function()
    require('legendary').setup()
    require('legendary').bind_whichkey(TS.leader_nmap, TS.leader_xmap)
  end,
}

Seems like we can't reproduce this issue. Regardless, I've updated setup so that you can make the config a little cleaner:
https://github.com/mrjones2014/legendary.nvim/commit/74229ea6b56fcc8df5b2be2e4af56ab5b54575d0


Make API calls return mongo-like results.

Refs #5

/cc @nevf
I converted most of the APIs, can you try this branch?
I'm going to merge this if it's good enough for you and release new version.

@nevf I released a new version with upsert and this PR

@YuriSolovyov Great and thanks.


TerserPlugin - exclude specific module / files

I haven't found a way to customize the TerserPlugin exclude option (and others).
Do you have any suggestion?
Terser is removing important functions from an external pure JavaScript library (bpmn-js) and I absolutely need to exclude it, while maintaining the build optimization.

In the future, please follow the issue template.  It is quite difficult to provide assistance without the requested information.
The package is marked as being side effect free (https://github.com/bpmn-io/bpmn-js/blob/master/package.json#L38).  If code is being removed, the most likely cause is that the package is not actually side effect free.
This line appears to confirm the suspicion: https://github.com/bpmn-io/bpmn-js/blob/master/lib/Viewer.js#L154

@clydin Sorry about the template.
And yes, exactly that function is removed. Can I override this behavior?

This is actually an issue with the package.  It is advertising itself as being side effect free when it is not.   The CLI is acting appropriately given the information it is being provided by the package.

@clydin yeah, I know, I've been reading issues regarding Terser/UglifyJs behavior.
But what if I cannot modify the external library?

If you need a temporary workaround, you can remove the side effects field from the installed package.  This would need to be done every time it is installed.  For completeness, a post install script for the project is a common method to augment errant packages.
If the library is being imported into the application then there is no separate file to exclude.  Only the final bundles are run through the minifier.

@clydin

If the package is being imported into the application then there is no separate file to exclude. Only the final bundles are run through the minifier.

Thanks for confirming that. I tought I was doing something wrong while debugging the builder.

a post install script for the project is a common method to augment errant packages.

I'll give this one a try, as I don't want to hand-edit the file every time.
Last question, if you can answer: the inherit function simply uses Object.create(...), what would be a side-effect free alternative?
if (typeof Object.create === 'function') {
  // implementation from standard node.js 'util' module
  module.exports = function inherits(ctor, superCtor) {
    ctor.super_ = superCtor
    ctor.prototype = Object.create(superCtor.prototype, {
      constructor: {
        value: ctor,
        enumerable: false,
        writable: true,
        configurable: true
      }
    });
  };
} else {
  // old school shim for old browsers
  module.exports = function inherits(ctor, superCtor) {
    ctor.super_ = superCtor
    var TempCtor = function () {}
    TempCtor.prototype = superCtor.prototype
    ctor.prototype = new TempCtor()
    ctor.prototype.constructor = ctor
  }
}

It's not actually the content of the function but rather the location of the call.  Since the call is at the module level, importing the module will result in execution and cause some type of effect by simply importing the module.


Improve local development and startup via docker

Motivation

Ensuring an easier start for new contributors.
Avoiding the need to install necessary development tools on the local system.
Ensuring the project runs in the same environment, eliminating potential issues associated with the system environment.

Purposes

[x] Add the ability to run a project locally in docker compose.
[x] Add dev container support.
[x] Document this in the README.

Known bugs

[x] An error may occur when launching an app container if the database was not ready to accept requests.
This often happens if the volume does not exist, in which case the container needs additional time to create the volume.
Solution: add a health check for the database container.

[x] Dev container fails at build.
Cause: dev container tries to override the configuration and fails.
Logs
[2024-06-15T19:18:33.904Z] Docker Compose override file for building image:
services:
  app:
    build:
      dockerfile: /var/folders/gz/srd0zmvj357gnb8_lk28mj400000gn/T/devcontainercli/container-features/0.64.0-1718479113903/Dockerfile-with-features
      args:
        - BUILDKIT_INLINE_CACHE=1
        - _DEV_CONTAINERS_BASE_IMAGE=dev_container_auto_added_stage_label

[2024-06-15T19:18:33.924Z] Start: Run: docker-compose --project-name dudes -f /Users/user/Desktop/dudes/compose-dev.yaml -f /Users/user/Library/Application Support/Code - Insiders/User/globalStorage/ms-vscode-remote.remote-containers/data/docker-compose/docker-compose.devcontainer.build-1718479113904.yml -f /Users/user/Library/Application Support/Code - Insiders/User/globalStorage/ms-vscode-remote.remote-containers/data/docker-compose/docker-compose.devcontainer.containerFeatures-1718479113923-c8851d69-5ce9-45d0-b5cc-8dc9b122edb3.yml up -d --no-recreate
[2024-06-15T19:18:33.973Z] service "app" declares mutualy exclusive dockerfile and dockerfile_inline: invalid compose project

[x] When opening http://localhost:4300/client on the server, you can’t find the user.
Not sure if this error is related to deployment. ~Maybe you need to run pnpm run db:generate.~
Update: This is not a deployment issue.
Logs
client_prisma_error.log

[x] There is no mounting of the volume with project files in the app container.
Currently, the project is copied during the creation of the image container.

Addition
At the moment, I can't ensure that the project works correctly in docker compose because the Twitch console refuses to register a new application due to the link not corresponding to the https protocol, even for links that have the https protocol.
It may be worth adding nginx to docker compose for development, which will allow testing some things locally.

Development in Docker is already working. But this is not yet documented in the README.
And there is still the problem of creating the database for the first time. It is worth adding a database health check and only then launching the app service.

Basically this is the finished stage.
But I would like to add nginx. I was able to add nginx and run it, but it requires me to change the nginx/default.conf file as it has all ports set to 3000. 🤔

[ ] Add some VS_Сode extensions to devcontainer.
[ ] Add pnpm run db:seed execution to Dockerfile.dev.
[ ] Move README style changes to a separate PR, leaving only those related to devcontainer and docker.
[ ] Add container with nginx.
[ ] Do not migrate when the container starts, but only generate types.
Migration will be performed at the discretion of the developer.
[ ] Is it possible to add caching for devcontainer files? Because the startup is too long, by my standards.

Pending approval

[ ] Move postgres into a separate compose file.
[ ] Move app service to .devcontainer.

Added nginx and website to the compose file. This now looks like the production version, except that it is running in dev mode.
nginx will also be able to proxy requests if the project is run locally, since it uses the host machine environment.
nginx works on port 8080, to make twitch application authorization work, it should be added to the list of allowed redirect urls.

@inferst do it please.

This PR is getting more complicated than I expected. It takes time to find out what happens.
Again, for me it's not convenient to have the whole app in dev container.

Again, for me it's not convenient to have the whole app in dev container.

I'm still working on it. It's the next thing I intend to do.
I'm turning PR back into draft until it's in the final stages. As soon as it works as intended and satisfies everyone and is fully documented in the README, I will reopen it for merging.

[ ] Replace docker compose run with docker compose up


Leaderboard UI component for Dashboard

[ ] design non/default state when not competing
[ ] design regular state when you started a trade
[ ] track top 10 accounts plus the current users position

https://www.figma.com/file/R9dygkd4HBA7PdxtE3iLis/External?node-id=435%3A731

<!-- Uploading "image.png"... —>


AWS Cognito NEW_PASSWORD_REQUIRED challenge requiring address, but it dosen't seems to accept any form of address types

Im trying to implement a spring boot back end for cognito, I managed to create one and perform signing in, after the sign in i got to the new password is required to change the default password, so I have submitted my request but got an exception saying:
Invalid attributes given, address is missing.

I have tried to send the address as string, and json object but the same exception occurred.
I have checked the read and write permissions in my cognito pool and they must be writable!
So any suggestions about this issue?

I think that the problem may be the way you configured your cognitive
account. There are a number of settings. If you have not done so, take a
look at my article on cognito on the topstonesoftware.com publications page.
Best,
Ian
On Sunday, February 10, 2019, khaledshamat<EMAIL_ADDRESS>wrote:

Im trying to implement a spring boot back end for cognito, I managed to
create one and perform signing in, after the sign in i got to the new
password is required to change the default password, so I have submitted my
request but got an exception saying:
Invalid attributes given, address is missing.
I have tried to send the address as string, and json object but the same
exception occurred.
I have checked the read and write permissions in my cognito pool and they
must be writable!
So any suggestions about this issue?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/IanLKaplan/CognitoDemo/issues/2, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFOFmg3JkOIaNlliu13f5Nj7iiy_n9rLks5vMJjCgaJpZM4azTnz
.

Could you share a print screen from your attributes tab, at your UserPool


Optimisation: do instant query count() without fetching samples

EDIT: the motivation for this was an instant query (from a recording rule) count by(job, env, team, cluster) ({__name__=~".+"}).
I think most administrators will run something similar any time they feel the number of metrics is too high, and want to know where they are coming from.
Currently count() is implemented as an aggregation, which is called within rangeEval() here: https://github.com/prometheus/prometheus/blob/d677aa4b29a8a0cf9f61af04bbf5bfdce893cf23/promql/engine.go#L1191-L1197
aggregation() takes as input []Sample, so the storage layer must fetch samples.
In the case it has been called from an instant query, the start and end of the range are equal, but nothing changes, it still fetches samples from storage.
It will be cheaper to compute count() the same way we do /series, without fetching samples: https://github.com/prometheus/prometheus/blob/c954cd9d1d4e3530be2939d39d8633c38b70913f/tsdb/querier.go#L140-L143
However this requires some refactoring to be able to run the aggregation (count by (foo,bar), etc.) outside of a function taking []Sample.
I suggest doing this only for the instant-query case, because over a time-range series can come and go, so we need per-timestamp information and may as well get that via samples.

I am not sure that this optimization is worth it. We have the risk of introducing bugs for a very niche usecase. The complexity seems too high to me.

I disagree this is "very niche"; my guess is every Prometheus admin runs such queries. I edited the description to add that info.
As a data point, on a v2.31.1 server with 2 million series count({__name__=~".+"}) took 24 seconds; profile says extracting the samples took 8.5 seconds of that.

Running count() is something I do a lot when auditing metrics usage, especially with big instances shared by many teams.
It is expensive, especially counting by __name__ so I mostly rely on job label and scrape metrics provided by Prometheus, but would love if count({__name__=~".+"}) was cheaper as it would help a lot.

Do we plan to implement this? I can try an implementation as a PR.

we would need to address backwards-compatibility

This came up again; we could have it as a different function name, e.g. count_for_block.
Naming is hard.

Hello from the bug scrub!
Since this came up again as recently as February (see above), we'll leave it open. However, it feels weird to introduce a new function just for a specific performance optimization. Maybe this should be an API call outside of PromQL?
(This reminds me of my wild meta data ideas, which might include information about start and end point of series, so that we could know about the end of a series without decoding samples to find staleness markers… With that in place, we could optimize in this way without breaking semantics.)


Add function that will pull API and Implement API changes at EOD.

Summary
Function needed that will pull API at the end of each day and implement price changes.
Things out of Scope
This function is independent of adding buy orders which is called out in issue #145, although this function will be working in tandem with the buy order function.
Criteria for Completion

This should be a bulleted list of items that, if satisfied, would result in an acceptably complete Pull Request. Be as specific as you need to be. For example:

[ ] At the end of each day, API should be pulled, and displayed stock prices should be changed.
[ ] This feature must be available for all users to view.
[ ] Add a test proving that it works

related issue in #56


Mail templates

Hi Sampoyigi!
The issue is still there. Mail templates can not be changed. Please try the following steps:

Login to your TastyIgniter site.
Go to "Design --> Mail Templates".
Click on the pencil of one preset template, e.g. last one "Contact form email to admin".
(Optional: make changes in the fiel "Markdown" - as I would like to do - translate some parts to German)
Click on the button "Save" or the the little arrow ...

... the site stucks completely. You can reload via F5 but no changes are saved.
Please let me know if you need further information. I will grant you access to my installation if that helps.
ps: I applied all updates available 2022-05-05.
Originally posted by @Graf-Buxtihude in https://github.com/tastyigniter/TastyIgniter/issues/919#issuecomment-1118289031

Yea access to your installation might help debug as i'm unable to reproduce. Direct message me on Discord

Fixed