net: net_context: zsock_connect can error out due to duplicate ports

Describe the bug
check_used_port can return false when the port is actually in use, leading to find_available_port returning an unavailable port.
This can be reliably triggered when sys_rand32_get is derived from the system clock, but can occur with a 1/2**16 chance in normal operation.
The initial net_context_get from zsock_socket assigns a random port to the net context:
https://github.com/zephyrproject-rtos/zephyr/blob/c059405da57b8b3b7bd22295c7f66ea3541cc55f/subsys/net/ip/net_context.c#L502-L503
net_context_connect from zsock_connect then generates another random port inside bind_default:
https://github.com/zephyrproject-rtos/zephyr/blob/c059405da57b8b3b7bd22295c7f66ea3541cc55f/subsys/net/ip/net_context.c#L656-L658
A duplicate port number at this stage is not detected because the context sin_addr pointer is currently NULL, which skips further checks here:
https://github.com/zephyrproject-rtos/zephyr/blob/c059405da57b8b3b7bd22295c7f66ea3541cc55f/subsys/net/ip/net_context.c#L247
Inside net_context_bind (called from bind_default), the sin_addr pointer is finally set (using the first generated port):
https://github.com/zephyrproject-rtos/zephyr/blob/c059405da57b8b3b7bd22295c7f66ea3541cc55f/subsys/net/ip/net_context.c#L884
And the second generated port is then checked again:
https://github.com/zephyrproject-rtos/zephyr/blob/c059405da57b8b3b7bd22295c7f66ea3541cc55f/subsys/net/ip/net_context.c#L893-L898
At this point check_used_port returns an error, and sock_connect errors out.
Expected behavior
It should not be possible for duplicate port numbers to be generated inside zsock_connect that cause an error without the user being able to intervene.
Environment (please complete the following information):

Zephyr v3.6 (Cannot see any meaningful changes on main)

I'm not sure whether the bug is in the implementation of check_used_port (i.e. the NULL check is not sufficient), or in one of the calling functions (context sin_addr pointer should be set earlier)

@JordanYates I am trying to replicate this but have so far failed. Do you have a setup / test app I could use?
What is baffling is that here in check_used_port() before the NULL check you mentioned,
https://github.com/zephyrproject-rtos/zephyr/blob/a495fd9475e324b453a02b543d917e0816cfce3a/subsys/net/ip/net_context.c#L173-L175
we check whether there is a local port (that is set by net_context_get()) already set. So I just wonder why we would get the duplicate port selected.

Are you using IPv4-to-IPv6 mapping  CONFIG_NET_IPV4_MAPPING_TO_IPV6 as that could explain this issue?

@JordanYates I am trying to replicate this but have so far failed. Do you have a setup / test app I could use?
What is baffling is that here in check_used_port() before the NULL check you mentioned,
https://github.com/zephyrproject-rtos/zephyr/blob/a495fd9475e324b453a02b543d917e0816cfce3a/subsys/net/ip/net_context.c#L173-L175
we check whether there is a local port (that is set by net_context_get()) already set. So I just wonder why we would get the duplicate port selected.

I can reliably trigger this in my local code by changing this line:
https://github.com/zephyrproject-rtos/zephyr/blob/c059405da57b8b3b7bd22295c7f66ea3541cc55f/subsys/net/ip/net_context.c#L315
to this:
local_port = k_uptime_get() / 2 | 0x8000;

Which in theory should just make the function loop for ~2 msec.

Are you using IPv4-to-IPv6 mapping CONFIG_NET_IPV4_MAPPING_TO_IPV6 as that could explain this issue?

This option is not enabled (CONFIG_IPV6 as a whole is disabled). I will try and generate a reproduction in NCS using a nRF7002DK.


Sample script includes messy code

The following character is displayed in the sample script last 3 lines:
Â

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

ID: 2a0836fd-4f07-3d69-08c0-2ed48b851e18
Version Independent ID: 9debb166-9302-d3b7-91a4-e7c9ed653b62
Content: Azure PowerShell Script Sample - Create a managed disk from a snapshot
Content Source: articles/virtual-machines/scripts/virtual-machines-windows-powershell-sample-create-managed-disk-from-snapshot.md
Service: virtual-machines-windows
GitHub Login: @ramankumarlive
Microsoft Alias: ramankum

@wellsluo Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@ramankumarlive @sptramer can either of you help correct the doc? I created a pull request but the script is stored in a different repo.

CC @cynthn

I have assigned the issue to the content author to investigate further and update the document as appropriate.

The issue is already fixed, please check it out:
https://docs.microsoft.com/en-us/azure/virtual-machines/scripts/virtual-machines-windows-powershell-sample-copy-managed-disks-vhd?toc=%2Fpowershell%2Fmodule%2Ftoc.json


How to run Spring Cloud Gateway with AWS Cognito

Is there some guide or information how I need to configure Spring Cloud Gateway to run with AWS Cognito in order to authenticate users? I would like to configure AWS Cognito to run behind gateway.

Yes - spring security question. Can you guide me is there some tutorial how to do this configuration?

I can not, you'll need to ask in spring-security


[Cognitive Services] Update endpoint URL template for TextAnalytics.

Detail background and explainations in here: https://github.com/Azure/azure-rest-api-specs/issues/3489 Cognitive Services URL template (endpoint)
Please hold for TextAnalytics team's review.
This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-node

Automation for azure-sdk-for-java

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-go

@assafi and @yangyuan - please let me know when your internal review is complete and I will review.

Hi @annatisch we've completed the internal review

Thanks @yangyuan - were you going to rename Endpoint -> baseEndpoint in this PR?

@annatisch not in this PR.
We have a hard deadline for this PR so will process first.


[client] Entity Details windows

This Module is a button in the main left navigation.
It contains information about the selected Entity.
Since Entities within Eternum can carry resources, and have other functions, this window is generic and dynamic enough to accomodate all Entities.
The high level sections are

Military Combat (Armies, Battles)
Commerce (resource information balances)
Build (can this Entity be built on)

Hi @ponderingdemocritus, may I take this?


Trouble w/ version of CRTM

We presently have a problem w/ the bundle here and the version of CRTM it picks. I have an issue in the jedi/crtm repo trying to identify a better version of CRTM to use. Presently, the c-tests are failing due to an inconsistent Aerosol coefficient file.
When I find out what version we should use this will need to be updated here.

Ok, please let me know what version you are trying to install.
The default CRTM version is defined below, but you can change that manually before building your bundle, while choosing which repos to build. I think a problem might arise when the CRTM version you are requesting is old and not compatible with the current develop JEDI repos.
https://github.com/GEOS-ESM/jedi_bundle/blob/develop/src/jedi_bundle/config/build.yaml

@rtodling , I need your help with this. Currently, tier2 tests involve building the latest JEDI every night and and there is a step during the build where CRTM coefficients need to be obtained from a server. See this comment for a detailed description. I see @danholdaway tried to resolve this issue with this PR but the tier2 CI build still fails while attempting to download CRTM coefficients.
Here is what I need to know:

What version of CRTM should we be using now?
What CRTM version are we aiming for next and how soon? I will mention this issue in the JEDI Wednesdays group as well.

Bonus, error message below:

@rtodling
I will change the YAML entries here:
https://github.com/GEOS-ESM/jedi_bundle/blob/develop/src/jedi_bundle/config/platforms/nccs_discover.yaml
crtm_coeffs_path will be the advda directory
crtm_coeffs_version will be tested with v2.4.1-jedi.1
So I will just need to download and store the crtm_coefficients, just FYI.

Hi Doruk,
This looks good to me - I will also generate the CRTM surf files later today or tomorrow and point you to them so we can have that closed. Thanks for this.


fix(scripts/artifacts): fix check-images-internal.sh

Also add Deno scripts:

scripts/artifacts/check-image-internal.ts
scripts/artifacts/check-tiup.sh
ops/search-lark-user-by-githubid.ts

Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

/approve


[FEATURE] use yt-dlp

Instead of doing it your self, you could rely on yt-dlp for some providers for getting the video links.
pornhub is supported.
hentaimama works with the generic provider.

mpv uses yt-dlp and mov-cli return the embed link the rest does mpv/yt-dlp


web: memory constantly grows then shrinks for seemingly no reason

Look at the memory panel if both of these examples.
Native behaviour (cargo r --release --features web -- examples/out/avocado.rrd):
https://user-images.githubusercontent.com/2910679/214511965-dbfcc85c-e94c-4a1a-8966-617a5e448b3f.mp4

Web behaviour (cargo r --release --features web -- --web-viewer examples/out/avocado.rrd):
<screwed up the recording, gotta make a new one>

The saw-toothing is because of wgpu. Allocation callstacks:
wgpu_hal::gles::device::<impl wgpu_hal::Device<wgpu_hal::gles::Api> for wgpu_hal::gles::Device>::create_buffer::h95a01789c68e5abb@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[3369]:0x5d7a94
wgpu_core::device::queue::prepare_staging_buffer::hb09c3e769e7dec34@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[7421]:0x7fc196
wgpu_core::device::queue::<impl wgpu_core::hub::Global<G>>::queue_write_buffer::h94c9d15eacad360a@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[2038]:0x49a5df
<wgpu::backend::direct::Context as wgpu::context::Context>::queue_write_buffer::he1b7a577c3aebc91@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[4387]:0x6868dc
<T as wgpu::context::DynContext>::queue_write_buffer::hb05a919c3e92827c@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[12381]:0x94730a
wgpu::Queue::write_buffer::h8777518da1c6e23c@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[14917]:0x994cda
egui_wgpu::renderer::Renderer::update_buffers::he1504de8afe9981f@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[1100]:0x324ad5
<eframe::web::web_painter_wgpu::WebPainterWgpu as eframe::web::web_painter::WebPainter>::paint_and_update_textures::h411d5e2d6aababda@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[876]:0x29bd3d
eframe::web::backend::AppRunner::paint::h584aff3234354fd5@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[3351]:0x5d4423
eframe::web::events::paint_and_schedule::hb13fba93f89741be@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[1700]:0x42c80c

wgpu_hal::gles::device::<impl wgpu_hal::Device<wgpu_hal::gles::Api> for wgpu_hal::gles::Device>::create_buffer::h95a01789c68e5abb@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[3369]:0x5d7a94
wgpu_core::device::queue::prepare_staging_buffer::hb09c3e769e7dec34@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[7421]:0x7fc196
wgpu_core::device::queue::<impl wgpu_core::hub::Global<G>>::queue_write_buffer::h94c9d15eacad360a@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[2038]:0x49a5df
<wgpu::backend::direct::Context as wgpu::context::Context>::queue_write_buffer::he1b7a577c3aebc91@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[4387]:0x6868dc
<T as wgpu::context::DynContext>::queue_write_buffer::hb05a919c3e92827c@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[12381]:0x94730a
wgpu::Queue::write_buffer::h8777518da1c6e23c@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[14917]:0x994cda
egui_wgpu::renderer::Renderer::update_buffers::he1504de8afe9981f@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[1100]:0x324b6e
<eframe::web::web_painter_wgpu::WebPainterWgpu as eframe::web::web_painter::WebPainter>::paint_and_update_textures::h411d5e2d6aababda@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[876]:0x29bd3d
eframe::web::backend::AppRunner::paint::h584aff3234354fd5@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[3351]:0x5d4423
eframe::web::events::paint_and_schedule::hb13fba93f89741be@http://<IP_ADDRESS>:9090/re_viewer.js line 366 > WebAssembly.instantiate:wasm-function[1700]:0x42c80c

I try this:
cargo r -p raw_mesh -- --save ../lego.rrd
cargo r -p rerun --features web_viewer -- --web-viewer ../lego.rrd

And I see this:

We have a decent picture of where these come from by now:

the bug can't be reproduced with a native GL renderer since the allocations we're seeing are "buffer backing memory" of wgpu::Buffers which is only needed when mapping buffers is not possible
it seems that the memory is exclusively from temp buffers on wgpu::Queue which are created on every write_buffer/write_texture call. This implies that any use of our CpuWriteGpuReadBelt is reducing the issue. However, we can't avoid it entirely right now since egui is still using the queue for uploading data every frame
the rise in (average) memory usage over time is merely egui using more memory every frame since the memory graph has more and more lines over time. The easiest way to demonstrate this is by panning in the panel so that egui clips all lines, saying there for a while and then going back: Without the lines, the memory spikes stay lower.
we do not know why the temp buffers are freed in bursts. Wgpu has no mechanism for delaying the buffer garbage collection. This directly originates from submission indices being thought of as in use or not in use

i.e. why does it go up and down if there is a constant stream of new frames?
it seems that the WebGL backend reports frames done (i.e. value on the signal handler) only every few frames
it should be possible to optimize the WebGL backend to discard the backing memory once it is known that a user won't use a buffer anymore (i.e. much earlier than when it is known that the gpu is no longer using a buffer)

Further investigation is needed, but with what we know the issue seems fairly contained, and not as worrisome as originally thought, especially if we continue to use the CpuWriteGpuReadBelt everywhere.


GroundOverlay Issue

Hi,
My Android app works perfectly with ground overlays, but when I port it to iOS I have to comment out some of the code in GroundOverlay.m to get it working.
Any idea what I'm doing wrong or is this a bug?
Javascript:
				map.addGroundOverlay({
				  'url': URL,
				  'bounds': bounds,
				  'opacity': 0.5,
				  'zIndex': 99
				},function(groundOverlay) {
				});

Commented Out To Avoid Issue:
NSURL *isUrl = [NSURL URLWithString:urlStr];
//if (!isUrl || !isUrl.scheme || isUrl.host) {
//  layer.icon = [UIImage imageNamed:urlStr];
//  [self.mapCtrl.overlayManager setObject:layer.icon forKey: id];
//  completionHandler(nil);
//} else {
dispatch_queue_t gueue = dispatch_queue_create("GoogleMap_createGroundOverlay", NULL);
dispatch_sync(gueue, ^{
NSURL *url = [NSURL URLWithString:urlStr];
NSData *data = [NSData dataWithContentsOfURL:url];
UIImage *layerImg = [UIImage imageWithData:data];
layer.icon = layerImg;
[self.mapCtrl.overlayManager setObject:layerImg forKey: id];
completionHandler(nil);
});
//}
Thanks!

This is already fixed in master

Great, I see it now, thank you!!

Comment on issue

Comment on issue

What you want to tell us?


Mounting /data on NFS

I'm trying to mount /data on an NFS mount. The idea is that I can boot up the image from any of my hosts in case one goes down. However, I'm getting the following error:
```chown: changing ownership of`/data/bind': Operation not permitted
Looks like the directories are initially created by nobody:nogroup.
Which user is attempting to change theh ownership?
Is this because of the docker bridge network?
How can I map /data to my NFS mount.

I was having the same problem, had to tweak the export and mount settings to fix the issue.

@diogogmt, what settings on the client/server for your NFS mounts did you use to resolve the issue?

How solve the issue?


Suggestion: Add label to readonly workspace folders

Something like the above could help make it clear where the readonly restriction is coming from.

#52573

The notion of readonlies in the ux side is file based, not workspace based. So in theory you could have some readonly files in a workspace and some which are writable.
Due to that I prefer this solution https://github.com/Microsoft/vscode/issues/53022


HUD not displaying data

Hi,
I am working on an ASP .NET MVC 5 app, I cannot get the Glimpse HUD to display any data... I get this:

No data has been found. This could be caused because:

the data is still loading by the client, or
no data has been received from the server (check to see if the data & metadata payloads are present), or
no plugin has been loaded, or
an error has been thrown in the client (please check your JavaScript console and let us know if anything is up).

I have installed from NuGet: Glimpse 1.8.6, Glimpse.ADO 1.7.3, Glimpse.AspNet 1.9.2, Glimpse.EF6 1.6.5, Glimpse.MVC5 1.5.3.
The erros I have in the logs are:

Exception loading 'Glimpse.Core.Extensibility.ITab' of type 'Glimpse.AspNet.Tab.Cache'. | System.Reflection.TargetInvocationException--Exception has been thrown by the target of an invocation.--   at System.RuntimeTypeHandle.CreateInstance(RuntimeType type, Boolean publicOnly, Boolean noCheck, Boolean& canBeCached, RuntimeMethodHandleInternal& ctor, Boolean& bNeedSecurityCheck)
at System.RuntimeType.CreateInstanceSlow(Boolean publicOnly, Boolean skipCheckThis, Boolean fillCache, StackCrawlMark& stackMark)
at System.RuntimeType.CreateInstanceDefaultCtor(Boolean publicOnly, Boolean skipCheckThis, Boolean fillCache, StackCrawlMark& stackMark)
at System.Activator.CreateInstance(Type type, Boolean nonPublic)
at System.Activator.CreateInstance(Type type)
at Glimpse.Core.Framework.ReflectionDiscoverableCollection1.GetConcreteTypes(Assembly assembly, List1 results)>>TypeInitializationException--The type initializer for 'Glimpse.AspNet.Tab.Cache' threw an exception.--Void .ctor()>>NullReferenceException--Object reference not set to an instance of an object.--Void .cctor()

and

2020-05-14 13:41:22.9789 | FATAL | AppDomain with Id: '18' and BaseDirectory: 'path to app here' has been unloaded. Any in memory data stores have been lost. Reason for shutdown: Change Notification for critical directories.;bin dir change or directory rename;HostingEnvironment initiated shutdown;Change Notification for critical directories.;bin dir change or directory rename;Change in C:\Windows\Microsoft.NET\Framework\v4.0.30319\Temporary ASP.NET Files\root\20b30f4c\6103a0c2\hash\hash.web;Change Notification for critical directories.;bin dir change or directory rename;Change Notification for critical directories.;bin dir change or directory rename;Change in C:\Windows\Microsoft.NET\Framework\v4.0.30319\Temporary ASP.NET Files\root\20b30f4c\6103a0c2\hash\hash.web;Change Notification for critical directories.;bin dir change or directory rename;Change Notification for critical directories.;bin dir change or directory rename;Change in C:\Windows\Microsoft.NET\Framework\v4.0.30319\Temporary ASP.NET Files\root\20b30f4c\6103a0c2\hash\hash.web;Change Notification for critical directories.;bin dir change or directory rename;Change Notification for critical directories.;bin dir change or directory rename;HostingEnvironment caused shutdown |

What could I do ? I have read all the google pages about this :(

this is what I call the HUD, don't really know why.. I have seen in in one of many pages I have read while investigating

I also have this in the Console:
"Failed to load resource: the server responded with a status of 404 () Glimpse.axd:1"


IE 11 - Unable to load in-context window

Description
What happens is, after you get on the confirmation page, the Paypal window opens but it closes after a few seconds with no action from the shopper.
https://store.kaspersky.fr/order/checkout.php?PRODS=4711615&QTY=1&CART=1&CARD=1&SHORT_FORM=1&LANGUAGES=fr&LANG=fr&CURRENCY=EUR&DCURRENCY=EUR&ARN=0&OPTIONS4711615=2a,4pc
Console log:
SCRIPT5: Access is denied.
File: component-meta, Line: 1, Column: 1
SEC7118: XMLHttpRequest for https://ing-district.clicktale.net/ctn_v2/wr/?1405346876354903&35&10&12&0&8&264&subsid=119933&fSchBi+Eu1nfNQvbfFBEfNBBIfFBPfFBgfFBIfNBCPfNBCJfNBCIfNBCIfFBIfNBCIfNBB7/fNBBIfNlh8BJmJlh9BB3fyCXCPAlF8MQfyIJFUAfNBCHfFCIftDDRftCCHftCBIftCBIftCBIfFBIfoBIftCCJftKHqftCCGftDCHftHCJfoELftEBEftFBJfoDIftPBhfoCGfoDIfoDIlJ8KGfphMAfoC8JcfoDGfpFBKIUCHTHLKqJq&KlUAfpNBAfpLBHfoLGfpNBJfoNHCjABAQU&CN4HLKTCJ&LljAfoOAfpwBwfocBfoHGfpFBGfpJBSfpHBGfpCBIfpECIfoDIfpCBJfpCBIfoCJfpCBGlU8JXfJBFAfJJOhIRCHTH1KoJq&DlRAfJQRAfJGFjIQCHTHBCByJqtEBJKhgA&OlQAfJxdAfJgOElNvfJQIAlFZfRBhZAfJUBAkSeAG5SdvG5CdvUECdvG5A9DBoP_({"a"%3A[{"a"%3A{"s"%3A"l%3A1px%3Bt%3A136px%3B"}%2C"n"%3A"div157"}%2C{"a"%3A{"s"%3A"l%3A0px%3Bt%3A0px%3Bw%3A1903px%3Bh%3A1284px%3B"}%2C"n"%3A"div155"}]})&Dm9BzaCkEUAQKcUAACUAAA0BA&CL/D/CBoE+&9J8ylkBf2EBEiAlh8CSfWP0BYAfFB8T9fFBEfFBKfFCGfFBGfFBQmBCl&ACUEdvtEBAhBA&GllAfNBDAfFBNfFCIfFBGfFCGfFBJfFCGfFCIfFCIfFCIfFBIfFDIfFBIfFBLfFEIfFFbfFBEfFBNfFDEfFCLfFCFfFCJfFDGfFCJfFZ8E5fFBXfFBQfFB1fFB8DpfFBQfFBHeC8GMeKbeKOeHNeEQeGJeEQeDIeEKeDReENeGaeHUeHOeEN&1086553995 required Cross Origin Resource Sharing (CORS).
File: verify.php
ppxo_paypal_legacy_multiple_init_paypal_checkout {"timestamp":1505847286059,"windowID":"2ff05774a0","pageID":"2d54f37567","referer":"store.kaspersky.fr","host":"store.kaspersky.fr","path":"/order/verify.php","env":"production","country":"US","lang":"en","uid":"ce81c458cd","ver":"4.0.130"}
SEC7118: XMLHttpRequest for https://ing-district.clicktale.net/ctn_v2/wr/?1405346876354903&35&10&13&3&0&105&subsid=119933 required Cross Origin Resource Sharing (CORS).
File: verify.php
ppxo_unhandled_error {"stack":"Error: Function expected\n   at stringifyError (https://www.paypalobjects.com/api/checkout.js:3436:13)\n   at Anonymous function (https://www.paypalobjects.com/api/checkout.js:2853:17)\n   at Anonymous function (https://www.paypalobjects.com/api/checkout.js:8075:37)\n   at ZalgoPromise.prototype.dispatch (https://www.paypalobjects.com/api/checkout.js:8062:26)\n   at ZalgoPromise.prototype.then (https://www.paypalobjects.com/api/checkout.js:8105:17)\n   at _RECEIVE_MESSAGE_TYPE[] (https://www.paypalobjects.com/api/checkout.js:2835:13)\n   at receiveMessage (https://www.paypalobjects.com/api/checkout.js:2770:25)\n   at messageListener (https://www.paypalobjects.com/api/checkout.js:2791:13)","errtype":"[object Error]","timestamp":1505847356215,"windowID":"7548e6d64d","pageID":"c75691c8cb","referer":"www.paypal.com","host":"www.paypal.com","path":"/webapps/hermes/component-meta","env":"production","country":"US","lang":"en","uid":"559ab5a8f6","ver":"4.0.130"}
SCRIPT5002: Function expected
File: checkout.js, Line: 7967, Column: 21
paypal.version responds with "4.0.130"
An attempt was made at 8:29AM Central on 9/20/17. It generated EC Token EC-7UB27141KD736245U
ppxo_paypal_legacy_component_error {"error":"Error: Error: No ack for postMessage xcomponent_init in 1000ms","timestamp":1505914145143,"windowID":"dfee8c9334","pageID":"742ca30a37","referer":"store.kaspersky.fr","host":"store.kaspersky.fr","path":"/order/verify.php","env":"production","country":"US","lang":"en","uid":"262eabce1d","ver":"4.0.130"}
ppxo_paypal_legacy_render_error_hijack_revert_target {"timestamp":1505914145150,"windowID":"dfee8c9334","pageID":"742ca30a37","referer":"store.kaspersky.fr","host":"store.kaspersky.fr","path":"/order/verify.php","env":"production","country":"US","lang":"en","uid":"262eabce1d","ver":"4.0.130"}
Steps to reproduce
What happens is, after you get on the confirmation page, the Paypal window opens but it closes after a few seconds with no action from the shopper.
Test link:
https://store.kaspersky.fr/order/checkout.php?PRODS=4711615&QTY=1&CART=1&CARD=1&SHORT_FORM=1&LANGUAGES=fr&LANG=fr&CURRENCY=EUR&DCURRENCY=EUR&ARN=0&OPTIONS4711615=2a,4pc
Steps to reproduce:
•             Open link in IE 11 (windows 7 preferably)
•             Insert billing info
•             Choose paypal and click continue
•             Click on Paypal pay button
Result:
Paypal window closes few seconds after it appears.
Affected browsers

[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge
[X] IE (Reproduced in version 11.0.9600.18762)
[ ] Chrome Mobile/Tablet
[ ] Safari Mobile/Tablet
[ ] Web View / Safari ViewController
[ ] Other

Thanks so much for providing all of these details.
One quick follow up question -- do they see the same issue when testing with IE11 on https://developer.paypal.com/demo/checkout/ ?

I do not experience the same issue at https://developer.paypal.com/demo/checkout/

Confirmed with the initial reporter the same. They do not experience the issue at developer.paypal.com, but do at the kapersky site.

Good to know, thanks. I'll keep trying to narrow this down.

An additional merchant is reporting similar issue:
https://origin1.tsc.ca
Seeing this in the console:
 No handler found for post message response for message: postrobot_ready from https://origin1.tsc.ca in https://www.paypal.com/webapps/hermes/button
[6:00 PM]
File: checkout.js, Line: 2865, Column: 27

Here is the code the tsc.ca uses to get the token:
   protected readonly postHeaders: Headers = new Headers({ 'Content-Type': 'application/x-www-form-urlencoded' });
   protected paypalSandboxBaseUrl = "https://api.sandbox.paypal.com";
  protected paypalLiveBaseUrl = "https://api.paypal.com";
   protected paypalBaseUrl = this.paypalLiveBaseUrl;

protected paymentUrl = this.paypalBaseUrl + "/v1/payments/payment";
/*
Use postman to create authorization key using clientid and secretkey
*/
protected createAccessToken(): Observable {
let postheader = this.postHeaders;
    if (this.paypalEnv == 'production') {
        /* TSC live paypal account */
        postheader.append("Authorization", "Basic xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx");
    } else {
        /* Dev sandbox account */
        postheader.append("Authorization", "Basic xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx");
    }

    let body = /*'userName=' + this.clientId + '&password=' + this.secretKey + */ 'grant_type=client_credentials';
    let url = this.paypalBaseUrl + this.accessTokenUrl;
    return this.http
        .post(url, body, { headers: postheader });
}

@Chaithi thanks so much for all of the details to help repro this one! It was difficult to track down, but I've made a series of fixes and I'm not able to repro any of these issues any more. Can you please clear your browser cache, make sure console.log(paypal.version) gives you 4.0.152, and let me know if you still see any issues in IE11? Thanks again!


jupyterlab-lsp 3.0.0rc0 plan

Over on https://github.com/krassowski/jupyterlab-lsp/pull/452 @krassowski, (building on @martinrenou's https://github.com/krassowski/jupyterlab-lsp/pull/402) is soooo close :ok_hand: to having a family of release candidates ready that works with JupyterLab 3 :tada:, but we don't want to break peoples' stuff just yet.
Here are some thoughts on how we move forward... I hope to get this done (later) today, but wanted to write down some stuff as I was researching.
rc label
Let's do an rc release, as we've done a few upstream dev releases, and this one is more complicated.
I'm thinking our conda_build_config.yml will be like:
channel_sources:
- conda-forge/label/jupyterlab_rc,conda-forge/label/jupyterlab_server_rc,conda-forge

channel_targets:
- conda-forge jupyter_lsp_rc

This will let us easily adapt to upstreams.
packages
jupyter-lsp will get a 1.0.0rc0 release which replaces notebook with jupyter_server, but is otherwise unchanged.
We'll need a new package, jupyterlab-lsp 3.0.0rc0 to require jupyterlab >=3,<4 and jupyter-lsp >=1.0.0rc0,<2. We'll have to update this when the real 3.0.0 comes out.
Luckily it's also a straight pip build, noarch: python, so no funny stuff there.
I don't know whether we'll want to change the outputs for the extra language server stuff, e.g. jupyterlab-lsp-python, etc. I'm leaning towards not, so we don't have two packages running around, and there's really no relationship between version of jupyterlab and the language servers.
versioning
We'll probably want the "magic" {{ version }} to be the jupyterlab-lsp one, as it gets released more frequently, no reason not to let the bot do some of the heavy lifting.

One potential change to upstream could be modification of install.json file. The main repo could provide test "pip uninstall jupyterlab-lsp" while here it could be "conda uninstall jupyterlab-lsp".

One potential change to upstream could be modification of install.json file. The main repo could provide test "pip uninstall jupyterlab-lsp" while here it could be "conda uninstall jupyterlab-lsp".

Well, rcs are up, and it kinda works on binder:
https://mybinder.org/v2/gist/bollwyvl/76a720a27be20aec815d171227dc28cd/HEAD?urlpath=lab%2Ftree%2Fenvironment.yml
We probably won't get bot PRs, but it'll be relatively easy to update. We can put some more jinja stuff back in when we (hand) migrate these changes over to master.

Well, rcs are up, and it kinda works on binder:
https://mybinder.org/v2/gist/bollwyvl/76a720a27be20aec815d171227dc28cd/HEAD?urlpath=lab%2Ftree%2Fenvironment.yml
We probably won't get bot PRs, but it'll be relatively easy to update. We can put some more jinja stuff back in when we (hand) migrate these changes over to master.


fluent ui integration part 1: icons (WIP) (dont merge)

I am still working at the intergration of fluent ui. this part brings all the icons.

[ ] change all icons
[ ] testing

(WIP: this pr is only for the feedback when I am done I change the name and say if it is able to merge.)
(⚠dont merge⚠)

#325

can someone build it for me?

at the moment this makes a bug with dark mode I am looking how to implement correctly


remove slp local validator

use insight details endpoint
query slpdb for token info

Pull Request Test Coverage Report for Build 2652

0 of 0   changed or added relevant lines in 0 files are covered.
456 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-1.3%) to 62.709%

Files with Coverage Reduction
New Missed Lines
%

dist/routes/v2/cashaccounts.js
15
76.36%

dist/routes/v2/slp.js
441
32.34%

Totals

Change from base Build 2637:
-1.3%

Covered Lines:
2204

Relevant Lines:
3347

💛  - Coveralls


Collapsible button icons aren't visible when browsers are overriding background colour

Firefox allows users to specify colour style overrides, one side effect of this is that icons tends to disappear. I'm guessing that's because they've been set using CSS background-image. More on overriding colour styles information here.

I'm not sure how to fix this yet, but I heard that @nickcolley has been fix these issues in the design system, he might have some advice.

I'm drafting a blog post for some of the techniques we're using: https://docs.google.com/document/d/1YWgszOPxDY-OdMHc43A9GKmucQUM-299EBBw4X-mrgo/edit?usp=sharing
You could consider:

Inline SVG with fill=currentColor
CSS Triangles

.collapsible__toggle-icon {
    position: absolute;
    top: 0;
    right: 0;
    left: 0;
    bottom: 0;
    margin: auto;
}

.collapsible__toggle-icon::after {
    display: block;
    content: '';
    position: absolute;
    top: 0;
    bottom: 0;
    right: 0;
    left: 0;
    margin: auto;
    width: 10px;
    height: 10px;
    transform: rotate(45deg);
    border: solid;
    border-width: 0 2px 2px 0;
}

.collapsible.is-open .collapsible__toggle-icon::after {
    transform: rotate(-135deg);
}

The second solution which was now implemented doesn't work with a black background. The border that creates the arrow is always black, even when it's been given a colour (like currentColor or inherit or even #f00).


Rebuild release version

Run grunt to rebuild versions

Closing this, since the release is built during the version bump.

Great, new version released :)


Mid-year review - TSTerm

Please share any additional details on this topic
Mid-year review - TSTerm
Detail what actions or feedback you would like from the TAC
@joenemo FYI - scheduled for June

Landscape change made at https://github.com/openmainframeproject/omp-landscape/commit/44ff2cc2589454183bd3598a2cb243871fbade3f

Closing in favor of #506


Support .tgz

And any other prevalent compressed files in the archive?

Now that I think about it, .tar.Z is moderately common for files from the early 90s.
I think all other compression formats are old platform-specific ones like .sit, .sea, .hqx, .bin (classic MacOS). Not a priority.

Closed by #29


Tab not added to group when opened in another group

Scenario

Group A with rule to add pages from domain A
Group B with rule to add pages from domain B
Page on domain A contains link which

leads to domain B page
opens link in new tab

Expected behavior
New tab is added to Group B
Current behavior
New tab opens in Group A

This refers to the same issue as #5.
As described there, this behavior is intentional, but I intend to offer an option to control it.
I'll close this as a duplicate, please subscribe to #5 to get notified about progress on this. 🙂


Support for SK6812 - RGBW LEDs

will be great if you can add support for the FabLab-Luenen/McLighting fork which supports SK6812 RGBW LEDs (additonal W channel). https://github.com/FabLab-Luenen/McLighting

0.0.12


Lost azure.mgmt.authorization.v2020_04_01_preview

Package Name: azure.mgmt.authorization
Package Version: 3.0.0
Operating System: RHEL8
Python Version: pip 21.3.1 from /usr/local/lib/python3.6/site-packages/pip (python 3.6)

Describe the bug
ModuleNotFoundError: No module named 'azure.mgmt.authorization.v2020_04_01_preview' when running commands related to azure.mgmt.authorization.
To Reproduce
Steps to reproduce the behavior:

pip install azure-mgmt.-authorization
az grafana create -g rg-poc-aks -l EastUS --name grafana-poc-aks1

Expected behavior
Should be No error in this step.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
12:13:19 pinhuang@rhel8 0 /usr/lib64/az/lib/python3.9/site-packages/azure/mgmt/authorization
$ ls
_authorization_management_client.py  models.py    _serialization.py  v2018_01_01_preview  v2021_01_01_preview  _version.py
_configuration.py                    __pycache__  v2015_06_01        v2019_08_01_preview  v2021_12_01_preview
__init__.py                          py.typed     v2015_07_01        v2020_10_01          v2022_04_01

12:30:37 pinhuang@rhel8 0 /usr/lib64/az/lib/python3.9/site-packages/azure/mgmt/authorization
$ cat _version.py
# coding=utf-8
# --------------------------------------------------------------------------
# Copyright (c) Microsoft Corporation. All rights reserved.
# Licensed under the MIT License. See License.txt in the project root for
# license information.
# --------------------------------------------------------------------------

VERSION = "3.0.0"

12:35:13 pinhuang@rhel8 0 /usr/lib64/az/lib/python3.9/site-packages/azure/mgmt./authorization
$ ls -la v2020_04_01_preview
ls: cannot access 'v2020_04_01_preview': No such file or directory

Related issue https://github.com/Azure/azure-cli/issues/26630

Thanks for the feedback @pichuang . We will investigate and get back to you asap.

This is an issue related with Azure CLI so I will transfer it to CLI repo.


#5209 activity tab

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[ ] Feature
[x] Refactoring

Context

[x] Closes #5209
[ ] Requires deployment <>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </bsx/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Had issue bounty label?

[x] Fill up your KSM address:
Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

the button does not work on the full width and height - please make the select are the whole button.

if the name is too long we can I think cut it so its not jumping to the second row - that is the ideal situation that no content is jumping to second row
can we also increase the vertical padding? something like 20 is good - between the buttons and the stroke.

Left:

[ ] if the name is too long we can I think cut it so its not jumping to the second row (somewhere on the edge with the second name

can this be done in another issue? bit tricky to change the class in oruga table as this is deeply nested and would be through unscoped css

fixed for div, for label , it seems is-clickable is mandatory as even if we wrap it in a , the label tag inside it still wont show pointer without is-clickable

good! works for me

pay 25 usd

😍 Perfect, I’ve sent the payout
💵 $25 @ 35.44 USD/KSM ~ 0.705 $KSM
🧗 EzGc4s9PgCPx1YnF3fqzhLzVHpHMTL4LWPScwpDrR8JKgSU
🔗 0xa86e1cab2a14ff1fed6612c4a0e0d059faec44cc8b3287c5a35a15f95e9db82f
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


hello world in kalam(hindi) programming language

Kalaam is the programming language in Hindi and has all the basic functionalities of a modern programming language.
Official language doc: https://github.com/Kalaam-Programming-Language/Kalaam

Thanks for the pr!


Does not open pdf

i dont know if is a error or is a possible feature to you implement,  but will be great has this working.

Does not open pdf

Are you referring to not being able to "open" the PDF file, or "preview" it?
If it's "preview," have you already installed the latest poppler as instructed in the README?

i refer to be able to open the pdf file. when press w to see the tasks running seems they run an open command and the process being finished and exited with status code 127.
![20230915_22h10m46s_grim](https://github.com/sxyazi/yazi/assets/98425664/9c84bdc8-f
![20230915_22h12m34s_grim]
(https://github.com/sxyazi/yazi/assets/98425664/1d088e48-6a4c-4da3-809a-b35c9a0152dc)
55f-4dfd-9e35-66e5eb22e450)

See https://github.com/sxyazi/yazi/issues/110

Please reopen this issue if the problem persists, closing then


.jpg Image can't decode

Yes I've run into this, too.
Will this somehow be updated? I think it's a nice idea.
Hopefully this Lib is not created and immediately abandoned.

Hi,
Can you provide a full example for me to reproduce the jpg issue?

Hopefully this Lib is not created and immediately abandoned.

Yeah it's be a bit sad. I created this lib mainly as an experiment to see what could be the performance of a dart-only QR decoder bind to the standard flutter camera stream.
I plan to have another look when Dart Isolate become faster (https://github.com/dart-lang/sdk/issues/46754).

Hi, sorry - it's now working with "decodeImage" instead of "decodeJpg". The PNG version worked then I tried JPG and had an error and so this issue so I thought it's the same. Now I have no errors and it works.
I use this on the server since we have users that have incredibly slow smartphones that struggle with detection.
Somehow this lib is able to decode QR from jpg with a quality of 0.05 (canvas.toDataUrl('image/jpeg', 0.05)) which is nice since for a scan I just send this low resolution image to the server which is 5kb.
This is really fine for our usecase and other libs don't have such a great detection.
Example (rectangle is to hide qr data):

This is detected just fine. We would also have use cases for bar codes. Thanks! :-)
Please keep it running I think it's really good to port such code bases.


add spice memecoin

Description

Adding memecoin Spice from this Streamswap
Checklist

Adding Assets

If adding a new asset, please ensure the following:

[x] Asset is registered to the Cosmos Chain Registry. PR pending here
[x] Add asset to bottom of zone_assets.json.

[x] chain_name and base_denom are provided and use values exactly as defined at the Chain Registry.
[x] path is provided, and the IBC channel referenced is registered at the Chain Registry (unless native to Osmosis).
[x] osmosis_verified is set to false
[x] listing_date_time_utc is specified and accurate
[x] Optional: transfer_methods, peg_mechanism, override_properties, canonical, categories, where necessary (see README for details).

[x] I am aware that upgrading an asset to 'verified' status requires an additional PR to this repo (checklist below).

Thanks for your submission. This assetlist will be listed on Osmosis Zone as an Unverified asset. Please create an additional PR to this repo if this asset should be verified. (Instructions)


adding support for https requests handling in OAuth2SecurityInterceptor

@sameera-jayasoma , please review and merge this.


Fancy indexing with a list instead of a tuple

Currently, MXNet's fancy indexing will fail when indexing with a list rather than a tuple (while it works with NumPy or PyTorch).
Minimal example to reproduce the issue:
import mxnet.ndarray as nd

# Creating a simple third order tensor
a = nd.arange(24).reshape((3, 4, 2))

# This works fine
a[slice(None), [1, 2, 1], [0, 1, 2]]

# This should work but does not
a[[slice(None), [1, 2, 1], [0, 1, 2]]]

# This works as expected
a[tuple([slice(None), [1, 2, 1], [0, 1, 2]])]

Thanks for reporting this. There are some concerns about inconsistent indexing behavior using lists in the numpy world.
https://github.com/numpy/numpy/issues/4940
https://github.com/numpy/numpy/issues/6564
That's why we don't want to introduce this kind of inconsistency into mxnet. From those threads, you can see that it's encouraged to avoid using lists in numpy, so as in mxnet.

Good point!
For the sake of user friendliness, would it be worth either accepting the list and throwing a warning or raising an informative error (just suggesting the user to use tuples)? Can be tricky to track down the source of the error otherwise :)

Good suggestion! We will make the error message clear and informative.

hi @reminisce was there a pr to make the changes to the error messages? if so can we close out this issue?

Btw. NumPy has a futurewarning to disallow lists being interpreted as tuple (forcing tuples for N-D indices). So you guys are already implementing the future I guess (did not check it carefully)


modify the feature of quick input, when overflowed the elipsses shows…

… on left

This PR fixes #143956

@kipackjeong can you attach a screenshot of the change?
Additionally, this isn't finished yet because we should make a new CSS class to do this behavior and then add it to the respective html tags that are created in the JS:
https://github.com/microsoft/vscode/blob/main/src/vs/base/parts/quickinput/browser/quickInputList.ts#L113
Then once you have that working we can talk about adding a setting to vscode that will allow you to set this behavior.

.

@TylerLeonhardt
Thank you for the feedback and please forgive my shortcomings, this was my first time contributing to the community project.
I have attached the screenshot of the changes. However, I do not understand why we have to make a new CSS class. I would appreciate in depth explanation on this please. Thank you!

@kipackjeong no worries and by the way, Draft PRs are totally fine to open. I'm happy to help you along the journey, just be patient with me as I have a few things I am working on at the same time.
We wanted the behavior you have introduced to be behind a setting like search.quickOpen.overflowOnLeft which would control whether the ellipsis would be shown on the right or left of the line.
In order to do this, I suggest making a CSS class (or a few if that's what is needed) that we can programmatically add here:
https://github.com/microsoft/vscode/blob/main/src/vs/base/parts/quickinput/browser/quickInputList.ts#L113
based on if our new setting is set.

@TylerLeonhardt I made changes respective to your comments, can you review this please.
Created new CSS Class: .left-ellipses
Added on labelContainer, nameContainer, and descriptionContainer in both the css file and in dom:

label-container.left-ellipses
name-container.left-ellipses
description-container.left-ellipses

Added this style on those three elements :
.quick-input-list .monaco-icon-label-container.left-ellipses {
    display: flex;
    align-items: center;
}

.quick-input-list .monaco-icon-name-container.left-ellipses {
    display:inline-block;
    block-size: 1.5rem;
}
.quick-input-list .monaco-icon-description-container.left-ellipses {
    display:inline-block;
    block-size: 1.5rem;
    overflow: hidden;
    text-overflow: ellipsis;
    width: 100%;
    direction: rtl;
    text-align: left;
}

For dom manipulation, when data.label is created with IconLabel class Instance, created and added new option ellipsesOnLeft
/* quickInputList.ts */
        data.label = new IconLabel(row1, { supportHighlights: true, supportDescriptionHighlights: true, supportIcons: true, ellipsesOnLeft: true });

Then added ellipsesOnLeft logic inside of IconLabel contsructor
/* iconLabel.ts */

if (options?.ellipsesOnLeft) {

            this.labelContainer.classList.add('left-ellipses');
            nameContainer.classList.add('left-ellipses');
            this.descriptionContainer.element.classList.add('left-ellipses');

        }

Now works well
When ellipsesOnLeft option is false

When it is true

Maybe give the setting a less specific name and make it an enum with only two values at the moment, left and right. That leaves scope for future enhancement.

@gjsjohnmurray I think that is a good idea. For instance like Ellipsis.Left and Ellipsis.Right?

@kipackjeong I'm thinking: EllipsisLocation with Left and Right and then hopefully in the near future we can figure out a safe and fast way to do ellipsis in the middle and we can have a Middle option.

Once you get Left and Right supported, you can make it a true VS Code setting by adding to the following:
https://github.com/microsoft/vscode/blob/dafc40bca5439520d7e8b1c4a5c15c744bf69c10/src/vs/workbench/contrib/search/browser/search.contribution.ts#L892-L911
Then the plumbing fun begins...
follow the reference here:
https://github.com/microsoft/vscode/blob/0e8fcebfbc9f5ecb1e7db0750fc3eb5a9a8cad69/src/vs/workbench/contrib/search/browser/anythingQuickAccess.ts#L197
alllllll the way down to the IconLabel.
I guess this issue was a bit more complicated than I thought... 😅

@TylerLeonhardt Thank you and sorry it has been over ten days of no reply, I was running so busy.
I added an option on
search.contribution.ts
and also added on
anythingQuickAccess.ts
	private get configuration() {
		const editorConfig = this.configurationService.getValue<IWorkbenchEditorConfiguration>().workbench?.editor;
		const searchConfig = this.configurationService.getValue<IWorkbenchSearchConfiguration>().search;
		const quickAccessConfig = this.configurationService.getValue<IWorkbenchQuickAccessConfiguration>().workbench.quickOpen;

		return {
			openEditorPinned: !editorConfig?.enablePreviewFromQuickOpen || !editorConfig?.enablePreview,
			openSideBySideDirection: editorConfig?.openSideBySideDirection,
			includeSymbols: searchConfig?.quickOpen.includeSymbols,
			includeHistory: searchConfig?.quickOpen.includeHistory,
			ellipsisLocation: searchConfig?.quickOpen.ellipsisLocation,
			historyFilterSortOrder: searchConfig?.quickOpen.history.filterSortOrder,
			shortAutoSaveDelay: this.filesConfigurationService.getAutoSaveMode() === AutoSaveMode.AFTER_SHORT_DELAY,
			preserveInput: quickAccessConfig.preserveInput
		};
	}

@TylerLeonhardt Thank you and sorry it has been over ten days of no reply, I was running so busy. I added an option on search.contribution.ts and also added on
  'search.quickOpen.ellipsisLocation': {
	  type: 'string',
	  default: 'right',
	  enum: ['left', 'right'],
	  enumDescriptions: [
		  nls.localize('quickInput.ellipsisLocation.right', 'The ellipsis will show on the right side when the file path string overflows'),
		  nls.localize('quickInput.ellipsisLocation.left', 'The ellipsis will show on the left side when the file path string overflows')
	  ]

  },

anythingQuickAccess.ts
	private get configuration() {
		const editorConfig = this.configurationService.getValue<IWorkbenchEditorConfiguration>().workbench?.editor;
		const searchConfig = this.configurationService.getValue<IWorkbenchSearchConfiguration>().search;
		const quickAccessConfig = this.configurationService.getValue<IWorkbenchQuickAccessConfiguration>().workbench.quickOpen;

		return {
			openEditorPinned: !editorConfig?.enablePreviewFromQuickOpen || !editorConfig?.enablePreview,
			openSideBySideDirection: editorConfig?.openSideBySideDirection,
			includeSymbols: searchConfig?.quickOpen.includeSymbols,
			includeHistory: searchConfig?.quickOpen.includeHistory,
			ellipsisLocation: searchConfig?.quickOpen.ellipsisLocation,
			historyFilterSortOrder: searchConfig?.quickOpen.history.filterSortOrder,
			shortAutoSaveDelay: this.filesConfigurationService.getAutoSaveMode() === AutoSaveMode.AFTER_SHORT_DELAY,
			preserveInput: quickAccessConfig.preserveInput
		};
	}

But couldn't figure out how I can use this in InputLabel to manipulate the dom.

@TylerLeonhardt Hello Tyler, any thoughts on it yet?

Referencing ticket: #156062
Exactly what I was looking for.
The path is truncated from the left, like in macOS Spotlight. Tested this patch with the latest dev version and it works fine.

Note: in a kind of sick outcome, #156062 got closed as a dup of #58040, and both are closed and locked.
Related on Stack Overflow: How do I disable the long path in visual studio code quick file opener?

aha why no reply then

Sorry for the delay, if you'd like to still continue, please handle the merge conflicts. I also would like to see a setting for this behavior so it's opt-in. Basically you will need to pass the value all the way down and add a CSS class to the InputBox depending on the setting value.


Adding optional evidence field to credential creation

Overview
Adding optional evidence field to create credential
Description
Adding evidence as described here - https://www.w3.org/TR/vc-data-model/#evidence
fixes - https://github.com/TBD54566975/ssi-service/issues/550
{
  "@context": [
    "https://www.w3.org/2018/credentials/v1",
    "https://www.w3.org/2018/credentials/examples/v1"
  ],
  "id": "http://example.edu/credentials/3732",
  "type": ["VerifiableCredential", "UniversityDegreeCredential"],
  "issuer": "https://example.edu/issuers/14",
  "issuanceDate": "2010-01-01T19:23:24Z",
  "credentialSubject": {
    "id": "did:example:ebfeb1f712ebc6f1c276e12ec21",
    "degree": {
      "type": "BachelorDegree",
      "name": "Bachelor of Science and Arts"
    }
  },
  "evidence": [{
    "id": "https://example.edu/evidence/f2aeec97-fc0d-42bf-8ca7-0548192d4231",
    "type": ["DocumentVerification"],
    "verifier": "https://example.edu/issuers/14",
    "evidenceDocument": "DriversLicense",
    "subjectPresence": "Physical",
    "documentPresence": "Physical",
    "licenseNumber": "123AB4567"
  }],
  "proof": {  }
}

How Has This Been Tested?

Added to integration test
Added to unit test

Should we add validation code to the sdk?
https://github.com/TBD54566975/ssi-sdk/blob/main/credential/model.go#L29
It seems like the only validation that is required from the spec is:
The precise content of each evidence scheme is determined by the specific evidence [type](https://www.w3.org/TR/vc-data-model/#dfn-type) definition.

@andresuribe87  Like only have "type": ["DocumentVerification"], and a few others?
And then make sure the DocumentVerification only matches the matches the DocumentVerification schema?
Do you know where can I find what DocumentVerification is and what fields it requires?

Like only have "type": ["DocumentVerification"], and a few others?

Similar. To be precise, the only type I see us supporting is what's specified in https://docs.google.com/document/d/1y65HA9ZP58ppySG-yiBhGdNwsFrJZ8_-zupOyasuqgk/edit#heading=h.pvfljv28woox (i.e. https://openid.bitbucket.io/ekyc/openid-connect-4-identity-assurance.html#name-verification-element).

Do you know where can I find what DocumentVerification is and what fields it requires?

AFAIK, no one has implemented the DocumentVerification type. We should implement the type described above, since that's going to be used for KYC purposes.

I don't think we should implement a specific type until it's in the VC specs directory. I'm in favor of leaving it more generic for now.

I don't think we should implement a specific type until it's in the VC specs directory. I'm in favor of leaving it more generic for now.

SGTM

Codecov Report

Merging #553 (1c6ee3c) into main (3761db2) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #553      +/-   ##
==========================================
- Coverage   20.00%   19.97%   -0.03%
==========================================
  Files          50       50
  Lines        6175     6182       +7
==========================================
  Hits         1235     1235
- Misses       4712     4719       +7
  Partials      228      228

Impacted Files
Coverage Δ

pkg/server/router/credential.go
2.93% <0.00%> (-0.02%)
:arrow_down:

pkg/service/credential/model.go
0.00% <0.00%> (ø)

pkg/service/credential/service.go
0.00% <0.00%> (ø)


ctrl-n skips lines

I'm using atomic-emacs 0.3.5 with Atom 0.158.0 on OS X 10.10.1

press ctrl-n to move to the next line
disable atomic-emacs package
enable atomic-emacs package
pressing ctrl-n now moves two lines (skipping one line)
disable then enable again; ctrl-n now moves three lines (skipping two lines)

I have to run "Window: Reload" to get back to normal behavior.

Thanks for the inputs everyone, it will help me debug the cause of this issue.

Ok, figured out why it's happening.
  previousLine: (event) ->
    if atom.workspaceView.find('.fuzzy-finder').view() or
       atom.workspaceView.find('.command-palette').view()
      debugger # After activating the fuzzy finder or the command palette, all subsequent calls to this will trigger the debugger
      event.abortKeyBinding()

    @editor.moveCursors (cursor) ->
      cursor.moveUp()

The lines above are there to exempt some views from atomic emacs keybindings, otherwise, those views won't have an up/down keybinding at all.
I'll be pushing a fix shortly ;)

Fixed by d92c41a748e5f12c727fc472d0e34e557c7a2693. Apparently, those lines aren't needed anymore. Please update to 0.3.6 :)

This appears to be happening again on version 0.170.0 of Atom

@bohlander looks like a new issue #37 is tracking this


add hugo-theme-yue

Hi, @TomasBankauskas . I searched yue in https://jamstackthemes.dev/#ssg=hugo and didn't find this theme. Is there anything wrong?


Added database collation to default configuration

As part of the development process, I have needed to drop and create the database using build.js from the extension-tools repository. The collation value is required. There will be an accompanying pull request for the other repository.

We don't have 'collation' parameter as far as I know.

That is true. Unfortunately, creating the database as part of the extension-tools repo requires a value for this. On a related note, it might be useful to add collation to the list of parameters for Espo because it is important for the consistency of the database installation.

I got an intention now.

Espo does not have the functionality that creates database. PostgreSQL does not have collation parameter. Hence no need to have this parameter. I'm not sure whether the database/schema collation has any effect of column collations, they are created with a proper collation by Espo.

Maybe I'm mistaken regarding creation functionality. During installation maybe it can create. Didn't use it for long, don't remember.

My intention isn't to change the Espo installer. This is only for development purposes, which meanas the developer might need access to the collation value during the databaseReset() process. If you do not specify a collation, mysql defaults to utf8mb4_0900_ai_ci, which is not what I would have expected. Regardless, I understand if you don't want to merge this and the databaseReset() functionlaity. I will keep the code in my fork and merge around it when I get updates to both repositories. I just thought this might be useful for other developers.

When it defaults utf8mb4_0900_ai_ci, does it cause any problems?

I have not encountered a problem yet and I use Espo for many hours per day, so probably not. It leaves the developer/installer to the mercy of MySQL's defaults, though. utf8mb4_0900_ai_ci is new as of MySQL 8.0, so some people may be unfamiliar with it.
It is possible to add the databaseReset functionality without the collation variable. Removing the last part of the string where the collation is set causes the collation to default to utf8mb4_0900_ai_ci, which seems fine if you don't want to add the collation parameter.

As far as I know, the database collation value should not have any effect on Espo, hence we can neglect it.


'dotnet restore' unhandled exception with NuGetCredentialProvider

Hey team,
I'm not sure if this is an issue with the dotnet CLI or with the credential provider. I'll try this repository first. I've also reviewed the other open issues and it this doesn't appear to be related.
We are using Azure Artifacts and restoring packages in a docker container:
WORKDIR /nuget_cache_warmer
COPY **/*.csproj ./
RUN for f in ./*.csproj; do dotnet restore $f --source $NUGET_SOURCE --source https://api.nuget.org/v3/index.jsonl; done

This is our base image: FROM microsoft/dotnet:2.1-sdk
Intermittently, I start to see:
Unhandled Exception:  [0m [91mSystem.Threading.Tasks.TaskCanceledException: A task was canceled.
NuGet.Protocol.Plugins.MessageDispatcher.DispatchWithNewContextAsync[TOutgoing,TIncoming](IConnection connection, MessageType type, MessageMethod method, TOutgoing payload, CancellationToken cancellationToken) at NuGet.Protocol.Plugins.SymmetricHandshake.HandshakeAsync(CancellationToken cancellationToken) at NuGet.Protocol.Plugins.Connection.ConnectAsync(CancellationToken cancellationToken)
at NuGet.Protocol.Plugins.PluginFactory.CreateFromCurrentProcessAsync(IRequestHandlers requestHandlers, ConnectionOptions options, CancellationToken sessionCancellationToken)
at NuGetCredentialProvider.Program.Main(String[] args)
at NuGetCredentialProvider.Program.<Main>(String[] args)

This happens more than once during the restore processing, but not for every .csproj file.
What is interesting compared to other issues is the restore process will complete, and I can run the next docker step to build my project successfully.
If it matters, the docker build is running on a Linux TeamCity agent. If I run dotnet restore --disable-parallel, the exception does not occur, however the restore takes twice as long (as expected).
Since I can still restore successfully this isn't a huge problem, though I thought I'd report it.

This could be a docker issue or related to this open issue with dotnet https://github.com/NuGet/Home/issues/7842.
FYI @shubham90 who worked on the docker implementation - I forgot if the TaskCanceledException was resolved with some of the docker images?

Hello @satbai, yes that seems somewhat related. What's odd though is I've tested it on two different projects and although I see the same exception, the restore process still completes successfully. Unlike the issue you've referenced, it appears that the restore process exits with error code 1.
I'd appreciate if @shubham90 has anything to share. I'm rolling this out to our org and I'd like to make sure it's safe to proceed.

Could you run it with detailed verbosity and share the logs please? I'm interested in seeing the detailed cred provider logs. The verbose logs should also show the cred provider version. Also, if you're not using the latest version cred provider, I'd recommend updating it.

Hey @satbai, can I get your e-mail address that I can send the logs to? I don't want to leak sensitive info in the logs, if there are any (it's pretty long).

Sure, its my github alias at microsoft dot com

After looking at the logs, I think what's happening is this:
The nuget.config file has two endpoints, an Azure Artifacts feed and api.nuget.org endpoint. Nuget thinks it needs credentials to get a package from the Artifacts feed so it launches the cred provider but all packages get fetched from nuget.org before the cred provider even starts logging. Since the task finished, the cred provider gets canceled. This is a valid cancellation scenario, so there seems to be a bug in the cancellation handling. I think the cancellation doesn't interrupt the build because the task it threw the exception is already finished so there's no task to stop processing. The build is already processing the next step.
To get rid of the exceptions, as a workaround, what I think will help is to remove one of the package source endpoints from nuget.config, because it looks like the same packages can be found in both endpoints.
There are some fixes in related areas in the next nuget/dotnet release that's coming soon, I think later this month, so I'd like to see if the new version will get rid of these exceptions. If this isn't fixed with the new version, we'll have to dive deeper to see why a valid cancellation scenario throws unhandled TaskCanceledExceptions.

Thanks a lot @Satbai! We don't cache packages on Azure Artifacts from the public feed so unfortunately we do pass both feeds using the --source parameter.
Do you think that this periodically fail a build if it does find a task to cancel?
I'm hoping I can get your advice: I'm migrating our systems to use Azure Artifacts and I am not sure if I should wait for a fix for this problem or go ahead with the change because the error appears to be harmless. What would you suggest based on your findings?

I might be mistaken, but if my assumptions are correct, theoretically it's possible that sometimes the build might actually fail. My confidence in this area is limited though, so adding @nkolev92 and @dtivel to make sure my theory makes sense in the nuget/dotnet side.

Do you agree that unhandled TaskCanceledException shouldn't be thrown if the restore task finishes using nuget.org package source endpoint while trying to launch the plugin cred provider to authenticate to an Artifacts feed, because the packages can be found in both sources? Instead, the cancellation in this case is valid and expected?
In my previous comment, I mentioned "I think the cancellation doesn't interrupt the build because the task that threw the exception is already finished so there's no task to stop processing. The build is already processing the next step." Does this theory make sense? Is that how nuget/dotnet handles exceptions?

"We don't cache packages on Azure Artifacts from the public feed so unfortunately we do pass both feeds using the --source parameter."

@AMoghrabi, do you mean that you prefer not to use nuget.org as an upstream source in the Artifacts feed? Setting that upstream source would likely solve this.

@Satbai Sorry for the delay!
That's correct -- we prefer not to set nuget.org as an upstream for two reasons:

If, for some reason an upstream package is deleted from Azure Artifacts, the version number cannot be reused. This may cause problems with retention policies. Actually I'm not really sure how the policies work in regards to cached packages, but we couldn't find any information on this online and we want to avoid it altogether.

We prefer not to pay for cached packages 😬 On our private nuget server that we're migrating from, we have about 10GB of cached packages from nuget.org.

@satbai, I agree that an unhandled TaskCanceledException should not be thrown in the scenario you describe.  A TaskCanceledException should be thrown, but it should be handled.
I recommend that you catch TaskCanceledException here and silently exit.

I can look into making that change if you're busy @satipai :)

@AMoghrabi thanks for preparing the PR and sorry for the delay - I was out of office for some time. I'll have a look at the PR today!

To your earlier point about not using upstreams - yes, cached packages do get charged the same manually pushed packages do.
When it comes to retention policies though, currently the packages that you don't want retention policies to delete, you can promote to a view. Packages that are in views are "safe" from retention policies because there's no way to "unpromote" other than manually deleting the package. However, we're planning to revisit the retention/immutability space in the near future to determine if the current experience is the right approach so, if you're interested in chatting about it more, the Artifacts program manager @elbatk is available.

Linking to https://github.com/NuGet/Home/issues/8528, as there may be another fix needed in NuGet to resolve this. I'm guessing we'll also still want to catch TaskCanceledException during credprovider handshake and exit cleanly (@AMoghrabi's PR), though I'm going to wait until the issue(s) are truly fully understood before merging that.

Thanks for the help everyone, closing this as Jonathan released a new version with my fix.


after 10 failed attempts, finally updating the model-image workflow the right way

@Gregory-Pereira

LGTM
@lmilbaum PTAL


NDArray: Adding to discontinuous indices

I have the following case. Assume that I have have an NDArray object, A, with the shape (D, dim) where dim is any arbitrary shape and D is the minibatch size. I have another NDArray object, B, with the shape (d, dim) where d < D. Now, I have a mapping between the axis-0 indices of B and A, such that entry B[i] is mapped to A[j] where j >= i. My aim is to add each B[i] to A[j]. The naivest thing I could to is to iterate on B's axis0 and do for each B[i] the operation A[j] += B[i]. But each such addition operation is costly. To reduce the it, I keep the track of each consequent j entries in the A object and do addition as A[j1:j2] += B[i1:i2]. While this lowers the number of additions, still I have to do multiple ones.
I wonder if such an operation is possible currently: Let's say I know all the j indices, [j1,j2, ..., jd]. In numpy, it is possible to do operations by using the index array as A[[j1,j2,...,jd]] += B. Trying the similar operation results in "ValueError: NDArray only support continuous slicing on axis 0" with MxNet NDArray. My purpose is to execute this addition in a single command. Is this possible with the current MxNet API?

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


(fix) have the buffer reader call wrapped in a try catch in case of exception

Summary

A customer found an exception when BuilderVersion is being read.  It  seems to be an class initialization issue.  So, we are wrapping it in a try catch and logging the error before returning version unknown.

The "why", or other context.
Test plan
No additional tests were written.
Issues

customer found issue

Pull Request Test Coverage Report for Build 774

6 of 6 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.007%) to 89.953%

Totals

Change from base Build 770:
0.007%

Covered Lines:
2695

Relevant Lines:
2996

💛  - Coveralls


Write even if there is no capacity.

In my usage, I need have to write to the ring buffer, when there is no space, just auto advance some space.
Can I make a pr for the function WriteWithAdvance

Are you sure you understand the typical way/purpose of a ring buffer? When it is full it should block writers until the reader(s) can catch up. Thus it provides flow control.
I can see using a ring buffer as a kind of temporal LRU cache... in which case it would make sense to WriteAndMaybeOverwriteOldestData().
If that's what you have in mind, I would accept a PR with that (deliberately long and obnoxious name--to warn unsuspecting users).

I know what is ring buffer, it also looks great to store log with it. Just drop the oldest log is need to let the newest log can write into it.

In passing, for your log, hopefully you realize that if the log lines vary in length, you might obliterate the start of the oldest lines, leaving the buffer to start at a partial line.
The PR, once you've made the 3 changes I noted in the comments, is okay.

I went ahead and applied my edits and committed.


When input doesn't exist ..

If the input file doesn't exist you panic, which is perhaps less clean than it could be:
frodo ~ $ monkey-lang file/not/exists
Failure to read file ''. Err: open file/not/exists: no such file or directorypanic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0x4cf1a0]

goroutine 1 [running]:
main.main()
	/home/skx/go/src/github.com/bradford-hamilton/monkey-lang/main.go:57 +0x360

Thanks for the feedback. I will put this on my list. The project is not intended to be used by anyone, and is entirely an educational tool for me/anyone interested so unfortunately I am not going to be actively fixing/updating anything like this, but thanks for surfacing!


append tags to files/folders via cookies/system.ini

**Is your feature request related to a problem?
A clear and concise description of what the problem is. append tags to files/folders via cookies/system.ini
Describe the solution you'd like
A clear and concise description of what you want to happen. append tags to files/folders via cookies/system.ini This will allow, searching / reaching / sorting files easily
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered. - no solution in windows yet
Additional context
Add any other context or screenshots about the feature request here. Ex: What device type/UI layout and app version?

Duplicate of #1161


Please add a contributor license agreement

As a possible contributor I would like to know how I could contribute...

Hello,
You can contribute by pointing to possible implementation issues/errors or providing feedback about how to improve the implementation using pull requests. There is no contributor license agreement. The RISE V2G project is published under the MIT license.


Local front matter editURL not functionning and breaks rendering

Description

When specifying a local editURL in front-mater, the page fails to render because we use a with clause to set the user defined URL. But inside the {{ with }} the context is ., not .Params.editURL.
Steps To Reproduce

In front matter, specify editURL: http://myurl

Expected Behavior
editURL is correctly set and is used to render the editURL

Actual Behavior
Render fails with:
template failed at <partial "toc.html" .>: error calling partial:
"[root_path]/_vendor/github.com/imfing/hextra/layouts/partials/toc.html:30:55":
execute of template failed at <.Params.editURL>: can’t evaluate field Params in type string

Replacing
      {{- with .Params.editURL -}}{{ $editURL = .Params.editURL }}{{- end -}}

With
      {{- with .Params.editURL -}}{{ $editURL = . }}{{- end -}}

Fixes the bug.

@davidovich thank you for spotting this. it's been fixed in #195


App doesn't boot

App does not boot (start screen freezes).
Version: 1.5.1
Tried with multiple Android phones.

@screeper Should be fixed with 7ae0b13c46f45cbf3c6c1a22b98cd8d9ce658f46 and released to the Apple & Play Store as 1.5.2. Can you check?

Works fine. Thanks :)


Warn if using output target module with tied embeddings

Context
Solving issue #2018.

Raise a warning if the user is requesting an output target_module when the embeddings are tied, because this could lead to errors, for example when merging the adapter.
Also refactored the code to get the model config

Todo

[x] Try if load with tie_word_embeddings=False is an actual option. Load Gemma2 with finetuned different lm_weights and check that the lm_head is not replaced with the embedding (even if cloned). If it works, try to merge an adapter to lm_weight and then load it to check if embed and lm_head are kept separate. (the main concern is that the loading model's architecture might ignore any lm_head weight present in safetensors, as it happens in llama.cpp for example).
[x] In the end I only checked how to save the base model as untied, then the user can work from there, although I DID NOT DO  A THOROUGH CHECK OF THIS APPROACH, only checked this:

from transformers import AutoModelForCausalLM
import torch

# Load original tied model
model = AutoModelForCausalLM.from_pretrained("google/gemma-2-2b-it", tie_word_embeddings=False)
# Set the randomly initialized lm_head to the previously tied embeddings
model.lm_head.weight.data = model.model.embed_tokens.weight.data.clone()
assert torch.equal(model.lm_head.weight.data, model.model.embed_tokens.weight.data)

# Save the untied model
untied_model_dir = "tmp_model"
model.save_pretrained(untied_model_dir)
model.config.save_pretrained(untied_model_dir)
# Now use the original model but in untied format
model = AutoModelForCausalLM.from_pretrained(untied_model_dir)

assert torch.equal(model.lm_head.weight.data, model.model.embed_tokens.weight.data)
assert model.model.embed_tokens.weight.data.data_ptr() != model.lm_head.weight.data.data_ptr()

[x] Add in warning about porting to other formats.

I know this is all a bit more complicated that initially thought and not necessarily what you "signed up for".

Not at all thanks, sounds really good, I'll have a go!

Not at all thanks, sounds really good, I'll have a go!

Thanks a lot.

@BenjaminBossan I made a version addressing your suggestions. (I can't run the whole test suite as I do not have a cuda-compatible gpu). Also, I refactored getting the model config in the code base.

However, the warning message should be a bit different.

I feel like the new message can be the same. Let me know.

Sure very happy to write tests! I'll put them in tests/test_tuners_utils.py.
Just one question: to mock models with tied embeddings, should I use the test model "HuggingFaceH4/tiny-random-LlamaForCausalLM" but loaded with:
model = AutoModelForCausalLM.from_pretrained("microsoft/Phi-3-mini-4k-instruct", tie_word_embeddings=True)

@BenjaminBossan I added the test here tests/test_tuners_utils.py 👍

@BenjaminBossan I think I've address the comments 👍

Thanks for the latest updates. I only have one more question, namely when it comes to how to untie the weights. In the script you provide, you clone the weights but is that even necessary if tie_word_embeddings=False is passed? To give an example, I tried this (I changed the model but only because I had already downloaded it):
>>> from transformers import AutoModelForCausalLM
>>> model = AutoModelForCausalLM.from_pretrained("google/gemma-2b", tie_word_embeddings=False)
>>> model.model.embed_tokens.weight.data_ptr()
126062054867008
>>> model.lm_head.weight.data_ptr()  # <= different data ptr
126051845931072
>>> model.model.embed_tokens.weight.sum()
tensor(952564.6250, grad_fn=<SumBackward0>)
>>> model.lm_head.weight.sum()
tensor(255.3427, grad_fn=<SumBackward0>)

>>> from peft import LoraConfig, get_peft_model
>>> config = LoraConfig(init_lora_weights=False, target_modules=["embed_tokens"])
>>> model = get_peft_model(model, config)
>>> unloaded = model.merge_and_unload()
>>> unloaded.model.embed_tokens.weight.sum()  # <= embed weights changed
tensor(985655.8125)
>>> unloaded.lm_head.weight.sum()  # <= lm head stayed the same
tensor(255.3427)

In the script you provide, you clone the weights but is that even necessary if tie_word_embeddings=False is passed? To give an example, I tried this (I changed the model but only because I had already downloaded it):

Yes I agree with your script but the user wants to fix lm_head after loading. Because when loading with tie_word_embeddings=False the lm_head is randomly initialized, so the user wants to set it to equal the embed layer, which means they want a clone of it.
This cloning also seems to allow to save it correctly. If you do not clone (beside actaully re-tieing the embeddings), then when you load the saved-untied model the last assertion below will fail, otherwise, if you clone, it will pass:
model = AutoModelForCausalLM.from_pretrained("google/gemma-2-2b-it", tie_word_embeddings=False)
# Set the randomly initialized lm_head to the previously tied embeddings
model.lm_head.weight.data = model.model.embed_tokens.weight.data
assert torch.equal(model.lm_head.weight.data, model.model.embed_tokens.weight.data)

# Save the untied model
untied_model_dir = "tmp_model"
model.save_pretrained(untied_model_dir)
model.config.save_pretrained(untied_model_dir)
# Now use the original model but in untied format
model = AutoModelForCausalLM.from_pretrained(untied_model_dir)

assert model.model.embed_tokens.weight.data.data_ptr() != model.lm_head.weight.data.data_ptr()
assert torch.equal(model.lm_head.weight.data, model.model.embed_tokens.weight.data)

Because when loading with tie_word_embeddings=False the lm_head is randomly initialized, so the user wants to set it to equal the embed layer, which means they want a clone of it.

Oh wow, I did not know that the LM head will be randomly initialized, that's quite surprising IMO. I would have expected to get the same parameter values, just not tied. Thanks for making me aware of that.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Not sure how to reproduce the error in the git actions
ruff check src tests examples docs scripts docker
All checks passed!
ruff format --check src tests examples docs scripts docker
189 files already formatted
doc-builder style src/peft tests docs/source --max_len 119 --check_only
Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.8.18/x64/bin/doc-builder", line 8, in <module>
    sys.exit(main())
  File "/opt/hostedtoolcache/Python/3.8.[18](https://github.com/huggingface/peft/actions/runs/10578337704/job/29356715757?pr=2025#step:5:19)/x64/lib/python3.8/site-packages/doc_builder/commands/doc_builder_cli.py", line 47, in main
    args.func(args)
  File "/opt/hostedtoolcache/Python/3.8.18/x64/lib/python3.8/site-packages/doc_builder/commands/style.py", line 28, in style_command
    raise ValueError(f"{len(changed)} files should be restyled!")
ValueError: 1 files should be restyled!
make: *** [Makefile:11: quality] Error 1
Error: Process completed with exit code 2.

@ltoniazzi could you please run make style to make the linter happy? The ruff version being used is 0.6.2.

I ran make style locally and this is the diff I get:
@@ -530,8 +530,8 @@ model = AutoModelForCausalLM.from_pretrained(untied_model_dir)
     @staticmethod
     def get_model_config(model: nn.Module) -> dict:
         """
-        This method gets the config from a model in dictionary form.
-        If model has not attribute config, then this method returns a default config.
+        This method gets the config from a model in dictionary form. If model has not attribute config, then this
+        method returns a default config.

could you please run make style

Done!

@BenjaminBossan Thanks so much for your help! ❤
Btw, a test on main failed, do you think it's related to this PR?

Btw, a test on main failed, do you think it's related to this PR?

Don't worry, this is a known issue with X-LoRA that came about with a recent change in transformers.


Where are gridlines draw?

I'm trying to edit it so that a line chart only displays gridlines horizontally and not vertically. I've searched the code but don't see where they're painted.

Closed per the merge of #611
Use the following options to toggle vertical and horizontal grid lines respectively
scaleShowHorizontalLines
scaleShowVerticalLines


Add interface for a command with result (ICommand)

The library has IQuery<in TQuery, out TResult> interface.
There should be a similar version for commands that will allow for a result object ICommand<in TCommand, out TResult>. It should work in similar way as query does.
Purists will argue that commands must not return anything, but there are several good reasons for that:

it reduces complexity and simplifies your application, especially when you're not using eventual consistency nor client-generated ids.
This information will dramatically improve the user experience of your system, because you don’t have to poll an external source for command execution result, you have it right away. It becomes trivial to validate commands, and to return error messages.
If you want to refresh the displayed data, you can use the aggregate’s new version to determine whether the view model reflects the executed command or not. No more displaying stale data.

Example use cases:

Return information if record was deleted or not: public class DeleteFoo : ICommand<DeleteFoo, bool>
Return number of affected rows: public class DeleteFoos : ICommand<DeleteFoos, int>
Return new version number of your aggredate public class ChangeFoo : ICommand<ChangeFoo, int>
Return object containing more-verbose result of the operation: public class PerformComplexOperation: ICommand<PerformComplexOperation, OperationResult>
Return ID of created object in relational databases: public class CreateFoo: ICommand<CreateFoo, int>

See also: https://vladikk.com/2017/03/20/tackling-complexity-in-cqrs/

Hi @mjamro, this is something I can consider. I've found it useful to return a return type for failures and this change would enable that. However, I still feel that it would be worth encouraging users to not use commands for queries and limiting returned information, which I'm not sure how best to do. Encouraging patterns like Get-Post-Redirect instead of returning updated entities is one case I'd particularly like to discourage.
Is this issue something that you'd be willing to make a pull request for?

Sorry, i've been busy with different project (👶🏻)
Yes, i can prepare changes for this if it's free to take.

However, I still feel that it would be worth encouraging users to not use commands for queries and limiting returned information, which I'm not sure how best to do. Encouraging patterns like Get-Post-Redirect instead of returning updated entities is one case I'd particularly like to discourage.

 Documentation with explanation would have to suffice.

No worries at all, we're all busy doing something.
I haven't found the time yet either. So yeah, feel free to start a PR!


Inifinite loop in some while-loops

Infinite loop in some while-loops produced when we have a class that don't share the same vertical, object-oriented inheritance. In my case is produced when I try to introspect NSProxy class.
Right know the loop is broken when a root class (aka "clazz = Nil") is detected.
I make a workaround on the affected classes but of course I don't know all the scope of the project. Just let me know your thoughts!

Thanks @Ricowere !
I don't see any issues. Will release within next few days as Typhoon 3.0.3. In the meantime:
`ruby
pod 'Typhoon', :head

(if you're using CocoaPods)

It was a pleasure contribute in that library!
Thanks!
David.
2015-03-24 4:25 GMT+01:00 Jasper Blues<EMAIL_ADDRESS>
Thanks @Ricowere https://github.com/Ricowere !
I don't see any issues. Will release within next few days as Typhoon 3.0.3.
In the meantime:
`ruby
pod 'Typhoon', :head
(if you're using CocoaPods)
—
Reply to this email directly or view it on GitHub
https://github.com/appsquickly/Typhoon/pull/331#issuecomment-85324986.

--
[image: David Rico]  David Rico<EMAIL_ADDRESS>iOS Architect &
Scrum master http://giantuser.com t +34 652 622 611
https://goo.gl/maps/imHnm a 28300 • Madrid
https://goo.gl/maps/imHnm [image:
LinkedIn] https://es.linkedin.com/in/davidricoiosdeveloper/en[image:
Twitter] https://twitter.com/ricowere[image: Github]
https://github.com/ricowere[image: Skype]


Array Statistics Operations Integer Overflow

Array Statistics contains functions that calculate the mean and standard deviation of Number Array types.
The Calculate Mean function of ByteArray uses a sum method with an Int return type, which will overflow with enough elements.
Originally posted by @DK96-OS in https://github.com/DK96-OS/MathTools/pull/32#discussion_r794927783

ByteArrayExt provides a sum method that uses a long return type.
This method is now implemented by the Array Statistics calculateMean functions, as of #55.

LongArrayExt provides a sum method that uses BigInteger return type.
This method is now implemented by Array Statistics calculateMean function.

The Guava library can be used.
Methods like checkedAddition are nice, but will they affect performance?


[Constraint solver] Exit immediately from solve() if we already have …

…a failed constraint.
As solveRec() already does, exit immediately if we have a failed
constraint from constraint generation. Not doing so means that we can
create a SolverScope in constraint propagation, and on destruction of
that SolverScope we clear out the failedConstraint field, obscuring the
fact that we had a failure.

@swift-ci Please smoke test and merge

@swift-ci Please smoke test and merge


Processor only accepts 3 channel images

Hello,
I have been trying to finetune this model using my own data which is single channel images and some text describing features of the images.
The processer only seems to handle 3 channel images. To get around this I have stacked the images onto themselves to fill 3 channels. However, this seems like it will have a significant impact on the amount of data I will train the model on.
Is there currently support for single channel in a pretrain BridgeTower checkpoint? If not are there plans to include it? Is stacking images the best approach when working with single channel images for this model?

Hello,
since BridgeTower use CLIP as our vision encoder, we are also subject to certain limitations of CLIP, such as only being able to process 3-channel images as you mentioned. You can try to solve this problem by searching for "CLIP processing on single-channel images".


Fix gitter badge for npm

This newline is breaking the link for the gitter badge on npm.

Thanks @billt2006 !


Godot Jolt Compatibility Issue: HeightmapCollider not a power of 2

If unfamiliar, Godot Jolt is a GDExtension implementation of the Jolt Physics Engine. Boasts pretty great performance but does have a few drawbacks. Mainly HeightmapColliders are required to be a mesh that is a power of 2 size.
The specific error is here:
E 0:00:11:0142   build: Failed to build height map shape with {height_count=1050625 width=1025 depth=1025}. Width/depth must be a power-of-two. This shape belongs to 'StaticBody3D:<StaticBody3D#75715577261>' and 0 other object(s).
  <C++ Error>    Condition "!is_power_of_2((uint32_t)width)" is true. Returning: {}
  <C++ Source>   src/shapes/jolt_height_map_shape_impl_3d.cpp:87 @ build()

I do believe it is very likely I can solve this through use of particular clipmap_levels and clipmap_size values. But reading through the wiki I wasn't able to find much documentation on what these values are and how they work. I have a surface-deep understanding of clipmap terrains, but I don't know what these values translate to, or how to force them to produce a power of two value. I'd definitely appreciate this being integrated into the addon, but a band-aid solution works for me now. I'm really loving this addon and saw Tokisan's video about it today. Excited to see what comes.

Clipmap levels are for the mesh, not the collision shape.
Duplicate, see discussion. You're welcome to offer a solution there is you have one.
https://github.com/outobugi/Terrain3D/discussions/152