[Switch] bug on Google Chrome 88

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Switches don't display properly after upgrading to Google Chrome v88. Cursor changes upon hovering to the area where switches are expected to show. You can click them but you can't toggle it back. Also upon clicking a switch, all other switches became visible on a brief moment before disappearing.
Expected Behavior 🤔
Switches display and toggle properly.
Steps to Reproduce 🕹
I can't replicate the bug on codesandbox maybe because this is a browser-related bug.

Steps:

Upgrade Google Chrome to latest version (v88.0.4324.150)
Switches go brrrr.

Context 🔦

Our forms are of limited use to unusable because of this. Un-toggling a switch is not possible because of this bug.
Your Environment 🌎

`npx @material-ui/envinfo`
  System:
    OS: Windows 10 10.0.18363
  Binaries:
    Node: 12.18.1 - D:\Program Files\nodejs\node.EXE
    Yarn: Not Found
    npm: 6.14.5 - D:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 88.0.4324.150
    Edge: Spartan (44.18362.449.0)
  npmPackages:
    @material-ui/core: ^4.11.0 => 4.11.3
    @material-ui/icons: ^4.9.1 => 4.11.2
    @material-ui/lab: ^4.0.0-alpha.56 => 4.0.0-alpha.57
    @material-ui/pickers: ^3.2.10 => 3.2.10
    @material-ui/styles:  4.11.3
    @material-ui/system:  4.11.3
    @material-ui/types:  5.1.0
    @material-ui/utils:  4.11.2
    @types/react:  17.0.0
    react: ^16.13.1 => 16.14.0
    react-dom: ^16.13.1 => 16.14.0

Thanks for the report.
Unfortunately, I'll have to close this since I cannot reproduce the issue. Feel free to mention me or open a new issue once you can reproduce it.

I can't replicate the bug on codesandbox maybe because this is a browser-related bug.

This might also indicate that there's a bug with your code.

The video shared in #24660 might surface the same underlying issue. But it's a speculation a this point.

@eps1lon Uhmm, I don't know what environment codesandbox is using, but the behavior is consistent on all the Chrome browsers in our office that was updated to v88. The switches also work on Firefox. There were no any code errors on the console and we haven't updated any part our code so the possibility of this error being a bug of ours is zero. It really just appeared after Google Chrome released their new update.

@eps1lon Here's a video showing the bug. https://youtu.be/ZPeecks4BgY

@ksdelacruz I can't reproduce either :/

@eps1lon @oliviertassinari I tried using switch on the other parts of the our website just to test it and the switch work. Maybe we'll just find a workaround for this. Tho I firmly believe the update of Chrome did messed up the functionalities of our website (as it works days before the update). Thanks.

@eps1lon @oliviertassinari I finally replicated it! https://codesandbox.io/s/material-ui-issue-forked-2p0b3?file=/src/Demo.js
It's similar to the issue here: https://github.com/mui-org/material-ui/issues/24825
Our switches were indeed wrapped inside FormControl and FormLabel.

The problem is with the ripple effect. Legend position is static by default so putting position relative stops ripple overflow. See https://github.com/mui-org/material-ui/issues/24825#issuecomment-774725487 @oliviertassinari and @eps1lon


Update v2 TOR addresses to v3 for GRS

Also adds regtest and signet mode for GRS

@SomberNight Care to review and merge this please?


Allow moreLink and lessLink options to use callbacks

I've had a request from a client to make the more/less links include the content of the first <h2> tag of the collapsed block. For example, "Read more about Contributing to Readmore.js". I don't really want to try and alter the text after the link has been generated, especially since it changes depending on the collapsed state. Allowing callbacks for moreLink and lessLink would make for a simple solution.

This will be included in Readmore.js v3.


Swagger API: NetworkConfig is not correct

(Note that the title is a joke, referring to swagger.yaml line 818: "TODO check is correct".)
In the swagger spec for inspecting a container (operationId: "ContainerInspect"), one of the object properties is NetworkSettings. For that value, the spec refers to the object definition at #/definitions/NetworkConfig. However, that definition is not complete.
Here is the incomplete NetworkConfig swagger spec:
NetworkConfig:
  description: "TODO: check is correct"
  type: "object"
  properties:
    Bridge:
      type: "string"
    Gateway:
      type: "string"
    Address:
      type: "string"
    IPPrefixLen:
      type: "integer"
    MacAddress:
      type: "string"
    PortMapping:
      type: "string"
    Ports:
      type: "array"
      items:
        $ref: "#/definitions/Port"

The spec for inspecting a container gives an example NetworkSettings with many more properties than are defined in the NetworkConfig spec that supposedly defines it:
NetworkSettings:
  Bridge: ""
  SandboxID: ""
  HairpinMode: false
  LinkLocalIPv6Address: ""
  LinkLocalIPv6PrefixLen: 0
  SandboxKey: ""
  SecondaryIPAddresses: null
  SecondaryIPv6Addresses: null
  EndpointID: ""
  Gateway: ""
  GlobalIPv6Address: ""
  GlobalIPv6PrefixLen: 0
  IPAddress: ""
  IPPrefixLen: 0
  IPv6Gateway: ""
  MacAddress: ""
  Networks:
    bridge:
      NetworkID: "7ea29fc1412292a2d7bba362f9253545fecdfa8ce9a6e37dd10ba8bee7129812"
      EndpointID: "7587b82f0dada3656fda26588aee72630c6fab1536d36e394b2bfbcf898c971d"
      Gateway: "<IP_ADDRESS>"
      IPAddress: "<IP_ADDRESS>"
      IPPrefixLen: 16
      IPv6Gateway: ""
      GlobalIPv6Address: ""
      GlobalIPv6PrefixLen: 0
      MacAddress: "02:42:ac:12:00:02"

The properties in the example should be added to the NetworkConfig definition. This should be mostly straightforward, with a couple exceptions. The Networks property will follow the same spec from ContainerSummary:
Networks:
  type: "object"
  additionalProperties:
    $ref: "#/definitions/EndpointSettings"

I'm not sure about the two null properties from the example: SecondaryIPAddresses and SecondaryIPv6Addresses. Are they lists of strings? Some object?

ping @sanimej @aboch PTAL

This seems to be fixed with commit b98ceb7121c5513c73d69081737ad824384c5391. Do you plan to backport the fix to previous versions? I'm currently working on updating the used docker-php version in an app and want to be compatible with API v1.25, though the incomplete NetworkSettings is blocking this.


Add tutorials section

Description
How can the docs be improved?
Right now we have getting-started guide and concepts section to get familiar with the project for end users but it would be more fruitful if they can understand to what potential our project could be extended.
We can do this by adding tutorials section where we can cover some uncommonly used parameters and put out the stylish (ofcourse for the better ) usecases of it.
By this way maybe we can make our project more adaptable and lovable

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Was there a specific tutorial you were thinking that would be useful here?
Since there's such a large dimensionality of use-cases in Kubernetes, we've found it better to host some examples of Karpenter CRDs in https://github.com/aws/karpenter/tree/main/examples and let users pick which features in the concepts section seem interesting to their use-case.
It might be cool to have a Drift tutorial, but that would be identical to the getting-started guide with one extra kubectl edit provisioner call at the end.

Ya there are good examples and often work as good templates for the users to use them directly and modifying for their usecase . I just thought like if we can have a tutorials section where we describe a scenario and show how exactly to leverage certain features of karpenter

Hey @sadath-12. We have limited resources to maintain our documentation, and try to keep it as focused an accurate as possible. The more content we add, the harder it is to maintain. We recommend folks write blog posts for this kind of content, rather than adding to the maintainership burden.


Create keycloak realm when creating address space in api server

At present, the keycloak realm of an address space is created by the address space controller in the realm controller. The historical reason for this is that the keycloak realm needed information about the address space console for redirection and auth.
Now that console auth is using RBAC, there is no need to create the realm from the address space controller anymore, and it can be created in the api server when creating the address space.

Sort of contradicting myself here after thinking more about this.
Moving AddressSpace to a CRD would become more difficult with this approach, as we would not be able to “intercept” the request.
Therefore I would think maybe a better approach to solve the “realm does yet exist” problem, now that console
Auth is out of the way, would be to share the same realm for multiple address spaces and use some sort of user and group prefixing.


Add project to maven repository

The pom should be modified and the project added to the maven repository.
See http://central.sonatype.org/pages/requirements.html

+1 for this:)

Thanks for your up votes of this issue, I aim at adding adding it to the maven repository during the coming week, where I have vacation.

+1

What happened to this? It'd be awesome to have it available on Maven Central. Right now the project is only accessible through JitPack.

@Hervian I can not find in  maven central repository

Is this project added to Maven Central yet? I see Hervian's comment  in 2016 about doing it in 2016

Thank you very much!


Deleting debts is odd

Clicking delete button next to debt doesn't always delete that one.
Pressing enter, clicks the delete button!! (Why)

fixed. blazor coding problem with loops. needed to use local variable buffer, not just a loop variable.


Move the list of YAML tests runned with darwin under src/darwin/Frame…

…work/templates/tests.js
Problem
The list of tests and the order of tests from src/darwin/Framework/CHIP/cluster-tests.zapt is hard to read and parse.
Change overview

Move the list into a js function

#fixes #10722
Testing
The only thing that has changed is the order and the way the list of tests is populated.

Fast track rationale: refactor existing code, affects tests only


Minor update to UG

Codecov Report

Merging #238 (3a69ad5) into master (dc44797) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #238   +/-   ##
=========================================
  Coverage     76.20%   76.20%
  Complexity      885      885
=========================================
  Files           130      130
  Lines          2564     2564
  Branches        302      302
=========================================
  Hits           1954     1954
  Misses          512      512
  Partials         98       98

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 dc44797...3a69ad5. Read the comment docs.


Update appveyor.yml

:white_check_mark: Build VTEX-SDK-dotnet 2.3.472 completed (commit https://github.com/guibranco/VTEX-SDK-dotnet/commit/e0cecf039a by @guibranco)


Exposed malloc_size on MacOSX

malloc_size is available in the OSX libc, but not exposed by rust libc. This change exposed it.
This is my first PR to libc, or to any rust library. So please be liberal with feedback.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Amanieu (or someone else) soon.
Please see the contribution instructions for more information.

Abandoning now that I found the contribution instructions.


Excessive egress data (gcp cloud storage) after update from 0.34.0 to 0.35.0 or 0.36.0

Thanos version: 0.34.0 - bitnami/thanos:0.34.0-debian-11-r3
Object Storage Provider: Google Cloud Storage
What happened: After update the thanos version the egress network data increase dramatically.  I log all the egress traffic in Microsoft sentinel and there I see a lots of requests to storage.googleapis.com with any version greater than 0.34.0
What you expected to happen: Egress traffic similar to 0.34.0 version
How to reproduce it (as minimally and precisely as possible): I installed thanos with the community helm chart from bitnami, in the images attached below shows how the egress data drops after rollback from 0.35.0 to 0.34.0

As the bot mentioned, can you try --block-discovery-strategy=recursive ?

Hi, thanks, I'll try. Which services needs this flag? I'm running compactor, query and storage gateway

Hi, thanks, I'll try.
I added it (in the compactor and in storage gateway) but I still see high egress traffic. To be precise, the charts in gcloud dashboards looks fine, but I still see lots of requests on sentinel...
Number of request to storage.googleapis.com per version in 30 minutes:

0.34.0: 5
0.36.1: 6000

Compactor and storage gateway

Do you use defaults for the discovery loops intervals?

Hello @MichaHoffmann, I tried with the flag --store.sd-interval in the querier service, but I still see a high number of request in my sentinel...
It's a different flag? Thanks for your time


Fixed ApplicationService request URL in case of empty path (root folder)

Hi Marius and Tomasz,
I found out that the new functions (get, delete, create, exists) don't handle properly the root folder under the Application tree (since this case the path is empty).
I've modified the functions and tested the functionality.
Cheers,
Adam

Thanks for the MR. Much appreciated!


Feature Request: Flag to Only Update Files With Comment Block

Is there a flag / option to only add a TOC if there's a comment block?

I suppose placing a block at the bottom of the file is the only way to ensure it won't get generated when you assign a dir of files.

I actually have a PR out (#68) that would let you do this with a bit of ack/ag magic. @thlorenz just needs to merge and push an npm update.

Does your pr do <-- doc toc skip -->  not sure if thats ideal. I think I'd rather not write comments for each doc. I'd like to see an explicit flag.
doctoc **/md.js --explicit

And only docs that have the
<!-- START doctoc -->
<!-- END doctoc -->

or
<!-- START doctoc generated TOC please keep comment here to allow auto update -->
<!-- DON'T EDIT THIS SECTION, INSTEAD RE-RUN doctoc TO UPDATE -->

<!-- END doctoc generated TOC please keep comment here to allow auto update -->

will be written to.

So my PR actually just lets you pass in a list of files to run doctoc on. If you want to exclude specific files, you can add 'DOCTOC SKIP' in comments to the files to exclude and run
doctoc $(ack -L 'DOCTOC SKIP')

but if you want something like --explicit, you can use -l (n.b.: lowercase)
doctoc $(ack -l 'START doctoc')

Of course, you can feel free to use any other program instead of ack that lets you search for files and list their names (ag, pt, grep) but ack is what I know so you'll have to consult the docs for the other programs.

Closing as duplicate of #66

@jez you just blew my mind, can't wait to give this a try.

I'm running ack -L 'DOCTOC SKIP' | xargs doctoc like the readme mentions. It's not updating at all.
ack -L 'DOCTOC SKIP' | xargs doctoc

then I get back:
Unknown option(s): -
Usage: doctoc [mode] [--entryprefix prefix] [--notitle | --title title] [--maxlevel level] <path> (where path is some path to a directory (e.g., .) or a file (e.g., README.md))

Available modes are:
  --bitbucket	bitbucket.org
  --nodejs	nodejs.org
  --github	github.com
  --gitlab	gitlab.com
  --ghost	ghost.org
Defaults to 'github.com'.

So it seems like the -L that is passed to ack is also being passed to doctoc, and it's erroring out thinking the -L was meant for itself maybe?

Could you give the output of just
ack -L 'DOCTOC SKIP'

?

Company Wiki/Certificate-Generator.md
Company Wiki/cheatsheets/typescript.md
Company Wiki/Code-Guidelines - add to .md
Company Wiki/Code-Guidelines.md
Company Wiki/example-files/openssl.cnf
Company Wiki/Git-Guidelines.md
Company Wiki/official-domains-and-accounts.md
Company Wiki/products.md
Company Wiki/README - add to.md
Company Wiki/Set-Up-Dev-Environment.md

Since your filenames have spaces in them, it's causing 'xargs' to mess up.
Perhaps try this:
ack --print0 -L 'DOCTOC SKIP' | xargs -0 doctoc

Returned an error. Good to know though. I guess the easiest path would just get rid of spaces and use dashes
ack --print0 -L 'DOCTOC SKIP' | xargs -0 doctoc
fs.js:907
  return binding.stat(pathModule._makeLong(path));
                 ^

Error: ENOENT: no such file or directory, stat 'Company\ Wiki/Certificate-Generator.md'
    at Object.fs.statSync (fs.js:907:18)
    at Object.<anonymous> (/usr/local/lib/node_modules/doctoc/doctoc.js:103:17)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.runMain (module.js:605:10)
    at run (bootstrap_node.js:420:7)
    at startup (bootstrap_node.js:139:9)

(As a general principle, I try to avoid spaces in filenames whenever they're not necessary, as things tend to break less frequently.)

Looks like you've found a bug in doctoc, relating to spaces in filenames. Feel free to open a separate issue.

Okay, will do. I'll most likely just get rid of spaces and use dashes. Thank you though for the pointers. First time ever using xargs

As @reggi mentioned earlier there is a difference between skipping files that have something in them and only updating files that contain something (START doctoc and END doctoc in this case). It is the difference about implicit vs. explicit effects.
Stating that you can use an additional (shell) tool for that other task of filtering the file, is not considering that most of the options are not as OS independent as doctoc is, meaning I need to prepare different command line script files or npm run scripts for each OS to support, just to not affect files that somebody might add.
So I don't agree with this being a duplicate to the skip option.
The question (to @jez or @thlorenz is, if I would invest time to provide a PR for this flag, does this have some chance of being added?
(I don't care so much about the name, I like explicit but update-only or something on that lines might be more easy to understand.)
Best Chrisian

Here's a quick one-liner I wrote to do this:
grep -rl --include '*.md' "START doctoc generated TOC" . | xargs doctoc


Add tcpudp plugin

Add plugin to use tcp_input without any extra parsing.

Lets go ahead and split this into two plugins


GEODE-1615: gfsh unable to destroy region that has an hyphen in its name

Added a test for whether or not the region contains a hyphen in the name.  If it does, it is necessary to wrap the region name within quotes.

the fix looks good. Where is the test?

I'm doing the test in a separate checkin once I have a final decision on what characters are valid.  Right now, it appears only alphanumeric, an underscore, and a hyphen are the only valid characters allowed.  Yet we can create regions with all sorts of characters.

For now, add at least a test that would cover deleting a region that has "-" and "_" in it. Later, you can expand your tests that would have all sorts of allowed characters.

Closing this.  The test worked the first time, but subsequent runs consistently fail.  I'll start a fresh pull request once I resolve it.


OnCustomProcessing in CU1380, Batch Processing Mgt.

In CU1380, Batch Processing Mgt. MS has created a great new event called OnCustomProcessing with a handled parameter.
My issue is that MS has used this event themselves in CU1371, Sales Batch Post Mgt. Rendering the event nearly completely useless since we cannot predict which subscription will be executed first and all events might set or reset the handled parameter in a different way.
I would like to propose MS either moves their code in function HandleOnCustomProcessing to a hook instead of an event subscriber or create a new event before the OnCustomProcessing event in CU1380.
Similar issues exist with the events OnVerifyRecord and OnBeforeBatchProcessing in CU1380.

This issue seems to have been resolved. Closing issue.


fix: allow deserializing UiAccount rent epoch > u64

Problem
allows deserializing larger max rent epoch values
closes #2950
Summary of Changes
deserialize rent epoch as u128 before clamping to u64::MAX

I'd also like to back port this fix to 1.16.* crates if possible

Sorry @jordy25519 but what exactly is the motivation here? Everywhere in the source, rent_epoch is a u64. Also, we can't backport to 1.16. The oldest branch we could backport to would be 2.0, and you'd have to make a case for the backport.

not sure right channel to broadcast to RPC providers and get this fixed as it shouldn't happen.
I assumed some intermediate infra is re-serializing the value incorrectly.
for completeness, more examples:

C# client issue: https://github.com/magicblock-labs/Solana.Unity-Core/issues/49
-blog post from RPC provider with bad value used in all examples: https://blog.syndica.io/understanding-solana-rpc-a-comprehensive-guide-for-developers/
bad response from getblock.io

Hey @jordy25519 sorry for delay here.
While I can appreciate the fact that rentEpoch is deprecated and, as such, using min(value, u64::MAX) doesn't really have much implication, I think a better approach would be to find the root of the problem.
When trying to reproduce some of the linked issues (https://github.com/magicblock-labs/Solana.Unity-Core/issues/49 and https://github.com/gagliardetto/solana-go/issues/172), using the RPC from curl gives me the correct value for rentEpoch: u64::MAX.
I'm going to move conversation to the issue for now, to see if we can narrow down where this issue might be rooted.


Empty repo?

Hi,
Why has the app been moved to duckduckgo/android-search-and-stories? Does this mean that there will be a new app?

yeah looks like the app is moved to https://github.com/duckduckgo/android-search-and-stories


TASK: Add a detailed documentation on how to run the flow-debugproxy in docker containers

Thanks, the next step will be to provide an official docker image, to make the setup a bit more easy ;)


Update bulma to latest version

Great package, could you upgrade bulma to 0.7.* ?
Thanks

Done :)

Thank you!


runtime/syntax/vim.vim: fixes for vimAugroupList

Similar issue, for when merging / picking this: https://github.com/vim/vim/issues/2450

Started based on this issue:
The second comment in the following code is not highlighted as a comment, but vimFilter vimAugroup:
augroup foo
  if 1  " comment
  endif
  if !1  " comment
  endif
augroup END

Codecov Report

Merging #4465 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4465      +/-   ##
==========================================
+ Coverage   80.49%    80.5%   +<.01%
==========================================
  Files         110      110
  Lines      143220   143220
==========================================
+ Hits       115283   115295      +12
+ Misses      27937    27925      -12

Impacted Files
Coverage Δ

src/netbeans.c
27.25% <0%> (-0.23%)
:arrow_down:

src/gui.c
58.51% <0%> (-0.11%)
:arrow_down:

src/term.c
73.86% <0%> (+0.05%)
:arrow_up:

src/ex_cmds.c
82.28% <0%> (+0.09%)
:arrow_up:

src/ex_cmds2.c
87.91% <0%> (+0.11%)
:arrow_up:

src/gui_gtk_x11.c
49.16% <0%> (+0.34%)
:arrow_up:

src/if_xcmdsrv.c
84.2% <0%> (+0.71%)
: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 7dd64a3...a4f4d62. Read the comment docs.


New release timeline

Subject of the issue
I'd like to inquire about a timeline for a new release, as we'd be very interested in being able to take advantage of 7a5e2c1aaf46800c7a81f8bb908fd94907dd4ca1.

It will be released in 3.11, but no promises on when exactly.
I understand the inconvenience of pointing to a ref in your Gemfile, and sorry for that.


Adding support for Sklearn linear regression in EvaDB

Starting this PR for integrating sklearn linear regression for EvaDb.

Currently the integration test is failing with the following error:
TypeError: Singleton array array('rental_price', dtype='<U12') cannot be considered a valid collection.
============================================================================================= short test summary info ==============================================================================================
FAILED test/integration_tests/long/test_model_train.py::ModelTrainTests::test_sklearn_regression - evadb.executor.executor_utils.ExecutorError: Singleton array array('rental_price', dtype='<U12') cannot be considered a valid collection.

Seems like something is wrong with return types somewhere. Need to debug further.

Unit test is passing with these changes.


Customize heatmap size

Thanks for this amazing categorical correlation matrix using Cramer's V and Theil's U! I'm working in Jupyter Notebook and the heat map is very small and I can't read all the categories. I'm learning on the mushroom dataset and want to plot all features, like you did in your article.
My code:
column_names = ['class','cap-shape','cap-surface','cap-color','bruises?','odor','gill-attachment','gill-spacing','gill-size','gill-color','stalk-shape','stalk-root','stalk-surface-above-ring','stalk-surface-below-ring','stalk-color-above-ring','stalk-color-below-ring','veil-type','veil-color','ring-number','ring-type','spore-print-color','population','habitat']

import pandas as pd
url = 'https://archive.ics.uci.edu/ml/machine-learning-databases/mushroom/agaricus-lepiota.data'
df = pd.read_csv(url, header=None, names=column_names)
df.drop(columns=['veil-type'], inplace=True)

from dython import nominal
nominal.associations(df, theil_u=True, nominal_columns='all')

Is there a way to display the heat map larger in Jupyter Notebook? Thanks for creating this!

Hi, yes - you can pass figsize=(10,10) (or any other numbers) to associations. You can also pass seaborn heatmap's annot and fmt parameters.

Awesome, thank you!

Duplicate of issue #11


Create a demo PR how to extend ApiScraper for API Testing

Based on https://github.com/MovingBlocks/Terasology/issues/2317#issuecomment-359257325:

Extend the ApiScraper to output a list of all packages, classes and (this is new) all methods with their signatures
Create one class with a main method to manually write this output to something like an API-File
Create an automated unit test which calls the api-scraper and then compares to output against the stored api-file.

Goal is to have an automated validation of all api classes and their signatures in the engine where the file can be re-generated manually if we release a new version. This does not have to be feature-complete in the first version but we need an example how it would look like. In the best case we can use the PR directly for API testing.

Closing as completed by #3253 :-)
Submitted three follow-up issues: #3320, #3321, #3322


Check for Microshift

Type of change

[ ] Refactor
[x] New feature
[ ] Bug fix
[ ] Optimization
[ ] Documentation Update

Description
Attempting to fix prom queries w/ microshift

Related Tickets & Documents

Related Issue #
Closes #

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please describe the System Under Test.
Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

It seems that OpenShift only relabels the node_cpu_seconds_total query and that requires the check in QueryNodeCPU() however the container_cpu_usage_seconds_total instance isn't relabeled and has the IP address instead.
I will just change my MicroShift prometheus config to use IP addresses only rather than the hostname. Appreciate the support @jtaleric!


collection time out

It showing collection time out error

CPU memory load very less but we are getting timeout for collection.
3.How to find what power shell modules are used in backend for collecting these metrics,Can you find the solution for this.

Hi @pechettibhaskar,
First a sidenote: I see that you are getting the events every five seconds. Note that this is a pretty intense scrape frequency, and you should evaluate if you really need it. It will increase the impact of your montoring significantly over the Prometheus default of 60s.
For your questions - what is your scrape_timeout configuration in Prometheus? The message will be logged if we were not able to complete a scrape before this period expires. WMI queries can take a few seconds to complete.
There are no Powershell modules involved in collecting the metrics. It is just Go code.

Hi @carlpett ,
How can set at scrape_timeout level in at expoter level, i have seen scrape_timeout in Prometheus server level. I want set in Prometheus expoter level. In the above I did nt set any scrape_timeout level, that is working on default >> 1 .  For exe file: .\wmi_exporter-0.9.0-amd64.exe --collectors.enabled="os,cpu,cs,dns,logical_disk,memory,net,process,service,system,tcp,iis,mssql" 2.  For msi file : msiexec /i C:\Users\Administrator\Downloads\wmi_exporter-0.9.0-amd64.msi  ENABLED_COLLECTORS=os,cpu,cs,dns,logical_disk,memory,net,process,service,system,tcp,iis,mssql LISTEN_PORT=9182 <<

Hi @pechettibhaskar

"Seconds to subtract from the timeout allowed by the client. Tune to allow for overhead or high loads."

You can set --% EXTRA_FLAGS="--scrape.timeout-margin=20" to 20 seconds.

The scrape timeout is set on the Prometheus server, the exporter cannot tell Prometheus to have a longer timeout. You can tune the timeout per job in Prometheus configuration, however, if that helps, see scrape_config docs.
Could you check the value of the metric scrape_duration_seconds for the wmi_exporter job in your Prometheus server?
@alekssako That config would mean that the exporter would try to get results in 20 seconds less than the timeout. With the default 10 second timeout, that would mean that we'd abort immediately.

@carlpett yes but is possible to tune scrape.timeout-margin based on prometheus scrape_config for high server loads.

@carlpett @alekssako We are not using the Prometheus server. We are taking the URL data of the exporter.
We are getting the timeout error after adding the scrape-timeout

error:
extfile.go:222"
WARN[0013] Collection timed out, still waiting for [logical_disk net os service system textfile cpu cs]  source="exporter.go:186"
ERRO[0013] Error reading textfile collector directory "C:\Program Files\wmi_exporter\textfile_inputs": open C:\Program Files\wmi_exporter\textfile_inputs: The system cannot find the path specified.  source="t
extfile.go:222"

As I mentioned in my previous reply, setting the suggested scrape.timeout-margin will effectively set the timeout to zero, which will immediately time out, unless you also pass a header with a larger timeout value.
But you are scraping the data and inserting it into some other system, without using Prometheus at all? That is an interesting use-case, but will make it harder for us to support.

Hello, @carlpett, As you said that I have tried by keeping the --scrape.timeout-margin=0.5, 20, 60, 120. Still, we are getting the timeout errors. The below image is the outputs of different Scarpe-timeout-margins.

We are taking the data of the exporter URL, in the Prometheus server also we don't get data in the exporter URL(http://ip:9182/metrics). Due to this timeout issue.

I understood that if I increase timeout it will work, but I am unable to figure out how to increase the timeout value. Can you provide the full command of that?
Suggest me a way to get the data without using scrape.timeout-margin.

@pechettibhaskar you can add collectors one by one to check from which one you are getting timeout. (or leave it blank to see whether the defaults do not give timeout)

@alekssako Thank you for a suggestion
when I giving this command = .\wmi_exporter-0.9.0-amd64.exe --collectors.enabled="[defulat],iis" , I getting data.

But when i given the complete command = .\wmi_exporter-0.9.0-amd64.exe --collectors.enabled="os,cpu,cs,dns,logical_disk,memory,net,process,service,system,tcp,iis,mssql" i am getting collection time out.

@pechettibhaskar The timeout configuration is on the Prometheus server side. What does your job configuration for the exporter look like there?
From looking at your screenshots, it seems all WMI-based collectors are timing out, while the ones that succeed are perflib-based. This suggests that something is not working with the WMI subsystem on your machine, could you try running Get-WmiObject Win32_ComputerSystem in Powershell?

Closing for lack of response. If this is still an issue, please update and we can reopen.


test: add some variants of tpc-h queries

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
The original TPC-H queries do not produce too many results, i.e. 0~20 rows per second, with low CPU and memory utilization.
Thus, it is hard to track and demonstrate the performance of the multiway join of Risingwave. And it is also hard to be a legit demo for POCs. This file only includes the queries that we intend to run (queries with a lot of joins): Q2, Q5, Q7, Q8, Q9, Q20, Q21.
Therefore, we remove all the filter conditions to increase the computation workload and memory usage.
Since it will be used for daily performance test.
I intend to put these modified queries into the frontend test to make sure nothing changed silently.

Checklist For Contributors

[x] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation

[x] My PR DOES NOT contain user-facing changes.

Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note

If the predicate is of the form XXX = constant or XXX between constant1 and constant2, then I removed all of them.
Nothing else gets changed


Policy config on EdgeOS (Edgerouter X)

Hey Folks, thanks for creating this great piece of software! 😄
I've been trying to get ctrld working on an Edgerouter X (running firmware 2.0.9-hotfix.7). All 2.x.x releases use Debian 9 (Stretch).
I have a pretty basic config which I would expect to work, however, it seems like the policy rules are not being considered at all, and all traffic is being sent to upstream.0. Here's the config that I've been using:
[service]
  log_level = "info"
  log_path = "/var/log/ctrld.log"

[listener]
  [listener.0]
    ip = '<IP_ADDRESS>'
    port = 5354

    [listener.0.policy]
      name = 'Main Policy'
      networks = [
        {'network.0' = ['upstream.0']},
        {'network.1' = ['upstream.0']}
      ]
      macs = [
        {'b0:be:83:2d:c6:50' = ['upstream.1']}
      ]

[network]
  [network.0]
    name = 'LAN'
    cidrs = ['<IP_ADDRESS>/25']

  [network.1]
    name = 'LAN-IoT'
    cidrs = ['<IP_ADDRESS>/27']

[upstream]
  [upstream.0]
    name = 'Cloudflare'
    type = 'dot'
    endpoint = '<IP_ADDRESS>:853'
    timeout = 5000
  [upstream.1]
    name = 'SmartDNS Proxy MTL'
    type = 'legacy'
    endpoint = '<IP_ADDRESS>'
    timeout = 5000

Looking at the config, I would expect DNS requests from both networks to go out upstream.0 and for the specified mac address to go out on upstream.1. However, when testing this from the machine with the mac address specified, and looking at the ctrld logs, there are never any requests hitting upstream.1.
Here's are snippets from the logs:
$ head /var/log/ctrld.log
{"level":"info","time":"2024-01-19T16:30:13Z.757","message":"starting ctrld v1.3.3"}
{"level":"info","time":"2024-01-19T16:30:13Z.767","message":"os: linux 4.14.54-UBNT"}
{"level":"info","bootstrap_ip":"<IP_ADDRESS>","time":"2024-01-19T16:30:14Z.584","message":"using bootstrap IP for upstream.0"}
{"level":"info","bootstrap_ip":"<IP_ADDRESS>","time":"2024-01-19T16:30:14Z.585","message":"using bootstrap IP for upstream.1"}
{"level":"warn","time":"2024-01-19T16:30:14Z.627","message":"no default route IP found"}
{"level":"info","time":"2024-01-19T16:30:14Z.629","message":"starting DNS server on listener.0: <IP_ADDRESS>:5354"}
{"level":"info","time":"2024-01-19T16:30:21Z.229","message":"[fbf13a] QUERY: <IP_ADDRESS>:39005 (gatekeepr) -> listener.0: A detectportal.firefox.com"}
{"level":"info","time":"2024-01-19T16:30:21Z.234","message":"[f84262] QUERY: <IP_ADDRESS>:48816 (gatekeepr) -> listener.0: PTR lb._dns-sd._udp.<IP_ADDRESS>.in-addr.arpa"}
{"level":"info","time":"2024-01-19T16:30:21Z.748","message":"[d907b4] QUERY: <IP_ADDRESS>:48816 (gatekeepr) -> listener.0: PTR lb._dns-sd._udp.<IP_ADDRESS>.in-addr.arpa"}
{"level":"info","time":"2024-01-19T16:30:21Z.908","message":"[f84262] REPLY: upstream.0 -> <IP_ADDRESS>:48816 (gatekeepr): NXDOMAIN"}

After making a request from machine with mac addrs b0:be:83:2d:c6:50, I don't see any hits on upstream.1:
cat /var/log/ctrld.log | grep upstream.1
{"level":"info","bootstrap_ip":"<IP_ADDRESS>","time":"2024-01-19T16:30:14Z.585","message":"using bootstrap IP for upstream.1"}

Instead of using the macs policy block, I also tried to use networks: [] instead, and use the source IP of the machine, i.e. <IP_ADDRESS>/32 but the results are no different. It seems that the policy evaluation doesn't happen at all?
ctrld is obviously working and routing all the traffic to the default upstream.0 however, so this is very strange. I've even checked to make sure that /etc/dnsmasq.d/dnsmasq-zzz-ctrld.conf is being created when ctrld is active, and everything seems to check out fine!
Hoping for a bit of help here, has anyone else seen this when testing on their device? Is it the config? Am I missing something? I can provide more detailed logs if that helps!
Thanks and much appreciated!

Just wanted to add that the mac address was obtained from the client list available through ctrld clients list so discovery is working correctly!

@jbialy How did you make the request to the router? Did you send request to router port 53 (dnsmasq) or you send requests directly to router port 5354 (ctrld)?

The request is coming from a wifi connected machine (on the <IP_ADDRESS>/25 subnet) which uses the Edgerouter as it's default gateway and also as the DNS resolver, i.e. <IP_ADDRESS>/32.
So, the request would be hitting the router on port 53 (dnsmasq) I would presume, and then get passed to ctrld on 5354?
Side note that I do have two nameservers set as part of DNS forwarding config on the Edgerouter:
show dns forwarding nameservers
-----------------------------------------------
   Nameservers configured for DNS forwarding
-----------------------------------------------
<IP_ADDRESS> available via 'system'
<IP_ADDRESS> available via 'system'
<IP_ADDRESS> available via 'ppp pppoe5'
<IP_ADDRESS> available via 'ppp pppoe5'

However, when ctrld is running, those don't get hit, I've verified this by looking at the Cloudflare status page and with ctrld running, I get Using DNS over TLS (DoT)set to YES, while without ctrld I only seeConnected to <IP_ADDRESS>` (legacy mode).

@jbialy Could you please run with debug log?

@cuonglm, I ran a quick test using the debug level. Please see the attached log, ctrld.log.1.gz.
Again, most of the request in this log are coming from the machine with mac b0:be:83:2d:c6:50, the rest are coming from either network.0 and some likely from network.1.

The client in question is {"level":"debug","time":"2024-01-19T17:24:45Z.776","message":"found hostname: \"Januszs-MacBook-Air\", ip: \"<IP_ADDRESS>\" via mdns"}. This debug log doesn't indicate its mac address but the given client should be matching the mac policy set for 0:be:83:2d:c6:50.

Is there a way to tell from the debug logs which policies are loaded? If they are at all loaded? As in, that the [listener.0.policy] block is being processed?

Does the ctrld clients list command show you the MAC address in the table? if not, then it wasn't discovered and therefore the policy cannot match. I took your config, modified it slightly like so:
[service]
  log_level = "debug"
  log_path = "/tmp/ctrld.log"

[listener]
  [listener.0]
    ip = '<IP_ADDRESS>'
    port = 53

    [listener.0.policy]
      name = 'Main Policy'
      networks = [
        {'network.0' = ['upstream.0']},
        {'network.1' = ['upstream.0']}
      ]
      macs = [
        {'00:0c:29:4a:5c:57' = ['upstream.1']}
      ]

[network]
  [network.0]
    name = 'LAN'
    cidrs = ['<IP_ADDRESS>/25']

  [network.1]
    name = 'LAN-IoT'
    cidrs = ['<IP_ADDRESS>/27']

[upstream]
  [upstream.0]
    name = 'Cloudflare'
    type = 'dot'
    endpoint = '<IP_ADDRESS>:853'
    timeout = 5000
  [upstream.1]
    name = 'Control D'
    type = 'legacy'
    endpoint = '<IP_ADDRESS>'
    timeout = 5000

client list table looks like this
test@test-virtual-machine:~$ sudo ctrld clients list
+-------------+----------------------+-------------------+------------+
|     IP      |       Hostname       |        Mac        | Discovered |
+-------------+----------------------+-------------------+------------+
| <IP_ADDRESS>   |                      | 00:50:56:9f:0e:84 | arp        |
| <IP_ADDRESS> | test-virtual-machine | 00:0c:29:4a:5c:57 | dhcp       |
| <IP_ADDRESS> |                      | 74:56:3c:44:eb:5e | arp        |
| <IP_ADDRESS>   | test-virtual-machine | 00:0c:29:4a:5c:57 | dhcp       |
+-------------+----------------------+-------------------+------------+

Notice the MAC address for localhost matches what's in the policy
{"level":"info","time":"2024-01-19T15:48:24-05:00.300","message":"[db1794] QUERY: <IP_ADDRESS>:33166 (test-virtual-machine) -> listener.0: A verify.controld.com"}
{"level":"debug","time":"2024-01-19T15:48:24-05:00.300","message":"[db1794] Main Policy, 00:0c:29:4a:5c:57, no rule -> [upstream.1]"}
{"level":"debug","time":"2024-01-19T15:48:24-05:00.300","message":"[db1794] sending query to upstream.1: Control D"}
{"level":"info","time":"2024-01-19T15:48:24-05:00.301","message":"[db1794] REPLY: upstream.1 -> <IP_ADDRESS>:33166 (test-virtual-machine): NOERROR"}
{"level":"debug","time":"2024-01-19T15:48:24-05:00.301","message":"[db1794] received response of 71 bytes in 1.813484ms"}

@yegors, thanks for testing this out and providing the config for me to try. What's interesting is that in your logs, I'm seeing Main Policy, 00:0c:29:4a:5c:57, no rule -> [upstream.1]"} and later sending query to upstream.1: Control D". That's explicit and clear that the policy filtering is working.
I've taken the config you posted and adjusted only the port number 53 -> 5354 so that dnsmasq can forward to ctrld. I can definitely see the client:
$ ctrld clients list | grep Janusz
| <IP_ADDRESS>              | Januszs-Air                          | b0:be:83:2d:c6:50 | arp,dhcp,mdns |

However, I'm still not seeing the policy being applied on b0:be:83:2d:c6:50, i.e. no explicit policy matched, using default routing -> [upstream.0]. I have no idea why except that it has something to do with the way the DNS query requests get passed to ctrld on EdgeOS. The dnsmasq auto generated conf file looks like this:
$ cat /etc/dnsmasq.d/dnsmasq-zzz-ctrld.conf
# GENERATED BY ctrld - DO NOT MODIFY
no-resolv
server=<IP_ADDRESS>#5354
max-cache-ttl=0

However, cross referencing this against what's in ConfigContentTmpl I'm wondering if I'm missing the add-mac and add-subnet=32,128 directives?
It looks to me that these two directives are set based on the condition determined in UpstreamSendClientInfo(). Where I figure that control-d must be set as one of the upstreams and be of type DOH/DOH3, or that ctrld is set to use NextDNS?
This is something that I should be able to easily verify.

Ah, indeed that was the culprit. Having one of the upstreams set to be control-d and use DoH3 has enabled the correct dnsmasq configuration.
...
[upstream]
  [upstream.0]
    name = 'Cloudflare'
    type = 'dot'
    endpoint = '<IP_ADDRESS>:853'
    timeout = 5000
  [upstream.1]
    name = 'Control D OISD'
    type = 'doh3'
    endpoint = 'https://freedns.controld.com/x-oisd'
    timeout = 5000
...

cat /etc/dnsmasq.d/dnsmasq-zzz-ctrld.conf
# GENERATED BY ctrld - DO NOT MODIFY
no-resolv
server=<IP_ADDRESS>#5354
add-mac
add-subnet=32,128
max-cache-ttl=0

And policy filtering is now working:
"level":"info","time":"2024-01-20T01:59:19Z.547","message":"[eab9ac] QUERY: <IP_ADDRESS>:47144 (Januszs-Air) -> listener.0: A az764295.vo.msecnd.net"}
{"level":"debug","time":"2024-01-20T01:59:19Z.548","message":"[eab9ac] Main Policy, b0:be:83:2d:c6:50, no rule -> [upstream.1]"}
{"level":"debug","time":"2024-01-20T01:59:19Z.549","message":"[eab9ac] including client info with the request"}
{"level":"debug","time":"2024-01-20T01:59:19Z.550","message":"[eab9ac] sending query to upstream.1: Control D OISD"}

Thanks for diving in, this appears to be a bug as that config should totally contain those 2 params. Will investigate further.

@jbialy You have to set send_client_info = true for non ControlD/nextdns upstream: https://github.com/Control-D-Inc/ctrld/blob/main/docs/config.md#send_client_info

Thanks @cuonglm, I think setting send_client_info = true makes sense for when needing to pass the mac information to the upstream but it shouldn't be required for enabling the two configuration directives in dnsmasq-zzz-ctrld.conf (i.e. add-mac and add-subnet=32,128).

Yes, we are going to fix this in next release.

Yea was going to say. Nothing in the docs for send_client_info suggests the current behavior. Local policy should logically work regardless of this setting.

This issue is resolved in v1.3.4 - the above flag is no longer required.


Fixed incorrect link

Link for 'built-in policies' was pointing to 'custom policies'.

@parakhj - Would you review the proposed changes? Thanks

#sign-off


Image not loaded when using ng-repeat with filter

I am witnessing an issue that prevents me from using lazy-image on a scrollable container with query text that filters a list:

                    {{item.text}}

The result that I am seeing, is that after entering a search query, the list shortens but the images (that are now in viewport) are not loaded until I manually perform a scroll.
Thank you for any input.

I've seen this happen as well... no solution at the moment.
+1

Can you add your sample to the sample repo? The current list shows now:
http://afklm.github.io/ng-lazy-image/sample-repeat.html

I can reproduce the issue (Mac/Chrome) with the repo's sample (http://afklm.github.io/ng-lazy-image/sample-repeat.html) by adding an option to display all as the default.
<option value>All</option>

Default to "all", all items display, do not scroll. Select filter 1, then filter 2. On one or the other filter will have some of the images near the bottom (at least on my screen, depending on your setup you may have to change the sizes or number generate a bit) will not load until you scroll.
It appears that if the span element that holds the image is added to the dom but not displayed (viewed), then the image is never loaded. The test as written passes because all of the spans are never added at the beginning. I don't know if that makes sense, but I'm going to try to fix it.

Inject $rootScope into the afklLazyImage directive.

Update the code around $on('$destroy' to read...
// Remove all events when destroy takes place
scope.$on('$destroy', function () {
$rootScope.$broadcast('resize');
return _eventsOff();
});
scope.$on('resize', _onResize);

This lets any child of root scope know that a lazy image was removed and maybe it should check its position.

Thanks, so we can do like this:
scope.$on('$destroy', function () {
    _checkIfNewImage();
    _onViewChange();
    return _eventsOff();
});

I see, on scope destroy i remove the image if it is there (in our case only ui-view destoyed the view). Then 300ms later the plugin looks if it has new images... So i need the timer here as well. Thanks Awalker!

Not sure if we are talking about the same thing here. Consider a page with a blue image and green image. The green image is far enough down the page that it is not show (and not loaded). Filtering the destroys the blue image. Invoking _onViewChange on the destroyed blue image does not solve the problem, with or without a timer. The problem is the green image's wrapper is now on screen from DOM manipulation without a resize or scroll event being fired. We need some way to notify the green image wrapper to load the green image.
Short video showing the issue
DOM MutationObserver exists, but I'm not sure how well supported it is. Broadcasting from the $rootScope works, but feels like a hack. I'm wondering if I could tap into ngRepeat animation support to achieve this goal. ngRepeat has support for a "move" animation event and that seems like a good place to look but it could require a dependency on ngAnimation.

Thanks Awalker. First i wanted to explain that our coding conventions disallowed us to misuse rootScope but since most of the concepts will not be there in 2.0 i decided for now it fits the goal which is using it inside a ng-repeat. Thanks a lot for your clear explanation!

It still doesn't solve other cases where you toggle other elements since it still doesn't see the change, but at least they can now use a trigger :)


Downloaded video files are empty (0 bytes)

When using downloadAllVideosFromUser, the created files are all empty.
I tried with different usernames, and also with a cookies.txt file, but I'm getting the same results.
There are no errors on the console.
EDIT: After some debugging, it turns out that the URLs return a 403 status code.
Any ideas how to get around this?
Let me know if I can help with any further details.

Are you using a VPN?

Using the fetchVideo method, I get downloadUrl and when I try to go to the address, I get a 403 response. Perhaps I somehow download the resource wrong, who in the topic show your implementation in the documentation of the solution did not see.

Is there a dependence on the region? If there is a vpn with which region should I use?

Using the fetchVideo method, I get downloadUrl and when I try to go to the address, I get a 403 response. Perhaps I somehow download the resource wrong, who in the topic show your implementation in the documentation of the solution did not see.

I was able to reproduce this issue and you are right. I will be looking into it. Thanks for the feedback

When using downloadAllVideosFromUser, the created files are all empty. I tried with different usernames, and also with a cookies.txt file, but I'm getting the same results. There are no errors on the console.
EDIT: After some debugging, it turns out that the URLs return a 403 status code. Any ideas how to get around this?
Let me know if I can help with any further details.

I guess this has something to do with the tiktok api endpoint that the library is using to get the videos. I guess they have added extra authentication to the links. I still did not look into it but from your feedback I guess this is the case. I will look into it and give you updates. Thanks for the feedback

Is there a dependence on the region? If there is a vpn with which region should I use?

As far as I know, you do not need vpn to access specific videos on tiktok so no

Any updates here by chance?

I used this: https://www.npmjs.com/package/follow-redirects
const tik = require("tiktok-scraper-ts");
const fs = require('fs');
const { http, https } = require('follow-redirects');
(async () => {
const video = await tik.fetchVideo("https://www.tiktok.com/@sbmowing/video/7227955368297647402?is_from_webapp=1&sender_device=pc",true);
https.get(video.downloadURL, resp => resp.pipe(fs.createWriteStream(./videos/${video.id}.${video.format})));
})();
Dont know if im doing it right, but in my case it works.


Not able to assign different aria-label to different handles

As the documentation of noUISlider:
https://refreshless.com/nouislider/slider-options/#section-handle-attributes
noUiSlider.create(slider, {
    start: [10, 120],
    handleAttributes: [
        { 'aria-label': 'lower' },
        { 'aria-label': 'upper' },
    ],
    range: {
        'min': 0,
        'max': 100
    }
});

It should be possible to assign different labels to the handles separately by passing a array

But recently the parameter aria of vueform slider is not supporting.
Where if we take a look on the source code:
if ((ariaLabelledby && ariaLabelledby.value) || (aria && Object.keys(aria.value).length)) {
  let handles = Array.isArray(value.value) ? value.value : [value.value]

  defaultOptions.handleAttributes = handles.map(h => (Object.assign({}, aria.value, ariaLabelledby && ariaLabelledby.value ? {
    'aria-labelledby': ariaLabelledby.value,
  }: {})))
}

aria is always passed as an object to defaultOptions.handleAttributes, which make such action becoming not possible.

Although developers can also try passing slider option with options parameter, it is not possible in this case.
While with the watcher:
watch(options, refresh, { immediate: false, deep: true })

Making the slider refreshing everytime when user dragging the controls and becoming unusable.

Possible solution:

Make aria to accept both object and array
Then have different action base on the type

Seems this repo is not being maintained..
For anyone else encountering same problem, you may try my version which applied #89
https://www.npmjs.com/package/@oliver139/vueform-slider

Seems this repo is not being maintained.. For anyone else encountering same problem, you may try my version which applied #89 https://www.npmjs.com/package/@oliver139/vueform-slider

can you provide an example how you are using it?
I'm currently setting:
:aria-labelledby="[type + '_min', type + '_max']"
but there are no ids referenced so lighthouse still throws an accessible names error. The aria-labelledby are applied.


proof-of-innocence WalletPOI.init in startRailgunEngineForPOINode

proof-of-innocence uses startRailgunEngineForPOINode, but it does not initialize WalletPOI, which engine requires in v6/v7

looks as if its implemented in latest


fix(render): Allow to render to multiple documents

Fixes #1178.

Warnings

:warning:
There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

  Generated by :no_entry_sign: dangerJS


Fixed: fails to generate with resources

Without this fix script fails:
[2015-12-11 16:44:33] local.ERROR: exception 'ErrorException' with message 'trim() expects parameter 1 to be string, resource given' in /home/nikolay-zakharov/PhpstormProjects/webhosts/project/vendor/barryvdh/laravel-ide-helper/src/Method.php:274
To reproduce:
https://gist.github.com/nikolay-zakharov/df3760693593df9dea3c

Thanks


Links to Markdown files

Would you consider an option to convert all links to Markdown files to HTML files? We're using Markdown for our documentation, linking to other Markdown files that are part of this. It would be useful if we could convert these links when generating HTML files.

Use a custom renderer for links.

If this custom renderer already exist, I will take it ;)

Closing as resolved. See #956


Generated updateShouldNotify

Is your feature request related to a problem? Please describe.
Currently, there is no consistent way to tell Riverpod if a value has changed. (or rather, how it should tell if it changed)
This is especially relevant for Lists and Sets
Describe the solution you'd like
Add an annotation field to tell Riverpod whether the value has changed.
For example (suggested by remi himself in discord)
@Riverpod(updateShouldNotify: function)

Describe alternatives you've considered
Directly overriding updateShouldNotify in generated classes, which does not have a function alternative
Additional context
This is useful for when our data is an object that does not implement value equality, or if devs would prefer to only compare specific values, for whatever reason.
This would allow us to avoid having to custom-implement == and hashcode for every object we need

Currently rivepod using identical(a,b) from dart:core to detect change.
But for some class, we expect to compare some attribute to decide equality. This could save some comparison computation between complex object values.
I wonder if we could in introduce a hook to modify updateShouldNotify() behavior for each provider at runtime

Currently rivepod using identical(a,b) from dart:core to detect change. But for detecting changes of values from some specific class, we expect to compare few attributes to decide equality, not all. This also could help to save some comparison computation between complex object values. I wonder if we could in introduce a hook to modify updateShouldNotify() behavior for each provider at runtime

I believe it is using ==

This use case is also relevant for me. For StateNotifier it is using identical(a,b) to determine if the value has changed. In some cases, I'm generating complex state, which if it changes can trigger the rebuild of large trees of widgets. So it is far less computationally expensive if it would be possible to customize operator == (or tell the provider some custom function to use for comparison).

You can override updateShouldNotify when using the class syntax. That should be good enough


Add promise resolution type

:+1:

:+1:  - feel free to merge once you checked the one feedback i have.


sources(curl): download multiple URLs with the same curl command

This PR is based on the excellent work of @supakeen in https://github.com/osbuild/osbuild/pull/1549 and the findings in https://github.com/osbuild/osbuild/pull/1023
Simon found that curl has a lot of fork/exec overhead in the way we use it. The other issue with our current use of curl is that we do not reuse connections and the overhead of the connection setup (especially for https) is there. So this branch uses a hybrid approach, it will spawn a number of workers and these workers will split the files evenly between them and download multiple files in parallel. The downside of this approach is of course that if a worker happens by chance to get multiple big files in their work partition the overall result may be slower (unless --parallel is utilized, see below).
Modern curl (7.68+) has a --parallel option that is even better but that is not available on RHEL8 so the code has fallbacks.
The numbers I see are relatively modest though, for me it's mostly I/O bound as my connections is not that fast maybe someone with a fast connection can also test this? I ran:
$ rm -rf .osbuild/ ; time python3 -m osbuild  --libdir . ./test/data/manifests/fedora-container.json
``
and see broadly the same total time except that the "user" time is significantly lower.

Even modest improvement is improvement. Can you rebase this and fix tox.ini so I can approve? :)

Remove the stale label for now, I still think this is a good idea. When there is time :)

I feel like; while it is useful for its performance improvements over the current curl stage, it seems to make the curl stage a lot more complex with all the setup, configuration files, and all these things as compared to an approach that doesn't go through a curl binary.
Would it perhaps be a good idea to instead use libcurl through pycurl, this is likely shipped in RHEL and gets rid of the subprocesses.

Sources run outside the build root if I recall correctly. You are right that this would apply cleanly to old manifests as it improves the existing curl source while only new manifests could use differently named sources.

Interesting, httpx is comparable to curl on my setup. :thinking: I think I'm still in favor of this PR, it will provide a very interesting self-contained speed improvement. Alternatively, we can focus on finishing https://github.com/osbuild/osbuild/pull/1304 which will bring support for handling mirror fallbacks properly.
old curl
7.72user 3.37system 1:26.58elapsed 12%CPU (0avgtext+0avgdata 30384maxresident)k
344inputs+0outputs (31major+542192minor)pagefaults 0swaps
new curl
1.68user 1.38system 0:44.46elapsed 6%CPU (0avgtext+0avgdata 30780maxresident)k
0inputs+0outputs (0major+12908minor)pagefaults 0swaps
new curl with tweaks (parallel-immediate)
1.97user 1.51system 0:47.14elapsed 7%CPU (0avgtext+0avgdata 38396maxresident)k
0inputs+0outputs (0major+17992minor)pagefaults 0swaps
httpx
8.82user 1.93system 0:51.83elapsed 20%CPU (0avgtext+0avgdata 82976maxresident)k
32inputs+1264outputs (0major+24259minor)pagefaults 0swaps

Interesting, httpx is comparable to curl on my setup. 🤔 I think I'm still in favor of this PR, it will provide a very interesting self-contained speed improvement. Alternatively, we can focus on finishing #1304 which will bring support for handling mirror fallbacks properly.
old curl
7.72user 3.37system 1:26.58elapsed 12%CPU (0avgtext+0avgdata 30384maxresident)k 344inputs+0outputs (31major+542192minor)pagefaults 0swaps
new curl
1.68user 1.38system 0:44.46elapsed 6%CPU (0avgtext+0avgdata 30780maxresident)k 0inputs+0outputs (0major+12908minor)pagefaults 0swaps
new curl with tweaks (parallel-immediate)
1.97user 1.51system 0:47.14elapsed 7%CPU (0avgtext+0avgdata 38396maxresident)k 0inputs+0outputs (0major+17992minor)pagefaults 0swaps
httpx
8.82user 1.93system 0:51.83elapsed 20%CPU (0avgtext+0avgdata 82976maxresident)k 32inputs+1264outputs (0major+24259minor)pagefaults 0swaps

Is this on the same manifest? I'd assume the percentage overhead of what needs to be done for curl to be larger the faster the pipe becomes.
And yes, this PR is still a nice speed up.

Is this on the same manifest? I'd assume the percentage overhead of what needs to be done for curl to be larger the faster the pipe becomes.

I'm not sure which manifest you used. I used the fedora-boot one. Not sure if that matters a lot, it's good to have multiple data points I think. :)

Is this on the same manifest? I'd assume the percentage overhead of what needs to be done for curl to be larger the faster the pipe becomes.

I'm not sure which manifest you used. I used the fedora-boot one. Not sure if that matters a lot, it's good to have multiple data points I think. :)

fedora-boot :)

They will only pull in curl (the binary) in the buildroot(AIUI?)

Sources run on the host before osbuild sets up a buildroot or bubblewrap containers.
Anything used by a source becomes a dependency of osbuild.

What if we add the ability for some pipelines to do networking so we can have a conceptual netroot as well ;)
/me runs away.

Had to do another rebase/conflict resolution because of https://github.com/osbuild/osbuild/pull/1809

There's a fixup! commit in the PR, otherwise LGTM.

There's a fixup! commit in the PR, otherwise LGTM.

Sorry for that! I removed that one now and it should be good again.


config: add annotation for exec isolated CPU affinity

The runc container runtime recently introduced functionality to direct exec operations to the first isolated CPU in a container cgroup cpuset. This is enabled through a new runc specific container annotation: org.opencontainers.runc.exec.isolated-cpu-affinity-transition
See https://github.com/opencontainers/runc/commit/afc23e3397 for the details.
This new functionality is also required in the crun container runtime and will be added soon. This purpose of this commit is to add a new annotation to the OCI runtime specification that would be used by any runtime that implements this new functionality:
org.opencontainers.runtime.exec.isolated-cpu-affinity-transition
The runc implementation should switch to using the new annotation once it is accepted.

Agree -- a runtime-specific annotation seems like a really great way for one runtime to experiment with adding functionality like this before it's fully standardized or proven out, but as soon as it's something desirable for interoperability, it seems worth making a proper "feature" instead.


Optimization 5&6 - Add exists(String file) and copy(String source, String dest) methods to FileBackend

Implement exists(String file) and copy(String source, String dest) methods as defined in https://github.com/perfectsense/gyro/issues/315 and https://github.com/perfectsense/gyro/issues/316

PR: https://github.com/perfectsense/gyro-google-provider/pull/196


Marketplace

I just downloaded phpStorm & am trying to change the code colors.  When I look in the plugin Marketplace, Rainglow is not listed.
I went to File: Settings: Plugins: Marketplace.
Typed Rainglow in search box.
[No results for this plugin.]
Have I missed a step?

@btn-ninja I had the same issue, using the semi-manual instructions from the readme I was able to install the themes.

Same for JetBrains IntelliJ IDE and Plugins marketplace. It's gone :'(

Importing the .jar crashed my PHPstorm 2020.2 twice.

Is there any update on this issue? I would love to see this plugin in the marketplace again.


set test_suite to tests to allow python setup.py test

Fixes #183.

This was the answer to ‘How to run unittest discover from “python setup.py test”?’.


LINEAR_TICKET_ID_REGEX should support multiple length of identifier

The current Linear identifier naming rule can support 1 ~ 5 characters.
However, the regex in synclinear.com is hard-coded by three characters.
https://github.com/calcom/synclinear.com/blob/b0277194852f583b43f4d6397dd13fd50104d077/utils/constants.ts#L65
In this case, if the linear team's identifier is not precisely three characters, it will confuse synclinear and create a duplicated linear issue when the users try to create a linear story with Public label.

Closed by #157


deviceCategoryDisplayName output mismatch with Intune UI for Unassigned devices

Version : 5.1.19041.1023
SchemaVersion : beta
Microsoft.Graph.Intune : 6.1907.1.0
Command :
Get-IntuneManagedDevice -Filter "contains(deviceName, 'AndroidforWork')" | Where-Object { $_.deviceCategoryDisplayName, 'Unknown' }
Expected Output: Current "AndroidforWork" devices falling into category "Unassigned".
Obtained Output: Current "AndroidforWork" devices falling into category "Unassigned" + Any other "AndroidforWork" device which belonged to category "Unassigned" upon a time.
That means, basically is not taking into account any manual category modification done by IT personnel.
From exactly where is getting that info? Those devices currently belong to other categories!.
Any idea? Thanks a lot! :-)

Hi community!
Can anyone take a look on this, please?

Sadly it doesn't look like there's anyone keeping an eye on this repo - I've had a codefix I've been trying to push for a year now.
Could you try:
(Get-IntuneManagedDevice -Filter "contains(deviceName, 'AndroidforWork')")[1].deviceCategoryDisplayName
You might need to change the index to find a device with another category than unassigned - but I expect you'll get multiple values. If you do, you can do this:
Get-IntuneManagedDevice -Filter "contains(deviceName, 'AndroidforWork')" | where-object {
    ($_.deviceCategoryDisplayName|measure-object).count -eq 1
 }

The logic is that all devices will have unassigned but any other categories applied will have unassigned+something, making it greater than 1. If it still returns everything, the property deviceCategoryDisplayName might be a string with newlines separating the elements, in which case, $_.deviceCategoryDisplayName.split("`n") should make an array of it.

Thank you so much for your assistance. I´ve tried with your suggestion but it´s still showing any device which was "Unassigned" at any time, and not only the current ones.
Anyway, this seems to be working for me;
Get-IntuneManagedDevice -Filter "contains(deviceName, 'Android')" | Where-Object {$_.deviceCategoryDisplayName, 'Unknown' }
Now is filtering properly.


cannot run this project in Linux

This project runs well in windows x64, but not in Linux arm64, maybe some reference doesn't support Linux arm64.

Are you getting some specific error or could you elaborate on what does not work?

Are you getting some specific error or could you elaborate on what does not work?

sorry, my fault, it works in linux, only one error about the colorpicker.js


Change Composite Writer to Sequential Writer

Which problem is this PR solving?

#929

Short description of the changes

As part of the necessary changes to add the Ingester, the Composite Writer will fail at the first error when writing spans as opposed to continuing with the next storage type.
Consider the case where the second storage type is not idempotent, and the first storage type fails to process a span. With a Composite Writer, on every retry, the second storage will have redundant spans written to it. With a Sequential Writer, the second storage type won't be attempted until the write to the first storage type succeeds.

Consider the case where the second storage type is not idempotent, and the first storage type fails to process a span. With a Composite Writer, on every retry, the second storage will have redundant spans written to it. With a Sequential Writer, the second storage type won't be attempted until the write to the first storage type succeeds.

What about the second of three writers fail?

This way you'll preserve the old opportunistic behavior.

Do we require this opportunistic behavior?
I like the name SequentialWriter or even SequentialFailFastWriter better because it is more descriptive. While CompositeWriter is implemented sequentially, the name itself doesn't allude to that behavior (A writer with the same name may write parallely for e.g.)

Changing the CompositeWriter was found to be unnecessary to complete the Ingester. In addition, the use case of a SequentialWriter is not yet well defined


crypto/sha/asm/sha512-ia64.pl: When checking assembler file names, ignore case

The use case is that uppercase .ASM extension may be used on some platforms,
and we were only testing for the lowercase extension.

... approvd if tests passed ...

Yeah well, GitHub Actions won't verify that, and VMS builds are only done on push (because much too slow for PRs), so you'll have to take my word for it that my manual builds compile that assembler fine with this change.

Is this causing a CI failure somewhere? I assume with VMS. If so agree urgent.

Merged
master:
119b7b5f2ad7efcf273f395e7633747f56ff3f95 crypto/sha/asm/sha512-ia64.pl: When checking assembler file names, ignore case
3.1:
733084ae065038e5ffb892fad1999cd028d3abdd crypto/sha/asm/sha512-ia64.pl: When checking assembler file names, ignore case
3.0:
22530d31789b57b2818ff28cbb95b6472b45c2d0 crypto/sha/asm/sha512-ia64.pl: When checking assembler file names, ignore case

And hopefully, there aren't more nits like this.  Buildbot will tell...


Combining results from multiple samples

With the updated database and scripts, I was able to get the relative abundance (Sample1_filtered_hits_eukfrac.txt) of different eukaryotes in my metagenome samples. What is the best way to combine my results from different samples ( I have 60 samples) into one so I can use various visualization tools?

Check out the newly added combine_rpks_eukfrac.py script in the eukdetect/ directory. Please reach out with any questions or issues.


fields generated out of order

What version of CUE are you using (cue version)?

$ cue version
cue version v0.8.2

go version go1.21.5
      -buildmode exe
       -compiler gc
     CGO_ENABLED 1
          GOARCH amd64
            GOOS linux
         GOAMD64 v1

Does this issue reproduce with the latest stable release?
yes
What did you do?

Cue File:
_environments: [
    "0",
    "1"
]

stages: [
	"lint",
	"diff",
	"apply"
]

".k8s-workloads-image": {
 	image: name: "registry.gitlab.com/gitlab-com/gl-infra/ci-images/k8s-workloads:1.2.0"
}

for k, _ in _environments {
	"\(k):dryrun": {
	  extends: ".k8s-workloads-image"
	  resource_group: "\(k)"
	  script: """
	make gen
	"""
	  stage: "dryrun"
	}
}

$ cue export test.cue --out yaml

What did you expect to see?
stages:
  - lint
  - diff
  - apply
.k8s-workloads-image:
  image:
    name: registry.gitlab.com/gitlab-com/gl-infra/ci-images/k8s-workloads:1.2.0
0:dryrun:
  extends: .k8s-workloads-image
  resource_group: "0"
  script: make gen
  stage: dryrun
1:dryrun:
  extends: .k8s-workloads-image
  resource_group: "1"
  script: make gen
  stage: dryrun

What did you see instead?
stages:
  - lint
  - diff
  - apply
0:dryrun:
  extends: .k8s-workloads-image
  resource_group: "0"
  script: make gen
  stage: dryrun
.k8s-workloads-image:
  image:
    name: registry.gitlab.com/gitlab-com/gl-infra/ci-images/k8s-workloads:1.2.0
1:dryrun:
  extends: .k8s-workloads-image
  resource_group: "1"
  script: make gen
  stage: dryrun

Thanks for confirming the details, @Awadabang. This is a duplicate of https://github.com/cue-lang/cue/issues/474.
Capturing your test case for completeness':
exec cue export --out yaml x.cue
cmp stdout stdout.golden

-- x.cue --
_environments: [
	"0",
	"1",
]

stages: [
	"lint",
	"diff",
	"apply",
]

".k8s-workloads-image": {
	image: name: "registry.gitlab.com/gitlab-com/gl-infra/ci-images/k8s-workloads:1.2.0"
}

for k, _ in _environments {
	"\(k):dryrun": {
		extends:        ".k8s-workloads-image"
		resource_group: "\(k)"
		script: """
			make gen
			"""
		stage: "dryrun"
	}
}
-- stdout.golden --
stages:
  - lint
  - diff
  - apply
.k8s-workloads-image:
  image:
    name: registry.gitlab.com/gitlab-com/gl-infra/ci-images/k8s-workloads:1.2.0
0:dryrun:
  extends: .k8s-workloads-image
  resource_group: "0"
  script: make gen
  stage: dryrun
1:dryrun:
  extends: .k8s-workloads-image
  resource_group: "1"
  script: make gen
  stage: dryrun

This fails with 819223954d3b for both the new and old evaluator.
Closing this as a dupe because it is fully covered by #474.


@ringcentral/sdk-utils not in npm registry

@ringcentral/sdk-utils not in npm registry
What to do?

What's the use case? You normally don't need this package, it's used for test/build configs.

I just cloned the repository and run npm install. And i received this error. The package is in package.json list.
By the way, did you integrate ringcentral with an react application?

Yes, there's a dedicated package for that. You should run npm install in the root of the cloned repo, lerna will link all packages. May I know what are you trying to do and why you cloned the repo? It's not needed if you only need to use it, not contribute.

I just want to integrate ringcentral into an blank react application, to have possibility to make/receive calls

Just follow the readme then, you don’t need to clone the repo.


Add cache detection for old AMD processors

In this pull request I added cache info detection for old AMD processors. We can get information about the TLB, but I don't know how to process this information as there is a difference in leaves, there is information for 4KB, 2MB, 4MB, 1GB pages, so I have defined these fields as UNDEF so far

@gchatelet, @Mizux, please review

rebased, @gchatelet, @Mizux, please review PR

@gchatelet, this PR has been here for a long time, please review or is this inappropriate PR and should we close it?

Looks much better thx!


Missing model.resblock BottleneckBlock implementation

It seems like in prefixLM.py, there is an import statement for BottleneckBlock from a file called resblock.py which is not in the model directory. Will you be adding this file to the repository? Thank you in advance!

thanks for your notification. I have added it.  if there exists any other problem, please let me know.

Thank you for the rapid response and upload! I really appreciate it.


SPARK-22021

What changes were proposed in this pull request?
This adds a new feature transformer which will take a javascript function and based on the logic provided in that, derives new features.
How was this patch tested?
There is proper unit test written for this transformer and also it's been testing in local and yarn-client mode.
Please review http://spark.apache.org/contributing.html before opening a pull request.

Can one of the admins verify this patch?


Use github terminology consistently

This PR it NOT backward compatible and users might have to update their hubtty.yaml configuration file to match the new option names. This is hopefully the last non-backward compatible change to hubtty.

Rename change to pull request:
We've inherited the change terminology from gertty. While descriptive
enough, we should be using the github terminology consistently and
adopt pull request.

Rename project to repository:
Project has a different meaning in Github than it has in Gerrit.

Let's merge this. This seems fine with my limited testing, merging this will to test more broadly before we cut a release.


Cert error while trying to access https://inspector.pypi.io/

Problem: Current certificate on inspector.pypi.io is invalid. This site uses HSTS this way you cannot bypass an exception in chrome / edge / firefox without disabling HSTS, but it is very insecure.
Firefox:
Websites prove their identity via certificates. Firefox does not trust this site because it uses a
certificate that is not valid for inspector.pypi.io. The certificate is only valid for the following
names: *.ingress.cmh1.psfhosted.org, test.pypi.org, upload.pypi.org, *.cmh1.psfhosted.com,
*.pyfound.org, *.ingress.cmh1.psfhosted.com, *.cmh1.psfhosted.org

This is resolved. The necessary SAN on the certificate was dropped unexpectedly when the cert auto renewed, but is now restored and a monitor has been setup for the endpoint.


Set up conductor-python releases to be published with Orkes Account

Resources:

email<EMAIL_ADDRESS>https://groups.google.com/my-groups

User: https://pypi.org/user/orkes/
Email<EMAIL_ADDRESS>


[fs] forbidden path error when using fs plugin to watch changes

I'm currently implementing the FS plugin to monitor changes in a specified folder. I’ve added the "watch" feature for the FS plugin in Cargo.toml and included the capabilities fs:allow-watch and fs:allow-unwatch. However, I'm encountering a forbidden path error. How can I solve this?

You need to add that path to the allow list
Use this to allow that path for watch
{
  "identifier": "fs:allow-watch",
  "allow": [
    {
      "path": "your-path"
    }
  ]
}

or this to allow all fs operations to that path
"fs:allow-watch",
{
  "identifier": "fs:scope",
  "allow": [
    {
      "path": "your-path"
    }
  ]
}

Can you help me with this?

There's a config if you need to do all paths and including dot files on unix systems
Put this in your tauri config
"plugins": {
  "fs": {
    "requireLiteralLeadingDot": false
  }
}

Or use $DESKTOP/local-repo/.git/refs/heads/** instead of ** if this path is fixed and you only need to watch this path

🎉🥳 It worked! A huge thank you to @Legend-Master for the help!
For anyone looking for this setting in the future, here’s the link: Tauri Configuration Reference - fsScope.


New Player 20dfca59 has broken YouTube extractor

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

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

Checklist

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

Region
No response
Provide a description that is worded well enough to be understood
A new player (20dfca59) has broken the YouTube extractor
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
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [f0993391e] (pip)
[debug] Python 3.12.3 (CPython AMD64 64bit) - Windows-11-10.0.22631-SP0 (OpenSSL 3.0.13 30 Jan 2024)
[debug] exe versions: ffmpeg N-116328-gfa5a605542-20240718 (setts), ffprobe N-116328-gfa5a605542-20240718
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.07.04, mutagen-1.47.0, requests-2.32.3, sqlite3-3.45.1, urllib3-2.2.2, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets
[debug] Loaded 1829 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:tab] Extracting URL: https://music.youtube.com/playlist?list=OLAK5uy_kOExsg4A5Wc7X3_DhtK4GB0YqdeFIIjhs
WARNING: [youtube:tab] YouTube Music is not directly supported. Redirecting to https://www.youtube.com/playlist?list=OLAK5uy_kOExsg4A5Wc7X3_DhtK4GB0YqdeFIIjhs
[youtube:tab] OLAK5uy_kOExsg4A5Wc7X3_DhtK4GB0YqdeFIIjhs: Downloading webpage
[debug] [youtube:tab] Extracted SAPISID cookie
[youtube:tab] OLAK5uy_kOExsg4A5Wc7X3_DhtK4GB0YqdeFIIjhs: Redownloading playlist API JSON with unavailable videos
[download] Downloading playlist: Album - Meteora 20th Anniversary Edition
[youtube:tab] Playlist Album - Meteora 20th Anniversary Edition: Downloading 89 items of 89
[download] Downloading item 1 of 89
[youtube] Extracting URL: https://music.youtube.com/watch?v=d_iv0n9ZpAg
[youtube] d_iv0n9ZpAg: Downloading webpage
[debug] [youtube] Extracted SAPISID cookie
[youtube] d_iv0n9ZpAg: Downloading ios player API JSON
[youtube] d_iv0n9ZpAg: Downloading ios music player API JSON
[youtube] d_iv0n9ZpAg: Downloading web music client config
[youtube] d_iv0n9ZpAg: Downloading player 20dfca59
[youtube] d_iv0n9ZpAg: Downloading web music player API JSON
[debug] [youtube] Extracting signature function js_20dfca59_107
[debug] Loading youtube-sigfuncs.js_20dfca59_107 from cache
WARNING: [youtube] d_iv0n9ZpAg: nsig extraction failed: Some formats may be missing
         n = A1Ou24bVhXz1jIVy ; player = https://www.youtube.com/s/player/20dfca59/player_ias.vflset/en_US/base.js
[debug] [youtube] Unable to extract nsig function code (caused by RegexNotFoundError('Unable to extract \x1b[0;94mInitial JS player n function name\x1b[0m; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U')); please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
WARNING: [youtube] d_iv0n9ZpAg: nsig extraction failed: Some formats may be missing
         n = -fLCxedkAk4LUTK2 ; player = https://www.youtube.com/s/player/20dfca59/player_ias.vflset/en_US/base.js
[debug] [youtube] Extracting signature function js_20dfca59_103
[debug] Loading youtube-sigfuncs.js_20dfca59_103 from cache
WARNING: [youtube] d_iv0n9ZpAg: nsig extraction failed: Some formats may be missing
         n = lfFDQouqFm1Pmkfk ; player = https://www.youtube.com/s/player/20dfca59/player_ias.vflset/en_US/base.js
WARNING: [youtube] d_iv0n9ZpAg: nsig extraction failed: Some formats may be missing
         n = N6FLr39qpAOmT5Ak ; player = https://www.youtube.com/s/player/20dfca59/player_ias.vflset/en_US/base.js

yeah I noticed an issue where mp4's download with no sound I assume this is related

I'm getting the same error

The referenced branch fixes it. You can wait for it to be merged to the main/nightly or download the YouTube.py and replace the one you have in extractors.

I'm not using the pip version I'm using the exe so i have to wait until it's merged.

I'm getting the same error

The referenced branch by bashonly fixes it. You can wait for it to be merged to the main/nightly or download the YouTube.py and replace the one you have in extractors.

Tomorrow I Download Videos Have No Problem.
Now After 12;00AM Get This Error.
WARNING: [youtube] vhgzkyCmesw: nsig extraction failed: Some formats may be missing
n = jVeOS8cTQ1N9R_Um ; player = https://www.youtube.com/s/player/20dfca59/player_ias.vflset/en_US/base.js

Any word when a new release will be minted with this fix?

@zackees https://github.com/yt-dlp/yt-dlp/releases/tag/2024.08.01


Highlight range on hover

Great repo first off! 😄
Just wondering if there is support for highlighting the range of dates from the selected startDate to before actually selecting endDate.
Example:
I select 16th as startDate
I hover over to 20th and there is a range 'light highlight' from 17th to 20th
I move over to 22nd and the range 'light highlight' extends to 22nd
I click on 22nd as endDate and the range gets the current implementation of 'dark highlight'

Try these 2  https://github.com/airbnb/react-dates and http://zippyui.com/docs/react-date-picker

Yeah sadly had to switch to react-dates to get that feature support out of
the box.
On Wed, 23 Nov 2016 at 21:40, vladi160<EMAIL_ADDRESS>wrote:

Try these 2 https://github.com/airbnb/react-dates and
http://zippyui.com/docs/react-date-picker
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Adphorus/react-date-range/issues/66#issuecomment-262558718,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHMoDG2_nrKOBax-dI3bSDQGNn802ICQks5rBGWPgaJpZM4JoG9n
.


fix watch&reload for sub folders under src/

when running dev in an ubuntu16.04 docker image (files mounted from a centos host), changes to src/componets/*.vue will not trigger recompilation thus no hotreloading. Adding an include entry solved this problem

Also, I can't really understand how this fixed your problem.

@LinusBorg I'm not sure how this worked either, but if I remove include, hotreloading will only work on app.vue, the rest .vue file in subfolders won't even recompile after been modified. I also tried to increase fs.inotify.max_user_watches, but doesn't seem to work. I think its a docker and webpack2 problem, so don't merge it. Just in case someone has the same problem with docker, try to add an include for src and maybe node_modules  too

Mabe you just need to change the watch mode of webpack-dev-middleware in dev-server.js?
See watchOptions -> Polling
https://github.com/webpack/webpack-dev-middleware

Adding this worked as well
var devMiddleware = require('webpack-dev-middleware')(compiler, {
  publicPath: webpackConfig.output.publicPath,
  quiet: true,
  watchOptions: {
    aggregateTimeout: 300,
    poll: true
  }
})

Can't this be update and merged ?

Well, polling is not a default in webpack-dev-middleware for a reason.
I'm rather thinking about making this an option you can activate in /config


Incompatible with protobuf v4

I know there was some discussion about this but now that #533 is closed without resolution it's unclear what the future holds.
The status has changed slightly since before. Protobuf v27.2 introduced some backwards compatibility with the older gencode, apparently allowing gRPC to function with Protobuf v27.2. However, this backwards compatibility doesn't extend to GeneratedMessageV3$Builder, thus the gencode used in cel-java runs into problems when using Protobuf v27.2. This is because the new shims give source compatibility with old gencode, but old gencode that is compiled against the old runtime will reference classes that still don't exist. It seems like a future version of protobuf may resolve this and offer a cleaner path forward, but it hasn't happened yet. See the discussion over here: https://github.com/protocolbuffers/protobuf/issues/17247
Meanwhile, though, on May 23rd, Protobuf v27 actually gave people a reason to upgrade: Protobuf v27 is the first version of Protobuf that enables Protobuf Editions by default. While Editions is mostly backwards compatible, using Protobuf Editions in protobuf schemas requires new gencode, and new gencode requires the new runtime.
This whole situation is a bit of a mess. With Protobuf v27.3 having been cut a few hours ago with no sign of any improvements to gencode compatibility, we are at a bit of a standstill. I wonder if the gencode compatibility improvements are perhaps good enough that cel-java can update? It will be a breaking change, but at least there shouldn't be a hold up with grpc-java.
Can't imagine this is an exciting issue to have to deal with but it's one that I don't think can be ignored so forgive me for having to raise it.
For people searching, this error can manifest in two ways:

From source code depending on old bytecode gencode with old runtime. This will give a compile-time error: error: cannot access Builder: class file for com.google.protobuf.GeneratedMessageV3$Builder not found
From bytecode depending on old bytecode gencode with old runtime. This will give a class not found error at runtime instead.

(When compiling old gencode from source with Protobuf v27.2, this issue should not occur.)

@jchadwick-buf thanks for your detailed issue report!
We've already seen the "v4 problem" a while ago via #533 and the "JLBP-7 violation" - and it's sad to hear that it's still a promise to have an ABI compatible upgrade path.
Our own use cases depend on the cel-standalone artifact, which has the necessary shaded dependencies.
The biggest issue I've seen when I naively tried to "just bump" to v4 were surprisingly the CEL conformance tests, which also suffered that time from the v4-upgrade issue.
I think, we have to bump protobuf (and gRPC for the conformance tests) to some newer version. Let me give that a try in the next days.

Pre-requisite: #584

Fixed via #588

Thanks for making quick work of this! I can confirm that it's working great so far.

Glad it help!


Error: buzzer_play_sound: mraa_pwm_period_us() failed, while executing UPM code example for Grove buzzer sensor with Firmata

While testing Grove Buzzer sensor connected to NUC with Arduino 101, following error occurred:
"buzzer_play_sound: mraa_pwm_period_us() failed"
Hardware setup: Grove Buzzer sensor connected to NUC (Windriver OS) with Arduino 101.
mraa version: 1.7.0
upm version: 1.3.0
Tested code: https://github.com/intel-iot-devkit/upm/blob/master/examples/javascript/buzzer.js
With following lines added for firmata:
"use strict";
const mraa = require('mraa');
console.log('MRAA Version: ' + mraa.getVersion());
// open connection to firmata
mraa.addSubplatform(mraa.GENERIC_FIRMATA, "/dev/ttyACM0");
Error log:
root@WR-IDP-4993:~# node buzzer.js
MRAA Version: v1.7.0
buzzer_play_sound: mraa_pwm_period_us() failed
/root/buzzer.js:54
console.log( myBuzzer.playSound(chords[chordIndex], 500000) );
^
Error: UPM Runtime Error: playSound: buzzer_play_sound() failed
at Error (native)
at melody [as _repeat] (/root/buzzer.js:54:31)
at wrapper [as _onTimeout] (timers.js:275:11)
at Timer.listOnTimeout (timers.js:92:15)

The mraa_pwm_write_period() function (called by mraa_pwm_period_us()) does not have a replace function in case of firmata. This leads to the error you see here. This is most likely because the pwm period for firmata is fixed. Working on a fix for this.

I am having the same error. Any progress?

Any progress on this issue? Thank you very much!

Hi @deadprogram this should be fixed now with the latest packages for MRAA/UPM. @priyanka-bagade and @malikabhi05 can confirm.


Fix an assertion in the PersistentActor to allow the SurgeEventModel to work

The SurgeEventModel forces publishStateOnly = true and eventWriteFormattingOpt = None since it doesn't publish any events. We have an assertion in the PersistentActor to double check that if we are publishing events the write formatting for events is defined, but it looks like that assertion was a bit off.

Codecov Report

Merging #114 (92676ef) into main (cfc0c5d) will increase coverage by 0.21%.
The diff coverage is 100.00%.

:exclamation: Current head 92676ef differs from pull request most recent head 5adba38. Consider uploading reports for the commit 5adba38 to get more accurate results

@@            Coverage Diff             @@
##             main     #114      +/-   ##
==========================================
+ Coverage   71.71%   71.92%   +0.21%
==========================================
  Files         147      147
  Lines        2814     2814
  Branches       75       75
==========================================
+ Hits         2018     2024       +6
+ Misses        796      790       -6

Impacted Files
Coverage Δ

...a/surge/internal/persistence/PersistentActor.scala
95.65% <100.00%> (ø)

...health/windows/actor/HealthSignalWindowActor.scala
91.95% <0.00%> (-1.15%)
:arrow_down:

...rnal/health/supervisor/HealthSupervisorActor.scala
74.45% <0.00%> (+2.18%)
:arrow_up:

...kafka/streams/KafkaStreamLifeCycleManagement.scala
65.51% <0.00%> (+4.59%)
: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 cfc0c5d...5adba38. Read the comment docs.