Fix tests on nightly Rust

Just some tweaks to compiler error messages

(gonna wait for travis)

r? @huonw
(rust_highfive has picked a reviewer for you, use r? to override)

:pushpin: Commit c25a753 has been approved by alexcrichton

@bors: r+

:hourglass: Testing commit c25a753 with merge 2257fce...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64


Broadcast transactions is a bit buggy

Background
The strangelove-venture's lens breaks when errors are returned when broadcasting a tx. This fixes it. It also adds a couple of more tests as well.
Changes

s/strangelove-ventures/volumefi/g
changed a logic where we do broadcasting
wrote tests to test broadcasting and new logic

Merging as I need to use this in the codebase from the master branch


Nitrous.io "Quickstart" Support

Hi Brunch!
Recently Nitrous.io have added a new feature to their platform named “Quickstarts”. This feature lets users create pre-configured development environments on the service at the click of a button. Nitrous are trying to set up these Quickstarts for users by requesting that projects like yourself host the Quickstart trigger buttons on your GitHub readme.
This would mean that if a Nitrous user wants a dev environment all set up and ready to develop Brunch applications with - they can click the HTML button embedded on your readme, and then the Nitrous API creates one for them.

https://www.nitrous.io/quickstarts/  - repositories so far we've got Quickstarts for.

All that’s needed from yourselves is that you keep the three API configuration files (seen in these proposed changes) inside your repo's root, and also that you have the HTML trigger button embedded in the main README.md - for people to click (this is also included in these changes).
Is this something you’d be up for helping us out with? It would really make using Brunch on the platform super easy.
To be clear there’s not much you have to do on your end, except merge these changes into your master. We (likely me) would make any new pull requests to yourself in the future, to maintain or ensure that these three files are up to date. None of your actual project code on GitHub gets touched or is involved at any point. The files here simply tell the API what to do when the button is clicked by  a Nitrous user.
Here’s some more links that might help explain:

Blog Post Explaining the Idea
Specifically How it Works

Any questions do ask.
Thanks guys.

I've started adding the Nitrous Quickstarts to a couple of my apps (which use Brunch) and it feels like this would be a better fit to add to the skeletons rather than to the main brunch repo.
This example installs three arbitrary? skeletons and lets you start with them which seems a bit odd to me.
Perhaps a brunch-nitrous repo or doc file with instructions on adding Nitrous support to your skeleton with some Brunch specific information (like setting the hostname). To take it a step further the skeletons with support could even have a little button on the skeletons list page.


missing files not flagged in compile

source: should_be_compile_error is table('duckdb:/no/such/file.csv')

This should show an error in the editor window, instead the file compiles file and you get errors when you try and run a query

There is a way for the schema fetcher to return an error, the duckdb dialect needs to do that


update ws to fix security vulnerabilities

there are 2 advisories with the version of ws currently in use:

https://nodesecurity.io/advisories/67
https://nodesecurity.io/advisories/120

they are medium and high vulnerabilities respectively

This will fix #66
I've already run npm test on Mac OSX, will run on Ubuntu Linux when I get home tonight.

Tests also pass on Ubuntu Linux.

Thank you for this. I will look at this shortly.

Thanks for merging! How long until this change is published to npm?

Soon.


Fix: external dns

Fixes the mismatch between the centralized secrets function and the https/ssh getRepoURL function.
Also, updates the translation of the auth secrets to the correct env key name for external dns on a case  by case basis.

sorry that close was a fat finger on mobile


fix: support complex schemas in commit

Codecov Report
Attention: Patch coverage is 0% with 36 lines in your changes are missing coverage. Please review.

Project coverage is 81.08%. Comparing base (aa23b00) to head (78d9627).

Files
Patch %
Lines

rust/lance/src/dataset/transaction.rs
0.00%
36 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2208      +/-   ##
==========================================
- Coverage   81.12%   81.08%   -0.05%
==========================================
  Files         184      184
  Lines       52286    52322      +36
  Branches    52286    52322      +36
==========================================
+ Hits        42417    42423       +6
- Misses       7421     7455      +34
+ Partials     2448     2444       -4

Flag
Coverage Δ

unittests
81.08% <0.00%> (-0.05%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Scrolling causes significant performance drops

Issue Type: Performance Issue
Earlier today I've been having a huge performance drop when scrolling in the editor tab, extension tab, anywhere that scrolls. Whether smooth scroll is toggled or all extensions are disabled, VSCode drops all the way to  ~20fps. I've tried reinstalling and restarting to no avail.
I'd be glad to provide profiling logs if needed although I've noticed something specifically pop up in red a bunch of times which may seem alarming?

 Some of the developer tools, depending on the settings used will reduce the frame rate of the page just by recording the events / stack traces / screenshots / etc.
I personally use this tool to measure frame rate, as it does not impact the frame rate:

It is extremely clear visually, while in the editor, that there is a huge performance drop as I scroll. Here is a gif of me scrolling using what you've suggested.

@ok-nick We have a known problem on Windows that might cause slowness (https://github.com/microsoft/vscode/issues/107016).   Could you please try to close all vscode windows and then launch with code --disable-renderer-accessibility ?

Thank you for trying. Some of the extensions you have installed (e.g. bracket-pair-colorizer-2) might add a lot of text editor decorations which might slow down the editor rendering, depending on your content. Does this reproduce when disabling all extensions?

Thanks for trying. Could you please create a JS CPU profile for the renderer when you are scrolling using these steps?

CPU-20210610T180252.zip

Thank you for the profile. I could not spot anything wrong, it looks like the JS that renders a frame is actually pretty fast on your machine, taking almost always <5ms to render a frame, which should give a great frame rate. IMHO the problem might be somewhere else, not on the renderer process.
@ok-nick To eliminate the problem that this is a GPU / driver issue, could you please also try to close all vscode windows and launch Code.exe --disable-gpu and see if there is any improvement.
Otherwise, this could be a problem with the main process which maybe does not send the user input events to the renderer process fast enough.

The --disable-gpu flag allows it to run perfectly, absolutely no lag

@deepak1556 Any thoughts on how to proceed?

Does this issue repro with latest stable 1.74 ?

It seems to be fixed! 👍

Not sure, we have had a couple of runtime version updates since the bug reported, very likely fixed through them.

The issue still persists. I must've opened it with the --disable-gpu flag which is why I thought it was fixed.

@ok-nick can you check if the issue is present with our exploration build that comes with a newer runtime. Thanks!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


PSNativeCommandPreserveBytePipe hangs on piping to python scripts in Windows

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
I'm trying out the preview feature PSNativeCommandPreserveBytePipe, and it seems to work in most cases but I've noticed it hangs when piping to Python scripts in certain situations. I have both Python 3.9.7 and 3.11.0 installed and the standard python launcher. I have some simple python scripts:
repr.py, which should print the escaped binary contents of stdin:
import sys
print(repr(sys.stdin.buffer.read()).lstrip('b'))

echo.py, which just prints it's arguments:
import sys
print(' '.join(sys.argv[1:]))

I also have cat.exe and echo.exe binaries from Cygwin 64-bit as well as WSL2 (Ubuntu).
The following commands hang with PSNativeCommandPreserveBytePipe enabled, until hitting Ctrl+C:
.\echo.py hello | .\repr.py
echo.exe hello | .\repr.py
wsl echo hello | .\repr.py
'hello' | cat.exe | .\repr.py
'hello' | wsl cat | .\repr.py
[byte[]](1, 2, 3) | cat.exe | .\repr.py
[byte[]](1, 2, 3) | wsl cat | .\repr.py

However, if I prefix the repr.py script with python.exe or py.exe (the python launcher) they work as expected:
> .\echo.py hello | py .\repr.py
'hello\r\n'
> [byte[]](1, 2, 3) | cat.exe | python .\repr.py
'\x01\x02\x03'

Also piping strings or bytes directly from Powershell to the un-prefixed script works:
> 'hello' | .\repr.py
'hello\r\n'
> [byte[]](1, 2, 3) | .\repr.py
'\x01\x02\x03'

So it seems that this only effects native-to-native commands involving the Python launcher launched implicitly via file association as the target of the pipe. I haven't tried using other scripting languages to see if it effects other implicitly launched binaries. I confirmed that this effects both Python 3.9 and 3.11.
Also assoc and ftype show the .py file association as:
> cmd /c assoc .py
.py=Python.File
> cmd /c ftype Python.File
Python.File="C:\WINDOWS\py.exe" "%L" %*

Expected behavior
The expected behavior for the first 5 hanging commands is to print `'hello\r\n'`, and the last 2 to print `'\x01\x02\x03'`.

Actual behavior
The commands hang and nothing happens until I hit Ctrl+C, which stops the command with no output.

Running the commands with PSNativeCommandPreserveBytePipe disabled does not hang and shows the legacy behavior (bytes are passed as line-separated strings).

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.4.0-preview.4
PSEdition                      Core
GitCommitId                    7.4.0-preview.4
OS                             Microsoft Windows 10.0.19045
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

Pipeline transmission of binary data is amazing to me.
Who can tell me the principle. How does it distinguish between binary data or not?
Can we make a common parameter, similar to 【-PipelineBinaryData】,
be used to force a single line script to only pass binary data through a pipeline?

I believe with the preview feature all native-to-native pipelines are treated as binary data, so no distinction is needed. Between Powershell commands and native binaries it depends on the type, so a byte array e.g. [byte[]](1,2,3) is passed as raw binary input. (There are some commands like Get-Content that can return byte arrays with the -AsByteStream flag).

I tried your script only on Linux. It works if repr.py has python shebang and  bit for execution.
#!/usr/bin/env python
import sys
print(repr(sys.stdin.buffer.read()).lstrip('b'))

$ gi ./repr.py

    Directory: ~/Desktop

UnixMode     Num   UID   GID      LastWriteTime    Size Name
--------     ---   ---   ---      -------------    ---- ----
-rwxr-xr-x   755  1000  1000   01.07.2023 07:58      82 repr.py

$ /usr/bin/echo "hello" | ./repr.py
'hello\n'

$ [byte[]](1, 2, 3) | /usr/bin/cat | ./repr.py
'\x01\x02\x03'

Oh probably should have made it clear this is windows. I imagine it is a detail of how programs are executed on windows and not a generic bug.

I can’t try on Windows, there is no python and no wsl. There are also no Unix utilities.
Could you give another example, without python, only with native Windows utilities?

I found that this also effects ruby in the same way. So calling a ruby script like
echo.exe hello | .\script.rb

hangs while
echo.exe hello | ruby .\script.rb

prints expected output.
This confirms it's not specific to Python/the Python launcher, but is an artifact of starting an interpreter via file association.

Ok I think this is a minimal example that hangs out-of-the-box on Windows:
cmd /c echo hi | .\hello.vbs

where hello.vbs is
WScript.Echo "hello"

Note that .vbs by default runs with the WScript binary which is a GUI app rather than console, and displays a windowed message box. With this pipeline it still shows the window but then hangs the terminal indefinitely even after the window is closed. If PSNativeCommandPreserveBytePipe is disabled or it is called like cmd /c echo hi | wscript .\hello.vbs then then it returns control to the terminal without hanging.

With Start-Process it works as expected. Especially unzipped 7.4.0-preview.4
cmd /c echo hi | Start-Process .\Desktop\hello.vbs -ea 0

I think the issue is really exists. The pwsh 7.4.0-preview.4 does not properly lead the file associations with enabled this experimental feature.

PS C:\ProgramData\kasini3000> echo "hi" | cscript.exe a:\pscode\TEMP_2023\temp152\aaa.vbs
Microsoft (R) Windows Script Host Version 5.812
版权所有(C) Microsoft Corporation。保留所有权利。

hello
PS C:\ProgramData\kasini3000>

Write-Output "hi" | a:\pscode\TEMP_2023\temp152\aaa.vbs

normal

hang
cmd /c "echo hi" |  a:\pscode\TEMP_2023\temp152\aaa.vbs

@237dmitry I think that Start-Process just discards the piped data though. Without the "-ea 0" powershell complains that the command doesn't take pipeline input, and modifying the vbs script to read from stdin it gets EOF. Whereas if I call it explicitly with wscript (or cscript) it receives the input.
I also confirmed that the scripts don't hang on startup; they execute ok until they first try to read stdin and then hang, or hang on exit if they never read stdin. Demonstrated with this script:
WScript.Echo "hello"
WScript.Echo WScript.StdIn.ReadLine

I can repro, thanks for the report @rmccampbell! Lookin' into it


fv3fit unit tests are slow

fv3fit unit tests are slow. They take about 10min to run on my Mac and more like 15-20min on CI. Development would be easier if the tests ran faster.
In #2055 I added output for the duration of the slowest tests.
Related: #2040

I'm surprised to see some tests from test_train_novelty_detection in here. I can work to speed or eliminate those.

Many of these test are marked with slow so can be skipped with
pytest -m 'not slow'

This runs in about 80s on my VM. Though coverage drops significantly for certain parts of the code base.

The somewhat ludicrous test time for autoencoder and cyclegan is something we could talk about @oliverwm1. These run much faster on my machine, but apparently only because I have M1 acceleration. Solutions include:

don't worry about it and run the long tests, use not slow for local tests
switch the tests to not check outputs but only test the training code runs for one epoch without exceptions
make this and other longer-running training tests optional, running under CI either periodically, when requested (like with the integration tests), or only manually/locally
some combination of the above

One issue to keep in mind is that the coverage drops a lot when excluding the slow tests (adding a marker for the cyclegan test. Here is the coverage of subdirs:
fv3fit/emulation 90%
fv3fit/pytorch 48%
fv3fit/keras     76%
fv3fit/_shared   87%
fv3fit/data    63%
fv3fit/sklearn 63%

Thanks @mcgibbon @nbren12. Agreed running the 'not slow' tests locally is a good idea for local dev. But as Noah points out we are relying on the slow tests for a lot of our coverage. Can we be more judicious about these tests (i.e. maintain similar coverage with fewer slow tests) or speed them up? I do think the long test time on CI is increasing friction for reviewing/merging PRs but I would also be a bit nervous about only doing slow tests infrequently.

I find almost all of the time a test that only runs the training code but doesn't check how well it trained will catch bugs introduced into the training codes (and have identical coverage, technically). The "skill" tests only need to be run when those training functions are edited, so we could treat them as a trigger and main test similar to the argo integration tests.
I did try hard to decrease the run time of the pytorch training tests, but pytorch is generally very slow on cpu because of how it runs python code every training epoch, and there's only so much we can do about it. The architectures implemented also just need more data/epochs to train.

Would it be worth writing more unit tests @mcgibbon of e.g. the model objects etc? top-level training tests can be hard to make fast, though I agree that they do tend to find bugs easily.

There's a tradeoff with that solution - It's hard to come up with these tests, and if you do rely extensively on unit tests of components of the model building / training functions, it means you will frequently need to refactor or update tests when you're only changing internal implementation details of the model, including style refactors. I don't know if it even can be done in this case, the best I could think of for sub-testing CycleGAN was to include the autoencoder which is already expensive to train.
The best answer may just be "training global models is expensive and tests of this will be expensive".


The "Customizing token validation" sample not working with .Net 5

The "Customizing token validation" section code is not working with .Net 5 . The error is :
Severity	Code	Description	Project	File	Line	Suppression State Error	CS1061	'BinderOptions' does not contain a definition for 'Events' and no accessible extension method 'Events' accepting a first argument of type 'BinderOptions' could be found (are you missing a using directive or an assembly reference?)	NotesModule.WebApi	C:\repos\Kaleido\NotesJS\NotesModule.WebApi\Startup.cs	36	Active
The option object type I use is Microsoft.AspNetCore.Authentication.JwtBearer.JwtBearerOptions.
Could you please update the sample with .Net 5 / 6 code please?

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

ID: 906f1086-776e-9abe-b6cc-bf93e84dc238
Version Independent ID: 50b27c9e-8f8a-3425-a1bb-b5d30e9d95e4
Content: Configure protected web API apps - Microsoft Entra
Content Source: articles/active-directory/develop/scenario-protected-web-api-app-configuration.md
Service: active-directory
Sub-service: develop
GitHub Login: @jmprieur
Microsoft Alias: jmprieur

@cotepatrice
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@cotepatrice :
The code works well in general (we use it all the time)
I think that you have another variable option of type BinderOptions
Use another variable name in the delegate:
services.AddAuthentication(JwtBearerDefaults.AuthenticationScheme)
        .AddMicrosoftIdentityWebApi(Configuration);
services.Configure<JwtBearerOptions>(JwtBearerDefaults.AuthenticationScheme, o =>
{
  var existingOnTokenValidatedHandler = o.Events.OnTokenValidated;
  o.Events.OnTokenValidated = async context =>
  {
       await existingOnTokenValidatedHandler(context);
      // Your code to add extra configuration that will be executed after the current event implementation.
      o.TokenValidationParameters.ValidIssuers = new[] { /* list of valid issuers */ };
      o.TokenValidationParameters.ValidAudiences = new[] { /* list of valid audiences */};
  };
});

#please-close


fix Broken Link for Create and run unit tests with Visual Studio

Fixes #4477

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

Thank you for your contribution, @spottedmahn! Your change will go live within the next 24 hours.


Two challenges, one after the other, doesn't open on Android

Describe the bug
I have a flow in my app where the user can pay with two cards, so if I need to do the 3ds for both cards, should open two challenges, one after the other, but it doesn't open, when the person buys with one card opens normally
To Reproduce
Steps to reproduce the behavior:

Go to page of product
Click to buy
select two credit cards
send information about payment
receive the challenges
3ds modal doesn't open the first modal and second modal, not doing the challenges

Expected behavior
open the first modal, the user completes the challenge, open the second modal, the user completes the challenge and follow the standard flow
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Android

Smartphone (please complete the following information):

Device: Nexus 5x
OS: Android 13.0

Additional context
this happens only on Android device, on iOS, open both modes, one after the other. The only error show to me is:
  {
    "nativeStackAndroid": [
      {
        "lineNumber": 25,
        "file": "AmountExtensions.kt",
        "methodName": "validate",
        "class": "com.adyen.checkout.components.core.internal.util.AmountExtensionsKt"
      },
      {
        "lineNumber": 113,
        "file": "BaseConfigurationBuilder.kt",
        "methodName": "setAmount",
        "class": "com.adyen.checkout.components.core.internal.BaseConfigurationBuilder"
      },
      {
        "lineNumber": 378,
        "file": "GooglePayConfiguration.kt",
        "methodName": "setAmount",
        "class": "com.adyen.checkout.googlepay.GooglePayConfiguration$Builder"
      },
      {
        "lineNumber": 416,
        "file": "GooglePayConfiguration.kt",
        "methodName": "googlePay",
        "class": "com.adyen.checkout.googlepay.GooglePayConfigurationKt"
      },
      {
        "lineNumber": 188,
        "file": "BaseModule.kt",
        "methodName": "invoke",
        "class": "com.adyenreactnativesdk.component.base.BaseModule$getCheckoutConfiguration$1"
      },
      {
        "lineNumber": 187,
        "file": "BaseModule.kt",
        "methodName": "invoke",
        "class": "com.adyenreactnativesdk.component.base.BaseModule$getCheckoutConfiguration$1"
      },
      {
        "lineNumber": 70,
        "file": "CheckoutConfiguration.kt",
        "methodName": "<init>",
        "class": "com.adyen.checkout.components.core.CheckoutConfiguration"
      },
      {
        "lineNumber": 181,
        "file": "BaseModule.kt",
        "methodName": "getCheckoutConfiguration",
        "class": "com.adyenreactnativesdk.component.base.BaseModule"
      },
      {
        "lineNumber": 46,
        "file": "ActionModule.kt",
        "methodName": "handle",
        "class": "com.adyenreactnativesdk.cse.ActionModule"
      },
      {
        "lineNumber": -2,
        "file": "Method.java",
        "methodName": "invoke",
        "class": "java.lang.reflect.Method"
      },
      {
        "lineNumber": 372,
        "file": "JavaMethodWrapper.java",
        "methodName": "invoke",
        "class": "com.facebook.react.bridge.JavaMethodWrapper"
      },
      {
        "lineNumber": 149,
        "file": "JavaModuleWrapper.java",
        "methodName": "invoke",
        "class": "com.facebook.react.bridge.JavaModuleWrapper"
      },
      {
        "lineNumber": -2,
        "file": "NativeRunnable.java",
        "methodName": "run",
        "class": "com.facebook.jni.NativeRunnable"
      },
      {
        "lineNumber": 942,
        "file": "Handler.java",
        "methodName": "handleCallback",
        "class": "android.os.Handler"
      },
      {
        "lineNumber": 99,
        "file": "Handler.java",
        "methodName": "dispatchMessage",
        "class": "android.os.Handler"
      },
      {
        "lineNumber": 29,
        "file": "MessageQueueThreadHandler.java",
        "methodName": "dispatchMessage",
        "class": "com.facebook.react.bridge.queue.MessageQueueThreadHandler"
      },
      {
        "lineNumber": 201,
        "file": "Looper.java",
        "methodName": "loopOnce",
        "class": "android.os.Looper"
      },
      {
        "lineNumber": 288,
        "file": "Looper.java",
        "methodName": "loop",
        "class": "android.os.Looper"
      },
      {
        "lineNumber": 234,
        "file": "MessageQueueThreadImpl.java",
        "methodName": "run",
        "class": "com.facebook.react.bridge.queue.MessageQueueThreadImpl$4"
      },
      {
        "lineNumber": 1012,
        "file": "Thread.java",
        "methodName": "run",
        "class": "java.lang.Thread"
      }
    ],
    "userInfo": null,
    "message": "Value cannot be less than 0.",
    "code": "parsingError"
  }

Hey @LeandroSimoes42

user can pay with two cards

This use-case is currently not possible, unfortunately.
You can have two sets of AdyenCheckout, but under the hood BaseModule is a singleton, so both instances will still rely on the same internal properties.
I can suggest using your custom UI and Client-side encryption as an alternative.
In future versions, we are planning to work on embedded views for Components, so there is a chance this limitation could be addressed. No promises just yet, since we haven't completed assessing this architecture.

Hey @LeandroSimoes42
What version of SDK is this?

Are you using "standalone action handling" ?
AdyenAction.handle(action, configuration)`

"message": "Value cannot be less than 0.",

Seem like it is failing, because amount is less then zero.
How is your configuration looks like?

@LeandroSimoes42
Any luck?

I'm going to close this ticket. Feel free to reopen or continue the discussion here.


Css import

How to import CSS from node_modules ? example for draft js

@m0rdreck Assume you're in App.js and you want to import bootstrap from node_modules :
import '../../../node_modules/bootstrap/dist/css/bootstrap.css';

i try but not working ... don't see any changement in styles i have add in style Html.js and in my containers.

@m0rdreck Actually, there's no different between but .css file in .src/theme and node_modules, make sure your path or css file is correct.


TheBloke/Xwin-LM-70B-V0.1-AWQ issue on 2GPU

I'm running on 2GPU A100 the following code:
sampling_params = SamplingParams(temperature=0.8, top_p=0.95)

llm = LLM(model=MODEL_NAME, quantization="awq", dtype="half", tensor_parallel_size=2)

and getting the incomplete error as

AssertionError                            Traceback (most recent call last)
File <command-557788572799157>, line 1
----> 1 outputs = benchmark_generation_speed(MODEL_NAME, PROMPTS)

File <command-557788572799156>, line 6, in benchmark_generation_speed(model_name, prompts)
      4 print(f"Benchmarking time to first token for {model_name}")
      5 start_time = time.time()
----> 6 _1 = llm.generate(prompts[0], sampling_params)  # Generate the first token
      7 print(_1) # Avoid spark lazy execution
      8 time_to_first_token = time.time() - start_time

File /local_disk0/.ephemeral_nfs/cluster_libraries/python/lib/python3.10/site-packages/vllm/entrypoints/llm.py:157, in LLM.generate(self, prompts, sampling_params, prompt_token_ids, use_tqdm)
    155         token_ids = prompt_token_ids[i]
    156     self._add_request(prompt, sampling_params, token_ids)
--> 157 return self._run_engine(use_tqdm)

File /local_disk0/.ephemeral_nfs/cluster_libraries/python/lib/python3.10/site-packages/vllm/entrypoints/llm.py:177, in LLM._run_engine(self, use_tqdm)
    175 outputs: List[RequestOutput] = []
    176 while self.llm_engine.has_unfinished_requests():
--> 177     step_outputs = self.llm_engine.step()
    178     for output in step_outputs:
    179         if output.finished:

File /local_disk0/.ephemeral_nfs/cluster_libraries/python/lib/python3.10/site-packages/vllm/engine/llm_engine.py:562, in LLMEngine.step(self)
    559     return ignored
    561 # Execute the model.
--> 562 output = self._run_workers(
    563     "execute_model",
    564     seq_group_metadata_list=seq_group_metadata_list,
    565     blocks_to_swap_in=scheduler_outputs.blocks_to_swap_in,
    566     blocks_to_swap_out=scheduler_outputs.blocks_to_swap_out,
    567     blocks_to_copy=scheduler_outputs.blocks_to_copy,
    568 )
    570 return self._process_model_outputs(output, scheduler_outputs) + ignored

File /local_disk0/.ephemeral_nfs/cluster_libraries/python/lib/python3.10/site-packages/vllm/engine/llm_engine.py:712, in LLMEngine._run_workers(self, method, get_all_outputs, *args, **kwargs)
    710 output = all_outputs[0]
    711 for other_output in all_outputs[1:]:
--> 712     assert output == other_output
    713 return output

AssertionError:

There is no message after Assertion Error so I have no idea what is the issue. Help is very much appreciated

https://github.com/vllm-project/vllm/pull/1389 fixed it for me.

@dimitry12 I just rerun and it pulled the new version and it is working now. Thank you so much


Add requirement.txt to install grpcio package

Need requirement.txt to install grpcio package

pip install dapr-client will install these dependencies now.


Implementation of  AppRole Auth

Hi guys. Thank you for great project.
I'm wondering if you could implement AppRole Auth.
It's really easy to implement becouse output of the AppRole is token which you can use in traditional way.
https://www.vaultproject.io/docs/auth/approle.html
AppRole provide way, how to authentificate against Vault using role-id and secret-id and get token from Vault for communication. Token can be then used, renewed etc.
Role-ID and Secret-ID can be configured to not expire, so it's ideal for automation and there is no problem when some of the system is down for some period of time.
It could be nice if your system can first use AppRole auth to get token and then continue in normal way.

Hi @rvojcik ! Thanks for using secrets-manager!
We have been considering this too to be honest :). But at some point we decided that was way more convenient to just use a token and renewing it.
We do use AppRole auth method internally, but we use it just to generate the token (attached to a policy) than later secrets-manger will consume in a pipeline that is fully automated. Wouldn't something like that work for you as well while we consider this feature?

Hi,
thank you for reply. Yes I'm using same process right now. Using gitlab CI/CD to do a approle auth, retrieve token and pass to next deploy job so I thing it's the same workaround like yours.
I'm pretty satisfied with it so I think there is no need to implement it. It could be nice to have it but it's not blocker for me now :)

Thanks! On one hand I find It nice as well. But there's no direct method in the go client library, you can always use the general logical client and write in the auth path though.
But I find It also that just letting secrets-manager renew a token make It more agnostic of the login mechanism.
So I'm kind of torn here XD

Yes I understand. I can describe you my use case, why I think it could be nice to have it.
Imagine that you have token from approle, but it's configured to be short, for example like 40m.
It's no problem when everything works, Secrets-Manager automaticly renew token when needed but when something broke, you can loose internet connection, or vault is down or sealed for some time, longer then TTL of the token. Then you have to redeploy Secrets-Manager to get new token.
It could be nice if Secrets-Manager detects token already expired and use Approle to get newone without need to redeploy it.

Hi @rvojcik !! I finally found the time (sorry for the delay) to really think about this and...
I decided to implenent approle as the only auth mechanism to Vault :) . Here's my reasoning:

AppRole was always our first goal, but the lack of native method in the go client and our rush to release it made us go with Token + renewal loop which is also a common approach
As you pointed out, re-login with approle will make secrets-manager more resilient to network failures that lead to potential token expiration. Increasing the token TTL is an option but not a good one.
I won't support both ways of login, it will make the code complex and can't find a very good reason for it.

I hope I can have a PR soon to share ;).
Thanks again for using it!

Hi @fcgravalos,
that is perfect ! Thx much for this :) I appreciate it :)
It helps a lot and now I can simplify my deployment.
Thanks !

@rvojcik you can build from master the latest version of secrets-manager and give it a try :)


Support slash commands

Hello, I wanted to know if it was possible to use slash commands with autocompletion?
Or is it something that can be added in future? :)
Thanks!

Hi! Yes, there will be such an opportunity, but after release discord.js(v13)


Lucide Icons Figma Plugin Doesn't Respect Settings

Package

[ ] lucide
[ ] lucide-angular
[ ] lucide-flutter
[ ] lucide-preact
[ ] lucide-react
[ ] lucide-react-native
[ ] lucide-solid
[ ] lucide-svelte
[ ] lucide-vue
[ ] lucide-vue-next
[X] Figma plugin
[ ] source/main
[ ] other/not relevant

Version
v0.452.0
Can you reproduce this in the latest version?

[X] Yes
[ ] No

Browser

[ ] Chrome/Chromium
[ ] Firefox
[ ] Safari
[ ] Edge
[ ] iOS Safari
[ ] Opera
[X] Other/not relevant

Operating system

[ ] Windows
[ ] Linux
[X] macOS
[ ] ChromeOS
[ ] iOS
[ ] Android
[ ] Other/not relevant

Description
Figma Plugin Lucide Icons doesn't respect settings.

Setting width to 1.67px stroke.

Create a new Frame
Select frame
Try to insert icon with stroke of 1.67
Icon Inserted has stroke of 1.31, and doesn't respect size settings

Bonus Bug: No feature to lock icon ratio for resizing. More important than offering to lock the icon layer

Steps to reproduce

Create a new Frame
Select frame
Try to insert icon with stroke of 1.67
Icon Inserted has stroke of 1.31, and doesn't respect size settings

Checklist

[X] I have searched if someone has submitted a similar issue before and there weren't any. (Please make sure to also search closed issues, as this issue might already have been resolved.)

Additional testing yields setting Stroke width: to: 1.9987, yields the desired stroke width of 1.67

Perhaps this solution will help you figure out the problem click


Add Swift Package Manager description.

Did some work to add SPM support, accepting this PM also means that a minor version of SwinjectStoryboard library including Package.swift will need to be released to make SPM work.

I'm sorry, I didn't read your comment. As you mentioned, we need more work to support SPM in this project.

I don't know much about SPM for iOS/macOS projects unlike SPM for Linux. What should we do to complete this pull request to support SPM for those projects?

I appreciate your pull request to try to support SPM in this project. If you have knowledge to apply SPM to an Xcode project, I would like you to guide me to support it😉


qemu-user-static and box64 do not work

Windows Version
Microsoft Windows [Version 10.0.26100.1457]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-microsoft-standard-WSL2
Distro Version
Ubuntu 24.04
Other Software

qemu-user-static/noble,now 1:8.2.2+ds-0ubuntu1 arm64 [installed]
box64/unknown,now 0.3.1+20240820.66da28a-1 arm64 [installed] (Installed from https://github.com/ryanfortner/box64-debs)

Repro Steps
Install qemu-user-static using:
sudo apt install qemu-user-static binfmt-support

Or install box64 using the steps listed here: https://github.com/ryanfortner/box64-debs
Then try downloading and running an x86_64 binary:
$ wget "https://github.com/aristocratos/btop/releases/download/v1.3.2/btop-x86_64-linux-musl.tbz"
$ tar -xf btop-x86_64-linux-musl.tbz
$ btop/bin/btop
-bash: btop/bin/btop: cannot execute binary file: Exec format error

Expected Behavior
I expect running x86_64 binaries to work on ARM via QEMU.
https://wiki.debian.org/QemuUserEmulation
Actual Behavior
Any attempts to launch an x86_64 binary fail with this error:
-bash: xxxxxxxxx: cannot execute binary file: Exec format error

/proc/sys/fs/binfmt_misc/ only shows a few entries:
$ ls /proc/sys/fs/binfmt_misc/
WSLInterop  WSLInterop-late  python3.12  register  status

Diagnostic Logs
$ strace -ff btop/bin/btop
execve("btop/bin/btop", ["btop/bin/btop"], 0xffffe1451d58 /* 27 vars */) = -1 ENOEXEC (Exec format error)
strace: exec: Exec format error
+++ exited with 1 +++

WslLogs-2024-08-21_09-22-41.zip

WslLogs-2024-08-21_09-41-42.zip

@ading2210: Based on the output, qemu is most likely missing its binfmt entry. What's the output of  sudo update-binfmts --display  ? Does this solve the issue ?

sudo update-binfmts --display does not show anything related to QEMU.
$ sudo update-binfmts --display
[sudo] password for allen:
python3.12 (enabled):
     package = python3.12
        type = magic
      offset = 0
       magic = \xcb\x0d\x0d\x0a
        mask =
 interpreter = /usr/bin/python3.12
    detector =
$ sudo apt install qemu-user-static
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
qemu-user-static is already the newest version (1:8.2.2+ds-0ubuntu1).
0 upgraded, 0 newly installed, 0 to remove and 3 not upgraded.

This is most likely the root cause of the issue. Do you have systemd enabled ? If and so, what's the output of systemctl -t service -a ?

I do have systemd enabled. The systemd-binfmt service is running normally.

        ACTIVE → The high-level unit activation state, i.e. generalization of SUB.
        SUB    → The low-level unit activation state, values depend on unit type.

134 loaded units listed.
To show all installed unit files use 'systemctl list-unit-files'.

It seems that the systemd-binfmt is not being started because it intentionally blacklists itself from running in WSL.
○ systemd-binfmt.service - Set Up Additional Binary Formats
     Loaded: loaded (/usr/lib/systemd/system/systemd-binfmt.service; static)
    Drop-In: /usr/lib/systemd/system/systemd-binfmt.service.d
             └─wsl.conf
     Active: inactive (dead)
  Condition: start condition unmet at Wed 2024-08-21 13:29:41 PDT; 3min 59s ago
       Docs: man:systemd-binfmt.service(8)
             man:binfmt.d(5)
             https://docs.kernel.org/admin-guide/binfmt-misc.html
             https://www.freedesktop.org/wiki/Software/systemd/APIFileSystems

Aug 21 09:41:59 LAPTOP-SL2KEVM7 systemd[1]: systemd-binfmt.service - Set Up Additional Binary Formats was skipped because of an unmet condition check (ConditionVirtualization=!wsl).
Aug 21 09:42:20 LAPTOP-SL2KEVM7 systemd[1]: systemd-binfmt.service - Set Up Additional Binary Formats was skipped because of an unmet condition check (ConditionVirtualization=!wsl).
Aug 21 13:29:41 LAPTOP-SL2KEVM7 systemd[1]: systemd-binfmt.service - Set Up Additional Binary Formats was skipped because of an unmet condition check (ConditionVirtualization=!wsl).

Ah I see. Your distro has manually disabled it when running inside WSL. That's definitely the problem.
If you manually edit that unit, you should be able to remove the condition to solve the issue.

Removing this line in /usr/lib/systemd/system/systemd-binfmt.service.d/wsl.conf fixes the issue, although it probably breaks other WSL functionality.
# systemd breaks WSL interoperability by rewriting the binfmt interpreters configuration.

[Unit]
ConditionVirtualization=!wsl

Now the service is running normally, and the QEMU binfmt entries show up in the kernel sysfs:
$ sudo systemctl status systemd-binfmt
● systemd-binfmt.service - Set Up Additional Binary Formats
     Loaded: loaded (/usr/lib/systemd/system/systemd-binfmt.service; static)
    Drop-In: /usr/lib/systemd/system/systemd-binfmt.service.d
             └─wsl.conf
     Active: active (exited) since Wed 2024-08-21 13:38:31 PDT; 43s ago
       Docs: man:systemd-binfmt.service(8)
             man:binfmt.d(5)
             https://docs.kernel.org/admin-guide/binfmt-misc.html
             https://www.freedesktop.org/wiki/Software/systemd/APIFileSystems
    Process: 40917 ExecStart=/usr/lib/systemd/systemd-binfmt (code=exited, status=0/SUCCESS)
   Main PID: 40917 (code=exited, status=0/SUCCESS)

Aug 21 13:38:31 LAPTOP-SL2KEVM7 systemd[1]: Starting systemd-binfmt.service - Set Up Additional Binary Formats...
Aug 21 13:38:31 LAPTOP-SL2KEVM7 systemd[1]: Finished systemd-binfmt.service - Set Up Additional Binary Formats.

ls /proc/sys/fs/binfmt_misc/
box64       qemu-cris     qemu-loongarch64  qemu-mips64    qemu-mipsn32el  qemu-riscv32  qemu-sh4eb        qemu-x86_64    status
python3.12  qemu-hexagon  qemu-m68k         qemu-mips64el  qemu-ppc        qemu-riscv64  qemu-sparc        qemu-xtensa
qemu-alpha  qemu-hppa     qemu-microblaze   qemu-mipsel    qemu-ppc64      qemu-s390x    qemu-sparc32plus  qemu-xtensaeb
qemu-armeb  qemu-i386     qemu-mips         qemu-mipsn32   qemu-ppc64le    qemu-sh4      qemu-sparc64      register

I'm going to close this issue as it it's the fault of Ubuntu, not WSL.


Incompatibility with Laravel 6

Laravel version: v6.18.8
Laravel Tracey: v1.11.0
PHP: PHP 7.2
Declaration of Recca0120\LaravelTracy\Exceptions\Handler::report(Throwable $e) must be compatible with Illuminate\Contracts\Debug\ExceptionHandler::report(Exception $e)
Looks like Recca0120\LaravelTracy\Exceptions\Handler.php report and render methods need to be updated to be compatible with the Laravel 6 version.
Cheers
Tom

fixed


VSCode-like Input Support

Hi, thank you for this awesome plugin! I love the live-preview for renaming, and live-command.nvim that expands this to other commands like norm.
Is there anyway to integrate this with filipdutescu/renamer.nvim or stevearc/dressing.nvim for a VSCode-like input (see below images)?
I opened a similar issue in renamer.nvim as well:
https://github.com/filipdutescu/renamer.nvim/issues/137
renamer.nvim

dressing.nvim

Hi, I'm glad you enjoy this plugin! I did not know about renamer.nvim, well this seems like a great match to add support for. I briefly looked at the source code and it looks like it shouldn't be too difficult to implement. I put it on my todo list -- a PR is always welcome of course to speed things up!

Hi, I'm glad you enjoy this plugin! I did not know about renamer.nvim, well this seems like a great match to add support for. I briefly looked at the source code and it looks like it shouldn't be too difficult to implement. I put it on my todo list -- a PR is always welcome of course to speed things up!

Thank you @smjonas! I appreciate the promptness and openness! :clap:


State methods are different in JavaScript and .NET.

state methods are different in JavaScript and .NET.
in JavaSCript we have read and write
in .NET we have load and save

(And incidentally the BotStateSet in .NET doesn't load on OnTurn because it doesn't need to - because the dialog set CreateDialogContext - via the accessor, does the load.)

The main difference is in the IStorage interface. The TypeScript defines a StoreItem structure, the C# doesn't bother. The IStorage mechanism is essentially just a plug in to the BotState world. Its not very widely scoped at all. Many implementations of state will by pass the BotState anyhow (because the whole reason you are implementing something else is because you want some other semantics). In other words the accessor is very much the main state abstraction.
The C# also defines a IStatePropertyInfo interface as a base for the IStatePropertyAccessor interface. This just provides the Name of the property. Its not there in the JavaScript but probably won't be missed.
Neither of these issues seem a high priority.


Investigate EXE extraction on Windoze

It is possible to extract an MSI .exe file from within a bash script using the following syntax:
#!/bin/bash
/cygdrive/c/Users/vagrant/Downloads/jdk-8u91-windows-x64.exe /s INSTALLDIR=C:\\temp\\jdk

Some observations:

perform chmod +x on .exe file before executing
specify the fully qualified path to the .exe file when executing
/s instructs a silent installation
INSTALLDIR must be specified as a windows path with escaped backslashes.
any nested installation folders will be created on demand by the command, no need for mkdir -p

IIRC I was able to get some older JDK's content without invoking the installer with the help of 7-zip. Depends on how you want to deal with possibly interfering registry settings or whether you completely want to take over the management of them.

It would be preferable to completely manage them like we do with all our candidates. I've actually managed to get it working on a vm with the above syntax.
I would prefer not to use 7-zip because of the dependency introduced. If MSI can do it out of the box, we get the extraction for free in a light weight manner.

I would prefer not to use 7-zip because of the dependency introduced.

I already thought this would be an issue.
My main point why suggesting 7-zip was that you never know what the installer does in addition to extracting the JDK files. I think one thing to cope with is the installer setting the installed JDK as the default. That will make the handling of a 'no' for the default question after the JDK installation a bit more difficult.
One thing: does the installer immediately return after call? If so, how will you know the installation has finished? (just curious)

AFAIK, when running the command as described, it doesn't do anything
besides extract the contents. The installer blocks the shell until
extraction is complete. We will then take the extracted folder and deal
with it as usual by placing it under ~.sdkman/candidates/java.
Important to note that we care only about the availability of the JDK on
the PATH in Cygwin terminals, and not about global Windows
installation/integration here.
On Thu, 21 Jul 2016 at 07:44 Andreas Schöneck<EMAIL_ADDRESS>wrote:

I would prefer not to use 7-zip because of the dependency introduced.
I already thought this would be an issue.
My main point why suggesting 7-zip was that you never know what the
installer does in addition to extracting the JDK files. I think one thing
to cope with is the installer setting the installed JDK as the default.
That will make the handling of a 'no' for the default question after the
JDK installation a bit more difficult.
One thing: does the installer immediately return after call? If so, how
will you know the installation has finished? (just curious)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sdkman/sdkman-cli/issues/441#issuecomment-234170598,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAI0BrhYb7CcQf3XQq77xU2qRiAuTzKyks5qXxVmgaJpZM4JKctb
.

Not sure about the installer not doing other stuff:
https://www.java.com/en/download/help/silent_install.xml
It's somehow unclear what Oracle means with

Have all features of the JRE installed

You could probably easily find out by creating the mentioned log, using /L <path-to>setup.log

Good point, will give that a try.


Sean, just an idea on YouTube shutting Streamus down...

I know that you've worked tremendously hard on this project and sincerely appreciate your work.
Instead of letting YouTube shut Streamus down because of its nature to be able to hide the video, market it AS a video player...
For instance, you currently have the ability to open streamus in a tab. Make this the ONLY functionality available to users and constantly have the streaming video open within this tab.
Users may then cause the tab to go to the background of their own doing, letting them listen to streaming music without the video.
Market streamus as a YouTube enhancer that allows users to create a "Rolling playlist" of videos. It would then be seen as none other than another YouTube video player.
You would keep Streamus going, and would be conforming to YouTube's standards.
Just an idea...

This rebranding is indeed going to be happening to some extent.
It's a bit unfortunate because the word "Streamus" stands for "Stream music", but yeah, am aware it would help improve their perception of me. Still a lot of coding stuff to go, though, too.

Sean, could you distribute my new project to your listeners. It's far from perfection, but I think they could use it until you go back online. I appreciate it! https://github.com/LNFWebsite/Streamly

Sorry, forgot to tell you I changed my username HelpingHand1 => LNFWebsite

I responded to you on Reddit. As stated, simply interacting with YouTube's content will make you liable to play by their set of rules. Simply trying to skirt around not using their API is insufficient... :/

Yeah... But as far as I can tell, I'm not breaking any rules... Also, without using their API, Streamly is essentially invisible to them. It can be put on any web server.

 We'll see where that takes me.

And all the best luck to you. Streamly does not break any laws. There is no server side code and never will be with Streamly. It is more of a fallback for when other services cannot be found.
I have seen that Streamus has its own dedicated server. That's where things get smelly in the legal realm.

Streamus server is not even part of the equation regarding my issues with YouTube's legal department. It has never even been brought up in any of our talks.

It kinda makes me wonder why... What does the server do?

localStorage is wiped whenever a Chrome extension is uninstalled and chrome.storage.sync is an incredibly small pipe for syncing data across PCs which can be easily filled.
The server stores a user's G+ ID associated with their playlists and songs so that their data is more persistent across machines / PC reformats / reinstalls. Additionally, it allows for sharing of playlists via streamus.com/share by offering an end-point for the URLs. Without this, you'd have to encode all the information into a URL which simply isn't realistic.

You can see all the table mappings here: https://github.com/MeoMix/StreamusServer/tree/development/Streamus/Dao/Mappings

Additionally, it allows for sharing of playlists via streamus.com/share by offering an end-point for the URLs. Without this, you'd have to encode all the information into a URL which simply isn't realistic.

Why isn't this realistic? Encode the playlist in JSON format and run it through as base64.
Check out how this works: http://jbt.github.io/markdown-editor/

Well, doing the math a bit it looks like it might be realistic, but still is just a horrible way to go about doing things. The maximum limit of a URL in Chrome is 2MB. Anything above that will start to crash the browser and, presumably, for weaker machines a lower value would be needed.
Base64 encoding size is: 4 * Math.ceil(string.length / 3) or 1500000 characters roughly for 2MB.
Stripping out everything except for the bare minimum needed a song would be on average around 100 characters. That gives us an upper limit of songs of 15,000. Streamus itself handles that just fine, and people do have 2k+ song playlists already, but it's something to consider. Additionally, if I ever need to represent more data... like whether a song is YouTube or some other API, then the # of songs able to be shared continues to go down.
Other stuff:

All shareable URLs would break whenever I make back-end changes for storing data or my back-end would need to be backwards compatible for all types of URLs.

It's pretty bad UX to try and share something that long with a friend. Just because you can paste it into your URL bar doesn't mean you can paste it into a Facebook post, Reddit post, etc... those all have maximum character lengths which would start to come into play.

Collaborative playlists are (were?) on the to do list. You cant' really collaborate with others on a playlist if the URL is static.

I didn't know about the 2MB limit on Chrome. Otherwise, every URL shortener could do away with the length troubles... But for the playlist collaboration, you would need the server side stuff.


checkPropTypes.js?a15c:20 Warning: Failed prop type: Invalid prop TextFieldComponent supplied to DateTextField.

I'm seeing a warning in the console. I am trying to use Datepicker component and want to clear all warning. This is the warning I'm getting.
Warning: Failed prop type: Invalid prop TextFieldComponent supplied to DateTextField.
in DateTextField (created by Context.Consumer)
in WithUtils(DateTextField) (created by ModalWrapper)
in ModalWrapper (created by BasePicker)
in BasePicker (created by Context.Consumer)
in WithUtils(BasePicker) (created by DatePickerModal)
in DatePickerModal (created by ForwardRef)
in ForwardRef (created by PickADate)
in MuiPickersUtilsProvider (created by PickADate)
in PickADate (created by Context.Consumer)
in div (created by ForwardRef(Grid))
in ForwardRef(Grid) (created by WithStyles(ForwardRef(Grid)))
in WithStyles(ForwardRef(Grid)) (created by Layout)
in div (created by ForwardRef(Grid))
in ForwardRef(Grid) (created by WithStyles(ForwardRef(Grid)))
in WithStyles(ForwardRef(Grid)) (created by Layout)
in Layout (created by Context.Consumer)
in Route (created by PrivateRoute)
in PrivateRoute
in Switch
in Route (created by App)
in Router (created by App)
in div (created by App)
in App (created by Connect(App))
in Connect(App)
in ThemeProvider
in CssBaseline
in Provider

Please check the code below
import React from 'react';
import { MuiPickersUtilsProvider, DatePicker } from 'material-ui-pickers';
import DateFnsUtils from '@date-io/date-fns';
const onChangeMethod = () => {
console.log('On change method is called!');
};
const PickADate = () => (

This is a date picker

);
export default PickADate;

Looking at the code we can understand that I am not writing DateTextField or its property TextFieldComponent anywhere in my code. It looks like this component is being rendered as it is invoked by another component. It is like a parent-child component tree and at some point, the DateTextField component is involved and it is being passed an invalid prop

Duplicate of #1086

I get the same error message and it seems that there is no real solution anywhere. This thread has been closed as duplicate, but the other referred issue does not provide any solution either. Do you think we could have some follow up, or a piece of advice? Thanks in advance.

@Zumwaloro We have no plan to solve the problem. You can upgrade to v4.0.0-alpha.7, we have addressed the issue.

Ok, thanks!


Opencontrail on Kubernetes flannel

This is a question not any issues.
When setting up opencontrail on Kubernetes does it requires flannel network?

No.
Regards
-Harshad

On May 19, 2017, at 10:03 PM, prasenforu<EMAIL_ADDRESS>wrote:
This is a question not any issues.
When setting up opencontrail on Kubernetes does it requires flannel network?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks for reply.
Can you please guide me how I can setup on Ubuntu with latest kubernetes kubeadm.
All are using flannel & whatever blog I checked all are quite old.
Please refer some current blog install kubernetes latest using kubeadm or any other procedure using opencontrail.


Try building files in windows to make codeql check we are building go code

Try building files in windows to make codeql check we are building go code

/azp run

/azp run


ServerSelector implementation needed to get a nacos server from server list

We need to implement ServerSelector class with different strategies (e.g.: Weighed, Random, Least used)

It's (almost) done by me :) 🥇

For Nacos server selection, we keep it simple to use random-select strategy to pick a start and iterate over all servers available until a success request is made
目前我们采用简单的随机选择起始点枚举所有可用服务器，一旦访问服务器成功，即返回请求结果


sometimes query&insert due to deadcycle

The test code is below. Similar to my project code, the insert action may revoke the query code, again and again.
public class MainActivity extends AppCompatActivity {
private DBHelper mDbhelper;
private BriteDatabase db;
@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);
    mDbhelper = new DBHelper(this);
    SqlBrite sqlBrite = SqlBrite.create();
    db = sqlBrite.wrapDatabaseHelper(mDbhelper, Schedulers.io());

    Subscription sub1 = getData("1")
            .subscribeOn(Schedulers.io())
            .observeOn(AndroidSchedulers.mainThread())
            .subscribe(new Action1<SqlBrite.Query>() {
                @Override
                public void call(SqlBrite.Query s) {
                    //http request for data

                    //save data
                    insertData();
                }
            });
    Log.i("test",""+sub1.isUnsubscribed());
}

private rx.Observable<SqlBrite.Query> getData(final String str){
    Log.i("test","getdata:"+ str);
    return db.createQuery("message","select * from message");
}
private void insertData(){
    Log.i("test","insertdata");
    ContentValues  cv = new ContentValues();
    cv.put("id", "2");
    try{
        db.insert("message",cv );
    } catch (SQLiteConstraintException e) {
        e.printStackTrace();
    }
}

}

There's not a whole lot we can do here. The problem is in your logic, not in the library. You created the loop with no way for it to break except when the insert throws an exception. One of these two operations should not happen as the result of the other and which one is up to you.

Thank u for you replay. I think my logic is common for app. Please let me describe the app logic.
step 1. check local data,if the data no exists, goto step2.
step2. get data from REST API
step3. save data to local stroage.

Your sample is missing the "if data exists" part that would break the cycle.

OMG. The sample made mistake, I will fix it. thanks a lot.


Removed indent from sides of install pages

None of the other pages have that offset, so, I removed it.
Also made the easy installation instructions a col-8 offset-4, so they line up with the download links.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @pcwalton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see CONTRIBUTING.md for more information.

:confetti_ball: :+1:


Add ownership privilidges to wookiee team for kcp

Description
Changes proposed in this pull request:

Add ownership privilidges to wookiee team for kcp

Related issue(s)

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


[#13][TECH][Karuppiah] Upgrade gradle version to v6.8.3

Fixes #13

Thank you @karuppiah7890 for this PR. Gonna merge this one.


Game of Life's slow frame rate is maybe due to scanForTouch()

scanForTouch() apparently takes 100 milliseconds, according to the source code:
// This whole procedure should take about 100 milliseconds
bool EverydayCalendar_touch::scanForTouch(){

If that's accurate, the best frame rate you could possibly hope for would be 10FPS even if the rest of the code took no time.
Suggested solutions:
ideal:
Find a fast way to check for touch - interrupt-based?
naive:
instead of scanForTouch every frame, do it only one in 5 or one in 10 "frames".  However, that will give us an unpleasant irregular frame duration. (e.g. 4 fast frames, 1 slow, etc)
better:
Model two modes: a "pattern programming" mode and a "run simulation" mode.  In pattern programming mode, you'd scanForTouch every frame to be nice and responsive to touch - we don't care about the slow frame rate when programming patterns.
In "run simulation" mode, however, only scan for touch rarely (e.g. once every 5 seconds?) - the user will have to hold a button down for a while to get out of simulation mode and back into pattern programming mode.
This would still give us one slower frame every 5 seconds but hopefully that's not too noticeable.
Maybe check only one key to get out of "run simulation" mode, if it's significantly faster to check one key for touch than all.

I guess I'm wrong - the bouncing balls sketch clearly has a good frame rate despite the fact you're doing scanForTouch in each iteration of loop


adding find-sum-of-cubes-of-numbers.py file

Why:

Closes #ISSUE_NUMBER
What's being changed:

853 resolved


Update of copyright year

Coverage remained the same at 88.414% when pulling afb695cb4ec2e312a782c8dd9ac781e8ee3071a2 on INM-6:update/copyright_year into f5c3e1524731ddf07a3ca873a5d1fcbb529f2a47 on NeuralEnsemble:master.


Web Part Property Pane Content is Overlapped by Apply Button Panel

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Expected or Desired Behavior
Web Part Property Pane content is fully visible in non-reactive mode
Observed Behavior
Web Part Property Pane content is partially overlapped by Apply Button Panel
Steps to Reproduce

scaffold default web part
disable reactive mode

protected get disableReactivePropertyChanges(): boolean {
    return true;
  }

add such amount property pane fields to make scroll appear (in my test I duplicated 'description' text field 13 times)
run the code
open property pane
scroll down the content of property pane
content is overlapped

Hello, @AJIXuMuK - Thanks for raising the concern. This is now fixed and will be available in the next release. I am closing the issue now but feel free to re-open it if the issue is not fixed in the next release.
Thanks,
Srikanth


Embedding Specific file from Gist containing multiple files

Describe the bug
Does Carbon support embedding a specific file from Gist containing multiple files?
To Reproduce

Gist at https://gist.github.com/sachinsu/b2d914a563b49d4dd50a4143166f27ec has multiple files
When trying to embed specific file (Cardwriter.cs) in Carbon App like https://carbon.now.sh/embed/b2d914a563b49d4dd50a4143166f27ec#file-cardwriter-cs. It still shows the first file (i.e. Card.cs) from Gist.

Am i missing something?

Code Snippet (If Applicable)

@sachinsu you are not missing anything! Currently, when using the GitHub Gist, we only show the first file.
I see that you are using the # hash location to try and specify the file name. Is this how you would expect to specify which file to show?

@mfix22 , Github uses # to link multiple files within same Gist so i just used it to give an example. It need not be # hash location and can be either in the form of Query parameter or Path itself.

Thanks for the idea, @sachinsu. You can now select which file you want using the filename query param.
Example Gist: https://gist.github.com/mfix22/4ee50f1c107be7e59e613724fc8ad949
File 1: https://carbon.now.sh/embed/4ee50f1c107be7e59e613724fc8ad949?filename=global.js
File 2: https://carbon.now.sh/embed/4ee50f1c107be7e59e613724fc8ad949?filename=local.js

Thank you @mfix22


chore: make bpftool installable in alpine linux

@trvll this solves the bpftool installation on the alpine container.

works! tks


HELP ME

Could you add me in discord to solve several errors that I have
Alexita#2023

Could you add me in discord to solve several errors that I have Alexita#2023

Hello,
I have sent a request to you on Discord.
Thanks.


neo4j writer

implement a neo4j writer to create a knowledge graph query using neo4j
write get_writter for neo4j writter

Looks Good!


fix(events): fallback to touch and mouse events

PR Checklist
Please review the guidelines for contributing to this repository.

[x] I am requesting to pull a topic/feature/bugfix branch (right side). In other words, not master.
[x] I have run yarn test against my changes and tests pass.
[x] I have added tests to prove my fix is affective or my feature works. This can be done in the form of unit tests in test/unit/ or a new or altered demo in demo/.
[x] I have added or edited necessary documentation in the README.md, or no docs changes are needed.

Description
Uses touch events or mouse events wherever pointer events are not supported. Tested in Desktop and iOS Safari.

:tada: This PR is included in version 4.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Feature/search user

Fixed the merge conflicts.
I have added link to the HTML page regarding the "search for a user" functionality in homepage. Didn't you test your frontend though?

I haven't tested the APIs. Those who will merge the branch must test the APIs, run the tests and test the frontend before merging.

Fixed merge conflicts with the practice_app.
Changed the name of the route. Now it's api/v1/users/username/<str:username> rather than api/v1/search_user/<str:username> since using verbs in APIs is not a good practice as stated here.
I have also changed api/v1/user/<int:id>/comment/<int:commentid> to api/v1/users/<int:id>/comment/<int:commentid> in order to keep consistency between our routes.
Test your API using Postman. Also tested the frontend. Run the tests. Everything works fine.

After merging I've deleted feature/search_user branch in the remote repository (here in this repo).
In order to delete it from local, you can follow these steps:
i) Delete the local branch using git branch -d <branch>. After this step, if you see that remotes/origin/feature/search_user still exists, then do the following.
ii) Delete the remote branch that doesn't have any correspondence in remote repository: git fetch --prune


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 212 ms

Resolved: ClamAV performance has improved in 9785ba5.


Fix typo in crl2pkcs documentation #13910

Fix typo in crl2pkcs documentation "form" to "from"
CLA: trivial

Checklist

I agree with CLA: trivial

I agree with CLA: trivial

@t-j-h do you agree this is trivial?

@t-j-h do you agree this is trivial?

Agree trivial.

Agree trivial.

This pull request is ready to merge

This pull request is ready to merge

Merged to master and 1.1.1 branch. Thank you for the contribution.

Merged to master and 1.1.1 branch. Thank you for the contribution.


Create issues on build failures

Is your feature request related to a problem? Please describe
We want to create a github issue in the respective plugins, opensearch or OpenSearch-dashboards repositories when the build failures.
Describe the solution you'd like

[x] Filter out issues from noise about build failure
[x] Create issues in respective repositories for build failures on jenkins.
[x] Tag owners in build failures on github once the issue is created.
[x] Duplicate issues are not created for build failures

Acceptance Criteria

[x] Github issue is created automatically in the respective repository when the build fails.
[x] Duplicate issues are not created
[x] Issues are filtered based on failures to avoid noise

The required PR has been merged and issue creation has been enabled.
Example: https://github.com/opensearch-project/performance-analyzer/issues/282.
Closing this issue.


Trying to create App level permission

Hi,
I am trying to achieve Application level permission, based on roles. What my idea is to use a field like 'appPermissions' simialr to 'indexPermissions' or 'clusterPermissions', from kibana.yml,  which will contain an array of permissible app ids, such as ['app:discover', 'app:dashboards',....], which further could be clubbed in action groups or could be used singly and associated with roles. Similar action like with clusterPermissions.
In the react code I created the configuration parameters, and the constant list, but I want to achieve this through the index, I want to save the permissions app_permissions field in the index, for a particular role, while trying to insert this field through API i got an error saying:
{
  "status" : "error",
  "reason" : "Invalid configuration",
  "invalid_keys" : {
    "keys" : "app_permissions"
  }
}

Following is the API query:
PUT _opendistro/_security/api/roles/approle
{
  "app_permissions": [
    "app:home",
    "app:discover",
    "app:dashboards",
    "app:visualize"
  ],
  "cluster_permissions": [
    "cluster_composite_ops",
    "indices_monitor"
  ],
  "index_permissions": [{
    "index_patterns": [
      "movies*"
    ],
    "dls": "",
    "fls": [],
    "masked_fields": [],
    "allowed_actions": [
      "read"
    ]
  }],
  "tenant_permissions": [{
    "tenant_patterns": [
      "human_resources"
    ],
    "allowed_actions": [
      "kibana_all_read"
    ]
  }]
}

Can anyone help me on where to start in the ES plugin end to achieve this?

"app_permissions" is not supported as part of the roles.

"app_permissions" is not supported as part of the roles.

Yeah I know that "app_permissions" is not supported. That was my question actually if I want to add the field then where should I start.

Yeah I know that "app_permissions" is not supported. That was my question actually if I want to add the field then where should I start.

There is no plan to add "app_permissions" or custom roles. You may start with creating proposal and justification (use cases) why such functionality will be useful for others.

There is no plan to add "app_permissions" or custom roles. You may start with creating proposal and justification (use cases) why such functionality will be useful for others.


Change Volume

Hi,
is in xZune.VLc a chance to set a Volume?
Thanks for answer
Ludek

Sorry,I'm so careless that forgot this.
I'll add it later.
Thank for use xZune.Vlc

Ok. Thanks a lot for your prompt answer.
Good luck with xZune.Vlc
Ludek

Ummm,xZune.Vlc is not perfect yet.
If you found any problems,please let me know 😊.

I have already found some problems. I write them to different threads.

 IsMute doesn't seem to work.  My testing code is as follows:
           vlcPlayer.IsMute = true;             vlcPlayer.LoadMedia(new Uri("<a rtsp url>"));             vlcPlayer.Play();


Create documentation

Documentation of all basic features. Place of the documentation has to be decided. VIVO wiki or here in the repo.

Initial documentation about the main steps and functionalities are added under (currently under intra TIB forum)
https://wiki.tib.eu/confluence/display/tapir/Documentation+of+the+TAPIR+reporting+tool
I will as well duplicate the documentation in the README.md of the VIVO and VItro in Tapir organization:
https://github.com/Project-TAPIR/vivo-reporting
https://github.com/Project-TAPIR/vitro-reporting


Save multiple backups on an interval

Hello @mtkennerly
I've started implementing this little addition. Plan is to use a symbolic link to the backup copy that ludusavi creates, then keep copying from that link while in-game so it doesn't collide with the files the game currently uses. When out of the game, that is when we enable a restore of a copy. The copy then overrides what you have there that ludusavi looks for.
I hope I don't get too many hiccups and that no one has been implementing this yet :).
Cheers!

Hi! Sorry for the very late reply, but here's my input on this one:

For the aspect of managing multiple copies of backups per game, that should rely on mtkennerly/ludusavi#74, rather than implementing it in the plugin itself. That way, you get consistent behavior in Ludusavi, the Playnite plugin, and in any other future plugins.
Once that's done, then adding a timer for backups during play would be great.

@ANRCorleone This is (finally) available now in v0.10.0 of the plugin.
Are there any other changes from the AutoSave fork that you think should be merged back here, or would it be safe to retire now?

Hello @mtkennerly sorry for the late reply, been busy with work. I quickly tested the ludusavi playnite plugin and I can't find option to pick an item from multiple backups that are generated automatically and restore one from there (like pick a backup from 3 hours ago, etc.). It was what I added on that fork, is that option available in ludusavi playnite now?

No worries! Yes, I did add that, but it requires Ludusavi 0.12.0 or newer, so that might be why you didn't see it.

If the extension can't get the list of backups (because you have an older Ludusavi, or the game just doesn't have any backups, or it does but the name doesn't match between Playnite and Ludusavi), then it will just show the original restore option without the extra menu level.

Thank you 😸 I'll archive that repo now then

Thanks!
Do you think it would be a good idea to make a new release of AutoSave so that the plugin status is also visible in Playnite? If you're just browsing in Playnite, you won't know that it's archived. Could update the notes shown in the plugin browser (below) and maybe even log a warning on startup.

All done 👍


ci(ti-community/test-infra): fix lint jobs

What problem does this PR solve?
Issue Number: close #xxx
Problem Summary:
What is changed and how it works?
What's Changed:
How it Works:

/merge


Java model loader does not set qualifying type correctly in some cases

Given this Java code:
package stuff;

public class Tree {

    static class Term {}
    static class Plus extends Term {}
    static class Minus extends Term {}

}

class Visitor {
    void visit(Tree.Plus plus) {}
}

The model loader gives the Value representing plus a Type with no qualifying type. This is the cause of bug https://github.com/ceylon/ceylon-ide-eclipse/issues/1742 in the IDE.

@tombentley or @FroMage any chance you could find a few mins to fix this one?

The reason is that to fix #2388 @FroMage explicitly prevented using the containers type as the qualifying type when the member type was static: https://github.com/ceylon/ceylon/issues/2388#issuecomment-156628074. But it seems that #2388 works just fine even when there is a qualifying type, so I've reverted that change.

Great, thanks!