Support Keyed Services in DryIoc.MS.DI for the Microsoft.Extensions.DependencyInjection V8

You probably have it on your radar, but if not then here is a reminder to look into expanding the support for using DryIoc as the container in applications using the Microsoft.Extensions.Hosting framework.
When .NET 8 is released, and the Microsoft.Extensions.Hosting package version 8 is released, Microsoft has finally added keyed service support. The bridge between DryIoc and IServiceCollection need to be added so that these can be used also when DryIoc is the underlying container.
The following example runs with the new hosting nuget package, but uncomment the DryIoc configuration and it throws an InvalidOperationException with the message This service descriptor is keyed. Your service provider may not support keyed services..
using ConsoleApp1;
using Microsoft.Extensions.DependencyInjection;
using Microsoft.Extensions.Hosting;

var builder = Host.CreateApplicationBuilder();
// builder.ConfigureContainer(new DryIocServiceProviderFactory(new Container()));
builder.Services.AddKeyedTransient<ITest, Test1>("T1");
builder.Services.AddKeyedTransient<ITest, Test2>("T2");

var host = builder.Build();
host.Services.GetKeyedService<ITest>("T1")?.Execute();
host.Services.GetKeyedService<ITest>("T2")?.Execute();

interface and classes for complete code
namespace ConsoleApp1;

public interface ITest
{
    void Execute();
}

public class Test1 : ITest
{
    public void Execute() => Console.WriteLine("Test1.Execute");
}

public class Test2 : ITest
{
    public void Execute() => Console.WriteLine("Test2.Execute");
}

@lassevk Let's keep this issue open until the IKeyedServiceProvider is implemented.

Interface https://learn.microsoft.com/en-us/dotnet/api/microsoft.extensions.dependencyinjection.ikeyedserviceprovider?view=dotnet-plat-ext-8.0
dotnet github issue https://github.com/dotnet/runtime/issues/64427

Let's address a challenge of the not-uniqueness of the MS.DI keyed services comparing to the uniqueness of DryIoc keyed services.
Solution 1
Create the composite key for the second, third, etc. MS key adding its index into the pair, e.g. "foo", ("foo", 1), ("foo", 2).
Pros:

For the majority of the cases when user hopefully will use a single key, there will be a parity and backward compatibility between MS and DryIoc.
Cons:
What happens when a User is resolving the single key in the presence of many? In this approach, we will resolve the first registered service. Or we need to adjust the SelectLastFactory rule to search for the duplicate keys.
When resolving the collection of the keyed services by default, DryIoc will return a single service, because the key is unique. It is not clear how to solve this problem

Solution 2
Represent the MS keys as DryIoc metadata
Pros:

DryIoc metadata is not unique, so multiple services with the same metadata are possible
SelectLastFactory need to be adjusted to filter based on metadata
Collection wrapper will return multiple filtered services based on the metadata

Cons:

Discrepancy between two features of MS and DryIoc, there should be a differently named APIs to resolve the services from the    MS and from the DryIoc. The documentation should clearly state this fact.
DryIoc may have the keys in addition to the metadata, which probably will add confusion.

v8 vs v7 API diff https://www.fuget.org/packages/Microsoft.Extensions.DependencyInjection.Abstractions/8.0.0/lib/netstandard2.1/diff/7.0.0/

Problems to solve
The multiple same key registrations
DryIoc treat the service key as unique id for the same service type. So it prevents you with registering the same service type with the same service key.
ServiceDecriptor.KeyedImplementationFactory has the key parameter
The type of KeyedImplementationFactory is Func<IServiceProvider, object?, object?> where the object? parameter is the key. Specifically, it is not the registration service key but the resolution/injection key, which may differ (e.g. it may be the AnyKey described below).
Problem is that DryIoc does not supply a service key into the FactoryDelegate which type is Func<IResolverContext, object>.
KeyedService.AnyKey
We need to support the KeyedService.AnyKey to resolve the service with any (not null) service key specified.
Therefore, I will be adding Registrator.AnyServiceKey on the DryIoc side.
So far, so good. But what is this thing?      See the next section for the reveal :-P
        [Fact]
        public void ResolveKeyedServiceSingletonFactoryWithAnyKeyIgnoreWrongType()
        {
            var serviceCollection = new ServiceCollection();
            serviceCollection.AddKeyedTransient<IService, ServiceWithIntKey>(KeyedService.AnyKey);

            var provider = CreateServiceProvider(serviceCollection);

            Assert.Null(provider.GetService<IService>());
            Assert.NotNull(provider.GetKeyedService<IService>(87));
            Assert.ThrowsAny<InvalidOperationException>(() => provider.GetKeyedService<IService>(new object()));
        }

ServiceKeyAttribute
Marks the parameter to be injected with the resolution service key as in the following case used by the test above:
        internal class ServiceWithIntKey : IService
        {
            private readonly int _id;

            public ServiceWithIntKey([ServiceKey] int id) => _id = id;
        }

Seems like we are doing the same Key for the same service Type in DryIoc.MefAttributedModel.
For this purpose, MEF has ServiceKeyStore stores the map from the Key to the all types registered (exported) with this key. And if Type is the same, it increases the number of such registrations and uses this number to augment the service Key and make it unique.
Question, can we use the MEF for MS.DI?...
Probably no, but we may still the ServiceKeyStore.
Btw, should we still the collection decorator too?
Btw, how to get all non-keyed services in the collection?
//cc @yallie

Ok, for the last 2 failing tests we need the #618

@dadhi, sorry, I'm on vacation, have no PC to look into it. Will return next Monday.
Moving the required code from MEF to DtryIoc core seems like a good idea 👍

@yallie Enjoy vacation, I have moved the thing already and optimized it along the way.

done


made the gear responsive when switching pages

This is when it's on the main page (/)

And this is on the settings page (/settings)

It indicates to the user that he is on the settings page


Support level-3 headings in CHANGELOG.md

Hello and thanks for this action which I implemented in many of my codes.
I have currently a problem with this changelog: https://github.com/SeismicSource/sourcespec/blob/master/CHANGELOG.md
The automatic release only picks up the text under the level-2 heading (ex.: ## v1.6 - 2022-08-02), but does not pick up the lines below level-3 headings (ex.: ### Input/output).
As a workaround, I added those lines manually, after the release was automatically created.
Would it be possible to support level-3 headings (and maybe any deeper level)?
Thanks!

Thanks!


Does RT utilize any add-on of react-addons?

Just curious, if RT components (which ones if any?) use any add-on of react-addons.
I have these add-ons in my node_modules:
react "version": "0.14.7"
react-addons-create-fragment "0.14.7"
react-addons-css-transition-group "0.14.7"
react-addons-pure-render-mixin "0.14.7"
react-addons-transition-group "0.14.7"
react-addons-update "0.14.7"
And in my webpack config I make vendors bundle providing both react and react-addons:
[
                'react', <===<
                'react-dom',
                'moment',
                'jquery',
                'react/addons', <===<
                'react-toolbox/lib/dropdown',
                'react-toolbox/lib/button',
            ]

And I get warning thrown in a browser console if I provide 'react/addons':
Warning: require('react/addons') is deprecated. Access using require('react-addons-{addon}') instead.
And if I remove  'react/addons' from vendor bundle (only react left), no warnings are throw in a browser and my app seems to work fine without react/addons.
My question. Do RT utilize any add-on of react-addons? If it does, which components do which add-ons utilize? Is it safe to completely remove react-addons from vendor bundle?

@mping

The only addon RT depends upon is react-addons-css-transition-group

Exactly! Thanks for answering @felipeleusin


wootility: 4.5.0 -> 4.6.15

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/wootility/versions.
meta.description for wootility is: A customization and management software for Wooting keyboards
meta.homepage for wootility is: https://wooting.io/wootility
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.6.14 in filename of file in /nix/store/2pa247qf826lr2br3dn74908k3d2i9wn-wootility-4.6.14

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
wootility

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/2pa247qf826lr2br3dn74908k3d2i9wn-wootility-4.6.14 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A wootility https://github.com/r-ryantm/nixpkgs/archive/8f747253a09895fd4e5c2305f70cbae402574d92.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/8f747253a09895fd4e5c2305f70cbae402574d92#wootility

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/2pa247qf826lr2br3dn74908k3d2i9wn-wootility-4.6.14
ls -la /nix/store/2pa247qf826lr2br3dn74908k3d2i9wn-wootility-4.6.14/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

wootility

Maintainer pings
cc @davidtwco for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/wootility/versions.
meta.description for wootility is: A customization and management software for Wooting keyboards
meta.homepage for wootility is: https://wooting.io/wootility
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.6.15 in filename of file in /nix/store/5grkiyr34s79cp4xbisq2gazv07ca3i8-wootility-4.6.15

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
wootility

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5grkiyr34s79cp4xbisq2gazv07ca3i8-wootility-4.6.15 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A wootility https://github.com/r-ryantm/nixpkgs/archive/06b0c7aeeef0ee7d283c7a60212f70fd762f22f2.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/06b0c7aeeef0ee7d283c7a60212f70fd762f22f2#wootility

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5grkiyr34s79cp4xbisq2gazv07ca3i8-wootility-4.6.15
ls -la /nix/store/5grkiyr34s79cp4xbisq2gazv07ca3i8-wootility-4.6.15/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

wootility

Maintainer pings
cc @davidtwco for testing.


Add ReactiveComponent without children in base Components

The demo should also be updated to use 3.0.0-rc.0 release.


Fix migrating Postgres fields with max character varchar custom type

Right now if we define in postgres field with max character varchar custom type
field.String("name", json.RawMessage{}).
			SchemaType(map[string]string{
				dialect.Postgres: "varchar(100)",
			}),

then first migration works as expected.
However the second time, migration operates and column already exists in the database, the migration fails with the following error:
sql/schema: creating changeset for \"my_table\": changing column type for \"name\" is invalid (varchar(100) != varchar):

This PR fixes the issue

@all-contributors please add @naormatania for code.


add caching management

retry guhundry pull request  #425 with latest version of fastimage

@DylanVann could you please take a look at this PR? Clearing cache can be quite beneficial 😬


Discard unnecessary commands when close workflow

What was changed
Discard unnecessary commands when close workflow
Why?
User code in defer function in workflow code could generate unwanted commands when workflow is closed. Those commands would not be handled anyway but they might cause validation error on server that result in workflow task to fail.
Checklist

Closes

How was this tested:

Any docs updates needed?

I'm not sure I love this. This needs to be implemented in every SDK and seems like something that can easily be handled server side at little cost. If we're just silently ignoring them here, why not just silently ignore them server side?
Why bother with this at all?


[Bug] [Rest-API] /hazelcast/rest/maps/running-jobs very slowly When there are more than 50 CDC tasks running

Search before asking

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

What happened
When there are more than 50
CDC tasks running, using the/hazelcast/rest/maps/running jobs rest interface to query the number of running tasks results in a very slow response, requiring more than one minute

SeaTunnel Version
2.3.5
SeaTunnel Config
none

Running Command
none

Error Exception
none

Zeta or Flink or Spark Version
No response
Java or Scala Version
No response
Screenshots
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

@Hisoka-X @liugddx help cc，/hazelcast/rest/maps/finished-jobs There is also a similar problem, but it is not as obvious

Have you checked jvm heap usage?

Have you checked jvm heap usage?

After investigation, it was found that the first few minutes after rolling restart would be very slow. It should be loading the corresponding class jar package, and then it became faster. However, during capacity testing, it was found that when running around 200 CDC tasks, querying the running tasks is not very fast, taking an average of 5-8 seconds, occasionally 20 seconds


WIP: cancle-pipeline-cancle-running-steps-in-time

see it as inspiration, else it's a personal "note" pr

Wait already does that


Kubernetes taints are not seen on the nodes in the UI

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

Deploy a cluster - 1 etcd, 1 control plane and 1 worker node.
Verify the etcd and control plane have kubernetes taints.

Expected Result:
kubernetes taints should be seen on the nodes in the UI.
Actual Result:
kubernetes taints are not seen on the UI on the nodes even though on an Edit node --> Taints section has them

Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): master-head - latest UI
Installation option (single install/HA): Single

Not Kubernetes taints, they are Kubernetes role taints. So you still can see kubernetes node not-ready taints in UI.
I would like to hide Kubernetes role taints in UI.
The reason is following:

To keep the UX the same with 2.2.x for users who don't use taints, users cannot see the very long  Kubernetes role taints in UI
We already have a Unschedule label and Roles column to indicate that if they are etcd/control plane and they are not unschedule.

Talked to Vince, we will remove the yellow Unschedule tag and just show the taints.

verified on master-head latest UI tag.
The Kubernetes role taints are seen on the UI and the yellow Unschedule tag is not seen anymore.


TableView Scrolling Inconsistency

When scrolling "down" through a TableView, instead of "smoothly" continuing down the list the view instead "jumps" several rows; the other rows are no longer visible unless you manually "scroll up".

As shown in the GIF above, once I scroll past line 10 on the screen the view "jumps" and displays lines 11-13; I can no longer see the other lines unless I "scroll up". I expected the view to "smoothly" scroll, showing lines 2-11, 3-12, 4-13 (etc) as I continue to "scroll down".
Scrolling up does seem to work "smoothly" instead of "jumping" the way scrolling down does.

Hi @rmorales87atx  thanks for bringing this up.  Please can you check out the fix in https://github.com/migueldeicaza/gui.cs/pull/1627 to make sure it is now behaving as you would expect.
I've fixed vertical scrolling down to only expose the minimum new rows for the newly selected row to be visible instead of jumping down to it.  Note that if AlwaysShowHeaders is false then scrolling down off the first page will still reveal multiple new rows since the header is no longer rendered and that space is instead filled with rows.
I've also added SmoothHorizontalScrolling which defaults to true.  It functions the same way for scrolling right.  Because TableView uses dynamic column sizes (based on content) not all columns are of equal width.  This means sometimes multiple new columns will be revealed scrolling right (e.g. when the column pushed off the left due to the scroll is very wide).

I just started to notice the horizontal scrolling, so I'm glad you managed to catch that.
I'll check out #1627 soon and report back, hopefully within the week. Thank you for the prompt attention.

Hi, I just pulled and tested your code with the fixed TableView. Your changes to vertical & horizontal scrolling work perfectly.


Refactor vectorization and preloading for pointwise fusions

This improves performance. For add_gelu in bert its 4x faster and for mul_add its 50% faster than what is on develop:
// This PR
gpu::code_object[code_object=13736,symbol_name=kernel,global=14155776,local=1024,]: 0.243313ms
// Develop
gpu::code_object[code_object=13736,symbol_name=kernel,global=153600,local=1024,]: 0.380066ms
gpu::mul_add: 0.475723ms

// This PR
gpu::code_object[code_object=14176,symbol_name=kernel,global=9437184,local=1024,]: 0.201901ms
// Develop
gpu::code_object[code_object=14016,symbol_name=kernel,global=153600,local=1024,]: 0.89556ms
gpu::add_gelu: 3.04435ms

This matches the performance on branch_for_ort2 for these same operators.

This change does improve performance.  Closer to what is seen in branch_for_ort2.


fluffychat: 1.11.2 -> 1.12.1

Also modify hash -> sha256 to make upgrading easier (able to use lib.fakeSha256 placeholder)
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Why not lib.fakeHash?


Fix non standard compliant usage of VLA

Variable length arrays are non-standard in C++ and don't compile in VS.

Sorry, @BMBurstein, I hadn't noticed this PR before - and have since addressed the issue (although need to go further). See https://github.com/philsquared/Clara/commit/7d5daa12f7d271b5d771269e5a6b1ed29a241664 (and the discussion there) for more.


paypal via braintree --or pay using label is displayed even when wallet payment buttons are not present on the checkout page

steps to reproduce

Bank of america connector>> Remove billing address >>>Allconnector123

PayPal via Braintree, PayPal button is not rendering.
Blocked on Backend to verify whether Braintree PayPal is currently supported

Duplicate issue. Already closed. As this is not even an issue as PayPal via braintree is button is not rendering but backend is sending as AllConnector123 is an old account so it will only happen in that case only.


Add protocol support for lists in headers

This updates the rest-xml and rest-json protocol serializers/parsers to support lists mapped to a header.

Codecov Report

Merging #2566 (f6adedd) into develop (2a7d0af) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop    #2566   +/-   ##
========================================
  Coverage    95.34%   95.34%
========================================
  Files           59       59
  Lines        11610    11618    +8
========================================
+ Hits         11069    11077    +8
  Misses         541      541

Impacted Files
Coverage Δ

botocore/parsers.py
99.45% <100.00%> (+<0.01%)
:arrow_up:

botocore/serialize.py
100.00% <100.00%> (ø)

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 2a7d0af...f6adedd. Read the comment docs.


Reshape error in XGBoost explainer.shap_values()

This is related to #580 where people experienced a reshape error when passing a matrix with more or less features to the explainer.shap_values than the number of features used to train a tree model.
I present the same bug, but I am passing a matrix with the correct number of features:
# Quick test that everything works ok:
dtest = xgb.DMatrix(features)
model.predict(dtest)
print('features are ok!')
print(features.shape)

# Shap implementation of model
explainer = shap.TreeExplainer(model)

X = shap.sample(features, 100)
print( type(X), X.shape )

model.predict(xgb.DMatrix(X))
shap_values = explainer.shap_values(xgb.DMatrix(X))

When I run it I get this:
features are ok!
(544401, 92)
<class 'pandas.core.frame.DataFrame'> (100, 92)
Traceback (most recent call last):
  File "/home/vladimir/.local/share/virtualenvs/ICFES-SocioEconomico-eS63OEPi/lib/python3.8/site-packages/shap/explainers/_tree.py", line 280, in shap_values
    phi = self.model.original_model.predict(
  File "/home/vladimir/.local/share/virtualenvs/ICFES-SocioEconomico-eS63OEPi/lib/python3.8/site-packages/xgboost/core.py", line 1397, in predict
    preds = preds.reshape(nrow, ngroup, data.num_col() + 1)
ValueError: cannot reshape array of size 23400 into shape (100,2,93)

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "shapsummaryplots.py", line 122, in <module>
    shap_summary_plots(2014, 2)
  File "shapsummaryplots.py", line 97, in shap_summary_plots
    shap_values = explainer.shap_values(xgb.DMatrix(X))
  File "/home/vladimir/.local/share/virtualenvs/ICFES-SocioEconomico-eS63OEPi/lib/python3.8/site-packages/shap/explainers/_tree.py", line 285, in shap_values
    raise ValueError("This reshape error is often caused by passing a bad data matrix to SHAP. " \
ValueError: This reshape error is often caused by passing a bad data matrix to SHAP. See https://github.com/slundberg/shap/issues/580

so out of nowhere it seems that SHAP is passing new data to XGBoost? (ValueError: cannot reshape array of size 23400 into shape (100,2,93))
Definitely I'm passing the correct number of columns, no matter if I sample or not the features dataframe, as model.predict(xgb.DMatrix(X)) works ok, but then when getting the shap values with the explainer it fails!
My shap version is 0.37 and my xgboost version is 1.2.1
Originally posted by @VolodyaCO in https://github.com/slundberg/shap/issues/580#issuecomment-745382015

I have the same error and need help with this!

Hi there !
I come late to the battle but finally got the solution to the issue.

explainer = shap.TreeExplainer(model)
shap_values = explainer.shap_values(X, y=y.values)

This worked for me. And be careful that your model matches with your X_train, in term of number of columns.
Hope this helps !
Cheers

Hi there !
I come late to the battle but finally got the solution to the issue.
explainer = shap.TreeExplainer(model)
shap_values = explainer.shap_values(X, y=y.values)

This worked for me. And be careful that your model matches with your X_train, in term of number of columns.
Hope this helps !
Cheers

This does not help me actually.. the columns are the same number as trained and being passed in the SHAP.

Hi there !
I come late to the battle but finally got the solution to the issue.
explainer = shap.TreeExplainer(model)
shap_values = explainer.shap_values(X, y=y.values)

This worked for me. And be careful that your model matches with your X_train, in term of number of columns.
Hope this helps ! Cheers

sorry for ask when so long have been
when i type
"explainer = shap.TreeExplainer(model)
shap_values = explainer.shap_values(X_test)"
it will report the error like this discuss, but when i try
"explainer = shap.TreeExplainer(model,X_test)
shap_values = explainer.shap_values(X_test)"
it work! it looks like different with the most answer ,may i ask why it work?and how this function work?or where i can find the guidebook about this function?

guys i find sometime maybe useful https://shap.readthedocs.io/en/latest/example_notebooks/api_examples/explainers/GPUTree.html


Update ClearSiteData default directives

Drop executionContexts from defaults
Drop cache from defaults, causes long logout times in Chrome

References:
https://github.com/nextcloud/server/issues/9179
https://bugs.chromium.org/p/chromium/issues/detail?id=898503

+1 works if we remove CLEAR_SITE_DATA_DIRECTIVES from our django_settings. Otherwise will use the settings' value and not use these changes.


Everything in console is missing

Issue Type: Bug
Hello, I think I have found a bug. I installed latest stable version of VS Code, found on official page. I tried to use console, but it's not possible. Everything is missing, except of my input.
VS Code version: Code 1.30.1 (dea8705087adb1b5e5ae1d9123278e178656186a, 2018-12-18T18:12:07.165Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6500U CPU @ 2.50GHz (4 x 2592)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
7.87GB (3.73GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

material-icon-theme
PKi
3.6.2

By console do you mean Terminal? https://code.visualstudio.com/docs/editor/integrated-terminal
Duplicate of #57803 ?


gitstatus: fix zsh integration

Motivation for this change
At some point the upstream changed and gitstatus.prompt.zsh stopped working. The issue was that this file expects to be able to run the "install" script in the same directory.

Install the "install" script and "build.info" file that gitstatus.prompt.zsh expects to be able to find when it runs.
Patch the install script so that it defaults to not automatically installing gitstatus. This can still be overridden by setting GITSTATUS_AUTO_INSTALL=1, although I'm not sure why anyone would do this if they've already installed gitstatus with Nix.
Add an install check phase that tests the zsh integration to prevent this derivation from breaking in the same way in the future. (This also ends up testing the binary itself.)

Fixes #96135.
Things done

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

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

cc @mmlb too—I can’t seem to add you to the reviewers list for whatever reason.

@GrahamcOfBorg build gitAndTools.gitstatus

@GrahamcOfBorg build gitAndTools.gitstatus

Tested on NixOS and this indeed fixes the zsh integration.

cc @mmlb too—I can’t seem to add you to the reviewers list for whatever reason.

Can only add org members afair.

Yep, I'm not part of the org (not sure how to get into it actually...).
@bdesham I don't use the zsh integration so am not really set up to test this. The diff looks good to me, the test makes sense too. That along with @ryneeverett's input I'd give this a :shipit:.
@mweinelt can you hit the merge button?


Crash: 1.1.3 NSBundle

NSBundle *bundle = [NSBundle bundleForClass:self.class];
NSURL *url = [bundle URLForResource:@"SVProgressHUD" withExtension:@"bundle"];
NSBundle *imageBundle = [NSBundle bundleWithURL:url];

this code block is in "SVIndefiniteAnimatedView.m" lines 60~62  and "SVProgressHUD.m" lines 265~267.
url == nil  ,  and carsh in   [NSBundle bundleWithURL:nil];
in SVProgressHUD 1.1.3 ,  Xcode 7.2 , cocoaPods 0.39.0

Have you run pod install? Did you tried the latest 2.0?

maybe i find the reason that i update the mac to 10.11.2 and xcode to 7.2 ..  and i run pod install when the xcode is updating... 3Q


MSDKUI-2285: Update dependencies to 3.16.3

Signed-off-by: Krzysztof Hubaczek<EMAIL_ADDRESS>

Codecov Report

Merging #185 into master will decrease coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #185      +/-   ##
==========================================
- Coverage   87.41%   87.27%   -0.14%
==========================================
  Files          89       59      -30
  Lines        5155     3199    -1956
==========================================
- Hits         4506     2792    -1714
+ Misses        649      407     -242

Flag
Coverage Δ

#ios
87.27% <ø> (-0.14%)
:arrow_down:

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

Impacted Files
Coverage Δ

MSDKUI_Demo/RouteViewController.swift

MSDKUI_Demo/LandingViewController.swift

MSDKUI_Demo/RouteOverviewViewController.swift

MSDKUI_Demo/MapRouteHandler.swift

MSDKUI_Demo/ExtensionString.swift

MSDKUI_Demo/GuidancePresentingViewController.swift

MSDKUI_Demo/OptionsViewController.swift

MSDKUI_Demo/ExtensionBundle.swift

...UI_Demo/GuidanceDashboardTableViewDataSource.swift

MSDKUI_Demo/ExtensionNMAGeocoder.swift

... and 20 more

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 1bf3cf6...e07451f. Read the comment docs.


Cleanup: Removed HAVE_CXX11 Issue #4284

Cleanup:
Removed ifdef HAVE_CXX11 and if HAVE_CXX11
Issue #4284

thnx for the patch! let's see what CI says and then i merge it


In Python3 range does not return a list

Now Pygam can be run on Python3(tested on Debian 9 with Python 3.5) without any problem;
It would return:
...
TypeError: 'range' object does not support item assignment
before.

@habedi lovely :)
could you write a quick test that proves that this bug is fixed?


Checkerboard BG

Helper sprite class to create checkerboard sprites on the spot
It also allows for two color channels to change, which removes some restrictions

https://github.com/Sword352/FNF-EternalEngine/assets/71162374/8bb2940e-4bb6-46af-8567-ba42bb9482f1

Thanks!


ENG-11083: Fix module references and bump vulnerable packages

Description of the change

Description here

Type of change

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

Checklists
Development

[ ] Lint rules pass locally
[ ] The code changed/added as part of this pull request has been covered with tests
[ ] All tests related to the changed code pass in development

Code review

[ ] This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
[ ] Jira issue referenced in commit message and/or PR title

Testing

Description of testing

@Yowgf , please provide a precise description in the PR body so @deepak-cyral can review it.


Porcupine Issue:

Have you checked the docs and existing issues?

[X] I have read all of the relevant Picovoice Porcupine docs
[X] I have searched the existing issues for Porcupine

SDK
Python
Porcupine package version
lasest, installed by pip
Framework version
lasest, installed by pip
Platform
Raspberry Pi
OS/Browser version
Armbian 23.11.1 Bookworm \l
Describe the bug
Traceback (most recent call last):
File "/opt/VoiceAssistant/./main.py", line 121, in
assistant = VoiceAssistant()
^^^^^^^^^^^^^^^^
File "/opt/VoiceAssistant/./main.py", line 35, in init
self.wakeword = WakeWord(lang=os.getenv("LANGUAGE", "ru"))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/opt/VoiceAssistant/wakeword_pvporcupine.py", line 46, in init
recognizer = pvporcupine.create(
^^^^^^^^^^^^^^^^^^^
File "/opt/VoiceAssistant/.venv/lib/python3.11/site-packages/pvporcupine/_factory.py", line 69, in create
return Porcupine(
^^^^^^^^^^
File "/opt/VoiceAssistant/.venv/lib/python3.11/site-packages/pvporcupine/_porcupine.py", line 198, in init
raise self._PICOVOICE_STATUS_TO_EXCEPTION[status](
pvporcupine._porcupine.PorcupineActivationRefusedError: Initialization failed:
[0] Picovoice Error (code 00000136)
[1] Picovoice Error (code 00000136)
[2] Picovoice Error (code 00000136)
Steps To Reproduce
https://github.com/dzianisv/VoiceAssistant/blob/main/src/wakeword_pvporcupine.py#L46
Expected Behavior
👍

You're trying to use Porcupine on Armbian OS, but it's not officially supported. We only support the latest Raspberry Pi OS. Closing this issue, but feel free to reopen if you face the same problem on Raspberry Pi OS with our demos.


Files App needs to be update due to crash and bugs

Description
i just brought this to replace the normal default file explore app but this app needs more updates and improvements please Fix
Steps To Reproduce

when i open the progream some of my icones like my drives are not displaying
2 when i serach for files in mp4 or mp3 it didn't work

Requirements

Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.4037
Log File
debug.log

Do the pinned items display after restarting Windows?

sometimes yes other times no

We're currently tracking this issue in #15410. In the meantime, you can workaround the issue by turning off the setting to keep Files in the background.

Can you try turning off Files settings > Advanced > Leave app running in the background then close and reopen Files.
As for the drive unplugged notice are you having that only when trying to open the this pc folder? If so you can use home it shows the drives with more details and has the other widgets

Can you try turning off Files settings > Advanced > Leave app running in the background then close and reopen Files. As for the drive unplugged notice are you having that only when trying to open the this pc folder? If so you can use home it shows the drives with more details and has the other widgets

i did that but i still get the Drives error

How are you opening Files?

yea


Friday, Dec 20: "Unable to retrieve Customer Details At the Moment"

The error is intermittent. So even if it works once, your code later may fail.

Dear Ramkumar,
Could you please check now and if you still face any issue please do let us know?
     Thanks & Regards,
      Hiten Solanki,
      Breeze API Team,
      ICICI Securities

From: Ramkumar Ramaswamy @.>
Sent: Friday, December 20, 2024 8:40:11 AM
To: Idirect-Tech/Breeze-Python-SDK @.>
Cc: Subscribed @.***>
Subject: Re: [Idirect-Tech/Breeze-Python-SDK] Friday, Dec 20: "Unable to retrieve Customer Details At the Moment" (Issue #175)
CAUTION: This email originated from outside the organization! Do not click links, open attachments or reply, unless you recognize the sender's email address! Use 'Report suspicious Email' button in the toolbar to report Phishing mails. Beware!
The error is intermittent. So even if it works once, your code later may fail.
—
Reply to this email directly, view it on GitHubhttps://github.com/Idirect-Tech/Breeze-Python-SDK/issues/175#issuecomment-2556189347, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A3TQZ463CN5R2A6U33ONFBT2GODBHAVCNFSM6AAAAABT6H2N2WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJWGE4DSMZUG4.
You are receiving this because you are subscribed to this thread.Message ID: @.***>
Print this mail only if absolutely necessary. Save Paper. Save Trees." "The information contained in this e-mail and any attachments to this message are intended for the exclusive use of the intended recipient and may contain proprietary, confidential or legally privileged information. If you are not the intended recipient, please note that you are not authorised to disseminate, distribute or copy this e-mail or any parts of it or act upon/rely on the contents of this e-mail in any manner. Please notify the sender immediately by e-mail and destroy all copies of this e-mail and any attachments. Please also note that ICICI Bank or its subsidiaries and associated companies, (collectively "ICICI Group"), are unable to exercise control or ensure the integrity of/over the contents of the information contained in e-mail transmissions and that any views expressed in this e-mail are not endorsed by/binding on the ICICI Group unless the sender does so expressly with due authority of ICICI Group. Before opening any attachments please check them for viruses and defects and please note that ICICI Group accepts no liability or responsibility for any damage caused by any virus that may be transmitted by this email. Any unauthorized use of this email or any attachment(s) may be treated as infringement of right(s) including right of privacy under the applicable laws of India including Information and Technology Act, 2000. Thank you for your cooperation. Regulatory Disclosures are available on https://www.icicisecurities.com (for institutional business) and www.icicidirect.com (for retail business).

Looks fixed, thank you!
Sent from my handphone.
On Fri, Dec 20, 2024, 9:20 AM Idirect-Tech @.***> wrote:

Dear Ramkumar,
Could you please check now and if you still face any issue please do let
us know?
Thanks & Regards,
Hiten Solanki,
Breeze API Team,
ICICI Securities

From: Ramkumar Ramaswamy @.>
Sent: Friday, December 20, 2024 8:40:11 AM
To: Idirect-Tech/Breeze-Python-SDK @.>
Cc: Subscribed @.***>
Subject: Re: [Idirect-Tech/Breeze-Python-SDK] Friday, Dec 20: "Unable to
retrieve Customer Details At the Moment" (Issue #175)
CAUTION: This email originated from outside the organization! Do not click
links, open attachments or reply, unless you recognize the sender's email
address! Use 'Report suspicious Email' button in the toolbar to report
Phishing mails. Beware!
The error is intermittent. So even if it works once, your code later may
fail.
—
Reply to this email directly, view it on GitHub<
https://github.com/Idirect-Tech/Breeze-Python-SDK/issues/175#issuecomment-2556189347>,
or unsubscribe<
https://github.com/notifications/unsubscribe-auth/A3TQZ463CN5R2A6U33ONFBT2GODBHAVCNFSM6AAAAABT6H2N2WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJWGE4DSMZUG4>.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>
Print this mail only if absolutely necessary. Save Paper. Save Trees."
"The information contained in this e-mail and any attachments to this
message are intended for the exclusive use of the intended recipient and
may contain proprietary, confidential or legally privileged information. If
you are not the intended recipient, please note that you are not authorised
to disseminate, distribute or copy this e-mail or any parts of it or act
upon/rely on the contents of this e-mail in any manner. Please notify the
sender immediately by e-mail and destroy all copies of this e-mail and any
attachments. Please also note that ICICI Bank or its subsidiaries and
associated companies, (collectively "ICICI Group"), are unable to exercise
control or ensure the integrity of/over the contents of the information
contained in e-mail transmissions and that any views expressed in this
e-mail are not endorsed by/binding on the ICICI Group unless the sender
does so expressly with due authority of ICICI Group. Before opening any
attachments please check them for viruses and defects and please note that
ICICI Group accepts no liability or responsibility for any damage caused by
any virus that may be transmitted by this email. Any unauthorized use of
this email or any attachment(s) may be treated as infringement of right(s)
including right of privacy under the applicable laws of India including
Information and Technology Act, 2000. Thank you for your cooperation.
Regulatory Disclosures are available on https://www.icicisecurities.com
(for institutional business) and www.icicidirect.com (for retail
business).
—
Reply to this email directly, view it on GitHub
https://github.com/Idirect-Tech/Breeze-Python-SDK/issues/175#issuecomment-2556232064,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQI36HMQNNGMLTHJCF7VQT2GOH2BAVCNFSM6AAAAABT6H2N2WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJWGIZTEMBWGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


[Feature Request] add rotation and reading mode button in reader

from official preview app, they are very useful. maybe replace with chapter and webview button

I would really like for this to be as well. I have been settling in SY for a while, and I found it most comfortable in use. I would not like to move back to Preview just for this. I wish for it to be considered as well. The Rotation and Readin Mode look more needed for the toolbar than the Webreader and the Chapters as well. Thank you @jferdous70.

This will be covered by #273 , so it can be closed @jobobby04


Digital Signature fields are not visible on non-Web platforms

Digital Signature fields are not visible on Windows, with the same issues as on Android.
In Web platform, that uses pdf.js instead of pdfium, digital signature fields are visible.

0.4.1 introduces PdfViewerParams.enableRenderAnnotations to control the visibility of PDF annotations on pages.
The following fragment illustrates the typical usage:
PdfViewer.asset(
  'assets/hello2.pdf',
  displayParams: PdfViewerParams(
    enableRenderAnnotations: true,
  ),
),

I'm sorry. Just tried on Windows and Android but it doesn't work.
Try with the following signed pdf.
sample3.pdf

Ah, signatures are not annotations... OK, need more work for that :(

Yes, exactly. signatures are a special type of annotation.
Consider that AndroidPdfViewer, which uses pdfium too, works like a charm with signatures:
https://github.com/barteksc/AndroidPdfViewer

0.4.4 changed the parameter to control the annotation rendering mode:
PdfViewer.asset(
  'assets/hello2.pdf',
  displayParams: PdfViewerParams(
    annotationRenderingMode: PdfAnnotationRenderingMode.annotationAndForms,
  ),
),

Sorry to comment on an old issue but I can't get any form to render on linux. Any help on how to debug / provide a traceback would be appreciated @espresso3389 !


⚠ Bofrost.com has degraded performance

In 72b9181, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 617 ms

Resolved: Bofrost.com performance has improved in 42ac5b0 after 5 hours, 3 minutes.


Add Mongo tracing

Simple auto configuration overriding the MongoClient and creating a TracingMongoClient by using the opentracing-mongo-driver

@vmaleze could you please also add a test to classpath profile to ensure that mongo classes are not on classpath. Otherwise the PR looks good to me!

@vmaleze I mean add test to NoDepsTest.class

0.1.13 is on the way
@vmaleze thanks for the PR!


More robust memory mapping, CI and C/CXX FFI

This PR is mainly robustness and quality of life improvements. I plan to use this as a stable base so I can look at algorithmic improvements without worrying about breaking something.
Refactoring
Refactored some reusable parts from main.rs into lib.rs so they can be used by library users & integration tests.
Integration Tests
Currently the code did not test if the actual sorted data are correct, I wrote an integration test to make sure that both mmaped and sequential processing yield the same data that is further compared with an expected value.
Memory Mapping Corrections
I got my hands on a MacOS system and fixed the mapping code. The main difference is transparent huge pages are not supported. I didn't find a credible source on how to make huge page happen on MacOS and remote debugging on GitHub Actions or my friend is just not worth the investment, but at least now with Integration Tests I can be sure mmap works without huge pages.
The same problem exists on some Windows systems too (like Github Actions) so I removed them too for compatibility.
C/CXX FFI
People may want to write bindings to your library, Rust is unfortunately still not popular enough in Bioinformatics. If we have a C/C++ API it would be much easier for them to integrate. I wrote a test too but I didn't include it in the CI because I feel changes to the API wrappers are infrequent enough that doesn't justify the large time investment in making the CI work across all platforms. The integration test is based on CMake with C++. The following should do it:
cd ci/c_ffi
mkdir -p build && cd build
cmake -DCMAKE_BUILD_TYPE=Release "-GUnix Makefiles" ..
make check

For usage see the test file. It's quite simple most lines are for assertions and testing.
Header files for full API: C C++
CI
I wrote GitHub actions workflow file which does the following on each of [macos, windows, linux] * [nightly, stable] So later we don't need to get 3 different systems to make sure we didn't break anything:

Checks the code lints and compiles with a minimal set of features and with a complete set of features
Checks the C and C++ header files are up to date (linux only).
Runs the unit tests and integration tests.
Uploads the header files, gtfsort binary and shared object as artifact. I tested on linux the only dependency is libc and I can just download it from GitHub actions and run the binary or link the shared object. This means automated multi-platform binary release on tag is possible!

Then if the commit message includes [ci benchmark], a benchmark is ran on each of the three platforms, compared to origin/master. Results are automatically submitted as a commit comment. Currently benchmark on windows and macos doesn't work because the current branch HEAD has compatibility problems with GitHub actions. After this is merged this should get resolved. Here's a sample comment: https://github.com/eternal-flame-AD/faster-gtfsort/commit/57651c378bfebe245fc60d218f40d4fe75dc16d4#commitcomment-145005867
here is a sample job log

@eternal-flame-AD
Amazing work and big thanks for your time!

People may want to write bindings to your library, Rust is unfortunately still not popular enough in Bioinformatics. If we have a C/C++ API it would be much easier for them to integrate. I wrote a test too but I didn't include it in the CI because I feel changes to the API wrappers are infrequent enough that doesn't justify the large time investment in making the CI work across all platforms.

I haven't had the opportunity to do C externals, but thanks for this additional feature. Now that the algorithm core is exposed as lib, make sense to also port that to python (will push that after merging this PR).
Let me know if you have any additional ideas/projects!

Thanks for the merge! I will look at the algorithm probably later next week to see if I can improve. I think the timing is good enough for most researches but memory wise I would like to try if I can do more.
As with the FFI I am more familiar with R than Python. I might write an R library for this. It will be a separate repo because R packages need to be in their own repository.

I am working on my own project on GFF too, it's called GFidx which produces a compact GFF index file for fast lookups over the file system or the network. I will publish it when it's polished enough .
Here's a presentation with some preliminary results if you are interested: https://blob.yumechi.jp/pub/20246_BIOL5382/presentation.html

Thanks for the merge! I will look at the algorithm probably later next week to see if I can improve. I think the timing is good enough for most researches but memory wise I would like to try if I can do more.

That will be amazing!

As with the FFI I am more familiar with R than Python. I might write an R library for this. It will be a separate repo because R packages need to be in their own repository.

I just pushed the py port, nothing fancy, though. With respect to the R port, I have not had the opportunity to extern rust-to-R; however I have seen that this is possible from a single repo by installing remotely from gh (check this example: r-oxbow). Would be nice to have all ports in 1 repo but feel free to continue with your desire approach, just let me know if I can help or share the repo link.

I am working on my own project on GFF too (partly why I found your project is because I think it would be good to have this as a preprocessor for my project), it's called GFidx which produces a compact GFF index file for fast lookups over the file system or the network. I will publish it when it's polished enough .

Your project looks pretty cool! Glad to have contributed (in some indirect way). I also created a pack of converters bed-to-tgf/gff and viceversa, all in rust. I think that they can definitely be enhanced with your algorithm. Right now they work with a pretty similar approach (not as cool and efficient as yours): 1) build a tree-like structure based on chr -> gene -> attributes and 2) perform coordinate calculations on each leaf in parallel.
At the pure bioinfo level I am "trying" to work on a rust library for chain files: chaintools. That version is really outdated and have changed a lot. I am trying to perform operations on genome alignments and pack everything using arrow to serialize huge files in the most efficient way possible (and also pyarrow to port everything 0-copy to python) and still have not come with a more clever algorithm to liftover things or play with alignment gaps and other stuff.  Feel free to reach out if you are interested or have some ideas!
I will be updating the README and paper (which are the only thing missing right now) in the following days.
Again, thanks for your time and pretty cool ideas, really looking forward to contribute to your project!
Alejandro

Would be nice to have all ports in 1 repo but feel free to continue with your desire approach, just let me know if I can help or share the repo link.

Will look at that probably tomorrow, if you are planning on bumping to 1.0.0 could you hold on to it for now?

At the pure bioinfo level I am "trying" to work on a rust library for chain files: chaintools.

I will look at it! Probably won't have time to write the friendly and Python API but will look at how to pack, serialize, de-serialize and unpack it as efficiently as possible!

I will be updating the README and paper (which are the only thing missing right now) in the following days.

Thanks!


Enable uniswap tests

Pre-submit checklist:

Branch

[x] Tests are provided (if possible)
[x] Commit sequence broadly makes sense
[x] Key commits have useful messages
[x] Relevant tickets are mentioned in commit messages
[x] Formatting, materialized Nix files, PNG optimization, etc. are updated

PR

[x] Self-reviewed the diff
[x] Useful pull request description
[x] Reviewer requested

We need to figure out a way to get this test to pass.

This PR puts back some tests that were disabled for the wrong reason.
By enabling them again, we expect to see an error in the balancer pop-up again. If there are Hydra errors, then that's also something that may need to be fixed, because we only expect a contract model test to fail, nothing else.

This test should be fixed with #419.

Ok good, then we await that PR being merged before we try to figure out what's up with Hydra here.

@sjoerdvisscher this still doesn't appear to be fixed. After fixing some annoying stuff to do with predicting ada on UTxOs in the tests (which you basically can't do...) I still get "value contains less than min ada" failures in the model. The counterexample is:
Actions
 [tok1 := Start,
  tok2 := SetupTokens,
  tok4 := CreatePool (Wallet 4) tok2.D 1 tok2.B 1,
  tok31 := CreatePool (Wallet 2) tok2.B 1 tok2.A 1,
  AddLiquidity (Wallet 4) tok2.B 0 tok2.A 3,
  tok64 := CreatePool (Wallet 4) tok2.A 1 tok2.C 1,
  AddLiquidity (Wallet 4) tok2.B 0 tok2.A 5]
Expected funds of W[4] to change by
  SymValue {symValMap = fromList [(tok2.A,999991),(tok2.B,999999),(tok2.C,999999),(tok2.D,999999),(tok4.Liquidity,1),(tok31.Liquidity,2),(tok64.Liquidity,1)], actualValPart = Value (Map [])}
but they changed by
  SymValue {symValMap = fromList [(tok2.A,999996),(tok2.B,999999),(tok2.C,999999),(tok2.D,999999),(tok4.Liquidity,1),(tok31.Liquidity,1),(tok64.Liquidity,1)], actualValPart = Value (Map [])}
a discrepancy of
  SymValue {symValMap = fromList [(tok2.A,5),(tok31.Liquidity,-1)], actualValPart = Value (Map [])}

                        Tx 7d49887431d4152d99dbfcc432f281253ed8242b86f9c3257b213c99c6eee88f:
                          {inputs:
                             - 2303b12237e7d94259ae7cf04b5b21219b021f2cb74b5a10195a6f9b42a6fe60!2

                             - 2303b12237e7d94259ae7cf04b5b21219b021f2cb74b5a10195a6f9b42a6fe60!3

                             - 2303b12237e7d94259ae7cf04b5b21219b021f2cb74b5a10195a6f9b42a6fe60!4

                             - 2fbac364045f0fec8a078ee11b6473366ff0eb6ac89aa33d977dd4ca1faffe0f!0
                               <>
                          collateral inputs:
                            - 2303b12237e7d94259ae7cf04b5b21219b021f2cb74b5a10195a6f9b42a6fe60!3

                          outputs:
                            - Value (Map [(,Map [("",2000000)]),(376c89700370ad6a1d5f11bbd8e6d070b0011ca8e09b67bb7669ff4a,Map [("A",9),("B",1)]),(a6ce5f802f1dadf50dae91880ade7787b2f6280ab14c4f3d33a2c296,Map [("Pool State",1)])]) addressed to
                              ScriptCredential: 4f750774388c4ac004d607ab84a8ab2c81544fd420e1aeb43206fa95 (no staking credential)
                            - Value (Map [(,Map [("",2000000)]),(a6ce5f802f1dadf50dae91880ade7787b2f6280ab14c4f3d33a2c296,Map [(0x57eeed951727bdc5729a319fad803dead5b3b2ec3363d254b66a0fb342555a00,1)])]) addressed to
                              PubKeyCredential: 557d23c0a533b4d295ac2dc14b783a7efc293bc23ede88a6fefd203d (no staking credential)
                            - Value (Map [(,Map [("",99999999965229293)])]) addressed to
                              PubKeyCredential: 557d23c0a533b4d295ac2dc14b783a7efc293bc23ede88a6fefd203d (no staking credential)
                            - Value (Map [(,Map [("",2000000)]),(376c89700370ad6a1d5f11bbd8e6d070b0011ca8e09b67bb7669ff4a,Map [("A",999991),("B",999999),("C",999999),("D",999999)]),(a6ce5f802f1dadf50dae91880ade7787b2f6280ab14c4f3d33a2c296,Map [(0x57eeed9
51727bdc5729a319fad803dead5b3b2ec3363d254b66a0fb342555a00,1),(0x77c201893baaafb0cdec91702749a7370c5c98e9f8b8164505349d3e05ffb444,1),(0x7cb6373701c4ceb558ec4159d079f27d6b998544cf1ba27eb12ab7959ae06680,1)])]) addressed to
                              PubKeyCredential: 557d23c0a533b4d295ac2dc14b783a7efc293bc23ede88a6fefd203d (no staking credential)
                          mint: Value (Map [(a6ce5f802f1dadf50dae91880ade7787b2f6280ab14c4f3d33a2c296,Map [(0x57eeed951727bdc5729a319fad803dead5b3b2ec3363d254b66a0fb342555a00,1)])])
                          fee: Value (Map [(,Map [("",7424514)])])
                          mps:
                            MintingPolicy { <script> }
                          signatures:
                          validity range: Interval {ivFrom = LowerBound NegInf True, ivTo = UpperBound PosInf True}
                          data:
                            <<<<"7l\137p\ETXp\173j\GS_\DC1\187\216\230\208p\176\SOH\FS\168\224\155g\187vi\255J",
                            "B">>,
                            <<"7l\137p\ETXp\173j\GS_\DC1\187\216\230\208p\176\SOH\FS\168\224\155g\187vi\255J",
                            "A">>>,
                            <2>>
                            <<<<"7l\137p\ETXp\173j\GS_\DC1\187\216\230\208p\176\SOH\FS\168\224\155g\187vi\255J",
                            "B">>,
                            <<"7l\137p\ETXp\173j\GS_\DC1\187\216\230\208p\176\SOH\FS\168\224\155g\187vi\255J",
                            "A">>>,
                            <3>>}```

@koslambrou and @sjoerdvisscher any chance we can get eyes on this latest bug in the balancer?

One thing I noticed is that it's no longer prop_Uniswap that's failing but the "can create a liquidity pool and add liquidity" test. So it seems the changes in this PR fix theprop_Uniswap test but have broken the other one.

I'm still having it fail but that may be because of the negative testing that we are doing. I'll have a look at it tomorrow and we can see where we are with this.

One thing I noticed is that it's no longer prop_Uniswap that's failing but the "can create a liquidity pool and add liquidity" test. So it seems the changes in this PR fix theprop_Uniswap test but have broken the other one.

I'm still seeing prop_Uniswap fail locally with less than min ada problems.

I can reproduce the issue indeed!
Can I easily make a stand-alone testcase from a list of actions like
       [tok1 := Start,
        tok2 := SetupTokens,
        tok3 := CreatePool (Wallet 6) tok2.A 1 tok2.B 1,
        tok6 := CreatePool (Wallet 3) tok2.C 1 tok2.D 1,
        tok7 := CreatePool (Wallet 6) tok2.C 1 tok2.B 1,
        AddLiquidity (Wallet 6) tok2.D 0 tok2.C 3,
        AddLiquidity (Wallet 6) tok2.A 0 tok2.B 3]

The easiest way I think is to translate it to a dynamic logic test using
the DL monad.
Best regards
Maximilian Algehed
On Fri, 8 Jul 2022 at 16:28, Sjoerd Visscher @.***>
wrote:

I can reproduce the issue indeed!
Can I easily make a stand-alone testcase from a list of actions like
   [tok1 := Start,
    tok2 := SetupTokens,
    tok3 := CreatePool (Wallet 6) tok2.A 1 tok2.B 1,
    tok6 := CreatePool (Wallet 3) tok2.C 1 tok2.D 1,
    tok7 := CreatePool (Wallet 6) tok2.C 1 tok2.B 1,
    AddLiquidity (Wallet 6) tok2.D 0 tok2.C 3,
    AddLiquidity (Wallet 6) tok2.A 0 tok2.B 3]

—
Reply to this email directly, view it on GitHub
https://github.com/input-output-hk/plutus-apps/pull/453#issuecomment-1179053186,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBPPQLBB7RUWBEBM3QGLDLVTA3JXANCNFSM5VRHFHIA
.
You are receiving this because you authored the thread.Message ID:
@.***>

@MaximilianAlgehed Thanks, that seems like a good idea! I tried this:
newTest = do
  action Start
  action SetupTokens
  UniswapModel {_exchangeableTokens = toList -> [a,b,c,d]} <- getContractState
  action $ CreatePool w6 a 1 b 1
  action $ CreatePool w3 c 1 d 1
  action $ CreatePool w6 c 1 b 1
  action $ AddLiquidity w6 d 0 c 3
  action $ AddLiquidity w6 a 0 b 3

testProperty "newTest" $ withMaxSuccess 1 $ forAllDL newTest propRunActions_

but that fails already at the first CreatePool action with a validation script failure (phase2 CekEvaluationFailure). What am I doing wrong? Can I not extract the tokens this way?

@sjoerdvisscher that's surprising to me. I've set this up locally and I'm going to see if I can make sense of what's going on there.

Ah, I think I've already figured out what might be wrong: add (defaultCheckOptionsContractModel & allowBigTransactions) and use propRunActionsWithOptions the way we do in prop_Uniswap.

@sjoerdvisscher I've pushed what might be a solution (I haven't had time to compile it yet 😅 )

Sorry, I updated my comment but it seems you already read the old one!

FYI, in #600 I fixed the minAda issue, but the tests still don't pass (might be a new issue I introduced with #600).

@sjoerdvisscher thanks, I'll rebase on main and see what I can do to work this out. It's possible that this is also a separate issue. What failures are you seeing?

This appears to still be a less-than-min-ada issue. Weird.

#600 hasn't been merged yet.

Ah... Derp.
Best regards
Maximilian Algehed
On Mon, 18 Jul 2022 at 11:57, Sjoerd Visscher @.***>
wrote:

#600 https://github.com/input-output-hk/plutus-apps/pull/600 hasn't
been merged yet.
—
Reply to this email directly, view it on GitHub
https://github.com/input-output-hk/plutus-apps/pull/453#issuecomment-1187000080,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBPPQJAGXOPVM4CY2NH433VUUS7BANCNFSM5VRHFHIA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Both...
On Wed, 11 May 2022, 20:07 koslambrou, @.***> wrote:

We need to figure out a way to get this test to pass.
You mean the Hydra CI or the actual tests for the uniswap use case?
—
Reply to this email directly, view it on GitHub
https://github.com/input-output-hk/plutus-apps/pull/453#issuecomment-1124115807,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBPPQN2P657LH6TJYLSOYTVJPZN5ANCNFSM5VRHFHIA
.
You are receiving this because you authored the thread.Message ID:
@.***>

I was hoping this would work on the new main branch where we now only have ledger validation. But I can't get it to work.


Genomics workflow - link or no link

Difficult to tell which text is links vs. no links

this was a part of the homepage prototype and we are not using this.


[Bug] Using TypeScript or ESModules for test-runner.ts does not work

Describe the bug
When using Storybook v7 and trying the test-runner.ts examples inaccessibility-testing, you encounter one these issues:
Using Typescript
running npm run test-storybook with a .storybook/test-runner.ts fails
test-runner.ts

import { injectAxe, checkA11y } from 'axe-playwright';

import type { TestRunnerConfig } from '@storybook/test-runner';

/*
 * See https://storybook.js.org/docs/react/writing-tests/test-runner#test-hook-api-experimental
 * to learn more about the test-runner hooks API.
 */
const a11yConfig: TestRunnerConfig = {
  async preRender(page) {
    await injectAxe(page);
  },
  async postRender(page) {
    await checkA11y(page, '#storybook-root', {
      detailedReport: true,
      detailedReportOptions: {
        html: true,
      },
    });
  },
};

export default a11yConfig

SyntaxError: ...test-runner.ts: Unexpected token, expected "from" (5:12)

5 | import type { TestRunnerConfig } from '@storybook/test-runner'
                ^

Using ESModules
running npm run test-storybook with a .storybook/test-runner.js fails
test-runner.js
import { injectAxe, checkA11y } from 'axe-playwright';

/*
 * See https://storybook.js.org/docs/react/writing-tests/test-runner#test-hook-api-experimental
 * to learn more about the test-runner hooks API.
 */
const a11yConfig: TestRunnerConfig = {
  async preRender(page) {
    await injectAxe(page);
  },
  async postRender(page) {
    await checkA11y(page, '#storybook-root', {
      detailedReport: true,
      detailedReportOptions: {
        html: true,
      },
    });
  },
};

export default a11yConfig

Jest encountered an unexpected token

...

Fest failed to parse a file.

...

Details:

import { injectAxe, checkA11y } from 'axe-playwright'
^^^^^^

SyntaxError: Cannot use import statement outside of a module

Steps to reproduce the behavior

Setup a simple react/storybook project
Set up storybook with test runner
Add a .storybook/test-runner.ts file
Try one of the examples from accessibility-testing
run npm run test-storybook
See error

Expected behavior
Using one of the examples from accessibility-testing should work out of the box if you're running storybook v7
Environment

OS: mac
Node.js version: v18.14.2
NPM version: 9.5.0

I have the exact same issue. Just using Vue instead of React.

Same issue here as well!

Same issue, tried setting this up with React according to the docs on SB v7 and ended up here.

Same problem here

I tried using the latest release 0.11.0-future.1 and it seems like the issue does not happen there. Only with 0.10.0. Should automatically be fixed once it's released then!

Harkunwar

Tried the version you suggested but that just threw another error: TypeError: rawArgv2.replace is not a function

I tried using the latest release 0.11.0-future.1 and it seems like the issue does not happen there. Only with 0.10.0. Should automatically be fixed once it's released then! Probably resolved due to #250

This worked for me, I used '0.11.0-next.1' with SvelteKit + the Svelte CSF add-on. Thanks!

I tried using the latest release 0.11.0-future.1 and it seems like the issue does not happen there. Only with 0.10.0. Should automatically be fixed once it's released then! Probably resolved due to #250

Tried 0.11.0-future.1 version you suggested but that just threw another error: TypeError: rawArgv2.replace is not a function

Hi, did you resolve it? Just got the same error. Thanks!

Since 0.11.0 is released, the problem seems to be resolved, doesn't occur again.

Hey there everyone! It seems like this issue is solved in the latest version of the test-runner, so I'll be closing it. Thanks for your patience, and do let us know if this issue occurs again!

I'm still getting this error on 0.19.1 with vite + react

can you provide are repro? Thanks!


msal angular and where is the msal config object in the sample ?

Please follow the issue template below. Failure to do so will result in a delay in answering your question.
Library

[ ]<EMAIL_ADDRESS>or<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[√]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Description
Hi i am looking at the sample here for Angular 9
https://github.com/AzureAD/microsoft-authentication-library-for-js/tree/dev/samples/angular9-sample-app
My question though is where do we configured the tenant_id,the client_id etc ?
When using the msal for angular with pkce where do i specify the token type is code ? just like in the oidc equivalent.

Hi
sorry wading through the documentation i was able to locate where this information is present kindly ignore this.


Update default display for portfolio and fixed white line

Updated Portfolio to show no client selected for view yet! text when open application
Update edit command to show updated portfolio after each command
Update add command to show no client selected for view yet! text instead of updated portfolio after each add person because addPerson cannot get person list (failed testcases, hence removed)
Update delete command to show default values after each deletion
Fixed white line showing at bottom of portfolio

Updated UI:
(when just open application)

(when editing a person)

Todo:

[ ] Update UG

Codecov Report
Base: 65.76% // Head: 65.57% // Decreases project coverage by -0.19% :warning:

Coverage data is based on head (568f083) compared to base (31b85fe).
Patch coverage: 16.12% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #105      +/-   ##
============================================
- Coverage     65.76%   65.57%   -0.20%
- Complexity      627      628       +1
============================================
  Files           101      101
  Lines          2337     2347      +10
  Branches        289      291       +2
============================================
+ Hits           1537     1539       +2
- Misses          692      700       +8
  Partials        108      108

Impacted Files
Coverage Δ

src/main/java/seedu/address/ui/MainWindow.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/address/ui/PortfolioWindow.java
0.00% <0.00%> (ø)

.../java/seedu/address/logic/commands/AddCommand.java
92.30% <100.00%> (ø)

...va/seedu/address/logic/commands/DeleteCommand.java
97.56% <100.00%> (ø)

...java/seedu/address/logic/commands/EditCommand.java
95.61% <100.00%> (ø)

...va/seedu/address/logic/commands/CommandResult.java
80.76% <0.00%> (+7.69%)
:arrow_up:

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

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

Just need to resolve the merge conflicts and your good to merge! :thumbsup:


fix: 잘못 배포된 버전을 바로잡기 위해 CHANGELOG.md를 수동으로 수정합니다

Overview

fix #262
#254 에서 추가 해주신 두 가지 함수가 2.2.0이 아닌, 2.2.1버전에 배포가 나갔습니다.
PR Checklist

[x] I read and included theses actions below

I have read the Contributing Guide
I have written documents and tests, if needed.

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

Project coverage is 99.53%. Comparing base (c1b078f) to head (5939b05).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #273   +/-   ##
=======================================
  Coverage   99.53%   99.53%
=======================================
  Files          38       38
  Lines         639      639
  Branches      154      154
=======================================
  Hits          636      636
  Misses          3        3


Page Looks Nothing Like Security Portal

My Security and Compliance Page and these instructions do not match what I see in the portal at https://protection.office.com/
Is there an update?

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

ID: 8d3a8799-0915-2446-b772-cf5e3c80a238
Version Independent ID: 31f10a22-3c95-5ebe-622d-7431c5cb76f1
Content: Deploy Cloud App Security
Content Source: CloudAppSecurityDocs/getting-started-with-cloud-app-security.md
Service: cloud-app-security
GitHub Login: @shsagir
Microsoft Alias: shsagir

I think you followed to the wrong URL. Right URL: https://security.microsoft.com/homepage

@VNJoe Thanks for your feedback. @GiantCrocodile is correct regarding the URL to use. We've added the URL and updated the Microsoft 365 Admin Center image and submitted a documentation change based on your suggestion, and the updated article should be live by tomorrow. Please let us know if there's anything more we can do for you. To follow up on this issue, leave us a comment with an @mention.


[FEATURE] Core fields for dataframe

Is your feature request related to a problem? Please describe.
I'm always frustrated when I realise the source processors are only really set up to work with wirc, and nothing else.
Describe the solution you'd like
We should have a list of core fields, like images. prv_candidates should be under that banner. More generally, there is so much overly-specific stuff in the source/alert generation that should not be there. Maybe we need make_alert, like load_raw_image, which is pipeline-specific.

Partly fixed by #566


Codespaces CLI release v0.2.0

Update to 2020-05-26 API version.
Plan creation supports subnets.
New az codespace update command.
New az codespace plan update command.

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

For new extensions:

[n/a] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

Codespaces

@fengzhou-msft as the pending reviewer, could you review this and merge if ready?

@yonzhan @haroldrandom @fengzhou-msft ping on this.
It's been a week since the PR was first created. We'd like to have the new updates available to our users.

@fengzhou-msft will help you merge this PR.

Ping on this again.

Merged. Sorry for the delay.


Watching for ForeignKey value changes seems to not trigger the hook

Using the below example in version 0.6.0, I am expecting to print a message any time someone changes a user's first name that is saved in SomeModel. From what I can tell from the documentation, I am setting everything up correctly. Am I misunderstanding how this works?
Assuming a model set up like this:
from django.conf import settings
from django.db import models
from django_lifecycle import LifecycleModel, hook

class SomeModel(LifecycleModel):
    user = models.ForeignKey(
        on_delete=models.CASCADE,
        to=settings.AUTH_USER_MODEL
    )
    # More fields here...

    @hook('after_update', when='user.first_name', has_changed=True)
    def user_first_name_changed(self):
        print(
            f"User's first_name has changed from "
            f"{self.initial_value('user.first_name')} to {user.first_name}!"
        )

When we then perform the following code, nothing prints:
from django.contrib.auth import get_user_model

# Create a test user (Jane Doe)
get_user_model().objects.create_user(
    username='test',
    password=None,
    first_name='Jane',
    last_name='Doe'
)

# Create an instance
SomeModel.objects.create(user=user)

# Retrieve a new instance of the user
user = get_user_model().objects.get(username='test')

# Change the name (John Doe)
user.first_name = 'John'
user.save()

# Nothing prints from the hook

In the tests, I see that it is calling user_account._clear_watched_fk_model_cache() explicitly after changing the Organization.name. However, from looking at the code, I do not see this call anywhere except for the overridden UserAccount.save() method. Thus, saving the Organization has no way to notify the UserAccount that a change has been made, and therefore, the hook cannot possibly be fired. The only reason that I can see that the test is passing is because of the explicit call to user_account._clear_watched_fk_model_cache().
    def test_has_changed_is_true_if_fk_related_model_field_has_changed(self):
        org = Organization.objects.create(name="Dunder Mifflin")
        UserAccount.objects.create(**self.stub_data, organization=org)
        user_account = UserAccount.objects.get()

        org.name = "Dwight's Paper Empire"
        org.save()
        user_account._clear_watched_fk_model_cache()
        self.assertTrue(user_account.has_changed("organization.name"))

I appreciate the detailed issue. I'll look into this.

I think I know what's happening. You want a model's hook to fire, upon making updates to its FK-related model. This isn't supported. The hook will only fire when you make the change to the model itself: the SomeModel instance in your example.
from django.contrib.auth import get_user_model

# Create a test user (Jane Doe)
get_user_model().objects.create_user(
    username='test',
    password=None,
    first_name='Jane',
    last_name='Doe'
)

# Create an instance
some_instance = SomeModel.objects.create(user=user)

# Retrieve a new instance of the user
user = get_user_model().objects.get(username='test')

# Change the name (John Doe)
user.first_name = 'John'
user.save()

# The hook will fire and it will print the message
some_instance.save()

Maybe the docs should be updated to clarify? I'm a little nervous about supporting the behavior you expect -- this could trigger a lot of additional ORM queries to answer the question "do any related models have hooks that require state tracking?"

Yes, that is exactly what I was expecting. The documentation on ForeignKey Field Value Changes shows an example of a hook of this nature, but doesn't show the example required to trigger it.
We (at my company) are writing an event-driven system where saves on one model can impact a callback chain on another. I have to do something like "when a user changes their name, go to this other model and update this other thing." As a "workaround," we added some hooks on the user model to achieve this. Now that I understand what the limitations are, I'll make sure that our code handles ForeignKey changes this way.
I do have some ideas for this in the future if you are interested. After creating this issue and looking at the code, I was fairly certain that what I was trying to do was not supported, so I tried figuring out a way to add support for it. Would you be interested in me researching this?

... Though, to be honest, I'm not quite sure if the "FK changes" feature should be included in the first place because of lack of support for handling it efficiently, ORM-wise.

I believe that if it became supported, efficiency would be in the hands of the developer. A nice little warning should suffice, just as it is now in the existing documentation.

Apologies for the radio silence - I appreciate the thought here. I just don't think I want to expand the scope of this project to support a feature of this complexity, given that it seems to be a pretty rare use case. Furthermore, handling chains of changes like this with application code in-memory seems a little shaky... I think it would be better solved with database triggers/functions.


[Bug] Server reports failed to verify certificate

Verify steps

[X] Is this something you can debug and fix? Send a pull request! Bug fixes and documentation fixes are welcome.
[X] I have read the Wiki, especially the FAQ page.
[X] I have searched on the issue tracker for a related issue.

MetaTube Plugin Version
2024.512.1925.0
MetaTube Server Version
1.2.3 (9126becc99dd)
What OS are you seeing the problem on?
Linux
Description
Server reports certificate signed by unknown authority.
Note: The URL in the following log is (obviously) NSFW
{"error":{"code":500,"message":"Get \"https://gyutto.com/data/item_img/1980/198093/198093.jpg\": GET https://gyutto.com/data/item_img/1980/198093/198093.jpg giving up after 4 attempt(s): Get \"https://image.gyutto.com/data/item_img/1980/198093/198093.jpg\": tls: failed to verify certificate: x509: certificate signed by unknown authority"}}

However chrome believes that the certificate is valid.

Tried updating ca-certificates in the container, with no luck
MetaTube Server
  metatube:
    image: "metatube/metatube-server"
    container_name: "metatube"
    user: "1000:1000"
    restart: "unless-stopped"
    logging:
      options:
        max-size: "16m"
    ports: ["8200:8080"]
    command: "-dsn /config/metatube.db"
    volumes:
      - "./metatube:/config"
    environment:
      TZ: "Asia/Shanghai"

Jellyfin/Emby Logs
{"error":{"code":500,"message":"Get \"https://gyutto.com/data/item_img/1980/198093/198093.jpg\": GET https://gyutto.com/data/item_img/1980/198093/198093.jpg giving up after 4 attempt(s): Get \"https://image.gyutto.com/data/item_img/1980/198093/198093.jpg\": tls: failed to verify certificate: x509: certificate signed by unknown authority"}}

MetaTube Server Logs
[GORM] 2024/05/31 11:22:50 github.com/metatube-community/metatube-sdk-go/engine/movie.go:197
[0.340ms] [rows:1] SELECT * FROM `movie_metadata` WHERE provider = "AVBASE" AND id = "4004925" COLLATE NOCASE ORDER BY `movie_metadata`.`id` LIMIT 1
[GIN] 2024/05/31 - 11:23:01 | 500 | 11.238473426s |        <IP_ADDRESS> | GET      "/v1/images/backdrop/AVBASE/4004925?ratio=-1&pos=-1&auto=False&quality=90"

How to Reproduce
Try parse GETCHU-004925

Thanks for the feedback. Don't know if it's a temporary issue from the site or something else. will fix it later.


Fix regression in 2.9 involving unsafe Context.get_all() usage

Since commit d67f0fd4cc2a4af08f51f4466150d49da7798729, callers
of Context.get_all() need to make a copy it they're going to
modify the result.
Fixes: d67f0fd4cc2a ("Generalize scoping.  This fixes #603")
See: https://github.com/ansible/ansible/issues/20494
@mitsuhiko, PTAL

This is cherry-picked now into 2.9-maintenance.


DOC: Update the docs of InterpolatedUnivariateSpline and LSQUnivariateSpline. .

Reference issue

Closes #14098.
What does this implement/fix?

1 .  It removed double white space in class UnivariateSpline.
2.  It updates the parameter section in the docs of LSQUnivariateSpline(UnivariateSpline), InterpolatedUnivariateSpline(UnivariateSpline).
3. Correct the spacing for bbox.

LGTM, thanks @Anselmoo. Do you have other changes planned before I merge?

@tupui thx for the support! In terms of the class UnivariateSpline.
In general, there are still some inconsistencies like see below, because default values are not defined. This should maybe become better a new issue like DOC-check of module ??
https://github.com/scipy/scipy/blob/a6b0a01484f428f83790995b78487b73d706c198/scipy/interpolate/fitpack2.py#L326-L341

In general, there are still some inconsistencies like see below, because default values are not defined. This should maybe become better a new issue like DOC-check of module ??

As I said, do not create other issues for this, as it creates unnecessary work for all maintainers. You can address them here as it's the same thing. A PR can touch multiple file/folders.

In general, there are still some inconsistencies like see below, because default values are not defined. This should maybe become better a new issue like DOC-check of module ??

As I said, do not create other issues for this, as it creates unnecessary work for all maintainers. You can address them here as it's the same thing. A PR can touch multiple file/folders.

Ic, that differs from some other communities. Then I will go ahead and fix it. I don't saw it in the first place!!

@rgommers did you intentionally close the PR or is this a side effect of the renaming? I've seen a few PR/issues I am following getting closed at the same time.

See https://github.com/scipy/scipy/pull/9291#issuecomment-1004248815 for the explanation and follow-up.

This is a very weird GitHub bug though, because https://github.com/Anselmoo/scipy still exists!

I see thanks. Yes sounds like a bug otherwise all PRs should be gone basically.

Ah, the message when hovering over this branch here is wrong:

The repo still exists, but the branch is not present. Please revive the PR if that makes sense (see https://github.com/scipy/scipy/pull/9291#issuecomment-1004248815)

Hum if you filter the PRs by is:pr is:closed sort:updated-desc. There are quite a few... Well if that can help cleaning stalled PRs...

I checked a few - in some cases the forked repo was gone (e.g., gh-12307), in other cases the branch was gone (this one is odd, because I thought removing a branch would already have auto-closed the PR).

Do you think we should send a quick email to warn maintainers and link maybe to the list of recently closed PRs so we all can have a look and quickly assess if PRs can stay closed or open?

Ah, was already typing that email:) Sent before I saw this.

We had this happen on the NX repo, too.  All the PRs that got closed for us were due to the originating PR's repo or branch being deleted.


zephir_parser extension not loaded

I'm running on ubuntu 14.04 and I installed php5.6. I'm trying to do the Hello world with Zephir but when I compile my project with zephir build but I got this error :
zephir_parser extension not loaded, compiling it
Parser statistics: 127 terminals, 99 nonterminals, 459 rules
                   947 states, 0 parser table entries, 0 conflicts
Parser statistics: 127 terminals, 99 nonterminals, 459 rules
                   947 states, 0 parser table entries, 0 conflicts
make: *** No rule to make target `clean'.  Stop.
Preparing for parser compilation...
sh: 1: phpize: not found
sh: 1: ./configure: not found
Compiling the parser...
make: *** No targets specified and no makefile found.  Stop.

PHP Warning:  copy(modules/zephir_parser.so): failed to open stream: No such file or directory in /root/zephir/Library/Compiler.php on line 282
Zephir\Exception: The zephir parser extension could not be found or compiled!

I reinstalled these packages :
php5-json php5-dev libpcre3-dev
I tried this command line :
./bin/zephir compile
And while compiling I have another error :
Copying new kernel files...
PHP Fatal error:  Call to undefined function Zephir\utf8_decode() in /root/zephir/Library/Compiler.php on line 2202

Any idea ?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/37582109-zephir_parser-extension-not-loaded?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github).

Could you please provide output of:
$ php -r 'var_dump(function_exists("utf8_decode"));'

I solved this error with :
apt-get install php5.6-xml
But I don't know if it's the good way.

@Johngtrs Yes, it is right way. You have to install xml extension