proxmox main os disk crashed - how to restore the haasos disks on new proxmox VM?

Hi I tried to make a new hassos install and and all started just fine. The I replaced the EFI and sata0 disk with the original ones (residing on a different disk than Proxmox OS) but now it boots to EFI prompt. Anyone know about this UEFI boot stuff?

Do you have backup of entire VM ?
do have following :
under hardware : Bios OVMF (UEFI) ?
Under options  boot order : Disk 'sata0'

Do you have backup of entire VM ? No (yes, but it vanished along with the OS disk)
do have following :
under hardware : Bios OVMF (UEFI) ? Yes
Under options boot order : Disk 'sata0' Yes

It was a hassos try-out, but eventually became quite useful with pretty advanced automations. The VM disk was luckily placed on another disk.  It's frustrating the VM disks are there right in front of me, but I cant get to them.
Thanks

You could try this. it worked for me:
Make sure VM is disabled:
ha-manager set vm: --state disabled
Open GDISK to modify disk partition map
gdisk /dev/zvol/rpool/vm--disk-<DISK#>
Once GDISK opens, then just use the W command to re-write the partiion map
Re-enable (start) VM to verify the VM boots using the disk
ha-manager set vm: --state enabled

@meichthys I could kiss you right now! This solved my problem after a failed hassova update!

No need for a kiss. A good hearty thank you will do 👍


Changing Cryptocurrency category name to Blockchain

While the blockchain is mainly known for cryptocurrency and decentralized finance, it has a lot of applications like decentralized autonomous organizations, NFT marketplaces, etc.
For this reason, there are some APIs like #2179 that are focused on studying other aspects of the various blockchain apart from cryptocurrencies.
What do you think?

Thanks @daviddavo for opening this issue and providing a suggestion. We appreciate it.
But I think changing the category name of Cryptocurrency to the Blockchain may result into some confusion while searching for the APIs on Cryptocurrencies for beginners.
So, to resolve this issue we'll be creating a new category for the Blockchain which will explicitly contains the projects based on blockchain technology other than cryptocurrencies. In this way we can have both eliminating any possibility of confusions that may be occur.
Thanks for bringing it in front of us.😊

Closing this issue. ref #2189
Feel Free to share your opinion @daviddavo

New catagorius


Revert "Add an option to SignalR to guarantee ordering of send events…

… (#10506)"
This reverts commit 4646c4a5e4412dbff6e40ade887792241f5a2c94.

Well... we can keep the change in SignalR and just remove all the infrastructure stuff.

Just keeping this prepped. I'd like to take a stab at fixing the build if I can with the change.

Just keeping this prepped. I'd like to take a stab at fixing the build if I can with the change.

Sure, but a better prep is to not remove the feature you made :D

@aspnet/build could you merge this in? The official build is currently broken due to this change.
cc @Pilchie \ @mkArtakMSFT


FMS 17 Schedules not appearing on FMS ACE calendar

Describe the bug
None of the scheduled scripts on our FMS 17 server appear in the FMS ACE calendar.
To Reproduce
Steps to reproduce the behavior:

Login to FMS ACE at https://ourserver.com/fms-ace/ with known-good FMS Admin credentials
Current view displayed is "Weekly" with current day highlighted in pale yellow
Change view to Month or Day - still no scheduled FileMaker script events from our FMS 17
I can add "Verify Schedule" or "Add Message Schedule" events from within FMS ACE, — or add a "Backup Schedule" from the FMS Admin Console — and they do appear on the calendar as expected.

Expected behavior
I would expect to see our Scheduled FileMaker Server Scripts appear on the calendar
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: macOS 10.13.6
Browser Safari
Version 11.1.2

Server:

OS: 10.12.6
FMS: <IP_ADDRESS>

Thank you very much for the thorough description of the reported issue, and for including some screenshots.
I tried replicating the behavior that you described, by creating 2 FileMaker script schedules via the FAC (FileMaker Admin Console), and then trying to view them in the FMS ACE calendar. I did this in a host machine running 10.12.6 and <IP_ADDRESS>, just like your environment. I used Safari 11.1.12 to launch ACE, the only difference being that I am on 10.12, and you reported being on 10.13.
I was not able to replicate what you described.
It's interesting that it's only FileMaker Script schedules that are now showing up in your copy of ACE. Would you be able to post another screenshot of the detailed settings of one of those 2 FMP Script schedules? I wonder if the combination of settings that you have chosen for those schedules, could be uncovering a bug in the code that we haven't run into thus far.
Now, you included a screenshot of the FM Script schedules in the FAC; can you confirm that you created those schedules via the FAC, and not via another means? I ask this because when you create schedules via the Admin API, you can set repeating settings for a schedule (for it to repeat every so many minutes or hours within a day, which is separate from the daily, weekly, etc frequency that you specify for the schedule). The repeatTask and repeatInterval API endpoint parameters cannot be specified from the FAC, and they are not supported in the current version of ACE. If you created those script schedules outside of the FAC and used those settings, that could be causing the issue you described.
Can you confirm what version of FMS ACE you are running? That should be displayed in the bottom left corner of the footer in the ACE app.
Finally, albeit unlikely that the following could be affecting this issue, let me ask:
a) are you using a custom or the default FMI certificate in your FMS?
b) can you replicate the issue when you use Firefox or Chrome instead of Safari?

I did this in a host machine running 10.12.6 and <IP_ADDRESS>, just like your environment.

Thanks for the effort. I seem to have reported there server config incorrectly as being 10.2.6 when it's actually 10.13.6, just like my client FMPA 17 system. I updated the comment to reflect the change, and added some additional info about the server.

I was not able to replicate what you described.

I apologize for the incorrect server OS version, and have edited the original comment with the updated configuration info. Perhaps the additional system configuration info I've added will assist you.

It's interesting that it's only FileMaker Script schedules that are now showing up in your copy of ACE. Would you be able to post another screenshot of the detailed settings of one of those 2 FMP Script schedules? I wonder if the combination of settings that you have chosen for those schedules, could be uncovering a bug in the code that we haven't run into thus far.

can you confirm that you created those schedules via the FAC, and not via another means?

The Schedules were solely created within the FMS 17 Admin Console web page.

Can you confirm what version of FMS ACE you are running?

Yes…it is displayed as "FMS ACE v1.0.0-1-g1fe1f3d"

a) are you using a custom or the default FMI certificate in your FMS?

We are not using the default certificate. I'm not certain of the criteria for what you refer to as a "custom" certificate.

b) can you replicate the issue when you use Firefox or Chrome instead of Safari?

The issue persists in Safari [Version 11.1.2 (13605.3.8)], FireFox [61.0.1 (64-bit)], and Google Chrome [Version 67.0.3396.99 (Official Build) (64-bit)].

Sorry for the delayed response here, I was not getting automated notifications for comments. Actually, Scott and I met at the FileMaker Developer Conference today, and we confirmed that the issue he reported affects FileMaker Script schedules that use the repeating option within a day, i.e. the "Every 1 Hours, 2:37 - 23:37" in the last screenshot uploaded. This is a feature not supported by the current version of ACE.
Please note that in my previous comment, I had stated that the repeatTask and repeatInterval API endpoint parameters cannot be specified from the FAC, and that was incorrect; these options can be specified via the FAC by choosing the "Between" option under "Time".

It appears that even Scheduled Server scripts created in the FileMaker Admin Console are not supported with FM ACE, so I will make a Feature Request, instead. Closing this Bug report.


Need routes for trafic returning to VPN users

I don't think traffic is being routed back to the VPN clients.  We need routes sending all <IP_ADDRESS>/21 to the OpenVPN server.

@felddy - I have no idea what you were seeing.  I assume you were talking to a FreeIPA server or replica, since those are the only instances deployed to the COOL right now other than the OpenVPN server.  The FreeIPA server and replicas are set up to only accept connections from the FreeIPA client and server security groups or certain privileged IPs.  Previously, only the OpenVPN server was in the client security group, only the replicas were in the server security group, and only the devs' home IPs were in the list of privileged IPs.  I suspect this is why you were not seeing traffic.
Security groups cannot be shared across VPCs, so I have added the entire COOL /9 CIDR to the list of privileged IPs.  (I also removed our home IPs since the FreeIPA server and replicas are now in private subnets.)  You can see this change at jsf9k/cool-tfvars@6a0de6f.
This change has been deployed, so please let me know if it resolves your issue.

Sorry I didn't go into more detail.  This was just a reminder for Monday.  VPN clients are unable to get DNS responses.  My theory is that the DNS service at <IP_ADDRESS> is unable to route responses back to the clients in the <IP_ADDRESS>/21 subnet.  So a route needs to be created sending that traffic to the VPN server for routing.   Theory...

@felddy, please see cisagov/cool-sharedservices-networking#12 and cisagov/cool-sharedservices-openvpn@f7ffb60c4b02f3cdb3b2ac77ad6c2c352e602e10.  I haven't deployed those changes yet because I can't do it without redeploying the OpenVPN server.  (This is for nonsense reasons - our isort pre-commit hook added some comments to the cloud-init python code.)
Let's work together tomorrow to deploy these changes and see if it fixes your issue.

Well this is a fine kettle of fish:
An execution plan has been generated and is shown below.
Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # aws_route.private_vpn_routes["<IP_ADDRESS>/24"] will be created
  + resource "aws_route" "private_vpn_routes" {
      + destination_cidr_block     = "<IP_ADDRESS>/21"
      + destination_prefix_list_id = (known after apply)
      + egress_only_gateway_id     = (known after apply)
      + gateway_id                 = (known after apply)
      + id                         = (known after apply)
      + instance_id                = "i-03d0d14056dafd1e7"
      + instance_owner_id          = (known after apply)
      + nat_gateway_id             = (known after apply)
      + network_interface_id       = (known after apply)
      + origin                     = (known after apply)
      + route_table_id             = "rtb-022632c9ceaa6c55e"
      + state                      = (known after apply)
    }

  # aws_route.private_vpn_routes["<IP_ADDRESS>/24"] will be created
  + resource "aws_route" "private_vpn_routes" {
      + destination_cidr_block     = "<IP_ADDRESS>/21"
      + destination_prefix_list_id = (known after apply)
      + egress_only_gateway_id     = (known after apply)
      + gateway_id                 = (known after apply)
      + id                         = (known after apply)
      + instance_id                = "i-03d0d14056dafd1e7"
      + instance_owner_id          = (known after apply)
      + nat_gateway_id             = (known after apply)
      + network_interface_id       = (known after apply)
      + origin                     = (known after apply)
      + route_table_id             = "rtb-0b451f1b5f6f43148"
      + state                      = (known after apply)
    }

  # aws_route.private_vpn_routes["<IP_ADDRESS>/24"] will be created
  + resource "aws_route" "private_vpn_routes" {
      + destination_cidr_block     = "<IP_ADDRESS>/21"
      + destination_prefix_list_id = (known after apply)
      + egress_only_gateway_id     = (known after apply)
      + gateway_id                 = (known after apply)
      + id                         = (known after apply)
      + instance_id                = "i-03d0d14056dafd1e7"
      + instance_owner_id          = (known after apply)
      + nat_gateway_id             = (known after apply)
      + network_interface_id       = (known after apply)
      + origin                     = (known after apply)
      + route_table_id             = "rtb-08ff45c22c66f3bb6"
      + state                      = (known after apply)
    }

  # aws_route.public_vpn_route will be created
  + resource "aws_route" "public_vpn_route" {
      + destination_cidr_block     = "<IP_ADDRESS>/21"
      + destination_prefix_list_id = (known after apply)
      + egress_only_gateway_id     = (known after apply)
      + gateway_id                 = (known after apply)
      + id                         = (known after apply)
      + instance_id                = "i-03d0d14056dafd1e7"
      + instance_owner_id          = (known after apply)
      + nat_gateway_id             = (known after apply)
      + network_interface_id       = (known after apply)
      + origin                     = (known after apply)
      + route_table_id             = "rtb-0fce54fa21948ed97"
      + state                      = (known after apply)
    }

Plan: 4 to add, 0 to change, 0 to destroy.

Do you want to perform these actions in workspace "production"?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

  Enter a value: yes

aws_route.public_vpn_route: Creating...
aws_route.private_vpn_routes["<IP_ADDRESS>/24"]: Creating...
aws_route.private_vpn_routes["<IP_ADDRESS>/24"]: Creating...
aws_route.private_vpn_routes["<IP_ADDRESS>/24"]: Creating...

Error: Error creating route: InvalidParameterValue: This route table is used by a subnet, and doesn't support route destination which are more specific than VPC local CIDR.
        status code: 400, request id: 01e8b982-22d2-428d-8a82-d9848ffceb17

  on vpc_routes.tf line 7, in resource "aws_route" "public_vpn_route":
   7: resource "aws_route" "public_vpn_route" {

Error: Error creating route: InvalidParameterValue: This route table is used by a subnet, and doesn't support route destination which are more specific than VPC local CIDR.
        status code: 400, request id: ebfe18e2-e31b-41a2-955d-a7fd9d153e80

  on vpc_routes.tf line 14, in resource "aws_route" "private_vpn_routes":
  14: resource "aws_route" "private_vpn_routes" {

Error: Error creating route: InvalidParameterValue: This route table is used by a subnet, and doesn't support route destination which are more specific than VPC local CIDR.
        status code: 400, request id: 5bef45ec-e0b6-47b1-b3c8-0be8a1c1d677

  on vpc_routes.tf line 14, in resource "aws_route" "private_vpn_routes":
  14: resource "aws_route" "private_vpn_routes" {

Error: Error creating route: InvalidParameterValue: This route table is used by a subnet, and doesn't support route destination which are more specific than VPC local CIDR.
        status code: 400, request id: dbe8da81-5287-4fa9-a062-275602f20e34

  on vpc_routes.tf line 14, in resource "aws_route" "private_vpn_routes":
  14: resource "aws_route" "private_vpn_routes" {

Closed the issue by mistake.  Sorry about that.

I'm unsure how to route this traffic.  Here is the documentation for the Terraform aws_route resource, although as you can see from the errors above I'm not allowed to specify anything more specific than a VPC local CIDR for VPC subnet routing tables.  Offhand, I can think of two things that might help:

Put the OpenVPN server in its own subnet with a CIDR identical to that of the client network.
Turn on NAT for the OpenVPN server.

These instructions indicate that NAT is not necessary, but they do not go into specifics.

I read those same docs.  Lets try the routing/subnet method first.  If that doesn't work we can fail back to the NAT.
Now might also be a good time to consider how this will scale with regards to load-balancing users across multiple VPN servers.  The NATing might make more sense in that case.

There is always this managed solution too, if we can get the certificate foo to work properly.

The routing method did not work, since there is no way to attach a routing table to a Transit Gateway attachment.  (One can attach route tables to Internet Gateways and Virtual Private Gateways, but not TGW attachments.)
We have decided to instead use NAT on the OpenVPN instance.  This ensures all traffic for teh clients is routed through the OpenVPN instance.

@felddy, do you think we can close this issue?  AWS Mike was pretty clear that we have to go with NAT in this case.

Yes.  NAT has solved this issue.  I'll open an issue to make the NAT changes to the OpenVPN server.


Update to latest Warg release to support registry federation

This is a small update to update to the latest Warg release. Supports federation across registries.

This PR is now ready for review

Thoughts on adding registry feature to the default feature set?

Closing in favor of #106


Make swift deployment optional

Swift is used to support one feature of contrail command, which is not necessary always used, but requires whole swift stack.
Deploy command also when SwiftInstance is not defined.

recheck ha

recheck ha

recheck ha

recheck

recheck ha


6513 pre tag helper

Closes #6513
What's Changed
Added pre_tag helper/filter which will wrap given value in

 tags.
Screenshots

TODO

[ ] Explanation of Change(s)
[ ] Added change log fragment(s) (for more information see the documentation)
[ ] Attached Screenshots, Payload Example
[ ] Unit, Integration Tests
[ ] Documentation Updates (when adding/changing features)
[ ] Example App Updates (when adding/changing features)
[ ] Outline Remaining Work, Constraints from Design

CI failure is due to #6509.


[MED] Fundraising Ideas Tool

Problem: It is difficult for students to come up with fundraising ideas.
Solution: Coaching is a large part of this solution. However, we can provide a database that allow students to filter ideas based on their interest, location, and more.
Priority: Medium

Wouldn't this just be action guides?

No, basically just simple table that has a huge list of like 200+ ideas filtered by tagged interest. But it's just the idea, not any info/resources on how to do it. Like for example "Yard Sale" or "Sell stickers". Eventually though we could use this data to figure out the kinds of ideas people are clicking on/not and why, how many they look through before they choose one, if they choose frmo list, etc

would this be per project or for everyone?


No Window Controls

Microsoft PowerToys version
0.71.0
Installation method
PowerToys auto-update
Running as admin
None
Area(s) with issue?
General
Steps to reproduce
Open PowerToys
No buttons to manage window [Minimize, Maximize and Close]

✔ Expected Behavior
Have buttons to minimize, maximize and close the window
❌ Actual Behavior
No buttons
Other Software
No response

/reportbug
cc: @niels9001

What happen if you hover or click where buttons should be placed? Maybe it's some theme/color issue we've seen before

Hi there!We need a bit more information to really debug this issue. Can you add a "Report Bug" zip file here?  Then drag the zipfile from your desktop onto the GitHub comment box in this issue. Thanks!

PowerToysReport_2023-07-12-22-58-35.zip

What happens if you hover or click where buttons should be placed? Maybe it's some theme/color issue we've seen before

The buttons seem to work. The spot where X would be turns red but they still don't appear

@niels9001 probably WASDK thingy, let's revisit after 1.4 is out :)

I just updated 0.73.0 and the problem still occurs

this is a need repro item.  .74 should have 1.4, this is very odd.

Looks like https://github.com/microsoft/microsoft-ui-xaml/issues/8899

Hi, i'm using PowerToys in dark mode and i face a similar issue for what i opened the ticket #27331
here are some captures to show what is happening on my side
With light theme i can see the buttons

When i switch from light to dark theme, the buttons are still visible

But as soon as the window loose the focus, the buttons goes dark and i no longer see them

they become visible again if i move my mouse over them

in light theme, the buttons become lighter when the window loose the focus but they still can be visible

After v0.74.1 update, the welcome window focused in dark mode

i can barely see the black cross in the upper right corner
and the settings window title bar, when focused

and when not focused

not sure this ticket and ticket #27331 have the same root cause

The buttons (minimize, maximize, and close) are there, but they're barely visible. Also, they only disappear when the affected window has focus.

Looks like there may be a fix we could implement to fix this now.
https://github.com/microsoft/WinUI-Gallery/blob/main/WinUIGallery/SettingsPage.xaml.cs#L97-L102

any WinUI app inside the code base would need this fix


moteus_tool and lib/python do not work with pi3hat by default

Simi on discord reported that the python library did not use their pi3hat without manually constructing a Pi3HatRouter.  It should use it automatically.

I've been unable to reproduce this, so considering it closed.


Data set list jumps back to the top when you select a data set

To reproduce: Load enough data sets that the Volumes widget has a scroll bar. Scroll to the bottom and select the last data set in the list. The data set is selected but the list jumps back to the top.
This makes it inconvenient to rapidly switch between data sets that happen to be at the end of the list.

Fixed in master, awaiting release

v0.9.5


End quote before period

Test case (from War and Peace):
"But what is there to say about me?" said Pierre, his face relaxing into a careless, merry smile. "What am I? An illegitimate son!" He suddenly blushed crimson, and it was plain that he had made a great effort to say this. "Without a name and without means... And it really..." But he did not say what "it really" was. "For the present I am free and am all right. Only I haven't the least idea what I am to do; I wanted to consult you seriously."
Expected output:
“But what is there to say about me?” said Pierre, his face relaxing into a careless, merry smile. “What am I? An illegitimate son!” He suddenly blushed crimson, and it was plain that he had made a great effort to say this. “Without a name and without means… And it really…” But he did not say what “it really” was. “For the present I am free and am all right. Only I haven’t the least idea what I am to do; I wanted to consult you seriously.”
Actual output:
“But what is there to say about me?” said Pierre, his face relaxing into a careless, merry smile. “What am I? An illegitimate son!” He suddenly blushed crimson, and it was plain that he had made a great effort to say this. “Without a name and without means… And it really…” But he did not say what “it really” was. “For the present I am free and am all right. Only I haven’t the least idea what I am to do; I wanted to consult you seriously”.
Note the last end quote.
Interestingly, if I convert just the last sentence, it is fine. So I'm guessing it's an interference with some other features, such as the ellipsis.
Btw, I didn't thank you yet for this package. Awesome job!

Good catch! Thanks for sharing that. I’ll look into that in one of the upcoming batches.
I’m humbled that you like the package, stay tuned for the improvements.

Fixed in version 2.3.2


Adding support for decimal and negative numbers

Opening a new ticket for adding support for decimal and negative numbers. More details for the same discussed here.
Expected outcomes
> parse("he scored fifty three point six percent in the finals")
he scored 53.6 percent in the finals

> parse_number("twelve point seven")
12.7

> parse_number("minus 9.4")
-9.4

It may make sense to keep separate issues for each.
Also, for decimal numbers, https://docs.python.org/3/library/decimal.html should probably either be used, or be usable (á la parse_float=Decimal of json.loads).

Regarding decimal numbers, I've seen that the CLDR files contain the infinity word for about 26 locales, so it could be a good idea to add them in the future as float(inf). I didn't think it deeply, just wanted to add this idea here to avoid forgetting it.


Exception when comparing List<Dictionary<string, string>>

When I run the following code
var bar1 = new List<Dictionary<string, string>>
{
    new Dictionary<string, string>
    {
        {"a", "b"},
        {"c", "d"},
        {"e", "f"},
        {"g", "h"},
    }
};

var bar2 = new List<Dictionary<string, string>>
{
    new Dictionary<string, string>
    {
        {"e", "f"},
        {"g", "h"},
        {"c", "d"},
        {"a", "b"},
    }
};

var comparer = new CompareLogic {Config = {IgnoreCollectionOrder = true}};
var res = comparer.Compare(bar1, bar2);

an exception is thrown:
System.Reflection.TargetParameterCountException : Parameter count mismatch.
   at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture, Boolean skipVisibilityChecks)
   at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at System.Reflection.RuntimePropertyInfo.GetValue(Object obj, Object[] index)
   at KellermanSoftware.CompareNetObjects.IgnoreOrderTypes.IgnoreOrderLogic.GetMatchIndex(ComparisonResult result, List`1 spec, Object currentObject) in C:\Users\test\Downloads\Compare-Net-Objects-master\Compare-NET-Objects\IgnoreOrderTypes\IgnoreOrderLogic.cs:line 238
   at KellermanSoftware.CompareNetObjects.IgnoreOrderTypes.IgnoreOrderLogic.CompareInOrder(CompareParms parms) in C:\Users\test\Downloads\Compare-Net-Objects-master\Compare-NET-Objects\IgnoreOrderTypes\IgnoreOrderLogic.cs:line 76
   at KellermanSoftware.CompareNetObjects.IgnoreOrderTypes.IgnoreOrderLogic.CompareEnumeratorIgnoreOrder(CompareParms parms, Boolean countsDifferent) in C:\Users\test\Downloads\Compare-Net-Objects-master\Compare-NET-Objects\IgnoreOrderTypes\IgnoreOrderLogic.cs:line 45
   at KellermanSoftware.CompareNetObjects.TypeComparers.ListComparer.CompareType(CompareParms parms) in C:\Users\test\Downloads\Compare-Net-Objects-master\Compare-NET-Objects\TypeComparers\ListComparer.cs:line 57
   at KellermanSoftware.CompareNetObjects.RootComparer.Compare(CompareParms parms) in C:\Users\test\Downloads\Compare-Net-Objects-master\Compare-NET-Objects\RootComparer.cs:line 51
   at KellermanSoftware.CompareNetObjects.CompareLogic.Compare(Object object1, Object object2) in C:\Users\test\Downloads\Compare-Net-Objects-master\Compare-NET-Objects\CompareLogic.cs:line 207

Do I need to set additional configuration parameters or it this case not supported?

It looks like this is a defect with the IgnoreCollectionOrder.  It will work without ignoring the order:
        [Test]
        public void DictionaryWithIgnoreOrder()
        {
            var bar1 = new List<Dictionary<string, string>>
            {
                new Dictionary<string, string>
                {
                    {"a", "b"},
                    {"c", "d"},
                    {"e", "f"},
                    {"g", "h"},
                }
            };

            var bar2 = new List<Dictionary<string, string>>
            {
                new Dictionary<string, string>
                {
                    {"e", "f"},
                    {"g", "h"},
                    {"c", "d"},
                    {"a", "b"},
                }
            };

            var comparer = new CompareLogic { Config = { IgnoreCollectionOrder = false } };
            var res = comparer.Compare(bar1, bar2);
        }

It will be a while until I can create a fix for this.

Thanks for the update. Do you think this will be difficult to fix? I do not know your code base but maybe I could have a look to see if this is something not that difficult to fix for a newcomer to your project!

My guess is that it just needs a change in the IgnoreOrderLogic and then not process dictionary types since that is already handled by other code.

@carsten-j Fix coming soon

Fixed in version 3.08
https://www.nuget.org/packages/CompareNETObjects/3.8.0

Thanks


feat: add console-logging doc

Description

Suggestions for improvements to #216.
Resolves #212
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The Reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@kenji-miyake Thank you.
It could be a ROS standard, not just Autoware.
So how about proposing this to the ROS2 community once?
It may not all be accepted, but I think some will be accepted.
I think it is good to make a proposal in order not to create big a difference between the ROS community and Autoware.

@yukkysaito Thank you for your comment.
I agree with your idea, but I personally feel it has a little low priority now because we don't use so many third-party ROS nodes yet.
If someone will propose this, then it would be helpful.

@mitsudome-r Yes, but I'd like to wait for comments from @alanmengg for a while.


feat: added support for rollbar multi token

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Mtrojan21

Challenge Name

Issue Description

Browser Information

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code
// If relevant, paste all of your challenge code in here

Screenshot

@mtrojan21 What's your issue?


Build pipeline

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


[blog]: Kubeflow spark operator launch blog

Description
This PR introduces the launch blog post for the new Kubeflow Spark Operator.
@andreyvelich

I gave it a quick read.   Looks pretty good.   Will read again and provide any detailed comments.  Thanks for driving this!

We want to publish this blog post this week. Please let us know for any changes/suggestions.
/assign @kubeflow/wg-data-leads @kubeflow/kubeflow-steering-committee @StefanoFioravanzo @akgraner

/hold

@terrytangyuan, the blog has been updated with the date for the first community meeting. Please review it, and if you're satisfied, it's ready for merge. Thank you!
@andreyvelich  @yuchaoran2011

/unhold


quiche: enable downstream HTTP3 in quic_protocol_integration_test

Commit Message: Enable protocol integration test for HTTP3
Additional Description: some tests are disabled; Added quic client connection factory to HttpIntegrationTest to create quic client connection; Move the config setup logic from quic_http_integration_test to ConfigHelper and used it in the base class HttpIntegrationTest.
Add fixes in server and client streams to make some tests pass:
Check whether :authority is header has valid host name.
Set :protocol and :path field in request headers if the :method is CONNECT.
Quic trailers concatenates header values with the same key with '\0', split them into multiple headers.
Risk Level: low
Part of #12930 #2557

The TSAN keeps failing at integration_admin_test: https://dev.azure.com/cncf/envoy/_build/results?buildId=69583&view=logs&j=d1f76054-8f79-554b-6f4a-11d6a63b8e00&t=266e17e3-d213-54b5-deef-0dcee01da137&l=15243. Seems unrelated?

/retest


Move redundant footer from posts to template

Atm. all the blog post AsciiDoc files have the same footer ("About Debezium", "Get Involved"). This should be extracted into the template. Might also go under the author info actually.
@mdrillin @indraraj @uidoyen Here's a first follow-up improvement.

pending PR #558

merged, closing


giflib port

giflib port added

@trazme,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@trazme, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

Thanks for the PR!


ADLdap delete entry

Hi
i am building a project with Laravel that includes OpenLadp. I know that Adldap is designed to work with AD anyway in my case i am trying to make it work with openldap.
I am already able to create new contact in the database the problem is i cannot delete it.
Creation procedure works fine with little modification in Ldap.php
$user = Adldap::make()->user([
'cn' => $cn,
]);
$user->save();
For deletion procedure i tried two approaches:
1.) find the user and than i use the
$user = Adldap::search()->whereEquals('cn', $cn)->get();
$user->delete() function.  >>Error:  "Method delete does not exist."
2.) find the user
$user = Adldap::search()->whereEquals('cn', $cn)->get();
$ad = new Adldap($configuration);
$deleted = $ad->user()->delete($cn);  >> Error: "Call to undefined method Adldap\Laravel\Facades\Adldap::user()"
I can't find a way out any help will be appreciated.
thx

Hi @simonvolk,
When you call get() you're actually receiving a collection of results, not a single result.
You need to call first() instead of get() to receive a since result:
$user = Adldap::search()->whereEquals('cn', $cn)->first();

$user->delete();
``

Hi stevebauman,
using first() it works,
thx


Hello,I run the script in centos, i got Out of memory error, why?? what should i do??

ninja: Entering directory `.'
[  0% 8/22706] Ensure Jack server is installed and started
Writing client settings in /home/aosp/.jack-settings
Installing jack server in "/home/aosp/.jack-server"
Launching Jack server java -XX:MaxJavaStackTraceDepth=-1 -Djava.io.tmpdir=/tmp -Dfile.encoding=UTF-8 -XX:+TieredCompilation -cp /home/aosp/.jack-server/launcher.jar com.android.jack.launcher.ServerLauncher
[  0% 15/22706] Building with Jack: out/target/common/obj/JAVA_LIBRARIES/core-all_intermediates/with-local/classes.dex
FAILED: /bin/bash out/target/common/obj/JAVA_LIBRARIES/core-all_intermediates/with-local/classes.dex.rsp
Out of memory error (version 1.2-rc4 'Carnac' (298900 f95d7bdecfceb327f9d201a1348397ed8a843843 by android-jack-team@google.com)).
Java heap space.
Try increasing heap size with java option '-Xmx'.
Warning: This may have produced partial or corrupted output.
[  0% 15/22706] build out/target/common/obj/JAVA_LIBRARIES/sdk_v17_intermediates/classes.jack
ninja: build stopped: subcommand failed.
build/core/ninja.mk:148: recipe for target 'ninja_wrapper' failed
make: *** [ninja_wrapper] Error 1

@acenodie  you can change this config file： /prebuilts/sdk/tools/jack-admin;
-Xmx4096M  ==  your  mem + swap / 4 。
  JACK_SERVER_COMMAND="java -XX:MaxJavaStackTraceDepth=-1 -Djava.io.tmpdir=$TMPDIR $JACK_SERVER_VM_ARGUMENTS -Xmx4096M  -cp $LAUNCHER_JAR $LAUNCHER_NAME"


...react-native-dotenv"' has no exported member 'USER_CLIENT_ID'.

I'm importing an environment variable into a typescript file, all in the app works well, except in my IDE I am getting ...
Module '"../../../../../../Users/chris/Projects/xxxxx/node_modules/@types/react-native-dotenv"' has no exported member 'USER_CLIENT_ID'.
..due to tyepscript when importing.
import { USER_CLIENT_ID } from 'react-native-dotenv';
Could anyone help out please?

I tried some things but for now just made this:
const isProduction = process.env.NODE_ENV === "production";

interface Config {
  API_URL: string;
}

const config: Config = {
  API_URL: isProduction
    ? "https://api.brabble.fm/graphql"
    : "http://localhost:3001/graphql"
};

Object.freeze(config);

export default config;

It's not rocket science but since this package only works with development/production env's it's not that much of a lost.

I've got it working with typescript by creating a file called env.d.ts which defines the keys you use, e.g.:
declare module 'react-native-dotenv' {
  export const USER_CLIENT_ID: string;
  export const ENV: 'dev' | 'prod';
}

The @types/react-native-dotenv is not enough to make it work. As @alexbrazier stated, a declaration file is also needed.


GQLDataFactory - Add GQL retry/resume/save/loading. Queries fail over time.

Motivation

When trying to get sufficient data from subgraph, errors happen.
There are no retries/resuming, so you need to start over. Workflow to generate plots is poor.
File deletion happens easily, improve this (especially important as we have multiple gql queries to make)
Assume there are gql sources with different start dates and that choosing jan-01-2023 as st_ut doesn't break stuff

Continue improving

[x] Right now, do not delete file. Simply append-to-end only.

DoD

[x] Extend from cli-yaml2
[x] Implement schema/validation so the data model is properly handled
[x] Update timestamp logic such that ss/pp (in ms) are resolved against gql and data logic (i.e. timestamp filtering) whose timestamp may be in seconds, or even minutes.
[x] Improve _update() such that file isn't deleted, and for now, will only be appended-to-end.
[x] Implement logic + tests such that all subgraph-predictions data is managed by GQLDataFactory
[x] Point existing stats/plot logic to use GQLDataFactory

Do Next:
Please read the following comments to organize next steps, and tackle the following items:

I think the last point Is "fairly done", without me doing more re-structuring....

[ ]  Point existing stats/plot logic to use GQLDataFactory, remove code from utils, eliminate DRY for subgraph-predictions

Scripts we need to move to data_eng / analytics:
/util/get_predictions_info.py
/util/get_predictoors_info.py
/util/get_traction_info.py
/util/predictoor_stats.py
/util/subgraph_slot.py
/util/subgraph_predictions.py

As an outcome of this we should:

Move all aggregate/summary logic to a /data_eng/data_pipeline/ folder
Move all plot logic to a /analytics/plots/ folder
Move majority of gql_queries, over to data_factory
Update some of these queries such that the results could be cached/lookedup locally... such as with get_all_contract_ids_by_owner where 99% of cases we care about the latest OPF contracts
Rewrite all statistics and analytics to use the data lake and dataframes

I think the last point Is "fairly done", without me doing more re-structuring....

I just created a new epic #447 and moved most of the checklist points into it.
And in the top of the epic, I added another issue I recently created: "Rename/move files & dirs for proper separation among lake, AI models, analytics" #446
What I explicitly did not add: creating sub-directories of lake/ or analytics/. This is premature optimization and adds unnecessary complexity. Shallow with more files in one directory is much easier to work with.

All tasks have been addressed, continuing w/ epic and closing this issue.


Flag to treat first positional parameter as end-of-options

Take in consideration a command as shown below:
class Cmd {
    @Option(names = "--alpha") String alpha;
    @Parameters(index = "0", arity = "1") String foo;
    @Parameters(index = "1..*", arity = "*") List<String> params;
}

It parses correctly a command line like:
<cmd> foo xx --alpha --beta

resulting:
cmd.foo == 'foo'
cmd.params == ['xx', '--alpha', '--beta']

Instead the command line:
<cmd> foo --xx --alpha --beta

throws the exception:
UnmatchedArgumentException: Unmatched argument [--xx]

Is this a feature or a bug?
I'm more inclined to the latter, also because I've noticed that when a command does not specify any double - option it is able to parse the above command line without reporting the unmatched argument exception.
I know that I could bypass that exception using a -- separator, however in any case it looks a bit incoherent behaviour.

Yes, that looks strange. I’ll take a look.

To me, the first result is unexpected (and wrong). A command like this:
<cmd> foo xx --alpha --beta

Should result in
alpha  = --beta
foo    = foo
params = [xx]

It appears that the arity = "*" causes the parser to consume remaining arguments greedily without checking whether the argument is an option. This is incorrect. Only after a -- argument should the parser no longer try to interpret arguments as options.
The second result looks correct. Command line arguments that look like an option should result in UnmatchedParameterException (unless unmatchedArgumentsAllowed is set to true).

It depends. I was looking exactly for this behaviour and the problem is that last example.
Let me give some other details about by use case: I need to implement a command line which behave somehow similar to the one of docker ie.
<tool> [--tool option].. <third-party-command> [command parameters]

Everything after the third-party-command should be delegated to the that command ie. picocli should not try to map option parameters. Basically it should work as when using the double-dash separator but the user should be be required to type the -- characters.

That sounds a bit like the use case described in #215.  To implement that, I propose to add a CommandLine.setStopOnUnmatched(boolean) method. If stopOnUnmatched is set to true, and an unknown option is encountered on the command line, the parser will stop interpreting the arguments and move that argument and all remaining arguments into the unmatchedArguments list. This allows the application to do the parsing in phases.
Would this meet your requirements?
About the first result, I do intend to fix that since the current behaviour does not follow the specification as described in the Mixing Options and Positional Parameters section of the user manual.

Nearly. But the proposed stopOnUnmatched could result on a wrong parsing just for a user typo. Maybe there should be a way to mark a parameter as greedy ie. that will consume everything that comes after.

In alternative it should be possible to disable the Mixing options and position parameters to allow the implementation of a positional parameter command line.

Ok, that may be a good idea.
By the way, I want to understand your use case better. Can you give a few examples of the application you are implementing with support for 3rd party commands? In addition, an example of how a user typo could give wrong parsing result would also be useful.

Think about Docker, for example:
docker run --rm -ti <image-name> [command line passed to the image]

The run command need to capture the options --rm and -ti (and eventually others) and the image-name parameter, after that it would be required a parameter capturing all values whether they are.
In my understanding the stopOnUnmatched stop the parsing as soon there's an unmatched option, let's say --rn instead of --rm. But I need to report an error in that case.

In your example above, I assume that your intention was to capture the 3rd party command name and its arguments in the params list. I want to clarify that with stopOnUnmatched, you shouldn't do this.
Instead, ensure that your command only has options and positional parameters that apply to your command. Avoid any potentially ambiguous options or positional parameters with variable arity (arity = "*").
Then the 3rd party command name and its arguments will be captured in the unmatchedArguments list instead of in the @Parameters field in your command:
class Cmd {
    @Option(names = "--alpha") String alpha;
    @Parameters(index = "0", arity = "1") String foo;

    public static void main(String... args) {
        CommandLine commandLine = new CommandLine(new Cmd())
            .setStopOnUnmatched(true);
        commandLine.parse(args);

        List<String> unmatched = commandLine.getUnmatchedArguments();
        invokeThirdPartyCommand(unmatched);
    }
}

Thanks for the docker example. That clarifies the spelling mistake concern you mentioned.
I see how you are better served with an option that disables the Mixing options and position parameters behaviour. Something like stopAtPositional?
stopAtPositional
When this option is enabled, the parser interprets the first positional parameter as "end of options" so the remaining arguments are all treated as positional parameters. The application must have a multi-value (array, Collection or Map) field to store the positional parameters, or a MaxValuesforFieldExceededException is thrown.
stopAtUnmatched
stopAtPositional is different from stopAtUnmatched. Picocli considers a command line argument "unmatched" when the argument does not match an option name and either of the following is true:

it "resembles" an option (has the same prefix as the other options, or starts with "-" if the command has no options)
there are no remaining slots for positional parameters (the command has no positional parameters or it has a limited number and they have all been given a value)

The stopAtPositional would work !

Fixed in master and 2.x branch. Please verify.

Released in 2.3.0


How to get started to make a dark theme?

Hello, I m new to this project and I would like to use the components in a dark theme.
Does a dark theme already exists?
What would be the best way to get started to have a default dark theme up ?
Thank you by advance

There is this guide https://github.com/material-components/material-web/blob/master/docs/theming.md

Modify the ones from the docs: https://github.com/material-components/material-web/blob/main/docs/theming.md
Or use the theme builder: https://m3.material.io/theme-builder#/custom

Closing since we've got some good replies in here. Feel free to start a discussion for more help!


Fancy UI for new version

Everyone loves good UI and this caught my eye to get me updating.<3
Something that we might want to consider?

Totally - we have some use of colour in some commands, but could definitely use some refreshing of old commands like version, same for help.

Firebase uses this package: update-notifier but i guess we can format it ourselves :)

The CLI recently got the no-update-check flag, I think also a force-update-check to force a check would be useful too.

update-notifier has many dependencies (over 60) - looks like they are working on this in master, so will wait until the next release to see how large the impact is.


Method reference : Enhancement or new feature

Java just started to support lambdas in its language, with this new feature, they brought into play some syntax sugar coating to make code more concise. I thought it look pretty neat and wanted to know if that could be an enhancement of the language.
So, for those who might not know what method reference actually is, I'm going to give a definition for this and then show you an example of how it is used.
Method reference : shortcuts that you can use anywhere you would use a lambda.
Oracle defines 4 types of method references
Reference to a static method     (ContainingClass::staticMethodName)
Reference to an instance method of a particular object (ContainingObject::instanceMethodName)
Reference to an instance method of an arbitrary object of a particular type  ContainingType::methodName
Reference to a constructor (ClassName::new)
As an example,
Here's the first version in Java (Lambda)
list.sort(comparing( (Apple a) -> a.getWeight() ));
Now here's the version using method reference
list.sort(comparing(Apple::getWeight));
It just makes code a bit shorter when using lambda. I'm well aware that Typescript has delegates and that it could be encapsulated (lambdas) within a delegate. I stumble upon that Java 8 feature and thought that a similar feature would be quite nice.
I'd like to start a discussion on this topic.

I'm a little confused - apart from `this

I'm well aware that Typescript has delegates

What do you mean by "delegates" exactly?
I think what you really want is to do is capture this properly when you use a method.
For instance,
class Apple {
    constructor(private weight: number) {
    }

    public getWeight() {
        return this.weight;
    }
}

So currently, if you have:

oh I see. Then I actually have some more reading to do on Typescript
because I completely over seen that.
On Tue, Apr 14, 2015 at 10:30 PM, Daniel Rosenwasser <
<EMAIL_ADDRESS>wrote:

I'm well aware that Typescript has delegates
What do you mean by "delegates" exactly?
I think what you really want is to do is capture this properly when you
use a method.
For instance,
class Apple {
constructor(private weight: number) {
}
public getWeight() {
    return this.weight;
}

}
So currently, if you have:
—
Reply to this email directly or view it on GitHub
https://github.com/Microsoft/TypeScript/issues/2769#issuecomment-93156820
.

Sorry @Kavignon, I hadn't finished that response and but accidentally posted (and deleted) it. I realized what you meant in the middle of responding though, which is decidedly different from what I had understood.

Based on what you understood, would you say that the topic I've raised is good enough to be discussed as a possible feature for either 1,6 or 2.0? @DanielRosenwasser

Before I could comment on something like that, we'd need a formal proposal. One thing I'll mentions is that we tend to be conservative on new syntax to avoid conflicting with future versions of ECMAScript.
An issue worth raising is whether this has a use-case outside of instance methods.

actually almost all examples given are static methods normal way to reference an instance method in java is this::methodName (unless your reference isn't this ). Seems like they would be very useful in chaining with promises
        this.container.registerTransient( publicKey, () => {
            return this.client.fetch( '' )
                .then( ResponseHelper::toJson )
                .then( RepsonseHelper::toHal )
                .then( RepsonseHelper::toLinks )
                .then( links => links["public"]);
        });
``

+1

Kevin would like to stay in touch on LinkedIn.
Kevin Avignon
Senior Software Engineer Student
Montreal, Canada Area
View profile https://www.linkedin.com/e/v2?e=0-ivzc5ec9-qv&a=publicProfile&tracking=eml-guest-invite-pprofile-link-cta&ek=invite_guest&vanityName=kevinavignon
Confirm that you know Kevin https://www.linkedin.com/comm/start/accept-invitation?sharedKey=60VMl3Ti&invitationId=6208290943456604162&trk=eml-guest-invite-cta&trkEmail=eml-invite_guest-null-112-null-null-0~ivzc5ec9~qv
.....................................
Unsubscribe: https://www.linkedin.com/e/v2?e=0-ivzc5ec9-qv&t=lun&midToken=AQFmrFVhGEd36Q&ek=invite_guest&li=119&m=unsub&ts=unsub&loid=AQG3dfHY5yK62QAAAVihH0J8poHUkq05iG1w_XYiFbBp_RHpKPPbJ5x1ZBIkFoqrjuwQapMppn8DEdN95thXFgGPhz5i-H_GTh27g3pX5p4DXqVNWqy65VJv4AU_QKFI5Y9hDkmVaILfWoLEyEmKEA7NC6xNASVuREVywIN5VJZqjRGdaqqUY2N6sGn8Kr94TowHicx2FZnver3Wuo_fAfnHmg&eid=0-ivzc5ec9-qv
Help: https://www.linkedin.com/e/v2?e=0-ivzc5ec9-qv&a=customerServiceUrl&ek=invite_guest&li=118&m=footer&ts=help&articleId=67
You are receiving Invitation emails. LinkedIn will use your email address to make suggestions to our members in features like People You May Know.
This email was sent to<EMAIL_ADDRESS>© 2016 LinkedIn Ireland Unlimited Company, Wilton Plaza, Wilton Place, Dublin 2. LinkedIn is a registered business name of LinkedIn Ireland Unlimited Company. LinkedIn and the LinkedIn logo are registered trademarks of LinkedIn.

A related feature #3508 might eventually supersede this issue.
(I came here searching for method references and dug down for the related ecmascript proposals)

Kevin Avignon would like to connect on LinkedIn. How would you like to respond?
Accept: https://www.linkedin.com/comm/start/accept-invitation?sharedKey=60VMl3Ti&invitationId=6208290943456604162&trk=eml-first_guest_reminder_01-hero-121-accept_text&trkEmail=eml-first_guest_reminder_01-hero-121-accept_text-null-~h4t84k~iw6msi7c~mf
View Kevin Avignon's profile: https://www.linkedin.com/comm/start/accept-invitation?sharedKey=60VMl3Ti&invitationId=6208290943456604162&trk=eml-first_guest_reminder_01-hero-3-profile_text&trkEmail=eml-first_guest_reminder_01-hero-3-profile_text-null-~h4t84k~iw6msi7c~mf
You received an invitation to connect. LinkedIn will use your email address to make suggestions to our members in features like People You May Know. Unsubscribe here: https://www.linkedin.com/e/v2?e=-h4t84k-iw6msi7c-mf&t=lun&midToken=AQFmrFVhGEd36Q&ek=first_guest_reminder_01&li=123&m=unsub&ts=HTML&eid=-h4t84k-iw6msi7c-mf&loid=AQHIqosvqxctiQAAAVi8PRGsi7YiEnxh1tLVWU2y2e9lIPPl65pCAUZlohi6ODiIs35SP9iKRA9EuEeCwNSsRfHjK7g_ep6QG1eEcWCXXC0AaMBYL8u1a4INaPOg7lYgRmvjym8AZwVPe_mQF6J-a6Zy9m0P48JP1JWx9eOO24qKg0XOzsEYSdN61q1pYgk1OOqYuyvKxVT76_ceC4OmZ2uLDM9vIw
This email was sent to<EMAIL_ADDRESS>If you need assistance or have questions, please contact LinkedIn Customer Service: https://www.linkedin.com/e/v2?e=-h4t84k-iw6msi7c-mf&a=customerServiceUrl&ek=first_guest_reminder_01
© 2016 LinkedIn Corporation, 2029 Stierlin Court, Mountain View CA 94043. LinkedIn and the LinkedIn logo are registered trademarks of LinkedIn.

+1


Azure Storage Blob: delete_blobs returns "tuple index out of range" when list_blobs are zero

Package Name:  azure-storage-blob
Package Version: 12.3.2
Operating System: Windows
Python Version:  3.8.1

I retrieve blobs matching a certain pattern and then delete them. Currently it is implemented with a (costly) workaround as you can see in the following issue: https://github.com/Azure/azure-sdk-for-python/issues/12778
When passing blob_paged directly to delete_blobs it works as long as blob_paged is not zero, but if it is zero (i.e. no blob matches the prefix) it returns the exception tuple index out of range.
Can you please adjust the behavior that delete_blobs can handle blob_paged even if it is zero?
Thanks

Hi @dr-dolittle
Thanks for reporting this issue! the pr above should resolve the problem!

I guess we can close this one since it's describing the same thing as #12778 right?

The fix has been release, so we will close this issue! Let me know if you want to reopen it or you have other questions!

The fix has been release, so we will close this issue! Let me know if you want to reopen it or you have other questions!


[BACKLOG-1511] - Remove pdi-monitoring-plugin from the plugins folder of kettle

@CodeOnCoffee , @pedrofvteixeira , @tkafalas  please review


Fix reset and dynamic behaviour (PR #4)

Fix the behaviour in experiment 2:
- add clear grid when use reset
- increase size for neighbours cells when interaction between two particles.

The fix seems to work!
Could you redo your PR from head ? (head includes my PR reverting your first PR).
Pull latest head, commit a revert my revert, add this fix commit to it. Redo a PR (I let you close this one)
(I tried to merge it as is but git do not get it easily it seems)

I'll probably fix the merge of this this week end, It's possible I do not see a way to do it properly with git, If I cant find it, I'll do another PR and maybe another changeset (a little sad if you lose ownership of this optim, that's why I suggest to see my previous com and redo a PR from the right state)

As said above (merge issue) I had to reject this PR but the changes are now on main (merge from a dedicated branch to bypass the strange merge issue) Thanks for the optim!


Register miniblocks only on sudden intervals

I think this change looks good. A added commit to disable this logic for replicated streams, and to restart node in test to make sure it can continue serving such stream.


1.4.0-post-1 version number is useless

The documentation refers to pip install straight.plugin being the canonical method of installing straight.plugin, so I'm going to work on the assumption that it's intended for that method to work.  However, the latest version downloaded by that command (1.4.0) is broken, which you acknowledged and fixed in #9.  Unfortunately you chose a new version number (1.4.0-post-1) that ensured users will never get the working version with the supplied instructions, since it fails to parse as an accepted version number and gets counted as a "pre-release" (according to pip documentation).
Please could you re-release that version under something sensible that won't leave loads of people downloading a broken version forever, especially since this project no longer seems to have any active development, and therefore may never have a 1.4.1 to eliminate this problem?  PEP 440 (and even it's predecessor PEP 386) suggests 1.4.0.post1 as more correct, but even 1.4.0-1, to match your version numbering in the past, would work.

Related to https://github.com/ironfroggy/straight.plugin/issues/17

Looks like this issue and #17 are now fixed with the release of 1.4.1.


Error on ArbTrade-resolved

Hi Mike,
Could you assist and see what I'm doing wrong? When running the arb Trade (lesson 2) using your codes I receive the following error:
Uniswap SAI Buy price is:
208893250405673813902
Kyber SAI Sell price is:
207119422357935884596
(node:5085) UnhandledPromiseRejectionWarning: Error: Transaction has been reverted by the EVM:
{
"blockHash": "0x05154fdce25d7a01598d2f3f79dc615130e8ad910953b8f9f04e9c9124bde81e",
"blockNumber": 10002651,
"contractAddress": null,
"cumulativeGasUsed": 8512917,
"from": "0x3ae5edbdd1342ce6acaa48342792315d52e7b067",
"gasUsed": 29253,
"logsBloom": "0x00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000",
"status": false,
"to": "0xc81d49da3d001c35b8528068dce02860e20f4aa5",
"transactionHash": "0x044bafee2087af8a244a85e177d3907d26a256744edcb3b0454ba9dff9481bbe",
"transactionIndex": 36,
"events": {}
}
Thanks in advance.

@andywmo was the issue resolved?

The above issue is resolved.
Now I'm just paying around with it to follow up with the next step (lesson 3 and 4).
Thanks and great video's..

Actually, do you also have documentation on how to build this Arb tool?

@andywmo this tool was built for identifying windows when the most arb is available: https://www.orfeed.org/defiarbfinder (I hope the lightboxed tutorial video at the top of the page is useful)
Soon, there may be an API that provides access to the above data, as well! Will post once that's up. Currently, working on a UI for executing the arb with more ease-of-use, and then this is the next priority.

Hey @mikedeshazer thanks for the reply.
I see the tool is very helpful indeed. I will look forward to the API connection soon.
Also I experience now that none of my Dai or Sai transactions are going through on Uniswap, even when speeding the gaslimit. Have you been experiencing the same and any ideas how to fix this?
Cheers,

@andywmo https://www.orfeed.org/angle


torchvision requirement

Hello,
I had to install torchvision==0.18.1 because of a compatibility with pytorch 2.3.1 and transformers 4.44, otherwise I would get this error
AttributeError: module 'torch.library' has no attribute 'register_fake'

May need to add to requirements.
Cheers,
Logan

Hi Logan, thanks for reporting.
Current setup (without torchvision) is for sure working, the problem is likely introduced by a new version of torchvision:
https://github.com/huggingface/diffusers/issues/8958

Looks like the issue had to do with my Azure instance holding a transformers 4.46 version even though I did a --force-reinstall --no-cache-dir... All set now, thanks

reopening to keep this visible for other people bumping into the same issue


What will/should the 'new window' icon on a card do?

@bgsandan  @jon571 , any objections to this issue being closed?

Nope.  I think we've decided that it opens a modal with a larger version of what went in the card, with potentially more options/interactions available.

Closing the issue on the understanding that "it opens a modal with a larger version of what went in the card, with potentially more options/interactions available."


node js dota 2 bot connecting broadcast failure in draft

The bot not connecting to the broadcast channel.

It's also not connecting in the unassigned players if the draft team (immortal draft) is enabled.


Unable to define custom colors while using defaults

Hi, I'm trying to change the default background color, while also using some of the original defaults to customize the existing hl-groups. Here's a minimal config to show what I mean:
local default_colors = require("kanagawa.colors")

local colors = {
	sumiInk1 = "#272727", -- new background color
}

local overrides = {
	Search = { bg = default_colors.surimiOrange }, -- customize search highlight color
}

require("kanagawa").setup({
	overrides = overrides,
	colors = colors,
})

vim.cmd("colorscheme kanagawa")

The customizations in overrides are applied, but the new background color in colors is not applied. However, this issue is fixed when I define my own colors as such, without reference to default_colors:
local colors = {
	sumiInk1 = "#272727",
	surimiOrange = "#FFA066",
}

local overrides = {
	Search = { bg = colors.surimiOrange },
}

It seems that reference to default_colors is overriding my customizations in colors. Is there another way to do this that I'm missing?
Thank you for creating this colorscheme! I'm really loving it.

Requiring 'kanagawa.colors' runs the module before the config table, thus default colors are assigned to "semantic" colors like fg, bg, visual, etc. I need to make some modifications in the internal logic to defer color assignment.
For the moment, if you want to declare default_colors, you can modify the bg color directly like this:
local default_colors = require("kanagawa.colors")

default_colors.bg = "#272727" -- new background color

local overrides = {
	Search = { bg = default_colors.surimiOrange }, -- customize search highlight color
}

require("kanagawa").setup({
	overrides = overrides
})

vim.cmd("colorscheme kanagawa")

Fixed in d1d785c0a71f28f8f535c090f6bb22618f69dd4f
please note that it is now best to use require('kanagawa').colors instead of require('kanagawa.colors').

Thanks for the quick fix, it works great!


Bank window

Initial opening of bank shows a grey window.
clicking on bag button to go to regent tab refreshes items to see in the original bank tab
right clicking again on the bag sometimes combines regent and bank slots.

This has been fixed and pushed to .36, which should be live now. Thanks for the report!


[bug]: Can't change model

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Windows
GPU
cuda
VRAM
24
What version did you experience this issue on?
2.3.5 post 2
What happened?
Got error when try to replace model

Model change requested: lofi_V21Inpaint
Current VRAM usage:  2.17G
Offloading lofi_V21 to CPU
Loading lofi_V21Inpaint from C:/vlad/automatic/models/Stable-diffusion/lofi_V21Inpaint.safetensors
Converting legacy checkpoint lofi_V21Inpaint into a diffusers model...
| global_step key not found in model
** model lofi_V21Inpaint could not be loaded: Error(s) in loading state_dict for UNet2DConditionModel:
size mismatch for conv_in.weight: copying a param with shape torch.Size([320, 9, 3, 3]) from checkpoint, the shape in current model is torch.Size([320, 4, 3, 3]).
Traceback (most recent call last):
File "C:\Users\derec\invokeai.venv\lib\site-packages\ldm\generate.py", line 964, in set_model
model_data = cache.get_model(model_name)
File "C:\Users\derec\invokeai.venv\lib\site-packages\ldm\invoke\model_manager.py", line 105, in get_model
requested_model, width, height, hash = self._load_model(model_name)
File "C:\Users\derec\invokeai.venv\lib\site-packages\ldm\invoke\model_manager.py", line 317, in _load_model
model, width, height, model_hash = self._load_ckpt_model(
File "C:\Users\derec\invokeai.venv\lib\site-packages\ldm\invoke\model_manager.py", line 376, in _load_ckpt_model
pipeline = load_pipeline_from_original_stable_diffusion_ckpt(
File "C:\Users\derec\invokeai.venv\lib\site-packages\ldm\invoke\ckpt_to_diffuser.py", line 1230, in load_pipeline_from_original_stable_diffusion_ckpt
unet.load_state_dict(converted_unet_checkpoint)
File "C:\Users\derec\invokeai.venv\lib\site-packages\torch\nn\modules\module.py", line 2041, in load_state_dict
raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
RuntimeError: Error(s) in loading state_dict for UNet2DConditionModel:
size mismatch for conv_in.weight: copying a param with shape torch.Size([320, 9, 3, 3]) from checkpoint, the shape in current model is torch.Size([320, 4, 3, 3]).

** trying to reload previous model

Retrieving model lofi_V21 from system RAM cache
Textual inversion triggers:
Setting Sampler to k_lms (LMSDiscreteScheduler)

Screenshots
No response
Additional context
No response
Contact Details
No response

This is an inpainting model - when you add it via the model manager, make sure you select the 'inpainting' type.


Update package location.

@cortner if there's a better way to do this, feel free to reject this PR and do that. But I've verified Elena's theory that this update fixes https://github.com/ACEsuit/IPFitting.jl/issues/37.

Resolves #2.

@cortner If this really fixes https://github.com/ACEsuit/IPFitting.jl/issues/37 , can it be merged soon?


chore: update repository to stacked framework

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

v3.1.2
GitHub release

Your semantic-release bot :package::rocket:


should make 'space' the second glyph gid1 unless glyphOrder explicitly says otherwise

TIL Glyphs.app always place the 'space' glyph after .notdef, even when the glyphOrder custom parameter says something else, unless you really really tell it to stop with yet another custom parameter which I cannot even find:
https://forum.glyphsapp.com/t/custom-glyphorder-is-not-respected/31555
Ufo2ft also just added something similar: if public.glyphOrder is absent or if it does not list the 'space' glyph while it is present in the font, it will make it so it is placed second after the .notdef:
issue: https://github.com/googlefonts/ufo2ft/issues/880
PR: https://github.com/googlefonts/ufo2ft/pull/881
All this apparently is because some old Windows wants gid1 to be a blank glyph because of some implementation detail while rendering COLRv0 fonts, or else it will overlay the colored glyphs on top of the non-empty gid1 contours.
I suppose we'll need to make sure both our glyphs2fontir and ufo2fontir frontends also behave accordingly, in order to match updated fontmake.

As of https://github.com/googlefonts/glyphsLib/releases/tag/v6.9.3 released 4 days ago, glyphsLib will also place the 'space' glyph as gid1 after the '.notdef' unless yet another "Keep GlyphOrder" custom parameter is set to true.
Although, I don't seem to be even able to set that parameter yet using the latest Glyphs.app beta 3.3 build 3324.
Probably it hasn't even shipped, not sure: https://forum.glyphsapp.com/t/custom-glyphorder-is-not-respected/31555/12
So to recap: ufo2ft checks for public.glyphOrder in the DS/UFO lib, if that is set it will use the glyph order at face value and will not do any reordering besides the required .notdef as gid0 (which fontc already supports); if public.glyphOrder is not specified, ufo2ft will place space after .notdef (this change is only in main and hasn't cut a release yet).
Whereas, glyphsLib >= 6.9.3 when exportig .glyphs to master UFOs will now set the UFO's public.glyphOrder such that space is always placed second after the .notdef, even if there is a glyphOrder custom parameter that says otherwise. In order to ensure glyphsLib actually obeys the stated glyphOrder one also needs to add the other "Keep GlyphOrder" custom parameter.
I expect none of the GF projects that fontc_crater runs against contain this new obscure parameter, so all the ones that didn't bother setting an explicit glyphOrder custom parameter with [.notdef, space] as the first two glyphs, wll suddenly show a diff in the GlyphOrder pseudo-table in the ttx_diff.py. A different glyph order may also produce other spurious diffs in other tables which sort things by the glyph order.

I actually decided to revert those changes in glyphsLib and ufo2ft, I don't think there anything to do for fontc. Maybe when we get the ability to build COLRv0, we could issue a warning in case gid1 is not blank saying this used to be required in some old Windows COLRv0 implementation.

I don't think there is anything to do for fontc

Can we close this then?

Yes


fix: Random security issue

Description

Related Issue

Fixes #20

Types of changes

[X] Bug fix (non-breaking change that solves an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (corrections, enhancements, or additions to documentation)
[ ] Other (please describe):

@tahadostifam can you have a review?


Consider switching to right-handed coordinate system

For historical reasons, Acoular currently uses a left-handed coordinate system for handling spatial information (grid definitions, microphone array geometries, flow properties, ray tracing, ...).
Since most of the world (including other python packages) by convention uses right-handed coordinates, we should consider making these the default in Acoular as well. This would mean a major compatibility break with older scripts in a lot of cases, but the benefits for Acoular in terms of interoperability and intuitive usage also weigh heavy in my opinion.
We could either

completely switch to right-handed coordinates or
have a flag that can be set to RH or LH coords in the config

Benefit of 2. would be that this allows easy support for older scripts using Acoular. Benefit of 1. would be keeping a better maintainability of the code base.

In the environment module, this affects

[ ] UniformFlowEnvironment (?)
[ ] Slot Jet
[ ] RotatingFlow
[ ] GeneralFlowEnviroment (?)

In the tbeamform module this affects

[ ] BeamformerTimeTraj (?)

Interestingly I could not find any other code, but I might be wrong.


[FIX] Frame flip in pre-processing causing left/right confusion

This PR removes the step in the pre-processing phase which was causing a confusion between left and right labels during predictions in gesture_recognition and fitness_tracker.

@guillaumebrg  @antoinemrcr  is this a possible bug in the pre-processing phase? @sunny-panchal  and I found that the display frame only gets flipped once here before making predictions.


Run all nix-builds with --sandbox

Fixes #28

selftest is failing for some reason on this. I'll fix it.

I think we'll need to run --sandbox on only certain builds. I haven't looked deeply into it yet but some things fail to build with this flag.

Interesting; I'm surprised some things fail.  I did add sandbox = true to the installation instructions, but having this as a fallback also seems good.

How does this relate to #55?

I think the idea here is enforcing sandbox whenever possible.
#55 is more about the kind of sandbox that should be used (be it recommendation or enforcing).

@3noch Looks like this needs to be rebased/merged.

Also: should we be using 'relaxed' somehow?

Hard to say at this point - it may very well be that #6 also happens with 'relaxed'.
Also, neither setting seems to completely accept obelisk currently.
true: https://github.com/obsidiansystems/obelisk/pull/55#issuecomment-395419688
relaxed : https://github.com/obsidiansystems/obelisk/issues/52#issuecomment-394168617

Should we close this then? I don't think it's correct.


Change eosiocpp linker ordering

Make dependent libraries go first.. as is tradition. Though I'm a little surprised llvm-link needed this. Resolves problem internally found by @andriantolie

#2564 fix eosiocpp link


Look at Azure deployment guide / depth & coverage of materials

The obvious route to follow is Docs -> Deploy and Setup lakeFS, which leads to https://docs.lakefs.io/deploy/azure.html. Makes sense so far.
This page needs a high-level summary/map of what's about to be done. ATM it launches into lots of detail about auth methods; I'm not sure if this is steps to follow, background info, etc etc.

[ ] Add a map, which is as follows

Choose storage (see existing list)
Choose Auth method

[ ] move details to an appendix

Create a database

[ ] add details for cosmosDB here

Configure lakeFS server, and then run it
Setup admin user and create your first repository

@guy-har @itaiad200 a few questions for you:

Does the above map look correct?
Would one of you or the team be able to write the content for CosmosDB?
Are there any other obvious gaps in our Azure-focussed material?

@rmoff Looks good, can't think of a gap that we currently have - though we might want to update some of the sections in this page.
In the database section - we should put emphasis on deploying using CosmosDB.
What are you missing in content for this section? @itaiad200 is it still beta?

Thanks @N-o-Z - content-wise I think https://github.com/treeverse/lakeFS/commit/e63bf51273bfcb1d8f01f25118b94702e5062193#diff-6220a5d37d042a9c89d26225623f6f95fa39fb3f93d810bc8d8bd235b568de4a which was merged yesterday probably covered it.
Which sections would you suggest we update?

I think we should definitely modify the database section.
We should remove the "Beta" tagging if Cosmos DB is no longer beta, and make it the primary DB option (I would also change these to tabs)
In addition, we might want to add information on how to deploy lakeFS in Azure as a container app (I think this should be the preferred option over a VM)

Got it, thanks. I'll make those changes.

how to deploy lakeFS in Azure as a container app

Do you have the content for this?

I think @guy-har had the context, but if he doesn't have anything written up we can compose something


Update commercial-camel-offerings.adoc

Hi,Thank you for approve and merge.But I am not able to see my change in the webpage. Should I wait for some time?
Regards,Sangeeta (Happy holidays!!!)
On Tuesday, December 28, 2021, 06:28:04 AM PST, Claus Ibsen @.***> wrote:
Merged #6597 into main.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>

I started a website build here:  https://ci-builds.apache.org/job/Camel/job/Camel.website/job/main/ https://ci-builds.apache.org/job/Camel/job/Camel.website/job/main/ and it should be done in an hour or so, at which point your changes will be visible.
David Jencks

On Dec 28, 2021, at 12:33 PM, Sangeeta @.***> wrote:
Hi,Thank you for approve and merge.But I am not able to see my change in the webpage. Should I wait for some time?
Regards,Sangeeta (Happy holidays!!!)
On Tuesday, December 28, 2021, 06:28:04 AM PST, Claus Ibsen @.***> wrote:
Merged #6597 into main.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>
—
Reply to this email directly, view it on GitHub https://github.com/apache/camel/pull/6597#issuecomment-1002271746, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAELDXU7SCDW7Y45CDEAYCTUTINKJANCNFSM5K3T6YYQ.
You are receiving this because you are subscribed to this thread.

Thank you so much for the detailed response.
Regards,Sangeeta    On Tuesday, December 28, 2021, 01:29:06 PM PST, djencks @.***> wrote:
I started a website build here: https://ci-builds.apache.org/job/Camel/job/Camel.website/job/main/ https://ci-builds.apache.org/job/Camel/job/Camel.website/job/main/ and it should be done in an hour or so, at which point your changes will be visible.
David Jencks

On Dec 28, 2021, at 12:33 PM, Sangeeta @.***> wrote:
Hi,Thank you for approve and merge.But I am not able to see my change in the webpage. Should I wait for some time?
Regards,Sangeeta (Happy holidays!!!)
On Tuesday, December 28, 2021, 06:28:04 AM PST, Claus Ibsen @.***> wrote:
Merged #6597 into main.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>
—
Reply to this email directly, view it on GitHub https://github.com/apache/camel/pull/6597#issuecomment-1002271746, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAELDXU7SCDW7Y45CDEAYCTUTINKJANCNFSM5K3T6YYQ.
You are receiving this because you are subscribed to this thread.

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>


musig-spec: Add motivation and design sections

Concept ACK
Maybe we should also say a little bit about "identifiable abort"/blaming using partial verification, i.e., what are the requirements to make that work (authenticated channels?). I think this would then go the actual spec, not to Design.

@real-or-random Agree, I'm planning to add notes about partial verification to the "Signing Flow" section (part of "Description") in a separate PR.


use vendored dependencies during generate

The code generation being done by the k8s.io/code-generator tools has become significantly slower with the change to go modules. Using the vendored dependencies makes it faster, although still slower than what it was before.

/lgtm


j.do should not be used except in bootstrapping

fine comb the codebase. Anything we're using should be ported to proper libs and used from there.

lets review together what we want to accomplish & spec appropriately


new: support for GKE kernels in the ubuntu-generic builder

What type of PR is this?
/kind bug
/kind feature
Any specific area of the project related to this PR?
/area pkg
What this PR does / why we need it:
Add supports for GKE kernels to the already existing ubuntu-generic builder.
Which issue(s) this PR fixes:
Fixes #67
Fixes #60
Special notes for your reviewer:
To help the reviewer I provide a YAML configuration file:
kernelversion: 46
kernelrelease: 4.15.0-1044-gke
target: ubuntu-generic
output:
  module: output/ae104eb20ff0198a5dcb0c91cc36c86e7c3f25c7/falco_ubuntu-generic_4.15.0-1044-gke_46.ko
  probe: output/ae104eb20ff0198a5dcb0c91cc36c86e7c3f25c7/falco_ubuntu-generic_4.15.0-1044-gke_46.o

Does this PR introduce a user-facing change?:
new: support for GKE kernels in the ubuntu-generic builder

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leodido after the PR has been reviewed.
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leodido after the PR has been reviewed.
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

LGTM label has been added.  Git tree hash: 419919c556e8d2023c92649346f0ad4da165e366

LGTM label has been added.  Git tree hash: 419919c556e8d2023c92649346f0ad4da165e366

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

Needs approval from an approver in each of these files:

OWNERS [fntlnz]

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

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

Needs approval from an approver in each of these files:

OWNERS [fntlnz]

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


Only first text token is rendered

With Mattermost Version 7.0.0 and the Memes plugin 1.4.0 only part of the text displays.
Using the example from the README: /meme everywhere "memes." "memes everywhere" results in:

Similarly the example from everywhere.yaml used as /meme geese. geese everywhere also displays just:

@senritsu thanks for opening this issue. However, I am unable to reproduce it using your example. Both strings are accepted and shown in the resulting meme. I also tried this on a v7.0.0 server and still don't see the issue.
Can you please confirm if it's still happening for you?

Unfortunately I cannot, I have recently changed company and no longer have access to the mattermost installation of my previous employer.

I can confirm the issue for Mattermost-Version: 7.1.2. I'm not able to check the plugin version but I can confirm the issue even with the example from /meme

Hi,
I confirm on 7.0.1, we have the issue.
Any ideas?
Thanks

@Gusser93 and @floviolleau Thanks for your help in confirming. 👍
I did find a server where I can repro the issue even though it has the same meme build deployed as other servers I've looked at.
I'm trying now to isolate the repro steps.

After a bit of investigation I am unable to determine why only some servers are affected. Tested matching server version, and plugin version.
The request shows the 2nd string appended

And in the response - I can see the & has been encodded

But these look exactly the same on a server where the issue does not occur.
Also noticed the Community server is affected by this issue.
Leaving in triage for dev input.

Hi,
What do you need as input?

Still reproducible on 7.7.1. I see this error in the log:
error [2023-01-31 22:58:11.967 +01:00] http: URL query contains semicolon, which is no longer a supported separator; parts of the query may be stripped when parsed; see golang.org/issue/25192 caller="log/log.go:195" source=httpserver

I provide the following information with this hint
https://mattermost.gitlab.rlp.net/api/v4/image?url=https://mattermost.gitlab.rlp.net/plugins/memes/templates/everywhere.jpg?text=Test&text=Test does not work (which is the request used by the mattermost web app). This leads to the request https://mattermost.gitlab.rlp.net/plugins/memes/templates/everywhere.jpg?text=Test&text=Test which does not work. Maybe the problem is the imageproxy in mattermost, which URL-encodes the & but does not reverse the process when redirecting here?
https://mattermost.gitlab.rlp.net/api/v4/image?url=https://mattermost.gitlab.rlp.net/plugins/memes/templates/everywhere.jpg?text=Test&text=Test2 and
https://mattermost.gitlab.rlp.net/api/v4/image?url=https://community.mattermost.com/plugins/memes/templates/everywhere.jpg?text=Test&text=Test do work, when used in the mattermost web app. Please note, that the & is not escaped here. This only works if I write the prompt myself without the /meme command.
I hope this helps.

One wild guess: Might it have to do with Mattermost being behind a reverse proxy or not?

Dunno if this adds anything useful, but this is what I see in the nginx reverse proxy log after entering /meme aliens "Test" "Test":
"GET /api/v4/image?url=https%3A%2F%2Fchat.redlink.de%2Fplugins%2Fmemes%2Ftemplates%2Fancient-aliens.jpg%3Ftext%3DTest%26amp%3Btext%3DTest HTTP/1.1"  302
"GET /plugins/memes/templates/ancient-aliens.jpg?text=Test&text=Test HTTP/1.1"  200

I have the very same issue on desktop and web version, but it renders correctly on the iOS version.

I digged into our Mattermost server. The link in the Post in the database is the correct one, without any semicolons. So the issue is that the Mattermost webapp renders this link into an encoded form and introduces an & in the process.
So either someone knows or finds out where that encoding happens and it get's fixed in the Mattermost webapp files, or the plugin catches this and just replaces & with &


Implement Create Op on ably_app resource

This PR implements Create op on the ably_app resource.
During this work, it was suggested that we move push related parameters to their own resource. As such, fcm_key, apns_certificate, apns_private_key, and apns_use_sandbox_endpoint. tf templates have been modified to account for this.
The playground terraform templates have also been updated to create two new apps
tfplugindocs has also been run to generate up-to-date docs.

https://user-images.githubusercontent.com/97962881/183637459-cb1e2948-aee4-46d4-ab3a-c5840ba41294.mov

When trying to create a "disabled" app
resource "ably_app" "app1" {
  name                      = "ably-tf-provider-app-0001"
  status                    = "enabled"
  tls_only                  = false
}

This is what I get:
╷
│ Error: Provider produced inconsistent result after apply
│
│ When applying changes to ably_app.app2, provider "provider[\"github.com/ably/ably\"]" produced an unexpected new value: .status: was
│ cty.StringVal("disabled"), but now cty.StringVal("enabled").
│
│ This is a bug in the provider, which should be reported in the provider's own issue tracker.
╵

After trying the API manually, what I see is that even when user tries to create "disabled" app, it creates it "enabled".
Overall this looks like a Control API bug. In the meantime I suggest we make status readonly, as a mitigation - @graham-russell wdyt?
Re Control API behaviour - @kennethkalmer what is the right place to raise Control API bugs?

Re Control API behaviour - @kennethkalmer what is the right place to raise Control API bugs?

#project-control-apis in Slack

Sent this privately, but now realise that I should post here instead.
@graham-russell do you mind just copy pasting documentation for provider fields from here when possible? https://ably.com/docs/api/control-api
I.e. The application name. instead of The Ably App Name,
or
The application status. Disabled applications will not accept new connections and will return an error to all clients. When creating a new application, ensure that it’s status is set to ‘enabled’ instead of The Ably App Status. When creating an App, ensure that it’s status is set to ‘enabled’. Values after app creating, can be ‘enabled’ or ‘disabled’
It will just make it easier as we can borrow from what docs team has written already (which means it is consistent with our styleguide and such )


Short solution needed: "Rust lang yaml" (rust)

Please help us write most modern and shortest code solution for this issue:
Rust lang yaml (technology: rust)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

How to read YAML file in Rust

How to read YAML from string in Rust

How to write YAML to file in Rust

How to convert YAML to struct in Rust

How to convert YAML to JSON in Rust

Rust YAML parser example

YAML serde example in Rust


Bugfix/list types

This resolves https://github.com/kongchen/swagger-maven-plugin/issues/505 and adds better test coverage for endpoints that return a list.

I did not add any new dependencies or modify the pom in any way, so I'm not sure why VersionEye is failing...


Update firebase

I have deliberately kept it enabled to enable all the features for https://template.karngyan.com as deploy is directly connected to this branch.
Ideally I'd turn it off if the deploy was from a separate branch but it increases my work to push it two branches everytime I add a new feature. (Lazy me)

Let me know if there was any other reason you raised this PR.

Oops I didn't close this request. Actually, this PR was for my forked repo, I fixed it later. Also, I could deploy my portfolio. Will post an update in the Awesome Forks section.


SLIM-691 refactor bundle schema

SonarQube analysis reported 1 issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 ActionMapperResponseService.java#L74: Private method 'postConstruct' is never used.

All is well.


Issue 87 - Make the navbar responsive

Fixes #87:
Hi, I see that your website is still working in progress with responsive. Then, all the pages haven't handle responsive for the Navbar section. I already implemented the menu burger button when users using your web on small screen.
The flow of work on each html file:

Add section with id: section-list under the  tag: That is used to generate the vertical navigation when clicking the hamburger button
Add script <script src="js/navbar_menu.js"></script> at the bottom of each html file: This script will handle the click event to hide and show the vertical navigation

Just let me know if you want to enhance anything.

Looks great :)


Optional params

Introduces optional parameters.
The implementation is finished and appears to be working.
Things left to do:

[x] Add missing comments
[x] Add tests
[x] Check for Col limit
[x] Resolve conflicts
[x] Split fixes and code clean-up of code not related to this PR into a separate PR
[x] Potentially rewrite parts of the code to make it more readable
[x] Rework the PreludeTypes to check if a correct version of Option type is in the environment

Nice to have:

[x] Create a test case for errors

resolves #28

@ppolesiuk The PR is ready for review


Enable deep crawling using Sparker

To enable deep crawling we would need two sparkler deployments configured in a way to enable deep crawling.
This would mean we would change the way we are deploying our system.
Instead of running an embedded solr within the sparkelr container we would separate it out in its own container.
We would then have 5 containers running
1 SOLR, 2 Sparklers, 1 DD Tool, 1 Firefox.

This is not what we did. Instead, we run deep crawls alternately with broad crawls in the same Sparkler container. First round is deep, second round is broad, etc...
@sujen1412, please amend if I'm missing anything relevant.


Reports: Dark Theme

It would be nice to have Dark Theme in reports.

@franciscotco @gpoulet Is this even feasible? I mean, these reports are static files. Can they store some cookie so that a Theme preference can be saved accros pages?

@slandelle it could use localStorage, though that is not a good idea when opening the reports locally (i.e. file:// url) as each browser might behave differently.
Maybe just a simple toggle that would add a query parameter to the page URL, something like index.html?theme=dark  would suffice? The js code could get this value with
const queryParams = new URLSearchParams(window.location.search);
const currentTheme = queryParams.has('theme') ? queryParams.get('theme') : 'default';

And toggle-switch the markup with
document.documentElement.setAttribute('data-theme', currentTheme);

Then the CSS would set variables for the colors
[data-theme="dark"] {
    --primary-color: #fff;
    --bg-color: #000
}

If users are linking to that report published somewhere then they could just append the theme value to the link to default it to the dark theme.

Can be this a parameter in gatling.conf  or/and additionall parameter when running -ro ?

Yes, I suspect that having an option to render light or dark mode at generation time is the only way.

I flagged this feature request as "contribution welcome" :)

Yes, I suspect that having an option to render light or dark mode at generation time is the only way.

Nah, with a bit JavaScript and CSS this is rather easy :) https://css-tricks.com/a-complete-guide-to-dark-mode-on-the-web

@slandelle see my MR - what do you think? :)

@slandelle the last release was quite a long time ago, any chance to release 3.10.0 soon?

@micheljung Gatling 3.10.3 is already out 😁