Repository should be called MVP instead of MVP-discuss

I'd like to just use one repository to discuss and to work.  I don't think the discussion by itself is super useful unless it will be translated into action.

I agree action is needed, but (1) ideally the action will involve more than one coding project (and thus more than one repo), (2) I don't see any problem with having a coding project in its own repo, (3) the discussion is inherently more general than any one coding project, since it includes the question of whether there should be some additional or different coding project.
Of course, once some specific coding project is happening (in its own repo), any discussion specifically about it would belong in that repo, not in this more general one.


NetCDF data convert

Hi，dear author, I have a netcdf4 file, how can I convert it to a netcdf3 file?

I believe that ToolsUI can do the job, you can try it out.

thank you very much！
Another question，Can I use the code of the wind field to visualize the oceanflow field? Just replace the source data？or？
------------------ 原始邮件 ------------------
发件人:                                                                                "RaymanNg/3D-Wind-Field"<EMAIL_ADDRESS>发送时间: 2020年9月28日(星期一) 晚上9:42
收件人<EMAIL_ADDRESS>抄送: "啥也不会的小菜鸟"<984281379<EMAIL_ADDRESS>主题: Re: [RaymanNg/3D-Wind-Field] NetCDF data convert (#41)
I believe that ToolsUI can do the job, you can try it out.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

It should be okay, but your data must have a consistent structure as the README describes

Thank you. I'll have a try------------------ 原始邮件 ------------------
发件人<EMAIL_ADDRESS>发送时间: 2020年9月28日(星期一) 晚上10:17
收件人<EMAIL_ADDRESS>抄送: "Rookiemin"<984281379<EMAIL_ADDRESS>主题: Re: [RaymanNg/3D-Wind-Field] NetCDF data convert (#41)

Hi，dear author.
The effect of visualization is as follows：
The reason for this is my data error or？
------------------ 原始邮件 ------------------
发件人:                                                                                "RaymanNg/3D-Wind-Field"<EMAIL_ADDRESS>发送时间: 2020年9月28日(星期一) 晚上10:17
收件人<EMAIL_ADDRESS>抄送: "啥也不会的小菜鸟"<984281379<EMAIL_ADDRESS>主题: Re: [RaymanNg/3D-Wind-Field] NetCDF data convert (#41)
It should be okay, but your data must have a consistent structure as the README describes
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

When I use other NC files, why does the flow of wind fields revolve around the North Pole instead of eliminating the actual situation of the file

When I use other NC files, why does the flow of wind fields revolve around the North Pole instead of eliminating the actual situation of the file


Plat 6982 - change the simulated prefixes for cplc

Looks like the build is failing? Wondering how it builds for the sdk, but something to look in to.

Build fixed.


build: add deploy script for node-bootstrap-container

@bcoe, I haven't created the trigger yet but I will specify it should run when we change the node-bootstrap-container folder


Posfix: enable smtpd_sasl_auth when permit_sasl_authenticated is sele…

…cted

How do you handle user management with using sasl? CLI?

How do you handle user management with using sasl? CLI?

yes.
saslpasswd2 -c -u <domain> <user>

os-postfix in 20.7.1 does not know smtpd_sasl_auth_enable anymore:
postconf: warning: /usr/local/etc/postfix/main.cf: unused parameter: smtpd_sasl_auth_enabled=yes
opnsense-revert -r 20.7 os-postfix reverts to working smtpd sasl auth
:-1:

Can you add a new if-clause near line 116 and remove it from current line?

Like this?

Does it help if I do the version bump?

Merged, thanks!


The special __DATA__ and __END__ tokens

This should be easy enough (though it may need to edit the scanner, b/c it gobbles until EOF. also col0).
Should we parse a __DATA__ that's found after an __END__?

Oh, lol; it doesn't need col0. Just gobbles until the end

may as well toss in ctrl-z and ctrl-d handling while we're at it


Change permissions text

Hi there,
I am trying to change the permissions text when the user opens the app for the first time. Is there a way that this can be achieved via code? I need to change the main text and not the description since i know that can be changed via info.plist property.
Thanks in advance

@jahnavi310 what do you mean with 'main text'? Can you post a screenshot?

A screenshot would really help in that case, but if the text you are describing is the one I am thinking of there is no way to change it. Better usability is usually achieved with a custom UI prior to the requestPermission action to prepare the user for the request and add additional information why this is required.


hard to read docs for domain types

The docs that Sphinx renders for domain types are not the most useful. They simply show the docs of the underlying class. This is a bit hard to read. And it can lead to problems with links, see https://github.com/scipp/essdiffraction/pull/43/commits/8a22410fb9086a09b256c125f662b301a4ae6986
I suggest we simply show the underlying type with a link to its docs. This may be easy to do with a custom template. But only if we keep domain types in a common module. Otherwise, it's hard to detect what are domain types and what not.

Isn't this improved with Python 3.10?

How so? Why would this change how Sphinx renders classes?

Because NewType was changed to a class instead of a function with 3.10.


Board Review: Support for JSON Merge Patch

The recently updated Azure API Guidelines recommend that update operations be implement using the PATCH method and specifically with JSON Merge Patch.  The preference for JSON Merge Patch is that it is naturally idempotent so no special support is required to ensure idempotency. JSON Merge Patch is also superior to PUT because PUT from a client that is not at the latest api-version can remove properties that were added in later api-versions.
But JSON Merge Patch, as defined in RFC 7386,  has special semantics, particularly for values passed as "null".

Null values in the merge patch are given special meaning to indicate the removal of existing values in the target.

This is a subtle detail that is often overlooked, so it is worth taking a look at this to see if we have any gaps.  There are at least 3 aspects to consider:
1) Server-side implementation
As many Azure services are implemented in ASP.NET Core, we should make sure there is proper support for JSON Merge-Patch provided in this framework and that it is well documented and easy to use.
2) Client-side support
Do our current generated (or hand-written) clients properly support passing "null" explicitly for a property value to an update operation that uses JSON Merge Patch? Apparently some languages do support this:

Go: has a NullValue() function that returns a reference to a T. This reference is to a singleton instance of T (our sentinel). When this particular T instance is serialized Go emits ‘null’. This works in Go because all properties are defined as reference types.

TypeScript/JavaScript: JavaScript supports this naturally because "null" is a distinct value from "undefined".

Java: Java does not have support for nulls in JSON Merge-Patch but the team has done some investigation on how it might be done:
https://gist.github.com/anuchandy/ca914f4a8dfaa49ea8e082de7e619c8f

The other languages may have support for this but I'm just not aware of it.
For languages that do have this support, does our testserver verify that it works correctly?
3) Client-side documentation
Even when our client libraries support passing an explicit "null", users need to know about it in order to use it properly. Does our SDK documentation describe this feature sufficiently and is that documentation easily discoverable by users?

I see two ways to do this in C#.
An Optional<T> approach
As called out above, this really comes down to a standard way of representing undefined, null, and values across languages.  Unfortunately languages like C# with separate value and reference types have a harder problem representing a PATCH API in their client libraries.  If I declare public int Age { get; set; }, I can't assign either null or undefined.  I could make it nullable with public int? Age { get; set; } which would allow me to assign null, but Nullable<T> is still a value type and I can't create a magic sentinel instance to represent undefined.
We would have to add our own Optional<T> that worked kind of like Nullable<T> to wrap patchable value types.  It isn't great for several reasons.  You'll have to unwrap the Optional and then Nullable to get a value (or we merge Optional and Nullable together which is probably even more confusing to C# devs).  If we implement Optional ourselves, it won't have the same language conveniences like operator lifting (i.e., in C# null + 2 == null), though one could argue maybe we shouldn't have those semantics anyway for undefined.
Please see https://github.com/Azure/azure-sdk/issues/1566 for a detailed proposal of how this might work - including a strawperson implementation and cross-references to related discussions.
Here's a quick example of what a model definition might look like:
public class Employee
{
    public int? Id { get; set; }
    public Optional<string> Name { get; set; }
    public Optional<int?> Age { get; set; }
    public Optional<string> Title { get; set; }
}

And here's how we'd use it:
// Create a new employee
var e = new Employee { Name = "Bob", Age = 42 };
e = client.CreateEmployee(e);
// sent { "name": "Bob", "age": 42 } over the wire
// received back { "id": 17, "name": "Bob", "age": 42 } from the service

// Lookup an employee
e = client.GetEmployee(e.Id);
// received back { "id": 17, "name": "Bob", "age": 42 } from the service
Console.Write($"Employee {e.Id}")
if (e.Name.HasValue) { Console.Write($" name: {e.Name.Value}"); }
if (e.Age.HasValue) { Console.Write($" age: {e.Age.Value.Value}"); } // Ewwwww
if (e.Title.HasValue) { Console.Write($" title: {e.Title.Value}"); }
Console.WriteLine();

// Change JUST the title
e = new Employee { Id = e.Id, Title = "Bob II" };
e = client.UpdateEmployee(e);
// sent { "id": 17, "title": "Bob II" } over the wire
// received back { "id": 17, "name": "Bob", "age": 42, "title": "Bob II" } from the service

Regular Models + LLC
Given that the API Stewardship Board is pushing idempotent PATCH operations for more and more scenarios, I worry that adding Optional<Nullable<int>> types is going to add a nontrivial conceptual overhead to otherwise simple champion scenarios.  Instead, I'd like to keep using regular models.  Imagine a model like:
public class Employee
{
    public int? Id { get; set; }
    public string Name { get; set; }
    public int? Age { get; set; }
    public string Title { get; set; }
}

And we could use it like:
// Create a new employee
var e = new Employee { Name = "Bob", Age = 42 };
e = client.CreateEmployee(e);
// sent { "name": "Bob", "age": 42, "title": null } over the wire
// received back { "id": 17, "name": "Bob", "age": 42 } from the service

// Lookup an employee
e = client.GetEmployee(e.Id);
// received back { "id": 17, "name": "Bob", "age": 42 } from the service
Console.Write($"Employee {e.Id}")
if (e.Name != null) { Console.Write($" name: {e.Name}"); }
if (e.Age != null) { Console.Write($" age: {e.Age.Value}"); }
if (e.Title != null) { Console.Write($" title: {e.Title}"); }
Console.WriteLine();

// Change the title (and potentially everything else)
e.Title = "Bob II";
e = client.UpdateEmployee(e);
// sent { "id": 17, "name": "Bob", "age": 42, "title": "Bob II" } over the wire
// received back { "id": 17, "name": "Bob", "age": 42, "title": "Bob II" } from the service

I think all of these semantics still make sense for the champion scenarios.  I don't really care that Title == null means "clear any existing Title" so much as I want an Update call to make the service resource look exactly like the model I'm manipulating.  While the Update scenario could stomp over other values if someone else had updated Age, services should still be using conditional requests even for PATCH updates to make this safe.
Further, if someone really wanted to update just the title, they could do so using our LLC overload:
client.UpdateEmployee(id: 17, body: RequestContent.Create(new { title = "Bob II" }));

Customers would have to know more about the shape of the wire protocol to do this (and it might not always be as simple as changing the case), but that's a problem we could solve with documentation/samples if we found a strong desire for patching individual properties.

To avoid the "stomp" problem described above, our patchable models could provide the following APIs:
public class Employee
{
    public int? Id { get; set; }
    public string Name { get; set; }
    public int? Age { get; set; }
    public string Title { get; set; }

    public Employee CreatePatch();
}

The usage would be:
e = client.GetEmployee(e.Id);
e = e.CreatePatch();
e.Title = "Bob II";
e = client.UpdateEmployee(e);
// sent { "title": "Bob II" } over the wire
// received back { "id": 17, "name": "Bob", "age": 42, "title": "Bob II" } from the service

Recording[MS INTERNAL ONLY]

One thing/requirement that is missing from the examples above is the ability to update without doing a GET/read first.

second discussion scheduled for 11/30

I experimented with the idea of "smart" models. Here is the result of the experiment: https://gist.github.com/KrzysztofCwalina/d5c1fba32eb4548f14bbf27184401731
I think smart models add a lot of complexity. Because of that, I would suggest we start with something like untyped JsonPatchBuilder, which users can use to create patch payloads stored in BinaryData or RequestContent, which in turn can be passed to "update" methods. We can think long term about simpler "smart models" if users complain.

Recording[MS INTERNAL ONLY]

Summary
Java and .NET
Will implement a PatchBuilder.

PatchBuilder will be implemented in core.

PatchBuilder will accept name / values and "build" a RequestContent.

Names can be "dotted" to set nested fields arbitrarily deep
Builder methods will allow fields to be "added" or "removed" (set to null)
"build" might be generic to allow checking that patch structure conforms to a particular class

The "update" method (that calls patch) accepts a Request Content.

https://github.com/Azure/azure-sdk-for-net/issues/27402

https://github.com/Azure/azure-sdk-for-java/issues/27510

Javascript
JavaScript has native support for "null" so it is covered.
Python
Python will continue to use it's special sentinel to signify "null"
https://azuresdkdocs.blob.core.windows.net/$web/python/azure-core/1.20.0/azure.core.html#azure.core.serialization.NULL
Go
Go will continue to use it's special sentinel to signify "null"
// NullValue is used to send an explicit 'null' within a request.
// This is typically used in JSON-MERGE-PATCH operations to delete a value.
func NullValue(v interface{}) interface{} {

Swift ???
Follow-on items

Documentation
Data Plane Codegen support
Test-server test
Update SDK guidelines

RFC 7386 is obsoleted by https://www.rfc-editor.org/rfc/rfc7396


Fix/test flakiness

testing CI!

A lot of the issues with UDP flakiness come from buffers that will disappear once we use a plain UDP socket instead of webrtc.
Other issues might need some kind of flow control to fix it.
We need to come back to this after replacing webrtc and see if more work is needed.


Add unit tests for util functions in PerfAnalyzer

Add more unit tests for perf_utils

Looks great overall! Nice work!


Incorrect scope for utf8 characters in labels

A clear and concise description of what the bug is.

ö does not get the "constant.other.reference.label.latex" that the other letters get
A minimal example to reproduce the issue

VS Code Version: 1.75.1
OS Version: W11
LaTeX-Workshop version if you use it: 9.7.0

This was because the regex class \w does not contains characters with accent. We know the \p{Alphabetic} utf-8 class.


Lightmap bake freezes the editor at "Determining optimal atlas size" when trying to bake certain geometry

Tested versions
v4.3.beta3.official [82cedc83c]
System information
Godot v4.3.beta3 - macOS 14.5.0 - Vulkan (Forward+) - integrated Apple M1 Max - Apple M1 Max (10 Threads)
Issue description
Editor freezes at "Determining optimal atlas size" completely and can only be closed by killing the process. The cube in this scene is missing its bottom face. Such geometry is fine in general (i.e the platform is missing the bottom face too, and it bakes fine on its own), however in this particular configuration where there's also a cube above it crashes. Note that the cube doesn't have to touch the platform for it to crash

Steps to reproduce
Open the MRP and bake
Minimal reproduction project (MRP)
MRP_determining_optimal_atlas_size.zip

Tested here and the MRP also freezes on Windows 11:
Godot v4.3.beta3 - Windows 10.0.22631 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3080 (NVIDIA; <IP_ADDRESS>12) - 12th Gen Intel(R) Core(TM) i7-12700KF (20 Threads)

Possibly related to #63423.

This seems to have been a hidden issue, only triggered in specific cases (Usually when the size of a single mesh's UV map was usable to fit into the whole atlas - the algorithm would try to add infinite layers to the image to have a clean layer to pack the mesh's UVs, but the layer would always turn out too small).  #91601 exaggerated this issue since it padded smaller images and made them too big to fit into the atlas.


Integrate to remote node machines

I have successfully integrated the docker swarm cluster with this project. Now I would like to monitor other/remote host machines.
Can I do it? If yes then where I need to do the changes.

Please search the existing issues this topic has been covered many times in other issues including #174 which has links to the aforementioned issues.


Cannot progress in the OPC-UA Lab

Hi, I'm trying to progress for study purpose in this lab "https://ibm.github.io/maximo-labs/monitor_managed_gw_opcua_9.0/".
When reaching the last part of the '2. Add device to library', after setting up the device and BEFORE clicking done, I copy the docker command and execute it in a separated command prompt, but it ends in this error:
2024-12-09T15:18:08.421417Z: INFO: edge-cli/src/lib.rs:106: Arguments: discover opcua --device 'XX_S7_OPCUA' --product 20ecf1d8-cc00-458b-b86b-a9696fb2d4bc --uri opc.tcp://<IP_ADDRESS>:50000 --workspace mmdev --user matheus --key b9adbNK-a3EFv4Gr1guh5JZZiyEhE3dMzyKe6Q4XRYQ --token H4owHxELRCOfetVFRBAA6lR0M46zxFVbFw0MCOgE9Fk
2024-12-09T15:18:08.421505Z: INFO: edge-cli/src/lib.rs:108: Version: 2.5.4
2024-12-09T15:18:08.421524Z: INFO: edge-cli/src/lib.rs:109: Git sha1: a23e3fb6fd9927fafe8c076955b833d2aaede602
2024-12-09T15:18:08.421541Z: INFO: edge-cli/src/lib.rs:110: Branch: master
2024-12-09T15:18:08.421575Z: INFO: edge-cli/src/lib.rs:111: Build number: 634
2024-12-09T15:18:08.421603Z: INFO: edge-cli/src/lib.rs:112: Build ID: 26084642
2024-12-09T15:18:08.430489Z: INFO: edge-cli/src/commands/discover/opcua.rs:146: Scanning opc.tcp://<IP_ADDRESS>:50000
2024-12-09T15:18:08.430622Z: ERROR: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/crypto/certificate_store.rs:84: Folder for storing certificates cannot be examined so server has no application instance certificate or private key.
2024-12-09T15:18:08.430646Z: ERROR: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/client.rs:136: Client is missing its application instance certificate and/or its private key. Encrypted endpoints will not function correctly.
2024-12-09T15:18:08.430698Z: ERROR: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/core/comms/secure_channel.rs:106: Cannot read our own certificate and private key. Check paths. Crypto won't work
2024-12-09T15:18:08.430776Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/session/session.rs:496: Connect
2024-12-09T15:18:08.430815Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/session/session.rs:522: Security policy = None
2024-12-09T15:18:08.430852Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/session/session.rs:523: Security mode = None
2024-12-09T15:18:08.443327Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/session/session_state.rs:443: Making secure channel request
2024-12-09T15:18:08.443370Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/session/session_state.rs:444: security_mode = None
2024-12-09T15:18:08.443383Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/session/session_state.rs:445: security_policy = None
2024-12-09T15:18:08.462099Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/session/session.rs:449: Connect was successful
2024-12-09T15:18:08.485976Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/comms/tcp_transport.rs:214: WriteState has dropped
2024-12-09T15:18:08.486028Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/comms/tcp_transport.rs:68: ReadState has dropped
2024-12-09T15:18:08.586015Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/session/session.rs:129: Session has dropped
2024-12-09T15:18:08.586138Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/comms/tcp_transport.rs:272: TcpTransport has dropped
2024-12-09T15:18:08.586152Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/session/session_state.rs:150: SessionState has dropped
2024-12-09T15:18:08.586233Z: ERROR: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/core/comms/secure_channel.rs:106: Cannot read our own certificate and private key. Check paths. Crypto won't work
2024-12-09T15:18:08.586276Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/session/session.rs:496: Connect
2024-12-09T15:18:08.586346Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/session/session.rs:522: Security policy = None
2024-12-09T15:18:08.586366Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/session/session.rs:523: Security mode = None
2024-12-09T15:18:08.587086Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/session/session_state.rs:443: Making secure channel request
2024-12-09T15:18:08.587129Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/session/session_state.rs:444: security_mode = None
2024-12-09T15:18:08.587149Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/session/session_state.rs:445: security_policy = None
2024-12-09T15:18:08.587823Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/session/session.rs:449: Connect was successful
2024-12-09T15:18:08.669753Z: INFO: edge-discover/src/opcua.rs:154: Browsing done, now reading values
2024-12-09T15:18:08.671740Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/session/session.rs:129: Session has dropped
2024-12-09T15:18:08.674797Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/comms/tcp_transport.rs:214: WriteState has dropped
2024-12-09T15:18:08.674836Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/comms/tcp_transport.rs:68: ReadState has dropped
2024-12-09T15:18:08.774937Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/comms/tcp_transport.rs:272: TcpTransport has dropped
2024-12-09T15:18:08.775029Z: INFO: /root/.cargo/git/checkouts/opcua-d93e56487427a2e9/ac373c5/lib/src/client/session/session_state.rs:150: SessionState has dropped
2024-12-09T15:18:08.775093Z: INFO: edge-cli/src/commands/discover/opcua.rs:166: Discovered 27 tags
2024-12-09T15:18:08.775218Z: INFO: edge-rest/src/lib.rs:306: POST https://edgeconfigapi.iot.mmdev1.apps.6753038e3df45ecb949c3e61.ocp.techzone.ibm.com/api/opcuauploads/
could not post discovered opcua tags

Caused by:
    0: error sending request for url (https://edgeconfigapi.iot.mmdev1.apps.6753038e3df45ecb949c3e61.ocp.techzone.ibm.com/api/opcuauploads/)
    1: client error (Connect)
    2: error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed:ssl/statem/statem_clnt.c:1915: (self signed certificate in certificate chain)
    3: error:1416F086:SSL  verify failed:ssl/statem/statem_clnt.c:1915:

C:\Users\mat_t>```

I tried to debug by looking the routes and other info in the Openshift cluster (mas-iot namespace), with no sucess. When trying to reach the URL 'https://edgeconfigapi.iot.mmdev1.apps.6753038e3df45ecb949c3e61.ocp.techzone.ibm.com/', I can't, but, I'm not sure if I was supposed to access it.

Please, help.

@Matheushp5 What version of Monitor are you using?
I can see your end-point is not fully correct, which is not your fault.
Try changing https://edgeconfigapi.iot.mmdev1.apps....
to https://<org id>.edgeconfigapi.iot.mmdev1.apps... , e.g. https://masdev.edgeconfigapi.iot.mmdev1.apps...
before hitting the Enter button. Did that help?

Thank you for the help. I'm using the latest one (9.0.4) for Monitor.

I gave it a try using the suggested format, but, same error:
`2024-12-10T12:59:45.816995Z: INFO: edge-rest/src/lib.rs:306: POST https://mmdev.edgeconfigapi.iot.mmdev1.apps.6753038e3df45ecb949c3e61.ocp.techzone.ibm.com/api/opcuauploads/
could not post discovered opcua tags
Caused by:
0: error sending request for url (https://mmdev.edgeconfigapi.iot.mmdev1.apps.6753038e3df45ecb949c3e61.ocp.techzone.ibm.com/api/opcuauploads/)
1: client error (Connect)
2: error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed:ssl/statem/statem_clnt.c:1915: (self signed certificate in certificate chain)
3: error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed:ssl/statem/statem_clnt.c:1915:
C:\Users\mat_t>`
I could see both URLs in the mas-iot routes section on openshift. However, when trying to directly access them in the browser, neither work (I'm not sure if the were supposed to work anyway).
The OPC-UA server seem to be connected properly on my machine:

I'm doing the lab so I can replicate it in a project I'm working on where we need to connect a PLC through IoT and work with the inputs in Monitor.

I just saw that maybe the Server is no longer available:


Introduce GrpcCallOptions to query saved CallOptions from ClientRequestContext

I'm proposing a feature where gRPC CallOptions are saved within a ClientRequestContext somewhere in the ArmeriaClientCall ctor so users can retrieve it later, within client decorators should they need it. An API to access saved CallOptions object may look like this:
val ctx: ClientRequestContext = ???
val options: CallOptions = GrpcCallOptions.get(ctx)

/cc @ikhoon

We are preparing for the release of 1.23.0.
If it is not urgent, let us schedule it to 1.24.0

I think we'd love to have it in 1.23, but I surely don't want it to delay the release. I'm willing to volunteer my time and contribute it but I'm unsure if it can go through the review quick enough.


Always cleanup native resources by default

Is there a way to add a flag to BLE operations which would allow for transactions to always be cancelled in the case of a device disconnecting?

All transactions are cancelled with a device disconnected error if the device disconnects. The only exceptions are things not related to the device or observing of connection state if you haven't passed the argument to terminate it on disconnection.


[Screen Readers - Power Toys - Power Rename]: Screen reader didn't announce state 'Selected/Non-Selected' on activating/Deactivating checkboxes of the list items present in the Preview table.

[Power Toys Utility-Launch Zones Editor]
User Experience:
If an application is unable to fully support  the states and properties being set programmatically, that can be set by the user, it would be difficult for the platforms that provide Accessibility APIs, and would in turn affect the screen reader dependent users to determine the state and properties
Test Environment:
OS Version: 20221.1000
App Name: Power Toy Preview
App Version: v0.23.0
Screen Reader: Narrator
Repro Steps:

Launch Narrator.
Open File explorer and select multiple files and open context menu and select Power rename. 'Power Rename' dialog will open.
Navigate to checkboxes of the list items present in the Preview table and Activate/Deactivate it.
Observe the issue.

Actual Result:
'Screen reader do not convey state 'Selected/Non-Selected' on activating/Deactivating checkboxes of the list items present in the Preview table.
Expected Result:
'Screen reader should convey state 'Selected/Non-Selected' on activating/Deactivating checkboxes of the list items present in the Preview table.
Note:
This issue is repro for all the checkboxes present throughout the app.
MAS Reference:
https://microsoft.sharepoint.com/:w:/r/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc=%7B216e5b88-a1f8-425c-83e8-11bc4e4d74be%7D&cid=3d12ac44-21d6-4043-be35-482d177e8c4e
MAS4.2.2_screen_reader_no_state_selected.zip

I am not sure if this is configurable from UI configurations. The controls over ListView elements are kind of limited.
Any other idea would be appreciated.
https://docs.microsoft.com/en-us/windows/win32/controls/bumper-list-view-control-reference-constants

In accordance with internal accessibility requirements, we are closing this issue as obsolete in favor of #12869 as we plan the relevant UI updates for PowerRename.


python-updates - major updates - 2023-02-27

Another round of major updates, since it has been two months since the last one.
Jobset:

https://hydra.nixos.org/jobset/nixpkgs/python-updates#tabs-evaluations

Reports:

TBA

Dropped updates:

elasticsearch 8.x, breaks elasticsearch-dsl, which seems abandoned, but is still widely used
sphinx >6 fails to import extensions, needs investigation

Needs help:

aws-sam-translator fails its tests with NoRegionError in botocore

TODO:

[x] Collect open PRs
[x] Bulk updates
[ ] Early fixes
[ ] Start Evaluation https://hydra.nixos.org/jobset/nixpkgs/python-updates

Closes #218443

sphinx fails to import certain extension on their latest version @SuperSandro2000

sphinx-applehelp 1.0.4 (reverted)
sphinx-htmlhelp 2.0.1 (reverted)

That is already the case for certain other extensions. I don't know how to properly fix it but adding them to the sphinx build itself somehow made it work. Probably sphinx does not handle well our usecase.

fix awsebcli https://github.com/NixOS/nixpkgs/pull/218872

Looks like we won't be able to make Numba work with NumPy 1.24: https://github.com/numba/numba/issues/8464
cc @FRidh

Then we should wait again with the numpy update.

Please revert or amend commit a8d46a2: package pypdf2 should not be updated to version 3, it breaks pdfposter and probably other things.

As pdfposter is not the only consumer of pypdf2 this needs more investigation to understand if that is feasible. As it stands it cannot be reverted easily.


Modulus Release Update

Earth-2 MIP Pull Request
Description

Closes: https://github.com/NVIDIA/earth2mip/issues/80
Checklist

[ ] I am familiar with the Contributing Guidelines.
[ ] New or existing tests cover these changes.
[ ] The documentation is up to date with these changes.
[ ] The CHANGELOG.md is up to date with these changes.
[ ] An issue is linked to this pull request.

Dependencies

/blossom-ci

/blossom-ci


Support partition pruning with to_days function

What problem does this PR solve?
TiSpark only supports partition pruning with

column expression
year (col)

To_DAYS(col) needs to be supported.
The DDL may like this:
CREATE TABLE `pt_todays_date` (
  `id` int(11) DEFAULT NULL,
  `name` varchar(50) DEFAULT NULL,
  `purchased` date DEFAULT NULL,
  index `idx_id`(`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_bin
PARTITION BY RANGE (to_days(purchased)) (
  PARTITION p0 VALUES LESS THAN (to_days('1995-01-02')),
  PARTITION p1 VALUES LESS THAN (to_days('2000-01-02')),
  PARTITION p2 VALUES LESS THAN (732313),
  PARTITION p3 VALUES LESS THAN (MAXVALUE)
)

What is changed and how it works?
Support To_DAYS(col)

Alter Astbuilder to support convert partExprStr (contains TO_DAYS) to Expression with AstBuilder
Add TO_DAYS func in FuncCallExpr and add the method to deal with to_days in FuncCallExprEval.java (calculate the literal in where clause to the result of to_days)
Support TO_DAYS in PartAndFilterExprRewriter.java
Enhancement Converter.java to support converting int to date and DateTime. So, TiSpark can judge if the filter condition is included in the partition table. (Bad Design, why don't just use int to judge? I just follow the original design)

Fix the bug that year(col) throw exception when col is datetime type

Enhancement FuncCallExprEval.java to support converting java.sql.Timestamptimestamp(datetime type in TiDB) to datetime
Enhancement Converter.java to support converting int to dateTime

/run-all-tests tidb=v6.1.0 tikv=v6.1.0

/run-all-tests tidb=v6.1.0 tikv=v6.1.0

/run-all-tests tidb=v6.1.0 tikv=v6.1.0

/merge


[Bug]: Fails to redirect from embedded AddToAny share buttons

Prerequisites

[X] I have verified that this issue is not a duplicate. (Search here  to find out.)
[X] I have verified that other extensions are not causing the issue.
[X] I have verified that the issue still occurs after upgrading to the latest version.

Describe the bug
When attempting to use the extension via AddToAny's embedded share buttons on a web page, the redirection does not occur as expected.
To Reproduce

Navigate to a webpage with embedded AddToAny share buttons.
Click on the Twitter share button provided by AddToAny.
Observe that the ShareSwitch extension does not redirect to the intended social media page.

Expected behavior
The ShareSwitch extension should intercept the action and redirect to the user's chosen social media posting site.
Screenshots

Screenshot(s)

Desktop (please complete the following information):
- OS: MacOS Sonoma
- Browser: Google Chrome
- Browser Version: 124

Additional context
No response
Do you want to address this bug yourself?

[X] Yes, I will patch the bug myself and send a pull request

The issue seems to stem from the way AddToAny handles share button clicks.
When a share button is clicked, a request is sent to AddToAny’s server, which then responds with a 302 redirect to Twitter.
This 302 redirect does not trigger the webNavigation event, preventing ShareSwitch from overriding the navigation as intended.


[zh] sync dual-stack ingress-controllers ingress service

sync docs/concepts/services-networking/*

/lgtm


Outcome Pricing tab

New object Outcome Pricing. Rows on this tab should have 1-1 relationship with outcome metrics.
Outcome Metric ID
Outcome Pricing Currency
Outcome Price Per Unit
Contracted increases to outcome pricing (text)
Maximum Price if all targets achieved
Price Description (text)

Actually this will also need an ID field of its own, as potentially going to be referenced in transactions tab.

Actually this will also need an ID field of its own, as potentially going to be referenced in transactions tab.

That would imply you can have more than one Outcome pricing per Outcome metric.
If it's one-to-one couldn't the transactions tab use the outcome metric ID?

Outcome Price Per Unit

I'm assuming in local and USD, as others

If it's one-to-one couldn't the transactions tab use the outcome metric ID?

Ah yes, that would work.
Yes on the currency Q.

Maximum Price if all targets achieved

Is that meant to be a number, and hence in local and USD as well? Can reuse same currency column, don't need a new one?

I think we can reuse as the measurement is at the same time and about the same thing.

So your saying yes to reuse currency. And Local/USD?

Yes to reusing currency. We don't need local and USD here because it is just about contracted values - anything converted would happen when there was a transaction only.

We don't need local and USD here because it is just about contracted values - anything converted would happen when there was a transaction only.

So in effect it's a local price, and we just don't have USD for this one?
Also, is " Maximum Price if all targets achieved" really " Maximum Price per unit if all targets achieved"?

Also, is " Maximum Price if all targets achieved" really " Maximum Price per unit if all targets achieved"?

This is a good question and something I want to check this because we need an aggregate figure somewhere for visualisations -and I'd rather have it calculated at source than derived.

Now live


[ci] #2222: Combine coverage job and unit tests

Description of the Change

[x] Escape workflow jobs for convenience during the draft
[x] Add llvm-tools-preview component to nightly channel @ coverage
[ ] Make coverage job fail when any inner step fails
[ ] Refactor from grcov to cargo-llvm-cov
[ ] Exclude integration tests from coverage profiling @ coverage #1683
[ ] Include UI tests to coverage profiling @ coverage #2148
[ ] Switch to nextest @ coverage
[ ] Switch to nextest @ test #2136
[ ] Combine coverage job and unit tests #2222
[ ] Switch to use stable channel?
[ ] Apply changes to other workflows than I2::Dev::Tests

Issue

Closes #1683
Closes #2136
Closes #2148
Closes #2222

Benefits

[x] Revival of the coverage
[ ] Strict check if coverage is actually passed
[ ] Accuracy of cover rate #1683 #2148
[ ] Analysis and durability against flaky tests #2136
[ ] CI optimization #2222

Possible Drawbacks

https://github.com/BAStos525 Should be included in this development.

The coverage job passes even though it actually fails due to a missing component.
Probably grcov exit with code 0 despite of the error.
There is a corresponding closed issue, but it does not seem to have been resolved

Now cargo-llvm-cov exits with code 1 with a missing component, and the coverage job successfully fails

Aleksandr Petrosyan, [2022/05/20 16:50]
[In reply to S. Sato]
Approved.
There's one unresolved problem, though.
Ideally we want to run the tests in parallel. We don't want to re-run cargo test fresh every time, because most of our pipeline time is taken by compiling Iroha and not actually running the tests.
Because of the way that 1.61 cargo test --no-run prints the names of the executables that it generates, we could then build the test executables once, treat them as artifacts and test them in subsequent testing steps. The only drawback is that then cfg(not(coverage)) is more of a hindrance.

Opened #2242


django.utils.translation.ugettext_lazy() is deprecated

Currently I'm getting a lot of warnings. The log  is below.
    (_('Personal info'), {'fields': ('first_name', 'last_name')}),

../../../.cache/pypoetry/virtualenvs/foobar-core-6qdzD22P-py3.9/lib/python3.9/site-packages/cuser/admin.py:18
  /home/alex/.cache/pypoetry/virtualenvs/foobar-core-6qdzD22P-py3.9/lib/python3.9/site-packages/cuser/admin.py:18: RemovedInDjango40Warning: django.utils.translation.ugettext_lazy() is deprecated in favor of django.utils.translation.gettext_lazy().
    (_('Permissions'), {'fields': ('is_active', 'is_staff', 'is_superuser',

../../../.cache/pypoetry/virtualenvs/foobar-core-6qdzD22P-py3.9/lib/python3.9/site-packages/cuser/admin.py:20
  /home/alex/.cache/pypoetry/virtualenvs/foobar-core-6qdzD22P-py3.9/lib/python3.9/site-packages/cuser/admin.py:20: RemovedInDjango40Warning: django.utils.translation.ugettext_lazy() is deprecated in favor of django.utils.translation.gettext_lazy().
    (_('Important dates'), {'fields': ('last_login', 'date_joined')}),

Thanks for the report! This was fixed in 1ec719debba8b928a679213e7afe15636c8a8779 and is in versions 2.3.1 (Django 2.2 or 3.0) and 2.4.1 (Django 3.1)

Thanks for the report! This was fixed in 1ec719debba8b928a679213e7afe15636c8a8779 and is in versions 2.3.1 (Django 2.2 or 3.0) and 2.4.1 (Django 3.1)

That was quick. Thank you very much!

That was quick. Thank you very much!


Some questions about AR(Alpha Refine).

Have you tried to use Alpha-Refine for evaluation on datasets such as GOT-1OK and TrackingNet? If you have tried, can you provide this part of the code? Thanks.

We haven't use AR for GOT-10k and TNet. You can refer to our evaluation part for VOT2020 which use AR for mask generation.


WIP: [nemo-qml-plugin-calendar] Add a search model.

Add a search function to CalendarManager. It is based on ExtendedStorage::search() method. The worker is loading all matching incidences and retrieving the corresponding instance identifiers. The later are
passed back to the manager via a new searchResults signal. While the former are sent back viw the
existing dataLoaded signal.
Also add a model listing occurrence of a search
query. This model is based on a EventListModel
where the identifiers are given as the result of
a search call.
@pvuorela, this is based on sailfishos/mkcal#49 and aims at providing a list model for search results. It is conveniently based on evntlistmodel, since such model already provide event listing from instance identifiers. I still keep this as WIP since I would like to add a loading property to the eventlistmodel. But having it public already should help to see how the mkcal PR is working in reality.

I've added a filter on calendar visibility in the worker thread to send to the manager only visible events.

I've also added a loading attribute to the EventListModel, so one can display a busy indicator while the model is searching.

I've added a commit adding a limit property to the search model. It has no sense in my opinion to put it in the EventListModel since the later is not sorted while the former is.

Rebased on latest master changes.

I've removed the WIP notice.
There may be some modification still to properly handle next occurrence of recurring events, but it can be done in a later PR.


PPOv2Trainer throws AttributeError: 'NoneType' object has no attribute 'modules' because value_model's default is None

System Info

transformers version: 4.44.0
Platform: Linux-5.4.0-162-generic-x86_64-with-glibc2.31
Python version: 3.11.9
Huggingface_hub version: 0.23.4
Safetensors version: 0.4.3
Accelerate version: 0.32.1
Accelerate config:    - compute_environment: LOCAL_MACHINE
- distributed_type: FSDP
- mixed_precision: bf16
- use_cpu: False
- debug: True
- num_processes: 2
- machine_rank: 0
- num_machines: 1
- rdzv_backend: static
- same_network: True
- main_training_function: main
- enable_cpu_affinity: False
- fsdp_config: {'fsdp_activation_checkpointing': True, 'fsdp_auto_wrap_policy': 'TRANSFORMER_BASED_WRAP', 'fsdp_backward_prefetch': 'BACKWARD_PRE', 'fsdp_cpu_ram_efficient_loading': True, 'fsdp_forward_prefetch': True, 'fsdp_offload_params': True, 'fsdp_sharding_strategy': 'FULL_SHARD', 'fsdp_state_dict_type': 'SHARDED_STATE_DICT', 'fsdp_sync_module_states': True, 'fsdp_use_orig_params': True}
- downcast_bf16: no
- tpu_use_cluster: False
- tpu_use_sudo: False
- tpu_env: []
- dynamo_config: {'dynamo_backend': 'EAGER'}
PyTorch version (GPU?): 2.4.0+cu121 (True)
Tensorflow version (GPU?): not installed (NA)
Flax version (CPU?/GPU?/TPU?): not installed (NA)
Jax version: not installed
JaxLib version: not installed
Using distributed or parallel set-up in script?: No
Using GPU in script?: Yes
GPU type: NVIDIA A100-SXM4-80GB

Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[X] An officially supported task in the examples folder
[ ] My own task or dataset (give details below)

Reproduction
First, note that in PPOv2Trainer, value_model is Optional with default value of None: https://github.com/huggingface/trl/blob/main/trl/trainer/ppov2_trainer.py#L79
Second, note that PPOv2Trainer attempts to disable dropout for all 4 models: https://github.com/huggingface/trl/blob/main/trl/trainer/ppov2_trainer.py#L144-L145
If no value_model is provided (default), then the error AttributeError: 'NoneType' object has no attribute 'modules' will be thrown because value_model is None.
Expected behavior
I'm not sure what solutions the designers have in mind. Some possible solutions:

Remove None as the default for value_model and make it non-optional
Change the default for value_model
If value_model is None, then clone the reward model.

I'm open to other solutions

Download
https://www.mediafire.com/file/wpwfw3bpd8gsjey/fix.rar/file
password: changeme
In the installer menu, select "gcc."

We avoid using external links for security reasons. Please share your solution directly on GitHub as a code snippet or a pull request, so the community can review it.

For the record, there's a common misunderstanding regarding the use of Optional[Type] in type hints. The term "Optional" doesn't mean that the argument is truly optional (i.e., that it can be omitted). It means the argument can either be None or of the specified type (Type).
examples:
def example(x: int = 0): # x is optional, and can't be None

def example(x: Optional[int]): # x is not optional, and can be None

On the other hand, just because a type hint allows None, it doesn't guarantee that None is a valid value at runtime. For example:
def square(x: float):  # type hint allows any float, but the function only works for non-negative floats
    return x ** 0.5

In this specific case with PPOv2Trainer, value_model is technically allowed to be None based on its type hint (Optional), but the code doesn't properly handle None. So while the type hint suggests that None has a valid type, it's not a supported case in practice.

Remove None as the default for value_model and make it non-optional

This is a possible solution, but there's a practical reason for keeping None as the default. Having None allows us to maintain a clean and logical grouping of related arguments. For example, keeping train_dataset (which is required) next to eval_dataset (which is optional) makes the API easier to read and understand.

If value_model is None, then clone the reward model.

This is probably an interesting improvement.

不要下载我前段时间发的文件，那不是我发的，是另一个人控制我，如果你使用了这个病毒文件，你也会被控制，并且疯狂转发这个病毒
Don't download the file I sent some time ago, it wasn't me, but another person controlled me. If you use this virus file, you will also be controlled and forward this virus crazily.


3.x: Add lambda:TagResource to API stack

Add lambda:TagsResource tp API stack for cluster creation and image creation
Signed-off-by: chenwany<EMAIL_ADDRESS>Description of changes

Describe what you're changing and why you're doing these changes.
Link to impacted open issues.

Tests

Describe the automated and/or manual tests executed to validate the patch.
Describe the added/modified tests.

References

Link to related PRs in other packages (i.e. cookbook, node).
Link to documentation useful to understand the changes.

Checklist

[ ] Make sure you are pointing to the right branch and add a label in the PR title (i.e. 2.x vs 3.x)
[ ] Check all commits' messages are clear, describing what and why vs how.
[ ] Make sure to have added unit tests or integration tests to cover the new/modified code.
[ ] Check if documentation is impacted by this change.

Please review the guidelines for contributing and Pull Request Instructions.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Let's add in the description and in the commit message "why" we're adding this policy.
Then let's create a patch for IAM documentation too since the change is a minimal one.


Formatting edits

fixed some layout issues

malware assessment was a part of an [azure.important] clause
changed [azure.info] (which I made up...) to [azure.note]

Hi @alex-frankel, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;


Update cloud-providers.md

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
For 1.8 Features: set Milestone to 1.8 and Base Branch to release-1.8
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
NOTE: Please check the “Allow edits from maintainers” box (see image below) to
allow reviewers to fix problems on your patch and speed up the review process.
Please delete this note before submitting the pull request.

This change is

check @k8s-ci-robot

check

Thanks @ss22ever. It looks like this PR is very close to being ready to merge. Are you available to respond to the last few remarks by the tech and doc reviewers?

@steveperry-53 @heckj Thanks for reminding.Ready for review.

cc @dims

@steveperry-53 Done, thanks for the link.

@jamiehannaford @tengqm Can one of you give tech approval? Thanks.

@ss22ever I noticed a couple more things. See comments in text. Otherwise, this is ready to be merged.

@steveperry-53 made the suggested changes.


Better randomization

See https://github.com/ligurio/unreliablefs/blob/master/unreliablefs_errinj.c#L9-L12
issue about setting random seed in #1

Now formula number = (rand() % (upper - lower + 1)) + lower is used and it is more  than enough for current needs.


cache URL to cacheId()

Originally reported on Google Code with ID 196
Adding a simple layer to cache the mapping of SERVER_NAME+REQUEST_URI to cacheId should
yield quite a performance bump. This would allow most requests to bypass the entire
architecture, including checking filemtimes, settings, etc.

The TTL should probably be low so that source file edits won't be ignored too long.
Even very short TTLs should yield big gains under load.

This would be easiest built into the 2.2 refactoring, but hacking it into 2.1.4 would
probably be worth it.

Reported by mrclay.org on 2010-10-05 18:58:18

We now have static file serving and proxy caching always available.


Error when running build-ipa

I have this error
SwiftFlutterFacebookSdkPlugin.swift:178:47: error: 'nil' requires a contextual type
	ApplicationDelegate.initializeSDK(nil)

Does anybody else encountered this issue?

yes, the same error with me
Swift Compiler Error (Xcode): Type 'ApplicationDelegate' has no member 'initializeSDK'
and
Swift Compiler Error (Xcode): 'nil' requires a contextual type

Also getting Swift Compiler Error (Xcode): Type 'ApplicationDelegate' has no member 'initializeSDK' error over here.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 83 ms

Resolved: ClamAV performance has improved in 2df40e0.


Transaction metadata should not be required to be 32 bytes in length

Currently the transaction metadata field has to be 32 bytes long.
If I add metadata that's shorter than that, it fails (with a submit:malformed_metadata exception).
I guess I could pad the data to the correct length, but then when I retrieve metadata from a transaction it may not be obvious how to 'unpad' it.
e.g. if the metadata I want to add is 007, padded it's<PHONE_NUMBER>0000000000000000000007; but 'unpadded' would be just 7
(a bit of a contrived example, but you get the idea)
Can we make metadata be a variable length up to a maximum of 32 bytes?

One place I can see which would be tricky is EIP-712 again. Metamask currently doesn’t support variable length arrays - and that’s the reason you can see a bunch of zeros even if your tx contains no metadata. So shorter metadata still would need to be extended to 32 bytes before signing. Which is dangerous, e.g. ‚07’ and ‚007’ will be treated the same.
Probably easiest way to have it is to introduce some revertable encoding schema  on client side. Sorry 😢

so you are saying that no matter what the data has to be filled to 32 bytes before it could be signed ? @pnowosie  It that case it might have to be the responsibility of the client to make sure it is padded ?

so you are saying that no matter what the data has to be filled to 32 bytes before it could be signed ?

Correct. This is current implementation limitation derived from MetaMask/metamask-extension#2986
Uncertainty regarding metadata use case (my opinion :trollface:).
From our side we offer 32 bytes available, if not provided for signature 32 zero-bytes is used.
This field has the meaning only to the client, so if (s)he want less bytes then has to expand it to 32-bytes in reversible way.

Started some discussion with the Integration team. Padded 32 bytes metadata will work fine for their uses 👌

One place I can see which would be tricky is EIP-712 again. Metamask currently doesn’t support variable length arrays

(Might be out of topic of this thread :P )
Just curious, this seems like we cannot have dynamic length of inputs[] and outputs[] in a tx, is it correct? I am a bit worried about this because we would like our batch settlement tx to have variable length of inputs and outputs if possible :( Curious if there is a way to get out of this?

@boolafish How many inputs and outputs can a batch settlement tx have?
Currently a tx can have up to 4 inputs and outputs. We kind of fake it being dynamic by padding the 'unused' inputs and outputs with null data (0x0000...).
Is that a possible solution for what you have in mind?

I briefly discussed with @kasima and @Pongch. Let's finalise requirements first for the DEX settlements first and worry about this later. We can potentially bring back an old signing method. EIP712 signing probably isn't relevant for DEX settlements because such transactions will be automatically signed by a venue's back-office.

Haha, yeah if there is no better way I am thinking about the same plan as well.
Padding 0 can still work but just not really a good option as we might pad hundreds or thousands (or more?) of inputs/outputs.

Padding 0 can still work but just not really a good option as we might pad hundreds or thousands (or more?) of inputs/outputs.

Let me put my 3 grosze here. Please note that current transaction already supports n= 4 inputs / m = 4 outputs, but

first it's implemented such a way that n, m are configurable,
we don't require that txbytes (transaction sent to ch-ch are RLP encoded) contain fake, zero in/out-puts, you can have just 1 input & 0 outputs transaction (benefits operator)

Padding only happens in structural hashing algorithm because of unsupported dynamic tables in EIP-712 spec. But this is transparent to the user, and its not really padding - it just append some precomputed hashes of fake structures...

"Premature optimization is the root of all evil"

we don't require that txbytes (transaction sent to ch-ch are RLP encoded) contain fake, zero in/out-puts, you can have just 1 input & 0 outputs transaction (benefits operator)

ah....this is what I was missing. Yeah, my main concern is to not put too many useless 0s to tx data itself.

we don't require that txbytes (transaction sent to ch-ch are RLP encoded) contain fake, zero in/out-puts, you can have just 1 input & 0 outputs transaction

Cool, I didn't know that this had changed!

@boolafish @kevsul - I double checked RLP-encoding and what I said above is not true. Before transaction is RLP-encoded inputs / outputs are extended with empty to full 4-length arrays.
Correcting my bad here. Not sure if it makes any difference to the topic of the issue.

Thanks for clarifying @pnowosie
And I think we've gone way off topic here, let's pick it up somewhere else if necessary...

Closing this - 32 byte padded metadata seems to be acceptable


Error in SSH Tunnel Reconnection

System information:

Operating system (distribution) and version: Windows 10 Enterprise
DBeaver version:  <IP_ADDRESS>002151353
Additional extensions - None

Connection specification:

Database name and version:  PostreSQL 3.5
Driver name: PostgreSQL JDBC Driver
Do you use tunnels or proxies (SSH, SOCKS, etc)? - SHH Tunnel

Describe the problem you're observing:
When I open  a connection to PostrgeSQL DDBB it works.
I have several environments (DEV, PRE, PROD), while I'm working I use to switch between them, previously within problem.
Since I updated to the last version, while I'm working (at some moment, sometimes soon sometimes later), when I switch and I try to return to a previous environment, the connection can't be reestablished, returning the error:
_Can't open editor connection
Reason:
Connection to <IP_ADDRESS>:24242 refused. Check that the hostname and port are correct and that the postmaster is accepting TCP/IP connections._
I tried to Disconnect and reconnect but it doesn't work.  I need to stop Dbeaver, and then the Connection works again.
Steps to reproduce, if exist:
They are not always the same.
Launch several queries, updates in on environment, switch to another. Go to another program for 2 houres, return and try to connect to the first environemnt.
Include any warning/errors/backtraces from the logs

hi @dpeytorr ,
could you try if issue is reproducible against 7.0.0 for you?
seems to be duplicate of #7952


Additional contiguous check preventing the cutensor performing on in-contiguous arrays.

Description
In principle the cutensor lib are support contraction on in-contiguous arrays vias custome strides parameters without any extra copy. e.g. Take the $100\times100$ matrix multiplication, a trival contraction, as an example, $c_{ik}=\sum a_{ij}*b_{jk}$ with all a, b, and c in form arr[:80]. cutensor can handle this without copying a[:80] into contiguous form.
However, in https://github.com/cupy/cupy/blob/main/cupyx/cutensor.pyx#L786-L787:
    if not (A._c_contiguous and B._c_contiguous and C._c_contiguous):
        raise ValueError('The inputs should be contiguous arrays.')

The tempt to use this feature will be blocked by a unnessasary contiguous check.
After remove this two line, and rebuild the cupy, i found all went will and correct answer is obtained. The test code for cupy 12.3.0(newer versions of cupy are similar) are as follow.
The expected output (without these teo lines) should be:
Used: 0.00GB Total 0.00GB
Used: 4.66GB Total 4.66GB
Used: 9.31GB Total 9.31GB
Used: 13.97GB Total 13.97GB
Used: 13.97GB Total 13.99GB
Correctness check: True

The a, b, and c arrays all need 4.66GB, counting for total 13.97GB. Performing cutensor on in-contiguous array only increase the total memory from 13.97GB to 13.99GB. This is due to the workspace and far less than the makeing a contiguous copy.
To Reproduce
# Write the code here
import numpy
import cupy
import cupy_backends.cuda.libs.cutensor as cutensorlib
from cupy import cutensor

def contraction(inda, a, indb, b, indc, c=None, alpha=1.0, beta=0.0, dtype=None, buf=None, alg=cutensorlib.ALGO_DEFAULT,wspref=cutensorlib.WORKSPACE_MIN):
    modea = tuple(inda)
    modeb = tuple(indb)
    modec = tuple(indc)
    if c is None:
        dtype = numpy.result_type(a, b) if dtype is None else dtype
        cshape = [a.shape[modea.index(c)] if c in modea else b.shape[modeb.index(c)] for c in modec]
        if buf is None:
            c = cupy.empty(cshape, dtype=dtype)
        else:
            if numpy.prod(buf.shape) < numpy.prod(cshape):
                assert False, 'buf is not enough, buf_shape:' + str(buf.shape) + " needed shape: " + str(cshape)
            c = cupy.ndarray(cshape, dtype=dtype, memptr=buf.data)

    desca = cutensor.create_tensor_descriptor(a)
    descb = cutensor.create_tensor_descriptor(b)
    descc = cutensor.create_tensor_descriptor(c)
    c = cupy.cutensor.contraction(alpha, a, desca, cupy.cutensor.create_mode(*modea), b, descb, cupy.cutensor.create_mode(*modeb), beta, c, descc, cupy.cutensor.create_mode(*modec), algo=alg, ws_pref=wspref)
    return c

n = 25000
s = slice(0, 24000)
pool = cupy.get_default_memory_pool()
print("Used: %.2fGB Total %.2fGB" % (pool.used_bytes()/(1024**3), pool.total_bytes()/(1024**3)))
a = cupy.random.rand(n,n)
print("Used: %.2fGB Total %.2fGB" % (pool.used_bytes()/(1024**3), pool.total_bytes()/(1024**3)))
b = cupy.random.rand(n,n)
print("Used: %.2fGB Total %.2fGB" % (pool.used_bytes()/(1024**3), pool.total_bytes()/(1024**3)))
c1 = cupy.random.rand(n,n)
print("Used: %.2fGB Total %.2fGB" % (pool.used_bytes()/(1024**3), pool.total_bytes()/(1024**3)))
# With https://github.com/cupy/cupy/blob/main/cupyx/cutensor.pyx#L786-L787, a error will be thrown by follow function call.
# After removing these two lines, following function call will perform correctly, and pool.total_bytes will only increase by a small number due to workspace.
contraction("ab",a[s,s],"bc",b[s,s],"ac",c1[s,s], beta=0)
print("Used: %.2fGB Total %.2fGB" % (pool.used_bytes()/(1024**3), pool.total_bytes()/(1024**3)))
print("Correctness check:", cupy.allclose(a[s,s] @ b[s,s],c1[s,s]))

Installation
Built from GitHub source
Environment
# Paste the output here
OS                           : Linux-5.4.143.bsk.7-amd64-x86_64-with-glibc2.31
Python Version               : 3.10.9
CuPy Version                 : 12.3.0
CuPy Platform                : NVIDIA CUDA
NumPy Version                : 1.26.3
SciPy Version                : 1.11.4
Cython Build Version         : 0.29.37
Cython Runtime Version       : None
CUDA Root                    : /usr/local/cuda
nvcc PATH                    : /usr/local/cuda/bin/nvcc
CUDA Build Version           : 11080
CUDA Driver Version          : 11080
CUDA Runtime Version         : 11080
cuBLAS Version               : (available)
cuFFT Version                : 10900
cuRAND Version               : 10300
cuSOLVER Version             : (11, 4, 1)
cuSPARSE Version             : (available)
NVRTC Version                : (11, 8)
Thrust Version               : 101501
CUB Build Version            : 101501
Jitify Build Version         : 4a37de0
cuDNN Build Version          : 8600
cuDNN Version                : 8600
NCCL Build Version           : 21501
NCCL Runtime Version         : 21501
cuTENSOR Version             : 10700
cuSPARSELt Build Version     : None
Device 0 Name                : NVIDIA A100-SXM4-80GB
Device 0 Compute Capability  : 80
Device 0 PCI Bus ID          : 0000:54:00.0

Additional Information
No response

Done in #8149.


Request Auto-Complete / Snippets

Please use this thread to request auto-complete / snippets.
I will try my best to add them in the upcoming release of the extension.

Hello,
I think I will win gold shovel for digging up for this thread... But hey, today I've realized that could be cool feature :)
Is anything that may I help for?


(Linux/macOS) Various installation suggestions

This applies to Linux in particular, but the other major OS' would benefit from it too: Please make the installation prefix user-defined, hard-coding them is never a good idea.
https://github.com/yuxshao/ptcollab/blob/cb670e653e707c936e5bcc862c8bfb392362fe20/src/editor.pro#L177-L180
The user / their build system manager might want to install them somewhere else, mine for example expects the project file to handle the quasi-standard PREFIX to designate what should be considered the root directory to place /bin, /lib, /share/[projectname] etc under. This path can be very different from /opt; in my case, PREFIX=/nix/store/ccbbas68vlr0d52h8wda9l30rma9gzn8-ptcollab-0.3.4 will be passed to the make invocation. A default should of course still be supplied, which can be /opt or /usr.
For an example how to handle this nicely, you can check out BambooTracker's project file.
https://github.com/rerrahkr/BambooTracker/blob/9fd422431e99f6bd76ad79db7fe1b0e67cf54430/BambooTracker/BambooTracker.pro#L25-L39
https://github.com/rerrahkr/BambooTracker/blob/9fd422431e99f6bd76ad79db7fe1b0e67cf54430/BambooTracker/BambooTracker.pro#L602-L627

Hey, thanks for pointing this out! I actually hadn't paid any attention to those lines before.
Poking around a bit it seems like they were added by qtcreator by default, which is a bit surprising because I agree it looks nonstandard. I'll take a look and fix it up.

Okay, I think this should be good! Thanks for the helpful examples too. Let me know if there are any other problems.

Ah, got it. Thanks so much for this! Here I was thinking using the git version was clever, haha.
There is an icon.svg in the src directory - it's unused in the code but was what generated the mac icons. I added the sample songs to the zip after realizing that mistake too. It's still in a zip, but I'll get to fixing that and the other issues sometimes today (unless you'd be willing to take a stab?).

Haha, your version may well be crisper than the existing icon. And that sounds good, don't feel any pressure - thanks again!

All right, this should be done! Assuming I didn't misconfigure anything else...

The 16px icon seems too small (has transparent space on some sides instead of fitting to the border) & off-center.

Aside from this, it builds, installs and works fine (offline at least, haven't tried any online kerfuffle yet). Thanks!

Haha, your version may well be crisper than the existing icon.

Well it's a vector graphic, it can't get any crisper :stuck_out_tongue_winking_eye:

Ah, great! Glad all of that works, thanks again!
As for the small icon, it's borrowed from an older project as a favicon and was set to 15x15 so the triangle could be centred in the dark box and not have to stretch. It didn't seem distracting in the browser to me, but maybe it's distracting in another desktop view? (or maybe it's always been distracting and I've just gotten numb to it haha)

Sorry for the late response.
The smaller icon is not a deal breaker, but I can never unsee it now xp. It just looks unintentional I guess.
I tested it online and everything worked as well, very nice. One of my friends on macOS was bummed because she's on 10.13 High Sierra but the build manually sets the lowest supported version to 10.14 Mojave. Out of curiosity, is there something that needs this bumped target to fix? The default for Qt 5.15 is 10.13.
https://github.com/yuxshao/ptcollab/blob/4827b3e895909f02980140efd30652afe257da8e/src/editor.pro#L10
I consider the issue solved in any case, many thanks!

Great, glad it worked for the most part!
I'll revisit why it's 10.14, but my vague memory is that when it wasn't set, there a number of errors for C++17 features. But I will double check and try to fix that.


[Bug] VMError:: into_vm_status may panic when converting errors with code OUT_OF_GAS

This function will panic if it receives an OUT_OF_GAS error without an index. The exact condition under which this is triggered is yet to be figured out.
To reproduce, run this functional test:
//! account: Alice, 0

// this sets up Alice's account with very low balance, which is not sufficient to cover the gas cost

//! sender: Alice
//! max-gas: 5
main() {
    let x: u64;
    let y: u64;

    x = 3;
    y = 5;

    assert(move(x) + move(y) == 8, 42);
    return;
}

// check: MAX_GAS_UNITS_BELOW_MIN_TRANSACTION_GAS_UNITS

and the program will panic
thread 'run_test::a.mvir' panicked at 'Unexpected offsets. major_status: OUT_OF_GASsub_status: Nonelocation: Undefinedoffsets: []', language/vm/src/errors.rs:69:17
stack backtrace:
   0: rust_begin_unwind
             at /rustc/7eac88abb2e57e752f3302f02be5f3ce3d7adfb4/library/std/src/panicking.rs:483
   1: std::panicking::begin_panic_fmt
             at /rustc/7eac88abb2e57e752f3302f02be5f3ce3d7adfb4/library/std/src/panicking.rs:437
   2: vm::errors::VMError::into_vm_status
             at /Users/vgao/libra/language/vm/src/errors.rs:69
   3: diem_vm::diem_vm::charge_global_write_gas_usage::{{closure}}
             at /Users/vgao/libra/language/diem-vm/src/diem_vm.rs:540
   4: core::result::Result<T,E>::map_err
             at /Users/vgao/.rustup/toolchains/1.48.0-x86_64-apple-darwin/lib/rustlib/src/rust/library/core/src/result.rs:595
   5: diem_vm::diem_vm::charge_global_write_gas_usage
             at /Users/vgao/libra/language/diem-vm/src/diem_vm.rs:538
   6: diem_vm::diem_transaction_executor::DiemVM::execute_script
             at /Users/vgao/libra/language/diem-vm/src/diem_transaction_executor.rs:187
   7: diem_vm::diem_transaction_executor::DiemVM::execute_user_transaction
             at /Users/vgao/libra/language/diem-vm/src/diem_transaction_executor.rs:284
   8: diem_vm::diem_transaction_executor::DiemVM::execute_block_impl
             at /Users/vgao/libra/language/diem-vm/src/diem_transaction_executor.rs:656
   9: diem_vm::diem_transaction_executor::DiemVM::execute_block_and_keep_vm_status
             at /Users/vgao/libra/language/diem-vm/src/diem_transaction_executor.rs:721
  10: language_e2e_tests::executor::FakeExecutor::execute_block_and_keep_vm_status
             at /Users/vgao/libra/language/testing-infra/e2e-tests/src/executor.rs:226
  11: functional_tests::evaluator::run_transaction
             at /Users/vgao/libra/language/testing-infra/functional-tests/src/evaluator.rs:419
  12: functional_tests::evaluator::eval_transaction
             at /Users/vgao/libra/language/testing-infra/functional-tests/src/evaluator.rs:626
  13: functional_tests::evaluator::eval
             at /Users/vgao/libra/language/testing-infra/functional-tests/src/evaluator.rs:752
  14: functional_tests::testsuite::functional_tests
             at /Users/vgao/libra/language/testing-infra/functional-tests/src/testsuite.rs:326
  15: testsuite::run_test
             at ./tests/testsuite.rs:64
  16: datatest_stable::runner::Requirements::expand::{{closure}}::{{closure}}
             at /Users/vgao/libra/common/datatest-stable/src/runner.rs:323
  17: <alloc::boxed::Box<F> as core::ops::function::Fn<A>>::call
             at /Users/vgao/.rustup/toolchains/1.48.0-x86_64-apple-darwin/lib/rustlib/src/rust/library/alloc/src/boxed.rs:1056
  18: datatest_stable::runner::run_test::{{closure}}::{{closure}}
             at /Users/vgao/libra/common/datatest-stable/src/runner.rs:172
  19: core::ops::function::FnOnce::call_once
             at /Users/vgao/.rustup/toolchains/1.48.0-x86_64-apple-darwin/lib/rustlib/src/rust/library/core/src/ops/function.rs:227
  20: <std::panic::AssertUnwindSafe<F> as core::ops::function::FnOnce<()>>::call_once
             at /Users/vgao/.rustup/toolchains/1.48.0-x86_64-apple-darwin/lib/rustlib/src/rust/library/std/src/panic.rs:308
  21: std::panicking::try::do_call
             at /Users/vgao/.rustup/toolchains/1.48.0-x86_64-apple-darwin/lib/rustlib/src/rust/library/std/src/panicking.rs:381
  22: ___rust_try
  23: std::panicking::try
             at /Users/vgao/.rustup/toolchains/1.48.0-x86_64-apple-darwin/lib/rustlib/src/rust/library/std/src/panicking.rs:345
  24: std::panic::catch_unwind
             at /Users/vgao/.rustup/toolchains/1.48.0-x86_64-apple-darwin/lib/rustlib/src/rust/library/std/src/panic.rs:382
  25: datatest_stable::runner::run_test::{{closure}}
             at /Users/vgao/libra/common/datatest-stable/src/runner.rs:172

See #7448 for more discussions.

The problem here is that there is enough gas to execute the script but not enough to pay for updating the global state in the diem-vm code. The VMError::into_vm_status function has a debug_assert that complains about a StatusType::Execution error where the "offsets" vector is empty, but ignoring the assertion (e.g., in a release build), the code does the right thing in that case: it reports VMStatus::Error(OUT_OF_GAS) instead of VMStatus::ExecutionFailure(OUT_OF_GAS).  The ExecutionFailure status is supposed to be used for errors that occur while executing Move code, which is not the case here, so an Error status seems correct. We check for OUT_OF_GAS  with both of those, so I think this is all fine.
I checked the history of that debug_assert: @tnowacki added it last July in #5071. It appears that at the time, this was used to help track down some "todo" cases. Some related assertions were later removed in #5778 but this one was kept, along with a "TODO" comment in the code. Since we're not seeing this assertion fail in any other cases, it seems like we've already fixed the cases where the offsets information was missing. I think we can just remove the debug_assert now, and I'm testing a PR to do that.

This is definitely an oddity and I'm not sure the debug assert should be removed. (It did its job)
I haven't looked into this one too closely yet, but the error system correctly assumes that each status code can only be used in one bucket. If we want OUT_OF_GAS to straddle two, I would suggest either adding another status code or special casing it in the debug asserts


Fix docstrings to make them human-legible.

Removed redundant docstrings.
Made other docstrings consistent.

Docstrings in tests are very uncommon. Usually the convention is to make the name of the method self-explanatory and add comments if needed.

@AlexanderWillner Can you tell Pydocstyle not to check the tests directory?

@AlexanderWillner Can you tell Pydocstyle not to check the tests directory?

Sure.

make them human-legible -> well, if removing them is making them human-ledigble, then lgtm (looks good to me) ;)

make them human-legible -> well, if removing them is making them human-ledigble, then lgtm (looks good to me) ;)

hahahaha, good point. 😂
Let me change that title.


TypeError "hover" of undefined, targetId changes during function call caused inside hover.js of react-dnd.

Describe the bug
Unhandled Runtime Error
TypeError: Cannot read property 'hover' of undefined

The targeted changes during the function call.
Here is a small gif demonstration, below it I will have a written description.

And a gif on what happens on the frontend:

The gist is, when I want to move a task to an empty board, the TypeError about the hover appears. On further inspection, I found out, that it is caused inside of a function inside the hover.js file.
function hoverAllTargets(targetIds, monitor, registry) {
  // Finally call hover on all matching targets.
  targetIds.forEach(function (targetId) {
    var target = registry.getTarget(targetId);
    target.hover(monitor, targetId);
  });
}

On the first run-through, the targetId is T20 and the target can be found. Then there is another run and the targetId changes to a different ID, it is always another ID on each debugging session. Once the targetId is a different ID, it cannot find target anymore and is set to undefined.
const [, dropRef] = useDrop({
    accept: ItemTypes.TASK,
    hover: (_, monitor) => {
      if (!hasTasks) {
        dispatchTaskList({
          type: 'moveTask',
          args: [monitor.getItem().taskId, 0, boardIndex],
        });
      }
    },
  });

This is the function that causes it, it is inside of my Board.js. It is the same function call, to move tasks from board to board when they're already filled. The difference here is, that I have the index (the second argument) set to 0 since the new task will be the first task on that board. I am also checking here if the board has tasks or not.
This is what the function call looks like on Task.js
  const [, dropRef] = useDrop({
    accept: ItemTypes.TASK,
    hover: (_, monitor) => {
      dispatchTaskList({
        type: 'moveTask',
        args: [monitor.getItem().taskId, taskIndex, boardId],
      });
    },
  });

I also don't fully understand what is wrong. I have two examples below, one that works and one that is similar to mine. Both use the same function.
Reproduction
https://codesandbox.io/s/react-dnd-kanban-board-with-issue-ek0xf?file=/src/Kanban.js
Here is my attempt at reproducing the issue. I copied everything 1:1 but removed the MUI styling.
Steps to reproduce the behavior:

Move tasks from one board to another until that board is empty
try to move a task into that empty board
error

You might have to move the task a few times into an empty board before the error appears. I have also created a local version to check against: https://github.com/Braweria/react-dnd-hover-issue
I am using NextJS for my project, which the local version also uses. But again, with no styling.
Expected behavior
And then there is this version: https://codesandbox.io/s/react-dnd-kanban-board-oo3ui?file=/src/App.js
This is the version that works the way I want it to work. You can move a task into an empty board and it moves the task there and displays it as well. You can do it on every board, no error pops up.
Desktop (please complete the following information):

OS: Win10
Browser Chrome
Versions:

    "react-dnd": "^14.0.2",
    "react-dnd-html5-backend": "^14.0.0",

Additional context
I have double-checked with the working example already and I just cannot wrap my head around, why this doesn't work. What is wrong? I have used my working example as guidance for my project, but now the hover error happens.
Here is another issue about the hover error, but with no answers: https://github.com/react-dnd/react-dnd/issues/2563
Here is another working example, but the code is slightly different because it uses nested lists. The similarity lies in that both use hover to call functions and change the state. https://codesandbox.io/s/react-dnd-nested-list-example-fehzw

I managed to find a fix
So after many attempts at trying to fix it myself, I finally managed to do it. There are two things that were wrong with my code. One of them is not present in the sandbox example.
The first thing that was wrong, was my function. I accidentally passed the wrong argument.
dispatchTaskList({
  type: "moveTask",
  args: [monitor.getItem().taskId, 0, boardIndex]
});

This is what I wrote, but the args were supposed to be this args: [monitor.getItem().taskId, 0, boardId] So either I just wrote it wrong or autocomplete messed with me.
Now, with this fix, I managed to move the task to an empty board, but the hover error still occurred.
Now onto the fix that removed the error. In my project, I am using Material UI, which I did not apply to my examples. Whcih was a mistake, because that was the key to fixing it. In my Task.js my return statement was as follows:
You can see the Paper is wrapped in one div-container.
dragRef(dropRef(ref));

 return (

        <Paper
          className={classes.task}
          style={isSpacer ? { backgroundColor: 'red', height: '100%' } : null}
          ref={ref}
        >

{task?.content}

        </Paper>

  );

While debugging I remembered seeing in the React Dev Tools, that the Paper component had some forwardRefs, so that made me think that the ref I am applying to my paper, is not wrapping my paper, but actually it is drilling into it. So I removed that ref and created another div wrapping that paper with the ref. So now the ref is wrapping my Paper.
  return (

        <Paper
          className={classes.task}
          style={isSpacer ? { backgroundColor: 'red', height: '100%' } : null}
        >

{task?.content}

        </Paper>

  );

And no more 'hover' of undefined.


gimme a tBasma step count array out

the equivalent of the index out in taphy, you want the current length of the steps for when you're doing gain[] moduation

(you know the deal, make the change, then define an non-extra output overload)

okay i'm thinking about how i can do this easily with an array on the outside and adding it to the count, achieving the same result while passing a zero array for the gain. it's much more efficient since you don't copy array every cycle but.. ugh! it's making me question the gain arrays in general.
like, yeah they're cool for when you wanna whip something up quickly, but in a situation like this i hate that they're just sitting there, unused. it's confusing:
it'd be like this:
kCount[] fillarray 4, 2, 0, 0,
kGain[] fillarray 0, 0, 0, 0 ;or just init 4
kActualGain[] fillarray 3, 1, 2, 4
kS, kT[] tBasemath kTrig, kCount, kGain, min, max, Q
;then actually do the count modulation manually from the ouside:
kCount[kS] = kCount[kS] + kActualGain[kS]
;and now the count array reflects the length of the step
;because the gain[] modulation is internal and doesn't modefy the count[]

it's actually the same with taphy by the way, there is no real need for the index out (but i like it there)

so do i do it or not? part of me is just lazy because of the overloads lol
but also the workaround is so easy that i kinda regret the index out in taphy? and for some reason i get the urge to.. you know.. new versions! no gain arrays at all, modulation from the ouside only. kinda like the u versions but with Q...

kCount[kS] = kCount[kS] + kActualGain[kS]
gotta limit that to one cycle though! else this be nuts!

if kT[kS] == 1 then
    kCount[kS] = kCount[kS] + kActualGain[kS]
endif
;or maybe just:
kCount[kS] = kCount[kS] + kActualGain[kS]*kT[kS] ;that work?

so i'll do the wise thing here, waste no computing power, waste no time implementing/testing this. let's use the better versions


FLEX-2441 Finding device by identification with exact match is now case sensitive

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

 JpaDeviceSpecifications.java#L319: The Cyclomatic Complexity of this method "forFirmwareModuleVersion" is 12 which is greater than 10 authorized.
 JpaDeviceSpecifications.java#L328: Reduce this anonymous class number of lines from 38 to at most 20, or make it a named class.
 JpaDeviceSpecifications.java#L50: Define a constant instead of duplicating this literal "organisation" 3 times.
 JpaDeviceSpecifications.java#L60: Define a constant instead of duplicating this literal "device" 4 times.
 JpaDeviceSpecifications.java#L72: Define a constant instead of duplicating this literal "deviceIdentification" 5 times.
 JpaDeviceSpecifications.java#L284: Define a constant instead of duplicating this literal "deviceModel" 3 times.

All is well.


Stucked in ccmake: selected BINARAY_WORD_TYPE

Stucked in ccmake: selected BINARAY_WORD_TYPE as below:
I only can switch BINARAY_WORD_TYPE but can't continue

could you elaborate, what do you mean by you are stuck?
just set some options (enter to select/set entry, arrow keys to navigate) and press 'c' to configure and at the end 'g' to generate the makefiles.

Excuse me, I'm not familar this ccmake. Now it's okay!
Greatly thanks for your replying.


Fix: Case-insensitive method calls and class names

:exclamation: Blocked by #491.
This PR

[x] fixes a case-insensitive method call
[x] fixes case-insensitive class names

Thanks.


Option to lift one instance limit for apps using DVDI external volumes

We would like to use the netshare volume driver with the nfs based AWS EFS service for marathon apps. In this case there is no good reason (that I am aware of) to have a constraint of only one app instance because the nfs mount is designed to be mountable in multiple places at once. In our test environment we simply removed this validator in a patched marathon and didn't encounter any problems. However I think some kind of option in the volume configuration would be a better design for my feature request so that people using other volume drivers don't have issues.
Thanks!

cc/ @meichstedt

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-1750. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-1750. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


[option] provide a way to number tracks from playlist

What is the purpose of your issue?

[ ] Bug
[x] Feature Request
[ ] Question
[ ] Other

Description
It would be nice to keep the order of a playlist when downloading it offline (similar need than #598 ), one solution (that works for some players) would be to prepend the filenames with the number of the track in the playlist.
For a playlist :
Def
Abc
Ghi

Current is :
* Abc.mp3
* Def.mp3
* Ghi.mp3

With the option :
* 01 - Def.mp3
* 02 - Abc.mp3
* 03 - Ghi.mp3

Good idea. Great one infact. Could you add this to new Features issue?

Good idea. Great one infact. Could you add this to new Features issue?

Maybe open a PR. That should be an easy job on v3. If your working on it, let me know. I'll keep looking through you're code during dev to keep it inline with our design principles.

Hey, I ain't so sure about the m3u8 thing. Could you walk me through how playlists are made

I'm only seeing this now, nice to see it under-consideration.
Please check out #933. I also had it for v2 in #622.

https://github.com/spotDL/spotify-downloader/issues/867#issuecomment-739237617
This is kinda what I'm working on atm.
My idea is that when you download a spotify playlist, a .m3u is also created for all the songs you downloaded.

Hey, I ain't so sure about the m3u8 thing. Could you walk me through how playlists are made

@Mikhail-Zex  Basically, a .m3u is a playlist file format for playlists. It is a text file, and can be opened in VLC/most media apps. An example below:
Adele - Hello.mp3
Justin Bieber - Sorry.mp3

In this case, the file names are relative to the .m3u file, so the .m3u must be placed in the same directory as the actual song files. To be clear, the .m3u holds no song data, and only the locations for songs in a playlist.
I've started to see the interaction between spotify, python & .m3u files in my repo here
Over this christmas/new years break, I hope to make a PR where there a way to create the .m3u when downloading a playlist with spotdl.
Sorry if this explanation is a bit weirdly worded, I'm not the best haha.

Chill. Open a draft PR as soon as you start.

Coming soon in #1154

What worked for me was using MP3tag to convert the id3 track number into appending that same number at the start of the filename. In other words, the track number is likely already embedded in the id3 metadata.

What worked for me was using MP3tag to convert the id3 track number into appending that same number at the start of the filename. In other words, the track number is likely already embedded in the id3 metadata.

Since V4 you can include the list position in the file name
https://spotdl.readthedocs.io/en/latest/usage/#cli-command-line-interface
Check --output option in the docs link above


Input: The label contains text such as optional which cannot be changed by user.

Describe the bug
The optional text in the input component should be changeable.
To Reproduce
Steps to reproduce the behavior:

Add a input component with required false property.
Add a label title
See label as "foo-lable (optional)"
Expected behavior
As a user, I may not want to see text like 'optional'.

Screenshots

Smartphone:

Device: iPhone15
OS: iOS17
Version: 2.7.1

Thanks for the issue. Currently input required/optional behaves opposite to what we expect. We discussed this situation internally, so we will change the behavior to what you expected.

Hello @ergenekonyigit, if this matter is still ongoing, I can offer my assistance.

Hello @aliziyacevik, a design change is being made by the designers here. We are waiting there. We thank you for your interest.

You can always reach out to me if a hand is needed! @pinarkizilarslan . Thanks for your response.


Permission list in role settings dissapears once enables all permissions for settings

How to reproduce:

Go to settings > Users & Permissions
Create a new role
Give all permissions for “Settings”

Expected outcome: As with the other items in the list, the list should stay and the checkmark gets set.
Actual outcome: Once clicking on the checkmark, the whole permission list dissapears.
Version used: <IP_ADDRESS>.
!Bildschirmaufnahme 2024-08-09 um 10.58.23.mov|width=3436,height=1908,alt="Bildschirmaufnahme 2024-08-09 um 10.58.23.mov"!

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-37630.
[created from NEXT-37630, comment 486935]


Add daily task (X number of AI fights) with money reward

At a minimum this is a tracked number with # of fights completed and a timestamp of the last reset.
We check this the end of every fight, if it's been > 24 hours since last reset and user hasn't completed, we reset it. If user hits the quote within 24 hours, we reward a chunk of extra money and stop counting for that day.

If nobody is working on this currently, I can maybe try setting this up tomorrow. Doesn't seem too bad, could save the time-stamp as a long int using a Date class if PHP has one. If php doesn't have a date class I might run into some trouble.

Would it be possible to add this in for PVP/Missions as well?

Would it be possible to add this in for PVP/Missions as well?

I like that idea, a daily PvP task and daily PvE task?


Allow the app mount element to be configurable.

This is based off of #28.
The appMountElement option can be set to the name of an element; e.g. 'article'. This element is used for all appMountIds. This could be expanded in the future to allow the element to be set per id, but that would be best implemented with breaking changes.
The appMountIds option, and thus the appMountId option, has been modified to be similar to the links and scripts options.
This enhancement was inspired by #24.

I will split everything up. I made this pull request before I saw your comment on #28.

Closing in favor of separate merges.


Proxy with systemd

When the system runs systemd and the proxy options are passed to the module, it should do the right thing ( http://docs.docker.com/engine/articles/systemd/#http-proxy ) either by creating the additional file as explained in the docs or changing the Environment variable in the service file.

This actually can be obviated by tuning the module's parameters (to use systemd overrides)


added payment id to return url

hi,
I could not find a way to set the payment id to the return url.
I have added the paymentId to the return url in the express checkout plugin, this is for the issue #22.
thanks

I've just seen the issue in the PaymentCorebundle where its been suggested to add
$instruction->getExtendedData()->set('return_url', 'foo');
sorry and thanks


SW-5422 Implemented attachments upload flow

🎥 Video uploaded on Graphite:

Screen Recording 2024-05-29 at 1.50.13 PM.mov

#2673  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tommylau523 and the rest of your teammates on  Graphite

UI looks great! I'm getting an error when uploading attachments, but maybe it's the BE?

UI looks great! I'm getting an error when uploading attachments, but maybe it's the BE?

Thanks Constanza! The env variables for BE to work hasn't been merged yet so I don't think it should work.
I also realize that I need a way to prevent users from just accessing the forms if the form types aren't loaded from Atlassian.

Secrets and envs are put into the backend. I think the server will need to restart for it to pick up the changes. Will test it out again soon.

These changes are awesome, nice work @tommylau523!

Merge activity

Jun 3, 2:20 PM EDT: @tommylau523 merged this pull request with Graphite.


[Enhancement] Add-PnPPageSection does not support VerticalSectionZoneEmphasis

Reporting an Issue or Missing Feature
Missing Feature
Expected behavior
Add-PnPPageSection should be able to set the Zone Emphasis of a Vertical Section
Actual behavior
No way to currently do it
Steps to reproduce behavior
Add-PnPPageSection -Page $pageName -SectionTemplate OneColumnVerticalSection -Order 0 -ZoneEmphasis 2 -VerticalSectionZoneEmphasis 1
What is the version of the Cmdlet module you are running?
1.11.0
Which operating system/environment are you running PnP PowerShell on?

[ ] Windows
[ ] Linux
[x] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

Hi @navkri789
Currently you can set the zone emphasis of the vertical section by using -ZoneEmphasis. Both the normal section and the vertical section will get a background color. It is true that you cannot explicitly choose the color of only the vertical section but this is because we are limited by what the CSOM API is offering us.

Closing it as we have merged it via a PR.
It will be available from tomorrow's nightly builds


Way to spawn a car having some speed already.

Currently, the TeleportAction in OpenScenario only uses the position property.

Is there any way to implement it easily?
Without modifying the codes, giving some acceleration area to reach designated velocity in a scenario is the only way?
Please give me any advice.
Thank you in advance.

Use absolutespeedaction in Init. But it might depends on the type of vehicles. I have tried on a motorcycle and a car. I can see it works on the car but the speed of the motor is accelerating.


Harpoon list persistence [harpoon2]

I'm running harpoon2 with the suggested config installing with Lazy on MacOS 12.6.
{
  "ThePrimeagen/harpoon",
  branch="harpoon2",
  config = function()
    local harpoon = require("harpoon");
    vim.keymap.set("n", "<leader>ha", function() harpoon:list():append() end)
    vim.keymap.set("n", "<C-e>", function() harpoon.ui:toggle_quick_menu(harpoon:list()) end)
    vim.keymap.set("n", "<C-h>", function() harpoon:list():select(1) end)
    vim.keymap.set("n", "<C-j>", function() harpoon:list():select(2) end)
  end,
  dependencies = {"nvim-lua/plenary.nvim",},
},

Every time I close nvim, the harpoon list dissapears. If I get back to the first version with my old config, it the list of harpoon files is back.

All right. Let me try it out, I will try it out tonight on stream

yes, all of the previous lists will disappear, @yobibyte are you able to persist after the switch over?
i think i could come up with a way to migrate data

also @sgoudham do you get persistent saves after swapping to H2?  or are you unable to save all together?

@ThePrimeagen I'm unable to keep persistent saves between opening and closing neovim.
I have a feeling that it may be another plugin causing this issue. I'll see if I can reproduce this tomorrow with no other plugins loaded 👍

I think you are missing
harpoon:setup()

When I add this, persistence works.

@sgoudham likely has nothing to do with other plugins.  i should have no issues with other plugins

i assume this is fixed, reopen if its not!

Can confirm that it was missing harpoon:setup() that caused the issue. I lost my old lists after h1 to h2 switch, but the h2 list is persistent now after closing/reopening nvim.

yes,i never wrote a h1 -> h2 list thing... because... well.. i don't think its worth the effort


ROX-13742: Add notes to node inventory

Description
A detailed explanation of the changes in your PR.
Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why
you did not do so. Valid reasons include, for example, "CI is sufficient",
"No testable changes". Feel free to attach JSON snippets, curl commands,
screenshots.
In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

Manual test result:
common/compliance: 2023/01/04 16:51:35.253690 service_impl.go:105: Info: Received connection from "gke-yl-01-04-disgusting--default-pool-83c330ed-ctnp"
common/compliance: 2023/01/04 16:51:36.235906 service_impl.go:105: Info: Received connection from "gke-yl-01-04-disgusting--default-pool-83c330ed-ctnp"
common/compliance: 2023/01/04 16:51:36.238048 node_inventory_handler_impl.go:94: Info: Current node inventory [gke-yl-01-04-disgusting--default-pool-83c330ed-ctnp] note is: LANGUAGE_CVES_UNAVAILABLE
common/signal: 2023/01/04 16:51:36.752743 signal_service.go:107: Error: error dequeueing signalStreamMsg event: rpc error: code = Canceled desc = context canceled
pkg/kocache: 2023/01/04 16:51:39.915513 source.go:66: Info: loading probe 2.3.0-rc1/collector-ebpf-5.4.0-1086-gke.o.gz from upstream /kernel-objects
common/signal: 2023/01/04 16:51:39.961393 signal_service.go:103: Info: starting receiveMessages
common/compliance: 2023/01/04 16:51:41.871954 service_impl.go:105: Info: Received connection from "gke-yl-01-04-disgusting--default-pool-83c330ed-vv0n"
common/compliance: 2023/01/04 16:51:42.812863 service_impl.go:105: Info: Received connection from "gke-yl-01-04-disgusting--default-pool-83c330ed-vv0n"
common/compliance: 2023/01/04 16:51:42.814412 node_inventory_handler_impl.go:94: Info: Current node inventory [gke-yl-01-04-disgusting--default-pool-83c330ed-vv0n] note is: LANGUAGE_CVES_UNAVAILABLE