All this CSRF auto-magic is nice but poses a problem when you cannot resolve where an error comes from

[EDIT by guardrex to change the article and update the Document ID]
Description
I added an OIDC provider as an authentication source to the new Blazor Web App. Redirecting to the login page works, logging in an redirecting back works as well.
Only then I get an error that the antiforgery-token is missing. That's nice. And for the life of me, I cannot figure out how to configure or disable this issue because you actively hid the token validator somewhere.
I'm talking about the included default "Blazor Web App" with "Individual Accounts" enabled. I have no chance to override the issue (or analyze anything really) because I cannot see where the problem is coming from. There are no visible "RequireAntiforgery" attributes anywhere.
Page URL
https://learn.microsoft.com/en-us/aspnet/core/blazor/security/?view=aspnetcore-8.0#antiforgery-support
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/blazor/security/index.md
Document ID
1247ee64-fbc9-ce29-0710-f6e59380f5ee
Article author
guardrex

Stand-by @Rhywden ... I'll get back to you shortly.

I'm BACK. 🏃♂
We do rely heavily on the main doc set for server-side (i.e., Blazor Web App) security guidance because this type of Blazor app relies on ASP.NET Core security/Identity API. However, we added a bit of guidance on antiforgery to the server-side Blazor security doc. Depending on how this is resolved, we'll probably end up at least covering it there. We may also cover something in the main doc set where you originally opened the issue. The current bit in the Blazor doc is at ...
https://learn.microsoft.com/en-us/aspnet/core/blazor/security/?view=aspnetcore-8.0#antiforgery-support
... which isn't helpful at the moment for the problem that you've run into. I've updated your opening comment to point to that document at least for the time being while we figure what's going on here.
Let's work the problem via the product unit's repo. Open an issue for the product unit at ...
https://github.com/dotnet/aspnetcore/issues
You'll need to add a bit more to your issue than you have here:

For "I get an error that the antiforgery-token is missing": Tell them exactly where are you getting the error and provide the exact error message (e.g., a stack trace is great).
If it's purely a template-created app with Individual Accounts, you won't need to provide a repro project. If you've added code, especially code that's throwing the error, then you'll probably need to put up a minimal repro project on GitHub for the product unit engineer to pull down and inspect/run.

Please add ...
cc: @guardrex https://github.com/dotnet/AspNetCore.Docs/issues/31082

... to the bottom of your opening comment so that I can follow along.
Leave this issue open. We'll use this issue for tracking the doc work. This issue will close automatically later when the docs PR is merged.

Well, I actually solved the issue by sheer blind luck - sorry if I was grumpy but I was fumbling around and there's a lot of not-helpful information out there. The issue seems to stem from the fact that any OpenID-server (like Keycloak) will do a POSTback (or similar) to hand over the access tokens (or at least I'm not seeing why else Blazor would complain about the missing antiforgery token).
The previous implementations I used in .NET7 didn't really care about that or where you pointed the Redirect URI to. It just worked. So when I copied that particular working code from a .NET7 Blazor application to the new .NET8 which seems to be more opinionated I ran into the issue I detailed above.
So, the previous (working) config for OpenID had, among other things, a config line like this:
RedirectUri = "https://foo/"

Now the whole thing needs to look like this:
builder.Services.AddAuthentication(options =>
    {
        options.DefaultScheme = IdentityConstants.ApplicationScheme;
        options.DefaultSignInScheme = IdentityConstants.ExternalScheme;
    }).AddOpenIdConnect(OpenIdConnectDefaults.AuthenticationScheme, opt =>
    {
        opt.Authority = "https://openid.foo/realms/master";
        opt.ClientId = "FooDev";
        opt.Scope.Add("openid");
        opt.Scope.Add("profile");
        opt.Scope.Add("roles");
        opt.Scope.Add("offline_access");
        opt.SaveTokens = true;
        opt.ResponseType = "code";

        opt.Events.OnRedirectToIdentityProvider = async n =>
        {
            n.ProtocolMessage.RedirectUri = "https://foo/signin-oidc";
            await Task.FromResult(0);
        };
    })
    .AddIdentityCookies();

The need for "https://foo/signin-oidc" instead of "https://foo/" is what threw me. Problem is that OIDC seems to be the readheaded stepchild in regards to documentation. I'd very much love it if you could expand the documentation to include a common OIDC configuration - because not everyone can use 3rd party auth if it resides on external servers (GDPR and such).

Thanks @Rhywden for the follow-up remarks. I'm glad you figured it out.
Currently, we only even have a vague mention of this OIDC API (OnRedirectToIdentityProvider) configuration in the MFA doc, which wouldn't be helpful for your issue. The only other place I readily found it was in some scattered Azure B2C docs coverage over in the MS Identity Platform docs. Again, none of that would've been helpful.

expand the documentation to include a common OIDC configuration

Yes, and that's what this issue is focused on ...
New doc: Configure OIDC web authentication
https://github.com/dotnet/AspNetCore.Docs/issues/30647
... and they're chatting about that right now over there. You might want to leave a brief comment over there about what happened here. I don't think you need to say much more than 'without the correct redirect URI provided to the IdP, the IdP's response got flagged 💥 by the Antiforgery Middleware' and cross-link to this issue. I assume that they will at least cover the API that you're referring to in a general way. If they decide to have a Troubleshoot section, then what happened here could be an entry in that section.
There are two other older issues that are loosely related because they ask for this type of coverage ...
Generic OpenIdConnect and OAuth concept docs
https://github.com/dotnet/AspNetCore.Docs/issues/7025
Add document explaining the authentication system
https://github.com/dotnet/AspNetCore.Docs/issues/4055
However, those last two aren't the best spot for a remark.
You can probably close this on the grounds that OIDC general docs are coming. I'll just reset the labels and title for historical purposes.

Hi @Rhywden and @guardrex
I came across the exactly same issue as you. Using Keycloak, upgrading from .NET 7 to .NET 8. Worked fine in .NET 7 but in .NET 8 I get reconnnected Keycloak, can login myself successfully and then get redirected to my application. There I get the following error message:
An unhandled exception occurred while processing the request.
InvalidOperationException: This form is being accessed with an invalid anti-forgery token. Validate the `IAntiforgeryValidationFeature` on the request before reading from the form.
Microsoft.AspNetCore.Http.Features.FormFeature.HandleUncheckedAntiforgeryValidationFeature()

AuthenticationFailureException: An error was encountered while handling the remote login.
Microsoft.AspNetCore.Authentication.RemoteAuthenticationHandler<TOptions>.HandleRequestAsync()

I copied your code:
builder.Services.AddAuthentication(options =>
    {
        options.DefaultScheme = IdentityConstants.ApplicationScheme;
        options.DefaultSignInScheme = IdentityConstants.ExternalScheme;
    }).AddOpenIdConnect(OpenIdConnectDefaults.AuthenticationScheme, opt =>
    {
        opt.Authority = "https://openid.foo/realms/master";
        opt.ClientId = "FooDev";
        opt.Scope.Add("openid");
        opt.Scope.Add("profile");
        opt.Scope.Add("roles");
        opt.Scope.Add("offline_access");
        opt.SaveTokens = true;
        opt.ResponseType = "code";

        opt.Events.OnRedirectToIdentityProvider = async n =>
        {
            n.ProtocolMessage.RedirectUri = "https://foo/signin-oidc";
            await Task.FromResult(0);
        };
    })
    .AddIdentityCookies();

but it is still not working. I replaced https://foo/signin-oidc with my local address https://localhost:7031/signin-oidc but nothing is changing.
As I searched through the comments the documentation is not finished yet. Could you help me solve the issue or do you have a source where I can get some further information.
Thank you very much.

The PR for the new doc is here ...
https://github.com/dotnet/AspNetCore.Docs/pull/31555
... but it isn't reviewed yet.
Open an issue on the product unit's repo for assistance ....
https://github.com/dotnet/aspnetcore/issues
Please add ...
cc: @guardrex https://github.com/dotnet/AspNetCore.Docs/pull/31555

... to the bottom of your opening comment so that I can follow along. I might add content to the PR based on their discussion with you about this.


GEODE-5271: Throw TransactionDataRebalancedException during commit wh…

…en bucket has been moved.
Co-authored-by: Ryan McMahon<EMAIL_ADDRESS>@dhemery  Please review.

We found that it the RegionDestroyedException shouldn't be thrown under normal operation during the commit, so handling it there is not the best solution.


Cannot create Digital Ocean Droplets

After generating the DO access token and copied into the conifg file, hidensneak says the key is invalid.
config/config.json
{
"aws_access_id": "XXXXXXXXXXXXXX",
"aws_secret_key": "XXXXXXXXXXXXXXXXXXXXXXXXX",
"aws_bucket_name": "XXXXXXX",
"digitalocean_token":"fba5251c93c381fbxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx",
"azure_tenant_id": "",
"azure_client_id": "",
"azure_client_secret": "",
"azure_location": "",
"azure_subscription_id": "",
"google_credentials_path": "",
"google_project": "",
"public_key":"/root/.ssh/id_rsa.pub",
"private_key":"/root/.ssh/id_rsa",
"do_user": "root",
"ec2_user": "ubuntu",
"google_user": "ubuntu"
}
./hidensneak instance deploy -c 1 -p DO --region-do nyc1
Creating Terraform Files...
Applying Terraform Changes...
Error: Error applying plan:
1 error(s) occurred:

module.doDropletDeploy1.module.do-nyc1.digitalocean_droplet.hideNsneak: 1 error(s) occurred:

digitalocean_droplet.hideNsneak: Error creating droplet: POST https://api.digitalocean.com/v2/droplets: 422 42:f8:d8:d4:ba:55:9e:1a:fb:9b:85:f7:9a:97:4b:fe are invalid key identifiers for Droplet creation.

Terraform does not automatically rollback in the face of errors.
Instead, your Terraform state file has been partially updated with
any resources that successfully completed. Please address the error
above and apply again to incrementally change your infrastructure.
Waiting for instances to initialize...
100% |████████████████████████████████████████|  [1m59s:0s]
Restricting Ports to only port 22...
[WARNING]: provided hosts list is empty, only localhost is available. Note
that the implicit localhost does not match 'all'
PLAY [install all packages] ****************************************************
skipping: no hosts matched
PLAY RECAP *********************************************************************

Found the solution for this. DO does not allow you to upload your own key via hidensneak. The ssh pub key defined in the config file must be manually added on digitalocean.com's console portal under Security -> Add SSH Keys.


Tool Slot Deletion Bug

What's New?

Fix a bug where, under certain circumstances, deleting a tool slot would cause a 500 error.
Add Devices::Dump
Remove rollbar debug point.

Coverage remained the same at 81.144% when pulling 221edfac630e630fa1782c288ab7a7cac2d7a771 on RickCarlino:toolslot_delete_bug into 64762082cb0185e62023a6d1179a5760ec9d6ad7 on FarmBot:staging.

Coverage remained the same at 81.144% when pulling d6ed4a2e825c1c97c32ca44448d4370549581268 on RickCarlino:toolslot_delete_bug into 64762082cb0185e62023a6d1179a5760ec9d6ad7 on FarmBot:staging.

Coverage remained the same at 81.144% when pulling 9bf154ec5267389c2674f0fbd2369dd09ea817fb on RickCarlino:toolslot_delete_bug into 64762082cb0185e62023a6d1179a5760ec9d6ad7 on FarmBot:staging.

Coverage remained the same at 81.144% when pulling 9bf154ec5267389c2674f0fbd2369dd09ea817fb on RickCarlino:toolslot_delete_bug into 64762082cb0185e62023a6d1179a5760ec9d6ad7 on FarmBot:staging.


Update overview.md

It was referring to another feature - entity linking.

@dem108 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks for this documentation update! We'll get this merged and published shortly.
#sign-off


This test/src/rules/extensions.js test should be failing with 2 errors not 1

This invalid test should fail with 2 errors – 1 for .js and another for .jsx.
https://github.com/benmosher/eslint-plugin-import/blob/a5844d57be0c3757248c7b824e4cc9c2c844db33/tests/src/rules/extensions.js#L129-L144
The error that I believe is missing:
{
    message: 'Unexpected use of file extension "jsx" for "./bar.jsx"',
    line: 2,
    column: 22,
},

We should probably audit all of the tests for this rule to see where the problem is.

I believe this rule behaves correctly because .js is listed as the first extension under settings > import/resolve > extensions, meaning that ./bar will always try to resolve first as ./bar.js, which exists, so the .jsx should be necessary in order to target bar.jsx specifically.
The fact that you can't import twice from the same module should be outside the scope of this rule and resolving extension shouldn't depend on the import order.

Hmm - if that's true, then a second test, that's an exact duplicate but without the first import line, would fail with just one error - the "missing" one referenced above.

The error is not there, I'll try to fix it.

Also without the first specified extension being .js.  This works:
test({
  code: [
    'import component from "./bar.jsx"',
    'import data from "./bar.json"',
  ].join('\n'),
  options: [ { json: 'always', js: 'never', jsx: 'never' } ],
  settings: { 'import/resolve': { 'extensions': [ '.jsx', '.json', '.js' ] } },
  errors: [
    {
        message: 'Unexpected use of file extension "jsx" for "./bar.jsx"',
        line: 1,
        column: 23,
    },
  ],
}),

This might seem like confusing behavior at first, but the alternative is impossible, so forcing the extension when omitting it would not yield the desired result is pretty smart.

Sounds like we might want to add more test cases, but everything's correct as-is?


feat: enumerate overloaded functions if they are nameless

Motivation
Close #543

Solution
generate aliases using the call's indices if the function params are nameless like
    abigen!(
        ConsoleLog,
        r#"[
            log()
            log(string, string)
            log(string)
    ]"#
    );

becomes
enum    ConsoleLogCalls {
    Log0,
    Log1(...),
    Log2(...),
}

PR Checklist

[x] Added Tests
[x] Added Documentation
[x] Updated the changelog

LGTM


[Logging] Send logs to backend

LGTM, although, there is no tests, but I'm wondering if it is really worth it in such case. I let you decide on this.

Basically I've based this on the sendCsm method which is untested and was wondering if that was worth testing, anyway, added a couple of tests.

LGTM, although, there is no tests, but I'm wondering if it is really worth it in such case. I let you decide on this.

Basically I've based this on the sendCsm method which is untested and was wondering if that was worth testing, anyway, added a couple of tests.

Thanks!

Thanks!


Improve repo documentation

Added CONTRIBUTING.md
Updated Towhee logo
Moved files under docs to root directory

Codecov Report

Merging #326 (99004e0) into main (1ca575c) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 99004e0 differs from pull request most recent head ca841e2. Consider uploading reports for the commit ca841e2 to get more accurate results

@@           Coverage Diff           @@
##             main     #326   +/-   ##
=======================================
  Coverage   69.17%   69.17%
=======================================
  Files         212      212
  Lines        7837     7837
  Branches     1063     1063
=======================================
  Hits         5421     5421
  Misses       2156     2156
  Partials      260      260

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 1ca575c...ca841e2. Read the comment docs.

One thing is that can you add an introduction about how to build towhee from source in README? For example:
git clone https://github.com/towhee-io/towhee.git
python setup.py install

/lgtm

/approve


Roll Flutter Engine from 0293a7cb7887 to ea484fe1d5ef (4 revisions)

https://github.com/flutter/engine/compare/0293a7cb7887...ea484fe1d5ef
2023-07-19<EMAIL_ADDRESS>Fix the rules for determining whether a blur image filter is valid (flutter/engine#43791)
2023-07-19<EMAIL_ADDRESS>Roll Skia from 035b12a03918 to c03050eb2b65 (1 revision) (flutter/engine#43802)
2023-07-19<EMAIL_ADDRESS>Roll Skia from 9062ca6a691c to 035b12a03918 (3 revisions) (flutter/engine#43801)
2023-07-19<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from SCshjyIlymHWD9W4D... to Iqf3gYJ8Vkq8k8l3O... (flutter/engine#43800)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from SCshjyIlymHW to Iqf3gYJ8Vkq8
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC bdero@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Automate input

This PR add automated input from ToL. Ignore the assets and .github changed files, they are the result of linting fixes.
PR checklist

[x] This comment contains a description of changes (with reason).
[x] If you've fixed a bug or added code that should be tested, add tests!

[x] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] If necessary, also make a PR on the nf-core/variantcalling branch on the nf-core/test-datasets repository.

[x] Make sure your code lints (nf-core lint).
[x] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

Why is there both modules/local/samtools_view.nf and modules/nf-core/modules/samtools/view/ ? I think only the former is included in the subworkflow ?

We are using the local version, but eventually I want to import fai without explicitly including it as a parameter, but that needs some engineering that is not  included in this sprint. So the nf-core is for future purposes. If you want I can exclude it for now?

OK, no that's fine. Let's leave it for now

The integration into a workflow is the main increment between this feature and https://github.com/sanger-tol/nf-core-pipeline/pull/2


When configured to use Github, in case of a Github error only block relevant calls

When Github is down or unreachable for whatever reason, we should make sure CG can continue to function as much as possible. So only calls that are dependant on Github should fail. Other calls that only do a search against Github while searching for the source of a cookbook, should just continue and treat the error as if the cookbook was not found on Github.
This will allow cookbook searches to successfully find all other cookbooks located in the Supermarket or an internal Github appliance without any errors.

Huge :+1: here

Closing this issue for now. I would be an improvement, but the last 2 years this happened maybe once. So there is not much incentive to spend time on this one.


Correction for raw affiliation G-EAU, AgroParisTech, Cirad, IRD, IRSTEA, Montpellier SupAgro, Univ Montpellier, Montpellier, France

Correction needed for raw affiliation G-EAU, AgroParisTech, Cirad, IRD, IRSTEA, Montpellier SupAgro, Univ Montpellier, Montpellier, France
raw_affiliation_name: G-EAU, AgroParisTech, Cirad, IRD, IRSTEA, Montpellier SupAgro, Univ Montpellier, Montpellier, France
new_rors: 051escj72;03rnk6m14;05kpkpg04;05q3vnk25;02kbmgc12;020g0rw41
previous_rors: 051escj72;03rnk6m14;05kpkpg04;05q3vnk25;02kbmgc12
works_examples: W4253403883
contact: 4a071705d290129f07ab7111b034b461:c86ef99607519ce0741e9e7cb0db03d34bfc @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


Allow defining API middleware group.

Note in docs that you will need to setup kernel accordingly.
SP will use the configured middleware group to assist routing.

Done and documented.


make adapter for GitHub Actions

Will to see
https://mail.google.com/mail/u/0/#inbox/FMfcgxwDsFgnZgZcVnMCjLRDVQjhCjfn

see https://code.visualstudio.com/docs/editor/tasks#_processing-task-output-with-problem-matchers

https://github.com/actions/toolkit/blob/main/docs/problem-matchers.md


Native for loop's head has destructuring declaration which must have an initializer.

Native for loop's head has destructuring declaration which must have an initializer.
Native for loop with destructuring does not enforce that it must have an initializer. For that purpose the ParseVariableDeclaration takes another param which is actually deducing whether native for loop is okay or not. We do this kind of checking as we don't know whether current loop turned out to be native loop or for..in/of loop.
If the current syntax turned out to be native for loop and during declaration we deduced that native for loop will not be okay then we throw error.
While playing with different test cases  I found that TrackAssignment assert requires adjustment as in the case of for..in loop we rescan the loop head again for validating destructuring grammar. Due that assert is not valid as the symbol span require re-adjustment.
Added few tests to cover more cases.

Hi @akroshg, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Akrosh Gandhi). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Fixes #225
@tcare , @ianwjhalliday  could you review pls?

Looks good

Thanks @tcare


Bug forbit variant column with filter, group by or order by

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


volume policy should enable user to delete data after corresponding volume is deleted

Volume Policy should enable user to mark a volume to be purged of its data once the PV is deleted.

We have shared kubernetes cluster for many customers. And we implement local pv dynamic provisioning based on LVM. When the local pv is deleted and we want to clear the user data on the LV. Now we use the dd-zero command to  cleanup, but it is so slow, for example when we have a 1000G LV, we need several hours to clean it. How can we speed the dd command? or do we have other method?

Above is a discussion at sig-storage.

Kubernetes PR that tries to solve this via a Job:

https://github.com/kubernetes-incubator/external-storage/pull/711/

Covered as part of https://github.com/openebs/openebs/issues/1246


allow custom blob key

Looks good to me, thank you! I'll see if @ahdinosaur has any input and he has to merge and make the release (or add us to the NPM repo).

also looks good to me. :+1: should have added you to the npm package now @daffl.


ImdsCredential.get_token failed: ManagedIdentityCredential authentication unavailable, no response from the IMDS endpoint.

Package Name: azure-identity
Package Version: 1.7.1
Operating System: linux
Python Version: 3.10.16

Describe the bug
ImdsCredential.get_token failed: ManagedIdentityCredential authentication unavailable, no response from the IMDS endpoint.
To Reproduce
Steps to reproduce the behavior:

Called ManagedIdentitfyCredential get_token() to Dynamics-365, and it will says IMDS endpoint not available.

Expected behavior
a token string returned by the service endpoint.
Screenshots

Additional context
---------------------------------------------------------------------------
ServiceRequestError                       Traceback (most recent call last)
File ~/cluster-env/clonedenv/lib/python3.10/site-packages/azure/identity/_credentials/imds.py:80, in ImdsCredential._request_token(self, *scopes, **kwargs)
     79 try:
---> 80     self._client.request_token(*scopes, connection_timeout=0.3, retry_total=0)
     81     self._endpoint_available = True

File ~/cluster-env/clonedenv/lib/python3.10/site-packages/azure/identity/_internal/managed_identity_client.py:123, in ManagedIdentityClient.request_token(self, *scopes, **kwargs)
    122 request_time = int(time.time())
--> 123 response = self._pipeline.run(request, retry_on_methods=[request.method], **kwargs)
    124 token = self._process_response(response, request_time)

File ~/cluster-env/clonedenv/lib/python3.10/site-packages/azure/core/pipeline/_base.py:211, in Pipeline.run(self, request, **kwargs)
    206 first_node = (
    207     self._impl_policies[0]
    208     if self._impl_policies
    209     else _TransportRunner(self._transport)
    210 )
--> 211 return first_node.send(pipeline_request)

File ~/cluster-env/clonedenv/lib/python3.10/site-packages/azure/core/pipeline/_base.py:71, in _SansIOHTTPPolicyRunner.send(self, request)
     70 try:
---> 71     response = self.next.send(request)
     72 except Exception:  # pylint: disable=broad-except

File ~/cluster-env/clonedenv/lib/python3.10/site-packages/azure/core/pipeline/_base.py:71, in _SansIOHTTPPolicyRunner.send(self, request)
     70 try:
---> 71     response = self.next.send(request)
     72 except Exception:  # pylint: disable=broad-except

    [... skipping similar frames: _SansIOHTTPPolicyRunner.send at line 71 (1 times)]

File ~/cluster-env/clonedenv/lib/python3.10/site-packages/azure/core/pipeline/_base.py:71, in _SansIOHTTPPolicyRunner.send(self, request)
     70 try:
---> 71     response = self.next.send(request)
     72 except Exception:  # pylint: disable=broad-except

File ~/cluster-env/clonedenv/lib/python3.10/site-packages/azure/core/pipeline/policies/_retry.py:467, in RetryPolicy.send(self, request)
    466             continue
--> 467     raise err
    468 finally:

File ~/cluster-env/clonedenv/lib/python3.10/site-packages/azure/core/pipeline/policies/_retry.py:445, in RetryPolicy.send(self, request)
    444 self._configure_timeout(request, absolute_timeout, is_response_error)
--> 445 response = self.next.send(request)
    446 if self.is_retry(retry_settings, response):

File ~/cluster-env/clonedenv/lib/python3.10/site-packages/azure/core/pipeline/_base.py:71, in _SansIOHTTPPolicyRunner.send(self, request)
     70 try:
---> 71     response = self.next.send(request)
     72 except Exception:  # pylint: disable=broad-except

File ~/cluster-env/clonedenv/lib/python3.10/site-packages/azure/core/pipeline/_base.py:71, in _SansIOHTTPPolicyRunner.send(self, request)
     70 try:
---> 71     response = self.next.send(request)
     72 except Exception:  # pylint: disable=broad-except

    [... skipping similar frames: _SansIOHTTPPolicyRunner.send at line 71 (1 times)]

File ~/cluster-env/clonedenv/lib/python3.10/site-packages/azure/core/pipeline/_base.py:71, in _SansIOHTTPPolicyRunner.send(self, request)
     70 try:
---> 71     response = self.next.send(request)
     72 except Exception:  # pylint: disable=broad-except

File ~/cluster-env/clonedenv/lib/python3.10/site-packages/azure/core/pipeline/_base.py:103, in _TransportRunner.send(self, request)
     94 """HTTP transport send method.
     95
     96 :param request: The PipelineRequest object.
   (...)
     99 :rtype: ~azure.core.pipeline.PipelineResponse
    100 """
    101 return PipelineResponse(
    102     request.http_request,
--> 103     self._sender.send(request.http_request, **request.context.options),
    104     context=request.context,
    105 )

File ~/cluster-env/clonedenv/lib/python3.10/site-packages/azure/core/pipeline/transport/_requests_basic.py:360, in RequestsTransport.send(self, request, **kwargs)
    359 if error:
--> 360     raise error
    361 if _is_rest(request):

ServiceRequestError: (<urllib3.connection.HTTPConnection object at 0x7f902dfc0c10>, 'Connection to <IP_ADDRESS> timed out. (connect timeout=0.3)')

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

CredentialUnavailableError                Traceback (most recent call last)
Cell In [11], line 26
     23 credential = ManagedIdentityCredential(client_id = uami_client_id, client_credential = None, authority = authority_2019_update)
     24 #scope = "https://graph.microsoft.com/.default"
     25 #credential = DefaultAzureCredential()
---> 26 token = credential.get_token(scope)
     27 print(token)

File ~/cluster-env/clonedenv/lib/python3.10/site-packages/azure/identity/_internal/decorators.py:30, in log_get_token.<locals>.decorator.<locals>.wrapper(*args, **kwargs)
     27 @functools.wraps(fn)
     28 def wrapper(*args, **kwargs):
     29     try:
---> 30         token = fn(*args, **kwargs)
     31         _LOGGER.log(
     32             logging.DEBUG if within_credential_chain.get() else logging.INFO, "%s succeeded", qualified_name
     33         )
     34         return token

File ~/cluster-env/clonedenv/lib/python3.10/site-packages/azure/identity/_credentials/managed_identity.py:119, in ManagedIdentityCredential.get_token(self, *scopes, **kwargs)
    112 if not self._credential:
    113     raise CredentialUnavailableError(
    114         message="No managed identity endpoint found. \n"
    115                 "The Target Azure platform could not be determined from environment variables. \n"
    116                 "Visit https://aka.ms/azsdk/python/identity/managedidentitycredential/troubleshoot to "
    117                 "troubleshoot this issue."
    118     )
--> 119 return self._credential.get_token(*scopes, **kwargs)

File ~/cluster-env/clonedenv/lib/python3.10/site-packages/azure/identity/_internal/get_token_mixin.py:76, in GetTokenMixin.get_token(self, *scopes, **kwargs)
     74 if not token:
     75     self._last_request_time = int(time.time())
---> 76     token = self._request_token(*scopes, **kwargs)
     77 elif self._should_refresh(token):
     78     try:

File ~/cluster-env/clonedenv/lib/python3.10/site-packages/azure/identity/_credentials/imds.py:91, in ImdsCredential._request_token(self, *scopes, **kwargs)
     87         self._endpoint_available = False
     88         self._error_message = (
     89             "ManagedIdentityCredential authentication unavailable, no response from the IMDS endpoint."
     90         )
---> 91         six.raise_from(CredentialUnavailableError(self._error_message), ex)
     93 if not self._endpoint_available:
     94     raise CredentialUnavailableError(self._error_message)

File <string>:3, in raise_from(value, from_value)

CredentialUnavailableError: ManagedIdentityCredential authentication unavailable, no response from the IMDS endpoint.

Hi, @vanlanchoy. It appears you are using a fairly old version of azure-identity. Is it possible for you to upgrade to the latest with pip install -U azure-identity and then retry?
Also, what Azure hosting environment/service are you running this code on? ManagedIdentityCredential only works in certain Azure hosts with Managed Identity enabled.

Hi @pvaneck , I've tried upgraded my Azure.Identity module to 1.16.0 msal-1.28.0, and I'm running my code in Azure Synapse Workspace and still getting the same error. Does the module currently supporting UAMI/SAMI?

Based on the docs here: https://learn.microsoft.com/azure/synapse-analytics/synapse-service-identity#user-assigned-managed-identity, "User-assigned Managed Identity is not currently supported in Synapse notebooks and Spark job definitions.". I'm not sure why this is the case, but this is the likely cause of the UAMI Credential being unable to work. Try using system-assigned MI, and see if you encounter the same problem.

I am seeing the same error for SAMI scenario as well.

Hmm, @xiangyan99 do you know if Managed Identity authentication is supported through our SDKs in Synapse Notebooks? I feel like it might not be as it doesn't seem like the IMDS endpoint is accessible from within the notebook/Spark pool, and we don't do any specific checks for Synapse specific environment variable within ManagedIdentityCredential...
You'd probably have to use service principals (ClientSecretCredential/EnvironmentCredential), or perhaps some of the msspark util functions listed here: https://learn.microsoft.com/azure/synapse-analytics/spark/microsoft-spark-utilities?pivots=programming-language-python

No. Azure identity library does not work in Synapse workspace.
In order to use managed identity in Synapse workspace, you need to use mssparkutils.
For more information: https://learn.microsoft.com/en-us/azure/synapse-analytics/spark/apache-spark-secure-credentials-with-tokenlibrary?pivots=programming-language-python

/unresolve
Hi @pvaneck @xiangyan99
We also have a similar use case and as you suggested we are using mssparkutils with System Assigned Managed Identity. However, our Synapse workspace is on Azure China (Mooncake) cloud and when we use gettoken with the audience type AzureOSSDB as described here, the function looks for the endpoint https://ossrdbms-aad.database.windows.net/ as per Azure Public Cloud (CORP tenant) which is incorrect.

The correct endpoint to look for on Mooncake cloud is *.database.chinacloudapi.cn as described here.
Please advise how can we set the correct cloud environment or override default endpoints in mssparkutils.

No. Azure identity library does not work in Synapse workspace.
In order to use managed identity in Synapse workspace, you need to use mssparkutils.
For more information: https://learn.microsoft.com/en-us/azure/synapse-analytics/spark/apache-spark-secure-credentials-with-tokenlibrary?pivots=programming-language-python

Hey @xiangyan99 I'm trying to also use managed identity to connect to a service from a workspace (to Azure Open AI in my case).
I don't understand how using mssparkutils can help with that. Is there any way to get the workspace or any managed identity from the notebook code in Synapse?

@xiangyan99 , I cannot use SPN due to security issues. I have to switch to MI. After switching to system assigned MI and following the steps mentioned at https://learn.microsoft.com/en-us/azure/machine-learning/how-to-authenticate-batch-endpoint?view=azureml-api-2&tabs=sdk#running-jobs-using-a-managed-identity, I am still facing the ImdsCredential.get_token failed

/unresolve

No. Azure identity library does not work in Synapse workspace.
In order to use managed identity in Synapse workspace, you need to use mssparkutils.
For more information: https://learn.microsoft.com/en-us/azure/synapse-analytics/spark/apache-spark-secure-credentials-with-tokenlibrary?pivots=programming-language-python

Hey @xiangyan99 I'm trying to also use managed identity to connect to a service from a workspace (to Azure Open AI in my case). I don't understand how using mssparkutils can help with that. Is there any way to get the workspace or any managed identity from the notebook code in Synapse?

I managed to get it working by creating a linkedservice to Azure ML workspace from Synapse after making sure that the Synapse's Managed Identity has at least a role of a Data Scientist to the Azure ML workspace.
I used the following code:
from notebookutils import mssparkutils

aml_token = mssparkutils.credentials.getConnectionStringOrCreds("ls_azureml")

where "ls_azureml" is the linkedservice name.
I also had to use a class to get the credential working through an access token from the linkedservice (thanks chatGPT):
# Assuming aml_token contains the access token
class AmlTokenCredential(TokenCredential):
    def __init__(self, token):
        self.token = token

    def get_token(self, *scopes, **kwargs):
        # Wrap the token in an AccessToken object with a valid expiry date
        return AccessToken(self.token, expires_on=int(time.time()) + 3600)

# Use the aml_token retrieved earlier
aml_token = mssparkutils.credentials.getConnectionStringOrCreds("ls_azureml")

# Create an instance of AmlTokenCredential with the token
credential = AmlTokenCredential(aml_token)

You can then use credential in your MLClient class to connect to the ML Workspace.
Hope this helps!


Unable to connect with Azure MFA and 'Sql Authentication Provider' enabled on Linux

Azure Data Studio Version: Latest
OS Version: Linux

Steps to Reproduce:

Set 'Mssql: Enable Sql Authentication Provider' setting to true.
Reload ADS
Try to connect to Azure SQL using Azure MFA
User receives error: "User account not received".

Reasoning
Issue occurs as SqlToolsService is not able to retrieve Encryption Keys for MSAL encrypted cache as they're stored using credential provider in ADS - which stores credentials and keys in VS Code Secret storage for Linux OS.
Solution
To resolve this issue, we need a mechanism to request credentials into service layer from NodeJS layer, with which STS will be able to request credentials from ADS > VSCode secret storage as needed.
Mitigation
Disable "Mssql: Enable Sql Authentication Provider" setting (set as default)
Additional Context
This issue also applies to VSCode MSSQL extension, due to same design gap.

Fixed with https://github.com/microsoft/azuredatastudio/pull/22384


Provisioning tools2

New features:

Support for multiple provisioning templates per project. Added "PnPTemplate" item type to "SharePoint PnP Site Template" project template. Now can have multiple provisioning templates in a single projects. All "PnPTemplate" items will be packed in a single .pnp package. Provisioning template metadata is now associated with "PnPTemplate" item, not project itself.

Support for adding "unreferenced" files in package. Added "PnPTemplateFile" item type to "SharePoint PnP Site Template". All "PnPTemplateFile" will be added to package even if not referenced in Files node of provisioning templates. For those files that can be used in custom extensibility providers.

Updated project properties "Provisioning Template" tab. Added UI controls to support applying of provisioning template when Debug starts. Back end logic is not implemented yet. Project configuration file updated accordingly. XML schema for project configuration file added.

Some bug fixes and small refactoring (renaming things)

Hi @ivanvagunin, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

thx Ivan, appreciated.


Revert "build(deps): bump actions/download-artifact and actions/upload-artifact from 3 to 4“

Thanks for contributing!

What type of PR is this?

release/none
release/bug

What this PR does / why we need it:
Upgraded actions/download-artifact in PR https://github.com/spidernet-io/spiderpool/pull/2943 and PR https://github.com/spidernet-io/spiderpool/pull/2942, but unfortunately it brought some additional problems, and this problem is not unique, many people reported this problem. refer to https://github.com/actions/download-artifact/issues/249
In this regard, first roll back the modified version and observe.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

just trigger a CI by yourself with this branch to test

just trigger a CI by yourself with this branch to test

Yes, I totally agree with what you said. Happily, I ran CI multiple times and everything was fine.
https://github.com/ty-dc/spiderpool/actions/runs/7318560150/job/19935405104


Surface winds

Add Ignition world plugin for simulation of surface winds (collaboration with Hamilton’s DOE).

One option is to use the existing WindEffects plugin. It applies a sinusoidal force that is configurable at runtime through a topic. It's a world plugin, so it applies to all models in the world equally.

BTW ignition::math already implements a Catmull-Rom spline.

It sounds like a good plan to me.

My only thought is to think about how complex is to make the function depend on x, y and z. That way it'll be a bit more generic for the future.

@caguero as we talked earlier, I think a simple yet generic way to make  forceApproximatingScalingFactor a function of x, y, and z would be to define it as f(x, y, z) = p(x).q(y).r(z) where p, q, and r are splines (e.g. piecewise cubic polynomials). Voxel grids and simple surfaces would be easy to define. Describing more general surfaces in those terms may be challenging though.

@caguero @arjo129 @braanan see quick demo for surface winds effects below:
https://user-images.githubusercontent.com/13500507/157509218-a17b622a-8b29-4ea8-9ae9-050f1804beec.mp4
I used the buoyant_tethys.sdf world as a basis. At first, you can see the vehicle getting pushed East by the wind. The vehicle then submerges and cruises downwards, then West, and then upwards. When it emerges, it starts getting pushed East by the wind again.

Thanks, @hidmic. Good progress. Do you happen to know if the vehicle was set to be positively buoyant in that run? Also, what wast the wind magnitude in that run?
Thanks

Argh, I thought I had replied.

Do you happen to know if the vehicle was set to be positively buoyant in that run?

I used buoyant_tethys.sdf defaults, and I did not deflate nor inflate the buoyancy engine from its neutral volume so perhaps it was positively buoyant (a sensible default?).

Also, what wast the wind magnitude in that run?

1 m/s, East.

@braanan as per f2f chat, I'll check what's going on with hydrodynamic drag here. Vehicle is moving too fast for such calm wind conditions.

@braanan there's a non-zero wrench due to hydrodynamic drag when winds are pushing the vehicle, so we're good on that front. I think it comes down to force approximation:
https://latex.codecogs.com/png.image?\dpi{110}F_w&space;=&space;s_f&space;m_b&space;(v_w&space;-&space;v_b)
In the short video above, I used a unit scaling factor above sea water level. Hydrodynamic drag forces are high as a result, but still, the vehicle moves fast. Reducing the scaling factor, e.g. to 0.1, gets us closer to the slow, expected motion. With such a simple approximation, we'll have to tweak it quite a bit.


No hover effects in new Interactive UI

No hover effects in new interactive UI when using pandas' MultiIndex.
Example:
import pandas as pd

url = 'https://api.census.gov/data/2018/acs/acs5/groups/B11002.html'
df = pd.read_html(url)[0]
df = df[['Label', 'Name']]

# only keep Estimate variables
df = df[df.Name.str.endswith('E')]

# make levels
df_level = df.Label.str.split('!!', expand=True).fillna('')
idx = pd.MultiIndex.from_frame(df_level)

# add levels to api variable name
df[['Name']].set_index(idx)

Before:

In Jupyter Lab:

In new UI now:

@StevenLi-DS Thanks for the report here and the clear repro. I repro the same thing. Just something we lost in the transition to the new UI. We'll look into it.

@StevenLi-DS @IanMatthewHuff fyi I mentioned this in #6546 as well


compiler segfaults on ubuntu because of global Time::Span variable

$one : Time::Span = 1.hour
def foo
end

works well on my arch linux, but the compiler segfaults on ubuntu 16.04 x86_64 and on https://play.crystal-lang.org/#/r/ywi

Thanks for reporting, I'll try to fix it in the 0.17 branch.
Workaround:
$one : Time::Span
$one = 1.hour
def foo
end

Fixed in the release/0.17 branch and master


[Caloris2][URL Connector] Add crawling settings

crawling settings
Crawling depth
The depth of links to follow.
Use cases:

A wikipedia page: you probably want to follow up to 1 depth, in order to crawl the wikipedia page itself and all the links it references.
A help center: You probably want to start from the main page of the help center and crawl up to a few level of links (somewhere between 2 and 5) in order to crawl all the articles.

Word matching
A list of words that must be in the URL for a page to be crawled.
The problem is that every page of a website references a bunch of "common pages" like Home, Contact Us, TOS etc, that you probably don't want to crawl.
If you are crawling a large website about a specific topic, you might want to crawl only the pages with an URL that contains a specific word.
For the Wikipedia example, if you are crawling the web page "Product Design", you can set a crawling depth of 1 and a word matching of "design" in order to crawl only pages that are likely related to "product design".

I presume this is done @lasryaric


Difficulty finding how to disable precompile workloads in docs

https://julialang.github.io/PrecompileTools.jl/stable/#Package-developers:-reducing-the-cost-of-precompilation-during-development
If this is missing something, let's fix it!

Oof. Sorry. This is a case of the Docs search not helping.
I knew the solution was Preferences based, so I searched for Preferences and got
https://julialang.github.io/PrecompileTools.jl/stable/search/?q=preferences
If you go onto the first 2 results and search for Preferences there's no results (it seems to be preferring similarity to "references" over actual occurrences of "preferences")
Perhaps making the title of section you linked to 'Disabling precompile workloads via Preferences` would've helped my case... But maybe the current title is better for people that don't even know about Preferences..
SEO is hard..
Feel free to close

New docs search is even worse

Yikes. CC @mortenpi, anything to be done about this? If you do a page-search with Firefox, "preferences" appears 8 times on the page.

No easy solution I suspect. Could be because of the way we write out the search index.

If it helps, PrecompileTools is a tiny package and thus has tiny docs compared to base Julia, which might be easier for debugging?
I hadn't previously dug into the search index, but I see a file, search_index.js, in the build directory. I've copied it into a gist, along with a variant that I converted from js to json so that I could more easily load it into Julia. I'm not quite sure how this data is used for searching, but I do notice some entries that contain "Preferences", e.g.,
{
      "location": "index.html",
      "page": "Home",
      "title": "Home",
      "text": "If you're frequently modifying one or more packages, you may not want to spend the extra time precompiling the full set of workloads that you've chosen to make fast for your \"shipped\" releases. One can locally reduce the cost of precompilation for selected packages using the Preferences.jl-based mechanism and the \"precompile_workload\" key: from within your development environment, use",
      "category": "page"
    }

Here's another funny thing: if I enter "Preferences" into the search bar I get nothing. If I enter "Preferences.jl" into the search bar, I get a set of hits that seem plain wrong, unless you count matching the ".jl" in "Powered by Documenter.jl and the Julia Programming Language" that appears in the footer of each page.


Makes mint adapter only accept mint messages during receive_message/2

PR to fix issue https://github.com/teamon/tesla/issues/398

cc @alex-strizhakov


Version mismatch when using in nwjs

Hi, I'm trying to include robotjs in a nwjs app and after installing via npm install robotjs I get a version mismatch error. Can you help me ?
module.js:625
return process.dlopen(module, path._makeLong(filename));
^
Error: Module version mismatch. Expected 48, got 49.
at Error (native)
at Object.Module._extensions..node (module.js:625:18)
at Module.load (module.js:497:32)
at tryModuleLoad (module.js:456:12)
at Function.Module._load (module.js:448:3)
at Module.require (module.js:507:17)
at require (internal/module.js:20:19)
at Object. (/test/node_modules/robotjs/index.js:1:77)
at Module._compile (module.js:580:32)
at Object.Module._extensions..js (module.js:595:10)

Hi there! With NW.js you need to rebuild the module. You can read more about this here:
https://github.com/nwjs/nw.js/wiki/build-native-modules-with-nw-gyp
I hope this helps!

Awesome, yup it worked. I had to build it with nw-gyp using the following commands !
sudo npm install nw-gyp -g
sudo nw-gyp configure --target=0.17.1 (0.17.1 = nwjs version) (in the robotjs folder)
sudo nw-gyp build --target=0.17.1
Thanks for your help ! Keep up the good work with your module

Awesome glad it worked! I'll get something added to the readme.

I created #236 to handle the docs.


Add bootstrap css

Hi,
First of all thanks for a great starter. I was having some problems trying to add bootstrap to the starter. I tried several ways .

Require the bootstrap.css at Html.js
Import the style , in general styles import section at App/index.js
Tried adding react-bootstrap(Compos work , but still import still fails )

The above methods always throw me an error with the fonts loader: woff -woff2-eot-svg etc. It saids it can not find the "file-loader". Given this I tried playing with the loaders order but still didnt work.
Any idea to get this working, thanks

actually looking in old issues, found the answer. Just needed to disable css modules, is this  correct ?

Hi @lgarro I'd recommend you import the Bootstrap in the ./src/App/index.js file. Yes, you're right just disable the CSS modules then you can use Bootstrap as normal css class :)


Make webnext CI-able

Since gulp tasks don't exit on errors (linter errors, build errors, etc), the linters & other tests can't be used with a continuous integration service like travis-ci.
Either find a way to get the gulp tasks to exit on error or move linting and other testing into npm scripts
Also maybe create a .travis.yml file to make webnext (almost) work out of the box with travis-ci.

Closed by 0860976f6db0ae3972ea90da6349806be87a808d


ValueError: 'axis' entry is out of bounds

In reference to:
https://github.com/andosa/treeinterpreter/blob/master/treeinterpreter/treeinterpreter.py#L60
Taking the sum along the first axis fails, if len(values.shape) == 1, a case which I just stumbled across.

This can also cause an issue in classification (vs regression) on line 64:
https://github.com/andosa/treeinterpreter/blob/master/treeinterpreter/treeinterpreter.py#L64


Get rid of z.hasTruncation in js/zamboni/init.js

Fixes #1320
:bomb: do not land yet :bomb:

Only to be landed after the tag.
r+wc


Update find-help-open-support-ticket.md

Updating steps around directory switching as current steps are inaccurate.  You cannot create techncial support cases from Azure AD B2C tenant due to lack of Azure subscription

@kengaderdus
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

#sign-off


Fix typo

Description

Issues Resolved

Screenshot

Testing the changes

Check List

[ ] All tests pass

[ ] yarn test:jest
[ ] yarn test:jest_integration
[ ] yarn test:ftr

[ ] New functionality includes testing.
[ ] New functionality has been documented.
[ ] Update CHANGELOG.md
[ ] Commits are signed per the DCO using --signoff

Codecov Report

Merging #176 (19fbfd7) into workspace (3ecaf7d) will increase coverage by 20.22%.
Report is 3 commits behind head on workspace.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           workspace     #176       +/-   ##
==============================================
+ Coverage      34.82%   55.05%   +20.22%
==============================================
  Files           1942     1191      -751
  Lines          38428    24051    -14377
  Branches        5499     4028     -1471
==============================================
- Hits           13384    13241      -143
+ Misses         24438    10155    -14283
- Partials         606      655       +49

Flag
Coverage Δ

Linux_1
?

Linux_2
55.05% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 2236 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Problem in script

Hi,
There are an issue in the script it doesn't work .
please can you fix it ?

Any detail about your issue?
I used this amazing script few months ago and everything worked fine.

Hello @tutoriel, as explained in #1 (but in french) you've to ask to ENI to enable PDF export option, this option may be disabled on your account. In this case, script cannot work.

Hi,
@Nicolas01 i don't have any message error . I can download the files but when i open the file i have a message box to insert the password
@Nainterceptor i have the PDF export option in my account
Thanks for help

@Nicolas01 @Nainterceptor  any solution to resolve the problem ?

No, honestly, this project was a one shot project, to help me on one pdf, it's not actively maintained, if you want something, fork & fix by yourself ;)


feat: To Include project id (optional attribute) in Create (Device, DLG, ACL, SSH Public key) and Get details by uuid(Device, DLG, ACL, SSH Public key)

Reference:https://docs.equinix.com/en-us/Content/IAM/iam-resources-projects.htm
Added changes required to allow customers to send Project ID (optional attribute) in

Create Device and Get Device by uuid
Create ACL Template and Get ACL template by uuid
Create SSH Public Key and Get SSH Public Key by uuid
Create DLG and get DLG details by uuid

Link to PR in ne-go repo
[https://github.com/equinix/ne-go/pull/28]

One thing to keep in mind while you're actively working on NE in terraform is that there are a few changes in progress that your team should plan for.  The biggest one is that we are introducing support for defining resources/datasource using https://github.com/hashicorp/terraform-plugin-framework/ instead of terraform-plugin-sdk, which Hashicorp has (in practice, if not in documentation) deprecated. SDK-based resources and datasources should continue to work for some time, but as we migrate more and more things to framework it will be important for your team to start thinking about and planning to migrate NE resources and data sources.


[5.1] Transform JsonSerializable content to Json response

This pull request transforms objects implementing JsonSerializable to Json in the setContent() method of Http/Response.php. By adding a check for that interface in the shouldBeJson() method.
Enabling us to do this:
// controller.php
public function index()
{
  $jsonSerializable = new classImplementingJsonSerializable;
  return $jsonSerializable;
}

Before this PR an \UnexpectedValueException with message The Response content must be a string or object implementing __toString() was thrown from Symfony's Response::setContent() method.

This is a breaking change IMO.

I'm not sure it is.
If you return an object implementing JsonSerializable the current behaviour is to just throw an \UnexpectedValueException and shut down.

After thinking about it I think @GrahamCampbell might be right.
There could be a situation where someone returns an object that both implements JsonSerializable and has a __toString() method.
Current behaviour would be to render whatever __toString() returns.
New behaviour would be to render it as Json.
Seems unlikely, but it's enough to be labeled as breaking IMO.
I'll close this PR and create a new one that commits into 5.2.
Kudos Graham :+1:


remove the horizontal scroll bar

Comment
Remove the horizontal scroll bar that distorts the view of small screens during the sound preview

Hi there, thanks for signing our CLA - you've been added to our list of excellent humans!
About this PR, I wanted to check - is there actually content that requires you to horizontally scroll on your sized screen or is the horizontal bar just showing?
If there is content off to the side, we probably need to look at how we improve the interface for screen sizes that have this behaviour.

The sound wave preview kind of overflows to the left on the small screen.
chrome: v107

Yeah right, my concern then is that if we remove the scrollbar, do we remove the ability to reach anything?
I'm not running this at all atm so hard for me to see exactly how it behaves.
If the audio preview content scrolls across as it plays and all navigation remains accessible, then that seems fine. But if primary navigation items become inaccessible then we clearly have a problem.


Add openFile() method to FileHelper

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Fixed issues
-

This method is similar to the PHP fopen() function, except that it suppresses the E_WARNING level error and throws the \RuntimeException exception if it can't open the file.

👍


Feature index training crash on macOS.

As title, we're not able to train feature index on macOS.
#695 also shown this issue. The latest MPS support allow us to train a model on macOS, but here's the issues:

python crashed after done training.
python crashed when I pressed the feature index training button.

So I assume that there's something wrong on feature index training under macOS, causing this issue.
start.sh: line 5: 49222 Segmentation fault: 11  python3.8 infer-web.py --pycmd python3.8
naozumi@n11649166039 RVC % /Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/multiprocessing/resource_tracker.py:216: UserWarning: resource_tracker: There appear to be 1 leaked semaphore objects to clean up at shutdown
  warnings.warn('resource_tracker: There appear to be %d '

(This error appeared on done training and feature index training button clicked)

Fixed by downgrade faiss-cpu to 1.7.0.


chore: "opn" is deprecated, it's renamed to "open"

@3cp please note, that v4 branch will be merged soon. If you want, you could add this functionality right here

I am lost, I didn't see opn there. Is that "open browser" feature removed from v4 (at least for now)?

@3cp it wasn't removed ;-) It's not implemented. Since v4 is a total re-write with immense speed boost and improvements.

Anyway, I encourage you to try https://github.com/fuse-box/react-example/tree/v4

I see :)
I'll close this one. I think that's easy for you to add the feature back to v4.

@3cp<EMAIL_ADDRESS>you can try it here.
Here are the options
devServer : { open : true }

thx!


Delete operation in iOS is not permitted?

Is this normal behavior?

Closing, eventually this is not related to this lib.


Avoid losing data types while patching nodes

PATCH was not using MetadataDecoders to convert interface{} values to
custom data types.
This commits modifies types.Nodes unmarshaling, using the one from
graph.Nodes which take into account MetadataDecoders.

@adrianlzt I answered on the issue you opened https://github.com/skydive-project/skydive/issues/2375 (with this simple fix https://github.com/lebauce/skydive/commit/80f2fa55bf9e1bc27fb4d91ccbf94c037346410f)

I have removed the test on tests/api_test.go. It was not testing that metadata fields defined in MetadataDecoders were processed correctly.
In fact, I don't know how to make that test, because in tests/api_test.go we only have access to the HTTP API interface, but to test that PATCH recover types correctly, we need to create a node in the graph with some fields used by MetadataEncoders, then apply the patch and check, in the graph, that types are correct.

Closed on favour of @lebauce solution on #2375


Use trilead-api plugin

Use https://github.com/jenkinsci/trilead-api-plugin and bump version.
If https://github.com/jenkinsci/bom/pull/110 get merged then the BOM could be bumped and the version for the trilead-api could be implicit instead.

Thanks @v1v
One piece of feedback. Otherwise LGTM


Can't Find Installed React Native Android App On Phone and Emulator

So something strange happened, when developing I noticed that my react-native app compiled but wasn't visible (not found on the apps list/tray) on my emulator although the react-native run-android launches it (both debug and release) but when I go to Settings > Apps I can see the app listed there, I thought this was some new thing to react for dev environments, but I noticed that after compiling my app as release  I can't find it on the phone until I go to Settings > Apps and even from there I can't launch.
Another thing I first noticed was, that the first time I compile the release, while trying to install a notice appeared on my screen saying, Google play connect rejected my app, and at first I couldn't install the app, but after rerunning the compilation I was able to, but now the app can't be launched.
I have generated a signed apk and have set it up in android > app > build.gradle
Please find below my environment's information.
Environment:
OS: macOS High Sierra 10.13.5
Node: 8.11.3
Yarn: 1.7.0
npm: 5.6.0
Watchman: 4.9.0
Xcode: Xcode 9.4.1 Build version 9F2000
Android Studio: 3.1 AI-173.4819257
Packages: (wanted => installed)
react: 16.3.1 => 16.3.1
react-native: 0.55.4 => 0.55.4

@react-native-bot have made the modifications.

the same issue, @thecodecafe did you solve this?

@abtuleshov I couldn't, I simply started a new react-native project, copied my source files and reinstalled my dependencies.

I had this problem and the issue is that i put <data android:scheme="@string/facebook_app_id" /> inside the same intent-filter as action and category.
Just take data out and put it in another intent-filter like:
<intent-filter>
                <action android:name="android.intent.action.MAIN" />
                <category android:name="android.intent.category.LAUNCHER" />
            </intent-filter>
            <intent-filter>
                <data android:scheme="@string/facebook_app_id" />
            </intent-filter>

Thats solve the problem for me :)

@GeraldHeng Thanks.
I'm actually have a same problem after doing a setup for Firebase Dynamic Link.
Here is my version of AndroidManifest.xml after fix.
<activity
        android:name=".MainActivity"
        android:label="@string/app_name"
        android:configChanges="keyboard|keyboardHidden|orientation|screenSize"
        android:windowSoftInputMode="adjustResize"
        android:launchMode="singleTop">
        <intent-filter>
            <action android:name="android.intent.action.MAIN" />
            <category android:name="android.intent.category.LAUNCHER" />
            <action android:name="android.intent.action.DOWNLOAD_COMPLETE"/>
        </intent-filter>
        <intent-filter android:autoVerify="true">
            <action android:name="android.intent.action.VIEW"/>
            <category android:name="android.intent.category.DEFAULT"/>
            <category android:name="android.intent.category.BROWSABLE"/>
            <data android:host="example.page.link" android:scheme="http"/>
            <data android:host="example.page.link" android:scheme="https"/>
        </intent-filter>
</activity>

@GeraldHeng huge thanks!

Eres grande GeraldHeng Gracias!!!!

I don't think it would be possible to write react-native apps without folks like you and others sharing these tips. THANK YOU!!


The setup_requires are not searched in cache

I'm trying to build a pex archive and I have all the package dependencies local but it looks like setup_requires are not checked in cache, but rather installed from pypi.
I'm trying to build a pex with pygit2 like this: pex -v --no-pypi --cache-dir=/tmp/build/packages/ -r 'pygit2', but I'm getting this error. The build machine has no internet connection:
pex.http: Crawling /tmp/build/packages/: 0.5ms
**** Failed to install pygit2-0.21.3. stdout:OKgW2

**** Failed to install pygit2-0.21.3. stderr:
Download error on https://pypi.python.org/simple/cffi/: [Errno 111] Connection refused -- Some packages may not be found!
Couldn't find index page for 'cffi' (maybe misspelled?)
Download error on https://pypi.python.org/simple/: [Errno 111] Connection refused -- Some packages may not be found!
No local packages or download links found for cffi
Traceback (most recent call last):
  File "<stdin>", line 6, in <module>
  File "setup.py", line 215, in <module>
    cmdclass=cmdclass)
  File "/usr/lib/python2.7/distutils/core.py", line 112, in setup
    _setup_distribution = dist = klass(attrs)
  File "/usr/lib/python2.7/dist-packages/setuptools/dist.py", line 239, in __init__
    self.fetch_build_eggs(attrs.pop('setup_requires'))
  File "/usr/lib/python2.7/dist-packages/setuptools/dist.py", line 264, in fetch_build_eggs
    replace_conflicting=True
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 620, in resolve
    dist = best[req.key] = env.best_match(req, ws, installer)
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 858, in best_match
    return self.obtain(req, installer) # try and download/install
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 870, in obtain
    return installer(requirement)
  File "/usr/lib/python2.7/dist-packages/setuptools/dist.py", line 314, in fetch_build_egg
    return cmd.easy_install(req)
  File "/usr/lib/python2.7/dist-packages/setuptools/command/easy_install.py", line 610, in easy_install
    raise DistutilsError(msg)
distutils.errors.DistutilsError: Could not find suitable distribution for Requirement.parse('cffi')

pex: Packaging pygit2: 88.0ms
pex:   Installing /tmp/tmp7OKgW2: 86.8ms
Traceback (most recent call last):
  File "/usr/bin/pex", line 9, in <module>
    load_entry_point('pex==0.7.0', 'console_scripts', 'pex')()
  File "/usr/lib/python2.7/dist-packages/pex/bin/pex.py", line 310, in main
    pex_builder = build_pex(args, options)
  File "/usr/lib/python2.7/dist-packages/pex/bin/pex.py", line 277, in build_pex
    platform=options.platform)
  File "/usr/lib/python2.7/dist-packages/pex/resolver.py", line 101, in resolve
    raise Unsatisfiable('Cannot satisfy requirements: %s' % requirement_set[requirement.key])
pex.resolver.Unsatisfiable: Cannot satisfy requirements: [Requirement.parse('pygit2==0.21.3')]

The contents of /tmp/build/packages/ is:
/tmp/build/packages/cffi-0.8.6.tar.gz
/tmp/build/packages/pycparser-2.10.tar.gz
/tmp/build/packages/pygit2-0.21.3.tar.gz

Ok, going back in the time machine with this Dockerfile:
FROM ubuntu:14.04

RUN apt-get update && \
    apt-get install -y \
        python-virtualenv \
        cmake \
        build-essential \
        libssl-dev \
        zlib1g-dev \
        libbz2-dev \
        libreadline-dev \
        libsqlite3-dev \
        curl \
        git \
        libncursesw5-dev \
        xz-utils \
        tk-dev \
        libxml2-dev \
        libxmlsec1-dev \
        libffi-dev \
        liblzma-dev

ENV WORKDIR=/pex/issues/17
WORKDIR $WORKDIR

RUN git clone https://github.com/libgit2/libgit2 && \
    cd libgit2 && \
    git reset --hard v0.21.3 && \
    mkdir build && \
    cd build && \
    cmake .. -DCMAKE_INSTALL_PREFIX=/usr/local && \
    cmake --build . --target install

ENV LD_LIBRARY_PATH=/usr/local/lib
ENV LIBGIT2=/usr/local

RUN git clone https://github.com/pyenv/pyenv && \
    cd pyenv && \
    src/configure && \
    make -C src

ENV PYENV_ROOT="$WORKDIR/pyenv"
ENV PATH="$PYENV_ROOT/bin:$PATH"

RUN pyenv install 2.7.18 && \
    pyenv global 2.7.18

RUN echo 'eval "$(pyenv init -)"' >> ~/.bashrc

RUN curl -fLO https://curl.se/ca/cacert.pem && \
    virtualenv -p "$PYENV_ROOT/versions/2.7.18/bin/python2.7" .venv && \
    .venv/bin/pip --cert cacert.pem install -U "pip==20.3.4" && \
    .venv/bin/pip install -U setuptools wheel pex

RUN echo 'source $WORKDIR/.venv/bin/activate' >> ~/.bashrc

CMD ["/bin/bash", "-i"]

I find modern Pex works for your use case @calind. The only change is to use --find-links instead of --cache-dir and that modern Pex no longer needs -r, you just list the requirements as positional args:
# Build the way back machine:
:; docker build -t issues-17 .
[+] Building 0.0s (13/13) FINISHED                                                                                docker:default
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 1.31kB                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [internal] load metadata for docker.io/library/ubuntu:14.04                                                                                0.0s
 => [1/9] FROM docker.io/library/ubuntu:14.04                                                                                0.0s
 => CACHED [2/9] RUN apt-get update &&     apt-get install -y         python-virtualenv         cmake  build-essential  libssl-dev  zlib1g-dev  libbz2-dev  libreadline-dev  libsqlite3-dev  cur  0.0s
 => CACHED [3/9] WORKDIR /pex/issues/17                                                                                0.0s
 => CACHED [4/9] RUN git clone https://github.com/libgit2/libgit2 &&     cd libgit2 &&     git reset --hard v0.21.3 &&     mkdir build &&     cd build &&     cmake .. -DCMAKE_INSTALL_PREFIX=/u  0.0s
 => CACHED [5/9] RUN git clone https://github.com/pyenv/pyenv &&     cd pyenv &&     src/configure &&     make -C src                                                                             0.0s
 => CACHED [6/9] RUN pyenv install 2.7.18 &&     pyenv global 2.7.18                                                                                0.0s
 => CACHED [7/9] RUN echo 'eval "$(pyenv init -)"' >> ~/.bashrc                                                                                0.0s
 => CACHED [8/9] RUN curl -fLO https://curl.se/ca/cacert.pem &&     virtualenv -p "/pex/issues/17/pyenv/versions/2.7.18/bin/python2.7" .venv &&     .venv/bin/pip --cert cacert.pem install -U "  0.0s
 => CACHED [9/9] RUN echo 'source $WORKDIR/.venv/bin/activate' >> ~/.bashrc                                                                                0.0s
 => exporting to image                                                                                0.0s
 => => exporting layers                                                                                0.0s
 => => writing image sha256:185cbc26f0335734afb4781d883bb9fbb446136b8eac9e5c41fa12ba63d52db5                                                                                0.0s
 => => naming to docker.io/library/issues-17

# Use it to test out modern Pex on this old setup:
:; docker run --rm -it issues-17
(.venv)root@eec0a01c04f3:/pex/issues/17# pip --no-python-version-warning -q download -d /tmp/build/packages/ --no-binary :all: pygit2==0.21.3
(.venv)root@eec0a01c04f3:/pex/issues/17# pip --no-python-version-warning -q download -d /tmp/build/packages/ wheel
(.venv)root@eec0a01c04f3:/pex/issues/17# ls -1 /tmp/build/packages/
cffi-1.15.1.tar.gz
pycparser-2.21.tar.gz
pygit2-0.21.3.tar.gz
wheel-0.37.1-py2.py3-none-any.whl
(.venv)root@eec0a01c04f3:/pex/issues/17# pex -V
2.16.2
(.venv)root@eec0a01c04f3:/pex/issues/17# pex --no-pypi --find-links /tmp/build/packages/ pygit2
Pex 2.16.2 ephemeral hermetic environment with 1 requirement and 3 activated distributions.
Python 2.7.18 (default, Aug 23 2024, 03:23:59)
[GCC 4.8.4] on linux2
Type "help", "pex", "copyright", "credits" or "license" for more information.
>>> pex()
Running from ephemeral loose PEX directory: /tmp/tmpBbXSdi
Requirements:
  pygit2
Activated Distributions:
  pygit2-0.21.3-cp27-cp27mu-linux_x86_64.whl
  cffi-1.15.1-cp27-cp27mu-linux_x86_64.whl
  pycparser-2.21-py2.py3-none-any.whl
>>> import pygit2
/root/.cache/pex/installed_wheels/af43fa63ac776b21d0c8ceb091e58910529a4b79bfbc14b0850e844cef382d5c/cffi-1.15.1-cp27-cp27mu-linux_x86_64.whl/cffi/model.py:537: UserWarning: 'git_checkout_notify_t' has no values explicitly defined; guessing that it is equivalent to 'unsigned int'
  % self._get_c_name())
>>> pygit2.__file__
'/root/.cache/pex/installed_wheels/fa752bed5fe70f92d18b842225a92ad01367cad9b18acb898f28e6acaaff1904/pygit2-0.21.3-cp27-cp27mu-linux_x86_64.whl/pygit2/__init__.py'
>>>
(.venv)root@eec0a01c04f3:/pex/issues/17# exit

So, @calind I'm going to close this as (finally) fixed somewhere along the way in the last 10 years.


Support Khmer language

Note to self: It seems that the pipeline/workflow approval has a time limit in a case that if certain times has passed, no info about a workflow being run or anything related to it. To re-trigger, close and re-open the PR


Manually inputting query params in history page causes a server error

Bug
When manually adjusting the query params for start date and end date in the income and expense history the site returns Server Error: 500
How To Reproduce

Go to financial history page
Enter any values that isn't a date with the format mm-dd-yy into the query params startDate & endDate

Expected behavior
The website returns no data available or some error message for that date.
Screenshots

User Specifications
OS: Windows 10
Browser: Chrome Version 131.0.6778.86

Fixed.


FR: Add ability to include calendar list in daily notes template

Hi! Is it possible to make the calendar list appear as part of a template? Something similar to an obsidian-tasks template where it generates the calendar for that day when you create a daily note automatically?

As an interim, I've tried to make a button for this plug-in as part of my Daily Notes template. But it seems to send the feed to the top of the page, rather than where the button is. I'm curious if anyone else has had success.

Thanks. Yes, that works. I'll raise the issue with Buttons.

I looked at the code and I can use javascript:
$= let x= await this.app.plugins.plugins['ics'].getEvents();return x;
to get the events, but can't figure out how to call the function that is called by obsidian for the command, which adds the reformatting.  Will see if I can add that to my snippet.  BUT, if the author can make a function that can be directly called or tell me how to access the registered callback, we can add it to the README and I am sure a lot of folks will copy/paste it

i Looked and getEvents is only called from the callback.  you could move the code in the callback into getEvents and have it return mdArray and then the callback would call events and use the results for the editor.replace call.  If you think there is a need to return the event object array, you could add an argument to getEvents to specify whether to return the events as objects or as the array of strings

Since I documented how to use Templater + Dataview to do this, closing this issue.

Can you say what the script is that you include in the documentation?
Should it be prefaced by <% or dataviewjs?
Thanks.

When I try the code:

I get back the events rendered:

@kmaustral I also added documentation for using Templater. Hope one of those works for your use case.


Low browser performance when scrolling page of many services

I suspect this is related to drawing the many background gradients

Yeah, the jenk is pretty serious at 150 services.

Taking tillery off here. I'm going to open a performance epic and tag him on that.


How to extract metadata from data sources

Where can I find the ETL scripts which extracts or crawls the metadata?
We have our data in various forms ex : in S3, CSV, parquets.
How can I feed or stream the metadata from these data sources to Kafka or Datahub system ?
My preference would be not to copy the data to external platforms.

the sample scripts are under metadata-ingestion folder
By running any of those scripts, for example, mysql-etl.py, it will connect to MySQL database, find the database you specified, and find all table schemas, assemble the schemas into Kafka message that Datahub recognize, and publish the message into Kafka.
Please take a look at the architecture diagram to understand how the system works.

Assuming the question has been answered. Feel free to reopen otherwise.


Translate docs to Russian

By doing so buffalo can better cover ru speaking audience. I'm ready to handle it by translating whole docs.

@markbates could you assign this issue to me please?

@orderbynull I just started the Spanish translation and the recommendation from @stanislas-m was to submit each page on its own branch/PR. We have a Russian team member and I'd love to see Russian documentation!


Increment version for digitaltwins releases

Increment version for digitaltwins releases

Can one of the admins verify this patch?


fix for non-seekable streams

prevents error message fseek(): stream does not support seeking

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
should not

Tests pass?
see CI

Fixed issues
Issue from Slack "eseperio 1.9.2020"

In our case we got streams (images) from AWS S3 which threw the above error.
We could not determine the underlying reason, but in all cases we need to check if a stream is seekable before calling fseek(), this might also apply to other code in the Response class.
CC: @eluhr @handcode

Related https://github.com/thephpleague/flysystem-aws-s3-v3/commit/c73ebc5b78076e971ec64fdab8a5a956d118b160

Should it be added as well for https://github.com/yiisoft/yii2/blob/master/framework/web/Response.php#L586 ?

Should it be added as well for https://github.com/yiisoft/yii2/blob/master/framework/web/Response.php#L586 ?

Yeah, I was unsure about it, since you can prevent a call here by specifying the fileSize.
Should I simply copy the code, which is not so nice?

Some private method would be better. I guess we cannot test it properly without some additional mock libraries (unless you know how, then please do). And please add changelog for this.

@samdark do you know how to mock stream without extra library?

https://stackoverflow.com/questions/41300828/how-to-fake-a-resourse-for-a-unit-test-in-php
And in theory u can just create a stream with, but I  wonder how u could test with that particular properties of a stream.
$stream = fopen('php://memory', 'r+');

Yes, I've seen it as well, question is whether it can simulate (non-)seekable one.

It's seekable:
'timed_out' => false
'blocked' => true
'eof' => false
'wrapper_type' => 'PHP'
'stream_type' => 'MEMORY'
'mode' => 'w+b'
'unread_bytes' => 0
'seekable' => true
'uri' => 'php://memory'

How to test nonseekable ones?

Heh, fopen('php://output', 'r+') is nonseekable.
@schmunk42 will you prepare unit tests and changelog line?

Heh, fopen('php://output', 'r+') is nonseekable.
@schmunk42 will you prepare unit tests and changelog line?

Yeah, will do next week. Also the private method.

How to fix The method sendContent() has a Cyclomatic Complexity of 10. The configured cyclomatic complexity threshold is 10. in https://codeclimate.com/github/yiisoft/yii2/pull/18290?

How to fix The method sendContent() [...]?

I wouldn't worry about that.

👍


Video size is broken after migrating from 1.4.1 to 1.5.0

Version
Media3 main branch
More version details
No response
Devices that reproduce the issue
Pixel 9 pro 14 android
Devices that do not reproduce the issue
No response
Reproducible in the demo app?
Not tested
Reproduction steps
Video link: https://static.brighty.app/onboarding/exchange.mp4
We use Exoplayer in Compose screen through AndroidView:
    AndroidView(
        factory = { context ->
            PlayerView(context).apply {
                this.useController = false
                this.player = player
                this.resizeMode = AspectRatioFrameLayout.RESIZE_MODE_FIT
            }
        },
        modifier = Modifier.fillMaxSize()
            .background(color = Theme.colors.positive)
    )

Version 1.5.0:
https://github.com/user-attachments/assets/631fe91c-3270-49f6-af81-853634244304
Version 1.4.1
https://github.com/user-attachments/assets/1ed92af4-f9b4-4b17-9991-23dde79c9068
Expected result
Size of resulting video should not change
Actual result
Size of video has changed
Media
https://static.brighty.app/onboarding/exchange.mp4
Bug Report

[ ] You will email the zip file produced by adb bugreport to<EMAIL_ADDRESS>after filing this issue.

Have you enabled to Compose-specific workaround for PlayerView as described in the 1.5.0 release notes?
See also https://github.com/androidx/media/issues/1594

I am seeing the same issue with the compose workaround enabled
AndroidView(
            modifier = Modifier.fillMaxSize(),
            factory = {
                PlayerView(context).apply {
                    player = exoPlayer
                    resizeMode = AspectRatioFrameLayout.RESIZE_MODE_ZOOM
                    useController = false
                    setEnableComposeSurfaceSyncWorkaround(true)
                    FrameLayout.LayoutParams(
                        ViewGroup.LayoutParams.MATCH_PARENT,
                        ViewGroup.LayoutParams.MATCH_PARENT
                    )
                }
            }
        )

@anton-begeima-okcupid Interesting - what do you observe on 1.4.1 (where the workaround should be enabled by default, and that method doesn't exist)? That will help us understand whether the workaround in general isn't working for you, or whether there's something wrong with the way we're toggling it in 1.5.0.

On 1.4.1 the video displayed correctly but on 1.5.0 is stretched width-wise. here is the video from Pixel 7 Android 15.
https://github.com/user-attachments/assets/af1fb20c-bac7-4d30-97e1-3853fcbc1a0b
https://github.com/user-attachments/assets/1ce27778-ea00-4391-8783-9c0da2f5efdf

The repro case we were provided with in https://github.com/androidx/media/issues/1237 works with the workaround on Android 14 and media3 1.5.0, and doesn't work without the workaround (I just checked).
@anton-begeima-okcupid Please can you provide a minimal reproducible example that demonstrates the problem in a way that we can build locally? Without this I don't think we can investigate further.
This could be an Android Studio project zipped up and sent to<EMAIL_ADDRESS>with the subject "Issue #1234". If emailing please also update this issue to indicate you’ve done this.

I sent the email with the minimal reproducible example attached. I reproduced the issue on the emulator Pixel 8a API 35. The file was a little too big for the email so I added the Google Drive link and granted permissions to the<EMAIL_ADDRESS>Let me know if it works, if not maybe I can just create a public repo.

This seems an issue with the Pixel phones, rather than an issue with the 1.5.0 code.
On a Pixel 4a I can see that the device provides a mediaFormat in MediaCodecVideoRenderer::onOutputFormatChanged() with sar-width and sar-height wrongly set to 1 (a surviving default assignment?) while they should be 360:427 as signalled in exchange.mp4 both in SPS VUI and in pasp. I've checked other devices and they simply do not carry the sar in mediaFormat.
One way to resolve it could be a Pixel workaround gating the block at https://github.com/androidx/media/blob/cc8439db93f256d2c5820b38ce9ffac9154a1203/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/MediaCodecVideoRenderer.java#L1366, changed in #PR1371. While a real case of sar!=1 from stream and sar=1 after post-processing would still fail on a Pixel phone it seems to us still an improvement on both 1.4.1 and 1.5.0.
We, as contributors of #PR1371, are willing to provide such a PR, please @icbaker let us know.

There's also more info/analysis of this same issue in internal b/378713448.

Hi Ian,
Thanks for the heads up, but how can I access your internal bug tracker?
Best,
Daniele
This email and any attachments are sent in strictest confidence for the sole use of the addressee and may contain legally privileged, confidential, proprietary and personal data. If you are not the intended recipient, please advise the sender by replying promptly to this email and then delete and destroy this email and any attachments without any further use, copying or forwarding.
For more information on how we use and protect your information, please review our Privacy Policy herehttps://www.v-nova.com/privacy-policy/
Please consider your environmental responsibility before printing this e-mail
From: Ian Baker @.>
Sent: 11 December 2024 10:15
To: androidx/media @.>
Cc: Daniele Sparano @.>; Mention @.>
Subject: Re: [androidx/media] Video size is broken after migrating from 1.4.1 to 1.5.0 (Issue #1955)
There's also more info/analysis of this same issue in internal b/378713448.
—
Reply to this email directly, view it on GitHubhttps://github.com/androidx/media/issues/1955#issuecomment-2535423128, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACYX2KJMWTSNGLLJ43L3ZHL2FAGCDAVCNFSM6AAAAABTCMSK4CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMZVGQZDGMJSHA.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@dsparano You cannot. This note is for us in the future to more easily follow the breadcrumbs from GitHub back to our internal issue tracker.


issue #1162 : Expose Jackson settings to allow configurability for JSON parsers

This PR addresses GitHub issue: #1162 and #1185 .
Briefly describe the changes proposed in this PR:

Exposes the various Jackson JSON parser features as RDF4J settings so they can be set by RDF4J users for JSONLD, RDF/JSON, and SPARQL/JSON. All of the new RDF4J settings are all set to the default Jackson values, so there is no change in behaviour for existing users
Bumps jsonld-java version to pickup a fix that is exposed by the new, non-standard, off-by-default, ability to parse NaN as a native JSON double value

I closed the request that was targeted at master, and this PR is now rebased and targeted at develop

@jeenbroekstra I don't have commit permissions so someone else will need to merge this in.


Missing contributors in License screen

All the contributors in the original LICENCE file should be displayed within the app.
https://git.tartarus.org/?p=simon/puzzles.git;a=blob;f=LICENCE;hb=HEAD

Good catch, thanks. I have a bugfix release coming out soon - I'll include them!

Resolved in version 1.2.1.


You? erroneous comments on Lodge

When You? enters the Lodge with you, they comment about the atmosphere being dangerous, or about the importance of a space suit, and so on, as if the Lodge were an unbreathable atmosphere.

Details
Possibly applies to some other NPCs. If the follower has dialogue specific to the Lodge, like the Constellation members do, then they don't comment on the atmosphere of the lodge. I have not tested it with other non-constellation followers
Game Version
1.7.36
Language
English (EN)
New Game
No
Reproduction Steps
Have You? as your follower
enter the lodge

You? will make this comment in various locations. Just heard this comment upon entry in a number of breathable locations inside Neon. Smelly sure... but definitely breathable.

this is the same as #977 - I think I can fix it, more details over there


dynamic deep form

Hello there,
I have the following form :
filters : {
 filter1:['asdasd','asdasd','asdasd'],
 filter2:['asdasd','asdasd'],
 filter3:['44444','asdasd','55555']
}
time: {
 start:'asdjansd',
 end:'asjdn'
}

lets say I have 15 potential filters fields that I want to add dynamically by user select.
but when I am trying to 'filters.addField()' \  filters.specific.addField() Its not working. (via onClick functionality from button inside the render scope of the Form)
I'v tried to set the following configurations on the reduxForm object fields field :
fields : ['filterBy[]',...]
fields : ['filterBy[].filter1,','['filterBy[].filter2',...]
fields : ['filterBy.filter1,','['filterBy.filter2',...]
fields : ['filterBy',...]
Any thoughts ?
Thanks in advance

Find a way to it.
maybe I will post it later, thanks.


API Explorer from openapi/swagger

Since openapi/swagger metadata can be used to generate both API documentation and API explorer, do you have any plans to create API explorer for the API interaction for a user ?
Swagger has a tool, named swagger-ui to do the same.

No, my goal when creating bootpring-swagger/bootpring-openapi was actually to provide printable documentation. I had not found any suitable tool at the time.
If you want an API-explorer, you should use swagger-ui. I don't think the OpenAPI-Specification has changed massively since OpenAPI was created from Swagger. They should still be the same so swagger-ui should work fine with OpenAPI-Specs.


Filesystem Bundles and custom content

Filesystem now saves objects in file without wrapping them in bundles, but can parse objects saved in bundles in files.
FileSystem and Memory tests are separated out into their own files.
Custom properties, objects, etc. can be added to and retrieved from FileSystem and Memory datastores.
Custom properties are returned by object.

Fixes #76, #85, and #87.

Codecov Report

Merging #88 into master will increase coverage by 0.22%.
The diff coverage is 95.61%.

@@            Coverage Diff             @@
##           master      #88      +/-   ##
==========================================
+ Coverage   96.55%   96.77%   +0.22%
==========================================
  Files          53       55       +2
  Lines        4875     4997     +122
==========================================
+ Hits         4707     4836     +129
+ Misses        168      161       -7

Impacted Files
Coverage Δ

stix2/test/test_data_sources.py
100% <100%> (ø)
:arrow_up:

stix2/test/test_memory.py
100% <100%> (ø)

stix2/test/test_filesystem.py
100% <100%> (ø)

stix2/test/test_custom.py
100% <100%> (ø)
:arrow_up:

stix2/base.py
100% <100%> (ø)
:arrow_up:

stix2/core.py
100% <100%> (ø)
:arrow_up:

stix2/sources/__init__.py
96.19% <100%> (ø)
:arrow_up:

stix2/test/test_bundle.py
100% <100%> (ø)
:arrow_up:

stix2/sources/taxii.py
34.61% <30.76%> (ø)
:arrow_up:

stix2/sources/memory.py
84.26% <86.66%> (+7.86%)
:arrow_up:

... and 4 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 09ef935...476cd1e. Read the comment docs.

Don't know why Travis failed yesterday, but I reran the job and it passed.


[Bug]: CloudFormation Stack fails to sync or get ready when it takes too long to deploy

Is there an existing issue for this?

[X] I have searched the existing issues

Affected Resource(s)

cloudformation.aws.upbound.io/v1beta1 - Stack

Resource MRs required to reproduce the bug
apiVersion: cloudformation.aws.upbound.io/v1beta1
kind: Stack
metadata:
  name: ct-stack
spec:
  forProvider:
    name: controltower-stack
    parameters:
      AuditAccountId: "xxxxxxxxxxxxxx"
      LogArchiveAccountId: "xxxxxxxxxxxxxx"
    region: us-east-1
    templateBody: |
      {
        "AWSTemplateFormatVersion": "2010-09-09",
        "Description": "AWS Control Tower Setup",
        "Parameters": {
          "AuditAccountId": {
            "Type": "String",
            "Description": "The ID of the Audit Account"
          },
          "LogArchiveAccountId": {
            "Type": "String",
            "Description": "The ID of the Log Archive Account"
          }
        },
        "Resources": {
          "ControlTowerLandingZone": {
            "Type": "AWS::ControlTower::LandingZone",
            "Properties": {
              "Manifest": {
                "governedRegions": ["us-east-1"],
                "organizationStructure": {
                  "security": { "name": "security" }
                },
                "centralizedLogging": {
                  "accountId": { "Ref": "LogArchiveAccountId" },
                  "configurations": {
                    "loggingBucket": { "retentionDays": 60 },
                    "accessLoggingBucket": { "retentionDays": 60 }
                  },
                  "enabled": true
                },
                "securityRoles": {
                  "accountId": { "Ref": "AuditAccountId" }
                },
                "accessManagement": { "enabled": true }
              },
              "Tags": [
                { "Key": "Name", "Value": "ControlTowerLandingZone" }
              ],
              "Version": "3.3"
            }
          }
        },
        "Outputs": {
          "LandingZoneId": {
            "Description": "The ID of the Control Tower Landing Zone",
            "Value": { "Ref": "ControlTowerLandingZone" }
          }
        }
      }

Steps to Reproduce

Create a CloudFormation Stack resource with the manifest mentioned above.
Wait for it to be provisioned on AWS (around 30 minutes)
Check the resource on crossplane and see that is not synced or ready and keeps trying to recreate.

What happened?
If it is a simple cloudformation template (for instance, creating a parameter store), it works fine and the resource gets Ready and Synced. But if the cloudformation template is complex and takes too long to complete (like enabling Control Tower on a master account) it never gets synced or Ready, and it keeps trying to recreate the stack, even thou it was created successfully on AWS.
Relevant Error Output Snippet
Warning CannotCreateExternalResource 2m57s (x215 over 3h33m) managed/cloudformation.aws.upbound.io/v1betal, kind-stack (combined from similar events): async create failed: failed to create the resource: [{0 creating CloudFormation Stack (FoundationControlTowerStack): operation error CloudFormation: CreateStack, https response error StatusCode: 400, RequestID: ec977b01-3963-49da-8069-1f6f134b055a, AlreadyExistsException: Stack [FoundationControlTowerStack] already exists []}]

Crossplane Version
1.17.0
Provider Version
1.14.0
Kubernetes Version
No response
Kubernetes Distribution
EKS
Additional Info
I have tried to create an Observe-only resource to import the Cloudformation stack that was created and it works, but only if I set the external-name metadata do the ID (last part of the ARN of the stack), not the full ARN or Name. I'm not sure if is related to the bug.

Just to add to this, I noticed that after exactly 15 minutes I get the "token expired" error on the provider logs, and after that it tries to recreate the stack, generating the AlreadyExistsException over and over again.
2024-10-02T18:10:11Z    DEBUG   provider-aws    Cannot create external resource {"controller": "managed/cloudformation.aws.upbound.io/v1beta1, kind=stack", "request": {"name":"ct-stack"}, "uid": "dd85052a-1483-438b-8d93-6e33aa384315", "version": "19203025", "external-name": "", "error": "async create failed: failed to create the resource: [{0 waiting for CloudFormation Stack (arn:aws:cloudformation:sa-east-1:096026687571:stack/controltower-stack/23403010-80e5-11ef-b07e-0a6a2c5bb2b9) create: operation error CloudFormation: DescribeStacks, https response error StatusCode: 403, RequestID: aa00246f-187c-4c34-b175-695ea3a91163, api error ExpiredToken: The security token included in the request is expired  []}]"}
2024-10-02T18:10:11Z    DEBUG   provider-aws    Async create starting...        {"trackerUID": "dd85052a-1483-438b-8d93-6e33aa384315", "resourceName": "ct-stack", "gvk": "cloudformation.aws.upbound.io/v1beta1, Kind=Stack", "tfID": ""}
2024-10-02T18:10:11Z    DEBUG   provider-aws    Creating the external resource  {"uid": "dd85052a-1483-438b-8d93-6e33aa384315", "name": "ct-stack", "gvk": "cloudformation.aws.upbound.io/v1beta1, Kind=Stack"}
2024-10-02T18:10:11Z    DEBUG   provider-aws    Async create ended.     {"trackerUID": "dd85052a-1483-438b-8d93-6e33aa384315", "resourceName": "ct-stack", "gvk": "cloudformation.aws.upbound.io/v1beta1, Kind=Stack", "error": "async create failed: failed to create the resource: [{0 creating CloudFormation Stack (controltower-stack): operation error CloudFormation: CreateStack, https response error StatusCode: 400, RequestID: 90d2e4b5-94f9-4ffe-85e8-6e764c42604e, AlreadyExistsException: Stack [controltower-stack] already exists  []}]", "tfID": ""}

Seems related to #1346, #1482 and https://github.com/crossplane/crossplane/issues/5918

I am facing the same issue, when creating CFN stacks, that takes longer to create. In my case I am creating CloudFront distribution in the stack. I also get security token in the request is expired error.


Refactor Postman Docs

This PR adds more info on how to use Postman, such as:

how to make API calls with video examples
info about Postman variables
how to use data visualization with video examples

Hey @laviniat1996 the build is failing every check, please run yarn build on your local branch to see why and push the changes when the build passes. Instructions were in the pull_request_template

any update on this?

any update on this?
Sorry, I didn't have time to work on this last week. I feel like the issue could be  Cloudflare Pages not allowing hosting for big video files. They have Cloudflare Stream too, so I suppose that's the option for adding videos. Not sure tho.

Definitely wont merge if cloudflare wont build, is it possible to remove the videos and instead embed a youtube video like this: https://docs.avax.network/learn/avalanche/avalanche-consensus#deep-dive-into-avalanche-consensus

@meaghanfitzgerald this is finally ready


User Request: Show nodes created with maximum arguments instead of minimal arguemnts.

Description
The documentation that the SDK has is good and working fine, but users could benefit if the example code for creating each node was made with all the possible arguments instead of just the required arguments.
Instead of the minimal code example with only the required arguments
# minimal file node
my_file = cript.File(
    name="my file node name",
    source="https://criptapp.org",
    type="calibration",
)

we should have code examples with the maximum amount of arguments
# maximal file node
my_file = cript.File(
    name="my file node name",
    source="https://criptapp.org",
    type="calibration",
    extension=".csv",
    data_dictionary="my file's data dictionary"
    notes="my notes for this file"
)

This will show users better how they can create their nodes.

Counter Point
If we do have examples with maximum amount of arguments the examples and docstrings can get big, hard to navigate, hard to maintain. It can also be too much documentation and overwhelm the user seeing everything all at once.