[Bug] Error when running Monocular 3D demo demo of FCOS3D

Prerequisite

[X] I have searched Issues and Discussions but cannot get the expected help.
[X] I have read the FAQ documentation but cannot get the expected help.
[X] The bug has not been fixed in the latest version (dev-1.x) or latest version (dev-1.0).

Task
I'm using the official example scripts/configs for the officially supported tasks/models/datasets.
Branch
main branch https://github.com/open-mmlab/mmdetection3d
Environment
sys.platform: linux
Python: 3.7.10 (default, Feb 26 2021, 18:47:35) [GCC 7.3.0]
CUDA available: True
numpy_random_seed:<PHONE_NUMBER>
GPU 0: Tesla V100-PCIE-16GB
CUDA_HOME: /usr/local/cuda
NVCC: Cuda compilation tools, release 11.1, V11.1.105
GCC: gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
PyTorch: 1.9.0
PyTorch compiling details: PyTorch built with:

GCC 7.3
C++ Version: 201402
Intel(R) oneAPI Math Kernel Library Version 2021.2-Product Build 20210312 for Intel(R) 64 architecture applications
Intel(R) MKL-DNN v2.1.2 (Git Hash 98be7e8afa711dc9b66c8ff3504129cb82013cdb)
OpenMP 201511 (a.k.a. OpenMP 4.5)
NNPACK is enabled
CPU capability usage: AVX2
CUDA Runtime 11.1
NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_80,code=sm_80;-gencode;arch=compute_86,code=sm_86;-gencode;arch=compute_37,code=compute_37
CuDNN 8.0.5
Magma 2.5.2
Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CUDA_VERSION=11.1, CUDNN_VERSION=8.0.5, CXX_COMPILER=/opt/rh/devtoolset-7/root/usr/bin/c++, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -fopenmp -DNDEBUG -DUSE_KINETO -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DSYMBOLICATE_MOBILE_DEBUG_HANDLE -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-variable -Wno-unused-function -Wno-unused-result -Wno-unused-local-typedefs -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_VERSION=1.9.0, USE_CUDA=ON, USE_CUDNN=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON,

TorchVision: 0.10.0
OpenCV: 4.9.0
MMEngine: 0.10.2
MMDetection: 3.3.0
MMDetection3D: 1.4.0+0ef13b8
spconv2.0: False
Reproduces the problem - code sample
File mono_det_demo.py
def main():
# TODO: Support inference of point cloud numpy file.
init_args, call_args = parse_args()
inferencer = MonoDet3DInferencer(**init_args)
inferencer(**call_args)

Reproduces the problem - command or script
python demo/mono_det_demo.py demo/data/nuscenes/n015-2018-07-24-11-22-45+0800__CAM_BACK__1532402927637525.jpg demo/data/nuscenes/n015-2018-07-24-11-22-45+0800.pkl  configs/fcos3d/fcos3d_r101-caffe-dcn_fpn_head-gn_8xb2-1x_nus-mono3d_finetune.py data/checkpoints/fcos3d_r101_caffe_fpn_gn-head_dcn_2x8_1x_nus-mono3d_finetune_20210717_095645-8d806dc2.pth  --cam-type CAM_BACK --out-dir FCOS3D/uno
Reproduces the problem - error message
Loads checkpoint by local backend from path: data/checkpoints/fcos3d_r101_caffe_fpn_gn-head_dcn_2x8_1x_nus-mono3d_finetune_20210717_095645-8d806dc2.pth
01/08 15:46:46 - mmengine - WARNING - Failed to search registry with scope "mmdet3d" in the "function" registry tree. As a workaround, the current "function" registry in "mmengine" is used to build instance. This may cause unexpected failure when running the built modules. Please check whether "mmdet3d" is a correct scope, or whether the registry is initialized.
/opt/conda/lib/python3.7/site-packages/mmengine/visualization/visualizer.py:196: UserWarning: Failed to add <class 'mmengine.visualization.vis_backend.LocalVisBackend'>, please provide the save_dir argument.
warnings.warn(f'Failed to add {vis_backend.class}, '
Traceback (most recent call last):
File "demo/mono_det_demo.py", line 98, in
main()
File "demo/mono_det_demo.py", line 88, in main
inferencer(**call_args)
File "/mmdetection3d/mmdet3d/apis/inferencers/base_3d_inferencer.py", line 204, in call
ori_inputs = self._inputs_to_list(inputs, cam_type=cam_type)
File "/mmdetection3d/mmdet3d/apis/inferencers/mono_det3d_inferencer.py", line 110, in _inputs_to_list
img_path = data_info['images'][cam_type]['img_path']
Additional information
I installed mmdetection3d via Docker. I only want to run the inference with a pretrained pth file. When running the official demo for inference with the PGD model and the SMOKE model on KITTI, it's running. For FCOS3D on NuScenes, this error occurs. I tried to hardcode the cam_type CAM_BACK into the files "base_3d_inferencer.py" and "mono_det3d_inferencer.py", because it seems that the CAM_BACK in the command I'm running is simply ignored.
When doing this, the following error occurs:
"File "/mmdetection3d/mmdet3d/apis/inferencers/mono_det3d_inferencer.py", line 125, in _inputs_to_list
lidar2img = cam2img @ lidar2cam
ValueError: matmul: Input operand 1 has a mismatch in its core dimension 0, with gufunc signature (n?,k),(k,m?)->(n?,m?) (size 4 is different from 3)"

Hello, I encountered the same problem. Have you solved this problem?
RUNNING FCOS3D inference demo:
(openmmlab) zz@zz-Ubuntu:~/project/mmdetection3d-dev-1.x$ python demo/mono_det_demo.py demo/data/nuscenes/n015-2018-07-24-11-22-45+0800__CAM_BACK__1532402927637525.jpg demo/data/nuscenes/n015-2018-07-24-11-22-45+0800.pkl  configs/fcos3d/fcos3d_r101-caffe-dcn_fpn_head-gn_8xb2-1x_nus-mono3d_finetune.py test_checkpoints/fcos3d_r101_caffe_fpn_gn-head_dcn_2x8_1x_nus-mono3d_finetune_20210717_095645-8d806dc2.pth --show --cam-type CAM_BACK
Error is same:
**Traceback (most recent call last):
File "demo/mono_det_demo.py", line 98, in
main()
File "demo/mono_det_demo.py", line 88, in main
inferencer(call_args)
File "/home/zz/project/mmdetection3d-dev-1.x/mmdet3d/apis/inferencers/base_3d_inferencer.py", line 204, in call
ori_inputs = self._inputs_to_list(inputs, cam_type=cam_type)
File "/home/zz/project/mmdetection3d-dev-1.x/mmdet3d/apis/inferencers/mono_det3d_inferencer.py", line 110, in _inputs_to_list
img_path = data_info['images'][cam_type]['img_path']
KeyError: 'CAM2'
My thoughts are the same as yours, the program ignores the input parameters ‘--cam-type CAM_BACK’. But I dont know how to solve it.

No, unfortunately not. I'm still waiting for help :-)

I think I have some clues. This error comes from an incorrect key in the dic 'kwargs'.
mmdet3d/apis/inferencers/base_3d_inferencer.py     line170
 if cam_type != '':
            kwargs['cam_type'] = cam_type
            # kwargs['cam_type_dir'] = cam_type

Then the script will correctly read the incoming parameter 'CAM_BACK'
But another error occurs.
Traceback (most recent call last):
  File "/home/zz/project/mmdetection3d-dev-1.x/demo/mono_det_demo.py", line 98, in <module>
    main()
  File "/home/zz/project/mmdetection3d-dev-1.x/demo/mono_det_demo.py", line 88, in main
    inferencer(**call_args)
  File "/home/zz/project/mmdetection3d-dev-1.x/mmdet3d/apis/inferencers/base_3d_inferencer.py", line 209, in __call__
    ori_inputs = self._inputs_to_list(inputs, cam_type=cam_type)
  File "/home/zz/project/mmdetection3d-dev-1.x/mmdet3d/apis/inferencers/mono_det3d_inferencer.py", line 127, in _inputs_to_list
    lidar2img = cam2img @ lidar2cam
ValueError: matmul: Input operand 1 has a mismatch in its core dimension 0, with gufunc signature (n?,k),(k,m?)->(n?,m?) (size 4 is different from 3)

The reason is that the number of columns of matrix cam2img is not equal to the number of rows of lidar2cam, so they cannot be multiplied.
cam2img is 3x3 and lidar2cam is 4x4, I don't know what this line of code means.
For running program, I changed 'lidar2cam' for no reason.
  else:
      # lidar2img = cam2img @ lidar2cam
      lidar2img = cam2img @ lidar2cam[0:3]

Finally it ran, and the results were unexpectedly correct. . .Of course I don't know why, I'm just getting started.

I think this program does not require lidar2cam or lidar2img because it is mono_det. So it's still normal and correct if delete all the code about lidar:
                # lidar2cam = np.asarray(
                #     data_info['images'][cam_type]['lidar2cam'],
                #     dtype=np.float32)
                # if 'lidar2img' in data_info['images'][cam_type]:
                #     lidar2img = np.asarray(
                #         data_info['images'][cam_type]['lidar2img'],
                #         dtype=np.float32)
                # else:
                #     lidar2img = cam2img @ lidar2cam
                input['cam2img'] = cam2img
                # input['lidar2cam'] = lidar2cam
                # input['lidar2img'] = lidar2img

I think this program does not require lidar2cam or lidar2img because it is mono_det. So it's still normal and correct if delete all the code about lidar:
mmdet3d/apis/inferencers/mono_det3d_inferencer.py   line 118
                # lidar2cam = np.asarray(
                #     data_info['images'][cam_type]['lidar2cam'],
                #     dtype=np.float32)
                # if 'lidar2img' in data_info['images'][cam_type]:
                #     lidar2img = np.asarray(
                #         data_info['images'][cam_type]['lidar2img'],
                #         dtype=np.float32)
                # else:
                #     lidar2img = cam2img @ lidar2cam

                input['cam2img'] = cam2img
                # input['lidar2cam'] = lidar2cam
                # input['lidar2img'] = lidar2img

Thank you! I want to make the inference on other input images, have you already done that? The pictures provided in the demo are listed in the .pkl files. If the name of the input pictures differs from the one in the demo, the inference won't run.

I want to add a few notes to this discussion. As @Aloaml  mentioned mono detection should not depend on lidar2cam and lidar2img matrices. If you are not using the public datasets support so far in MMDet3D you might encounter errors as the code will try to load those matrices by default.
So far I have not found a proper way to bypass this without changing the source code, and the same thing holds for inference codes which are called in the demo files.
I think it calls for a bit modifications for the next release, but for now your best shot is changing the source code to ignore those matrices.
while we are at it, maybe having a demo that runs on img arrays instead of files would be a great asset as well.

to by pass you should update the mmdet3D/apis/inferencers/mono_det3d_inferencer.py
with the following code. It will allow you to run demo on datasets that don't have lidar.
`def _inputs_to_list(self,
inputs: Union[dict, list],
cam_type='CAM2',
**kwargs) -> list:
"""Preprocess the inputs to a list.
Preprocess inputs to a list according to its type:
    - list or tuple: return inputs
    - dict: the value with key 'img' is
        - Directory path: return all files in the directory
        - other cases: return a list containing the string. The string
          could be a path to file, a url or other types of string according
          to the task.

    Args:
        inputs (Union[dict, list]): Inputs for the inferencer.

    Returns:
        list: List of input for the :meth:`preprocess`.
    """
    if isinstance(inputs, dict):
        assert 'infos' in inputs
        infos = inputs.pop('infos')

        if isinstance(inputs['img'], str):
            img = inputs['img']
            backend = get_file_backend(img)
            if hasattr(backend, 'isdir') and isdir(img):
                # Backends like HttpsBackend do not implement `isdir`, so
                # only those backends that implement `isdir` could accept
                # the inputs as a directory
                filename_list = list_dir_or_file(img, list_dir=False)
                inputs = [{
                    'img': join_path(img, filename)
                } for filename in filename_list]

        if not isinstance(inputs, (list, tuple)):
            inputs = [inputs]

        # get cam2img, lidar2cam and lidar2img from infos
        info_list = mmengine.load(infos)['data_list']
        assert len(info_list) == len(inputs)
        for index, input in enumerate(inputs):
            data_info = info_list[index]
            img_path = data_info['images'][cam_type]['img_path']
            if isinstance(input['img'], str) and \
                    osp.basename(img_path) != osp.basename(input['img']):
                raise ValueError(
                    f'the info file of {img_path} is not provided.')
            cam2img = np.asarray(
                data_info['images'][cam_type]['cam2img'], dtype=np.float32)
            if 'lidar2cam' in data_info['images'][cam_type]:
                lidar2cam = np.asarray(
                    data_info['images'][cam_type]['lidar2cam'],
                    dtype=np.float32)
                if 'lidar2img' in data_info['images'][cam_type]:
                    lidar2img = np.asarray(
                        data_info['images'][cam_type]['lidar2img'],
                        dtype=np.float32)
                else:
                    lidar2img = cam2img @ lidar2cam
                input['lidar2cam'] = lidar2cam
                input['lidar2img'] = lidar2img
            input['cam2img'] = cam2img
    elif isinstance(inputs, (list, tuple)):
        # get cam2img, lidar2cam and lidar2img from infos
        for input in inputs:
            assert 'infos' in input
            infos = input.pop('infos')
            info_list = mmengine.load(infos)['data_list']
            assert len(info_list) == 1, 'Only support single sample info' \
                'in `.pkl`, when inputs is a list.'
            data_info = info_list[0]
            img_path = data_info['images'][cam_type]['img_path']
            if isinstance(input['img'], str) and \
                    osp.basename(img_path) != osp.basename(input['img']):
                raise ValueError(
                    f'the info file of {img_path} is not provided.')
            cam2img = np.asarray(
                data_info['images'][cam_type]['cam2img'], dtype=np.float32)
            if 'lidar2cam' in data_info['images'][cam_type]:
                lidar2cam = np.asarray(
                    data_info['images'][cam_type]['lidar2cam'],
                    dtype=np.float32)
                if 'lidar2img' in data_info['images'][cam_type]:
                    lidar2img = np.asarray(
                        data_info['images'][cam_type]['lidar2img'],
                        dtype=np.float32)
                else:
                    lidar2img = cam2img @ lidar2cam

                input['lidar2cam'] = lidar2cam
                input['lidar2img'] = lidar2img
            input['cam2img'] = cam2img

    return list(inputs)

`

Same problem here! The workaround described by @Aloaml worked, but would be cool to get this fixed pemanently.


double-beep: Watch evensi.com

double-beep requests the watch of the watch_keyword evensi\.com. See the MS search here and the Stack Exchange search here.

evensi\.com has been seen in 2 true positives, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


changed one line for resources#destroy

Description of changes
Just changed the one line to be more descriptive
Issue Resolved

Fixes #

@kenny - Can you please add more description to what this PR is solving/addressing?  Also, in the "issue resolved" section, please link up the associated issue #
You can see an example of that linkage in this PR's description: https://github.com/OperationCode/operationcode_backend/pull/272


CIF-912 - introduce new parent pom.xml to prepare for reactor release

Description
Introduce new parent pom.xml to prepare for project release.
Motivation and Context
To be able to release this project as a joined reactor release sone which then can be used as Maven dependency by AEM projects we need to restructure and introduce a common parent pom.xml.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] I have signed the Adobe Open Source CLA.
[ ] My change requires a change to the documentation.
[X] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes and the overall coverage did not decrease.
[ ] All unit tests pass on CircleCi.
[ ] I ran all tests locally and they pass.

Codecov Report

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

@@            Coverage Diff            @@
##             master      #50   +/-   ##
=========================================
  Coverage     56.88%   56.88%
  Complexity      135      135
=========================================
  Files            25       25
  Lines           842      842
  Branches         49       49
=========================================
  Hits            479      479
  Misses          340      340
  Partials         23       23

Flag
Coverage Δ
Complexity Δ

#karma
95.62% <ø> (ø)
0 <ø> (ø)
:arrow_down:

#unittests
46.13% <ø> (ø)
135 <ø> (ø)
:arrow_down:

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 3f9256f...856b55f. Read the comment docs.


DCR file?

[This should include some clues about what to do with the DCR file once it is generated]

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

ID: 483faea0-2cc4-58e4-311b-219e537680eb
Version Independent ID: 6f4c9892-a0fd-ccf9-81b0-d920f690aea9
Content: Migration guidance from Change Tracking and inventory using Log Analytics to Azure Monitoring Agent
Content Source: articles/automation/change-tracking/guidance-migration-log-analytics-monitoring-agent.md
Service: automation
Sub-service: change-inventory-management
GitHub Login: @SnehaSudhirG
Microsoft Alias: sudhirsneha

@kjmartinHEAT
It would be great if you could add more about query to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!!

The last step in the directions for using Powershell is "Generate and
associate a new DCR to transfer the settings to the Change Tracking and
Inventory using AMA."  .
There is no further information about how to "associate a new DCR".
Kevin Martin
On Tue, Apr 30, 2024 at 11:32 AM PavanPesala-MSFT @.***>
wrote:

@kjmartinHEAT https://github.com/kjmartinHEAT
It would be great if you could add more about query to the documentation
you are following for these steps? This would help us redirect the issue to
the appropriate team. Thanks!!
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/122107#issuecomment-2085893718,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFZ63IXQOCEOLEAP5DUDDT3Y77BTDAVCNFSM6AAAAABHALXHROVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBVHA4TGNZRHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Kevin Martin
OIT - MPT

@kjmartinHEAT
Thanks for your feedback! We will investigate and update as appropriate.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary based on resourcing, so we've created an internal work item to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary.
#please-close


Add Richard Linz to members

Config changes proposed in this pull request

For new organization members
I acknowledge that

[x] I have read and understood the Open Source Guidelines
[x] I agree to act accordingly (with due dilligence) to our guidelines
[x] I have no open questions regarding the topics covered (please delete the "open questions" section)

Open questions

@RiLi2001 lgtm


Anivia W (the wall)

When used, only the mist effect was shown, the figure of the wall was gone.

#1


Example for tink/go/hybrid/subtle package

Is there any example or reference code which shows how to use "github.com/google/tink/go/hybrid/subtle" package properly?
P.S.: I know "github.com/google/tink/go/hybrid" package but I want to avoid protocol buffers.

1/ The unit tests are good examples: https://github.com/google/tink/blob/master/go/hybrid/ecies_aead_hkdf_hybrid_encrypt_test.go.
2/ The github.com/google/tink/go/hybrid delegates crypto operations to the github.com/google/tink/go/hybrid/subtle package. It provides a good example of how to use the latter package properly.
Hope this helps. Ping back if you have any question.


[React CRUD] Bogus error message on duplicated dataset creation

A clear and concise description of what the bug is.
Expected results
Error msg telling that dataset already exists
Actual results
Error saving dataset Object Object
Screenshots
If applicable, add screenshots to help explain your problem.

How to reproduce the bug

dataset list view in react CRUD
create dataset
try to create same dataset
See error

Environment
(please complete the following information):

superset version: superset version
python version: python --version
node.js version: node -v
npm version: npm -v

latest master
cc @nytai

@lilykuang @nytai

should have been fixed with https://github.com/apache/incubator-superset/commit/78cad9a4a86179c2305036d08740c0f01c4b0695#diff-910af846ee243a9628839783c7553af5R100


Switching to existing branch may fail

In GitLab by @arisilon on Sep 23, 2020, 10:02
The code splits on '/' but that is also the branch name separator so it may try to create a branch that already exists. Need to just remove refs/heads and refs/remotes/origin to get the list of existing branches. Would also make sense to fetch --all first.

In GitLab by @arisilon on Dec 30, 2020, 21:57
mentioned in commit a62b4b8bdc0336b34717273abd88aab9d01391a9


inference for 2s 720p took 7 minutes on A100

Hi there. thank for your great work!
i just try your model, the result is awesome, but I encounter an issue.
I run gradio demo on A100 - 80G GPU, install everything as in readme file. but it took about 7 minutes for image-to-video within 2 seconds video. I didn't change anything.
When I disable cpu off load and save_memory=False, it took about 22 minutes.
I have tried your huggingface demo, it only took about 2 minutes for 4s video.
I'm running on cuda 12.2 in conda env.
My question is: How can I can config to reduce reference time?

Could you share the GPU memory usage to help us locate the problem? A quick fix is to use video_generation_demo.ipynb instead of Gradio demo, as we have received similar issues that are resolved in this way.

I spend 15min for 10 seconds video, I wonder if cpu capacity will restrict the inference speed?


"来園者" needs to be replaced with "ゲスト"

Strongly recommend to replace "来園者" with "ゲスト" , I can see "来園者" is used as a localized Japanese word of "guest" in English, however, it should be "ゲスト" which stands for guest user of Microsoft Teams.
https://docs.microsoft.com/ja-jp/microsoftteams/teams-adoption-governance-quick-start

ドキュメントの詳細
⚠ このセクションを編集しないでください。 docs.microsoft.com で必須です ➟ GitHub の問題のリンク。

ID: 630d8bc9-ba1d-8d41-f3b1-00a1ea03a43d
Version Independent ID: 3d15b49e-5fe1-dba7-9ba8-9c4eac728e14
Content: Microsoft Teams のガバナンスのクイック スタート
Content Source: Teams/teams-adoption-governance-quick-start.md
Service: msteams
GitHub Login: @karuanag
Microsoft Alias: karuanag

@hyoshioka0128 - Thank you for submitting this feedback.
I will get this over to for investigation. Thank you for reporting and making the docs better. Much appreciated.

@officedocsbot assign @scanum

@OfficeGlobal

@OfficeGlobal following up on this issue? Could you please update status? Thanks

@hyoshioka0128 , @scanum string was updated and published live. The issue is fixed

@hyoshioka0128 Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

@officedocsbot close


SF suggestions doesn't work with very long segments

There is something limiting this - for very long verses something pops up and says "this verse is too long for suggestions."  Can we fix/resolve this?

When a source segment is too long, decoding can take a long time. We put in a hard limit of 200 words in order to be able to generate suggestions for a segment. If we increased the CPU on the server, then we might be able to increase the limit a bit.

The limit of 200 tokens is actually a aspect of the of NLLB 200 itself.  I don’t think we can extend the token limit meaningfully, but rather we should split up the long segments.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Damien Daspit @.>
Sent: Thursday, August 10, 2023 4:59:37 PM
To: sillsdev/serval @.>
Cc: John Lambert @.>; Assign @.>
Subject: Re: [sillsdev/serval] SF suggestions doesn't work with very long segments (Issue #76)
When a source segment is too long, decoding can take a long time. We put in a hard limit of 200 words in order to be able to generate suggestions for a segment. If we increased the CPU on the server, then we might be able to increase the limit a bit.
—
Reply to this email directly, view it on GitHubhttps://github.com/sillsdev/serval/issues/76#issuecomment-1673911523, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADIY5NAQU24T4CG5UTH6D73XUVDTTANCNFSM6AAAAAA3I3EYMA.
You are receiving this because you were assigned.Message ID: @.***>

Actually, in this case, I was referring to the SMT model, but yes, you are correct NLLB also has a limit of 200 tokens.

Sorry, I was misunderstanding the context.

If the CPU is the main thing limiting the word graph, I would be more inclined to pay for more CPU. A few hundred dollars a year is worth the increased performance. What do you think?

We can also use grafana to measure the word graph performance in production.

We would need to do testing to see how much we could increase the limit. At this point, SMT suggestions are not being used very much, so it isn't a priority to increase this limit.

This issue is related to https://github.com/sillsdev/machine.js/issues/19.  After this bug is fixed, it may not be a big issue anymore.


DBeaver: System setting EXTRA SPACES has no effect when viewing data grid in TEXT and ONE RECORD a time

DBeaver - Version 6.3.4 CE EA
DBeaver driver: MS SQL Server  / Microsoft Driver (the new Microsoft Driver)
Operating System: Windows 7 / Windows 8.1 / Windows 10
Database Server: Microsoft SQL Express 2014, 2016, 2017
The systemsetting for Extra spaces has no effect when viewing a table grid data in text and records as one record a time (ie. vertical grid).
System setting in question is:
  DBeaver -> Editors -> Presentation -> Plain text -> Extra spaces

thanks for the bug report

Fixed

verified


HADOOP-18590. Publish SBOM artifacts

Description of PR
This is a second try of #5281 with new cyclonedx plugin 2.7.6 and tested with Maven 3.9.1.
This PR aims to publish SBOM artifacts.

https://cwiki.apache.org/confluence/display/COMDEV/SBOM

Here is an article to give some context.

https://www.activestate.com/blog/why-the-us-government-is-mandating-software-bill-of-materials-sbom/

Software Bill of Materials (SBOM) are additional artifacts containing the aggregate of all direct and transitive dependencies of a project. The US Government (based on NIST recommendations) currently accepts only the three most popular SBOM standards as valid, namely: CycloneDX, Software Identification (SWID) tag, Software Package Data Exchange® (SPDX).
This PR uses CycloneDX maven plugin v2.7.6, a lightweight software bill of materials (SBOM) standard designed for use in application security contexts and supply chain component analysis. cyclonedx-maven-plugin 2.7.6 is verified in Apache ORC/Parquet/Avro/Arrow/Spark community as of today.

https://github.com/apache/orc/pull/1463
https://github.com/apache/parquet-mr/pull/1057
https://github.com/apache/avro/pull/2174
https://github.com/apache/arrow/pull/35092
https://github.com/apache/spark/pull/40726

How was this patch tested?
Manually. For example, hadoop-auth-3.4.0-SNAPSHOT.jar will have hadoop-auth-3.4.0-SNAPSHOT-cyclonedx.xml and hadoop-auth-3.4.0-SNAPSHOT-cyclonedx.json SBOM files additionally.
$ mvn --version
Apache Maven 3.9.1 (2e178502fcdbffc201671fb2537d0cb4b4cc58f8)
Maven home: /opt/homebrew/Cellar/maven/3.9.1/libexec
Java version: 11.0.18, vendor: Apple Inc., runtime: /Library/Java/JavaVirtualMachines/applejdk-<IP_ADDRESS>.1.jdk/Contents/Home
Default locale: en_US, platform encoding: UTF-8
OS name: "mac os x", version: "13.3", arch: "aarch64", family: "mac"

$ ls -l ~/.m2/repository/org/apache/hadoop/hadoop-auth/3.4.0-SNAPSHOT
total 1008
-rw-r--r--  1 dongjoon  staff     373 Apr 13 13:43 _remote.repositories
-rw-r--r--  1 dongjoon  staff  100849 Apr 13 13:42 hadoop-auth-3.4.0-SNAPSHOT-cyclonedx.json
-rw-r--r--  1 dongjoon  staff   85972 Apr 13 13:42 hadoop-auth-3.4.0-SNAPSHOT-cyclonedx.xml
-rw-r--r--  1 dongjoon  staff   84455 Apr 13 13:42 hadoop-auth-3.4.0-SNAPSHOT-sources.jar
-rw-r--r--  1 dongjoon  staff  113598 Apr 13 13:42 hadoop-auth-3.4.0-SNAPSHOT-tests.jar
-rw-r--r--  1 dongjoon  staff  106123 Apr 13 13:42 hadoop-auth-3.4.0-SNAPSHOT.jar
-rw-r--r--  1 dongjoon  staff    8246 Apr 13 13:38 hadoop-auth-3.4.0-SNAPSHOT.pom
-rw-r--r--  1 dongjoon  staff    1537 Apr 13 13:43 maven-metadata-local.xml

For code changes:

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

cc @steveloughran

Do you have a regular test pipeline with -Pdist, @ayushtkn ?

Anyway, I moved it to under dist profile according to your advice, @ayushtkn .

Thanx, We don't have a regular test with -Pdist, I thought this was required only during release time, we moved this to -Pdist in hive as well, recently in favour of saving time during normal dev work.
If you want this to be executed for every PR, we can explore adding -Pdist as well during execution, it will increase our cost for every build, may be have github actions for just Pdist or something of that sort

Ya, I agree with your decision, @ayushtkn .

BTW, do we have a regular SNAPSHOT publishing job? Apache ORC and Apache Spark has a daily SNAPSHOT publishing jobs and SBOM is tested there independently.

Also, cc @sunchao

We have one here: https://ci-hadoop.apache.org/view/Hadoop/job/Hadoop-trunk-Commit/
I can add -Pdist here if you say, but we don't check the results of this build :)
We have a post commit github action as well to deploy trunk website, can add -Pdist over there if you want
https://github.com/apache/hadoop/blob/trunk/.github/workflows/website.yml#L47

Got it. Thank you for the pointers.

Thank you, @ayushtkn and @cnauroth !


fix(e2e): Add installation with helm,kustomize,olm securityContext re…

Fix #4786
Description
Add e2e tests installation with helm,kustomize,olm securityContext restricted validation

Release Note
fix(e2e): Add installation with helm,kustomize,olm securityContext restricted

Looks good. However, in general, I'd avoid to introduce reflection.

Noted. Do you want me to change the current PR ?

@gansheer can you please confirm if you're finally going to address the reflection or can I merge this the way it is? Thanks!

@gansheer can you please confirm if you're finally going to address the reflection or can I merge this the way it is? Thanks!

I knew I forgot something :sweat_smile: . Yes I am taking care of it and will fix the CLI test as well.


Logout button is a need

There is no logout button in the panel; Having one can make a better user experience.

Hi
We are using HackaPanel for contests which the login info will never be used afterwards and there is no need of logging out but if you need to switch the logged PC the way is to just closing the tab on the other one and try logging in with the new PC. HackaPanel understands that the other one is logged out.
By the way it would be a good help if you can do that with a pull request and we'll be happy.

@shahabbasi  If you have a login form you SHOULD consider a way for people to logout or at least set a message for them to make it clear how they can logout of HackaPanel. It doesn't matter if you use it in a certain way because it's a general idea of how user experience should works!
By the way I'm trying my best to contribute as much as I can afford.

@EYazdpour logout button has been made here
Thumbs up for giving useful feedbacks!

And this is how the button looks like right now:)


Discussion: Inline constants in the generated code

Discussion
What aspect of the language would you like to see improved?
I'm curious whether we can propagate known constants in Crystal to the emitted LLVM so they can be stored as data.
What are the reasons?
It should ideally result in more efficient code and less strain on the garbage collector.
Include practical examples to illustrate your points.
The specific example that made me think of this is that I want the UI of my app to provide hints for keyboard shortcuts. For example:
MODIFIER_KEY_STR = {% if flag?(:darwin) %} "Cmd" {% else %} "Ctrl" {% end %}
{"Fullscreen", "{MODIFIER_KEY_STR}+F"}

Today, this generates a call to String.interpolation, which allocates a new string on every call. This also happens when concatenating the strings with +. Since MODIFIER_KEY_STR is a constant, I'd hope that we could concatenate the strings at compile-time and store the resulting constant in the binary.
Optionally add one (or more) proposals to improve the current situation.
I have no idea what this would look like in the compiler.

I had a branch for that but I didn't think it was that important. I lost it but I can redo it.

To be fair, I haven't compared measured the impact of performance. I don't actually know whether it's significant. My app barely uses the GC today though, so this topic came to mind when writing the code. It's a few hundred allocations per second that probably don't need to exist

String.interpolation is not guaranteed to be a constant expression, since it is only a regular method. It does not even exist in an empty prelude. Instead you could do the following to force constant evaluation:
{% begin %}
  MODIFIER_KEY_STR = ...
{% end %}
{{ "#{MODIFIER_KEY_STR.id}+F" }}

I think the proposal here is for the compiler to notice that the interpolations interpolates a constant, so it could very well interpolate things at compile time.
I actually had a branch with this working, I just didn't sent it to avoid overwhelming the team with such PRs at that time.

Doing so involves running an arbitrary amount of non-macro code during compilation, because there is no bound on the #to_s method's complexity. At this time I wouldn't worry about such an optimization or whitelist certain interpolations when the {{ "#{}" }} workaround suffices.

@HertzDevil For a start, this could be limited to constants of type String (and maybe some primitives?).
But since constants are not immutable, this wouldn't work for arbitrary types anyway, because the constant could have a different value each time it is used in string expansion (simple example: it's an array and elements get added and removed at runtime).

I think there's a misunderstanding here. This is about constants whose value is a string literal. Just that.

MODIFIER_KEY_STR's initializer is a MacroIf that only gets expanded in the main phase, rather than a StringLiteral. The two are not quite the same.

The idea would be to try to resolve that before expanding the interpolation. I'll send a PR later during this week.

Here's the PR: https://github.com/crystal-lang/crystal/pull/12524/files

#12671 was eventually reverted in https://github.com/crystal-lang/crystal/pull/14155 due to https://github.com/crystal-lang/crystal/issues/14150.


[Feat]: Add parameter to struct for pushing block request for stopping scaling

This should allow a caller to set the channel and then push to the channel to stop scaling and otherwise allow scaling. This will likely require a routine middle layer for translating a push to a stop to keep the scaler working normally.

I'm not sure how I feel about this one. I'm going to test out the new scaling duration and max routine stuff before moving onto this.
The idea here was to allow an external user to push events over a channel to stop the scaling of go routines. This would allow for a user monitoring memory or cpu usage to halt scaling when exceeding some limit. It adds quite a bit of complexity as well as an additional backend goroutine and a select case with potential performance impact.
If the new additions for tuning are not sufficient then this may be necessary but for now I'm going to hold off on this.


Remove unused remote config for apm tracing

What does this PR do?
Remove code for listening to apm tracing RC Product.
Motivation
This was originally the intended approach to handle RC of single step instrumentation, but has been replaced with fleet automation policies. This code was only used in a proof of concept and there should be no users of this file.
Describe how to test/QA your changes
existing tests are sufficient here.
Possible Drawbacks / Trade-offs
Additional Notes

/merge


application crash

I faced this error after open the app, unity indeed does not export libmain.so and other .so files , I made the apk and decompiled it and got libmain.so ,libil2cpp.so and other files i put them into unity export but invain
E/flutter (13362): [ERROR:flutter/shell/platform/android/platform_view_android_jni.cc(39)] java.lang.UnsatisfiedLinkError: dalvik.system.PathClassLoader[DexPathList[[zip file "/data/app/com.rexraphael.flutterunitywidgetexample-TxKj4BthLTvMKnaWmKmMOw==/base.apk"],nativeLibraryDirectories=[/data/app/com.rexraphael.flutterunitywidgetexample-TxKj4BthLTvMKnaWmKmMOw==/lib/x86, /data/app/com.rexraphael.flutterunitywidgetexample-TxKj4BthLTvMKnaWmKmMOw==/base.apk!/lib/x86, /system/lib]]] couldn't find "libmain.so"

I think it maybe because unity does not support x86 architecture anymore,is that right?
unity version 2019.3.0f3

I ran into the same issue. Have yet to find a solution, so I changed directions to utilize the new Unity as a Library support.
Please let us know if you find a fix. Would be much more convenient for flutter development to use this widget.

Here is my exact error:

E/flutter ( 9958): [ERROR:flutter/shell/platform/android/platform_view_android_jni.cc(39)] java.lang.UnsatisfiedLinkError: dalvik.system.PathClassLoader[DexPathList[[zip file "/data/app/com.rexraphael.flutterunitywidgetexample-BcfwL8VOVFiWptQasnB55Q==/base.apk"],nativeLibraryDirectories=[/data/app/com.rexraphael.flutterunitywidgetexample-BcfwL8VOVFiWptQasnB55Q==/lib/x86, /data/app/com.rexraphael.flutterunitywidgetexample-BcfwL8VOVFiWptQasnB55Q==/base.apk!/lib/x86, /system/lib, /system/product/lib]]] couldn't find "libmain.so"

My build.gradle:
def localProperties = new Properties()
def localPropertiesFile = rootProject.file('local.properties')
if (localPropertiesFile.exists()) {
    localPropertiesFile.withReader('UTF-8') { reader ->
        localProperties.load(reader)
    }
}

def flutterRoot = localProperties.getProperty('flutter.sdk')
if (flutterRoot == null) {
    throw new GradleException("Flutter SDK not found. Define location with flutter.sdk in the local.properties file.")
}

def flutterVersionCode = localProperties.getProperty('flutter.versionCode')
if (flutterVersionCode == null) {
    flutterVersionCode = '1'
}

def flutterVersionName = localProperties.getProperty('flutter.versionName')
if (flutterVersionName == null) {
    flutterVersionName = '1.0'
}

apply plugin: 'com.android.application'
apply from: "$flutterRoot/packages/flutter_tools/gradle/flutter.gradle"

android {
compileSdkVersion 28

lintOptions {
    disable 'InvalidPackage'
}

defaultConfig {
    // TODO: Specify your own unique Application ID (https://developer.android.com/studio/build/application-id.html).
    applicationId "com.rexraphael.flutterunitywidgetexample"
    minSdkVersion 19
    targetSdkVersion 28
    versionCode flutterVersionCode.toInteger()
    versionName flutterVersionName
    testInstrumentationRunner "android.support.test.runner.AndroidJUnitRunner"
}

buildTypes {
    release {
        // TODO: Add your own signing config for the release build.
        // Signing with the debug keys for now, so `flutter run --release` works.
        signingConfig signingConfigs.debug
    }
}
}

flutter {
    source '../..'
}

dependencies {
    testImplementation 'junit:junit:4.12'
    androidTestImplementation 'com.android.support.test:runner:1.0.2'
    androidTestImplementation 'com.android.support.test.espresso:espresso-core:3.0.2'
}

And settings.gradle:
include ':app', ':unity-classes'

def flutterProjectRoot = rootProject.projectDir.parentFile.toPath()

def plugins = new Properties()
def pluginsFile = new File(flutterProjectRoot.toFile(), '.flutter-plugins')
if (pluginsFile.exists()) {
    pluginsFile.withReader('UTF-8') { reader -> plugins.load(reader) }
}

plugins.each { name, path ->
    def pluginDirectory = flutterProjectRoot.resolve(path).resolve('android').toFile()
    include ":$name"
    project(":$name").projectDir = pluginDirectory
}

include ":UnityExport"
project(":UnityExport").projectDir = file("./UnityExport")

i'm pretty sure i had this issue when using unity as a library with arfoundation and various other 3rd party unity libs, and not adding their aar-files inside UnityExport/libs as project-dependencies
if you have additional libs there other than unity-classes.jar, add them as modules like you did with unity-classes.jar, and in my case with latest unity 2019.3 beta build, change your build.gradle in UnityExport to use:
 implementation project(':unity-classes')     implementation project(':ARPresto')    ...
for an example instead of having the dependencies added by their filenames directly (as the unity flutter export script does)
add all of those "projects" as includes to the "android" projects settings.gradle as you did with unity-classes and make sure you sync the gradle scripts
make sure the imported aar-files as modules are inside the UnityExport directory instead of the flutter project root, the chapter "Vuforia" in the readme hints about this (and also the way how to add the dependencies as written above), this finally made things work for me (well until i found issues with androidx libs, sdk versions and flutter-plugins, but that is whole other story ...)

Thank you for your time@subjectdenied , I did the steps as you mentioned but i guess the core problem is that i used emulator which working only on x86 architecture and unity does not support x86 in the latest version , i ran into other problem when i install the app into my phone MI 9T

Thank you for your time@subjectdenied , I did the steps as you mentioned but i guess the core problem is that i used emulator which working only on x86 architecture and unity does not support x86 in the latest version , i ran into other problem when i install the app into my phone MI 9T

I also got the same issue as you. I manage it to work by change isARScene: true, where dart file include the UnityWidget.

yeah it works now , thank you @sarawin008 you solved my problem ,Thank you all


Include a demo app with the repo

This will be useful when testing pull requests to
react-storybook repo

Could you add more info on this?
If we are going to take this, I need to change the README of the demo app to add the info we only need.
Just mention it's a CRA app and that's enough.

Okay. Take this.


tests: add SwCanvas unit tests

This requires this patch: https://github.com/Samsung/thorvg/commit/eda022365f7e166a857cdb775acee437074e0849


⚠ Fortnite has degraded performance

In ac66075, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1645 ms

Resolved: Fortnite performance has improved in 8c66569.


Add docker

Add docker to the boilerplate.
Readme has been updated.

Hi,
Thank you for your PR.
But, Can you create a PR to another branch not master?

Done,
Sorry for the inconvenience :)


fix: NavigationStart only hide currentStep if route is specified eg. routing used

Dear isaac
Introduction
We had a issue with the altered code, represeting a call to the hide function if any NavigationStart event happens. We are using some advanced routing mechanism, where the user will trigger interactively a router change but effectively he will stay on the same screen.
Issue description
The current implementation will hide the currentStep no matter if a custom router url per step is defined or not. this causing the tour to end, because we do not have any custom route defined for the following step.
Issue solution
The PR does only contain a little check for the property "route": this property will be set if any routing is used by the tourSteps and/or by the "stepDefaults". If it is being used, the mechanism will work as before. If there is no route property present (and therefore no handling required), it will just skip the hideStep call.
Testing
I've used the integrated demo/documentation pages and tested all of the modules. I had complete success with all modules, there were no sideeffect I've noticed.
Thanks for integrating the PR!
If you have any questions feel free to ask.
Greetings
Chris from enersis

Looks good to me.  Merging

Thanks for merging into, the new version works like a charm!


⚠ Bofrost.es has degraded performance

In b8a9868, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1914 ms

Resolved: Bofrost.es performance has improved in 8219af7 after 9 minutes.


remote-cli not being added to PATH with "terminal.integrated.inheritEnv": false

Fixes microsoft/vscode-remote-release#6289

@alexdima I'm out next week. Can you review and push?

New parameter withUserShellEnvironment for buildUserEnvironment
just moved the PATH computation down a bit (only for cosmetic reasons)

There's a this._logService.trace('*'); I removed. Not sure it was of any use.

I had to revert this because it consistently broke integration tests on windows 🤷 . Let's continue in https://github.com/microsoft/vscode/pull/143202


Deprecation warning : defineLocale from french locale file

Hello,
I got a warning (console) while using french locale fr.js file (since version 1.15.x and with 1.16.0) :
moment.updateLocale(localeName, config) to change an existing locale. moment.defineLocale(localeName, config) should only be used for creating a new locale See http://momentjs.com/guides/#/warnings/define-locale/ for more info.
Am i supposed to do something to avoid this ? If i can help on the french file, it would be pleasure.
In all case : thanks you for this amazing lib which works flawlessly with fullcalendar.
Best regards,
Vincent

If you are using the default locale file and not overriding any part of it in your code, then my guess is you are loading the file twice. If you override any part of the locale file in your code, then you need to change that code to use .updateLocale instead of .locale.

Thanks you for your answer, here is what i load :

min/moment.min.js
then locale/fr.js (warning is fire at this moment)

Since i'm using files "as it", without any change, i was thinking about a depreciation within the locale/fr.js

If you use fullcalendar it might be the cause (cf. #3076).

Yes, i've reported it here : https://github.com/fullcalendar/fullcalendar/issues/3471


CDN publishing needs a way to prepend CDN url fragment in front of bundle.js and related paths

When Phenomic populates the Helmet script and css paths, it hard-codes them to live in the root of the website but many sites will want to place them on a CDN that's separate from the hosting of the html file

We might solve this by accepting a final url for the website, like in v0.x. So scripts and css might be prefixed by this. That should solve your problem?

I don't think a final url specifier is sufficient. The issue is when using a CDN the js and css files have a different URL from the site. For example the site might be http://Foo.com, with the html file being http://Foo.com/index.html while the js and cs are at http://SomeCDN.com/yadayada/phenomic.browser.123423.js and http://SomeCDN.com/yadayada/phenomic.browser.123423.css

I guess it's doable from custom Html component directly, by not using props send by phenomic and defining script/link yourself. Does this look ok for you? I am afraid that introducing a specific option when you can already handle it will add confusion.

My concern is that the user doesn't have a good way to know which css file is the one that needs path mangling. Yes, you could walk the list of entries in the helmet.link array looking for the one that appears to match the pattern of a phenomic css file and mangle just that one, but that feels kind of sketchy.
The body.script object I think may be simpler/safer, but from here I can't tell whether that's guaranteed to just contain a single script tag whose path could then be mangled or whether it potentially has more complex stuff in it.

Providing your own Html component should be the way to go. We dont want to have so many config and flags like 0.2x
Btw, if your website generated by Phenomic is fully static, instead of deploying css, js and images to a CDN, why dont you deploy the whole website there? Your website will be blazing fast.

For a variety of tech stack reasons I need to deploy my HTML to the web server, in part because the entire site isn't able to be static, but I still want to make use of a CDN for the css and js

What we can probably do is to send another props to Html compo that have a list of url, no components directly. Will think about that.

I guess we'll need to add a default DefinePlugin to PhenomicWebpack which would pass automatically everything found in process.env so that users could do:
<body {...helmet.bodyAttributes.toComponent()}>
  {props.body}
  {props.state}
  {React.cloneElement(props.script, { src: process.env.URL + props.script.props.src })}
</body>

Ok for you?

That looks good for the props.script (which I think is browser.js), but I'm not sure it's enough to let us easily/safely do the same with browser.css. Does it make sense to call out browser.css separately from the other css links, so that it can be handled this way (I think currently browser.js is separate from other user-included js files but browser.css is thrown in with other user-included css files, but I might be misreading that)

Will be part of beta 1


consider adding a deployment option to automatically shut down all servers

Potential config API:
deployments:
  - name: production
    restartServers: true

Would this be a useful feature for developers? I feel like this could be a bit heavy-handed, especially for patch releases. Perhaps it should be a command-line flag (e.g. --restart-servers-post-deploy) instead so that it is more of a one-time thing?


Added option for top-level sinlge sub-configs to the keyring

I pulled out the part of the code that dealt with the keyring into a separate function for simplicity and added a check for whether a given keyring entry was a dictionary. If so, it treats it as before, otherwise, it tries to find a subconfig with the given value.

I forgot to remove a breakpoint before making the pull request.


feat: row write protocol

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?

datanode's row inserter
standalone's row inserter
influxdb line protocol  row inserter

Checklist

[ ]  I have written the necessary rustdoc comments.
[x]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)

@waynexia @MichaelScofield  PTAL

@evenyag PTAL


Ctrl+Enter doesn't work

While editing the query in the right-side panel, Ctrl+Enter doesn't run the SQL.
Chrome 48 on Ubuntu.

Same on Ubuntu/Chrome 52.0.2743.82 (64-bit)


feat: add more project templates

What :computer:
This PR adds more template projects to be chosen from namely an FT and an NFT project additional to the existing Hello World project.
The functionality has a default parameter value and is thus optional. This means this does not introduce any breaking changes to how the CLI currently works. If the user desires to initialise a project with the other templates that can be done using the flags of the CLI as:
zksync-cli create-project hh-vyper-nft --framework hardhat_vyper --project non_fungible_token

or
zksync-cli create-project hh-sol-ft --framework hardhat_solidity --project fungible_token

Why :hand:
Gives developers more flexibility when starting projects.
Evidence :camera:

A couple points on this PR:

In order to connect to the network in a GitHub CI runner with the action make sure the provider / hardhat.config is using <IP_ADDRESS>
Thank you for sourcing the templates remotely 💯
A monorepo of starter templates with a breadth of options with dedicated documentation / external contributions is more organized and easier to manage. For inspiration, see these starter kit templates. This to me will be much easier to manage then many template repos.

Lastly, Rather than focusing on the specifics of the contracts, such as ERC20 or NFTs, our priority should be catering to the developer's holistic experience.
Developers typically place significant emphasis on their intended project setup. For instance, while it's beneficial to provide a starter contract for an NFT project, what truly adds value is how efficiently they can bootstrap their project with their desired tools and configurations. In essence, it's not really about the contracts because they will likely write their own entirely for their specific needs, but it's about how seamlessly they can integrate into a preferred environment quickly.
Considering this, I propose that our template suite emphasizes the following areas:

Contract Development:

Hardhat + Solidity
Forge + Solidity
Hardhat + Vyper
Forge + Vyper

Web dApp (with contract integration, maybe even wallet support):

React
Next.js
Vite

Mobile:

react native
ios

Scripting:

node
bun

This structure addresses the core needs of a developer, facilitating a quick action into actual development with minimal setup overhead.  This would be genuinely enhancing the developer experience, making our offering more pragmatic and valuable. This also goes hand in hand with sourcing templates from a monorepo. Consider how awesome it would be to bootstrap a zksync web app quickly, or a script to debug / test something with ease and simplicity.
My apologies on the long-winded post here.


TestXMLDocument: Un-XFAIL DTD tests.

Serve the DTD from the built in HTTPServer so the tests
will work without internet access.

Resolves rdar://31567922.

This LGTM. What do you think, @pushkarnk ?

More tests are nice, and this was open long enough that we can open a separate PR for feedback if needed.

@swift-ci please test and merge


Generators and Iterators - wrong task goal

As task says:

Write a generator function factorial that, given an input number, starts at 1 and goes
up to the number, yielding the factorial of each number along the way.

So, what I expect to do is:

factorial(5):
1! = 1
2! = 2
3! = 6
4! = 24
5! = 120

But answer is wrong, because what seems to be checked is:

Write a generator function factorial that, given an input number, yielding the intermediate steps of factorial computation.

Should it be corrected in some way?

isRuslan/learn-generators#5
Yes, need to correct that way. Agreed.

Seems to be better if factorial will actually yield factorial of each number from [1..n] sequence.
Especially since it is written in the task :)
Will fixed in 1.2.0.

@isRuslan yep. Thanks! :star2:


swap height, width

https://github.com/waldo-seg/waldo/blob/540c3cbe1488e487c2c00a72240d95aaf9e65c4d/scripts/waldo/segmenter.py#L233
I think after we created the code we changed the indexing conventions.
this may be a purely cosmetic change.

also see line 263, they seem swapped there too.

Fixed in #57


Create more files

feat: create file 2
chore: new file 3

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #10

PR #11   👈

This stack of pull requests is managed by Graphite.


proposal: testing: allow regexps to specify example output

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.7.4 linux/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/eyal/go:/home/eyal/home/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build162642716=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"

What did you do?
Wrote a testable example.
What did you expect to see?
A possibility to define Output lines with a regular expression.
What did you see instead?
Only plan text is supported.

Why do you need this, exactly? Is the plaintext output too big or non-deterministic?
The point of output lines is readability and simplicity, and I thibk regexes would just add complexity.

I have a program that part of the output there is a time duration, that i would like to give an example for.
I think that making a regular expression optional won't harm anything in the simplicity of the example tests.
In the current example system I can't test this program.

Note that you'll also need to introduce an alternative prefix (not output:) or you'll break existing tests for people that for example wrote \d expecting a literal match with "\d".

you could sanitize the output before printing.
(e.g. use regexp to replace the number with a fixed number before print
them.)
Not only does this not require a new facility, it also allows the users to
use older versions of Go.

I saw in the code that there is unsorted option for the output, you use it by prefixing the output: unsorted Output:. (it is not documented)
We could do something similar, so it won't break non-regex output.

I think we'd prefer not to further complicate output tests. Doing something like replacing the time with "TIMESTAMP" in the output would be better. Or just don't print it in the first place.

I also don't want to do anything here. The whole point of examples to be simple.
Once you need a way to normalize the output, you've already failed at being simple.

Declining for @golang/proposal-review


Blob is not defined

It gives "UnhandledPromiseRejectionWarning: ReferenceError: Blob is not defined" error.
imageConversion.compressAccurately(data, {       size: 100, //The compressed image size is 100kb       accuracy: 0.9, //the accuracy of image compression size,range 0.8-0.99,default 0.95;       //this means if the picture size is set to 1000Kb and the       //accuracy is 0.9, the image with the compression result       //of 900Kb-1100Kb is considered acceptable;       type: 'image/jpeg',       width: 300,       height: 200,       orientation: 2,       scale: 0.5     });

What is your browser and os version

Hi,
I have similar error:
ReferenceError: Blob is not defined

ReferenceError: Blob is not defined
[1]     at Module. (/home/kiw/Programming/BP_code/app/node_modules/image-conversion/build/conversion.js:1:6142)
[1]     at Generator.next ()
[1]     at /home/kiw/Programming/BP_code/app/node_modules/image-conversion/build/conversion.js:1:5551
[1]     at new Promise ()
[1]     at y (/home/kiw/Programming/BP_code/app/node_modules/image-conversion/build/conversion.js:1:5296)
[1]     at Module.b (/home/kiw/Programming/BP_code/app/node_modules/image-conversion/build/conversion.js:1:6090)
[1]     at createMiniature (/home/kiw/Programming/BP_code/app/app/electron/communication.js:71:38)
[1]     at /home/kiw/Programming/BP_code/app/app/electron/communication.js:249:41
[1]     at tryCatcher (/home/kiw/Programming/BP_code/app/node_modules/bluebird/js/release/util.js:16:23)

I use it like this:
const result = imageConversion.compressAccurately(pathToPhoto, 150)         .then((res) => {         console.log(res)       }).catch((e) => {           console.log(e)         });
What am I doing wrong?
I run it in Electron

Same


Type of id impacts findDoc behaviour

db.table.findDoc(1) -> { id: 1 }
db.table.findDoc('1') -> [{ id: 1 }]

I'm not sure what's actually happening in the second statement, but it seems like the parameter is just ignored entirely (db.table.findDoc('abc') yields the same result).
I would expect it to either magically work through type coercion, or not return any results because '1' != 1; however, it behaving like a regular find all query is not what I expected.

Yeah, this could be clearer :). If the parameter you send in (which in this case is id) corresponds to the primary key and the value is an integer (or a UUID), then result will be a single record.
Otherwise it's treated as a simple where clause which returns multiple records.

Closing this unless something is to be done.


Can a QUIC server connect to another QUIC server?

For example, if a QUIC server is bound to a local port number 9999, can it use this port number as an outbound connection to connect to another QUIC server?

you are talking about a proxy and yes for sure this works


Bump openmdx patch version to 2.18.9

There was a regression case in OpenMDX which caused a bump in patch version. If you were to use version 2.18.8 of OpenMDX instead of actually used version 2.18-20231111 to build OpenCRX before upgrading to 5.3.0, I'm pretty confident it wouldn't have worked.
See https://github.com/openmdx/openmdx/commits/openmdx-v2.18.9

Then I wonder how that was possible. I tried building OpenCRX with 2.18.8 locally, was failing just like some other projects.

> java --version
openjdk version "<IP_ADDRESS>" 2023-08-24
OpenJDK Runtime Environment (build <IP_ADDRESS>+1-Ubuntu-0ubuntu122.04)
OpenJDK 64-Bit Server VM (build <IP_ADDRESS>+1-Ubuntu-0ubuntu122.04, mixed mode, sharing)

> git clone https://github.com/opencrx/opencrx.git opencrx
> cd opencrx
> git checkout opencrx-v5.3.0
> ./gradlew --refresh-dependencies clean
> ./gradlew --refresh-dependencies assemble

BUILD SUCCESSFUL in 1m 24s
29 actionable tasks: 29 executed

Sorry for the confusion, it's the schema generation that is failing with 2.18.8
...
[ant:mappingtool] 	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
[ant:mappingtool] 	at java.base/java.lang.Thread.run(Thread.java:829)
[ant:mappingtool] Caused by: <openjpa-3.2.2-re5933d6 fatal user error> org.apache.openjpa.util.MetaDataException: The id class specified by type "class org.opencrx.kernel.jpa3.Segment$Slice" does not match the primary key fields of the class.  Make sure your identity class has the same primary keys as your persistent type, including pk field types. Mismatched property: "openmdxjdoIdentity"
[ant:mappingtool] 	at org.apache.openjpa.meta.ClassMetaData.validateAppIdClassPKs(ClassMetaData.java:2228)
[ant:mappingtool] 	at org.apache.openjpa.meta.ClassMetaData.validateAppIdClass(ClassMetaData.java:2102)
[ant:mappingtool] 	at org.apache.openjpa.meta.ClassMetaData.validateIdentity(ClassMetaData.java:2038)
[ant:mappingtool] 	at org.apache.openjpa.meta.ClassMetaData.validateMeta(ClassMetaData.java:1950)
[ant:mappingtool] 	at org.apache.openjpa.meta.ClassMetaData.resolve(ClassMetaData.java:1811)
[ant:mappingtool] 	at org.apache.openjpa.meta.MetaDataRepository.processBuffer(MetaDataRepository.java:838)
[ant:mappingtool] 	at org.apache.openjpa.meta.MetaDataRepository.resolveMeta(MetaDataRepository.java:734)
[ant:mappingtool] 	at org.apache.openjpa.meta.MetaDataRepository.resolve(MetaDataRepository.java:655)
[ant:mappingtool] 	... 185 more
[ant:mappingtool] NestedThrowables:
[ant:mappingtool] <openjpa-3.2.2-re5933d6 fatal user error> org.apache.openjpa.util.MetaDataException: Attempt to map "org.opencrx.kernel.jpa3.Segment$Slice.openmdxjdoIdentity" failed: the owning entity is not mapped.
[ant:mappingtool] 	at org.apache.openjpa.jdbc.meta.MappingInfo.assertTable(MappingInfo.java:632)
[ant:mappingtool] 	at org.apache.openjpa.jdbc.meta.MappingInfo.createForeignKey(MappingInfo.java:1084)
[ant:mappingtool] 	at org.apache.openjpa.jdbc.meta.ValueMappingInfo.getTypeJoin(ValueMappingInfo.java:117)
...

Correct.
./gradlew -Pdatabase.name=hsqldb-2 create-schema
does not work with openmdx-v2.18.9 and later. The reason is a missing META-INF/orm.xml in openmdx-base.jar. I have reported this to @DlRTY-HARRY without ticket. I have now made up for this: https://github.com/openmdx/openmdx/issues/98
This is why opencrx master is using openmdx-v2.18-20231111
This bug was detected after releasing opencrx-v5.3.0. opencrx-v5.3.1 will be released when this and other bugs are fixed.

I just tried it, the orm.xml file was there for openmdx version 2.18.7 through 2.18.9
I would suggest that you once more try ./gradlew -Pdatabase.name=hsqldb-2 create-schema with opencrx-5.3.0 (which has openmdx-2.18.8)
It should fail with this: https://github.com/opencrx/opencrx/pull/134#issuecomment-1838155429 from above,
Then update to openmdx-2.18.9 and the schema generation should work just fine.
If anything diverges from above, then there has to be a concerning flakiness around because I'm getting these really consistently.

See https://github.com/openmdx/openmdx/issues/98 how to reproduce an openmdx build which consistently builds openmdx-base.jar without META-INF/orm.xml.
opencrx will not be upgraded to something newer than openmdx-v2.18-20231111 until https://github.com/openmdx/openmdx/issues/98 is fixed.
Will close here. Please move discussion to https://github.com/openmdx/openmdx/issues/98


Update LaCE Team Onboarding Guide

The LaCE Team Onboarding guide currently contains a bunch of solutions to common problems faced by team members while working on issues, such as populating data by creating topics, skills, etc. However, it doesn't provide enough information about what the LaCE Team is about.
In short, the current onboarding guide of the LaCE Team doesn't resemble an onboarding guide. Therefore, we would like to update it.
You can review the Contributor Dashboard team's onboarding guide for reference on how an onboarding guide should look: Contributor Dashboard Onboarding Guide
LaCE Team onboarding guide: https://github.com/oppia/oppia/wiki/LaCE-onboarding-guide

/cc @HardikGoyal2003, @HardikGoyal2003.

Hey @Ash-2k3 Please can you assign this issue to me?

We would want to explain the terms like Topic, Exploration, Classroom, etc and their relationships in the new wiki.
We can also fold in some contents from the User Documentation: https://oppia.github.io/#/


Prevent Retry from happening endlessly

[x] Add test case(s)

[x] Update CHANGES.txt

[x] Fix bugs in TestNG

@juherr - Can you please have a quick look ?


Pop comments

In this pull request I :
-create a popup  that loads, the webapp and retrieves data from the API to show the item comments
-apply gitflow
-refactor code accoding to US6 guidelines

@Kidd254 Great work Lawerence, I have seen you added retrieving the length in the pop display. It was a good call!
Is ready to merge!🙌🙌


Failed to build split-monitor-workspaces on latest Hyprland built from source

Plugin is not building since updated to latest Hyprland release built from source, was working fine before v0.41.0.
I'd be happy to try and learn about this and get it working if I can get some guidance.
Cheers.
[v] version returned: Hyprland, built from branch main at commit a9d7befc36f2890f080d02b8c04b678778ded080 dirty (formats: fixup for legacy_renderer builds).
Date: Wed Jun 12 23:29:24 2024
Tag: v0.41.0-21-ga9d7befc, commits: 4815

flags: (if any)

[v] parsed commit a9d7befc36f2890f080d02b8c04b678778ded080 at branch main on Wed Jun 12 23:29:24 2024, commits 4815
✔ adding a new plugin repository from https://github.com/Duckonaut/split-monitor-workspaces
  MAKE SURE that you trust the authors. DO NOT install random plugins without verifying the code and author.
  Are you sure? [Y/n] Y

 → Cloning https://github.com/Duckonaut/split-monitor-workspaces
✔ cloned
✔ found hyprpm manifest
✔ parsed manifest, found 1 plugins:
 → split-monitor-workspaces by Duckonaut version
 → Manifest has 4 pins, checking
✔ Hyprland headers OK
 → Building split-monitor-workspaces
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━╍━━━━━━━━━━━━━━━━━━━  3 / 5  Building plugin(s)[v] shell returned:  -> cd /tmp/hyprpm/msviridov && PKG_CONFIG_PATH="/home/msviridov/.local/share/hyprpm/headersRoot/share/pkgconfig" meson setup build --wipe
The Meson build system
Version: 1.4.1
Source dir: /tmp/hyprpm/msviridov
Build dir: /tmp/hyprpm/msviridov/build
Build type: native build
Project name: split-monitor-workspaces
Project version: 0.1
C++ compiler for the host machine: c++ (gcc 14.1.1 "c++ (GCC) 14.1.1 20240522")
C++ linker for the host machine: c++ ld.bfd 2.42.0
Host machine cpu family: x86_64
Host machine cpu: x86_64
Compiler for C++ supports arguments -std=c++23: YES
meson.build:8: WARNING: Consider using the built-in option for language standard version instead of using "-std=c++23".
Found pkg-config: YES (/usr/bin/pkg-config) 2.1.1
Run-time dependency hyprland found: YES 0.41.0
Run-time dependency pixman-1 found: YES 0.43.4
Run-time dependency libdrm found: YES 2.4.121
Build targets in project: 1

Found ninja-1.12.1 at /usr/bin/ninja

 -> cd /tmp/hyprpm/msviridov && PKG_CONFIG_PATH="/home/msviridov/.local/share/hyprpm/headersRoot/share/pkgconfig" meson compile -C build
ninja: Entering directory `/tmp/hyprpm/msviridov/build'
[1/2] Compiling C++ object libsplit-monitor-workspaces.so.p/src_main.cpp.o
FAILED: libsplit-monitor-workspaces.so.p/src_main.cpp.o
c++ -Ilibsplit-monitor-workspaces.so.p -I. -I.. -I../include -I/home/msviridov/.local/share/hyprpm/headersRoot/include -I/home/msviridov/.local/share/hyprpm/headersRoot/include/hyprland/protocols -I/home/msviridov/.local/share/hyprpm/headersRoot/include/hyprland -I/home/msviridov/.local/share/hyprpm/headersRoot/include/hyprland/wlr -I/usr/include/pixman-1 -I/usr/include/libdrm -fdiagnostics-color=always -D_GLIBCXX_ASSERTIONS=1 -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -O3 -std=c++23 -DWLR_USE_UNSTABLE -fPIC -MD -MQ libsplit-monitor-workspaces.so.p/src_main.cpp.o -MF libsplit-monitor-workspaces.so.p/src_main.cpp.o.d -o libsplit-monitor-workspaces.so.p/src_main.cpp.o -c ../src/main.cpp
In file included from ../src/main.cpp:7:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:158:20: error: reference to ‘CSharedPointer_’ is ambiguous
  158 |     CSharedPointer(CSharedPointer_::impl_base* implementation) noexcept {
      |                    ^~~~~~~~~~~~~~~
In file included from /usr/include/hyprutils/memory/WeakPtr.hpp:3,
                 from /home/msviridov/.local/share/hyprpm/headersRoot/include/hyprland/src/helpers/../helpers/memory/Memory.hpp:3,
                 from /home/msviridov/.local/share/hyprpm/headersRoot/include/hyprland/src/helpers/../macros.hpp:7,
                 from /home/msviridov/.local/share/hyprpm/headersRoot/include/hyprland/src/helpers/Vector2D.hpp:5,
                 from /home/msviridov/.local/share/hyprpm/headersRoot/include/hyprland/src/includes.hpp:113,
                 from /home/msviridov/.local/share/hyprpm/headersRoot/include/hyprland/src/defines.hpp:1,
                 from /home/msviridov/.local/share/hyprpm/headersRoot/include/hyprland/src/Compositor.hpp:7,
                 from ../src/main.cpp:3:
/usr/include/hyprutils/memory/SharedPtr.hpp:20:19: note: candidates are: ‘namespace Hyprutils::Memory::CSharedPointer_ { }’
   20 |         namespace CSharedPointer_ {
      |                   ^~~~~~~~~~~~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:21:11: note:                 ‘namespace CSharedPointer_ { }’
   21 | namespace CSharedPointer_ {
      |           ^~~~~~~~~~~~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:158:46: error: expected ‘)’ before ‘*’ token
  158 |     CSharedPointer(CSharedPointer_::impl_base* implementation) noexcept {
      |                   ~                          ^
      |                                              )
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:252:5: error: reference to ‘CSharedPointer_’ is ambiguous
  252 |     CSharedPointer_::impl_base* impl_ = nullptr;
      |     ^~~~~~~~~~~~~~~
/usr/include/hyprutils/memory/SharedPtr.hpp:20:19: note: candidates are: ‘namespace Hyprutils::Memory::CSharedPointer_ { }’
   20 |         namespace CSharedPointer_ {
      |                   ^~~~~~~~~~~~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:21:11: note:                 ‘namespace CSharedPointer_ { }’
   21 | namespace CSharedPointer_ {
      |           ^~~~~~~~~~~~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: In constructor ‘CSharedPointer<T>::CSharedPointer(T*)’:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:132:9: error: ‘impl_’ was not declared in this scope
  132 |         impl_ = new CSharedPointer_::impl<T>(object);
      |         ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:132:21: error: reference to ‘CSharedPointer_’ is ambiguous
  132 |         impl_ = new CSharedPointer_::impl<T>(object);
      |                     ^~~~~~~~~~~~~~~
/usr/include/hyprutils/memory/SharedPtr.hpp:20:19: note: candidates are: ‘namespace Hyprutils::Memory::CSharedPointer_ { }’
   20 |         namespace CSharedPointer_ {
      |                   ^~~~~~~~~~~~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:21:11: note:                 ‘namespace CSharedPointer_ { }’
   21 | namespace CSharedPointer_ {
      |           ^~~~~~~~~~~~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: In constructor ‘CSharedPointer<T>::CSharedPointer(const CSharedPointer<U>&)’:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:139:9: error: ‘impl_’ was not declared in this scope
  139 |         impl_ = ref.impl_;
      |         ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: In copy constructor ‘CSharedPointer<T>::CSharedPointer(const CSharedPointer<T>&)’:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:144:9: error: ‘impl_’ was not declared in this scope
  144 |         impl_ = ref.impl_;
      |         ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: In constructor ‘CSharedPointer<T>::CSharedPointer(CSharedPointer<U>&&)’:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:150:19: error: ‘impl_’ was not declared in this scope
  150 |         std::swap(impl_, ref.impl_);
      |                   ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: In constructor ‘CSharedPointer<T>::CSharedPointer(CSharedPointer<T>&&)’:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:154:19: error: ‘impl_’ was not declared in this scope
  154 |         std::swap(impl_, ref.impl_);
      |                   ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: In destructor ‘CSharedPointer<T>::~CSharedPointer()’:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:177:13: error: ‘impl_’ was not declared in this scope
  177 |         if (impl_ && impl_->ref() == 1)
      |             ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: In member function ‘CSharedPointer<T>::validHierarchy<const CSharedPointer<U>&> CSharedPointer<T>::operator=(const CSharedPointer<U>&)’:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:185:13: error: ‘impl_’ was not declared in this scope
  185 |         if (impl_ == rhs.impl_)
      |             ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:189:9: error: ‘impl_’ was not declared in this scope
  189 |         impl_ = rhs.impl_;
      |         ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: In member function ‘CSharedPointer<T>& CSharedPointer<T>::operator=(const CSharedPointer<T>&)’:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:195:13: error: ‘impl_’ was not declared in this scope
  195 |         if (impl_ == rhs.impl_)
      |             ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:199:9: error: ‘impl_’ was not declared in this scope
  199 |         impl_ = rhs.impl_;
      |         ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: In member function ‘CSharedPointer<T>::validHierarchy<const CSharedPointer<U>&> CSharedPointer<T>::operator=(CSharedPointer<U>&&)’:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:206:19: error: ‘impl_’ was not declared in this scope
  206 |         std::swap(impl_, rhs.impl_);
      |                   ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: In member function ‘CSharedPointer<T>& CSharedPointer<T>::operator=(CSharedPointer<T>&&)’:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:211:19: error: ‘impl_’ was not declared in this scope
  211 |         std::swap(impl_, rhs.impl_);
      |                   ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: In member function ‘CSharedPointer<T>::operator bool() const’:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:216:16: error: ‘impl_’ was not declared in this scope
  216 |         return impl_ && impl_->dataNonNull();
      |                ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: In member function ‘bool CSharedPointer<T>::operator==(const CSharedPointer<T>&) const’:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:220:16: error: ‘impl_’ was not declared in this scope
  220 |         return impl_ == rhs.impl_;
      |                ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: In member function ‘bool CSharedPointer<T>::operator<(const CSharedPointer<T>&) const’:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:228:27: error: ‘impl_’ was not declared in this scope
  228 |         return (uintptr_t)impl_ < (uintptr_t)rhs.impl_;
      |                           ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: In member function ‘void CSharedPointer<T>::reset()’:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:241:9: error: ‘impl_’ was not declared in this scope
  241 |         impl_ = nullptr;
      |         ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: In member function ‘T* CSharedPointer<T>::get() const’:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:245:21: error: ‘impl_’ was not declared in this scope
  245 |         return (T*)(impl_ ? static_cast<CSharedPointer_::impl<T>*>(impl_)->_data : nullptr);
      |                     ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:245:41: error: reference to ‘CSharedPointer_’ is ambiguous
  245 |         return (T*)(impl_ ? static_cast<CSharedPointer_::impl<T>*>(impl_)->_data : nullptr);
      |                                         ^~~~~~~~~~~~~~~
/usr/include/hyprutils/memory/SharedPtr.hpp:20:19: note: candidates are: ‘namespace Hyprutils::Memory::CSharedPointer_ { }’
   20 |         namespace CSharedPointer_ {
      |                   ^~~~~~~~~~~~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:21:11: note:                 ‘namespace CSharedPointer_ { }’
   21 | namespace CSharedPointer_ {
      |           ^~~~~~~~~~~~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:245:56: error: expected ‘>’ before ‘::’ token
  245 |         return (T*)(impl_ ? static_cast<CSharedPointer_::impl<T>*>(impl_)->_data : nullptr);
      |                                                        ^~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:245:56: error: expected ‘(’ before ‘::’ token
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:245:58: error: ‘::impl’ has not been declared
  245 |         return (T*)(impl_ ? static_cast<CSharedPointer_::impl<T>*>(impl_)->_data : nullptr);
      |                                                          ^~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:245:58: note: suggested alternatives:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:39:11: note:   ‘CSharedPointer_::impl’
   39 |     class impl : public impl_base {
      |           ^~~~
/usr/include/hyprutils/memory/SharedPtr.hpp:37:19: note:   ‘Hyprutils::Memory::CSharedPointer_::impl’
   37 |             class impl : public impl_base {
      |                   ^~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:245:64: error: expected primary-expression before ‘>’ token
  245 |         return (T*)(impl_ ? static_cast<CSharedPointer_::impl<T>*>(impl_)->_data : nullptr);
      |                                                                ^
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:245:66: error: expected primary-expression before ‘>’ token
  245 |         return (T*)(impl_ ? static_cast<CSharedPointer_::impl<T>*>(impl_)->_data : nullptr);
      |                                                                  ^
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:245:82: error: expected ‘)’ before ‘:’ token
  245 |         return (T*)(impl_ ? static_cast<CSharedPointer_::impl<T>*>(impl_)->_data : nullptr);
      |                                                                                ^
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: In member function ‘unsigned int CSharedPointer<T>::strongRef() const’:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:249:16: error: ‘impl_’ was not declared in this scope
  249 |         return impl_ ? impl_->ref() : 0;
      |                ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: In member function ‘void CSharedPointer<T>::decrement()’:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:261:14: error: ‘impl_’ was not declared in this scope
  261 |         if (!impl_)
      |              ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:264:9: error: ‘impl_’ was not declared in this scope
  264 |         impl_->dec();
      |         ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: In member function ‘void CSharedPointer<T>::increment()’:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:272:14: error: ‘impl_’ was not declared in this scope
  272 |         if (!impl_)
      |              ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:275:9: error: ‘impl_’ was not declared in this scope
  275 |         impl_->inc();
      |         ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: In member function ‘void CSharedPointer<T>::destroyImpl()’:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:282:9: error: ‘impl_’ was not declared in this scope
  282 |         impl_->destroy();
      |         ^~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp: At global scope:
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:293:8: error: reference to ‘CSharedPointer’ is ambiguous
  293 | static CSharedPointer<U> makeShared(Args&&... args) {
      |        ^~~~~~~~~~~~~~
/usr/include/hyprutils/memory/SharedPtr.hpp:120:15: note: candidates are: ‘template<class T> class Hyprutils::Memory::CSharedPointer’
  120 |         class CSharedPointer {
      |               ^~~~~~~~~~~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:122:7: note:                 ‘template<class T> class CSharedPointer’
  122 | class CSharedPointer {
      |       ^~~~~~~~~~~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:298:34: error: template argument 1 is invalid
  298 | struct std::hash<CSharedPointer<T>> {
      |                                  ^~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:298:13: error: template class without a name
  298 | struct std::hash<CSharedPointer<T>> {
      |             ^~~~~~~~~~~~~~~~~~~~~~~
../src/main.cpp:26:8: error: re
   26 | static SP<HOOK_CALLBACK_FN> e_monitorAddedHandle = nullptr;
      |        ^~
/usr/include/hyprutils/memory/SharedPtr.hpp:120:15: note: candidates are: ‘template<class T> class Hyprutils::Memory::CSharedPointer’
  120 |         class CSharedPointer {
      |               ^~~~~~~~~~~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:122:7: note:                 ‘template<class T> class CSharedPointer’
  122 | class CSharedPointer {
      |       ^~~~~~~~~~~~~~
../src/main.cpp:27:8: error: reference to ‘CSharedPointer’ is ambiguous
   27 | static SP<HOOK_CALLBACK_FN> e_monitorRemovedHandle = nullptr;
      |        ^~
/usr/include/hyprutils/memory/SharedPtr.hpp:120:15: note: candidates are: ‘template<class T> class Hyprutils::Memory::CSharedPointer’
  120 |         class CSharedPointer {
      |               ^~~~~~~~~~~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:122:7: note:                 ‘template<class T> class CSharedPointer’
  122 | class CSharedPointer {
      |       ^~~~~~~~~~~~~~
../src/main.cpp:28:8: error: reference to ‘CSharedPointer’ is ambiguous
   28 | static SP<HOOK_CALLBACK_FN> e_configReloadedHandle = nullptr;
      |        ^~
/usr/include/hyprutils/memory/SharedPtr.hpp:120:15: note: candidates are: ‘template<class T> class Hyprutils::Memory::CSharedPointer’
  120 |         class CSharedPointer {
      |               ^~~~~~~~~~~~~~
/usr/local/include/hyprland/src/helpers/memory/SharedPtr.hpp:122:7: note:                 ‘template<class T> class CSharedPointer’
  122 | class CSharedPointer {
      |       ^~~~~~~~~~~~~~
../src/main.cpp: In function ‘PLUGIN_DESCRIPTION_INFO pluginInit(void*)’:
../src/main.cpp:326:5: error: ‘e_monitorAddedHandle’ was not declared in this scope
  326 |     e_monitorAddedHandle = HyprlandAPI::registerCallbackDynamic(PHANDLE, "monitorAdded", refreshMapping);
      |     ^~~~~~~~~~~~~~~~~~~~
../src/main.cpp:327:5: error: ‘e_monitorRemovedHandle’ was not declared in this scope
  327 |     e_monitorRemovedHandle = HyprlandAPI::registerCallbackDynamic(PHANDLE, "monitorRemoved", refreshMapping);
      |     ^~~~~~~~~~~~~~~~~~~~~~
../src/main.cpp:328:5: error: ‘e_configReloadedHandle’ was not declared in this scope
  328 |     e_configReloadedHandle = HyprlandAPI::registerCallbackDynamic(PHANDLE, "configReloaded", configReloadedCallback);
      |     ^~~~~~~~~~~~~~~~~~~~~~
ninja: build stopped: subcommand failed.
INFO: autodetecting backend as ninja
INFO: calculating backend command to run: /usr/bin/ninja -C /tmp/hyprpm/msviridov/build

✖ Plugin split-monitor-workspaces failed to build.
  This likely means that the plugin is either outdated, not yet available for your version, or broken.
  If you are on -git, update first.
  Try re-running with -v to see more verbose output.

✔ all plugins built
✔ installed repository
✔ you can now enable the plugin(s) with hyprpm enable
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━  5 / 5  Done!

You seem to have a hyprland installation installed in /usr/local/, but also in /usr/. Delete the one in /usr/local and see what happens.

Looks like this is fixed with https://github.com/Duckonaut/split-monitor-workspaces/pull/96. I'll try to get that merged asap

Great, thank you! Is there any way to get the plugin working until the merge gets done? Cheers.

Should be merged now, can you try installing it again with hyprpm? If that doesn't work, try manually building and loading.

Just opened a similar issue, i'll try building it manually


Files will not open

I see the latest version fixed an issue with not allowing files to be opened. This exact issue is still happening. (v1.1.4)

Please go to IntelliJ IDEA -> About IntelliJ IDEA (Mac) or Help -> About IntelliJ IDEA (Windows) and copy the infos
to the clipboard and paste it here.
It should contain:
* OS (Windows, Linux, Mac)
* IntelliJ Product + version (IDEA, RubyMine...)
* Plugin version
* A list of enabled plugins
What are the steps to reproduce the issue? Provide a video by dropping it in here if able to.

Windows 10 Home - Version 22H2
Android Stuido Dolphin | 2021.3.1 Patch 1 Ai-213.7172.25.2113.9123335
Anime Memes 1.1.4 Unthrottled
+
Anime Memes - Android Extension 0.2.0 Unthrottled

.ignore 4.4.0 JetBrains
CodeGlance Pro 1.3.8-Last2021.3.x Nasller
Extra ToolWindow Colorful Icons 1.24.0 Jonathan Lermitage
Gerry Themes 2023.1.10 Gerry
Key Promoter X 2021.3 halirutan
Notes Tree 1.2.2 Dmitrii Priporov
Nyan Progress Bar 1.14 Dmitry Barkovich
Rainbow Brackets 2023.1.0-ij izhangzhihao
Spec & Math Symbols 1.5.0 Alexander Galagutskiy
StopCoding 1.2.2 jogeen

Process, from plugin uninstalled

Open IDE, go to project or home
Settings > Plugins > Anime Memes; click Enable, choose Yes for installing extension
Allow IDE to install both AMII and AMII extension
Close IDE

(A) Try to open directly to most recent project

Window blinks then dissapears, process still running according to Task Manager
(tricky to get out of this loop, had to move the project to a new file so the IDE couldn't open the most recent project and defaulted to home)

(B) Try to open to home

Home opens, double click on any recent project
Menu blinks, does not open project
Able to navigate menus, but all clickable options do nothing
Only option is to close IDE and restart, which re-enables clickable options
(easier to get out of, but I still can't open any files until I disable the AMII plugin)

Sorry for no video, I'm learning and haven't figured that out yet. Thank you for your help!

Okay, that's enough information for me. I think I know what the issue might be.  Give me some time and I'll get it fixed later this week. Thanks!


fix(rust): forced-killed foreground nodes can be recreated with ockam node create

Follow up of https://github.com/build-trust/ockam/pull/5166
These changes fix a problem where a foreground node forcibly killed (e.g. due to an OS restart) was unable to get recreated, as it wasn't removing the previous tcp-listener and we were trying to use it as the "default-tcp-listener", resulting in a "peer not found" error.
To avoid this kind of misuse, the list of transports stored as part of the node config has been replaced by a single transport, which will be overwritten upon restarting a node.
To provide more context:

We were storing a list of transports so we could recreate them after a node restart, but this feature was never developed
We were using the first tcp transport of that list to enable communication between the command and the api, which felt unnecessary and can lead to problems if it's not keep properly updated

The following snippet now works as expected:
ockam node create n -f &
kill -9 $(cat $OCKAM_HOME/nodes/n/pid)
ockam node create n -f &

⚠ UPDATE: I've disabled the tests that were using kill to stop the node, as it was having an unexpected behavior on CI. I tried several different approaches but none worked. I'll move those tests to rust code as soon as I can.


NFD add test_branch function

Signed-off-by: Carlos Eduardo Arango Gutierrez<EMAIL_ADDRESS>

/uncc jmencak

/assign @kpouget

Run ansible-lint on the playbooks and roles / build (pull_request)
LOL I didn't modified any role 🙃

/lgtm
/hold

render_group

+ ansible-lint -v --force-color -c config/ansible-lint.yml playbooks roles
INFO     Added ANSIBLE_ROLES_PATH=~/.ansible/roles:/usr/share/ansible/roles:/etc/ansible/roles:roles
Traceback (most recent call last):
  File "/usr/local/bin/ansible-lint", line 8, in <module>
    sys.exit(_run_cli_entrypoint())
  File "/usr/local/lib/python3.9/site-packages/ansiblelint/__main__.py", line 276, in _run_cli_entrypoint
    sys.exit(main(sys.argv))
  File "/usr/local/lib/python3.9/site-packages/ansiblelint/__main__.py", line 192, in main
    from ansiblelint.generate_docs import rules_as_rich, rules_as_rst
  File "/usr/local/lib/python3.9/site-packages/ansiblelint/generate_docs.py", line 6, in <module>
    from rich.console import render_group
ImportError: cannot import name 'render_group' from 'rich.console' (/usr/local/lib/python3.9/site-packages/rich/console.py)

first time I see this, my other PR don't fail on it ...

Because it was removed 2 days ago: https://github.com/Textualize/rich/commit/ad9debf2eb5a04cea709d35a7bc03f316bf0431c

cool, thanks for checking @fcami, so the issue should solve itself soon :)

/approve
/unhold

PR doesn't affects how GPU's get tested
/override ci/prow/gpu-operator-e2e


Correct 'Teaching demonstration' reason for rejection

Context
James A spotted a couple of inconsistencies with the SR4R design prototype...

Changes proposed in this pull request
Changes legend for radios

Correct 'Teaching demonstration' reason label

Guidance to review

Link to Trello card

Things to check

[x] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[x] This code does not rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] Required environment variables have been updated added to the Azure KeyVault


Tibco/Ems receiver sleep interrupted java.lang.InterruptedException: null

Error message on the Console: Receiver [ProcessDPDHTitan] sleep interrupted, java.lang.InterruptedException: null.
The tibco/ems receiver does not start after ConnectionWaitTimeoutException's happened.
The Receiver can only be started manually, through the console.
Versie: FF! 7.8.3-hotfix1
Reporter
Richard van der Stel
Additional Context
Sent by mail to Niels: logfile and adapter xml.

Hello @rvdstel,
We've investigated this issue on the interrupt exceptions together with the SocketTimeoutException (Read timed out) messages, found in the log file.
First, we've concluded that in F!F version 7.9 and higher, the error handling works quite differently and is improved already.
Secondly, it is hard to get clear who exactly interrupted our receivers. That stack is quite short somehow.
Thirdly, we think that it is okay that it moves to the state STOPPED after somebody interrupted our receivers. That's with the current information we have.
If you encounter the same issue in the future again, with version 7.9 or higher, we like to investigate this once more. So, please then let us know. For now, we're just doing some minor cleanups, but no functional changes on this logic.
Thanks,
Jacobjob


T5 MoE docs need updates

There is no description of T5 MoE Support in docs/t5_guide.md; updates are needed; thanks!
Another question: are MoE Support in  examples/pytorch/t5/translate_example.py thoroughly tested? I found some suspicious bugs in it.

There is no description of T5 MoE Support in docs/t5_guide.md; updates are needed; thanks!
Another question: are MoE Support in examples/pytorch/t5/translate_example.py thoroughly tested? I found some suspicious bugs in it.

We don't have public checkpoint to demo.

Does the kernels even work? I set random weights to set up an MoE T5, but I continuously get errors regarding internal errors in CUTLASS MoE GEMM kernel. Any thoughts?

Also, Switch Transformers is pretty much the MoE version of T5.
The weights are publicly available everywhere, namely huggingface.

There is no description of T5 MoE Support in docs/t5_guide.md; updates are needed; thanks!
Another question: are MoE Support in examples/pytorch/t5/translate_example.py thoroughly tested? I found some suspicious bugs in it.

We don't have public checkpoint to demo.

I see, thank you. But it's very important for us users to follow this work. I'll appreciate it if you can provide some checkpoints and detailed docs about T5 Moe Support.

Hello:
Has this problem been fixed? How can I use FT to support T5-MoE? please~

+1