[HOLD for payment 2024-12-30] [$250] QBD - QBD is displayed like an established connection without error after dismissing the RHP

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.71-1
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5297104
Email or phone of affected tester (no customers): N/A
Issue reported by: Applause Internal Team
Action Performed:

Go to ND and log in with a new expensifail account
Create a new collect workspace
Go to "More Features" in workspace settings
Enable the Accounting toggle
Navigate to Accounting
Click on the connect button next to QBD
Upgrade your workspace to Control plan
Dismiss the RHP before the setup link appears
Navigate to any other page in workspace settings
Go back to Accounting

Expected Result:
QuickBooks Desktop should not be displayed as if a successful connection was made due to the RHP being dismissed even before the setup link appears. The behavior should be similar to the other integrations.
Actual Result:
QBD is displayed like an established connection without error after dismissing the RHP before the setup link appears.
Workaround:
Unknown
Platforms:

[ ] Android: Standalone
[ ] Android: HybridApp
[x] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/1e07d67c-0a1b-467f-901f-f1582d49770b
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021865081766703229432
Upwork Job ID:<PHONE_NUMBER>703229432
Last Price Increase: 2024-12-06
Automatic offers:

ikevin127 | Contributor | 105312550

Issue OwnerCurrent Issue Owner: @abekkala

Proposal
Please re-state the problem that we are trying to solve in this issue
QBD is displayed like an established connection without error after dismissing the RHP before the setup link appears.
What is the root cause of that problem?
https://github.com/Expensify/App/blob/b595f5d24141a403fcc3f44c73d00cd6951dca7d/src/pages/workspace/accounting/qbd/QuickBooksDesktopSetupPage.tsx#L62-L69
The code block above calls ConnectPolicyToQuickbooksDesktop when the RHP modal mounts, regardless of it being in the loading state wich leads to the QBD connection to be established even when the RHP modal is closed while still loading.
What changes do you think we should make in order to solve the problem?
https://github.com/Expensify/App/blob/b595f5d24141a403fcc3f44c73d00cd6951dca7d/src/pages/workspace/accounting/qbd/QuickBooksDesktopSetupPage.tsx#L116
We should use the same loading variables used to display the "links to connect" content to call ConnectPolicyToQuickbooksDesktop as this ensures the QBD connection will only be established once the "links to connect" content is shown in the RHP modal and not before that (in case we close the modal before content loading is complete).

Proposal
Please re-state the problem that we are trying to solve in this issue
QBD is displayed like an established connection without error after dismissing the RHP before the setup link appears.
What is the root cause of that problem?
https://github.com/Expensify/App/blob/b595f5d24141a403fcc3f44c73d00cd6951dca7d/src/pages/workspace/accounting/qbd/QuickBooksDesktopSetupPage.tsx#L62-L69
The code block above calls ConnectPolicyToQuickbooksDesktop when the RHP modal mounts, the modal starting with isLoading state variable as true which leads to the QBD connection to be established even when the RHP modal is closed while still loading because the call is initialized on mount.
The issue is that this component was designed such that the setup link appears only when the loading state becomes false and hasResultOfFetchingSetupLink becomes true as a result of the ConnectPolicyToQuickbooksDesktop call being completed, therefore the appearance of the setup link depends on the loading state which depends on the call being made.
What changes do you think we should make in order to solve the problem?
Given that the setup link being shown depends on the loading state becoming false and hasResultOfFetchingSetupLink being true, which depends on the ConnectPolicyToQuickbooksDesktop call being completed, and the fact that the Expected result requires that the call should only be made once the setup link is visible, this is my suggested approach:

remove the isLoading state variable completely from the RequireQuickBooksDesktopModal component
remove the shouldShowLoading variable as well as the FullScreenLoadingIndicator which comes with it

this way the setup link page content is visible right away except for the copy to clipboard link where we will show a small ActivityIndicator until ConnectPolicyToQuickbooksDesktop call is completed - this way there's no issue anymore since we eliminated the full screen loading loading indicator which led to this issue.
This would pose another issue when it comes to the setup link content page because the link only becomes available once ConnectPolicyToQuickbooksDesktop call is completed and hasResultOfFetchingSetupLink becomes true.
For this we have to conditionally show the CopyTextToClipboard component only when hasResultOfFetchingSetupLink is true, if it's false we display a small ActivityIndicator:
{!hasResultOfFetchingSetupLink ? (
    <ActivityIndicator
        color={theme.spinner}
        size="small"
    />
) : (
    <CopyTextToClipboard
        text={codatSetupLink}
        textStyles={[styles.textSupporting]}
    />
)}

Here's the complete DIFF for clarity
diff --git a/src/pages/workspace/accounting/qbd/QuickBooksDesktopSetupPage.tsx b/src/pages/workspace/accounting/qbd/QuickBooksDesktopSetupPage.tsx
index 5e90c3db4ed..02a0744e356 100644
--- a/src/pages/workspace/accounting/qbd/QuickBooksDesktopSetupPage.tsx
+++ b/src/pages/workspace/accounting/qbd/QuickBooksDesktopSetupPage.tsx
@@ -1,11 +1,11 @@
+import {useTheme} from '@react-navigation/native';
 import React, {useCallback, useEffect, useState} from 'react';
-import {View} from 'react-native';
+import {ActivityIndicator, View} from 'react-native';
 import Computer from<EMAIL_ADDRESS> import FullPageOfflineBlockingView from '@components/BlockingViews/FullPageOfflineBlockingView';
 import Button from '@components/Button';
 import CopyTextToClipboard from '@components/CopyTextToClipboard';
 import FixedFooter from '@components/FixedFooter';
-import FullScreenLoadingIndicator from '@components/FullscreenLoadingIndicator';
 import HeaderWithBackButton from '@components/HeaderWithBackButton';
 import Icon from '@components/Icon';
 import * as Illustrations from '@components/Icon/Illustrations';
@@ -31,10 +31,10 @@ type RequireQuickBooksDesktopModalProps = PlatformStackScreenProps<SettingsNavig

 function RequireQuickBooksDesktopModal({route}: RequireQuickBooksDesktopModalProps) {
     const {translate} = useLocalize();
+    const theme = useTheme();
     const styles = useThemeStyles();
     const {environmentURL} = useEnvironment();
     const policyID: string = route.params.policyID;
-    const [isLoading, setIsLoading] = useState(true);
     const [hasError, setHasError] = useState(false);
     const [codatSetupLink, setCodatSetupLink] = useState<string>('');
     const hasResultOfFetchingSetupLink = !!codatSetupLink || hasError;
@@ -42,20 +42,19 @@ function RequireQuickBooksDesktopModal({route}: RequireQuickBooksDesktopModalPro
     const ContentWrapper = hasResultOfFetchingSetupLink ? ({children}: React.PropsWithChildren) => children : FullPageOfflineBlockingView;

     const fetchSetupLink = useCallback(() => {
-        setIsLoading(true);
         setHasError(false);
         // eslint-disable-next-line rulesdir/no-thenable-actions-in-views
         QuickbooksDesktop.getQuickbooksDesktopCodatSetupLink(policyID).then((response) => {
-            if (response?.jsonCode) {
-                if (response.jsonCode === CONST.JSON_CODE.SUCCESS) {
-                    setCodatSetupLink(String(response?.setupUrl ?? ''));
-                } else {
-                    setConnectionError(policyID, CONST.POLICY.CONNECTIONS.NAME.QBD, translate('workspace.qbd.setupPage.setupErrorTitle'));
-                    setHasError(true);
-                }
+            if (!response?.jsonCode) {
+                return;
             }

-            setIsLoading(false);
+            if (response.jsonCode === CONST.JSON_CODE.SUCCESS) {
+                setCodatSetupLink(String(response?.setupUrl ?? ''));
+            } else {
+                setConnectionError(policyID, CONST.POLICY.CONNECTIONS.NAME.QBD, translate('workspace.qbd.setupPage.setupErrorTitle'));
+                setHasError(true);
+            }
         });
     }, [policyID, translate]);

@@ -77,8 +76,7 @@ function RequireQuickBooksDesktopModal({route}: RequireQuickBooksDesktopModalPro
         },
     });

-    const shouldShowLoading = isLoading || !hasResultOfFetchingSetupLink;
-    const shouldShowError = !shouldShowLoading && hasError;
+    const shouldShowError = hasError;

     return (
         <ScreenWrapper
@@ -92,7 +90,6 @@ function RequireQuickBooksDesktopModal({route}: RequireQuickBooksDesktopModalPro
                 onBackButtonPress={() => Navigation.dismissModal()}
             />
             <ContentWrapper>
-                {shouldShowLoading && <FullScreenLoadingIndicator style={[styles.flex1, styles.pRelative]} />}
                 {shouldShowError && (
                     <View style={[styles.flex1, styles.justifyContentCenter, styles.alignItemsCenter, styles.ph5, styles.mb9]}>
                         <Icon
@@ -113,7 +110,7 @@ function RequireQuickBooksDesktopModal({route}: RequireQuickBooksDesktopModalPro
                         </Text>
                     </View>
                 )}
-                {!shouldShowLoading && !shouldShowError && (
+                {!shouldShowError && (
                     <View style={[styles.flex1, styles.ph5]}>
                         <View style={[styles.alignSelfCenter, styles.computerIllustrationContainer, styles.pv6]}>
                             <ImageSVG src={Computer} />
@@ -122,10 +119,17 @@ function RequireQuickBooksDesktopModal({route}: RequireQuickBooksDesktopModalPro
                         <Text style={[styles.textHeadlineH1, styles.pt5]}>{translate('workspace.qbd.setupPage.title')}</Text>
                         <Text style={[styles.textSupporting, styles.textNormal, styles.pt4]}>{translate('workspace.qbd.setupPage.body')}</Text>
                         <View style={[styles.qbdSetupLinkBox, styles.mt5]}>
-                            <CopyTextToClipboard
-                                text={codatSetupLink}
-                                textStyles={[styles.textSupporting]}
-                            />
+                            {!hasResultOfFetchingSetupLink ? (
+                                <ActivityIndicator
+                                    color={theme.spinner}
+                                    size="small"
+                                />
+                            ) : (
+                                <CopyTextToClipboard
+                                    text={codatSetupLink}
+                                    textStyles={[styles.textSupporting]}
+                                />
+                            )}
                         </View>
                         <FixedFooter style={[styles.mtAuto, styles.ph0]}>
                             <Button

This is how the UI would look with this solution:

MacOS: Web
https://github.com/user-attachments/assets/2ffc6cda-a783-4378-903b-65ff6c25ac48

cc @Expensify/design for their take on this approach UI / UX wise

That looks pretty good to me. cc @kadiealexander as you worked on QBD too.

Yeah I think it looks good too.

@kadiealexander do you have any comments irt this one before proceeding?

@dannymcclain @shawnborton I think we're good to proceed with the proposal from @ikevin127
@getusha do you want to review the proposal as well

Yes, reviewing.

@ikevin127 @abekkala I see the same behavior after applying the proposal.

QBD is displayed like an established connection without error after dismissing the RHP before the setup link appears.

https://github.com/user-attachments/assets/666eb218-6db6-4cf8-ba31-692a43e5cc4b
And does the link being displayed mean the connection was established successfully? don't we expect the user to complete the setup before determining that?

@getusha Te way QBD logic was designed is that the call is made as soon as the modal mounts and when this call is completed the connection will appear in Accounting. The issue states that the current implementation should not trigger the call if the RHP is closed while in the loading state, but that's impossible given that the current logic performs the call on mount and that loading state is basically linked to that API call.
I proposed not showing the loading indicator as a full page spinner initially when the RHP opens, but instead showing the setup link UI right away, and showing a small ActivityIndicator while the copy to clipboard link is being fetched by the API call which is still triggered on mount.
To clarify: I'm not changing the fact that the API call is triggered on RHP mount, but instead changing where and how we show the loading indicator such that it doesn't give the impression that the call still went through and the connection was made when the user closed the RHP while loading, making it more clear for the user that as soon as the setup link UI is shown (RHP is mounted), the connection was already made and is pending QBD setup via the link (once fetched) - and we're just waiting for the copy to clipboard link to appear when the API call is completed.
This was the only way that I could think of that would solve the issue in a way, except for the The behavior should be similar to the other integrations. part which is not possible in our case because QBD connection works differently than QBO and the other ones which are web based connections compared to QBD which requires a desktop app to be installed.

If I'm wrong in my assumptions and this is not a desirable solution, then the only solution left would be to reredesign the flow completely, starting with the BE.

Thanks for clarifying, @ikevin127. However, I assume we don't want to show the connection as established before displaying the link either way.  Pulling in an internal engineer since I'm not sure if we'll actually be fixing the issue by changing where we display the loading indicator.
🎀 👀 🎀

Sounds good, I like the proposed solution.
When we generate the setup link in the backend, we have to store some details about it in the policy, so if you switch between different sections like in @getusha's video and one of them makes an API call to get updated policy details, eventually, one of them is going to include details that make it look like the policy is being connected to QBD. I don't think there's a straightforward way to avoid that and it sounds like a rare edge case, so let's go with the current proposal that addresses the main issue in the OP.

PR https://github.com/Expensify/App/pull/54087 is ready for review!

♻ Status update: PR has been open and ready for review for 4 days - awaiting review from @getusha.

BugZero Checklist:

[ ] [Contributor] Classify the bug:

Bug classification

Source of bug:

[ ] 1a. Result of the original design (eg. a case wasn't considered)
[ ] 1b. Mistake during implementation
[ ] 1c. Backend bug
[ ] 1z. Other:

Where bug was reported:

[ ] 2a. Reported on production (eg. bug slipped through the normal regression and PR testing process on staging)
[ ] 2b. Reported on staging (eg. found during regression or PR testing)
[ ] 2d. Reported on a PR
[ ] 2z. Other:

Who reported the bug:

[ ] 3a. Expensify user
[ ] 3b. Expensify employee
[ ] 3c. Contributor
[ ] 3d. QA
[ ] 3z. Other:

[ ] [Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
Link to comment:

[ ] [Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
Link to discussion:

[ ] [Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.

Regression Test Proposal Template

[ ] [BugZero Assignee] Create a GH issue for creating/updating the regression test once above steps have been agreed upon.
Link to issue:

Regression Test Proposal
Precondition:

Test:

Do we agree 👍 or 👎

Payment summary:
@ikevin127 paid $250 via Upwork
@getusha to be paid $250 via NewDot manual request


Getting TypeError: RayTaskError.as_instanceof_cause..cls.init() takes 2 positional arguments but 4 were given

After running below code
# Trainer
trainer = TorchTrainer(
    train_loop_per_worker=train_loop_per_worker,
    train_loop_config=train_loop_config,
    scaling_config=scaling_config,
    run_config=run_config,
    datasets={"train": train_ds, "val": val_ds},
    dataset_config=dataset_config,
    metadata={"class_to_index": preprocessor.class_to_index}
)

%%time
# Train
results = trainer.fit()

Setuptools version - 69.5.1
Ray version - 2.7.0 also tried 2.7.1 & 2.7.2
getting below error -

TypeError                                 Traceback (most recent call last)
File :2
File /opt/anaconda3/envs/madewithml/lib/python3.10/site-packages/ray/train/base_trainer.py:653, in BaseTrainer.fit(self)
647 restore_msg = TrainingFailedError._RESTORE_MSG.format(
648     trainer_cls_name=self.class.name,
649     path=str(experiment_local_path),
650 )
652 try:
--> 653     result_grid = tuner.fit()
654 except TuneError as e:
655     # Catch any TuneErrors raised by the Tuner.fit call.
656     # Unwrap the TuneError if needed.
657     parent_error = e.cause or e
File /opt/anaconda3/envs/madewithml/lib/python3.10/site-packages/ray/tune/tuner.py:372, in Tuner.fit(self)
370 if not self._is_ray_client:
371     try:
--> 372         return self._local_tuner.fit()
373     except TuneError as e:
374         raise TuneError(
375             _TUNER_FAILED_MSG.format(
376                 path=self._local_tuner.get_experiment_checkpoint_dir()
377             )
378         ) from e
File /opt/anaconda3/envs/madewithml/lib/python3.10/site-packages/ray/tune/impl/tuner_internal.py:585, in TunerInternal.fit(self)
581     analysis = self._fit_resume(trainable, param_space)
583 self._experiment_analysis = analysis
--> 585 return ResultGrid(self._experiment_analysis)
File /opt/anaconda3/envs/madewithml/lib/python3.10/site-packages/ray/tune/result_grid.py:83, in ResultGrid.init(self, experiment_analysis)
78 def init(
79     self,
80     experiment_analysis: ExperimentAnalysis,
81 ):
82     self._experiment_analysis = experiment_analysis
---> 83     self._results = [
84         self._trial_to_result(trial) for trial in self._experiment_analysis.trials
85     ]
File /opt/anaconda3/envs/madewithml/lib/python3.10/site-packages/ray/tune/result_grid.py:84, in (.0)
78 def init(
79     self,
80     experiment_analysis: ExperimentAnalysis,
81 ):
82     self._experiment_analysis = experiment_analysis
83     self._results = [
---> 84         self._trial_to_result(trial) for trial in self._experiment_analysis.trials
85     ]
File /opt/anaconda3/envs/madewithml/lib/python3.10/site-packages/ray/tune/result_grid.py:317, in ResultGrid._trial_to_result(self, trial)
309 else:
310     metrics_df = self._experiment_analysis.trial_dataframes.get(
311         trial.local_path
312     )
314 result = Result(
315     checkpoint=checkpoint,
316     metrics=trial.last_result.copy(),
--> 317     error=self._populate_exception(trial),
318     _local_path=trial.local_path,
319     _remote_path=trial.remote_path,
320     _storage_filesystem=(
321         self._experiment_analysis._fs
322         if isinstance(self._experiment_analysis, ExperimentAnalysis)
323         else None
324     ),
325     metrics_dataframe=metrics_df,
326     best_checkpoints=best_checkpoints,
327 )
328 return result
File /opt/anaconda3/envs/madewithml/lib/python3.10/site-packages/ray/tune/result_grid.py:262, in ResultGrid._populate_exception(trial)
260 if trial.pickled_error_file and os.path.exists(trial.pickled_error_file):
261     with open(trial.pickled_error_file, "rb") as f:
--> 262         e = cloudpickle.load(f)
263         return e
264 elif trial.error_file and os.path.exists(trial.error_file):
TypeError: RayTaskError.as_instanceof_cause..cls.init() takes 2 positional arguments but 4 were given

same here


Fix/staking reactive updates for relaychain staking

#8693g0ewg
We use eraCalculatorFlow, which is endless flow, as an inner flow inside combineTransform for unbondings and stake summary when staking state is "waiting for next era". Howerver, combineTransform does not process next upstream item until inner flow completes, so in case of "waiting for next era" no updates after initial state were processed
Fixed by replacing combineTransform with combineToPair(...).flatMapLatest{...}

Task linked: CU-8693g0ewg Android. Inactive state after start staking on Westend


Update Composer dependencies (2022-01-28-00-13)

Visual regression test passed!
View the visual regression test report


Add Intel's GPU / XPU support to TIMM's validation script

Background
This PR is mainly about increasing the diversity of accelerators. Intel has a new GPU released last year and it is promoting to different platforms. This is a first step to incorporate the new hardware into TIMM. More optimizations and engineering efforts will be applied.
Related materials:
https://pytorch.org/tutorials/recipes/intel_extension_for_pytorch.html
https://github.com/intel/intel-extension-for-pytorch/tree/xpu-master
Environment
PyTorch version: 2.1.0a0+cxx11.abi
PyTorch CXX11 ABI: Yes
IPEX version: 2.1.10+xpu
OS: Ubuntu 22.04.3 LTS (x86_64)
GCC version: (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
GPU: Intel(R) Data Center GPU Max 1550
Dataset: Imagenet-1K validation
Result
Without autocast
python validate.py --data-dir ../datasets/ --dataset hfds/imagenet-1k --split validation --device xpu --model resnet50.a1_in1k --batch-size 256 --pretrained

With autocast:
python validate.py --data-dir ../datasets/ --dataset hfds/imagenet-1k --split validation --device xpu --model resnet50.a1_in1k --batch-size 256 --pretrained --amp

@cfgfung Hi, thanks for the PR, I have to come up with a better plan for handling the growing number of accel differences, with #2109 etc need a better way to centralize this code and support easy addition without needing checks and imports in every script...
I had an idea in the works in the bitsandtpu branch for TPU support, but haven't had a chance to unify that here with what I've learned since.

Hi Ross,
Thanks for the prompt review! Do you have any design/draft plan in your mind? We can have some discussions on unifying the interface/APIs. I am working on Intel's GPU and Gaudi. Might be able to provide some insights for those. Please feel free to contact me through email<EMAIL_ADDRESS>Related information:
https://www.intc.com/news-events/press-releases/detail/1689/intel-unleashes-enterprise-ai-with-gaudi-3-ai-open-systems
https://www.intel.com/content/www/us/en/products/details/discrete-gpus/data-center-gpu/max-series.html

Hi @cfgfung PyTorch 2.5 is here, there is native support for some Intel accelerators now, and also finally https://pytorch.org/tutorials/prototype/python_extension_autoload.html
I've got #2308 this to cleanup a few old api calls, and there's an updated NPU merge ready to go (assuming the autoload capability) #2109  ... with that in mind, what's best to support Intel devices that aren't native... habana should work via autoload now/soon right?

Hi @rwightman,
Yes, PyTorch 2.5 now has native support (prototype) of Intel GPU/XPU.
For the Habana/Gaudi product, we need to have a similar design like the NPU (https://github.com/huggingface/pytorch-image-models/pull/2109). My colleague @ZhengHongming888 will follow up on that.

Hi @rwightman ,
Thanks for the considering on the Intel's hpu (Gaudi Habana). yes if you agree i will submit the PRs for enablement the inference/training (lazy/graph mode both two) with Intel's HPU around next week. Thanks.
Hongming


Swarm does not start in Nano Server TP5

I've tried to put swarm.exe built with Go 1.5.4 into a nanoserver image on Windows Server 2016 TP5. My work in progress can be found here https://github.com/StefanScherer/dockerfiles-windows/pull/4
But the exe panics with
panic: Failed to load shell32.dll: The specified module could not be found.

goroutine 1 [running]:
syscall.(*LazyProc).mustFind(0xc082030ae0)
        c:/go/src/syscall/dll_windows.go:280 +0x6a
syscall.(*LazyProc).Addr(0xc082030ae0, 0x7ff804e68e40)
        c:/go/src/syscall/dll_windows.go:287 +0x28
syscall.CommandLineToArgv(0x1e81762, 0xc08202bdec, 0x1, 0x0, 0x0)
        c:/go/src/syscall/zsyscall_windows.go:885 +0x3c
os.init.1()
        c:/go/src/os/exec_windows.go:100 +0x4a
os.init()
        c:/go/src/os/types_windows.go:107 +0x303
fmt.init()
        c:/go/src/fmt/scan.go:1195 +0x5d
github.com/docker/swarm/vendor/github.com/docker/docker/pkg/discovery/file.init()
        C:/Go/src/github.com/docker/swarm/vendor/github.com/docker/docker/pkg/discovery/file/file.go:109 +0x42
main.init()
        C:/Go/src/github.com/docker/swarm/main.go:14 +0x42

There is a workaround in Docker Engine to explicit load the shell32.dll
https://github.com/docker/docker/commit/fa82c0aa10cfac8c6d5e2446876dc79b2b0c1bf9
So probably we need something like this in Swarm to workaround the issue in Go 1.5.4, 1.6.1, 1.6.2.
This may be fixed with Go 1.7 https://go-review.googlesource.com/#/c/22054/

Fixed in newer versions.


Test/cli

Description
This PR fixes #635

Hey @banipreetr 👋
Thank you for the review 😄
@Mr-Sunglasses , this PR is dependant on the scaffolding PR(https://github.com/sparckles/Robyn/pull/584) . We can have a look at this one once 584 is merged.


Impossible to do a npm install on a project generated from the 1.1.x Images

Hello, i'm facing an issue while trying to create project compatible with SP 2016
Describe the bug
Impossible to do a npm install on a project generated from the 1.1.x Images
To Reproduce
Steps to reproduce the behavior:
1 - Run the a container
2 - Generate a project via yo @microsoft/sharepoint
3 - run a npm install
Expected behavior
a complete npm install
Screenshots

Desktop (please complete the following information):

OS: Windows, macOS
Shell [PowerShell, Zsh]
Image version 1.1.0,1.1.1,1.1.3]

Additional context
N/A

Sorry to hear that. Let me see if I can repro the issue that you're seeing and what we could do about it.

I can repro your issue and it turns out that it's a broader ecosystem problem. SPFx uses dependencies that use dependencies that don't have their versions locked. As a result, they now point to modern versions of the dependencies that aren't compatible with the old version of Node and npm that's used in the image. Typically, you'd solve this by having a lockfile that's pointing to the supported versions of dependencies, but since you're creating a new project, you don't have one. As a result the installation fails. One way to solve this issue, is find supported version of each failing package and install it manually, which should allow you to run npm install in the end. For example, in your screenshot, you're getting error about @types/mime (ignore the -hash suffix). You can reproduce it by doing npm i -D @types/mime. Go back the list of versions until you find one that's working, eg. v0.0.29 which is roughly as old as SPFx v1.1.0. Then install it by running npm i -D<EMAIL_ADDRESS>With that version installed, re-rerun npm i. Repeat this until npm i succeeds. Not a great and easy solution, but should work eventually.

Hi @waldekmastykarz, i'll give it a try.
Thanks a lot for your support, and all the work ur doing for the community.

My pleasure


Fails to compile when SYS_PM_DIRECT_FORCE_MODE is true

To reproduce:

Use the sample samples\boards\nrf\system_off (or any other power oriented sample).
Change the prj.conf to add the line
CONFIG_SYS_PM_DIRECT_FORCE_MODE=y
Try to build

Produces the error:
-c C:/z/zephyr/subsys/power/power.c
In file included from C:/z/zephyr/include/kernel_includes.h:41,
from C:/z/zephyr/include/kernel.h:17,
from C:/z/zephyr/include/zephyr.h:18,
from C:/z/zephyr/subsys/power/power.c:7:
C:/z/zephyr/subsys/power/power.c: In function 'sys_pm_force_power_state':
C:/z/zephyr/include/sys_clock.h:85:28: error: incompatible type for argument 1 of '_sys_suspend'
85 | #define Z_TIMEOUT_TICKS(t) ((k_timeout_t) { .ticks = (t) })
…..
C:/z/zephyr/include/power/power.h:288:38: note: expected 's32_t' {aka 'int'} but argument is of type 'k_timeout_t' {aka 'struct '}
288 | enum power_states _sys_suspend(s32_t ticks);
….      |                                ~~^
C:/z/zephyr/subsys/power/power.c:87:15: note: in expansion of macro 'K_FOREVER'
87 |  _sys_suspend(K_FOREVER);
Expected behavior: build should succeed
So, the call to _sys_suspend is passing in K_FOREVER in power.c line 87 but _sys_suspend is expecting an s32_t ticks count not a k_timeout setting.
Environment: Windows 10 pro
Compiler: GNUARMEMB_TOOLCHAIN_PATH=C:\Program Files (x86)\GNU Tools ARM Embedded\9 2019-q4-major
Commit:
commit d898ada917782289dbd5ceb6de70547189a141d0 (HEAD -> master, origin/master)
Author: Maureen Helm<EMAIL_ADDRESS>Date:   Tue Apr 28 20:34:44 2020 -0500
Thanks.

This was fixed by 04d1104ee51754193efb557d6d4d0d1d9452e965 which is in current master.


minor code refactor

I'm not sure I'd call this a refactor. Mostly just cosmetic changes to the code. Thanks a lot for this PR Anderson.

Thank you @rabernat! I concur. This PR didn't have much in it. I probably should have opened this PR in a draft mode ( I was planning to add more commits to it 🙂). I'll open a separate PR later

Sorry for the premature merge. 🤦  Yes if something is not a draft, I generally assume it is considered ready to be merged.

Lots of great stylistic improvements here, and I definitely learned some things, including |=.
Thanks, @andersy005!


Fight Redd𝚒t UFC 306 Streams - Crackstreams StreamEast #8730

Fight Redd𝚒t UFC 306 Streams - Crackstreams StreamEast
The UFC heads to the Sphere in Las Vegas for the first time with two title fights lined up for UFC 306.
UFC 306 LIVE Stream Free
UFC 306 LIVE Stream Here
Sean O'Malley and Alexa Grasso will make their second title defences against Merab Dvalishvili and Valentina Shevchenko.
UFC
View Latest Odds
How to watch UFC 306 and live stream
UFC 306 will be shown live on TNT Sports in the UK, while the prelims and early prelims will be streamed live on UFC Fight Pass.
UFC 306 start time, date and venue
The early prelims for UFC 306 will start at 00:30 BST. The prelims will be 01:00 and the main card will be 03:00. The card will be held at the Sphere in Las Vegas in the early hours of 15th September.
LAS VEGAS – Sean O'Malley and Merab Dvalishvili had their final bit of pre-fight interaction at UFC 306 ceremonial weigh-ins Friday with one last staredown.
O'Malley (18-1 MMA, 10-1 UFC) will put his bantamweight belt on the line against Dvalishvili (17-4 MMA, 10-2 UFC) in the main event of Noche UFC (pay-per-view, ESPNews, ESPN+) at Sphere in what UFC CEO Dana White has touted to be a one-of-a-kind fight card, which airs on ESPN+ pay-per-view following prelims on ESPNews and ESPN+.
After both made weight earlier in the day, they engaged in the final staredown in front of fans and media. It was intense and both men were locked in. O'Malley looks to log a second consecutive title defense while Dvalishvili looks to capture gold and set the UFC record for longest winning streak in 135-pound history.
Check out the video above to see O'Malley and Dvalishvili face off at UFC 306 ceremonial weigh-ins.
How to watch UFC 306 main card
Live stream: ESPN+ ($79.99 PPV fee plus $10.99/month membership)
UFC 306 is available on ESPN+ and ESPN+ PPV in the U.S.
UFC 306 fight card
Main Card
Sean O'Malley (c) vs. Merab Dvalishvili for the UFC bantamweight title
Alexa Grasso (c) vs. Valentina Shevchenko 3 for the UFC flyweight title
Brian Ortega vs. Diego Lopes; Featherweight
Daniel Zellhuber vs. Esteban Ribovics; Lightweights
Ronaldo Rodríguez vs. Ode' Osbourne; Flyweights
Prelims
Ignacio Bahamondes vs. Manuel Torres; Lightweights
Irena Aldana vs. Norma Dumont; Bantamweights
Edgar Chairez vs. Joshua Van; Flyweights
Yazmin Jauregui vs. Ketlen Souza; Strawweights
Early Prelims
Raul Rosas Jr. vs. Aoriqileng; Bantamweights
UFC 306 start time
Date: Saturday, September 14
Location: The Sphere - Las Vegas
Time: 10 p.m. ET (8 p.m. ET for prelims)
The UFC 306 main card starts at 10 p.m. ET.

World's no1 website for watching all sports in HD
https://allin1.cx/
World's no1 website for watching all sports in HD
https://allin1.cx/
World's no1 website for watching all sports in HD
https://allin1.cx/
World's no1 website for watching all sports in HD
https://allin1.cx/


Replace staticfiles tag with static to fix django 2.1 deprecation warnings

Description
Since 2.1 release Django deprecated statcifiles tag in favour of static. For more information please check Features deprecated in 2.1
Type of change

Bug fix (non-breaking change which fixes an issue)

Coverage remained the same at 73.739% when pulling 2d0ce2bc8302fa7c6c31d2c81e98c633901234dd on spinnerok:fix/fix-django-warnings into 58e657456541f003724af0f40312c9fed27d3854 on sanoma:develop.

Closing since this request was reissued from from the same repository for simplicity.


Error in export to TRT int8

Hi,
I have a question regarding the TRT-export with int8.
I do the following command:
python export.py --trt --weights edgeyolo_visdrone.pth --batch 1 --workspace 8 --int8 --dataset params/dataset/visdrone_coco.yaml --num-imgs 16
Unfortunately after the calibration part I get the following error.
Do you have any idea or suggestion what I am doing wrong?
Thanks in advance!
2023-03-03 12:39:32.768 | INFO     | edgeyolo.models:__init__:50 - loading models from weight /opt/ssd/visiontools/edgeyolo/edgeyolo_visdrone.pth
/opt/ssd/visiontools/edgeyolo/venv/lib/python3.8/site-packages/torch/functional.py:568: UserWarning: torch.meshgrid: in an upcoming release, it will be required to pass the indexing argument. (Triggered internally at  ../aten/src/ATen/native/TensorShape.cpp:2311.)
  return _VF.meshgrid(tensors, **kwargs)  # type: ignore[attr-defined]
Reparameterizing models...
2023-03-03 12:39:37.982 | INFO     | edgeyolo.export.calib:__init__:43 - used images: 16
[03/03/2023-12:39:46] [TRT] [I] [MemUsageChange] Init CUDA: CPU +221, GPU +0, now: CPU 763, GPU 9107 (MiB)
[03/03/2023-12:39:49] [TRT] [I] [MemUsageChange] Init builder kernel library: CPU +302, GPU +404, now: CPU 1088, GPU 9512 (MiB)
2023-03-03 12:40:04.763 | INFO     | edgeyolo.export.pth2trt:torch2onnx2trt:137 - start to simplify ONNX...
2023-03-03 12:40:08.507 | INFO     | edgeyolo.export.pth2trt:torch2onnx2trt:140 - simplified ONNX successfully.
[03/03/2023-12:40:08] [TRT] [W] onnx2trt_utils.cpp:375: Your ONNX model has been generated with INT64 weights, while TensorRT does not natively support INT64. Attempting to cast down to INT32.
[03/03/2023-12:40:11] [TRT] [I] [MemUsageChange] Init cuBLAS/cuBLASLt: CPU +535, GPU +539, now: CPU 2739, GPU 11713 (MiB)
[03/03/2023-12:40:11] [TRT] [I] [MemUsageChange] Init cuDNN: CPU +82, GPU +122, now: CPU 2821, GPU 11835 (MiB)
[03/03/2023-12:40:11] [TRT] [I] Timing cache disabled. Turning it on will improve builder speed.
[03/03/2023-12:40:17] [TRT] [I] Total Activation Memory:<PHONE_NUMBER>
[03/03/2023-12:40:17] [TRT] [I] Detected 1 inputs and 4 output network tensors.
[03/03/2023-12:40:18] [TRT] [I] Total Host Persistent Memory: 239856
[03/03/2023-12:40:18] [TRT] [I] Total Device Persistent Memory: 0
[03/03/2023-12:40:18] [TRT] [I] Total Scratch Memory: 0
[03/03/2023-12:40:18] [TRT] [I] [MemUsageStats] Peak memory usage of TRT CPU/GPU memory allocators: CPU 18 MiB, GPU 384 MiB
[03/03/2023-12:40:18] [TRT] [I] [BlockAssignment] Started assigning block shifts. This will take 372 steps to complete.
[03/03/2023-12:40:18] [TRT] [I] [BlockAssignment] Algorithm ShiftNTopDown took 47.7322ms to assign 10 blocks to 372 nodes requiring 182681600 bytes.
[03/03/2023-12:40:18] [TRT] [I] Total Activation Memory: 182681600
[03/03/2023-12:40:18] [TRT] [I] [MemUsageChange] Init cuDNN: CPU +0, GPU +3, now: CPU 3143, GPU 12633 (MiB)
[03/03/2023-12:40:18] [TRT] [I] [MemUsageChange] Init cuDNN: CPU +0, GPU +5, now: CPU 3143, GPU 12618 (MiB)
[03/03/2023-12:40:18] [TRT] [I] [MemUsageChange] TensorRT-managed allocation in IExecutionContext creation: CPU +0, GPU +174, now: CPU 0, GPU 430 (MiB)
[03/03/2023-12:40:18] [TRT] [I] Starting Calibration.
[03/03/2023-12:40:20] [TRT] [I]   Calibrated batch 0 in 1.90551 seconds.
[03/03/2023-12:40:22] [TRT] [I]   Calibrated batch 1 in 1.72534 seconds.
[03/03/2023-12:40:24] [TRT] [I]   Calibrated batch 2 in 1.71749 seconds.
[03/03/2023-12:40:25] [TRT] [I]   Calibrated batch 3 in 1.73021 seconds.
[03/03/2023-12:40:27] [TRT] [I]   Calibrated batch 4 in 1.71629 seconds.
[03/03/2023-12:40:29] [TRT] [I]   Calibrated batch 5 in 1.7292 seconds.
[03/03/2023-12:40:31] [TRT] [I]   Calibrated batch 6 in 1.72344 seconds.
[03/03/2023-12:40:32] [TRT] [I]   Calibrated batch 7 in 1.74846 seconds.
[03/03/2023-12:40:34] [TRT] [I]   Calibrated batch 8 in 1.73344 seconds.
[03/03/2023-12:40:36] [TRT] [I]   Calibrated batch 9 in 1.72423 seconds.
[03/03/2023-12:40:38] [TRT] [I]   Calibrated batch 10 in 1.728 seconds.
[03/03/2023-12:40:39] [TRT] [I]   Calibrated batch 11 in 1.74652 seconds.
[03/03/2023-12:40:41] [TRT] [I]   Calibrated batch 12 in 1.75374 seconds.
[03/03/2023-12:40:43] [TRT] [I]   Calibrated batch 13 in 1.7546 seconds.
[03/03/2023-12:40:45] [TRT] [I]   Calibrated batch 14 in 1.75687 seconds.
[03/03/2023-12:40:47] [TRT] [I]   Calibrated batch 15 in 1.7355 seconds.
[03/03/2023-12:40:54] [TRT] [E] 2: [quantization.cpp::DynamicRange::80] Error Code 2: Internal Error (Assertion min_ <= max_ failed. )

2023-03-03 12:40:54.210 | ERROR    | __main__:<module>:182 - An error has been caught in function '<module>', process 'MainProcess' (469532), thread 'MainThread'<PHONE_NUMBER>37648):
Traceback (most recent call last):

> File "export.py", line 182, in <module>
    main()
    └ <function main at 0xfffeeae42940>

  File "/opt/ssd/visiontools/edgeyolo/venv/lib/python3.8/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
    return func(*args, **kwargs)
           │     │       └ {}
           │     └ ()
           └ <function main at 0xffff0bf304c0>

  File "export.py", line 169, in main
    data_save["model"] = model_trt.state_dict()
    │                    │         └ <function Module.state_dict at 0xffff21304160>
    │                    └ TRTModule()
    └ {'names': ['pedestrian', 'people', 'bicycle', 'car', 'van', 'truck', 'tricycle', 'awning-tricycle', 'bus', 'motor'], 'img_siz...

  File "/opt/ssd/visiontools/edgeyolo/venv/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1321, in state_dict
    hook_result = hook(self, destination, prefix, local_metadata)
                  │    │     │            │       └ {'version': 1}
                  │    │     │            └ ''
                  │    │     └ OrderedDict()
                  │    └ TRTModule()
                  └ <function TRTModule._on_state_dict at 0xfffed7158700>
  File "/opt/ssd/visiontools/edgeyolo/venv/lib/python3.8/site-packages/torch2trt-0.4.0-py3.8.egg/torch2trt/torch2trt.py", line 572, in _on_state_dict
    state_dict[prefix + "engine"] = bytearray(self.engine.serialize())
    │          │                              │    └ None
    │          │                              └ TRTModule()
    │          └ ''
    └ OrderedDict()

AttributeError: 'NoneType' object has no attribute 'serialize'

https://github.com/NVIDIA/TensorRT/issues/1634

in edgeyolo/export/calib.py, change this function to
class CalibDataset(Dataset):
    def __getitem__(self, item):
        import numpy as np
        ret = []
        for file in self.path_list[item:item + self.batch]:
            im, _ = preproc(cv2.imread(file), self.input_size)
            if self.norm:
                im /= 255.0
            im = np.ascontiguousarray(im, dtype=np.float16)
            ret.append(torch.from_numpy(im).unsqueeze(0).cuda())
        return [torch.cat(ret)]

I don't know whether it is useful because i didn't occur this problem. You can have a try

Unfortunately the change in the getitem did not help it.
Do you have another idea?

maybe change the same version of CUDA、cuDNN and tensorrt as mine?
my enviroment:
windows 10
CUDA 11.1
cuDNN 8.0.5
TensorRT <IP_ADDRESS>

or
ubuntu18.04
CUDA 10.2
cuDNN 7.X(I don't remenber exactly)
TensorRT <IP_ADDRESS>


Documentation: Readme - Add workload philosophy section.

Add a workload philosophy section to Readme.md
The Workload philosophy section gives some information about why an
educational workload layout is preferable to a concise one liner.
Inspired by: https://github.com/exercism/xruby/pull/567#pullrequestreview-34227224

Looks great to me! 👍

Thank you @hilary and @Insti .


Remove withIsCardanoEra workaround

BlockInMode now carries a IsCardanoEra constraint. We don't need this workaround anymore.

Pre-submit checklist:

Branch

[ ] Tests are provided (if possible)
[ ] Commit sequence broadly makes sense
[ ] Key commits have useful messages
[ ] Formatting, PNG optimization, etc. are updated

PR

[ ] Self-reviewed the diff
[ ] Useful pull request description
[ ] Reference the ADR in the PR and reference the PR in the ADR (if revelant)
[ ] Reviewer requested

Maybe this should be a PR on main? That also fixes the readthedocs build issue.

@andreabedini @sjoerdvisscher I'm merging main into next-node. I'll ping you when it's done.


Fix tests.

Codecov Report

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

@@          Coverage Diff          @@
##           master     #5   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines         181    181
=====================================
  Hits          181    181

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 3175fdb...d4f1c8f. Read the comment docs.

Lets bump the sleep to 1.25 seconds and keep c.delete.

I still have transient failures with a sleep(1.25).

I decided to remove the assertion in f94f2cb as it's mainly testing beanstalkd behavior anyways. Release 1.0.1 is on PyPI with this fix, thanks @dargor.


Allow dash (-) in aliases

When a dash (-) is present in an alias, the parser fails.
Example:
alias CVE-2022-26923 {
    #code
}

I added a dash to the t_ID regular expression, after which the CNA is successfully parsed. This was a quick fix on my end and I don't have much knowledge of lexers. Thus, I'm not sure if this will cause other things to break.

Thank you Max! All tests passed so merging 🎉


user ui : remove console log

Scope of changes
user ui : remove console log
Type of change

[x] bug fix
[ ] new feature
[ ] documentation
[ ] other (describe)

Acceptance criteria
user ui : remove console log
Author checklist

[ ] I have manually tested the change and/or added automation in the form of unit tests or integration tests
[ ]  I have updated the dependencies list
[ ]  I have recompiled and included new protocol buffers to reflect changes I made
[ ]  I have added new test fixtures as needed to support added tests
[x]   Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)
[x]  I have moved the associated Shortcut story to "Ready for Review"

Reviewer(s) checklist

[ ] Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.

Codecov Report

Merging #732 (0239dda) into main (338cebb) will decrease coverage by 5.26%.
The diff coverage is n/a.

:exclamation: Current head 0239dda differs from pull request most recent head 1b33025. Consider uploading reports for the commit 1b33025 to get more accurate results

@@            Coverage Diff             @@
##             main     #732      +/-   ##
==========================================
- Coverage   43.39%   38.13%   -5.27%
==========================================
  Files         583      155     -428
  Lines       17510     2431   -15079
  Branches     1310      618     -692
==========================================
- Hits         7599      927    -6672
+ Misses       8723     1327    -7396
+ Partials     1188      177    -1011

Impacted Files
Coverage Δ

...b/gds-user-ui/src/components/BasicDetail/index.tsx

...eb/gds-user-ui/src/components/FileUpload/index.tsx

...nents/TestnetProgress/CertificateStepContainer.tsx

...i/src/components/TrisaImplementationForm/index.tsx

...er-ui/src/components/ui/InputFormControl/index.tsx

web/gds-user-ui/src/hooks/useAuth.ts

...b/gds-user-ui/src/modules/auth/login/user.slice.ts

...ules/dashboard/certificate/lib/validationSchema.ts

...src/modules/dashboard/certificate/registration.tsx

...ds-user-ui/src/modules/dashboard/profile/index.tsx

... and 418 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


virtcontainers: vhost-user-blk/scsi are block device nodes

When checking if a device is an emulated vhost-user-blk or
vhost-user-scsi one, we should not only check for their major number but
also their device node type. They must be block devices.
Fixes: #2822
Signed-off-by: Samuel Ortiz<EMAIL_ADDRESS>

/test

PR ported to 2.0: https://github.com/kata-containers/kata-containers/pull/402

/test


SIGSEGV while running (docker version)

Describe the bug
SIGSEGV while running bridging IRC and telegram channels. It needs investigation if it is misconfiguration of library in matterbridge's code or bug in 3rd party library (lrstanley/girc).
To Reproduce
Random. It happened rarely, but today it happened 19 times.
Expected behavior
Well, do not SIGSEGV.
Screenshots/debug logs
time="2024-04-16T13:51:32Z" level=info msg="Running version 1.26.1-dev 56e7bd0" prefix=main

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x28 pc=0xb52fa6]

goroutine 13269432 [running]:
github.com/lrstanley/girc.(*Client).readLoop(0xc00068a700, {0x26a1d90, 0xc0000d6550})
        /go/src/matterbridge/vendor/github.com/lrstanley/girc/conn.go:440 +0x266
github.com/lrstanley/girc/internal/ctxgroup.(*Group).Go.func1()
        /go/src/matterbridge/vendor/github.com/lrstanley/girc/internal/ctxgroup/ctxgroup.go:58 +0x6e
created by github.com/lrstanley/girc/internal/ctxgroup.(*Group).Go
        /go/src/matterbridge/vendor/github.com/lrstanley/girc/internal/ctxgroup/ctxgroup.go:55 +0x8d

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x28 pc=0xb52fa6]

goroutine 8 [running]:
github.com/lrstanley/girc.(*Client).readLoop(0xc000254700, {0x26a1d90, 0xc0002e6140})
        /go/src/matterbridge/vendor/github.com/lrstanley/girc/conn.go:440 +0x266
github.com/lrstanley/girc/internal/ctxgroup.(*Group).Go.func1()
        /go/src/matterbridge/vendor/github.com/lrstanley/girc/internal/ctxgroup/ctxgroup.go:58 +0x6e
created by github.com/lrstanley/girc/internal/ctxgroup.(*Group).Go
        /go/src/matterbridge/vendor/github.com/lrstanley/girc/internal/ctxgroup/ctxgroup.go:55 +0x8d

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x28 pc=0xb52fa6]

goroutine 564 [running]:
github.com/lrstanley/girc.(*Client).readLoop(0xc0009381c0, {0x26a1d90, 0xc0000df270})
        /go/src/matterbridge/vendor/github.com/lrstanley/girc/conn.go:440 +0x266
github.com/lrstanley/girc/internal/ctxgroup.(*Group).Go.func1()
        /go/src/matterbridge/vendor/github.com/lrstanley/girc/internal/ctxgroup/ctxgroup.go:58 +0x6e
created by github.com/lrstanley/girc/internal/ctxgroup.(*Group).Go
        /go/src/matterbridge/vendor/github.com/lrstanley/girc/internal/ctxgroup/ctxgroup.go:55 +0x8d
time="2024-04-16T14:13:38Z" level=info msg="Running version 1.26.1-dev 56e7bd0" prefix=main

Environment (please complete the following information):

OS: linux
Matterbridge version: 1.26.1-dev 56e7bd0, docker image
If self compiled: I am using most recent, "official" docker image (6beb4da71858)

The girc library has been updated on master, try running lastest docker image


Update spec. to support "dependencies" grammar...

Please see: https://github.com/openwhisk/openwhisk-wskdeploy/pull/243
which has the proposed grammar (and experimental code added to support).
FYI, @houshengbo @daisy-ycguo @lzbj
Other issues to consider (or open as separate):

Current impl. is a list (array); we need a true dep. graph
Dep. graph should assure:
a. No cycles
b. dependency order (if this cannot be derived, we need "-" to change grammar to ordered list to impose author provided order).
c. Version resolution; that is, if diff. packages ref. the same dependency, they must be at the same version.
d. Provide warnings for unused dependencies.

Fixed in spec. version 0.8.5 uploaded 2017-04-21 via PR https://github.com/apache/incubator-openwhisk-wskdeploy/pull/267


Custom email server

I have implemented a custom email server option, subject prefix, and fixed a bug in the log function that causes an infinite loop.

This is great!  Merging now.  Thanks so much for your contribution.


Fix: Reset cut items opacity when copying other items

Resolved / Related Issues

[ ] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
Closes #12249

I'm wondering if we should implement Esc to cancel cut as in file explorer....
Validation
How did you test these changes?

[x] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[x] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[x] Are there any other steps that were used to validate these changes?

Open app
Cut some items
Select other items and select them
See that items opacity resets

I'm wondering if we should implement Esc to cancel cut as in file explorer....

@hishitetsu what do you think about this?

I'm wondering if we should implement Esc to cancel cut as in file explorer....

@hishitetsu what do you think about this?

I didn't know about this feature. I don't need it. I think it is unnecessary until someone requests it.


The final model (GPT-NEO-1.3B) doesn't save after training.

System Info
- `Accelerate` version: 0.15.0
- Platform: Linux-5.15.0-58-generic-x86_64-with-glibc2.35
- Python version: 3.10.6
- Numpy version: 1.24.1
- PyTorch version (GPU?): 1.13.1+cu117 (True)
- `Accelerate` default config:
        - compute_environment: LOCAL_MACHINE
        - distributed_type: DEEPSPEED
        - mixed_precision: bf16
        - use_cpu: False
        - dynamo_backend: NO
        - num_processes: 2
        - machine_rank: 0
        - num_machines: 1
        - gpu_ids: None
        - main_process_ip: None
        - main_process_port: None
        - rdzv_backend: static
        - same_network: True
        - main_training_function: main
        - deepspeed_config: {'gradient_accumulation_steps': 1, 'offload_optimizer_device': 'cpu', 'offload_param_device': 'cpu', 'zero3_init_flag': True, 'zero3_save_16bit_model': True, 'zero_stage': 3}
        - fsdp_config: {}
        - megatron_lm_config: {}
        - downcast_bf16: no
        - tpu_name: None
        - tpu_zone: None
        - command_file: None
        - commands: None

Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[X] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[ ] My own task or dataset (give details below)

Reproduction
I try to train GPT-NEO-1.3B on my dataset. But faced with a problem after training, the size of pytorch_model.bin is only 98 Mbytes. For training, I used the following script. For example, checkpoints were saved correctly.
Expected behavior
the size of `pytorch_model.bin` will be ~5GB

Please provide us the command you used to run the script.

No, the training was finished without errors and interruptions.

Please let us know if that solves the issue

@pacman100 Thanx it's working


weird algorithm

in this problem there is an algorithm that takes as input a positive integer n. If n is even, the algorithm divides it by two, and if n is odd, the algorithm multiplies it by three and adds one. The algorithm repeats this, until n is one.
Hey, I am Vabhv1903 a GSSOC'21 Participant. I am having experience in C++ Coding as well as I am doing Competitive coding in the C++ programming language.
Please assign this task to me :)
Thank-you.
@anubhavitis @dheerajkotwani @tarun26091999 @todi-2000 @geekquad @plazzy99

I want to work on this issue. Please assign to me.

We do not accept the problems available on competitive sites. We only accept standard algorithms and questions.
https://www.codechef.com/problems/WALG


problems with transfer

Hello I would like to know if you can cancel a transfer of lumens because I made the transfer and for not knowing, do not put the memo code and it is already 24 hours and the transfer does not reach my wallet ... what can I do?

Yo también tengo la misma problema y no se a quien reclamar


When using mqtt.js to connect to asp.net core, requests are always made

When using https://github.com/mqttjs/MQTT.js to connect to asp.net core, requests are always made,but use https://github.com/eclipse/paho.mqtt.javascript ,it's ok

Hi, sorry but I don't understand your question. Please provide more details.
Best regards
Christian


Revert PR #518 to re-enable test cases

PR #518 disabled some test cases due to an issue with Mono 4.2.x. The bug was fixed later in the 4.2.x builds. The current stable mono build is now 4.8.x, and anyone who's working with this repo (and running the test cases) should be running at least 4.6.x, so let's re-enable these test cases for parity with the Microsoft/visualfsharp repo.

I restarted the failed job (I think the failure was unrelated)

The macOS job with mono-latest failed again. Both times it was due to an error in travis-ci's internal scripts, where the script wasn't able to fetch the mono package.

@jack-pappas Thanks for this change


Introspection Mapping to Existing Database (Relation with Extra Columns) returns SQL Error

I am looking at the docs (https://www.prisma.io/docs/data-model-and-migrations/introspection-mapping-to-existing-db-soi1/), because I am having an issue with a specific join type.
Similar to what is in the docs here
type Bill @pgTable(name: "bill") {
  bill: String!
  id: Int! @unique
  bill_products: [Bill_product]
}

type Bill_product @pgTable(name: "bill_product") {
  bill: Bill @pgRelation(column: "bill_id")
  product: Product @pgRelation(column: "product_id")
  some_other_column: String!
}

type Product @pgTable(name: "product") {
  id: Int! @unique
  product: String!
  bill_products: [Bill_product]
}

I am trying to access some_other_column data when I query
query {
  bill(where: { id: 1 }){
    id
    bill_products {
      product {
        id
        product
      }
      some_other_column
    }
  }
}

When I run the SQL from the docs, add the types to my app.prisma file then run prisma deploy I get 3 errors
Bill_product ✖ The required field 'id' is missing and has to have the format: id: ID! @unique or id: UUID! @unique or id: Int! @unique.

Bill ✖ The relation field ‘bill_products’ has the wrong format: ‘[Bill_product]’ Possible Formats
: ‘Bill_Product’, ‘Bill_product!‘, ‘[Bill_product!]\!’

Products ✖ The relation field ‘bill_products’ has the wrong format: ‘[Bill_product]’ Possible Formats
: ‘Bill_Product’, ‘Bill_product!‘, ‘[Bill_product!]\!’

When I resolve these 3 errors and run prisma deploy all is well
When I try to query
query {
  bill(where: { id: 1 }){
    id
    bill_products {
      product {
        id
        product
      }
      some_other_column
    }
  }
}

I get errors in PostgreSQL
ERROR:  column Alias.id does not exist at character 217
STATEMENT:  select
  "Alias"."id",
  "RelationTable"."id" as "__RelatedModel__",
  "RelationTable"."bill_id" as "__ParentModel__"
from "public"."bill_product" as "Alias"
  join "public"."bill_product" as "RelationTable"
  on "Alias"."id" = "RelationTable"."id"
where "RelationTable"."bill_id" in ($1)
order by "RelationTable"."id" asc

Versions:

OS: OS X High Sierra
prisma CLI: prisma/1.20.7 (darwin-x64) node-v10.10.0
Prisma Server: 1.19
graphql-cli, prisma-binding

@epitaphmike : This is a current limitation of the Postgres connector. The connector requires each specified type to be backed by a table that has an id column. In your example Bill_product seems to be a relation table which does not have an id column. We are currently working on lifting this limitation.

In your example Bill_product seems to be a relation table which does not have an id column.

Thanks @mavilein. I do want to point out that this isn't my example. This is an example from the Prisma Docs under Relations in the generated datamodel > Relation Tables.
https://www.prisma.io/docs/data-model-and-migrations/introspection-mapping-to-existing-db-soi1/#relations-in-the-generated-datamodel

Thanks @mavilein. If there is anything I can do to assist please let me know.


Cluster Express Server

It would greatly help the performance of the image commands if they were clustered. I'm not sure whether the cluster would be of server.js or the individual image commands as I don't have much experience with clustering.
For reference, there are usually 10k-20k image commands ran per day, so single thread is hurting.

Maybe you're looking after something like pm2 and its cluster mode.

#4


Browser debug console errors after update to 3.7.2

This is  a bug.

Prerequisites

[ x] Are you running the latest version?
[ x] Are you reporting to the correct repository?
(enso is made of many specialized packages: https://github.com/laravel-enso)
[ x] Did you check the documentation?
[ x] Did you perform a cursory search?

Description

After login the following errors appear in debugger console. They seems to be related to the new vue-cli build..
Steps to Reproduce

Update to Enso version 3.7.2
Build the resources using new vue-cli
Login to application and open browser debug console
Check the console for error messages

Expected behavior
The debug console should not contain any errors regarding the build
Actual behavior
The debug console shows a bunch of errors related probably to vue-cli build

I am unable to replicate this.
Do you have your websockets setup right?

Are there any extra changes on the enso 3.7.2 regarding sockets? I haven't
had this issue on 3.6.1
I mention that I run enso on windows with apache, if that matters.
Thanks
On Mon, Nov 25, 2019 at 7:39 AM Adrian Ocneanu<EMAIL_ADDRESS>wrote:

I am unable to replicate this.
Do you have your websockets setup right?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/laravel-enso/enso/issues/269?email_source=notifications&email_token=AN23K2V7NRLUWQ73BNLYZG3QVNQHJA5CNFSM4JQOHA32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFBGDHQ#issuecomment-557998494,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN23K2QX7BY6LBOYRYZRI3LQVNQHJANCNFSM4JQOHA3Q
.

This is no longer an issue after we updated our code part.
Thank you!


Runtime errors for compiled examples

Library examples are compiled without errors.
Errors appear at runtime.
A compiled example of "basics" is available here.
I also want to note that the "libs" folder contains old versions of the library, which creates inconvenience.
I used the latest pixijs from haxelib and haxe 4.1.1

The examples and the lib folder are outdated. Here's what you need to change:
https://github.com/pixijs/pixi-haxe/pull/171/commits/28041113d0d8f37dd477f38465ab9e80c1ec33e3
The result using haxe 4.0.5/pixi 5:
https://notboring.github.io/demos/haxe-pixi/basics.html


How does one introduce new category classes to the ConCat plugin?

Do you have any documentation on how we introduce new category classes to the plugin, and how we identify which Haskell functions should be represented in the abstraction by methods of those classes?

Pinging @ryantrinkle

Nothing explicit. There are two phases of translation. One turns an existing non-class operation into a corresponding invocation a class operation with a (->) dictionary. The other is to translate such method from (->) to another category.
The plugin attempts the second phase automatically when it sees an operation applied to (->). There's a tricky point about avoiding GHC immediately inlining class operations before the plugin can translate class operations. My workaround is illustrated in ConCat.AltCat and described in section 10.3 of Compiling to categories.
The first phase of translation is mostly handled internal to the plugin, turning things like ccc (&&) or ccc $fNumFloat_$cnegate (GHC Core) into (non-inlining aliases of) methods (curry andC and negateC, respectively). I think you can simply add these translations as GHC rewrite rules. I'd like to get all such specifics out of the plugin and into transparent rewrite rules. I haven't done so yet, as I'd expect compilation to slow down with a lot of such rules, due to GHC current implementation of rewrite rule search. Simon PJ has mentioned that he's keen to have the rewrite rule implementation improved to use a trie-based approach, which I think would help tremendously, reducing lookup from linear to something like logarithmic in the number of rules.

I'm afraid I don't quite follow yet as to what I'm actually supposed to do.  Let's say I want to make teletypeCat, with methods getC and putC, and the arrows in (->) I want represented for this are get :: Teletype Char and putC :: Char -> Teletype ().
Once I've defined the typeclass, how do I teach the plugin the do the mapping?  Am I just writing rewrite rules to do this? What would such a rule look like?

As long as you prevent your class operations from getting inlined (into dictionary accessors), you can just refer to them in your code, and the plugin will try converting to your target category. You don't have to add rewrite rules. The tip-off is that getC and putC have (->) as their first argument in the Core. Seeing this pattern in a ccc call, the plugin tries to replace (->) by your target category and then replace all of the dictionaries with ones synthesized to fit the target. The only awkward bit (sigh) is preventing GHC from inlining away getC and putC before the plugin can see them, which is the whole unfortunate point of ConCat.AltCat. You'll need to do something similar (late-inlining aliases for getC and putC). There's some interest in fixing this unfortunate behavior of GHC.

When you say "just refer to them in your code", do you mean change my code to call getC and putC directly from teletypeCat?  That is, I basically create instances for my categories in (->), and then use those categories for my programming?  I guess that makes sense, because then ccc really has no work to do, as long as the instances aren't being chosen before ccc has a chance to examine the term.

I mean refer to the late-inlining getC and putC in code that uses your TeletypeCat class, not defines it. Similarly, the test code I compile imports ConCat.AltCat (aliases) rather than ConCat.Category. You will have defined a (probably bogus) TeletypeCat (->) instance, allowing your original code to type-check. The ccc transformations will then end up changing to a category with the TeletypeCat instance you want.
If it's still unclear what to do, we could have a Skype chat to clarify, and then you could add your understanding here as a favor to those who find this thread later.

Conal has done some more work on this, and now we're at the point where we just have one coercion error standing in the way, even if the instance being reported is defined:
ghc: panic! (the 'impossible' happened)
  (GHC version 8.0.2 for x86_64-apple-darwin):
	ccc - couldn't build dictionary for
  coerceC @ (->) @ (V 'V2 Int) @ Int :: CoerceCat
                                          (->) (V 'V2 Int) Int =>
                                        V 'V2 Int -> Int:
  no bindings

Not only is V l v coercible to v for any l, but I also have the instance defined:
instance CoerceCat (->) (V l v) v where
    coerceC (V v) = v

This happens when switching from the type class instances in ConCat.Category, to those in ConCat.AltCat, to avoid GHC inlining the members of custom categorical structures.

I should mention, this bug can be seen by building my https://github.com/jwiegley/categorical project, which I've created as a testbed for trying out ConCat ideas.

Hmm - I've tried running your example and it gives me this even when the COERCE_BUG is not defined.
Hello, Haskell!
Main.hs: internal error: stg_ap_p_ret
    (GHC version 8.0.2 for x86_64_unknown_linux)
    Please report this as a GHC bug:  http://www.haskell.org/ghc/reportabug
[1]    12477 abort (core dumped)  cabal exec -- runghc Main.hs

This is happening because there was a "residual": a construction that the plugin was unable to abstract.  This is the next problem I'm having with this example. And then there's one more, once we get past these two.

Ah. How does the plugin associate e.g. Floating with FloatingCat? By name?
I tried to make the simplest case I could think of, but I'm guessing I need a much deeper understanding of how the re-write rules work... but I'm very much blundering around in the dark with no idea what I'm doing.
class Show a => Foo a where
  foo :: a -> String
  foo = show
  {-# NOINLINE foo #-}

instance Show a => Foo a

class (Show a, Ok k a) => FooCat k a where
  fooC :: a `k` String

instance (Show a) => FooCat (->) a where
  fooC = P.show
  {-# NOINLINE fooC #-}

program :: (Int, Int) -> String
program (x, y) = foo (x + y)

@conal I wonder if the missing CoerceCat might have to do with the use of DataKinds in my example?

So I finally understand what the discussion above is about. Instead of having a class FooCat and a class Foo I only need a class FooCat and an instance FooCat (->). It's confusing because the standard instances are all of this pattern e.g. Floating and FloatingCat, I guess this is necessary because the types don't always match up with regard to currying?
This all works - as long as I have my instances in another module and use the AltCat trick.

@Saulzar Could you share your example perhaps? I think that collecting together some real examples would help future learners of concat. I'd also like to compile a file filled with notes of the form "if you see this error, it means this, and here's how you can debug it."

I second John's request. This project has just been me so far and it has some rough edges. I'm grateful for contributed tips, examples, etc.

I've put up my few tinkerings here:
https://github.com/Saulzar/category_tests
I'll add more as I go...
My interest here is doing things like deep learning, but there would seem to be an enormous number of DSLs which would be much simpler!
Playing games with the inliner reminds me of trying to write a reflex implementation to avoid having weak references disappear, not something which was much fun!

Thanks, @Saulzar.

I'd like to take a stab at the McIlRoy power series example you mentioned.  Did you have a rough outline of what you envisioned? I also have a project enumerating properties of small finite functions if you want some transformation semigroup classes.

@chadbrewbaker I don't have a rough outline other than the following generic approach:

Choose a representation for power series.
Define instances for Category, ProductCat, NumCat, and perhaps a few others.
Bonus: prove these instances correct via a "semantic function" that maps the power series type to the usual type of functions, together with saying that the semantic function is a homomorphism w.r.t. those classes.
Better yet, derive the instances from this homomorphic specification.

Any recommendations on adding mutable operations? I'm thinking about adding something that will be compiled into Block RAM in Verilog and stay as mutable pointer in ST monad in Haskell.


Information needed : How can we implement WebDriverManager in QAF project

QAF Version
2.1.10
Information needed : How can we implement WebDriverManager in QAF project so that we don't need to provide driver for the browser and the latest version would be automatically downloaded.

Here I have implemented WebDriverManager in QAF. You can refer this example.
https://github.com/amitbhoraniya/Databean-Component-Example

@amitbhoraniya Hi, why we have to provide browser driver in application.properties file like you did in your project
webdriver.chrome.driver =/Users/amit.bhoraniya/Documents/drivers/chromedriver,
if we are using webdrivermanager and it automatically downloads the driver.

You don't need to provide that.

@amitbhoraniya But it's throwing error if I am not providing that property.

That example uses driver listener did you created and registered listener?

@cjayswal @amitbhoraniya thanks for the suggestions. After adding WDListener class, i am able to run my script without the need of providing the browser driver.


python312Packages.pytensor: 2.25.4 -> 2.25.6

Things done
Upstream has re-released latest version of pymc so I refreshed the hash as necessary.

python312Packages.pytensor: 2.25.4 -> 2.25.6
python312Packages.pymc: update hash

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 346130

x86_64-linux

:fast_forward: 2 packages marked as broken and skipped:

python311Packages.bambi
python311Packages.bambi.dist

:white_check_mark: 4 packages built:

python311Packages.pymc
python311Packages.pymc.dist
python311Packages.pytensor
python311Packages.pytensor.dist

aarch64-linux

:fast_forward: 2 packages marked as broken and skipped:

python311Packages.bambi
python311Packages.bambi.dist

:white_check_mark: 4 packages built:

python311Packages.pymc
python311Packages.pymc.dist
python311Packages.pytensor
python311Packages.pytensor.dist

x86_64-darwin

:fast_forward: 6 packages marked as broken and skipped:

python311Packages.bambi
python311Packages.bambi.dist
python311Packages.pymc
python311Packages.pymc.dist
python311Packages.pytensor
python311Packages.pytensor.dist

aarch64-darwin

:fast_forward: 6 packages marked as broken and skipped:

python311Packages.bambi
python311Packages.bambi.dist
python311Packages.pymc
python311Packages.pymc.dist
python311Packages.pytensor
python311Packages.pytensor.dist


move license to root level

@alukach I'm moving the LICENSE to the root level. Was it intentionally in the spec directory?

Hmmm... I have no idea how this ended up in spec. Good change


'getTimeouts' error appium server

Hello, I am encountering an error like this when starting the application with Appium inspector. My app is in debug mode.
Any ideas on what I'm doing wrong?
[FlutterDriver@3cc8 (04bfc84e)] Responding to client with driver.getTimeouts() result: {"command":3600000,"implicit":0}
[debug] [FlutterDriver@3cc8 (04bfc84e)] Responding to client with driver.getTimeouts() result: {"command":3600000,"implicit":0}
[HTTP] <-- GET /session/04bfc84e-c5ae-42df-ab09-2fce3945c3a5/timeouts 304 2 ms - -

As the error message, appium-flutter-driver's FLUTTER context does not support the page_source.
https://github.com/appium-userland/appium-flutter-driver#note


add new tags for netcore versions of transaction samples.

read_only_transaction_core
read_write_transaction_core
also adds a new tag for showing code on how to use the transient fault app block to handle transaction retries.
read_write_retry
(only one is needed as its the same in netcore and net45)

I'll give the tests a go.  shouldnt be hard if there are tests covering the net45 version

I've added tests and they pass.  However, with the way the commands and tests are structured, you need to have previously set up the sample database with the following commands:
(replace spanneref with your own projectid)
spanner createSampleDatabase spanneref my-instance my-database
spanner insertSampleData spanneref my-instance my-database
spanner addColumn spanneref my-instance my-database
spanner writeDataToNewColumn spanneref my-instance my-database
I recommend making the tests runnable from scratch -- but that's a bit of work and we need to get Sandra unblocked asap.


Inconsistent de-indentation of jsdoc comment between  format selection and format document

Bug Report
🔎 Search Terms

format
formatting
formatter
format selection

🕗 Version & Regression Information
TypeScript version tested: 4.2.0-dev.20210112
💻 Code
{
    /**
     * Some doc comment
     */
    const a = 1;
}

First run format document
Notice no change
Then select just the doc comment and run format selection

🙁 Actual behavior
The doc comment is de-indented:
{
/**
 * Some doc comment
 */
    const a = 1;
}

If you run format document, this indent is added back
🙂 Expected behavior
No change. The document is already formatted

Related #42309
Also marking this as a VS Code priority. It is easy to hit if you run the format modified command or set modified as the default mode for format on save using "editor.formatOnSaveMode": "modified"

Related #42309
Also marking this as a VS Code priority. It is easy to hit if you run the format modified command or set modified as the default mode for format on save using "editor.formatOnSaveMode": "modified"


[19.0.x] WFLY-13099 Standard configuration files for MicroProfile use cases

https://issues.redhat.com/browse/WFLY-13099
Upstream: #13015

retest this please

retest this please

retest this please

I have tried this with jboss.dist and got inconsistent results.
mvn clean install -DskipTests
cd testsuite
mvn clean test -Dts.standalone.microprofile -Djboss.dist=/home/okotek/test/server -fae

Web and Smoke modules seem to ignore the server referenced by jboss.dist. I have checked WildFly Full <version> (WildFly Core <version>) starting message in target/wildfly*/standalone/log/server.log files of the modules.
Also, in case the referenced server dosn't contain the new configs, I would expect all the test modules to fail. However, Web, Smoke, and MicroProfile - OpenAPI TCK pass:
[INFO] WildFly Test Suite: Aggregator ..................... SUCCESS [  1.540 s]
[INFO] WildFly Test Suite: Integration (parent) ........... SUCCESS [  0.998 s]
[INFO] WildFly Test Suite: Integration - Web .............. SUCCESS [ 33.075 s]
[INFO] WildFly Test Suite: Integration - Smoke ............ SUCCESS [ 23.196 s]
[INFO] WildFly Test Suite: Integration - Basic ............ FAILURE [ 23.915 s]
[INFO] WildFly Test Suite: Integration - MicroProfile ..... FAILURE [  3.656 s]
[INFO] WildFly Test Suite: Integration - MicroProfile TCK . SUCCESS [  0.181 s]
[INFO] WildFly Test Suite: Integration - MicroProfile TCK - Config FAILURE [  4.677 s]
[INFO] WildFly Test Suite: Integration - MicroProfile TCK - Fault Tolerance FAILURE [  6.073 s]
[INFO] WildFly Test Suite: Integration - MicroProfile TCK - Health FAILURE [  5.913 s]
[INFO] WildFly Test Suite: Integration - MicroProfile - JWT FAILURE [  7.486 s]
[INFO] WildFly Test Suite: Integration - MicroProfile TCK - Metrics FAILURE [  4.516 s]
[INFO] WildFly Test Suite: Integration - MicroProfile - OpenAPI TCK SUCCESS [ 49.448 s]
[INFO] WildFly Test Suite: Integration - MicroProfile TCK - OpenTracing FAILURE [  4.862 s]
[INFO] WildFly Test Suite: Integration - Clustering ....... FAILURE [ 10.986 s]

Also, it seems that modules directory of the referenced server is used by Basic test module for testing somehow, the modified timestamp is updated.

@OndrejKotek That's for those findings. I've added 3 commits that should address the issues with smoke, web and microprofile-tck/openapi.
I see nothing that would alter whether the modules dir gets written to in this PRs change to ts/integration/basic. Does this not occur with master/19.0.x?  There may be a test writing temporary modules there. If so we should file an issue to get that cleaned up.

retest this please

@bstansberry Thank you for the fix. Smoke still passes despite the new configs missing in the server referenced by jboss.dist. According to target/wildfly/standalone/log/server.log the referenced server is used, however there is standalone-microprofile.xml config (used) and original modules (not used?) in target/wildfly.
Regarding modules directory of the referenced server used by Basic test module, it's an issue unrelated to this PR, I'll file it separately.

@OndrejKotek  Thanks. I'll try again with 'smoke'. I thought I'd made it fail when pointing -Djboss.dist at output from the ee-dist module but I probably created a false negative somehow. Or somehow broke it again.
Thanks for filing something re writing to 'modules'. That kind of thing has come up before and probably got cleaned up for some specific test where it came up but we probably never cleaned up the whole testsuite.

@OndrejKotek I added a commit dealing with 'smoke'. It would fail if -Djboss.dist pointed at a dist without MP, but the failure would be because modules referred to in the config file would not be found. The smoke ts is different from the others because by default it doesn't copy stuff from jboss.dist into target/wildfly. It uses Galleon to provision a server. For the ts.standalone.microprofile layer I turned that off and went back to the copying approach. So there will no longer be a target/wildfly/standalone/configuration/standalone-microprofile.xml if jboss.dist didn't have one. That means the server will immediately fail to boot due to a non-existent config file.


SHA-3 and SHAKE

This is #479, rebased to current development (2.13), with the following main changes:

Combine everything into a single module. The Keccak modules are not useful on their own. SHA3 without SHAKE or vice versa represents a negligible gain in code size. Therefore separating the modules isn't worth the trouble.
Add cSHAKE from NIST SP800-185 (which hadn't been published yet when #479 was made). I haven't added the other algorithms from SP800-185, but they can be implemented using cSHAKE as a black box.
Update to the latest library features and conventions:

Message digest functions now return error codes.
Use the new data_t type for binary data as arguments to unit tests.
Add HW_FAILED errors.
Declare only context types in #ifndef xxx_ALT blocks, not functions.

Shrink the SHA3 state from 400B+change to 200B+change.
Documentation and style changes to bring it closer to our guidelines.
Various test coverage improvement. In particular, add test cases for SHA-3 in the generic md test suite.

The original version of this PR was for 2.8. Its history is available at https://github.com/gilles-peskine-arm/mbedtls/tree/sha3-2.8-1 but this is of little interest except to follow the first round of review.

Please fix compilation warnings on mingw:

[win32-mingw] C:\builds\ws\mbedtls-pr\library\sha3.c: In function 'mbedtls_sha3_kat_test':
[win32-mingw] C:\builds\ws\mbedtls-pr\library\sha3.c:353:29: warning: unknown conversion type character 'z' in format [-Wformat=]
[win32-mingw]              mbedtls_printf( "  %s test %zi error code: %i\n",
[win32-mingw]                              ^
[win32-mingw] C:\builds\ws\mbedtls-pr\library\sha3.c:353:29: warning: format '%i' expects argument of type 'int', but argument 3 has type 'size_t {aka long long unsigned int}' [-Wformat=]
[win32-mingw] C:\builds\ws\mbedtls-pr\library\sha3.c:353:29: warning: too many arguments for format [-Wformat-extra-args]
[win32-mingw] C:\builds\ws\mbedtls-pr\library\sha3.c:380:29: warning: unknown conversion type character 'z' in format [-Wformat=]
[win32-mingw]              mbedtls_printf( "  %s test %zi failed\n", type_name, test_num );
[win32-mingw]                              ^
[win32-mingw] C:\builds\ws\mbedtls-pr\library\sha3.c:380:29: warning: too many arguments for format [-Wformat-extra-args]
[win32-mingw] C:\builds\ws\mbedtls-pr\library\sha3.c:388:25: warning: unknown conversion type character 'z' in format [-Wformat=]
[win32-mingw]          mbedtls_printf( "  %s test %zi passed\n", type_name, test_num );
[win32-mingw]                          ^
[win32-mingw] C:\builds\ws\mbedtls-pr\library\sha3.c:388:25: warning: too many arguments for format [-Wformat-extra-args]
[win32-mingw] [ 14%] Building C object library/CMakeFiles/mbedcrypto.dir/sha512.c.obj
[win32-mingw] [ 15%] Building C object library/CMakeFiles/mbedcrypto.dir/shake.c.obj
[win32-mingw] C:\builds\ws\mbedtls-pr\library\shake.c: In function 'mbedtls_shake_self_test':
[win32-mingw] C:\builds\ws\mbedtls-pr\library\shake.c:261:29: warning: unknown conversion type character 'z' in format [-Wformat=]
[win32-mingw]              mbedtls_printf( "  SHAKE128 test %zi ", i );
[win32-mingw]                              ^
[win32-mingw] C:\builds\ws\mbedtls-pr\library\shake.c:261:29: warning: too many arguments for format [-Wformat-extra-args]
[win32-mingw] C:\builds\ws\mbedtls-pr\library\shake.c:287:29: warning: unknown conversion type character 'z' in format [-Wformat=]
[win32-mingw]              mbedtls_printf( "  SHAKE256 test %zi ", i );
[win32-mingw]                              ^

retest

Adding label "needs: work" due to the numerous conflicts.

Is this PR pending and waiting for help? If so, I want to give it a try

@HowJMay Thanks for the offer! Yes, this PR is still pending. It needs significant work to adapt to the library conventions in Mbed TLS 3.0. It will probably be easier to build a new history than to try to rebase this one. You can see a modern example of adding a new hash to the library with SM3.
The Mbed TLS team is currently focusing on preparing Mbed TLS 3.0. As a consequence, we won't be able to review or assist much until July.

Thanks @gilles-peskine-arm. It is ok, I need some time on studying the convention, too.

Will this apply cleanly to a 2.16 version ?

@InspireSemi Our main bottleneck is review. We don't have time to review all the work we do, plus all the contributions we're getting, plus the backlog of contributions.
We can prioritize certain contributions based on demand, but we have a significant backlog even for that.
Please note that Mbed TLS 2.16 is no longer maintained since December 2021 and will not receive any further bug fixes. You should upgrade to 2.28 or 3.x as soon as possible.

Closing in favour of https://github.com/Mbed-TLS/mbedtls/pull/5820 and https://github.com/Mbed-TLS/mbedtls/pull/5821 - which now have backlinks to this PR, so reviewers can check that nothing useful was lost (in particular, tests).


Init firebase. Some users report that the application crashes

if this is not done is not done

Note that this change has been reverted, because it does not fix the issue


[FEAT]: Add "keep_alive" Parameter Configuration in UI for Ollama Models

What would you like to see?
Dear Maintainers,
I hope this message finds you well. First and foremost, I would like to express my gratitude for your continuous efforts in maintaining and improving Anything LLM. Your work is highly appreciated by the community.
I am writing to request the addition of a feature that would greatly enhance the usability and flexibility of the application, specifically when using Ollama models. Currently, the Ollama models support a "keep_alive" parameter, as documented here on ollama and here in langchain. (I've seen in the source code that you use langchain as a wrapper around the call to ollama which is why I'm providing that link.)
The "keep_alive" parameter is useful for keeping ollama models in memory longer - and as of now, there is no option to configure this parameter directly from the UI.
Problem:
The model always defaults to a 5-minute load in memory only. This behavior can be observed using the "ollama ps" command once you start a chat in Anything LLM. This default setting leads to the model unloading from memory after 5 minutes, which is not ideal for scenarios requiring persistent connections. (Additionally, I've tested the OLLAMA_KEEP_ALIVE environment variable implemented by Ollama does not get respected when using the API/chat endpoint from Anything LLM). I've also tried the curl http://localhost:11434/api/generate -d '{"model": "llama3", "keep_alive": -1}' command (docs) but a new call from AnythingLLM seems to override this setting and default back to 5 mins.
Feature Request:

Add "keep_alive" Configuration in UI:

Introduce ani nput field in the ollama-specific UI that allows users to enter a value into the "keep_alive" parameter when configuring Ollama models.
Optionally, provide a brief description or tooltip explaining the purpose and impact of the "keep_alive" parameter?

Benefits:

Improved Performance: Allowing users to configure the "keep_alive" parameter can help maintain persistent connections, reducing latency and improving overall performance when making requests with a >5 min break in between.
Enhanced Flexibility: Users can tailor the behavior of the Ollama models to better suit their specific use cases.
Resolve Current Limitation: Address the current limitation where the model defaults to a 5-minute memory load.

I believe this feature will be a valuable addition to Anything LLM and will enhance the user experience for many. Thank you for considering this request. If you need any further information or assistance, please do not hesitate to contact me.
Best regards,
Simon

Also, if it helps, I'm currently running Anything LLM in a Docker container in WSL2 and accompanied by a direct native install of Ollama in WSL 2 on Windows (due to my company restrictions on installing files).

Update: I've been searching and found this is a dup of #1588 which has been moved to #1585

Very well written, thank you. Will move the conversation to this issue since its more action-oriented.


🚨  Move markdown tests.

Markdown tests are now integration tests rather than acceptance.
Refs: https://github.com/TryGhost/Ghost/issues/8201

Coverage decreased (-0.02%) to 75.012% when pulling 65b68fd46f062140430e867912c83b9a98c8de8a on disordinary:integration-tests into 9adb99058c0731fe1e50f5e57006058e86d1c47b on TryGhost:master.


Drop support for Internet Explorer and non Chromium-based Edge browsers.

Users of Internet Explorer or non Chromium-based Edge browsers may wish to upgrade their browser if possible, or continue to use WebUI 10.x (latest).

Completed.


Allow specific user-agent string for Liquibase

Cloud Spanner now has community supported Liquibase integration. It would be interesting to be able to track the usage of it by allowing a specific user-agent string to be used when working with Liquibase. This would mean expanding the list of allowed client lib tokens with one for Liquibase.

Added in https://github.com/googleapis/java-spanner/pull/925


Spock tests for libconsensus

Once a JVM binding is available for libconsensus, Spock tests could be a very good way to test both the JVM binding and libconsensus itself.
There is not much documentation on libconsensus at this point, but it is mentioned int the Bitcoin Core 0.10.0 Release Notes
See also:

https://github.com/bitcoin/bitcoin/pull/5235 [Merged]
https://github.com/bitcoin/bitcoin/issues/5311 [Docs and Examples]

Note: Now the non-Omni-related Spock test functionality has been moved to bitcoinj-addons from OmniJ I'm opening this issue here and closing https://github.com/OmniLayer/OmniJ/issues/55

Currently it's really just one function to verify, whether some script executes correctly:

https://github.com/bitcoin/bitcoin/blob/master/src/script/bitcoinconsensus.h

No JVM bindings yet, according to Google and doc/shared-libraries.md. How hard do you think they would be to create?

In February I asked on the bitcoinj mailing list in a post titled libconsensus JVM bindings and the link to the Lighthouse project was taken down.
I did some work with JNI 10+ years ago, but from the comments it sounds like JNA is a better way to go.
Do you have any interest in working on this @dexX7 ?

@msgilligan: I'm only a bit familiar with this, but I was curious and gave it a try! :)

https://github.com/dexX7/java-libbitcoinconsensus

The actual binding sits in LibBitcoinConsensus.java and I started with a more convenient wrapper in BitcoinConsensus.java.
Rudamentary JUnit tests are available.

Nice, @dexX7 ! Can't wait to try it out.

Would you accept a pull request with some Spock tests?

Sure! I also added you as collaborator, just in case. I think there are a few interesting paths from here: it could become a plain wrapper, or combined with bitcoinj (I'm mostly thinking about the script/transaction representing classes), and I don't have specific plans.

Well, it seems the logical place for the Spock Tests would be in your new project, so I'm going to close this issue and forward it to https://github.com/dexX7/java-libbitcoinconsensus/issues/1.

I did a checkout, build and test on Mac OS X Yosemite. The tests all passed!
One little nit is that the Gradle build didn't seem to notice if I move or change (touch) the libbitcoinconsensus.dylib file. I also discovered that it works if the dylib is in project root or in src/main/resources


fix: update next peer dep to 12.3.0

fixes #325

Should we make this ^12 so it expects a peer dep of next 12.x.x?


cuelang.org: cue eval -p documentation is incorrect

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

$ cue version
cue version 0.4.0 linux/amd64

Does this issue reproduce with the latest release?
yes
What did you do?
Given 2 files foo.cue and bar.cue
src/foo.cue
package foo

foo: "hello"

src/bar.cue
package bar

bar: "world"

cue eval -p foo ./src
found packages "foo" (foo.cue) and "bar" (bar.cue) in ".../src"

What did you expect to see?
I expect the error message to be printed when using cue eval ./src and to disappear when adding the -p option as in the documentation at https://cuelang.org/docs/concepts/packages/#files-belonging-to-a-package
What did you see instead?
Same behavior with or without -p

Hmm, this appears to be a bug in the docs. Because, to my knowledge, -p has never been used for this purpose. -p is used for specifying a package where one cannot be inferred from context, e.g. where OpenAPI is consumed as input, and the user wants to specify a package as part of the output.
In the case you link to from the docs, the package selection happens as part of the package pattern specified on the command line:
# One package in the current directory
exec cue eval

# Two packages in the x directory
! exec cue eval ./x

# Choose package x
exec cue eval ./x:x

# Choose package y
exec cue eval ./x:y

-- a.cue --
package a

import "mod.com/x"

x
-- cue.mod/module.cue --
module: "mod.com"
-- x/x.cue --
package x

x: 5
-- x/y.cue --
package y

y: 5

Gives:
# One package in the current directory (0.048s)
> exec cue eval
[stdout]
x: 5
# Two packages in the x directory (0.033s)
> ! exec cue eval ./x
[stderr]
found packages "x" (x.cue) and y (y.cue) in "$WORK/x"
[exit status 1]
# Choose package x (0.041s)
> exec cue eval ./x:x
[stdout]
x: 5
# Choose package y (0.039s)
> exec cue eval ./x:y
[stdout]
y: 5
PASS

So I will repurpose this issue for fixing up those docs.
But I've also raised https://github.com/cue-lang/cue/issues/1138, because I think the behaviour of imports and cmd/cue package resolution should be consistent with respect to ambiguous package selection.

Was able to figure out how to export a specific package thanks to this issue.
$ cue export
found packages "config" (config.cue) and hello (hello.cue) in "..."

$ cue export :config
{
    "foo": "bar"
}

$ cue export :hello
{
    "hello": "world"
}


Add README-DEVEL to searchengine

While working on my first PR I had troubles developing searchengine locally. This README might be helpful for future contributors.

LGTM, thanks! I simplified UWSGI logging a bit (it was configured incorrectly before, and will amend the docs accordingly)


noto-fonts-cjk-serif-static: init at Serif2.003

This PR packages the static version of noto cjk serif fonts. Currently nixpkgs only has the variable width version.
Would close #351244.
As for the test, I have checked the nix expression for the variable version of noto fonts already packaged. I saw that there's a test here, but I'm not sure how simply taking a screenshot can test if the font is working.
Could someone help me understand how this test work, so I can eventually add an equivalent for my package too?
Thanks :)
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Wouldn't it be preferable to alter the existing noto-fonts-cjk-serif package and add a new static optional argument defaulting to false? A top-level package noto-fonts-cjk-serif-static calling the package and setting static to true would ensure consistency between the static and the variable version of the font if both are installed on the same system.

@samueltardieu
That sounds like a good idea. I have thought about it, but it makes more sense now you say it haha.
Would that be considered an update to the noto-font-cjk-serif package then?

Maybe you could add the alias to pkgs/toplevel/all-packages.nix, something like:
noto-fonts-cjk-serif-static  = callPackage ../by-name/no/noto-fonts-cjk-serif/package.nix { static = true; };


[profiling] Use actions matrix for fuzz

What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
How to test the change?
Describe here in detail how the change can be validated.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 68.56%. Comparing base (de8d3b6) to head (8636311).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #462   +/-   ##
=======================================
  Coverage   68.56%   68.56%
=======================================
  Files         193      193
  Lines       25086    25086
=======================================
  Hits        17201    17201
  Misses       7885     7885

Components
Coverage Δ

crashtracker
19.34% <ø> (ø)

datadog-alloc
98.76% <ø> (ø)

data-pipeline
51.45% <ø> (ø)

data-pipeline-ffi
0.00% <ø> (ø)

ddcommon
85.24% <ø> (ø)

ddcommon-ffi
74.93% <ø> (ø)

ddtelemetry
56.09% <ø> (ø)

ipc
81.69% <ø> (ø)

profiling
77.98% <ø> (ø)

profiling-ffi
60.05% <ø> (ø)

serverless
0.00% <ø> (ø)

sidecar
37.89% <ø> (ø)

sidecar-ffi
0.00% <ø> (ø)

spawn-worker
54.98% <ø> (ø)

trace-mini-agent
69.12% <ø> (ø)

trace-normalization
97.79% <ø> (ø)

trace-obfuscation
95.74% <ø> (ø)

trace-protobuf
30.76% <ø> (ø)

trace-utils
90.00% <ø> (ø)


Instance size flexibility ratios CSV is not up to date with the Ev5 and Dv5 series

The Instance size flexibility ratios CSV file was not updated after Dv5 and Ev5 series went GA. We are still seeing Internal_Dv5 and Internal_Ev5 values in the InstanceSizeFlexibilityGroup and ArmSkuName columns. These should be updated to the Standard designations, according to what is made available for the other VM sizes.

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

ID: a20f52cb-4eb1-6faa-9c02-bcaf4e5974b1
Version Independent ID: d1a9bcf3-7d25-3373-5306-bba1ce351430
Content: Virtual machine size flexibility -Azure Reserved VM Instances - Azure Virtual Machines
Content Source: articles/virtual-machines/reserved-vm-instance-size-flexibility.md
Service: virtual-machines
Sub-service: billing
GitHub Login: @cynthn
Microsoft Alias: cynthn

@helderpinto
I have assigned the issue to the content author for further review.

#please-close Thank you for the feedback. Engineering is working on updating the .csv file to show the new VM sizes.