Exclude from sitemap.xml attribute persists in the SEO section even after removing it from the Attributes section.

Steps to recreate:
1.) Fill in page type defaults for a page to include "exclude from sitemap.xml" in attributes, and have it be checked.
2.) Create the page
3.) Edit the page and go to the page properties > Attributes
4.) Click the minus button on the attribute for Exclude from sitemap.xml
5.) Save
6.) Reload the page
7.) Note that under attributes it appears to not be applied, but under the SEO section it is still present and checked. It seems that removing it from attributes isn't making it also be set to false in the SEO section.

Can anyone else replicate this? I cannot.

I cannot reproduce this issue.
When reaching step 7, the "Exclude From sitemap.xml" is not checked.


Add more options

this pr adds 4 new options to the cli
--no-relations: Do not include relations
--optional-relations: Make relations optional types
--no-hidden: Do not include hidden model attributes
--timestamp-strings: Output timestamps as strings

this fixes #31 without breaking changes and addresses #32 as well

Thanks for the pr! reviewing shortly

@tcampbPPU what do you think about the TS date type expecting an object vs a Laravel endpoint response always coming back as a json - do we keep it an option to go text? or make the switch making it an option to make it a date?

added another option --optional-nullables: Output nullable attributes as optional fields

yea I think union type works fine from @nurdism suggestion

@nurdism if you can please:

default dates to string since Date is wrong either way
make the option --timestamp-dates for people that use this and convert the response themselves
add a small section to the readme showing these options - it can be between Requiresments and Custom Interfaces - it can easily be a dump of ./artisan model:typer --help

@acidjazz updated the requested changes


Crash in ~ChannelImpl() - should remove() also detach()?

Hi! If you run the following program and then "force close" the connection from the RabbitMQ management GUI, it crashes in ChannelImpl::~ChannelImpl().
#include <optional>
#include <amqpcpp.h>
#include <amqpcpp/libuv.h>

class MyHandler : public AMQP::LibUvHandler {
private:
    std::optional <AMQP::TcpChannel> _channel;

public:
    MyHandler(uv_loop_t *loop) :
            AMQP::LibUvHandler(loop),
            _channel(std::nullopt) {
    }

    void onConnected(AMQP::TcpConnection *connection) override {
        _channel.emplace(connection);
    }

    void onError(AMQP::TcpConnection *connection, const char *message) override {
        // Uncommenting this fixes the crash since the channel will be
        // destructed before the connection.
        //_channel = std::nullopt;
    }
};

int main() {
    auto loop = uv_default_loop();
    MyHandler handler(loop);
    AMQP::TcpConnection conn(&handler, AMQP::Address("amqp://guest:guest@localhost"));
    AMQP::TcpChannel chan(&conn);
    uv_run(loop, UV_RUN_DEFAULT);
    return 0;
}

Because MyHandler is holding on to a TcpChannel, the channel is destructed after the TcpConnection. That would be ok if the connection had called detach() on the channel when it was destructed, but that never happened because of ChannelImpl::reportError():
    // the connection no longer has to know that this channel exists,
    // because the channel ID is no longer in use
    if (_connection) _connection->remove(this);

Should that remove be followed up with a detach?

Hmm, this is an interesting point. For now, I would say that this is not a bug, but a consequence of the architecture of this library.
The constructor of the Channel class requires you to pass in a pointer-to-a-Connection. This implies that this pointer must stay valid for the entire lifetime of the Channel object. If you do destruct the Connection before you destruct the Channel, you will indeed run into errors. Even when it seems logical to destruct the connection.
This is a design principle of the library: pointers passed in from user space are considered to be safe. It is also true for other pointers that you can pass to the library, like the pointer to the ConnectionHandler that you pass to a Connection instance. You cannot destruct this object either while the Connection object is still alive.
Although in your specific code sample if may seem logical to destruct the connection first and then the channel, it is not what this library expects from user-space programs. I would recommend to restructure your code a little so that there can never be a scenario in which connections are destructed before channels, or more general: in which "parent" objects are destructed before objects that depend on / refer to them.

Shouldn't the API uses smart pointers to prevent these kind of issues ?

Not really, because that would force all user space programs in the world to always use smart pointers for connection and handler objects, even when they do not even destruct connections while channels are still in use, only because one person uses this peculiar architecture.

not specifically, the current constructor could be kept and build an internal-only smart pointer with a custom no-op destructor. That way, both case could be handled, giving the flexibility to use smart pointers.

Can you show me a example where the user-space program does not use a smart pointer, and explicitly destructs the connection object, while the amqp-library channel object still keeps it valid because it uses a smart pointer internally?

This is what I have in mind:
struct F
{
        F(std::shared_ptr<int> p) :
                p(p)
        {}

        F(int *p) :
                F(std::shared_ptr<int>(p, [](int* p){ std::cout << "no-op deleter for " << p << std::endl; }))
        {}

        std::shared_ptr<int> p;
};

int main()
{
        int unmanaged(0);
        auto managed = std::make_shared<int>(0);

        std::cout << "unmanaged ptr at " << &unmanaged << std::endl;

        F f0(&unmanaged);
        F f1(managed);

        return 0;
}

Internally, the library uses smart pointers.
F(int *p) allows legacy software continues to use a raw, unmanaged interface. They keep being responsible for object lifetime (potentially leading to the issue raised by @jgallagher).
F(std::shared_ptr<int> p) allows more complex code to be safeguarded automatically about lifetime issues.

I can see what you mean. It is interesting and I may even like the approach. However, at this point in time, my opinion is that there is not much wrong with the requirement that in C++ objects sometimes depend on or refer to other objects, and that destruction therefore sometimes has to follow a certain order in order not to break things. But I will let this sink in for a while any maybe I am going to use this.


Request: new gem release

It has been 3 years since the last release, and fixes such as this are not in the latest gem version. Using github in my gemfile for now, but this is less than ideal.

Good point. I will push a minor version soon.

Still planning on doing this soon, once I get enough time to put it together. Sorry for the delay.

@nertzy just a little nudge

v2.3.7 is released!


Enhancement Req: Expose more User-friendly logging

I have been playing around with this extension on Windows and Linux, and both environments have a few different hiccups in the bootstrapping process. Some of them are varying degrees of host misconfiguration or race conditions, but it's difficult as an end user to determine if the process is stuck, still running, timed out, or otherwise unable to finish.
It would be a better UI/UX if this extension had the ability to export it's log files in plaintext via the preferences menu rather than relying on what appears to be a somewhat broken console UI inside podman desktop. Having the log file is a lot easier to grok than relying on the console UI and trying to catch an error message at the second it happens. This also helps when podman-desktop's UI gets unresponsive in the moment (IE when starting fairly resource intensive processes).
To avoid issues like podman-desktop being run inside flatpak or windows vs linux paths, it would be best to serve the file through the UI so the user doesn't have to know things like what path the files actually go to inside a running flatpak.

Hi @DaveWK UI agree.
at the moment we have some challenges here, as some of the infrastructure does not exist yet in PD, and some is our own from before. We will be looking at this for sure.
We are looking into a new events-based model, but if linking to our current log gathering can help; crc.log and crcd.log.
The issue with flatpak has to be solved generic and is not part of the extension. We adopted a 'broken' model until this can be fixed by PD.
Is there anything that we can help with already to address your issues? In the worst case, you can rely on the CLI to perform these steps; like crc setup, crc start, etc.


The authentication failure could not throw the expected IllegalStateException

Added a test and fix when the authentication fails we can compose the exceptıon properly when no credentials exist. Without the fix the log was trying to print a null object which causes shared_ptr dereference assertion.

Windows test PASSed.

Linux test PASSed.


Changelog missing (Migration guide link broken)

New Issue Checklist
🚫 If this template is not filled out your issue will be closed with no comment. 🚫

[x] I have looked at the Documentation
[x] I have read the F.A.Q.
[x] I have filled out this issue template.

Issue Info
Not needed
Issue Description
The migration guide:
https://github.com/SnapKit/SnapKit/blob/master/Documentation/SnapKit 3.0 Migration Guide.md
Has a link to a CHANGELOG that is broken:
https://github.com/SnapKit/SnapKit/blob/master/CHANGELOG.md
I see that the file is actually in another branch so you can fix it by linking to this branch or copying the CHANGELOG to develop as well:
https://github.com/SnapKit/SnapKit/blob/feature/0.40.0/CHANGELOG.md

@lluisgerard thanks for this! I'm migrating the changelog strictly to GitHub releases hence it's gone missing in master. I'll update the migration guide.


Update eval_path_list method to recognize entries with semicolon.

Updated eval_path_list method to recognize path entries separated by semicolon. It allows different formats and moreover, the usage of environment variables within the :paths: key.
Below an example using an environment variable being used on the :source: folders. The user can define this variable with all the necessary folders separated by a semicolon (e.g):
"../folderA;../folderB/"
:paths:
:test:
- test/**
- -:test/support/**
:source:
- "#{ENV['INCLUDE_DIR']}"

Two issues:
(1) It appears that instead of adding the ability to list multiple paths specified with a semicolon, you've replaced the existing array method?
(2) A semicolon isn't cross-platform. At a minimum we should also support the colon in this case, yes?

@mvandervoord please take a look below:
(1) I didn't remove the ability of using array. It will be handled by the flatten, and by the end the point is that instead of having a multidimensional array as before, it will have only an 1-D array. But we're still able to specify arrays in the paths.
(2) I will update the semicolon to use File::PATH_SEPARATOR instead of hard coded semicolon.
Please, let me know your thoughts.

:) awesome. sounds perfect!
Also, I should have started my previous post with thanking your for your help! I appreciate it and didn't mean to jump right into critiques!

The solution proposed, when using File::PATH_SEPARATOR, it conflicts with the Array definitions using comma for Linux. With that, it seems to be not the best approach to be able to use environment variables in path configuration.


[xy-chart] add support for confidence bands

This PR

adds support for confidence bands in the <AreaSeries /> component and updates the readme + tests. To implement this it bumps to<EMAIL_ADDRESS>which added support for <Area /> components
bumps to<EMAIL_ADDRESS>which has adds less-strict tickValue props
preserves user-set onMouseMove and onMouseLeave props for all child series instead of having XYChart overriding them. this also makes it possible to disable mouse events for specific subsets of child series by setting these handlers to null

Coverage decreased (-0.06%) to 84.856% when pulling 98de4a4d262892cb7ea68b699982d00cf6b9f716 on chris--band-series into 74e88a21089cc84b24bf5765018e91720253f4c6 on master.

Coverage decreased (-0.06%) to 84.856% when pulling 98de4a4d262892cb7ea68b699982d00cf6b9f716 on chris--band-series into 74e88a21089cc84b24bf5765018e91720253f4c6 on master.

Coverage decreased (-0.2%) to 84.703% when pulling 3f3e2b1247737e4653f06513f6a42b1bb409bc8b on chris--band-series into 74e88a21089cc84b24bf5765018e91720253f4c6 on master.


Why I am not eligible for zkSync Era nft sumonxyx nft

I do all work in zk sync development my volume in zkSync Era is 1150 dollar above
My address is 0x88686F7A5Db76c4462efcc0b7CAE5c4F9A3c15F0
Please why I am not eligible for zkSy

nc Era nft matter labs

@Armankhan566 please avoid spamming in Matter Labs repositories.
Further attemps will result into a permanent ban in our organization.
Thanks for understanding!


Buttons have incorrect styles in 7.1.0

Step 2: Describe your environment

FirebaseUI version: 7.1.0

Step 3: Describe the problem:
Steps to reproduce:

Customize button appearance by overriding styles
Update from 6.2 to 7.1
Buttons are no longer styled appropriately

Here's how I'm doing the customization:
<style name="FirebaseUI.Button.AccountChooser.GoogleButton">
  <item name="drawableStartCompat">@drawable/ic_sign_in_google</item>
  <item name="android:background">@color/auth_button_bg_google</item>
  <item name="android:textColor">@color/actually_white</item>
</style>

I checked and the style hasn't changed names or anything:
https://github.com/firebase/FirebaseUI-Android/blob/aef1d2f94901295a6f7b66eeea75f1ec7a90cd37/auth/src/main/res/values/styles.xml#L231
Looks like this is happening because of a switch from com.firebase.ui.auth.util.ui.SupportVectorDrawablesButton to Button in 5342e3dc69221069745945ef613be7eeaf0573eb .  I can get it working again if I use drawableLeft but obviously that's not ideal for RTL users.  What was the motivation for that change?  It seems to have even broken the default UI.  If I remove all my style customizations, here's what the screen looks like:

Same here. It seems MDC changes <Button> to MaterialButton, and then styles are lost.

@edenman @DeweyReed thanks for reporting this.  That change was just a cleanup, if it's breaking we can revert it for version 7.1.1.

This has been fixed and released in version 7.1.1

Confirmed: fixed in 7.1.1


ledger nano s fix

nano s signing issues, had to modify go.mod to use zondax/ledger-go v0.14.3 instead of the v0.14.1 then rebuild, then was able to sign

Just in case, a PR already cover this https://github.com/Stride-Labs/stride/pull/1099

closing in favor of #1099


[FR] Functions

@julienXX Could you review this, please?

@julienXX dis-moi quand tu as fini :)

@tchoutri je crois que c'est bon pour mes remarques :)

@julienXX Okay je vais les appliquer :)

@julienXX I think you can confirm the merge to @pjhampton if you think everything's alright. :)

@tchoutri you might want to rebase in a single commit with a better message. Otherwise @pjhampton :shipit: :)

Awesome @tchoutri and  @julienXX!
Yeah, @tchoutri could please squash these? @doomspork prefers to keep each commit unique and meaningful for obvious reasons :)

Okay, just let me learn again about git rebase >_>

@julienXX Okay j'ai fini ma première revue. Ton avis ?

@doomspork can this be merged with the two en/ overlaps?

This branch isn't currently merge-able, it needs to be rebased.

This one needs to be resolved first, it may clean up #244

We have implemented a new feature that automatically generates the table of contents: #356.
This may help you avoid copypaste, typos and broken anchors. In order to use it you can replace the manually written TOC with {% include toc.html %}.

Hi @tchoutri, any updates here?

@doomspork yeah I've got a problem with my forked repository and ruby throws me a bunch of error now :<
I'm forking it again to have a clean repo.

No worries @tchoutri, let me know if you need a hand 😀

well… now would be the moment for someone to handle it since I can't help but messing everything up. :/. @julienXX if you have some time, I wouldn't say no.

@tchoutri I copied over your lesson, I'll open a PR and ping you 👍

Merged in #521


Enhance Documentation and Code Readability for ContactForm Component

The ContactForm component requires improved documentation and code clarity. This enhancement focuses on:
Adding meaningful comments to describe the structure and logic of the component.
Improving readability with better formatting and consistent indentation.
Ensuring future maintainability by clearly defining component behavior and state updates.
Acceptance Criteria:
Clear and meaningful comments explaining key parts of the component.
Consistent formatting and indentation.
No changes to the original functionality.

Hey @Sumanth077s👋! Thanks for raising this issue and demonstrating your interest in contributing to our project!🚀
We value your engagement and eagerly anticipate your contributions. Should you wish to get involved, please use the /assignme command to self-assign the issue.
This issue provides an excellent opportunity for us to enhance our project, and we're excited to have you join us! Please take a moment to review our CONTRIBUTING GUIDELINES before getting started.
If you have any questions or need assistance, feel free to reach out to us on LinkedIn.❤

Hello @Sumanth077s! We'd like to inform you that the issue has been resolved and is now closed. We appreciate your understanding and look forward to your continued involvement with our repository!🤗
If you encounter any further issues, please don't hesitate to reach out to us on LinkedIn. Thank you for your support!❣
Happy coding!✨ We hope to see you again soon!❣


Clarify how tree sizes are computed

We compute tree sizes in a few places in the code, including:

The size of the full tree used to train the GLM. The resulting tree size is used to generate the master table of per-site mutation rates by context.
The size of the clade subtree. This tree size is used to convert from rates to predicted counts.

Based on this function, it looks to me like we use slightly different methods for computing tree sizes in these two cases. In the first case, it looks like we compute the tree's size by summing the actual counts of synonymous mutations from the curated dataframe (train_df). In the second case, it looks like we compute the tree's size by summing the actual counts of synonymous mutations from a non-curated dataframe (count_clade_syn). Is that correct? If so, this seems a bit discordant to me.
@lucasesta: on slack, you said we discussed with Richard and concluded that the most consistent strategy would be to retain only the sites from the curated dataset when computing the evolutionary time for clade-specific subtrees. It should not make a significant difference since the approach we are using explicitly accounts for the potential variation in the number of sites.
I like the idea of retaining only the sites from the curated dataframe when computing the evolutionary time for clade-specific subtrees, so just wanted to double check that this is what the code is doing. And if so, it doesn't seem like we need to account for variation in the number of sites. The curated dataframe only includes the set of sites that are conserved in all founder sequences. So, if we only use those sites, the number and composition of those sites should be exactly the same, right?

@Haddox: we updated the code so to only consider the set of sites from the curated datasets when computing the evolutionary times of every clade-specific subtree. As you correctly pointed out, in this scenario the clade-specific genome composition is no longer relevant, as it is the same for all.
With this commit, we pushed the updated outputs for the novel convention.

Sounds good. Thanks @lucasesta! I'll go ahead and close this issue.


The runtime_manager management interface in autoware version 12.0 can not be updated

The runtime_manager management interface in autoware version 12.0 can not be updated.
Operating system and version:
Ubuntu 18.04
Autoware installation type:
-From source
Autoware version or commit hash
12.0
ROS distribution and version:
Melodic
ROS installation type:
From source
Package or library, if applicable:
velodyne_driver

How can I solve this problem?      help~

Thanks for using Autoware and for your question. However the issue tracker is for posting confirmed bugs and feature requests.
We ask that you please ask questions at the ROS Answers website following our support guidelines: https://gitlab.com/autowarefoundation/autoware.ai/autoware/wikis/Support-guidelines#ros-answers. Please pay particular attention to the information we ask you to provide.


[Hub Generated] Review request for Microsoft.Devices to add version stable/2023-06-30

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
ARM API Information (Control Plane)
Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

    },

are these create/update only?

Refers to: specification/iothub/resource-manager/Microsoft.Devices/stable/2023-06-30/iothub.json:3255 in 09336e3. [](commit_id = 09336e3a5ddf329ae15f5c593921f5cc06086eb8, deletion_comment = False)

    },

Having name, subscriptionId, and resourceGroup, and id seems redundant - are they all identifying the same resource? Or are there diffeent resources here?

Refers to: specification/iothub/resource-manager/Microsoft.Devices/stable/2023-06-30/iothub.json:3225 in 09336e3. [](commit_id = 09336e3a5ddf329ae15f5c593921f5cc06086eb8, deletion_comment = False)

      "description": "Id of the cosmos DB sql container endpoint",

Want to include an example?

Refers to: specification/iothub/resource-manager/Microsoft.Devices/stable/2023-06-30/iothub.json:3215 in 09336e3. [](commit_id = 09336e3a5ddf329ae15f5c593921f5cc06086eb8, deletion_comment = False)

    },

format: uri?

Refers to: specification/iothub/resource-manager/Microsoft.Devices/stable/2023-06-30/iothub.json:3229 in 09336e3. [](commit_id = 09336e3a5ddf329ae15f5c593921f5cc06086eb8, deletion_comment = False)

      "description": "The template for generating a synthetic partition key value for use with this cosmos DB sql container. The template must include at least one of the following placeholders: {iothub}, {deviceid}, {DD}, {MM}, and {YYYY}. Any one placeholder may be specified at most once, but order and non-placeholder components are arbitrary. This parameter is only required if PartitionKeyName is specified.",

Why is it required if PartitionKeyName is specified?

Refers to: specification/iothub/resource-manager/Microsoft.Devices/stable/2023-06-30/iothub.json:3269 in 09336e3. [](commit_id = 09336e3a5ddf329ae15f5c593921f5cc06086eb8, deletion_comment = False)

      "description": "The user assigned identity."

objectid of?

Refers to: specification/iothub/resource-manager/Microsoft.Devices/stable/2023-06-30/iothub.json:4485 in 09336e3. [](commit_id = 09336e3a5ddf329ae15f5c593921f5cc06086eb8, deletion_comment = False)

    "cosmosDBSqlContainers": {

Are you going to add an example for this? It'd be nice if your examples cover all the different kind of routing endpoints with a common set of parameters.

Refers to: specification/iothub/resource-manager/Microsoft.Devices/stable/2023-06-30/iothub.json:2952 in 09336e3. [](commit_id = 09336e3a5ddf329ae15f5c593921f5cc06086eb8, deletion_comment = False)

    },

not sure I understand question

In reply to:<PHONE_NUMBER>

Refers to: specification/iothub/resource-manager/Microsoft.Devices/stable/2023-06-30/iothub.json:3255 in 09336e3. [](commit_id = 09336e3a5ddf329ae15f5c593921f5cc06086eb8, deletion_comment = False)

    },

what is annotating as a resource id? do you have any documentation link?

In reply to:<PHONE_NUMBER>

Refers to: specification/iothub/resource-manager/Microsoft.Devices/stable/2023-06-30/iothub.json:3217 in 09336e3. [](commit_id = 09336e3a5ddf329ae15f5c593921f5cc06086eb8, deletion_comment = False)

    },

name - name of endpoint set by customer
id  - internal id
resourceGroup - it is azure resource group where cosmosdb account reside

In reply to:<PHONE_NUMBER>

Refers to: specification/iothub/resource-manager/Microsoft.Devices/stable/2023-06-30/iothub.json:3225 in 09336e3. [](commit_id = 09336e3a5ddf329ae15f5c593921f5cc06086eb8, deletion_comment = False)

    },

the example is: https://ailn-systemstore-test2.documents.azure.com:443/
I dont see we use format in this doc at all

In reply to:<PHONE_NUMBER>

Refers to: specification/iothub/resource-manager/Microsoft.Devices/stable/2023-06-30/iothub.json:3229 in 09336e3. [](commit_id = 09336e3a5ddf329ae15f5c593921f5cc06086eb8, deletion_comment = False)

      "description": "The template for generating a synthetic partition key value for use with this cosmos DB sql container. The template must include at least one of the following placeholders: {iothub}, {deviceid}, {DD}, {MM}, and {YYYY}. Any one placeholder may be specified at most once, but order and non-placeholder components are arbitrary. This parameter is only required if PartitionKeyName is specified.",

partitionkeyname is name stamped to document
partitinkeytemplate is value of stamped document

In reply to:<PHONE_NUMBER>

Refers to: specification/iothub/resource-manager/Microsoft.Devices/stable/2023-06-30/iothub.json:3269 in 09336e3. [](commit_id = 09336e3a5ddf329ae15f5c593921f5cc06086eb8, deletion_comment = False)

    "cosmosDBSqlContainers": {

I will add example for cosmosdb endpoint, as far as this PR is for cosmosdb endpoint

In reply to:<PHONE_NUMBER>

Refers to: specification/iothub/resource-manager/Microsoft.Devices/stable/2023-06-30/iothub.json:2952 in 09336e3. [](commit_id = 09336e3a5ddf329ae15f5c593921f5cc06086eb8, deletion_comment = False)

    "cosmosDBSqlContainers": {

added

In reply to:<PHONE_NUMBER>

Refers to: specification/iothub/resource-manager/Microsoft.Devices/stable/2023-06-30/iothub.json:2952 in 09336e3. [](commit_id = 09336e3a5ddf329ae15f5c593921f5cc06086eb8, deletion_comment = False)

    },

So now, looks like you're using x-ms-secret, which implies x-ms-mutability is create/update only. I think its probably good to make that explicit though.
Do I understand right that you allow the customer to pass the keys they like, to enable iothub to connect to their cosmosDB instance?
This sounds like a pretty painful way of doing it, for anyone who wants to or has to (compliance) implementi key autorotations. Firstly, they can't even tell/audit what key values are in use.

In reply to:<PHONE_NUMBER>

Refers to: specification/iothub/resource-manager/Microsoft.Devices/stable/2023-06-30/iothub.json:3255 in 09336e3. [](commit_id = 09336e3a5ddf329ae15f5c593921f5cc06086eb8, deletion_comment = False)

    },

Is either databaseName or name, specifically the name of the ARM cosmosDB resource? Trying to understand intended usage.

Refers to: specification/iothub/resource-manager/Microsoft.Devices/stable/2023-06-30/iothub.json:3261 in 13514e2. [](commit_id = 13514e28bd3f4c8c3236a093234a7af885448028, deletion_comment = False)

    },

name of database created within cosmosdb resource

In reply to:<PHONE_NUMBER>

Refers to: specification/iothub/resource-manager/Microsoft.Devices/stable/2023-06-30/iothub.json:3261 in 13514e2. [](commit_id = 13514e28bd3f4c8c3236a093234a7af885448028, deletion_comment = False)

Added DoNotMerge at requet of PR author

/azp run

Merge conflicts need resolving now..

@rkmanda I wonder why powershell fails. Seems like its something to do wtih the tags in the readme maybe? Its a bit beyond me.

WARNING: /mnt/vss/_work/1/s/azure-rest-api-specs/specification
04:26:26.784 cmdout 	[psci.ps1] WARNING: There should be only one tag  in /mnt/vss/_work/1/s/azure-rest-api-specs/specification/marketplacecatalog/resource-manager/readme.md
04:26:26.792 cmdout 	[psci.ps1] WARNING: There should be only one tag  in /mnt/vss/_work/1/s/azure-rest-api-specs/specification/marketplacecatalog/resource-manager/Solutions/readme.md
04:26:28.003 cmdout 	[psci.ps1] WARNING: There should be only one tag  in /mnt/vss/_work/1/s/azure-rest-api-specs/specification/machinelearning/resource-manager/readme.md


await Moralis.LogInAsync(username, password) Does Not Set AuthenticationKitState to Connected

When calling  await Moralis.LogInAsync(username, password), the AuthenticationKitState does not change to Connected. However, once the user has been logged in and the application is restarted, Moralis reads the LoginAsync user from cache and does change the state to Connected. This leads to other bugs because the user is connected but the AuthenticationKitState is still in Initialized state.
Workaround is to set the state manually after successful login but the State is private, so need to change the packaging code.

Hi @vrxgaminginc, currently the AuthenticationKitState does not handle external state changes but this will be added in the near future. For now you could do a await  AuthenticationKit.InitializeAsync(); after the user is connected with Moralis.LogInAsync to work around this.


Explicit control over granting refresh tokens

It would be nice if clients of this library could easily control whether or not a particular request received a refresh token along with an access token.

We've merged the server into this, so the whole client/library distinction is void now.


PLUTO-733 Author Selenium-based tests for the Pluto V3 demos

@ngriffin7a, please review and merge all these commits and do not squash them. Thanks!
cc @vsingleton.

Merged, thank you.


feat: Implement the rule "react"

Describe your changes
Implement the ruleset by referring to the eslint-plugin-react:recommended and the react of eslint-config-airbnb.
However, the following rules are set by us:

jsx-handler-names

Enable it

jsx-sort-props

Enable it

jsx-uses-react

Disable it as it's no longer needed from React 17.

jsx-uses-vars

Disable it as it's no longer needed from React 17.

react-in-jsx-scope

Disable it as it may not be necessary depending on the builder and transpiler settings.

no-array-index-key

Disable it as there're many situations where you have to use index as key.

require-default-props

Disable it as it's not needed in TypeScript.

forbid-foreign-prop-types

Disable it  as it's not needed in TypeScript.

default-props-match-prop-types

Disable it as it's not needed in TypeScript.

no-redundant-should-component-update

Disable it

button-has-type

Disable it as it doesn't allow specification by variables containing Prop.

function-component-definition

Change namedComponents and unnamedComponents to arrow-function in function-component-definition rule for TypeScript

prefer-exact-props

Disable it as it's not needed in TypeScript.

Issue ticket number and link
Checklist before requesting a review

[ ] I have performed a self-review of my code
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics?
[ ] Will this be part of a product update? If yes, please write one phrase about this update.

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Object has already been returned to this pool or is invalid

Describe the bug
In 'edc-controlplane-postgresql' an exception occurs during startup
Reported in Jira
To Reproduce
On startup from controlplane
Screenshots/Error Messages
SEVERE 2023-04-05T08:06:08.375372 StateMachineManager [transfer-process] error caught java.lang.IllegalStateException: Object has already been returned to this pool or is invalid     at org.apache.commons.pool2.impl.BaseGenericObjectPool.markReturningState(BaseGenericObjectPool.java:1251)     at org.apache.commons.pool2.impl.GenericObjectPool.returnObject(GenericObjectPool.java:1029)     at org.eclipse.edc.sql.pool.commons.CommonsConnectionPool.returnConnection(CommonsConnectionPool.java:76)     at org.eclipse.edc.sql.datasource.PooledDataSourceConnection.close(PooledDataSourceConnection.java:96)     at org.eclipse.edc.connector.store.sql.transferprocess.store.SqlTransferProcessStore.lambda$nextForState$1(SqlTransferProcessStore.java:80)     at org.eclipse.edc.transaction.spi.NoopTransactionContext.execute(NoopTransactionContext.java:34)     at org.eclipse.edc.connector.store.sql.transferprocess.store.SqlTransferProcessStore.nextForState(SqlTransferProcessStore.java:69)     at org.eclipse.edc.connector.transfer.process.TransferProcessManagerImpl.lambda$processTransfersInState$21(TransferProcessManagerImpl.java:591)     at org.eclipse.edc.statemachine.StateProcessorImpl.process(StateProcessorImpl.java:46)     at java.base/java.util.stream.ReferencePipeline$5$1.accept(Unknown Source)     at java.base/java.util.ArrayList$ArrayListSpliterator.forEachRemaining(Unknown Source)     at java.base/java.util.stream.AbstractPipeline.copyInto(Unknown Source)     at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(Unknown Source)     at java.base/java.util.stream.ReduceOps$ReduceOp.evaluateSequential(Unknown Source)     at java.base/java.util.stream.AbstractPipeline.evaluate(Unknown Source)     at java.base/java.util.stream.LongPipeline.reduce(Unknown Source)     at java.base/java.util.stream.LongPipeline.sum(Unknown Source)     at org.eclipse.edc.statemachine.StateMachineManager.performLogic(StateMachineManager.java:118)     at org.eclipse.edc.statemachine.StateMachineManager.lambda$loop$2(StateMachineManager.java:106)     at io.micrometer.core.instrument.composite.CompositeTimer.record(CompositeTimer.java:79)     at io.micrometer.core.instrument.Timer.lambda$wrap$0(Timer.java:160)     at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Unknown Source)     at java.base/java.util.concurrent.FutureTask.run(Unknown Source)     at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(Unknown Source)     at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)     at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)     at java.base/java.lang.Thread.run(Unknown Source)

Can you please improve the issue with some more details? With the current content, we won't know what the problem was in a year. Also the full stack trace would be helpful.

Can you please improve the issue with some more details? With the current content, we won't know what the problem was in a year. Also the full stack trace would be helpful.

Hi @florianrusch-zf I have added the whole stack trace and linked the Jira ticket. I hope this helps people to understand it better after one year :smile:


Webhooks refuse IPv6 URLs as invalid

Gogs version (or commit ref): <IP_ADDRESS>7 / 0.11 RC
Git version: 2.1.4
Operating system: debian jessie
Database (use [x]):

[ ] PostgreSQL
[X] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gogs.io:

[X] Yes (provide example URL) - create a new repo and try to add a webhook to the URL below. For example, try it on https://try.gogs.io/sanfred/herpaderpa/settings/hooks/gogs/new
[ ] No
[ ] Not relevant

Log gist (usually found in log/gogs.log): no relevant entry

Description
I'm trying to add a webhook with a URL that looks like this:
http://[2002:d572:90f9:0:a00:26ff:feab:c501]:8080/gogs-webhook/?job=ci-test
I get the following error message, and cannot continue:

PayloadURL is not a valid URL.

As far as I know, this URL is valid. It works with firefox and curl with no warnings.

It's worth noting that Go's standard library accepts this format for URL. Maybe some regex or other filter doesn't think [] are valid characters in a URL.
RFC 3986 Section 3.2.2 describes a valid URL.

Patch has pushed to fix this issue, please test on develop branch or https://try.gogs.io.


Removed the indexed consumer code

2.10.X supports multi subject filter at a consumer so removing indexed consumer creating code, Also add support to bind to existing consumers with same or different account.

Closing as duplicate from the changes introduced in 1.2.0


Connect with John N. - Clean Growth Compliance

John has insight into Clean Growth Branch operations and industry needs.
Acceptance criteria:

[x] Send calendar invite
[x] Write questions

Excellent interview and very insightful. It is always a delight to speak with John. He mentioned a few really key points for further exploration - especially around permits and improved accessibility of the data.


Gem Load Error is: undefined method type_cast_from_database' for class ArJdbc::MSSQL::UUIDType'

Hey guys,
I'm struggling for many hours now using activerecord-jdbcmssql-adapter with Rails 5.1 (or 5.2). I've installed ant to be able to compile the gem during the bundling. The bundle process runs fine but when I want to start rails I'm just getting
Bundler::GemRequireError: There was an error while trying to load the gem 'activerecord-jdbcmssql-adapter'.
Gem Load Error is: undefined method `type_cast_from_database' for class `ArJdbc::MSSQL::UUIDType'

I've googled a lot but didn't find a solution or a hint what causes this error. Here is some more information:
Java version: Java(TM) SE Runtime Environment (build 1.8.0_171-b11)
Ruby: jruby-<IP_ADDRESS>
OS: Win10
Part of the Gemfile
ruby '2.3.3'

# Bundle edge Rails instead: gem 'rails', github: 'rails/rails'
gem 'rails', '~> 5.1.6'

#gem 'activerecord-jdbcsqlite3-adapter'
gem 'activerecord-jdbc-adapter', github: "jruby/activerecord-jdbc-adapter", :branch => "51-stable"
gem 'activerecord-jdbcmssql-adapter',github: "jruby/activerecord-jdbc-adapter",:branch => "51-stable"
gem 'activerecord-oracle_enhanced-adapter','~> 1.8.0'

The Backtrace
Q:\Ruby\jruby-<IP_ADDRESS>\bin\jruby.exe -S rails s
Bundler::GemRequireError: There was an error while trying to load the gem 'activerecord-jdbcmssql-adapter'.
Gem Load Error is: undefined method `type_cast_from_database' for class `ArJdbc::MSSQL::UUIDType'
Backtrace for gem load error is:
org/jruby/RubyModule.java:2841:in `alias_method'
o:/bundler/bundler/gems/activerecord-jdbc-adapter-4eebab4df17d/lib/arjdbc/mssql/types.rb:319:in `<class:UUIDType>'
o:/bundler/bundler/gems/activerecord-jdbc-adapter-4eebab4df17d/lib/arjdbc/mssql/types.rb:316:in `<module:MSSQL>'
o:/bundler/bundler/gems/activerecord-jdbc-adapter-4eebab4df17d/lib/arjdbc/mssql/types.rb:2:in `<module:ArJdbc>'
o:/bundler/bundler/gems/activerecord-jdbc-adapter-4eebab4df17d/lib/arjdbc/mssql/types.rb:1:in `<main>'
org/jruby/RubyKernel.java:956:in `require'
o:/bundler/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:292:in `block in require'
o:/bundler/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:258:in `load_dependency'
o:/bundler/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:292:in `require'
o:/bundler/bundler/gems/activerecord-jdbc-adapter-4eebab4df17d/lib/arjdbc/mssql/adapter.rb:1:in `<module:(root)>'
o:/bundler/bundler/gems/activerecord-jdbc-adapter-4eebab4df17d/lib/arjdbc/mssql/adapter.rb:37:in `<module:MSSQL>'
o:/bundler/bundler/gems/activerecord-jdbc-adapter-4eebab4df17d/lib/arjdbc/mssql/adapter.rb:30:in `<main>'
org/jruby/RubyKernel.java:956:in `require'
o:/bundler/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:292:in `block in require'
o:/bundler/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:258:in `load_dependency'
o:/bundler/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:292:in `require'
o:/bundler/bundler/gems/activerecord-jdbc-adapter-4eebab4df17d/lib/arjdbc/mssql/adapter.rb:29:in `<main>'
org/jruby/RubyKernel.java:956:in `require'
o:/bundler/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:292:in `block in require'
o:/bundler/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:258:in `load_dependency'
o:/bundler/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:292:in `require'
o:/bundler/bundler/gems/activerecord-jdbc-adapter-4eebab4df17d/lib/arjdbc/mssql.rb:1:in `<main>'
org/jruby/RubyKernel.java:956:in `require'
o:/bundler/bundler/gems/activerecord-jdbc-adapter-4eebab4df17d/lib/arjdbc/mssql.rb:2:in `block in (root)'
org/jruby/RubyArray.java:1735:in `each'
o:/bundler/bundler/gems/activerecord-jdbc-adapter-4eebab4df17d/activerecord-jdbcmssql-adapter/lib/activerecord-jdbcmssql-adapter.rb:1:in `block in (root)'
org/jruby/RubyArray.java:1735:in `each'
o:/bundler/bundler/gems/activerecord-jdbc-adapter-4eebab4df17d/activerecord-jdbcmssql-adapter/lib/activerecord-jdbcmssql-adapter.rb:2:in `(root)'
Q:/Ruby/jruby-<IP_ADDRESS>/lib/ruby/gems/shared/gems/bundler-1.16.2/lib/bundler/runtime.rb:1:in `(root)'
Q:/Ruby/jruby-<IP_ADDRESS>/lib/ruby/gems/shared/gems/bundler-1.16.2/lib/bundler/runtime.rb:81:in `<main>'
org/jruby/RubyKernel.java:956:in `require'
Q:/Ruby/jruby-<IP_ADDRESS>/lib/ruby/gems/shared/gems/bundler-1.16.2/lib/bundler/runtime.rb:76:in `block in require'
org/jruby/RubyKernel.java:1765:in `tap'
Q:/Ruby/jruby-<IP_ADDRESS>/lib/ruby/gems/shared/gems/bundler-1.16.2/lib/bundler/runtime.rb:65:in `require'
Q:/Ruby/jruby-<IP_ADDRESS>/lib/ruby/gems/shared/gems/bundler-1.16.2/lib/bundler.rb:114:in `require'
O:/git/xxx/config/application.rb:7:in `(root)'
o:/bundler/gems/railties-5.1.6/lib/rails/commands/server/server_command.rb:1:in `(root)'
o:/bundler/gems/railties-5.1.6/lib/rails/commands/server/server_command.rb:133:in `perform'
o:/bundler/gems/railties-5.1.6/lib/rails/commands/server/server_command.rb:130:in `perform'
o:/bundler/gems/thor-0.20.0/lib/thor/command.rb:27:in `<main>'
org/jruby/RubyKernel.java:956:in `require'
bin/rails:4:in `<main>'
Bundler Error Backtrace:

  block in require at Q:/Ruby/jruby-<IP_ADDRESS>/lib/ruby/gems/shared/gems/bundler-1.16.2/lib/bundler/runtime.rb:84
              each at org/jruby/RubyArray.java:1735
  block in require at Q:/Ruby/jruby-<IP_ADDRESS>/lib/ruby/gems/shared/gems/bundler-1.16.2/lib/bundler/runtime.rb:76
              each at org/jruby/RubyArray.java:1735
           require at Q:/Ruby/jruby-<IP_ADDRESS>/lib/ruby/gems/shared/gems/bundler-1.16.2/lib/bundler/runtime.rb:65
           require at Q:/Ruby/jruby-<IP_ADDRESS>/lib/ruby/gems/shared/gems/bundler-1.16.2/lib/bundler.rb:114
            <main> at O:/git/zak3_data/config/application.rb:7
           require at org/jruby/RubyKernel.java:956
   block in (root) at o:/bundler/gems/railties-5.1.6/lib/rails/commands/server/server_command.rb:1
               tap at org/jruby/RubyKernel.java:1765
           perform at o:/bundler/gems/railties-5.1.6/lib/rails/commands/server/server_command.rb:133
           perform at o:/bundler/gems/railties-5.1.6/lib/rails/commands/server/server_command.rb:130
               run at o:/bundler/gems/thor-0.20.0/lib/thor/command.rb:27
    invoke_command at o:/bundler/gems/thor-0.20.0/lib/thor/invocation.rb:126
          dispatch at o:/bundler/gems/thor-0.20.0/lib/thor.rb:387
           perform at o:/bundler/gems/railties-5.1.6/lib/rails/command/base.rb:63
            <main> at o:/bundler/gems/railties-5.1.6/lib/rails/command.rb:44
           require at org/jruby/R

MSSQL still isn't supported for Rails 5.x. You can get updates here: https://github.com/jruby/activerecord-jdbc-adapter/issues/784 and I'm hoping that support can be handled by https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/pull/599 going forward. Unfortunately, it hasn't had much movement either.

Thank you for referencing the issue!


bird-watcher: add "for loop" concept

Fixes: #1617

Updated config.json with concept and concept exercise
Added concept exercise directory with minimum valid spec
Added concept directory with about, introduction.

I marked this as a draft PR. Click on "ready for review" to when you are ready to change it back to a normal PR.

@sachinmk27 - any chance you will have time to work on this? This is also blocking ErikSchierboom's new exercise interest-is-interesting (unless we remove the prerequisite and add it later).

@jmrunkle Unfortunately I will not be able to make time for the next 4-5 days. Kindly reassign it since it's already delayed.

@sachinmk27 Could you give me an update again whether you want to finish this PR or whether we should re-assign the task?

Pls reassign the task. Sorry, I couldn't make time for it.

Pls reassign the task. Sorry, I couldn't make time for it.

No worries, thanks for letting us know :)

@andrerfcsantos I only reviewed about.md so fixes you do there need to be applied to the other files as well. ... you probably know the drill. :wink:

@junedev Gotcha. Ready to review again.

@andrerfcsantos The only thing left is this comment: https://github.com/exercism/go/pull/1775#discussion_r739879670
I approved so you can merge yourself after that last fix.


Make alternate app icon buttons accessible

References #676

Description:
Whereas previously the alternate app icon screen's buttons weren't accessible, this makes sure the buttons are seen as accessibility elements, adds the expected traits, and adds an accessibility label.
Steps to test this PR:

In VoiceOver, go to Settings > App Icon
Verify that the buttons are now accessible, show correct traits (button, button + selected if selected)

Device Testing:

[ ] iPhone SE (1st Gen)
[ ] iPhone 8
[ ] iPhone X
[x] iPad

OS Testing:

[ ] iOS 10
[ ] iOS 11
[ ] iOS 12
[x] iOS 13

Thanks again @BasThomas :)

Is this something you'd want an entry to the release notes for, or is that something you will take care of?

We'll take care of that, thanks @BasThomas  :)


Goan and text box in awards menu state

Hola the title says it all. goan is a game over character. and the box behind text is optinal. soon ill fix the text color but yeah!!

What is this

DO NOT MERGE THIS.
There is already a way to edit the gameover characters.
Not to mention, you are making ENTIRE SCRIPTS TO EXPLAIN THESE COMMANDS WHICH IS SOMETHING NONE OF THE OTHER ONES HAVE.
  There is already a way to edit the gameover characters. Not to mention, you are making ENTIRE SCRIPTS TO EXPLAIN THESE COMMANDS WHICH IS SOMETHING NONE OF THE OTHER ONES HAVE.


Update docker.io/renovate/renovate Docker tag to v24.34.0

This PR contains the following updates:

Package
Update
Change

docker.io/renovate/renovate
minor
24.32.5 -> 24.34.0

Release Notes

renovatebot/renovate
v24.34.0
Features

template field filtering (#8504) (7c40d26)

Miscellaneous Chores

migrate renovate.json (f3d6c18)

v24.33.0
Features

add redwood monorepo (#8498) (fe8b3e1)

Renovate configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
As this PR has been closed unmerged, Renovate will now ignore this update (24.34.0). You will still receive a PR once a newer version is released, so if you wish to permanently ignore this dependency, please add it to the ignoreDeps array of your renovate config.
If this PR was closed by mistake or you changed your mind, you can simply rename this PR and you will soon get a fresh replacement PR opened.


[FIX] Fix various bugs in sample JSON config files

Closes #280
Closes #292
Closes #295

LGTM 👨🍳


not plug-and-play; doesn't start

Thanks for making such a package at all; however, it has some non-standard preconditions.
ahk is not a standard file extension and not installed here, so it does not work.

I compile *.ahk to *.exe in build.bat (and download ahk2exe in prepare.sh),  wsl-terminal-ver.7z only include *.exe.

I see. I hadn't noticed the first time that build.bat failed to run that because after unpacking under cygwin, the tools don't have execute permission. Setting chmod +x doesn't work either because next time the files are freshly unzipped. Running build.bat from cmd.exe doesn't work either because there is no unzip.
If I fix these issues manually, open-wsl.exe fails with an error popup:
---------------------------
open-wsl.exe
---------------------------
Error:  Failed attempt to launch program or document:
Action: <C:\cygwin\home\towo\wsl-terminal-master\src\bin\mintty -t "        " -e /bin/wslbridge -t "bash>
Params: <>

Specifically: Das System kann die angegebene Datei nicht finden.

	Line#
--->	015: Run,%A_ScriptDir%\bin\mintty -t "%title%" -e /bin/wslbridge -t "%shell%

The current thread will exit.
---------------------------
OK
---------------------------

I have no Cygwin/MSYS2 installed, build.sh will run *.sh with /bin/bash in WSL, and default permissions of all files are 0777. wget/tar/xz/gzip/unzip/p7zip should be installed before running build.sh.
After running build.sh, chdir to C:\cygwin\home\towo\wsl-terminal-master\output, and run open-wsl.exe,  instead of C:\cygwin\home\towo\wsl-terminal-master\src\open-wsl.exe. Maybe I should remove *.exe from src directory.

There is no wsl-terminal-master\output directory; the source package does not build without additional dependencies or/and manual intervention in any environment (cygwin, WSL, cmd). In a plain Windows cmd window, e.g., it fails because there is no unzip. A script build.sh does not exist. Not sure how the source package is supposed to be built.
[Maybe I should create separate issues, but let me summarize my findings once more for now]
About the binary package: The howto says "download and run open-wsl" but that does not work because it needs to be unpacked first which does not work on a plain system because 7z is not part of a plain standard system configuration. I suggest to avoid such dependencies.
About the package contents: open-wsl displays a mintty popup:
/usr/bin/mintty: could not load icon file '/Cygwin-Terminal.ico':

then runs WSL in the current directory.
Major use cases are not addressed yet:

How to start terminal from WSL?
How to start terminal with login bash, in home dir?
How to start terminal from desktop shortcut?
How to configure an icon; maybe some kind of overlay of the WSL icon (which is round on transparent background) on the mintty terminal icon might be nice.
About the latter, I will add a comment in the respective mintty issue https://github.com/mintty/mintty/issues/558

If wsl-terminal-master\output is not exit, build.bat failed to run. I add some error messages (https://github.com/goreliu/wsl-terminal/commit/d19e990c7bfc1d08c103f40d25fc1f3cb9dfc4d6). WSL (instead of Cygwin) is depended to run build.bat, make sure build.bat will call C:\Windows\System32\bash.exe (or C:\Windows\SysNative\bash.exe), instead of C:\Cygwin64\bin\bash.exe.
And I add a zip package (https://github.com/goreliu/wsl-terminal/commit/446a7836609b4cb84cda3d0ef42465f87d32e1ad).
open-wsl will not display a mintty popup:
/usr/bin/mintty: could not load icon file '/Cygwin-Terminal.ico':
``

Because open-wsl call /bin/mintty instead of /usr/bin/mintty, and it will not find Cygwin-Terminal.ico.

How to start terminal from WSL?

The only way is using wstart, wstart open-wsl.

How to start terminal with login bash, in home dir?

To start terminal with login bash, set shell=bash -l in etc/wsl-terminal.conf (https://github.com/goreliu/wsl-terminal/commit/bff5ee0bc0599cbb6c0f67383f60896e62813b35 fix it).
To start in home dir, add cd $HOME in .bash_profile, and I will think other ways.

How to start terminal from desktop shortcut?

Create a shortcut to open-wsl.exe in explorer.exe.

I will add a icon= config in wsl-terminal later.
I use  %LOCALAPPDATA%\lxss\bash.ico in run-wsl-file.exe because if a file is associated with run-wsl-file.exe, bash.ico seems better then a AutoHotKey icon (a letter "H"). It may cause a copyright issue, I will find other ways, or find another icon.

I have replaced all icons of *.exe (https://github.com/goreliu/wsl-terminal/commit/818cc5ddb1d0832cd0e8d4be532af752304f69d0, https://github.com/goreliu/wsl-terminal/commit/f0b8c0ff8872f8a5feba73c93c2b0bb39780d1fd), I downloaded them from http://www.iconfont.cn, they are free-to-use images.

I may have been misleading about the icon; I think referring to %LOCALAPPDATA%\lxss\bash.ico is a good choice as long as mintty has no icon overlay feature (i.e. combining the bash icon with its own dynamically). (The idea of such a feature is to avoid inclusion of any bash.ico-derived file in the package.)
About wstart open-wsl, bash -l, and a desktop shortcut, these were rather meant as suggestions to polish the package as to provide basic features out-of-the-box for less-expert users. I know it's fiddly to optimize a package to that aim and may take some time. I'm also considering to provide a very basic package myself, which you will be welcome to reuse for your more enhanced package.


there is some distance between toolbar and keyboard!

General information

JSQMessagesViewController version:
iOS version: iOS11
Devices/Simulators: both of them
Reproducible in the demo project? (Yes/No):
Any related issues:

What happened?
when i ran my apps and make toolbar become first response.there is some distance between toolbar and keyboard!

This library is unsupported, please look into alternatives like MessageKit. Thank you!


Dev/mibi/arc inheritance circle

What type of PR is this? (check all applicable)

[X] ♻ Refactor
[X] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 👷 Optimization
[ ] 📝 Documentation Update
[ ] 🔖 Release
[ ] 🚩 Other

Description
This PR makes Arc inheriting from Circle.
Related Tickets & Documents
This PR closes #262
Added tests?

[X] 👍 yes
[ ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📓 docs
[X] 🙅 no documentation needed

@cesarecaoduro and @d3ssy probably some tests between Arc and Circle are repetitive, what do you think?
This Pr only makes the inheritance of Circle into Arc, next step will be to integrate the new methods for ICurve and remove the properties for the NURBS and expose the ToNurbs method.

@cesarecaoduro and @d3ssy probably some tests between Arc and Circle are repetitive, what do you think?
This Pr only makes the inheritance of Circle into Arc, next step will be to integrate the new methods for ICurve and remove the properties for the NURBS and expose the ToNurbs method.

Yes, since arc inherits from circle, any tests written for circle don't need to be repeated for arc. ArcTests should be testing any additional behavior if any.

@sonomirco and @d3ssy I would probably keep both for readability


New package: EarthSciMLData v0.1.0

Registering package: EarthSciMLData
Repository: https://github.com/EarthSciML/EarthSciMLData.jl
Created by: @ctessum
Version: v0.1.0
Commit: 8d7003e27aa73c1c25189d587a69bd7c8ae10e0f
Reviewed by: @ctessum
Reference: https://github.com/EarthSciML/EarthSciMLData.jl/commit/8d7003e27aa73c1c25189d587a69bd7c8ae10e0f#commitcomment-99338436

Hello! This package has a similar name to another one because they are meant to be part of a related ecosystem of packages. The ecosystem is named EarthSciML, one of the packages provides shared basic functionality, and the other one (this one) provides interfaces to data. Thank you for your consideration! Suggestions welcome.

Hello! I'd like to withdraw this request in favor of one with a different name.

This one: https://github.com/JuliaRegistries/General/pull/77035


Issue with 3.21_AnimationBlueprint in UE 5.1

I'm using Unreal Engine 5.1 and have encountered an issue with the 3.21_AnimationBlueprint character animations. After importing them, the animations play no problem when previewing, however when the game starts, the character runs for a second or two and then holds position. Any ideas why this might be?

Hi,
It is possible that the speed variable is set to a higher value than 0 or the spawning is happening in midair, causing velocity to be produced.

Where would I change the speed variable? The character keeps moving forward in the game, but the character does not continue the animation. It stops moving in the middle of the sequence and keeps going with arms frozen out.

Hi,
Regarding this query, please check to make sure the animation is set to "loop". It sounds like it may be stopping after playing once.

Thanks, that worked. In the CharacterAnims_Run (state) under Details there is a Loop Animation checkbox that once checked resolved the issue!


Cache Drive Backups

Description
If our application is unable to connect to Redis, a CriticalError exception is now thrown.
CacheFactory catches the CriticalError exceptions that are thrown when Redis, Memcache and Predis fail to connect, it logs the exception and then attempts to initialize the backup cache driver (and if that fails it initializes dummy).
Checklist:

[ ] Securely signed commits
[x] Component(s) with PHPdocs
[x] Unit testing, with >80% coverage
[x] User guide updated
[x] Conforms to style guide

Was originally pull request https://github.com/codeigniter4/CodeIgniter4/pull/2057, but i had some issues... So here it is again (Working this time hopefully).

I like the idea of this a lot. I made one comment on the code, but could you add the same process to the other drivers?

@lonnieezell I've attempted to implement a similar process for the other drives as requested.
The DummyHander, FileHander and WincacheHandler's initialize() method contain no logic, so I have ignored these.
I modelled my solution on how PredisHandler initialize() method, so I've just added some comments there.
I've changed MemcachedHandler initialize() a fair bit, I've broken it down into two sections contained within the if/elseif block.
I've tested the Redis, Predis, Memcached drivers and I'm happy with how they work (If they can't connect to the cache server, they throw a CriticalError exception). I'm unable to test the Memcache driver (I can't seem to get it to work with PHP 7+).
I'm confident with the code I've written to check if the cache server is contactable when using Memcache, I followed https://www.php.net/manual/en/memcache.connect.php, which suggests that if connect() returns false, the sever can not be contacted, which will trigger our CriticalError exception.
Is it worth performing similar checks on the Handlers I've ignored? Or does the isSupported() method handle it?

I believe isSupported should be sufficient, but it's been a while since I've been in those handlers, honestly.
Thanks for this!
In the future, though, we would prefer to have signed commits. There's instructions in the docs to help with that.


nuxt-unstyled-tailwind | ERROR  Pre-transform error: Octal literal in strict mode

The downloaded project from the nuxt-unstyled-tailwind folder does not run.
When running the npm run dev script
I get an error in the console.
ERROR  Pre-transform error: Octal literal in strict mode (14:29)
Nuxt 3.9.3 with Nitro 2.8.1                                                                                23:10:29
23:10:30
➜ Local:    http://localhost:3000/
➜ Network:  use --host to expose
ℹ Using default Tailwind CSS file                                                                                nuxt:tailwindcss 23:10:34
➜ DevTools: press Shift + Alt + D in the browser (v1.0.8)                                                                                23:10:37
ℹ Tailwind Viewer: http://localhost:3000/_tailwind/                                                                                nuxt:tailwindcss 23:10:37
ℹ Compiled plugins/server.mjs in 787.03ms                                                                                23:10:38
ℹ Compiled plugins/client.mjs in 812.99ms                                                                                23:10:38
ℹ Compiled types/plugins.d.ts in 812.02ms                                                                                23:10:38
ERROR  Pre-transform error: Octal literal in strict mode (14:29)                                                                                23:10:40
ℹ Vite client warmed up in 6968ms                                                                                23:10:47
ℹ Vite server warmed up in 6869ms                                                                                23:10:48
✔ Nitro built in 2245 ms                                                                                nitro 23:10:50

There is an error in the browser
500
[vite-node] [plugin:unctx:transform] [VITE_ERROR] virtual:nuxt:C:/Projects/1-zip/front-zip-shop/front-zip-shop/.nuxt/primevue-plugin.mjs:14:29
12 |  import FocusTrap from 'primevue/focustrap';
13 |  import AnimateOnScroll from 'primevue/animateonscroll';
14 |  import Lara from 'C:\Projects\1-zip\front-zip-shop\front-zip-shop\presets\lara';
|                               ^
15 |
16 |
at virtual:nuxt:C:/Projects/1-zip/front-zip-shop/front-zip-shop/.nuxt/primevue-plugin.mjs:14:29

Similar issue here. I found something interesting.
In windows, I get this:
eposprimevue-examplesolve import "C:UsersUsuariosource
uxt-unstyled-tailwindpresetslara" from "virtual:nuxt:C:\Users\Usuario\source\repos\primevue-examples\nuxt-unstyled-tailwind\.nuxt\primevue-plugin.mjs". Does the file exist?

But using an Ubuntu in WSL, it seems to work.
Windows Node version: 18.18.2
Ubuntu Node version: 21.6.1

I had a problem in the path to the presets folder in Winodws in the nuxt.config.ts file
You need to put a dot in front of the /presets folder
importPT: { as: 'Lara', from: path.resolve(__dirname, './presets/lara/') },

I had a problem in the path to the presets folder in Winodws in the nuxt.config.ts file
You need to put a dot in front of the /presets folder
importPT: { as: 'Lara', from: path.resolve(__dirname, './presets/lara/') },
Also did that. Didn't work either. The full project cloned does not work on windows for me.

@Alejandro-SB just get rid of path resolve and use relative path prefixed with ~ shortcut
importPT: { as: 'Lara', from: '~/presets/lara/' }

It will work both on Windows and Linux.


Prototype Pollution vulnerability affecting<EMAIL_ADDRESS>

Overview
Affected versions of this package are vulnerable to Prototype Pollution. An attacker can manipulate the prototype of an object, potentially leading to the alteration of behavior of all objects inheriting from the affected prototype by modify built-in Object.prototype through reachable special properties __proto__ and constructor.prototype. Thus, the attacker can use one of these properties to pollute the application logic that can be escalated to Denial of service, remote code execution or cross-site scripting attacks.
PoC:
(async () => {
  const lib = await import('@75lb/deep-merge');

var BAD_JSON = JSON.parse('{"__proto__":{"polluted":true}}');
var victim = {}
console.log("Before Attack: ", JSON.stringify(victim.__proto__));
try {
  lib.default ({}, BAD_JSON)
} catch (e) { }
console.log("After Attack: ", JSON.stringify(victim.__proto__));
delete Object.prototype.polluted;
})();

Output:
Before Attack:  {}

After Attack:  {"polluted":true}
How to prevent:

Freeze the root prototype using Object.freeze
Require schema validation of JSON input.
Avoid using unsafe recursive merge functions.
Consider using objects without prototypes (for example, Object.create(null)), breaking the prototype chain and preventing pollution.
As a best practice use Map instead of Object

Beside pollution risk, is this issue causing you any operational problems?

The pollution is a the first step to implement further attacks based on the implementation of the module, such as denial of service, commands execution, privilege escalation or any other OWASP 10 vulnerabilities. Please refer to the details in this article Snyk.io

You answered a question I did not ask - I know what the vulnerability is, read my question again please. Is this issue currently causing you any specific operational issues? For example, is a security policy blocking you using the module, or an app which uses the module?

Just to clarify, I'm not personally using this package. However, as this issue poses a potential risk to users, I've opened it to collaborate on addressing the vulnerability for the community's safety.

yes, that's fine - I was just trying to assess the level of urgency..

This project currently uses lodash to merge objects - this issue has already been reported upstream, will monitor it..

This issue has been reported and is now reporting on npm audit: https://gist.github.com/mestrtee/b20c3aee8bea16e1863933778da6e4cb
The issue is resolved in later versions of Lodash. https://github.com/lodash/lodash/blob/6a2cc1dfcf7634fea70d1bc5bd22db453df67b42/test/proto-property-bugs.spec.js#L26-L54
This package imports lodash.assignwith which is still vulnerable. Lodash no longer applies security updates to dot-suffix "micro packages" (https://lodash.com/per-method-packages, https://github.com/lodash/lodash/issues/5894#issuecomment-2197551540).
This package should be updated to depend on lodash directly and import lodash/assignWith to ensure the minimum amount of code is bundled.

I replaced lodash.assignwith with lodash/assignWith as suggested. Released in @75lb/deep-merge v1.1.2. I also fixed a similar issue in table-layout (used by command-line-usage).
Thanks for the report, let me know if there's anything else.


HADOOP-17643 WASB : Make metadata checks case insensitive

3102

 Quality gate: Passed
 Everything looks good
Details

 Quality gate: Passed
 Everything looks good
Details


[UI] X in upper right corner of Logs screen doesn't funtion

The x in the upper right corner of the logs screen should close the window.  It does not do anything.

Fixed in rancherio/ui#105

Verified Fixed


Cleanup for 1.0 release

added *.log to .gitignore to prevent any npm-debug.log from
accidentally making its way into git

added .jshintignore, jshint to devDependencies, and a 'lint'
npm script to fill out support for linting in the project

added 'ci' npm script to add linting to the travis build job

added Node 4 & Node 6 to travis versions to test against

added license to package.json

removed .npmignore since npm follows .gitignore in the absence
of .npmignore and the files were identical

removed test/common.js because it was setting up a lot of
boilerplate that this project doesn't use.

removed sinon and sinon-chai from devDependencies because they
are not used.

updated date in LICENSE

updated mocha and chai

updated version to 1.0

fixed double-quote lint error in test/index.test.js

fixed typo in name of iterate function

A bit of a spring clean as well? Thanks.


Require ConnectionConfig Write Access to Mask Data [#40]

Purpose
Currently the query builder will still try to generate erasure statements for database tables that the configured connection has no write access to. If no write access is granted, Fidesops should not generate these statements or try to execute them.
Changes

Add a check in GraphTask. erasure_request that verifies that the related ConnectionConfig for the particular TraversalNode has write_access or it skips building and executing an update statement.
Fixes some of our integration erasure unit tests that were using connection configs with read access

Ticket
Resolves https://github.com/ethyca/fidesops/issues/40

whoa thanks for that quick turnaround @eastandwestwind ⚡ ⚡


Added add-/move-/remove-entry event to list

Looks good, should we raise an issue for website doc?


Refactoring ModelData with JuMP.Model.

Much easier to keep track of variables this way, and also easier for benchmarking. This is going to take a while to get right!

Jeez, speed of evaluation of Exprs has become a serious problem. I am coming up with a Expr -> Function based workaround.

This completes all of #8 so far. Amped to merge...


Triage on ostracon#126

Opinion

Not to apply

Reason

Hard to apply to upstream, because of compatibility

ex) BlockIntervalSeconds metrics.Gauge != BlockIntervalSeconds metrics.Histogram


input sample_name

Hi,
I have a question. Does this pipeline only read one pair of samples? What if I have multiple samples?
Thanks

Hi
The tool only accepts one pair-end sample per run.
You can put all the fastq files in one folder, then run the bulk pipeline by specifying sample name per run, with the same configuration file if you do not want to change anything except the sample name.


Not yet published in npm

@TorstenDittmann, we already got appwrite for the web SDK, node-appwrite for the Node.js server SDK, Should I publish as svelte-appwrite?

@eldadfux svelte-appwrite fits very well.
I have done 50% of writing proper documentations for this and fixing consistency for naming, callbacks etc. Next weekend I will very likely have a production ready version 👍

Still haven't found it in npm

@TorstenDittmann are we good to go with the npm publish?

Yes, just need to finish the docs soon afterwards :+1:

@eldadfux I just double checked. Functionality is given and works, so could be published. I would like to change the version in package.json first! The documentation will be added little by little.

@TorstenDittmann  Everything looks good to me. Ready to publish?

@TorstenDittmann let me know when version is set and I'll publish the package. @Glide7 we should probably add an announcement about this in the community, @TorstenDittmann any chance you could help with it?

I will leave it on 0.0.1-alpha+001 for now, until the documentation is finished with examples and anything.
Code wise, I don't see anything blocking a release right now 👍
@Glide7 feel free to contact me if you need informations 🙇

Done: https://www.npmjs.com/package/svelte-appwrite Awesome work @TorstenDittmann 🎉


agario-client fixed

hey i using agario-client but fixed:
Error:
Bot_152: Connection Failed: Error: Socket Closed
Oke, please update.

i can help you with that

Bots are still patched, why are we worrying about this?


Can't deploy dashboard

Follow the blog on a Control Tower standard organization (main region eu-west-3)
When trying to deploy dashboard (from security account same as quicksight), I get Unrecognized resource types: [AWS::QuickSight::DataSet]
make deploy-dashboard-hub aws-region=eu-west-3 quicksight-user-arn=<userarn>
An error occurred (ValidationError) when calling the CreateChangeSet operation: Template format error: Unrecognized resource types: [AWS::QuickSight::DataSet]
make: *** [deploy-dashboard-hub] Error 254
same using CloudFormation template directly
error creating changeset: Template format error: Unrecognized resource types: [AWS::QuickSight::DataSet]

Issue is link to eu-west-3 region that seems to not support yet all quicksight resources  (or its from cloudformation support of resources)
deploy on eu-est-1 encounter an error on creation
AccessAnalyzerFindingsTemplate trigger TemplateError(type=ACCESS_DENIED, message=You do not have permissions to access the source)]
the source template propose seams not accessible
Minor improvement could be :

the cleaning part could improve to also empty bucket first (script or CustomResource  ;) )
pS3BucketName should be randomized to give possibility to retry without failure
add region that support this artefact

Thank you for the feedback and the improvement ideas.
We are looking into your issue and will suggest a fix if necessary.

Issue fixed with this commit. Please re-open it if you encounter any difficulty deploying the implementation.


Cannot able to change the default download directory and browser closes after last command without quit command

My code is
from msedge.selenium_tools import Edge, EdgeOptions
from webdriver_manager.microsoft import EdgeChromiumDriverManager
import os, time

options = EdgeOptions()
# options.use_chromium = True
# options.add_argument("headless")
# options.add_argument("disable-gpu")
prefs = {'safebrowsing.enabled': 'false','download.default_directory' : os.getcwd() + os.path.sep}
options.add_experimental_option("prefs", prefs)
options.add_experimental_option("useAutomationExtension", False)

driver.get('https://github.com/')

Please help me on this
I want to run this on headless and download the files to a desired directory
especially from wher the code is triggered

@krishna3008 I don't see where the driver is created in the sample code, but you need to create and configure the EdgeOptions first, and then pass them to the driver. EdgeOptions cannot be changed after the driver is created. For example:
options = EdgeOptions()
options.add_argument("headless")
prefs = {'safebrowsing.enabled': 'false','download.default_directory' : os.getcwd() + os.path.sep}
options.add_experimental_option("prefs", prefs)

driver = Edge(options = options)

I'm going to close this issue since this repo is now deprecated. Please upgrade to Selenium 4 which has built-in support for Microsoft Edge (Chromium). For help with upgrading, see https://docs.microsoft.com/en-us/microsoft-edge/webdriver-chromium/#upgrading-from-selenium-3. If you need further help with this issue, you can contact the Edge DevTools team at https://docs.microsoft.com/en-us/microsoft-edge/devtools-guide-chromium/contact


Add support for capture field on nncp deployment

Short description:
Add support for supplying captures for more generic NNCPs
More details:
What this PR does / why we need it:
Which issue(s) this PR fixes:
Special notes for reviewer:
Bug:

/wip


Get-DbaDatabase / Change suser_name to suser_sname

SID, not principal ID, is being passed to the function.
suser_name takes principal id
suser_sname takes sid
This change allows the query to return the correct owner name instead of NULL for AD logins.

Type of Change

[x] Bug fix (non-breaking change, fixes https://github.com/sqlcollaborative/dbatools/issues/6103)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose

Approach

Commands to test

Screenshots

Learning

Oops; merged into wrong branch; will attempt to fix...

Please use 6104 instead of 6102.


how to preprare diy dataset

@wlz987 ,
Preparing the dataset is simple.

Download the dataset from:

Google drive: Link to dataset
Baidu drive: Link to dataset

Unzip the dataset and modify this file and set the directories for the dataset and masks.

face_sep_mask = './Datasets/CelebAMask-HQ-mask-anno'  # specify the path
mask_path = './Datasets/CelebAMask-HQ/mask'  # specify the path to save result masks

Run the file:

python prepare_labels.py


[$500] [Report fields] Clean-up some logic with invoice report fields

Coming from here.
Bug 1: invoice report fields are showing on expense reports:

The fieldList[0].target can be expense|invoice|paycheck
Lets make sure we only show the particular report fields on the respective report types
Bug 2: Report fields in More features is auto-enabled when connecting to an accounting solution (i.e QBO) despite not having any coding selected to import as report fields:

CC: @mountiny @JmillsExpensify
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~012b274cfc3d17eda0
Upwork Job ID:<PHONE_NUMBER>157983346
Last Price Increase: 2024-08-05

Issue OwnerCurrent Issue Owner: @jayeshmangwani

@mountiny How can we import the report field via QBO?

@nkdengineer i have asked Tom if he has some steps in mind but also feel free to make proposal for the first part

Not Overdue, Awaiting Proposal

I just connected a workspace to QBO and then the Report fields menu item appeared in the LHN of the workspace settings.

Not Overdue, Awaiting Proposals

Increasing the price so we can prioritize a fix for this issue.

@hungvu193 you have a QBO account, right? Perhaps you want to take a shot at this.. :)

@hungvu193 you have a QBO account, right? Perhaps you want to take a shot at this.. :)

Yeah. Let me take a look at this one 😄

Ok I can reproduce. I'll raise a PR soon

@trjExpensify  For the first issue, the draft PR is here.
For the second one, I assume that you want to add an import section for QBO's Import Page right? If so, is there any docs for it?

Discussing here.

Not overdue, Draft PR is here https://github.com/Expensify/App/pull/46858

@mountiny Can you please assign me to this issue so it won't be slipped from my K2 list? TIA 😄

Done!

@trjExpensify It appears that the automation failed, but this PR https://github.com/Expensify/App/pull/46858#issuecomment-2294133960 went to production on August 15th. The issue is now ready to be paid out

Alrighty, payment summary as follows:

$250 to @jayeshmangwani for the C+ review
$250 to @hungvu193 for the fix

Both of you can go ahead and request on ND, closing!

Requested on ND as per https://github.com/Expensify/App/issues/46303#issuecomment-2312456251

$250 approved for @hungvu193

$250 approved for @jayeshmangwani


[SPARK-50430][CORE] Use the standard Properties.clone instead of manual clone

What changes were proposed in this pull request?
This PR proposes to use the standard Properties.clone instead of manual clone
Why are the changes needed?
In a very rare condition, when the properties were changed during the clone of Properties, it might throw an exception as below:
: java.util.ConcurrentModificationException
	at java.util.Hashtable$Enumerator.next(Hashtable.java:1408)
	at java.util.Hashtable.putAll(Hashtable.java:523)
	at org.apache.spark.util.Utils$.cloneProperties(Utils.scala:3474)
	at org.apache.spark.SparkContext.getCredentialResolvedProperties(SparkContext.scala:523)
	at org.apache.spark.SparkContext.runJobInternal(SparkContext.scala:3157)
	at org.apache.spark.rdd.RDD.$anonfun$collect$1(RDD.scala:1104)
	at org.apache.spark.rdd.RDDOperationScope$.withScope(RDDOperationScope.scala:165)
	at org.apache.spark.rdd.RDDOperationScope$.withScope(RDDOperationScope.scala:125)
	at org.apache.spark.rdd.RDDOperationScope$.withScope(RDDOperationScope.scala:112)
	at org.apache.spark.rdd.RDD.withScope(RDD.scala:454)
	at org.apache.spark.rdd.RDD.collect(RDD.scala:1102)
	at org.apache.spark.mllib.evaluation.AreaUnderCurve$.of(AreaUnderCurve.scala:44)
	at org.apache.spark.mllib.evaluation.BinaryClassificationMetrics.areaUnderROC(BinaryClassificationMetrics.scala:127)
	at org.apache.spark.ml.evaluation.BinaryClassificationEvaluator.evaluate(BinaryClassificationEvaluator.scala:101)
	at sun.reflect.GeneratedMethodAccessor323.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at py4j.reflection.MethodInvoker.invoke(MethodInvoker.java:244)
	at py4j.reflection.ReflectionEngine.invoke(ReflectionEngine.java:397)
	at py4j.Gateway.invoke(Gateway.java:306)
	at py4j.commands.AbstractCommand.invokeMethod(AbstractCommand.java:132)
	at py4j.commands.CallCommand.execute(CallCommand.java:79)
	at py4j.ClientServerConnection.waitForCommands(ClientServerConnection.java:199)
	at py4j.ClientServerConnection.run(ClientServerConnection.java:119)
	at java.lang.Thread.run(Thread.java:750)

We should use the standard clone method.
Does this PR introduce any user-facing change?
It fixes a very corner case bug as described above.
How was this patch tested?
It's difficult to test because the issue is from concurrent execution.
Was this patch authored or co-authored using generative AI tooling?
No.

Merged to master and branch-3.5

cloneProperties was using forEach right ? The stacetrace does not seem to match the code ?

There was a bit of code change here. It will be matched after https://github.com/apache/spark/pull/49036 and https://github.com/apache/spark/pull/48993 ..

Thank you for reverting this from branch-3.5.

https://github.com/apache/spark/commit/f7c48fe24fd8fb5f7f3eb9b3bec2659d99e4307c


Create function to poll the counter from substraTEE-worker

Create a function to poll the counter from the https://github.com/scs/substraTEE-worker

Action: compose correct poll message
Action: send request to substraTEE-worker

can we close this?

Yes. This was implemented with 1e3e30c


Add support for content-type while uploading files through ObjectStore API

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
I use ObjectStore in a project that requires uploading files (mostly PDF) to AzureBlobStorage, and sometimes to S3. These files are later displayed on a web page using the  tag.
Since object_store doesn't support including content-type header with the file to be uploaded, it becomes a problem with AzureBlobStorage, which defaults the content-type to application/octet-stream. In such case,  downloads the file instead of displaying it. Also mentioned in this StackOverflow question.
Describe the solution you'd like
Have the ability to add a content-type header while uploading a file using ObjectStore.
Describe alternatives you've considered
I've written some code modifying the ObjectStore trait's method definition, adding an additional argument content_type: Option<&str>
pub async fn put_request<T: Serialize + crate::Debug + ?Sized + Sync>(
        &self,
        path: &Path,
        bytes: Option<Bytes>,
        is_block_op: bool,
        query: &T,
        content_type: Option<&str>,
    ) -> Result<Response> {
        let credential = self.get_credential().await?;
        let url = self.config.path_url(path);

        let mut builder = self.client.request(Method::PUT, url);

        if !is_block_op {
            builder = builder.header(&BLOB_TYPE, "BlockBlob").query(query);
        } else {
            builder = builder.query(query);
        }

        if let Some(ct) = content_type {
            builder = builder.header(&CONTENT_TYPE, ct);
        }

        if let Some(bytes) = bytes {
            builder = builder
                .header(CONTENT_LENGTH, HeaderValue::from(bytes.len()))
                .body(bytes)
        } else {
            builder = builder.header(CONTENT_LENGTH, HeaderValue::from_static("0"));
        }

        let response = builder
            .with_azure_authorization(&credential, &self.config.account)
            .send_retry(&self.config.retry_config)
            .await
            .context(PutRequestSnafu {
                path: path.as_ref(),
            })?;

        Ok(response)
    }

Additional context
This solution works very well for me, and I have it in my local setup. The problem I face is that I also use delta-rs crate, where I need to use ObjectStore as the backend. Therefore I cannot use my code, because it then says trait not satisfied.

This is tricky, the aim of ObjectStore is to provide a common set of functionality across filesystems, memory, and object stores. Here the complication is filesystems don't have a concept of a MIME type, and so I don't think we can easily add this to the ObjectStore trait itself.
Perhaps we could add mime settings to ClientOptions?
Perhaps something like
/// Set the default CONTENT_TYPE for uploads
fn with_default_mime_type(self, mime: impl Into<String>)

/// Set the CONTENT_TYPE for a given file extension
fn with_mime_type_for_suffix(self, mime: impl Into<String>, extension: impl Into<String>)

Would that work for your use-case?

I think so, yes. Because the first problem I faced was the inability to make any changes to the HTTP client, since that's where one would add any header.
In the changes I've made, I was able to add it to the trait itself, but had to ignore the content_type argument for local file system by underscoring it. Even though it works without any problems, I'm afraid its presence will confuse users using local file system as the backend.
In such a case, should I write the code for the solution you've proposed above? I think it's rather straightforward with one question.
What's the purpose of having both mime and extension in fn with_content_type_for_suffix(self, mime: impl Into<String>, extension: impl Into<String>)? I mean, couldn't we have a mime type for the given file type anyway?

should I write the code

If you would like to, I'd be happy to review it. Otherwise I can add this to my list

What's the purpose of having both mime and extension in fn

Mainly to avoid having to bring in a dependency on a mime inference crate, and also to support non-standard file extensions.
So you'd be able to do something like
with_content_type_for_extension("myjson", "application/json")

Sounds like a plan. I'll write the code and get back once I have something tangible ready.

This may sound like a stupid question, but I can you tell me in which header should I keep the extension, and if it's supposed to be kept as an optional parameter in the content-type header after the forward slash? I'm not sure if I should keep it inside content-disposition either. This seems new to me.
In other words, would be nice if you explained the flow to me when it comes to handling the extension.

This may sound like a stupid question, but I can you tell me in which header should I keep the extension

I meant for the config to contain a HashMap from file extension to ContentType, on writing a file it would first lookup the extension in this HashMap, on match it would use the corresponding ContentType entry, otherwise it would use the default. Does that make sense?

Yep, totally. Correct me if I've misunderstood.
The extension is the key to get the corresponding content-type (e.g. application/json if extension is json) already stored in a HashMap. The content-type argument is the default value if that fails. And I can populate the said HashMap from something like here.

I see. This explaination is so much better. Thanks.

I've committed the solution. Though this still won't work for GCP. The reason for this is the fact that calling build() on any of the cloud provider's builder e.g., MicrosoftAzureBuilder::default().build() generates the underlying client for that provider by calling .client() on ClientOptions (default or user provided).
This client obviously cannot handle our requirement above, and therefore the content type must be handled within the implementation of put(). And we need to be able to access ClientOptions for that to be possible.
While the upload object for Azure and AWS still contain the underlying ClientOptions when we call build() on their builders, GCP does not. Therefore I couldn't handle it now. Also couldn't think of a way to achieve it without changing any of the original structs.

Feel free to change the GCP implementation as required, none of those details are public and can therefore be changed without issue

I've made the necessary changes in the PR, including the test cases for filename() and extension(). Please take a look and let me know if everything looks correct.

label_issue.py automatically added labels {'object-store'} from #3358


I am confused how can I use it for s3 images?

is it same as adding https://wsrv.nl/?url=s3 image url....?  I am getting 404 this way

Images on public S3 buckets can be reached via <bucket-name>.s3.<region>.amazonaws.com, for example:
https://wsrv.nl/?url=https://libvips-packaging.s3.eu-west-1.amazonaws.com/zebra.jpg
Private S3 buckets requires HTTP basic authentication which is not something we'd support on our public service (see #341). If you're able to host your own solution, you can support this by doing something like this within the ngnix configuration:
location /s3/ {
    weserv filter;

    # The S3 bucket we use
    proxy_pass http://libvips-packaging.s3.amazonaws.com/;
    proxy_set_header Host libvips-packaging.s3.amazonaws.com;

    # Authorization should not be needed on public buckets
    proxy_set_header Authorization '';

    # Header configuration
    proxy_hide_header x-amz-id-2;
    proxy_hide_header x-amz-request-id;
    proxy_hide_header x-amz-meta-server-side-encryption;
    proxy_hide_header x-amz-server-side-encryption;
    proxy_hide_header Set-Cookie;
    proxy_ignore_headers Set-Cookie;

    # Intercept proxy errors
    proxy_intercept_errors on;

    # Enable the upstream persistent connection
    proxy_http_version 1.1;
    proxy_set_header Connection '';
}

$ curl -s -o /dev/null -w "%{http_code}" http://localhost/s3/zebra.jpg?w=512
200

I hope this information helped. Please feel free to re-open if questions remain.


Fixed broken link in the pod.

Hi @d-ash
Please review the PR.
Many Thanks.
Best Regards,
Mohammad S Anwar

@manwar I am presently the maintainer - thanks very much for the fix!  Always a pleasure to hear from famous people :D .

I think the correct way to write a link to an email address is:
<EMAIL_ADDRESS>
instead of:
<EMAIL_ADDRESS>
unless you explicitly do not want to turn the email into a link.


Improve the scaffold of the gem and some awesome things

Changes
This changes are ordered with the changes you can see in the Files changed for easy tracking.

[x] Add binstubs: bin/console and bin/setup.
[x] Add some common paths to .gitignore (this are extracted from bundle gem).
[x] Configure RSpec to load spec_helper and print by default documentation format. (see .rspec file)
[x] Add CODE_OF_CONDUCT.md
[x] Improve the  Gemfile and counter_culture.gemspec
[x] Add Development and Releasing section to the README.md.
[x] Remove unused code from Rakefile and clean it up.
[x] Add CounterCulture::VERSION
[x] Move test configuration enviroment from spec/counter_culture_spec.rb to spec/spec_helper.rb

Thanks for the contribution, but I don't want to just wholesale merge this. Also, I think this will mess with how I manage gem releases with jeweler (I know, it's old). There's changes here that I'd be willing to pick over, but I don't have time to do this manually just now. For stuff like this having multiple commits with one change per commit would have made this a bit easier for me.
Thanks for your contributions, though!

I understand. I thought that you didn't use jeweler because is old, sorry.
I will "spam" you with small PR. 1 for each item of the checklist :)


Recommendations client

Adds new client to return User and Job recommendations in a hash with the recid (and other metadata). We need this RecID field to start doing NDCG on recs calls.
Done side by side with existing code (which is in the poorly name Recommendation class) for a transition and to avoid the major version bump.

CC is for identical code (this started as a copy paste code). The old code will eventually be removed, so I don't see any point in changing this code to appease the CC gods, and I don't see any value in refactoring to make these private methods available to both.

:thumbsup: LGTM. I dig the side-by-siding to avoid the major bump for now!

@tkling updated with Map, and added spec for single hash result (since i didn't get what the stupid [].flatten was doing at first and nearly removed it)

@Casao :thumbsup: :thumbsup: mergin'


Not so much a bug as a question about editing the run-applio.bat

I use the same settings (advanced settings) for each TTS session.  Is it possible for me to make them the default settings on startup?  I assume I can edit the run-applio.bat.  If this is possible I need to know how to properly implement the presets.

not possible atm


[Feature] [docs] docs document structure  Elasticsearch-flink.md

…k-plugin/
Signed-off-by: kodalien<EMAIL_ADDRESS>
Purpose of this pull request

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in you PR, please add License Notice according
New License Guide
[x] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

Where is the corresponding English document?

hi @Kodalien, some actions have been cancelled due to timeout, you need to trigger them.

Where is the corresponding English document?

https://github.com/apache/incubator-seatunnel/blob/dev/docs/en/flink/configuration/sink-plugins/Elasticsearch.md

对应的英文文件在哪里？

https://github.com/apache/incubator-seatunnel/blob/dev/docs/en/flink/configuration/sink-plugins/Elasticsearch.md

嗨@Kodalien，由于超时，某些操作已被取消，您需要触发它们。

 You should go through the logs to fix it instead of doing pointless rerun.

Since there is no update for a long time, I will close this PR, if there is any follow-up response, please leave a message.


Chef 13: Alert on dot notation of resources

From our deprecations page:
"Both lines in the example will cause separate deprecation warnings.
node.chef.server = "https://my.chef.server"
chef_server = node.chef.server"

iirc there is a handful of exceptions to dot notation, .chef_environment is one, don't recall others.


[Fix #11511] Fix a false negative for Style/YodaCondition

Fixes #11511.
This PR fixes a false negative for Style/YodaCondition when using constant.
Yoda conditions targets constant but only handles literal. This patch makes constant aware.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

I'd also update the cop examples to reflect this. Does this affect the other Yoga cop as well?

Indeed! I've update example of this cop and opened https://github.com/rubocop/rubocop/pull/11520 to affect other Yoda cop as the same.

Thanks!


Test import with dask-expr on CI

Addresses #1308.
This PR is a first step toward ensuring compatibility with dask-expr. After getting the import to work at all, I recommend running the entire test suite with dataframe.query-planning=True on CI.

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

Project coverage is 85.09%. Comparing base (93bb1e5) to head (082fd48).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1309      +/-   ##
==========================================
- Coverage   85.54%   85.09%   -0.46%
==========================================
  Files          77       77
  Lines        4261     4259       -2
  Branches      795      753      -42
==========================================
- Hits         3645     3624      -21
- Misses        447      466      +19
  Partials      169      169

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

Superseded by #1319


Please update the SR Linux base to v6.8 for resolving the BSA and SBSA Linux failures for v23.09_2.0.0 / v24.03_2.1.0 ACS images.

Comment from Chetan:
"As suspected the issue was still observed since the fix in linux mainline is available from on v6.8-rc6 revisions onwards
https://github.com/torvalds/linux/blob/v6.8-rc6/drivers/pci/msi/irqdomain.c#L64 and the latest ACS SR image is based on Linux v6.7.
Can you please raise a request on arm-systemready github to update the SR linux base to v6.8"

Hello @nvidiaarm,
SR ACS image is updated to build with v6.8 kernel.
https://github.com/ARM-software/arm-systemready/actions/workflows/SystemReady_Images_Daily_Build.yml
Daily Build number 11 onwards will have the changes
Please take SR image from this link: https://github.com/ARM-software/arm-systemready/actions/runs/8942694960
Thanks,
ACS team