subscription.pull() should return a list of objects rather than tuples.

Copied from comments in #910
Currently, when pulling messages via a subscription, I'd write:
received = subscription.pull()
for ack_id, message in received:
  print ack_id
  print message['attr1']
  print message.data  # ?
  print message.id  # ?

I think it'd be better to have the return value be an iterable of some more complex object (maybe a pubsub.Message?):
messages = subscription.pull()
for message in messages:
  print message.get_ack_id()  # ReceivedMessage.ackId (string)
  print message['attr1']  # PubsubMessage.attributes (dict)
  print message.get_data()  # PubsubMessage.data (bytes)
  print message.get_id()  # corresponds to PubsubMessage.messageId (string)

What is the point of message.get_data() when we can just say message.data?
@tseaver Are there any issues with making ack_id an optional attribute of Message?

What is the point of message.get_data() when we can just say message.data?

I'm fine with just .data.

Are there any issues with making ack_id an optional attribute of Message?

Can we ack multiple messages at a time? that is, could the following be possible in our current implementation?
received = subscription.pull()
for ack_id, messages in received:
  for message in messages:
    print message.data
  acknowledge(ack_id)

Per the spec, each message in a multi-message pull response has its own ackId, but that ID is not part of the message object, which has its own messageId.
It is kind of like the difference between the "envelope From" and the "payload From: header" in SMTP.  The "transfer agent" cares about the "envelope" ackId (because it must use it to acknowlege the message in order to prevent re-transmission), while the "normal" part of the application would care about the messageId.

Sure, so the idea then is... if we had a Message class, we could put the ackId into the instance, and even provide an acknowledge() method that handles this, right?

We do have a message class, but putting the acknowledge method on it would be wrong:  it is much more closely tied in the API to the subscription (again, "envelope" vs. "payload").

I get that it is logically a separate thing, however conceptually, the only thing I can acknowledge is a single message, right?
I'm OK with the acknowledge method living elsewhere, but wouldn't it make sense to put the ackId into the message?

The bits of the application that deal with the actual message payload are going to be very different from the part that does the pull + acknowlege part:  the latter is likely to be "frameworky".
I think keeping the ackId out of the "payload" is actually a win.

Can you give any more background on why that's a win? If we put it in the message the list returns a list of Message objects. If we don't it's a list of tuples, that has more specific unpacking requirements.
So the code might look like:
for message in subscription.pull():
  acknowledgement_system.acknowledge_receipt(message.ack_id)
  # or even...
  acknowledgement_system.acknowledge_receipt(message)
  message_processing_system.process_message(message)

In other words, I'm saying "I want to acknowledge the receipt of a message", treating the ackId as something that's part of the message. The "payload" is message.data... So a Message object is the envelope, and you can get the payload by looking at .data ..
Is this crazy?

In my mind the "payload" is the data, attrs, and messageId (analogous to headers and body of an SMTP message), while the "envelope" is the ackId.
I'm sorry that I don't seem to be communicating here my sense of the importance of separation of concerns, in addition to staying as close a reasonably possible to the concepts used by the back-end API:

The attributes of the current Message class all come from the PubsubMessage resource, and represent non-ephemeral data about the given message.  They are the same whether the message is obtained via a pull() on a subscription, or via a push to an endpoint.

The ackId is part of the ReceivedMessage resource (along with the message, which is the PubsubMessage):  it is only relevant for pull():  in push, the endpoint's HTTP response code acts as the acknowedgement.

Except in toy examples, the code dealing with pull() and acknowledge() will not care about the message object itself:  it will dispatch it to other code which does care about the message, and then send the acknowlegement.  The dispatched-to code which handles the message object itself won't care about the ackId at all.

OK, and if another attribute comes back with the ReceivedMessage (maybe an expiration or something of that nature) ? Will we continue unpacking tuples still?
It seems like if ReceivedMessage had more than just an ackId + PubsubMessage, we'd have wrapped it in an object. So my main point is...

It seems like it'd be nice if .pull() gets back a list of objects.
Those objects have ways to get the things you want from them

Envelope, aka ackId
Payload, aka PubsubMessage

Since the ackId is the only piece of data, and it has a 1-to-1 relationship with the PubsubMessage itself (I asked about this above), it seemed reasonable to put this together and say .pull() returns an object that let's you "get" the different things you want.

OK, and if another attribute comes back with the ReceivedMessage (maybe an expiration or something of that nature) ? Will we continue unpacking tuples still?

"We" don't unpack them -- the caller does.  If it makes sense, we could bundle the two as a namedtuple, which would allow the caller to access the items so:
for received in subscription.pull():
    if 'log_me' in received.message.attrs:
        print received.message.data
    acknowledge(received.ack_id)

I'd be fine with the named tuple, but then it leads to the question: why would we use a named tuple here instead of an object for ReceivedMessage?
Are we making the decision that "if there's only N properties and 0 methods, we should use a named tuple"?

Also, using your example code, I'm a tiny bit confused why that's preferable to:
for message in subscription.pull():
    if 'log_me' in message.attrs:
        print message.data
    acknowledge(message.ack_id)

Is there some example you can share as to why that is a horrible idea ?

I've tried to share it above:  the Message class we have now is a transport-neutral mapping of the PubsubMessage structure.  Capturing the "pull-only" bits of ReceivedMessage' in a separate class (or the namedtuple`) maintains that clarity / separation of concerns.

So what about the part asking about the named tuple vs object?


feat(iroh-net): upgrade to Quinn 0.11 and Rustls 0.23

Description
Upgrades the Quinn and Rustls dependencies.  This touches a lot of dependencies and affects a lot of the internal API due to the changes in Quinn.
Breaking Changes

iroh::net::endpoint::Endpoint::accept now returns Incoming instead of Connecting. This allows rejecting incoming connections earlier, notify the initiating side to retry later and more. To get back Connecting, simply use Incoming::accept(). If all you did with Connecting before was to .await it, you can do the same with Incoming, too, and get back a Connection.
Removed iroh::net::endpoint::Builder::concurrent_connections. The amount of concurrent connections is now controlled by either calling Incoming::accept or Incoming::refuse after Endpoint::accept.

Notes & open questions
See #2273 for the tracking issue coordinating all this work.  This is the last piece of the puzzle.
Change checklist

[x] Self-review.
[x] Documentation updates following the style guide, if relevant.
[x] Tests if relevant.
[x] All breaking changes documented.

When I try to run sendme send ... with a dependency on this branch, I get:
thread 'main' panicked at /home/philipp/program/work/iroh-quinn11/iroh-net/src/tls.rs:77:6:
Server cert key DER is valid; qed: InvalidCertificate(Other(OtherError(UnsupportedCriticalExtension)))

Seems to be related to the libp2p extension perhaps? I'm investigating.

Information on the windows test failures
Quinn UdpSocket updates on windows
Part of the updates between quinn 0.10 and quinn 0.11 are a bunch of new features to their socket impl: https://github.com/quinn-rs/quinn/pull/1701
Previously, the quinn::Transmit::src_ip: Option<IpAddr> field was simply ignored, now it's actually set via IP_PKTINFO socket options.
Now, in windows it's apparently an error when src_ip: Some(<IP_ADDRESS>), or any other unspecified IP address, since that doesn't help windows figure out which IP interface to use. It's also an error when the src_ip is Ipv6, but the destination is an Ipv4 socket address (and I assume the other way around, too? not sure). In the error msg above you can see both problems.
So, where does the src_ip come from?
It's actually only set on the server side, or in our case, whatever the side ends up being that calls accept, instead of connect.
The mechanism here is, once you accept, you've received a packet, and that packet usually contains the IP address that it arrived at (the dst_ip), and from that quinn determines "oh that's now the interface we should use to answer", which makes sense.
However, in magicsock, we do overwrite the dst_ip when we receive packets, namely with self.normalized_local_addr, which unfortunately resolves to :: or <IP_ADDRESS>.
This dst_ip then gets passed back to quinn, wihch uses it in the src_ip for transmits & it infects the try_send calls quinn makes, causing us to call the socket badly.
What do we do about it?
Not sure. I'd love people to tell me ideas!
The only thing that I think reliably made the test suite pass so far is always setting dst_ip: None.
What hasn't worked, and I find that quite surprising is not overwriting the dst_ip anymore, and set it to None when the packet is "received" from the relay. (Perhaps worth diagnosing this further?)
Other ideas we had:

Set src_ip in the Transmits to results from netcheck
Use a "fake IP address" in relay receives, so that, conceptually, the "relay" is just another "IP interface" that we recieve on (just a fake one).

This PR
For now, this PR does this:
#[cfg(not(windows))]
let dst_ip = self.normalized_local_addr().ok().map(|addr| addr.ip());
#[cfg(windows)]
let dst_ip = None;

Which seems to pass on Linux, Mac and Windows.

Information on the recent gossip_net_smoke flakyness: https://github.com/quinn-rs/quinn/issues/1974

don't we need a ! on the PR description?


accepted image size?

Hello,
I've got the project running, but I'm getting the following error:
Error shape '[1, 3, 2304, 1536]' is invalid for input of size 169869312 IMG_8608_downsized
What image size should I be using to work with this?
Thanks!

Same，testing on different image size ,got the same result .
wondering what is the required image size .

hi, I uploaded the 'inference.py' in the repository, now you can try it again!
I have fixed up some bugs. I ignored the tiling process in the previous version.
Now you can run it easily, it works on my computer. Thanks!


Support the server to specify the folder to the playlist

Are there plans to support the server to specify the folder to the playlist?

@nt1m

@mhb0925 Can you elaborate ? Which kind of server ?

@nt1m Such as https://github.com/andrewpthorp/simple-http-server, support for playing media


API - graphiql error on change

certain functions prevent changes in graphiql (e.g. drop tables) from working.
Does not cause any production issues but makes change via graphiql difficult
error message:
Deleting view failed
the following tracked function(s) cannot be overloaded: filter_rule_nwobj_resolveds, filter_rule_svc_resolveds, filter_rule_user_resolveds

https://stackoverflow.com/questions/64288754/graphql-overloaded-function-error-when-adding-a-computed-field
https://github.com/hasura/graphql-engine/issues/8383 --> stop tracking my function as a query the error goes away for me

functions that are used as computed fields need not be tracked!


TLS context leak

We are currently in the process of closing sockets that have a TLS context. In our closing function, both uCellSockClose and the cleanup function uSockCleanUp are invoked. However, the TLS context remains in the gpcontextlist. It cannot be freed from the asynchronous callback closedCallback because the socket information has already been cleared by the cleanup. Furthermore, the cleanup does not clear the TLS context. Over time, the list becomes full and no socket with a TLS context can be created. This leads to an ‘out of memory’ error.
The question then is whether we should use this cleanup, and if so, when would be the appropriate time to do so?

Hi there, thanks for posting and sorry your have problems with this.
Just to clarify the sequence, I guess you're following the kind of pattern used in the example main_tls.c, which is as follows:

uSockCreate(() a TCP socket,
uSockSecurity() on that socket,
uSockConnect() the socket,
use the socket,
uSockShutdown() the socket (this doesn't actually do very much, it's just there so that the usual BSD sockets pattern can be followed),
uSockClose() the socket,
uSockCleanUp() to clear up any resources.

[...noting that iIn this pattern uCellSockClose() will be called by the uSockClose() function (i.e. it wouldn't be called directly)].
Is that a correct understanding of your flow, and in doing this the memory for the TLS socket is not being marked as free?

Supplementary question: which cellular module type is this being run with?

Good morning,
Thank you for your quick response. Indeed, you are correct. My apologies for the oversight; we naturally call uSockClose, not uCellSockClose. We are utilizing the SARA R422-M8S modem.
When we encounter issues with the server, the TLS context is not eliminated from the gpContextList.
The Socket from the gStaticContainers has already been freed and removed from the list with uSockCleanUp. However, the closedCallback (from u_Sock.c) gets called later and pContainer is then NULL.
I have now attempted without the uSockCleanUp, and everything was cleaned up accordingly.

Excellent: just to be clear then, I think the issue was as follows (please correct me if I'm wrong):

Sockets were being closed, with uSockClose(),
However, socket close on SARA-R422 is asynchronous, so the socket might not close at that time,
Calling uSockCleanUp() before the socket has actually, really, been closed (which would be after the close callback has been called, if you have registered one of those with uSockRegisterCallbackClosed()) results in this leak,

If this is correct I can update the documentation of the uSockCleanUp() function to point out the loop-hole.  Note that if you happen to be opening no more than U_SOCK_NUM_STATIC_SOCKETS (7) sockets, uSockCleanUp() won't actually do anything anyway, since no malloc() will have been needed.

Hey @RobMeades, just wanted to let you know that Florian (@FloriVeit) is my successor and will probably report more issues in the future 😄
Your description is right, that's the problem we observed.
I have two questions regarding this issue:

Is it right that we don't need to call uSockCleanUp() at all? If I remember correctly, SARA R422 only supports 7 sockets anyway, so I guess we shouldn't even be able to exceed U_SOCK_NUM_STATIC_SOCKETS, right?
I understand that calling uSockCleanUp() before the socket was fully closed on the modem side might be undefined behavior, but I wonder if that behavior of uSockCleanUp() makes sense in this case. Wouldn't it be better to just free the TLS context in uSockCleanUp() to avoid theleak? In its current state it's pretty hard for a complex application to safely use uSockCleanUp(), e.g. when multiple threads on a system use the socket layer.

Thanks for your quick answers as always!

Oh, hi @johannesmay, I've got the context now.
I was probably over-thinking when I introduced uSockCleanUp() in the first place: you are correct that, for the cellular case at least, we have U_SOCK_NUM_STATIC_SOCKETS set to the maximum number of sockets that the current set of cellular modules support (to remove the risk of malloc()ed/free()ed memory being referenced); I just wanted to be generic for future cellular modules, or maybe short-range modules (which uses the same code for Wi-Fi-based sockets, and I had heard that there was an extension to BLE in the wings which would do sockets) that might want more simultaneous connections.
But I can describe this in the function header of uSockCleanUp() also, so that people don't have to read the code to understand what's going on.
On the "when to free" topic, I'm basically sh*t scared of asynchronicity holes; bastards to find/debug, so I always err on the side of leaving some RAM hanging, rather than free()ing: the way things are coded [or at least, the intention of the coder :-)] is that any memory that has been allocated and not yet free()ed, when more stuff is required, will be re-used in that new stuff; it doesn't "build up", if you like, it is just an extending pool, so calling uSockCleanUp() would likely be something you do at end of day, when your application is done with comms, that kind of thing.  I can explain this in the function header also.

On the "when to free" topic, I'm basically sh*t scared of asynchronicity holes; bastards to find/debug, so I always err on the side of leaving some RAM hanging, rather than free()ing: the way things are coded [or at least, the intention of the coder :-)] is that any memory that has been allocated and not yet free()ed, when more stuff is required, will be re-used in that new stuff; it doesn't "build up", if you like, it is just an extending pool, so calling uSockCleanUp() would likely be something you do at end of day, when your application is done with comms, that kind of thing. I can explain this in the function header also.

Sure, that makes sense. You basically have a kind of vector-linked-list hybrid where nodes in the list may be empty and reused later on - so far so good.
But I think what we are seeing is not that - in pNewContext() in u_cell_sec_tls.c you allocate the cellular TLS context. The pointers to all TLS contexts are maintained using the static gpContextList. Our issue is that after causing that leak using uSockCleanUp() the gpContextList ends up with lingering contexts that are not referenced by any sockets anymore. The issue is that once that list is full, no more sockets can enable TLS, because pNewContext() will return NULL and thus pUCellSecSecTlsAdd() returns U_ERROR_COMMON_NO_MEMORY.
That was the root cause that triggered our investigation - after a few reconnections from our side, no connection is possible at all without restarting the device.
I would have expected uSockCleanup() to free the TLS context for static sockets it is setting to the CLOSED state here: https://github.com/u-blox/ubxlib/blob/master/common/sock/src/u_sock.c#L1499-L1504

Apologies for the delay: I have made the changes to hopefully free the security context if uSockCleanup() happens to be called while a secured socket is in the "closing but not yet closed" state.  I have pushed a preview branch of the change here:
https://github.com/u-blox/ubxlib/tree/preview_sock_sec_free_rmea
Could one or both of you take a look and confirm that the change is as you expect?

I tested it, and it works flawlessly to fix the problem. Thanks for the fix.
Unfortunately, we don’t have C++ support in our project. With the u_geofence_geodesic.cpp, we can’t build our project anymore. I have opened a separate issue for this.

Excellent, thanks, once this is approved internally I will get it pushed to master here and delete the preview branch.
Will also address the C++ thingy...


Closures not triggered

Xcode: Version 10.1 (10B61)
iPhone 6s Plus 16gb - iOS 12.1.4
Manual install, i am using Reachability inside Cocoa Touch Framework
I tried this example of closures from the README:
//declare this property where it won't go out of scope relative to your listener
let reachability = Reachability()!

reachability.whenReachable = { reachability in
    if reachability.connection == .wifi {
        print("Reachable via WiFi")
    } else {
        print("Reachable via Cellular")
    }
}
reachability.whenUnreachable = { _ in
    print("Not reachable")
}

do {
    try reachability.startNotifier()
} catch {
    print("Unable to start notifier")
}

But no print or error shown in console.
However, then i tried this:
let reachability = Reachability()!
 if reachability.connection != .none {
     if reachability.connection == .wifi {
         print("Via WiFi")
    } else {
         print("Via Cellular")
    }
 } else {
    print("None")
 }

the print statement is called, works fine for detecting network.
Any idea why?

inconsistent in reachability.

@winadiw / @vtkumaresan have you found a workaround for this issue?

I face the same issue

For me it's only in simulator


OWLS-66101, Site documentation for persistent volumes

Comment from Tom B:
My first cut comment:   (1) I don’t think we’re restricting the PV type to NFS & HOST_PATH  (2) the k8s object names in the readme are arbitrary (suggested) not required, so maybe that should be made clear…

One more comment, if a PV/PVC is associated with a particular domain-uid, it’s a best practice to label it with weblogic.domainUID=mydomainuid — this makes it easy to search for the resources associated with a particular domain, it also helps the domain cleanup sample script (which uses labels).

@doxiao Yes, in fact it is based on that readme. I guess this is meant as a general PV documentation whereas the sample one is more specific.

Two docs are mostly duplicate. If we want to keep both, we need to simplify one of them and pointing to the other.

I am wondering if this works.
The main site doc does not provide any samples/templates, instead it just introduces the PV ad PVC, where they can be used with the Operator - say domain home on pv use case or log persistence, dedicated PV vs shared PVC, as well as the common problem with PV.  The sample readme provides the details with samples. The main site doc points to the sample readme.

I agree with Dongbo that the main documentation should describe PV/PVC and the use in this project to persist optionally the domain home, the server logs, and the application binaries. It should describe that if thedomain home is persisted to the PV/PVC then it must be on a shared storage across all the servers in the WebLogic domain.  This main document should describe any issues that might be encountered with PV/PVCs (they are described in the blog https://blogs.oracle.com/weblogicserver/how-to-run-weblogic-clusters-on-the-oracle-cloud-infrastructure-container-engine-for-kubernetes).  This document should link to the sample which describes how to create a PV/PVC in detail.

@tbarnes-us @rosemarymarano I think I resolved your comments, thank you!

@mriccell @doxiao I am looking into combining this with the samples.

Can we keep the configuration, templates, examples and checking of the results in the sample readme  as we agreed upon?

@doxiao Thank you! Is that better now?

@doxiao Thank you again. I took out the naming convention section, but kept the generic parts regarding YAML files, plus I rewrote the dedicated domain part.

@doxiao @mriccell I kept the generic kubectl commands used to create and query kubernetes resources. I do think they belong in the general doc as they are important to the process of creating persistent volumes.

@doxiao @mriccell @rosemarymarano Hi again, sorry to bug everyone. I would like to close this one out, so can we go over what is still missing?

@rosemarymarano @mriccell @jgrundback Cycling back to this one, as well.

I think it looks OK. I'll have a better opportunity to compare it to the PV doc in the samples after it is committed.

@mriccell @doxiao I added what I think we need from the review earlier this week.

@rosemarymarano You can review the rendered version here:
https://github.com/oracle/weblogic-kubernetes-operator/blob/feature/OWLS-66101/site/persistent-volumes.md

It looks fine to me.


Excel sheet for user logins

Needs to make an excel sheet with 2 columns : "Username" and "Participant name"
Username column will have "user01" to "user50".
The participants can take one username and add their names to the "Paticipant name" column. This way each person has their own username in the jupyterhub instance. Need to probably make a big QR code that links to this excel sheet,. Might also need to link the excel sheet in the slides as well.

Ben created excel sheet https://docs.google.com/spreadsheets/d/1-NSPIyJLGA5p2nX_oOhCn1KjIsv2BGnIM2WpFd2cWes/edit?usp=sharing
Closing issue


Merge into Socket component and deprecate SocketClient component

Opening this for the reference and to raise awareness. Note that there's an ongoing discussion as to whether this component should be merged into the Socket component.
See https://github.com/reactphp/socket/issues/74 for details.

This component has now been merged in the the Socket component as per https://github.com/reactphp/socket/issues/74.
If you've previously used the SocketClient component to establish outgoing
client connections, upgrading should take no longer than a few minutes.
All classes have been merged as-is from the latest v0.7.0 release with no
other changes, so you can simply update your code to use the updated namespace
like this:
// old from SocketClient component and namespace
$connector = new React\SocketClient\Connector($loop);
$connector->connect('google.com:80')->then(function (ConnectionInterface $conn) {
    $connection->write('…');
});

// new
$connector = new React\Socket\Connector($loop);
$connector->connect('google.com:80')->then(function (ConnectionInterface $conn) {
    $connection->write('…');
});

See https://github.com/reactphp/socket for more details.


avoid locale dependent test

uses a String transformation from double value to string (in default Locale) so that Scanner instance will be able to parse again the value correctly.

we fixed by 0d585cd5e7c65b154699e99b3d1d1320fe8f1ffc


createOrder not returning order details ?

How can i get current place order ? , i think createOrder should return current order id and order details as well ?

The order details will be present in the order object after calling createOrder()
$client->createOrder($order);
$order->getCode(); // will return the code for the created order
$order->getWhatever();

thanks dear . :+1:


fix typo of heathsite

Coverage remained the same at 39.163% when pulling c1c3fa24003182590049db7ba665e695bde96002 on meomancer:fix-typo into b7e420ff1c1c134a3496d79b5a565aed1e465baf on healthsites:develop.


Pretrained models

Hi authors,
Are there pre-trained models I can use? I found it take a long time to pre-train.
Thank you!

Hi! Here you have! https://drive.google.com/drive/folders/1M1ALzZax4FTv1tjWMAYPhYXTJoH_iriZ?usp=sharing
Sorry for the delay, I had to look for them.

Thanks!
Could you please also release the fine-tuned models?

Here they are https://drive.google.com/drive/folders/1RJptAcZsBixkdJmhKpDoYhPKxlApTcQW?usp=sharing

Thanks!


[BUG] fix plot_crossplot_std for predict_var return

This PR fixes an unreported bug where plot_crossplot_std would crash for for a predict_var return (as opposed to a full distributional prediction).
This is fixed, and a test is added.

Codecov Report
Patch coverage: 100.00% and project coverage change: -0.03% :warning:

Comparison is base (54a73a1) 71.35% compared to head (3acc465) 71.32%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #71      +/-   ##
==========================================
- Coverage   71.35%   71.32%   -0.03%
==========================================
  Files          92       92
  Lines        4940     4942       +2
  Branches      897      897
==========================================
  Hits         3525     3525
- Misses       1179     1183       +4
+ Partials      236      234       -2

Files Changed
Coverage Δ

skpro/utils/plotting.py
94.73% <100.00%> (+1.87%)
:arrow_up:

skpro/utils/tests/test_plots.py
100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Pull upstream changes in netflix/bless into lyft's fork lyft/bless

We did not have any lyft specific code on lyft/bless and git merge from netflix/bless was a fast forward.

👀 @skiptomyliu

Sorry i misclicked


Result of numpy.sum with float32 input is float64

IIUC the standard states that input dtype is to be preserved for floating point types when dtype is unspecified.

dtype - data type of the returned array. If None, the returned array must have the same data type as x, unless x has an integer data type

However,
from array_api_compat import numpy as np
x = np.arange(10, dtype=np.float32)
np.sum(x).dtype  # dtype('float64')

This is part of #127. This was changed in 2023.12. I don't see a need to keep the previously standardized behavior intact. Unfortunately, I haven't yet started work on updating array-api-compat for 2023.12, but I plan to start that work now.

Interesting. I didn't notice that the old standard required that the output dtype be the default floating point type. Glad that was changed!


Separate the handling of the three Android Platform View modes

In support of https://github.com/flutter/flutter/issues/107313 and after reading through PlatformViewsController.java, I think we should separate the logic of the three PlatformView modes with delegates.
The organization of PlatformViewsController.java, makes it hard to decipher what is specifically being done in for each mode. I'm not entirely sure how interconnected each mode is, but there are many methods, fields, and classes that are only used by a single mode.
My suggestion is to create PlatformViewsControllerDelegates to help separate the logic:
io/flutter/plugin/platform
\
 \_PlatformViewsController.java
\
 \_controllerDelegates
  \
   \_virtualDisplay
       \
        \_VirtualDisplayDelegate.java
  \
   \_hybridComposition
       \
        \_HybridCompositionDelegate.java
  \
   \_textureLayerHybridComposition
       \
        \_TextureLayerHybridCompositionDelegate.java

[ ] Virtual Display
[ ] Hybrid Composition
[ ] Texture Layer Hybrid Composition

cc @cyanglaz @stuartmorgan @dnfield @GaryQian

I haven't dug into all of the implementation details, but at a high level this certainly sounds good to me.

In support of #107313

It occurs to me that, somewhat counterintuitively, we should actually do this after #107313 if at all possible. I would like to maximize the chances of us being able to cherry pick the solution there to the current beta branch, and having other PRs that it depends on will complicate that.


refractor: add error handling for fetching contributor's list

Related Issue
#394
Description

add async/await for fetching function
add axios for api call

there are some errors. we need to solve it
[01:29:22.115] Cloning github.com/priyankarpal/ProjectsHut (Branch: api-handling, Commit: a00ef8a)
[01:29:22.496] Cloning completed: 380.966ms
[01:29:25.505] Restored build cache
[01:29:25.535] Running "vercel build"
[01:29:26.046] Vercel CLI 29.3.6
[01:29:26.702] Detected `pnpm-lock.yaml` version 6 generated by pnpm 8...
[01:29:26.717] Installing dependencies...
[01:29:27.381] Lockfile is up to date, resolution step is skipped
[01:29:27.488] Already up to date
[01:29:28.079]
[01:29:28.079] . prepare$ husky install
[01:29:28.134] . prepare: husky - Git hooks installed
[01:29:28.139] . prepare: Done
[01:29:28.140] Done in 1.3s
[01:29:28.184] Running "pnpm run build"
[01:29:28.718]
[01:29:28.718] ><EMAIL_ADDRESS>build /vercel/path0
[01:29:28.718] > vite build
[01:29:28.718]
[01:29:28.972] vite v4.3.8 building for production...
[01:29:29.018] transforming...
[01:29:31.351] ✓ 56 modules transformed.
[01:29:31.352] ✓ built in 2.38s
[01:29:31.352] [vite]: Rollup failed to resolve import "axios" from "/vercel/path0/src/components/ContributorsCard.jsx".
[01:29:31.352] This is most likely unintended because it can break your application at runtime.
[01:29:31.352] If you do want to externalize this module explicitly add it to
[01:29:31.352] `build.rollupOptions.external`
[01:29:31.354] error during build:
[01:29:31.354] Error: [vite]: Rollup failed to resolve import "axios" from "/vercel/path0/src/components/ContributorsCard.jsx".
[01:29:31.354] This is most likely unintended because it can break your application at runtime.
[01:29:31.354] If you do want to externalize this module explicitly add it to
[01:29:31.354] `build.rollupOptions.external`
[01:29:31.354]     at viteWarn (file:///vercel/path0/node_modules/.pnpm/vite@4.3.8_@types+node@20.2.1/node_modules/vite/dist/node/chunks/dep-4d3eff22.js:46597:23)
[01:29:31.354]     at onwarn (file:///vercel/path0/node_modules/.pnpm/@vitejs+plugin-react-swc@3.3.1_vite@4.3.8/node_modules/@vitejs/plugin-react-swc/index.mjs:179:9)
[01:29:31.354]     at onRollupWarning (file:///vercel/path0/node_modules/.pnpm/vite@4.3.8_@types+node@20.2.1/node_modules/vite/dist/node/chunks/dep-4d3eff22.js:46618:9)
[01:29:31.355]     at onwarn (file:///vercel/path0/node_modules/.pnpm/vite@4.3.8_@types+node@20.2.1/node_modules/vite/dist/node/chunks/dep-4d3eff22.js:46368:13)
[01:29:31.355]     at Object.onwarn (file:///vercel/path0/node_modules/.pnpm/rollup@3.22.0/node_modules/rollup/dist/es/shared/node-entry.js:25346:13)
[01:29:31.355]     at ModuleLoader.handleInvalidResolvedId (file:///vercel/path0/node_modules/.pnpm/rollup@3.22.0/node_modules/rollup/dist/es/shared/node-entry.js:23981:26)
[01:29:31.355]     at file:///vercel/path0/node_modules/.pnpm/rollup@3.22.0/node_modules/rollup/dist/es/shared/node-entry.js:23941:26
[01:29:31.355]     at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
[01:29:31.371]  ELIFECYCLE  Command failed with exit code 1.
[01:29:31.393] Error: Command "pnpm run build" exited with 1
[01:29:31.998] Deployment completed
[01:29:31.607] BUILD_UTILS_SPAWN_1: Command "pnpm run build" exited with 1

install the dependencies

Based on the guidelines of GSSoC'23, it appears that this particular issue has not been resolved within the past few days. We would like to kindly inform you that we will be closing this PR.
Thank you for your understanding.


feat: allow multiple devimint clients with the same name

allow you to run latency tests twice on same test dir for example

allow you to run latency tests twice on same test dir for example

Is this fundamentally sound? Two tests using same client at the same time seems like asking for trouble.

it is not same client though, second one uses a different client dir. (foo-1)

it lets you do
latency_test(dev_fed).await;
latency_test(dev_fed).await;

currently it error that client dir already exists

OK. Makese sense. :)

What's status here?

now CI is passing


MAUI: first steps

Supersedes https://github.com/nblockchain/geewallet/pull/199

@webwarrior-ws 2 things:

Can you squash 24baa4d commit with the previous one (but mentioning in commit msg), in the same way that we didn't have a single commit for its addition.
In the commit that introduces the JDK at CI, please add the URL https://stackoverflow.com/a/77519085/544947 to the commit msg for reference.

Superseded by https://github.com/nblockchain/geewallet/pull/274


fix gf gen ctrl import

Codecov Report
Patch coverage has no change and project coverage change: -0.02 :warning:

Comparison is base (41c0dde) 79.13% compared to head (97f57c4) 79.11%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2771      +/-   ##
==========================================
- Coverage   79.13%   79.11%   -0.02%
==========================================
  Files         632      629       -3
  Lines       52014    51675     -339
==========================================
- Hits        41161    40885     -276
+ Misses       8820     8769      -51
+ Partials     2033     2021      -12

Flag
Coverage Δ

go-1.15-386
?

go-1.15-amd64
?

go-1.16-386
?

go-1.16-amd64
?

go-1.17-386
?

go-1.17-amd64
?

go-1.18-386
?

go-1.18-amd64
?

go-1.19-386
?

go-1.19-amd64
?

go-1.20-386
?

go-1.20-amd64
79.11% <ø> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 12 files with indirect coverage changes

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


Reorganize the test infra code

Move the shared test infrastructure code to the project root and make
small adjustments to ensure greater consistency with the rest of the
Camel Core

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] If you're unsure, you can format the pull request title like [CAMEL-XXX] Fixes bug in camel-file component, where you replace CAMEL-XXX with the appropriate JIRA issue.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Run mvn clean install -Psourcecheck in your module with source check enabled to make sure basic checks pass and there are no checkstyle violations. A more thorough check will be performed on your pull request automatically.
Below are the contribution guidelines:
https://github.com/apache/camel/blob/master/CONTRIBUTING.md

@davsclaus @oscerd Here's a proposal for a small reorganization of the camel-test-infra based on your comments on the dev list. What do you think?

I made a few small adjustments, like: renamed top base module camel-test-infra to test-infra to make it consistent with the rest of the modules in the root directory.


Expose Selection Event and Method for Workflow Version Retrieval

This issue proposes two enhancements to Elsa Studio aimed at improving developer experience and interaction with the workflow editor component.
Expose Event on Version Selection from History Tab
Currently, when a user selects a different version of a workflow from the History tab, there's no straightforward way for external components or scripts to react to this change. To address this, we propose adding an event that is fired every time a user selects a version from the History tab. This would enable external components to listen to this event and react accordingly, enhancing the integration capabilities of Elsa Studio.
Add Method to Imperatively Get Selected Workflow Definition Version
To further improve interaction with the Elsa workflow editor, we suggest adding a method that allows developers to imperatively retrieve the currently selected workflow definition version. This method would facilitate scenarios where the external code needs to know which version of the workflow is being edited without relying on UI events or other indirect methods.
These enhancements aim to provide a more interactive and responsive development experience in Elsa Studio, making it easier for developers to integrate and extend the workflow editor's functionality.

Done via https://github.com/elsa-workflows/elsa-studio/pull/130


[Question] How to handle user id in the webchat?

Bot Info

SDK Platform: Node.js
SDK Version: 3.13.1
Active Channels:  Direct Line
Deployment Environment: Azure Bot Service

Issue Description
Question: How to handle user id in the webchat?
I am using the web-chat with a Direct Line API as a channel.  The configuration requires setting a user id and a bot id.
      BotChat.App({
        directLine: { secret: direct_line_secret },
        user: { id: 'userid' },
        bot: { id: 'botid' },
        resize: 'detect'
      }, document.getElementById("bot"));

I tried to access state information from different chatbot instances and they are the same. What is the recommended way to make sure that user state information stays unique?
How should choose the bot id ?

If you have a way of saving independent user data (assuming you're passing a unique userid for each instance), you can set up an object to store the user data and allow it to persist.  Of course, you'll want to write to a file or database, as the basic example I'm about to show will reset every time app.js is reuploaded.
Basic Example:
function UserData(id) {     this.userId = id;     this.messageCount = 0; }
...
`    let user = session.message.address.user.id;
let userData = users.filter((u) => {return u.userId === user;});
let currentUser = userData.length < 1 ? new UserData(user) : userData[0];

session.send(`You said ${session.message.text} as Message # ${++currentUser.messageCount}`);

if (userData.length < 1) users.push(currentUser);

`
I just tested this in a couple browsers, and it persisted the user's message count for each one separately.
If you need the bot ID, it'll be session.message.address.bot.id


Extract pg model

No issue to close
related to #377
Description
simplify pg_model module
extract connection module where Receiver Sender and high level hand shake logic resides
use pg_wire crate instead of protocol module
Is it a feature that change user experience?
no
Client output
no changes

Pull Request Test Coverage Report for Build 322653780

113 of 188 (60.11%)  changed or added relevant lines in 3 files are covered.
20 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.4%) to 88.902%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/node/src/lib.rs
0
1
0.0%

src/connection/src/lib.rs
111
185
60.0%

Files with Coverage Reduction
New Missed Lines
%

src/node/src/lib.rs
20
0%

Totals

Change from base Build 318329816:
-0.4%

Covered Lines:
7546

Relevant Lines:
8488

💛  - Coveralls


Email notifications are not being sent, for anything.

Expected behavior:
When an account is created, an order is created, an order is updated, or any change I expect an email to be sent.
Actual behavior:
Nothing is sent.
Reproduce steps:
Place an order, advance the order and select notify customer.
Version:
Beta 3?
Additional Information:
I already tested sending the test email and it succeeds.
Tried looking in the logs for any errors regarding the emails and couldn't find anything.

https://forum.tastyigniter.com/d/1966-new-order-emails-not-working-in-v304-beta221/9
Changed them to run immediately.


Add program.cs code from dashboard

Resolves #71
Remove the default identity login and replace it with azure AD authentication

@ashleybot the azure AD authentication is working, however, there are some unknowns that i'd like to talk about whenever you get a chance to get into checking over this.
right now, you are able to login with your active directory credentials, but it seems that the scopes are currently unused. Would we want to set up another card to get the scopes to be required and set, or is that something we want to set now. I tried removing the scopes in the current code, but it still allowed me to sign in with my linkblue.

@ashleybot The site right now will launch the user into a sign-in prompt upon visiting the site. I've set up the researchers site to wait for the user to click the sign in link before asking for authentication. I can set it up like that here as well if we would like to allow for unauthenticated access to parts of the site.
we mentioned that this project will be on a different branch for internal purposes, so it may be safe to assume that everyone coming to this branch of the website will be Microsoft Entra (azure AD) accounts?

@ashleybot do you want me to set up the uds web production app registration and set up the appsettings.json for it in this task?
I wasn't super sure if we set up the registration on production registrations after we get the custom domain set up, since the redirect on our other prod apps seems to use the custom domain for auth redirects

PR has been squashed and merged into "release-azure-ad" destination branch via github desktop application and was not tracked on the github web UI.
Closing branch manually.


Refactor MediaSession functions to remove session end closure

Description
Refactor MediaSession, MediaRealTimeSession, and MediaEventProcessor to bring implementation in line with Android implementation.
Remove "sessionEndHandler" closure from Media Session and replace with better logging in MediaRealTimeSession and a bulk remove remove function in MediaEventProcessor to remove sessions which are not active and have no queued events.  Adds getQueueSize() function to MediaSession to get the current queue size, which is used by MediaEventProcessor, along with the existing isMediaSessionActive to determine if a session can be removed from the processor.
Removes trackerSessionId and dispatchQueue from MediaSession as they are not used.
Removes trackerSessionId and trackerConfig from MediaEventProcessor.createSession() as they are not used.
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Just one comment. Rest everything looks good!


TranslocoHttpLoader doesn't load the translation file

Is there an existing issue for this?

[X] I have searched the existing issues

Which Transloco package(s) are the source of the bug?
Transloco
Is this a regression?
Yes
Current behavior
TranslocoHttpLoader doesn't load the translation file without using TranslocoPipe inside the component.
I'm injecting the TranslocoService inside a custom TitleService to handle the browser tab title.
The current behavior is that the TranslocoService can't get the translation because the loader doesn't load it.
Expected behavior
TranslocoHttpLoader should load the translation file even if I don't use the TranslocoPipe.
It should load the translation file at least when injecting the TranslocoService.
Please provide a link to a minimal reproduction of the bug, if you won't provide a link the issue won't be handled.
https://codesandbox.io/p/sandbox/transloco-loader-issue-t2hvyp
Transloco Config
provideHttpClient(),
provideTransloco({
  config: {
    availableLangs: ["en-US", "ar-JO"],
    defaultLang: "en-US",
    reRenderOnLangChange: true,
    prodMode: !isDevMode(),
  },
  loader: TranslocoHttpLoader,
}),

Please provide the environment you discovered this bug in
Transloco: 7.5.0
Angular: 18.2.9
Node: 20.12.2
Package Manager: npm
OS: Windows

Browser
Edge (latest)

Additional context
you have to uncomment the code inside the OnPush Component and import the TranslocoPipe.
I would like to make a pull request for this bug
No

Hey, this isn't a bug; this is how the library works. If you want to load the translations manually, you can use the load method on the Transloco service.

Great! Thanks for your help. I did it and it's working fine.


Add use case unapply

This PR introduces a use case of the unapply function (reverse lookup).
User can add the property optimize to allow optimization of lookup based extraction filter.
The optimization layer will run on the broker and it will rewrite the extraction filter as clause of selector filters.

I was reading the documentation related to this; as part of this PR, could you fix a couple of things in lookups.md?
"funcitonality"  (could almost be a musical term, but not intended as such)
add a period at the end of that sentence

@cheddar and @rasahner  thanks,  done

So, at a high level I think the idea behind this PR is fine, but I don't like the centralization of the logic within the Toolchest. I think it'd be nice to have another layer which is doing query optimization and rewrite that we can extend in the future to do some types of query optimization.
If we treat this new layer as the start of a query planner for Druid, there should be no need for the optimize flag and the planner should do these things automatically.
You can probably ignore most of my code review comments as they concern what I was thinking as I was reading the code.

@fjy and @cheddar please check the new version.

I like the move into an optimize method inside the filter.
@fjy I agree with your comments that having a separation of query -> plan -> rewrite -> plan -> execution totally makes sense.  However, I don't think it makes sense for the current query path.  I think that work should come as part of a new query path.  It shouldn't be too difficult to adjust the internals to enable us to have a different query path that operates on top of the same set of segments, once that's in place it should be possible to experiment with a new query path.
For now, I don't think that should block this PR.
I'm :+1:.

Well, I'm :+1: once tests pass anyway.

@b-slim can you rebase from current master to fix transient failures?

@fjy done, please check it out

I'm :+1: with changes


We can select a complete XML element and then extract it to a local macro (directly in the tool wrapper) or into an external macro file. If there are various imported macro files you can choose where to put it or if there is no imported file it can be created and imported directly.

Resolves https://github.com/galaxyproject/galaxy-language-server/issues/136

This is just magic! Very cool!


[Feature Request] need table VirtualScrollView

Existing Component
否
Component Name
VirtualScrollView
Description

no Pagination use
2.scoll bar is all data height
3 when scroll some position,loading from remote

Duplicate of #9761


request: AnimationSheetBuilder implementation

Is there an existing feature request for this?

[X] I have searched the existing issues.

Command
Implementation of AnimationSheetBuilder for testing animated widgets
Description
Flutter doc: https://api.flutter.dev/flutter/flutter_test/AnimationSheetBuilder-class.html
This feature will help you get images that clearly display the process of complex animations.
Reasoning
useful in cases where you need to compare the results of widget animation with golden.
Additional context and comments
No response

@lesleysin could you elaborate on your request more? I'm having trouble understanding what the ask here is. If you could share a use case, e.g the code of a Widget you're unable to fully test with Alchemist and how the feature you're requesting would help solve that, it would be very helpful. Thank you.

Previously I used golden_toolkit to create golden tests. But due to problems with CI, I started using Alchemist.
I had this example test:

Previously I used golden_toolkit to create golden tests. But due to problems with CI, I started using Alchemist.
I had this example test:
    testWidgets("DuitText props animations", (tester) async {
      final AnimationSheetBuilder animationSheet = AnimationSheetBuilder(
        frameSize: const Size(100, 100),
      );
      addTearDown(animationSheet.dispose);

      final driver = DuitDriver.static(
        _textWithPropAnimation,
        transportOptions: HttpTransportOptions(),
        enableDevMetrics: false,
      );

      final widget = Material(
        child: Directionality(
          textDirection: TextDirection.ltr,
          child: DuitViewHost(driver: driver),
        ),
      );

      await tester.pumpFrames(
        animationSheet.record(widget),
        const Duration(milliseconds: 1000),
      );

      await expectLater(
        animationSheet.collate(10),
        matchesGoldenFile("goldens/d_text_animation.png"),
      );
    });

As a result of running this code, I received the following screenshot, which captures the widget animation process:

I'd like to see an opportunity to do something like this within Alchemist

Gotcha -- this reminds me of https://github.com/Betterment/alchemist/pull/50. @Kirpal was that just lost due to time constraints (totally fine if that's the case) or was there some other blocking reason?

Any news on this issue?

@btrautmann I had built out an implementation of it, but never could get it to reliably pass across local and CI. I believe there were pixel-level diffs on certain frames of the animation. I do think this would be a valuable feature to include, but I'm not really sure what else to try in order to make it pass across platforms, and I don't have the time currently to dig into it again.
As far as I remember, the feat/animations branch has the latest attempts I had made, so someone could merge it with main and try it again, but I'm not too confident in the approach working.
I know this answer's not very helpful with the problem you're facing @lesleysin. The way I've gotten around this in the past is by composing the animation out of multiple components. For example, you could structure the text animation in two widgets, one of which controls the animation (TextAnimation), and one which renders the animation at a specific point (TextAnimationContent). Then, you can test the TextAnimation widget with testWidgets, by checking that it renders TextAnimationContent with the correct values. Finally, you can test TextAnimationContent with golden tests at different points in the animation. I hope that's helpful 🙂
class TextAnimation extends StatefulWidget {
  const TextAnimation({super.key});

  @override
  State<TextAnimation> createState() => _TextAnimationState();
}

class _TextAnimationState extends State<TextAnimation> {
  late AnimationController controller;

  /// Animation logic

  @override
  Widget build(BuildContext context) {
    return AnimatedBuilder(
      animation: controller,
      builder: (context, _) => TextAnimationContent(progress: controller.value),
    );
  }
}

class TextAnimationContent extends StatelessWidget {
  const TextAnimationContent({
    required this.progress,
    super.key,
  });

  final double progress;

  TextStyle get _style => TextStyleTween(
        begin: TextStyle(),
        end: TextStyle(),
      ).transform(progress);

  @override
  Widget build(BuildContext context) {
    return Text('Hello World', style: _style);
  }
}


cannot composer install

using php7.4.9
using composer2.0.9
ran
composer install

got the following output:
Installing dependencies from lock file (including require-dev)
Verifying lock file contents can be installed on current platform.
Warning: The lock file is not up to date with the latest changes in composer.json. You may be getting outdated dependencies. It is recommended that you run `composer update` or `composer update <package name>`.
Your lock file does not contain a compatible set of packages. Please run composer update.

  Problem 1
    - Root composer.json requires php ^8.0 but your php version (7.4.9) does not satisfy that requirement.

could be fixed via
composer require 'php:^7.1.3 || ^8.0'

this issue is caused by #21


I think this is a BC break, too.

I don't think it's a BC because it only changes implementation details.
Anyway your new PRs will be so it does not play a role now.

I'd say it is because it changes how you have to configure the cached converter.


Medal.TV requires input for first-time visit

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Germany
Provide a description that is worded well enough to be understood
As of a few weeks ago, Medal.tv started to show a "Download Medal" screen that required a manual user input (button click) to proceed to the video page itself. Clicking the "Continue in Browser" button instead appends the URL param ?mobilebypass=true at the end of the link and reloads with the now accessible video page.
Manually adding ?mobilebypass=true to the end of the URL works and successfully fetches the video.
I'm aware you don't like screenshots, but this is for the actual relevant webpage, not code

Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['https://medal.tv/de/games/euro-truck-sim-2/clips/1FBcjEIkRoypA6/vFFvmZEfbk2g', '-vU']
[debug] Home config "yt-dlp.conf": ['-o', '~/Videos/ydl/%(id)s_%(title)s.%(ext)s', '--merge-output-format', 'mp4', '-f', '617+ba/137+ba/136+ba/135+ba/22+ba/18+ba/bestvideo+bestaudio', '--audio-format', 'mp3']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [24f827875] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.19045-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: ffmpeg 6.0-full_build-www.gyan.dev (setts), ffprobe 6.0-full_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.19.0, brotli-1.1.0, certifi-2023.07.22, mutagen-1.47.0, requests-2.31.0, sqlite3-3.35.5, urllib3-2.1.0, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests
[debug] Loaded 1901 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[generic] Extracting URL: https://medal.tv/de/games/euro-truck-sim-2/clips/1FBcjEIkRoypA6/vFFvmZEfbk2g
[generic] vFFvmZEfbk2g: Downloading webpage
[redirect] Following redirect to https://medal.tv/games/euro-truck-sim-2/clips/1FBcjEIkRoypA6/vFFvmZEfbk2g/
[MedalTV] Extracting URL: https://medal.tv/games/euro-truck-sim-2/clips/1FBcjEIkRoypA6/vFFvmZEfbk2g/
[MedalTV] 1FBcjEIkRoypA6: Downloading webpage
WARNING: [MedalTV] unable to extract next data; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
ERROR: [MedalTV] 1FBcjEIkRoypA6: Could not find video information.; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U

  File "yt_dlp\extractor\common.py", line 715, in extract
  File "yt_dlp\extractor\medaltv.py", line 92, in _real_extract

@bashonly I would like to work on the problem. Can you please provide me the related test cases so that I can ensure I have correct changes before pushing.

https://github.com/yt-dlp/yt-dlp/blob/master/yt_dlp/extractor/medaltv.py

Thank you for the ?mobilebypass=true tip! hope this gets fixed


0.19 backports

@bilelmoussaoui Should we update gir-files etc in the 0.19 branch for the final versions of the GNOME 46 release?


default value of core.abbrev is different on Mac OS, leading to 404

error
I believe this should fix, probably, #1078 once the release happens after being merged, [but I'm not 100% sure. At the very least it is harmless though.

@dwightguth maybe you should also change the script src/main/scripts/brew-update-to-local as well?

No, that script doesn't use --short


python3Packages.identify: 2.4.4 -> 2.4.5

Motivation for this change
Update to latest upstream release 2.4.5
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 156471 run on x86_64-linux 1

3 packages built:

pre-commit
python310Packages.identify
python39Packages.identify


"Desești", "Hărnicești" and "Mara" are listed in the drop-down menu

Description:
"Desești", "Hărnicești" and "Mara" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Desești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Desești, MARAMUREȘ (Desești)" and "Comuna Desești, MARAMUREȘ".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Hărnicești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Hărnicești, MARAMUREȘ (Desești)".

Step 5
Type "Mara" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Mara, MARAMUREȘ (Desești)".


Add benchmark suite :carousel_horse:

Codecov Report

Merging #13 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #13   +/-   ##
=======================================
  Coverage   80.15%   80.15%
=======================================
  Files           5        5
  Lines         378      378
=======================================
  Hits          303      303
  Misses         75       75

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 16665f1...252bed9. Read the comment docs.


[SMALLFIX]javadoc for AbstractAlluxioShellTest#readContent

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14184/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14272/
Test PASSed.

Merged build finished. Test PASSed.


Codify install instructions

Do note, this does hard code the branch to master. But I think it would be fine.

LGTM

Thanks, improved it slightly.


Some problems about security

When my friends were developing with your framework, they wanted me to test their website for bugs (my friends are not very good at PHP programming), then I found some problems with cross-site scripting attacks, or also called "XSS". In more detail, when I enter "" in the search box and click the button, the page will display an alert. Also, when I enter "" in the content input of the confession wall, the JavaScript will run as expected, and then, others cannot enter the home page (the page will jump to its previous page). My friends think you might be able to add a protection system of network attack, but maybe it's unnecessary, who will attack a confession wall? But my friends think it's a no small problem, so I wrote this issue. In my opinion this won't be a big problem, but who knows? :-)

Thank you. I've already told my friend this. By the way, my friends are Chinese, I just help them find loopholes. Any way I think this is a great project. :-)


rtx verion file support

Description:
node-version-file should support rtx too. heres what .rtx.toml looks like
[tools]
yarn = "1.22.19"
nodejs = "18.16.0"

I saw the regex here doesn't work for this file.
Justification:
We support a few files, let's support one more popular one.
Are you willing to submit a PR?
Yes.

Hello @pvinis. Thank you for your feature request. We'll take a look at it.

't would be amazing if all the setup- actions could use RTX under the hood - but I also understand not wanting to be dependent on something that's not as popular as nvm, rbenv, etc ...

RTX has been renamed to MISE and you can see their action here: https://github.com/jdx/mise-action


"extraneous params: transaction_id" when claiming a reward

Seems like there might be some extra param whitelisting needed on the backend. This particular error occurred when clicking the button to claim the channel surfing reward on the rewards page.

fixed


I found a broken link

Page: https://success.docker.com/article/how-do-i-set-the-docker-daemon-options
Clicked link on word 'docs' (marked with docs) in text:
"
All the Docker daemon options are documented at in the Docker docs.
..."
Went to dead link

Thanks for sharing this issue. It appears that this issue is no longer applicable to the current docs because the parent page no longer exists. If you feel this issue still exists, please let us know.


Adding flags to disable notifications and surveys to avoid unneeded d…

…ecide requests

these config options will disable a/b tests as well right? why not have one configuration option to just turn off decide rather than multiple options where the end user will not know turning off one also turns off the other features?

@ryanseams nice catch and I agree with the idea of turning decide check of.

This is not a good solution in general. Besides what @ryanseams said, current implementation will disable everything (tweaks, notifications, surveys...) only by having notifications OR surveys off.

duplicate.. closing


List sugar

Perhaps use curly braces?

Actually, since maps already use () it might be best to use something like (| |)


Map istio Health status value to envoy to simpify code

Please provide a description of this PR:
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[x] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
a9fce7e76350bf4c953097aa78ff3719a9b533d5
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
a9fce7e76350bf4c953097aa78ff3719a9b533d5
link
true
/test release-notes

unit-tests-arm64_istio
a9fce7e76350bf4c953097aa78ff3719a9b533d5
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
a9fce7e76350bf4c953097aa78ff3719a9b533d5
link
true
/test release-notes

unit-tests-arm64_istio
a9fce7e76350bf4c953097aa78ff3719a9b533d5
link
true
/test unit-tests-arm64

unit-tests_istio
a9fce7e76350bf4c953097aa78ff3719a9b533d5
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
a9fce7e76350bf4c953097aa78ff3719a9b533d5
link
true
/test unit-tests-arm64

unit-tests_istio
a9fce7e76350bf4c953097aa78ff3719a9b533d5
link
true
/test unit-tests

release-notes_istio
073018781d38fad1cbe5da2467545b6aee4db05f
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test release-notes


[BLE] set pairing status based on network provision status

IsPairedToAccount is bound with Weave fabric and service which won't be
introduced in a short time. Set to use network status instead.

@woody-apple ?


Feat: support hostpath in storage and vela cli

Signed-off-by: FogDong<EMAIL_ADDRESS>Description of your changes

Fixes #
I have:

[x] Read and followed KubeVela's contribution process.
[ ] Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
[x] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Special notes for your reviewer

what can hostpath be used?


Using GO it's impossible to set ViewersCanCopyContent to false

file: drive-gen.go
package drive // import "google.golang.org/api/drive/v3"
type File struct {
....
// ViewersCanCopyContent: Whether users with only reader or commenter
// permission can copy the file's content. This affects copy, download,
// and print operations.
ViewersCanCopyContent bool json:"viewersCanCopyContent,omitempty"
}
When you create or update a file, ViewersCanCopyContent is serialized only if it's set to true.
This means that you cannot set "Restrict download, print & copy actions on this file for commenters & viewers" using the GO library, since this option requires ViewersCanCopyContent to be false.
I "fixed" this issue by removing 'omitempty', but that makes the default on create/update the opposite of what one expects.

Try this:
f := &File{
    ViewersCanCopyContent: false,
    ForceSendFields: []string{"ViewersCanCopyContent"},
}

I'm finding the same problem with updating User in google.golang.org/api/admin/directory/v1. The ChangePasswordAtNextLogin attribute is being left out if set to false, and adding ForceSendFields:  []string{"ChangePasswordAtNextLogin"}, doesn't seem to be working. So I don't seem to be able to remove the "change password" flag from the user account if it's previously set.

Try setting the flag: ForceSendFields: []string{"ChangePasswordAtNextLogin
"},
On Tue, Oct 31, 2017 at 2:36 PM, steverippl<EMAIL_ADDRESS>wrote:

I'm finding the same problem with updating User in
google.golang.org/api/admin/directory/v1. The ChangePasswordAtNextLogin
attribute is being left out if set to false, and adding ForceSendFields:
[]string{"ChangePasswordAtNextLogin"}, doesn't seem to be working. So I
don't seem to be able to remove the "change password" flag from the user
account if it's previously set.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/google-api-go-client/issues/229#issuecomment-340914687,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQLiS6KIaPPtrFte3Z_SIrMBzPpipkCTks5sx5LNgaJpZM4PRgQl
.

I'm sorry, I don't understand what you mean by "setting the flag" in the context of the struct field? If I add this as a field, as in...
user := admin.User{
	ChangePasswordAtNextLogin: false,
	HashFunction:              "MD5",
	Password:                  fmt.Sprintf("%x", md5.Sum([]byte(newPassword))),
	ForceSendFields:           []string{"ChangePasswordAtNextLogin"},
}

and then
_, err := srv.Users.Update(email, &user).Do()

It doesn't work, the field won't be updated from true to false.

Using the field ForceSendFields as part of the structure File to specify
that a given field should be serialized, worked for my use case.
Another thing that would work, is to modify the struct definition in the
Google Library so that the field is always serialized.
On Tue, Oct 31, 2017 at 3:53 PM, steverippl<EMAIL_ADDRESS>wrote:

I'm sorry, I don't understand what you mean by "setting the flag" in the
context of the struct field? If I add this as a field, as in...
user := admin.User{
ChangePasswordAtNextLogin: false,
HashFunction:              "MD5",
Password:                  fmt.Sprintf("%x", md5.Sum([]byte(newPassword))),
ForceSendFields:           []string{"ChangePasswordAtNextLogin"},
}
and then
_, err := srv.Users.Update(email, &user).Do()
It doesn't work, the field won't be updated from true to false.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/google-api-go-client/issues/229#issuecomment-340930894,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQLiS67juVxTMzJIi43vqDOgFIGk1mnhks5sx6TPgaJpZM4PRgQl
.

@steverippl Sorry for the delay in getting back to you on this.
I confirmed that your code sends the right JSON body:
{"changePasswordAtNextLogin":false,"hashFunction":"MD5","password":"..."}

If it's still not working, I suggest contacting support for the Admin API, because the problem would be there rather than with this client.


Black Screen on SDDM start since Plasma 5.12

Issue description
When booting NixOS after updating to KDE 5.12 with the latest nixpkgs commit SDDM no longer starts.
Steps to reproduce

Fresh install of NixOS (with KDE and SDDM enabled)
Clone nixpkgs repository from GitHub
Run nixos-rebuild boot --upgrade -I nixpkgs=/my/path/to/nixpkgs
Reboot

Technical details
Hardware Configuration:
Encountered same problem on both VMWare Guest and Thinkpad X220 core-i7.
I had noticed this problem when building the KDE 5.12 packages before they were in the Nix cache and assumed that there was something wrong with my environment at the time.  I've now updated to KDE 5.12 using the latest nixpkgs repository commit which pulled in KDE packages from the cache server and the problem still occurs.
Relevent errors from the log:
Feb 15 09:23:11 nixos sddm-greeter[862]: High-DPI autoscaling not Enabled
Feb 15 09:23:11 nixos sddm-greeter[862]: Reading from "/nix/store/ldyf3y8v0wd7h694lpva9h6vd5dksh2w-desktops/plasma5.desktop"
Feb 15 09:23:11 nixos sddm-greeter[862]: Reading from "/nix/store/ldyf3y8v0wd7h694lpva9h6vd5dksh2w-desktops/xterm.desktop"
Feb 15 09:23:11 nixos sddm-greeter[862]: inotify_add_watch("/usr/share/wayland-sessions") failed: "No such file or directory"
Feb 15 09:23:11 nixos sddm-greeter[862]: Connected to the daemon.
Feb 15 09:23:11 nixos sddm-greeter[862]: Loading file:///run/current-system/sw/share/sddm/themes/breeze/Main.qml...
Feb 15 09:23:12 nixos sddm-greeter[862]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:23:12 nixos sddm-greeter[862]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:23:12 nixos sddm-greeter[862]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:23:12 nixos sddm-greeter[862]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:23:12 nixos sddm-greeter[862]: QObject::installEventFilter(): Cannot filter events for objects in a different thread.
Feb 15 09:23:12 nixos sddm-greeter[862]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:23:12 nixos sddm-greeter[862]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:23:12 nixos sddm-greeter[862]: QObject::installEventFilter(): Cannot filter events for objects in a different thread.
Feb 15 09:23:12 nixos sddm-greeter[862]: file:///run/current-system/sw/share/sddm/themes/breeze/components/VirtualKeyboard.qml:20:1: module "QtQuick.VirtualKeyboard" is not installed
Feb 15 09:23:12 nixos sddm-greeter[862]: Adding view for "Virtual1" QRect(0,0 1024x768)
Feb 15 09:23:12 nixos sddm-greeter[862]: QDBusConnection: name 'org.freedesktop.UDisks2' had owner '' but we thought it was ':1.6'
Feb 15 09:23:12 nixos sddm-greeter[862]: Message received from daemon: Capabilities
Feb 15 09:23:12 nixos sddm-greeter[862]: Message received from daemon: HostName
Feb 15 09:23:14 nixos sddm-greeter[862]: Reading from "/nix/store/ldyf3y8v0wd7h694lpva9h6vd5dksh2w-desktops/plasma5.desktop"
Feb 15 09:23:16 nixos sddm-greeter[862]: Message received from daemon: LoginFailed
Feb 15 09:23:16 nixos sddm-greeter[862]: Message received from daemon: LoginFailed
Feb 15 09:23:44 nixos sddm-greeter[862]: The X11 connection broke: I/O error (code 1)
Feb 15 09:23:51 nixos sddm-greeter[874]: High-DPI autoscaling not Enabled
Feb 15 09:23:51 nixos sddm-greeter[874]: Reading from "/nix/store/ldyf3y8v0wd7h694lpva9h6vd5dksh2w-desktops/plasma5.desktop"
Feb 15 09:23:51 nixos sddm-greeter[874]: Reading from "/nix/store/ldyf3y8v0wd7h694lpva9h6vd5dksh2w-desktops/xterm.desktop"
Feb 15 09:23:51 nixos sddm-greeter[874]: inotify_add_watch("/usr/share/wayland-sessions") failed: "No such file or directory"
Feb 15 09:23:51 nixos sddm-greeter[874]: Connected to the daemon.
Feb 15 09:23:51 nixos sddm-greeter[874]: Loading file:///run/current-system/sw/share/sddm/themes/breeze/Main.qml...
Feb 15 09:23:51 nixos sddm-greeter[874]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:23:51 nixos sddm-greeter[874]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:23:51 nixos sddm-greeter[874]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:23:51 nixos sddm-greeter[874]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:23:51 nixos sddm-greeter[874]: QObject::installEventFilter(): Cannot filter events for objects in a different thread.
Feb 15 09:23:51 nixos sddm-greeter[874]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:23:52 nixos sddm-greeter[874]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:23:52 nixos sddm-greeter[874]: QObject::installEventFilter(): Cannot filter events for objects in a different thread.
Feb 15 09:23:52 nixos sddm-greeter[874]: file:///run/current-system/sw/share/sddm/themes/breeze/components/VirtualKeyboard.qml:20:1: module "QtQuick.VirtualKeyboard" is not installed
Feb 15 09:23:52 nixos sddm-greeter[874]: Adding view for "Virtual1" QRect(0,0 1024x768)
Feb 15 09:23:52 nixos sddm-greeter[874]: QDBusConnection: name 'org.freedesktop.UDisks2' had owner '' but we thought it was ':1.6'
Feb 15 09:23:52 nixos sddm-greeter[874]: Message received from daemon: Capabilities
Feb 15 09:23:52 nixos sddm-greeter[874]: Message received from daemon: HostName
Feb 15 09:23:53 nixos sddm-greeter[874]: Reading from "/nix/store/ldyf3y8v0wd7h694lpva9h6vd5dksh2w-desktops/plasma5.desktop"
Feb 15 09:23:53 nixos sddm-greeter[874]: Message received from daemon: LoginSucceeded
Feb 15 09:25:08 nixos sddm-greeter[860]: High-DPI autoscaling not Enabled
Feb 15 09:25:08 nixos sddm-greeter[860]: Reading from "/nix/store/ldyf3y8v0wd7h694lpva9h6vd5dksh2w-desktops/plasma5.desktop"
Feb 15 09:25:08 nixos sddm-greeter[860]: Reading from "/nix/store/ldyf3y8v0wd7h694lpva9h6vd5dksh2w-desktops/xterm.desktop"
Feb 15 09:25:08 nixos sddm-greeter[860]: inotify_add_watch("/usr/share/wayland-sessions") failed: "No such file or directory"
Feb 15 09:25:08 nixos sddm-greeter[860]: Connected to the daemon.
Feb 15 09:25:08 nixos sddm-greeter[860]: Loading file:///run/current-system/sw/share/sddm/themes/breeze/Main.qml...
Feb 15 09:25:09 nixos sddm-greeter[860]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:25:09 nixos sddm-greeter[860]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:25:09 nixos sddm-greeter[860]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:25:09 nixos sddm-greeter[860]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:25:09 nixos sddm-greeter[860]: QObject::installEventFilter(): Cannot filter events for objects in a different thread.
Feb 15 09:25:09 nixos sddm-greeter[860]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:25:09 nixos sddm-greeter[860]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:25:09 nixos sddm-greeter[860]: QObject::installEventFilter(): Cannot filter events for objects in a different thread.
Feb 15 09:25:09 nixos sddm-greeter[860]: file:///run/current-system/sw/share/sddm/themes/breeze/components/VirtualKeyboard.qml:20:1: module "QtQuick.VirtualKeyboard" is not installed
Feb 15 09:25:09 nixos sddm-greeter[860]: Adding view for "Virtual1" QRect(0,0 1024x768)
Feb 15 09:25:09 nixos sddm-greeter[860]: QDBusConnection: name 'org.freedesktop.UDisks2' had owner '' but we thought it was ':1.6'
Feb 15 09:25:09 nixos sddm-greeter[860]: Message received from daemon: Capabilities
Feb 15 09:25:09 nixos sddm-greeter[860]: Message received from daemon: HostName
Feb 15 09:25:45 nixos sddm-greeter[860]: Reading from "/nix/store/ldyf3y8v0wd7h694lpva9h6vd5dksh2w-desktops/plasma5.desktop"
Feb 15 09:25:45 nixos sddm-greeter[860]: Message received from daemon: LoginSucceeded
Feb 15 09:39:16 nixos sddm-greeter[886]: High-DPI autoscaling not Enabled
Feb 15 09:39:16 nixos sddm-greeter[886]: Loading theme configuration from "/run/current-system/sw/share/sddm/themes/breeze/theme.conf"
Feb 15 09:39:16 nixos sddm-greeter[886]: Reading from "/nix/store/71w3sf41c36ci6kdpxsdbp9ch2bnsj2r-desktops/plasma5.desktop"
Feb 15 09:39:16 nixos sddm-greeter[886]: Reading from "/nix/store/71w3sf41c36ci6kdpxsdbp9ch2bnsj2r-desktops/xterm.desktop"
Feb 15 09:39:16 nixos sddm-greeter[886]: inotify_add_watch("/usr/share/wayland-sessions") failed: "No such file or directory"
Feb 15 09:39:16 nixos sddm-greeter[886]: Connected to the daemon.
Feb 15 09:39:17 nixos sddm-greeter[886]: Loading file:///run/current-system/sw/share/sddm/themes/breeze/Main.qml...
Feb 15 09:39:17 nixos sddm-greeter[886]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:39:17 nixos sddm-greeter[886]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:39:17 nixos sddm-greeter[886]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:39:17 nixos sddm-greeter[886]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:39:17 nixos sddm-greeter[886]: QObject::installEventFilter(): Cannot filter events for objects in a different thread.
Feb 15 09:39:17 nixos sddm-greeter[886]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:39:17 nixos sddm-greeter[886]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:39:17 nixos sddm-greeter[886]: QObject::installEventFilter(): Cannot filter events for objects in a different thread.
Feb 15 09:39:17 nixos sddm-greeter[886]: Empty filename passed to function
Feb 15 09:39:17 nixos sddm-greeter[886]: Empty filename passed to function
Feb 15 09:39:17 nixos sddm-greeter[886]: Empty filename passed to function
Feb 15 09:39:17 nixos sddm-greeter[886]: Adding view for "Virtual1" QRect(0,0 1024x768)
Feb 15 09:39:17 nixos sddm-greeter[886]: QDBusConnection: name 'org.freedesktop.UDisks2' had owner '' but we thought it was ':1.6'
Feb 15 09:39:17 nixos sddm-greeter[886]: Message received from daemon: Capabilities
Feb 15 09:39:17 nixos sddm-greeter[886]: Message received from daemon: HostName
Feb 15 09:39:38 nixos sddm-greeter[886]: Reading from "/nix/store/71w3sf41c36ci6kdpxsdbp9ch2bnsj2r-desktops/plasma5.desktop"
Feb 15 09:39:38 nixos sddm-greeter[886]: Message received from daemon: LoginSucceeded
Feb 15 09:47:36 nixos sddm-greeter[860]: High-DPI autoscaling not Enabled
Feb 15 09:47:36 nixos sddm-greeter[860]: Loading theme configuration from "/run/current-system/sw/share/sddm/themes/breeze/theme.conf"
Feb 15 09:47:36 nixos sddm-greeter[860]: Reading from "/nix/store/209rnrb281yaypi033djq5gc646dyv0z-desktops/plasma5.desktop"
Feb 15 09:47:36 nixos sddm-greeter[860]: Reading from "/nix/store/209rnrb281yaypi033djq5gc646dyv0z-desktops/xterm.desktop"
Feb 15 09:47:36 nixos sddm-greeter[860]: inotify_add_watch("/usr/share/wayland-sessions") failed: "No such file or directory"
Feb 15 09:47:36 nixos sddm-greeter[860]: Connected to the daemon.
Feb 15 09:47:36 nixos sddm-greeter[860]: Loading file:///run/current-system/sw/share/sddm/themes/breeze/Main.qml...
Feb 15 09:47:37 nixos sddm-greeter[860]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:47:37 nixos sddm-greeter[860]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:47:37 nixos sddm-greeter[860]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:47:37 nixos sddm-greeter[860]: QObject: Cannot create children for a parent that is in a different thread.
Feb 15 09:47:37 nixos sddm-greeter[860]: QObject::installEventFilter(): Cannot filter events for objects in a different thread.
Feb 15 09:47:37 nixos sddm-greeter[860]: QObject: Cannot create children for a parent that is in a different thread.

I did some research on the error message and found that people have run into this with different versions of KDE but I wasn't able to determine a common root cause.  So far I haven't had any luck in fixing it.
Related PR:  #34726

Could you please delete the sddm qml cache dir?
sudo rm -fr /var/lib/sddm/.cache/sddm-greeter/qmlcache
If that fixes it, this belongs to:
https://github.com/NixOS/nixpkgs/issues/32577

Yes, removing the cache fixed the problem for me.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/sddm-fails-to-start-x11-session-for-non-root-user-in-kde-plasma-using-nvidia-after-upgrading-to-22-11/24139/1


[2.0] must be of the type array, object given

Bug Report

Q
A

BC Break
dk

Version
2.0.0-alpha1

Hi,
I'm trying out 2.0.0-alpha1 and I get a lot of these: must be of the type array, object given. Please consider the following excerpt from Doctrine\ODM\MongoDB\Persisters\DocumentPersister:
    /**
     * Creates or fills a single document object from an query result.
     *
     * @param object $result   The query result.
     * @param object $document The document object to fill, if any.
     * @param array  $hints    Hints for document creation.
     *
     * @return object The filled and managed document object or NULL, if the query result is empty.
     */
    private function createDocument($result, ?object $document = null, array $hints = []) : ?object
    {
        if ($result === null) {
            return null;
        }

        if ($document !== null) {
            $hints[Query::HINT_REFRESH] = true;
            $id                         = $this->class->getPHPIdentifierValue($result['_id']);
            $this->uow->registerManaged($document, $id, $result);
        }

        return $this->uow->getOrCreateDocument($this->class->name, $result, $hints, $document);
    }

$result is said to be an object (the |null is missing). In the test suite it's always an array, but in my application it's often a MongoDB\Model\BSONDocument, which can be used as an array because it extends ArrayObject. The problem is that both registerManaged() and getOrCreateDocument() expect $result to be an array.
From what I gathered from my tests, this happens when I'm searching for a document, using findOneBy() for instance.
        $family = $this->repository->findOneBy([

            Family::PROPERTY_COUNTRY => (string) $country,
            Family::PROPERTY_HANDLE => $handle,

        ]);

Raises the following exception:
1) HelloFresh\MenuService\Application\Command\Menu\IntegrationTest::shouldFailToActivateAnInvalidMenu
Failed asserting that exception of type "TypeError" matches expected exception "\HelloFresh\MenuService\Application\Exception\ValidationException". Message was: "Argument 2 passed to Doctrine\ODM\MongoDB\UnitOfWork::getOrCreateDocument() must be of the type array, object given, called in /app/vendor/doctrine/mongodb-odm/lib/Doctrine/ODM/MongoDB/Persisters/DocumentPersister.php on line 601" at
/app/vendor/doctrine/mongodb-odm/lib/Doctrine/ODM/MongoDB/UnitOfWork.php:2467
/app/vendor/doctrine/mongodb-odm/lib/Doctrine/ODM/MongoDB/Persisters/DocumentPersister.php:601
/app/vendor/doctrine/mongodb-odm/lib/Doctrine/ODM/MongoDB/Persisters/DocumentPersister.php:473
/app/vendor/doctrine/mongodb-odm/lib/Doctrine/ODM/MongoDB/Repository/DocumentRepository.php:166
/app/src/Infrastructure/Persistence/Finder/FamilyFinder.php:32

I tried to converting $result into a array when a BSONDocument, but then it's another method failing for the same reason. What I don't understand is that I'm using the exact same driver both Docker images, one I created for mongodb-odm and the one for my service. I don't get it :(

Found the solution in the tests:
$client = new Client(
    getenv('DOCTRINE_MONGODB_SERVER') ?: DOCTRINE_MONGODB_SERVER,
    [],
    ['typeMap' => ['root' => 'array', 'document' => 'array']]
);

I was missing that ['typeMap' => ['root' => 'array', 'document' => 'array']] part. We really need an updated documentation, it still talk about Doctrine\MongoDB\Connection when we need a Mongo\Client.
https://github.com/doctrine/mongodb-odm/blob/f3ed76beb4dea085f06a407c9e12681e8aa59375/docs/en/reference/introduction.rst#setup
I'll see what I can do to help.

Interesting, did you create the client yourself? The type map should be applied automatically. We may however need to override it if a client is provided.
I’ll work on updating the documentation soon, completely missed the old classes there 😒

Looks like the default typeMap is wrong: https://github.com/mongodb/mongo-php-library/blob/bd148eab0493e38354e45e2cd7db59b90fdcad79/src/Client.php#L36
    private static $defaultTypeMap = [
        'array' => 'MongoDB\Model\BSONArray',
        'document' => 'MongoDB\Model\BSONDocument',
        'root' => 'MongoDB\Model\BSONDocument',
    ];

We set a type map when creating a document manager, but not when a client is provided, hence my question whether you create the client yourself.


#538 remove autobind, refactor functions to arrow notation methods

Fixes #538

Will do a little bit more testing tomorrow, not ready for merge yet

@andreievg How did that testing go?

no extensive testing yet. 'shallow' testing went well.. will endeavour to do a full test asap

Probably also fixes other this.props bugs: #548 #545 #546

Did a thorough test, haven't come across any issues.


Bump to version 21.37.0

Add heading option to warning component (PR #1415)
Add the "distribution" property to the dataset machine readable metadata (PR #1416)
Fix the dataset machine readable metadata erroring if there is no description set (PR #1416)

Thanks @alex-ju - I'd just realised myself too 😅


benchmark with Postgres throws relation error

I get the following error while trying:
nodejs benchmark postgresql -a myhost1 -t all

INFO using server address myhost1
INFO start
INFO version [{"version":"PostgreSQL 9.5.6 on x86_64-pc-linux-gnu, compiled by gcc (Ubuntu 5.4.0-6ubuntu1~16.04.4) 5.4.0 20160609, 64-bit"}]
ERROR error: relation "relations" does not exist

I get the feeling I am missing something out. Do I have to preload the database? How?

hi @felixkretschmer Did you get this one working?

Nop. Sorry.

Marcin Kubica<EMAIL_ADDRESS>hat am 27. November 2017 um 12:37 geschrieben:
hi @felixkretschmer Did you get this one working?
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/weinberger/nosql-tests/issues/16#issuecomment-347156851


Improve docs of flatten, reshape

This is PR for improving docs of functions/links. Related issue: #2182
Modified functions/links:

functions/array/flatten
functions/array/reshape

GPU test passed. It is ready to be merged.

Thank you fort the reply! I will wait for the merge.


Create page via the vue cli plugin library

[ ] We need when setting up the project with vue cli plugin library to have a dynamic page created that will  use for the name the url of the page instead of a hardcoded one

@jfayad Please to elaborate more on how this should be done.
Currently the vue cli plugin ask the user to input his projectUrl and his projectId.
If projectUrl is http://localhost:3000 for example, we should extract the page name from it ? And in this case if no page was specified in the url (/homePage), what will be the created section name ?
Also currently the section that is being added is inside generator template and it is called Sections.vue So I am not sure the vue cli plugin can rename this page to what we need but I need to check if it is possible.


Restangular.copy causing the route in the object to become 'undefined'

Hey guys
I am trying to make a PUT request by copying the original restangular object, editing its properties and calling put.
But when I do a restangular.copy, it does not make a correct copy of the original. I was able to figure out that it sets the 'route' property to undefined.
See the fiddle below, it demonstrates the copy behavior.
http://jsfiddle.net/uma1rkhan/HB7LU/4678/
I dont know if this is correct or not but I am not able to make a successful PUT request with the copied object.
thanks :)

+1, reqParams is also undefined after copy. Maybe other stuff too...

+1, Restangular.copy() on a collection breaks the collection. If I do a collectionName.post(newObj) after the copy(), there is an extra "/[object%20Object]" added to the route

+1

+1

Yup this is definitely broken. Confirmed with the latest version of Restangular:
http://plnkr.co/edit/zDuP4zWH6fD4ATSxPb03?p=preview
If someone wants to track this down and submit a PR, that would be awesome.

It's been a while.

Is this closed because its resolved? I get the same issue using restangular 1.5.2, where the copy is not done properly. I followed the plunker example. (http://plnkr.co/edit/d6yDka?p=preview)
 {   "_id":{     "$oid":"57f2a51ebd966f4fb6b51548"   },   "name":"Sample 2",   "description":"asdas so this is important and its using schema form.",   "site":"http://www.google.com",   "route":"projects",   "reqParams":{     "apiKey":"4f847ad3e4b08a2eed5f3b54"   },   "restangularized":true,   "fromServer":true,   "parentResource":null,   "restangularCollection":false }
{   "_id":{     "$oid":"57f2a51ebd966f4fb6b51548"   },   "name":"Sample 2",   "description":"asdas so this is important and its using schema form.",   "site":"http://www.google.com",   "route":"projects",   "reqParams":null,   "restangularized":true,   "fromServer":true,   "parentResource":null,   "restangularCollection":false }

Hi,
Still happening on 1.6.1
http://plnkr.co/edit/gwSsimaEDmC6IJHorcNA?p=preview
Regards,
Andy


Support boolean expression rule for gray release

For now there are 4 matching rules: match, =, regEx, like. A new customized rule with parameters will make gray release more flexible. The previous discussion can be found at https://github.com/dromara/soul/issues/1288

I suggest that we add a boolean expression rule, if the expression is true then the query will be routed to the new version.
The provision may like:

Set the parameter(from URI or Header), like 'user_id'.
Set the expression, like ${user_id}%5=0 , ${user_id} in (1,2,3)

With this rule, users can simply control the new version proportion by adjust the expression instead of design the URI or Header.
Maybe we could use groovy script parse the expression, or something else.

The groovy script is more flexible

groovy script

The groovy script is more flexible, but user shoud be programmer .we need to consider the cost of study.

The groovy script is more flexible, but user shoud be programmer .we need to consider the cost of study.

Agree. I wonder how to parse a common expression, any suggestions?

Spring Expression Language (SpEL)

Spring Expression Language (SpEL)

good idea

The groovy script is more flexible

good idea

groovy script + SpEL  ?

groovy script + SpEL ?

the two strategy


added minor change to allVotelogYaml

Added sort to allVotelogYaml. It will fetch data in descending order.
console.log added in render just for demonstration purpose that top 10 votes will be first 10 elements of the array
resolved conflicts with master. i accepted incoming changes wherever i didn't make any change.

@sprakash57  Thanks for PR. I'll accept what you did on votelog.yaml and sorting allVotelogYaml.
But the changes on /src/page/index.js inside IndexPage is a bit confusing (there's code that was deleted!) and you shouldn't commit console.log. Please don't do anything on this part.

@rapee Sincere apologies!! and thank you for letting me work on this interesting project. console.log() was a mistake. I was testing the response. But i didn't delete any code. i only merged master in my forked repo and accepted all incoming changes. I don't have prettier extension installed. That could be the reason of formatting code. Should that be part of my development? If you want i can raise a fresh PR and closing this one. little guidance would be appreciated.

As I mentioned, you can commit just votelog.yaml and the sorting allVotelogYaml part in index.js. You can discard other changes.
And yes we use prettier for formatting in this project. If you use VSCode there's prettier plugin. However, I recently added husky and lint-staged to automatically do prettier on git commit.