[Snyk] Upgrade react-router-dom from 6.13.0 to 6.14.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade react-router-dom from 6.13.0 to 6.14.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 3 versions ahead of your current version.
The recommended version was released 24 days ago, on 2023-06-23.

Release notes

Package name: react-router-dom

6.14.0 - 2023-06-23

6.14.0-pre.1 - 2023-06-21

6.14.0-pre.0 - 2023-06-21

6.13.0 - 2023-06-14

    from react-router-dom GitHub release notes

Commit messages

Package name: react-router-dom

09b6cbe chore: Update version for release (#10636)
0ec67e6 Exit prerelease mode
d4ec21b changelog/docs updates
9debb5c Update changelogs
d1bc7cf Update changeset + chanbgelog
6cbd0b1 chore: Update version for release (pre) (#10628)
19a71df fix FormData submitter check (#10627)
8b95f2b chore: Update version for release (pre) (#10626)
d9b3941 Enter prerelease mode
ccf63e2 Merge branch 'main' into release-next
fff84b5 fix empty submissions (#10625)
718ec1f Avoid calling shouldRevalidate on interrupted initial load fetchers (#10623)
96e1fc1 Inline startTransitionImpl to avoid .d.ts issues with skipLibCheck:false (#10622)
bc25528 Fixing typo (#10619)
7e2a122 chore: sort contributors list
80072b2 docs(DEVELOPMENT.md): fix typos (#10613)
73cd01e Fix generatePath when passed a zero value param (#10612)
3d4868b chore: add `deduplicate-yarn` workflow (#10583)
f63774c fix(react-router-dom): fix submitter serialization (#9865)
a3b21eb Merge branch 'release-next' into dev
dedafeb Merge branch 'release-next'
ce54655 chore: dedupe `yarn.lock` (#10582)
2b204d8 chore: upgrade typescript (#10581)
834383b Fix unstable_useBlocker key issues in strict mode (#10573)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Dependency update solved in #64


Tell them it's for testing

Yes, telling them it's for testing is the best. :)

@StacysMum  thanks for the comment. That's what I've been told, but I know a few customers use it for caching in production. A well architected cache can recover from failure.


Resolve Discovered Launch Issues (docs.ms track 2 docs)

[x] Move management.msi to Active Directory section
[x] Move FluentCore packages to Resources
[x] Make a pass over the implementation namespaces. Remove all where present.

PR Here Azure/azure-docs-sdk-java#966

Merging to live today.


Fix internal-debug syncz not working with agent, and add support for debug type config_dump

Please provide a description of this PR:
Currently if I do:
istioctl x internal-debug syncz  --xds-address localhost:15004 --plaintext

I get no response.
Also, replace x ps with x internal-debug syncz. This is a prerequisite of https://github.com/istio/istio/pull/47740.
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Dual Stack
[ ] Installation
[x] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[x] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[x] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

@hanxiaop: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
0bc51c22b6cfcffd3d003c45c2f680b4145bacca
link
true
/test lint

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.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
0bc51c22b6cfcffd3d003c45c2f680b4145bacca
link
true
/test lint

gencheck_istio
0bc51c22b6cfcffd3d003c45c2f680b4145bacca
link
true
/test gencheck

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.

I am not even aware there is an internal-debug sub command, do we have any docs related

@hzxuzhonghu I don't think so. It is a command marked as hidden. I haven't investigated the intention of the command further, but right now the command example seems to use only syncz to provide debug interface via xds, which is identical to istioctl x ps.

Is it just similar? Can we use istioctl x ps to replace that one

Is it just similar? Can we use istioctl x ps to replace that one

@hzxuzhonghu Sorry for the misunderstanding, I meant that istioctl x internal-debug syncz is identical to istioct x ps, x ps is like a subset of internal-debug. The internal-debug command can be used to obtain additional info like config_dump, but that will require to use resource name which will be like istioctl x internal-debug config_dump <resource name>.

@hzxuzhonghu Basically it's generate through:
https://github.com/istio/istio/blob/572acf2859888645467345cb4c7aa418176b757d/pilot/pkg/xds/statusgen.go#L83-L85

@hanxiaop: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security_istio
21028bc791300c41f093d217ea712654b1201758
link
unknown
/test integ-security

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.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security_istio
21028bc791300c41f093d217ea712654b1201758
link
unknown
/test integ-security

integ-security-multicluster_istio
21028bc791300c41f093d217ea712654b1201758
link
unknown
/test integ-security-multicluster

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.

/retest

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security_istio
21028bc791300c41f093d217ea712654b1201758
link
unknown
/test integ-security

integ-security-multicluster_istio
21028bc791300c41f093d217ea712654b1201758
link
unknown
/test integ-security-multicluster

integ-security-istiodremote_istio
21028bc791300c41f093d217ea712654b1201758
link
unknown
/test integ-security-istiodremote

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.

/test integ-security-istiodremote


While giving backup from local to server, give wrong datetime at created_at fields.

Everything works fine but when create backup online, there is difference of 4 to 5 hours of created_at field

It's Timezone issue, while exporting using mysqldump, by default data is exported timezone with '00:00' so that's why its changed the time-stamp's (created_at,updated_at) value.
Can you please tell us, how can i configure timezone for mysqldump


operator: add missing IaaDevicePlugin finalizers RBAC

Fixes: #1471

Codecov Report

Merging #1475 (0061d64) into main (0337834) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 0061d64 differs from pull request most recent head 34baf98. Consider uploading reports for the commit 34baf98 to get more accurate results

@@           Coverage Diff           @@
##             main    #1475   +/-   ##
=======================================
  Coverage   50.10%   50.10%
=======================================
  Files          43       43
  Lines        4878     4878
=======================================
  Hits         2444     2444
  Misses       2295     2295
  Partials      139      139

Impacted Files
Coverage Δ

pkg/controllers/iaa/controller.go
6.89% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


An enum property isn't mapped to the PostgreSQL enum inside value objects

The problem occurs only for value objects inside complex objects. The following example maps PhoneNumberType to int:
public enum PhoneNumberType
{
    Mobile,
    Home,
    Work
}

public class PhoneNumber
{
    public PhoneNumberType Type { get; set; }
    public int Number { get; set; }
}

public class Contact
{
    public PhoneNumber Phone { get; set; }
}

builder.OwnsOne(e => e.Phone).Property(p => p.Type);
builder.OwnsOne(e => e.Phone).Property(p => p.Number);

@YohDeadfall Do you mind if I wrap this into #605 which attempts to address some other enum edge cases?

I think that such changes should be made by separate pull requests. One issue - one request. But let's leave it as is this time.

I'm going to add this to 2.2.0 for now, hoping we can fix it by the time we release.

FYI starting to look into this now.
Notes

This looks similar to #651. The triage for that issue notes that it could be reproduced without an owned entity using just a one-to-one relationship.

[ ] Test whether or not this can be reproduced with a simple one-to-one relationship.

Actually, I can't reproduce this in the test suite. The 2.2 release notes do reference changes to owned type behavior, so it's possible that this was simply resolved upstream.
@YohDeadfall Here's the setup I used to try and reproduce this. Does the configuration look right? Are you still able to reproduce this locally?
[Fact]
public void Where_with_owned_type()
{
    using (var ctx = Fixture.CreateContext())
    {
        var _ = ctx.SomeEntities.Single(e => e.OwnedTypeWithEnum.MappedEnum == MappedEnum.Happy);

        AssertContainsInSql("WHERE e.\"OwnedTypeWithEnum_MappedEnum\" = 'happy'::test.mapped_enum");
    }
}

public class SomeEnumEntity
{
    public int Id { get; set; }
    public MappedEnum MappedEnum { get; set; }
    public UnmappedEnum UnmappedEnum { get; set; }
    public InferredEnum InferredEnum { get; set; }
    public SchemaQualifiedEnum SchemaQualifiedEnum { get; set; }
    public int EnumValue { get; set; }

    public OwnedTypeWithEnum OwnedTypeWithEnum { get; set; }
}

public class OwnedTypeWithEnum
{
    public MappedEnum MappedEnum { get; set; }
    public UnmappedEnum UnmappedEnum { get; set; }
    public InferredEnum InferredEnum { get; set; }
    public SchemaQualifiedEnum SchemaQualifiedEnum { get; set; }
    public int EnumValue { get; set; }
}

protected override void OnModelCreating(ModelBuilder builder)
{
    builder.ForNpgsqlHasEnum("mapped_enum", new[] { "happy", "sad" })
           .ForNpgsqlHasEnum<InferredEnum>()
           .HasDefaultSchema("test")
           .ForNpgsqlHasEnum<SchemaQualifiedEnum>();

    builder.Entity<SomeEnumEntity>().OwnsOne(x => x.OwnedTypeWithEnum).Property(x => x.MappedEnum);
    builder.Entity<SomeEnumEntity>().OwnsOne(x => x.OwnedTypeWithEnum).Property(x => x.UnmappedEnum);
    builder.Entity<SomeEnumEntity>().OwnsOne(x => x.OwnedTypeWithEnum).Property(x => x.InferredEnum);
    builder.Entity<SomeEnumEntity>().OwnsOne(x => x.OwnedTypeWithEnum).Property(x => x.SchemaQualifiedEnum);
    builder.Entity<SomeEnumEntity>().OwnsOne(x => x.OwnedTypeWithEnum).Property(x => x.EnumValue);
}

Note that I tested this in a branch following #685, so the schema handling may look different from the current dev branch.

We have moved out from EF for the current project (: Will check it when I have half an hour to spare.

@YohDeadfall hope it's nothing bad with EF :) Do you want to continue being active and reviewing EF stuff or only Npgsql?

It's very limited for our goals and too slow in comparison with pure Npgsql and SQL, but we will continue to use it for migrations only which are written in plain SQL (: Yes, I want to work on EF too, but currently I'm busy with the presentation I mentioned in person. It covers npgsql/npgsql#2180.

OK, great - we can also use your work to write that page on migrating from MSSQL. Good luck with the presentation!

Looks like I've omitted some part that triggers the problem or most likely had an issue in application... Closing as I can't reproduce it too. Thanks for taking a look!


fix issue #1682

Description
The local vector database configuration does not allow customizing the collection_name through the settings.
Fixes #1682
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Please delete options that are not relevant.

[ ] Unit Test
[ ] Test Script (please provide)

import os
from mem0 import Memory

# Set the OpenAI API key

class PersonalMemory:
    MEM0_CONFIG = {
        "vector_store": {
            "provider": "qdrant",
            "config": {
                "collection_name": "msw_mem0_test",
                "host": "<server-ip>",
                "port": 6333,
            }
        },
    }

    def __init__(self, user_id):
        self.user_id = user_id

        self.memory = Memory.from_config(self.MEM0_CONFIG)

    def add_memory(self, content):
        self.memory.add(content, user_id=self.user_id)

    def get_memories(self):
        return self.memory.get_all(user_id=self.user_id)

    def search_memories(self, query):
        return self.memory.search(query, user_id=self.user_id)

# Usage example
user_id = "msw_test_3"
pm = PersonalMemory(user_id=user_id)

def add_memory():
    while True:
        content = input("content: ")
        if content.lower() in ['q', 'exit']:
            print("Exiting...")
            break

        answer = pm.add_memory(content)
        print(f"Answer: {answer}")
        memories = pm.get_memories()
        print("Memories:")
        for memory in memories:
            print(f"- {memory}")
        print("-----")

def ask_question():
    question = input("question: ")
    memories = pm.search_memories(question)
    print("Memories:")
    for memory in memories:
        print(f"- {memory}")

if __name__ == "__main__":
    add_memory()

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have checked my code and corrected any misspellings

Maintainer Checklist

[ ] closes #xxxx (Replace xxxx with the GitHub issue number)
[ ] Made sure Checks passed

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @WiSmile Can you please resolve the merge conflicts?

It seems that someone has solved this problem with a better solution before me


font problem

I have set  enable cjk font to 1, but after reboot the container, the language is still shown as 口口口口, how should i fix this,thanks.( run on synology ds920 ,dsm 6.2.4)

Fixed，I checked the log and found that fetch and excute font may need some time,and after about 10 minutes everyting went well.


Nesting an Office 365 Group in an Azure AD Group

As per discussion in below MSDN thread it seems this is currently not supported. Please clarify.
https://social.msdn.microsoft.com/Forums/en-US/a6ff37c5-1a7c-4788-9b44-430f70cdd11d/nesting-an-office-365-group-in-an-azure-ad-group?forum=WindowsAzureAD

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

ID: fe42976d-da2c-c20a-25f1-f1548dcffbfb
Version Independent ID: fbeaa46b-eff9-b43a-e90e-6486149a3103
Content: How to add or remove a group from another group in Azure Active Directory
Content Source: articles/active-directory/fundamentals/active-directory-groups-membership-azure-portal.md
Service: active-directory
GitHub Login: @eross-msft
Microsoft Alias: lizross

@eross-msft There is discussion on below MSDN thread which states that nesting of groups is not supported. Customer points to this document. Please help clarify if this document is stating a valid scenerio.
https://social.msdn.microsoft.com/Forums/en-US/a6ff37c5-1a7c-4788-9b44-430f70cdd11d/nesting-an-office-365-group-in-an-azure-ad-group?forum=WindowsAzureAD

@MohitGargMSFT  This is correct, Office 365 doesn't support nested groups. I'm updating our article to address this by changing the section to be about Security groups and to specifically say that we don’t currently support adding Office 365 groups to other groups (also known as nesting groups), assigning apps to nested groups, or applying licenses to nested groups.

do we know the GA of AAD nested groups?

Hi,
Related to Azure AD group nesting options, Microsoft team documented here
https://docs.microsoft.com/en-us/azure/active-directory/fundamentals/active-directory-groups-membership-azure-portal
the options they would not support...
We don't currently support:
Adding groups to a group synced with on-premises Active Directory.
Adding Security groups to Microsoft 365 groups.
Adding Microsoft 365 groups to Security groups or other Microsoft 365 groups.
Assigning apps to nested groups.
Applying licenses to nested groups.
Adding distribution groups in nesting scenarios.
As any progress been made or we have still same limitations?
Thanks,
Alex


[Cloud messaging] Send scheduled push notification

Is there way to send scheduled push notification using admin SDK? I want to send scheduled push notification instead of using a cloud function pubsub schedule to send notification on a specific date and time.

To my understanding the Admin SDK or the Firebase Cloud Messaging does not have an API to schedule notifications.
It looks like a common way to handle scheduled notifications is to store the message information in a database and periodically check for notifications that need to be sent. You could achieve this using Cloud Functions or Cloud Tasks API. If you do not want to use either of those, you could try using node-schedule (assuming you will not use it in a Cloud Function).
You could also send a silent message to your clients and handle the display of the notification later on the client side.
Please see this for a more detailed answer. Hope this helps!

Firebase Console also supports message scheduling. The HTTP send API currently does not support scheduling. Admin SDK also does not support scheduling since the SDK talks to the HTTP API.

@chong-shao Any information on when it's gonna be supported in HTTP API, or if it's ever be supported?

Is there any way to achieve this?

The lack of such a feature is a great drawback with firebase, to acheive this functionality we have to write our own cron jobs @kasvith

@Kernelogy true. Although its in the console its a shame they did not expose on firebase admin


does this theme not work with github.io site?

I tried to use this theme to host my github.io site and I followed the set of deployment steps from herehttps://gohugo.io/hosting-and-deployment/hosting-on-github/
but I am getting 404 site not found even though the htmls are committed. Does this theme requires netlify?

Hi @TheThreeCats ,
I  used this theme successfully for my blog (https://tagalong.page) which is still under construction.
Try adding publishDir: docs parameter in config.toml or config.yaml file ... and then in your github repo setting (GitHub Pages sections) make sure to select /docs from master branch.

Hi @TheThreeCats ,
I used this theme successfully for my blog (https://tagalong.page) which is still under construction.
Try adding publishDir: docs parameter in config.toml or config.yaml file ... and then in your github repo setting (GitHub Pages sections) make sure to select /docs from master branch.

It seems like you decided to switch from this theme to mainroad theme. But it doesn't mean that chunky poster theme does not work well with github.io webpage, right?

Hi @TheThreeCats ,
I used this theme successfully for my blog (https://tagalong.page) which is still under construction.
Try adding publishDir: docs parameter in config.toml or config.yaml file ... and then in your github repo setting (GitHub Pages sections) make sure to select /docs from master branch.

It seems like you decided to switch from this theme to mainroad theme. But it doesn't mean that chunky poster theme does not work well with github.io webpage, right?


add impala engineconn-plugin

What is the purpose of the change
EngineConn-Plugin for Impala
Related issues/PRs
Related issues: #4180
Brief change log

Define an executor base on Impala thrift client;
Fix some code styles along with Trino engineconn-plugin;

Checklist

[x] I have read the Contributing Guidelines on pull requests.
[x] I have explained the need for this PR and the problem it solves
[x] I have explained the changes or the new features added to this PR
[x] I have added tests corresponding to this change
[ ] I have updated the documentation to reflect this change
[x] I have verified that this change is backward compatible (If not, please discuss on the Linkis mailing list first)
[x] If this is a code change: I have written unit tests to fully verify the new behavior.

The unit test failed at compile time, please see why.
And the code seems to conflict.


🚀 - create TUI_TIME_VALUE_TRANSFORMER for tui-input-time component

Description
I am encountering issues integrating tui-input-time component due to the absence of a token for handling data types.
I seek the ability to change data types for input and output, similar to what is available in tui-input-date-time. Implementing this feature would greatly improve component flexibility and integration efficiency.

Sounds reasonable. There are already good examples on how to do it, so I'll mark this issue for community contributions, PRs are welcome.

I'd like to submit a PR for this.


clarify agentless task code example

The attribute of pool: server is not clear that server is a meaningful value and not just an example used for the code snippet.
It is not clear enough to immediately know that in order to make the task agentless the value of the pool attribute must be set to the string server.

@juliakm

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@chadbaldwin Thank you! This is a very helpful addition.
#sign-off


Fix SegmentFetcherFactory configuration to be different for controller and server

Codecov Report

Merging #2200 into master will decrease coverage by 0.05%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master    #2200      +/-   ##
==========================================
- Coverage   69.88%   69.82%   -0.06%
==========================================
  Files         824      824
  Lines       39052    39053       +1
  Branches     5246     5246
==========================================
- Hits        27290    27268      -22
- Misses       9966     9988      +22
- Partials     1796     1797       +1

Impacted Files
Coverage Δ

.../common/segment/fetcher/SegmentFetcherFactory.java
84.72% <ø> (-0.21%)
:arrow_down:

...m/linkedin/pinot/common/utils/CommonConstants.java
35.48% <0%> (-0.59%)
:arrow_down:

...m/linkedin/pinot/controller/ControllerStarter.java
59.37% <100%> (ø)
:arrow_up:

.../server/starter/helix/SegmentFetcherAndLoader.java
57.25% <100%> (+0.34%)
:arrow_up:

...in/pinot/core/query/pruner/ValidSegmentPruner.java
57.14% <0%> (-28.58%)
:arrow_down:

...core/operator/dociditerators/AndDocIdIterator.java
51.72% <0%> (-24.14%)
:arrow_down:

...ore/query/scheduler/resources/ResourceManager.java
81.81% <0%> (-13.64%)
:arrow_down:

.../pinot/core/query/reduce/InAndNotInComparison.java
61.29% <0%> (-12.91%)
:arrow_down:

...e/operator/dociditerators/SortedDocIdIterator.java
55.55% <0%> (-8.34%)
:arrow_down:

.../pinot/core/query/scheduler/PriorityScheduler.java
77.46% <0%> (-5.64%)
:arrow_down:

... and 16 more

Continue to review full report at Codecov.

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

This commit changes the way controller is configured with fetchers. Earlier configurations looked like: "pinot.server.fetcher.foo.bar". These will no longer be honored. Instead, the configuration should be changed to "pinot.controller.fetcher.foo.bar".
Server's configurations can remain unchanged.


Statistics & RRD time-table for traffic data per client/overall

It would be great to see some statistics in the WGDashboard.
I would like to have an RRD chart for incoming and outgoing traffic per hour, day, week, month and year.
You could implement a parallel task that checks all clients every 60 seconds and writes the traffic data to the RRD database.
This task could also store the status of the clients to get some statistical data on who was online when and for how long.

same here I need these features as well.

This is available is it not?

Closing for inactivity

This feature is not implemented in the latest version 4.1.0, would still be nice to have some statistics. :-)


Network Mapped Drive Access

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to 'SideBar Drive Lift
Click on '..Network Mapped Drive(LetterPath).'
Files UWP will say "this folder is empty"

Expected behavior
When opening network drive path, files and folders stored on drive should be visible in explorer
Screenshots

Desktop (please complete the following information):

OS Version: [e.g. Windows 10 pro 2004 19041.329]
App version: [v<IP_ADDRESS>]

Additional context
Add any other context about the problem here. Does this problem occur again after restarting the app?

Duplicate of #642


fix #14 - resolve auto arrays

Only works for a single dimension for now (tested in DCD)

Now also works for multi dims arrays that don't use another auto array declaration for the elements.

Legal weed suddenly turned Brian Schott into a genius.


CLOUD-56733 added custom tag to CF stack created by cloudbreak

@biharitomi

Jenkins build failed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/2433/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/2435/


Update emsdk to 1.39.20

#75716 has the same changes, let's sync it!

@JohnTitor: no appropriate reviewer found, use r? to override


Revert channel parts of #1720

Fixes #

Proposed Changes

Revert channel parts of #1720
It breaks conformance tests

Release Note

:wastebasket: Reverted the channel related changes done in https://github.com/knative-sandbox/eventing-kafka-broker/pull/1720. The reference information for metrics was not built successfully for channels.

Docs

/test pull-knative-sandbox-eventing-kafka-broker-integration-tests

/cherypick release-1.2

/cherypick release-1.2

FYI, If you want this PR to be part of tomorrow's patch release we need a patch before we can merge this one.

@pierDipi https://github.com/knative-sandbox/eventing-kafka-broker/pull/1835
Thanks!

/retest

/test pull-knative-sandbox-eventing-kafka-broker-upgrade-tests


Remove CNode implementation

A couple of things:

we always use NIF implementation
in the next PR I would like to remove GenServer and convert this repo to a plain module to limit message passing
Erlang relies on OpenSSL so I would assume it's pretty safe to use it as a NIF
I didn't see people writing CNodes

I didn't see people writing CNodes

One day we'll make Unifex support ports, and migration from C nodes should be seamless ;) But I agree with the rest of your points


Stream play return printIn need a message

Hi community,
I try to use this module for play audio sound in streaming.
After running play method, i've this error :
printIn needs a message
You can see my code :
async function play(remoteURL) {
  const player = new Player(remoteURL, {
    continuesToPlayInBackground: true
  });

  player.play(() => {
    alert("yeah");
  });
}

My remoteURL is a audio streaming.
Anyone have already this issue ?
Thank you !

Same problem here.


configurable pvc wait timeout

Could PVC creation timeout be made configurable. Not it hardcoded to 15 times retires after each 4 seconds. In our conditions it takes a little bit more to bound pvc so the cluster can never up.

Not it hardcoded to 15 times retires after each 4 seconds.

We could make it more robust by retrying and backoff.

Could PVC creation timeout be made configurable.

I don't think this is user configuration. User shouldn't know how much time waiting for PVC to be bound.

We could make it more robust by retrying and backoff.

Should an issue be created for that?

I don't think this is user configuration. User shouldn't know how much time waiting for PVC to be bound.
Why it could be a problem to have the number of retries not hardcoded but configurable vie env variable for example? It could be defaulted to 15 by default so it will be a black box for most of users.

Let's use this issue.
It is implementation details. We shouldn't expose that.


Fix handling of CUBICSPLINE morph animations

This PR extracts the sparse animation data for the CUBICSPLINE case (which has 3 values instead of just 1) and creates the curve with inTangent and outTangent. This was quite tricky to get right from existing models, so I made my own model for testing all values:
gltf_unsampled.zip
Then I compiled a little image which explains the sparse data layout:

And a little script to quickly test the indexing code:
console.clear();
values = [0, 0, 0.10000000149011612, -0.23000000417232513, 0.021052630618214607, 0.1789473593235016, -0.02105262689292431, 0.029034702107310295, 0.7623739242553711, 1.5165784358978271, -0.013775895349681377, 0.018977485597133636, 0.05862068384885788, -0.02413792908191681, -0.20000000298023224, 0.11999999731779099, 0, 0]
numValues = 3;
numCurves = 2;
offsetPerTime = 6;
for (i = 0; i < numCurves; i++) {
  for (j = 0; j<3; j++) {
    var x = j * offsetPerTime + i;
    var y = j * offsetPerTime + numCurves + i;
    var z = j * offsetPerTime + numCurves * 2 + i;
    console.log('curve', i, 'time', j, 'inTan', x, values[x]);
    console.log('curve', i, 'time', j, 'value', y, values[y]);
    console.log('curve', i, 'time', j, 'ouTan', z, values[z]);
  }
}

Morphed CUBICSPLINE animations also fail for the built-in engine glTF loader: https://playcanvas.com/viewer (I don't know if that is the same issue of reading the animation data correctly)
I confirm I have signed the Contributor License Agreement.

Closing this PR since this repo is now retired in favor of the glTF parser in the core engine repo.


Azure Function SQL Trigger Binding not firing once automatically stopped

I’ve been using the Trigger Binding for SQL lately, and I noticed that at some point I get a “Stopped the listener”, if the trigger doesn’t detect changes for some time (1 hour for example) message

And after that, the trigger doesn’t work anymore. If I make some changes to the database, they do not fire up the trigger. I order to make everything working again, I have to go to the Azure Portal, and interact in some way with the deployed Azure function, for example use the “Live metrics” feature.
Is this a known issue or I’m doing something wrong?

Maybe related to this issue #892?

Hi @yorek, could you share the following information:
Timestamp:
Function App version:
Invocation ID:
Region:

@fabiocav Do you know why the listeners would be stopped? If our listener is stopped then it's definitely expected you won't get any more trigger activations...but that's not something we control AFAIK.

Hi @yorek, could you share the following information: Timestamp: Function App version: Invocation ID: Region:

Timestamp: 2023-11-10T19:38:53.3543291Z
Invocation ID: 11ee1096-b090-4156-8373-98e838ae9c7b
Region: Central US
Function App version: 4
This is when the function was executed. After a while (hours? minutes? not sure...) you'll see that it stops listening to the SQL changes

Not sure if it's related, but a quick search shows that other people have had similar looking issues in the past with other triggers.
https://stackoverflow.com/questions/63446501/azure-function-trigger-listener-stops-running-abruptly
Given that the extension itself doesn't control anything about when the listener is started/stopped it seems likely this is something with the host, but we'll look into the logs and see if we can find any more information out.


[KeyVault] - Migrate Key Vault Admin package to Core V2

What

Migrate @azure/keyvault-admin to core V2
Migrate KeyVaultBackupClient and KeyVaultAccessControlClient to core CAE
Bring @azure/core-rest-pipeline back down to core-tracing preview 11

Why
This PR proves out two important things: it demonstrates that core continuous access evaluation works for both container
registry (already done) and Key Vault (this PR). It also demonstrates the migration path for Core V2 for Key Vault.
Finally, I noticed that core-client is still on core-rest-pipeline 1.0.3 which means we cannot use core-tracing preview 12 yet. This
change ensures all core V2 packages reference the same version of core-tracing.

test comment

/azp run js - keyvault - ci

/azp run js - keyvault - ci

/azp run js - keyvault - ci

@jeremymeng yeah I regenerated from swagger and also do not see any userAgent changes come up in generated code... Not sure if that was added after I regenerated possibly?
cc @sarangan12


TiClient Region Error OPS on a new setup of TiDB 2.0.8

Question
I set up an empty cluster of TiKV and ran a few create table commands. I started receiving TiClient Errors.

Next, I ran bin/tikv-ctl --db /data/tikv/db bad-regions  command and received below response
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: "IO error: While lock file: /data/tikv/db/LOCK: Resource temporarily unavailable"', libcore/result.rs:945:5
stack backtrace:
   0: std::sys::unix::backtrace::tracing::imp::unwind_backtrace
             at libstd/sys/unix/backtrace/tracing/gcc_s.rs:49
   1: std::panicking::default_hook::{{closure}}
             at libstd/sys_common/backtrace.rs:71
   2: std::panicking::rust_panic_with_hook
             at libstd/panicking.rs:227
             at libstd/panicking.rs:475
   3: std::panicking::continue_panic_fmt
             at libstd/panicking.rs:390
   4: rust_begin_unwind
             at libstd/panicking.rs:325
   5: core::panicking::panic_fmt
             at libcore/panicking.rs:77
   6: core::result::unwrap_failed
             at /checkout/src/libcore/macros.rs:26
   7: tikv_ctl::new_debug_executor
             at /checkout/src/libcore/result.rs:782
             at src/bin/tikv-ctl.rs:94
   8: tikv_ctl::main
             at src/bin/tikv-ctl.rs:1247
   9: std::rt::lang_start::{{closure}}
             at /checkout/src/libstd/rt.rs:74
  10: main
  11: __libc_start_main
  12: <unknown>

Before asking a question, make sure you have:

Searched existing Stack Overflow questions.
Yes
Googled your question.
Yes
Searched open and closed GitHub issues
Read the documentation:

TiKV Readme
TiKV Doc
TiKV-Ctl Doc
TiDB & TiKV Stack Doc

Hi

When you are operating tikv-ctl over local data, you need to make sure that no other TiKV instance is using it (i.e. offline). If you want to use it online, you can specify a TiKV address.

TiClient Region Error is not a critical error, which will be automatically retried. For example, during a request when a region is split there will be region error. However this request will be retried internally.


Right alignment behaviour different to what's stated in the docs

With 'right' alignment and when the right screen-edge obscures the Popup, I find that it's the right edge of the Popup that repositions itself to the left edge of the PlacementTarget and not the left edge of the Popup as stated in the docs.
Parragraph:
  | A position of the Popup control that aligns its left edge with the right edge of the PlacementTarget and aligns its upper edge with the upper edge of the PlacementTarget. If the right screen-edge obscures the Popup, the control repositions itself so that its left edge aligns with the left edge of the PlacementTarget. If the left screen-edge obscures the Popup, the control repositions itself so that its left edge aligns with the left screen-edge. If the upper or lower screen-edge obscures the Popup, the control then repositions itself to align with the obscuring screen edge.
-- | --

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

ID: 8e09cd6b-9b1f-473e-7ba7-d1fa8dd2a0ac
Version Independent ID: 34de618c-16b1-e504-b94a-032df08a6c8d
Content: PlacementMode Enum (System.Windows.Controls.Primitives)
Content Source: xml/System.Windows.Controls.Primitives/PlacementMode.xml
Product: dotnet-api
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

@RaulBastian Thank you for pointing this out. We'll add this to our backlog and investigate. Thanks again!


verify data

verify the data output, if raw data is the same as the output in the json and csv files.

verfiy manually
verify by agreggating values
check country which has two tables
check country which has two names
check first and last table which gets parsed
check with different country name structure: before table, first row, second row
Check this for all reports.

compare total values with sum of single values


[Xenoblade X] Anti-Aliasing pack broken, causes strange artifacting +potential fix

Here is a snippet of a screenshot of xenoblade x at 1440p with the anti-aliasing graphics pack set to 'Disable Native AA'

The issue here is two-fold. Firstly, there is still some sort of anti aliasing. (Look at the branches of the tree in the background)
Second, and far worse, is enabling the pack, regardless of your settings, causes strange vertical artifacts throughout the screen. (Note the odd scattering of blue pixels around the item diamond and the line next to it. There are also some strange 'cutouts' on my character's arm piece.)
I've tested with OpenGL and Vulkan, both have the same issues.
I only have an AMD gpu, so I don't know if this happens on other platforms. (If you have an Nvidia gpu please test it yourself!)
I do not know what causes this, other than the fact that it is caused by the pack.
I've attached a zip of an alternate version of the pack, which just blanks out the anti-aliasing shader by returning (0,0,0,0).
This doesn't have the options of the original pack, only disabling anti-aliasing, but does actually work.
AntiAliasingRemover.zip

I feel like I released a AA pack for this game. But it doesn't go by that name. One of these days I'll catch up and see whats up.

I have a NVIDIA GeForce GTX 1070 and can confirm this issue.

Here is an alternate fix - that works with both openGL and vulkan. Put monolib.cpk from this zip over the monolib.cpk in Xenoblade X > content
This disables anti-aliasing in the game's internal, not user-accessible graphics config file.
monolib - no aa.zip

Would it be worth it to remove the remove-AA option from the current AA mod and add threethan's graphics pack above labeled as 'Anti-Aliasing Removal' next to the current 'Anti-Aliasing' mod?


[DOC] Dask processing options docs updates

Fixes #2882 and addresses the first two points of #2897.

Thanks @dkillick.


IdP sends successful LogoutResponse from AuthnRequest?

I am using the onelogin/php-saml library to allow a Service Provider (SP) to initiate auth requests to an IdP (OneLogin) with SP, IdP, organisation and security settings that roughly described below:
 idp:
      entityId: "https://app.onelogin.com/saml/metadata/<metadataID>"
      singleLogoutService:
        binding: "urn:oasis:names:tc:SAML:2.0:bindings:HTTP-Redirect"
        url: "https://<subdomain>.onelogin.com/client/apps"
      singleSignOnService:
        binding: "urn:oasis:names:tc:SAML:2.0:bindings:HTTP-Redirect"
        url: "<http-redirect-slo-url>"
      # public key for x509 cert in OneLogin
      certFingerprint: "<pubkey>"
      certFingerprintAlgorithm: "sha256"
sp:
      assertionConsumerService:
        binding: "urn:oasis:names:tc:SAML:2.0:bindings:HTTP-Post"
        url: "<SPACSEndpoint>"
      entityId: "<SPMetadataEndpoint>"
      singleLogoutService:
        binding: "urn:oasis:names:tc:SAML:2.0:bindings:HTTP-Redirect"
        url: "<SPLogoutEndpoint>"
      NameIDFormat: "urn:oasis:names:tc:SAML:1.1:nameid-format:emailAddress"

security:
      authnRequestsSigned: false
      logoutRequestSigned: false
      logoutResponseSigned: false
      nameIdEncrypted: false
      requestedAuthnContext: true
      signMetadata: false
      signatureAlgorithm: "http://www.w3.org/2001/04/xmldsig-more#rsa-sha256"
      wantAssertionsSigned: false
      wantMessagesSigned: false
      wantNameIdEncrypted: false
      wantXMLValidation: true

I make a GET request on the Service Provider to saml/login that checks if the user is already authenticated on the SP, and if not, to call php-saml's ->login() method which sends a GET SAMLRequest to the IdP using the http-redirect/slo endpoint.  It looks something like this:
<samlp:AuthnRequest xmlns:samlp="urn:oasis:names:tc:SAML:2.0:protocol"
                    xmlns:saml="urn:oasis:names:tc:SAML:2.0:assertion"
                    ID=""
                    Version="2.0"
                    ProviderName="<placeholder>"
                    IsPassive="true"
                    IssueInstant="2018-07-11T23:13:29Z"
                    Destination="https://<domain>.onelogin.com/trust/saml2/http-redirect/slo/<id>"
                    ProtocolBinding="urn:oasis:names:tc:SAML:2.0:bindings:HTTP-Post"
                    AssertionConsumerServiceURL="<SPACSEndpoint>"
                    >
    <saml:Issuer><SPMetadataEndpoint></saml:Issuer>
    <samlp:NameIDPolicy Format="urn:oasis:names:tc:SAML:1.1:nameid-format:emailAddress"
                        AllowCreate="true"
                        />
    <samlp:RequestedAuthnContext Comparison="exact">
        <saml:AuthnContextClassRef>urn:oasis:names:tc:SAML:2.0:ac:classes:PasswordProtectedTransport</saml:AuthnContextClassRef>
    </samlp:RequestedAuthnContext>
</samlp:AuthnRequest>

the IdP then comes back with a LogoutResponse (even though the user is already authenticated at the IdP).
<samlp:LogoutResponse IssueInstant="2018-07-11T23:13:27Z"
                      Version="2.0"
                      ID=""
                      xmlns:samlp="urn:oasis:names:tc:SAML:2.0:protocol"
                      > <saml:Issuer xmlns:saml="urn:oasis:names:tc:SAML:2.0:assertion">https://app.onelogin.com/saml/metadata/<metadataId></saml:Issuer> <samlp:Status> <samlp:StatusCode Value="urn:oasis:names:tc:SAML:2.0:status:Success" /> <samlp:StatusMessage>Successfully logged out from service </samlp:StatusMessage> </samlp:Status> </samlp:LogoutResponse>

which ultimately redirects the user to the IdP's login page.
I was expecting the IdP to call the ACS endpoint on the SP to tell the SP the user is already authenticated on the IdP, for the SP to authenticate the user.
I suspect this is caused by a mis-configuration.  I don't have visibility into what the IdP is doing to determine sending back a LogoutResponse instead of calling our ACS endpoint.
The ultimate problem I'm trying to solve here is for OneLogin to not prompt for MFA when being accessed from the IdP side if the SP/IdP is already authenticated.
Any help/guidance would be appreciated.

For SSO flow, you are using wrong IdP endpoint.
You may use
http-redirect/sso

not
http-redirect/slo

I missed that, thanks so much @pitbulk

@pitbulk Do you happen to know if there is a way for the IdP OneLogin to call the Login URL first, before prompting for MFA only when it's ready to call ACS?


ReadMe need to be updated to reflex ENV variables changes

Duplicate issue related to ENV vars, Closing

duplicate issue

duplicate issue


Update 2022-12-12-kubernetes-release-artifact-signing.md

Fixed provenance url to match the correct bucket.

/kind cleanup

/retitle Fix broken link in “Kubernetes 1.26: We're now signing our binary release artifacts!”
/remove-kind cleanup


Remove game.mapforced

Just duplicates game.map

Running tests on release locally works fine but explodes on github sooooo that's fun

This change is part of my GameMapManager refactor/removal.


⚠ Bofrost.lu has degraded performance

In c6a871e, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2629 ms

Resolved: Bofrost.lu performance has improved in 11a9859 after 6 minutes.


Added performance benchmarks for platform channels

As outlined in the doc: https://docs.google.com/document/d/1oNLxJr_ZqjENVhF94-PqxsGPx0qGXx-pRJxXL6LSagc/edit
This just adds tests for iOS and Android for BasicMessageChannels with the StandardMessageCodec.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

There's a lot of boilerplate in this PR, I hope I reviewed all the right files...
Would probably be good if we found a way to not duplicate all this boilerplate for different benchmarks.

I was able to remove most of that boilerplate.  The test is a bit more fragile now since I needed to keep some of android/ and ios/ to work.  If the format of the template changes it could break these tests.  It also adds a slight complication if people want to run the tests outside of devicelab for debugging purposes.  I added a note in the README file about it.


Swiping events in app

In my app, I need to act when a swipe event occurs.

Would exposing the Gallery.Msg type sufficient for your use case?
You can then handle any event you would like to act by case matching on the msg in your own update function.

Yes that sounds perfect.

It seems Msg is already exposed, but it needs to be Msg(..) to expose the message types like "Next".
I'll send through a PR for this.


FormatSheet display behind OS keyboard on IOS 17

I update to Xcode15 - IOS 17 and this bug happen, the format text sheet is behind the os keyboard

Oh snap! This doesn't happen in iOS 16, right?
Are you using the latest release? I changed the format toolbar to use a medium size and to not dim the background.
So it may not be an iOS 17 thing, but rather that I overlooked this scenario.

It's oke when I test with 16.4 but it maybe still happen on IOS > 16.4
I'm using Xcode 15.0.1 now

func prefersMediumSize() -> some View {
        if #available(iOS 16.4, *) {
            self.presentationDetents([.medium])
                .presentationBackgroundInteraction(
                    .enabled(upThrough: .medium)
                )
        } else if #available(iOS 16, *) {
            self.presentationDetents([.medium])
        } else {
            self
        }
    }

I found that everything is working oke when I just use self.presentationDetents([.medium]) for IOS higher than 16.4. I wonder why you have to this code
.presentationBackgroundInteraction(
                    .enabled(upThrough: .medium)
                )

Hello @minhtien1403 are you able to solve this problem. I am trying to use the library on IOS and i am having the same issue. IOS 16.6

@hardeyem yeah, I have explain my solution about and its work, you have to edit source code of this library or waiting for a hotfix from  @danielsaidi
https://developer.apple.com/documentation/xcode/editing-a-package-dependency-as-a-local-package

Thank you @minhtien1403

I'll revert this change.

Ok, I have reverted the 0.9 addition that kept the background active when the format sheet was presented, since this caused the sheet to display behind the keyboard.
I would love for this behave like in Notes, where the format sheet dismisses the keyboard but keeps the text field editable. When I tried this and dismissed the keyboard when presenting the sheet, the text field loses focus.
Anyhow, we can improve this over time. For now, reverting this will restore the previous behavior. 👍


S3 Download Url with attachment attribute (mp3 opens file instead of downloading)

Hi there,
I've been scratching my head for a while to be able to bring a download link (for a mp3 file) to work.
I tried the redirect method but it opens the file in the browser instead of downloading it.
I also tried with send data but it copies the file first which makes it useless (it takes too much processing time).
What I want to achieve is to stream the S3 file so that the user can download it directly.
I tried to add the attribute "content_disposition: "Attachment" in my carrier wave initializer without success.
I suspect this could be achieved with the S3 object
def download_url
  S3 = AWS::S3.new.buckets[ 'bucket_name' ] # This can be done elsewhere as well.

  S3.objects[ self.path ].url_for( :read,
    expires_in: 60.minutes,
    use_ssl:    true,
    response_content_disposition: "attachment; filename='#{attachment_file_name}'" ).to_s
end

But I don't know how to get access to the S3 object with carrierwave-aws (and I don't even know if the above code would work).
I'm sure this is pretty straight-forward to do but I'm stuck.
Any ideas?
Thanks!

Vincent

You don't get direct access to the S3 object, but you can add custom attributes to it with aws_read_options. This is something you can do for a particular uploader:
  configure do |config|
    config.aws_read_options = {
      content_disposition: 'attachment; filename='hardcoded_filename.mp3'
    }
  end

That will help with reading, but that still requires pulling the data down. Instead, you may want to set the content_disposition when writing the content. In that case, use:
  configure do |config|
    config.aws_write_options = {
      content_disposition: 'attachment'
    }
  end

Hope that helps!

I tried to do what you mentioned but this doesn't change the fact that it copies the file first which takes 10-15sec to process. First option is not possible since I need to retrieve the filename from the database.
Still looking!
Thanks for your help,

Vincent

If you write the header along with the content you can just use the generated url rather than copying the file locally. If you have to use read there is no way around downloading the file to the local server.

hi @musixite Your code is working for aws-sdk-v1, since carrierwave-aws relies on aws-sdk v2, it is not working anymore. However, in v2, aws-sdk has a method :presigned_url defined in Aws::S3::Object, you can use this method to construct the direct download url for a s3 object:
s3 = Aws::S3::Resource.new
# YOUR-OBJECT-KEY should be the relative path of the object like 'uploads/user/logo/123/pic.png'
obj = s3.bucket('YOUR-BUCKET-NAME').object('YOUR-OBJECT-KEY')
url = obj.presined_url(:get, expires_in: 3600, response_content_disposition: "attachment; filename='FILENAME'")


Windows Standalone with IL2CPP does not work

Changing from IL2CPP to mono and the build completes successfully

I have the same problem but im on mono :(

nevermind i just need to switch to .net 2.0 rather than .net 2.0 subset

Confirm. I can't build with Scripting backend IL2CPP. The same error: ArgumentException: The Assembly Mono.WebBrowser is referenced by System.Windows.Forms

Confirmed, as IL2CPP becomes more and more important in Unity...

I can't either 2019.3.13

I think this doesn,t work with IL2CPP backend. I have tried with .NET 4.x and .NET (standar) 2.0 on Unity 2019.4.15 (LTS).
With Mono everything works fine.
What are the right configuration to make it work with IL2CPP? because on the "README" says that "Mono/IL2CPP backends supported."

I had the same problem, and solved by this.
I don't understand how dll works in Unity very well, but anyway it works in my local.

Unity 2021.1.9f1
IL2CPP
.NET 4.X

make directory Assets/Plugins
copy Mono.WebBlorwser.dll and Mono.Posix.dll from <Your Unity Installation Directory>\Unity\2021.1.9f1\Editor\Data\MonoBleedingEdge\lib\mono\4.7-api\
confirm in the inspector window that these the dll assets are used for Standalone (maybe windows only?) and Editor is excluded.


ci: Add quarantine capabilities to k8s-all jenkinsfile

test-missed-k8s

test failure was expected, will merge


Roadmap design change

In the revision of the LP design, we agreed to put the design on the S model, with more distributed information for the scroll bar.

It looks like this

Nice!


Use conditionals for GOOS and GOARCH to allow cross-compiles

As discussed in prometheus/consul_exporter#1, this change make cross-compiling possible with the common Makefiles.
One thing I think needs to be thought through before merge, though: Because $(ARCHIVE) builds off $(BINARY), which doesn't include suffix, running a make archive with a binary sitting there will use the binary regardless of whether it is the correct architecture. One answer to this would be to make BINARY be $(TARGET).$(SUFFIX), and then maybe include a little convenience step in default that would cp the last binary compile to the unsuffixed name if necessary.
Currently you have to make sure to do a make clean before each cross-compile.

:+1: in general.
Not sure yet what I prefer around the $(ARCHIVE) dependency handling. Separately, we might even want to put the $(SUFFIX) into the archive name itself. (Actually there was even a PR in the node exporter (https://github.com/prometheus/node_exporter/pull/49) to optionally put the git SHA in there).
For now I'd even be happy to merge this as it is if that's ok with you, since that gives us some more time to think about what the proper way here is.
I can imagine @grobie or @matthiasr could possibly have opinions on this...

:+1: for this change, since it just honors the wishes of whoever is building.
Long term, if we want to provide building all the architectures at once each should put the artifacts in separate directories so that they can be tarred up separately and kept separately. I'd have to research what the most popular convention is but I would feel no surprise if prometheus-x.y.z.tar.xz is the source tarball, prometheus-x.y.z-amd64.tar.xz is the AMD64 binaries but inside the file names are architecture independent. Tying into the cookbook/deployment discussion the other day, this would sensibly allow determining which tarball to download and unpack from the machine architecture but require no gymnastics after unpacking.

When I say "we" I of course mean The Prometheus Contributors.

Great, thanks for your input! I'll just merge this one for now then and welcome future changes to implement what you're suggesting :)

Cool. Suffix does end up in the archive name, so there's no problem there. It's just the intermediate step.
Platform/arch build dirs seems like a good solution for keeping those artifacts separate. I agree, @matthiasr, you do want the filenames to be agnostic once you extract them.


How can I use substrate_frame_rpc_system & sc_client_api modules on substrate-node-template project?

Hi, Everyone.
I am getting this error.
Please help me.
Compiling node-template v4.0.0-dev (/home/mersaid/Documents/HARRY/Task/Climacoin/node)
error[E0432]: unresolved import substrate_frame_rpc_system::LightSystem
--> node/src/rpc.rs:196:35
|
196 |     use substrate_frame_rpc_system::{LightSystem, SystemApi};
|                                      ^^^^^^^^^^^ no LightSystem in the root
error[E0433]: failed to resolve: could not find light in sc_client_api
--> node/src/rpc.rs:61:48
|
61 |     pub remote_blockchain: Arc<dyn sc_client_api::light::RemoteBlockchain>,
|                                                   ^^^^^ could not find light in sc_client_api
error[E0433]: failed to resolve: could not find light in sc_client_api
--> node/src/rpc.rs:192:20
|
192 |     F: sc_client_api::light::Fetcher + 'static,
|                       ^^^^^ could not find light in sc_client_api
Here is Error Image.

How can I solve this problem?
Best Regards.

Here is my Codebase.
Please check carefully and help me.
https://github.com/WB-Expert5007/Poa_to_Pos
Best Regards.


Issues regarding performance

I used the open-source model LLM2CLIP-EVA02-L-14-336 to test the following images and sentences, but the results were very poor. Did I use it incorrectly? I didn't use xformers.

captions = ["a student with a red hat"]
sim=[0.1643, 0.0739]
However, the results with model EVA02-L-14-336 were actually correct.
sim = [0.1539, 0.1591]

You can test it yourself with images and descriptions; similar descriptions might actually have lower similarity.

Did you make changes here? I don't see any difference.

I'm curious about what you changed. If it's the part below, it should have thrown an error before the modification, right?

captions = ["a deer"]
LLM2CLIP-EVA02-L-14-336 sim=[[0.3325, 0.2966]]
EVA02-L-14-336 sim = [[0.1321, 0.2197]]
The performance is still very poor.

Please pull the latest code. I have tested it on the new example you provided, and here are the test results.
sim = [0.2690, 0.2939]

I have already updated the code under the llm2clip/eva_clip path, but the results are still incorrect.

I have already updated the code under the llm2clip/eva_clip path, but the results are still incorrect.

I am conducting the test on the commit ca5634faf. Please verify that the code version is consistent.

Yes

Here is my test code, and this is the output result
tensor([[0.2690, 0.2939]], device='cuda:0', dtype=torch.float16)
import os

os.environ["CUDA_VISIBLE_DEVICES"] = "0"
import torch
from PIL import Image
from llm2vec import LLM2Vec
from eva_clip import create_model_and_transforms
from transformers import AutoModel, AutoConfig, AutoTokenizer

model, _, preprocess_val  = create_model_and_transforms('EVA02-CLIP-L-14-336', force_custom_clip=True)
ckpt = torch.load('LLM2CLIP-EVA02-L-14-336.pt')
model.load_state_dict(ckpt)
model = model.cuda().eval()

llm_model_name = 'microsoft/LLM2CLIP-Llama-3-8B-Instruct-CC-Finetuned'
config = AutoConfig.from_pretrained(
    llm_model_name, trust_remote_code=True
)

llm_model_name = 'microsoft/LLM2CLIP-Llama-3-8B-Instruct-CC-Finetuned'
config = AutoConfig.from_pretrained(
    llm_model_name, trust_remote_code=True
)
llm_model = AutoModel.from_pretrained(llm_model_name, torch_dtype=torch.bfloat16, config=config, trust_remote_code=True)
tokenizer = AutoTokenizer.from_pretrained(llm_model_name)
llm_model.config._name_or_path = 'meta-llama/Meta-Llama-3-8B-Instruct' #  Workaround for LLM2VEC
l2v = LLM2Vec(llm_model, tokenizer, pooling_mode="mean", max_length=512, doc_max_length=512)

captions = ["a deer"]

image_path_1 = "im1.jpg"
image_path_2 = "im2.jpg"

image_1 = preprocess_val(Image.open(image_path_1)).cuda()
image_2 = preprocess_val(Image.open(image_path_2)).cuda()
image = torch.stack([image_1, image_2])

text_features = l2v.encode(captions, convert_to_tensor=True).to('cuda')

with torch.no_grad(), torch.cuda.amp.autocast():
    image_features = model.encode_image(image)
    text_features = model.encode_text(text_features)

    image_features /= image_features.norm(dim=-1, keepdim=True)
    text_features /= text_features.norm(dim=-1, keepdim=True)

print("sim:", text_features @ image_features.T)

Here is my test code, and this is the output result tensor([[0.2690, 0.2939]], device='cuda:0', dtype=torch.float16)
import os
os.environ["CUDA_VISIBLE_DEVICES"] = "0"
import torch
from PIL import Image
from llm2vec import LLM2Vec
from eva_clip import create_model_and_transforms
from transformers import AutoModel, AutoConfig, AutoTokenizer
model, _, preprocess_val  = create_model_and_transforms('EVA02-CLIP-L-14-336', force_custom_clip=True)
ckpt = torch.load('LLM2CLIP-EVA02-L-14-336.pt')
model.load_state_dict(ckpt)
model = model.cuda().eval()
llm_model_name = 'microsoft/LLM2CLIP-Llama-3-8B-Instruct-CC-Finetuned'
config = AutoConfig.from_pretrained(
llm_model_name, trust_remote_code=True
)
llm_model_name = 'microsoft/LLM2CLIP-Llama-3-8B-Instruct-CC-Finetuned'
config = AutoConfig.from_pretrained(
llm_model_name, trust_remote_code=True
)
llm_model = AutoModel.from_pretrained(llm_model_name, torch_dtype=torch.bfloat16, config=config, trust_remote_code=True)
tokenizer = AutoTokenizer.from_pretrained(llm_model_name)
llm_model.config._name_or_path = 'meta-llama/Meta-Llama-3-8B-Instruct' #  Workaround for LLM2VEC
l2v = LLM2Vec(llm_model, tokenizer, pooling_mode="mean", max_length=512, doc_max_length=512)
captions = ["a deer"]
image_path_1 = "im1.jpg"
image_path_2 = "im2.jpg"
image_1 = preprocess_val(Image.open(image_path_1)).cuda()
image_2 = preprocess_val(Image.open(image_path_2)).cuda()
image = torch.stack([image_1, image_2])
text_features = l2v.encode(captions, convert_to_tensor=True).to('cuda')
with torch.no_grad(), torch.cuda.amp.autocast():
image_features = model.encode_image(image)
text_features = model.encode_text(text_features)
image_features /= image_features.norm(dim=-1, keepdim=True)
text_features /= text_features.norm(dim=-1, keepdim=True)

print("sim:", text_features @ image_features.T)

Here are the two images you provided. If there are no unexpected issues, you should be able to obtain the same results.

I only changed trust_remote_code to False, and the result was incorrect.
[[0.3323, 0.2966]]

Setting trust_remote_code=True is essential.

I have already downloaded the LLM2CLIP-Llama-3-8B-Instruct-CC-Finetuned model. What else should I do to run it offline?

Am I missing any files?

I have already downloaded the LLM2CLIP-Llama-3-8B-Instruct-CC-Finetuned model. What else should I do to run it offline?

The trust_remote_code=True flag is used for models whose code lives on the hub rather than natively in the Transformers library.
The use of trust_remote_code does not conflict with offline operation.
After downloading the model, it can be executed offline.


[SUPPORT] Exit code 137 (interrupted by signal 9: SIGKILL) when StreamWriteFunction detect object size

Describe the problem you faced
In StreamWriteFunction#detect, when ObjectSizeCalculator#getObjectSize is executed and the process would exit with code 137 (interrupted by signal 9: SIGKILL). After debugging, I found that the process was killed by system when executing ServiceabilityAgentSupport#needSudo in the ServiceabilityAgentSupport's constructor function. Specifically, it is exit when executing out.flush() in ServiceabilityAgentSupport#callAgent(null, true, style). And the current user has sudo with NOPASSWD. The code was running well with hudi version 0.11.0 and it is running well in windows. Anything i missed?
Environment Description

Hudi version : 0.14.0-SNAPSHOT

JDK version : 1.8.0+362

System version : macOS Monterey 12.0.1

Hadoop version : 3.1.2

Storage (HDFS/S3/GCS..) : HDFS

Running on Docker? (yes/no) : no

Interesting, not an expert of MacOS, we can have more search here

Interesting, not an expert of MacOS, we can have more search here

I can't find any solution in the internet, even the openjdk-jol-core's issues also have nothing to help this.
And today, i try to use jdk-11 to run the code, it works.
I believe that the macOS cause this problem, but i don't know how to fix it.

Yeah, we can keep this issue open and still thanks for the feedback.

Yeah, it is introduced by https://github.com/apache/hudi/pull/6657, @codope , do you have any thoughts here?

https://stackoverflow.com/questions/64593567/run-jol-core-exception-the-exception-message-is-process-finished-with-exit-cod
A stack overflow question documenting this. Pasting the answer/solution that one of the commenter + OP agreed on.
OP
After testing I have found the problem and documented it here in the hope that it will help you. The problem occurs because jol-core is an extension for the OpenJDK and cannot be used on the Oracle Java Development Kit. For this reason I changed the local JDK to OpenJDK and ran the code in question to get the correct result

Commentor
Because the JDK used by IDEA by default is not OpenJDK, just replace the Project SDK with Open JDK

Swapping from an Oracle JDK (which will throw the error) to OpenJDK fixed my issue.

Close because we already have a solution ~


Where will atom go when Microsoft(VScode) own github?

As I was saying,  If github becomes part of Microsoft, atom as well .As a company which needs the profits, it would be such a waste to focus on both projects, I was just wondering if there will be a choice between them(atom & VScode), and if there is, enlighten me which gonna survive, plz?

It's been asked via issues about 100 times.

@nkkollaw
I know it's a boring question, I'm just so obssessed to see big companies fight with each other :) .
And I'm a little sad github has to be one of them, it's like she's gonna lose her neutral when it happens.

@g1mi: it's not a boring question, it's been asked and answer before many times, which means every time someone spams issues with this question, someone wastes his time.

Thanks for your concern.
Atom remains key to GitHub. Our product roadmap is set, and the team will continue all of their work.

If you want to be further reassured that Atom will continue, see this comment by GitHub's future CEO: https://www.reddit.com/r/AMA/comments/8pc8mf/im_nat_friedman_future_ceo_of_github_ama/e0a235q/

Developers are really particular about their setup, and choosing an editor is one of the most personal decisions a developer makes.  The last thing I would want to do is take that decision away from Atom users.
Atom is a fantastic editor with a healthy community, adoring fans, excellent design, and a promising foray into real-time collaboration. At Microsoft, we already use every editor from Atom to VS Code to Sublime to Vim, and we want developers to use any editor they prefer with GitHub.
So we will continue to develop and support both Atom and VS Code going forward.


Bungeecord API

Hello,
Would you be able to update the API for bungeecord luckperms?

The newest API version is bundled in the newest LuckPerms version, you can download it from https://luckperms.net/download


[new locale] Added locale ug-cn

Hey guys, here is the locale and test file for uyghur language.
Thanks.

Hi @marwahaha could you plz tell me what's wrong with my title?
@jsf-clabot i only have one github account, what should i do to finish the signing?

I fixed your title.
@boyaq  - some of your commits are made with an email address that's linked to your Github account, but some aren't. You can tell by looking at the Commits tab on Github. Notice that some have your profile and some don't. You can add the missing email address to your Github account and it will work out!

ok, let me open a new pull request with a fresh fork


refactor: clean-up components

remove editor component
move layout to VariablesEditor
introduce VariablesMasterToolbar component
clean-up css
remove ValidationMessagesRows component

Next I am going to look into react contexts to reduce the arguments of components.

Next I am going to look into react contexts to reduce the arguments of components.

E.g. you can have a look at the form editor


don't use the clang node's USR for subscript decls

Resolves rdar://117130545
When importing subscript methods from Objective-C, we currently use the Clang node's USR as the USR for the subscript decl as a whole. However, in the following code, both classes' subscripts receive the same USR:
@interface MyClass : NSObject
- (NSUInteger)objectAtIndexedSubscript:(NSUInteger)idx;
@end

@interface MyDerivedClass : MyClass
- (void)setObject:(NSUInteger)obj atIndexedSubscript:(NSUInteger)idx;
@end

This is because the derived class's subscript inherits its subscript getter from its base class. In the Clang importer, a duplicate subscript getter is generated that overrides the base class's method and receives the same USR.
This PR changes USR generation to specifically skip using the Clang node's USR for subscript decls, to avoid this situation.

@swift-ci Please test

Is this a problem for computed properties as well?

@slavapestov Are computed properties allowed to have a separate setter/getter between a superclass and subclass? The thing that makes subscripts trigger this issue is that you can implement the getter (i.e. objectAtIndexedSubscript:) in a superclass, then only implement the setter method (i.e. setObject:atIndexedSubscript:) in a subclass while using the inherited getter, like with NSArray and NSMutableArray. I'm not familiar with how computed properties in Objective-C work, nor how they get imported into Swift; if this is possible i'd need to know the right methods to write in the test. 😅

@swift-ci Please test

@slavapestov Are computed properties allowed to have a separate setter/getter between a superclass and subclass?

I definitely remember this coming up in the past. Maybe @beccadax can point you to a specific example?

Are computed properties allowed to have a separate setter/getter between a superclass and subclass?

A subclass can override a readonly computed property to make it readwrite, and Swift will import that correctly.
The key difference between properties and subscripts in ClangImporter is that ObjC has explicitly declared properties, so there is a specific clang decl that, when passed to importDecl(), will map directly to the Swift property decl. There is no explicit ObjC decl for subscripts, so importDecl() has to basically create the Swift subscript decl as a side effect of importing the accessor methods.

@beccadax So you think that properties won't be affected by this issue? Is there any other situation where the Clang importer effectively generates a decl or are subscripts the only situation?
@bnbarham Can you take another look at this PR?


Add TLS to MySqlSessionStorage

Overview
Currently there is no way to use TLS with MySQLSessionStorage.
Type

[ ] New feature
[x] Changes to existing features

Motivation

What inspired this feature request? What problems were you facing?

I'm trying to connect my Azure MySQL database to MySQLSessionStorage and I need it to work with tls, but in the current iteration the dburl is coded in as a string and there is no way for me to add the option.
What I would like is to be able to add the tls option in like:
SESSION_STORAGE: new Shopify.Session.MySQLSessionStorage({   url: dbUrl,   ssl: SSL_OPTIONS })
Area

[ ] Add any relevant Area: <area> labels to this issue

Checklist

[x] I have described this feature request in a way that is actionable (if possible)

Overview
Currently there is no way to use TLS with MySQLSessionStorage.
Type

[ ]  New feature
[x]  Changes to existing features

Motivation

What inspired this feature request? What problems were you facing?

I'm trying to connect my Azure MySQL database to MySQLSessionStorage and I need it to work with tls, but in the current iteration the dburl is coded in as a string and there is no way for me to add the option.
What I would like is to be able to add the tls option in like:
SESSION_STORAGE: new Shopify.Session.MySQLSessionStorage({ url: dbUrl, ssl: SSL_OPTIONS })
Area

[ ]  Add any relevant Area: <area> labels to this issue

Checklist

[x]  I have described this feature request in a way that is actionable (if possible)


Incognito Mode: Cookies are blocked

The problem
Docs say it is possibel to follow the steps in Incognito Mode, but that leads to the following warning on the screen:
Cookies are blocked
To continue, change your browser settings to allow third-party cookies and try again.

What version of this integration (ha-nest-protect) has the issue?
latest
What version of Home Assistant Core has the issue?
latest
Device / Model
No response
Diagnostics information
No response
Home Assistant log
Logs
```
Copy/paste any log here, between the starting and ending backticks (`)
```

Additional information
No response

sorry, was meant for @nicoinch 's fork


CompleteInput support use the same dir separator as the input param on Windows #2219

Summary of the new feature/enhancement
Some time ago I created this issue in PSReadline about using tab-completion using a forward slash in PowerShell running on Windows (to better simulate Unix ). For instance, today the ls C:/Users followed by the tab is automatically converted to C:\Users\. I would expect C:/Users/.
PSReadline team said it couldn't be done because CompleteInput is coded to use the system separator.
Therefore, my feature request is that the CompleteInput return respects the same dir separator used in the input param.
Proposed technical implementation details (optional)
No response

@alanlivio, I found this issue searching for a resolution to this exact scenario. I can think of reasons why this shouldn't be the default for all installations, but it'd make things easier for me if this was an option.

Hi @MartinGC94, any update on the PR? Thanks.

The path settings have been implemented and if you want to try them out you can create a build from my branch. I'm actually just waiting for another PR of mine to get merged: https://github.com/PowerShell/PowerShell/pull/18955 so I can add those options as well.
It should be noted that PSReadLine interferes a little with my changes because of the way it auto inserts a separator for directories, see: https://github.com/PowerShell/PSReadLine/issues/3699 so that probably needs to be fixed as well before my changes makes its way into a production build.

Hi, again @MartinGC94. It seems that your PR has been closed.

Yeah I know. I'm still waiting on: https://github.com/PowerShell/PowerShell/pull/18955 to get merged. I could of course just leave that feature out for now and resubmit it as is but I suspect it won't make the PS team merge these changes any faster. It seems like they are not really merging any new community changes at the moment where they are preparing for an official 7.4.0 release.
My best estimate is that they'll "pick up the slack" in January and start merging old PRs and then hopefully it will make its way into a 7.5.0 preview 1 release.

Recently, PowerShell, or PSReadLine, stopped expanding ~ to your user profile, which makes the slash issue much worse. Now, if I type a "less" command line (using git's less) like this:
less ~/somefi

and press tab, it gets completed to
less ~\somefile.txt

which doesn't work in less. Before, it completed to the full path (with ) which did work. Since I use less constantly, this is a pretty big annoyance.

Dear @cyanite, I am also very used to typing /.
The most important thing to me is that the windows will accept the / if you send it. But the PowerShell CompleteInput force used to use the .

Dear @cyanite, I am also very used to typing /. The most important thing to me is that the windows will accept the / if you send it. But the PowerShell CompleteInput force used to use the .

Right. For me, the most important would be if I could tell it to always complete with /. But any variant will do :)

I hope #18955 will make it in soon, the autocomplete forcing the use of backslash breaks my use of WSL commands from Windows.
I'll try to run commands like so:
lx kak ./client.go # lx is an alias for wsl -e
but the auto-complete changes it to:
lx kak .\client.go
which breaks how the command is interpreted by WSL and instead of opening the existing file, it opens a new empty text file in the WSL home directory.
Anytime I use a WSL command in Windows via PowerShell, I have to type the names/paths by hand to circumvent this problem, which is a nuisance to say the least.

Thanks @dyskos. I am really looking forward to that as well.


python 3.11 support

Codecov Report

Merging #70 (9057b7f) into main (ec4a927) will decrease coverage by 0.19%.
The diff coverage is 79.59%.

@@            Coverage Diff             @@
##             main      #70      +/-   ##
==========================================
- Coverage   94.29%   94.10%   -0.19%
==========================================
  Files          24       24
  Lines        1507     1511       +4
  Branches      311      373      +62
==========================================
+ Hits         1421     1422       +1
- Misses         53       54       +1
- Partials       33       35       +2

Flag
Coverage Δ

unittests
94.10% <79.59%> (-0.19%)
:arrow_down:

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

Impacted Files
Coverage Δ

heksher/main.py
75.00% <72.22%> (-3.27%)
:arrow_down:

heksher/app.py
81.39% <100.00%> (-0.75%)
:arrow_down:

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


Jkl 227 implement filtree sim ops

@dahn510 I finished all the other msgs take a look when u have a chance
Also, we pushing this to master or bouncy dog?

Sim runs fine locally for me, not sure why it failed here 😵

investigating the issue


Bookmarks activity / display bookmarks as treeview

Will need to decide where bookmarks are stored? Server would require login. Add login suggestion if not logged in? Support feature only when logged in? If bookmarks are stored locally (temporarily) how are they merged with cloud bookmarks once you login?


Ghost objects to disable collision response

Adds ability to mark a rigid body as a ghost (just as we check kinematic) in the editor. This makes the rigid body disregard all collisions; it doesn't interact with other rigid bodies at all. This functionality is useful if we want to get trigger events for a rigid body but are not interested in actually colliding objects.
Disclaimer: The desired approach would be to put this functionality in the shape component instead, and have trigger components listen for shapes. In order to progress towards a delivery, this solution is fine for our purposes.

SonarQube analysis reported 5 issues

 5 minor

Watch the comments in this conversation to review them.
3 extra issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AncientFungus:LGEPR: Cppcheck cannot find all the include files (use --check-config for details)
 TriggerObserver.cpp#L19: The function 'GetPhase' is never used.
 TriggerObserver.cpp#L119: The function 'addSingleResult' is never used.

SonarQube analysis reported 5 issues

 5 minor

Watch the comments in this conversation to review them.
3 extra issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AncientFungus:LGEPR: Cppcheck cannot find all the include files (use --check-config for details)
 TriggerObserver.cpp#L19: The function 'GetPhase' is never used.
 TriggerObserver.cpp#L119: The function 'addSingleResult' is never used.


Morning Peer to peer code Review

@BucurEva87 , good work so far in this project. How about you add an anti-scientific notation rule to validation checks?

Hi @BucurEva87
Great job so far.


Require minimum duration for kinetic animation

Fixes #6861. Thanks @ogo-adp for the research and the suggested fix.

Looks sensible.


Added OpenCollective integration

We have been approved for OpenCollective and everything looks good. In this PR:

Moved from Flattr/bit.ly integration to OpenCollective
Fixes #66

Notes:
This does bloat up the README but it will automatically fill in backer images as we get new ones (defaults to 1px images otherwise)
/cc @rpdelaney

Seems legit. :+1:

Woot =D This has been merged/released in 0.26.7

We've got our first backer 🎉 https://opencollective.com/sexy-bash-prompt


⚠ RPC server - Conflux has degraded performance

In 797e990, RPC server - Conflux ($CONFLUX_RPC_URL) experienced degraded performance:

HTTP code: 200
Response time: 1528 ms

Resolved: RPC server - Conflux performance has improved in fabedfe after 9 minutes.


Accurate date on file history

The human language "2 months ago" is nice but not useful when trying to debug an order of events/commits

Brackets 1.1.0-15558 (mac), Git 1.9.3 (Apple Git-50) & Brackets-Git 0.14.9

Hello @carstenroselundberg,
you can change the date formatting via settings. Open the Brackets-Git panel and click on the settings icon.

How come I didn't think of that ... (shameful)
Loving this extension more and more - has completely replaced my git client

:+1:


Why do the resources not implement Properties?

Easy as that! We're deriving Default so that when the plugin is initialized it starts with a None value, but we could provide an initial value in case we wanted by implementing FromResources. You can see how it works in this Bevy example.

The liked example in the last sentence states

Registered components must implement the Properties and FromResources traits.`

Aren't our resources missing Properties? Do we need it? If not, why not?

Ah, I think I confused Resources and Components here. In our case, we have a resource and you talk about implementing FromResources, but in the linked bevy example FromResources is discussed for Components. Could you please shed some light on that?

Looks like I linked the wrong example! That one demonstrates implementing FromResources for a Component, while in the tutorial I was talking about implementing it for a Resource.  I haven't really worked much with implementing FromResources and Properties for a Component, so I can't really tell how they compare.
It has been fixed, it now links to a different (and now correct) example. Thanks for catching that!


Creating VS Solution Yields: "TypeError: get_host_target() missing 1 required positional argument: 'msvc_version':"

Godot version
main
System information
Win 10, AMD Ryzen 5 PRO 2400GE w/ Radeon Vega Graphics
Issue description
I tried creating a Visual Studio Solution by following the compilation instructions:
https://docs.godotengine.org/en/stable/development/compiling/compiling_for_windows.html
using the following code:
scons p=windows vsproj=yes
but then this happens:

I'm able to compile the project right using:
scons -j8 platform=windows
the bin file ran as expected. But I got stuck in prepping it for VS.
I tried reinstalling VS but that didn't work. Am I missing something?
What I've got installed so far:
+Visual Studio Community 2022 with both Desktop Development C++ and Game development with C++
+Python 3.10.15
+Git.2.37.1
+SCons: v4.4.0.
Steps to reproduce
basically I followed this tutorial:
https://youtu.be/Ivs5lLcY5ug
Minimal reproduction project
No response

I figured out what was wrong
I looked at the methods.py file:

the get_host_target function was missing an argument. I looked at the Scons Repo and found the function:

...I tweaked the methods.py file a bit:

I got it to work:

Might be time to update this part of the file:

But I'm not so sure if changing this would be safe for anyone not using the latest version of SCons.
Anyways, hope this helps :)

Thanks for the details. Sounds like SCons 4.4.0 added this new mandatory argument so we'll have to use either firm depending on which SCons version is used.


alphaess-developer OpenAPI repository gone?

Is it just me or has the repo for the open API gone from alphaess-developer?
I went to look up the recent thread on using the API to bind an inverter rather than using the website and I can't find any of that discussion at all any more.

Im not sure if this relates to elevanfan job being "transferred" or not. He sent me a slack DM a few weeks ago saying:
"My job has been transferred and I will no longer be responsible for this API. If you have any questions in the future, please contact our after-sales service."

I have a feeling he was one of the main people who worked on the github and maintained it, and obv with him gone it was for some reason wiped...

Oh dear.  Well we will see if the API keeps running.  I guess if it develops problems in the long run we could go back to scraping from the unofficial client again.


Cannot read properties of undefined (reading 'odometer')

I am using react and gatsby. I have followed the gatsby instructions and all seems to be working, but when i include the   line within the rendered html I get the following error. Any ideas? @inferusvv  inferusvv

Hi @inferusvv, I just wanted to see if there was any feedback on this issue? Is this something you have come across before? Any help much appreciated. Thank you!

Hi @inferusvv, I just wanted to see if there was any feedback on this issue? Is this something you have come across before? Any help much appreciated. Thank you!

I will try to check soon. Could you create a demo on code sandbox for example?

Thank you for getting back to me. Its a very simple example. Shall I record a short video of recreating the issue, im sure its something really obvious I am missing. Or maybe a 10 min pair session, whichever you prefer

@inferusvv Incase it is something simple, I have a gatsby & react app set up. I used yarn to install react-odometerjs, when importing the node_module directly I get the "Failed to Server Render (SSR). Error message: ReferenceError: document is not defined" error which is why the gatsby section in your docs exists I guess.
I have then added loadable, and using the Odometer.js file you mention in the gatsby docs.
It loads fine but then when the script runs client-side to pick up the elements and animate them I get the error above.
I have the example:
      <div className="odometer-container">
        <Odometer value={1234} format="(ddd),dd" />
      </div>

given my example is such a simple use of your script I thought maybe you had come across options.el being undefined when you wrote the gatsby docs?

I also found the below 2 errors at the top of the console log which I believe to be the route cause. I am not sure what the fix would be and why this would be any different to the other gatsby implementations of react- odometerjs

@UntappedSolutions  I think, that I need a demo from your side. Maybe create a public repository.
Here is my code:
import * as React from "react"
import type { HeadFC, PageProps } from "gatsby"
import loadable from "@loadable/component"
import "../styles/odometer.css"

const Odometer = loadable(() => import('react-odometerjs'));

const pageStyles = {
  color: "#232129",
  fontSize: "24px",
  padding: 96,
  fontFamily: "-apple-system, Roboto, sans-serif, serif",
}

const IndexPage: React.FC<PageProps> = () => {
  const [count, setCount] = React.useState(1000);
  React.useEffect(() => {
    const timeoutId = setTimeout(() => {
      setCount(4321);
    }, 1000);
    return () => clearTimeout(timeoutId);
  }, []);
  return (
    <main style={pageStyles}>
      <Odometer duration={1000} value={count} />
    </main>
  )
}

export default IndexPage

export const Head: HeadFC = () => <title>Home Page</title>

And here is a result:
https://github.com/inferusvv/react-odometerjs/assets/3227120/d7185f29-7764-4cc2-89e1-29f4ac50d17f

Hi @inferusvv, I notice that you are using const Odometer = loadable(() => import('react-odometerjs')); where as I followed the Gatsby docs which suggested creating an extra file with the following inside:
`import Odometer from 'react-odometerjs'
export default Odometer;`
This has solved the immediate error.
I have now got it working. I needed to download the css and import it directly, not a big issue.
I'd suggest the key update to the docs would be to use loadable to load the node module and not the extra file. Thank you for your messages, they really helped get to the bottom of it quickly.

No matter. I checked both. With extra file and without it should work.
Honestly, section about Gatsby was added by contributor, not by me, but guide is working as I see


fix: Send JSON response instead of plain text

Closes #59
Thanks for submitting a PR! Please check the boxes below:

[x] I have run pre-commit to check linting
[x] I have filled in the "Changes" section below?
[x] I have filled in the "How did you test this code" section below?
[x] I have used a Conventional Commit title for this Pull Request

Changes
Send JSONReponse instead of plain text.
How did you test this code?

make test

Codecov Report
Patch coverage: 50.00% and no project coverage change.

Comparison is base (55a9ef7) 95.48% compared to head (25bd3a3) 95.48%.
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2739   +/-   ##
=======================================
  Coverage   95.48%   95.48%
=======================================
  Files         994      994
  Lines       27939    27939
=======================================
  Hits        26677    26677
  Misses       1262     1262

Files Changed
Coverage Δ

api/app/settings/common.py
87.95% <ø> (ø)

api/users/views.py
91.21% <50.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@tushar5526 can you fix the linting errors here?

@tushar5526 can you fix the linting errors here?

Done!


Operations allow me to specify an interface as a return type. Well. Kind of. Should they?

Only the openapi emitter complains about operations returning interfaces. But otherwise it seem to compile just fine. Is that intentional?
Playground Link

Think this is the same base problem as #561

fix in the compiler

pri: 1
est: 5

@markcowl, questions on this.

should the label be Required for DPG 2.0?
Tim says it's the same problem as #561 but that is on the backlog. Just confirming this is something we want to do this sprint.
Are we saying this is a bug (it's not labeled as such)? When you say "fix in compiler" do you mean that this should emit an error (warning?) when this happens?

This is not required for DPG 2.0.

This is not required for DPG 2.0.

Then is this really Pri 1 for this sprint?

Yeah, the only reason this was marked as 'required' was in case it heralded a larger problem - if Timothee identified the problem, then this should no longer be required.  I downgraded it in the sprint.