WAF rules show diff and update on every preview / up

What happened?
I have some unchanging WAF rules that show as changing every time my project is deployed. See the example section below for the details on one rule.
I note that the code defines an ordered array, in which rules go from priority 0 to priority 6. The inputs section of the exported stack agrees. However, the outputs section of the stack has the (correct) rules in order 2, 6, 1, 3, 4, 5, 0.  If that was important I would expect to see more updates or replaces in the diff below; instead, it shows only adds.
Example
Just one of the rules:
    const ipReputationList: aws.types.input.wafv2.WebAclRule = {
      name: "AWS-AWSManagedRulesAmazonIpRepulationList",
      priority: 0,
      statement: {
        managedRuleGroupStatement: {
          name: "AWSManagedRulesAmazonIpReputationList",
          vendorName: "AWS",
        },
      },
      overrideAction: {
        none: {},
      },
      visibilityConfig: {
        cloudwatchMetricsEnabled: true,
        metricName: `AWS-AWSManagedRulesAmazonIpReputationList`,
        sampledRequestsEnabled: true,
      },
    };

Which produces this diff every time:
          ~ rules: [
              ~ [0]: {
                      + name            : "AWS-AWSManagedRulesAmazonIpRepulationList"
                      + priority        : 0
                      ~ statement       : {
                          ~ managedRuleGroupStatement: {
                              + name      : "AWSManagedRulesAmazonIpReputationList"
                              + vendorName: "AWS"
                            }
                        }
                      ~ visibilityConfig: {
                          + cloudwatchMetricsEnabled: true
                          + metricName              : "AWS-AWSManagedRulesAmazonIpReputationList"
                          + sampledRequestsEnabled  : true
                        }
                    }
 ...

Which shows correctly in the AWS WAF console:
{
  "Name": "AWS-AWSManagedRulesAmazonIpRepulationList",
  "Priority": 0,
  "Statement": {
    "ManagedRuleGroupStatement": {
      "VendorName": "AWS",
      "Name": "AWSManagedRulesAmazonIpReputationList"
    }
  },
  "OverrideAction": {
    "None": {}
  },
  "VisibilityConfig": {
    "SampledRequestsEnabled": true,
    "CloudWatchMetricsEnabled": true,
    "MetricName": "AWS-AWSManagedRulesAmazonIpReputationList"
  }
}

I've exported the state, here's that rule in the inputs section:
                        {
                            "__defaults": [],
                            "name": "AWS-AWSManagedRulesAmazonIpRepulationList",
                            "overrideAction": {
                                "__defaults": [],
                                "none": {
                                    "__defaults": []
                                }
                            },
                            "priority": 0,
                            "statement": {
                                "__defaults": [],
                                "managedRuleGroupStatement": {
                                    "__defaults": [],
                                    "name": "AWSManagedRulesAmazonIpReputationList",
                                    "vendorName": "AWS"
                                }
                            },
                            "visibilityConfig": {
                                "__defaults": [],
                                "cloudwatchMetricsEnabled": true,
                                "metricName": "AWS-AWSManagedRulesAmazonIpReputationList",
                                "sampledRequestsEnabled": true
                            }
                        },

And here it is in the outputs section:
                        {
                            "action": null,
                            "name": "AWS-AWSManagedRulesAmazonIpRepulationList",
                            "overrideAction": {
                                "count": null,
                                "none": null
                            },
                            "priority": 0,
                            "ruleLabels": [],
                            "statement": {
                                "andStatement": null,
                                "byteMatchStatement": null,
                                "geoMatchStatement": null,
                                "ipSetReferenceStatement": null,
                                "labelMatchStatement": null,
                                "managedRuleGroupStatement": {
                                    "excludedRules": [],
                                    "name": "AWSManagedRulesAmazonIpReputationList",
                                    "scopeDownStatement": null,
                                    "vendorName": "AWS"
                                },
                                "notStatement": null,
                                "orStatement": null,
                                "rateBasedStatement": null,
                                "regexPatternSetReferenceStatement": null,
                                "ruleGroupReferenceStatement": null,
                                "sizeConstraintStatement": null,
                                "sqliMatchStatement": null,
                                "xssMatchStatement": null
                            },
                            "visibilityConfig": {
                                "cloudwatchMetricsEnabled": true,
                                "metricName": "AWS-AWSManagedRulesAmazonIpReputationList",
                                "sampledRequestsEnabled": true
                            }
                        }
                    ],

Output of pulumi about
pulumi about doesn't show plugin versions (except of my default unused plugins -- why are they even there?), and it does show strings I don't want posted here. Can you update either pulumi about to show only what's needed here, or change the tip on this section of the bug form?
I'm currently using Pulumi 3.86.0,   pulumi-aws 4.38.1.
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

@paul-hicks-mahana This sounds like a duplication of https://github.com/pulumi/pulumi-aws/issues/1775 or https://github.com/pulumi/pulumi-aws/issues/1423. You would need to upgrade to a modern version of the provider (6.x) to get those fixed. Is that an option for you?


Setting rpiCameraROI "exits unexpectedly"

Which version are you using?
0.22.2
Which operating system are you using?

[ ] Linux amd64 standard
[ ] Linux amd64 Docker
[ ] Linux arm64 standard
[ ] Linux arm64 Docker
[ ] Linux arm7 standard
[ ] Linux arm7 Docker
[x] Linux arm6 standard
[ ] Linux arm6 Docker
[ ] Windows amd64 standard
[ ] Windows amd64 Docker (WSL backend)
[ ] macOS amd64 standard
[ ] macOS amd64 Docker
[ ] Other (please describe)

Describe the issue
Using this config:
  cam:
    source: rpiCamera
    rpiCameraWidth: 1920
    rpiCameraHeight: 1080
    rpiCameraFPS: 5
    rpiCameraBitrate: 1600000
    # Region of interest, in format x,y,width,height
    rpiCameraROI: 10,10,1024,768

Apr 16 13:24:49 pi-cam mediamtx[886]: 2023/04/16 13:24:44 INF MediaMTX / rtsp-simple-server v0.22.2
Apr 16 13:24:49 pi-cam mediamtx[886]: 2023/04/16 13:24:44 DEB [path cam] created
Apr 16 13:24:49 pi-cam mediamtx[886]: 2023/04/16 13:24:44 DEB path manager created
Apr 16 13:24:49 pi-cam mediamtx[886]: 2023/04/16 13:24:44 INF [RTSP] listener opened on :8554 (TCP), :8000 (UDP/RTP), :8001 (UDP/RTCP)
Apr 16 13:24:49 pi-cam mediamtx[886]: 2023/04/16 13:24:44 INF [path cam] [rpicamera source] started
Apr 16 13:24:49 pi-cam mediamtx[886]: 2023/04/16 13:24:44 INF [path cam] [rpicamera source] ERR: process exited unexpectedly

$ libcamera-hello  --list-cameras
Available cameras
-----------------
0 : ov5647 [2592x1944] (/base/soc/i2c0mux/i2c@1/ov5647@36)
    Modes: 'SGBRG10_CSI2P' : 640x480 [30.00 fps - (0, 0)/0x0 crop]
                             1296x972 [30.00 fps - (0, 0)/0x0 crop]
                             1920x1080 [30.00 fps - (0, 0)/0x0 crop]
                             2592x1944 [30.00 fps - (0, 0)/0x0 crop]

Describe how to replicate the issue
N/A
Did you attach the server logs?
yes
Did you attach a network dump?
N/A

Oops, was using wrong format. It's working now.
libcamera-vid --help|grep roi
--roi arg (=0,0,0,0)                  Set region of interest (digital zoom) e.g. 0.25,0.25,0.5,0.5


Open in VesselView button

As long as no file has been selected, the button should be green and show: "Open this module with a file" (with the appropriate icon).
If a recent file is selected, it should just show "Open this module".

In  "Open this module with a file"  mode:

The application should open a dialog so the user can open some data.
The type of data to load should be determined by the module's prefered data type

In  "Open this module"  mode:

Happens whenever the application has loaded at least one file (succesfully)


usePassage crashes on bad reference

When the reference is 3JN 1:1 the code works, but whenever we try changing that value the code crashes.
Error: "Uncaught TypeError: Cannot read properties of undefined (reading 'length')"
If we initialize the reference to "3JN 1:" we get this error: "Uncaught TypeError: Cannot destructure property 'groups' of 'r.match(...)' as it is null."
You can see this yourself at: https://github.com/Proskomma/diegesis-reboot/tree/elias-pinero-browsePassage

Can you try on the latest release(s)? This should be resolved now.
The last 1.x should address it and the 2.x release as well, however, you may need to update your usage of useCatalog when updating to v2.x


Added test case for CCITTFaxDecoder

Copy of #642

Thanks for this great project!
The tostring method has been deprecated since Python 3.2 and was
removed altogether in 3.9. In Python 3.2 the method was renamed
to "tobytes"
Without this fix, using the ccit module will lead to an AttributeError and a crash on Python 3.9+.
Will close #641
How Has This Been Tested?
Ran locally with python 3.9
Checklist
I have added tests that prove my fix is effective or that my feature
works.
I have added docstrings to newly created methods and classes not relevant
I have optimized the code at least one time after creating the initial
version
I have updated the README.md or I am verified that this
is not necessary
I have updated the readthedocs documentation or I
verified that this is not necessary
I have added a consice human-readable description of the change to
CHANGELOG.md

Actual error was already fixed by #661. So only adding test case.


Version Request

hi there, i really hope that this apps could be installed in our apple devices. its should be cool hey. more user would install this apps.
thank you for the attention.

Sorry, no iOS version planned.

https://github.com/yukuku/androidbible/issues/66


asynchronous and partial completion of a payment

Summary
An unattended ECR has the problem where it has to decide if it finishes the transaction with the payment terminal first, and then issues the goods to the customer, or vice versa. For vending machines with mechanical parts, or other cash registers where the failure is an option the workflow according to the ZVT standard is, that the goods are issued first and only then the transaction is finalized in the PT. If the issuing of goods fails, an auto reversal is triggered. This is essential in unattended use cases (compared to attended ones) because the card does not need to be presented again to the PT.
For us such failures are an option and the current implementation does not support a callback for issuing goods. This PR is a proposal to implement such a callback.
Changes
This branch builds on #21 (required) and #22 (optional) so the diff is larger then necessary. For the actual changes you can only view the last commit (5bbd3b8098b4e5f2f747017137e1bd7cc3d87ff4).
In order for the ProcessData function of the ZvtCommunication class to decide if a successful payment was made, the processing functions were refactored to return a ProcessData object which includes the State aswell as the parsed message.
As mentioned in the https://github.com/Portalum/Portalum.Zvt/issues/11#issuecomment-1161685860 otherwise it is hard to determine if and when a successful payment has been made.
The first prototype did not differentiate between response types, so AskForCompletion was always called, regardless if it was a payment or other command. With the changes in the processing pipeline the ProcessData can now check if it is a successful status information.
I have Introduced an additional callback which is currently only used for payments StartAsyncCompletion. This is called once per payment, as otherwise users need to be aware that StatusInformation's with ErrorCode == 0 will be called multiple times. This avoids that each user needs to implement the "start issuing goods only once" gate in their code. A not yet added documentation should clearly outline that this function should be used when using async completion.
Open questions

Timeouts: currently the command timeout is extended by the repeatet status information messages. This behaviour is different in async mode compared to immediate completion. The only timeout is the one imposed by the PT's max status information

Tests
Real World Tests: Worldline VALINA from Card Complete

contrary to the standard, the terminal only queries 3 times with a status information and then ends the transaction. this is why i have explicitly added the max-status information parameter in the registration call.

Testcases: i have included 3 tests for asynchronous completion, and added two for non-asynchronous completion which show that the async callbacks will not be called in this case.
Backwards Compatibility
From the protocol side, nothing should change as long as the GetAsyncCompletionInfo is not registered. The PamynetAsync method also avoids setting the additional parameter if asynchronous completion is not used.
From the code side the main internal change is that now the processed state is not passed as a State enum or a boolean, but always as ProcessData object which includes the State and the parsed message.

Do we still have discussion needs in the Open Questions section? Or should the implementation fit so far for the merge?

About the open question: i have read the diff wrong, your prototype is using the same timeout behaviour. This voids the question, there is no difference as previously mentioned.
The PR is fit to merge from our side, we have not seen any issues or made further changes within the last week using this code.

i don't know what the issue here is, but i am having trouble getting the ci to run through lately. Do you have a different ci setup, i could not replicate the issues in my repository.

I need to look at this in detail on the quick I can not determine the reason. But it will take a few days until I find time for it.

I think it should fit now.
Due to the changeover from private to protected, we now have many places where the summary is missing. Is protected really needed everywhere?

Thanks for your work on the testcases!
I have reduced the protected / virtual members to what i think is reasonable, and what we currently use to catch some special cases. Most other members can be intercepted or overwritten in the constructors. For the remaining ones i have added documentation.

@rettier I just tried to change the amount to a lower amount with our test terminal but had no success. Have you tested this variant with you?

@tinohager to be honest i have never, and i can observe the same thing on our Worldline VALINA (Card Complete), i am receiving 6C (=Abort) as a respones to the change amount message.
https://github.com/Portalum/Portalum.Zvt/issues/11#issuecomment-1079999689 mentioned it working, looking at the zvt standard i cannot see what we are doing wrong.

our message seems fine: Portalum.Zvt.TcpNetworkDeviceCommunication[0] SendAsync - 84-9D-07-04-00-00-00-00-00-10
CardComplete is also not mentioning any differences to the standard:

@rettier I have just consulted with CardComplete. I was told that the function of changing the amount is not supported.

@rettier So from my point of view we are actually so far through would like to incorporate the pull request into the main code next week. 👍🏻
Before that I would just like to discuss the naming of the events again. Whether the names make sense.
I don't know if Async makes sense in the event name. And that we have no verbs here. naming convention and style
Current

LineReceived
ReceiptReceived
StatusInformationReceived
IntermediateStatusInformationReceived

New

StartAsyncCompletion
GetAsyncCompletionInfo

@tinohager great, thanks for working with us on that PR. I'm glad your project exists, and even more so that we were able to contribute something to it.
I am fine with a overhaul of the event names and i do like your proposal.
I was considering "issue goods" as the base for event names. But looking at the current event names the events are named what has happened, not what the effect is.
i.e. ReceiptReceived and not PrintReceipt
thus naming them IssueGoods / GetIssueGoodsStatus would be equally bad names.


Read task environment storage limits from manifest

AI R&D tasks often need quite a bit of storage space (e.g. to checkpoint models while training), but they also shouldn't be unlimited. We'll use TASK_ENVIRONMENT_STORAGE_GB to increase it globally for now, but this seems a good use of manifest.yaml

Done with https://github.com/METR/vivaria/pull/399


java.lang.IllegalStateException: ServiceLocatorImpl(GuiceServiceLocator-9,8,1066656936) has been shut down

I'm getting these since upgrading to jersey 2.21...in the constructor of AbstractRuntimeDelegate jersey now shuts down the ServiceLocator that it is given, so when Guice tries to access it it gets this error.

Any news on this?

I'm running into the same issue on dropwizard 0.8.2

We do not have this issue with dropwizard 0.8.2 and 0.8.3. Maybe you need use the BootstrapUtils in your tests as advised in dropwizard-guice?

I'm confused as to why I'm running into the issue actually, since dropwizard 0.8.2 uses jersey 2.19, and the presumed culprit (https://github.com/jersey/jersey/commit/ad54f0033c73144f1ad23706583714a3fe84bee7) was introduced in jersey 2.20... I'll try modifying the test as you suggest and will post the result here

No dice, just ignoring that test for now :(

I got the same issue when upgrading from jersey-common 2.17 to 2.22.

I created a fork with a workaround for the issue:
https://github.com/rvs-fluid-it/jersey2-guice
But I'm not sure if the workaround does not have any side effects (resources leaking ...)
Hence I currently do not have enough confidence in the solution to prepare a pull request.
But if you want to take it for a test drive, please feel free :-).

I used a hacky workaround which was to decorate the ServiceLocator before passing it to jersey2-guice and override the shutdown method to do nothing

Hello,
I've just pushed a new build to Maven Central. There are some breaking API changes but this problem should be fixed. Please take a look at the updated documentation which should be straight forward if you're running your code from a main() method (e.g. Dropwizard).

If you want to see this fixed in Jersey proper, please put pressure on https://java.net/jira/browse/JERSEY-2627.

If you want to see this fixed in Jersey proper, please put pressure on https://java.net/jira/browse/JERSEY-2627.


Customize Tag Slugs

I am using Middleman-Blog and I am quite happy with it. It has only one minor issue that tag slug does not work well with my language (Vietnamese) and this is understandable. I tried to search on Google and this repo to find someone who has same issue but unfortunately I can't find it :(.
I would like to ask, is there any simple way to customize tag slugs? I just want to replace some characters with another.
Any help or a simple example would be greatly appreciated.

I don't think we have any hooks for this right now. But you could override like so:
module CustomLink
  def link(original_tag)
    new_tag = original_tag.dup
    # Do something to new_tag to change characters for urls/links
    super(new_tag)
  end
end

::Middleman::Blog::TagPages.send :include CustomLink

@tdreyno Thank you for your help. Could you please let me know where should I put this file?

config.rb
Sorry, that it involves writing a little ruby

@tdreyno Thank you for your help, I am okay with a little ruby writing ruby if this sample work, however I can't make it :(.
I put your code in config.rb and get this error. config.rb:48:in block in initialize': uninitialized constant Middleman::Blog::TagPages (NameError)`. Could you please help ?
Again, thank you and I am appreciated it

Maybe add this up top:
require 'middleman-blog/tag_pages'

@tdreyno Thank you for your kind help. I am really appreciated it
I used your code and for example I just want to replace all v character with -. Below is my code
require 'middleman-blog/tag_pages'

module CustomLink
  def link(original_tag)
    new_tag = original_tag.dup
    newtag.gsub("v", "-")
    super(new_tag)
  end
end

::Middleman::Blog::TagPages.send :include, CustomLink

It doesn't work and I don't know why. Could you please help?

well, newtag should be new_tag
require 'middleman-blog/tag_pages'

module CustomLink
  def link(original_tag)
    new_tag = original_tag.dup
    new_tag.gsub!("v", "-")
    super(new_tag)
  end
end

::Middleman::Blog::TagPages.send :include, CustomLink

@tdreyno Sorry for the typo. I used exact your code and it still doesn't work. Does it work for you ?

Like I said, you'll have to write some ruby for now. I'm afraid I don't have the time to do it myself.

@tdreyno Thank you for your kind help.


enum length

In order to get the length of an enum, source generator can introduce a public constant variable what is faster.

Method
Mean
Error
StdDev
Ratio
Gen 0
Allocated

EnumLength
41.9690 ns
0.2502 ns
0.2218 ns
1.000
0.0076
48 B

EnumLengthExtensions
9.0046 ns
0.0649 ns
0.0607 ns
0.214
0.0076
48 B

EnumLengthProperty
0.0214 ns
0.0046 ns
0.0043 ns
0.001
-
-

@andrewlock this is very useful, are you considering to merge this?

Yes, sounds perfect. :)

I thought maybe it is useful to add a documentation comment which clarifies what this property reflects.
Because it is a non distinct count of the names (if I'm correct).
A user could have an enum defined with different names, but same values.
using System;
using System.Linq;

public class Program
{
    public enum Foo
    {
        First = 0,
        Second = 1,
        Third = 1,
    }

    public static void Main()
    {
        var nameCount = Enum.GetNames(typeof(Foo)).Length;
        var valueCount = Enum.GetValues(typeof(Foo)).Length;
        var distinctNameCount = Enum.GetNames(typeof(Foo)).Distinct().Count();
        var distinctValueCount = ((Foo[])Enum.GetValues(typeof(Foo))).Distinct().Count();

        Console.WriteLine($"{nameof(nameCount)}: {nameCount}");
        Console.WriteLine($"{nameof(valueCount)}: {valueCount}");
        Console.WriteLine($"{nameof(distinctNameCount)}: {distinctNameCount}");
        Console.WriteLine($"{nameof(distinctValueCount)}: {distinctValueCount}");
    }
}

nameCount: 3
valueCount: 3
distinctNameCount: 3
distinctValueCount: 2

That's a good point. I guess one question I have is: when would you use this 🤔 i.e. when would you want to get the number of names/values but not get the actual name/values themselves?

I usually use the number of names, when I want to create a new List<T> with an initial size of the enum.
I can't think of anything right now, when would I use the distinct value count.

Thanks all, looks good to me (I added the xmldocs for clarity)


No route matches when I go an ember router

Hello,
I'm new with ember and I'm trying ember-cli-rails. I configured the initializer like this :
EmberCLI.configure do |c|
  c.app :frontend, path: '../frontend'
  c.build_timeout = 30
end

And I copied/pasted codes on the README to my application/index.html.erb and application_controller.rb files. When I go at http://localhost:3000 it works. I added a route and a link lie this :

Welcome to Ember.js

{{#link-to 'users'}}Test{{/link-to}}
{{outlet}}

When I click on the link, the current paths is http://localhost:3000/users and it works. The problem is, when I reload, I have a No route matches [GET] "/users" and all routes I see on the error page comes from Rails.
Should the ember's routes be mounted? Is there a way to fix that issue? What's the best practice?
Thanks!

I had an answer here : http://stackoverflow.com/questions/32168466/ember-cli-rails-no-route-match-for-embers-paths. I had to add :
get '/:all', to: "application#index"

to my routes.
It should be in the Readme!

It really should be in the README!

Thanks to add it to read me. I'm having the same problem but on a little bit more complex app.
http://stackoverflow.com/questions/32712198/ember-cli-rails-routing-close-to-a-classic-rails-app

It should be get '/*all', to: "application#index" to include all pathes.


prevent grouping created action message

Details
prevent grouping created action message
https://github.com/Expensify/App/pull/18672/files#r1195263666
Fixed Issues
$ https://github.com/Expensify/App/issues/19010
PROPOSAL: https://expensify.slack.com/archives/C01GTK53T8Q/p1684242722161919?thread_ts=1684242655.160749&cid=C01GTK53T8Q
Tests

Login any account
Create new workspace
Send new message in announce room
Verify that current user avatar and display name show in action item

[x] Verify that no errors appear in the JS console

Offline tests
Same as Tests
QA Steps
Same as Tests

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/97473779/2cf3897f-61fd-401f-98f6-96b951c32f23

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

🚀 Deployed to production by https://github.com/yuwenmemon in version: 1.3.14-14 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Cherry-picked to staging by https://github.com/AndrewGable in version: 1.3.28-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.3.28-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Fix 7360. Delete pod dir only after volume TearDown.

The volume dir is deleted before mounted volumes are torn down. As
a result all the content of the mounted volume is deleted.
This patch reorders the calls so that volumes and torn down before
the pod dir is deleted, and it moves both calls to killPod().
This fixes #7360

Closing this since #7503 was merged


fix cannot detect valid service IP

fix rancher/longhorn#56 cannot detect valid service IP

Can you explain why this change is needed? And it does work in the current form.

I will close this PR for now as we have decided not to change the releasename and using the longhorn as default for each release, so dynamic input is not necessary for now

with all longhorn-system pods, logs show me this error: Cannot detect valid service IP, retrying
What can I do?


Docs: Added root folders description

Closes #543

LGTM. The folder structure of the different packages like the graphql proxy will be then specified each one on a separate page no @JoseDuSV

LGTM. The folder structure of the different packages like the graphql proxy will be then specified each one on a separate page no @JoseDuSV

Right. Root folder is not that interesting and we already have separate sections for apps https://kiwicom.github.io/margarita/docs/apps/ and packages https://kiwicom.github.io/margarita/docs/packages/


[SPARK-47360][SQL] Collation support: Overlay, FormatString, Length, BitLength, OctetLength, SoundEx, Luhncheck

What changes were proposed in this pull request?

Add collation support for batch of string expressions mentioned in PR title.
Why are the changes needed?

Add collations support in string functions.
Does this PR introduce any user-facing change?

Yes, it changes behavior of string functions when string parameters have collation.
How was this patch tested?

Added checks to CollationStringExpressionsSuite.
Was this patch authored or co-authored using generative AI tooling?

No.

thanks, merging to master!


Single \Predis\Connection\Parameters instance should be wrapped in array when using sentinel replication.

Config below results in  ERR unknown command 'SET' when trying to set session data. Parsed DSN is passed as an instance of \Predis\Connection\Parameters to \Predis\Client which results in sending SET/GET to Redis Sentinel instead of real Redis node.
snc_redis:
    clients:
        session:
            type: predis
            alias: session
            dsn:
                - redis://redis-sentinel:26379
            options:
                replication: sentinel
                service: master

This issue is related to: https://github.com/nrk/predis/issues/476

Hello @dzolnierz
I just ran into the same issue. Did you find a solution/workaround ?

Our temporary workaround to this issue is to just duplicate the declaration, causing it to be correctly converted to an array.
snc_redis:
    clients:
        default:
            type: predis
            alias: default
            dsn:
                # These are duplicated as a workaround to: https://github.com/snc/SncRedisBundle/issues/381
                - redis://redis-sentinel:26379
                - redis://redis-sentinel:26379

@petitphp
Yeah, we do exactly the same thing as @adammwood. Duplicate the DSN.

Would any of you have a go at issuing a PR to fix this?


st-util returns "unknown chip id! 0" / STM32F429

Hello.
Board is STM32F4 Discovery kit for STM32F429
$ st-util
2015-07-26T16:28:00 INFO src/stlink-common.c: Loading device parameters....
2015-07-26T16:28:00 WARN src/stlink-common.c: unknown chip id! 0
$ st-info --chipid
0x0419
stlink build from 1.1.0 source
st-util -v99
2015-07-26T16:48:08 DEBUG src/stlink-common.c: stlink current mode: debug (jtag or swd)
2015-07-26T16:48:08 DEBUG src/stlink-common.c: stlink current mode: debug (jtag or swd)
2015-07-26T16:48:08 DEBUG src/stlink-common.c: *** looking up stlink version
2015-07-26T16:48:08 DEBUG src/stlink-common.c: st vid         = 0x0483 (expect 0x0483)
2015-07-26T16:48:08 DEBUG src/stlink-common.c: stlink pid     = 0x3748
2015-07-26T16:48:08 DEBUG src/stlink-common.c: stlink version = 0x2
2015-07-26T16:48:08 DEBUG src/stlink-common.c: jtag version   = 0x11
2015-07-26T16:48:08 DEBUG src/stlink-common.c: swim version   = 0x0
2015-07-26T16:48:08 DEBUG src/stlink-common.c:     notice: the firmware doesn't support a swim interface
2015-07-26T16:48:08 INFO src/stlink-common.c: Loading device parameters....
2015-07-26T16:48:08 DEBUG src/stlink-common.c: *** stlink_core_id ***
2015-07-26T16:48:08 DEBUG src/stlink-common.c: core_id = 0x2ba01477
2015-07-26T16:48:08 DEBUG src/stlink-common.c: *** stlink_read_debug32 0 is 0xe0042000
2015-07-26T16:48:08 DEBUG src/stlink-common.c: *** stlink_read_debug32 0 is 0x40015800
2015-07-26T16:48:08 WARN src/stlink-common.c: unknown chip id! 0
2015-07-26T16:48:08 DEBUG src/stlink-common.c: *** stlink_close ***

I'm facing the same problem in Mac OS X.

The problem is Nutx specific. Have a look here: http://www.nuttx.org/doku.php?id=wiki:howtos:jtag-debugging

Could some of you verify it still exists with v1.2.0 or master ?

I still get this after checking out latest master and copy-pastaing all text from the readme:
st-util
2016-05-17T14:47:32 INFO /Users/kenz/Documents/sandbox/stlink/src/common.c: Loading device parameters....
2016-05-17T14:47:32 WARN /Users/kenz/Documents/sandbox/stlink/src/common.c: unknown chip id! 0x3001641b

P.S. There doesn't seem to be a --version flag for st-flash.

@kubark42 yes I know about --version as most people would expect. This is currently not implemented because of not very nice implemented commandline argument parsing and will be resolved in milestone v2.0.0.
I'm trying to dig up the chipid 0x3001641b from the datasheets but i'm unable to find it. Could you help me out?

@xor-gate Crap, I was trying to update this before anyone got a chance to resolve. I hadn't realized I was using an STM32F4Discovery board that still had its jumpers in place. It's no surprise that this was a GIGO situation.
This result should make you happier:
STM32F427
 st-util
2016-05-17T14:57:24 INFO /Users/kenz/Documents/sandbox/stlink/src/common.c: Loading device parameters....
2016-05-17T14:57:24 INFO /Users/kenz/Documents/sandbox/stlink/src/common.c: Device connected is: F42x and F43x device, id 0x20016419
2016-05-17T14:57:24 INFO /Users/kenz/Documents/sandbox/stlink/src/common.c: SRAM size: 0x40000 bytes (256 KiB), Flash: 0x200000 bytes (2048 KiB) in pages of 16384 bytes
2016-05-17T14:57:24 INFO /Users/kenz/Documents/sandbox/stlink/src/gdbserver/gdb-server.c: Chip ID is 00000419, Core ID is  2ba01477.
2016-05-17T14:57:24 INFO /Users/kenz/Documents/sandbox/stlink/src/gdbserver/gdb-server.c: Target voltage is 2881 mV.
2016-05-17T14:57:24 INFO /Users/kenz/Documents/sandbox/stlink/src/gdbserver/gdb-server.c: Listening at *:4242...

STM32F429
$ st-util
2016-05-17T14:58:05 INFO /Users/kenz/Documents/sandbox/stlink/src/usb.c: -- exit_dfu_mode
2016-05-17T14:58:05 INFO /Users/kenz/Documents/sandbox/stlink/src/common.c: Loading device parameters....
2016-05-17T14:58:05 INFO /Users/kenz/Documents/sandbox/stlink/src/common.c: Device connected is: F42x and F43x device, id 0x10036419
2016-05-17T14:58:05 INFO /Users/kenz/Documents/sandbox/stlink/src/common.c: SRAM size: 0x40000 bytes (256 KiB), Flash: 0x200000 bytes (2048 KiB) in pages of 16384 bytes
2016-05-17T14:58:05 INFO /Users/kenz/Documents/sandbox/stlink/src/gdbserver/gdb-server.c: Chip ID is 00000419, Core ID is  2ba01477.
2016-05-17T14:58:05 INFO /Users/kenz/Documents/sandbox/stlink/src/gdbserver/gdb-server.c: Target voltage is 2880 mV.
2016-05-17T14:58:05 INFO /Users/kenz/Documents/sandbox/stlink/src/gdbserver/gdb-server.c: Listening at *:4242...

So I can confirm that I'm not seeing the problem on an STM32F429.

Yeah, those things happen. Still we are error-creating-humans 👍. Thanks for testing. I have cross-referenced about the commandline arguments and edited your post. @kubark42 basicly this means we can close this issue then?

@xor-gate Thanks for the understanding, and glad after sorting it out I could help test. It does indeed look like there is full st-link support for the STM32F42x chips.

Great!


NiFi-2829: Add Date and Time Format Support for PutSQL

Fix unit test for Date and Time type time zone problem while testing PutSQL processor
@paulgibeault made the original PR #1073, #1468
@patricker add support of DATE and TIME in Epoch format for PutSQL processor.
I’ve fix the unit test in different time zone problem.
The detail is list below
The originally problem with unit test happens because of different time zone.
Internally without specifying time zone, java.sql.Date and java.sql.Time will use local time zone to parse the time.
As a result, different time zone will have different format result for a given constant time value.
This is mentioned by @mattyb149 in https://github.com/apache/nifi/pull/1524
Currently solve the problem by giving time zone before insert and parse result with same time zone. (GMT)
Currently build and test successfully with NiFi newest version on GitHub which is 1.4.0-SNAPSHOT.
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Well, lesson learned.
Last build fail with Travis-CI is trailing whitespace problem.
Just fix that and retry the PR.

Thanks for the reply and detail review.
Just update the commit title, document part and remove unnecessary if condition.
Seems AppVevor fail at same part.
Travis CI only fail at build 2 but pass build4,5.
Not sure why.
I'll keep track of it.

@yjhyjhyjh0 Thanks for the updates!
For AppVeyor test, it has been failing and Travis test failure is caused by nifi-persistent-provenance-repository, which has also been failing occasionally, so please don't worry about those.
I'll continue reviewing..

Thanks for the review and feedback.
I’ve add your enhancement of handing time precision at milliseconds in PutSQL.
Also I'm very willing to add a unit test for it.
However after some experiment, I figure out Derby DB has same behavior as MySQL in Time type just like what you mentioned.
I’ve tested with derby, MySQL and PostgreSQL.
Only PostgreSQL will handle millisecond in Time type properly.
About the experiment, it store Time type well with value "00:01:01.111" in PutSQL.
But once client fetch via jdbc of Derby.
It truncate the milliseconds.
(actually has different long value in client side) (java jdbc client and IJ tool of derby)
So even with correct format to parse, the result in client side is "00:01:01.000”.
Thus, I only update document to remind this situation instead of providing a unit test for this format.
Please let me know if there is any problem.
Thanks for your patient.

I've made a slight change to the documentation to state MySQL and Derby are some database engines that do not support milliseconds, but those may not be the only two. I am fine with not having unit test for it as we use Derby, clarify it via docs should be enough.
Thanks @yjhyjhyjh0 , this has been merged to master!

As this PR is based on #1524 which was submitted by @patricker , I cherry-picked #1524 first, then cherry-picked this PR to give @patricker a credit. Thanks again!


Votes aren't updated dynamically on TFE when they are voted or votes are changed on Console and vice versa

Votes aren't updated dynamically on TFE when they are voted or votes are changed on Console and vice versa

To picked up later.


Pointing chart to gateway 3.3.1

Pointing chart to gateway 3.3.1.

i don't think charts/gateway/conduktor-gateway-3.3.0.tgz should be pushed?

i don't think charts/gateway/conduktor-gateway-3.3.0.tgz should be pushed?

Yep mistake from where I was doing some local CS testing! didn't spot it, thank you!

i don't think charts/gateway/conduktor-gateway-3.3.0.tgz should be pushed?

Yep mistake from where I was doing some local CS testing! didn't spot it, thank you!

no worries. don't forget you can use .gitignore ;-)


Does ValuesFrom support secret in gzip format

Sometimes because the data of the secret is too large, it will use gzip for compression. Does the ValuesFrom of helm-controller support obtaining the data of the gzip type? For example, you can judge this Annotations: encoding: gzip . If it is yes, then you can The data is decompressed, and then the value is obtained according to its key.
My case:
I create my infrastructure through terrafrom, and set its backend to k8s, then the state of its backend is stored through secret, because the data of the state may be too large, so it is compressed by a gzip, and I create it through terraform The load-balance needs to get the load-balance-id to fill in helm's service.yaml, so I assign values through ValuesFrom, so I need to get this value from a gzip secret.
If this feature is not available, I'd be happy to submit a PR
this is secret:

This is how I get his data through kubctl:

This is a normal secret:

This is not something we support.
I also do not think that we are open to supporting it as proposed, as the encoding: gzip annotation is a Terraform specific annotation and not conform Kubernetes standards, for which a custom Secret Type would have been more fitting.
This is for example done by Helm, which also stores releases in a compressed format:
Name:         sh.helm.release.v1.podinfo-1648201816.v1
Namespace:    source-system
Labels:       modifiedAt=1648201817
              name=podinfo-1648201816
              owner=helm
              status=deployed
              version=1
Annotations:  <none>

Type:  helm.sh/release.v1

Data
====
release:  28176 bytes

@hiddeco
thanks a lot for your answer
If this is not a good choice, can I ask a question? I saw that your weave blog mentioned that the infrastructure in gitops is implemented by terraform. I use flux's CRD(such as helm controller) in the application layer. I am very confused about how we connect the facilities created by terraform with the components of flux, such as I gave an example of the transfer of clb-id, how to use the components of the infrastructure I created in terraform in helm-controller, because it seems that the components in flux are all at the application level.
I am very confused about this question, thank you very much for your answer

@huzerun0306 a maintainer of the Terraform controller here.
Here's my assumption of your setup. Please correct me if I'm wrong.

You have your infra provisioned by Terraform.
You're already using Kubernetes backend for it, so that's secret is your TFSTATE managed by Terrform

What would you like is to pick out something from TFSTATE file to feed into a Helm Release. Is that right?
If so, it seems to be a good Terraform/Helm use case.
I'll check if we could have a simple example / tutorial to connect them for you. Please ping me again next week.
Is that OK for you?

@chanwit  Yes, my case is what you said, we created an infrastructure through terraform-controller, and helm-controller references the information of this infrastructure from its state file. We are looking for a suitable gitops solution to do infrastructure and application level correlation. thank you very much

@chanwit

@chanwit

@huzerun0306 you may want to consider Kyverno and create policy for replicating the secret create by tf-controller in other namespaces, see https://kyverno.io/policies/other/sync_secrets/
If tf-controller breaks the namespace boundary that it can't be used on multi-tenant clusters, and I think @chanwit doesn't want that.

Like @stefanprodan  suggested, please use other tools to sync secrets across the namespace boundary.
Yes, for multi-tenancy & security reasons - TF-controller will not support writing secrets into the other namespaces.


Request for detailed document

We are using kong ingress in k8s cluster. Can you kindly provide more detailed document about how to integrate this with our kong-ingress.
Thanks a lot for the great work.

Hi @BrikerMan sure thing, this repo was just to register to the Kong hackathon.
The solution in that repo has pretty bad scalability because it relies on the gateway plugin... and does synchronous calls during the packet handling.
Together with @ludovic-pourrat we have made a second implementation which adds tracing data afterwards using the logging phase handler (which is async).
I can check with him whether we should opensource it and provide some docs.

So, I have checked with @ludovic-pourrat and we have done nothing more than what is described here:
https://docs.konghq.com/gateway/2.7.x/reference/external-plugins/
As for publishing the new version (with improved performance) we have to wait approval from the newly created OSPO in our bank. I would not expect anything in the next 3 months unfortunately.
At least from a usage point of view, both plugins should deliver the same experience.

Please let me know if go build doesn't work for you.

Thanks for your reply. I will update you about go build next month.
We are preparing for the Chinese new year, so will be busy with other tasks till the end of next month. Is that okay I keep this issue open for 2 months?

Sure, no problem. Happy new year!

@BrikerMan I have added some resources, please check the README and docker-compose files.


ValueFactory attempted to access the Value property of this instance.

Another report

This issue was submitted by the add-in's exception handler.

Exception:
System.InvalidOperationException: ValueFactory attempted to access the Value property of this instance.
   at System.Lazy`1.CreateValue()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Lazy`1.get_Value()
   at XLToolbox.Dispatcher.Execute(Command cmd) in X:\Code\xltoolbox\NG\XLToolbox\Dispatcher.cs:line 83

Message: ValueFactory attempted to access the Value property of this instance.
Environment

Parameter
Value

Version
7.3.4

OS
Microsoft Windows NT 6.2.9200.0, 64-bit

Excel
2010 SP2 (14.0.7239), 32-bit

CLR
4.0.30319.42000, 32-bit

VSTO runtime
10.0.40305

ClickOnce
False

Bovender
<IP_ADDRESS>

FreeImage
3.18.0

Identifier
2d193fb3da4c7d2204e924665e2e1965c4ef981651af64e081fa0a5551779f3e

Another report

This issue was submitted by the add-in's exception handler.

Exception:
System.InvalidOperationException: ValueFactory attempted to access the Value property of this instance.
   at System.Lazy`1.CreateValue()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Lazy`1.get_Value()
   at XLToolbox.Dispatcher.Execute(Command cmd) in X:\Code\xltoolbox\NG\XLToolbox\Dispatcher.cs:line 83

Message: ValueFactory attempted to access the Value property of this instance.
Environment

Parameter
Value

Version
7.3.4

OS
Microsoft Windows NT 6.2.9200.0, 64-bit

Excel
2010 SP2 (14.0.7239), 32-bit

CLR
4.0.30319.42000, 32-bit

VSTO runtime
10.0.40305

ClickOnce
False

Bovender
<IP_ADDRESS>

FreeImage
3.18.0

Identifier
2d193fb3da4c7d2204e924665e2e1965c4ef981651af64e081fa0a5551779f3e

Another report

This issue was submitted by the add-in's exception handler.

Exception:
System.InvalidOperationException: ValueFactory attempted to access the Value property of this instance.
   at System.Lazy`1.CreateValue()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Lazy`1.get_Value()
   at XLToolbox.Dispatcher.Execute(Command cmd) in X:\Code\xltoolbox\NG\XLToolbox\Dispatcher.cs:line 83

Message: ValueFactory attempted to access the Value property of this instance.
Environment

Parameter
Value

Version
7.3.4

OS
Microsoft Windows NT 10.0.18362.0, 64-bit

Excel
365 (16.0.4966), 64-bit

CLR
4.0.30319.42000, 64-bit

VSTO runtime
10.0.40820

ClickOnce
False

Bovender
<IP_ADDRESS>

FreeImage
3.18.0

Identifier
2d193fb3da4c7d2204e924665e2e1965c4ef981651af64e081fa0a5551779f3e


Unable to establish SSL connection when "tailscale up"

root@iStoreOS:~# tailscale up --accept-dns=false --advertise-routes=<IP_ADDRESS>/24
--2023-06-17 16:39:59--  https://pkgs.tailscale.com/stable/
Resolving pkgs.tailscale.com... <IP_ADDRESS>
Connecting to pkgs.tailscale.com|<IP_ADDRESS>|:443... connected.
Unable to establish SSL connection.
Downloading Tailscale _arm64 ..
--2023-06-17 16:39:59--  https://pkgs.tailscale.com/stable/tailscale__arm64.tgz
Resolving pkgs.tailscale.com... <IP_ADDRESS>
Connecting to pkgs.tailscale.com|<IP_ADDRESS>|:443... connected.
Unable to establish SSL connection.
gzip: invalid magic
tar: Child returned status 1
tar: Error is not recoverable: exiting now
root@iStoreOS:~#

Could you install/update the libustream-openssl and ca-bundle packages and retry? That's the download of Tailscale failing.

Yes, It's OK for now today, the file name has version as below. Thanks.
root@iStoreOS:~# tailscale up --accept-dns=false --advertise-routes=<IP_ADDRESS>/24
--2023-06-18 18:20:59--  https://pkgs.tailscale.com/stable/
Resolving pkgs.tailscale.com... <IP_ADDRESS>
Connecting to pkgs.tailscale.com|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: unspecified [text/html]
Saving to: 'STDOUT'

-                                         [  <=>                                                                     ]  30.51K   128KB/s    in 0.2s

2023-06-18 18:21:01 (128 KB/s) - written to stdout [31240]

Downloading Tailscale 1.42.0_arm64 ..
--2023-06-18 18:21:01--  https://pkgs.tailscale.com/stable/tailscale_1.42.0_arm64.tgz
Resolving pkgs.tailscale.com... <IP_ADDRESS>
Connecting to pkgs.tailscale.com|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 22142063 (21M) [application/x-compressed-tar]
Saving to: 'STDOUT'

-                                       0%[                                                                          ]  14.01K  56.3KB/s               tailscale_1.42.0_arm64/tailscaled
-                                      58%[==========================================>                               ]  12.43M  3.85MB/s    eta 3s     tailscale_1.42.0_arm64/tailscale
-                                     100%[=========================================================================>]  21.12M  5.95MB/s    in 4.0s

2023-06-18 18:21:06 (5.27 MB/s) - written to stdout [22142063/22142063]

Done!


The Required Datasets for Training.

As indicated on GitHub, training our own model(superpoint-lightglue) requires two steps and a total of two datasets: 1(1M-image distractor set of the Oxford-Paris retrieval dataset.) and 2(MegaDepth dataset). However, I am not sure which part of these two datasets I should download. Is there anyone who can share the download links and tell me which parts to download?

The dataset code have the dataset message you need.you can check in the code and fine the dataset url  and file.


Fix epoll spliceTo file descriptor with offset

Motivation
The AbstractEpollStreamChannel::spliceTo(FileDescriptor, ...) methods take an offset parameter but this was effectively ignored due to what looks like a typo in the corresponding JNI function impl. Instead it would always use the file's own native offset.
Modification

Fix typo in netty_epoll_native_splice0() and offset accounting in AbstractEpollStreamChannel::SpliceFdTask.
Modify unit test to include an invocation of the public spliceTo method using non-zero offset.

Result
spliceTo FD methods work as expected when an offset is provided. Note this does change the prior behaviour in that the file's native offset was previously modified but will now be unchanged - we could make a further modification to preserve that though if necessary.

@netty-bot test this please


brain-score.org submission (user:386) | (public:False)

Plugin-only changes from brain-score.org submission. Generated via Jenkins job create_github_pr, build 245. Plugins added: models: [eBarlow_lmda_001_2]

travis timeout


[CN] translate basics string lesson

This commit translate basics/strings to Chinese, thus solving #598

Thank you again @cizixs 👍


Add timeout mechanic for chunk requests

Pull Request Etiquette

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the contributing guidelines.

Changes

[x] Internal code
[ ] Library interface (affecting end-user code)
[ ] Documentation
[ ] Other: _____

Closes Issue: NaN
Description
Now the controller will attempt to request chunks again if 10 seconds have passed since the last request. This should eliminate issues with resumes where its unclear if the requests were actually received.

I have been able to successfully timeout chunk requests and recover a session with this new handling.


Update free-courses-hi.md

What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources
Description
Added assembly tutorial in hindi
Why is this valuable (or not)?
Teaches hindi speakers how to code in assembly
How do we know it's really free?
it is a youtube video
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

For youtube, we prefer "playlist" links and the name of the producer/creator. docs/CONTRIBUTING.md#platforms-and-access-notes
Make any changes in this PR - don't close and open a new one.

Hi, finished the change

Thanks, and welcome to free-programming-books!


feat(dialog): Add tests and usage examples

Shows how to use the dialog (prompt and custom)
Add unit tests

Hmm. It looks like this doesn't have the latest code I sent. For example, I changed up the controller so that it would not reject promises on cancel. After trying this out in a real app, it proved to be problematic because in many cases, the developer doesn't care about the cancel case and so doesn't register a catch block. This results in errors in the console. We'll get this merged in and then I may make some changes based on that.

Ok. This was merged...but it was kind of strange the way the branching worked out. In any case, there is one broken test "the Dialog Renderer calls the corresponding controller methods"

Yeah I don't know what code was missed but if you can send me the updated version I'll diff it.  I'll squash the commits before merging to clean up history.


Migrate e2e test for Elasticsearch to Go

Proposal
Relates to https://github.com/kedacore/keda/issues/2357, https://github.com/kedacore/keda/issues/2737.
Use-Case
No response
Anything else?
No response

I'll take that. it might take some time because I see some complexity here (some direct exec commands)


[RFC] Add character literals

One thing I've found myself missing lately is character literals. My code seems to have a lot of magic numbers such as 48, 65, 90, 97 amd 122 which is OK when you write it but not so good when you come back to it weeks or months later or when other people are trying to read your code.
When programming in other C-family languages I can generally replace these numbers with the character literals '0', 'A', 'Z', 'a' and 'z' respectively which are self-documenting. Now I know you can do stuff such as "A".bytes[0] to get 65 but it's hardly nice or efficient as you're having to create a String object and call a method on it just to get a number!
I'm not suggesting here that we need a new Char type - we don't - just a simple alias for integer numbers in the code-point range 0 to 0x10ffff. So 'A' and 65 would mean exactly the same thing.
We don't currently use single quotes for anything else (and are unlikely to use them in the future) so there are no compatibility issues here.
A slight difficulty is escape sequences. Clearly we'd need a new one '\'' to represent the single quote character itself as a character literal. However, it would be unnecessary to escape double quotes and the percent sign (no interpolation of course) in character literals so '"' and '%' could be valid.
The question really is whether in string literals should "\'" be valid and whether in character literals should '\"' and '\%' be valid alternatives to their un-escaped versions? My personal opinion is that they should be but I don't have a strong view either way.
So what do y'all think - a worthwhile addition to the language or not?

@munificent clearly stated that he was against such addition. That said, in particular when you are manipulating strings/parsing the lack of easy conversion between char, string and number is really annoying. So if C char encoding is adopted, I would go all the way with a Char unicode type.
But we can already simulate it with external library, and do like:
Char.fromCodePoint(Char.fromString("a").codePoint + foo)

With trivial use of static maps, it can be quite efficient term of speed and memory (by making Chars immutable).

Well my personal taste would be to have a native type for Char. It would solve a bunch of problems, but I wounder how costly it could be in the VM to have this extra native type.

Well my personal taste would be to have a native type for Char. It would solve a bunch of problems, but I wounder how costly it could be in the VM to have this extra native type.

It probably could be a new NAN-tagged type? But I understand NAN-tagging is optional, right?

The are 2 value implementation in wren. So yes it would mean to have
implemented in Nan tagging and in union tagging. Union tagging is more
simple, so the issue is more to find room with Nan tagging.

I’ve added 32bit int literals for my implementation of Lox language, whose VM is quite similar to Wren. I’d say that adding a new native atomic type for NAN-tagging shouldn’t be an issue at all, I don’t notice any performance problem either. In theory a total of 8 native atomic types can be implemented, and Wren has 5 right now(NAN, NULL, TRUE, FALSE and UNDEFINED).
A better question is whether this is actually needed. To my understanding, most dynamically typed languages do not have character literals, and for a good reason. This will depend on the future direction for Wren how it will evolve. If there’s plan to make it gradually typed or even statically typed in version 1.0, then it makes sense to add dedicated Char type. If it will stay like a simple dynamically typed language, then it may not be necessary really.

I agree that it's not necessary to have a separate 'Char' type in Wren and all I'm asking for here is character literals as aliases for the corresponding code-point numbers so you don't have to remember what these are.
Having said that, I know that @mhermier would like a native 'Char' type for technical reasons and is incorporating it into  his 'veery' language - see #1159.

Small Talk has Strings with only one quote on one side, like this:
#iamaSingleWordLitteral. Which is equivalent to 'iamaSingleWordLitteral' (Small Talk uses single quotes for string). So you could type #A as synonym for "A". Not the same thing as a char-type, though.

While that feature would be cool, unfortunately '#' is used for shebang and attributes. If I remember well, that feature was also discussed somewhere else...

Well # in Smalltalk is called Symbol, basically immutable strings. It’s worth noting that strings are mutable in Smalltalk, which seems to be the norm in that era. It also has character literal with $.


Brainstorm enhancing UX

We are adding more information to statuses of the various APIs we have (https://github.com/kubernetes-sigs/kueue/issues/7 and https://github.com/kubernetes-sigs/kueue/issues/5); but I am wondering what other UX-related enhancements we should pursue for the two personas: batch admin and batch user.
UX gets users excited about the system and I think should be a focal point as Kueue evolves.

[ ] Prometheus integration for resource consumption of QueuedWorkloads / Queues

A kubectl plugin will be quite useful here for both admins and users.

I think we should update the QueuedWorkloadAdmitted condition to True in a best-effort basis.
This means that we don't need to do it in the scheduling cycle. But we can do it in the controller (we will get an event for the addition of the .spec.admission field).

Created https://github.com/kubernetes-sigs/kueue/issues/201 and https://github.com/kubernetes-sigs/kueue/issues/199 to track updating the workload condition and adding metrics. Closing this one.


Readme not showing in extension after deploying new version

After deploying version 1.0.1 the readme in the extension in VSCode is not showing anymore. Also the "overview" on marketplace.visualstudio.com is not updating to version 1.0.1.

Fixed in release 1.0.2


Virtual panel 1x4

I launch by placing 4 panels on top of each other. Connection from bottom left to top.
In the first lower module, the x coordinates are the same, and in the 3 upper ones they are shifted by one pixel. In the 2nd and 4th panels, x is shifted to the left, and in the 3rd panel to the right by one pixel.
The у coordinates are the same.
Controller HD-WF2.
64x32 panels, FM6124 driver.
`#define R1_PIN 2
#define R2_PIN 3
#define G1_PIN 6
#define G2_PIN 7
#define B1_PIN 10
#define B2_PIN 11
#define A_PIN 39
#define B_PIN 38
#define C_PIN 37
#define D_PIN 36
#define E_PIN 21
#define OE_PIN 35
#define CLK_PIN 34
#define LAT_PIN 33
#define PANEL_RES_X 64      // Number of pixels wide of each INDIVIDUAL panel module.
#define PANEL_RES_Y 32     // Number of pixels tall of each INDIVIDUAL panel module.
#define NUM_ROWS 4 // Number of rows of chained INDIVIDUAL PANELS
#define NUM_COLS 1 // Number of INDIVIDUAL PANELS per ROW
#define PANEL_CHAIN NUM_ROWS*NUM_COLS    // total number of panels chained one to another
#include <ESP32-VirtualMatrixPanel-I2S-DMA.h>
HUB75_I2S_CFG::i2s_pins _pins_x1 = {
R1_PIN,
G1_PIN,
B1_PIN,
R2_PIN,
G2_PIN,
B2_PIN,
A_PIN,
B_PIN,
C_PIN,
D_PIN,
E_PIN,
LAT_PIN,
OE_PIN,
CLK_PIN
};
#define VIRTUAL_MATRIX_CHAIN_TYPE CHAIN_BOTTOM_LEFT_UP
MatrixPanel_I2S_DMA *dma_display = nullptr;
// placeholder for the virtual display object
VirtualMatrixPanel  *virtualDisp = nullptr;
void setup()
{
HUB75_I2S_CFG mxconfig(
PANEL_RES_X,   // module width
PANEL_RES_Y,   // module height
PANEL_CHAIN,   // Chain length
_pins_x1       // pin mapping for port X1
);
mxconfig.i2sspeed = HUB75_I2S_CFG::HZ_20M;
mxconfig.latch_blanking = 4;
//mxconfig.clkphase = false;
mxconfig.driver = HUB75_I2S_CFG::FM6124;
mxconfig.double_buff = 0;
//mxconfig.min_refresh_rate = 30;
dma_display = new MatrixPanel_I2S_DMA(mxconfig);
dma_display->setBrightness8(255); //0-255
dma_display->begin();
dma_display->clearScreen();
virtualDisp = new VirtualMatrixPanel((*dma_display), NUM_ROWS, NUM_COLS, PANEL_RES_X, PANEL_RES_Y, VIRTUAL_MATRIX_CHAIN_TYPE);
virtualDisp->fillScreen(virtualDisp->color444(0,0,0));
virtualDisp->setPhysicalPanelScanRate(NORMAL_TWO_SCAN);
}
void loop()
{
if((!(0))) {
virtualDisp->drawPixel((5), (5), virtualDisp->color444((255), (0), (255)));
}
if((!(0))) {
virtualDisp->drawLine((1), (1), (1), (126), virtualDisp->color444((0), (255), (250)));
}
}`

I had to look here, now it's normal. https://github.com/mrcodetastic/ESP32-HUB75-MatrixPanel-DMA/issues/134#issue-919612284

Another problem was found: the color shades of modules 1 and 3 are different from 2 and 4 in the example above. The panels are normal, when swapped, the shades remain the same.


Update preservica-client-fs2 to 0.0.80

About this PR
📦 Updates uk.gov.nationalarchives:preservica-client-fs2 from 0.0.78 to 0.0.80
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "uk.gov.nationalarchives", artifactId = "preservica-client-fs2" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "uk.gov.nationalarchives", artifactId = "preservica-client-fs2" }
}]

labels: library-update, early-semver-major, semver-spec-patch, commit-count:1

Superseded by #86.


Update to GEOSchem_GridComp v1.3.5

This is a zero-diff update to Chem to support MAPL 2.2

This is a sibling PR to https://github.com/GEOS-ESM/GEOSgcm_GridComp/pull/295


anytype: 0.43.8 -> 0.44.0; build from source

Upstream has no aarch64-linux builds :/
Either supersedes or follows #366778. Personally, I'd prefer rebasing after that PR is merged so that the packaging changes are isolated. I tried splitting the rewrite and update and packaging 0.43.8 from source, but anytype-heart at 0.37.4 uses go 1.23 iterators but go.mod declares they use 1.22, so that'd need some extra patching and is probably not worth the effort.

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Hey. I thought that, since Anytype is a proprietary application, the build will not be cached by Hydra, right? That would mean that we would force every user to compile the entirety of Anytype themselves each update. If that is the case, I am not sure if the source Anytype version should entirely replace the existing binary package. Maybe create a new package, something like anytype and anytype-bin, then?


Deep Learning

I think that would be interesting if the GPUs had hardware dedicated to data compression and decompression. It would be incredible in tools such as deep learning where there is constantly a large volume of data in transfer. But now, Is it possible to implement it in deep learning frameworks like pytorch? I see new API direct storage features https://devblogs.microsoft.com/directx/directstorage-1-1-coming-soon/

hi @johnnynunez, do you have any particular use case in mind for fast compression/decompression for deep learning?

hi @johnnynunez, do you have any particular use case in mind for fast compression/decompression for deep learning?

In the hypothetical case that all my preprocessing is done offline. Wouldn't it be possible to load the batches from the SSD in compressed form?

hi @johnnynunez, do you have any particular use case in mind for fast compression/decompression for deep learning?

hi @johnnynunez, do you have any particular use case in mind for fast compression/decompression for deep learning?

In the hypothetical case that all my preprocessing is done offline. Wouldn't it be possible to load the batches from the SSD in compressed form?

hi @johnnynunez, do you have any particular use case in mind for fast compression/decompression for deep learning?

In the hypothetical case that all my preprocessing is done offline. Wouldn't it be possible to load the batches from the SSD in compressed form?

@johnnynunez for loading data directly from SSD bypassing the CPU you need to use GPUDirect Storage (GDS), it's orthogonal to the compression, and we have an example of using nvCOMP with GDS https://github.com/NVIDIA/nvcomp/blob/main/examples/nvcomp_gds.cu
The link you referenced introduces a new format GDeflate, we support both compression and decompression of GDeflate format in nvCOMP on the GPU.
BTW, is your platform Windows or Linux?

@johnnynunez for loading data directly from SSD bypassing the CPU you need to use GPUDirect Storage (GDS), it's orthogonal to the compression, and we have an example of using nvCOMP with GDS https://github.com/NVIDIA/nvcomp/blob/main/examples/nvcomp_gds.cu
The link you referenced introduces a new format GDeflate, we support both compression and decompression of GDeflate format in nvCOMP on the GPU.
BTW, is your platform Windows or Linux?

Linux


Does not compile on windows.

Fails to build with the following errors:
151               typ: PosixRlimitType::RlimitNofile,
^^^^^^^^^^^^ variant or associated item not found in PosixRlimitType
191 | pub enum PosixRlimitType {
------------------------ variant or associated item RlimitNofile not found for this enum
191 | pub enum PosixRlimitType {
------------------------ variant or associated item RlimitCpu not found for this enum
265       Self::RlimitCpu
^^^^^^^^^ variant or associated item not found in PosixRlimitType

That's a duplicate of https://github.com/containers/oci-spec-rs/issues/179, we're working on a new release to fix that issue.


adding markdown samples

This is to solve #10. The test worked for me, with the output here:
sample.pdf
Developer Certificate of Origin is already amended, it seems the below checks is not automatically updated.

@weiwu-zhang thanks for the contribution.


Unit tests should be runnable

This package should be able to run unit tests in Travis/CircleCI/whatever for a full integration test suite, but also ideally have some subset available locally if the config isn't available (i.e. for external contributors)

Fixed in #20 @msluther


Fix for errors when removing entities

Override entity.remove to avoid removing all colyseus schema fields on resetting an entity.

Tests are failing because the required _ComponentTypes etc aren't present until the applyMixin is called.  Might be able create a separate Mixin class with remove which is applied before EcsyEntity. Unless anybody can think of a different approach?
Can look at this tomorrow.

Hi @will-hart, I'm afraid we'd have to clear the internal refId as well (I'm already doing this for components, missed for entities)
https://github.com/endel/ecs/blob/03e6c9d1fde808e66976e4580a847a934acec074/src/index.ts#L26-L30

The errors caught by the CI none of those properties actually exist in the Entity class. Because the EcsyEntity is injected later into the Entity, you can see all EcsyEntity references have the // @ts-ignore flag to avoid compilation errors :eyes:

I'm afraid we'd have to clear the internal refId

Ahh good pickup thanks.

`// @ts-ignore

Oh I didn't think of that 🤣


Impossible to debug failing tests

I have a failing pub test, but no way to debug it.
I can find ways to run the test, but the IDE does not provide any way to set a breakpoint and debug the code. (At least I couldn't find any).
Eventually I ended up creating a new build-configuration that just executes 'dart' directly on the failing test file. That at least, gave me a way to set a breakpoint in the main of the that file, but I was unable to set a breakpoint in the package that was failing.
Update: the breakpoint not triggering is probably because Pub is running commands...
Edit: removed breakpoint screenshot.

/cc @jwren for triage (and possibly moving to the YouTrack tracker)

YouTrack, area-analyzer?

but I was unable to set a breakpoint in the package that was failing

When you run test as an ordinary Dart app breakpoints should works anywhere: own code, packages and SDK. In case of file from package make sure you set breakpoint in the original file, not in its symlink from any of numerous 'packages' folders.

Debugging VM tests that are started via 'pub run test' is not possible at the moment. Tracked as https://github.com/dart-lang/test/issues/50 and/or https://github.com/dart-lang/test/issues/459.
In my understanding the problem could also be solved by being able to pass VM options to the VM that executes tests (when starting tests via pub run test). For example this could be solved if pub run started program in a separate isolate (currently it spawns separate OS process). Related issues are https://github.com/dart-lang/pub/issues/1018 and https://github.com/dart-lang/pub/issues/1204.

Thanks @alexander-doroshko !

Debugging of tests executed by the Dart VM is possible starting from IntelliJ IDEA 2017.1 (available as EAP now) and Dart SDK 1.22+.


Hyprland 0.41.0

Draft PR for hyprland 0.41.0
@WhySoBad I created the dev branch, let me know if you want to participate

I did, but then completely forgot to reply. Also, if you happen to have telegram, feel free to add me there (https://t.me/Levnikmyskin).
I'll give you some rights to contribute to this repo btw

Hi, if that's me then it must be something I created to only enter once and take a look. I don't use discord and I don't even have it installed actually. If you don't have telegram, you can find my email in the commit log, i.e., if you use git log in the repo you should see my email

I created a matrix room (it's also linked in the main readme now).
@WhySoBad


Piercing Spell (Metamagic) - Plea

Please, is it possible to implement Piercing Spell Feat (Metamagic) , please see https://www.d20pfsrd.com/feats/metamagic-feats/piercing-spell-metamagic/
Thanks:)

Will be in 1.11.0

cool! Thanks a lot:)
---------- Původní e-mail ----------
Od: Sean Petrie @.>
Komu: Vek17/WrathMods-TabletopTweaks @.
github.com>
Datum: 13. 1. 2022 22:48:38
Předmět: Re: [Vek17/WrathMods-TabletopTweaks] Piercing Spell (Metamagic) -
Plea (Issue #348)
"
Will be in 1.11.0
—
Reply to this email directly, view it on GitHub
(https://github.com/Vek17/WrathMods-TabletopTweaks/issues/348#issuecomment-1012542270)
, or unsubscribe
(https://github.com/notifications/unsubscribe-auth/AH7WAQGKVOCL6OT2PHN6HWTUV5CDBANCNFSM5LCXUCFQ)
.
Triage notifications on the go with GitHub Mobile for iOS
(https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675)
or Android
(https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub)
.
You are receiving this because you authored the thread. Message ID: <Vek17/
@.***>
"

Dear Mr. Petrie,
is that yet any possibility to join discord community please? The link
https://discord.gg/bQVwsP7cky is not working for me..
Many thanks
Petr
---------- Původní e-mail ----------
Od: Sean Petrie @.>
Komu: Vek17/WrathMods-TabletopTweaks @.
github.com>
Datum: 13. 1. 2022 22:48:38
Předmět: Re: [Vek17/WrathMods-TabletopTweaks] Piercing Spell (Metamagic) -
Plea (Issue #348)
"
Will be in 1.11.0
—
Reply to this email directly, view it on GitHub
(https://github.com/Vek17/WrathMods-TabletopTweaks/issues/348#issuecomment-1012542270)
, or unsubscribe
(https://github.com/notifications/unsubscribe-auth/AH7WAQGKVOCL6OT2PHN6HWTUV5CDBANCNFSM5LCXUCFQ)
.
Triage notifications on the go with GitHub Mobile for iOS
(https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675)
or Android
(https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub)
.
You are receiving this because you authored the thread. Message ID: <Vek17/
@.***>
"


Is /OOBE required?

In the documentation it states we should run "sysprep.exe /oobe /generalize /shutdown"
Is /oobe mandatory? If I'm going to use the image for VMSS I don't want it stopping at the Out-Of-Box-Experience but I don't know if Azure is doing something during this process.
Thanks!

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

ID: 83917921-2dc0-42a9-1b99-cce2fca9961a
Version Independent ID: 3812c21e-a5f6-af8d-17b5-05a2635d8b1d
Content: Deprovision or generalize a VM before creating an image - Azure Virtual Machines
Content Source: articles/virtual-machines/generalize.md
Service: virtual-machines
Sub-service: imaging
GitHub Login: @cynthn
Microsoft Alias: cynthn

I have just tested this and without OOBE it doesn't run...
How would you go about setting up a Virtual Machine Scale Set with custom software if you need to avoid OOBE?

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

@DarylGraves and @SaibabaBalapur-MSFT    It absolutely shouldn't have /oobe. What OS SKU and version are you trying to generalize? I suspect there might have been a change to sysprep recently if sysprep.exe /generalize /shutdown isn't working. Can you tell me what exactly is happening?
I've removed /oobe  from the syntax in the doc. There is some debate about whether /mode:vm helps, so you could try that. I'm also going to try to recreate this with whatever version you were working with.
Thank you!

@cynthn Thanks for your quick responses. Please share the pull request link once the document has been updated with the changes.
@DarylGraves
if there are any further questions regarding the documentation, please tag here with your reply and we will be happy to continue the conversation.

@SaibabaBalapur-MSFT The change is already live :)

@DarylGraves
Thanks for reporting this! We have created a PR for this issue and the changes is live now.


[Question] How to properly change the username and password of the MySQL db?

Hello. I am new to docker/docker-compose so pardon my ignorance.
I changed the username and password variables in the .env file. Then I ran docker-compose build. But after doing this, PHPMyAdmin cannot access the database anymore.
Thanks for making this repo public. It's very helpful. :)

Hi,
I guess you started the stack previously. You need to remove everything from data/mysql/*. Afterwards mysql will start a new instance and will initialize with you new credentials.

It did work but I think the problem lies with the particular password I have chosen for the $MYSQL_PASSWORD environment variable since it contains symbols which are causing problems. Changing the password to something simpler like testpass works perfectly. I will test a bit more and let you know. Thanks a lot for the help.

Updated to docker-compose 1.26.2 (latest) which uses python-dotenv to parse the .env file. The issue still persists but I will close the issue because it looks like it is related to a limitation of docker-compose itself. Thanks for your help.


fix: Notification popup click event does not open relevant chat window.

Closes #2091 and #2007.
When a notification arises, if user clicks on that notification, nothing happens. With this fix, it will open relevant chat window (with proper user) as supposed to be.
Tested locally.

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

Nice! Waiting for review!
Thank you @gencer

tested, works fine ! thank you @gencer

Thanks for your effort @gencer. I will take a look and test it on a Windows install on a physical machine (not VM) as soon as possible.


Bug / Potential error in the documentation - GET/POST /predict request

In the docs:
https://labelstud.io/guide/ml.html#Get-predictions-from-a-model
There is the paragraph:
If you want to retrieve predictions manually for a list of tasks using only an ML backend, make a GET request to the /predict URL of your ML backend with a payload of the tasks that you want to see predictions for, formatted like the following example: [...]
With a GET request I get the following Error:
requests.exceptions.HTTPError: 405 Client Error: METHOD NOT ALLOWED
But with a POST request it seams to work just fine.

@FelixJuch Thank you very much, it's a misspell. I will fix it asap.

Fixed: https://github.com/heartexlabs/label-studio/pull/2298/files


Consider calling didMount() before build() for StatefulComponents

Currently, we build first and then call didMount.  That makes the state machine for StatefulComponents really complex.  For example, if you get unmounted, you can be asked to build before you're remounted.  If you draw out the state machine, it's much cleaner if didMount comes before build and didUnmount is always followed by didMount before build.  That gives you the invariant that you only ever build while mounted.
/cc @Hixie

It's possible to be built without being mounted ever (e.g. the MixedViewport widget does it to its children).

The state machine for stateful components is a lot more rational now that we have fn3.


Remove the word "Social" from README's headline

I have the feeling that it's not just about social icons anymore, right?
What do you think?

Good plan. Do it!


fix: AccountId.toStringWithChecksum() errors

Description

AccountId.toStringWithChecksum() errors when invoked

Steps to reproduce
(1) Install the following dependencies:
{
    "type": "module",
    "dependencies": {
        "@hashgraph/sdk": "2.31.0",
        "dotenv": "16.3.1"
    }
}

(2) run the following script
import {
    Client,
    AccountId,
    PrivateKey,
} from '@hashgraph/sdk';
import dotenv from 'dotenv';

dotenv.config();

if (!process.env.OPERATOR_ID || !process.env.OPERATOR_KEY) {
    throw new Error('Must set OPERATOR_ID and OPERATOR_KEY in .env');
}
const operatorId = AccountId.fromString(process.env.OPERATOR_ID);
const operatorKey = PrivateKey.fromString(process.env.OPERATOR_KEY);
const client = Client.forTestnet().setOperator(operatorId, operatorKey);

async function main() {
    console.log('Operator Account ID:', operatorId.toString());
    console.log('Operator Account ID with checksum:', operatorId.toStringWithChecksum());

    process.exit(0);
}

main();

(3) Observe the output:

operatorId.toString() succeeds
operatorId.toStringWithChecksum() fails with this error

node_modules/@hashgraph/sdk/src/EntityIdHelper.js:397
    if (client._network._ledgerId == null) {
               ^

TypeError: Cannot read properties of undefined (reading '_network')
    at Module.toStringWithChecksum (file:///Users/user/code/hedera/hedera-code-snippets/multisig-account/node_modules/@hashgraph/sdk/src/EntityIdHelper.js:397:16)
    at AccountId.toStringWithChecksum (file:///Users/user/code/hedera/hedera-code-snippets/multisig-account/node_modules/@hashgraph/sdk/src/account/AccountId.js:297:26)
    at main (file:///Users/user/code/hedera/hedera-code-snippets/multisig-account/multisig-1-of-2.js:24:52)
    at file:///Users/user/code/hedera/hedera-code-snippets/multisig-account/multisig-1-of-2.js:29:1
    at ModuleJob.run (node:internal/modules/esm/module_job:194:25)

Node.js v18.15.0

Additional context
No response
Hedera network
testnet
Version
0.31.0
Operating system
macOS

This is normal behaviour - you are supposed to pass the client to the method in order to get the _network value. I just added a handler to indicate this to the users as well


⚠ ClamAV has degraded performance

In a48dab0, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 197 ms

Resolved: ClamAV performance has improved in 94f44c3.


Port custom component

To implement
Except if Bootstrap v5 added equivalents components to customize (if so, try to customize Bootstrap's ones first — and in case of improvements, backport them to BS).

[x] fluid-margins full-width containers. Dropped in v5
[x] focus-visible polyfill and styles: #139 moved to 5.0.0-alpha milestone
[x] link with arrow (#520)
[x] Buttons: (#521)

[x] inversed buttons: should use custom properties to switch colors depending on context
[x] icons buttons: close button should inherit those in any way— as well as pagination with icons only, isn't it?
[x] social buttons

[xm] accordion: multiple openable Dropped
[x] megamenu: should totaly refactor this, starting by defining behaviours — and should also handle aria-current, I guess (#522)
[x] footer: still required as a component Example firtst, see #410
[x] Navbar: (#525)

[x] icons in navbar
[x] minimizing behaviour on scroll
[x] local + priority navigation: shouldn't this be a single component, since priority nav is based on local nav? Moreover, local nav is basically a navbar with borders…
[x] supra nav: mostly a navbar variant, isn't it?
[x] nav-tabs-light → done in #362, using a custom property --bs-tabs-spacing to reduce code overrides

[x] Scroll up (#523)
[x] Stepbar: also ensure to have a look at bs-stepper
[x] Tables: (#526)

[x] responsive tables (#222)
[x] sortable tables (#526)

https://dequeuniversity.com/library/aria/table-sortable

[x] icons in tables
[x] row selection ~~ Example only (#410)

[x] Carousels: see #366 (Bootstrap-side)
[x] tabs: maybe try to update Bootstrap's ones to comply with Heydon Pickering Inclusive tabbed Interface Component? see #140 #150 #456

To enhance

[x] Colors: enforce contrasts by coupling color with background-color in utilities
[x] color-yiq to use WCAG contrast algorithm? See #292
[x] background utilities using supporting colors instead of functionnal ones

To question
Those Boosted v4 components duplicates existing Bootstrap components, so I'd recommend to try to customize Bootstrap's ones before porting those to v5.

[x] Orange popin: what's differing from Bootstrap modal? → almost nothing since responsive content is achievable using grid, utilities and components…
[x] Custom carousel: shouldn't we simply override Bootstrap one? → confirmed, check #366
[x] Orange navbar: only a navbar example with content… I guess only supra nav should remain
[x] 3-way toggles: duplicated by "Buttons" → "Toggle states"
[x] O-tabs: tabs component (in Bootstrap's nav) seems to do the job (see #140 #150)

Closing this after triage: either dedicated issues exist or feature/component will be dropped in v5.


Secondary Oauth

I'm trying to use the library to process OAuth2 requests for Mixer.com.  Mixer allows you to sign in with a Microsoft, Twitter or Discord account.  The electron-oauth2 window will authenticate them with password login, however if they click sign in with Microsoft, they are presented with a blank login.  I tried setting sandbox: true in the webPreferences of the parameters passed to the constructor, and that allows the child windows to display and authenticate with their respective services, however when I come back to Mixer and allow the oauth connection, it goes to a white screen and doesn't close the window or return the Auth and refresh tokens.  Do you know if there's a way to return the credential with sandbox on, or is there another way to let these child windows be displayed without having to resort to sandbox mode?

Just adding some screenshots to try to describe the flow a bit.

Is anything happening with this?
I am also affected by this, and not sure what to do. Should I just add the changes from #26 ?


fix(auth): Do not ask to log-in on every page refresh

Closes https://github.com/backstage/backstage/issues/5109
Relates https://github.com/backstage/backstage/discussions/5389
The SingleSignInPage component is used when users add a custom SignInPage to their Backstage app with just one Sign In provider (for example). This is a different component than the MultiSignInPage which is used when there are more than one Sign In providers.
To fix the "user being logged out on every page refresh" bug, this PR makes the following changes to the SingleSignInPage component

Set the default value of retry state to be true. Without this, the Sign in page loads but makes no attempt to automatically log in the user if they have existing session.

Use a autoShowPopup state to determine whether or not to show log-in popup if the user is not signed in.

In the login function, perform a check beforehand to see if a session exists before requesting a new Backstage identity.

Use a showLoginPage state and a progress bar to prevent a glitch-like experience in case when the sign-in page is displayed for a split second when the user is already logged-in.

Hi, @OrkoHunter , seems there have an issue I didn't test it yesterday. If set auto to false for sign in. the page couldn't be loaded and remains on progress. This is because the setLoginPage is init to false, and login method will never be called to setLoginPage again. I think we might can set as:
return showLoginPage || !retry ? (
    ...
  ) : (
    <Progress />
  );

though the behavior is not correct as the sign in dialog will always popup even refresh.

Thanks for the heads-up @falconbin ! Let me test things when auto is set to false.

Thanks @falconbin ! I tested the behaviour with auto set as true and false and all the flows seem to be in place.
@Rugvip another review please 🙏 🙂

Hi all, how to apply this patch to the standalone app?
Tried to update the core https://backstage.io/docs/getting-started/keeping-backstage-updated
But no luck, still having the same problem on my end. :(
Also nothing about login in  manual patches or maybe it missed anything, can someone advice, please? 🙏

Don't mind ^ https://github.com/backstage/backstage/pull/5702#issuecomment-855645721 ! It looks like its related to this issue using GitLab authentication  https://github.com/backstage/backstage/issues/3044


cli-tips: fix update script

Just a small fix for the update script for my package.
https://nixpkgs-update-logs.nix-community.org/cli-tips/2024-12-11.log
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4993


fix: use module augmentation instead of extending config

also changes defineConfig to withPwa

@brc-dd vitepress-alpha15 seems to broke the pwa: the viteConfig.build.ssr is true inside the transformHead hook, we're getting the api reference from the client build, but it seems to be mutated (shared between Vitepress builds)

You can merge this PR when you solve the ts problem.

@brc-dd you can check how Vitest do the augmentation here: https://github.com/vitest-dev/vitest/blob/main/packages/vitest/src/config.ts#L3


Security question: DiskAttribute uses user provided filename for temp file name

Hi there,
maybe I'm paranoid, but I have a security concern.
If you use DiskFileUpload the temp file name is partly the user provided file name, only validation is this:
protected String getDiskFilename() {
    File file = new File(filename);
    return file.getName();
}

This catches the most obvious security gotchas (like /../../../somethingEvil) but almost everything else I tried is passed directly to the file system.
Is this safe? Why not simply use some random string?
Thanks
Aron

@eun-ice we love PRs ... Maybe you want to submit one with a "fix" ?

Here it is: https://github.com/netty/netty/pull/5481


About CopyIntoStream problem

eg:
create stream stream_1 (data text);
CopyIntoStream -> stream_1  ing....
but now
 create continuous view v as select data from stream_1;
pipelinedb is segmentfault,GeLocalStreamReader  and searchCatCache is error.
about Transaction?  every rows copy into stream need start new transaction?

@derekjn  0.9.8

fn_addr ptr is bad...  what happen ? I restart PipelineDB is OK.

I know what happen in this segment fault.
I create MemoryContext use CurrentMemoryContext is bad,I need use TopMemoryContext.