Return 1 as exchange rate when currencies are identical

When I ask Swap::latest('EUR/EUR') for the latest exchange rate, it appears that it's actually contacting all the providers:
The chain resulted in 3 exception(s):
Exchanger\Exception\UnsupportedCurrencyPairException: The currency pair "EUR/EUR" is not supported by the service "Exchanger\Service\Fixer".
RuntimeException: Unable to parse JSON data: JSON_ERROR_SYNTAX - Syntax error, malformed JSON
Exchanger\Exception\UnsupportedCurrencyPairException: The currency pair "EUR/EUR" is not supported by the service "Exchanger\Service\EuropeanCentralBank".

This doesn't seem right to me.
I looked at the code (both Swap and Exchanger) to see if I could create a PR to address this. I found out that in https://github.com/florianv/exchanger/blob/master/src/Exchanger.php#L46 it is smart enough to return new ExchangeRate(1) when the currencies are identical.
I could add these checks to all ExchangeRateProvider implementations, but that does seem a bit ugly to me. What if an implementation forgets to add this.
I could also add a check in Swap\Swap::quote to check if the currencies are identical. But the issue is that I need to use Exchanger\CurrencyPair::createFromString (instead of the contract Exchanger\Contract\CurrencyPair to use CurrencyPair::isIdentical.
What's your view on this?

Hey, Swap is not contacting any provider for identical currency pairs as you can see in this example:
<?php

require_once __DIR__ . '/vendor/autoload.php';

use Swap\Builder;

// Build Swap with Fixer.io
$swap = (new Builder())
    ->add('fixer')
    ->add('yahoo')
    ->add('google')
    ->build();

$rate = $swap->latest('EUR/EUR');

@ruudk Maybe you are using an older version of Exchanger before the identical currency case was supported ?


Execute UWP Unit tests projects on Windows Server 2016 Build Controller with TFS 2015 Build Agent (non xaml)

On a Windows Server 2016 machine with a TFS 2015 Build Agent (TFS Build System - not the old XAML Builds) the execution of UWP Unit Tests hangs/fails.
When logging on to the server the UWP unit test app is started, but it seems to fail to communicate back to the test runner. The test runner runs into a timeout, when looking into the logs.

@dersteve Can you give more information on how you are running your tests and share your build logs with us? [please do enable system.debug=true in build variables

Closing this issue as there's no further information available. Please do reopen if needed

Here is the test log: You can see that there is timeout at the end.
V, 3464, 1, 2017/02/15, 10:21:54.487,<PHONE_NUMBER>072, vstest.console.exe, AddPackage(eb61e043-d453-43ae-bda3-569724225115_<IP_ADDRESS>_x64__41zkwv185hb04) Error Code:0 and Error Message:
V, 3464, 1, 2017/02/15, 10:21:54.675,<PHONE_NUMBER>193, vstest.console.exe, Established network loopback.
V, 3464, 1, 2017/02/15, 10:21:54.675,<PHONE_NUMBER>047, vstest.console.exe, AppContainerUtilities.InstallPacakge(eb61e043-d453-43ae-bda3-569724225115_<IP_ADDRESS>_x64__41zkwv185hb04) returned:0
V, 3464, 1, 2017/02/15, 10:21:54.675,<PHONE_NUMBER>408, vstest.console.exe, AppContainerUtilities.InstallPacakge: Time taken for Installing pacakge 00:00:03.5453938
I, 3464, 1, 2017/02/15, 10:21:54.675,<PHONE_NUMBER>155, vstest.console.exe, RunTestsArgumentProcessor:Execute: Test run is starting.
V, 3464, 1, 2017/02/15, 10:21:54.737,<PHONE_NUMBER>077, vstest.console.exe, Package C:[....]ABCDEF.Tests_Test[.....].Tests.appx has architecture X64.
V, 3464, 1, 2017/02/15, 10:21:54.753,<PHONE_NUMBER>400, vstest.console.exe, Using effective platform:X64 effective framework:Framework45
I, 3464, 1, 2017/02/15, 10:21:54.831,<PHONE_NUMBER>428, vstest.console.exe, Using platformType OutOfProc for inIsolation=True.
V, 3464, 1, 2017/02/15, 10:21:54.847,<PHONE_NUMBER>690, vstest.console.exe, TestRunRequest.ExecuteAsync: Creating test run request.
I, 3464, 1, 2017/02/15, 10:21:54.862,<PHONE_NUMBER>971, vstest.console.exe, AssemblyResolver: Microsoft.VisualStudio.Utilities.Internal, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a: Resolving assembly.
I, 3464, 1, 2017/02/15, 10:21:54.862,<PHONE_NUMBER>643, vstest.console.exe, AssemblyResolver: Microsoft.VisualStudio.Utilities.Internal, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a: Resolved assembly.
I, 3464, 1, 2017/02/15, 10:21:54.862,<PHONE_NUMBER>027, vstest.console.exe, AssemblyResolver: Microsoft.VisualStudio.Telemetry, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a: Resolving assembly.
I, 3464, 1, 2017/02/15, 10:21:54.878,<PHONE_NUMBER>081, vstest.console.exe, AssemblyResolver: Microsoft.VisualStudio.Telemetry, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a: Resolved assembly.
V, 3464, 1, 2017/02/15, 10:21:54.987,<PHONE_NUMBER>015, vstest.console.exe, TestRunRequest.ExecuteAsync: Created test run request.
V, 3464, 1, 2017/02/15, 10:21:54.987,<PHONE_NUMBER>755, vstest.console.exe, TestRunRequest.ExecuteAsync: Starting.
I, 3464, 1, 2017/02/15, 10:21:54.987,<PHONE_NUMBER>772, vstest.console.exe, TestRunRequest.ExecuteAsync: Starting run with settings:TestRunCriteria:
KeepAlive=False,FrequencyOfRunStatsChangeEvent=10,RunStatsChangeEventTimeout=00:00:01.5000000,TestCaseFilter=Priority!=0,TestExecutorLauncher=Microsoft.VisualStudio.TestPlatform.Utilities.UniversalAppTestExecutorLauncher
Settingsxml=

C:\TestResults
X64
Framework45

V, 3464, 1, 2017/02/15, 10:21:54.987,<PHONE_NUMBER>388, vstest.console.exe, TestRunRequest.ExecuteAsync: Wait for the first run request is over.
V, 3464, 1, 2017/02/15, 10:21:54.987,<PHONE_NUMBER>433, vstest.console.exe, TestExecutorServiceClient: StartTestRun: Validated the parameters and updated the active context.
V, 3464, 1, 2017/02/15, 10:21:55.003,<PHONE_NUMBER>180, vstest.console.exe, TestExecutorServiceClient: ShouldRelaunchExecutorProcess: Restart executor process True. Previous launcher '', Current launcher 'Microsoft.VisualStudio.TestPlatform.Utilities.UniversalAppTestExecutorLauncher'
V, 3464, 1, 2017/02/15, 10:21:55.003,<PHONE_NUMBER>653, vstest.console.exe, TestExecutorServiceClient: EnsureInitialized: Relaunching executor.
V, 3464, 1, 2017/02/15, 10:21:55.050,<PHONE_NUMBER>992, vstest.console.exe, TestExecutorServiceClient: Initializing Proxy connection.
I, 3464, 1, 2017/02/15, 10:21:55.050,<PHONE_NUMBER>095, vstest.console.exe, UniversalTestExecutorProxy: Creating a test executor service using EndPointAddres: 'net.tcp://<IP_ADDRESS>:11000/UniversalTestExecutorService'
V, 3464, 1, 2017/02/15, 10:21:55.409,<PHONE_NUMBER>517, vstest.console.exe, Activating vstest.executionengine from elevated process
V, 3464, 1, 2017/02/15, 10:21:56.128,<PHONE_NUMBER>050, vstest.console.exe, LaunchActivator: Returned with processId:4932
V, 3464, 1, 2017/02/15, 10:21:56.128,<PHONE_NUMBER>704, vstest.console.exe, AppContainerUtilities.LaunchAppContainerTestExecutor: Started Windows app process: pid=4932 PackageFullName:eb61e043-d453-43ae-bda3-569724225115_<IP_ADDRESS>_x64__41zkwv185hb04 AppId:vstest.executionengine.universal.App.
V, 3464, 1, 2017/02/15, 10:21:56.128,<PHONE_NUMBER>346, vstest.console.exe, AppContainerUtilites.LaunchAppContainerTestExecutor: Time taken for launching Windows app Test Executor 00:00:00.7417164
I, 3464, 1, 2017/02/15, 10:22:56.181,<PHONE_NUMBER>409, vstest.console.exe, TestExecutorServiceClient: StartTestRun: Started the test run on the executor.
I, 3464, 1, 2017/02/15, 10:22:56.181,<PHONE_NUMBER>579, vstest.console.exe, TestRunRequest.ExecuteAsync: Started.
V, 3464, 1, 2017/02/15, 10:22:56.181,<PHONE_NUMBER>952, vstest.console.exe, RunTestsArgumentProcessor:Execute: Test run is queued.
V, 3464, 1, 2017/02/15, 10:22:56.181,<PHONE_NUMBER>426, vstest.console.exe, TestRunRequest.WaitForCompletion: Waiting with timeout -1

Please reopen the ticket @nigurr and sorry for the small delay in my answer.

@dersteve I didn't get the issue. Can you please describe in details what's expected and what's happening

I have a Windows Server 2016 machine where there is a TFS Build Agent running.
My build definition builds via the VS-Build task a solution that contains a UWP Test Project to execute UWP Unit tests. The test runner is vstest.console.exe as we use the normal VS/MS Unit Test libraries/infrastructure.
When I execute the build the test execution task fails with a timeout. When opening up the Windows Server 2016 the UWP Unit Test App is still open. It looks like the app can't communicate back with the test runner . The logs from the test runner task are the logs I've just added to this issue.

@dersteve Are you able to run the test on the 2016 build machine from cmdline directly ?

@cltshivash I get the same problem, when executing them from the cmdline - it also hangs - via vstest.console.exe

@dersteve I believe the changes called out here 2016 RTM are impacting this scenario. Please try running the tests on windows 10 and use a build agent on windows 10.


command line almost there

Description of the new feature/enhancement
I open 15 tabs and run programs process data during hours and print during those hours. I am actually using pyautogui to generate tabs, write command lines on each one, change tab color an title. And it works ok (sometimes there are some errors). Recently I learn the command line feature is actually doing this work and I tested it and work very efficiently (I did not chek all features).  The probllem I found (perhaps I did not fount the solution that already exists) is that once the program running in the tab finishs the tab is closed and I lose the printed results (20.000 lines or more in each console). programs/tab titles/tab colors/etc.

See above.
Proposed technical implementation details (optional)
My suggestion is that the default behaviour should not close the tab and might be a switch for closing it in the command line of creation of tabs/runing

run a bat with the whole command line that gets up all the consoles (opens tabs, run programs and keep all the information available)

These options may be of interest.

I think that solves the problem. I found (not easily) the menu you show.
Perhaps tomorrow I can try to configure all tabs.
thank you 😀
El lun, 26 abr 2021 a las 0:01, vefatica @.***>)
escribió:

These options may be of interest.
[image: image]
https://user-images.githubusercontent.com/61856645/116023440-15cc4080-a61a-11eb-886b-e334f4efa569.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/terminal/issues/9948#issuecomment-826468143,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT2XGAATAFBFYO75KONCQPLTKTJQLANCNFSM43R6MJGA
.

I'm glad it helps.  In the JSON file, you can get all profiles by putting it in "defaults".  The default setting can be overridden for specific profiles.
        "defaults":
        {
            "closeOnExit": "never",
            // ...
      },

Great! Glad you got this figured out ☺

Dear Mike, the first time I tested my computer freezed and I had to reset
it. I did not further tests until this weekend and everything worked fine
(I am not sure what happened before but I consider it a isolated case that
will not repeat). I have two questions:

When my programs end (launching them in this fashion (multiple consoles
form command line) there is an exit code 0 and the console can not be used
again. Is it possible to configure the console to return the command prompt
as is usually done at the end of running a program?

Is there a linux version of WINDOWS TERMINAL? Not to be run in WSL but
in linux.

regards,
Daniel
El jue, 29 abr 2021 a las 17:13, Mike Griese @.***>)
escribió:

Closed #9948 https://github.com/microsoft/terminal/issues/9948.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/terminal/issues/9948#event-4665026496, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AT2XGAHQSZCKKI2PRFW4SZ3TLG4VHANCNFSM43R6MJGA
.

I'm pretty sure the /k flag in cmd.exe will do what you're looking for. cmd.exe /k dir will execute a dir, then return to the cmd prompt. I'm sure there's a different way of doing it for PowerShell, but I'm more of a cmd.exe person myself 😄
Nope, and there's a whole plethora of threads elsewhere on the repo as to why that is.

I am reading your answer.
About... gives this version.
Windows Terminal
Versión: 1.11.2921.0
El jue, 21 oct 2021 a las 18:27, Mike Griese @.***>)
escribió:

This sounds like the recent change to the Terminal where tabs opened on
the commandline use the "base layer" rather than the "default profile" by
default. Could you confirm which version you're on now?
The two quick solutions would be:

for each subcommand in the commandline, manually specify the profile
who's settings you'd like to use (e.g. wt nt -p "Command Prompt" ; nt
-p "Command Prompt" -- pwsh.exe ; sp -p "Command Prompt", etc).
Move the settings you'd like to apply to all profiles into the
profiles.defaults section in your settings.json. E.g. these:
[image: image]
https://user-images.githubusercontent.com/18356694/138359094-6a235bb0-4764-431f-becc-b22cd24f5292.png

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/terminal/issues/9948#issuecomment-949016136,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT2XGAHFLD3APEVZXY6OBMDUICAURANCNFSM43R6MJGA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

adding "-p cmd" seems to solve the problem. I have to wait until all
process is done but at the beginning I have the proper background color and
the correct characters per line, so the aspect was corrected. I will wait
the processes to end to check how windows closes, but seems to correct that
also since the default is "closeOnExit": "graceful" and my setting is for
"closeOnExit": "never".
If so, I am back in my previous condition of operation. => SOLVED.
Thank you again, Daniel. 😀
El jue, 21 oct 2021 a las 18:53, Daniel Futterweit @.***>)
escribió:

I am reading your answer.
About... gives this version.
Windows Terminal
Versión: 1.11.2921.0
El jue, 21 oct 2021 a las 18:27, Mike Griese @.***>)
escribió:

This sounds like the recent change to the Terminal where tabs opened on
the commandline use the "base layer" rather than the "default profile" by
default. Could you confirm which version you're on now?
The two quick solutions would be:

for each subcommand in the commandline, manually specify the
profile who's settings you'd like to use (e.g. wt nt -p "Command
Prompt" ; nt -p "Command Prompt" -- pwsh.exe ; sp -p "Command Prompt",
etc).
Move the settings you'd like to apply to all profiles into the
profiles.defaults section in your settings.json. E.g. these:
[image: image]
https://user-images.githubusercontent.com/18356694/138359094-6a235bb0-4764-431f-becc-b22cd24f5292.png

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/terminal/issues/9948#issuecomment-949016136,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT2XGAHFLD3APEVZXY6OBMDUICAURANCNFSM43R6MJGA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Use defaults for unset monitor options

Using Nones causes mismatch with remote (which converts Nones to Azure's defaults) triggering record updates. This is a fix for that.
Also simplifying the very cumbersome-to-type monitor option keys (will update README after #2 goes in).

Sorry. Looks like the move of tests.py to tests/ caused a conflict. Hopefully simple to sort out.

nbd, git merge did its thing.


Can not generate a makefile

I have check that I install the SDL2 lib on my MacBook, and I try to use cmake to configure the makefile but it encounter some error
❯ cmake ..
CMake Error at CMakeLists.txt:17 (find_package):
Found package configuration file:
/Library/Frameworks/SDL2.framework/Resources/CMake/sdl2-config.cmake

but it set SDL2_FOUND to FALSE so package "SDL2" is considered to be NOT
FOUND.
-- Configuring incomplete, errors occurred!

I have solved it.

Closing this issue then 🙂
Please share your solution so that others can learn from it.


Implement generic zulip notes

The motive of adding BaseNotes was to support monokey patching
temporary attributes to objects (such as .trigger on Message) when
working on the django-stubs migration in #18777.
Testing plan:
This has been tested with existing test suites.

@abhijeetbodas2001 FYI -- do you think we could more cleanly handle trigger with a refactor using MessageNotificationData?
(I think the refactor to have a BaseNotes isn't a bad idea in any case)

I think it would be cleanest to just pass trigger as a new parameter to those functions which utilize it, because it's just a single field.
diff --git a/zerver/lib/push_notifications.py b/zerver/lib/push_notifications.py
index 3df81400ea..e0af7b07d5 100644
--- a/zerver/lib/push_notifications.py
+++ b/zerver/lib/push_notifications.py
@@ -718,19 +718,19 @@ def get_apns_alert_title(message: Message) -> str:

 def get_apns_alert_subtitle(
-    user_profile: UserProfile, message: Message, mentioned_user_group_name: Optional[str] = None
+    user_profile: UserProfile, message: Message, trigger: str, mentioned_user_group_name: Optional[str] = None
 ) -> str:
     """
     On an iOS notification, this is the second bolded line.
     """
-    if message.trigger == NotificationTriggers.MENTION:
+    if trigger == NotificationTriggers.MENTION:
         if mentioned_user_group_name is not None:
             return _("{full_name} mentioned @{user_group_name}:").format(
                 full_name=message.sender.full_name, user_group_name=mentioned_user_group_name
             )
         else:
             return _("{full_name} mentioned you:").format(full_name=message.sender.full_name)
-    elif message.trigger == NotificationTriggers.WILDCARD_MENTION:
+    elif trigger == NotificationTriggers.WILDCARD_MENTION:
         return _("{full_name} mentioned everyone:").format(full_name=message.sender.full_name)
     elif message.recipient.type == Recipient.PERSONAL:
         return ""

(We will still be creating MessageNotificationData instances here as part of #19451, but I don't think there's a need to pass those around even when we do so)

Yeah, that seems pretty reasonable for this.

OK! The requested changes have been made in the PR.

OK, this is great, merged, after doing a bit of editing to the docstrings.  Thanks @PIG208!
@andersk it might be worth your doing a quick look at zerver/lib/notes.py; I suspect that the Anys there can be replaced, but didn't have time to investigate.  (We could also decide to defer figuring that out until we finish #18777, we'll surely come across the Any when we next do a sweep for those).


upgrade dependencies

Codecov Report

Merging #26 (df629a0) into main (a7ab481) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #26   +/-   ##
=======================================
  Coverage   79.31%   79.31%
=======================================
  Files          10       10
  Lines         817      817
=======================================
  Hits          648      648
  Misses        118      118
  Partials       51       51

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 a7ab481...df629a0. Read the comment docs.


fix undefined constant error on dynamic slot names as key of object

It's solution for #42932
I've described problem in the issue.
With 9.15 and after we cannot use <x-slot :name="$model->name"> because it is rendering as
<?php $__env->slot($model>name, null, []); ?>
instead
<?php $__env->slot($model->name, null, []); ?>

<x-slot :name="$model->name">

That should work

@dennisprudlo I guess they want to pass it as a literal string.

@dennisprudlo I updated the example. Forgot to put :.

Please resubmit with a test.


Missing the example to set a valid chartPullSecret

If you want to install a helm chart from a private helm repository, please set a valid chartPullSecret by referring this example.
Here in the above page the exaple url redirects to empty page. There is no example to check how things can be implemented.
Please check this.
URL for the page is : https://clusternet.io/docs/tutorials/multi-cluster-apps/replication-scheduling-to-multiple-clusters/

This is a valid URL:  https://github.com/clusternet/clusternet/blob/main/deploy/templates/helm-chart-private-repo.yaml


Apr 3rd 2024, Community Meeting Agenda (#100)

📞 Dapr Community Meeting
Every two weeks we host a community meeting to highlight community created content, showcase new features, review upcoming milestones, and engage in Q&A with the Dapr community - all are welcome, and we encourage participation.
The purpose of this thread is to form a discussion amongst the Dapr community on potential topics to highlight during the meeting. If you have a topic you wish to present or learn more about, please comment and be sure to include your name and a short description of the topic.
Since this is Community Call 100 we're doing a special edition. We'll be live streaming to YouTube and not be using the regular Zoom call.

The conversations in this thread will directly influence the meeting agenda for the community call on , at 9 AM PST (See it in your local time). The meeting agenda is finalized by a Community Call Host prior to the meeting.
See you at the community call! 👋
Join the live YouTube Stream
Agenda

Introduction
Discussion with Yaron Schneider, Mark Fussell, Paul Yuknewicz, and Hal Spang, about the past, present, and future of Dapr.
Dapr Community Championship program
Kahoot Quiz
Closing

Checklists for the host
Before the meeting

[ ] Find speakers
[ ] Collect Dapr news (blog posts & videos found on social media, and show-and-tell Discord channel)
[ ] Prepare slide deck
[ ] Send meeting invite to the speakers
[ ] Announce the meeting on social media

[ ] 1 week before
[ ] 1 day before
[ ] 1 hour before

During the meeting

Welcome & announce that the meeting will be recorded
Start the recording
Show the agenda
Welcome first speaker
First session
Q&A first session
Welcome second speaker
Second session
Q&A second session
Show Dapr related news, posts, videos.
Open discussion
Closing & mention Dapr Discord channel

After the meeting

[ ] Upload the meeting to YouTube
[ ] Add chapters to the video
[ ] Add links to the video description

[ ] Read-only link to slides
[ ] Dapr Discord link
[ ] Other relevant links related to the sessions

[ ] Announce the published video on social media
[ ] Close this issue

We should do something special for this one. To start, it would be great to all the original creators and early adopters on.  Maybe invite Mark Russinovich too?
It might be interesting have a conversation about the early days of Dapr and talk about how the project started. Then we could transition to a conversation about where we all see the project going.
We should also consider doing some giveaways to the community.
Thoughts?

Recording: https://www.youtube.com/watch?v=5vyF6QACPwU&t=6s


preread, support TLCP ClientHello

TLCP, Transport layer cryptography protocol, GB/T 38636-2020

 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.


multivalue-control menu opens behind workitem window

Hi
I have TFS 2017 update 1 and I have installed version 1.0.5. of the control
I have a field populated with  SUGGESTEDVALUES just like I used to do in TFS 2015
but the menu (after looking for it...)  seems to open under the workitem window
I have tried chrome , chrome incognito window and IE ... all behave the same (under windows 10).
see attached screen capture for more details

@icnaan The screen shot you show does not look like this multi value control. This is what it looks like:


Populated police records with 20 contacts

Codecov Report

Merging #230 into master will increase coverage by 0.27%.
The diff coverage is 95.23%.

@@             Coverage Diff              @@
##             master     #230      +/-   ##
============================================
+ Coverage     71.93%   72.21%   +0.27%
- Complexity      587      588       +1
============================================
  Files            64       64
  Lines          2188     2206      +18
  Branches        226      226
============================================
+ Hits           1574     1593      +19
  Misses          534      534
+ Partials         80       79       -1

Impacted Files
Coverage Δ
Complexity Δ

src/seedu/addressbook/ui/Gui.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

src/seedu/addressbook/storage/StorageFile.java
82.08% <100%> (+6.57%)
9 <0> (ø)
:arrow_down:

src/seedu/addressbook/data/person/Offense.java
96.87% <0%> (+1.56%)
18% <0%> (+1%)
: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 ce96379...1206b2b. Read the comment docs.


Feature request: devbox shell -p somepackage

It makes it feel a lot like npx. Nothing to install or configure, I just run an isolated command and get results.

I think something like devbox run <package> could be a better syntax. It follows that of docker. The run signifies that devbox will execute the isolated command and exit. As opposed to shell, which given its current function, could imply that it will stay active after the command has been completed (i.e., a way to install a package in a shell without defining it in the config).

We're not totally set on how the CLI will do it, but we are looking into devbox run like functionality for an upcoming release. See #94 for more details and discussion

This is now available in Devbox 0.0.7! You can run a one off command in your devbox shell using:
devbox shell -- <cmd>

This will launch a shell based on your devbox.json, run your command, and then exit back to your normal shell. For example:
~ $  devbox shell -- echo "Hello World"
Installing nix packages. This may take a while...
Starting a devbox shell...
Hello World
~ $

More info is in our new CLI Docs: https://www.jetpack.io/devbox/docs/cli_reference/devbox_shell/


[DOC] Feedback on Search topics

What do you want to do?
x Request a change to existing documentation
Tell us about your request. Provide a summary of the request and all versions that are affected.
Feedback from the community on Search topics:

[ ] 20230501 | /docs/latest/search-plugins/knn/knn-index/ |The documentation doesn't provide a way how we can create an index with method as IVF and engine as Faiss. There is no sample payload. It is unusable

[x] 20230606 | /docs/latest/search-plugins/search-pipelines/index/ | Missing the flags needed to enable the experimental feature for testing.

[ ] 20230504 |/docs/latest/search-plugins/searching-data/paginate/ |none of the methods are not working, when I put from as 8000 and size as 10000 the throwing error , like they adding from and size together and saying it is more than 10000 .

[ ] 20230525 |/docs/latest/search-plugins/searching-data/sort/ |Creating a sort filter on the Discovery page does not provide correct results.  Also, the Time column

[ ] 20230410 |/docs/latest/search-plugins/sql/ppl/functions/ |How to search for a text

[ ] 20230522 | /docs/latest/search-plugins/sql/sql/basic/ COUNT (distinct xxx) is not supported

[ ] 20230522 |/docs/latest/search-plugins/sql/sql/complex/ |no support for distinct

[ ] 20230428 |/docs/2.3/search-plugins/sql/index/ |0 examples, nothing useful other than the statement that SQL and PPL are supported

[ ] 20230505 |/docs/2.3/search-plugins/sql/sql-ppl-api/ | Missing more information on _explain.
Like using 'WHERE field IN' parameters

[ ] 20230302 |/docs/2.4/search-plugins/sql/sql/complex/ |subquery barely works. often doesn't work at all even for queries same to examples

[ ] 20230425 |/docs/2.5/search-plugins/sql/full-text/ |I am looking for more detailed description about multi_match options. In this article you have only a list of options and a misleading statement in the bottom "Please, refer to multi_match query documentation for parameter description and supported values." which points to itself.

[ ] 20230228 | /docs/2.5/neural-search-plugin/index/ | - Reverse order documentation to follow it

Show me the pipeline name

Try to add placeholder for

What other resources are available? Provide links to related issues, POCs, steps for testing, etc.
https://opensearch.org/docs/latest/search-plugins/index/

@kolchfa-aws , does this PR https://github.com/opensearch-project/documentation-website/pull/4476 address 20230501 above? Thanks.

@hdhalter No, it doesn't

@hdhalter This one is Dashboards-related and should be moved to the Dashboards feedback

[ ] 20230525 |/docs/latest/search-plugins/searching-data/sort/ |Creating a sort filter on the Discovery page does not provide correct results. Also, the Time column

[ ] 20231219 | /docs/2.11/search-plugins/sql/sql/partiql/ | I tried this with v2.11.1 but the query is retruning error: QueryShardException[failed to create query: [nested] nested object under path [projects] is not of nested type]
[ ] 20231204 | /docs/latest/search-plugins/neural-search-tutorial/ | Add python code examples on how to use these.
[ ] 20231222 | /docs/latest/search-plugins/searching-data/paginate/ | Scroll operation not clearly understandable, like "/_search?scroll=10m"in this meaning of sroll arg is not explained,
[ ] 20231127 | /docs/latest/search-plugins/sql/ppl/syntax/ | with 3 basic example? is this a seroius question?


Snap applications don't show

Sorry if this is a dupe of 82, but it seems App Outlet isn't even trying to get a list of available snap apps. If that is the case, is it pointing to a larger problem in the future or it's just a bug to be fixed (and App Outlet is temporarily disabling the population of the Snap listing). If it's a bug, I would like to help as much as I can and do some investigation on various guest distros in a VM on my Debian. Looking forward to your answer.

When we get a list of available apps, we do not do it separately. they are fetched all together in the same request. The filter of the type of package is made locally.


(docs) Updated integration.md: Added Walrus & Walrus Site Builder section

Closes #104
Also refers to https://github.com/MystenLabs/walrus-sites/issues/94

@mario4tier Please review my Walrus integration guide when you have time. Thx

All good. Thanks.

Thanks


allow updating chromadb version

Notes:

I disabled the decorator @retry_with_exponential_backoff inside the callable class and solved the chroma problem.
The current errors are due to the absence of hf-embeddings. Shouldn't hf-embeddings be installed as part of Langroid, otherwise, the test cases should be under extra.

@Mohannadcse this looks good
HF had an outage yesterday, that may be the reason for the hf-embed err
Also I restored the @Retry decorator, and tests seem to pass. Let me know if you noticed some other issue.

never mind, I see the issue with the retry wrapper now. I had not updated the chroma version in my local.

I still need to fix these errors. Before pushing make check wasn't showing them, but I noticed them in my last push.

ok . for the retry, use tenacity as shown here in example 1 https://platform.openai.com/docs/guides/rate-limits/error-mitigation?context=tier-free
inside the __call__, define a fn

I tried this suggestion but didn't work, was raising an error


Ignored custom eslint plugin rules

Hello, I noticed a format bug with a custom plugin.
Here I leave an example https://github.com/vladi-strilets/vs-code-prettier-eslint
Just clone the master, do yarn, go to the src/test.js and format with the prettier-eslint extension.
I have added a new plugin https://github.com/kusotenpa/eslint-plugin-destructuring-newline
The problem is that this code has no been formatted.
const { a, b } = foo;

Expected result:
const {
  a,
  b,
} = foo;

I have installed prettier-eslint-cli to check if I was a problem with prettier-eslint, but it works well, you can check it by;
yarn prettier-eslint --write src/test.js

Any ideas, thanks!

@vladi-strilets Thank you for making an issue!! Thanks for making a reproducible example repo as well. I'll check into ASAP

@vladi-strilets What version of the prettier-eslint-cli did you install<EMAIL_ADDRESS>requires ESLint 7 and my plugin only supports 6.8.0 for the time being.

@idahogurl
I found a stranger behavior with prettier-eslint.
When I pass the text param to the  format function, it returns a formated string but without applying the object restructuring rule.
Meanwhile, passing the filePath prop of the file, the object restructuring rule is applied well.
Here I have updated the example:
https://github.com/vladi-strilets/vs-code-prettier-eslint/blob/master/src/test.js
Just node src/test.js
Anyway, I don't know why, but now my VSCode applies the rule for the test2.js file. However, in my other project, the same rule just underlines the problem but can't fix it. Maybe it's some config conflict.
One more thing, is it possible to update eslint and prettier versions of the package?
Thanks!

@vladi-strilets If the format function doesn't work then my extension won't ever work. Could you follow the troubleshooting steps I have in the README to see if an error shows up in the Output panel?

Closing due to over 30 days of inactivity.


PostgreSQL autocomplete is not properly escaping table names

Description
This was working previously but is broken in version <IP_ADDRESS>310081633 on macOS.
If I write a postgresql query for a table with Pascal casing (e.g. Users), the autocomplete is incorrectly suggesting the table name without quotes. This query will throw an error in postgresql.
SELECT * FROM Users;
Previously, dBeaver would correctly surround the table name with quotes:
SELECT * FROM "Users";
DBeaver Version
<IP_ADDRESS>310081633
Operating System
macOS Sonoma 14.0
Database and driver
Database: PostgreSQL 14.6 running in Docker container with alpine-linux
Steps to reproduce
Write any query with a table name that starts with a capital letter. DBeaver autocomplete is not wrapping in quotes in the autocomplete.
Additional context
No response

I can't reproduce you issue.
The suggestions

The query after applying suggestion

Could you show your Code Completion preference page for the connection?

Sorry - not having the exact same issue as op:
It is correctly quoting table names for me but it is NOT quoting column names in a where clause.  It does quote column names in the result columns though - very weird.
So this is the result with all of the quoting applied as it's happening for me:
select "Name" from "Employee" where HireDate > '2023-10-01'
Only "HireDate" is not quoting for me.

Ugh. My apologies, it is not the table name population, it is the column names in the where clause for me as well. Will close and open a new ticket.

Closed due to inaccurate description- use this issue instead https://github.com/dbeaver/dbeaver/issues/21589


⚠ Bofrost.lu has degraded performance

In 27b8af7, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2815 ms

Resolved: Bofrost.lu performance has improved in f044a8a after 14 minutes.


Adds support for associated types

Adds support for accessing all associated types declared in a protocol including their type constraints.
Fixes krzysztofzablocki/Sourcery#539

@krzysztofzablocki @ilyapuchka please merge if seems fine or let me know if improvements needed, Thanks.

thanks for contributing 🙇


Data Science with Python : Sentiment Analysis

Description 📜
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes #419

Type of change 📝

[ ] Audio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Video (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev (Flutter, Java, Kotlin)
[ ] Backend Dev (Java, .NET, Python)
[ ] C/CPP
[ ] Competitive Programming
[ ] Cyber Security
[ ] DSA
[ ] Database
[x] Datascience with Python
[ ] Datascience with R
[ ] Frontend Dev HTML/CSS/JS
[ ] Frontend Dev React/Angular/Vue
[ ] Golang
[ ] Interview Prep
[ ] Java_Domain
[ ] JavaScript
[ ] MERN
[ ] Machine Learning
[ ] Open Source
[ ] Python_Domain
[ ] Rust
[ ] Statistics
[ ] UI/UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

This PR is currently under reviewing process

Ok will work on it.

@prathimacode-hub I cannot upload the dataset which I have used because it's a large file greater than 25MB. So I have mentioned the link in README.md and also in my .ipynb file.
I have done all changes that were required. Let me know if there are any other changes.👍🏻


make deploy-to-bosh-lite compatible with CentOS 7

The git package version in CentOS 7 repo is <IP_ADDRESS>, which doesn't support -C option.

I don't have a CLA with CF org. And it seems too much hassle to attain one. If the policy of CF doesn't allow merging changes from developer without CLA, feel free to take my changes and resummit it by someone else with the authority.

We have done a large re-write of this script to use some newer tooling (golang bosh cli, bosh-deployment repo and cf-deployment repo).  The script no longer calls git at all.


Bump h2

Problem
Cargo audit failure
Summary of Changes
Bump h2 to v0.3.26

Sorry, I forgot that we have to run cargo update on programs/sbf/Cargo.toml directly instead of ./scripts/cargo-for-all-lock-files.sh tree


How to download highlight.js in Windows

Hi,
I am trying to deploy my docs in my SoilWater-ToolBox
I migrated from GitHub pages to Franklin.jl . I think that one of the problems is that I have not installed highlight.js
The problem is that the example you give seems to only work on Linux and not on Windows. Would you kindly let me know how to download highlight.js in Windows?
julia> using NodeJS julia> run(sudo $(npm_cmd()) install highlight.js)
Many thanks for any help you may provide,
Joseph

Hello, I can see your site at the address you pointed, is there something that doesn't work?
The message about highlight can be ignored, if Franklin doesn't find it, it will use a fallback mechanism (instead of pre-rendering the code into HTML, the highlight JavaScript library is loaded on the browser to do the same thing)
To answer your question, NodeJS.jl is the portable way of installing the lib, did you encounter an issue running the installation that way (calling the NodeJS command) or is it just that Franklin complains it can't find it?

Ah you may not have set your prepath correctly, can you check? see here: https://franklinjl.org/workflow/deploy/

Dear Thibaut,
A hallo from New Zealand.
A great thanks for your kindness. I spend till midnight last night trying to get the  pure-sm team and I believe that I followed https://franklinjl.org/workflow/deploy/https://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Ffranklinjl.org%2Fworkflow%2Fdeploy%2F&data=04|01|pollaccoj%40landcareresearch.co.nz|6ad87f9e4f674c95638a08da1138136e|43050530b3c74cd2a11cb826b2604b5b|0|0|637841229825301031|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=eKWGwIfCRk0ZfgBzYITII%2FybfNR6uDdCu3JG84yRAg0%3D&reserved=0, which to my greatest regret failed. Also the maths katex failed so I used MathJax instead. I am wandering if a quick Teams would not work, for that I would need your email so I can send you an invitation. It is now 17h00 and I am free until 18h00.
There is also some strange issues with @def which seems to be an old way of doing things.
https://franklinjl.org/workflow/deploy/https://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Ffranklinjl.org%2Fworkflow%2Fdeploy%2F&data=04|01|pollaccoj%40landcareresearch.co.nz|6ad87f9e4f674c95638a08da1138136e|43050530b3c74cd2a11cb826b2604b5b|0|0|637841229825301031|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=eKWGwIfCRk0ZfgBzYITII%2FybfNR6uDdCu3JG84yRAg0%3D&reserved=0
@.***
Many thanks for any help you may provide,
Joseph
From: Thibaut Lienart @.>
Sent: Tuesday, 29 March 2022 4:56 PM
To: tlienart/Franklin.jl @.>
Cc: Joseph Pollacco @.>; Author @.>
Subject: Re: [tlienart/Franklin.jl] How to download highlight.js in Windows (Issue #951)
Ah you may not have set your prepath correctly, can you check? see here: https://franklinjl.org/workflow/deploy/https://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Ffranklinjl.org%2Fworkflow%2Fdeploy%2F&data=04|01|pollaccoj%40landcareresearch.co.nz|6ad87f9e4f674c95638a08da1138136e|43050530b3c74cd2a11cb826b2604b5b|0|0|637841229825301031|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=eKWGwIfCRk0ZfgBzYITII%2FybfNR6uDdCu3JG84yRAg0%3D&reserved=0

Reply to this email directly, view it on GitHubhttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftlienart%2FFranklin.jl%2Fissues%2F951%23issuecomment-1081373898&data=04|01|pollaccoj%40landcareresearch.co.nz|6ad87f9e4f674c95638a08da1138136e|43050530b3c74cd2a11cb826b2604b5b|0|0|637841229825301031|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=fPEim5QLV3wOSlvDlf0YWGI8HJGkvVOF9i2JwAibNfs%3D&reserved=0, or unsubscribehttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACVGEEDNJYVRJJAEIRLD5RDVCJ5N5ANCNFSM5R5CCMQA&data=04|01|pollaccoj%40landcareresearch.co.nz|6ad87f9e4f674c95638a08da1138136e|43050530b3c74cd2a11cb826b2604b5b|0|0|637841229825301031|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=cVd5G0T5edxhWVJy2q3fCfV7mbNHVi%2B1Z4v%2BAAJ87Ns%3D&reserved=0.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
Please consider the environment before printing this email
Warning: This electronic message together with any attachments is confidential. If you receive it in error: (i) you must not read, use, disclose, copy or retain it; (ii) please contact the sender immediately by reply email and then delete the emails.
The views expressed in this email may not be those of Landcare Research New Zealand Limited. http://www.landcareresearch.co.nz

I'm about to board a plane, so not the most convenient time for a chat.
It would be useful if you told me what failed (what error). Also if you were able to get the website to work locally (when you call serve) or not.
In the latter case, can you try setting up a new sandbox site newsite("testSite") in a separate folder, call serve() without having modified anything, and tell me whether the math and code show properly in your browser? (It should)
If it's just when you deploy your website that things don't work, I believe your prepath is not set correctly, eg if you want your landing page to be (...).io/SoilWater.jl/ you need to specify the prepath in config.md to prepath = "SoilWater.jl", if you do not do that, your paths to the CSS and JavaScript will be incorrect when you deploy and the math and code (and styling) wont work.
There's no need for highlight.js nor mathjax, this is independent of the platform you're on.

Dear Thibaut Lienart,
Thanks for providing me with your explanations. The maths is now working without using mathjax.
When I deploy by using serve() it works and the website is beautiful. But when I deploy to github than it does not deploy properly as you could see:
https://manaakiwhenua.github.io/SoilWater_ToolBox/HyPix/HyPix_Output
My GitHub repository is as follow:
https://github.com/manaakiwhenua/SoilWater_ToolBox
Below is the Config.md file:

+++
author = "Joseph Pollacco, Jesus-0Fernandez"
mintoclevel = 2
Add here files or directories that should be ignored by Franklin, otherwise
these files might be copied and, if markdown, processed by Franklin which
you might not want. Indicate directories by ending the name with a /.
Base files such as LICENSE.md and README.md are ignored by default.
ignore = ["node_modules/"]
RSS (the website_{title, descr, url} must be defined to get RSS)
generate_rss = true
website_title = "SoilWater ToolBox"
website_descr = "SoilWater_ToolBox"
website_url   = https://manaakiwhenua.github.io/SoilWater_ToolBox/
prepath = "SoilWater_ToolBox"
+++

\newcommand{\R}{\mathbb R}
\newcommand{\scal}[1]{\langle #1 \rangle}
Many thanks for all the help you may provide,
Joseph
From: Thibaut Lienart @.>
Sent: Tuesday, 29 March 2022 5:35 PM
To: tlienart/Franklin.jl @.>
Cc: Joseph Pollacco @.>; Author @.>
Subject: Re: [tlienart/Franklin.jl] How to download highlight.js in Windows (Issue #951)
I'm about to board a plane, so not the most convenient time for a chat.
It would be useful if you told me what failed (what error). Also if you were able to get the website to work locally (when you call serve) or not.
In the latter case, can you try setting up a new sandbox site newsite("testSite") in a separate folder, call serve() without having modified anything, and tell me whether the math and code show properly in your browser? (It should)
If it's just when you deploy your website that things don't work, I believe your prepath is not set correctly, eg if you want your landing page to be (...).io/SoilWater.jl/ you need to specify the prepath in config.md to prepath = "SoilWater.jl", if you do not do that, your paths to the CSS and JavaScript will be incorrect when you deploy and the math and code (and styling) wont work.
There's no need for highlight.js nor mathjax, this is independent of the platform you're on.

Reply to this email directly, view it on GitHubhttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftlienart%2FFranklin.jl%2Fissues%2F951%23issuecomment-1081392503&data=04|01|pollaccoj%40landcareresearch.co.nz|ebc99c13c8844058eddd08da113d7aa4|43050530b3c74cd2a11cb826b2604b5b|0|0|637841252999147173|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=kFnOZ0wMCBjORlT51fYsYFuCJ5iRFXztT7aG2k84RV8%3D&reserved=0, or unsubscribehttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACVGEEFNEUNBIJTR3LZK4B3VCKB65ANCNFSM5R5CCMQA&data=04|01|pollaccoj%40landcareresearch.co.nz|ebc99c13c8844058eddd08da113d7aa4|43050530b3c74cd2a11cb826b2604b5b|0|0|637841252999147173|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=DGcCD9eEXkR6kw0hl8deEvB%2FUdDEp374JFtjKdf4uwY%3D&reserved=0.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
Please consider the environment before printing this email
Warning: This electronic message together with any attachments is confidential. If you receive it in error: (i) you must not read, use, disclose, copy or retain it; (ii) please contact the sender immediately by reply email and then delete the emails.
The views expressed in this email may not be those of Landcare Research New Zealand Limited. http://www.landcareresearch.co.nz

I'll close this as I think we fixed the issues on the repo and the website is live

Dear Thibaut,
A great thanks for your help the website is working well will need now to complete it.
Have a good easter,
Joseph
From: Thibaut Lienart @.>
Sent: Tuesday, 12 April 2022 6:59 PM
To: tlienart/Franklin.jl @.>
Cc: Joseph Pollacco @.>; Author @.>
Subject: Re: [tlienart/Franklin.jl] How to download highlight.js in Windows (Issue #951)
I'll close this as I think we fixed the issues on the repo and the website is live

Reply to this email directly, view it on GitHubhttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftlienart%2FFranklin.jl%2Fissues%2F951%23issuecomment-1096200040&data=04|01|pollaccoj%40landcareresearch.co.nz|5824aa9e60ee4d326c6a08da1c51f979|43050530b3c74cd2a11cb826b2604b5b|0|0|637853435653835851|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=wTF6L9GbUYCvd%2BeFcfZ6udVS8dpNaMV%2BMKzIKudr0FM%3D&reserved=0, or unsubscribehttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACVGEEFQWXNDZ36F2OBAE7LVEUNMRANCNFSM5R5CCMQA&data=04|01|pollaccoj%40landcareresearch.co.nz|5824aa9e60ee4d326c6a08da1c51f979|43050530b3c74cd2a11cb826b2604b5b|0|0|637853435653992096|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=75mFPFY6uLP92GGdwmPKl3IcWxwSOoq9%2BaXXA4K%2BiPQ%3D&reserved=0.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
Please consider the environment before printing this email
Warning: This electronic message together with any attachments is confidential. If you receive it in error: (i) you must not read, use, disclose, copy or retain it; (ii) please contact the sender immediately by reply email and then delete the emails.
The views expressed in this email may not be those of Landcare Research New Zealand Limited. http://www.landcareresearch.co.nz


Appstudio update test-component-pac-xihd

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-xihd-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
14 seconds
clone-repository

✅ Succeeded
37 seconds
build-container

✅ Succeeded
22 seconds
deprecated-base-image-check

✅ Succeeded
23 seconds
inspect-image

✅ Succeeded
8 seconds
clair-scan

✅ Succeeded
35 seconds
clamav-scan

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
22 seconds
label-check

✅ Succeeded
7 seconds
show-sbom

✅ Succeeded
6 seconds
show-summary


ContextSwitchDeadlock exception

it raised an exceptiono named "ContextSwitchDeadlock" while I'm using the below code:
xrmBrowser.LoginPage.Login(_xrmUri, _username, _password);
xrmBrowser.GuidedHelp.CloseGuidedHelp();
xrmBrowser.ThinkTime(500);
xrmBrowser.CommandBar.ClickCommand("CREATE SR");
xrmBrowser.Driver.WaitUntilVisible(OpenQA.Selenium.By.Id("hw_contactphonenumber"));  //raised at the line
xrmBrowser.Entity.SetValue("hw_contactphonenum", "15914314812");
or  set the think time more longer ,like 3000ms , it also raised this exception .

Thanks for raising this @skytown32. First off - we want to apologize for the apparent delay in reply to your submission. We've been monitoring issues for quite some time and this one never appeared in the issues list up until two days ago. We spent some time looking into it, but have been unable to determine why this issue was not originally visible to us.
In regards to your exception, the ContextSwitchDeadlock doesn't necessarily mean your code has an issue, just that there is a potential for problem. If you go to Debug > Exceptions in the menu and expand the Managed Debugging Assistants, you will find ContextSwitchDeadlock is enabled. If you disable this, VS will no longer warn you when items are taking a long time to process. In some cases you may validly have a long-running operation. It's also helpful if you are debugging and have stopped on a line while this is processing - you don't want it to complain before you've had a chance to dig into an issue.

Is this exception still occurring for you with the latest release? I'd expect so since this is related to Visual Studio configuration, but would like to confirm.

Is the exception blocking your tests from completed successfully with a pass marker?

Regards,
Tyler

@skytown32 - If you're still facing this issue, please re-open for additional review.
Thanks,
Tyler


[incubator-kie-issues-1131] test migration from V7 to code generation-17

In addition to previous changes associated,
Migrated tests within VariableTagsTest.java
The test can be identified by referring to VariableTagsTest.java:
https://github.com/apache/incubator-kie-kogito-runtimes/blob/main/jbpm/jbpm-tests/src/test/java/org/jbpm/bpmn2/VariableTagsTest.java
Closes https://github.com/apache/incubator-kie-issues/issues/1131.

PR job #1 was: UNSTABLE
Possible explanation: This should be test failures

Reproducer
build-chain build full_downstream  -f 'https://raw.githubusercontent.com/${AUTHOR:apache}/incubator-kie-kogito-pipelines/${BRANCH:main}/.ci/buildchain-config-pr-cdb.yaml' -o 'bc' -p apache/incubator-kie-kogito-runtimes -u https://github.com/apache/incubator-kie-kogito-runtimes/pull/3618 --skipParallelCheckout
NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution

Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest_jobs/job/kogito-runtimes-pr/job/PR-3618/1/display/redirect
Test results:

PASSED: 3290
FAILED: 1

Those are the test failures:

io.quarkus.it.kogito.devmode.DevMojoIT.testDRLHotReload
Condition with io.quarkus.it.kogito.devmode.DevMojoIT was not fulfilled within 2 minutes.


Maximum update depth exceeded when draggin a very simple node.

Describe the Bug
I am getting the react maximum update depth exceeded when dragin a node that is very simple, i am also noting lag when draggin nodes:
react_devtools_backend.js:2357  Warning: Maximum update depth exceeded. This can happen when a component calls setState inside useEffect, but useEffect either doesn't have a dependency array, or one of the dependencies changes on every render.
at InputBase2 (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/chunk-UNTGWZEE.js?v=e205334d:354:17)
at OutlinedInput2 (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/chunk-UNTGWZEE.js?v=e205334d:1875:17)
at div
at http://<IP_ADDRESS>:5174/node_modules/.vite/deps/chunk-LDKA362W.js?v=e205334d:3630:51
at FormControl2 (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/chunk-UNTGWZEE.js?v=e205334d:2562:17)
at http://<IP_ADDRESS>:5174/node_modules/.vite/deps/chunk-LDKA362W.js?v=e205334d:3630:51
at TextField2 (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/chunk-UNTGWZEE.js?v=e205334d:4113:17)
at PostIt (http://<IP_ADDRESS>:5174/src/customNodes/PostIt.jsx?t=1682685762093:22:3)
at div
at NodeWrapper (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/reactflow.js?v=e205334d:5180:30)
at div
at NodeRenderer (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/reactflow.js?v=e205334d:6360:115)
at div
at Viewport (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/reactflow.js?v=e205334d:6567:31)
at div
at http://<IP_ADDRESS>:5174/node_modules/.vite/deps/reactflow.js?v=e205334d:6179:38
at div
at ZoomPane (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/reactflow.js?v=e205334d:5840:19)
at FlowRenderer (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/reactflow.js?v=e205334d:6336:33)
at GraphView (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/reactflow.js?v=e205334d:6659:20)
at Wrapper (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/reactflow.js?v=e205334d:6930:28)
at div
at http://<IP_ADDRESS>:5174/node_modules/.vite/deps/reactflow.js?v=e205334d:6975:49
at div
at Workflow (http://<IP_ADDRESS>:5174/src/screens/Workflow.jsx?t=1682686183853:57:17)
at ReactFlowProvider (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/reactflow.js?v=e205334d:6922:38)
at WorkflowWithProvider
at RenderedRoute (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/react-router-dom.js?v=e205334d:3115:5)
at Outlet (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/react-router-dom.js?v=e205334d:3423:26)
at Dashboard (http://<IP_ADDRESS>:5174/src/screens/Dashboard.jsx:40:17)
at RenderedRoute (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/react-router-dom.js?v=e205334d:3115:5)
at Routes (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/react-router-dom.js?v=e205334d:3483:5)
at ThemeProvider2 (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/chunk-LDKA362W.js?v=e205334d:6819:5)
at ThemeProvider3 (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/chunk-LDKA362W.js?v=e205334d:6906:5)
at ThemeProvider (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/chunk-JI7TGQF7.js?v=e205334d:285:12)
at QueryClientProvider (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/@tanstack_react-query.js?v=e205334d:2663:3)
at App (http://<IP_ADDRESS>:5174/src/App.jsx?t=1682686183853:29:27)
at div
at Router (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/react-router-dom.js?v=e205334d:3430:15)
at BrowserRouter (http://<IP_ADDRESS>:5174/node_modules/.vite/deps/react-router-dom.js?v=e205334d:3851:5)
overrideMethod @ react_devtools_backend.js:2357
printWarning @ react-dom.development.js:86
error @ react-dom.development.js:60
checkForNestedUpdates @ react-dom.development.js:27300
scheduleUpdateOnFiber @ react-dom.development.js:25475
dispatchSetState @ react-dom.development.js:17527
(anonymous) @ InputBase.js:410
commitHookEffectListMount @ react-dom.development.js:23150
commitPassiveMountOnFiber @ react-dom.development.js:24926
commitPassiveMountEffects_complete @ react-dom.development.js:24891
commitPassiveMountEffects_begin @ react-dom.development.js:24878
commitPassiveMountEffects @ react-dom.development.js:24866
flushPassiveEffectsImpl @ react-dom.development.js:27039
flushPassiveEffects @ react-dom.development.js:26984
commitRootImpl @ react-dom.development.js:26935
commitRoot @ react-dom.development.js:26682
performSyncWorkOnRoot @ react-dom.development.js:26117
flushSyncCallbacks @ react-dom.development.js:12042
(anonymous) @ react-dom.development.js:25651
Show 18 more frames
react_devtools_backend.js:2357  Warning: Maximum update depth exceeded.
Add some new nodes, including a post it node.
Complete the nodes with some data.
Start dragging the post it node until you get the error.

Expected behavior
I would like to have a smooth draggin experience without the maximum update depth exceeded react error.
Screenshots or Videos
https://user-images.githubusercontent.com/23506803/235236315-b5444fbc-ee3e-4bcc-ab1d-0f01566e2347.mov
Platform

OS: macOS 13.3.1
Browser: Edge
Version: 112.0.1722.58 (arm64)

Additional context
No response

Hey! Can you create a codesandbox for this? https://new.reactflow.dev/react-js

i create it here:
https://codesandbox.io/s/musing-merkle-f18de6
if you navigate to /Workflows/1 you can see an example similar to my video.
But i am noting that in the sandbox i dont have the lag on drag and not having the same error of Maximum update depth exceeded, any ideas?
Thanks!

Does it lag locally when you close the dev tools?

I am experiencing something similar to this as well, 67 nodes and panning becomes almost impossible with dev tools, however it becomes smooth when dev tools are closed.

Also facing the same issue when try to move the node getting maximum depth error.

Please provide a repo or codesandbox. I can't reproduce this locally.

@moklick Sorry but the issue isn't with React Flow itself 😅 It's the meta information Offcanvas component that I've implemented which when selecting a node causes it to open. I've commented it out and it works fine now

@FarrukhXeb how are you accessing the nodes in the meta info panel?

@moklick I'm trying to access the selected node and open up the meta info panel
 const [nodes, setNodes, onNodesChange] = useNodesState(initialNodes);
 // I think this is causing an issue or somehow
  const selectedNode = useMemo(
    () => nodes.find((node) => node.selected && node.type ==='rectangle'),
    [nodes],
  );

   return (
   <ReactFlowProvider>
     <ReactFlow {...someProps} />
     <Offcanvas
        isOpen={!!selectedNode}
        direction='bottom'
        title={selectedNode?.data?.label}
        backdrop={false}
      >
        <MetaInfo node={selectedNode!} />
      </Offcanvas>
   </ReactFlowProvider/>
  )

Don't pass anything to MetaInfo but access the selected node like this (in MetaInfo)
const selectedNode = useStore(s => s.getNodes().filter(node => node.selected && node.type ==='rectangle'));

@moklick thanks a bunch!
https://github.com/wbkd/react-flow/assets/43153347/3de80fe4-aa13-46f2-b843-bd3235b6e81c
This is what I did I created wrapper of Offcanvas
import { useStore } from 'reactflow';
import Offcanvas from '../CommonElements/Offcanvas';
import MetaInfo from './MetaInfo';
import { NODE_TYPES } from './Nodes';
const MetaInfoCanvas = () => {
  const { resetSelectedElements } = useStore((store) => store);
  const selectedNode = useStore(
    (s) => s.getNodes().filter((node) => node.selected && node.type === NODE_TYPES.CUSTOM_NODE)[0],
  );
  return (
    <Offcanvas
      isOpen={!!selectedNode}
      toggle={() => {
        resetSelectedElements();
      }}
      direction='bottom'
      title={selectedNode?.data?.label}
      backdrop={false}
    >
      <MetaInfo />
    </Offcanvas>
  );
};

export default MetaInfoCanvas;

you don't want to do this:
const { resetSelectedElements } = useStore((store) => store);

but
const resetSelectedElements = useStore((store) => store.resetSelectedElements);

otherwise the component would re-render whenever any item in the store changes.

Good Afternoon, Im using Redux Toolkit with react flow, and while using "onNodesChange" from the library itself in the store, the node lags like hell and if I use use useNodesState in the component itself then it works fine. Early help would really be appreciated.


data-device: check if there's a source before sending dnd_finish

This could cause a segfault in data_offer_destroy if the offer has version < 3.
Fixes https://github.com/swaywm/wlroots/issues/1570

Thanks!


Wrong link in documentation for Control.Layers

How to reproduce

Leaflet version I'm using: N/A
Browser (with version) I'm using: N/A
OS/Platform (with version) I'm using: N/A
step 1: Go to http://leafletjs.com/reference-1.0.3.html#control-layers
step 2: Click " detailed example"

What behaviour I'm expecting and which behaviour I'm seeing
I expect to be taken to the layer groups example (http://leafletjs.com/examples/layers-control/)
But Instead I'm being taken to an error page.

Good catch!
Could you please edit https://github.com/Leaflet/Leaflet/blob/master/src/control/Control.Layers.js#L13 and send a pull request with the change? That'll speed things up.

Closed by #5568.


Use latest version of DiagnosticSource in NSwag.Console for .NET 4.

This makes it possible to use it with projects which reference latest version of DiagnosticSource.
Fixes #3292

I'm a bit worried that this is a breaking change for some.

@RicoSuter NSwag.Console already had transitive reference to System.Diagnostics.DiagnosticSource, so all this does is makes it explicit in order to bump up the referenced version. In what scenario do you think it might be a breaking change?

I think it's fine. Thanks for the PR.

In what scenario do you think it might be a breaking change?

Usually i'm quite hesitant to upgrade libs in own libs - as this might force consumers to upgrade... but here it's the host and probably fine.


Update dead link to Go Chaincode SDK

Type of change

Documentation update

Description
The link to the go chaincode SDK was missing and the documentation was pointing to the old location of the shim package.
Additional details
N/A
Related issues
N/A

Need to check if these is needed in master as well...

It makes sense to have the 1.4 doc point to the release-1.4 branch of the shim package. The master branch already points to the new version of the chaincode sdk and API.

@fasmat I agree 1.4 doc should point to the relevant release-1.4 code. This PR has been merged, but you could update in a subsequent PR.

See #479


Docs.MS Release Updates for Azure.Storage.Blobs.ChangeFeed

Update docs metadata and targeting for release of Azure.Storage.Blobs.ChangeFeed

Docs Build status updates of commit 78bd03b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/overview/azure/storage.blobs.changefeed-readme-pre.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Created Arduino Code for Autonomous Rover

Hey there,
In response to our last email conversation, I wasn't aware of the stated
fact. I'll create another PR in the repo that you mentioned in your
previous email.
And for this repo, I'll look to create a hello world code too. Kindly
consider the PR if you find it appropriate for the repository.
Thanks & Best Regards,
Abhishek Gaurav
On Mon, 3 Oct, 2022, 2:43 am Rishit Saiya, @.***> wrote:

@.**** requested changes on this pull request.
Hey @gauravabhishek4U https://github.com/gauravabhishek4U
We are taking HelloWorld related programs here. However we do have other
repo:
https://github.com/6-CSE/AnyLangHack for all such things. We will happily
merge it there for this Hacktoberfest.
—
Reply to this email directly, view it on GitHub
https://github.com/6-CSE/HelloWorld/pull/247#pullrequestreview-1127693919,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKT45YU22Y5LA72OCLCCZXTWBH3IBANCNFSM6AAAAAAQ2NNJ5A
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey @gauravabhishek4U
That sounds good. We are looking forward to your PRs. We will merge it if they are within our scopes. If not, we will recommend for changes and we can work on from that to get your PRs done.
Cheers


improve/clarify sol session start time in multiple timezones

i think sol start time doesn't need to worry about timezones at all in displaying to the user?

When choosing a time for SOL weather, it says "Asia/Shanghai" but it's actually circuit's local time. So I want this "(Asia/Shanghai)" to be removed.
Also, when pressing SAVE to save the custom race and edit again, the time will become the time + 8 hours, which is 21:00 so every time we want to edit again even only change the circuit we use, we should always set the time again which is kind of annoying.

https://www.racedepartment.com/threads/ac-server-manager.165662/page-15#post-3006235

Also fix editing the event adding eight hours to the start time, bloody timezones.

Also server welcome message is apparently:
"This server is running Sol with a 0 time progression multiplier" when the setting is actually 1. Probably related, definitely close by.


publish<EMAIL_ADDRESS>

Requested by: @timfish
Merge target: (default)
Quick links:

View changes
View check runs

Assign the accepted label to this issue to approve the release.
Leave a comment containing #retract under this issue to retract the release (original issuer only).
Targets

[ ] npm
[ ] registry
[ ] github

Targets marked with a checkbox have already been executed.  Administrators can manually tick a checkbox to force craft to skip it.

@mydea this is ready to release!


Testing

There are a bunch of stuff we should do regarding documentation and testing:

Provide information in documentation on how to manually test
Setup automated testing

I would like to add a travis file here, and put in some testing.
prescript:

build the image
start a container with the image

jobs:

test that the container didn't crash.

I might start with a travis file in a different image first.

https://github.com/wunderkraut/image-fuzzy-alpine-php-fpm/pull/22 delivers initial travis testing

#22 now has some actual run tests in it, but only a container run test so far.


[WFLY-16317] Upgrade JOSE4j to 0.7.11

Fixes https://issues.redhat.com/browse/WFLY-16317

@fjuma Could you review and approve, please?


Fix s3 upload issue

Iam testing thanos sidecar using s3 buckets and it fails with error:
vel=error ts=2018-05-15T15:06:42.013968661Z caller=shipper.go:152 msg="shipping failed" block=01CDJ51CFFYG9F7TBRDVXP00KP err="upload chunks: upload file /prometheus/data/thanos/upload/01CDJ51CFFYG9F7TBRDVXP00KP/chunks/000001 as 01CDJ51CFFYG9F7TBRDVXP00KP/chunks/000001: upload s3 object: Your proposed upload size ‘-1’ is below the minimum allowed object size ‘0B’ for single PUT operation.

Looking at minio-go code turns out filesize -1 is only supported in GCP.
There is an explanation i comment:
https://github.com/minio/minio-go/blob/c6412e9f72738208830f9d16014896124028c4ce/api-put-object-streaming.go#L340

How it worked before for other folks? 0.o


Please describe and detail the PMC commands alongside the CLI commands

[Enter feedback here]
It would be very helpful to show and explain the PMC commands along side the CLI commands.  Simply referencing the package needed (at the bottom of the article) to use the commands is not enough.

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

ID: e56e675b-5964-a75b-7e45-6f6d1a607e65
Version Independent ID: 78e47218-58a3-f8d9-e091-306126871a67
Content: Tutorial Part 5, apply migrations to the Contoso University sample
Content Source: aspnetcore/data/ef-mvc/migrations.md
Product: aspnet-core
Technology: aspnetcore-data
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Thanks for the suggestion. This is an advanced tutorial so we don't duplicate PMC

Thanks for the suggestion. This is an advanced tutorial so we don't duplicate PMC


Pending LLVM Dependent Patches

We keep this issue open to track local patches that should be applied once integration has caught up.
This issue is pinned to the #llvm-integrate channel on Discord.
In order to add a required patch, please add a comment to this issue with the following information:

Upstream PR/commit link
Branch/commit that should be applied in conjunction with the above (PSA: please squash into one commit that can be cherry-picked)
Any additional notes or condolences for the integrator

https://github.com/llvm/llvm-project/commit/11ac97c67a9315dce48fd938d68ae991e3559f10 is reverted in the recent integrate because it introduces many regressions in benchmark suite. The tracking issue is https://github.com/openxla/iree/pull/16055. We should be able to drop the revert once the issue is fixed.

https://github.com/llvm/llvm-project/commit/af78e5daf0791135485dbd7972ffedb927727a6b is reverted. Issue: https://github.com/openxla/iree/issues/16173 We can drop the revert after we bump tf-nightly to include the commit.

https://github.com/llvm/llvm-project/commit/e47b507562624bf291ab2515699d39c2669b6131 is a breaking change in the EmitC dialect. https://github.com/iree-org/iree/pull/18284 should have all the changes needed in iree.

Upstreaming of a pattern in IREE: https://github.com/llvm/llvm-project/pull/107302
Once pulled in can be dropped by cherry-picking this: https://github.com/iree-org/iree/pull/18465

Breaking change in EmitC syntax
https://github.com/llvm/llvm-project/commit/68a3908148c6b6424b1ad4d0ed19d56435252832 is a breaking change in the EmitC syntax. https://github.com/iree-org/iree/pull/19201 contains the changes needed in iree.


Scan druid datasources request is not defined error

Superset 0.34 installed with druid same machine. I have druid working with some datasourses.
I create in superset a druid cluster, localhost 8082, then I go to superset scan druid datasources.
Expected results
It should refresh druid datasources and list them in superset. but i get the following error : connection refused.
Actual results
ConnectionRefusedError: [Errno 111] Connection refused
2019-10-26 11:49:39,659:ERROR:root:name 'requests' is not defined
Traceback (most recent call last):
File "/var/www/html/venv/lib/python3.6/site-packages/superset/connectors/druid/views.py", line 426, in refresh_datasources
cluster.refresh_datasources(refreshAll=refreshAll)
File "/var/www/html/venv/lib/python3.6/site-packages/superset/connectors/druid/models.py", line 193, in refresh_datasources
ds_list = self.get_datasources()
File "/var/www/html/venv/lib/python3.6/site-packages/superset/connectors/druid/models.py", line 174, in get_datasources
auth = requests.auth.HTTPBasicAuth(self.broker_user, self.broker_pass)
How to reproduce the bug

Go to 'superset'
Click on 'Scan new datasources'
See error Error while processing cluster 'cluster_druid' name 'requests' is not defined

Environment
(please complete the following information):

superset version: superset version 0.34
python version: python --version  3.6
node.js version: node -v not installed
npm version: npm -v not installed

Checklist
Make sure these boxes are checked before submitting your issue - thank you!

[X ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[X] I have reproduced the issue with at least the latest released version of superset.
[ X] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
I have Druid and superset installed as services. No dockers.
I checked to access Druid broker from command line using
curl http://localhost:8082/druid/v2/datasources
It worked.

Resolved installing pydruid.
pip install pydruid
but exception handling in connectors/druid/models.py should print proper message instead of just pass.
Pep.

Hi @jakubczaplicki,
Glad you were able to solve it. A nice way to install the druid extra requirements:
pip install "apache-superset[druid]"


Can't use single quotes with Jackson, "in" predicate operator

The README says, in section Filter Operators

String literals must be enclosed by single or double quotes<EMAIL_ADDRESS>== 'blue')] or<EMAIL_ADDRESS>== "blue")]).

When using a JacksonJsonProvider or JacksonJsonNodeJsonProvider, and when testing a predicate with the in operator, an InvalidJsonException is thrown.
I've uploaded a test class to a gist: https://gist.github.com/johnflavin/31aea6ac790f32cbbb528124eceeb4df. In it, I try out all the different json provider classes, with single and double quotes, in predicates using ?(@.foo == 'bar') and ?(@.foo in ['bar']). Everything works as expected, except for Jackson providers, single quotes, and in.

Use " double quotes
<EMAIL_ADDRESS>==\ "driverContactId\").value
works for me

I think you missed the point of the issue. I know that double quotes work. I know that == works. I provided a test class that tests this case and shows that it passes.
The only thing I am saying does not work is a predicate with a JacksonMappingProvider, using single quotes, and the in operator.

Works as expected.


Add configuration support for "textlint" field in package.json

Closes https://github.com/textlint/textlint/issues/643

Thanks for Pull Request!


Add natchez-fs2 & natchez-testkit

These modules are a bit more experimental than the others but what natchez-fs2 does is break Span[F] out of a Resource[F, Span[F]] and give you an AllocatedSpan that can be manually submitted as part of an FS2 stream, much like a committable Kafka Record in fs2-kafka

Ok, I skimmed through and I'm trying to wrap my head around this, but I'm not sure if I have any brainpower left today to reason about queues and spans 😄

Looks good, I'm excited to give this a try 👍

Just a suggestion, but would be nice to add all the new modules to the README :)


Firefox Inconsistencies

Figure out what is causing Firefox inconsistencies.
For example

Update: Apparently media queries in Firefox do not apply if the units are in pixels.
e.g.@media only screen and (max-device-width: 732px)


Get PosixPath NotImplementedError in Windows

Error originates from linting/cli.py. Using PurePosixPath instead seems to fix this.

Please consider creating a PR, and also check with master first.

This was reported for Neomake in https://github.com/neomake/neomake/issues/2172.
It appears to be fixed in 4.0a1 apparently - try pip install --pre vim-vint.
Hopefully there will be a release shortly.

The local reference to PosixPath was removed in 3a2729eb , which is only released as part of pre-release v0.4a1 , so 0.3.x is still effected.  Code is https://github.com/Kuniwak/vint/blob/stable-0.3/vint/linting/cli.py#L129
Note I am happy to wait, as I dont use Windows - I was just a little confused when it didnt appear to be fixed  in the latest release for the coala CI.

@jayvdb
Thanks for the reference.
The 0.3.x releases were only to fix critical things.
We should really have 0.4 sometime soon, until then I recommend to use master.


isUnitary assert in dDifference requires high precision, does not assert on 0-vector

Since 0d0e1e4999335da660de6bf63393448fa4d7302d was merged, we are having the isUnitary assert triggered regularly when passing states in where the quaternion is not explicitly normalised right before we pass it to dDifference. The off-axis terms are often in the range of 1e-8 - but they still trigger the isUnitary check since no precision is specified.
However, it still does not trigger when I pass a full zero-vector in (which also shows as isUnitary 0):
dDifference: q1 = 0.00000000000000000000e+00 7.07106781186535915396e-01 0.00000000000000000000e+00 7.07106781186559008034e-01, norm=9.99999999999999777955e-01, unitary? 1
dDifference: q2 = 0.00000000000000000000e+00 0.00000000000000000000e+00 0.00000000000000000000e+00 0.00000000000000000000e+00, norm=0.00000000000000000000e+00, unitary? 0

q2 here does not trigger (but should).
When q1 was slightly off (1e-8 terms, when initialised from a quaternion with 0 0.7071068 0 0.7071068), it triggered.
In short:

The default precision for isUnitary is not set and requires a really high precision, or else it asserts. We could set a precision ourselves.
It does not throw on a Zero(7) vector, and I believe it should (?)

It does not throw on a Zero(7) vector, and I believe it should (?)

I think so too. Eigen states that the conversion from quaternion to rotation matrix yield an undefined result if the quaternion is not normalized. I have checked what it returns when given a zero quaterion.
This can only be fixed by checking the quaternion norm.

We should maybe provide a isUnitary function dedicated to Quaternions.
The remarks made by @wxmerkt mean that we are not checking sufficient cases inside unit testing.
It also means that at some point in Croco, re-normalization of the state vector should be enforced to avoid numerical drifts.

I've implemented a tiny example to check it, but I think the asserts are correct, in the sense that they were not present before, because we were not checking them, this was a mistake from the Pinocchio side.
One solution would be to allow the users to relax this constraint, by defining his/her own tolerance value. @wxmerkt This something you would use?
Here is the script compilation command:
g++ $(pkg-config --cflags eigen3) -g  test.cpp -o test

test.txt

Regarding (1): I am now renormalising the state and the assert no longer gets triggered. From experience/theory, is there are better tolerance than dummy_precision we should pass by default?
Regarding (2): Perhaps we should add another check checking that the norm is approximately 1 as @jmirabel also suggested: The all-zeros-uninitialised case would be caught this way.

Solved by #1182.

I still get this triggered by the latest master. Here is the use case - it involves using integrate and difference a lot, and in sequence: It's a finite difference method for a unit test (pseudo-code):
dx = zeros(NDX)
dx[i] = eps / 2.0
q_plus = pin.integrate(q, dx[:nv])
q_minus = pin.integrate(q, -dx[:nv])
jacobian[:,i] = pin.difference(q_plus, q_minus) / eps
This fails in the difference operation saying that the quaternion is not normalised. Note, I used pin.integrate to get q_plus and q_minus.
Any ideas? Is this covered by a unit test?


Mayastor's etcd chart's initialClusterState value is pinned to "new"

Describe the bug
It seems like etcd is supposed to be scalable via Helm upgrades but mayastor.etcd.initialClusterState is set to "new". Is this intentional?
Maybe related: https://github.com/openebs/mayastor/issues/1659
To Reproduce
Install with mayastor.etcd.replicaCount: 1 then set it to 2 and run helm upgrade.
Expected behavior
Let the etcd chart handle upgrades.
** OS info (please complete the following information):**

Distro: Talos
6.6.33
openebs 4.1

Duplicate of https://github.com/openebs/mayastor/issues/1659


Use cuSPARSE Generic API instead of older one documented to be removed

Rel. #3513
Some cuSPARSE functions are announced to be removed in the next CUDA major release on its document. This PR fixes CuPy functions which use such older functions to use cuSPARSE Generic API instead.
TODO

cupyx.scipy.sparse

cupyx/scipy/sparse/linalg/_solve.py

[x] fix spsolve_triangular to use spsm first (currently uses csrsm2)
[x] fix SuperLU.solve to use spsm first (curently uses csrsm2)

cupyx/scipy/sparse/_csc.py

[x] fix csc_matrix.__mul__ to use spgemm

cupy.cusparse

[x] update _available_cusparse_version
add spsm function

[x] CUDA
[x] HIP

cupy_backends

add compat stubs

[x] add CUDA backend stubs for API to be removed in CUDA 12

add spsm bindings or stubs

[x] Cython
[x] cuda
[x] hip
[x] stub

cuSPARSE API to be removed (32 cuSPARSE doc items)

cusparseAlgMode_t
csrgemm2Info_t
csrsm2Info_t
csrsv2Info_t
cusparseCreateCsrsv2Info
cusparseDestroyCsrsv2Info
cusparseCreateCsrsm2Info
cusparseDestroyCsrsm2Info
cusparseCreateCsrgemm2Info
cusparseDestroyCsrgemm2Info
cusparse<t>axpyi
cusparse<t>gthr
cusparse<t>gthrz
cusparse<t>roti
cusparse<t>sctr
cusparseCsrmvEx
cusparse<t>csrsv2_bufferSize
cusparse<t>csrsv2_analysis
cusparse<t>csrsv2_solve
cusparseXcsrsv2_zeroPivot
cusparse<t>csrsm2_bufferSizeExt
cusparse<t>csrsm2_analysis
cusparse<t>csrsm2_solve
cusparseXcsrsm2_zeroPivot
cusparse<t>gemmi
cusparse<t>csrgemm2
cusparse<t>csc2dense
cusparse<t>csr2dense
cusparse<t>dense2csc
cusparse<t>dense2csr
cusparseSpMatSetStridedBatch
cusparseConstrainedGeMM

I checked that cusparseSpSM_* was introduced in CUDA 11.3.1, and hipsparseSpSM_* in ROCm 5.0.0.

hipsparseSpSM_solve has an extra parameter externalBuffer that cusparseSpSM_solve does not have. I'll disable SpSM on HIP this time.

/test mini

/test mini

/test mini

/test mini

@anaruse Is it safe to give the same matrix with B to the matrix C in cusparseSpSM_solve to overwrite B to store the solution?
https://docs.nvidia.com/cuda/cusparse/index.html#cusparse-generic-function-spsm

/test full

I'll kick full test after code unfrozen.

/test full

/test full

/test mini

/test full

Let me run another full test after pre-review tests pass.

/test full

Thanks for the hard work @takagi! Could you briefly check the performance difference before/after the change for these affected functions?

I've a comparison between cupy.cusparse.csrsm2 and cupy.cusparse.spsm. The result weirdly says spsm is much slower than csrsm2 (and even than scipy's spsolve_triangular). I've also profiled the script with Python cProfile and found almost all of the spsm time is used by spSM_analysis (in cupy_backends).

m
n
nnz
scipy[ms]
csrsm[ms]
spsm[ms]

100
10
1133
1.399
1.047
3.165

1000
10
100871
15.201
1.836
16.932

10000
10
10008206
872.663
5.945
1225.227

click to see benchmark script
import numpy
import scipy

import cupy
from cupy import testing
import cupyx
import cupyx.scipy.linalg
from cupyx import profiler

def make_sparse(m, density, dtype):
    a_shape = m, m
    a = testing.shaped_random(a_shape, numpy, dtype=dtype, scale=0.001)
    a -= numpy.diag(a)
    a_mask = testing.shaped_random(a_shape, numpy, dtype='f', scale=1)
    a[a_mask > density] = 0
    a_diag = numpy.random.uniform(-0.001, 0.001, size=(m,)).astype(dtype) + 1
    a_diag = numpy.diag(a_diag)
    a += a_diag
    a = numpy.tril(a)
    assert abs(numpy.linalg.det(a) - 1) < 0.1
    a = scipy.sparse.csr_matrix(a)
    a.sum_duplicates()
    return a

def make_dense(m, n, dtype):
    return numpy.array(numpy.random.uniform(0, 1, (m, n)).astype(dtype), order='f')

def verify(a_np, b_np, a_cp, b_cp):
    # verify with scipy
    x = scipy.sparse.linalg.spsolve_triangular(a_np, b_np,lower=True)
    testing.assert_allclose(a_np.todense() @ x, b_np, rtol=1e-5, atol=1e-5)

    # verify with csrsm2
    x = cupy.copy(b_cp)  # f-order
    assert x._f_contiguous
    cupy.cusparse.csrsm2(a_cp, x)
    testing.assert_allclose(a_cp.todense() @ x, b_cp, rtol=1e-5, atol=1e-5)

    # verify with spsm
    x = cupy.cusparse.spsm(a_cp, b_cp)
    testing.assert_allclose(a_cp.todense() @ x, b_cp, rtol=1e-5, atol=1e-5)

def get_time(perf):
    cpu_time = perf.cpu_times.mean()
    gpu_time = perf.gpu_times.mean()
    return max(cpu_time, gpu_time) * 1000  # ms

def benchmark(a_np, b_np, a_cp, b_cp):
    times = []

    # scipy spsolve_triangular
    perf = profiler.benchmark(scipy.sparse.linalg.spsolve_triangular, (a_np, b_np), n_repeat=10, name='scipy spsolve_triangular')
    times.append(get_time(perf))

    # csrsm2
    def _csrsm2(a, b):
        x = cupy.copy(b)  # f-order
        assert x._f_contiguous
        cupy.cusparse.csrsm2(a, x)
    perf = profiler.benchmark(_csrsm2, (a_cp, b_cp), n_repeat=10, name='cusparse.csrsm2')
    times.append(get_time(perf))

    # spsm
    perf = profiler.benchmark(cupy.cusparse.spsm, (a_cp, b_cp), n_repeat=10, name='cusparse.spsm')
    times.append(get_time(perf))

    return times

dtype = cupy.float32
n = 10
density = 0.2

print('m | n | nnz | scipy[ms] | csrsm[ms] | spsm[ms]')
print('--|--|--|--|--|--')
for m in [100, 1000, 10000]:
    a_np = make_sparse(m, density, dtype)
    b_np = make_dense(m, n, dtype)
    a_cp = cupyx.scipy.sparse.csr_matrix(a_np)
    b_cp = cupy.asarray(b_np, order='f')
    verify(a_np, b_np, a_cp, b_cp)
    ret = benchmark(a_np, b_np, a_cp, b_cp)
    print(f'{m} | {n} | {a_np.nnz} | {ret[0]:.3f} | {ret[1]:.3f} | {ret[2]:.3f}')

@anaruse Have you ever heard any cases like this about the performance of cusparseSpSM? (I'm afraid that I don't immediately have naive C/C++ replicator)

@anaruse Have you ever heard any cases like this about the performance of cusparseSpSM? (I'm afraid that I don't immediately have naive C/C++ replicator)

It is quite slow.. Please wait a little while while we investigate as well.

cusparseSpSM may take a long time to compute when the given input matrix has many non-zero elements. The amount of nnz at https://github.com/cupy/cupy/pull/7052#issuecomment-1275512137 is high enough (20%) compared to typical cases where cusparseSpSM is used. Thanks, @anaruse!

/test mini

/test mini

@asi1024 Failed to backport automatically.

Not cleanly cherry-picked

conflict: https://github.com/cupy/cupy/pull/7013

I'll make its backport

@asi1024 This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?


System.IO.FileNotFoundException at TuaRua.FreSharp.FreSharpHelper.DispatchEvent

This error is driving my crazy... In my dev computer everything works perfect, but when I run my app in the client test computer (Windows 8.1 Pro 32-bit), I get this exception:
System.IO.FileNotFoundException: Could not load file or assembly 'FreSharpCore.dll' or one of its dependencies. The specified module could not be found.
File name: 'FreSharpCore.dll'
   at TuaRua.FreSharp.FreSharpHelper.DispatchEvent(IntPtr& freContext, String name, String value)
   at PlayerBridgeANELib.MainController.RaiseEvent(String eventType, String jsonEventInfo)
   at PlayerBridgeANELib.MainController.NotifyException(Exception ex)
   at PlayerBridgeANELib.MainController.WatchSpontania()
   at PlayerBridgeANELib.MainController.InitController(IntPtr ctx, UInt32 argc, IntPtr[] argv)

The exception line is in my code is in the SendEvent(...) method call.

.Net 4.6 installed ✔
VC++ redist installed ✔
My C# ANE lib dll, FreeSharp.dll, FreSharpCore.dll in the .exe folder ✔

If the DispatchEvent method is reached, I guess FreSharpCore.dll is loaded but not one of it dependencies, but FreSharp.dll is in the same folder. So which is the missing file? Or maybe I have to place dlls in other location?
The Windows Event viewer and the Windows Error Reporting file don't give any useful information (or I don't interpreting it properly...).
Any clue is welcome.

Well the message says it can't load FreSharpCore.dll, you say you guess it is already loaded but are you making any other FRE calls before the sendEvent ? - if not that it is not certain it is loaded.
Are you building your C++ dll with v140 targeting 8.1 like the sample projects?
FreSharpCore.dll uses VC++ Dist 2015
That and .NET is Core's only dependency.
You are sending an event when an Exception occurs. What is this exception? are you able to run a very basic ANE on the machine.
I have had no problems using FreSharp on other non-dev machines.

You nailed it. After struggling a lot I realized that I was using v120 (cause I use VS 2013) and inspecting the FreSharp source project I saw it uses v140.
I installed VC++ redist 2015 and worked again. I only wish the error messages where more clear...
Thanks again for your help.

Yes the VC++ dist has to match only. It's common for projects to have components that need different versions of the dist.


1.18.2 waystones , 3-4 spawn per village

so heres an image of a village on a newly generated world.. there are 4 waystones in this village... default config.. no changes at all.
{
"worldgen": {
"generate_in_villages": true,
"village_waystone_weight": 5
},
"teleportation_cost": {
"cost_type": "level",
"cost_item": "minecraft:ender_pearl",
"base_cost": 1,
"cost_per_block_distance": 0.0,
"cost_multiplier_between_dimensions": 1.0
},
"discover_with_item": "none",
"take_amount_from_discover_item": 0,
"consume_infinite_knowledge_scroll_on_use": false,
"consume_local_void_on_use": true,
"free_local_void_teleport": true,
"store_waystone_data_on_sneak_break": true,
"can_owners_redeem_payments": false,
"waystone_block_hardness": 4.0,
"waystone_block_required_mining_level": 1,
"prevent_non_owners_from_breaking_waystone": false,
"can_players_toggle_global_mode": true,
"teleportation_cooldown": {
"cooldown_ticks_when_hurt": 0,
"cooldown_ticks_from_abyss_watcher": 0,
"cooldown_ticks_from_pocket_wormhole": 0,
"cooldown_ticks_from_local_void": 0,
"cooldown_ticks_from_waystone": 0
}
}

I changed the way village generation works. For now, lower the "village_waystone_weight" value, in the future I might make it more configurable if I have time. If you (or anyone else) have a good suggestion for a good default weight, feel free to comment.


How to reject unknown query params?

Hey, I see in Poem there is deny_unknown_fields for when deserializing objects. I wonder if there is a similar thing for query params. For example, if I have a function like this:
    async fn get_account_modules(
        &self,
        accept: Accept,
        address: Path<Address>,
        ledger_version: Query<Option<U64>>,
    )

I want it that if someone sends a request like ?version=54 instead of ?ledger_version=54, it returns a 400.
Thanks!

@sunli829 could we re-open this one and add a "do not mark stale" tag? I think it's a legit feature request.