release: v3.6.5

Codecov Report

Merging #784 (7322f9b) into master (c190163) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 7322f9b differs from pull request most recent head 15c6641. Consider uploading reports for the commit 15c6641 to get more accurate results

@@           Coverage Diff           @@
##           master     #784   +/-   ##
=======================================
  Coverage   52.12%   52.12%
=======================================
  Files          46       46
  Lines        1224     1224
  Branches      257      257
=======================================
  Hits          638      638
  Misses        586      586


Add basic description to README

Basic README description

I'll implement the changes in another PR, we need to get this out quick RN


Redfish URI response for CI (1 pt)

https://gerrit.openbmc-project.xyz/c/openbmc/openbmc-test-automation/+/27425


Question about command's auto-complete mechanism

Hi, I am neophyte of Emacs, and I started using this configuration on April 2016. At that time, it's very easy to find command after typing M-X. I thought it's something like auto-full-complete the command at that time. I can just type a few key words of the command and it could show me the thing which exactly I want. But I didn't remember since what time, the command's completion mechanism was changed. It request me to input keywords more strictly. I was confused by it for so long time without asking about it. However, I thing I should know what made this change and how can I modify it to previous. Then maybe I could understand a little more about Emacs than before.
Thank you,
Jiashuai Lu

Hi @jslu0418! Yes, I changed from ido to ivy a while ago, and you can read about ivy to get a full understanding. But here's what you need to know: the matching now is essentially a regular expression, and spaces expand to .*, ie. "anything". So to run the command package-list-packages with M-x, previously with ido you could have typed packlist or listpack. Now, with ivy, you would need to type pack list or  list pack. A little retraining is necessary, then, but I've found that quite easy and I recommend you try it too.
(If you really, really can't retrain yourself, you can try calling sanityinc/enable-ivy-flx-matching, which should make the matching work more like ido, but it might also slow down completion sometimes.)

Hi Purcell, Thank you for your prompt reply, yeah I tried it just now. It's really easy for me to use too. I am really sorry for bothering you for this small question. I found the new mechanism kind of respect the regular expression, but I find it's quite annoying for typing too much .* and .* . Since you told me the 'space' expand to .* . I feel the convenient world is back, you really made my day. I may still should read about ivy to learn more about it.
Thank you again, you are soooooooooo nice!!!

I'm happy to help, and don't feel bad about asking. Happy hacking!


How to disable auto update

Which version of the AzCopy was used?
Note: The version is visible when running AzCopy without any argument
User-Agent: AzCopy/10.21.0
Which platform are you using? (ex: Windows, Mac, Linux)
Linux hosted in azure
What command did you run?
Note: Please remove the SAS to avoid exposing your credentials. If you cannot remember the exact command, please retrieve it from the beginning of the log file.
azcopy sync
We are syncing millions of small files around 4TB
What problem was encountered?
azcopy tries to get metadata from the internet and fails as there is no internet access on the server
2024/08/28 13:20:20 ==> REQUEST/RESPONSE (Try=1/21.464366ms, OpTime=21.465915ms) -- REQUEST ERROR#012   GET https://azcopyvnextrelease.blob.core.windows.net/releasemetadata/latest_version.txt#012   Accept: application/xml#012   User-Agent: AzCopy/10.21.0 azsdk-go-azblob/v1.1.0 (go1.19.12; linux)#012   X-Ms-Client-Request-Id: ea1b1b8c-8006-4f77-7d42-f4f6ea787e93#012   x-ms-version: #012   --------------------------------------------------------------------------------#012   ERROR:#012Get "https://azcopyvnextrelease.blob.core.windows.net/releasemetadata/latest_version.txt": EOF#012goroutine 53 [running]:#012github.com/Azure/azure-storage-azcopy/v10/ste.stack()#012#011/home/vsts/work/1/s/ste/xferLogPolicy.go:239 +0x65#012github.com/Azure/azure-storage-azcopy/v10/ste.logPolicy.Do({{{0xb2d05e00, 0x0}, 0x109e878, 0xc00040c030}, 0xc0004021e0, 0xc000402210, 0xc000402240}, 0xc000408400)#012#011/home/vsts/work/1/s/ste/xferLogPolicy.go:433 +0xb5d#012github.com/Azure/azure-sdk-for-go/sdk/azcore/internal/exported.(*Request).Next(0xc0004083c0)#012#011/home/vsts/go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/azcore@v1.7.1/internal/exported/request.go:84 +0xf8#012github.com/Azure/azure-storage-azcopy/v10/ste.(*trailingDotPolicy).Do(0xf4d340?, 0x181b280?)#012#011/home/vsts/work/1/s/ste/xferVersionPolicy.go:119 +0x225#012github.com/Azure/azure-sdk-for-go/sdk/azcore/internal/exported.(*Request).Next(0xc000408380)#012#011/home/vsts/go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/azcore@v1.7.1/internal/exported/request.go:84 +0xf8#012github.com/Azure/azure-storage-azcopy/v10/ste.(*coldTierPolicy).Do(0xc0003ca5e8?, 0xc000408380)#012#011/home/vsts/work/1/s/ste/xferVersionPolicy.go:83 +0xf9#012github.com/Azure/azure-sdk-for-go/sdk/azcore/internal/exported.(*Request).Next(0xc000408340)#012#011/home/vsts/go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/azcore@v1.7.1/internal/exported/request.go:84 +0xf8#012github.com/Azure/azure-storage-azcopy/v10/ste.(*versionPolicy).Do(0xc0003ca6c0?, 0xc000408340)#012#011/home/vsts/work/1/s/ste/xferVersionPolicy.go:68 +0x134#012github.com/Azure/azure-sdk-for-go/sdk/azcore/internal/exported.(*Request).Next(0xc000408300)#012#011/home/vsts/go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/azcore@v1.7.1/internal/exported/request.go:84 +0xf8#012github.com/Azure/azure-storage-azcopy/v10/ste.(*retryNotificationPolicy).Do(0x0?, 0xc000408300)#012#011/home/vsts/work/1/s/ste/xferRetryNotificationPolicy.go:90 +0x25#012github.com/Azure/azure-sdk-for-go/sdk/azcore/internal/exported.(*Request).Next(0xc0003ca8c0)#012#011/home/vsts/go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/azcore@v1.7.1/internal/exported/request.go:84 +0xf8#012github.com/Azure/azure-sdk-for-go/sdk/azcore/runtime.(*retryPolicy).Do(0xe12b60?, 0xc000408240)#012#011/home/vsts/go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/azcore@v1.7.1/runtime/policy_retry.go:134 +0x5b4#012github.com/Azure/azure-sdk-for-go/sdk/azcore/internal/exported.(*Request).Next(0xc000408200)#012#011/home/vsts/go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/azcore@v1.7.1/internal/exported/request.go:84 +0xf8#012github.com/Azure/azure-sdk-for-go/sdk/azcore/runtime.(*requestIDPolicy).Do(0xc0003caab8?, 0xc000408200)#012#011/home/vsts/go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/azcore@v1.7.1/runtime/policy_request_id.go:33 +0x146#012github.com/Azure/azure-sdk-for-go/sdk/azcore/internal/exported.(*Request).Next(0xc0004081c0)#012#011/home/vsts/go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/azcore@v1.7.1/internal/exported/request.go:84 +0xf8#012github.com/Azure/azure-sdk-for-go/sdk/azcore/runtime.telemetryPolicy.Do({{0xc00017c040?, 0xf49980?}}, 0xc0004081c0)#012#011/home/vsts/go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/azcore@v1.7.1/runtime/policy_telemetry.go:66 +0x1c5#012github.com/Azure/azure-sdk-for-go/sdk/azcore/internal/exported.(*Request).Next(0xc000408180)#012#011/home/vsts/go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/azcore@v1.7.1/internal/exported/request.go:84 +0xf8#012github.com/Azure/azure-sdk-for-go/sdk/azcore/runtime.includeResponsePolicy(0xc000408180)#012#011/home/vsts/go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/azcore@v1.7.1/runtime/policy_include_response.go:19 +0x25#012github.com/Azure/azure-sdk-for-go/sdk/azcore/internal/exported.PolicyFunc.Do(0xc000402300?, 0xc0003d0088?)#012#011/home/vsts/go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/azcore@v1.7.1/internal/exported/request.go:181 +0x1f#012github.com/Azure/azure-sdk-for-go/sdk/azcore/internal/exported.(*Request).Next(0xc000408140)#012#011/home/vsts/go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/azcore@v1.7.1/internal/exported/request.go:84 +0xf8#012github.com/Azure/azure-sdk-for-go/sdk/azcore/internal/exported.Pipeline.Do({{0xc000002140?, 0x11e4b88?, 0xc00003c090?}}, 0xc000408140)#012#011/home/vsts/go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/azcore@v1.7.1/internal/exported/pipeline.go:96 +0x199#012github.com/Azure/azure-sdk-for-go/sdk/storage/azblob/internal/generated.(*BlobClient).Download(_, {_, _}, _, _, _, _)#012#011/home/vsts/go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/storage/azblob@v1.1.0/internal/generated/zz_blob_client.go:878 +0xa8#012github.com/Azure/azure-sdk-for-go/sdk/storage/azblob/blob.(*Client).DownloadStream(_, {_, _}, _)#012#011/home/vsts/go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/storage/azblob@v1.1.0/blob/client.go:392 +0x148#012github.com/Azure/azure-storage-azcopy/v10/cmd.beginDetectNewVersion.func1()#012#011/home/vsts/work/1/s/cmd/root.go:262 +0x169#012created by github.com/Azure/azure-storage-azcopy/v10/cmd.beginDetectNewVersion#012#011/home/vsts/work/1/s/cmd/root.go:244 +0x6a
Also, there is one more problem. When running the azcopy sync the memory (128GB) on the server is almost entirely utilised. How can we limit the memory used by azcopy. By configuring AZCOPY_BUFFER_GB?
How can we reproduce the problem in the simplest way?
Have you found a mitigation/solution?
Disabling auto update of azcopy might fix the issue but there is no official documentation on how to disable the auto update of azcopy. We have tried --skip-version-check with sync but still failed.
I came across two variables AZCOPY_DISABLE_AUTO_UPDATE and AZCOPY_AUTO_UPDATE, are these supported.
Best
Reddy

Environment Variables -
You can disable the auto-update feature by setting specific environment variables. The two environment variables you mentioned, AZCOPY_DISABLE_AUTO_UPDATE and AZCOPY_AUTO_UPDATE, are the correct ones to use.
AZCOPY_DISABLE_AUTO_UPDATE: Setting this variable to true disables the auto-update check. This prevents AzCopy from attempting to check for and download the latest version.
AZCOPY_AUTO_UPDATE: Setting this variable to false also prevents AzCopy from auto-updating.
Memory Utilization Issue
Regarding the high memory usage (128GB) during the azcopy sync operation, AzCopy is indeed known to consume a large amount of memory when dealing with numerous small files.
AZCOPY_BUFFER_GB: You can control the memory usage by setting the AZCOPY_BUFFER_GB environment variable. This variable limits the amount of memory AzCopy uses for buffering data.
Additionally, you might want to consider splitting the sync operation into smaller chunks or using the --cap-mbps flag to throttle the data transfer rate, which could indirectly reduce memory usage.

@tanyasethi-msft Thanks for the inputs. I have tried AZCOPY_BUFFER_GB, but it seems to be not working. I kept it to half of the memory but it still consumed almost all the memory most of it is shown as cached memory on the server.
I will give the speed throttle a try.
Best
Reddy

@tanyasethi-msft I have checked the both auto update variables AZCOPY_DISABLE_AUTO_UPDATE and AZCOPY_AUTO_UPDATE and they are not working. It is still trying to reach internet to check for latest version.
Since version 10.22.0 cache feature is included, https://github.com/Azure/azure-storage-azcopy/pull/2402.
So I created a latest_version.txt file under folder .azcopy containing 10.24.0 and a expiry date less than 1 day but it is still going to internet and failing. Is there any way at all to stop the azcopy from going to the internet or atleast do not fail the operation when the internet access is failing.


[BUG] Min aggregate of a date in Transform Job

Describe the bug
When doing Min aggregate of a date field in Transform Job, it gives a long number in place of a date, it is very very disturbing because when I want to create an index pattern, it tells me that there is no field of type date.
For some indices it works as expected and for others it gives this problem. Why the way of working of min aggregate of date is not consistent.

@habid-altice Could you provide more details on this bug?

Can you share the versions of OpenSearch and any other relevant plugins you're using?
Are there specific configurations or mappings on the indices where you see the issue vs. where you don't?
A small sample or replica of your data could be very helpful.
Are there any error messages or logs that might provide more context?

Closing this for now. @habid-altice, please feel free to reopen if you're still experiencing this issue. Alternatively, you may want to create an issue in https://github.com/opensearch-project/index-management.


feat: add fractional noise generation via cholesky decomposition

@avhz I have applied the change let me note if you need further modifications. :)

@avhz you do mean, I should create a submodule fractional_brownian_motion as override the euler scheme in its own struct?

@avhz should I have to something more, because I would add more fractional processes :)


Deu party riverplot

added riverplot in separate subfolder (nothing changed besides README)

@lwarode fine with me and see minor revisions. I wonder if the partycoloR part can be optional: save colors in csv file, read from there and partycoloR dependency optional in if block only (as with "pf_pg_deu_merge.csv").

@lwarode fine with me and see minor revisions. I wonder if the partycoloR part can be optional: save colors in csv file, read from there and partycoloR dependency optional in if block only (as with "pf_pg_deu_merge.csv").


go1.8 and gb test with latest release (v0.4.3) do not play nice.

It fails with something like:
Given this fixture from the heroku buildpack I get this when using gb @ v0.4.3:
$ gb test
# cmd/fixture/testmain
_testmain.go:48: cannot use matchString (type func(string, string) (bool, error)) as type testing.testDeps in argument to testing.MainStart:
	func(string, string) (bool, error) does not implement testing.testDeps (missing MatchString method)
# sub/testmain
_testmain.go:56: cannot use matchString (type func(string, string) (bool, error)) as type testing.testDeps in argument to testing.MainStart:
	func(string, string) (bool, error) does not implement testing.testDeps (missing MatchString method)
FATAL: command "test" failed: exit status 2

This does not happen when I checkout master, recompile gb and re-run the tests.
Can we get a new release?

I think this is fixed with #692. Please reopen if this is not the case. A test case was added to the integration test suite based on the sample code in the error report.

Thanks, can I get a new release ? That was my main concern, to have a released version that supports go1.8. Thanks for all of your work!


Request - Different Taskbar Positions per Orientation

I use a Surface Pro with Windows 11. Frequently when in tablet mode (Portrait) I prefer the taskbar at the top so that it's easier to use the buttons (especially when laying down).
However, in most cases in laptop mode I would prefer the taskbar to be at the bottom (speciaically, so that I am not bothered by the bugs with switching between multiple windows of the same app).
It would be nice if the app supported different options per orientation.

Hi there! I think this option is definitely something that can be added in the application. Do you have suggestions on how to display this in de user interface? Currently I am thinking of adding a checkbox which allows the user to select their desired taskbar location for landscape and horizontal mode.
A requirement would be that you keep the application running in the background, so the orientation change can be detected.

Regarding suggestion for user interface, the one in Taskbar Shuffler 1.2  used for windows 10 is a good one.
If asks - for each of portrait and landscape,
choose -left/right/top/bottom - 4 checkboxes for portrait, and 4 checkboxes for landscape.

Hi there! Please note that I have not forgotten to fix the issue, there is some slight delay though.
I wanted to implement another function which adjusts the registry to disable the new context menu (which is shown on right mouse button click). However, something is modifying the corresponding registry key. I'm currently not sure if this is due to another application or Windows, so I'll have to create a virtual machine to be sure. I hoped to implement this feature alongside your suggestion for the next version. For this reason it might take a bit longer.

Hello! Thank you for considering this request. As a suggestion for a UI you could look at https://github.com/AutoDarkMode/Windows-Auto-Night-Mode - which has a similar UI where you can configure conditions.
However, perhaps the easiest way would be to keep it simple and have a "Landscape" and "Portrait" section, where the user could configure any of the options per orientation.
If you think in the future you may want to add more "conditions" (Plugged in, on battery, landscape, portrait, etc), then it may be worth the effort to create something a bit more flexible

Hi! Please note the feature has been implemented in the new release, which you can download here. Let me know if you have any suggestions.

@jetspiking Thank you for implementing this! I was having troubles getting it working due to the crashing bug (fixed in 5.0.2), but it works well now!
My only suggestions would be:

Find a way to have the application minimize to the tray - having it open all the time is fine, but showing the icon on the taskbar always can be intrusive
There should be an option to start with Windows (for the automatic orientation switching to work)
The taskbar seems to reload / rerender when moved - small nuisance, but would be nice if this could be addressed.

Hi there!
The first two issues are related to the fact that the application is portable. A solution for the first problem is to add a button that closes Taskbar11 and hides it on the taskbar. It can not be displayed in the tray-icon though, since the application needs to be a service for that (which makes the application less portable).
The second option could also be implemented. I will look into a solution to add the current assembly to the registry path for startup applications.
I do not have a solution for the third issue.
The taskbar reloads due to restarting the process "explorer.exe" after adjusting the registry settings. Currently I am not aware of a faster way to reload the taskbar.
I will look into the first two issues for the next version of Taskbar11.

Thanks for the quick response. Here's my suggestion:

Add a command line parameter: "windowless"
When registering the exe as a startup app you can have it pass the above parameter
When the user manual starts the exe, it checks to see if there's a running instance (make it a singleton application), and triggers that one to become visible
In the app you could perhaps also add a "close to background" checkbox


specification xml3d

I understand that the project has been discontinued. But I have several projects developed in xml3d. I can't find the specifications anywhere and the page where I was is no longer available. Is it possible that they can publish these specifications on github? At least the last two. Is there any other place where I can find them.

Hi juceo,
the specification document is also here in the repository:
https://github.com/xml3d/xml3d.js/tree/master/spec
The easiest way to access it would be to clone the repo and open it locally. The specification for older versions can be accessed by switching to the appropriate tag in the repository, eg git checkout 5.1.4 for v5.1.4


Added support for 05-15 stable API version

Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

cosmosDB

@evelyn-ys Other than the linter/styling issues, do you see any important omissions in the PR? I believe I have provided all the necessary changes related to the recent API version update, but just wanted to know if you see any other additions to this PR. I'll resolve the styling issues parallely.

Another question, in case there are no big issues with this PR and this PR gets ready for merge by today, when is the earliest the CLI package can be released for this update?

You can see the milestone for this sprint

@evelyn-ys Let me know if there are any other review comments as well.
Regarding milestone cycle, is it possible to make an OOB release for CLI?

If you want to bump API version, you need to re-record all the related tests because their recordings are using old API version. This is why CI fails. There's an internal pipeline you can leverage, see the bump version doc
I will review after all CI pass. CLI only releases hotfix version if there's critical blocking issue or security issue. Here's our OOB release doc

@evelyn-ys @necusjz please merge the PR. Thanks!


Page will not open when a Viewer contains an attribute which is not readable nor writeable

Thanks for the widget, it is beautiful!
When the viewer contains an attribute which is not readable nor writeable (so you don't have any access) it will block loading the entire page you try to open.
In Chrome Developer Tools it will give you the following error:
An error occurred while handling queued requests: mxui_widget_DataView_95.applyContext: Error: CKEditorForMendix_widget_CKEditorViewerForMendix_6.applyContext: Error: CKEditorForMendix_widget_CKEditorViewerForMendix_6.applyContext: TypeError: Cannot read property 'split' of null
Our expectation:
We would expect that you can't view the attribute value, but this would not block loading a page. You simply won't see any value (the normal behavior of an attribute when using access rules).
If you need some more info, please contact me.
Regards,
Dave

Can you test this version and let me know if this fixes your problem?

Hi Jelte,
It still not working unfortunately.
We created a testproject (modellere 6.5.1).
Explanation:

Log in as MxAdmin - 1
Create a new record
Go to demo user: demo_user
Click the buttons: 'Lees en schrijf', 'Lees', 'Geen rechten'.
The last one will not give any result.

Demo_user has no rights on the element which is shown in the form 'Geen rechten'.
Because the demo_user has no rights at all on this attribute, the page will not load and when you open the developer tools in Chrome you will see the error.
Hopefully this can help you fixing the issue.
Thanks!
Ps. unzip first, then you got the .mpk
Test app CKEditor.zip

Thanks. I'll have look at it this week and see if we can further fix it.

Due to Mendix World I am moving this issue back to later this week

Jelte, did you already talk with R&D about this issue?

Nope. Due to sick-leave (and my colleague being on vacation) I haven't had the time to resolve this issue. It's on my todo for next week

We have run into a similar issue in 6.9.1, where we receive the following error:
mxui.js?636128950893645105:29 mendix.sys.Data.get: error in handler for<PHONE_NUMBER>4364145: CKEditorForMendix_widget_CKEditorForMendix_0.applyContext: Error: CKEditorForMendix_widget_CKEditorForMendix_0.applyContext: TypeError: Cannot read property 'setReadOnly' of undefined


Backend change to install with separate command

ex:
latte install brew

Actually happening:
/bin/bash -c "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/HEAD/install.sh)"

Drafted due to #15

#17


Quick search Control + K not working on Windows

Current Behavior
Most likely need to preventDefault and/or stopPropigation for the events
Steps To Reproduce

Open in Chrome for Windows
Hit control + k
Expect to be in the site search
Instead you end up in the Chrome URL bar search

Anything else?
Tailwind docs allow this behavior, so it is possible.

I've opted for stopPropigation. We'll need to monitor this to ensure it doesn't introduce new issues, but should work as expected.


Virtual service validation doesn't know about "mesh" or Istio system resources

To reproduce: istioctl x analyze tests/e2e/tests/pilot/testdata/rbac/v1alpha1/istio-egressgateway.yaml
The output Referenced gateway not found: "mesh" is not valid.  https://istio.io/docs/reference/config/networking/v1alpha3/virtual-service/#VirtualService says the user may explicitly use mesh as one of the gateway names.
The output Referenced gateway not found: "istio-system/istio-egressgateway" seems bogus.  This is the conundrum I pointed out in https://github.com/istio/istio/pull/17410#issuecomment-535517844 .  If the user specifies Istio system resources without specifying --use-kube, they probably don't want to hear from Istio that it doesn't know about system resources.
The output Referenced host not found: "istio-egressgateway.istio-system.svc.cluster.local" has the same dubious quality as the previous.

The first issue should be fixed by https://github.com/istio/istio/pull/17733.

Revisiting this:
First issue is solved.
Second issue is still occurring. @esnible Is istio-system/istio-egressgateway a special value that is recognized even if no such Gateway CR actually exists in the cluster? If so, I can't find docs indicating that. The repro file defines an istio-egressgateway Gateway, but doesn't put it in istio-system specifically. I'm wondering how this is supposed to work, or if there's a test bug lurking here.
Third issue is not currently occurring when looking only at a file. It only comes up if looking at a file with --discovery=true explicitly set. This seems at least mostly correct.

When Istio is installed it can be installed with or without "Egress Gateway".  Most users do not install it.  The instructions for adding "Egress Gateway" are at https://istio.io/docs/tasks/traffic-management/egress/egress-gateway/#deploy-istio-egress-gateway
If the user analyzes with --use-kube then we want analyze to report an error for tests/e2e/tests/pilot/testdata/rbac/v1alpha1/istio-egressgateway.yaml.
We need a rule of thumb for what to do about unsatisfied references in istio-system.  I see three possibilities:

Do nothing, user sees Referenced gateway not found: "istio-system/istio-egressgateway"
Suppress all references to things in istio-system if --use-kube is ignored
Have a special analysis code: INFO IST0112 Skipped check of reference to control plane; use --use-kube that replaces all errors related to the Istio control plane without getting info from a live cluster.

Thanks for the clarification.
Setting aside for a moment the "rule of thumb" question, I feel like the test YAML is slightly buggy. It includes an entry that defines istio-egressgateway, but the one that it defines ends up in namespace default, and never gets referenced. If the test file is updated so that the istio-egressgateway Gateway is in namespace istio-system, then we don't actually generate an error.
If I'm right about this, it mostly highlights that there are cases where we'd want to show this error even if --use-kube is off.

Another option to consider: when --use-kube is off, we could include a limited set of default istio-system resource definitions. This would involve less specialized plumbing and would prevent the analyzers from needing to know the state of --use-kube.
The risks are that this could become painful if the default definitions end up changing between versions, or if we need to handle different configuration options. If we're reasonably confident that this is unlikely to happen for the resources we hard-code, then it may be worth it.
I believe the only ones we need so far are istio-ingressgateway and istio-egressgateway.

It makes sense to provide some default resource definitions if --use-kube is off.  It is especially important for pipelines to be able to validate generated YAML without an actual cluster available.
I have defaultIngressGatewayPorts in galley/pkg/config/analysis/analyzers/gateway/gateway.go and that is the kind of stuff that should be in the default.
I am not clear on how we generate istio-system resources somewhere else for users who want to deploy Istio to a different namespace.  Not sure it is needed at this point.  istioctl has a command line argument for that, but not for various other deployment time settings.   It seems like too much to pass in a IstioControlPlane custom resource for this purpose, but that would be very general.

The tricky aspect here is that we don't have a good way to distinguish between the following scenarios:

The user ran with files only, and didn't include any istio resources
The user ran with files only, including istio resources (e.g. their istio manifest), but they had configured it with the egress gateway disabled.

In the first case, we probably wouldn't want to report an error if something referenced the default egress gateway, but in the second case, we would.
If we included a default egress gateway, we'd avoid the false positive in the first case, but then fail to generate a a warning in the second case.

I should note that as of 1.4 the Referenced gateway not found: "istio-system/istio-egressgateway" message no longer appears, and Referenced host not found: "istio-egressgateway.istio-system.svc.cluster.local" only appears if --discovery is enabled. I think the current behavior is reasonable for now, so I'm going to close this out.


icons: Replace previous bot icon with new single antenna bot icon.

This PR replaces the previous bot icon with new single antenna bot icon, and also cleanups the redundant .fa-group icon related styling.
As suggested by @karlstolley in https://github.com/zulip/zulip/pull/31350#issuecomment-2332434467, I went ahead and tweaked the svg itself, lifting the icon by 2px – which resulted in the removal of the previously required adjustments to compensate for the bottom heavy bot icon.
Fixes: CZO#1 and CZO#2

Screenshots and screen captures:

Before
After

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Looks good to me in screenshots and light manual testing!

Merged, thanks @sayamsamal and @alexmv!


[core][java-convert] Method decompilation failed - missing block/lost code

This sample can't be decompiled with java-convert plugin. A missing block with lost code is reported. My test was made with openjdk8 and latest git version:
public class MissingBlockTest {
  private volatile boolean flag;

  public String test() {
    String result = "Hello";
    if (flag) {
      doSomething();
      if (!result.isEmpty()) {
        return result;
      }
    }

    if (flag) {
      doSomething();
    }
    return result;
  }

  private void doSomething() {
  }
}

Java-input plugin works fine. This sample can be used for unit tests.

@nitram84 thanks. This issue similar to code-duplication issues, here return should be duplicated to allow code restructure. Java-input plugin works fine because java bytecode keeps several return instructions, unlike dx, which merge all returns in one, so jadx trying to guess how to split them.


ISPN-9814 Publisher forEach conversion

https://issues.redhat.com/browse/ISPN-9814
The actual changes are in the first commit. The second is removing all the now unneeded code.

+322 −9,053
That is quite awesome :)

Yeah, I have finally been able to remove all the old distributed stream code :)

@wburns I think AbstractCacheStream::supplierForSegments and ::nonNullKeyFunction can be removed now that TxDistributedCacheStream has gone.

Removed and rebased.

Thanks @wburns


Update https://github.com/Alexpux/MSYS2-packages/blob/master/mintty/PKGBUILD file

Hi please update https://github.com/Alexpux/MSYS2-packages/blob/master/mintty/PKGBUILD file because mintty is no longer active on google code and have moved to GitHub where they have started developing it again. new link for mintty is https://github.com/mintty/mintty

Done by https://github.com/Alexpux/MSYS2-packages/commit/dc548f4df607ffe5064986c4ee65714ecf3563ba

Thanks.


Fixing tsr imports via absolute_import

To try and address issue #9

I've added absolute_import and corrected the other call to TSRLibrary which @pengjujin discovered.

Thanks! Merging.


redirect to home

Why is the login action redirecting me to the home instead of the last visited page?
Any ideas?
PS: I read here in some older issues that this should work, but not for me.
Can somebody please check that this is supposed to work right now?

Do you mean that you are not redirected back to the page where access for guests is forbidden after loggin in? Or you mean that if you click login link from any page and after logging in you're not redirected back? If the latter, then there is nothing wrong, because you're not supposed to be redirected back. If the former, then I don't know really, I've never experienced such issue.

You are right. I tought that it will return me to the page where I clicked on the login button.
I am now storing the referer in a session var for this purpose.
Thanks for your help.


Failed to Refresh Profile

failed to refresh profile with gh data:  Error: timeout of 30000ms exceeded

Duplicate of https://github.com/sourcerer-io/sourcerer-app/issues/558.


When loading higher volume of data for two graphs at the same time, one of the graphs doesnt render properly

The data loaded is an array of 10 columns with 300 values.
Please see the code below, if I set the timeout for redrawGraph2 both of the graphs render correctly. Otherwise without it only one renders correctly, the other is blank(but the axis is rendered ok, and when zoomed the columns appear). I also tried to call the redrawGraph2 function via done: or onrendered:, but it didn't help.
Can you please provide me more info or help? thank you
function redrawGraph(data, graphType) {
    redrawGraph1(data.1, graphType);
    setTimeout(function(){redrawGraph2(data.2, graphType);}, 2000);
}

function redrawGraph1(data, graphType) {
    chartRX.load({
        columns: data,
        type: graphType,
        unload: true,
    });
}

function redrawGraph2(data, graphType) {
    chartTX.load({
        columns: data,
        type: graphType,
        unload: true
    });
}

Could you please provide a JSfiddle? I'm not entirely sure what's going on with your code. Thanks!

Hi @aendrew ,
Here is the fiddle: https://jsfiddle.net/koLavwgn/20/
If you click on the button it will generate a 1000 data points for each set of data and will load that into the graph. Although it renders only one chart correctly. The other is redrawn after clicking on it and trying to zoom in.
Thanks for help
Peter


Aurora R7 won't show light control options.

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:
Open program and the lights tab doesn't show up.
Expected behavior
A clear and concise description of what you expected to happen. Should show a light control tab and it doesn't.
Screenshots
If applicable, add screenshots to help explain your problem.

System (please complete the following information):

Model: Alienware Aurora R7
Model details: Tower
OS: Windows 11 Stable Release

Correct.
R7 and R6 the only uncommon systems which have ACPI-controlled lights (all other hardware have USB-controlled).
Unfortunately, it's still under development for latest Fan SDK V2 (yes, lights controlled by fan subsystem at this gear), but you can use Fan V1 version i provide among other release files - it supported there (BTW, this fact mentioned into every release notes).
Please read readme notes into V1 archive first!

I'm sorry.  I totally read the readme but I didn't understand which version I needed sorry!

It's OK.
 Check WIKI page why it happened.
You can help me to support it into newest SDK, if you have time.

I have the same issue with the Alienware 15 R4. Should also go back?

No, R4 have 2 USB devices, so the source of your issue is different. Is it visible in Device Manager? What alienfx-cli probe prints?

the -cli.exe files have refused to function for me regardless of the version I've used.

Ah, I joined the discord cuz I figured something like this you'd rather deal with there and saw your pinned message. If you aren't able to respond for a while I totally get it. I'll just keep troubleshooting on my side. I'm Luck13 on the discord if it's easier there.

CLI = Command Line Interface. So first open CMD first (any for FX, Administrator for Fan) then run it from it - you will see the log.
But in case we are talking about AlienFX-CLI - this is bad news for you - most probably, you will see something like "Both high and low-level devices not detected, quit" - this mean no light chip detected among PnP hardware, so you need to check what you have into Device Manager.

Have command prompt open as I try to run AlienFX-CLI ? Tried that even running both as administrator. I don't see "Both high and low-level devices not detected, quit". I don't see anything. It's as if nothing even tried to run. Nothing in the CMD either. Where am I looking in Device manager and what for. Although somehow I doubt it has to do with recognized devices because on 8.1.2 which I'd been running previously I could control my lighting. I even tried rolling back to 8.1.2 but... would you rather make a thread in the discord support server? I have a feeling this problem is gonna hide from us.

It randomly fixed itself?! Uhm. Problem solved?

Check about restart/reboot.
Yes, you can lose devices not because of my app, but because Windows update and hardware fault.
About device manager - you should have "Vendor defined device" among "Human interface devices" with VID_187C&PID_0550 - this is tron lights. Check among "Unknown devices" as well. I don't remember exact keyboard lights device IDs, but it should be one with VID_0d62.
PS: This is well-known bug about monitor can hang both AFX and AWCC in some cases. Also, it's a way to reset light system - see afx readme ("Known issues").


Add support for AWS Dynamo DB for terraform testing infrastructure

Add permissions and terraform files to support conformance and certification tests for Dynamo DB component

@addjuarez  I wonder if this PR: #2384  took care of this issue. In there, I added the Terraform, Conformance and Certification Tests for DynamoDB.

Yes I think your PR addressed this @robertojrojas - assuming there were no permissions problems (which there don't appear to be).


Set up autoUpdater

Needs review. Closes #26

This requires the app to be signed for autoUpdater to work on mac. This will require getting an Apple Developer account


Maintenance

I think this is an interesting project but a lot of PRs and issues don't seem to get the attention they need which usually leads to contriutor frustration, I'd be willing to contribute but not if things won't get merged eventually.
@davidchua are you the only maintainer? @Cretezy seems to be quite active too.
Why are the builds failing on master and nobody seems to care?
And finally is it worth my time to contribute or should I fork it? I could possibly help maintaining things related to features I need but I can't make promises at this stage.

I'm going to be forking this project soon. Stay updated

https://github.com/Cretezy/pymessenger2/


Remove dead code

This is 7-year old CMake code that never did anything because the function invocation is missing the PROPERTIES keyword. Adding the keyword results in build errors, so just drop the dead code since everything seems to be working without it anyway.

Merging as discussed in the April 16th teleconference.


Fix broken shiet

[ ] Crashes & remaining issues with message embeds
[ ] Channel DM special case
[ ] Make component rerender on settings changes so you don't have to swap channels

isnt the rerendering issue fixed?

isnt the rerendering issue fixed?

Not the settings preview, the actual quotes in the channel you are currently on, a rerender does not occur until you swap channels (after closing settings).


[error] node.js ERR_MODULE_NOT_FOUND

I've tried running
import init, { planetary_system } from 'accrete-wasm';
async function run() {
    await init();
    const seed = 1;
    const stellar_mass = 1;
    const output = planetary_system(BigInt(seed), stellar_mass);
}
run();

but I got
node:internal/errors:464
    ErrorCaptureStackTrace(err);
    ^

Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'C:\Users\jayki\Documents\Open Source\space-game\node_modules\accrete-wasm\' imported from C:\Users\jayki\Documents\Open Source\space-game\index.js
    at new NodeError (node:internal/errors:371:5)
    at legacyMainResolve (node:internal/modules/esm/resolve:336:9)
    at packageResolve (node:internal/modules/esm/resolve:917:14)
    at moduleResolve (node:internal/modules/esm/resolve:976:18)
    at defaultResolve (node:internal/modules/esm/resolve:1078:11)
    at ESMLoader.resolve (node:internal/modules/esm/loader:530:30)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:251:18)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:79:40)
    at link (node:internal/modules/esm/module_job:78:36) {
  code: 'ERR_MODULE_NOT_FOUND'
}

It seems it's unable to detect the accrete.js file.

I will look into it soon. Thank you!
Probably some packaging issues...
For now you can just download “pkg” folder from this repo and import accrete.js from there (check examples/wasm/index.html).

@Jaykingamez Just to clarify: you trying to run script from node or in browser?

Script, a simple node index.js

@Jaykingamez ok, thank you! I will rebuilt package so you will be able to use it in nodejs and let you.

@Jaykingamez Please update to<EMAIL_ADDRESS>and check an example (you have to import methods from accrete-wasm/accrete-node):
https://github.com/LeonidGrr/accrete/blob/master/examples/nodejs/index.js


Encode/decode fields

I ran into this problem where I wanted to store a record in NSData so I could temporarily cache it and share among targets (ie. the app and a today extension) but none of the fields were populated anymore.
So I added an iterator to loop over the fields to the encoder/decoder of the resource and store/load those values.

Thanks for your PR Roy! I've thought about adding such functionality multiple times before. Every time I decide not to do so though. I think the base Resource class should be responsible for decoding/encoding its own properties. It's up to the developer to decide and implement the coding for subclasses. Perhaps not every field should be encoded, or perhaps they should be encoding in a different way. This is pretty much how it's done all throughout the Cocoa frameworks, you need to implement your own coding, and call super in both methods.
I appreciate your work but I don't think I'll merge it at this point. If you want this behaviour you can subclass Resource and the subclass that into concrete resource classes.

That's a fair reasoning. I'll subclass Resource then :) Thanks!

I seem to be having problems subclassing resource.
I created a new file, lets say MyResource.swift with the following contents:
import Foundation
import Spine

public class MyResource : Resource {

    public required init(coder: NSCoder) {
        super.init(coder: coder)

        for field in fields {
            if coder.decodeObjectForKey(field.name) != nil { // here
                setValue(coder.decodeObjectForKey(field.name), forField: field.name) // here
            }
        }
    }

    override public func encodeWithCoder(coder: NSCoder) {
        super.encodeWithCoder(coder)

        for field in fields {
            coder.encodeObject(valueForField(field.name), forKey: field.name) // here
        }
    }
}

However it complains that Value of type 'Field' has no member 'name' while following fields clearly reveals it uses the Field class, which does have the property name, this exact same code worked while it was in your Resource class.
Any ideas what I'm missing here?

The name variable is internal to the Spine module, so it's not visible in your application code. You can use the serializedName variable.

Okay, making that public solved one issue. But in my own class I had to "override" the init(coder) function (without actually using the word override) but now it expects me to add the coder parameter to all places where I call my resource (ie. Event)
spine.registerResource(Event.resourceType) { Event() }

throws Missing argument for parameter 'coder' in call

I managed to fix the above issue by overriding the regular init
public override init() {
    super.init()
}

Instead of making serializedName public, please make name public because name is the one that is used to store the data in the model whereas serializedName is the one used in the JSON response and doesn't necessarily correspond to the key in the model.

I'm thinking about getting rid of the name variable as a whole. It don't that that it is an implicitly unwrapped optional to support fieldsFromDictionary. I think changing Resource.fields to [String: Field], getting rid of fieldsFromDictionary is the way to go.


Sidebar isn't fixed

I think the sidebar is meant to be fixed. I think you need to change position:absolute to position:fixed. This would match the style of your aviator docs too.

It depends on the usecase, we've tried to make it flexible so it can be used in both ways.


Websocket Connections close after 1 minute for user channel

Coinbase Pro Asyncronous Websocket Client version:
Python version:3.6
Operating System: Amazon Linux

Description
When connecting to the user channel, there is no data instantly, in that case, the websocket connection shuts down and reconnects after 1 minute and just continuously reconnects every 1 minute. Is there a keep-alive that can be implemented or heartbeat to ensure that the connection is open?

This can definitely be done, and I will look into adding into the client initialization, but in the meantime, I believe the following should work. I have not tested it yet, so I could be wrong.
copra.websocket.client.Client extends autobahn.asyncio.websocket.WebSocketServerFactory.  The autobahn base class allows for really fine grained control of the protocol that manages the connection. You should be able to use setProtocolOptions() to configure the ping.  You would have to do this before the client is added to the event loop. As it stands the protocol is a singleton created when Client.add_as_task_to_loop() is run, and setProtocolOptions() must be run before the protocol is initialized. To ensure this, the client would need to be initialized with auto_connect = False.
The process would look something like:
import asyncio
from copra.websocket.client import Channel, Client

KEY = YOUR_API_KEY
SECRET = YOUR_API_SECRET
PASSPHRASE = YOUR_API_PASSPHRASE

loop = asyncio.get_event_loop()

channel = Channel('user', 'BTC-USD')

client = Client(loop, channel, auth=True, key=KEY, secret=SECRET, passphrase=PASSPHRASE, auto_connect=False)

client.setProtocolOptions(autoPingInterval=30)

client.add_as_task_to_loop()

try:
    loop.run_forever()
except KeyboardInterrupt:
    loop.run_until_complete(client.close())
    loop.close()

Again, I haven't yet tried this, but it should work. If you do try it, I would greatly appreciate your feedback. I will move forward with adding the option of configuring the ping during client initialization, but hopefully this will be a bandaid until I get it integrated.

Update: I have tested the above solution and it seem to work.
An alternative to using a ping would be to subscribe to the heartbeat channel. It may be simpler to set it up this way, and it will keep your connection open, but it is also pushing data to you once per second that you may not need.

I tried something different which worked too. Just a ping to keep connection alive.
Added this to the ClientProtocol class:
def schedule_keepalive(self):
        loop=asyncio.get_event_loop()
        loop.call_later(30,self.sendPing)

def sendPing(self, payload = None):
        if self.state != WebSocketClientProtocol.STATE_OPEN:
            return
        if payload:
            l = len(payload)
            if l > 125:
                raise Exception("invalid payload for PING (payload length must be <= 125, was %d)" % l)
            self.sendFrame(opcode = 9, payload = payload)
            self.schedule_keepalive()

Changed the onOpen Function to this.
def onOpen(self):
        print("Websocket Open")
        self.last_ping_time = time.time()
        self.schedule_keepalive()
        self.factory.on_open()

Thank you for sharing your workaround. Any of these three methods should work to keep the connection alive. I am going to continue forward with implementing ping configuration in the client initialization since it makes the most sense to use the underlying functionality provided by autobahn. Since there already are multiple, if not convenient, ways to skin this cat, I am going to close this issue.


[mesheryctl] incorporate traffic.proto into perf test using SMP config

Yes, we will likely need a traffic proto. For now, HTTP GET, gRPC ping, and TCP open/close can be supported by logic in mesheryctl only.
Originally posted by @leecalcote in https://github.com/layer5io/meshery/pull/1156

Done


properly store binaries with kviator

kviator can't store binary data (see ceph/ceph-docker #220 ).  This works around that limit via sharutils and uuencoding/decoding the monmap.

Just a minor comment other than that LGTM.
Thanks @fmeppo!

Cleaned up, should be good.

I know it's related to your patch and we can fix that later but I still wonder why we have installation step into the entrypoint.
We can fix that another PR though.
Thanks!


CLI: Warn when input file does not exist

Current behavior
In  CLI when referencing a file that does not exist, the linter finishes without any errors.
Expected behavior
Warning message about missing input file.

You will not be able to do this if it is a glob pattern, unfortunately.


improve search parameter registry logging

also wrap resource search parameter map

Very nice. I understand what you meant about the wrapped map being more readable now that I've seen an example.


AES standby test still having old org interface to check BMC state at one place.

AES standby test having old org interface to check BMC state. This is due to "Reboot and Wait for BMC Online" keyword used here. Seem like we can remove this check as we have new interfaces in place.
    ${rest_status}=  Run Keyword And Return Status  Initialize OpenBMC
    Run Keyword If  '${rest_status}' == '${False}'
    ...  Reboot and Wait for BMC Online

openbmc/openbmc-test-automation#375


Open To LAN

I've observed (probably because of Open To LAN) that sometimes upgrades disappear, like if another player on the LAN looks in the surgery box before I go in.  Is Open To LAN supported, and if so, what should we avoid to prevent things disappearing (i.e.:  I have a second set of my eyeballs, even though the cyber-eye upgrade is gone, and at one point, it looked like the other player had 'stolen' a cardio-power upgrade)

I'm seeing that when you die all your upgrades go away.  If this is intentional, then I suppose it might explain what appeared to be upgrades disappearing.


[GSSOC' 23 ]  HOVERING IS NOT WORKING FOR TWO BUTTONS

I  am an Open source Contributor
DESCRIPTION:-
The hovering effect of the Instagram and LinkedIn icons  aren't working in the site so please give me an opportunity to correct those and also I would like to change the ui a bit to make it more impressive.

I CAN FIX THIS AND MAKE THIS MORE GOOD WITH GOOD ICONS
CODE OF CONDUCT
-I Follow Contributing Guidelines of this project

Hello, I already worked on a food recipe app, so I'm good to handle this issue. please assign me this issue.
I'm GSSOC'23 contributor.


How to resume the training with FCOS3D

Hi, I've just train a job with 6 epochs, the 6th.pth has been saved. But it seems that our model still need more epochs to train, I wanna know how to start a training process with 6 epochs as the start. Thanks a lot.

you can add resume_from = '/nfs/fcos3d/epoch_6.pth' (your checkpoint dir) in your config.


Change hotkey defaults using "Ctrl" to "Meta" for users on MacOS

Is your feature request related to a problem? Please describe.
MacOS behavior dictates the "Meta" key be used in place of the "Ctrl" key in many commands where "Ctrl" is used comparably on Linux or Windows
Describe the solution you'd like
Change the client defaults for hotkeys to use "Meta" instead of "Ctrl" on MacOS. Nothing should change for existing users.
Describe alternatives you've considered
Ask users if they'd like to switch on next client start, if they're using MacOS.

Arent the hotkeys already configurable? Or idk what do you mean

This suggestion is specifically addressing the defaults being set by RL on first-time-run. Yes the settings can be changed, but imo this would make runelite follow macOS best practice without user intervention

Doing OS-specific configuration is very bad idea unless there is very good reason to do it (and this isnt) because the configuration is synced (and so, if you would just use meta for OSX on first run, it will simply not work at all on synced windows machine)

I had forgotten about hotkey syncing in the client. While non-trivial, it may be worth thinking on the issue as the CTRL key on Mac can be more awkward, but obviously this issue is in no way urgent.

Imo as long as it is configurable, its fine as it is now


Due date

Overview

Do not show expired tasks
Add expires today (red)
Add expires tomorrow (red)
Add expires in n days (red, within 7 days)
Add n days left (black)

Screenshots

 Improve how the due date is displayed on the cards


Convert pdf into markdown

this one https://github.com/OWASP/user-security-stories/blob/master/user-security-stories.pdf

If you would be up for some help, I could take a stab at this for yah.

I foked the repo the other day with the intention of doing this, but @obihann feel free to take a stab at it if you want!


Integrate MemGPT

Feature request
Implement MemGPT for better memory management with local and hosted models:
https://github.com/cpacker/MemGPT
Motivation
This can help automate the manually creation and retrieval of vector stores.
Your contribution
I'm not really familiar with the langchain repo at the moment, I could provide feedback on the implementation from a users perspective.

@torrinworx can you give an estimate on how long this would take to complete? Assuming I have a few people to work with and this will be our first time contributing to langchain

@HassanA01 I'd guess it would take about a week if we had 3 developers who are familiar with LLM's and strong python programming skills. I have time this week if you are still interested in working on this.

@hwchase17 any opinion on this? is memgpt & langchain both are differnt ? is there any plans to support the memgpt with langchian?

Highly interested also. Would appreciate any update if implentatation is in pipeline(?)

Closely watching as well, memgpt seems to be a very interesting architecture.

https://memgpt.readme.io/docs/python_client
MemGPT has a python client. Someone should be able to leverage this with LangChain's memory system. Don't think it would be too hard.

hey, this integration with MemGPT seems essential


Radar showing multiple arrows for same hunt

When first detecting a hunt Radar works fine. After detection if you move out of ACT detection range, then back into it, it places another non functioning arrow on the overlay.
https://imgur.com/a/ly1hXn8
Edit: This arrow does not clear on zone swapping.

The code doesn't lowercase the mob name properly, so it doesn't see its already on the list. I fixed this by adding the following just before line 96: if (!(mob_name in this.targetMonsters)) {
this.RemoveMonster(mob_name.toLowerCase());
Added benefit, it pops up the latest ping to the top with this.

That's in cactbot/ui/radar/radar.js that is. Passable workaround until its fixed.

Thanks! Yeah, the lowercasing was a change right before the cactbot release and I clearly didn't test it enough.  Thanks for the pointer to the bug.


Unable to install pendulum==2.1.2 using uv

uv -v pip install pendulum==2.1.2 seems to error for me in a couple different environments.
I first noticed the error in my macOS venv. I was then able to reproduce it in a Docker container
The sequence of steps that reproduced the error for me on an Apple Silcon MacBook Pro were as follows. I also have attached the terminal output as well.
Reproducible Example

Create an interactive session in a basic Python 3.12 Docker container.

docker run -it --rm python:3.12 /bin/bash
# error also occurs when emulating x86
docker run -it --rm --platform linux/amd64 python:3.12 /bin/bash

Install the latest version of uv using the official installer script

curl --proto '=https' --tlsv1.2 -LsSf https://github.com/astral-sh/uv/releases/download/0.2.8/uv-installer.sh | sh
source ~/.cargo/env

Note: This also occurs in version 0.2.5

Create a uv venv and activate it

uv venv && source .venv/bin/activate

Fail to install pendulum==2.1.2

uv pip install pendulum==2.1.2

Output of Reproducible Example
❯ docker run -it --rm python:3.12 /bin/bash
root@f6719cf5dd68:/# curl --proto '=https' --tlsv1.2 -LsSf https://github.com/astral-sh/uv/releases/download/0.2.8/uv-installer.sh | sh
downloading uv 0.2.8 aarch64-unknown-linux-gnu
installing to /root/.cargo/bin
  uv
everything's installed!

To add $HOME/.cargo/bin to your PATH, either restart your shell or run:

    source $HOME/.cargo/env (sh, bash, zsh)
    source $HOME/.cargo/env.fish (fish)

root@f6719cf5dd68:/# source ~/.cargo/env
root@f6719cf5dd68:/# uv venv
Using Python 3.12.3 interpreter at: usr/local/bin/python3
Creating virtualenv at: .venv
root@f6719cf5dd68:/# source .venv/bin/activate
(.venv) root@f6719cf5dd68:/# uv -v pip install pendulum==2.1.2
DEBUG Searching for Python interpreter in virtual environments
DEBUG Found CPython 3.12.3 at `/.venv/bin/python3` (active virtual environment)
DEBUG Using Python 3.12.3 environment at .venv/bin/python3
DEBUG Acquired lock for `.venv`
DEBUG At least one requirement is not satisfied: pendulum==2.1.2
DEBUG Using registry request timeout of 30s
DEBUG Solving with installed Python version: 3.12.3
DEBUG Adding direct dependency: pendulum==2.1.2
DEBUG No cache entry for: https://pypi.org/simple/pendulum/
DEBUG Searching for a compatible version of pendulum (==2.1.2)
DEBUG Selecting: pendulum==2.1.2 (pendulum-2.1.2.tar.gz)
DEBUG No cache entry for: https://files.pythonhosted.org/packages/95/13/e7598039a7c2be1c74538125c035f866189897291f12da029a2a1008dc3e/pendulum-2.1.2-cp27-cp27m-macosx_10_15_x86_64.whl.metadata
DEBUG Adding transitive dependency for pendulum==2.1.2: python-dateutil>=2.6, <3.0
DEBUG Adding transitive dependency for pendulum==2.1.2: pytzdata>=2020.1
DEBUG No cache entry for: https://pypi.org/simple/python-dateutil/
DEBUG No cache entry for: https://pypi.org/simple/pytzdata/
DEBUG Searching for a compatible version of python-dateutil (>=2.6, <3.0)
DEBUG Selecting: python-dateutil==2.9.0.post0 (python_dateutil-2.9.0.post0-py2.py3-none-any.whl)
DEBUG No cache entry for: https://files.pythonhosted.org/packages/ec/57/56b9bcc3c9c6a792fcbaf139543cee77261f3651ca9da0c93f5c1221264b/python_dateutil-2.9.0.post0-py2.py3-none-any.whl.metadata
DEBUG Adding transitive dependency for python-dateutil==2.9.0.post0: six>=1.5
DEBUG No cache entry for: https://pypi.org/simple/six/
DEBUG Searching for a compatible version of pytzdata (>=2020.1)
DEBUG Selecting: pytzdata==2020.1 (pytzdata-2020.1-py2.py3-none-any.whl)
DEBUG No cache entry for: https://files.pythonhosted.org/packages/e0/4f/4474bda990ee740a020cbc3eb271925ef7daa7c8444240d34ff62c8442a3/pytzdata-2020.1-py2.py3-none-any.whl.metadata
DEBUG Searching for a compatible version of six (>=1.5)
DEBUG Selecting: six==1.16.0 (six-1.16.0-py2.py3-none-any.whl)
DEBUG No cache entry for: https://files.pythonhosted.org/packages/d9/5a/e7c31adbe875f2abbb91bd84cf2dc52d792b5a01506781dbcf25c91daf11/six-1.16.0-py2.py3-none-any.whl.metadata
DEBUG Tried 5 versions: pendulum 1, python-dateutil 1, pytzdata 1, root 1, six 1
Resolved 4 packages in 845ms
DEBUG Identified uncached requirement: pendulum==2.1.2
DEBUG Identified uncached requirement: python-dateutil==2.9.0.post0
DEBUG Identified uncached requirement: pytzdata==2020.1
DEBUG Identified uncached requirement: six==1.16.0
DEBUG No cache entry for: https://files.pythonhosted.org/packages/e0/4f/4474bda990ee740a020cbc3eb271925ef7daa7c8444240d34ff62c8442a3/pytzdata-2020.1-py2.py3-none-any.whl
DEBUG Acquired lock for `/root/.cache/uv/built-wheels-v3/pypi/pendulum/2.1.2`
DEBUG No cache entry for: https://files.pythonhosted.org/packages/ec/57/56b9bcc3c9c6a792fcbaf139543cee77261f3651ca9da0c93f5c1221264b/python_dateutil-2.9.0.post0-py2.py3-none-any.whl
DEBUG No cache entry for: https://files.pythonhosted.org/packages/d9/5a/e7c31adbe875f2abbb91bd84cf2dc52d792b5a01506781dbcf25c91daf11/six-1.16.0-py2.py3-none-any.whl
DEBUG No cache entry for: https://files.pythonhosted.org/packages/db/15/6e89ae7cde7907118769ed3d2481566d05b5fd362724025198bb95faf599/pendulum-2.1.2.tar.gz
DEBUG Downloading source distribution: pendulum==2.1.2
DEBUG Building: pendulum==2.1.2
DEBUG Solving with installed Python version: 3.12.3
DEBUG Adding direct dependency: poetry-core>=1.0.0a9
DEBUG No cache entry for: https://pypi.org/simple/poetry-core/
DEBUG Searching for a compatible version of poetry-core (>=1.0.0a9)
DEBUG Selecting: poetry-core==1.9.0 (poetry_core-1.9.0-py3-none-any.whl)
DEBUG No cache entry for: https://files.pythonhosted.org/packages/a1/8d/85fcf9bcbfefcc53a1402450f28e5acf39dcfde3aabb996a1d98481ac829/poetry_core-1.9.0-py3-none-any.whl.metadata
DEBUG Tried 2 versions: poetry-core 1, root 1
DEBUG Installing in poetry-core==1.9.0 in /root/.cache/uv/.tmpIyOnb6/.venv
DEBUG Identified uncached requirement: poetry-core==1.9.0
DEBUG Downloading and building requirement for build: poetry-core==1.9.0
DEBUG No cache entry for: https://files.pythonhosted.org/packages/a1/8d/85fcf9bcbfefcc53a1402450f28e5acf39dcfde3aabb996a1d98481ac829/poetry_core-1.9.0-py3-none-any.whl
DEBUG Installing build requirement: poetry-core==1.9.0
DEBUG Calling `poetry.core.masonry.api.get_requires_for_build_wheel()`
DEBUG Calling `poetry.core.masonry.api.build_wheel("/root/.cache/uv/built-wheels-v3/pypi/pendulum/2.1.2/ByoYWHVpQUGO2IqO_ZlPb/.tmplVuW65", {}, None)`
error: Failed to download distributions
  Caused by: Failed to fetch wheel: pendulum==2.1.2
  Caused by: Failed to build: `pendulum==2.1.2`
  Caused by: Build backend failed to build wheel through `build_wheel()` with exit status: 1
--- stdout:

--- stderr:
Traceback (most recent call last):
  File "/root/.cache/uv/built-wheels-v3/pypi/pendulum/2.1.2/ByoYWHVpQUGO2IqO_ZlPb/pendulum-2.1.2.tar.gz/build.py", line 5, in <module>
    from distutils.command.build_ext import build_ext
ModuleNotFoundError: No module named 'distutils'
Traceback (most recent call last):
  File "<string>", line 11, in <module>
  File "/root/.cache/uv/.tmpIyOnb6/.venv/lib/python3.12/site-packages/poetry/core/masonry/api.py", line 58, in build_wheel
    return WheelBuilder.make_in(
           ^^^^^^^^^^^^^^^^^^^^^
  File "/root/.cache/uv/.tmpIyOnb6/.venv/lib/python3.12/site-packages/poetry/core/masonry/builders/wheel.py", line 88, in make_in
    wb.build(target_dir=directory)
  File "/root/.cache/uv/.tmpIyOnb6/.venv/lib/python3.12/site-packages/poetry/core/masonry/builders/wheel.py", line 123, in build
    self._build(zip_file)
  File "/root/.cache/uv/.tmpIyOnb6/.venv/lib/python3.12/site-packages/poetry/core/masonry/builders/wheel.py", line 172, in _build
    self._run_build_script(self._package.build_script)
  File "/root/.cache/uv/.tmpIyOnb6/.venv/lib/python3.12/site-packages/poetry/core/masonry/builders/wheel.py", line 262, in _run_build_script
    subprocess.check_call([self.executable.as_posix(), build_script])
  File "/usr/local/lib/python3.12/subprocess.py", line 413, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['/root/.cache/uv/.tmpIyOnb6/.venv/bin/python', 'build.py']' returned non-zero exit status 1.
---
(.venv) root@f6719cf5dd68:/#

uv Info

macOS: uv 0.2.5, arm64
Linux: uv 0.2.8, arm64

Hi @mihirsamdarshi
I believe the problem is that distutils was removed in Python 3.12 so the Pendulum build script fails to import it.

👍 I don't think pip is able to install this either. It's an issue with the pendulum package itself.

Thanks, my bad, I should've check that

No prob.


Creating a Issue

My First in GitHub. Creating a new issue

Issue Created Successfull. Closing it now


Follow-up new member

Summary
When project submission is accepted, adding a comment about what the project maintainer can to follow-up the Ecosystem.
Details and comments

[x] Create new job in submission wf
[x] Tests run

Closes #525

@frankharkins ready for your review :)
(I'll clean my fork for future PR ^^')


Android crashes on .stopStreaming()

Describe the bug
Using the example code in the readme, streaming works fine but as soon as I stop recording the app crashes.
To Reproduce
Steps to reproduce the behavior:

Use example code
Press record button to stop streamin

Expected behavior
The app to not crash
Smartphone (please complete the following information):

Device: Realme 8
OS: 12

Additional context
I'm using Expo, consistently works for starting the stream but as soon as I stop it crashes. Majority of the time it crashes on the first time I stop streaming, very occasionally I'll stop streaming and it'll be fine, I then start and stop again and it crashes

log.txt
log file attached

import React, { useCallback, useEffect, useRef, useState } from 'react';
import { View, TouchableOpacity } from 'react-native';
import { LiveStreamView } from<EMAIL_ADDRESS>
const RecordStream = () => {
  const ref = useRef(null);
  const [streaming, setStreaming] = useState(false);

  return (
    <View style={{ flex: 1, alignItems: 'center', backgroundColor: 'grey' }}>
      <>
        <LiveStreamView
          style={{ flex: 1, backgroundColor: 'black', alignSelf: 'stretch', backgroundColor: 'lightgrey' }}
          ref={ref}
          camera='back'
          video={{
            fps: 30,
            resolution: '720p',
            bitrate: 128000, // # 2 Mbps
          }}
          audio={{
            bitrate: 128000,
            sampleRate: 44100,
            isStereo: true,
          }}
          isMuted={false}
          onConnectionSuccess={() => {
            console.log('success')
          }}
          onConnectionFailed={(e) => {
            console.log('failed')
            console.error(e)
          }}
          onDisconnect={() => {
            console.log('disconnect')
          }}
        />
        <View style={{ position: 'absolute', bottom: 40 }}>
          <TouchableOpacity
            style={{
              borderRadius: 50,
              backgroundColor: streaming ? 'red' : 'white',
              width: 50,
              height: 50,
            }}
            onPress={() => {
              if (streaming) {
                ref.current?.stopStreaming();
                setStreaming(false);
              } else {
                ref.current?.startStreaming(API_KEY_HERE);
                setStreaming(true);
              }
            }}
          />
        </View>
      </>
      {/* )} */}
    </View>
  );
}

export default RecordStream;

Hi,
What version of the library do you use?
When you call startStreaming, the library creates a new encoder object to be ready for the next live and I don't know why but the system could not create it. Usually it is because the encoding settings are not supported (like resolution is not supported,...). Could you perform the same test with our Android native library (StreamPack)?
For the example code, could you please give a full project instead?
By the way 128000 bps for video is way too low.

1.2.0
Is there any way I can determine which parameters are supported? I've been trying to perform the same Android native however I'm not sure your example is up-to-date, I'm trying to implement the ConnectionChecker and I can't find it to reference.
Example code https://drive.google.com/file/d/1z94XDG3r6Cd6VDHiAB_M7LjoSZ70EkdH/view?usp=sharing
Thanks, I've changed that now

1.2.0
Ok perfect!

Is there any way I can determine which parameters are supported?
Not on the RN part yet

I've been trying to perform the same Android native however I'm not sure your example is up-to-date, I'm trying to implement the ConnectionChecker and I can't find it to reference.
Just use the example code or the example from StreamPack.

I've just spent several hours trying to implement both your native SDK and StreamPack. I've managed to implement it with your SDK and the same thing happens, I'm struggling to get StreamPack working to be honest as my android experience is limited and the example code leaves some stuff out. I may well be doing something wrong but I'm reaching the limit of my knowledge at this point.
To run my example it requires using expo's eas build, it's basically a yarn install and then use the expo cli to build and install. I can send you a dev build if you like. I should also state it doesn't crash in an Android emulator.
The only remaining thing I can try is to build in a bare react native project

Is there any chance you could send me a full StreamPack example to try and run?

Yes, of course, I should have said that before : https://github.com/ThibaultBee/StreamPack/suites/7900508529/artifacts/335986000
If it does not crash on an Android emulator, I am pretty sure it won't crash with my device either. Unfortunately it looks like an issue with this model of phones...

Thanks for your reply. I tried with SteamPack and it worked fine on the original device and a new one. I went back to your library to try and it wouldn't work on either device

I ran your example code but  the application did not crash. Is there specific step to reproduce your issue?

Strange.. Unfortunately not. It was just a case of starting the stream and then stopping it. I think occasionally it would work but 90% of the time it wouldn't


Cluster long names need to be truncated in home page cluster list

Setup

Rancher version: v2.9-e2023b437f3ccba16169b1e51db18419832ca5d7-head

Describe the bug
Cluster long names override the cluster provider in the home page cluster list.
To Reproduce

Create a downstream cluster with a long name (more than 60 characters)
Go to the home page and observe the table listing all the existing clusters

Result
Cluster long names override the cluster provider in the home page cluster list.
Expected Result
Cluster long names need to be truncated in the home page cluster list, fitting the standard width of the column designated for the name. Tooltips should be added to show the cluster's full name.
A better solution would be to limit the number of characters for the cluster name in the UI as a field validator. As seen in the screenshot, the cluster fails to be created, as GKE does not support cluster names with more than 40 characters.
Screenshots

Found while testing https://github.com/rancher/dashboard/issues/8854


SignEncrypt: paramaterize sigPrefix, move chunkNum up

make sigPrefix an arg
move chunkNum up so that in the next PR exposing a non-streaming interface will be easier.

This test passes for me. Is this about endianness?
23:22:04 23:22:04 --- FAIL: TestVectors (0.00s)
23:22:04 23:22:04 	codec_test.go:632: i:0 sealed bytes not equal

The test fails for me. The first 80 bytes of ciphertext (16 byte Poly1305 tag + encrypted 64 byte sig) are all different between the two, though the rest is the same. So probably nothing about the encryption is different, but something has changed in the sig input?
I have absolutely no theory for why it would pass on your box and not mine / CI though.

The sig also fails to verify when I comment out that first failure.


replace links to /docs/concepts/containers/runtime-class/ with /ja/docs/

closes #28799

/assign @nasa9084

/lgtm

/approve


LLVM Corrections

I recently used this in a project compiled with LLVM and several corrections/fixes/modifications were needed to pass the compiler.  I've assembled them into small commits to make it easier (so review by commit).
The last commit is actually a pretty serious issue if a non-existant file is path is passed into the parser, it will just enter an infinite loop and never exit because it never checked whether the stream was actually open.

Thank you very much, @gbooker!


chore(stable): Update stable

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Allow subclassing of AIProxyCertificatePinningDelegate

This allows us to create a subclass of the session delegate, to override functions for tracking progress (whilst still maintaining certificate pinning) such as:
func urlSession(_ session: URLSession, task: URLSessionTask, didSendBodyData bytesSent: Int64, totalBytesSent: Int64, totalBytesExpectedToSend: Int64)

and
func urlSession(_ session: URLSession, dataTask: URLSessionDataTask, didReceive data: Data)

We can use it by simply overriding the delegate:
final class ProgressMonitoringSessionDelegate: AIProxyCertificatePinningDelegate {
...
}

AIProxyURLSession.delegate = ProgressMonitoringSessionDelegate()

Very nice, thank you for the contribution!

Available in release https://github.com/lzell/AIProxySwift/releases/tag/v0.49.0


Website broken on Firefox

working on Mac chrome & safari
https://user-images.githubusercontent.com/6603389/123415488-a2ad5f00-d5b5-11eb-98b2-d0a90ebcea86.mp4

I'll close this issue as we updated our website, feel free to ping if you still have problem


Fix typos (theadpool -> threadpool)

Came across Theadpool in an exception. Found another one in doc-string :)

Thanks!


odb: add detele/clear methods

Mostly convenience, to avoid manually messing with odb.fs.

Codecov Report
Base: 69.02% // Head: 68.57% // Decreases project coverage by -0.44% :warning:

Coverage data is based on head (77930e6) compared to base (bc253bb).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #149      +/-   ##
==========================================
- Coverage   69.02%   68.57%   -0.45%
==========================================
  Files          25       25
  Lines        1556     1588      +32
  Branches      219      186      -33
==========================================
+ Hits         1074     1089      +15
- Misses        452      470      +18
+ Partials       30       29       -1

Impacted Files
Coverage Δ

src/dvc_objects/db.py
97.80% <100.00%> (+0.06%)
:arrow_up:

tests/test_odb.py
100.00% <100.00%> (ø)

src/dvc_objects/fs/system.py
63.23% <0.00%> (-20.59%)
:arrow_down:

tests/test_reflink.py
87.50% <0.00%> (-12.50%)
:arrow_down:

src/dvc_objects/fs/base.py
56.48% <0.00%> (+0.38%)
:arrow_up:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Request - Display as vacuum in iOS18

Now that iOS 18 will allow support for vacuums in HomeKit, can this plugin be reworked to display correctly please?

+1 to vote for this update!!


Add Pagination support

We need to add pagination support

I was looking at pagination using gcloud. It seems that finding out the number of pages may be tricky using cloud-datastore. I'm not really happy with any solutions given in the article.
I've fixed the existing "More" button. We may need to add a "Previous Page".

I think we should migrate pagination to use skip / limit and if the database grows enough that it starts being an issue, we change? WDYT? @juliantoledo @ithinkihaveacat

Changed to skip / limit on #160

Closed by #160, improved by #161

Closed on #160 and #161. UI update being tracked on #141


add disabled type for filter dropdown

This is good when a user wants to disable an option within the filter dropdown. It can be done by setting FilterField type value as FilterType.DISABLED

Deploy preview for patternfly-ng ready!
Built with commit 2ad7cb36bb59eec590e3324bf0f87bb28ea1b822
https://deploy-preview-439--patternfly-ng.netlify.com


API for stateless step definitions

Currently, the only way to pass state between step definitions is by storing state as instance variables.
This is cumbersome in some languages, in particular for Java where it becomes tricky to share state between classes. DI is the only way to address this.
For other languages (such as Ruby and JavaScript) the World often becomes a huge object holding all kinds of different state used by various step definitions.
I would like to propose an alternative API for Step Definitions where:

Each step definition (or hook) can return a value
This value is passed to the next step definition
Statically typed languages can use custom types

This would remove the need for a World or DI. It would reduce coupling and make it more explicit what is passed between step definitions.
We'll use this issue to discuss the implications of this in various implementations, and to come up with an API for various languages such as Java, Ruby and JavaScript (at least).
Once we're happy with the API we can create new issues to implement this in various Cucumbers.
(I've been thinking about this for a while, and I'm glad I am not the only one! See cucumber/cucumber-jvm#1136)

I think this solution trades a coupling a between step definitions and the World/DI modules for a coupling to the order in which steps in gherkin must be written. So I don't think this can be used to completely replace the World or DI modules. In the worst case you end up passing the World between steps.
For example take a website like Gmail that personalizes your account even when you  are not logged in because it recognizes that nobody else ever uses that machine.
Given I am a specific kind of customer
And I have a login account that was previously used to login
When use the website
Then I am greeted with my name

In this scenario I'd have to pass the information about the customer (his name) from the first step all the way to last step through one or more steps that don't use the information.
But other then that yeah this can useful.

I found some past discussion about the same issue in Scala.
https://github.com/cucumber/cucumber-jvm/issues/451

I am very much in favour of pure functions, or better of allowing both pure and impure step definitions for languages that were FP is not the main paradigm.
I think DI framework support is still needed to inject what is not part of the scenario context into the step definition class: components from the application or shared test dependencies (e.g. Selenium). In this way we don't need to handle different scopes for context parameters, as they should always have scenario scope, but we'd be flexible.


Add hot reloading in development

Here's a proof of concept of hot reloading. I'm not sure whether this is useful with stateful spring system, but you can play with it :-)
npm install
npm start
open http://localhost:3000/demo1

You can also make a production build
npm run build

and open the regular HTML file.

:+1: :D


pari 2.9.1

Created with brew bump-formula-pr.

Thanks for your contribution to Homebrew!  You rock!


Adding section for AppAnalytics

Adding section for AppAnalytics

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@vkonev : Thanks for your contribution to the Azure documentation! The author, @alancameronwills, has been notified to review your proposed change.

We shouldn't use internal jargon in public facing documents. We call the service "Application Insights Analytics."  Some customers won't know when they should use the CDN addresses.
For what purpose would customers use the information in the note?

There was a CSS escalation when one of the large customers were complaining that not all Application Endpoints are listed in our documentation (including CDN, AppAnalytics, etc). I can loop you in the thread if you want, but here is the quote from CSS:
“The Bank allow to open the internal network through proxy only to specific FQDN addresses,
The bank configure the connection to Application Insight using the Article IP addresses used by Application Insightshttps://docs.microsoft.com/en-us/azure/application-insights/app-insights-ip-addresses but last update is from Nov-2016 and currently when we are trying to use Application Insight (with fiddler debugging) we have many URL’s that missing.”
Thanks,
Vitaliy
From: Alan Cameron Wills<EMAIL_ADDRESS>Sent: Thursday, May 18, 2017 3:18 PM
To: Microsoft/azure-docs<EMAIL_ADDRESS>Cc: Vitaliy Konev<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/azure-docs] Adding section for AppAnalytics (#1800)
We shouldn't use internal jargon in public facing documents. We call the service "Application Insights Analytics." Some customers won't know when they should use the CDN addresses.
For what purpose would customers use the information in the note?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fazure-docs%2Fpull%2F1800%23issuecomment-302557035&data=02|01|vkonev%40microsoft.com|203451c8d6e54cda1fe308d49e3bb114|72f988bf86f141af91ab2d7cd011db47|1|0|636307426592170217&sdata=0IW4khYeBA2CCtqjiNodKbBv8mC4X6IF30cMeisf%2FNg%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAT0F37fAF6_ibcxvhh-Fad-I5S_7OCMqks5r7MN_gaJpZM4Nfmn1&data=02|01|vkonev%40microsoft.com|203451c8d6e54cda1fe308d49e3bb114|72f988bf86f141af91ab2d7cd011db47|1|0|636307426592170217&sdata=tglKDIHk6zb0IKOt1oI%2FaZtZmZJXTcRb4gfTeSZc1Tc%3D&reserved=0.

OK, but let's fix the name - we don't use AppAnalytics

ok, makes sense. Updated the name

#sign-off


Error: EPERM: operation not permitted, unlink  errno: -4048

I take screenshots of more than 10,000 websites. But sometimes in a way that I don't understand, the program crashes in this way and my program does not continue. Is there a solution for this? At the very least, I want the program to continue.

The error is coming from Puppeteer, not Pageres.

The error is coming from Puppeteer, not Pageres.

Hi, Where exactly does Pageres use Puppeteer? maybe i can try to patch it.

Try the latest version: https://github.com/sindresorhus/pageres/releases/tag/v7.0.0 It may have resolved the issue.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-source-version-target/pull/93 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-source-version-target.git

Docs Build status updates of commit eb1ecc1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

versioningArchiveSourceFolder/TOC.yml
:white_check_mark:Succeeded
View (sql-server-2014 || azuresqldb-current || azuresqldb-mi-current || azure-sqldw-latest || >= aps-pdw-2016 || >= sql-server-2016 || >= sql-server-linux-2017 || sql-server-previousversions || sqlallproducts-allversions)

versioningArchiveSourceFolder/docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


calling display on a large Sound object crashes Source Academy

Ways to reproduce:         display(sine_sound(500, 20));

This issue will go away with the newest version of the sound missions that the team SOUND is working on.


fix: add local modifier to simp attribute

Restrict the scope of the simp attribute to Std.Data.String.Lemmas.

This PR will debug the errors raised by the linter in https://github.com/leanprover-community/mathlib4/pull/3966.

as the comment says, this is deliberate. You should change the mathlib simp lemma as suggested by the linter, and remove or un-simp the (singleton c).toList lemma.

(Arguably it would be better if String.data didn't exist in the first place, and the projection was called toList, but this requires changes in core.)


[Bug Report] v-loading does not clear the el-loading-parent when the component is destroyed

Element UI version
2.0.8
OS/Browsers version
macOS Sierra 10.12.6
Vue version
2.5.11
Reproduction Link
https://jsfiddle.net/zet1vyLg/
Steps to reproduce

Create a component using v-loading.fullscreen.lock
Destroy the component that contains the v-loading directive while it is still loading
The el-loading-parent classes (el-loading-parent--hidden ...) are still there (preventing the user to scroll etc)

What is Expected?
Remove the el-loading-parent classes
What is actually happening?
The el-loading-parent classes (el-loading-parent--hidden ...) are still there (preventing the user to scroll etc)

Fixed in https://github.com/ElemeFE/element/pull/8914

Thank you ! What is the expected release date ?

Should be released before next Monday.


Hello, why is the recognition effect so bad

Hello, why is the recognition effect so bad

I also has this problem.
Hope the Readme an be update with more detail.

Try to set up different thresholds and see what happens


Adding missing stream icons for execution plan

Missing icons:

Fixes https://github.com/microsoft/azuredatastudio/issues/20747
Changes in STS: https://github.com/microsoft/sqltoolsservice/compare/<IP_ADDRESS>...<IP_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 37.423%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
14135

Relevant Lines:
34727

💛  - Coveralls


ENH: Use itkPrintSelfObjectMacro to print objects that can be null

Use itkPrintSelfObjectMacro to print objects that can be null.
PR Checklist

[X] No API changes were made (or the changes have been approved)
[X] No major design changes were made (or the changes have been approved)

Left behind in #2989.


Support fallback of buck2.file_watcher to 'notify' if 'watchman' doesn't work?

As I noted in https://github.com/facebookincubator/buck2/issues/57#issuecomment-1362318423, I found a way to enable watchman support via the config option buck2.file_watcher. See this corresponding commit, in particular the highlighted lines: https://github.com/thoughtpolice/buck2-nix/commit/e47fc8f52bc599f1c36a1fbbcf2ea3ce38b66d4f#diff-d33e979799a45c7c51752e9c8d96a3e452015d1a40b1e4b6ec6a98e92c4d8430R92-R105 — I also recommend the commit message for a summary.
It's very handy. In short, I use direnv to automatically use systemd-run --user to launch a transient user service for watchman, then export an appropriate WATCHMAN_SOCK variable for the repository. Works great! Except...
I forgot to enable it in CI, which caused this failure:
[2022-12-22T15:45:13.628+00:00] 2022-12-22T15:45:13.626994Z  WARN buck2_server::file_watcher::watchman::core: Connecting to Watchman failed (will re-attempt): Error reconnecting to Watchman: While invoking the watchman CLI to discover the server connection details: reader error while deserializing, stderr=`2022-12-22T15:45:13,625: [watchman] while computing sockname: failed to create /usr/local/var/run/watchman/runner-state: No such file or directory
[2022-12-22T15:45:13.630+00:00] `
[2022-12-22T15:45:13.663+00:00] Build ID: 6d28aa[89](https://github.com/thoughtpolice/buck2-nix/actions/runs/3754737959/jobs/6379213447#step:8:90)-1dee-49fd-80c2-b8a54e1c63[96](https://github.com/thoughtpolice/buck2-nix/actions/runs/3754737959/jobs/6379213447#step:8:97)
[2022-12-22T15:45:13.665+00:00] Command failed:
[2022-12-22T15:45:13.665+00:00] SyncableQueryHandler returned an error
[2022-12-22T15:45:13.665+00:00]
[2022-12-22T15:45:13.665+00:00] Caused by:
[2022-12-22T15:45:13.665+00:00]     0: No Watchman connection
[2022-12-22T15:45:13.665+00:00]     1: Error reconnecting to Watchman
[2022-12-22T15:45:13.665+00:00]     2: Error reconnecting to Watchman
[2022-12-22T15:45:13.665+00:00]     3: While invoking the watchman CLI to discover the server connection details: reader error while deserializing, stderr=`2022-12-22T15:45:13,633: [watchman] while computing sockname: failed to create /usr/local/var/run/watchman/runner-state: No such file or directory
[2022-12-22T15:45:13.665+00:00]        `
BUILD FAILED

So there's two things going on here:

I'm using a watchman binary from the repository, the Ubuntu 22.04 one. But I didn't install it with dpkg, I installed it with Nix, since you can't rely on the user having it. A consequence of this is that some directories, such as /usr/local/var/run/watchman/, don't exist, which cause spurious failures, since watchman binaries implicitly have a STATEDIR set to /usr/local at build time. And if there is no WATCHMAN_SOCK variable set, the watchman_client Rust library will query the CLI. Therefore calls like watchman get-sock or whatever will fail, because they always probe a non-existent statedir which causes part of the stack trace. I can work around this in some way with Nix, and will file a possible bug report with upstream watchman about it; but I just wanted to clarify this since it's in the above error.

Note that the transient systemd unit sets an explicit path to statefile, logfile, and pidfile, which are the 3 required variables that watchman needs; if these are set, the implicit STATEDIR is not used.
This is why I set WATCHMAN_SOCK instead with my setup, because if it is set, the watchman_client library uses it above all else, and doesn't need to query the CLI binary. So it "just works"

The build fails outright if the file_watcher can't be configured.

Would it be possible to optionally fallback to buck2.file_watcher=notify if watchman_client detects and captures a failure like above? This would at least allow users to continuing developing in the repository without a strange error if they don't enable watchman, even if they would detect file changes watchman would otherwise ignore. I'm trying to make my repository 'plug and play', and it feels bad if you cd $src; buck build ... and it immediately fails like this without watchman.

I can probably try to write a patch for this too, if it's deemed acceptable and easy to review/integrate; I don't know how difficult it is to "back-port" changes from GH into the Meta repository in this manner, so if it's harder than just making the change on all of y'alls end, I can wait.

I don’t think an unconditional fallback would be desirable (it’s certainly
not something we’d ever want to do internally, so merging that would be
undesirable), but allowing e.g. a comma separated list of file watchers to
try, or a separate config value that means « watchman if available but
inotify otherwise » seems like it’d be reasonable (the former seems like
it’d be preferable to me).
Submitting a PR is fine, it’d quite easy to pull in, but bear in mind a
good portion of the team is on holiday, so it might take a little while to
get back to you if you do.
On Thu, 22 Dec 2022 at 17:27, Austin Seipp @.***> wrote:

I can probably try to write a patch for this too, if it's deemed
acceptable and easy to review/integrate; I don't know how difficult it is
to "back-port" changes from GH into the Meta repository in this manner, so
if it's harder than just making the change on all of y'alls end, I can wait.
—
Reply to this email directly, view it on GitHub
https://github.com/facebookincubator/buck2/issues/59#issuecomment-1363042886,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANIHVXIYVGTK24QVRMFGJ3WOR6QJANCNFSM6AAAAAATG4IRZM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

We do have https://github.com/facebookincubator/buck2/blob/main/CONTRIBUTING.md, but I've just revised that and put up an internal diff to make it also read:

Buck2 is currently developed in Meta's internal repositories. Code that is developed internally gets reviewed, sent through CI, committed, and then automatically mirrored out to GitHub every 15 minutes. Code that arrives through a PR is reviewed by a Meta developer on GitHub, then once accepted, moved into our internal workflow where it is reviewed, sent through CI, committed and added to the repo. We maintain both external CI (the results of which are visible on GitHub) and a more thorough internal CI (building internal projects etc). Alas, our full test suite is not yet mirrored to the open source repo, but we hope to fix that in due course.

TLDR: We welcome PRs, and they are actually remarkably easy to merge (if we are around, but as of now, I think almost no one is!)
If you want to make your repo plug and play, using inotify is probably fine. Watchman is really useful if you have a huge repo, or are using a Sapling-backed virtual disk. If not, then inotify works great. That's the reason we pick inotify as the default for open source - to increase plug and play ability.

I tried implementing this over the holiday break but got stuck. I initially tried a naive approach where configuration is parsed, here. Basically: split on , in the string, and try every option in a for loop until one worked.
But that failed. I think what's happening in this code is that the buck client program is telling the daemon which of the two file watcher APIs to use. It then tries to enable one, e.g. watchman, and fails, which causes the daemon to fail instead, not the client program, and that causes an unrecoverable failure. I think. Maybe it is all in the server. I don't quite understand the code layout yet and I'm pretty new at Rust, especially on a codebase this size.
I'll post my initial attempt later but needless to say I'd still quite like this. As a workaround, at the moment I simply hardcode aarch64-linux to use notify and ignore the configuration setting entirely using this patch. This roughly gets me the UX I want on multiple platforms, and once this is implemented, should be much less hacky.

Generally the file watcher lives in the daemon, so I wouldn't expect much to happen in the client (broadly speaking, the client is just a argument parser and output console). When you say fail, do you mean with a panic or with a anyhow::Error? If it's a panic, that's probably the bug and should be fixed. If it's an Error, that should be easy to ignore and select the next one on.
Alternatively, if Watchman provided binaries for aarch64-linux, would that solve the problem? Should be easy to reach out to that team.
Separately, I am curious why you want to default to watchman at all? Wouldn't notify be sufficient? For really big repos, especially ones backed by Sapling/Mercurial, Watchman is a great idea. Otherwise, I imagine notify is good enough.

So, following up on this: I got on a phone call with Neil a while back and explained that one use case for watchman was integration with Sapling's virtual filesystem, which I intend to explore once it's stable and in a FOSS release. That's a major motivation for integrating watchman reliably and early on.
And recently, after a miraculous amount of work from the Nixpkgs community, a recent change added an up-to-date watchman binary, as well as all the dependencies -- and it works on aarch64-linux! See https://github.com/thoughtpolice/buck2-nix/commit/81a2e8c043cbefa44ef265ec0529f34001c1fb31 and https://github.com/NixOS/nixpkgs/pull/181787 for info.
So my need for this feature has basically gone away. Nixpkgs can provide aarch64-linux binaries for my users instead.
I also made an upstream issue about this, so non-Nix users can also have binary builds; see https://github.com/facebook/watchman/issues/1094 — but it remains unresolved, due to a lack of appropriate aarch64 builders for GitHub Actions. We'll have to wait until GitHub deploys Ampere builders at some point, I guess...
Closing this for now to reduce issue clutter and because the original (niche) motivation has gone away.


BBBB in copybook

Background
I am trying to read a cool playbook that has "BBBB".
Looks something like this.

Version of spark -
3.4.1
Scala - 2.12
Cobol_parser - 2.12:2.6.9
Spark-cool- 2.12.:2.6.9
Question
Is it possible to handle such "BBBB" defines? Has anyone come across this before. I currently get syntax error with Invalid input "BBBB" at position ...

Hi @sou-joshi, we haven't encountered 'B' in field PIC definitions. Could you send some info or reference to documentation  on how it works, please. I couldn't find it easily.

Hi, sorry, I've missed the reply. The duplicate error happens when a copybook contains a field having the same name on the same level. Spark does not duplicate names, so the way to workaround it is to rename one of duplicate fields.


Filling "null" as a method parameter

Filling "null" as a method parameter in method call is very bad practice. It's little bit dangerous. If you want to crate metjod like this:
method(String param) and method(Properties param) and you call method in code like this: method(null). Then java is confused which one should use.

@lcir ; can you please provide more info.

@garethahealy Sorry, i can't. I'm not using this mapper for 3 years. Details about this issue disappeared from my head long time ago :)