chore(main): release 2.141.0

:robot: I have created a release beep boop
2.141.0 (2024-09-13)
Features

Add hubspot cookie tracking (#4539) (6714384)
Add subscription cache for new organisations (#4587) (b2a1899)
Add use_identity_overrides_in_local_eval setting (#4612) (f8a048e)
Detect unchanged feature states when saving versions (#4609) (0e53baf)
Move versioned feature history into feature details modal (#4499) (ae47db1)

Bug Fixes

404 when last organisation doesn't exist (#4624) (d60b3b7)
Allow switching organisations if current one is blocked (#4606) (6ef774b)
Don't include null traits in transient identifier (#4598) (4bf7b9d)
Handle null cancellation dates (#4589) (603889c)
ignore old versions when validating segment override limit (#4618) (52b9780)
incorrect Java SDK installation and initialization code examples (#4596) (d12cf8b)
versioning: fix issue creating duplicate priority segment overrides (#4603) (1e357b8)
versioning: use transaction.atomic to prevent corrupt versions being created (#4617) (7ac05cd)
webhook/changed_by: Return name of the master api key (#4602) (1b22cf5)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v2.141.0
:sunflower:


[BUG] pyproject.toml based builds using setuptools.build_meta don't allow setup.py to import from the local project

setuptools version
75.6.0
Python version
3.10
OS
Linux Mint 21.3 Cinnamon
Additional environment information
I doubt that the problem is specific to the above details to any significant extent.
Description
Imports from setup.py to the local code fail (even within setup.py's directory) when setuptools.build_meta is used as the backend. The path to setup.py is missing from sys.path, leading to the traceback shown below.
The build succeeds if sys.path is appropriately hacked in setup.py, e.g.
from pathlib import Path
import sys
sys.path.insert(0, str(Path(__file__).parent))
import content

; or if setuptools.build_meta:__legacy__ is used instead (or if the build-backend key is omitted from pyproject.toml entirely.
The problem is not fixed by disabling build isolation, nor by using a different front-end (e.g. pip wheel .).
Expected behavior
The build should succeed as is, since the historical user expectation is that setup.py can import code from the local project. (For example, this has been used to retrieve a __version__ value and compute dynamic version metadata. While I understand that Setuptools provides its own mechanisms for this now, this is just one example of a use case generally expected to work.)
How to Reproduce
$ mkdir test_project && cd test_project
$ touch content.py
$ cat > setup.py
from setuptools import setup
import content

setup()
$ cat > pyproject.toml
[project]
name = "test"
version = "1.0.0"

[build-system]
requires = ["setuptools==75.6.0"]
build-backend = "setuptools.build_meta"
$ pyproject-build

Output
Using pyproject-build (i.e., build installed by Pipx) - home path elided:
$ pyproject-build
* Creating isolated environment: venv+pip...
* Installing packages in isolated environment:
  - setuptools==75.6.0
* Getting build dependencies for sdist...
Traceback (most recent call last):
  File "~/.local/pipx/venvs/build/lib/python3.10/site-packages/pyproject_hooks/_in_process/_in_process.py", line 389, in <module>
    main()
  File "~/.local/pipx/venvs/build/lib/python3.10/site-packages/pyproject_hooks/_in_process/_in_process.py", line 373, in main
    json_out["return_val"] = hook(**hook_input["kwargs"])
  File "~/.local/pipx/venvs/build/lib/python3.10/site-packages/pyproject_hooks/_in_process/_in_process.py", line 317, in get_requires_for_build_sdist
    return hook(config_settings)
  File "/tmp/build-env-e17m7bci/lib/python3.10/site-packages/setuptools/build_meta.py", line 337, in get_requires_for_build_sdist
    return self._get_build_requires(config_settings, requirements=[])
  File "/tmp/build-env-e17m7bci/lib/python3.10/site-packages/setuptools/build_meta.py", line 304, in _get_build_requires
    self.run_setup()
  File "/tmp/build-env-e17m7bci/lib/python3.10/site-packages/setuptools/build_meta.py", line 320, in run_setup
    exec(code, locals())
  File "<string>", line 2, in <module>
ModuleNotFoundError: No module named 'content'

ERROR Backend subprocess exited when trying to invoke get_requires_for_build_sdist

Similarly via Pip (aliased to Pipx's copy):
$ pip wheel .
Processing /path/to/test_project
  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [15 lines of output]
      Traceback (most recent call last):
        File "~/.local/pipx/shared/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
          main()
        File "~/.local/pipx/shared/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
        File "~/.local/pipx/shared/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 118, in get_requires_for_build_wheel
          return hook(config_settings)
        File "/tmp/pip-build-env-aafckipn/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 334, in get_requires_for_build_wheel
          return self._get_build_requires(config_settings, requirements=[])
        File "/tmp/pip-build-env-aafckipn/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 304, in _get_build_requires
          self.run_setup()
        File "/tmp/pip-build-env-aafckipn/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 320, in run_setup
          exec(code, locals())
        File "<string>", line 2, in <module>
      ModuleNotFoundError: No module named 'content'
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

By design setuptools.build_meta does not add the current workdir to sys.path. This change was introduced on purpose many years ago upon the adoption of PEP 517 to avoid common pitfalls of local files overwriting modules from the stdlib or other 3rd party libs.
If you wish to import local files please add the adequate statements (e.g. import sys; sys.path.insert(...) before the relevant imports in your setup.py script.

Is the change documented somewhere? It makes sense and I agree with it, but I can imagine many people being confused when they attempt to migrate from legacy setup.py-only build processes.

To be honest, it is actually a direct consequence of the way PEP 517 (so there was no real change in setuptools itself, just a decision to not add anything new given the previous rationale).
I.e., when you call python setup.py ... Python adds the current directory to sys.path. If you don't call that, then nothing is added to sys.path. Please feel free tonsend a PR If you would like this to be mentioned explicitly in the PRs (not sure if it already is...)

Ah, of course - the old workflows were all invoking it directly, and nobody would go out of their way to use -I or whatever, so naturally it would seem to work.
I guess what I'm really looking for is more detail in the modernization guide offered on packaging.python.org. If I figure out how to write it up effectively, I'll offer a PR over there instead.
Thanks for helping me think through it clearly.


Realtime demo on MMSKELETON project

Hi,
Does the old ST-GCN realtime demo recongition has been integrated on the mmskeleton project ?

Look forward !

Hi, Yes I made some modifications on the mmcv library, now I can train dataset with a model trained with the mmskeleton project and test the real time demo with the old project st-gcn.

@mejdidallel can you show us the modifications you have done ?

@mejdidallel can you show us the modifications you have done ?

@mejdidallel can you show us modification details on mmcv to run on real time? Did you use the st-gcn demo file trun on real time ?

Use this command to locate your mmcv package :
python -c "import mmcv as _; print(_.__path__)"
Then search for mmcv /runner/checkpoint.py and replace it with this :
# Copyright (c) Open-MMLab. All rights reserved.
import os
import os.path as osp
import pkgutil
import time
import warnings
from collections import OrderedDict
from importlib import import_module

import torch
import torchvision
from torch.utils import model_zoo

import mmcv
from .dist_utils import get_dist_info

open_mmlab_model_urls = {
    'vgg16_caffe': 'https://s3.ap-northeast-2.amazonaws.com/open-mmlab/pretrain/third_party/vgg16_caffe-292e1171.pth',  # noqa: E501
    'resnet50_caffe': 'https://s3.ap-northeast-2.amazonaws.com/open-mmlab/pretrain/third_party/resnet50_caffe-788b5fa3.pth',  # noqa: E501
    'resnet101_caffe': 'https://s3.ap-northeast-2.amazonaws.com/open-mmlab/pretrain/third_party/resnet101_caffe-3ad79236.pth',  # noqa: E501
    'resnext50_32x4d': 'https://s3.ap-northeast-2.amazonaws.com/open-mmlab/pretrain/third_party/resnext50-32x4d-0ab1a123.pth',  # noqa: E501
    'resnext101_32x4d': 'https://s3.ap-northeast-2.amazonaws.com/open-mmlab/pretrain/third_party/resnext101_32x4d-a5af3160.pth',  # noqa: E501
    'resnext101_64x4d': 'https://s3.ap-northeast-2.amazonaws.com/open-mmlab/pretrain/third_party/resnext101_64x4d-ee2c6f71.pth',  # noqa: E501
    'contrib/resnet50_gn': 'https://s3.ap-northeast-2.amazonaws.com/open-mmlab/pretrain/third_party/resnet50_gn_thangvubk-ad1730dd.pth',  # noqa: E501
    'detectron/resnet50_gn': 'https://s3.ap-northeast-2.amazonaws.com/open-mmlab/pretrain/third_party/resnet50_gn-9186a21c.pth',  # noqa: E501
    'detectron/resnet101_gn': 'https://s3.ap-northeast-2.amazonaws.com/open-mmlab/pretrain/third_party/resnet101_gn-cac0ab98.pth',  # noqa: E501
    'jhu/resnet50_gn_ws': 'https://s3.ap-northeast-2.amazonaws.com/open-mmlab/pretrain/third_party/resnet50_gn_ws-15beedd8.pth',  # noqa: E501
    'jhu/resnet101_gn_ws': 'https://s3.ap-northeast-2.amazonaws.com/open-mmlab/pretrain/third_party/resnet101_gn_ws-3e3c308c.pth',  # noqa: E501
    'jhu/resnext50_32x4d_gn_ws': 'https://s3.ap-northeast-2.amazonaws.com/open-mmlab/pretrain/third_party/resnext50_32x4d_gn_ws-0d87ac85.pth',  # noqa: E501
    'jhu/resnext101_32x4d_gn_ws': 'https://s3.ap-northeast-2.amazonaws.com/open-mmlab/pretrain/third_party/resnext101_32x4d_gn_ws-34ac1a9e.pth',  # noqa: E501
    'jhu/resnext50_32x4d_gn': 'https://s3.ap-northeast-2.amazonaws.com/open-mmlab/pretrain/third_party/resnext50_32x4d_gn-c7e8b754.pth',  # noqa: E501
    'jhu/resnext101_32x4d_gn': 'https://s3.ap-northeast-2.amazonaws.com/open-mmlab/pretrain/third_party/resnext101_32x4d_gn-ac3bb84e.pth',  # noqa: E501
    'msra/hrnetv2_w18': 'https://s3.ap-northeast-2.amazonaws.com/open-mmlab/pretrain/third_party/hrnetv2_w18-00eb2006.pth',  # noqa: E501
    'msra/hrnetv2_w32': 'https://s3.ap-northeast-2.amazonaws.com/open-mmlab/pretrain/third_party/hrnetv2_w32-dc9eeb4f.pth',  # noqa: E501
    'msra/hrnetv2_w40': 'https://s3.ap-northeast-2.amazonaws.com/open-mmlab/pretrain/third_party/hrnetv2_w40-ed0b031c.pth',  # noqa: E501
    'bninception_caffe': 'https://open-mmlab.s3.ap-northeast-2.amazonaws.com/pretrain/third_party/bn_inception_caffe-ed2e8665.pth',  # noqa: E501
    'kin400/i3d_r50_f32s2_k400': 'https://open-mmlab.s3.ap-northeast-2.amazonaws.com/pretrain/third_party/i3d_r50_f32s2_k400-2c57e077.pth',  # noqa: E501
    'kin400/nl3d_r50_f32s2_k400': 'https://open-mmlab.s3.ap-northeast-2.amazonaws.com/pretrain/third_party/nl3d_r50_f32s2_k400-fa7e7caa.pth',  # noqa: E501
}  # yapf: disable

def load_state_dict(module, state_dict, strict=False, logger=None):
    """Load state_dict to a module.

    This method is modified from :meth:`torch.nn.Module.load_state_dict`.
    Default value for ``strict`` is set to ``False`` and the message for
    param mismatch will be shown even if strict is False.

    Args:
        module (Module): Module that receives the state_dict.
        state_dict (OrderedDict): Weights.
        strict (bool): whether to strictly enforce that the keys
            in :attr:`state_dict` match the keys returned by this module's
            :meth:`~torch.nn.Module.state_dict` function. Default: ``False``.
        logger (:obj:`logging.Logger`, optional): Logger to log the error
            message. If not specified, print function will be used.
    """
    unexpected_keys = []
    all_missing_keys = []
    err_msg = []

    metadata = getattr(state_dict, '_metadata', None)
    state_dict = state_dict.copy()
    if metadata is not None:
        state_dict._metadata = metadata

    # use _load_from_state_dict to enable checkpoint version control
    def load(module, prefix=''):
        local_metadata = {} if metadata is None else metadata.get(
            prefix[:-1], {})
        module._load_from_state_dict(state_dict, prefix, local_metadata, True,
                                     all_missing_keys, unexpected_keys,
                                     err_msg)
        for name, child in module._modules.items():
            if child is not None:
                load(child, prefix + name + '.')

    load(module)
    load = None  # break load->load reference cycle

    # ignore "num_batches_tracked" of BN layers
    missing_keys = [
        key for key in all_missing_keys if 'num_batches_tracked' not in key
    ]

    if unexpected_keys:
        err_msg.append('unexpected key in source state_dict: {}\n'.format(
            ', '.join(unexpected_keys)))
    if missing_keys:
        err_msg.append('missing keys in source state_dict: {}\n'.format(
            ', '.join(missing_keys)))

    rank, _ = get_dist_info()
    if len(err_msg) > 0 and rank == 0:
        err_msg.insert(
            0, 'The model and loaded state dict do not match exactly\n')
        err_msg = '\n'.join(err_msg)
        if strict:
            raise RuntimeError(err_msg)
        elif logger is not None:
            logger.warning(err_msg)
        else:
            print(err_msg)

def load_url_dist(url):
    """ In distributed setting, this function only download checkpoint at
    local rank 0 """
    rank, world_size = get_dist_info()
    rank = int(os.environ.get('LOCAL_RANK', rank))
    if rank == 0:
        checkpoint = model_zoo.load_url(url)
    if world_size > 1:
        torch.distributed.barrier()
        if rank > 0:
            checkpoint = model_zoo.load_url(url)
    return checkpoint

def get_torchvision_models():
    model_urls = dict()
    for _, name, ispkg in pkgutil.walk_packages(torchvision.models.__path__):
        if ispkg:
            continue
        _zoo = import_module('torchvision.models.{}'.format(name))
        if hasattr(_zoo, 'model_urls'):
            _urls = getattr(_zoo, 'model_urls')
            model_urls.update(_urls)
    return model_urls

def load_checkpoint(model,
                    filename,
                    map_location=None,
                    strict=False,
                    logger=None):
    """Load checkpoint from a file or URI.

    Args:
        model (Module): Module to load checkpoint.
        filename (str): Either a filepath or URL or modelzoo://xxxxxxx.
        map_location (str): Same as :func:`torch.load`.
        strict (bool): Whether to allow different params for the model and
            checkpoint.
        logger (:mod:`logging.Logger` or None): The logger for error message.

    Returns:
        dict or OrderedDict: The loaded checkpoint.
    """
    # load checkpoint from modelzoo or file or url
    if filename.startswith('modelzoo://'):
        warnings.warn('The URL scheme of "modelzoo://" is deprecated, please '
                      'use "torchvision://" instead')
        model_urls = get_torchvision_models()
        model_name = filename[11:]
        checkpoint = load_url_dist(model_urls[model_name])
    elif filename.startswith('torchvision://'):
        model_urls = get_torchvision_models()
        model_name = filename[14:]
        checkpoint = load_url_dist(model_urls[model_name])
    elif filename.startswith('open-mmlab://'):
        model_name = filename[13:]
        checkpoint = load_url_dist(open_mmlab_model_urls[model_name])
    elif filename.startswith(('http://', 'https://')):
        checkpoint = load_url_dist(filename)
    else:
        if not osp.isfile(filename):
            raise IOError('{} is not a checkpoint file'.format(filename))
        checkpoint = torch.load(filename, map_location=map_location)
    # get state_dict from checkpoint
    if isinstance(checkpoint, OrderedDict):
        state_dict = checkpoint
    elif isinstance(checkpoint, dict) and 'state_dict' in checkpoint:
        state_dict = checkpoint['state_dict']
    else:
        raise RuntimeError(
            'No state_dict found in checkpoint file {}'.format(filename))
    # strip prefix of state_dict
    if list(state_dict.keys())[0].startswith('module.'):
        state_dict = {k[7:]: v for k, v in checkpoint['state_dict'].items()}
    # load state_dict
    if hasattr(model, 'module'):
        load_state_dict(model.module, state_dict, strict, logger)
    else:
        load_state_dict(model, state_dict, strict, logger)
    return checkpoint

def weights_to_cpu(state_dict):
    """Copy a model state_dict to cpu.

    Args:
        state_dict (OrderedDict): Model weights on GPU.

    Returns:
        OrderedDict: Model weights on GPU.
    """
    state_dict_cpu = OrderedDict()
    for key, val in state_dict.items():
        state_dict_cpu[key] = val.cpu()
    return state_dict_cpu

def save_checkpoint(model, filename, optimizer=None, meta=None):
    """Save checkpoint to file.

    The checkpoint will have 3 fields: ``meta``, ``state_dict`` and
    ``optimizer``. By default ``meta`` will contain version and time info.

    Args:
        model (Module): Module whose params are to be saved.
        filename (str): Checkpoint filename.
        optimizer (:obj:`Optimizer`, optional): Optimizer to be saved.
        meta (dict, optional): Metadata to be saved in checkpoint.
    """
    #if meta is None:
        #meta = {}
    #elif not isinstance(meta, dict):
        #raise TypeError('meta must be a dict or None, but got {}'.format(
            #type(meta)))
    #meta.update(mmcv_version=mmcv.__version__, time=time.asctime())

    mmcv.mkdir_or_exist(osp.dirname(filename))
    if hasattr(model, 'module'):
        model = model.module

    #checkpoint = {
        #'meta': meta,
        #'state_dict': weights_to_cpu(model.state_dict())
    #}
    #if optimizer is not None:
        #checkpoint['optimizer'] = optimizer.state_dict()

    #torch.save(checkpoint, filename)
    torch.save(model.state_dict(), filename)

Then re-train you model with your dataset (with the mmskeleton project) and then you can use the generated models in realtime recognition code of the ST-GCN project.

Thanks, i'm still getting "Can not find Openpose Python API." on the ST-GCN project while trying to run the demo even if BUILD_PYTHON flag is on I'm not seeing .so files.

Thanks, i'm still getting "Can not find Openpose Python API." on the ST-GCN project while trying to run the demo even if BUILD_PYTHON flag is on I'm not seeing .so files.

You have to correctly install the ST-GCN project first with the correct dependencies with all libraries including OpenPose.
Check the ST-GCN and Openpose documentations.

how to use demo_offline.py  to visualize using hrnet instead of using openpose API

@mejdidallel  how to use demo_offline.py to visualize using hrnet instead of using openpose API


Request Seam ID in player list

Can you please add steamID64 in player list?
Thank you for this amazing project.

I will need to check if I can get the id via RCON but I dont think so.
The playerlist is provided by rcon and I would somehow need to match the rcon entry to the ingame player entity. Dunno if thats possible in a reliable way

this has been added in v3.5.0


eslint.fix flag does not commit changes

When there are autofixable lint errors detected by eslint, the order of operations is:

arc diff
if there are uncommited changes, amend or commit
arc lint, which autofixes and returns successful
autofixed changes are not committed nor staged
diff is pushed to phabricator without changes

This means that arc diff fails to properly apply lint changes before creating a diff.
See messages for a file provided by<EMAIL_ADDRESS>        "messages": [
            {
                "ruleId": "prettier/prettier",
                "severity": 2,
                "message": "Replace `(flow.component·&&·flow.component.archived)` with `flow.component·&&·flow.component.archived`",
                "line": 61,
                "column": 10,
                "nodeType": null,
                "messageId": "replace",
                "endLine": 61,
                "endColumn": 53,
                "fix": {
                    "range": [
                        1462,
                        1505
                    ],
                    "text": "flow.component && flow.component.archived"
                }
            },
            {
                "ruleId": "prettier/prettier",
                "severity": 2,
                "message": "Insert `(⏎··········`",
                "line": 65,
                "column": 13,
                "nodeType": null,
                "messageId": "insert",
                "endLine": 65,
                "endColumn": 13,
                "fix": {
                    "range": [
                        1675,
                        1675
                    ],
                    "text": "(\n          "
                }
            },
            {
                "ruleId": "prettier/prettier",
                "severity": 2,
                "message": "Replace `⏎⏎⏎········` with `········)`",
                "line": 66,
                "column": 1,
                "nodeType": null,
                "messageId": "replace",
                "endLine": 69,
                "endColumn": 9,
                "fix": {
                    "range": [
                        1685,
                        1696
                    ],
                    "text": "        )"
                }
            }
        ],

Fixed by #79


Python Formatting

Description
Adds a step to the common PR workflow to run yapf over all python files.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: jwaldrop107:x: github-actions[bot]You have signed the CLA already but the status is still pending? Let us recheck it.

🚀 [bumpr] Bumped! New version:v0.3.42 Changes:v0.3.41...v0.3.42


[Map] Allow GeoJSON to be tappable

Builds on #782.
Next steps in fast-follow PR(s):

Parse GeoJSON at load time rather than rendering time.
Use on shared GeoJSONLayer rather than one for each feature. Set parsed geometries on that layer.
Let view model provide GeoJSON styles rather than setting them in MapAdapter.
View  model updates GeoJSON styles to reflect whether feature is selected.

@scolsen This PR includes your commits as well. PTAL?

@googlebot I consent.


Connect Wallet v2

@MananTank @iketw Some comments on usability, DX:

supportedWallets should not be mandatory in ThirdwebProvider the comment says it'll use some defaults but still requires me to pass something
useContract is should be re-exported in /react - don't make me import anythign from /react-core since it'll be an internal dependency
ConnectWallet requires a theme attribute? feel like we could also default to something (prob dark) and not require any props.
passing an empty array for supportedWallets results in a broken UI, should probably throw an error if empty array is passed explicitely
code comment for supportedWallets should give me a few examples of wallets to pass (helps ppl find the rest of them) - its also not obvious that i need to pass class names rather than instances

on Device wallet:

is the second login flow implemented? it always shows the "create" password screen for me

is the second login flow implemented? it always shows the "create" password screen for me

That's weird.


Refactor and updates

Overview
Closes https://gitlab.com/architect-io/product/-/issues/268
Changes

Updated starter project with new standards
Updated Dockerfiles
Added new DB service to component

Pictures
architect dev

architect dev --debug=false

architect deploy nextjs -e example-environment --auto-approve

TODO List
Before merging, a PR will need

[ ] Signoff from product if there are UX changes
[ ] Signoff from one other engineer at Architect.io

@ryan-cahill I'm getting an error when running this with --debug=false:
 => => writing image sha256:6c0ced5cae0474cb9e3722165e2cd80dd7549ffbf642374a412b968c60bbb851                                0.0s
 => => naming to docker.io/library/nextjs--app                                                                              0.0s
Building containers... done

Once the containers are running they will be accessible via the following urls:
https://app.localhost.architect.sh:443/ => nextjs--app

http://localhost:443/ => gateway:443
http://localhost:8080/ => gateway:8080
http://localhost:50000/ => nextjs--app:3000
http://localhost:50001/ => nextjs--database:5432

Starting containers...

nextjs--database-1  |
nextjs--database-1  |  PostgreSQL Database directory appears to contain a database; Skipping initialization
nextjs--database-1  |
gateway-1           |  time="2023-03-21T20:40:01Z" level=info msg="Configuration loaded from flags."
nextjs--app-1       |  Prisma schema loaded from prisma/schema.prisma
nextjs--app-1       |  Datasource "db": PostgreSQL database "default_db", schema "public" at "nextjs--database:5432"
nextjs--app-1       |
nextjs--app-1       |  1 migration found in prisma/migrations
nextjs--app-1       |
ERROR: nextjs.services.app has encountered an error and is being restarted.
d0e9c6f011bc21e5aa8ab4eda79fb97cbae25434585442bc5769ad24cbc41d95
nextjs--app-1       |  Prisma schema loaded from prisma/schema.prisma
nextjs--app-1       |  Datasource "db": PostgreSQL database "default_db", schema "public" at "nextjs--database:5432"
nextjs--app-1       |
nextjs--app-1       |  1 migration found in prisma/migrations
nextjs--app-1       |
ERROR: nextjs.services.app has encountered an error and is being restarted.
d0e9c6f011bc21e5aa8ab4eda79fb97cbae25434585442bc5769ad24cbc41d95

Update - I couldn't actually reproduce this... I think everything in this PR looks fine, we should see if @Zach-at-Arch or somebody else runs into this issue but things seem to be working just fine now


Add support for retry-after header

Add support for the Retry-After header to robust. I placed it behind an optional flag respect_retry_header, so default behaviour is unchanged. It's a bit of a niche feature, but it could be useful when using robust directly in an api client.
Supports both seconds and http date format of the header.
Includes tests.

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


TypeError: 'FilesIndex' object is not iterable

The code is from the tutorial 1.
Traceback (most recent call last):
File "C:/Users/Predator/Documents/WORK/Project_CT/radio/radio_tut1.py", line 17, in
batch = (luna_dataset >> preprocessing).next_batch(batch_size=2, shuffle=False)  # execution starts when you call next_batch
File "C:\Users\Predator\AppData\Local\Programs\Python\Python35\lib\site-packages\dataset\pipeline.py", line 1080, in next_batch
batch_res = next(self._batch_generator)
File "C:\Users\Predator\AppData\Local\Programs\Python\Python35\lib\site-packages\dataset\pipeline.py", line 1053, in gen_batch
batch_res = self.execute_for(batch)
File "C:\Users\Predator\AppData\Local\Programs\Python\Python35\lib\site-packages\dataset\pipeline.py", line 597, in execute_for
batch_res = self._exec_all_actions(batch)
File "C:\Users\Predator\AppData\Local\Programs\Python\Python35\lib\site-packages\dataset\pipeline.py", line 569, in _exec_all_actions
batch = self._exec_one_action(batch, _action, _action_args, _action['kwargs'])
File "C:\Users\Predator\AppData\Local\Programs\Python\Python35\lib\site-packages\dataset\pipeline.py", line 517, in _exec_one_action
batch = action_method(*args, **kwargs)
File "C:\Users\Predator\Documents\WORK\Project_CT\radio\radio\dataset\dataset\decorators.py", line 37, in _action_wrapper
_res = action_method(action_self, *args, **kwargs)
File "C:\Users\Predator\Documents\WORK\Project_CT\radio\radio\preprocessing\ct_batch.py", line 367, in load
self._load_raw()                # pylint: disable=no-value-for-parameter
File "C:\Users\Predator\Documents\WORK\Project_CT\radio\radio\preprocessing\ct_batch.py", line 541, in _load_raw
for patient_id in self.indices:
TypeError: 'FilesIndex' object is not iterable

Hey, @Screenium!
The issue looks familiar. Though, for better understanding, I would like to see the content of your Dataset-index. So, I suggest you run the following line (before calling next_batch):
luna_index.indices[:5]
and share the results with us.
Thanks for reporting this and other issues, truly appreciated.
Best,
Alex.

Hello, I switched to mac but still ahving this issue. I put the line before calling next_batch as suggested.
Here is the result:
 ['<IP_ADDRESS>.4.1.145<IP_ADDRESS>.6279.6001.105756658031515062000744821260'
 '<IP_ADDRESS>.4.1.145<IP_ADDRESS>.6279.6001.108197895896446896160048741492'
 '<IP_ADDRESS>.4.1.145<IP_ADDRESS>.6279.6001.117383608379722740629083782428'
 '<IP_ADDRESS>.4.1.145<IP_ADDRESS>.6279.6001.124663713663969377020085460568'
 '<IP_ADDRESS>.4.1.145<IP_ADDRESS>.6279.6001.126121460017257137098781143514']
Traceback (most recent call last):
  File "/Users/Welcome/Documents/work/radio/radio_tut1.py", line 19, in <module>
    batch = (luna_dataset >> preprocessing).next_batch(batch_size=2, shuffle=False)  # execution starts when you call next_batch
  File "/usr/local/lib/python3.6/site-packages/dataset/pipeline.py", line 1080, in next_batch
    batch_res = next(self._batch_generator)
  File "/usr/local/lib/python3.6/site-packages/dataset/pipeline.py", line 1053, in gen_batch
    batch_res = self.execute_for(batch)
  File "/usr/local/lib/python3.6/site-packages/dataset/pipeline.py", line 597, in execute_for
    batch_res = self._exec_all_actions(batch)
  File "/usr/local/lib/python3.6/site-packages/dataset/pipeline.py", line 569, in _exec_all_actions
    batch = self._exec_one_action(batch, _action, _action_args, _action['kwargs'])
  File "/usr/local/lib/python3.6/site-packages/dataset/pipeline.py", line 517, in _exec_one_action
    batch = action_method(*args, **kwargs)
  File "/Users/Welcome/Documents/work/radio/radio/dataset/dataset/decorators.py", line 37, in _action_wrapper
    _res = action_method(action_self, *args, **kwargs)
  File "/Users/Welcome/Documents/work/radio/radio/preprocessing/ct_batch.py", line 367, in load
    self._load_raw()                # pylint: disable=no-value-for-parameter
  File "/Users/Welcome/Documents/work/radio/radio/preprocessing/ct_batch.py", line 541, in _load_raw
    for patient_id in self.indices:
TypeError: 'FilesIndex' object is not iterable

Process finished with exit code 1

Closing as non-active for a long time.
Feel free to reopen if the problem persists.


Support dots in Kind name segment

CDI spec states that dots ('.') are allowed.
Add support in parser for this.
reference: https://github.com/container-orchestrated-devices/container-device-interface/blob/main/SPEC.md#kind

Ack, thx for the feedback will work on adding the needed check & test

One thing that would also have to be added is that we would have to add a check and an associated test to the requiresV060 function in pkg/cdi/version.go. This would mean that a lower spec version can be generated if a non-dotted class name is used to ensure better compatibility with our clients.

Done.

lgtm
@klihub @bart0sh any comments here?

Thanks for the contribution @adrianchiris. I don't think I have an issue with this.
@klihub can you recall why dots were not included in device classes when this was initially implemented?

I think it was simply considered unnecessary (there was no immediate need for it) and cleaner to only accept dots in the vendor part where it was clearly needed (to allow for DNS-like vendor prefixes).

lgtm
@klihub @bart0sh any comments here?

I have no strong feelings. I don't see why dots would be necessary or useful in the class part. But you guys have been playing around much more with generating Specs and if you say it is necessary/helpful, then I believe you.

CDI spec states that dots ('.') are allowed.
Add support in parser for this.

A much simpler alternative would have been to simply update the specs reflect the implementation, stating that dots are only allowed in the vendor prefix. But as I said, no strong feelings and I'm fine with this.

Some context on what triggered me to add this:
im working on adding support via DRA to network resources for NVIDIA NICs
part of this work is to generate CDI files for container runtime to consume.
there may be different network device types (e.g SRIOV virtual function[1] and subfunctions[2])
So, i was thinking on how to name them in CDI.
as spec allowed dots in name segment this is what I had in mind for the kind string (and its what triggered this PR):
nvidia.com/net.sf=<SF device name>
nvidia.com/net.vf=<VF device address>

since the first part usually refers to the vendor (at least that what i got from the examples within CDI proj)
it made sense to me to chose the kind it as above.
some alternatives that conform to current implementation (without dot in name segment)
nvidia.com/net-sf=<SF device name>
nvidia.com/net-vf=<VF device address>

net.nvidia.com/sf=<SF device name>
net.nvidia.com/vf=<VF device address>

[1] https://en.wikipedia.org/wiki/Single-root_input/output_virtualization
[2] https://docs.kernel.org/networking/devlink/devlink-port.html#subfunction

I discussed this briefly in the COD Working group meeting, and have no objections. This also aligns with what is allowed for Kubernetes Annotations and does provide more flexibility from a user perspective.

Thanks for bringing this one up for discussion.

@adrianchiris as a last update, please add a line to the "Released versions of the spec are available as Git tags." table in SPEC.md that indicates this relaxation. (I think we're missing one for the relaxation of the device name too which I will add in a follow-up).

Done. updated v0.6.0 entry. LMK if wording is OK

@elezar Isn't it a backward-incompatible change? If it is we should consider incrementing spec version.

@elezar Isn't it a backward-incompatible change? If it is we should consider incrementing spec version.

It is backward incompatible, but we haven't released (tagged) the v0.6.0 spec yet. Meaning that this change would be included as part of the v0.6.0 spec once we tag that.

@bart0sh @elezar we OK with merging this one ?


v11 - Trace in RELEASE build

Some Trace is being logged in TMPE.log in the v11 RELEASE build (built from master branch prior to #451 merge).
Trace<PHONE_NUMBER>: TrafficLightSimulation.Update(): called for node 2623
Trace<PHONE_NUMBER>: TrafficLightSimulation.Update(): called for node 4469
Trace<PHONE_NUMBER>: TrafficLightSimulation.Update(): called for node 5008
Trace<PHONE_NUMBER>: TrafficLightSimulation.Update(): called for node 9883

Full log, savegame, etc., can be found in #452

I'll pay Log class a visit in a couple hours

Still have the trace showing up in my log file. Don't know if the V11 alpha 6 has been updated to remove those... and for information, all of those traces are for traffic lights that have more than one node to control.
TMPE.log

Sub-project TMPE.API has TRACE defined for Release build.

I'll push another v11 alpha to workshop tonight with the trace removed. Thanks @kvakvs and thanks for the report @TLHeart60 👍


Livepeer dStorage playback demo application

An demo application that illustrates how Livepeer can be used as a video processing and caching layer on top of content available on IPFS or the Arweave gateway for optimized video playback.
Demo: https://livepeer-dstorage.vercel.app/
GitHub Repo: https://github.com/suhailkakar/livepeer-dStorage-playback


Fix leaks by disposing in detach

Description
Fixes #14204.
Tests
I have no idea how to test this - if the existing tests pass, this should be fine I reckon. However, I can add tests if someone can tell me how this would be done.
Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change

[x] No, no existing tests failed, so this is not a breaking change.

cc @Hixie
Can you help me regarding tests?

Cannot fix the leaks for _RenderSlider because the Slider has correct animations when reparented test in test/material/slider_test.dart prevents disposing the gesture recognizers..

It turns out that these aren't actually leaks... with the animation controller, the render object starts and stops it on attach/detach already, so there's no resources being consumed while the render object is detached.
For the gesture recognizer it's a bit trickier. We can't do the solution that's in this PR because attach/detach happens during GlobalKey reparenting, so for example if someone were to move the Slider around the tree while the finger was down, it would cancel the gesture if we were to dispose/recreate the recognizers.
On the other hand, it's true that currently if you put your finger down, then the Slider goes away, then keep moving your finger, the recognizer will keep tracking the finger and may even interfere with the arena if there's other things trying to win the gesture. I'm not 100% sure how to address this, gesture recognizers aren't really compatible with the render object lifecycle... we could move the objects up to the State I guess, and pass them down to the _CupertinoSwitchRenderObjectWidget. The State's dispose is not called during GlobalKey reparenting. The other option would be to add a dispose method to _CupertinoSwitchRenderObjectWidget and have the RenderObjectWidget call it, as a special case for Slider.
We should audit the rest of the code to see if other widgets have these problems.

BTW, re tests: leaks in AnimationControllers are already checked for by the test framework, so to see if there's a leak you need but instantiate and initialize the leaking owner (the render object here, attaching it being the key), then dispose of it (detach it, in the case of render objects). For GestureRecognizers there's no particularly good way to test if they've been disposed currently. I would focus on what the symptoms would be... e.g. attach the render object (pump a frame with the widget), start a gesture but don't complete it, detach the render object (pump a new frame with different widgets), and see if the pointer router table is empty, or the gesture arenas are empty, or something like that.


NME: Add a CSS class to all graph nodes

Use this new feature to widen the LightInformationBlock.
Closes #13329

Just a quick question, does that not conflict on any of the current nodes ? (I guess not cause in the worst case, it might double the className I believe)

Just a quick question, does that not conflict on any of the current nodes ? (I guess not cause in the worst case, it might double the className I believe)

As I could see it, there is only a single CSS class added to graph node elements, and this class seems to be auto-generated as it is not readable at all:

The PR is adding a second class on the main div element of the node at creation time, so I don't think it's possible to have this class doubled(?)

I added @RaananW and @carolhmj for review :-)


Xenos should be able to drag some dead bodies

Description

Specifically they have to be over stage 1 infected and not rotted. Parity.
Reproduction

Screenshots

Additional context

Does this include nesting

This doesn't work in CM because it never returns true after checking...oh well


About   Charset (Encoding)

How to get a file's Charset?
How to convert utf-8 to gbk,or gbk to utf-8?

@huangsir0
if you need assistance with your code please follow the link I want help writing my application in the new issue menu

@huangsir0
Please use above referred link to get more help on your question.
Closing from here.


security & IOT Questions analysis from stack overflow

resolves #345

@sanjay-kv please merge this pr along with level and label of gssoc'24

Hi i appreciate the contribution, but there is something wrong, you creating another library structure. i can see that, we need to merge into the existing streamlit


Encryptions of access and refresh tokens

The document mentions that the tokens are encrypted and stored, Can you provide more details on this part?
like which algorithm is used for encryptions, where are tokens stored,  and where the keys are stored.

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

ID: 54a92920-3605-2fe5-65e0-e295505bd3e2
Version Independent ID: 604fab7b-7654-e46a-1d5a-30d1afd97acd
Content: About API authorizations in Azure API Management
Content Source: articles/api-management/authorizations-overview.md
Service: api-management
GitHub Login: @dlepow
Microsoft Alias: danlep

@utkarsh24 Thanks for reaching out to us and reporting this issue. The documentation mentions the below:

The access token and other authorization secrets (for example, client secrets) are encrypted with an envelope encryption and stored in an internal, multitenant storage. The data are encrypted with AES-128 using a key that is unique per data. Those keys are encrypted asymmetrically with a master certificate stored in Azure Key Vault and rotated every month.

Hope this answers.

@utkarsh24 I hope you had a chance to look at the above comment. If you have any follow-up questions or need any further assistance on this issue, please feel free to reopen this thread. I would be happy to help.


We lose the bazel server way too easily

Every time I run bazel run //metropolis:launch-cluster inside the Nix shell, Bazel appears to launch a brand-new server and recompute the entire graph, which is really expensive:
$ bazel run //metropolis:launch-cluster
IntelliJ found at /home/leopold/.local/share/JetBrains/IntelliJIdea2024.1, aspect repository already patched.
Starting local Bazel server and connecting to it...
Computing main repo mapping:
    Fetching repository @@crate_index; Restarting. 10s
    Fetching repository @@rust_linux_x86_64__x86_64-unknown-linux-gnu__stable_tools; starting 10s

I looked into this at one point, most likely a namespace-related issue (fhsenv uses that). I couldn't fix it in reasonable time so I just moved my devenv into the nix-shell (with remote extension host).


Update on connections settings

Hey there!
Is there an ETA on updating dbtcloud_connection and dbtcloud_project_connection? Since no so long ago connections were moved into the account object now? I'm down to work on this if needed

After testing I realized this is not an enhancement anymore but an issue. Connections can't be created with the latest version of the provider. The schema
POST url: https://cloud.getdbt.com/api/v3/accounts/4954/projects/377492/connections/, status: 400, body: {"status":{"code":400,"is_success":false,"user_message":"The request was invalid. Please double check the provided data and try again.","developer_message":""}

I am actively working on allowing global connections in TF. For that we will add a connection_id to dbtcloud_environment and create a new dbtcloud_global_connection resource.
But what surprises me is that the current resources should still work though.
Could you share a sanitized version of the resource that is failing?

Awesome! And thanks for the quick reply
Here you go:
resource "dbtcloud_connection" "snowflake" {
  project_id = dbtcloud_project.project.id
  account    = "xxx"
  type       = "snowflake"
  name       = "Snowflake Connection"
  database   = "xxx"
}

resource "dbtcloud_project_connection" "snowfalke_connection" {
  project_id    = dbtcloud_project.project_data_privacy.id
  connection_id = dbtcloud_connection.snowflake.connection_id
}

Sanitized error log here:
dbtcloud_connection.snowflake: Creating...
╷
│ Error: POST url: https://cloud.getdbt.com/api/v3/accounts/xxx/projects/xxx/connections/, status: 400, body: {"status":{"code":400,"is_success":false,"user_message":"The request was invalid. Please double check the provided data and try again.","developer_message":""},"data":{"details":"{'account': 'xxxx, 'database': 'xxx', 'allow_sso': False, 'client_session_keep_alive': False} is not valid under any of the given schemas"},"extra":{},"error_code":null}
│
│   with dbtcloud_connection.snowflake,
│   on connections.tf line 2, in resource "dbtcloud_connection" "snowflake":
│    2: resource "dbtcloud_connection" "snowflake" {
│
╵
❌  exit status 1

From DBT's API spec it looks like that endpoint is not there anymore. Closest to that one would be:
https://docs.getdbt.com/dbt-cloud/api-v3#/operations/Create Account Connection

OK, so I think that this is not related to the global connection change but you are just missing the warehouse config for the data warehouse. Could you try again and let me know?
The new dbtcloud_global_connection resource that will be released soon should be better at highlighting missing mandatory fields.

The "old" endpoints are removed from the docs as people should be using the new global/account connection going forward, but they should still work from the time being.

Confirmed!! Adding the warehouse fixed it. The provider docs didn't mention it as mandatory so I just skiped it. Thanks for the support!


support win32 thread

support multithread on the windows.

Hello @wesley-wu , thank you for this PR, please give me some time to review and evaluate. Thank you.

Hi, @MichalPrincNXP
I will try to setup win32 build testing as another PR.

Hello @MichalPrincNXP, It's OK, you can route the win32 port issues to me.

hello @MichalPrincNXP

hello @MichalPrincNXP , It's OK, you can route the win32 port issues to me.

hello @MichalPrincNXP , It's OK, you can route the win32 port issues to me.
I will try to setup win32 build testing on another PR.

OK, great!


improve error handling on containerised flash tools and fix jetson flash image name

@balena-ci I self-certify!


Using ruff for code formatting

Thank you for submitting this pull request! We appreciate you spending the time to work on these changes.
What is the motivation?

Using ruff to check code formatting.
Type of Change

[x] 📚 Examples / docs / tutorials / dependencies update

What does this change do?

Adds ruff to the Poetry dev-dependencies.
Adds ruff to pre-commit.
Changes docstrings to Google style.

ruff selects include:

Flake8
Isort
PEP8 naming
pydocstyle
PyUpgrade

What is your testing strategy?

Two main commands for ruff:
poetry run ruff check .
and for automated fixing:
poetry run ruff --fix .
Is this related to any issues?

#41 - This PR only implements the ruff part of this issue. A separate PR should be made for mypy.
Have you read the Contributing Guidelines?

[x] I have read the Contributing Guidelines

I didn't realise, this PR fixes https://github.com/surrealdb/surrealdb.py/pull/36, too.


The problem with maintainability

🙏 Thank you for this awesome React boilerplate 🙏
Is your feature request related to a problem? Please describe.
There was a problem of maintaining (keeping track of the boilerplate changes) the project created with react-boilerplate, which was fixed to some extent by moving it to cra-template. Now, webpack and babel configs can be updated just by updating the react-script package.
For the rest of the configurations and generators, it is still difficult to update them by checking the changelog and diff.
Describe the solution you'd like
Move the project to a mono repo. Create separate packages for the generators, utils and configs (ESLINT, PREETIER etc). It will be very easy to upgrade to a new version of boilerplate this way.
Describe alternatives you've considered
Additional context

I dont follow much. Utils and App folders are tightly coupled. There was never a case you need to modify just one without touching other. What good would it do to make them separate packages? Configs and lint prettier etc they hardly ever change anyway.
It can be never easy to just 'upgrade' to new version, sorry I have given lot of thoughts about it and still no good trade-off.
A monorepo would be way harder to maintain and keep it simple.

I will try to share a PR by this week

just don't touch this shitty boilerplate, like me you are in to wasting your time, that's already precious.

@Can-Sahin this is still WIP. https://github.com/sabarnix/react-boilerplate-cra-template/tree/wip. See if you like the idea.

thanks will check it out.

Ok I checked it and its pretty much what I imagined at first. But still struggling to understand how helpful would this be?

dotfiles folder almost NEVER change. The last change was probably 2 years ago.
internals file will be updated mostly 80% of the time when template folder is updated. Starting templates and  generators are directly targeting the real app so its highly coupled. Meaning, almost all the releases will have 2 release with first template package and then internals package.

This monorepo would only come handy when doing version bumps for template package because you cant just bump the version of react there and dont touch other packages.
I like the idea actually but I feel like the components of the boilerplate is all together single unit of work. There are not really independent pieces isolated from others. I have several future plans like switching to redux toolkit query and ditching redux-saga etc but considering those with this monorepo approach I have to 3 releases (each package) and it would even complicate things for everybody willing to update


{api,http}: add HTTP 418 to list of named response status codes

Signed-off-by: Chet Nichols III<EMAIL_ADDRESS>Commit Message: {api,http}: add HTTP 418 to list of named response status codes
Additional Details: While not officially part of RFC 2616, and intended as an April Fools' joke (per RFC 2324 and RFC 7168), the HTTP 418 I'm a teapot response code has enough historical popularity to be listed in certain locations as a [recognized] HTTP status code, with even Google tossing in their own easter egg.
And while IANA might not formally list 418 with a purpose (and instead lists it as "Unused"), they do go as far as saying it is effectively reserved for said purpose:
15.5.19.  418 (Unused)

   [RFC2324] was an April 1 RFC that lampooned the various ways HTTP was
   abused; one such abuse was the definition of an application-specific
   418 status code.  In the intervening years, this status code has been
   widely implemented as an "Easter Egg", and therefore is effectively
   consumed by this use.

All said, I've been using HTTP 418 throughout much of my career for various reasons (both production and internal testing), and noticed it was not listed in the StatusCode enum when working on some go-control-plane efforts today.
I am hoping to rectify that with support from the community.
Risk Level: Low
Testing: Leveraged existing unit tests
Docs Changes: None
Release Notes: None
Platform Specific Features: None

@wrowe do you approve of teapots? :)

the methods BREW, WHEN - should these be added to MethodValues as well?

I agree if we are going to support the RFC, we need this as well.

@chet please consider my suggestions, and merge main to try and clear the CodeQL hiccup. Thanks!

Outside of completing the 2324 implementation, let's not consider RFC 7168 in-scope for this PR. There is significant debate over whether international considerations were appropriately applied to cha. c.f. https://thelanguagenerds.com/2019/tea-if-by-sea-cha-if-by-land-why-the-world-had-only-two-words-for-tea/

/wait

/retest

These look like typical testing regressions that have nothing to do with your submission, rechecking to be sure
@chet are you willing to add the BREW and WHEN methods to complete the RFC's API?

/wait


comply with votifier protocol

send the mod version in the reply header to match the votifier protocol
fixes #16

Thanks!


PackageManager: Use {} for an empty function body

Because Martin likes that better than "= Unit" :-)
Signed-off-by: Sebastian Schuberth<EMAIL_ADDRESS>

This change is

Merging without another review as the commit was just rebased.


Refactor Flink Segment Writer to use File-based Segment writer

Codecov Report

Merging #8312 (93d2226) into master (e41d86b) will increase coverage by 14.73%.
The diff coverage is 18.18%.

@@              Coverage Diff              @@
##             master    #8312       +/-   ##
=============================================
+ Coverage     14.18%   28.92%   +14.73%
=============================================
  Files          1586     1623       +37
  Lines         83608    85306     +1698
  Branches      12674    12859      +185
=============================================
+ Hits          11862    24676    +12814
+ Misses        70864    58364    -12500
- Partials        882     2266     +1384

Flag
Coverage Δ

integration1
28.92% <18.18%> (?)

unittests2
?

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

Impacted Files
Coverage Δ

...ot/spi/ingestion/segment/writer/SegmentWriter.java
0.00% <0.00%> (ø)

...egmentwriter/filebased/FileBasedSegmentWriter.java
83.56% <28.57%> (-13.78%)
:arrow_down:

...pinot/controller/recommender/io/ConfigManager.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...troller/recommender/io/metadata/FieldMetadata.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...roller/recommender/rules/impl/BloomFilterRule.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...oller/api/resources/PinotControllerAppConfigs.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ler/recommender/data/generator/BytesGenerator.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...er/recommender/io/metadata/SchemaWithMetaData.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...r/recommender/rules/impl/AggregateMetricsRule.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../recommender/exceptions/InvalidInputException.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 800 more

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 e41d86b...93d2226. Read the comment docs.


changed action regulation to process regulation per #178

@cameronmore The definitions of permits, prohibits, requires, and their inverses still reference regulating the involvement of an agent in each process, which means these definitions still are describing acts rather than processes per se. I recommend this language also be removed so that, for instance, mere processes not obviously involving agents as causal actors (e.g. the states of affairs of a cyber system) may be prohibited, permitted, and required.

@cameronmore MRO is built automatically as part of the release process. There is no need to make changes manually, unless you can guarantee that every change to any property in CCO from v1.5 to v1.6 is accounted for.


[Windows version] pulmonary vein clipping plate parallel to axial slices

Prerequisites
Put an X between the brackets on the following lines if you have done all of the following:

[X] Checked the wiki board for common solutions: https://github.com/CemrgAppDevelopers/CemrgApp/wiki
[X] Checked that your issue isn't already filed: https://github.com/search?q=+is%3Aissue+user%3ACemrgAppDevelopers
[X] Are you providing the log? https://github.com/CemrgAppDevelopers/CemrgApp/wiki/Troubleshooting

Description
When Display Clippers button is clicked, the displayed clipping plate seems to be parallel to axial slice image planes.

Steps to Reproduce

Loading segmentation.
Loading segmentation.
Combine segmentation.
PV Clipper.
Unpick Flip Lines.
Find Centreliens.
Display Clippers.

Expected behaviour:
Clipping plate is expected to be perpendicular to the traced centre line.
Actual behavior:
Clipping plate seems to be parallel to axial slice image planes.
Reproduces how often:
All time.
Additional Information
NVIDIA driver is updated to the latest version.
´´´

After running some tests, it seems to be a problem with the view renderer in Windows. The automatic clipper itself does the job even if the clippers look completely wrong.
However, from a visual point of view, it's difficult to assess the clippers if they're not oriented correctly.
the actual problem comes in the CemrgAtriaClipper module, probably in function CalcParamsOfPlane(...). the normals are not being calculated or not being stored properly.
Workarounds

Use the manual (green) clippers (press the arrow keys when selecting a clipper).
trust the automatic (yellow) clippers and just carry on pressing the next button (again, the clippers are fine, they're just displayed weirdly).

Tasks

Perhaps provide the option to clip automatically (as in the scar plugin)

For workarounds:

When pressing the arrow keys, the clipper disappears.
The automatic clippers do not work. Correct display of the clipper is important especially when the segmentation of veins are connected.

Pushed commit 4b5b14e with fix. Theproblem was on the CemrgAtriaClipper class.  For some reason, windows wasn't storing the correct values in the correct x_s, y_s, z_s  variables, so I changed it from:
x_s = line->GetPoint(position+1)[0] - line->GetPoint(position)[0];
y_s = line->GetPoint(position+1)[1] - line->GetPoint(position)[1];
z_s = line->GetPoint(position+1)[2] - line->GetPoint(position)[2];

to
double pAtPos[3], pAtPosPP[3];
line->GetPoint(position+1, pAtPosPP);
line->GetPoint(position, pAtPos);
x_s = pAtPosPP[0] - pAtPos[0];
y_s = pAtPosPP[1] - pAtPos[1];
z_s = pAtPosPP[2] - pAtPos[2];


feat(cli): support -l flag for template list. Fixes #13309

Fixes #13309
Motivation

support -l selector param for template client
Modifications

support -l selector param for template client

Verification

run locally and tested

For future reference, please use text of logs instead of screenshots as text is much more accessible

Also looks like you were using commands/list.go as a reference instead of commands/cron/list.go, right? That would be helpful to know in your PR description as it makes reviewing easier.


page-ranges doesn't check syntax

If I try a command like
lp doc.pdf -o page-ranges=8-

then Cups will print the whole document. It should either print pages 8 onward, or tell me that I've made a syntax error in the page specification. Any other behavior is a waste of my time and my paper and toner.

What OS/Linux distribution?
What version of CUPS?

Arch Linux
Linux 4.9.18-1-lts #1 SMP x86_64 GNU/Linux
cups 2.2.2-2
Let me know if you need more help reproducing this. I had almost thought it would be a known bug. It seems much more general, indeed if I execute
lp -o blah t.pdf

then the file is printed with no error, even though "blah" is not a recognized option (is it?). I will be surprised if this is system-specific.

It is, indeed, OS-specific. The bug lies in the print filters used on Linux, specifically the cups-filters package. Please report this to the openprinting folks.
(the "blah" option is just silently ignored...)

Can you give me a URL for that?

Here is the bug tracker for cups-filters:
https://bugs.launchpad.net/ubuntu/+source/cups-filters

Thanks. Reported here: https://bugs.launchpad.net/ubuntu/+source/cups-filters/+bug/1689948


Update README.md

Codecov Report

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

@@           Coverage Diff           @@
##           master       #9   +/-   ##
=======================================
  Coverage   88.49%   88.49%
=======================================
  Files           7        7
  Lines         704      704
=======================================
  Hits          623      623
  Misses         81       81

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 5d7032f...2c7cdb9. Read the comment docs.


fix: updated deps

:tada: This PR is included in version 1.13.108 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


StringIndexOutOfBoundsException when building project

I get:
java.lang.StringIndexOutOfBoundsException: String index out of range: 1 at org.parceler.internal.ParcelableAnalysis.getPropertyName(ParcelableAnalysis.java:726)
when building my project. This is what's at line 726:
 return name.substring(0, 1).toLowerCase(Locale.getDefault()) + name.substring(1);
I get no trace of which one of my methods is the culpit. Still investigating.

One of my @Parcel(Parcel.Serialization.BEAN) annotated methods had a get() method. Seems that adding @Transient to it fixed the issue for now.


Adds a pool controller

This PR add a controller to the pool, which allows one to send transactions to the pool.
The new command is of form:
memcoin [--config CONFIG] pool add --key KEY [{--args KEY --args VALUE} ...] [--nonce NONCE]

Pull Request Test Coverage Report for Build 530935666

113 of 117 (96.58%)  changed or added relevant lines in 6 files are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.04%) to 99.102%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core/ordering/cosipbft/pbft/mod.go
2
6
33.33%

Files with Coverage Reduction
New Missed Lines
%

core/ordering/pow/block.go
1
98.91%

mino/proxy/http/mod.go
5
93.48%

Totals

Change from base Build 435513261:
0.04%

Covered Lines:
12143

Relevant Lines:
12253

💛  - Coveralls


Fix: Missing defaultMeta in log entry while using Profiler

fixes #1665

Since the logger instance is the thing actually doing the "write" operation (sorta) I feel it most appropriate to let it also run the "_addDefaultMetadata" logic instead of redefining it on the Profiler class. As such I've implemented this fix with the fixes for the overarching metadata issues in #1989
Ref: #2029

@HonzaMac Are you alright with me closing this and favoring the overarching implementation I've been working through in the above mentioned PR?

This could be closed in favour od better implemention you work on.


[6.0.0][1.11.2] UseItemStackEvent.Start and LaunchProjectileEvent Not Functioning

Sponge Build: spongeforge-1.11.2-2227-6.0.0-BETA-2159
Sponge API Build: 6.0.0-20170217.091623-41
Forge Build: forge-1.11.2-<IP_ADDRESS>7-universal
Java Version: 1.8.0_121-b13
Hello, I have the following listeners that do not function at all. Unless I'm doing something wrong, I would like to know if this can be looked at. Thanks!
@Listener
public void onPlayerConsumption(UseItemStackEvent.Start event) {
event.setCancelled(true);
}
@Listener
public void onPotionThrow(LaunchProjectileEvent event) {
event.setCancelled(true);
}

are you registering the event listener?

The listeners are registered and I've updated to spongeforge-1.10.2-2281-5.2.0-BETA-2401, but the issues are still present to this day. My issue was originally posted here months ago, but was told to re-post it here.

I can also confirm that neither TargetProjectileEvent nor any of its children are fired using
spongevanilla-1.12-7.0.0-BETA-290
spongeapi-7.0.0-20170703.084035-18

I can confirm that LaunchProjectile event did not fire for players, skeletons, or dispensers shooting arrows on SpongeForge Bleeding 7789e67bbd1ef2ed2411d08d23c1ea006ef7aa56
by using https://gist.github.com/ryantheleach/2152cebe0e3c90874c39e69566df4b66

Bumping up report.

still an issue with https://github.com/SpongePowered/SpongeCommon/commit/caccfa4f325396c866bb803d51ee0c3763e8d073 tested both LaunchProjectileEvent & UseItemStackEvent.Start

@ImMorpheus
Want to work on LaunchProjectileEvent?

@Zidane Oh, I started working on LaunchProjectileEvent, but I will need some time to do it properly (modded support mostly)

https://github.com/SpongePowered/SpongeCommon/pull/1769

PR merged


Add ability to change font colors for headers

For my game's credit screen I wanted the titles to be different colours than the text
I used vscode to copy over my changes, not realizing it would trim all the extra whitespace, sorry about that :)
Your little library is really great

Thanks! this is a great addition! I didn't get a chance to test it but viewing the code everything seems fine :)

haha I guess you could test it @ https://halkeye.itch.io/not-the-plants
Thanks for merging this! Love the library.
I was able to do scrolling credits with it - https://github.com/halkeye/AlienAttackGodot/blob/main/scenes/menus/credits.gd


Option to display logo instead of title in nav bar

@gcushen thanks for this great theme.  I am migrating my research lab website from Wordpress to blogdown and found the academic theme perfect for what I need. I would like to display the mini logo at the top left (as in lithium theme) instead of the title page. Is this implemented? thoughts?

Thanks for the feedback. This option is planned and we hope to add it within the next couple of weeks. In the meantime, you can edit the template directly to include your logo.

Done.
To utilize this feature, update Academic to latest version (following the Release Notes) and add logo = "logo.png" to [params] config, assuming a file named logo.png is placed in static/img/.

Done.
To utilize this feature, update Academic to latest version (following the Release Notes) and add logo = "logo.png" to [params] config, assuming a file named logo.png is placed in static/img/.

Could you please describe with more details where (and in which file) exactly put the logo = logo.png-command and the file? Thanks.


feat: move observe-content to cdk

Moves the observe content directive to the CDK package.

Caretaker note: will need a build rule


WebSocket.read_message is too complex (31) (MC0001)

Codacy detected an issue:
Message: WebSocket.read_message is too complex (31) (MC0001)
Occurred on:

Commit:  74422ce03601b774f905220e64d3fe98e3185de9
File:    wsocket.py
LineNum: 321
Code:    def read_message(self):

Currently on:

Commit:  9bf919181dc6868a72b426a6b29f1b821b7c21b3
File:    wsocket.py
LineNum: 319

Stale issue message

Stale issue message


Update setup.sh

See changes below:

Fix an error [!] when the user tries to run the following command: pod install
Remove log functions.
It's better use for loops instead if's

[!] Your Podfile requires that the plugin `cocoapods-acknowledgements` be installed.
Please install it and try installation again.

O CI falhou por este motivo:
Error: An unsatisfied requirement failed this build.
The command "brew upgrade" failed and exited with 1 during.

Não é um problema relacionado ao PR.


az webapp missing in 2.0.46

Hi,
After installing the latest version of the cli (2.0.46) the az webapp funcion is missing. The debugging reports that its not compatible. For the windows I managed to install 2.0.45 and that works, biut for mac OS I cannot find how to install the previous version.
Regards,
Robin

please reactivate with traces for me to take a look


Rendering issue with modal

The first, is when using the reveal modal.  When I first load the page, the first time I call the reveal modal, the modal appears to slide up, rather than down and ends up so that I cannot see anything except the bottom of the modal itself.  This is on chrome, and seems to be rather consistent.  Subsequent calls to the modal seems to bring it back to functioning the way I would expect.
I don't have the site live yet, so giving access to it isn't really much of an option, however I did send a screenshot in via email.

If we didn't tackle this via email please let us know.


Can't use as slave bound to localhost

Why is there a check that prevents using a slave that also binds to localhost: https://github.com/arioch/puppet-redis/blob/master/manifests/init.pp#L251?
I set up an instance with this configuration and it works fine.

I think this was because before you couldn't have multiple redis instances on one host with this module, so it made sense then. But as you say, this isn't neccsary now 👍


Slack + Discord bot

new endpoint and share validation and request logic with existing netlify function

Can you share more information?
I've done discord bots with nodejs before but never embedded into a web application.

It would need you to investigate hosting a discord bot on Netlify Functions @alhassanv. Netlify Functions are the same as AWS Lambda, if that helps. I will change this issue into Discord bot and make a separate Slack bot issue as that is the easier one.

can i make an bot for this purpose. i have some code which works: hacktoberbot

@lukeocodes I might be able to do this 🤔

Closing this because it appears it is not possible using Netlify Functions.


Update README.md

Add note about the project not being maintained anymore.

Proposed text: Note: After the conclusion of the IMLS project 2021, this repository is not longer actively maintained by the M-Lab team. Community contributions are welcome.

2: Placed after the paragraph describing the grant.

Since archiving, we can take out note about community contributions.

Done!


The submap tips over

I am working with a tof camera and my problem is this:

Then this happens:

I have looked at the imu (rostopic echo /imu) and I couldn't see any changes in there. So I think there is no problem with the imu or the odometry.
This is my lua:
include "map_builder.lua"
include "trajectory_builder.lua"

options = {
  map_builder = MAP_BUILDER,
  trajectory_builder = TRAJECTORY_BUILDER,
  map_frame = "map",
  tracking_frame = "imu",

  published_frame = "odom",
  odom_frame = "odom",
  provide_odom_frame = false,
  publish_frame_projected_to_2d = false,
  use_odometry = true,
  use_nav_sat = false,
  use_landmarks = false,
  num_laser_scans = 0,
  num_multi_echo_laser_scans = 0,
  num_subdivisions_per_laser_scan = 1,
  num_point_clouds = 1,
  lookup_transform_timeout_sec = 0.2,--0.1
  submap_publish_period_sec = 0.5,--0.1
  pose_publish_period_sec = 5e-3,
  trajectory_publish_period_sec = 30e-5,
  rangefinder_sampling_ratio = .5,
  odometry_sampling_ratio = 1.,
  fixed_frame_pose_sampling_ratio = 1.,
  imu_sampling_ratio = 1.,
  landmarks_sampling_ratio = 1.,

}

-- if point cloud frequency is low e.g. 10 hz then decrease submaps.num_range_data to 5, optimize_every_n_nodes to 30

MAP_BUILDER.use_trajectory_builder_3d = true
MAP_BUILDER.num_background_threads = 6

TRAJECTORY_BUILDER_3D.use_online_correlative_scan_matching = true

TRAJECTORY_BUILDER_3D.submaps.num_range_data = 100 --120

TRAJECTORY_BUILDER_3D.high_resolution_adaptive_voxel_filter.min_num_points = 1050 --150
TRAJECTORY_BUILDER_3D.high_resolution_adaptive_voxel_filter.max_length = 10. --2.
TRAJECTORY_BUILDER_3D.high_resolution_adaptive_voxel_filter.max_range = 15. --15.

Do you also have hints for tuning?
This is the bag file:
https://www.dropbox.com/s/cuc9fmy3zcrjz6a/2019-03-12-Neu.bag?dl=0

I still suspect IMU, please post the data contents of a few IMU messages.
Also, please post the text output of rosbag_validate as the issue template describes.
Cartographer initializes submap orientation with IMU gravity direction.

TRAJECTORY_BUILDER_3D.use_online_correlative_scan_matching = true

Why do you use the correlative scan matcher?

use_odometry = true,

For tuning, always start with as few input as possible, so no odometry. Only add after it works with the minimum sensor set.

Thank you for your answer! I put the correlative scan matcher out and it works fine without. Thanks!

Hey @gaschler I thought the problem was solved but it's not. The tip over is not that big anymore. Thats why I thought it is solved but I have here the rosbag validate: https://gist.github.com/Semih67/671cf968358a3178c9abb34e8e26f9f3
and this is my imu data:
header:
  seq: 1359
  stamp:
    secs:<PHONE_NUMBER>
    nsecs: 922173471
  frame_id: "imu"
orientation:
  x: 0.0
  y: 0.0
  z: 0.0
  w: 0.0
orientation_covariance: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]
angular_velocity:
  x: -0.00111111111111
  y: -0.00444444444444
  z: -0.00111111111111
angular_velocity_covariance: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]

I hope you can help me with this

And here is more:
---
header:
  seq: 1075
  stamp:
    secs:<PHONE_NUMBER>
    nsecs: 156453738
  frame_id: "imu"
orientation:
  x: 0.0
  y: 0.0
  z: 0.0
  w: 0.0
orientation_covariance: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]
angular_velocity:
  x: 0.0
  y: 0.00111111111111
  z: 0.0
angular_velocity_covariance: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]
linear_acceleration:
  x: -0.13
  y: 0.24
  z: 9.49
linear_acceleration_covariance: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]
---
header:
  seq: 1076
  stamp:
    secs:<PHONE_NUMBER>
    nsecs: 166435749
  frame_id: "imu"
orientation:
  x: 0.0
  y: 0.0
  z: 0.0
  w: 0.0
orientation_covariance: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]
angular_velocity:
  x: -0.00111111111111
  y: 0.0
  z: 0.0
angular_velocity_covariance: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]
linear_acceleration:
  x: -0.16
  y: 0.27
  z: 9.52
linear_acceleration_covariance: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]
---
header:
  seq: 1077
  stamp:
    secs:<PHONE_NUMBER>
    nsecs: 176433824
  frame_id: "imu"
orientation:
  x: 0.0
  y: 0.0
  z: 0.0
  w: 0.0
orientation_covariance: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]
angular_velocity:
  x: -0.00111111111111
  y: 0.00111111111111
  z: 0.0
angular_velocity_covariance: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]
linear_acceleration:
  x: -0.14
  y: 0.23
  z: 9.55
linear_acceleration_covariance: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]
---
header:
  seq: 1078
  stamp:
    secs:<PHONE_NUMBER>
    nsecs: 186434875
  frame_id: "imu"
orientation:
  x: 0.0
  y: 0.0
  z: 0.0
  w: 0.0
orientation_covariance: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]
angular_velocity:
  x: -0.00222222222222
  y: 0.00333333333333
  z: -0.00111111111111
angular_velocity_covariance: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]
linear_acceleration:
  x: -0.16
  y: 0.22
  z: 9.55
linear_acceleration_covariance: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]

The jump in rotation during global slam (pose graph optimization) is the exact issue I observe (#1201) with a dual lidar system. The result of local slam is relatively stable but there is something about the global optimization which I don't understand and have likely configured incorrectly. Do you notice this if you start from a stationary pose or if you are moving at the beginning of the data set?


can not use Prisma.JsonNull or Prisma.DbNull with Json[] columns

Bug description
can't use Prisma.DbNull or Prisma.JsonNull with Json[] columns
however, an empty create({}) will write a row with a NULL value in the jsonb[] column

this makes a copy operation really awkward:

const orig = await prisma.withJsonArray.create({})

await prisma.withJsonArray.create({
    data: {
      j: orig.j // error
    }
  })

await prisma.withJsonArray.create({
    data: {
      j: orig.j === null ? Prisma.DbNull : (orig.j as Prisma.InputJsonValue[]) // error, same with Prisma.JsonNull
    },
  });

// works, but writes a different value to column `j` (an empty array) than `orig.j` (DbNull)
await prisma.withJsonArray.create({
    data: {
      j: orig.j === null ? [] : (orig.j as Prisma.InputJsonValue[]),
    },
  });

additionally (and even more confusingly), none of the above code actually results in a find producing a null value on the typescript side of prisma:

const found = await prisma.withJsonArray.findFirstOrThrow()
type info:
const found: {
    id: number;
    j: Prisma.JsonValue[]; // i can't write nulls in the array or as the top-level value, but they are possible to read?
}

none of the code above actually reads values where found.j === null, but you still have to account for it when copying

also, the type system says array elements can be null on read, but i can't write null (or DbNull/JsonNull) array elements

await prisma.withJsonArray.create({
    data: {
      j: orig.j.map((el) =>
        el === null ? Prisma.DbNull : (el as Prisma.InputJsonValue)
      ),
    },
  });

// Type '(DbNull | InputJsonValue)[]' is not assignable to type 'WithJsonArrayCreatejInput | InputJsonValue[] | undefined'.
  Type '(DbNull | InputJsonValue)[]' is not assignable to type 'InputJsonValue[]'.
    Type 'DbNull | InputJsonValue' is not assignable to type 'InputJsonValue'.
      Type 'DbNull' is not assignable to type 'InputJsonValue'.ts(2322)

How to reproduce
use the prisma schema and ts code below
Type 'DbNull' is not assignable to type 'WithJsonArrayCreatejInput | InputJsonValue[] | undefined'.ts(2322)
(property) j: Prisma.NullTypes.DbNull

also:
Type 'JsonNull' is not assignable to type 'WithJsonArrayCreatejInput | InputJsonValue[] | undefined'.ts(2322)
(property) j: Prisma.NullTypes.JsonNull

Expected behavior
the docs say you should be able to use Prisma.DbNull / Prisma.JsonNull with Json columns, either this is a bug or the docs should note this doesn't work for Json[] columns
Prisma information
model WithJsonArray {
  id Int    @id @default(autoincrement())
  j  Json[]
}

const first = await prisma.withJsonArray.create({
    data: {
      j: Prisma.DbNull,
    },
  });

const second = await prisma.withJsonArray.create({
    data: {
      j: Prisma.JsonNull,
    },
  });

Environment & setup

OS: macOS 14.2
Database: PostgreSQL
Node.js version: 20.4.0

Prisma Version
prisma                  : 5.4.2
@prisma/client          : 5.4.2
Current platform        : darwin-arm64
Query Engine (Node-API) : libquery-engine ac9d7041ed77bcc8a8dbd2ab6616b39013829574 (at<EMAIL_ADDRESS>Schema Engine           : schema-engine-cli ac9d7041ed77bcc8a8dbd2ab6616b39013829574 (at node_modules/@prisma/engines/schema-engine-darwin-arm64)
Schema Wasm             : @prisma/prisma-schema-wasm 5.4.1-2.ac9d7041ed77bcc8a8dbd2ab6616b39013829574
Default Engines Hash    : ac9d7041ed77bcc8a8dbd2ab6616b39013829574
Studio                  : 0.494.0

@shawnjones253 did you hit the same problem in newer versions of prisma? eg v5.7.1 ?

@shawnjones253 did you hit the same problem in newer versions of prisma? eg v5.7.1 ?

@miguelff yes, i've just reproduced this with 5.7.1

Running into this on 5.8.1. Seems like the same as #9247, but in array form.
The array makes it a bit more tricky, but you can use filter to get rid of the nulls with a type guard, similar to this: https://stackoverflow.com/a/54317362/629356
Still feels weird though since for my use case there never will be nulls.

Confirmed, there is a bug, the docs express:

If we used the following schema:
model WithJsonArray {
  id Int    @id @default(autoincrement())
  array  Json[]
  json  Json
}

The congruent behavior would creating a record with an array field such as:
  const second = await prisma.withJsonArray.create({
    data: {
      array: [Prisma.JsonNull],
    },
  })

Given the following operation is indeed possible
  await prisma.withJsonArray.create({
    data: {
      json: Prisma.JsonNull,
    },

The type of JSON is JsonInputValue | JsonNull
The type of JSON[] is []JsonInputValue, while it should be something more along the lines of Array<JsonInputValue | JsonNull>  | DbNull
Further discussion and opinions:
I believe it should not be expected to be able to create the following record, because an JSON (value, right hand of assignment), is not of type Array of Json (column, left hand of assingment).
await prisma.withJsonArray.create({
    data: {
      array: Prisma.JsonNull,
    },
  });

But this should also be expected, as there can be a null value for the column (representing absence of value for that column)
await prisma.withJsonArray.create({
    data: {
      json: Prisma.DbNull,
    },
  });

But this should not be expected to work, as the field is an array of JSON, and we are assigning the null value for a column, which expresses nullability of a column, not nullability of an element within the column.
await prisma.withJsonArray.create({
    data: {
      array: [Prisma.DbNull],
    },
  });


Error in reference. Line: 1

VM64:1 Uncaught ReferenceError:     is not defined
    at <anonymous>:1:1

That's an easy fix.
Try:

Thanks! That worked like a charm.
Closing.

Had same issue, this helped, thanks


update documentation on modular configurations

as configuration required by the ApplicationFactory is now strictly array in 1.1.x

1.1.0 had a bug, as we were still supporting Aura.Di versions less than 3.1.0, which do not allow array values for services. As such, a fix in #447 ensures we continue to support ArrayObject for config service values.
Thanks for taking the time to make this patch, though!


Use OkReplay for UI tests

Dependent on #23

https://github.com/airbnb/okreplay
https://medium.com/airbnb-engineering/writing-fast-deterministic-and-accurate-android-integration-tests-c56811bd14e2


Fixes : #26 Added the Import and export option to the NavBar

Made the Required chnages on #26 , as stated in #28

Cleared all the conflicts!

@jinx-vi-0 @jindalpriyanshu101 Here is the fix , with all the suggested changes


System.UriFormatException : Invalid URI: The hostname could not be parsed.

From @nulltoken on July 2, 2018 12:39
Hello,
It's possible I may have found a bug (or at the very least a surprising behavior :wink: ).
Running an integration test, using the TestServer, against an Mvc backed API (monitored by ApplicationInsights), throws a UriFormatException.
It looks like (cf. the screenshots below) that the host being parsed end up being "localhost,localhost".
I've managed to trim down the code as much as possible into a repro case available at https://github.com/nulltoken/UriFormatException
Beware, that looks pretty random. The repro case breaks most of time with one failing test, and from time to time two tests fail.
FWIW, removing this line from the repro case makes the problem disappear.
Versions:

.Net Core 2.1.1
ApplicationInsightsAspNetCore 2.4-beta1

Copied from original issue: aspnet/Mvc#7997

From @nulltoken on July 2, 2018 19:9
FWIW this was working against .NET Core 2.0.5

From @pranavkm on July 2, 2018 20:25
cc @Tratcher

@Tratcher Would you mind if I tried and gave a try at fixing it?

Go for it.

Go for it.

@Tratcher Done. That was indeed a pretty easy fix. You wonderfully paved the way with your detailed explanation. Thanks a lot!
The only things that keeps on bugging me is that in my initial repro, not all the tests were failing. Only some of them.
Condering that ClientHandler.SendAsync() performs this normalization for each and every call, all the tests should fail, shouldn't they?

I don't have a good answer for that. The error wouldn't happen until the Uri was constructed, and only AI does that. Either that wasn't happening on some requests, or the error was being handled differently on some code paths.


Bug fix: swatches() was not returning LightMutedSwatch

Fail. How did I miss that.
Can you run gulp and commit again so the compiled files get updated too?

Thanks for the quick response! I'm having an issue running gulp, I followed the instructions on your page and got:
Error: Command failed: java -jar -XX:+TieredCompilation bower_components/closure-compiler/lib/vendor/compiler.jar --flagfile="/var/folders/gb/0pbsww4147776g9l7s6qqr4r0000gn/T/621cedf3-e84a-4d96-9c82-bdb7e0e8a5ab" --js_output_file="Vibrant.min.js"```
Unable to access jarfile bower_components/closure-compiler/lib/vendor/compiler.jar

It seems that bower installed closure-compiler#~0.2.6 successfully but the folder vendor/ doesn't exist.

#43 fixes the close compiler problem and this bug


Clarification on using project scoped feeds

Since all new feeds in Azure DevOps are now scoped to a project it might be helpful to note that you cannot just use feedName for new feeds, but instead have to use <projectName>/<feedName>.  Using just the feedName will result in a FeedNotFoundException when the pipeline is run.

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

ID: d4cb48a1-d0bf-40b7-4645-910d4006fee9
Version Independent ID: 9bddd101-2bc2-f65c-6a12-e395b76a2b98
Content: Publishing npm packages - Azure Pipelines and TFS
Content Source: docs/pipelines/artifacts/npm.md
Product: devops
Technology: devops-cicd
GitHub Login: @chasewilson
Microsoft Alias: phwilson

@ma-kern Thank you for your feedback. The new version will go live shortly.


chore(NODE-4348): enhance skip reason filtering in unified runner

Description
What is changing?
This PR enhances the skip test functionality of the unified runner.  Rather than accepting an array of tests to skip, the runner now takes a function that is evaluated to determine whether a test should be skipped.
This PR also updates the existing usages of runUnifedSuite that skip tests to use the new function approach.
Finally, some tests relating to auto connect and change stream resumability were unskipped.
Is there new documentation needed for these changes?
Negative.
Double check the following

[x] Ran npm run check:lint script
[x] Self-review completed using the steps outlined here
[x] PR title follows the correct format: <type>(NODE-xxxx)<!>: <description>
[x] Changes are covered by tests
[x] New TODOs have a related JIRA ticket

Also 1 lint issue.


Keybindings editor: steals focus in this case

Steps to Reproduce:

open keybindings editor and some other files
open quick open (Cmd+P) and pick some of the other files and press Arrow-right
the other files focus in the background but quick open stays open
now try this on the keybindings editor

=> focus is lost to the keybindings editor, it eagerly puts focus into its input field
//cc @sandy081

Thanks @bpasero


Update and rename outreach-outcomes to outreach-outcomes.md

👷 Deploy Preview for famous-marshmallow-cfbd61 processing.

Name
Link

🔨 Latest commit
a54b961f496071edeca79cd54f41c9f3447f9b81

🔍 Latest deploy log
https://app.netlify.com/sites/famous-marshmallow-cfbd61/deploys/64dd943158482400082375dc


Migrate extension_lang_map to a global Talon list

I think we should migrate the variable extension_lang_map to a Talon list.
Ideally, someone implementing support for a new programming language would not have to modify any existing code, and each language mode would append the programming language name and its file extensions to the global list, which would trigger the creation of a new language mode. I assume that this requires registering an event handler in code.py, rather than the current loop which creates language modes once based on a constant mapping.
In the current setup, the mapping is not exposed, which leads to developers copying it into their own project whenever they need to know about language modes and file extensions. For instance, see talon_hud. Such copies naturally get out of sync with the copy in this repository. Furthermore, someone implementing support for a new language must also know to update the list in talon_hud/status_bar_poller.py and any other project which uses this mapping.
If approved, I should be able to make this change with minimal guidance, and I would be happy to also update chaosparrot/talon_hud.

I don't know of a way to append to a talon list rather than simply overriding it. Also, talon lists are intended for use in voice commands, not as generic associative-arrays. For instance, I think grabbing the value of a talon list currently requires mucking with talon.registry, which is mostly intended for debug use iirc.
It would definitely be nice to support (a) adding to the set of language extensions on a modular way and (b) reading the language extension map from other files. I don't immediately see a good way to do (a). I think (b) could be done by having an action that returns the list of extension mappings, which talon_hud etc would need to call when they try to figure out what language a file uses. Another option would be to keep the mapping in a CSV file.

What about declaring a global dict in code.py and adding to it in each language mode?

We generally try to avoid cross-file imports in Talon python scripts, it causes cascading failures (eg. if I'm editing code.py and make a syntax error and save, now every python file that imports code.py will fail to load, possibly making some commands stop working).
This particular case might also interact badly with Talon's auto-reloading of python scripts. If a module has top-level side effects (like mutating another module's global) and then you change the module, it will get reloaded, and the effect will reoccur. That wouldn't be so bad here, since setting a dictionary entry twice is idempotent. It would mean that deleting that file wouldn't remove the corresponding dictionary entry, though.
So, I think that approach would mostly work, but it feels icky.

This feels like a problem that would need solving in order to move to a "plugin" model of talon scripts (instead of the current knausj-centric approach), which I know is on @lunixbochs long-term agenda. Might be worth asking for a Talon feature to handle this.

Currently the functions and libraries GUI stuff use global variables, so I don't see why we'd need a new Talon feature?

I don't think we take a global variable defined in one module and mutate it in another. The library and function globals don't seem to do this; afaict they're just caches used for gui state. Cross-module mutation is what worries me, because (as mentioned earlier) I'm not confident it'll always play nice with Talon's auto-reloading, and it requires importing one python module from another.

@rntz I don't think we take a global variable defined in one module and mutate it in another.

It's bad practice in general, but, e.g., matplotlib does it.

I like the external csv/json file idea --- is that not feasible?

I've been using it in my personal set up, and it works just fine.

Closing in favor of https://github.com/talonhub/community/issues/1585


Search UI should normalize query parameters

Describe the bug
A page on OpenWDL's github (https://github.com/openwdl/wdl) links to
https://dockstore.org/search?_type=workflow&descriptorType=wdl&searchMode=files
Since descriptorType is wdl, rather than WDL, no workflows are displayed.  Then, if you select "WDL" in the facet, the workflows are displayed, but in the updated URL, the descriptorType parameter is set twice (to both "wdl" and "WDL").
To Reproduce
Steps to reproduce the behavior:

Go to https://dockstore.org/search?_type=workflow&descriptorType=wdl&searchMode=files
See zero workflow results

Expected behavior
Gracefully handle cases such as the above.
Additional context
Webservice - 1.13.0
UI - 2.10.0
Compose Setup - 1.13.0
Deploy - 1.12.6
galaxyParsingLambdaVersion: 0.0.7
┆Issue is synchronized with this Jira Story
┆fixVersions: Dockstore 1.14
┆friendlyId: DOCK-2267
┆sprint: Backlog
┆taskType: Story

Another (additional?) option would be to submit a PR against the openwdl repo.

Another (additional?) option would be to submit a PR against the openwdl repo.

Inevitably, some inbound links will be malformed, and if the problems are easy to correct (in this case, by uppercasing), we should do it.  Patching openwdl is fine, but doesn't generalize to a scalable solution.

The OpenWDL link also has a _type=workflow query parameter, which should be _entryType=workflows. I think it's OK though, unless it leads to a variant of #5207

PR was merged to openwdl/wdl so link is now good as a temporary measure.


Changes for network infrastructure services

Changes for ntp, new dns, dhcp and tftp roles with basic configs.

OK, pasa.


Microsoft.Cache\Redis ARM template missing publicNetworkAccess property

The ARM template reference for Microsoft.Cache\Redis does not specify the publicNetworkAccess property.

@sean-keane25
Thanks for the feedback and bringing this to our notice. At this time we are reviewing the feedback and will provide an update as appropriate.

@sean-keane25
Thanks for the feedback and bringing this to our notice. At this time we are reviewing the feedback and will provide an update as appropriate.

@sean-keane25 The flag is supported but disabled by default.
You can submit a PATCH operation to change it.
PATCH  https://management.azure.com/subscriptions/{subscription}/resourceGroups/{resourcegroup}/providers/Microsoft.Cache/Redis/{cache}?api-version=2020-06-01
{    "properties": {
       "publicNetworkAccess":"Disabled"
   }
}

Do let us know if you have any questions

@sean-keane25 The flag is supported but disabled by default.
You can submit a PATCH operation to change it.
PATCH  https://management.azure.com/subscriptions/{subscription}/resourceGroups/{resourcegroup}/providers/Microsoft.Cache/Redis/{cache}?api-version=2020-06-01
{    "properties": {
       "publicNetworkAccess":"Disabled"
   }
}

Do let us know if you have any questions

Shouldn't this flag be documented in the arm template ? Thats the point of this github issue.

Shouldn't this flag be documented in the arm template ? Thats the point of this github issue.

@sean-keane25 This is documented on Azure Cache for Redis docs. You can find it here
If there is some other doc where you think this information is relevant, please provide the links and we will make the changes.
Thank you providing feedback and helping us improve our documentation.

@sean-keane25 This is documented on Azure Cache for Redis docs. You can find it here
If there is some other doc where you think this information is relevant, please provide the links and we will make the changes.
Thank you providing feedback and helping us improve our documentation.

Yes, I am aware that it is document at the link you provided. This doc github issue was opened because the ARM template reference for Microsoft.Cache\Redis does not specify the publicNetworkAccess property.

Yes, I am aware that it is document at the link you provided. This doc github issue was opened because the ARM template reference for Microsoft.Cache\Redis does not specify the publicNetworkAccess property.

Got it. Thank you for the additional context.
Private link integration with Azure Cache for Redis is currently in preview. The parameter should be included in a new API version once the integration is GA and the template reference will be updated accordingly.
We will proceed to close this issue now.
If there are further questions regarding this matter, please comment and we will gladly continue the discussion.

Got it. Thank you for the additional context.
Private link integration with Azure Cache for Redis is currently in preview. The parameter should be included in a new API version once the integration is GA and the template reference will be updated accordingly.
We will proceed to close this issue now.
If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Scala samples

Scala samples or usage pattern are welcomed.

+1 would be good to see an example to use a delta table as a source syncing updates in real-time to an sql server dB table.

@rajmera3 any scala samples?
@slonikix @pixie79 You can find scala usage at https://spark.apache.org/docs/latest/sql-data-sources-jdbc.html
Note : Here's a sample snippet to get you going. Note : interface is same as spark jdbc. Just replace the name of the connector with new connector name.
https://spark.apache.org/docs/latest/sql-data-sources-jdbc.html
df.write.
format("com.microsoft.sqlserver.jdbc.spark").
mode("append").
option("url", url).
option("dbtable", dbtable).
option("user", user).
option("password", password).
option("truncate", true).
save()

@DaniBunny assigning this to you for samples. Refer test app. Test app is an elaborate Scala  sample on using connector in scala

Will close, documented here as necessary. CC @DaniBunny
#please-close


fix typo in docs

Simple typo fix in docs on grids.

Thanks a lot for catching this and submitting a PR. Will merge now.


Refactor rollback execution

What this PR does / why we need it:
In order to provide more logging around rollback workflow execution, the code is refactored.
In addition, it will enable the support of linked-rollbacks if it will be needed by simple additional work.
Change type

[ ] New feature
[ ] Bug fix
[ ] Unit tests
[ ] Integration tests
[ ] CI
[ ] Documentation
[ ] Auto-generated SDK code

Impacted services

[ ] Workflow Service
[ ] Notification Service

Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.

/lgtm


misc: add device guard for kernels

plan

[x] Check all kernels and add device guard
[x] Complete the  tests

FIX: https://github.com/flashinfer-ai/flashinfer/issues/452

Hi @jeejeelee , thanks for bringing this up, would you mind adding device guard for JIT templates as well?

https://github.com/flashinfer-ai/flashinfer/blob/a3360ff9c85e7d0bae8bb4ca6dbbca69cfadea37/python/flashinfer/jit/batch_decode_mla_templ.py
https://github.com/flashinfer-ai/flashinfer/blob/a3360ff9c85e7d0bae8bb4ca6dbbca69cfadea37/python/flashinfer/jit/batch_decode_templ.py
https://github.com/flashinfer-ai/flashinfer/blob/a3360ff9c85e7d0bae8bb4ca6dbbca69cfadea37/python/flashinfer/jit/batch_prefill_templ.py
https://github.com/flashinfer-ai/flashinfer/blob/a3360ff9c85e7d0bae8bb4ca6dbbca69cfadea37/python/flashinfer/jit/single_decode_templ.py
https://github.com/flashinfer-ai/flashinfer/blob/a3360ff9c85e7d0bae8bb4ca6dbbca69cfadea37/python/flashinfer/jit/single_prefill_templ.py

Okay, I will

@yzh119  I have checked all kernels and added the device guards. Additionally, I tested the ROPE and NORM kernel in a CUDA 11.8 environment to verify they pass the tests. Please let me know if any additional testing is needed.