Import League of Legends

This is a stand alone game, and not part of a launcher, but we should be able to import it all the same.
Not required for initial release.

Can this issue be expanded to include all riot games that use the riot launcher?


Log compaction fails due to timeout

Describe the bug
Log compaction fails due to a timeout.
To Reproduce

Start pulsar standalone
Create producer
Produce messages at high throughput
Trigger topic compaction
Read compaction status

Detailed steps to reproduce:

Clear all state

# rm -r data logs # !!! careful !!! this wipes all your topics etc.

Start pulsar standalone

bin/pulsar standalone

Create producer

PulsarClient pulsarClient = PulsarClient.builder()
    .serviceUrl("pulsar://<IP_ADDRESS>:6650")
    .build();
Producer<byte[]> producer = pulsarClient.newProducer()
    .topic("my-topic")
    .create();

Produce messages at high throughput

int msgCount = 500000;
long before = System.currentTimeMillis();
for (int i = 0; i < msgCount; i++) {
    producer.newMessage()
            .key("asdf")
            .value(new byte[] {1, 2, 3, 4, 5})
            .sendAsync();
}
System.out.println("Sent " + msgCount + " messages in " + (System.currentTimeMillis() - before) + " msec");

Trigger topic compaction

$ bin/pulsar-admin topics compact persistent://public/default/my-topic
Topic compaction requested for persistent://public/default/my-topic

Read compaction status

$ bin/pulsar-admin topics compaction-status persistent://public/default/my-topic
Compaction is currently running

$ # WAIT FOR 10 SECONDS

$ bin/pulsar-admin topics compaction-status persistent://public/default/my-topic
Error in compaction
null

Reason: Error compacting: java.util.concurrent.TimeoutException: Timeout

Expected behavior
Compaction and no error
Desktop (please complete the following information):

Pulsar version: 2.5.0 (tar.gz from website)
OS: Linux 4.19.0-4-amd64 #1 SMP Debian 4.19.28-2 (2019-03-15) x86_64 unknown unknown GNU/Linux

Logs
The complete logs from the start of compaction until the error was read via compaction status
09:47:54.409 [pulsar-web-57-8] INFO  org.apache.pulsar.client.impl.ConsumerStatsRecorderImpl - Starting Pulsar consumer status recorder with config: {
  "topicNames" : [ "persistent://public/default/my-topic" ],
  "topicsPattern" : null,
  "subscriptionName" : "__compaction",
  "subscriptionType" : "Exclusive",
  "receiverQueueSize" : 1000,
  "acknowledgementsGroupTimeMicros" : 100000,
  "negativeAckRedeliveryDelayMicros" : 60000000,
  "maxTotalReceiverQueueSizeAcrossPartitions" : 50000,
  "consumerName" : null,
  "ackTimeoutMillis" : 0,
  "tickDurationMillis" : 1000,
  "priorityLevel" : 0,
  "cryptoFailureAction" : "FAIL",
  "properties" : { },
  "readCompacted" : true,
  "subscriptionInitialPosition" : "Latest",
  "patternAutoDiscoveryPeriod" : 1,
  "regexSubscriptionMode" : "PersistentOnly",
  "deadLetterPolicy" : null,
  "autoUpdatePartitions" : true,
  "replicateSubscriptionState" : false,
  "resetIncludeHead" : false,
  "keySharedPolicy" : null
}
09:47:54.410 [pulsar-web-57-8] INFO  org.apache.pulsar.client.impl.ConsumerStatsRecorderImpl - Pulsar client config: {
  "serviceUrl" : "pulsar://localhost:6650",
  "authPluginClassName" : null,
  "authParams" : null,
  "operationTimeoutMs" : 30000,
  "statsIntervalSeconds" : 60,
  "numIoThreads" : 1,
  "numListenerThreads" : 1,
  "connectionsPerBroker" : 1,
  "useTcpNoDelay" : true,
  "useTls" : false,
  "tlsTrustCertsFilePath" : null,
  "tlsAllowInsecureConnection" : false,
  "tlsHostnameVerificationEnable" : false,
  "concurrentLookupRequest" : 5000,
  "maxLookupRequest" : 50000,
  "maxNumberOfRejectedRequestPerConnection" : 50,
  "keepAliveIntervalSeconds" : 30,
  "connectionTimeoutMs" : 10000,
  "requestTimeoutMs" : 60000,
  "initialBackoffIntervalNanos" : 100000000,
  "maxBackoffIntervalNanos" :<PHONE_NUMBER>0
}
09:47:54.411 [pulsar-web-57-8] INFO  org.eclipse.jetty.server.RequestLog - <IP_ADDRESS> - - [31/1/2020:09:47:54 +0900] "PUT /admin/v2/persistent/public/default/my-topic/compaction HTTP/1.1" 204 0 "-" "Pulsar-Java-v2.5.0" 6
09:47:54.411 [pulsar-client-io-85-1] INFO  org.apache.pulsar.client.impl.ConsumerImpl - [persistent://public/default/my-topic][__compaction] Subscribing to topic on cnx [id: 0xeb973738, L:/<IP_ADDRESS>:60180 - R:localhost/<IP_ADDRESS>:6650]
09:47:54.412 [pulsar-io-50-7] INFO  org.apache.pulsar.broker.service.ServerCnx - [/<IP_ADDRESS>:60180] Subscribing on topic persistent://public/default/my-topic / __compaction
09:47:54.412 [pulsar-io-50-7] INFO  org.apache.bookkeeper.mledger.impl.ManagedCursorImpl - [public/default/persistent/my-topic-__compaction] Rewind from 1544:100 to 1544:0
09:47:54.412 [pulsar-io-50-7] INFO  org.apache.pulsar.broker.service.persistent.PersistentTopic - [persistent://public/default/my-topic] There are no replicated subscriptions on the topic
09:47:54.412 [pulsar-io-50-7] INFO  org.apache.pulsar.broker.service.persistent.PersistentTopic - [persistent://public/default/my-topic][__compaction] Created new subscription for 1
09:47:54.412 [pulsar-io-50-7] INFO  org.apache.pulsar.broker.service.ServerCnx - [/<IP_ADDRESS>:60180] Created subscription on topic persistent://public/default/my-topic / __compaction
09:47:54.413 [pulsar-client-io-85-1] INFO  org.apache.pulsar.client.impl.ConsumerImpl - [persistent://public/default/my-topic][__compaction] Subscribed to topic on localhost/<IP_ADDRESS>:6650 -- consumer: 1
09:47:54.413 [pulsar-client-io-85-1] INFO  org.apache.pulsar.client.impl.ConsumerImpl - [persistent://public/default/my-topic][__compaction] Seek subscription to message id -1:-1:-1
09:47:54.415 [pulsar-io-50-7] INFO  org.apache.pulsar.broker.service.Consumer - Disconnecting consumer: Consumer{subscription=CompactorSubscription{topic=persistent://public/default/my-topic, name=__compaction}, consumerId=1, consumerName=b411f, address=/<IP_ADDRESS>:60180}
09:47:54.416 [pulsar-io-50-7] INFO  org.apache.pulsar.broker.service.AbstractDispatcherSingleActiveConsumer - Removing consumer Consumer{subscription=CompactorSubscription{topic=persistent://public/default/my-topic, name=__compaction}, consumerId=1, consumerName=b411f, address=/<IP_ADDRESS>:60180}
09:47:54.416 [pulsar-client-io-85-1] INFO  org.apache.pulsar.client.impl.ClientCnx - [localhost/<IP_ADDRESS>:6650] Broker notification of Closed consumer: 1
09:47:54.416 [pulsar-client-io-85-1] INFO  org.apache.pulsar.client.impl.ConnectionHandler - [persistent://public/default/my-topic] [__compaction] Closed connection [id: 0xeb973738, L:/<IP_ADDRESS>:60180 - R:localhost/<IP_ADDRESS>:6650] -- Will try again in 0.1 s
09:47:54.416 [pulsar-io-50-7] INFO  org.apache.pulsar.broker.service.persistent.PersistentSubscription - [persistent://public/default/my-topic][__compaction] Successfully disconnected consumers from subscription, proceeding with cursor reset
09:47:54.416 [bookkeeper-ml-workers-OrderedExecutor-2-0] INFO  org.apache.bookkeeper.mledger.impl.ManagedCursorImpl - [public/default/persistent/my-topic] Initiate reset position to 1544:-1 on cursor __compaction
09:47:54.424 [BookKeeperClientWorker-OrderedExecutor-1-0] INFO  org.apache.bookkeeper.mledger.impl.ManagedCursorImpl - [public/default/persistent/my-topic] reset position to 1544:-1 before current read position 1544:0 on cursor __compaction
09:47:54.425 [BookKeeperClientWorker-OrderedExecutor-1-0] INFO  org.apache.pulsar.broker.service.ServerCnx - [/<IP_ADDRESS>:60180] [persistent://public/default/my-topic][__compaction] Reset subscription to message id -1:-1
09:47:54.426 [pulsar-client-io-85-1] INFO  org.apache.pulsar.client.impl.ConsumerImpl - [persistent://public/default/my-topic][__compaction] Successfully reset subscription to message id -1:-1:-1
09:47:54.516 [pulsar-timer-87-1] INFO  org.apache.pulsar.client.impl.ConnectionHandler - [persistent://public/default/my-topic] [__compaction] Reconnecting after timeout
09:47:54.517 [pulsar-client-io-85-1] INFO  org.apache.pulsar.client.impl.ConsumerImpl - [persistent://public/default/my-topic][__compaction] Subscribing to topic on cnx [id: 0xeb973738, L:/<IP_ADDRESS>:60180 - R:localhost/<IP_ADDRESS>:6650]
09:47:54.517 [pulsar-io-50-7] INFO  org.apache.pulsar.broker.service.ServerCnx - [/<IP_ADDRESS>:60180] Subscribing on topic persistent://public/default/my-topic / __compaction
09:47:54.518 [pulsar-io-50-7] INFO  org.apache.bookkeeper.mledger.impl.ManagedCursorImpl - [public/default/persistent/my-topic-__compaction] Rewind from 1544:-1 to 1544:0
09:47:54.518 [pulsar-io-50-7] INFO  org.apache.pulsar.broker.service.persistent.PersistentTopic - [persistent://public/default/my-topic] There are no replicated subscriptions on the topic
09:47:54.518 [pulsar-io-50-7] INFO  org.apache.pulsar.broker.service.persistent.PersistentTopic - [persistent://public/default/my-topic][__compaction] Created new subscription for 1
09:47:54.518 [pulsar-io-50-7] INFO  org.apache.pulsar.broker.service.ServerCnx - [/<IP_ADDRESS>:60180] Created subscription on topic persistent://public/default/my-topic / __compaction
09:47:54.518 [pulsar-client-io-85-1] INFO  org.apache.pulsar.client.impl.ConsumerImpl - [persistent://public/default/my-topic][__compaction] Subscribed to topic on localhost/<IP_ADDRESS>:6650 -- consumer: 1
09:47:54.526 [pulsar-external-listener-86-1] WARN  org.apache.pulsar.client.impl.ConsumerImpl - [persistent://public/default/my-topic] [__compaction] Could not get connection while getLastMessageId -- Will try again in 100 ms
09:47:54.527 [pulsar-external-listener-86-1] INFO  org.apache.pulsar.client.impl.ConsumerImpl - [persistent://public/default/my-topic][__compaction] Get topic last message Id
09:47:54.527 [pulsar-client-io-85-1] INFO  org.apache.pulsar.client.impl.ConsumerImpl - [persistent://public/default/my-topic][__compaction] Successfully getLastMessageId 1544:37344
09:47:54.527 [pulsar-client-io-85-1] INFO  org.apache.pulsar.compaction.TwoPhaseCompactor - Commencing phase one of compaction for persistent://public/default/my-topic, reading to 1544:37344:-1
09:47:57.416 [pulsar-web-57-12] INFO  org.eclipse.jetty.server.RequestLog - <IP_ADDRESS> - - [31/1/2020:09:47:57 +0900] "GET /admin/v2/persistent/public/functions/coordinate/stats HTTP/1.1" 200 887 "-" "Pulsar-Java-v2.5.0" 5
09:48:02.838 [pulsar-web-57-1] INFO  org.eclipse.jetty.server.RequestLog - <IP_ADDRESS> - - [31/1/2020:09:48:02 +0900] "GET /admin/v2/persistent/public/default/my-topic/compaction HTTP/1.1" 200 35 "-" "Pulsar-Java-v2.5.0" 2
09:48:04.646 [pulsar-io-50-7] INFO  org.apache.pulsar.broker.service.ServerCnx - [/<IP_ADDRESS>:60180] Closing consumer: 1
09:48:04.646 [pulsar-io-50-7] INFO  org.apache.pulsar.broker.service.AbstractDispatcherSingleActiveConsumer - Removing consumer Consumer{subscription=CompactorSubscription{topic=persistent://public/default/my-topic, name=__compaction}, consumerId=1, consumerName=b411f, address=/<IP_ADDRESS>:60180}
09:48:04.646 [pulsar-io-50-7] INFO  org.apache.pulsar.broker.service.ServerCnx - [/<IP_ADDRESS>:60180] Closed consumer Consumer{subscription=CompactorSubscription{topic=persistent://public/default/my-topic, name=__compaction}, consumerId=1, consumerName=b411f, address=/<IP_ADDRESS>:60180}
09:48:04.646 [pulsar-client-io-85-1] INFO  org.apache.pulsar.client.impl.ConsumerImpl - [persistent://public/default/my-topic] [__compaction] Closed consumer
09:48:05.941 [pulsar-web-57-3] INFO  org.eclipse.jetty.server.RequestLog - <IP_ADDRESS> - - [31/1/2020:09:48:05 +0900] "GET /admin/v2/persistent/public/default/my-topic/compaction HTTP/1.1" 200 79 "-" "Pulsar-Java-v2.5.0" 2
09:48:27.417 [pulsar-web-57-6] INFO  org.eclipse.jetty.server.RequestLog - <IP_ADDRESS> - - [31/1/2020:09:48:27 +0900] "GET /admin/v2/persistent/public/functions/coordinate/stats HTTP/1.1" 200 887 "-" "Pulsar-Java-v2.5.0" 5

The error can be reproduced if the topic is empty, as described by @KannarFr , as it will try to read the next message which is not there, and a scheduled timeout will kick in after ten seconds and complete the compaction with a TimeoutException:
https://github.com/apache/pulsar/blob/13d8ecd20a3c6795405fbf5946c1907e9c90dd91/pulsar-broker/src/main/java/org/apache/pulsar/compaction/TwoPhaseCompactor.java#L106
And even if a message is produced to the empty topic after the compaction started, it will still fail with a TimeoutException as the condition to end the async loop will not be true because lastMessageId will be MessageId.earliest and always less than the id of the message just received in this case:
https://github.com/apache/pulsar/blob/13d8ecd20a3c6795405fbf5946c1907e9c90dd91/pulsar-broker/src/main/java/org/apache/pulsar/compaction/TwoPhaseCompactor.java#L139
However, I have not reproduced the error when compacting a non-empty topic yet, follow the steps as @EugenDueck described. Does it reproduce the problem every time? And what if you trigger the compaction again after you see the timeout error?  Could you provide a complete code snippet of exactly how you produced the messages? Thanks! @EugenDueck
Also, the problem describled by @pienio7 seems to be a different one due to the empty value of the message, and I'll dig into it further.

@fantapsody
I've now stripped this down to a self-contained example and have updated the instructions to reproduce the problem. I can reliably replicate it using above snippet.
I've checked that the contents of neither key nor value are a problem, by using a single, hard-coded value for each. So when sending thousands of identical messages with the same key and value at low throughput (500 msg/sec or less), the topic can be compacted. But when sending thousands of identical messages at high throughput (I could replicate at more than 900 msg/sec), the topic becomes uncompactable.
One more observations while playing with this: Even when not a single message has been produced to a topic, the same timeout error occurs. This is not that big a deal, but I guess should be fixed.

Thanks for all the details @EugenDueck provided, I can reliably reproduce the problem as described now. I found a few problems for log compaction here:

Fail for empty topics. This can be fixed by an additional check before compaction.
Unable to finish the compaction if the value of the last message is empty when the compaction is triggered. This can be fixed by adding the missing check in the phase 2 of the compaction. Please notice that a message with an empty value indicates a deletion @pienio7
Fail for massive writes. This is a bit complicated, as the raw reader for compaction stops reading new messages after the first batch (100 entries) of raw messages(which may contain many more sub-messages), and increasing the timeout duration doesn't help. It seems there is something wrong with flow control here, and I have to do more test and analysis.
https://github.com/apache/pulsar/blob/13d8ecd20a3c6795405fbf5946c1907e9c90dd91/pulsar-broker/src/main/java/org/apache/pulsar/compaction/TwoPhaseCompactor.java#L106

For problem 3, I got the stats of the topic when the compaction stucks, and some metrics like availablePermits, lastAckedTimestamp seems to be anomalous here.  Maybe someone could provide some insight or advice based on these metrics and the problems described? @ivankelly @sijie
{
  "msgRateIn" :<PHONE_NUMBER>883373,
  "msgThroughputIn" : 7316116.5272862585,
  "msgRateOut" : 0.0,
  "msgThroughputOut" : 0.0,
  "averageMsgSize" : 23.497099420201696,
  "storageSize" : 238074383,
  "backlogSize" : 238074383,
  "publishers" : [ {
    "msgRateIn" :<PHONE_NUMBER>883373,
    "msgThroughputIn" : 7316116.5272862585,
    "averageMsgSize" : 23.0,
    "producerId" : 0,
    "metadata" : { },
    "address" : "/<IP_ADDRESS>:57444",
    "producerName" : "standalone-1-3",
    "connectedSince" : "2020-02-05T22:29:10.529+08:00",
    "clientVersion" : "2.5.0"
  } ],
  "subscriptions" : {
    "__compaction" : {
      "msgRateOut" : 0.0,
      "msgThroughputOut" : 0.0,
      "msgRateRedeliver" : 0.0,
      "msgBacklog" : 11181,
      "blockedSubscriptionOnUnackedMsgs" : false,
      "msgDelayed" : 0,
      "unackedMessages" : 40122,
      "type" : "Exclusive",
      "activeConsumerName" : "3e96d",
      "msgRateExpired" : 0.0,
      "lastExpireTimestamp" : 0,
      "lastConsumedFlowTimestamp" :<PHONE_NUMBER>315,
      "lastConsumedTimestamp" :<PHONE_NUMBER>386,
      "lastAckedTimestamp" : 0,
      "consumers" : [ {
        "msgRateOut" : 0.0,
        "msgThroughputOut" : 0.0,
        "msgRateRedeliver" : 0.0,
        "consumerName" : "3e96d",
        "availablePermits" : -39122,
        "unackedMessages" : 40122,
        "blockedConsumerOnUnackedMsgs" : false,
        "lastAckedTimestamp" : 0,
        "lastConsumedTimestamp" :<PHONE_NUMBER>386,
        "metadata" : { },
        "address" : "/<IP_ADDRESS>:56969",
        "connectedSince" : "2020-02-05T22:29:32.314+08:00",
        "clientVersion" : "2.6.0-SNAPSHOT"
      } ],
      "isReplicated" : false
    }
  },
  "replication" : { },
  "deduplicationStatus" : "Disabled",
  "bytesInCounter" : 238088065,
  "msgInCounter" : 9985337
}

Another point, the log compaction is run on event, like foreach 100MB of data, right? Can it be on the fly, like foreach message received on the topic, it's added to the compacted topic too?

@KannarFr : Currently it doesn't support compacting events on the fly. Its is an interesting feature request to consider although it is probably for a separate discussion for the features and improvements that we need to add to compaction.

@sijie You are right, I will write an issue.

(loop @codelipenghui in this thread since he has good experiences on dealing with permits related issues on production)
@fantapsody
regarding the 3) question you posted in https://github.com/apache/pulsar/issues/6173#issuecomment-582480075, it does look like a flow-control problem.
I think the main problem is here: https://github.com/apache/pulsar/blob/402ecec9c3731711fd1bf700c30d09678aa9b1e5/pulsar-broker/src/main/java/org/apache/pulsar/client/impl/RawReaderImpl.java#L144
The "permits" in Pulsar's flow control logic is designed for "messages" not "batches". That says one permit per message. So broker decrementing the permits after it dispatched messages. It is okay to see negative "permits" at the broker side because it means that broker dispatched more messages that the consumer requested.
However, it is NOT okay if the consumer counts the permits by batches. For example, a consumer/reader requests 100 permits (the receiver queue size), the broker dispatches 1000 "messages" (10 message per batch). Hence the "permits" at broker side will become -900. Since RawReaderImpl treats one permit per batch, it will request 50 permits again from the broker once the available permits at reader side reach the threshold (which is half of the receiver queue size).
The broker receives 50 permits and increases its available permits back to -850. But it is still negative. So broker will not read any more messages and cause raw reader to be stuck at waiting for new messages.
The problem of 3) can be addressed by fixing https://github.com/apache/pulsar/blob/402ecec9c3731711fd1bf700c30d09678aa9b1e5/pulsar-broker/src/main/java/org/apache/pulsar/client/impl/RawReaderImpl.java#L144 to use number of messages as permits.

Besides this problem, @codelipenghui has seen a lot of "consumer stuck" problems related to permits. We have been discussing how to introduce a self-healing mechanism into current flow-control logic to avoid similar mistakes in client implementation. I think we should discuss a more generic approach to improve flow-control after fixing this issue here.


Warning running on OS X Yosemite DP 5

After installing lsusb I get the following error when running it:
2014-08-11 09:36:25.561 system_profiler[9043:960982] Error loading /System/Library/SystemProfiler/SPNVMeReporter.spreporter: Error Domain=NSCocoaErrorDomain Code=4 "The bundle “SPNVMeReporter.spreporter” couldn’t be loaded because its executable couldn’t be located." UserInfo=0x7fc052500290 {NSLocalizedFailureReason=The bundle’s executable couldn’t be located., NSLocalizedRecoverySuggestion=Try reinstalling the bundle., NSBundlePath=/System/Library/SystemProfiler/SPNVMeReporter.spreporter, NSLocalizedDescription=The bundle “SPNVMeReporter.spreporter” couldn’t be loaded because its executable couldn’t be located.}
2014-08-11 09:36:25.759 system_profiler[9042:960956] Error loading /System/Library/SystemProfiler/SPNVMeReporter.spreporter: Error Domain=NSCocoaErrorDomain Code=4 "The bundle “SPNVMeReporter.spreporter” couldn’t be loaded because its executable couldn’t be located." UserInfo=0x7fd950f03610 {NSLocalizedFailureReason=The bundle’s executable couldn’t be located., NSLocalizedRecoverySuggestion=Try reinstalling the bundle., NSBundlePath=/System/Library/SystemProfiler/SPNVMeReporter.spreporter, NSLocalizedDescription=The bundle “SPNVMeReporter.spreporter” couldn’t be loaded because its executable couldn’t be located.}

... after which, the attached USB devices are listed.

I'm closing this since it works as expected in the official release


ASIHTTPRequest warnings

Can we somehow fix the warnings (format strings, etc.) in ASIHTTPRequest. Maybe with a private pod?

@JanX2 Replacing ASIHTTP with AFNetworking or AlamoFire (bridged to Objective-C) is high on our list of priorities. I'm having trouble with getting the asynchronous networking working reliably though. Do you have experience with that?

Our calls to ASIHTTPRequest are in RefreshManager and GoogleReader. Could we directly use NSURLSession, rather than introduce a third party class ?

I think so but we would still have to change our logic to asynchronous as all of the synchronous stuff has been deprecated.

This duplicates #344


Claim 7:  The non-cellular (extracellular) space accounts for 6% of the total volume, less than half the extracellular space estimates from living brains.

Do Something:

[ ] Download mask volume and count voxels
[ ] Count volume of all objects in mask slices
[ ] Compute percentage of extracellular space

Potential Improvements:

[ ] Restrict to cylinders with extracellular space (automatic cylinder may not have any!)

this is done?

obe - refactoring


Cmdlets for 1P Guest App Permissions

Edits to Get-SPOApplication to include guest app permissions to 1P SPE Apps
Addition of Set-SPOApplicationPermission to document adding guest app permissions

@jacktremper @maggierui Please help review and sign off on content.


UI component - Filter Chip variant

Functional Criteria

[ ] Be able to create a text only badge
[ ] Be able to create a badge with a leading icon
[ ] Be able to create badge with a delete icon

Visual Criteria

[ ] focus state
[ ] hover state
[ ] active state

Components

[ ] Web component and tests
[ ] React component and tests

Storybook

[ ] Text only
[ ] Leading icon
[ ] Delete icon and event handling

Resources
https://www.figma.com/file/3pb2IK8s2QUqWieH79KdN7/❖-Components-|-DIO-DS?node-id=911%3A5909
https://www.figma.com/file/wXpcgeeOarkCv5ItVRTVFZ/[XD-000]-Design-Tokens---Spacing?node-id=3%3A378

@ambroseyau will work with Chris on design issues in component design specs.

@chrisolsen
Spacing and text size + line height specs is here:
https://www.figma.com/file/wXpcgeeOarkCv5ItVRTVFZ/[XD-000]-Design-Tokens---Spacing?node-id=3%3A378

After looking at the design specs there are some issues that need to be ironed out before this task can be completed.

The badge has a focus state, like a button, but yet only the X icon is clickable
The delete icon is clickable, but does not have any hover or active states.

There are other issues that were discussed with @ambroseyau that also need to be looked at.

05/02/2022

had a discussion with Chris to identify all the problems

05/03/2022

Went through the design with Mark and Tom
Investigate chip design

background color is not WCAG AA compliant

05/04/2022

New design is complete and reviewed.
https://www.figma.com/file/3pb2IK8s2QUqWieH79KdN7/❖-Components-|-DDI-DS?node-id=911%3A5909

@chrisolsen please let me know if you have any questions

@ambroseyau I have questions :)

Had a discussion with Chris to clear all the questions (hover states, keyboard interactions, and accessibility) and provided additional specs (spacing, icon size tokens)
https://www.figma.com/file/3pb2IK8s2QUqWieH79KdN7/❖-Components-|-DDI-DS?node-id=911%3A5909

Hey @chrisolsen Looks like there might have been some confusion on which chip design is the source of truth. I have clarified in my design review comments above.

:tada: This issue has been resolved in version 4.0.0-alpha.5 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 4.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Item: Calcined Ash

The impure result you get from items calcined in the Calciner

Textures merged in https://github.com/pahimar/Equivalent-Exchange/pull/54


[SPARK-5904][SQL] DataFrame API fixes.

Column is no longer a DataFrame to simplify class hierarchy.
Don't use varargs on abstract methods (see Scala compiler bug SI-9013).

Still need to add Java API tests.

Test build #27704 has started for   PR 4686 at commit 5221530.

This patch merges cleanly.

Test build #27704 has finished for   PR 4686 at commit 5221530.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27704/
Test FAILed.

Test build #27708 has started for   PR 4686 at commit df25cef.

This patch merges cleanly.

Test build #27708 has finished for   PR 4686 at commit df25cef.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27708/
Test FAILed.

Test build #27714 has started for   PR 4686 at commit fd9b199.

This patch merges cleanly.

Test build #27714 has finished for   PR 4686 at commit fd9b199.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27714/
Test PASSed.

LGTM

Btw, this fixed my issue with Java examples using DataFrame


chore: move from prom-client to OpenTelemetry metrics

Description
Adopts OpenTelemetry metrics for metrics and move away from using prom-client.
This change adds these new Backstage specific metrics:

catalog.stitching.duration
catalog.stitching.queue.length
catalog.stitching.queue.length
backend_tasks.task.runs.count
backend_tasks.task.runs.duration

However, OpenTelemetry doesn’t collect Node.js specific metrics like nodejs_eventloop_lag_seconds.

Note: Node.js specific metrics are WIP in this PR
Edit: this has been merged and released, these missing metrics are now included with OpenTelemetry

The application metrics will be exposed at localhost:9464/metrics

See sample log file: application.log

The host/process metrics will be exposed at localhost:9463/metrics

See sample log file: host.log

Which issue(s) does this PR fix

Fixes RHIDP-4404

Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

TODO: need to update monitoring-and-logging.md, will go through this set up & update the docs


Website 【Talking Head(?) Anime from a Single Image 3】 Page not found

When I open Talking Head(?) Anime from a Single Image 3: Now the Body Too, shows a page not found.

I have to take down the web pages because of copyright issues: https://twitter.com/dragonmeteor/status/1550712533342101504. I'm not actively working to bring them back at the moment due to lack of motivation and time.

I understand.
I hope someday the article can come back, I like your job and want to study the technology behind it.

An archived version of the webpage appears to be at https://web.archive.org/web/20220606073558/https://pkhungurn.github.io/talking-head-anime-3/index.html

Got it.
Thank you so much.


Release 0.0.15

Arrays have description

Pull Request Test Coverage Report for Build 476496121

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

Totals

Change from base Build 476213419:
0.07%

Covered Lines:
243

Relevant Lines:
312

💛  - Coveralls

Pull Request Test Coverage Report for Build 476496121

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

Totals

Change from base Build 476213419:
0.07%

Covered Lines:
243

Relevant Lines:
312

💛  - Coveralls


fix(1153) Indicate which fields are required

fixes https://github.com/KaotoIO/kaoto-ui/issues/1153
The required fields are highlighted with (*)

yes, the validation will be I think next step. I know the pushing of the deploy button could be a place where we could validate the filled forms, but with current flow I'm not sure how this would work.
This is probably an area for the UX team :)


[Service] Error 500 when an integer type is sent as offset parameter

Error 500 when I am trying to list the services (/services) with an integer type on the offset param.
$ http GET :8001/services?offset=2

Kong version: 0.13.x
Log message:
2018/04/05 15:00:08 [error] 112#0: *153753 lua coroutine: runtime error: /usr/local/share/lua/5.1/kong/api/endpoints.lua:52: offset must be a string
stack traceback:
coroutine 0:
	[C]: in function 'error'
	/usr/local/share/lua/5.1/kong/db/dao/init.lua:214: in function 'page'
	/usr/local/share/lua/5.1/kong/api/endpoints.lua:52: in function </usr/local/share/lua/5.1/kong/api/endpoints.lua:51>
coroutine 1:
	[C]: in function 'resume'
	/usr/local/share/lua/5.1/lapis/application.lua:393: in function 'handler'
	/usr/local/share/lua/5.1/lapis/application.lua:130: in function 'resolve'
	/usr/local/share/lua/5.1/lapis/application.lua:161: in function </usr/local/share/lua/5.1/lapis/application.lua:159>
	[C]: in function 'xpcall'
	/usr/local/share/lua/5.1/lapis/application.lua:159: in function 'dispatch'
	/usr/local/share/lua/5.1/lapis/nginx.lua:215: in function 'serve_admin_api'
	content_by_lua(nginx-kong.conf:148):2: in function <content_by_lua(nginx-kong.conf:148):1>, client: <IP_ADDRESS>, server: kong_admin, request: "GET /services?offset=2 HTTP/1.1", host: "localhost:8001"
2018/04/05 15:00:08 [error] 112#0: *153753 [lua] init.lua:133: handle_error(): /usr/local/share/lua/5.1/lapis/application.lua:397: /usr/local/share/lua/5.1/kong/api/endpoints.lua:52: offset must be a string
stack traceback:
	[C]: in function 'error'
	/usr/local/share/lua/5.1/kong/db/dao/init.lua:214: in function 'page'
	/usr/local/share/lua/5.1/kong/api/endpoints.lua:52: in function </usr/local/share/lua/5.1/kong/api/endpoints.lua:51>

stack traceback:
	[C]: in function 'error'
	/usr/local/share/lua/5.1/lapis/application.lua:397: in function 'handler'
	/usr/local/share/lua/5.1/lapis/application.lua:130: in function 'resolve'
	/usr/local/share/lua/5.1/lapis/application.lua:161: in function </usr/local/share/lua/5.1/lapis/application.lua:159>
	[C]: in function 'xpcall'
	/usr/local/share/lua/5.1/lapis/application.lua:159: in function 'dispatch'
	/usr/local/share/lua/5.1/lapis/nginx.lua:215: in function 'serve_admin_api'
	content_by_lua(nginx-kong.conf:148):2: in function <content_by_lua(nginx-kong.conf:148):1>, client: <IP_ADDRESS>, server: kong_admin, request: "GET /services?offset=2 HTTP/1.1", host: "localhost:8001"
<IP_ADDRESS> - - [05/Apr/2018:15:00:08 +0000] "GET /services?offset=2 HTTP/1.1" 500 54 "-" "HTTPie/0.9.9"

Offset is meant to be a Kong service uuid not numeric.

Hi @jeremyjpj0916
I am aware of it. I created the issue because the status 500 represents something went wrong in the application/server, and not on the client side (the request).
If you try to search for some invalid uuid, you gonna see the specification of the problem with the properly status.
There are a couple of issues created with similar problems.

Ah I see your point then, thought you were just unintentionally mixing up offset/size :) . Should be returning a HTTP 400 for a bad request in this case then  👍 .

Yes, we will add schema aware arguments parser at some point, to fix these wrongly (to number) type converted arguments. Until then we can consider it a bug. Fix is coming, it is just not yet on our table.

Soon: https://github.com/Kong/kong/tree/feat/admin-schema-aware-parser

Fixed with #3416 (will be included in 0.14).


 Perfect switching off the LEDs would also be in idle?

Simply write a shutdown script to set the colors to black before shutting down.
You should find enough tutorials for it on the web. It's not an Hyperion issue, imho.

change the stop section by the initscript under /etc/init.d/hyperion
stop)
        if [ -f $PIDFILE ]
        then
                printf "%s\n" "Set LEDs off"
                hyperion-remote -c black
                sleep 1
                printf "%-50s" "Stopping $NAME"
                    PID=`cat $PIDFILE`
                    cd $DAEMON_PATH
                if [ -f $PIDFILE ]; then
                    kill -HUP $PID
                    printf "%s\n" "Ok"
                    rm -f $PIDFILE
                else
                    printf "%s\n" "pidfile not found"
                fi
        else
                printf "%-50s\n" "No PID file $NAME not running?"
        fi
;;

Where is the like button?
+1 for it to make it a standard function!

I don´t know to create a pull request, but the initscript is located under github:
hyperion/bin/hyperion.init.sh

I will submit this, with the new install script, thanks

@Spaddel can be closed?

OSMC uses systemd and the new script will install this accordingly. If you push the shutdown button, the lights will be set to off. (But it is better so power off the leds (at least) over night).
Thank you


move-request-helper-to-utils

Codecov Report

Merging #240 into master will not change coverage.
The diff coverage is 90.9%.

@@           Coverage Diff           @@
##           master     #240   +/-   ##
=======================================
  Coverage   58.93%   58.93%
=======================================
  Files          40       40
  Lines        3244     3244
  Branches      390      390
=======================================
  Hits         1912     1912
  Misses       1226     1226
  Partials      106      106

Impacted Files
Coverage Δ

core/sessions.py
78.43% <ø> (-1.69%)
:arrow_down:

vmpool/clone.py
51.46% <ø> (-0.15%)
:arrow_down:

vmmaster/webdriver/helpers.py
62.11% <100%> (+0.23%)
:arrow_up:

vmmaster/webdriver/commands.py
68.3% <50%> (ø)
:arrow_up:

core/utils/network_utils.py
63.71% <94.44%> (+5.82%)
:arrow_up:

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 f026f70...8d1c851. Read the comment docs.

!test


Value of string literal type as property name.

For example:
type Method = 'get' | 'post';

let app = {
    get() { },
    post() { },
    foo() { }
    bar: 123
};

let method = 'get';
let handler = app[method];

See also #394.

@SaschaNaz Thanks for the reference. So basically now we may have memberof operator return a string literal type, that's interesting. Hopefully somebody would take that PR.

I know @weswigham has brought this up, though I'm not sure if there's an existing issue for it.

@vilic What does the snippet demonstrate?

@masaeedu Hmm, I think it's about finding a common type among several properties (within the range of string literals union).

This should be addressed by the new indexing support for literal types in https://github.com/Microsoft/TypeScript/pull/11929


interlock: supports only one path per host

Current docs show single context root and the option is singular, but we've had a few folks try for multiple contexts on the same hostname.  Added a call out to avoid this issue.

/cc @ehazlett

Deploy preview for modest-edison-79d82e ready!
Built with commit 6f0252bbf910ad7e0145ac33be4c1134ac35a97b
https://deploy-preview-7045--modest-edison-79d82e.netlify.com

Thanks.  Dropped **Note:**.  Added {: .important}.


it seems we might need an explicit brew update before brew install git-annex

see https://github.com/Homebrew/homebrew-core/issues/75243#issuecomment-820482307
not sure if we should unconditionally do brew update before any install to follow, or do it as a mediation step if initial brew install fails, or somehow detect if it is needed.  I will let you @jwodder to decide on that.  unconditional brew update if any install to follow sounds like the easiest.

@yarikoptic Please merge #51 first in order to avoid a future merge conflict.


Moves pid file operations to /pkg/

Fixes #10958 by moving /utils/daemon to /pkg/pidfile.
I also added some test cases, and the daemon now uses the new pidfile package.

can you add a closes #10958 so it will actually close the issue if its
merged, thanks
On Wed, Feb 25, 2015 at 11:26 AM, Rick Wieman<EMAIL_ADDRESS>wrote:

As requested in #10958 https://github.com/docker/docker/issues/10958,
this PR moves /utils/daemon to /pkg/pidfile.
I also added some test cases, and the daemon now uses the new pidfile
package.
You can view, comment on, or merge this pull request online at:
https://github.com/docker/docker/pull/11018
Commit Summary

Moved pidfile from utils to pkg

File Changes

M daemon/daemon.go
https://github.com/docker/docker/pull/11018/files#diff-0 (6)
A pkg/pidfile/pidfile.go
https://github.com/docker/docker/pull/11018/files#diff-1 (51)
A pkg/pidfile/pidfile_test.go
https://github.com/docker/docker/pull/11018/files#diff-2 (31)
D utils/daemon.go
https://github.com/docker/docker/pull/11018/files#diff-3 (36)

Patch Links:

https://github.com/docker/docker/pull/11018.patch
https://github.com/docker/docker/pull/11018.diff

—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/11018.

Like that, @jfrazelle?

perfect :)

should we also have a test to verify contents of the created pidfile as well?

@ahmetalpbalkan I just pushed the changes :smiley:

Alright, I replaced FindProcess with another way of checking whether the process is still running. This works at least on Linux and Mac OS.

@RickWieman -1 for sending a kill signal. in such cases we have multiple files e.g.
proc_windows.go
// +build windows

func isProcessRunning(pid int) bool {
    windows specific implementation
}

proc_linux:go
// +build linux

func isProcessRunning(pid int) bool {
    linux-specific implementation with /proc/pid
}

do not bother creating OS X implementation at this point, we probably are not going to have docker daemon for OS X. Or if you do this way OS X can also be easily implemented in a separate file.
And from your code you will just call isProcessRunning.
Also your test file probably should start a process (e.g. sleep 1000) and test should create a new pidfile for that with specified pid and then you should use New to create the new pidfile. Otherwise you're not really testing all the functionality and as you said tests can just provide false positives like that.
If you don't want to do all these, feel free to proceed with these, I can send a PR afterwards.

There are various sources on the internet saying you can use such a kill signal to check the status of a pid, that's why I implemented it in the first place (just maximizing compatibility).
I didn't know about OS specific build files yet, so thanks for mentioning that. I don't know when I will be able to add all this stuff though.

@RickWieman Feel free to revert to an earlier commit where you use /proc and I can totally take from there. (don't forget to squash your commits into one) Don't let my comments getting this PR from merged, it started as a nice refactoring after all.  :smile:
We should be providing clear platform-dependent implementations rather than inline hacks trying to cover all cases and that's probably a separate PR.

You're right. I just reverted to the initial commit, but I kept the filepath and ioutils parts. I also used filepath.Join for the /proc implementation.

Removing it (as it was a full duplicate) sounded like the best idea :+1:

LGTM

Fixed it :+1:

@crosbymichael is this what you had in mind?

LGTM ping @crosbymichael

Just rebased on master :wink:

@RickWieman thanks!
LGTM


Bug in PageHeader.SecondaryCommands list

Description:
Exception is thrown whenever I try to scroll through the secondary commands' list of the page header.
The exception message is "Layout cycle detected. Layout could not complete."
Steps to reproduce:

Create a new app using a hamburger or minimal template
Build and launch the app
Click a button with three dots on it in the top right corner of the app
Hover any item in the list with your mouse pointer and try to scroll in any direction

Result:

Version? Do you have any Ellipsis Behaviors in markup?

Plugin version: 1.18
Package version (in the solution): 1.1.10 or 1.1.11
I didn't touch XAML after the application creation.

so no changes bare bones, hamburger application?

That's right.

Pc or 2in1, the template will determine screen size and reacts accordingly

PC

Could you please review this known issue? Then report.
https://github.com/Windows-XAML/Template10/issues/857

Yes, I've seen it but my case is a bit different: when the app is created, it already has T10 v. 1.1.10 (I can update it to 1.1.11 but it won't change anything) and I  also don't change anything in the app.

New question. Are you using a <Hub /> control?

Hi @JerryNixon ! Hub is not used. Nothing is added to the code the template already has, so it is just a clean template.
Again, here are the steps that I do to reproduce this issue on my machine:

I start my Visual Studio Community 2015 (fresh, no modifications applied to it);
I click Create Project link on the start page in Visual Studio;
I select the Hamburger (Template 10) template and click OK;
I click F5 to run this newly created project;
When the application starts, I click the ellipsis button in the top right corner of the app, align the mouse pointer over any element of the menu that has just opened and turn the scroll wheel up or down couple of times.
That's it. After this I get the exception.

Thanks for reporting this. Well, there's nothing I can do to repro this. Let me ask you another question. Are you using any scale factor? Does your machine read 100%?

It's 125%

Yes, whent the scaling is set to 100% it works just fine. No scroll bar is visible when I try to scroll.

Oh cool, People app is failing too. And the Windows Feedback app does not. It seems when the scaling factor is set to more than 100%, any app will fail if it has more than one secondary command in the command bar.

Good. We figured it out. Let me reach out to the control lead.

The team says this is fixed in RS1. Let's hope so.


build: detect-secrets false positives

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


attempted fix for auth lockup

@elvece try this

@dr-bonez I remember testing this at the time, but I forget what the original issue was, do you recall?


Arrow not positioned correctly until a scroll happens

Reproduction demo
% is the arrow in this example. It is at the left of the element at initialisation when it should be centered.
Only when a scroll starts does it fix the issue.
https://codesandbox.io/s/react-popper-v2x-issue-template-forked-2fj5t?file=/src/index.js
Codesandebox sometimes does something with its layout that triggers the repositioning of the arrow so hit the refresh of the browser view or go to it directly https://2fj5t.csb.app/

Steps to reproduce the problem

Create a react-popper top or bottom with arrow
Arrow not centered

What is the expected behavior?
Arrow should be centered immediately
What went wrong?
Any other comments?
If I bypass react-popper and use popperjs directly inside a useLayoutEffect it works as expected
Packages versions

Popper.js: 2.5.4
react-popper: 2.2.4

The issue is with the custom update modifier:
  updateStateModifier,
  { name: 'applyStyles', enabled: false },

updateStateModifier is not receiving the correct initial arrow styling so it's possible the issue is in poperjs/core.
The old components Manager, Target, etc... does not create a custom updateStateModifier so it does not have the issue the hook has

@skimi @FezVrasta I still see this as an issue after updating to 2.5.5.
The initialization of the arrow is now near the target element, but it's slightly to the right of the element. Also, where the content of the popper spawns isn't working.
In the below examples, I have my placement set to bottom-end and my arrow element has a padding of 12. I have a modifier in place to set the offset to [ 12, 12 ]
Here is the initial arrow position:

And here it is if scroll the page (the desired position):

How did you fix this @skimi ?


Upgrade Spring Boot to the Latest Version

Update the guide to use the most recent Spring Boot version.
Files that require changes are:
initial/build.gradle
initial/pom.xml
complete/build.gradle
complete/pom.xml

wanted to submit a PR

Closing this issue as dependabot is now configured in this project for ongoing version updates.


Shader loading fails when using a jar version of the library

I get an exception in Program.kt around line 294:
val lines = File(url.toURI()).readLines()
Looking at the url it looks like:
jar:file:/C:/Users/<myusername>/.m2/repository/com/github/kotlin-graphics/imgui/-v1.51-gfcc47f9-11/imgui--v1.51-gfcc47f9-11.jar!/shader.vert
It might be an issue exposed when grabbing imgui as a jar file from maven rather than referring directly to the code. It seems Files can't be created from a path pointing into a jar.
Using url.openStream() seem to solve this in my local tests (I don't have an environment allowing me to rebuild the imgui sources at this point so I have not tried it on the imgui code)

Disregard. This error happened in uno-sdk

Hi,
I'm interested in pursuing the issue also if this regards uno-sdk..
I'm used to work using dependencies managed by Gradle.. so I don't have the possibility normally to test other setups and this would be a perfect chance to do that
Let me know if you still have problems, we cant work on that :)

It's certainly still a problem. I made a cowboy fix looking like this after
getting the source code for uno:
val url = ClassLoader.getSystemResource(path)
var str = url.openStream().bufferedReader()
//val lines = File(url.toURI()).readLines()
val lines = str.readLines()
Let me know if it is something you want me to clean up and make a pull
request out of (it needs to be changed in both functions called
createShader).
I have not gone deeper down this rabbit hole since I ran into some font
loading issue with imgui that blocked me from showing anything meaningful
so there might be other places where the same issue happens. Also, I have
not tried it on a source code build so I don't know if this works when
referring the source directly rather than the jar.
David
On Sun, Sep 10, 2017 at 11:55 PM, Giuseppe Barbieri <
<EMAIL_ADDRESS>wrote:

Hi,
I'm interested in pursuing the issue also if this regards uno-sdk..
I'm used to work using dependencies managed by Gradle.. so I don't have
the possibility normally to test other setups and this would be a perfect
chance to do that
Let me know if you still have problems, we cant work on that :)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/kotlin-graphics/imgui/issues/2#issuecomment-328434352,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABswGzgqtwafCH2sE3GFKZ2W6h8UvsXQks5shNlcgaJpZM4PSlWa
.

One late addition, I solved the problem you had, I had to use getSystemResourceAsStream


Proposal to Add an example

I found using the library very difficult given the lack of examples. I created a simple project detailing how to use the library in a real way, that is, a practical way that it actually gets used in a setting I had to implement it in.
I wanted to propose that this example be added to the examples. Here is the link to my repository:
https://github.com/granitDev/rPGP-Working-Example

very cool, would you like to make a PR?


18+ HOT Sean Diddy MMS VIRAL LEA𝚔ED VIDEO TELEGRAM LINKS XNXX - Hot Se𝚡 po𝚛n Videos XXX

Sean 'Diddy' Combs' lawyers say feds leaked video of Cassie assault, seek to exclude evidence
Lawyers for Sean “Diddy” Combs on Wednesday accused the government of waging a campaign to leak evidence — including the 2016 video that...
P Diddy's Lawyers Accuse DOJ of Pre Trial Cassie Video Leak.
Sean 'Diddy' Combs' Lawyers Have Accused the Government of Leaking a 2016 Cassie Video to 'Savage' His Reputation Before Trial.
Did US govt leak video of Sean ‘Diddy’ Combs assaulting Cassie Ventura? Lawyers for the music mogul say… | Today News
Sean 'Diddy' Combs' lawyers claim the U.S. government leaked damaging information, including a 2016 assault video, which harmed his chances...
Diddy court case: Sean Combs lawyers accuse government of video leak
Lawyers for Sean “Diddy” Combs, who is facing sex-trafficking charges at the federal level, have accused the government of leaking a video...
.34 minutes ago
Attorneys for Sean ‘Diddy’ Combs allege government leaked evidence to the media, impeding ‘fair trial’
Attorneys for Sean “Diddy” Combs filed a motion late Wednesday accusing the government of leaking a video of the hip-hop mogul beating his...o
Diddy's Legal Team Implies Feds Leaked Cassie Beating Video
Shocking new claim from Diddy and his legal team ... they believe the federal government "orchestrated" the leak of the Cassie beating video...
Diddy Ripped For ‘False’ Claim That Feds Leaked Cassie Ventura Assault Video
Federal prosecutors tore into Sean “Diddy” Combs' for his “false and misleading” accusations that the government leaked the shocking footage...
Prosecutors deny leaking hotel video of Sean 'Diddy' Combs attacking Cassie Ventura
Prosecutors denied leaking the hotel security camera video that showed Sean "Diddy" Combs attacking his then-girlfriend Casandra Ventura at...
Sean 'Diddy' Combs' Lawyers Accuse Government of Leaking 2016 Cassie Video to 'Savage' Reputation Before Trial, Feds Deny Claim
Sean 'Diddy' Combs' attorneys are alleging that the government purposefully leaked 2016 footage of Combs physically assaulting his...

hgjhkj


Input shaper orientations

"Klipper does not to my knowledge currently support input shaper tuning in other orientations"
FYI, this can be done by using axes_map under [adxl345]. See here.

The axes_map does allow you to use an adxl345 in other orientations, but only ones that align its' axes with those of movement.
This doesn't let you mount the board to a sailfin or orbiter motor like with a sherpa mini.
Here is the blurb explaining axes_map from the kilpper docs:
#axes_map: x, y, z
The accelerometer axis for each of the printer's X, Y, and Z axes.
This may be useful if the accelerometer is mounted in an
orientation that does not match the printer orientation. For
example, one could set this to "y, x, z" to swap the X and Y axes.
It is also possible to negate an axis if the accelerometer
direction is reversed (eg, "x, z, -y"). The default is "x, y, z".

axes_map would not help. Nonetheless the orientation does not matter for input shaper as it does not look at a specific axis from the sensor but takes the input from all 3 axis. So no need to align your axis for IS to work properly.
https://github.com/Klipper3d/klipper/pull/5979#issuecomment-1379486909


Fix StringName Inequality Operators

Fixes #36511
This one got me for a good 4 hours!

It fixes the script error but there is something weird going on, I gave it a try, applying over 1e255532e90fa35148167e4c153ad39f77010a04
try running this script
tool
extends EditorScript

func test(opts):
	#var should_be_true = opts.should_be_true # with this:  # Invalid get index 'should_be_true' (on base: 'Dictionary').
	var should_be_true = opts.get("should_be_true", false)
	print("should_be_true: ", should_be_true)

func _run():
	test({ should_be_true = true })  # outputs: should_be_true: False
	test({ "should_be_true": true }) # outputs: should_be_true: True

so the first dictionary literal syntax doesn't define the properties

I've tested this to check if this problem is solved too: https://github.com/godotengine/godot/issues/36900 but it's not.
Any idea what could cause it?

I believe this is now obsolete given the issue was fixed in another way and this code was refactored. Thanks for the contribution anyway!


Check if pip is outdated in provision.py.

Since the automatic update in provision.py doesn't work for
some edge cases, we need to manually check if the pip version
used is recent enough for provisioning (see discussion in #3237
for further details).

Automated message from Dropbox CLA bot
@derAnfaenger, it looks like you've already signed the Dropbox CLA. Thanks!

This looks good to me, pending Travis.

I think that this will fail; the problem is the pip in apt is much older, and we only install pip 9 when we construct the virtualenv inside setup_venvs.py.

Given the construction, I don't see how it should be possible for Zulip to end up using an old version of pip for actually installing packages other than pip itself.

@timabbott is it possible that it is installing the pip package with the wrong command, so that the new version gets installed


Add override capability for kafka topic per log call

TODO Refactor test code to check topics

closes #12


Default to prepare_v2 if prepare_v3 not found

So it seems that travis is still stuck on Ubuntu 16.04 causing the library to fail (hence all the sqflite tests - I added an issue here https://travis-ci.community/t/change-ubuntu-distribution-from-16-04/9320 but I'm not sure when/if it will get fixed).
Here I propose to try to load prepare_v3 and load prepare_v2 if not found.
I also added travis tests (see the build history https://travis-ci.org/github/tekartikdev/sqlite3.dart/builds to see that it was failing before - I made 2 attemps to fix the issue). You might want to change the implementation.
Thanks!

Thanks for the PR! Unfortunately the .ffi.dart file is auto-generated, so your changes would be overridden. With the linked commit, we'll just look those functions up outside of that file. I also made the CI run on Ubuntu 16.04.

Ah ok sorry for missing that. I guess I just throw something to see that this was indeed the only remaining issue (as I could not test it locally). Great indeed that you test on ubuntu 16.04. Thanks! Looking forward for a new published version!

Just published version 0.1.4 which should contain this logic.

Hooray! : https://travis-ci.org/github/tekartik/sqflite/builds/708061710
I published sqflite_common_ffi 1.1.0 based on your lib. Thanks again for your work. Bravo!


Enhancements to apply-changes to make a more robust selective deploy

apply-changes will reattach to an already running apply changes on a given director if run more than once(to prevent failure if there's an accidental disconnect from om).
When using a selective deploy (--product-name), this will always still reattach to a running apply changes, and return failure or success. However, a selective deploy will not differentiate if the already running apply changes is for the selective deploy given or if it is just some other apply changes.
The Ops Manager API returns the following when hitting the /api/v0/installations endpoint:
{
  installations: [
    {
      user_name<EMAIL_ADDRESS>      finished_at: null,
      started_at: "2019-02-20T16:56:41.829Z",
      status: "running",
      additions: [ ],
      deletions: [ ],
      updates: [ ],
      unchanged: [
        {
          identifier: "cf",
          label: "Pivotal Application Service",
          product_version: "2.4.3",
          deployment_status: "pending",
          change_type: "unchanged",
          guid: "cf-randomsha"
        },
        {
          identifier: "p-bosh",
          label: "BOSH Director",
          product_version: "2.4-build.152",
          deployment_status: "running",
          change_type: "unchanged",
          guid: "p-bosh-randomsha"
        }
      ],
      id: 20
    },
  ]
}

If we could intelligently have selective deployments check the status of a "running" installation, and validate the unchanged or changed products match what you are trying to selectively deploy, that would make the feature more robust. right now, it is very misleading, and could lead to a false success of the selective deploy, even if it wasn't the current deploy running.
Expected Behavior:
if using selective deploy and the product defined is not the product(s) currently being deployed
I want to return a failure to maintain honestly of the command.

After some discussion, we've actually chosen to make the re-attachment to an apply-changes optional by the user.
If a user runs apply-changes from the CLI (#1), makes changes, and runs apply-changes again (#2).

If #1 is still running, the #2 re-attaches to the currently running #1. From a CI standpoint, it is not clear that #2 never actually successfully ran.
If #1 is done, then #2 runs it own apply changes.

Since the former, can be disruptive, especially for selective apply changes, we are adding the options --reattach to om apply-changes. The current behaviour of auto-attachment will be removed. This provides more control and intention to a user.
This will be a breaking change in 4.0.


The results of the 'Mapped Types' distribution have changed.

Bug Report
The conversion result of 'Mapped Types' has changed in 4.3.2 or later typescript.
💻 Code
type Base = { 100: { a: number }; 200: { b: string } }
type NewType<T = Base> = {
  [P in keyof T ]: {
    code: P
    body: T[P]
  }
} extends {
  [P in any]: infer R
}
  ? R
  : never

let test: NewType

🙁 Actual behavior

4.3.0-beta and earlier

let test: {
    code: 100;
    body: {
        a: number;
    };
} | {
    code: 200;
    body: {
        b: string;
    };
}

4.3.2

let test: {
    code: keyof Base;
    body: {
        a: number;
    } | {
        b: string;
    };
}

🙂 Expected behavior
Hoping to return to pre-4.3.0-beta behavior!

Related: https://github.com/microsoft/TypeScript/issues/44143

I was able to temporarily work around the problem by inserting 'extends' once!
This code also works in 4.3.2 and previous versions.
type Base = { 100: { a: number }; 200: { b: string } }
type NewType<T = Base> = {
  [P in keyof T ]:(({
    code: P
    body: T[P]
  }) )
} extends {
  //[P in keyof T ]: infer R
  [P in keyof T ]: unknown extends unknown ? infer R : never
}
  ? R
  : never

let test: NewType

Another workaround (tried on ts 4.4.0-dev.20210530):
type NewType<T> = {
  [P in keyof T]: {
    code: P
    body: T[P]
  }
}[keyof T]

type Base = { 100: { a: number }; 200: { b: string } }
declare const test: NewType<Base>

Looks like this was broken by #43649.

@weswigham I'm not sure about the changes in #43649. They break the example in this issue because, as we check that inferences for R are assignable to the constraint of R (which is computed by #43649), the constraint contains un-instantiated references to T and the assignability check therefore fails. When then pick the constraint instead, which subsequently is instantiated with the type passed for T. But that's too late and not the result we want. We could potentially consider adding logic to somehow instantiate the constraint before the assignability check, but that would further complicate the inference process in conditional types and I'm not even sure I agree it's the right thing to do. I think it is perfectly fine for the inferred R to not have a constraint--and, honestly, there are many similar situations in which we don't produce constraints (e.g. if the check type is just a regular object type).
My recommendation would be to back out #43649. That of means that #43357, the issue it fixes, needs some other solution. As you yourself point out, it never really should have worked, and I think the right solution is to explicitly intersect with keyof T in the KeysWithoutStringIndex<T> type, i.e.
type KeysWithoutStringIndex<T> =
    ({ [K in keyof T]: string extends K ? never : K } extends { [_ in keyof T]: infer U } ? U : never) & keyof T;

I think our syntactic inference constraint rules are kiiiiiinda arbitrary to begin with (what type parameters we match up across what locations has kinda been chosen by request) - I "fixed" the issue by adding one because the change in behavior was technically a regression, and adding those syntactic constraints is sort-of free to do. The extra instantiation would probably be the best thing we could do to keep the old behavior going. We could back it out... But then we'd be reintroducing the old regression...


Custom icons not set for all configs

Regarding the icons is see two minor issues.

We could improve the selected icons and not use the label everywhere.
Sometimes there is a space missing after the icon.

Proposed fix: go through all HTML files and check the space. Change icons if better ones are available.

When you do that could you please also have a look for missing description. i know that there are some nodes, but not where :)

Put the missing descriptions I found and could not fix immediately into the new issue #29 .


Add a "Testimonials" docs page

I'd love to have a page with actual testimonials from real end users who are using Redux to do nifty stuff, especially things that really show the benefits of Redux (middleware that manipulate actions, apps that persist or sync state, scaled-up codebases, etc).

I would love to have that, as well as CodeSandbox examples, and webinars that allow us to learn from others. Blogs are fun but, they have limited interactivity. We need another medium for cool technologies, especially the ones that start to mature.
JS, needs to stop inventing and replacing framework after framework, but built upon and around 4-5-6 existing ones, in order to really help people create amazing stuff.
If you need help or volunteers, reach out. I know a few people.

Hi! It's my first time to open source projects. Is this open for contribution? I've created one design for this if this is open. Let me know. Thank you! :D

Hiya.  Yeah, this is open in general, although to be honest I haven't really given it any thought since I created the issue.
I guess the biggest thing that needs to be done is actually getting folks who use Redux to contribute quotes and blurbs that we can post :)  Once we've got some actual testimonials to put up there, then yeah, some good formatting would be helpful.
Lemme see if I can maybe tweet a request tomorrow for folks who would like to be featured here or something.

Yeah, I agree. Just mention me anytime when you get folks willing to get their blurbs posted and when you will give this issue a go. Thanks :D

I would be also interested to contribute to this. I can collect the quotes from response thread. Posting on reddit might generate a better response.

Slightly belated, but just tweeted that request: https://twitter.com/acemarke/status/1120535529936891905

👋 big fan. I am happy to contribute

Where you after a general comment on the library itself, or more of a why and how we are using it?

more of the "why and how", with the intended usage being including it in that eventual docs page attributed to you.

Here we go:

redux is a simple, yet fantastically powerful and fast state management solution. We chose redux for react-beautiful-dnd in order to facilitate fast deep tree updates in react (with react-redux), and for middleware which allows for complex data flow logic. Developing with redux is a pleasure thanks to it's extensive documentation, tests, passionate maintainers, and large community.

Wow, I could write multiple quotable comments on how and why we use redux at IOOF. I stand by that tweet, but could also talk about testability, predictability, the strange things we do with middleware (e.g. auth, analytics), the amazing docs, the incredible Dev tools, and probably more if I put my mind to it.
Too much for one testimonial, so let me know which one(s) you're most interested in.

* Who you are and where you're using Redux

Kumar McMillan, Staff Software Engineer at Mozilla.
My team uses React and Redux on https://addons.mozilla.org (code: https://github.com/mozilla/addons-frontend/)

* How it's helping your development

What if you could stop an infinite loop by pressing a pause button? With Redux you can! I can't imagine building a UI without Redux and its developer tools. Debugging is easy. Testing is easy. Bouncing between states while designing screens is easy. Everything is just easy.

* _Why_ you're using Redux

It makes development fun and easy.

* Some of the scale of your application in terms of SLOC, reducers, number of users, etc

11 million monthly users
96K SLOC
29 reducers

Closing in favor of #3602


Frontend Library Split RFC

Proposal for a split of the frontend library

+100 to this! I just created a JS monorepo lately and it is way more effective and would make working in Amundsen more efficient.
However, I don't know how to manage a mixed python/JS monorepo, I guess a lot of stuff needs to be done manually vs the JS only projects using Lerna, right?

I haven't done this exact combination in a few years... but if there's some bubbling support for it, I'd be happy to propose an RFC and do the necessary research.

I wasn't really proposing to change the API server to be node based, although this change would allow to do it easily if we wanted to go that way.

ohh, sorry, i misread the next.js mention. Makes sense.

Hi team,
To wrap up this RFC, here is a summary:

We agree that the migration path would be the most important challenge when doing something like this
Most of us think that keeping an API proxy layer would be useful for now
All of this would be a lot easier if we were in a monorepo setup

Because of that, I will close this RFC and instead create another one to suggest moving Amundsen into a monorepo configuration, then we would be able to do the FE split a lot easier.

Hi team,
To wrap up this RFC, here is a summary:

We agree that the migration path would be the most important challenge when doing something like this
Most of us think that keeping an API proxy layer would be useful for now
All of this would be a lot easier if we were in a monorepo setup

Because of that, I will close this RFC and instead create another one to suggest moving Amundsen into a monorepo configuration, then we would be able to do the FE split a lot easier.


[Loggable] Allow for Embedded documents to also have child documents

In the current implementation if a embedded class has  child embedded class the fields for this child's do not get logged. This is solving that problem. I'm not sure how to update and test the other mappings here.

thanks ;)


Update with IF

Search around and still could NOT figure out how to do UPDATE with IF. Is this supported? Thanks. -ky

Hi @yuekui2, GoCassa currently only supports updating single documents, for more advanced queries I would recommend using GoCQL directly.


[YouTube] No longer extracting like_count for video metadata

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
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
No response
Provide a description that is worded well enough to be understood
The field like_count no longer gets parsed in the YouTube info json metadata. It looks like they have moved things around in the topLevelButtons, but I couldn't quite track down how to access that correctly with traverse_obj.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] 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
In [1]: import yt_dlp

In [2]: obs = {"skip_download": True, "verbose": True}
   ...: url = "https://www.youtube.com/watch?v=hNZy603as5w"
   ...: info_json = yt_dlp.YoutubeDL(obs).extract_info(url)
   ...: info_json["like_count"]
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [24f827875] API
[debug] params: {'skip_download': True, 'verbose': True, 'compat_opts': set(), 'http_headers': {'User-Agent': 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/90.0.4430.212 Safari/537.36', 'Accept': 'text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8', 'Accept-Language': 'en-us,en;q=0.5', 'Sec-Fetch-Mode': 'navigate'}}
[debug] Python 3.11.6 (CPython x86_64 64bit) - Linux-6.1.66-1-lts-x86_64-with-glibc2.38 (OpenSSL 3.1.4 24 Oct 2023, glibc 2.38)
[debug] exe versions: ffmpeg 6.1 (setts), ffprobe 6.1
[debug] Optional libraries: certifi-2023.11.17, requests-2.31.0, sqlite3-3.44.2, urllib3-1.26.18
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests
[debug] Loaded 1901 extractors
[youtube] Extracting URL: https://www.youtube.com/watch?v=hNZy603as5w
[youtube] hNZy603as5w: Downloading webpage
[youtube] hNZy603as5w: Downloading ios player API JSON
[youtube] hNZy603as5w: Downloading android player API JSON
[youtube] hNZy603as5w: Downloading m3u8 information
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, vcodec:vp9.2, channels, acodec, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), channels, acodec, lang, proto, size, br, asr, vext, aext, hasaud, id
[debug] Default format spec: bestvideo*+bestaudio/best
[info] hNZy603as5w: Downloading 1 format(s): 313+251
---------------------------------------------------------------------------
KeyError                                  Traceback (most recent call last)
Cell In[2], line 4
      2 url = "https://www.youtube.com/watch?v=hNZy603as5w"
      3 info_json = yt_dlp.YoutubeDL(obs).extract_info(url)
----> 4 info_json["like_count"]

KeyError: 'like_count'

@Ganesh910 sure, feel free to open a PR. I had already begun working on it, this is what I came up with:
diff --git a/yt_dlp/extractor/youtube.py b/yt_dlp/extractor/youtube.py
index 86818a9dc..5b14b187a 100644
--- a/yt_dlp/extractor/youtube.py
+++ b/yt_dlp/extractor/youtube.py
@@ -4480,14 +4480,13 @@ def process_language(container, base_url, lang_code, sub_name, query):
                             if mobj:
                                 info[mobj.group('type') + '_count'] = str_to_int(mobj.group('count'))
                                 break
-            sbr_tooltip = try_get(
-                vpir, lambda x: x['sentimentBar']['sentimentBarRenderer']['tooltip'])
-            if sbr_tooltip:
-                like_count, dislike_count = sbr_tooltip.split(' / ')
-                info.update({
-                    'like_count': str_to_int(like_count),
-                    'dislike_count': str_to_int(dislike_count),
-                })
+
+            info['like_count'] = traverse_obj(vpir, (
+                'videoActions', 'menuRenderer', 'topLevelButtons', ...,
+                'segmentedLikeDislikeButtonViewModel', 'likeButtonViewModel', 'likeButtonViewModel',
+                'toggleButtonViewModel', 'toggleButtonViewModel', 'defaultButtonViewModel',
+                'buttonViewModel', 'accessibilityText', {parse_count}), get_all=False)
+
             vcr = traverse_obj(vpir, ('viewCount', 'videoViewCountRenderer'))
             if vcr:
                 vc = self._get_count(vcr, 'viewCount')

@bashonly I am new to open source contribution but am good at python. So I need some help to get started!
I am unable to reproduce the error.
I forked and cloned the repo and first ran
pip install -r requirements.txt
Then i pip installed yt-dlp also.
And then i ran the command which is mentioned in the bug
yt-dlp -vU https://www.youtube.com/watch?v=hNZy603as5w
And the video was downloaded on my system. I got no error.
Here is output
yt-dlp -vU https://www.youtube.com/watch?v=hNZy603as5w
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [24f827875] (pip)
[debug] Python 3.10.12 (CPython x86_64 64bit) - Linux-6.5.6-76060506-generic-x86_64-with-glibc2.35 (OpenSSL 3.0.2 15 Mar 2022, glibc 2.35)
[debug] exe versions: none
[debug] Optional libraries: Cryptodome-3.19.0, brotli-1.1.0, certifi-2023.11.17, mutagen-1.47.0, requests-2.31.0, sqlite3-3.37.2, 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)
[youtube] Extracting URL: https://www.youtube.com/watch?v=hNZy603as5w
[youtube] hNZy603as5w: Downloading webpage
[youtube] hNZy603as5w: Downloading ios player API JSON
[youtube] hNZy603as5w: Downloading android player API JSON
[youtube] hNZy603as5w: Downloading m3u8 information
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, vcodec:vp9.2, channels, acodec, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), channels, acodec, lang, proto, size, br, asr, vext, aext, hasaud, id
[debug] Default format spec: best/bestvideo+bestaudio
[info] hNZy603as5w: Downloading 1 format(s): 22
[debug] Invoking http downloader on "https://rr2---sn-gwpa-pmjk.googlevideo.com/videoplayback?expire=1702421441&ei=YY94ZYP0HZ6B9fwP--qwoAg&ip=2409%3A40d0%3A102d%3Aaa8e%3A5897%3Ab4bb%3A3e07%3A3dd2&id=o-AM373uWndqJhNjq9QfIjs2Z5LDBjh5tDYkk3FykXfnrN&itag=22&source=youtube&requiressl=yes&xpc=EgVo2aDSNQ%3D%3D&mh=Jv&mm=31%2C29&mn=sn-gwpa-pmjk%2Csn-gwpa-qxae7&ms=au%2Crdu&mv=m&mvi=2&pl=36&initcwndbps=480000&spc=UWF9f_ZPLR7wI363PhsbAc2brCdEqP8&vprv=1&svpuc=1&mime=video%2Fmp4&cnr=14&ratebypass=yes&dur=323.662&lmt=1702379385757469&mt=1702399522&fvip=1&fexp=24007246&c=ANDROID&txp=4432434&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cxpc%2Cspc%2Cvprv%2Csvpuc%2Cmime%2Ccnr%2Cratebypass%2Cdur%2Clmt&sig=AJfQdSswRQIgGb0d9dBgXahmuf2hiaYwe3HMyoCj_aVSRjXDK7ZdzVICIQDn4mAx8bhjUoNkuMd9AxBihXj7NrKiUoYxDRMg_hP1QQ%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AAO5W4owRQIhANdq3KLmHEeCXv8ldpJnBjV8byMJxIKBANosKMNSnU9dAiAA2a5A8O4OXrBSuTfmmegXA3Se6TzHbmGBG2ogP9awMA%3D%3D"
[download] Destination: A robot just swapped my electric car's battery [hNZy603as5w].mp4
[download] 100% of   20.48MiB in 00:00:02 at 8.56MiB/s```

You'll need to git clone the repo so you can run the tests.
Run this:
python test/test_download.py TestDownload.test_Youtube
and you should see it fail due to a missing value for like_count

@bashonly After debugging for some time, This is what I get to know
line 4445 in youtube.py
vpir = get_first(contents, 'videoPrimaryInfoRenderer')
vpir is able to get the values
But in line 4483
sbr_tooltip = try_get(vpir, lambda x: x['sentimentBar']['sentimentBarRenderer']['tooltip'])
It is unable to get any value because there isn't ['sentimentBar']['sentimentBarRenderer']['tooltip'] in vpir
And since sbr_tooltip is none here, these next lines are not run
                like_count, dislike_count = sbr_tooltip.split(' / ')
                info.update({
                    'like_count': str_to_int(like_count),
                    'dislike_count': str_to_int(dislike_count),
                })```

 Hence getting the error.
 I just don't know why those values have not come in vpir. What should I do next?

I think my patch above should be sufficient, but I've only tested it with a couple of videos

@bashonly I think Now i got eveything!
I took me some time as I am beginner. But you updated the like_count directly. But You seem to saving the whole string for the likes inside info['like_count']
I used different attribs to get the number of likes only and then saved it
And dislike is not updated as they are now being given in the json
Can I submit PR if my changes are passing the test you gave?

parse_count parses the count from the accessibility text string. That is the only field in the initial data where the true/accurate like_count is found. The other fields will be approximate (e.g. 16K) for videos with more than ~1000 likes

Okay, can you please help me to change the url of the video in test case. How do i change that so that i can test.
Also should I now not work on it as it seems your change is indeed sufficient?
What should I do now?

as I said above,

feel free to open a PR

further discussion can take place in the PR
you shouldn't need to edit any test cases

@bashonly one case failed. I am unable to understand which one. Can you please help so that i can fix it?


[BUG] When provisioning an RKE1 node drivercluster Harvester went down

Describe the bug

When I was provisioning a RKE1 node driver cluster (1 node) my Harvester cluster lost network connectivity
To Reproduce
Steps to reproduce the behavior:
I'm not sure if this is reproducible. I'm looking into it

Provision RKE1 node driver cluster (1 node) on single node Harvester cluster.
Wait for RKE1 node to go active

Expected behavior

Harvester networking shouldn't crash
Support bundle

supportbundle_d3bbb44a-07f9-44ce-9d87-d31e6ef4ace6_2021-09-16T17-22-10Z.zip
Environment:

Harvester ISO version: master-3d096946-head
Rancher version: 2.6.3-rc5
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): KVM virtual machine 16 cores 16GB RAM

Additional context
Add any other context about the problem here.
The RKE1 cluster went active then seemed to go down. When looking into it I realized that I couldn't ping or SSH into my harvester host. I rebooted the machine and it came back up fine. This was a RKE1 cluster with the Harvester cloud provider included. In the RKE1 node template I added this in the user data
#cloud-config
password: password
chpasswd: {expire: False}

@noahgildersleeve can u please re-check if this can still be reproduced with the latest master version, thanks.

Tested in Harvester master-da3f1778-head and Rancher v2.6-3f9109c3f3a1ea482f66f7e3700e495d36517e10-head.
Verification steps

Import Harvester into Rancher
Provision RKE1 cluster in Harvester
Check if you still have connectivity with Harvester, Rancher, and RKE1


Add title attribute to h1 input so the user knows its editable

In my fork I had a title attribute on the h1 input that read "Click to edit". it's not obvious that it can be edited.

Not sure if the tooltip is obvious enough. Maybe a css hover style? I had to tell Cheery that it was editable while she was using it on stream.


Commands not being defined/sourced

Hi,
I cannot figure this out. No matter what sort of file I open, in whatever directory, I cannot seem to get the 'CMake*' commands available.
I have to manually call:
call cmake#commands#apply_global_commands()
call cmake#commands#apply_buffer_commands()

And then I get the  various commands available, but the few I tried do not work because some buffer variables are not defined.
Basically, it seems that the plugin for some reason or another does not see any of files as part of a valid CMake project.
I have tried this with a clean configuration (i.e., with cmake.vim being the only plugin and no vimrc, and get the same results.).
Any ideas what might be going on?

Same issue here on vim 7.4.475.

@jeetsukumaran A few questions:

How do you define your project? You mind doing a find . -type f of the tree structure?
What files in particular are you trying to open? A source file or a CMake file itself?

As you can see here, cmake.vim only begins to set those options for buffers after obtaining its filetype1. It used to work off file extensions but that was biased on my use of .{h,c}pp and .{hh,cc}.

Hi, it would be great if this plugin could support arbitrary file types, as CMake is used in a very wide variety of projects.
In any case, I do not get the commands defined even in my C++ projects:
./cmake/Modules/DisableInSourceBuild.cmake
./cmake/Modules/ListFilter.cmake
./cmake/Modules/TrackGitRevision.cmake
./cmake/Modules/TrackGitRevision.cmake.in
./CMakeLists.txt
./LICENSE.txt
./README.txt
./src/CMakeLists.txt
./src/pstrudel/character.cpp
./src/pstrudel/character.hpp
./src/pstrudel/CMakeLists.txt
./src/pstrudel/dataio.hpp
./src/pstrudel/echo_pstrudel_info.sh
./src/pstrudel/profile.cpp
./src/pstrudel/profile.hpp
./src/pstrudel/pstrudel.hpp
./src/pstrudel/pstrudel_trees_main.cpp
./src/pstrudel/split.cpp
./src/pstrudel/split.hpp
./src/pstrudel/treeshape.cpp
./src/pstrudel/treeshape.hpp

regardless of the file I open.
As an example of a non-C++ project, I also have CMake as a build system in my LaTeX project:
./cmake/Modules/DisableInSourceBuild.cmake
./cmake/Modules/ListFilter.cmake
./cmake/Modules/UseLATEX.cmake
./CMakeLists.txt
./src/archipelago-model/archipelago-model.bib
./src/archipelago-model/archipelago-model.tex
./src/archipelago-model/CMakeLists.txt
./src/CMakeLists.txt

Interesting. This is a design flaw then. I can get to this. My reasoning to fixing it to specific file types was to prevent the pollution of one's buffer commands in a particular buffer. That use of having CMake build LaTeX projects is new to me :dizzy:!
Also, I think I see the bigger problem here. You're disabling in-source builds? I haven't tested the plugin to do out-of-source builds. I'd have to refactor the tests to handle this. Thanks for the feedback @jeetsukumaran.

This is my fault for not populating the documentation adquetely but have you tried setting g:cmake_build_directories?

It certainly makes sense to avoid polluting the command namespace! Though so many different types of files can be part of a CMake project (e.g., even a C++ project may include its tex, markdown, pandoc, doxygen, etc. files as part of the CMake build) that I think this might be difficult to restrict based on filetype. I think the commands should, in fact, be available globally rather than buffer specific? For example, I can see having a test output buffer open, making a change in a project file then back to the test output buffer (or even a totally non-related file) and invoking the build from there. It would be cumbersome to navigate back to a particular buffer that happens to have the build commands defined. Of course, this makes identifying the build directory tricky.

This is my fault for not populating the documentation adquetely but have you tried setting g:cmake_build_directories?

No! That might be the problem. Does this need to be defined before a buffer is loaded, e.g., in ~/.vimrc?

This is my fault for not populating the documentation adquetely but have you tried setting g:cmake_build_directories?
No! That might be the problem. Does this need to be defined before a buffer is loaded, e.g., in ~/.vimrc?
Ok, just tried:

let g:cmake_build_directories = ['build']

in my ~/.vimrc and no joy on the C++ project.

I think the commands should, in fact, be available globally rather than buffer specific?

It's funny you mention this, it originally was globally set for most of the commands. I think that for this, it'd make sense to search for the target of a particular buffer, set b:cmake_target accordingly and then have the global command use that value to do what's necessary (like run a test, clean or build). For the most part, that should be as easy as moving some lines from cmake#commands#apply_buffer_commands() to ...#apply_global_commands().
I'm actively working on fixing the test suite so once I can get it at 100% passing, I can vet PRs for this .

Thanks! Appreciate your work on this plugin!

Thank you for reporting this!

Okay so #54 should have handled the act of loading the commands globally instead of doing it on a buffer-by-buffer basis. Now for getting the target of said buffers.

You'd be able to invoke :CMakeBuild normally in one of these files below if you used in a in-source build setup.
./src/pstrudel/character.cpp
./src/pstrudel/character.hpp
./src/pstrudel/CMakeLists.txt
./src/pstrudel/dataio.hpp
./src/pstrudel/echo_pstrudel_info.sh
./src/pstrudel/profile.cpp
./src/pstrudel/profile.hpp
./src/pstrudel/pstrudel.hpp
./src/pstrudel/pstrudel_trees_main.cpp
./src/pstrudel/split.cpp
./src/pstrudel/split.hpp
./src/pstrudel/treeshape.cpp
./src/pstrudel/treeshape.hpp

Having a out-of-source build is a bit tricky for auto-detection. What's the typical convention for defining out-of-source builds?

Alright, so I just added some changes for out-of-source builds. The thing is, you'd have to manually set g:cmake_root_binary_dir for your project since there isn't a defined practice for handling out of source builds. I'm debating doing a sibling folder check for a CMakeCache.txt but that can lead to conflicts if you have a bunch of projects with out-of-source builds. Any tips + suggestions are welcome :)

Alrighty, version 0.5.2 should help you out. Let me know if anything goes wrong.

Hi Jacky,
Thank you so much for your work on this! It works!
I think setting a variable to define the CMake root binary directory is fine. I wonder if making it a global string g:cmake_root_binary_dir may provide sufficient flexility, though? E.g., some projects may have multiple build directories (build/release, build/debug). Perhaps the global string variable can serve as a default, and then commands that require it can take a flag that overrides? E.g., CMakeBuild -b build/debug? But if this is too complicated or breaks other things, then the current set up is fine.
Thank you again!

I wonder if making it a global string g:cmake_root_binary_dir may
provide sufficient flexility, though? E.g., some projects may have multiple
build directories (build/release, build/debug). Perhaps the global
string variable can serve as a default, and then commands that require it
can take a flag that overrides? E.g., CMakeBuild -b build/debug? But if
this is too complicated or breaks other things, then the current set up is
fine.

Shouldn’t be too hard to get going. Now, my only concern, though a little
crazy, would be different build directories that make use of different
generators. With your example, if someone used Makefiles for build/release
but Ninja for build/debug, the plugin might get a bit confused since it
requires input about that. I’d have to probably add a detection for the kind of
project that’d be built.
I’ll look into it for later into the week. Thanks again for the feedback!

Took me a bit longer than I expected but I might have to punt on having multiple build directory support. It's only because that'd require a bit of rewiring so that the plug-in's is less deterministic in what paths to choose and how it uses the build directory (it uses it for everything.) I'm open to patches to work on as always!

Hi Jacky,
I would be happy to contribute a patch, but, I too am swamped right now. I will look into it a little bit down the road. Thanks for all the work so far!

Hey @jeetsukumaran, you mind if I close this issue? I'd make an new issue referring to the multiple build system idea (I just ran into it when beginning to cross-compile a project, heh) and targeted it for the next minor release.

No problems at all. Looking forward to the developments!

Dope, I'll flesh it out before the weekend over on #60.


Any specific need for the --skip-grant-tables?

I would like to secure the DB, does MariaDB4j needs the --skip-grant-tables argument? Can it be removed safely?

@kaismh how about we add some method to REMOVE parameters which MariaDB4j adds by default? A bit similar to https://github.com/vorburger/MariaDB4j/pull/23, but sort of the opposite of it... thoughts? Any PRs welcome, BTW.. ;-)

@kaismh are you alive? I finally have time to pick up MariaDB4j maintenance, and if you would like to contribute a PR with what you had in mind above, I'd certainly look at and merge it ..

I would also find this useful.  Let me taker a look at the code this weekend.

Closing as this is DONE via https://github.com/vorburger/MariaDB4j/pull/83 - big TX to all who contributed in the discussion here, and to @marcelvanderperk for the code!


Appended run.py with comm arg support

Note: This commit can be ignored for a while, till the bot takes a stable form.

Hm, does not work for me.

the args get parsed but python cannot really see them.
Are we sure this is implemented correctly in Piston?

it is in piston for sure, not sure what's happening here though

Heh found the problem.
This is the url used in felix/pBot: 'https://emkc.org/api/internal/piston/execute'
changing that to v1 makes it work

Thanks @Minecraftian14 - this might actually be quite useful

Thanks a lot for the appreciation!
I am sorry I couldn't reply before.


Some iOS users crash with SIGABRT: Failed to grow buffer

please provide the version of installed library and RN project.

      "react-native": "0.66.2",
    "react-native-blob-util": "^0.13.18",

We have reports from iOS users, that when they start uploading photos/videos (with a single endpoint call) the app crashes.
This is what we get in crash report:
SIGABRT: Failed to grow buffer
Any ideas if this is a bug (seems a low memory issues?) or we can change something for these crashes not to occur?

Do you have a line of code for it?
It is some kind of problem with the memory allocation, yes.
But it could  be a wrong length for the data or sth like that.

Sorry @RonRadtke, the crash report did not give me the line number of the JS.

If I see that error again I will let you know, meanwhile we get reported that users tried to upload for example 34 photos in a direct upload and it managed to download some and then crashed.
Will it help memory-wise to split the single large upload of all files into separate uploads? (one per 2 files for example?)

@RonRadtke I have more info on this now:
ReactNativeBlobUtilReqBuilder.m, line 63

See also screenshot:

Same problem. We've tried some other libraries as well, which end up in the same exception.
The problem is reproducible with large files (900MB) on iOS.

I assume that the app can't allocate as much memory as it would have to and thus fails.
So the only option would be some way of streaming the file in chunks I think.

So the only option would be some way of streaming the file in chunks I think. - How can we do it?


Adding jmeter-driver

Signed-off-by: Shruthi<EMAIL_ADDRESS>

Looks like there is a conflict with the file .gitignore, the PR cannot be merged without the conflicts being resolved.

Looks like there is a conflict with the file .gitignore, the PR cannot be merged without the conflicts being resolved.


Improving return types of action creators by adding generic args type

The reason for this is to have better autocomplete on the created actions.
It seems like a very niche use, but I ran into it and would like to have better autocompletes.
Also improves the LocationState type inside LocationDescriptorObject, before it was aways any.

Hey @supasate, Hmm there might be if someone is implementing the CallHistoryMethodAction and LocationActionPayload interfaces, I can't find a reason anyone would do that, but one way of making it backwards compatible would be to default those generics to any, something like export interface LocationActionPayload<A = any> {
Should I do the change?

Hey @supasate, I made the changes, what do you think?

LGTM


[doctor] Ignore files that don't exist

Minor fix to ignore files that don't exist.
Without this change, the following error occurs on running bin/doom doctor if savehist or projectile.cache are not present.
! Attempt to load DOOM failed
  (number-or-marker-p nil)

I came up with a slightly better solution in 962459e which should resolve this issue. In any case, thanks for bringing it to my attention!


Authenticate using CLI

I would like to access the server behind caddy-security with a powershell script.
I setup the authroization portal like described in: https://authp.github.io/docs/authenticate/oauth/backend-oauth2-0011-keycloak
And using the browser everything works fine.
I am able to authenticate to keycloak and I am getting a valid token from the same client that is used by caddy-security.
The variable $token.access_token has a valid keycloak token from the same keycloak client.

$local:header = New-Object "System.Collections.Generic.Dictionary[[String],[String]]"
$local:header.Add("Accept", "application/json")
$local:header.Add("Content-Type", "application/json")
$local:header.Add("Authorization", "Bearer $($token.access_token)")
$local:body = New-Object "System.Collections.Generic.Dictionary[[String],[String]]"
$local:body.Add("Authorization", "Bearer $($token.access_token)")
$local:result = Invoke-WebRequest -Method Post -Uri "https://auth.example.com/login" -Headers $local:header -SkipHttpErrorCheck -Body $local:body
$local:result

I am trying the approach you suggested here: https://authp.github.io/docs/authenticate/misc#json-api
Caddy Log shows: "keystore: failed to parse token"
Any hint how I can get this to work?

@jb-home , do you have bearer token support enabled in caddy security config? https://authp.github.io/docs/authorize/token-discovery

I tried:
'validate bearer header' and 'set token sources header query cookie' and both at the same time.
I saw that the token I get from Keycloak has a different ALG. I think it is RS256, and auth.example.com has a HS512. So I set HS512 in the Keycloak client as default. Even than the token does not get accepted.
Do I have to set the same crypto-key? If so, do you know how to request a token with specific crypto-key?

@jb-home , please share your config.

Here my Caddyfile

{
	order authenticate before respond
	order authorize before basicauth
security {
	oauth identity provider keycloak {
		driver generic
		realm keycloak
		client_id lpauthp
		client_secret <CLIENTSECRET>
		scopes openid email profile
		metadata_url https://keycloak.example.com/realms/MYREALM/.well-known/openid-configuration
	}

	authentication portal myportal {
		crypto default token lifetime 28800
		crypto key sign-verify <MYSIGNVERIFYKEY>
		enable identity provider keycloak
		cookie domain something.example.com
		ui {
			links {
				"My Identity" "/whoami" icon "las la-user"
			}
		}
	}

	authorization policy workflowpolicy {
		set token sources header query cookie
		validate bearer header
		set auth url https://auth.example.com/
		allow roles workflow/user
		crypto key verify <MYSIGNVERIFYKEY>
	}
}

grace_period 5s

}
(headers) {
header {
-Server
Permissions-Policy interest-cohort=()
X-Xss-Protection "1; mode=block"
X-Robots-Tag none
X-Frame-Options SAMEORIGIN
Cache-Control no-cache
Referer-Policy no-referer-when-downgrade
X-Content-Type-Options nosniff
Strict-Transport-Security "max-age=31536000; includeSubDomains"
Content-Security-Policy upgrade-insecure-requests
}
}
:80 {
redir / https://{hostonly}{uri} 301
}
auth.example.com {
import headers
authenticate with myportal
}
workflows.example.com {
import headers
authorize with workflowpolicy
reverse_proxy http://nodered:1880 {
lb_policy ip_hash
}
}

@jb-home , are you using HTTPS or HTTP? If it is 80, then secure cookies are not propagated. In that case, you have to use “cookie insecure on” directive.

Also, I am not sure what effect the headers have on the cookies.

I am using HTTPS only.
The headers work fine using a browser. So I expect that CLI should be kind of transparent for the service.

@jb-home , the cli login works only with local auth store. It would not work with OAuth user store.

Is there a way you could make it work?
As I pass the valid token from OAuth with same realm and same client, your plug-in should be able to verify that token.
I wonder that nobody else is using such a construct.

As I pass the valid token from OAuth with same realm and same client, your plug-in should be able to verify that token. I wonder that nobody else is using such a construct.

There is, but it requires development time.
What needs to happen is connecting the identities from various store and linking them to local store. See https://github.com/greenpau/caddy-security/issues/23


Point NPM Package (global) to Organisation NPM package.

At this time there are 2 hosted NPM packages for the Token-Negotiator.
This task is to update the Token-Negotiator README - to explain that the @Alpha-Wallet/Token-Negotiator should be used.
This I feel will help bring more knowledge of Alpha Wallet and the products are creating inside the Open Source Blockchain space.

Closing this issue for now. Because of the NPM / Github work flow, it makes sense that the examples use the direct code inside this repo for the token-negotiator, so the examples can be tested before publishing new NPM package versions.