fix: PodcastConfigs::loads exists when the config is empty

All code branches that operates on PodcastConfigs are happy if it's empty, except the load function that called std::process::exit(1).
Both add and search tries to push entry to the PodcastConfigs map, but as push tries to load (100% logical to prevent data loss) it existed when the config was empty. As a result we couldn't add new entries until we added an entry manually (which is a bit tricky as the format is documented and shouldn't be an expected behaviour to add entries manually before a user can use --add or --search.

I think it's fine to print out the error message if the list is empty in all cases as additional operations are logged also:
Enter index of podcast to add
1: xxxxxxxxxxxxxx
1
No podcasts configured!
You can add podcasts with the following methods:

* "talecast --search <name of podcast>"
* "talecast --add <feed url>  <name of podcast>"
*  Manually configuring the "/home/efertone/.config/talecast/podcasts.toml" file.
'xxxxxxxxxxxxxx' added!

If we want to hide it we would have to either add the error message on commands that may need it, or add an extra flag to loads. Latter is pretty ugly, the other one makes more sense to all stdout/err handled only on app layer and not config and other, I don't think it's worth the effort and maybe fix when a refactor is needed.

Thank you! That's pretty critical, didn't think about that when i was moving some code around 😬


API V1: Add auth0

can you add .idea folder to the gitignore please?

what are the new environment variables?

what are the new environment variables?


Package upgrade: device_info 2.0.0 --> 2.0.2

ITS-PRO: MA-95
Package upgrade: device_info 2.0.0 --> 2.0.2
This is done when:

[x] [PR 1399 Test Plan] (https://ucsdcloud.sharepoint.com/sites/WorkplaceTechnologyServices-CampusMobileBuilds/Shared Documents/Campus Mobile Builds/Pull Request Testing/PR-1399-Test-Plan-7.8.0-QA-1454.xlsx?web=1) PASS on iOS (@kevdesilva01)
[x] PR 1399 Test Plan PASS on Android

PASS on iOS


Fix travis build failure

should be fixed now
cc: @ronanki
ref: https://github.com/CSTR-Edinburgh/merlin/commit/1af513638aaf0bebe225e297e956a09da0280106#commitcomment-25811630

@r9y9 Thank you for the update.


Azure Storage Queue scaler is reporting wrong number of messages in queue

Report
Hello
I think the azure_queue monitor is not reading the number of messages in the queue as it should
Looking at the code here I think that when Keda calculates the "visible messages" as 0 it returns 0, even though the actual number of messages in the queue is much higher
For example:
We see "showing 0 of 128503 messages in queue" in the portal, and Keda is also reporting 0 for the metric (even though the actual count is 128k) - see the right side in the screenshot
When we work with the c# SDK for azure storage we're getting the right numbers - see left side in the screen shot (csharp code below):

public async Task Run()
        {
            m_tracer.TraceInformation($"{nameof(StorageAccountMetricsLogger)} - Starting work.");
            CloudStorageAccount storageAccount = m_aadStorageProvider.GetStorageAccount(m_queueStorageName);
            var queueClient = storageAccount.CreateCloudQueueClient();
            var queueNames = queueClient.ListQueues();
            foreach (var cloudQueue in queueNames)
            {
                var dimension = new Dimensions();
                dimension.SetDimension("Name", cloudQueue.Name);
                var instrumentor = new Instrumentor(dimension);
                await cloudQueue.FetchAttributesAsync();
                if (!cloudQueue.ApproximateMessageCount.HasValue)
                {
                    m_tracer.TraceWarning(0, $"{nameof(StorageAccountMetricsLogger)} - Failed to retrieve message count for storage queue {cloudQueue.Name}.");
                    continue;
                }
                instrumentor.LogMetric(s_monitoringStorageQueueActiveMessageCountMetricName, cloudQueue.ApproximateMessageCount.Value);
                m_tracer.TraceInformation($"{nameof(StorageAccountMetricsLogger)} - Metrics for storage queue name \"{cloudQueue.Name}\" messages = {cloudQueue.ApproximateMessageCount}");
            }
            m_tracer.TraceInformation($"{nameof(StorageAccountMetricsLogger)} - Done.");
        }

Expected Behavior
keda should report the actual # of messages in queue
Actual Behavior
keda is reporting 0 messages in queue
Steps to Reproduce the Problem

add messages to azure queue
query the queue with keda

Logs from KEDA operator
No response
KEDA Version
2.4.0
Kubernetes Version
1.19
Platform
Other
Scaler Details
Azure storage queue
Anything else?
No response

Hi @jungopro
Could you share KEDA's logs? (from operator and metric server)

Hi @jungopro
I was unable to reproduce the issue. Note that the scaler code also peeks at the queue to try to get an accurate number in order not to use the approximation if a definite number can be obtained.
What is the size of the messages in the queue ? If they are large it may be a result of a timeout during the peek phase.

I've just run into the same problem, and it's a big one for my use case.
It happens when the queue messages are not deleted within seconds after being popped from the queue.
The scaler peeks messages, and peek returns the visibile messages in the queue.
When a client Gets a message from the queue, it doesn't delete it immediately: it only hides it for a configurable period of time (see: visibility timeout) which defaults to 30 seconds.
The message must be deleted with a separate Delete operation.
Let's say I enqueue 10 messages.
Keda peeks 10 messages and creates 10 pods.
Each pod gets a message from the queue, processes it (e.g: calls a webservice, sends an email, etc) and then deletes the message.
If the time needed to process the message is high (e.g: a few hours), Keda will peek 0 messages and will try to scale in to 0 pods.
Proposed solution:
The scaler should Get Queue Metadata insteak of Peek, and then use the x-ms-approximate-messages-count response header to scale in/out.
x-ms-approximate-messages-count includes hidden messages too.

I am total Azure (Storage) noob, but in general, if it takes a few hours to process a message, then ScaledObject might not be the right option, you should try to use ScaleJob.

Hey @antonio-policastro
I think that the implementation can recover the expectation of more than 32 messages.
I'm not an expert in azure sdk, but I think that we check if they are less than 32 and if not, we check the estimate amount of them so we can get more than 32

Oh, I see. Thanks, i completely missed that you were already using ApproximateMessagesCount when the queue has more thank 32 messages.
During my tests I always had less than 32 messages.
I believe that the scaler should always use ApproximateMessagesCount and never Peek, because Peek can't see hidden messages.
What do you think?
From docs:

If you need to dynamically determine whether to adjust workloads to handle message volume, you can query approximate message count on each queue and then respond with the appropriate action. Use the Get Queue Metadata REST operation or use any of the supported Blob storage SDKs to get the approximate message count.

I'm not sure about it, peek returns the value with accuracy.  ApproximateMessagesCount (at least in .NET SDK) could return more messages than actual count

Right. Approximate may return >0 count when in fact there aren't any
messages at all.
This looks more and more like something that we can resolve with a scaler
parameter that will determine whether to use peek or not
On Tue, Mar 1, 2022 at 2:34 PM Jorge Turrado Ferrero <
@.***> wrote:

I'm not sure about it, peek returns the value with accuracy.
ApproximateMessagesCount (at least in .NET SDK) could return more messages
than actual count
—
Reply to this email directly, view it on GitHub
https://github.com/kedacore/keda/issues/2385#issuecomment-1055398053,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATJ7GSL2JAHS5X5HKN2V7LU5YFGJANCNFSM5JMYOAVQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

A scaler parameter would solve my problem (and jungopro's too), and it wouldn't be a breaking change. Upvote!
I didn't know that ApproximateMessageCount could return a wrong (higher) number. Can you point to some documentation about it? I can't find anything.
Thanks everyone for your prompt answers.

hey @antonio-policastro
This is the .NET docs about that method.

Did anything happen with this?
I am in exactly the same situation as jungopro both with the scaling issue with visible messages, and the viability of scaled jobs.
Not including invisible messages doesn't seem right. If I have KEDA set to scale 5 messages per pod, and I have 5 messages in progress, it would take another 6 messages to make KEDA request a second pod. It also results in the race condition observed here were if all of the in progress messages are not completed within the cooldown time frame, KEDA will scale in while they are still processing.
Having the option to use approximate message count instead seems ideal.
On a side note: when using Azure functions on an App Service Plan, you can scale based on storage queue length. The implementation Azure has used for this is approximate message count, and does not offer any other method of scaling based on queue length.
A screenshot from the app service plan scale rules GUI:

I am not sure if my earlier comment will trigger this issue to be reopened, so I raised a feature request as well.


feat: add env inputs

I suggest using env and buildEnv as the names as opposed to the pluralized versions. In my mind, env can already be considered plural (similar to deer/fish/etc.) and this would match more closely with the CLI flag names.

@TooTallNate Thanks for the feedback, fixed it!

@TooTallNate Any idea when this could be merged?


There should be a fieldName option for filterBy-based queries

When using any query that reduces to ModelCriteria::filterBy (e.g. MyTableQuery->findOneByArray), you MUST use phpNames for all columns. This is because filterBy uses getRealColumnName, which is hardcoded to use phpNames. This is inconsistent with the ActiveRecord functions (getByName, toArray, etc.) which are controlled by the generator.objectModel.defaultKeyType config setting.
So I can set defaultKeyType=fieldName for my project and use raw field names on my get & set operations, but I'm still forced to use phpNames in my queries. It seems to me like that config setting should apply to both.

Is someone able to make a PR here with suggested changes?


Fixing a typo in the spec

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

I am updating one letter. This is part of a live site issue causing multiple entries on the TOC:
https://docs.microsoft.com/en-us/rest/api/consumption/pricesheet

Did a commit to Azure/azure-sdk-for-python:
https://github.com/Azure/azure-sdk-for-python/commit/e60bc64dd48d808ce7e10082265d210cc45e497b

This commit was treated and no generation was made for Azure/azure-sdk-for-go

This commit was treated and no generation was made for Azure/azure-sdk-for-go

Swagger to SDK encountered an unknown error: (Azure/azure-sdk-for-go)
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/github_tools.py", line 29, in exception_to_github
    yield context
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/restapi/github.py", line 180, in rest_handle_action
    return rest_pull_close(body, github_con, restapi_repo, sdk_pr_target_repo, sdkbase)
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/restapi/github.py", line 311, in rest_pull_close
    rest_pr.create_issue_comment("Was unable to create SDK %s PR for this closed PR.", sdkid)
TypeError: create_issue_comment() takes 2 positional arguments but 3 were given


Versioning/Releasing things where the output of a package isn't the package itself

This isn't meant to be a let's implement a thing now issue, more a this is just some ideas, we probably won't do anything with for a bit issue.

@JedWatson and I were chatting last week about this.
The first thing I think is separating the parts of Changesets that are the concept of Changesets and the implementation of Changesets.

A contributor creates a changeset which declares their intent to release packages with semver bumps and a change summary
"Things" are versioned
"Things" are released somewhere

The implementation of Changesets adds on some things, most prominently I think they are:

Bumping dependents
Publishing to npm

These two things aren't necessary for publishing things where the output of a package isn't the package itself(sites/apps). A problem that we talked about for this as an example is the Keystone site, we have the problem that the site could be ahead of the packages that are published to npm so the information would be wrong for what's actually published - If we versioned the website and only published it when we did releases, we wouldn't have the problem. I think another use case is the style guides - we want to version the style guides and publish the style guide and its parts to npm but I think we could also "publish" from the "source" markdown in packages to the root README and etc.

Clarity thing - this is all hypothetically possible now right? As in, if things are set up as packages within the multi-package repo, they can/will be versioned? And if it depends on packages, updating those packages will cause versioning?
(The atlaskit website gets versioned all the time, but we don't do anything with that fact)
Technically to push changes you need to make your website (or other asset) have a dependency, possibly a fixed version dep, on the package, which could be made easier.

this is all hypothetically possible now right? As in, if things are set up as packages within the multi-package repo, they can/will be versioned? And if it depends on packages, updating those packages will cause versioning?

Yes. The main question is how releasing works. This probably won't be a thing in Changesets itself but I thought this would probably be the best place to start a discussion about this.

As a possible extension of this discussion, I am interested in using changesets in a monorepo with multiple languages (in this case, js and rust). Would it be possible to allow the configuration of an arbitrary command for each of the "lifecycles"? I think this potential solution that would cover both my case and the original issue.


tsconfig outDir is ignored.

While using a tsconfig.json located in /app/assets/ the setting outDir is fully ignored.

The way I interpret the README, this is intended. Support for tsconfig.json only exists in version 0.3.0 and up, but then there only exist two settings: configFile and sourceRoot. outDir only existed prior to 0.3.0.


U7: putting it all together

Content is split into three sections:

Speech-to-speech translation (combine speech translation + text-to-speech to map from speech in language X to speech in English)
Voice assistant (similar to Alexa or Siri: wake word detection, speech-to-text, LLM query, then text-to-speech)
Transcribe meetings (with text + speaker turns)

Hey @mishig25 - Build PR Documentation has failed a few times here, not sure exactly what the error is though: https://github.com/huggingface/audio-transformers-course/actions/runs/5488104466/jobs/10000477696?pr=84
Is it on my end or is there something we need to fix with the doc builder?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Support function activator workflow

Summary

Support function activator workflow
How did you test this change?

[x] Test(s) added
[x] Manual testing (please provide screenshots/recordings)
[ ] No testing (please provide an explanation)

https://github.com/finos/legend-studio/assets/73408381/d345c8b6-b8bd-4a50-89ed-ce7ed6e84782

More comments. and the next task is to move over to extension. we can work on that in the next PR.


Refactor Variable.addgrad test

Refactor Variable.addgrad test to use inject_backend_tests.
Preparation for supporting Chainerx (for #6983, #6857)

Jenkins, test this please.

Jenkins failure seem unrelated.

Jenkins CI test (for commit 6a8da137857049b24ee62f4c09b676a5b1f57bd5, target branch master) failed with status FAILURE.

Jenkins, test this please.

Jenkins CI test (for commit 6a8da137857049b24ee62f4c09b676a5b1f57bd5, target branch master) failed with status FAILURE.

LGTM!


Ensure webhook is started

This ensures webhook is started when controller-manager deployment becomes ready.

/test ovn-operator-build-deploy-kuttl


Fixes #2391 cleanup merged closed prs

Description

Auto-removes branches linked to merged or closed pull-requests on a cron.
Checklist Before Merging

[ ] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[ ] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[ ] My commits or Pull-Request Title contain my JIRA information, or I do not have a JIRA.

I think it is possible to have the license checker ignore specific files, though, if for some reason you don't want to add the license header to the yaml file you're checking in here


Templates - Custom Templates window: 'Save to SDF...' button is in wrong position

Steps to Reproduce

Open Ketcher.
Click the 'Custom Templates' button at the bottom.

Expected behavior
'Save to SDF...' button is displayed on the right.

Actual behavior
'Save to SDF...' button is displayed on the left.

Desktop:
OS: Windows
Browser: Chrome Version 93.0.4577.63, Firefox Version 92.0, Microsoft Edge Version 93.0.961.47

Release/2.4 Closed as fixed.


Add an addition wait to fix race in Web socket connect test

Add an addition wait to fix race in Web socket connect test

@DDEfromOR also I did do a small fix in the product code last weekend - it was to grab the Task from the TaskCompletionSource before it was ever possible for it to be set to null - the race condition on connect caused the occasional null exception

Thanks @johnataylor !


Automation?

Hi, is there a way we can activate roles with Powershell or the REST api? Our users would like to be able to do this from the command line. We have 2-factor auth with the authenticator app when activating PIM roles.

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

ID: 3a831056-b759-8539-ca86-9fbb84f331d7
Version Independent ID: 8fca63cc-bd36-d774-1f31-eca19502b006
Content: Activate roles for Azure resources by using Privileged Identity Management
Content Source: articles/active-directory/privileged-identity-management/pim-resource-roles-activate-your-roles.md
Service: active-directory
GitHub Login: @rolyon
Microsoft Alias: rolyon

@mikeoleary Thanks for your feedback! We will investigate and update as appropriate.

@mikeoleary Please note that you can use Microsoft graph API or Powershell for PIM. Please refer to below links to check available commands.
Powershell - Microsoft.Azure.ActiveDirectory.PIM.PSModule
Graph API - Azure AD Privileged Identity Management

Great to know, thanks Mohit.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Mohit Garg<EMAIL_ADDRESS>Sent: Wednesday, July 11, 2018 1:31:02 PM
To: MicrosoftDocs/azure-docs
Cc: mikeoleary; Mention
Subject: Re: [MicrosoftDocs/azure-docs] Automation? (#11550)
@mikeolearyhttps://github.com/mikeoleary Please note that you can use Microsoft graph API or Powershell for PIM. Please refer to below links to check available commands.
Powershell - Microsoft.Azure.ActiveDirectory.PIM.PSModulehttps://www.powershellgallery.com/packages/Microsoft.Azure.ActiveDirectory.PIM.PSModule/<IP_ADDRESS>3
Graph API - Azure AD Privileged Identity Managementhttps://developer.microsoft.com/en-us/graph/docs/api-reference/beta/resources/privilegedidentitymanagement_root
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/11550#issuecomment-404250030, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AlC5_Mdz570vlE-KIyDLVqWCgz7mgabzks5uFjZWgaJpZM4VLJfP.

@mikeoleary We will now proceed to close this thread. If there are further questions regarding this matter, please open a new issue and we will gladly continue the discussion.


Modify the comment copy of the approval process

Modify the comment copy of the approval process

Codecov Report

Merging #154 (142703a) into main (f9fc17b) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff            @@
##               main     #154   +/-   ##
=========================================
  Coverage     65.49%   65.49%
  Complexity      525      525
=========================================
  Files           162      162
  Lines          4509     4509
  Branches        586      586
=========================================
  Hits           2953     2953
  Misses         1421     1421
  Partials        135      135

Flag
Coverage Δ

*Carryforward flag

api_ut
54.12% <ø> (ø)

Carriedforward from f9fc17b

server
70.90% <ø> (ø)

Carriedforward from f9fc17b

ui
94.51% <ø> (ø)

*This pull request uses carry forward flags. Click here to find out more.


projects dependencies

the lastest version can support  one project dependencies other projects ?  how to configure?

good question. but diffcult to make it .


updated deploy scripts

@dominiek ok added docs for the new scripts and brushed up documentation a bit as well


docs(order): fix bad inline config

alphabetize option in order rule, does not permit boolean

Coverage remained the same at 97.738% when pulling 4050030af526a2b0004a7ac4fbd16e596c2bb477 on nickofthyme:patch-1 into 3f46ccfe203fa5aad5767699df339a9b2f0294e3 on benmosher:master.

Coverage remained the same at 97.738% when pulling 4050030af526a2b0004a7ac4fbd16e596c2bb477 on nickofthyme:patch-1 into 3f46ccfe203fa5aad5767699df339a9b2f0294e3 on benmosher:master.

Coverage remained the same at 97.738% when pulling 4050030af526a2b0004a7ac4fbd16e596c2bb477 on nickofthyme:patch-1 into 3f46ccfe203fa5aad5767699df339a9b2f0294e3 on benmosher:master.

Coverage remained the same at 97.738% when pulling 4050030af526a2b0004a7ac4fbd16e596c2bb477 on nickofthyme:patch-1 into 3f46ccfe203fa5aad5767699df339a9b2f0294e3 on benmosher:master.


Embedded h5p elements from Twillo not properly loaded

When embedding h5p elements from Twillo, there are some icons not shown or sometimes the file cannot be loaded at all. On Twillo, the elements look fine.

Hi @JuliaHeiken ... could you provide a small code-example, how you embed the H5P elements? Either here as a markdown-code-block or you create an example at
https://liascript.github.io/LiveEditor/
and then share it by clicking on menu and then on snapshot-url ... this way, the URL will contain the entire content ...

Sure, for example
<iframe src="https://www.twillo.de/edu-sharing/eduservlet/render?node_id=291a02e3-2fd0-4276-b49d-37c205fe945d" width="100%" height="840" frameborder="0" allowfullscreen="allowfullscreen" title="Research Data Management in the Energy Sector"></iframe>

You can try the following, add a persistent instruction to the global header-comment or locally per slide, where you add an IFrame ...

You can try this out here: LiveEditor

I hope this fixes your issue?

Unfortunately, it doesn't. I think the issue is browser-dependent. The issue appears in chrome, but I just tried in Firefox and there it's working.

Ah, I see ... These little icons are actually fonts:

The problem you are facing is related to how different browsers handle the loading of web fonts from an iframe. Firefox seems to be able to handle the situation gracefully and only shows a warning, while Chrome fails with a 400 Bad Request error.
You could either try to use real images for this purpose or you have to ask your administrator to change the CORS settings for this case. This link should help solve the issue:
https://stackoverflow.com/questions/33197751/fonts-are-blocked-in-web-client-cors


在运行项目时出现的错误

Compiler message:
/F:/Flutter/flutter/.pub-cache/hosted/pub.flutter-io.cn/dio-2.2.2/lib/src/dio_http_headers.dart:55:8: Error: The method 'DioHttpHeaders.add' has fewer named arguments than those of overridden method 'HttpHeaders.add'.
void add(String name, value) {
^
org-dartlang-sdk:///third_party/dart/sdk/lib/_http/http.dart:694:8: Context: This is the overridden method ('add').
void add(String name, Object value,
^
/F:/Flutter/flutter/.pub-cache/hosted/pub.flutter-io.cn/dio-2.2.2/lib/src/dio_http_headers.dart:70:8: Error: The method 'DioHttpHeaders.set' has fewer named arguments than those of overridden method 'HttpHeaders.set'.
void set(String name, Object value) {
^
org-dartlang-sdk:///third_party/dart/sdk/lib/_http/http.dart:703:8: Context: This is the overridden method ('set').
void set(String name, Object value,
^
Compiler message:
/F:/Flutter/flutter/.pub-cache/hosted/pub.flutter-io.cn/dio-2.2.2/lib/src/dio_http_headers.dart:55:8: Error: The method 'DioHttpHeaders.add' has fewer named arguments than those of overridden method 'HttpHeaders.add'.
void add(String name, value) {
^
org-dartlang-sdk:///third_party/dart/sdk/lib/_http/http.dart:694:8: Context: This is the overridden method ('add').
void add(String name, Object value,
^
/F:/Flutter/flutter/.pub-cache/hosted/pub.flutter-io.cn/dio-2.2.2/lib/src/dio_http_headers.dart:70:8: Error: The method 'DioHttpHeaders.set' has fewer named arguments than those of overridden method 'HttpHeaders.set'.
void set(String name, Object value) {
^
org-dartlang-sdk:///third_party/dart/sdk/lib/_http/http.dart:703:8: Context: This is the overridden method ('set').
void set(String name, Object value,
^
Target kernel_snapshot failed: Exception: Errors during snapshot creation: null
build failed.
FAILURE: Build failed with an exception.

Where:
Script 'F:\Flutter\flutter\packages\flutter_tools\gradle\flutter.gradle' line: 886

What went wrong:
Execution failed for task ':app:compileFlutterBuildDebug'.

Process 'command 'F:\Flutter\flutter\bin\flutter.bat'' finished with non-zero exit value 1

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 22s
Exception: Gradle task assembleDebug failed with exit code 1

你Flutter什么版本

你Flutter什么版本

谢谢 我已经解决这个问题  回退版本就可以了  之前版本太高了

flutter v1.17.1 会报这个错误

问题解决了把 Issues 关掉呀

@KingOn3 how to did you solve it, what versions are you using?

其实这个改dio版本就行吧


Set GOCACHE environment value within THUMBAI

@radutopala reported an issue related with GOCACHE env value
2018-12-20 20:34:17.396 INFO  THUMBAI Executing /usr/local/go/bin/go mod download
2018-12-20 20:34:17.414 ERROR THUMBAI go: cannot use modules with build cache disabled

This was done in the commit 9fe5c0e1fadaa939fe176fe23e8374fef205cd93, so closing it 😄

@radutopala Thumbai would setGOCACHE value automatically.
GOCACHE directory is used for go build cache and it is different from mod cache. So pointing to $GOPATH/pkg/mod/cache would pollute the Mod cache directory.
So, Thumbai would derive GOCACHE value as follows -

First via go env GOCACHE
If any errors from above then it would set it $GOPATH/pkg/go-build-cache


Modal Title Bar Text Color Should be White

From a consistency standpoint the header should have a while text color if the background is blue.

My opinion is that text color should be white for every color case on modal, even with black.

Fixed by https://github.com/DigitalRiver/react-atlas/pull/505


Correction for raw affiliation Hakai Institute, British Columbia, Canada

Correction needed for raw affiliation Hakai Institute, British Columbia, Canada
raw_affiliation_name: Hakai Institute, British Columbia, Canada
new_rors: 02pry0c91
previous_rors: 01a583453
works_examples: W2424161488
contact: 746abd07bad390ccf1cb909cc228f056:6f01aaa6f6bd6444e414 @ recherche.gouv.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.


Update bar heading and done buttons

Reviewer: Caine
Asana:
• https://app.asana.com/0/361428290920652/379553182484189
• https://app.asana.com/0/361428290920652/379475105160937
Description:
Add settings heading and move done buttons to the right side of the screen
Steps to test this PR:

Open settings and note that it now has a heading and the done button is on the right side of the screen
Open bookmarks and note that the done button is on the right side of the screen

Reviewer Checklist:

[ ] Ensure the PR solves the problem
[ ] Review every line of code
[ ] Ensure the PR does no harm by testing the changes thoroughly
[ ] Get help if you're uncomfortable with any of the above!
[ ] Determine if there are any quick wins that improve the implementation

PR DRI Checklist:

[ ] Get advice or leverage existing code
[ ] Agree on technical approach with reviewer (if the changes are nuanced)
[ ] Ensure that there is a testing strategy (and documented non-automated tests)
[ ] Ensure there is a documented monitoring strategy (if necessary)
[ ] Consider systems implications (Database connections, Grafana stats, CPU)

Approved by Caine over hipchat


support edge tls termination

For deployment in kubernetes/openshift and similar environment it makes sense to support edge tls termination: Hydra would not terminate TLS itself but trust HTTP headers (e.g. X-Forwarded-Protocol).

right now you can force hydra to disable https using --force-dangerous-http

HTTP headers shouldn't really be trusted unless those security sensitive parameters can't be spoofed. we could add something like trust-http-headers or edge-tls-termination but

you can already disable https with --force-dangerous-http
X-Forwarded-Protocol is not a standard header and might vary from proxy to proxy. AWS for example uses X-Forwarded-Proto

I am aware of the security implications of trusting headers: I am not advocating blindly trusting them. In a way, an edge-tls-termination flag is different from force-dangerous-http, because hydra would still generate only https-links. Also logging the right IP (from e.g. X-Forwarded-IP) is only necessary then.
(BTW: Of course, all X- headers are non-standard. Unfortunately the standard PROXY header support is not widespread yet).

an edge-tls-termination flag is different from force-dangerous-http, because hydra would still generate only https-links.

This is a very good point and I classified it as a bug.

(BTW: Of course, all X- headers are non-standard. Unfortunately the standard PROXY header support is not widespread yet).

How could we solve this issue then? My best bet right now is a flag like --enable-edge-tls-termination. What would you suggest?

I believe that right now the only piece of code that would be affected by this issue is the consent challenge

I prefer an option like "--edge-tls-termination-from" where you can specify one or more netmasks from which you trust the headers.
But I am fine with any way that makes it possible to use it behind a tls terminating router.
In a way, I like using RFC 7239, the FORWARDING header (confused above with PROXY), but it's not as straightforward to parse and not very widespread.
And don't you log IP addresses somewhere? That part will most likely benefit by inspecting the headers and logging the client's IP.

Logging HTTP requests is usually the task of the proxy/gateway/loadbalancer. However, it should be fairly easy to add a middleware (e.g. negroni) for it. If you think this is a neccessity, please open another issue for it.
I think it's smart to let hydra detect if the original request was HTTPS or not. I checked out RFC7239 and some resources online, it looks like X-Forwarded-Proto is a de-facto standard. So why not use it!
Additionally, I like the idea to whitelist tls termination origins: --edge-tls-termination-from=[<IP_ADDRESS>/16, ...]

@ibotty what do you think of the following:
hostCmd.Flags().StringSlice("allow-tls-termination-from", []string{}, "Whitelist one or multiple CIDR address ranges and allow them to terminate TLS connections. Be aware that the `X-Forwarded-Proto` header must be set and must never be modifiable by anyone but your proxy / gateway / load balancer.")

do you have an idea how to describe this better in fewer words?

I like it. I can't think of anything shorter which is as precise.

Instead, this is an env var! Makes more sense for production environments IMHO

- HTTPS_ALLOW_TERMINATION_FROM: Whitelist one or multiple CIDR address ranges and allow them to terminate TLS connections.
	Be aware that the X-Forwarded-Proto header must be set and must never be modifiable by anyone but
	your proxy / gateway / load balancer.
	Example: HTTPS_ALLOW_TERMINATION_FROM=["<IP_ADDRESS>/16","<IP_ADDRESS>/16"]

https://github.com/ory-am/hydra/pull/165/commits/53b2f830436f8c1b2fcd43e1dec4abdc8de17b36

I hope it is parsed as a comma-separated list, without the brackets. If so, the example should read
 Example: HTTPS_ALLOW_TERMINATION_FROM=<IP_ADDRESS>/16,<IP_ADDRESS>/16

btw: what about ipv6?

hope it is parsed as a comma-separated list, without the brackets. If so, the example should read

done

btw: what about ipv6?

will be included. do you know if there's something i need to take care of when using go and ipv6?

Sorry, I don't know anything about go and IPv6.


improve Dependabot Alert UI; allow markdown comments

This is regarding the User Interface.
In my comment to close a Dependabot Alert, I created a markdown comment. That comment was saved as a plain-text comment with newlines removed. My closure explanation is difficult to read.
I would like to be able to create a normal markdown comment when closing a Dependabot Alert.
The closed Alert is here https://github.com/jtmoon79/super-speedy-syslog-searcher/security/dependabot/3
Screenshot at https://imgur.com/a/2Hvitg7

Dependabot Alerts PM here 👋. I've forwarded this to our design team to get their thoughts!

+1 to this feature request; there's usually blocking issues/tickets/ongoing discussions and being able to referencing them in the Dependabot UI via simple markdown comments (like the standard Github PR commenting UI) would be extremely useful (especially for enterprise GH customers)


Fix all auto-fixable Problems doesn't yield the same result as eslint file.js --fix

I have configured ESLint for an already written project, but the problem is when I click Fix all auto-fixable Problems it doesn't do a great job of fixing indents whereas running eslint file.js --fix does it well (to be precise sometimes after two times of running it ).
Below is a code sample with which Fix all auto-fixable Problems doesn't work well:
    <LanguageContext.Consumer>
        {(value) => {
            const vocabs = getTranslator(value.language);
            return (
                <Grid container direction="row"
                      justify="center" alignItems="center">
                    <ButtonGroup size={"small"}>
                        <Button variant={props.authFormState === 'login' ? 'contained' : 'outlined'} color='primary'
                                className={props.authFormState === 'login' ? classes.disabledButtonRight : classes.switchButtonRight}
                                onClick={changeFormStateToLogin}>
                            {vocabs('login')}
                        </Button>
                        <Button variant={props.authFormState === 'register' ? 'contained' : 'outlined'} color='primary'
                                className={props.authFormState === 'register' ? classes.disabledButtonLeft : classes.switchButtonLeft}
                                onClick={changeFormStateToRegister}>
                            {vocabs('register')}
                        </Button>
                    </ButtonGroup>
                </Grid>
            )
        }}
    </LanguageContext.Consumer>

This got addressed for the next version of ESLint. Pls see https://github.com/microsoft/vscode-eslint/issues/815 if you want to help test this.

2.0.4 now shipped.


Adding the bank balance to the account header

I reconcile often.  To reconcile, I need the actual bank balance to check that it matches YNAB before reconciling.  If I've been careful, I can reconcile without opening the bank's web page.
Would it be possible to add "Imported Balance" to the current "Cleared Balance", "Uncleared Balance", and "Working Balance"?

@hmspain how do you define "imported balance" and how would you use it?

When you reconcile an account, it is the number you are trying to match.  The toolkit may not have access to this number.  Perhaps it is only made available to YNAB when they import (but they don't use it).
If I've cleared the appropriate transactions in YNAB, my YNAB balance should match the bank's "available balance".
By showing this in ynab, I don't have to go into my bank's web page to confirm the balance.

Would this just be the current reconciled balance?
As far as I can tell there are three numbers in the account header:
Cleared Balance - Everything that is locked + All "Cleared" transactions marked green
Uncleared Balance - Sum of everything that's not been cleared or reconciled
Working Balance - (Cleared balance - uncleared balance)
If you just want to show the current reconciled amount (excluding cleared and uncleared transactions) then you may benefit from https://github.com/toolkit-for-ynab/toolkit-for-ynab/pull/2165
Show Reconciled Balance which shows the sum of all locked transactions.

The "imported balance" is often reported as the "current balance" or the "available balance".  It's the number you want to match when you reconcile a YNAB account.

Another way of looking at this would be to say;
Cleared Balance = YNAB Cleared Balance
Uncleared Balance = YNAB Uncleared Balance
Working Balance = YNAB Working Balance
Imported Balance = BANK Balance

Unfortunately (kinda), this is not available for us to see. Direct import is something that YNAB does on the backend probably on some sort of job that runs daily. When they run this, I don't think they even care about the actual balance of your account, they're probably only getting transactions and creating YNAB transactions that are the same. Because of this, the only thing we can see is that a transaction is one they created as an "imported" transaction and we never actually know info about the account balance.
Though it's unfortunate we can't implement what you're asking about (at this time) I will say it's nice to see that only the pertinent information makes it from a bank account to the browser 😄

Thank you Josh!  I thought this was the case, but was not sure :-).  Let's hope YNAB sees the benefit :-).

On Oct 26, 2020, at 9:34 PM, Josh Madewell<EMAIL_ADDRESS>wrote:

Unfortunately (kinda), this is not available for us to see. Direct import is something that YNAB does on the backend probably on some sort of job that runs daily. When they run this, I don't think they even care about the actual balance of your account, they're probably only getting transactions and creating YNAB transactions that are the same. Because of this, the only thing we can see is that a transaction is one they created as an "imported" transaction and we never actually know info about the account balance.
Though it's unfortunate we can't implement what you're asking about (at this time) I will say it's nice to see that only the pertinent information makes it from a bank account to the browser 😄
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Top-level await makes Nuxt3 app crash when using NPM import

Can't seem to be using Lucid via npm with Nuxt3 because of the following error:
Top-level await is not available in the configured target environment

Tried adapting targets to no avail. Any idea of a workaround?

I am not an expert in Nuxt, however are you sure you have to "await" the imports? What happens if you try to remove the "await",  are those actually dynamic imports? Please update.

Well, those are in the library itself, not in my code :/


Rename W2DImage to PlatformImage

With this changes we can avoid this issue:

@jonlipsky Why was the W2DImage an internal class?
Fixes #457
Fixes #402

We can't do this as it will cause issues. We need to make the namespaces consistent as well for net7 or something. This is also a breaking change for Maui. It has to wait for net7


[question] how to fit plyer to outer div size

Hi
this is more of a question (either css ot plyr related). please see my example pen: http://codepen.io/guykatz/pen/LbOrgE
my container div has a "strange ratio" and an overflow hidden. playr overflows and I cant see the controls and the lower part of the movie is clipped.
adding the css:
.container .plyr{
height:100%;
}
will make the controls to come back byt the movie is still overflowing.
any suggestions?
thanks.

You don't need to set the height. If you want responsiveness, you should also just use max-width. Not sure you need overflow: hidden either.
If you need to set the height and width then you need to do the 16:9 trick:
https://css-tricks.com/NetMag/FluidWidthVideo/Article-FluidWidthVideo.php

thanks.
I have a flexbox layout with tiles like items. there are flex items with 25% width and overlow hidden.
the overflow is so that the layout is not interupted - this is very common.
inside one of the tiles I want to put plyer. I do not control the tile width as it changes from one screen resolution to the other. also, I do not know the ratio of the movie I am going to load.
I want to keep my such that it fits inside my tile.
my example was simplifying the situation. i'll gladly write a new pen with the whole layout.
is your answer still the same? I mean, it would be nice if plyer 'new' the parent's dims and fit itself such that the movie is not cropped.
if there is no plyer feature for this maybe there should be?
thanks!

I'd try that first or perhaps try using flexbox if you need to set the height.

wow that was fast - thanks!
I will play around with it and worse case scenario if I still am in trouble I will write a new pen and write another reply here.
thanks again.


A Tutorial Guide for Crypt Beginners

If a beginner is using Ethereum for the first time, they are often confused about what to do when they visit Stakes.social.
I think we need a tutorial guide for beginners so that they can use Stakes.social with ease. That page will work for "read this first."
What kind of content do we need for that page? I think it would be very helpful if interactions such as wallet connection could actually be manipulated. What kind of interactions do we need for that page?

I think the idea to solve this is to change the content posted on the TOP page of Stakes.social.
First of all, I would like to submit the design of the TOP page that I think


Ability to get many items (aka. getItems())

After doing some performance measurements on my app I notice I suffer a lot from doing this:
localForage.getItem('all_ids', function(ids) {
    _.each(ids, function(id) {
        localForage.get(id, function(value) {
            ...
        });
    })
})

This is causing a lot of excess work for the browser if the list is long (e.g. 1,000+).
I know the localStorage spec doesn't have support getItems() but in most cases IndexDB is supported and it appears to have low level support for getting multiple keys which could hopefully reduce the overhead in total.
For the localStorage fallback, this would be trivial to wrap up with a for loop of kinds.

If something like this was to be implemented, how the return object should look like?

Array of KeyValues

[{
  key:'test1',
  value: 'value1'
}, {
  key:'test2',
  value: 'value2'
}]

or just an object like

{
  'test1': 'value1',
  'test2': 'value2'
}

???
What's most preferable of the two in your opinion?
What are the pros and cons in each case in your opinion?
For me:
+1 for obj result
obj pros:

It is more straightforward/efficient in retrieving a specific value.
array pros
Someone might argue that the array is more convenient for iterating over the results.

To iterate over the results they can always use Object.keys, I think the second (object) one is nicer as it’s a basic return object that can be extended but is easy to grab items out of. Using the array the syntax is more annoying if you know exactly what you want. It would also mean we’d either need to guarantee order or you’d have to iterate through to get a key.
Object for sure!
-tofumatt
On 10 May 2015 at 10:24:45, Thodoris Greasidis<EMAIL_ADDRESS>wrote:
If something like this was to be implemented, how the return object should look like?
Array of KeyValues
[{
key:'test1',
value: 'value1'
}, {
key:'test2',
value: 'value2'
}]
or just an object like
{
'test1': 'value1',
'test2': 'value2'
}
???
What's most preferable of the two in your opinion?
What are the pros and cons in each case in your opinion?
For me:
+1 for obj result
obj pros:
It is more straightforward/efficient in retrieving a specific value. array pros
Someone might argue that the array is more convenient for iterating over the results.
—
Reply to this email directly or view it on GitHub.

@tofumatt I was just wandering if anyone could report any edge case or strange behavior when setting any special/reserved key as a property to an object.
PS: I also like for..in with Object.hasOwnProperty to support pre-es5 browsers. Hope it has the same results.

I can’t think of too many and I’d think it would crop up before saving to localForage though… it would be a problem in the data BEFORE we did a get/set. I think ^_^
-tofumatt
On 10 May 2015 at 11:27:24, Thodoris Greasidis<EMAIL_ADDRESS>wrote:
@tofumatt I was just wandering if anyone could report any edge case or strange behavior when setting any special/reserved key as a property to an object.
PS: I also like for..in with Object.hasOwnProperty to support pre-es5 browsers. Hope it has the same results.
—
Reply to this email directly or view it on GitHub.

FYI: Chrome is about to implement IDBObjectStore.getAll.
https://code.google.com/p/chromium/issues/detail?id=457450

Oh, that is awesome!

Can you give a try to thgreasi/localForage-getItems ?
I thought it would be better for developing and reviewing if it was written (initially, until merged) as an extension library for localforage.
The jsperf testcases (found in README) gave me mixed results.
For big enough number of requested keys the gains are pretty good.
For small number of requested keys, chrome does a pretty god job parallelizing the independent read-only requests. As a result the generic method is sometimes faster.
The generic method is based on the code described above and it should be the default implementation for any driver that can't offer a better (driver specific) implementation (eg localstorage).

This is probably not the best place to discus it, but I have also been experimenting with a startsWith implementation.

whats the status of this feature?

It's not available yet, but we'll update the issue when it is. I'm not sure anyone is working on it right now.

You can use the aforementioned plugin if you need it now.

@tofumatt The plugin itself was structured in a way that would ease the creation of a PR. I could open one after the holidays if we want something like this as part of the main library. In order to reduce duplication, we could wire up the existing getItem method to use the composite version with a single item as a parameter.
On the other hand setItems gave me much better performance gains than getItems.
It seems that doing concurrent getItem requests and composing them with Promise.all() works faster than fetching the items in a single transaction when there is no one else writing on the db at the same time.

I do like the plugin way, I’m starting to feel like we can strike a balance on library size and features. For now I’d recommend using the plugin and I'll think about whether we want to merge this in directly.

:+1: on the plugins... that way we can push new experimental features
As a side note & correction on my earlier performance related comments,
I just had a rerun of the jsperf benchmarks and it seems that the current versions of Firefox (v44) and Chrome (v47) give better result on getItems() than multiple generic concurent getItem() calls.

Yes, it would be convenient.
Please consider adding that feature. :godmode:


⚠ Samsung has degraded performance

In b693baa, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2689 ms

Resolved: Samsung performance has improved in 86c79e0.


Usability issue in infinite scrolling

i have select2 with infinite scrolling,
the problem is whenever select2 fetches more results from remote data-source the highlight is shifted to the first element in the list instead of highlighting the next element in list
for repo : on your examples page

try Loading remote data example
enter some keyword
use down arrow key to load more data and go to last element so that it will load more data
the focus goes to first element of drop down after loading new data

any update on this?

I ran into the same problem. We currently use version 3.5 and we have plans to migrate to version 4, but in order to do that it must support "keyboard-only" model well, so this behavior "as is" will not work for us.
So I did some investigation and it appears to me that the problem is with setClasses function. It does not follow "separation of concerns" principle: instead of just setting classes on newly loaded items like its name suggests it does also change highlighted item (lines after $options.each() cycle). This leads me to three points:

Changing highlighted item is definitely a separate concert and it should not happen here, regardless of considering anything else;
"results:append" event handler (through setClasses() call or by any other means) should not change highlighted item in dropdown at all. It should still call setClasses() though to update "aria-selected" on any newly loaded items if they've already been selected before current data loading;
For performance reasons it should probably go through only newly loaded items, not all of them, cause previously loaded data had already been processed on previous data loads and user actions. Even if personally I didn't see any performance problems yet it doesn't mean that it should not be optimized.

So my conclusion is that setClasses() definitely should be rewritten.
Kevin - what do you think? I can prepare pull request to fix it if it works for you.

Kevin - what do you think? I can prepare pull request to fix it if it works for you.

If you can think of a better way to handle setClasses, I'm all for it. I'll gladly review any pull requests that are sent in.


Adding resources

Adding "Data Structures and Algorithms - Self Paced", "Data Structures Easy to Advanced Course-FreeCodeCamp" in CS Resources and "100 Days of Machine Learning", "Machine Learning Course by Andrew NG" in Machine Learning Resources.

Great! Seems good
You need to add 5 new resources for hacktoberfest


Update METADATA.yaml to list @enriqe as facilitator

/to @Enriqe, @ericfs, or @mrjoro for approval


Convert Windows Services topic to use Powershell

Fixes #10833
Thanks @pherbel ... I used your ACL bits.
Notes:

We need PS6 for this: Not only is it the latest, the Remove-Service cmdlet didn't become available until PS6, and that's a nice-to-have here.

WRT granting Log on as a service rights to the user: We can't use the UserRights script:

That script doesn't work in PS6.
It's not supported by MS anyway.

However, we certainly can use a security template as I do here. @blowdart will be The First Decider on what we ultimately do.

There's no OOB way AFAICT to create a local user in PS6. For example, the New-LocalUser command of PS5.1 doesn't work. I stick with the net user command here.

As I don't know powershell that well @TravisEz13 might have some thoughts about it.

@blowdart What about the use of the security template and secedit.exe command to give the user Log on as a service privileges? It's supported and it's easy to show-'n-tell for the reader to perform. There's no supported PS way to do it ... not as far as I can tell anyway.

That's certainly novel :)

Well ... it's not as bad as my usual 😝 RexHacks 😝 around here. @shirhatti may have one or more alternatives. This is fairly easy to grok/perform tho. Seems to work well.

New-LocalUser should work in PS6.2 on windows.

I see ... I was using 6.1.3. They seem to be saying that that's the latest. Is that correct? Is 6.2 in preview?

Yeah ... I see it. It's in RC. I'll open an issue to replace that bit when 6.2 RTMs.

I just verified that New-LocalUser works at least when you don't let it prompt for a password.  I'll have to open an issue on the prompt, but that may never get fixed.

Can you point me to the UserRights script?

Cool ... I'll take care of swapping for the net user command when 6.2 RTMs.
Here it is ... https://gallery.technet.microsoft.com/scriptcenter/Grant-Revoke-Query-user-26e259b0
It broke when I tried to use it. However, it isn't supported (unless management wants to give me a pass here ... to cover my b-hind :smile:).

I think that script can be fixed (not making any claims about supportability or maintainability.  Actually, I believe it has similar issue to why New-LocalUser had to be pulled from PowerShell 6)
It's actually all C# code that is broken (So far), which is being compiled at runtime in PowerShell.  This is not a great design for cross framework compatibility.
I'm not sure I have time, but if someone want to work on it, my recommendation would be to convert it to a Net/PowerShell Standard/Windows Compat based C# PowerShell module.  This would work on Windows PowerShell (in Windows 10) and PowerShell Core.

Thanks @TravisEz13
I'll have an :ear: open for @shirhatti's feedback.

Ping @shirhatti - This one is also fairly short.
The main concern with the updates here is granting the Log on as a service right to the service account.
This PR includes a move from icacls.exe to secedit.exe (with a security template), which is fairly pointless. Now that I've :zzz:🛌:zzz: on it, I like the icacls.exe approach better.
UserRights doesn't work with PS6 and imo presents significant security risks.
Does engineering want to come up with a PS script for this step (akin to the UserRights approach) that we can trust and include with the topic? Want me to hack something together as a first pass? ... good for a laugh anyway :smile:.

Maybe I missed the memo on this, but why are we converting this doc to use Powershell?

https://github.com/aspnet/Docs/pull/10837#pullrequestreview-201724257
We can close this if you want. In addition to the inability to grant Log on as a service, New-LocalUser doesn't return until 6.2 RTM (it's RC right now; I opened https://github.com/aspnet/Docs/issues/11344 to cover it). If we do close the PR, do you want to close the issue, as well, or do you just want me to push the issue to the Backlog? (If we close both the PR and the issue, I won't bill MS for any of this work.)

Since @blowdart asked for it, I'll let him make the call.

@blowdart Ur call ... which way do u want to go? ...

icacls.exe
secedit.exe (with a security template)

I'm leaning back toward what we have in the topic today (icacls.exe) just because it doesn't require the extra template bit. It's just a command that they can execute in the shell.

... and just note to all that we're not going to make a PS script today for the Log on as a service permission, but I would like to reconsider it in the future. I'm going to open a backlog issue (P3) so that we can take another look down the road at it.

icacls please


Integration tests: No tree view with id 'test.treeId' registered

Running integration tests, I think from
https://github.com/microsoft/vscode/blob/376716d1aeef7f64002068308ea45a91fab25a96/extensions/vscode-api-tests/src/singlefolder-tests/rpc.test.ts#L87-L87
No tree view with id 'test.treeId' registered.: Error: No tree view with id 'test.treeId' registered.
    at ExtHostTreeViews.$hasResolve (/Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/api/common/file:/Users/bpasero/Development/Microsoft/monaco/src/vs/workbench/api/common/extHostTreeViews.ts:131:10)
    at RPCProtocol._doInvokeHandler (/Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/services/extensions/common/file:/Users/bpasero/Development/Microsoft/monaco/src/vs/workbench/services/extensions/common/rpcProtocol.ts:417:17)
    at RPCProtocol._invokeHandler (/Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/services/extensions/common/file:/Users/bpasero/Development/Microsoft/monaco/src/vs/workbench/services/extensions/common/rpcProtocol.ts:402:32)
    at RPCProtocol._receiveRequest (/Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/services/extensions/common/file:/Users/bpasero/Development/Microsoft/monaco/src/vs/workbench/services/extensions/common/rpcProtocol.ts:318:19)
    at RPCProtocol._receiveOneMessage (/Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/services/extensions/common/file:/Users/bpasero/Development/Microsoft/monaco/src/vs/workbench/services/extensions/common/rpcProtocol.ts:245:10)
    at /Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/services/extensions/common/file:/Users/bpasero/Development/Microsoft/monaco/src/vs/workbench/services/extensions/common/rpcProtocol.ts:110:42
    at Emitter.fire (/Users/bpasero/Development/Microsoft/monaco/out/vs/base/common/file:/Users/bpasero/Development/Microsoft/monaco/src/vs/base/common/event.ts:622:16)
    at BufferedEmitter.fire (/Users/bpasero/Development/Microsoft/monaco/out/vs/base/parts/ipc/common/file:/Users/bpasero/Development/Microsoft/monaco/src/vs/base/parts/ipc/common/ipc.net.ts:468:19)
    at /Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/services/extensions/node/file:/Users/bpasero/Development/Microsoft/monaco/src/vs/workbench/services/extensions/node/extensionHostProcessSetup.ts:198:22
    at Emitter.fire (/Users/bpasero/Development/Microsoft/monaco/out/vs/base/common/file:/Users/bpasero/Development/Microsoft/monaco/src/vs/base/common/event.ts:622:16)
    at BufferedEmitter.fire (/Users/bpasero/Development/Microsoft/monaco/out/vs/base/parts/ipc/common/file:/Users/bpasero/Development/Microsoft/monaco/src/vs/base/parts/ipc/common/ipc.net.ts:468:19)
    at PersistentProtocol._receiveMessage (/Users/bpasero/Development/Microsoft/monaco/out/vs/base/parts/ipc/common/file:/Users/bpasero/Development/Microsoft/monaco/src/vs/base/parts/ipc/common/ipc.net.ts:821:22)
    at /Users/bpasero/Development/Microsoft/monaco/out/vs/base/parts/ipc/common/file:/Users/bpasero/Development/Microsoft/monaco/src/vs/base/parts/ipc/common/ipc.net.ts:660:73
    at Emitter.fire (/Users/bpasero/Development/Microsoft/monaco/out/vs/base/common/file:/Users/bpasero/Development/Microsoft/monaco/src/vs/base/common/event.ts:622:16)
    at ProtocolReader.acceptChunk (/Users/bpasero/Development/Microsoft/monaco/out/vs/base/parts/ipc/common/file:/Users/bpasero/Development/Microsoft/monaco/src/vs/base/parts/ipc/common/ipc.net.ts:241:21)
    at /Users/bpasero/Development/Microsoft/monaco/out/vs/base/parts/ipc/common/file:/Users/bpasero/Development/Microsoft/monaco/src/vs/base/parts/ipc/common/ipc.net.ts:202:51
    at Socket.listener (/Users/bpasero/Development/Microsoft/monaco/out/vs/base/parts/ipc/node/file:/Users/bpasero/Development/Microsoft/monaco/src/vs/base/parts/ipc/node/ipc.net.ts:32:38)
    at Socket.emit (events.js:315:20)
    at addChunk (_stream_readable.js:295:12)
    at readableAddChunk (_stream_readable.js:271:9)
    at Socket.push (_stream_readable.js:212:10)
    at Pipe.onStreamRead (internal/stream_base_commons.js:186:23)

@jrieken, I am seeing the following error in the OSS console and I believe that it is related to this fix:
vscode.vscode-api-tests]: property name is mandatory and must be of type string

Thanks for that @lszomoru. Seems like the package.json isn't validated properly


'Create Issue' option in the editor's context menu

It would be wonderful if the option to create an issue in the tracker were available on the editor's context menu. This would allow creating issues without switching to the TODOs window, which can significantly improve user experience. I could create an issue on the fly without disturbing the code-writing process.

Additional Context:

This one could be related to #18.

There are plans to make it available from the gutter (#20). Isn't this the same? It will be available in the editor, just like you want.

@ForNeVeR Oh, it's called gutter. My fault, sorry... Should I close this one?

Yeah let's close ad duplicate of #20.


Add forward etc properties to Spatial

Godot version:

a5370b1b1bed3744941c90b4a05d4516aabd4c67
OS/device including version:

Arch Linux
Issue description:

While it's possible to get the orientation of a spatial by referring to its transform.basis, such as -transform.basis.z as a manner of determining the transform's forward direction, I think it would be more intuitive, clear and convenient to have properties represent these values, such as forward referring to -transform.basis.z, right to transform.basis.x, and up to transform.basis.y (and likewise, global_forward etc for its global_transform.)
It would also be good to be able to assign to these properties, e.g. assigning a vector to forward would orient the object to face that direction, similarly to look_at(), but also with up and right for easier ways of, for example, getting an object (such as a Sprite3D or MeshInstance) to align with a floor (e.g. setting up to the floor's normal.)  Currently when assigning to an object's basis, the object deforms, which is often not desired; getting an object to orient itself so that its up points in a particular direction requires a roundabout method, such as using look_at() then rotating around an axis (often requiring trial and error to get correct.)  Being able to just assign up to a particular direction (such as by subtracting one position from another) would make this matter much easier.

Note that being able to assign the 3 axes from Spatial is actually not just simplifying the case, it also brings a bunch of assumptions which limits it more than you think.
Setting forward only would basically have no choice but to do some partially default look_at(direction, Vector3(0, 1, 0)), which by the way still needs to expect a negative vector because OpenGL convention. Also what about right and up? You can't just set them either without defaulting another axis under the hood to fix all possible cases, which may result in one of the many valid orientations that you might not want (which is what your trial and error is about, finding the axis that YOU want).
If the object deforms when setting basis, it might be that you are setting an invalid basis to begin with.

Seems like duplicate of #17200

@KoBeWi your right, good catch. Closing as duplicate of #17200


Workload Identity + ORAS + cosign being enabled by default breaks the chart

Cosign is now enabled by default in the chart as of November (post beta2, so included in RC1). This is causing for isSuccess:false responses from Ratify when validating subjects. It doesn't work with workload identity + ORAS store, so right now the chart is ~broken out of the box.
Ratify was installed using:
helm install ratify/ratify \
    --namespace $RATIFY_NAMESPACE --create-namespace \
    --atomic \
    --set provider.auth="tls" \
    --set provider.tls.skipVerify="false" \
    --set provider.tls.cabundle="$(cat $RATIFY_TMP_CERTS_DIR/ca.crt | base64 | tr -d '\n\r')" \
    --set provider.tls.key="$(cat $RATIFY_TMP_CERTS_DIR/server.key)" \
    --set provider.tls.crt="$(cat $RATIFY_TMP_CERTS_DIR/server.crt)" \
    --set image.tag=$RATIFY_VERSION \
    --set azureWorkloadIdentity.clientId=$WORKLOAD_IDENTITY_ID \
    --set ratifyTestCert="$(cat ./azure-ca.crt)" \
    --set oras.authProviders.azureWorkloadIdentityEnabled=true \
    --set logLevel=info

When validating subjects the response will be Error from server (Forbidden): admission webhook "validation.gatekeeper.sh" denied the request: [ratify-constraint] Subject failed verification: <redacted>.azurecr.io/poi:<redacted>
Ratify logs show:
time="2023-02-01T14:19:03Z" level=info msg="received request POST /ratify/gatekeeper/v1/verify "
time="2023-02-01T14:19:03Z" level=info msg="start request POST /ratify/gatekeeper/v1/verify"
time="2023-02-01T14:19:03Z" level=info msg="verifying subject <redacted>.azurecr.io/trips@sha256:<redacted>"
time="2023-02-01T14:19:03Z" level=info msg="received request POST /ratify/gatekeeper/v1/verify "
time="2023-02-01T14:19:03Z" level=info msg="start request POST /ratify/gatekeeper/v1/verify"
time="2023-02-01T14:19:03Z" level=info msg="verifying subject <redacted>.azurecr.io/poi@sha256:<redacted>"
time="2023-02-01T14:19:03Z" level=info msg="Resolve of the image completed successfully the digest is sha256:<redacted>"
{
  "isSuccess": false,
  "verifierReports": [
    {
      "subject": "<redacted>.azurecr.io/poi@sha256:<redacted>",
      "isSuccess": false,
      "message": "verification failed: GET https://<redacted>.azurecr.io/oauth2/token?scope=repository%3Apoi%3Apull\u0026service=<redacted>.azurecr.io: UNAUTHORIZED: authentication required, visit https://aka.ms/acr/authorization for more information."
    }
  ]
}

The temporary fix is to disable cosign and everything works as expected.
helm upgrade ratify ratify/ratify --reuse-values --set cosign.enabled=false -n gatekeeper-system

What did you expect to happen:
Chart should work out of the box with given defaults.
Environment:
GATEKEEPER_VERSION=3.11.0
RATIFY_VERSION=v1.0.0-rc.1
clientVersion:
buildDate: "2023-01-18T15:58:16Z"
compiler: gc
gitCommit: 8f94681cd294aa8cfd3407b8191f6c70214973a4
gitTreeState: clean
gitVersion: v1.26.1
goVersion: go1.19.5
major: "1"
minor: "26"
platform: linux/arm64
kustomizeVersion: v4.5.7
serverVersion:
buildDate: "2022-12-16T19:44:08Z"
compiler: gc
gitCommit: b969368e201e1f09440892d03007c62e791091f8
gitTreeState: clean
gitVersion: v1.25.4
goVersion: go1.19.3
major: "1"
minor: "25"
platform: linux/amd64

A question: Should the cosign feature be enabled by default? This is a (unintended breaking) change from beta 2
I am a data point of N=1, but I would prefer personally to opt-in to features rather than have stores/verifiers enabled that I'm not planning to use
Possibly relevant: we have AKS e2e tests. We've noted that they use Managed Identity rather than Workload Identity. Might be an opportunity to increase test coverage here to catch/prevent this from happening going forward https://github.com/deislabs/ratify/blob/a345e80318b1b06330c266c033548b85fcea88ed/scripts/azure-ci-test.sh#L69-L70

thanks for creating this issue @mluker.  This is caused by Cosign code path not supporting auth. I agree with Noel , our action item:
Short Term ( RC2):

Disable Cosign in the default chart
Better documentation to describe current state of the cosign verifier. Currently the limitation "The Cosign verifier with ratify today only supports images in a public (non-authenticated) registry." is documented here

Long term:
We have a tracking issue to improve the cosign store. Please comment on 194 if you like to increase the priority of this item.

+1 on prioritizing fixing cosign auth. For now can we document that cosign might need to be disabled and have an RC2 milestone where we fixing.

+1 instead of using helm value to disable cosign.
I found we need to delete the verifier verifier-cosign and then edit store to set cosignEnabled: false. Can we remove the spec cosignEnabled spec in store and directly check it according to if exists cosign verifier?

PR #616 updated chart so cosign no longer conflicts with workload identity.  We will actively working on issue #194  and #230 to improve the cosign experience working with private registries.

+1 instead of using helm value to disable cosign. I found we need to delete the verifier verifier-cosign and then edit store to set cosignEnabled: false. Can we remove the spec cosignEnabled in store and directly check it according to if exists cosign verifier?

good Point @fseldow, we will take this requirement into consideration.


Update docs for remote attach

Codecov Report

Merging #202 (8ae8fb4) into master (a22c619) will decrease coverage by 0.80%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##            master     #202      +/-   ##
===========================================
- Coverage   100.00%   99.19%   -0.81%
===========================================
  Files           23       23
  Lines         2597     2597
===========================================
- Hits          2597     2576      -21
- Misses           0       21      +21

Impacted Files
Coverage Δ

src/viztracer/patch.py
94.62% <0.00%> (-5.38%)
:arrow_down:

src/viztracer/viztracer.py
95.67% <0.00%> (-4.33%)
:arrow_down:

src/viztracer/main.py
98.37% <0.00%> (-1.63%)
:arrow_down:

src/viztracer/simulator.py
98.38% <0.00%> (-1.62%)
:arrow_down:

src/viztracer/viewer.py
99.60% <0.00%> (-0.40%)
:arrow_down:

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 a22c619...8ae8fb4. Read the comment docs.


bug in origin_time_source_matcher

Messed up somehow what I started this and got this empty box.  Actual comment is below.

Messed up starting this somehow, but I found what is without doubt a bug in the normalization class `origin_time_source_matcher.   The following method will fail if the input is an ensemble:
    def _get_test_time(self, d, time):
        """
        Private method that safely computes a time for testing from
        the "starttime" attribute of d.
        """
        if time == None:
            if isinstance(
                d, (TimeSeries, Seismogram, TimeSeriesEnsemble, SeismogramEnsemble)
            ):
                test_time = d.t0 - self.t0offset
            else:
                if d.is_defined("starttime"):
                    test_time = d["starttime"] - self.t0offset
                else:  #   t0 can't be extracted from the object
                    return None
        else:
            test_time = time - self.t0offset
        return test_time

The problem is that t0 is defined only for TimeSeries or Seismogram data.  An ensemble does not always have the concept of a start time.   I think the idea of this private method is to try to safely always retrieve a valid time for testing, but this will fail often if passed Time==None.
We need this algorithm to work with ensembles as common source gathers are a very very common construct.   I think what we need is to add a time key as an optional argument to the constructor.  The logic should be something like this:
if self.time_key:
   if d.is_defined(self.time_key):
     test_time = d[self.tim_key] - self.t0offset
   else:
     return None   # matches logic but may not be right approach here
else:
   if isinstance(d, (TimeSeries, Seismogram)):
      test_time = d.t0 - self.t0offset
   else:
     return None
return test_time

or something like that.

I got that fixed in #324 and will merge it soon after checks passed.


Add convention for PersistenceDiagram

This PR adds a convention for scitype(::PersistenceDiagram, ::MLJ) = PersistenceDiagram.
I also added entries into the docs. Let me know if I missed anything.
cc: @ablaom

Codecov Report

Merging #48 into master will decrease coverage by 2.26%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #48      +/-   ##
==========================================
- Coverage   99.11%   96.84%   -2.27%
==========================================
  Files           7        7
  Lines         225      222       -3
==========================================
- Hits          223      215       -8
- Misses          2        7       +5

Impacted Files
Coverage Δ

src/MLJScientificTypes.jl
100.00% <ø> (ø)

src/convention/scitype.jl
82.14% <100.00%> (-14.16%)
:arrow_down:

src/autotype.jl
98.00% <0.00%> (-2.00%)
:arrow_down:

src/convention/coerce.jl
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4b5f971...b40ebd5. Read the comment docs.

And can you switch the PR target from master to dev
https://github.com/alan-turing-institute/MLJ.jl/blob/master/CONTRIBUTING.md .

Just a quick question. If I understand this correctly, Scitype needs to be overloaded for the case when you have an Array of non-concrete types. PersistenceDiagram has no type parameters. Is overloading Scitype still necessary in this case?

Yes, it is still necessary.
Otherwise scitype(v) will compute the union of all the scitypes of the elements of v, whenever eltype(v) <: PersistentDiagram. This is a long story.

OK, thanks for the clarification. I added the definition.


GET /spots

GET /spots
Story

As a developer
I want an api that lists spot information
so that I can query a list of spots

Acceptance Criteria

Update the GET /spots api do use a mongo collection called 'spots'
For the find query object, use {}
Update your test to only make the following assertion

res.body.should.be.an.Array

Approach

Use the API listed here:

http://mongodb.github.io/node-mongodb-native/2.2/api/Db.html
http://mongodb.github.io/node-mongodb-native/2.2/api/Collection.html

@dreagle95 Sprint due Tuesday 3/28

Will you be sending the .env file with the username and password for testing purposes?

Up


Fix .btn background on

Clear browser default background off buttons for consistency.
Before:

After:

💃 🍻

Just released your latest updates as<EMAIL_ADDRESS>Thanks again!


Fix Register Clobbering

Haswell (and probably also knl/skx) hadn't had rbx and rcx clobbered, and hence failed sometimes due to that. Thus, we now add the registers to the starting_regs instead.

Thanks for the approval, @montrie


Pre-training model

when I load the resnet_v1_50,ckpt in my model,there is the error in my pycharm:
NotFoundError (see above for traceback): Tensor name "arcface_loss/embedding_weights" not found in checkpoint files ./output/ckpt/resnet_v1_50.ckpt
[[Node: save_1/RestoreV2 = RestoreV2[dtypes=[DT_FLOAT, DT_FLOAT, DT_INT32, DT_FLOAT, DT_FLOAT, ..., DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save_1/Const_0_0, save_1/RestoreV2/tensor_names, save_1/RestoreV2/shape_and_slices)]]
what should I do ? please?

I have uploaded two new model, you can test with the new ckpt file.

@auroua The new model is the  resnet_v1_50? I want to test it, please?

It's using resnet in L_Resnet_E_IR.py.

I'm so sorry because I'm a new player.If I want to use the pre-training model that is the  resnet_v1_50 pretrained model download link. I have loaded it.I revised "# restore_saver.restore(sess, '/home/aurora/workspaces2018/InsightFace_TF/output/ckpt/InsightFace_iter_1110000.ckpt') " to "
restore_saver.restore(sess, 'home/user//InsightFace_TF/output/ckpt/resnet_v1_50)",But it can't use.
can you help me? @auroua

and I meet the same error, no Tensor name "arcface_loss/embedding_weights" not found.
I use the newest model and project.


Result levels are different or not all existing comparing to the help text

Expected Behavior
That the four result levels named in the --help text are recognized.
Actual Behavior
rlog and extended are not recognized: "Option --result-level defined but not recognised. Using 'aggregate' by default."
Steps to Reproduce the Problem

Run docker container: docker run --rm -it -p 80:80 --entrypoint="" aksw/rdfunit /bin/bash
Execute RDFUnit with --result-level and rlog or extended as parameter value.

Specifications

Version: https://hub.docker.com/r/aksw/rdfunit
Platform: Mac OS
Subsystem: docker engine

If I remember correctly I had removed these options as deprecated in favor of shacl-lite (equivalent to rlog) and shacl (equivalent to extended).
these new formats provide the same level of detail and are also based on the SHACL vocabulary which makes them more compatible with other systems


DotNetCoreCLI loc handback

Closed and reopened against releases/tfs2018rtw instead of releases/m122

reviewer: take or close

Closed it since the rtw branch is done for now


Allow greater customization of lightbox elements

Is it possible to add toggles to the toml file to control the inclusion or visibility of the elements of the lightbox? Specifically the right-left control arrows, magnifying glass icon, and image count.
The functionality would remain the same, but the visible presence of these elements would be optional.

@kowalabearhugs yes. foto is passing everything defined in foto.toml to the template. So technically you can do this without binary update. This will require some modification on both the toml file and template html file, like this.
However this seems an interesting feature so I just included it in the newest release today. You still need to re-create the whole directory or copy the new template file to it though.

Wonderful. Your prompt response and enhancements are much appreciated.
I just ran some tests using Firefox and Safari. When configured in this instance no arrows appear: show_arrow_prev = false, show_arrow_next = true
If prev is true and next false then it correctly only shows the prev arrow.
All other configurations appear to work correctly. This really doesn't affect my use-case as I intend to always pass the same value to both arrows. IMO consolidating the arrows into a single toggle option would be a a suitable offering.

@kowalabearhugs thanks for the testing! I also found this and thought it should be a bug on PhotoSwipe. But consolidating the arrows is a great idea. I'll push a fix later.

@kowalabearhugs fixed here and pushed 1.5.1.

I tested a 1.5.1 site build on FF and Safari and the lightbox toggles work in all configurations. Thank you!

Thanks for testing!


Update the Android demo to use retrained models

The Android example currently uses an older Inception model. It requires some changes to load the one used by the label_image, classify_image, and image_retraining examples. These include altering the names of the input and output layers, ensuring the larger model can be loaded, and updating the mean, standard deviation, and image size parameters.
Once this is done, people should be able to transfer retrained models to a device easily. This issue also covers documenting that process.

This is now fixed, closing.


Just something to think about adding

Since it's based off of Catppuccin, Maybe add some more of the dark colors like their core, mantle, and base.

And/Or darker versions of each color


Uncaught TypeError: objectInfluences is undefined

This is my first day using gltfjsx and I got this error immediately. I haven't modified any code.

I'm using vanilla jsx.
import React, { useRef } from "react";
import { useGLTF } from "@react-three/drei";

export function Avatar(props) {
  const { nodes, materials } = useGLTF("models/khun.glb");
  return (
    <group {...props} dispose={null}>
      <primitive object={nodes.Hips} />
      <skinnedMesh
        geometry={nodes.Wolf3D_Hair.geometry}
        material={materials.Wolf3D_Hair}
        skeleton={nodes.Wolf3D_Hair.skeleton}
      />
      <skinnedMesh
        geometry={nodes.Wolf3D_Glasses.geometry}
        material={materials.Wolf3D_Glasses}
        skeleton={nodes.Wolf3D_Glasses.skeleton}
      />
      <skinnedMesh
        geometry={nodes.Wolf3D_Body.geometry}
        material={materials.Wolf3D_Body}
        skeleton={nodes.Wolf3D_Body.skeleton}
      />
      <skinnedMesh
        geometry={nodes.Wolf3D_Outfit_Bottom.geometry}
        material={materials.Wolf3D_Outfit_Bottom}
        skeleton={nodes.Wolf3D_Outfit_Bottom.skeleton}
      />
      <skinnedMesh
        geometry={nodes.Wolf3D_Outfit_Footwear.geometry}
        material={materials.Wolf3D_Outfit_Footwear}
        skeleton={nodes.Wolf3D_Outfit_Footwear.skeleton}
      />
      <skinnedMesh
        geometry={nodes.Wolf3D_Outfit_Top.geometry}
        material={materials.Wolf3D_Outfit_Top}
        skeleton={nodes.Wolf3D_Outfit_Top.skeleton}
      />
      <skinnedMesh
        geometry={nodes.EyeLeft.geometry}
        material={materials.Wolf3D_Eye}
        skeleton={nodes.EyeLeft.skeleton}
      />
      <skinnedMesh
        geometry={nodes.EyeRight.geometry}
        material={materials.Wolf3D_Eye}
        skeleton={nodes.EyeRight.skeleton}
      />
      <skinnedMesh
        geometry={nodes.Wolf3D_Head.geometry}
        material={materials.Wolf3D_Skin}
        skeleton={nodes.Wolf3D_Head.skeleton}
      />
      <skinnedMesh
        geometry={nodes.Wolf3D_Teeth.geometry}
        material={materials.Wolf3D_Teeth}
        skeleton={nodes.Wolf3D_Teeth.skeleton}
      />
    </group>
  );
}

useGLTF.preload("models/khun.glb");

How I use it
import { OrbitControls } from "@react-three/drei";
import { Avatar } from "./Avatar";

export const Experience = () => {
  return (
    <>
      <OrbitControls />
      <Avatar />
    </>
  );
};

File paths are correct

Welcome to gltfjsx!
I have created a quick working demo of this project.
Github: https://github.com/justindhillon/gltfjsx-issue--249
Demo: https://gltfjsx-issue-249.vercel.app/
Screencast from 2024-03-10 23-57-11.webm
Your implementation lacks <Canvas>.  Here is how it is supposed to look like:
"use client"

import { Canvas } from "@react-three/fiber";
import { OrbitControls, PerspectiveCamera } from "@react-three/drei";

import { Avatar } from "./Avatar";

export default function Experience() {
  return (
    <Canvas>
      <OrbitControls />
      <Avatar />
      <PerspectiveCamera makeDefault position={[30, 30, 20]} />
    </Canvas>
  );
}

It could always be something wrong with your model or rest of your project files. If you still have issues, link your github repo or send me questions via linkedIn. I'm always happy to help!
LinkedIn: https://www.linkedin.com/in/justin-dhillon/

Welcome to gltfjsx!
I have created a quick working demo of this project.
Github: https://github.com/justindhillon/gltfjsx-issue--249 Demo: https://gltfjsx-issue-249.vercel.app/
Screencast.from.2024-03-10.23-57-11.webm
Your implementation lacks <Canvas>. Here is how it is supposed to look like:
"use client"

import { Canvas } from "@react-three/fiber";
import { OrbitControls, PerspectiveCamera } from "@react-three/drei";

import { Avatar } from "./Avatar";

export default function Experience() {
  return (
    <Canvas>
      <OrbitControls />
      <Avatar />
      <PerspectiveCamera makeDefault position={[30, 30, 20]} />
    </Canvas>
  );
}

It could always be something wrong with your model or rest of your project files. If you still have issues, link your github repo or send me questions via linkedIn. I'm always happy to help!
LinkedIn: https://www.linkedin.com/in/justin-dhillon/

Thanks for your time.
My code works now but this is how I'm done.
Instead of using cli:
npx gltfjsx <path-to-my-.glb-file>

I uploaded the same .glb file to
https://gltf.pmnd.rs/
and it generates the very similar code, and that code works.
Comparing the two code, I found that in skinnedMesh generated by cli doesn't come up with ''name'' and other props
<skinnedMesh
  geometry={nodes.EyeLeft.geometry}
  material={materials.Wolf3D_Eye}
  skeleton={nodes.EyeLeft.skeleton}
/>

The one generated by that website comes up with this:
<skinnedMesh
          name="EyeLeft"
          geometry={nodes.EyeLeft.geometry}
          material={materials.Wolf3D_Eye}
          skeleton={nodes.EyeLeft.skeleton}
          morphTargetDictionary={nodes.EyeLeft.morphTargetDictionary}
          morphTargetInfluences={nodes.EyeLeft.morphTargetInfluences}
        />

I'm not sure but I think that's the case


👷 Set PYTHONWARNDEFAULTENCODING in GitHub Workflow

Fix #91
This merge request sets the PYTHONWARNDEFAULTENCODING variable in the GitHub workflow per the documentation at https://docs.github.com/en/actions/learn-github-actions/variables.
I've chosen to set PYTHONWARNDEFAULTENCODING to the value true to ensure it is clear that this is enabling the feature. With that said, https://docs.python.org/3/library/io.html#opt-in-encodingwarning does not indicate that there is a value of the environment variable that would disable this, so I understand if a different value would be preferred.

Happy to help out 🙂  looks like there were two warnings in the tests, so shouldn't be too bad to get those addressed.
=============================== warnings summary ===============================
tests/integration/test_cli.py::test_command_line
tests/integration/test_cli.py::test_command_line
  /home/runner/work/bump-pydantic/bump-pydantic/bump_pydantic/codemods/mypy_visitor.py:34: EncodingWarning: 'encoding' argument not specified
    result = build(files, opt, stdout=sys.stdout, stderr=sys.stderr)

tests/integration/test_cli.py::test_command_line
  /home/runner/work/bump-pydantic/bump-pydantic/bump_pydantic/main.py:83: EncodingWarning: 'encoding' argument not specified
    log_fp = log_file.open("a+")

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html

Happy to help out 🙂 looks like there were two warnings in the tests, so shouldn't be too bad to get those addressed.
=============================== warnings summary ===============================
tests/integration/test_cli.py::test_command_line
tests/integration/test_cli.py::test_command_line
  /home/runner/work/bump-pydantic/bump-pydantic/bump_pydantic/codemods/mypy_visitor.py:34: EncodingWarning: 'encoding' argument not specified
    result = build(files, opt, stdout=sys.stdout, stderr=sys.stderr)

tests/integration/test_cli.py::test_command_line
  /home/runner/work/bump-pydantic/bump-pydantic/bump_pydantic/main.py:83: EncodingWarning: 'encoding' argument not specified
    log_fp = log_file.open("a+")

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html

Wanna help? 👀 🙏

Happy to, it'll be later today before I can get to that but shouldn't be a problem at all. I'll need to read the docs on failing on warnings but I'd be surprised if pytest made that a difficult thing to configure. It seems like it'd be something many teams would want.

I think those 2 lines need to be added for the tests to fail on warning: https://github.com/encode/starlette/blob/844220819f1cfcf2726d1628dd4ea597182d6f3e/pyproject.toml#L77-L79


Can this framework call xctool cmd?

I want use this for call xctool  to build my project and then uplaod to my sever. Could you give me some suggestion or a demo for me. Thanks!

Sure, do you want to make a commandline application written in Swift to do this? What xctool commands do you want to run? What type of server are you uploading to?

Thx for your reply. I had call the xcbuild to build and package the project. Your framework is very good ! And could you me some suggestion about this https://github.com/myafer/HHSSJJ/blob/master/HHSSJJ/main.swift

Okay, I have some comments:

use main.currentdirectory = projectPath. This way you don't have to do "cd " + projectPath + "; all the time.
instead of using bash, call the command directly, like this: try runAndPrint("xcodebuild", "archive", "-scheme", "TTTT", "-configuration", "Release", "-archivePath", archivepath). This way you don't have to remember to use ". All the arguments are sent directly to xcodebuild exactly the way you write them, even if they contain '*' or '?'.
instead of "CommandLine.arguments" you can use "main.arguments"; it contains just the arguments to the script, the first item is not the path of the script as it is with "CommandLine.arguments".

OK， thank you very much!


Cancelling ChangeBlockEvent.Modify Does Not Cancel Redstone Output

In the case of buttons, levers, pressure plates, tripwire and any other block that outputs a redstone signal, cancelling the ChangeBlockEvent.Modify event does not cancel the redstone signal that is outputted immediately. There is a very short redstone signal output that is still sent out from the block, which is just long enough for a redstone repeater to detect. I would recommend that you use the test world that I provided to test this issue, as orientation of the redstone repeater can mean everything with this bug (redstone bugs are known to behave differently depending on the orientation). I have also provided a video, recorded in the test world, showing you the bug. I recorded this video from a second player, who is viewing the first player producing the bug, to eliminate any suspicion of client-side "ghosting".
Here is the sample code that I used to produce this bug:
	@Listener
	public void onBlockModify(ChangeBlockEvent.Modify event, @Root Player player) {
		event.setCancelled(true);
	}

The world file can be found at http://www.risingcolonies.com/files/world.zip. The area I'm testing at is located at -840,4, 1075
And here's the video: https://www.youtube.com/watch?v=5cgStgLYLq0.  This should happen before the modify and should fix the issue you are having. Just check permissions on all neighbor blocks and if any do not have correct permission, remove it from the list. The filterPredicate method is bugged so dont use that, just remove it directly from the neighbor list passed.

@clucky I'm implementing CollideBlockEvent which should be the best way to handle these type of events as the player will be the root cause.

@bloodmc I have not yet tried the NotifyNeighborBlockEvent. Would this CollideBlockEvent replace the need to do NotifyNeighborBlockEvent for pressure plates, or would that event still be required?

@clucky Cancelling CollideBlockEvent will remove the need to worry about NotifyNeigborBlockEvent. I'll be pushing it shortly.

@clucky Please confirm cancelling the CollideBlockEvent fixes your issue. Thanks!

@bloodmc It appears to correct the issue. Thanks


Allow unambiguous FQDNs

Fixes https://github.com/istio/istio/issues/14404

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: howardjohn
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

galley/OWNERS
pilot/OWNERS [howardjohn]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: howardjohn, sdake
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

galley/OWNERS
pilot/OWNERS [howardjohn]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/assign @ozevren

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
51689717e2733bb1f4b078c2043de20d9b8bb7be
link
/test integ-security-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn, ozevren, sdake
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

galley/OWNERS [ozevren]
pilot/OWNERS [howardjohn]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
51689717e2733bb1f4b078c2043de20d9b8bb7be
link
/test integ-security-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Create executables in a separate folder ./bin

Otherwise .gitignore file should contain all the executable names.
If this is implemented, just add bin/ in .gitignore, and remove the current list of explicit executable names from .gitignore

Right, in general we should at some point move sources to src/ folder and exectuables to build/ or bin/
As for now I added to .gitignore to omit all files without extension, this should solve the problem temporarily.


Portfolio Design System Alignment + Creating 3rd Grey in MM DS

Justin to check in with Chris M on what the DS Wishlist is
What are the limitations of MM?
Create 3rd Gray color that is between Alternative and Default

This has become a bigger assignment, and so I will be deprecating this.


MAINT: Validate codeowners

Inspired by #11989, let's see if it works

Okay this action appears not to work. I just installed and ran it locally instead with:
REPOSITORYPATH=. GITHUB_ACCESS_TOKEN="$OWNERS_VALIDATOR_GITHUB_SECRET" OWNER_CHECKER_REPOSITORY="mne-tools/mne-python" EXPERIMENTAL_CHECKS="avoid-shadowing" $(go env GOPATH)/bin/codeowners-validator

all we got was false alarms for shadowing with /*.* (the tool doesn't recognize the leading / as meaning "root-level only") so I'll go ahead and close this. We'll just have to be vigilant until the automated tools mature :shrug:


Merge StickyE project into HARK

The StickyE_prepareMerge branch is up to date with econ-ark/master and contains the changes made during the StickyE project, both to the original HARK files and all the additional files contained in the cAndCwithStickyE folder

For the historical record: The first new commit on this branch (fce7d45) is a squash merge of several months of commits from ESC and me on the cAndCwithStickyE project, which were done on his private fork.  We are now moving the completed work for that project into the main HARK repo as another example of a real application that uses the toolkit.
The big squash commit was to avoid cluttering up HARK's commit history with what is (in great part) a research dialog between Edmund and me, not development of "core" HARK functionality.  In executing this project, we coded up the RepAgent and ConsAggMarkov models as new additions to HARK.  The RepAgent model has been merged into this repo in a separate pull request; ConsAggMarkov has an outstanding PR, but this PR probably goes beyond that branch, fixing various issues.
I need to investigate what is in my original ConsAggMarkov PR vs what's here, and make sure that this version fully supercedes that one.  I know I did a small bit of work on ConsIndShockModel.py on that branch as well.

This PR now incorporates and supercedes #125 ... The development commit history for ConsAggMarkov from last summer has been added to this branch.

Made a few small changes to this branch, then tested StickyE and various model demos.  Everything seems to work properly, merging (then closing #125).


Add transform_row and scalar_parser documentation and make them easier to use

Purpose
As a follow up to #117, we wanted to document the new capabilities in the README and to make it a bit easier to use the new scalar_parser configuration of queries.
Overview

Add an attr_accessor for scalar_parser to Query
Pass this along in column_value_parsers
Fix an issue where the shorthand used in Query#transform_rows was incorrect (sorry!)
Add an example of using transform_row and scalar_parser to the README.

@fionaochs paired on this with me 🎉
Checklist

[x] Code compiles correctly
[x] Created tests which fail without the change (if possible)
[ ] All tests passing
[x] Extended the README / documentation, if necessary

👋 I noticed that we didn't increase the version number for the last change I did:
https://github.com/treasure-data/trino-client-ruby/blob/e14251cebb8ebae968a0e7b7c8e68d6b5918dd93/lib/trino/client/version.rb#L18
Do you want me to bump that to 2.1.1 in this PR?

@kmcq Thanks. I forgot to do git-push.
https://github.com/treasure-data/trino-client-ruby/commit/d1078c0a491dbd0fc133abc98978afebea257ad9

@kmcq We released v2.2.0. Thanks for your contribution.
https://rubygems.org/gems/trino-client/versions/2.2.0