deal service

I had a problem while updating (probably creating also) deals. simply I wanted to change stage id like that;
$r = $baseclient->deals->update($dealID,['stage_id'=>$quoteSentStageID]);
and I have got that error.
Fatal error: Uncaught exception 'BaseCRM\Errors\ResourceError' with message 'resource=deal field=/value code=invalid_type message=attribute value has an invalid type' in vendor/basecrm/basecrm-php/lib/HttpClient.php:294
Stack trace:
#0 vendor/basecrm/basecrm-php/lib/HttpClient.php(280): BaseCRM\HttpClient->handleErrorResponse(422, '{"errors":[{"er...', Array)
#1 vendor/basecrm/basecrm-php/lib/HttpClient.php(162): BaseCRM\HttpClient->handleResponse(422, '{"errors":[{"er...')
#2 vendor/basecrm/basecrm-php/lib/HttpClient.php(62): BaseCRM\HttpClient->request('PUT', '/deals/31156905', NULL, Array, Array)
#3 vendor/basecrm/basecrm-php/lib/DealsService.php(112): BaseCRM\HttpClient->put('/deals/31156905', Array)
#4 x.php(82): BaseCRM\DealsService->update(31156905, Array)
#5 in vendor/basecrm/basecrm-php/lib/HttpClient.php on line 294
as I see
$attributes["value"] = Coercion::toStringValue($attributes['value']);
lines(#62,#108) in create and update functions are wrong. I commented them and it worked.

Hi @v0na, thanks for the note. It's fixed in version 1.3.1.


Help with ingress (nginx)

Hey guys,
I can't get my head around the docs for using helm charts:
Due to not having traefik running as ingress controller pre-shipped (I think it's standard for k3s for having it included but it's not part of zart's k3s), I wanted to install nginx-ingress.
There is an installation guide presenting this helm command:
helm upgrade --install ingress-nginx ingress-nginx \
  --repo https://kubernetes.github.io/ingress-nginx \
  --namespace ingress-nginx --create-namespace

I tried tons of ways to let zarf know to download the required images and deploy the helm config afterwards but always something is missing.
I guess I am just missing out a key concept for using helm... Could you please help and let me know how to deploy a standard helm?
Thanks a lot!

the zarf.yaml
kind: ZarfPackageConfig
metadata:
  name: nginx-ingress
  description: "ingress based on nginx"

components:
  - name: nginx-ingress
    required: true
    charts:
      - name: ingress-nginx
        namespace: ingress-nginx
        version: 4.6.0
        url: https://kubernetes.github.io/ingress-nginx
    images:
      - nginx/nginx-ingress:3.1.0

Alternatively, I followed the full nginx-ingress guide to have a full k8s config to run the nginx-ingress. See the darf.yaml here with correlating file ingress-controller.yaml.txt:
kind: ZarfPackageConfig
metadata:
  name: nginx-ingress
  description: "nginx-ingress"

components:
  - name: nginx-ingress
    required: true
    manifests:
      - name: nginx-ingress
        namespace: nginx-ingress
        files:
          - manifest/ingress-controller.yaml
    images:
      - nginx/nginx-ingress:3.1.0

Its not well documented but you should be able to turn back on traefik by specifying empty K3S_ARGS like this:
zarf init --set K3S_ARGS=""

You can see where that is pulled in here: https://github.com/defenseunicorns/zarf/blob/main/packages/distros/k3s/common/zarf.yaml#L8 (if it is not set it gets defaulted to --disable traefik)

That said, it looks like that is the wrong image for that chart.  This seemed to work for me:
# zarf.yaml
kind: ZarfPackageConfig
metadata:
  name: nginx-ingress
  description: "ingress based on nginx"

components:
  - name: nginx-ingress
    required: true
    charts:
      - name: ingress-nginx
        namespace: ingress-nginx
        version: 4.6.0
        url: https://kubernetes.github.io/ingress-nginx
        valuesFiles:
          - "values.yaml"
    images:
      - registry.k8s.io/ingress-nginx/controller:v1.7.0
      - registry.k8s.io/ingress-nginx/kube-webhook-certgen:v20230312-helm-chart-4.5.2-28-g66a760794

# values.yaml
controller:
  image:
    registry: registry.k8s.io
    image: ingress-nginx/controller
    tag: "v1.7.0"
    digest:
    digestChroot:
  admissionWebhooks:
    patch:
      enabled: true
      image:
        registry: registry.k8s.io
        image: ingress-nginx/kube-webhook-certgen
        tag: v20230312-helm-chart-4.5.2-28-g66a760794
        digest:

I found the first image running zarf prepare find-images:
$ zarf prepare find-images .

Saving log file to /tmp/zarf-2023-04-19-09-56-32-1267556376.log

Using base directory .
components:
  ✔  Processing helm chart ingress-nginx:4.6.0 from repo https://kubernetes.github.io/ingress-nginx
  ✔  Templating helm chart ingress-nginx

  - name: nginx-ingress
    images:
      - registry.k8s.io/ingress-nginx/controller:v1.7.0
      - registry.k8s.io/ingress-nginx/kube-webhook-certgen:v20230312-helm-chart-4.5.2-28-g66a760794

The second I found running zarf tools monitor (which launches k9s) and pressing d on the ImagePullBackOff error I saw then looking through the values file for it here: https://github.com/kubernetes/ingress-nginx/blob/helm-chart-4.6.0/charts/ingress-nginx/values.yaml (note I was on the tag for the 4.6.0 helm chart).

Note I dropped the SHAs and overrode them using the valuesFiles key since Zarf currently will normalize all manifests to the OCI format and then push them to the remote - we have discussed ways to make it so that SHAs are kept in tact but that is still on our future roadmap.

Hi @Racer159,
thanks for your help: I was able to start both, traefik and nginx ingress on different zarf clusters 👍
Since traefik is the default for k3s I guess it would be highly appreciated to have this config a bit better documented... I am not quite sure concerning the concepts of zarf deploying the k3s but then not using it's default settings... Maybe it's worth describing this a bit better in docs?
In terms of the images with nginx:
First I think your last code block is showing both images though only the first one was unveiled during find-images, at least only the first one is visible for me running this command.
Do you have an idea, why the second image wasn't found by find-images command?
Secondly, I guess the zarf.yaml option for defining images is due to the specification, which images from the components are required to get shipped with this zarf package.
Supposing I am right, one could have a first zarf package, containing image xyz, and then a second package, also requiring image xyz but since it's already shipped previously there is no need to have it shipped again.
In case my assumptions are correct: I think from a user's perspective an "autoload" feature would be nice → if images isn't specified for the component, find-images would get run internally and be added to the package.

I see what you did with the values.yaml. Yet, I am not sure if this is relevant for each usage of a given helm chart? If so, I guess a support document on how to use helm charts would be highly appreciated...

Some of the auto mode may also be cover-able in #1651

@Racer159 thank you!
Your decisions make sense, also thx for the explanation 👍

No problem! We also added a Package Create Tutorial that should help future folks as well, and with that now in place, this issue should be complete.


Building Omni Core <IP_ADDRESS> release

Just for the reference and completness, so we're on the same level.
The previous steps are based on:

https://github.com/OmniLayer/omnicore/blob/omnicore-0.0.10/doc/gitian-building.md

If needed, refresh the inputs (from ~/gitian-builder):
make -C ../omnicore/depends download SOURCES_PATH=`pwd`/cache/common

If needed, download the SDK for OS X (stored in ~/gitian-builder/inputs):
wget https://s3.amazonaws.com/ci.omnicore.private/depends-sources/sdks/MacOSX10.7.sdk.tar.gz

Update the local copy of Omni Core:
git clone https://github.com/OmniLayer/omnicore.git

Then set the environment:
SIGNER=dexX7
COMMIT=v<IP_ADDRESS>
RELEASE=<IP_ADDRESS>

Note: the release is tagged with v<IP_ADDRESS> instead of v<IP_ADDRESS>-rel, even though the version suffix in Omni Core itself is -rel.
Build for Windows:
./bin/gbuild --commit omnicore=${COMMIT} ../omnicore/contrib/gitian-descriptors/gitian-win.yml
./bin/gsign --signer ${SIGNER} --release ${RELEASE}-win --destination ../gitian.sigs/ ../omnicore/contrib/gitian-descriptors/gitian-win.yml
mv build/out/omnicore-*.zip build/out/omnicore-*.exe ../

Build Linux:
./bin/gbuild --commit omnicore=${COMMIT} ../omnicore/contrib/gitian-descriptors/gitian-linux.yml
./bin/gsign --signer $SIGNER --release ${RELEASE}-linux --destination ../gitian.sigs/ ../omnicore/contrib/gitian-descriptors/gitian-linux.yml
mv build/out/omnicore-*.tar.gz build/out/src/omnicore-*.tar.gz ../

Build for OS X:
./bin/gbuild --commit omnicore=${COMMIT} ../omnicore/contrib/gitian-descriptors/gitian-osx.yml
./bin/gsign --signer $SIGNER --release ${RELEASE}-osx-unsigned --destination ../gitian.sigs/ ../omnicore/contrib/gitian-descriptors/gitian-osx.yml
mv build/out/omnicore-*.tar.gz build/out/omnicore-*.dmg ../

Finally continue with signing the results, as described in the gitian.sigs guide:

https://github.com/OmniLayer/gitian.sigs/blob/master/CONTRIBUTING.md

For discussion about building <IP_ADDRESS>, see #412.

Windows:
19c20cdbe0fe145a73c918924e98a64eca9d0d965ba0e2673bd40f0835f5e17b  omnicore-<IP_ADDRESS>-rel-win32-setup.exe
547c0f9765b6e3fe10a50c59886ddadfc4414499172ff452bda9c3d98d184f1a  omnicore-<IP_ADDRESS>-rel-win32.zip
020358218ebacd6301c630c5f98834ec1ac087055d7ce6a0107b44764e442c14  omnicore-<IP_ADDRESS>-rel-win64-setup.exe
87795c03ffa192ffcc2c19f876780aa748e2d10f5dc3f401f11bcf98ab17b521  omnicore-<IP_ADDRESS>-rel-win64.zip
22fd8b42b96fe5c89116595a1b5a35594de9f4a94edcefaf7d8007619ea19c58  src/omnicore-<IP_ADDRESS>-rel.tar.gz

Linux:
218d97ccc03c4caeb2c80579997c603516038aec80c5c99edd33e49d11a1b487  omnicore-<IP_ADDRESS>-rel-linux32.tar.gz
8b1b0d082834b2fea3da466bd674f7401cb883d754ad314625cf850265ef7f9f  omnicore-<IP_ADDRESS>-rel-linux64.tar.gz
22fd8b42b96fe5c89116595a1b5a35594de9f4a94edcefaf7d8007619ea19c58  src/omnicore-<IP_ADDRESS>-rel.tar.gz

OS X:
13838ce75be95fbe9ecbffadc9826aef8b70f03d565f90379046901fc833025c  omnicore-<IP_ADDRESS>-rel-osx-unsigned.dmg
156bccb41a8a94a8b06d8a150f5eca79a641358621cd2013a92cf0b87b338c18  omnicore-<IP_ADDRESS>-rel-osx-unsigned.tar.gz
ca913d241108e159acdf7dc562ffc4aed84369171ed3b6ba8465334cddcf1a90  omnicore-<IP_ADDRESS>-rel-osx64.tar.gz
22fd8b42b96fe5c89116595a1b5a35594de9f4a94edcefaf7d8007619ea19c58  src/omnicore-<IP_ADDRESS>-rel.tar.gz

19c20cdbe0fe145a73c918924e98a64eca9d0d965ba0e2673bd40f0835f5e17b  omnicore-<IP_ADDRESS>-rel-win32-setup.exe
547c0f9765b6e3fe10a50c59886ddadfc4414499172ff452bda9c3d98d184f1a  omnicore-<IP_ADDRESS>-rel-win32.zip
020358218ebacd6301c630c5f98834ec1ac087055d7ce6a0107b44764e442c14  omnicore-<IP_ADDRESS>-rel-win64-setup.exe
87795c03ffa192ffcc2c19f876780aa748e2d10f5dc3f401f11bcf98ab17b521  omnicore-<IP_ADDRESS>-rel-win64.zip
22fd8b42b96fe5c89116595a1b5a35594de9f4a94edcefaf7d8007619ea19c58  src/omnicore-<IP_ADDRESS>-rel.tar.gz
9f78d56e1e8f03a047638d0069248e3225be50e7f3b68ee94060720a93b7a452  omnicore-win-0.10-res.yml

218d97ccc03c4caeb2c80579997c603516038aec80c5c99edd33e49d11a1b487  omnicore-<IP_ADDRESS>-rel-linux32.tar.gz
8b1b0d082834b2fea3da466bd674f7401cb883d754ad314625cf850265ef7f9f  omnicore-<IP_ADDRESS>-rel-linux64.tar.gz
22fd8b42b96fe5c89116595a1b5a35594de9f4a94edcefaf7d8007619ea19c58  src/omnicore-<IP_ADDRESS>-rel.tar.gz
03b89673c6683e3bd49002598903264b1d139cb239cb477ff0c998229a1281f8  omnicore-linux-0.10-res.yml

13838ce75be95fbe9ecbffadc9826aef8b70f03d565f90379046901fc833025c  omnicore-<IP_ADDRESS>-rel-osx-unsigned.dmg
156bccb41a8a94a8b06d8a150f5eca79a641358621cd2013a92cf0b87b338c18  omnicore-<IP_ADDRESS>-rel-osx-unsigned.tar.gz
ca913d241108e159acdf7dc562ffc4aed84369171ed3b6ba8465334cddcf1a90  omnicore-<IP_ADDRESS>-rel-osx64.tar.gz
22fd8b42b96fe5c89116595a1b5a35594de9f4a94edcefaf7d8007619ea19c58  src/omnicore-<IP_ADDRESS>-rel.tar.gz
a1c5d3dc36636f288d2a4b75e2bde5233c123bed9ee387ead3d66a09c41f42f9  omnicore-osx-0.10-res.yml

Signatures submitted here https://github.com/OmniLayer/gitian.sigs/pull/36
Thanks for organizing @dexX7 :)

P.S. looks like we'll be needing another release for the fees fixes...  Well done there mate.

P.S. looks like we'll be needing another release for the fees fixes... Well done there mate.

Yup, let's use #433 as new issue. :)


update temaki to v5.10.0

Bump the @rapideditor/temaki dependency version to ^5.10.0. This version of temaki has a new icon in it that I made, indended for a preset that I'm currently working on adding to id-tagging-schema. 🙂

Gotcha, thanks! I'll close this PR then.


Solidworks Macro for Balloons in drawing

Hello,
I've been searching the internet everywhere for a macro that will allow me to check for missing balloons and possibly add them to a drawing in solidworks.
I deal with assemblies with 500+ parts at times so its a very painful process at the moment.
thanks
John

Cool idea! I will be looking into this. The main trick here is proper positioning of the balloon so I might just skip this step. Instead, macro will scan the view, find all missing balloons, add ones into default location and notify which ones were added for the user to manually align them. Do you think this would be a viable solution?

Yes I think that solution would be an awesome starting point.


Make GitReleaseManager to correctly handle async methods

@Cheesebaron I assume you wish to be assigned?
If so, please comment on the issue, then we can assign you

I'll have a go at this


x/build: docker image for linux-x86-std-kube Makefile doesn't match coordinator?

@evandbrown, it looks like this CL,
https://go-review.googlesource.com/19021
refactored the Makefile of env/linux-x86-std-kube/Makefile which would've changed its image name from gcr.io/symbolic-datum-552/linux-x86-std:latest to gcr.io/symbolic-datum-552/linux-x86-std-kube:latest but the dashboard/builders.go file was never updated.
One of the two files needs to change so they're in sync.
So far we've just not needed to change anything and have gotten lucky I think.

Was fixed by https://go-review.googlesource.com/28416


Update python for tests/pages and add poetry lockfile

I think it's generally considered best practices to commit the lockfile for projects like this.

Maybe disable the poetry actions on CI?

That's a strange error:
Failed to resolve 'pypi.org'

Maybe disable the poetry actions on CI?

Disabled the lock action and got it working.

Maybe disable the poetry actions on CI?

Disabled the lock action and got it working.

If it works when you push it though, then it just needs to be disabled in pre-commit-ci


Function name mismatch between M2M_arch.py and taichi_wrapper.py

The M2M_arch.py file attempts to import softsplat_func with the following line:
     from ..ops.softsplat import softsplat_func'

However, the function is actually defined as softsplat in taichi_wrapper.py:
def softsplat(
    tenIn: torch.Tensor, tenFlow: torch.Tensor, tenMetric: torch.Tensor, strMode: str
):

This discrepancy leads to the following error:
ImportError: cannot import name 'softsplat_func' from 'models.ops.softsplat'

Fixed


Title Field Sanitization Broken

When using a double quote in the title field. It breaks the form input.

edit vendor\cnvs\easel\resources\views\backend\post\partials\form.blade.php line 41
using '' instead of "" for value attr will work.
but its a temperal fix

Fixed with https://github.com/cnvs/easel/commit/456e2458fb0bf4cb0ad89749c4120adf20c95741


Add S3 list-parts UTs

What changes are proposed in this pull request?
add S3 list-parts UTs.
Why are the changes needed?
test for s3 list parts
Does this PR introduce any user facing changes?
none

alluxio-bot, merge this please


Add archiver again

Codecov Report

Merging #79 into master will increase coverage by 19.75%.
The diff coverage is 92.59%.

@@             Coverage Diff             @@
##           master      #79       +/-   ##
===========================================
+ Coverage   50.14%   69.89%   +19.75%
===========================================
  Files          13       13
  Lines         357      392       +35
===========================================
+ Hits          179      274       +95
+ Misses        178      118       -60

Impacted Files
Coverage Δ

historical_system_profiles/deleter.py
0.00% <0.00%> (ø)

historical_system_profiles/listener_logging.py
0.00% <ø> (ø)

historical_system_profiles/archiver.py
69.04% <93.75%> (+69.04%)
:arrow_up:

historical_system_profiles/config.py
100.00% <100.00%> (+6.06%)
:arrow_up:

historical_system_profiles/db_interface.py
78.04% <100.00%> (+11.38%)
:arrow_up:

historical_system_profiles/listener_metrics.py
100.00% <100.00%> (+100.00%)
:arrow_up:

historical_system_profiles/models.py
98.14% <100.00%> (+27.77%)
:arrow_up:

historical_system_profiles/views/v1.py
98.57% <100.00%> (+31.90%)
:arrow_up:

... and 3 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 eec31bf...22b482e. Read the comment docs.


Access violation/segmentation fault connecting signal (with QFileSystemWatcher/QTimer)

Good things come in threes. #2086, #2492, and now this. Note this code likely doesn't do anything useful anymore, it is highly reduced to reproduce the segmentation fault.
crash.py
# nuitka-project: --onefile
# nuitka-project: --static-libpython=no
# nuitka-project: --enable-plugin=pyside6
# nuitka-project: --nofollow-import-to=tkinter
# possibly remove the above since 1.7
# nuitka-project: --noinclude-default-mode=error
# nuitka-project-if: {OS} == "Windows":
#    nuitka-project: --output-filename=crash.exe
# nuitka-project-if: {OS} == "Linux":
#    nuitka-project: --output-filename=crash.bin
# nuitka-project: --output-dir=dist-nuitka
# nuitka-project: --assume-yes-for-downloads

# With tests:
# nuitka-project: --noinclude-pytest-mode=allow
# nuitka-project: --noinclude-unittest-mode=allow
# nuitka-project: --include-package=_pytest
# nuitka-project: --include-package=pytestqt
# nuitka-project: --include-module=PySide6.QtTest
# nuitka-project: --include-module=test_crash
# nuitka-project: --nofollow-import-to=doctest
# nuitka-project: --include-data-file=test_crash.py=.

from pathlib import Path

import pytest

if __name__ == "__main__":
    pytest.main([Path(__file__).parent, "-s"], ["pytestqt.plugin"])

test_crash.py
from contextlib import contextmanager, nullcontext

from window import Window

def test_crash(qtbot, tmp_path):
    tmp_file = tmp_path / "tmp_file.raw"

    window = Window(tmp_file)
    qtbot.addWidget(window)

    def write(contents):
        with tmp_file.open("wb") as file:
            file.write(contents.encode("utf-8"))

    @contextmanager
    def wait(fixed=0):
        signal_wait = qtbot.waitSignal(window.signal, timeout=5000)
        with nullcontext() if fixed else signal_wait:
            yield
        if fixed:
            qtbot.wait(fixed)

    write("1")

    with wait():
        window.start()

    with wait(fixed=200):
        write("12")

    with wait():
        qtbot.wait(300)

window.py
from PySide6.QtCore import QFileSystemWatcher, QTimer, Signal
from PySide6.QtWidgets import QMainWindow

class Window(QMainWindow):
    signal = Signal()

    def __init__(self, path):
        super().__init__()
        self.path = path
        self.fsize = self.get_fsize()
        self.watcher = QFileSystemWatcher()
        self.watcher.fileChanged.connect(self.file_changed)

    def get_fsize(self):
        try:
            return self.path.stat().st_size
        except FileNotFoundError:
            return 0

    def file_changed(self):
        if (new_size := self.get_fsize()) != self.fsize:
            self.fsize = new_size
            QTimer.singleShot(300, self.file_changed)
            return
        self.start()

    def start(self):
        self.signal.emit()
        self.watcher.addPath(str(self.path))

Setup:
Python=3.11.6
Nuitka==1.9rc5  # 11a3d44ab12c706a54e85c21d5161e2983e1bc01
PySide6-Essentials==6.5.3
pytest==7.4.2
pytest-qt==4.2.0

Run
pytest  # works
python -m nuitka crash.py
dist-nuitka/crash.bin  # segfaults

Output
====================================================================================== test session starts ======================================================================================platform linux -- Python 3.11.6, pytest-7.4.2, pluggy-1.3.0
PySide6 6.5.3 -- Qt runtime 6.5.3 -- Qt compiled 6.5.3
rootdir: /tmp/onefile_11874_1697322110_687089
collected 1 item

../../../../tmp/onefile_11874_1697322110_687089/test_crash.py Fatal Python error: Segmentation fault

Current thread 0x00007f34dcefab80 (most recent call first):
  File "/tmp/onefile_11874_1697322110_687089/pytestqt/wait_signal.py", line 193 in connect
  File "/tmp/onefile_11874_1697322110_687089/pytestqt/qtbot.py", line 289 in waitSignal
  File "/tmp/onefile_11874_1697322110_687089/test_crash.py", line 18 in wait
  File "/tmp/onefile_11874_1697322110_687089/contextlib.py", line 137 in __enter__
  File "/tmp/onefile_11874_1697322110_687089/test_crash.py", line 32 in test_crash
  File "/tmp/onefile_11874_1697322110_687089/_pytest/python.py", line 194 in pytest_pyfunc_call
  File "/tmp/onefile_11874_1697322110_687089/pluggy/_callers.py", line 27 in _multicall
  File "/tmp/onefile_11874_1697322110_687089/pluggy/_manager.py", line 106 in _hookexec
  File "/tmp/onefile_11874_1697322110_687089/pluggy/_hooks.py", line 479 in __call__
  File "/tmp/onefile_11874_1697322110_687089/_pytest/python.py", line 1792 in runtest
  File "/tmp/onefile_11874_1697322110_687089/_pytest/runner.py", line 169 in pytest_runtest_call
  File "/tmp/onefile_11874_1697322110_687089/pluggy/_callers.py", line 27 in _multicall
  File "/tmp/onefile_11874_1697322110_687089/pluggy/_manager.py", line 106 in _hookexec
  File "/tmp/onefile_11874_1697322110_687089/pluggy/_hooks.py", line 479 in __call__
  File "/tmp/onefile_11874_1697322110_687089/_pytest/runner.py", line 262 in <lambda>
  File "/tmp/onefile_11874_1697322110_687089/_pytest/runner.py", line 341 in from_call
  File "/tmp/onefile_11874_1697322110_687089/_pytest/runner.py", line 261 in call_runtest_hook
  File "/tmp/onefile_11874_1697322110_687089/_pytest/runner.py", line 222 in call_and_report
  File "/tmp/onefile_11874_1697322110_687089/_pytest/runner.py", line 133 in runtestprotocol
  File "/tmp/onefile_11874_1697322110_687089/_pytest/runner.py", line 114 in pytest_runtest_protocol
  File "/tmp/onefile_11874_1697322110_687089/pluggy/_callers.py", line 27 in _multicall
  File "/tmp/onefile_11874_1697322110_687089/pluggy/_manager.py", line 106 in _hookexec
  File "/tmp/onefile_11874_1697322110_687089/pluggy/_hooks.py", line 479 in __call__
  File "/tmp/onefile_11874_1697322110_687089/_pytest/main.py", line 350 in pytest_runtestloop
  File "/tmp/onefile_11874_1697322110_687089/pluggy/_callers.py", line 27 in _multicall
  File "/tmp/onefile_11874_1697322110_687089/pluggy/_manager.py", line 106 in _hookexec
  File "/tmp/onefile_11874_1697322110_687089/pluggy/_hooks.py", line 479 in __call__
  File "/tmp/onefile_11874_1697322110_687089/_pytest/main.py", line 325 in _main
  File "/tmp/onefile_11874_1697322110_687089/_pytest/main.py", line 271 in wrap_session
  File "/tmp/onefile_11874_1697322110_687089/_pytest/main.py", line 318 in pytest_cmdline_main
  File "/tmp/onefile_11874_1697322110_687089/pluggy/_callers.py", line 27 in _multicall
  File "/tmp/onefile_11874_1697322110_687089/pluggy/_manager.py", line 106 in _hookexec
  File "/tmp/onefile_11874_1697322110_687089/pluggy/_hooks.py", line 479 in __call__
  File "/tmp/onefile_11874_1697322110_687089/_pytest/config/__init__.py", line 169 in main

Can you retry this one, I am pretty sure ultimately #2492 is a duplicate of this one, and let me know if it needs a re-open.

I can still reproduce this:
1.9.2
Commercial: None
Python: 3.11.6 (tags/v3.11.6:8b6ee5b, Oct  2 2023, 14:57:12) [MSC v.1935 64 bit (AMD64)]
Flavor: Unknown
Executable: c:\Build\project\.venv\Scripts\python.exe
OS: Windows
Arch: x86_64
WindowsRelease: 10
Version C compiler: C:\Build\NuitkaCache\downloads\gcc\x86_64\13.2.0-16.0.6-11.0.1-msvcrt-r1\mingw64\bin\gcc.exe (gcc 13.2.0).

===================================================================================== test session starts ======================================================================================
platform win32 -- Python 3.11.6, pytest-7.4.3, pluggy-1.3.0
PySide6 6.6.0 -- Qt runtime 6.6.0 -- Qt compiled 6.6.0
rootdir: C:\Users\bers\AppData\Local\Temp\ONEFIL~1
collected 1 item

..\..\..\Users\bers\AppData\Local\Temp\ONEFIL~1\test_crash.py Windows fatal exception: access violation

Current thread 0x00004634 (most recent call first):
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\pytestqt\wait_signal.py", line 193 in connect
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\pytestqt\qtbot.py", line 289 in waitSignal
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\test_crash.py", line 18 in wait
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\contextlib.py", line 137 in __enter__
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\test_crash.py", line 29 in test_crash
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\_pytest\python.py", line 194 in pytest_pyfunc_call
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\pluggy\_callers.py", line 27 in _multicall
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\pluggy\_manager.py", line 106 in _hookexec
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\pluggy\_hooks.py", line 479 in __call__
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\_pytest\python.py", line 1792 in runtest
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\_pytest\runner.py", line 169 in pytest_runtest_call
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\pluggy\_callers.py", line 27 in _multicall
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\pluggy\_manager.py", line 106 in _hookexec
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\pluggy\_hooks.py", line 479 in __call__
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\_pytest\runner.py", line 262 in <lambda>
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\_pytest\runner.py", line 341 in from_call
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\_pytest\runner.py", line 261 in call_runtest_hook
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\_pytest\runner.py", line 222 in call_and_report
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\_pytest\runner.py", line 133 in runtestprotocol
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\_pytest\runner.py", line 114 in pytest_runtest_protocol
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\pluggy\_callers.py", line 27 in _multicall
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\pluggy\_manager.py", line 106 in _hookexec
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\pluggy\_hooks.py", line 479 in __call__
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\_pytest\main.py", line 350 in pytest_runtestloop
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\pluggy\_callers.py", line 27 in _multicall
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\pluggy\_manager.py", line 106 in _hookexec
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\pluggy\_hooks.py", line 479 in __call__
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\_pytest\main.py", line 325 in _main
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\_pytest\main.py", line 271 in wrap_session
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\_pytest\main.py", line 318 in pytest_cmdline_main
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\pluggy\_callers.py", line 27 in _multicall
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\pluggy\_manager.py", line 106 in _hookexec
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\pluggy\_hooks.py", line 479 in __call__
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\_pytest\config\__init__.py", line 169 in main
  File "C:\Users\bers\AppData\Local\Temp\ONEFIL~1\crash.py", line 1 in <module>

Hm, very strange, maybe 6.6 didn't really fix it after all, it surely disables the workaround. Can you try with a 6.5 downgrade?

Can you try with a 6.5 downgrade?

Yes, same thing. Using nuitka 1.9.2 from pip:
=========================== test session starts ===========================
platform win32 -- Python 3.11.6, pytest-7.4.3, pluggy-1.3.0
PySide6 6.5.3 -- Qt runtime 6.5.3 -- Qt compiled 6.5.3
rootdir: C:\Users\bers\AppData\Local\Temp\ONEFIL~1
collected 1 item

..\..\Users\z004j05c\AppData\Local\Temp\ONEFIL~1\test_crash.py Windows fatal exception: access violation

Thanks, looks like there is more to find. It's strange since this is also the code that triggered me to realize that disconnect is the actual problem, and that seemed to solve some things.

I recently did another fix that is maybe on factory only right now, that revives a fix that I made. Does look to me like you bottom posted supposely fixed issues? I don't track those at all.

Does look to me like you bottom posted supposely fixed issues? I don't track those at all.

That's a pity because you may miss out on people thanking you :) but also, how else are people expected to tell you a fix is not actually fixing the issue?

I recently did another fix that is maybe on factory only right now, that revives a fix that I made.

Will re-test reproductions in both #2492 and #2499.

Both #2492 and #2499 are working on Windows now!

platform win32 -- Python 3.11.7, pytest-7.4.4, pluggy-1.3.0
PySide6 6.6.1 -- Qt runtime 6.6.1 -- Qt compiled 6.6.1

On Linux now, too!

platform linux -- Python 3.11.7, pytest-7.4.4, pluggy-1.3.0
PySide6 6.6.1 -- Qt runtime 6.6.1 -- Qt compiled 6.6.1

Both are using d6ae88493988b9391c4ac1ee941134eedb2cd997 from factory, by the way.
Interested in seeing if my full app works now, too!

Interested in seeing if my full app works now, too!

It does! Thanks a lot!

I think we are saying, this is fixed with 2.0 then, aren't we. And bottom posting is bad, because after I decided it's fixed, an issue can resurface indeed, but it can be a different one, with a similar symptom but often in a different setup, so I hate that.
One of my PySide fixes seems to have caused crashes for other cases, due to not being allowed to set name on some Qt methods without a crasher, and once that code was reverted, other problems resurface. I always need a full new report to stay sane and not spend time needlessly. My time in the project is limited. If I spend it on managing bottom posts, that might not be the most effective use of my time, and there is not enough time available.

I think we are saying, this is fixed with 2.0 then, aren't we.

I'd say yes - assuming current factory will be included there. I haven't check develop (and am not planning to, to be honest).

This is now on the develop and the commercial staging branch and part of the latest pre-release, will be part of the 2.0 release that will eventually follow.

This is part of the stable release 2.0 that I just made.


docs: remove redundant trailing spaces and newlines

This will likely force me to reapprove all the related localizations in crowdin. So, I don't think it is worth it

Do crowdin translation contains trailing whitespaces?

I just checked out crowdin page for pnpm, and unfortunately, its sources do contain trailing whitespaces.
If there is a way to download crowdin translation, we can run a script to automatically update all the sources to remove trailing whitespaces.

is there some linting for preventing trailing spaces in the future? maybe remark can be used

There are plenty of markdown linter if one searches for it in Google:

https://github.com/markdownlint/markdownlint
https://github.com/DavidAnson/markdownlint
https://github.com/robertohuertasm/mdlint
https://github.com/dprint/dprint-plugin-markdown

But the tool I used to create this PR was a simple Node.js script.

IMO, the formatting of markdown files in this repo doesn't matter. People can even edit the files in the github web interface. We don't care about the format. So, I'd just recommend to turn off autoformat in the editor for this repo.


No documentation for mapping storage to containers

A preview feature is to allow custom path mapping of storage accounts to App Service. The feature links to this page for more information on how to access mapped storage paths. These paths are not restricted to the /home folder, etc.

The Learn more link here references the Configure multi-container apps section and this addresses Docker Compose examples.
The problem is that there is an undocumented requirement that volume mounts begin with ${WEBAPP_STORAGE_HOME} even though path mappings are not constrained to this.
2019-10-13 01:28:30.553 ERROR - Exception in multi-container config parsing: (Line: 5, Col: 9, Idx: 66) - (Line: 5, Col: 32, Idx: 89): Bind mount must start with ${WEBAPP_STORAGE_HOME}.

Please add documentation and examples for mounting a separate storage account (not the built-in app service storage) using path mappings to /custom/path or similar path that is not under /home/

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

ID: 6489b871-9cf8-7c99-9834-076ec646d0e6
Version Independent ID: fa5d2995-5833-c04b-2530-c100bcd1e76b
Content: Configure custom container - Azure App Service
Content Source: articles/app-service/containers/configure-custom-container.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@mbrancato  . Thanks for the feedback .We have assigned the issue to the content author . They will evaluate & update the document as appropriate

Hi @mbrancato, we have information on mounting storage:
https://docs.microsoft.com/en-us/azure/app-service/containers/how-to-serve-content-from-azure-storage As you mentioned, it's in preview.
What page links to this one for mounting storage? Please let us know so we can update it.

@msangapu-msft It's the fwlink in the portal page: https://go.microsoft.com/fwlink/?linkid=2088516. Can you get it redirected?

Looks like @btardif owns the link. Byron, can you help update the FW Link (2088516) to: https://docs.microsoft.com/en-us/azure/app-service/containers/how-to-serve-content-from-azure-storage

I have updated the fwlink as requested. #please-close


Missing documentation ?

Hey,
The link supposed to show the documentation is only displaying samples, but without any code ?
Why ?
Regards

It's only because I didn't take the time to do it at the moment.

Code sample are available in git repo : https://github.com/KoRiGaN/Vue2Leaflet/tree/master/examples

https://github.com/KoRiGaN/Vue2Leaflet/issues/185#issuecomment-423886038

Closing this since new docs are just rolled out! (feedback welcomed)


Revert "Install latest policy-framework"

Policy framework 0.10.0 was released last week, so we can use it instead of latest version.
This reverts commit df2533b0ee0370d672c798a9855dae509e5a3865.

Testing the change:
$ drenv start ocm.yaml
2023-02-23 03:05:00,337 INFO    [ocm] Starting environment
2023-02-23 03:05:00,339 INFO    [hub] Starting cluster
2023-02-23 03:05:01,339 INFO    [dr1] Starting cluster
2023-02-23 03:05:02,340 INFO    [dr2] Starting cluster
2023-02-23 03:05:42,950 INFO    [hub] Cluster started in 42.61 seconds
2023-02-23 03:05:42,950 INFO    [hub/0] Running ocm-hub/start
2023-02-23 03:06:07,900 INFO    [dr1] Cluster started in 66.56 seconds
2023-02-23 03:06:07,900 INFO    [dr1/0] Running ocm-cluster/start
2023-02-23 03:06:29,507 INFO    [dr2] Cluster started in 87.17 seconds
2023-02-23 03:06:29,507 INFO    [dr2/0] Running ocm-cluster/start
2023-02-23 03:07:00,000 INFO    [hub/0] ocm-hub/start completed in 77.05 seconds
2023-02-23 03:07:00,000 INFO    [hub/0] Running ocm-controller/start
2023-02-23 03:07:16,912 INFO    [hub/0] ocm-controller/start completed in 16.91 seconds
2023-02-23 03:08:26,729 INFO    [dr1/0] ocm-cluster/start completed in 138.83 seconds
2023-02-23 03:08:26,729 INFO    [dr1/0] Running ocm-cluster/test
2023-02-23 03:08:32,247 INFO    [dr1/0] ocm-cluster/test completed in 5.52 seconds
2023-02-23 03:08:41,644 INFO    [dr2/0] ocm-cluster/start completed in 132.14 seconds
2023-02-23 03:08:41,644 INFO    [dr2/0] Running ocm-cluster/test
2023-02-23 03:08:45,949 INFO    [dr2/0] ocm-cluster/test completed in 4.31 seconds
2023-02-23 03:08:45,950 INFO    [ocm] Environment started in 225.61 seconds

$ kubectl describe deploy -n open-cluster-management --context hub | grep policy | grep Image
    Image:      quay.io/open-cluster-management/governance-policy-addon-controller:v0.10.0
    Image:      quay.io/open-cluster-management/governance-policy-propagator:v0.10.0


Use gradle instead of maven

This PR disables maven support and moves the project to be managed with gradle. This enables us to:

delete all files in /.idea
let gradle generate a pom.xml
extend build.gradle easily

After merging this PR the project files will be generated with gradle. Just open a shell or cmd and execute gradlew idea on the project root. This will generate all needed files and also integrate google-style and file-encoding.

Hi @serious6, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;

:+1: to get rid of /.idea files from source control!

ref. #235

closed for now


Add spinning icon next to npm under dependencies node in Visual Studio

While creating a new ASP.NET CORE 2.0 MVC application with Angular, the npm package restore takes a lot of time. It takes around 3 minutes. Is this expected? Am I doing something wrong? I have latest VS 15.7.1 update installed and have windows 10 anniversary update. If this is normal then I have a feature request.
Feature request.
What if just like the latest spinning icon that were added for test explorer window, if there was a similar spinning icon next to npm under dependencies tree. It will give a visual indication that something is still processing. There is a processing icon at the bottom left but it is not obvious as a first time user. When I see yellow mark, it feels like I have done something wrong.

@barrytang, would this go through VS feedback?

Yes, to report VS issues, use the Report a Problem tool.


gnupatch: update from 2.7.1 to 2.7.3

Potentially fixes CVE-2015-1196.
Signed-off-by: Edward O'Callaghan<EMAIL_ADDRESS>

I believe this is a mass rebuild, cc @vcunat

Yes, gnupatch is included in our stdenv.

Test builds will run at http://hydra.nixos.org/jobset/nixpkgs/mass-rebuild-update.

@peti why not staging?

Because staging is not for testing. Staging is for patches that have been tested already and that can be merged at any time we want without a risk of breaking master.

Well, yes, it's good to first try building at least a few complex packages before merging to staging (to keep it manageable). I typically switch my system to such changes before pushing them.

The results from Hydra look fine. There are additional failures compared to trunk, but those appear to be unrelated to this patch, i.e. most failures occur because our revision has an invalid URL for imagemagick. I believe this patch can go to staging. Does anyone disagree?

I tested as well, and pushed.


set filter chain name to per request stream info

Commit Message: set filter chain name to per request stream info
Additional Description:
to close #28775
Risk Level: low.
Testing: n/a. wait.
Docs Changes: n/a.
Release Notes: n/a.
Platform Specific Features: n/a.

It's no doubt that the filter chain name will have longer life time than all related connections/requests. So, seems store a string view in the stream info object is enough to avoid unnecessay (and possible) stack allocation.
This is a minor ajustment. If the community think use std::string is better, I will revert it.

/wait-any

Looks like this needs a main merge, at least.
/wait

According to previous dicusssion, the implementation is refactored completely. So a force push was used to refresh the code.

friendly ping @ggreenway


Remove all initial metadata and add title has ERC handbook

No need to remove all the metadata. For the pages that we are keeping(like documentation) we can edit the text as and when we add the contents


Multiprocess scheduler handles unpickling errors

Previously if a task failed to be cleanly deserialized, the error wasn't handled gracefully and it would cause the scheduler to lock up. This fixes that issue by special casing the apply_async implementation in dask.multiprocessing. This means that it no longer matches the call signature of pool.apply_async, but in reality we always call apply_async with the same arguments. The special casing allows for deserialization errors to be handled the same as all other errors, and allows for a clean exit.
Also did a few small pep8 cleanups in the touched files.
Fixes #1342.

Errors are due to new bokeh release. Fixing in a separate PR.

I have a slight preference to remain consistent to the standard interface if easy.

I've pointed people to this implementation before when they've needed to handle multiprocessing with dill

Unfortunately it's not easy, as we need access to the queue (and the execute_task function) to be able to handle errors in a clean way. I'm not sure of another way to fix this.

Oh well. OK.  +1 on the general idea then.

Merging later today if no comment.


typo

This change is

signed

/lgtm

@mingyang91 thanks

Coverage remained the same at 46.597% when pulling 721b228cefd441f04c3a26a3838e015a7328495e on mingyang91:patch-1 into d13b293619b979bdf1c71c3471b7d0fb7eef1fbc on kubernetes:master.


$.extend() does not with with noconflict

I am using jQuery in noconflict mode and I was getting a error when trying to the fileuploader.
on Line 3937 data.headers = $.extend({}, data.headers, { where the error was occurring. I changed it to data.headers = jQuery.extend({}, data.headers, { and it works

I'm sorry for my response delay @hrtrulz.
Can you please make sure you're using the latest version of our library? It seems that line 3937 is different than what the code line you referred to (changed in 2.1.3 last September).

Closing this issue due to the time elapsed. Please feel free to either re-open the issue, contact our support http://support.cloudinary.com or create a new ticket if you have any additional issues.


Add 'show/hide totals' option to pivot table vis

Allow user to disable the totals row/columns on pivot tables.

Awesome. Thanks for the contribution!


The earliest date on tonmon.xyz

Didn't find where it is better to ask the question: the earliest date available on tonmon.xyz is July 3rd, 2021, how does this date relate to TON blockchain?

Nothing special, just switched to another database.


Report on how many dotfiles/resource forks

So I've a question that relates a bit to #83 , I often see several thousand AppleDouble Resource Forks in reports - the donor doesn't expect to donate these to us, they're dotfiles so they're hidden on MacOS, so outside of a disk image or zip, they won't be selected for preservation. I'd love for demystify to be able to say:

How many files are Resource Forks
How many files are dotfiles/hidden files
Perhaps how many windows ADS files too?
How many files are neither resource forks or dotfiles

More thought might be needed in terms of how this might be most useful, and what kinds of other files might need to be lumped into the aggregate value.
I know that sometimes resourceforks and dotfiles will be selected for preservation in some contexts, but it would be good to know how many are there, and how many files are not resource forks or hidden files.
Curious to know what you think anyhow - just tried demystify-lite for the first time and I'm so impressed!

I can conceive of adding this in a section on it's own, it's a good idea. Will also have a think. If you have any good Siegfried/DROID exports with some of this information in, it will be a great help, and we'll anonymize that and put it into the unit tests.
Also, in the meantime, perhaps have a look at how the denylist impacts the output for you and see if it begins to get close to what you're thinking.

https://github.com/exponential-decay/demystify/blob/c8f527585f74d3cdeee71c112e4773e531df67ad/denylist.cfg

Regarding implementation, then there's some work to do to make each section more customizable, something like the strategy pattern to enable different sections to be configured for use, and output, as desired by the caller. That's a little bit further down the line once the Py3 release is stabilized and packaging works. Until then, adding another section might feel quite onerous for anyone taking it on - but I'm also more than happy to take a look with the right data to help support it.


Support for default tags in azurerm_firewall_policy resource

Overview/Summary
Changed from null to local.tags for the Firewall Policy (both the traditional hub-spoke edition and the Virtual WAN edition). With this change the default_tags are also applied to the Firewall Policy.
This PR fixes/adds/changes/removes

Adds support for default tags for the Firewall Policy resource.
Fixes #627

Breaking Changes
None.
Testing Evidence
Before the change:

After the change:

As part of this Pull Request I have

[X] Checked for duplicate Pull Requests
[X] Associated it with relevant issues, for tracking and closure.
[X] Ensured my code/branch is up-to-date with the latest changes in the main branch
[X] Performed testing and provided evidence.
[ ] Updated relevant and associated documentation.
[ ] Updated the "RELEASE" page with details needed for when this code change is released.

Thanks for the PR @robertbrandso
Will get this merged soon

/azp run unit

@robertbrandso The terraform plan failed. Did you test this locally?
We are getting the error on the firewall policy resource:
a tag key "deployedBy" expected to be all in lowercase

@matt-FFFFFF Confirmed working when testing locally (with Virtual WAN configuration):

@robertbrandso I see it's an issue with older provider versions. Looks like we may have to raise the minimum :(

I see. No worries. Release in the next major release of the module then?

@microsoft-github-policy-service agree

raising min azurerm verison to 3.35.0

/azp run unit


Incorrect size of modal on tablet (V3.3.6)

If the size of container is more little than modal, the modal is not visible correctly.
A demo here :
http://www.kalaninformatique.fr/modal/demo.html
Test with windows 7 : IE, Firefox, Safari, Chrome => no problem
Test with Samsung Galaxy : Chrome => ok
Test with ipad : Firefox, chrome and Safari =>** BUG**

Tested with v4, the same problem.

@ldesmeules

What version of iOS are you using?
What model of iPad are you using?
Could you post a screenshot of the problem?

iOS 9.2.1 Ipad2

IOS 9.2.1 Ipad2
On computer:

On phone samsung

BUG on IPAD2:

Apparent duplicate of https://bugs.webkit.org/show_bug.cgi?id=150715


[HOLD on prioritizing P2P] IOU - Button doesn't show the expected text "Pay" or "Pay Elsewhere"

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

Case 1 - Short Pay button label in the report preview
Action Performed:

As userA, request $10 from userB
As user B, check the button in the IOU

Expected Result:
The IOU should display only one button with no dropdown option saying simply "Pay". Expected result extracted from this PR.
https://github.com/Expensify/App/pull/19817
Actual Result:
The button displays "Pay Elsewhere" instead of "Pay".

Case 2 - Pay Elsewhere shows instead of I'll settle up elsewhere in case of expense reports
Action Performed:

UserA creates a workspace and invites UserB as a member
UserB requests money from this workspace
As UserA (admin), go to the expense report
Verify you can see Pay elsewhere option and you cannot see I'll settle up elsewhere option

Expected Result:
The IOU should display only one button with no dropdown option saying simply "Pay Elsewhere". Expected result extracted from this PR.
https://github.com/Expensify/App/pull/19817
Actual Result:
The button displays "Pay" instead of "Pay Elsewhere" and when navigating to the IOU, the user can see the option to "Pay Elsewhere" in the dropdown.

Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: 1.3.24-1
Reproducible in staging?: Yes
Reproducible in production?: No, this is a new change to the button
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal team
Slack conversation:
View all open jobs on GitHub
Issue found while executing PR https://github.com/Expensify/App/pull/19817 - Assigning @mountiny as he was the author of the PR.

@isagoico Thanks for raising it but these are expected, maybe it was not clear in the test steps, sorry it was a lot of changes and hard to describe.
The single Pay button from the chat should show:

Pay if the methods are Wallet or VBBA which would be used upon click
Pay with Paypal if the method is paypal
Pay elsewhere if the method used would be elsewhere

Then the Expense report detail (workspace request) pay button options should be or:

Pay with Expensify
Pay elsewhere

In 1:1 request you wont have the Pay elsewhere, but you will have I'll settle up elsewhere

I will close this out, I believe there are no testrail steps to update right now, but if yes, please let us know in slack


Code quality settings for developers

pre-commit hooks (if installed) are run every time you try to commit they check:

the ordering of imports at the top of the file (isort configured in .isort.cfg)
coding style conventions are followed (black configured in pyproject.toml)
some pylint rules are adhered to (flake8 configured in setup.cfg) e.g. no unused imports
the process is configured in .pre-commit-config.yaml

The black coding style is rigid, and I quite like that.  Basically, it insists on:

Strings with double quotes
A specific kind of line continuation within brackets
spaces between shit

To check this change hasn't broken stuff, please may you:

Check the install instructions for both the standard install and with the dev option pass all tests
With the dev environment, try and commit things which are invalid to see how pre-commit hooks work.

Tests pass, ACME created correctly.


Allow dropdownToggle directive to inherit parent scope

When paired with another directive on the same element, Angular may complain about multiple directives trying to create a new scope.  This directive (dropdownToggle) doesn't seem to really require its own scope, so we should be able to remove it and get the information we need from the "attrs" parameter instead.

:+1: Looks good to me. Thanks!


How to handle scientific notation for timestamp

I have data points in the following format and I want to generate alerts when service was started in last 15 seconds for that I need to take a diff of the current time and the gauge tag in the following points, I could not find anyway to do so , any help is appreciated.
time                gauge        service
----                -----            ---
1512435230000000000 1.512421379e+09  test-service

New 1.04 release has now() and unixNano() function which can be used for such purposed, closing it.


Added resource for AWS

Added resource for AWS (official documentation for AWS Cloud)

Thanks for contributing, it looks great!

Glad that I was able to contribute some resource. These prove to be helpful to students in the long.


Memory Leak

We noticed that our pipeline was leaking memory equal to the audio/video data
passing through it. The BEAM was not reporting a leak, ps -o rss was. For this
reason, we suspected the leakage was to be found within the nifs.
This mail
exchange
made us double check the av_interleave_write function and we noticed that
packet->data field was set directly: libav tells instead to add a reference
counted buf to the packet so that the data payload can be freed by the library
as soon as nobody is referencing it.
We probed the memory, draw some graphs: the leakage is here!

Solved with #31 !


feat(DRFT-1361): Add systemd fact to policies

This PR addresses: DRFT-1361

/retest

/retest

@jpramos123 please follow https://issues.redhat.com/browse/RHINENG-775 for the test failures

/retest


pour_bottle?: Pour local bottles without sha256

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[x] Have you successfully run brew tests with your changes locally?

Pouring a local bottle for a formula without a bottle sha256
in the formula for that OS would unexpectedly install from
source instead.

Thanks again @sjackman!

My pleasure. Thanks for the feedback, and thanks for merging, Mike.


Add SetPositionEntityModifier and SetShowArmsEntityModifier

Hi, I added these two modifiers and also bumped the version of a few things.
SetPositionEntityModifier's syntax is a little different from vdvman1's suggestion because I found it a better fit as it is now. It can also be modified a bit to also include rotation since the methods for teleporting also require it but I left it out for now.
This is kind of a learning exercise for me as I'm not very familiar with the language, Minecraft's codebase and PRs, so I'm looking forward to see what I could have done better. I'm Velizar#8409 on Discord. Cheers!

I just thought of another way that solves the problem and would like to know your opinion about it. This method utilises the position types used in commands (DefaultPosArgument and LookingPosArgument) in a similar way, except here we make use of a registry.
This allows for any combination of absolute and relative coordinates and doesn't get in the way of incompatible arguments (world and local) and different naming (x, y and z and left, up and forwards)
Examples are
{
	"function": "minecraft:set_position",
	"position": {
		"type": "minecraft:world",
		"x": {
			"value": 1,
			"relative": true
		},
		"y": {
			"value": 2,
			"relative": false
		},
		"z": 3 // shorthand for {"value": 3, "relative": false}
	}
}

This would be ~1 2 3.
{
	"function": "minecraft:set_position",
	"position": {
		"type": "minecraft:local",
		"left": 1,
		"up": 2,
		"forwards": 3
	}
}

This would be ^1 ^2 ^3.

Heya, I'm here again. (After a way too long time, busy stuff ftw)
I think it might be better to have one pull request for each feature (so one for minecraft:set_position and one for minecraft:set_show_arms) in case one needs additional changes. It would keep everything a bit tidier. So I'm kindly asking you to submit a new pull request specifically for minecraft:set_position and/or minecraft:set_arms_visible if you would want to. (After all, it's been almost 5 months, so I get it if you don't want to redo all of this.)
I'll make an issue for minecraft:set_position with the current draft as I still like it.


Performance/reduce clones

Looks like there's a CI issue. Let's get that fixed before we merge.

Nice. Have you considered using ref+lifetime instead of Rc? I know lifetimes can kinda get out of hand, so I don't blame you if you want to stick with a happy medium.

@MitchTurner I'm down to try ref + lifetime. I wanted to go that route originally but then had issues constructing the ast in the parser, I got confused lol. Let's use this improvement for now and make it a thing that we intend on optimizing even further?

Nice. Have you considered using ref+lifetime instead of Rc? I know lifetimes can kinda get out of hand, so I don't blame you if you want to stick with a happy medium.

I'd be more than happy to switch as well. Just got to deal with lifetimes, but I'll need some help with that.

Yeah. I think it is a good goal in the long run. But there is no rush.
I'd be down to pair on it some time. Don't want to hold this up.


Add go1.16.15 patches

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve

closes https://github.com/aws/eks-distro-build-tooling/issues/979 , https://github.com/aws/eks-distro-build-tooling/issues/980 , https://github.com/aws/eks-distro-build-tooling/issues/981


How to add different highlight for dunder methods

Hi there,
How can we add different highlight for __dunder__ methods a.k.a. magic methods? Is there a TS highlight group for that?
I know there is pythonTSConstructor for __init__ method that I can customize, but I couldn't figure out for other magic methods.

Hi, they are currently under the TSConstBuiltin hl group,
https://github.com/nvim-treesitter/nvim-treesitter/blob/5c364707442965ae52a1ac89c359080e32958445/queries/python/highlights.scm#L16-L17
Also, you should ask this at the https://github.com/nvim-treesitter/nvim-treesitter repo.

Sorry my later reply.
No, it's not TSConstBuiltin. It appears that it's for const/var like __name__.
This regex matches with dunder methods as well, but they are overwritten by TSMethod: image
Oh, ok, I thought opening here since specific to python would be more appropriate. Closing this then.
P.S:
Lukas gave some insights in this thread: thread


"aria-controls" attribute on buttons

Hello,
In order to allow assistive technologies to focus on content, there should be an aria-controls attribute on buttons with the "id" of the content to expand as value.
More details: https://www.w3.org/TR/wai-aria-practices/#accordion

Hello,
it is already set up, at the beginning of the script:
    var attr_control = 'data-controls',
        attr_expanded = 'aria-expanded',
        attr_labelledby = 'data-labelledby',
        attr_hidden = 'data-hidden',

However, for "practical reasons" (especially on voice over, to avoid application mode if I remember well - ping @goetsu you will probably be able to explain why), it has been decided not to follow exactly the DP.
;)

I understand the reason but another accessibility expert, in an audit, told me to add "aria-controls", so...
I don't like to modify a library but if we have to... :-)
Thank you for your answer.

@juliemoynat you don't have to modify it because it's already something you can choose to add or not in the configuration ;)
@nico3333fr yes you remember well

In fact, if I remember well, for Voice over: if it detects certains ARIA attributes, it switches to application mode, and this mode implies more complicated keyboard combinations for the user.
As this component should be simple, this is why we set it up like mentionned before: it works without being more complicated for the users (difference between "accessibilité de terrain" - don't know how to say it in english and "accessibilité des référentiels").
However, your a11y expert and mine's are not wrong, it depends on the environment of the project and the point of view.
In a near future, I will rewrite this plugin in the same style of coding that the accordion one, allowing to pass a config on the call, and it should fix this issue without updating the code of the plugin.

Ok, so I was not so wrong about the fact that we have to modify plugin's code. Adding a variable will not just work, so we have to modify several lines of code (or maybe I'm wrong about the way I should do it...?).

At the beginning of the script, you need to replace
var attr_control = 'data-controls',

by
var attr_control = 'aria-controls',

that's all ;)

Ok!! So cool :-)
Thank you


Feature: improving the create insight page

Type of feature
🍕 Feature
Current behavior
The page right now does not have a title for the added repositories neither showing the create page from first look, which could be confusing:

https://insights.opensauced.pizza/hub/insights/new
Suggested solution
It would be great to:

have a title for the repos
have the create page button visible always

Additional context
I got this when I tried to understand the UX from this issue: Related to: https://github.com/open-sauced/insights/issues/1425
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[ ] I agree to follow this project's Contribution Docs

if we go with quick fix we can do the next:
When no repo is added (with suggestions):

When no repo is add (without suggestions):

.take

Closing as resolved


How to set hyper-parameters? "The right recipe!"

Hi
@DMTSource
@awjuliani
Is there a way to set hyper parameters?

Reward value
Parameter Initialization method
LSTM length
Learning rate
Optimizer (Adam or RMSProp)
Gradient Clipping value

After 10s of experiments, I found that any tiny change in one of these affects the whole training dramatically, usually in a bad way.
It is also not logic to conduct a grid search over different parameters, because a single experiment may take hours or days, and cost a lot of money
One trick I usually use, is to use large network and dropout to reduce/eliminate over fitting, but what about all of the above?
Another trick, try to adjust the learning rate * gradient = 1e-3 parameters. (In other works make the parameter update around 1/1000 of the parameter value, to prevent too large to too small updates)
What do you recommend?

Very common issue in machine learning in general. Some quick reading might show you suggestions like
baysean > random > gridsearch. That is a SUPER simple view, cause I have had seen many try baysean opt and fail, more so in architecture vs hyperparam opt.
I would DEF not opt things yourself or use a grid search. Just let it run via Spearmiint for a few days to save your sanity.
https://github.com/HIPS/Spearmint

@DMTSource  thank you, helpful as usual :)
Moreover, I have seen the following
This short video
and this nice article
What are super simple ways to perform optimization like Spearmint?

The Spearmint examples show how to place you code inside a function and then call it from another. This is used by Spearmint to automatically determine the best values for whatever you let it optimize.
For example you could set ranges for your: learning rate, skip rate, gamma and max experience buffer len, and then have Spearmint automatically, "intelligently"(Bayesian) explore the param space in a quicker way than a grid or random search might. The trick would be evaluation. You would have to take the max of the average score or some other grading factor to judge each experiment against each other in a meaningful way. If you use multi factors you could try (top score, time to score) and try to discover a network that works best for you. The idea being even if it took a month its totally automatic. Smartish Brute force, woooo!
If you have the resources, Spearmint can speed up its efforts by running multiple experiments at once.


Added fix for Error: Expected only one oneOf fields to be set: got 0

@sasikeda Can you please review the changes.


[web] Images pixelated on CanvasKit

Hi there, I wanted to raise this issue again since it is locked:
https://github.com/flutter/flutter/issues/79846
The issue seems to be evident even on the sample apps, for example rally:
https://gallery.flutter.dev/#/rally/login
This is what the rally logo looks like on my screen:

If it is not the same issue with the sample app, I am happy to share a sample of my app with the issue.

possibly related to https://github.com/flutter/flutter/issues/89605

@adendiamond Thank you for raising this issue and @iapicca thank you for the additional information on it :) I can see the reproducing on the Gallery app via the link OP shared of the Rally app. I'll label this issue for further insights from the team.
cc: @yjbanov

Yep, on web it would seem any image is rendered in a very pixelated way, yet on desktop and mobile platforms the images are rendered smoothly and resized with appropriate antialiasing. Looks as though the filterQuality is ignore as it produce zero noticeable difference on web regardless of the setting. This is on Flutter 3.7.0-1.1.pre • channel beta

Image on the left that is originally a 1024x300 png and text font rendered by the web app on the left.

Vomit !!

Hi there, I wanted to raise this issue again since it is locked: #79846
The issue seems to be evident even on the sample apps, for example rally: https://gallery.flutter.dev/#/rally/login
This is what the rally logo looks like on my screen:
If it is not the same issue with the sample app, I am happy to share a sample of my app with the issue.

Thanks for filing the issue. Would you be able to share your sample app with the issue?

I tried to repro the issue locally with the latest version of gallery and the images are not pixelated like I see with the gallery from the link. If the repro with your code has the same pixelation issues that I can repro locally, that might help me figure out whats wrong.

You don't have to use the repo, just run the gallery on any browser. It looks awful and precious screenshots are an example.
If you are saying a new version of the gallery doesn't have this problem, then what changed to fix it? As surely the live web version should have that fix too right?
Are you saying that Chrome or Safari currently show the Gallery published on the website with perfect resolution image?

You don't have to use the repo, just run the gallery on any browser. It looks awful and previous screenshots are an example.
If you're saying a new version of the gallery doesn't have this problem, then what changed to fix it? As surely the live web version should have that fix too right?
Are you saying that Chrome or Safari currently show the Gallery published on the website with perfect resolution image?

Yes, the gallery published on the website does have the pixelated image issues. However, I believe the gallery on the website is not using the latest versions of flutter. When I clone the gallery code and run it locally using the latest versions of flutter, I do not see any pixelation issues.
As far as what's fixed, that's what I'm not sure about, and where repro code of some kind could help. I'd need something to repro locally in order for me to properly debug the issue. I'm guessing it could be something that was fixed on the skia side though.

What are you using when you say "latest version of Flutter" specifically?
I'm seeing issues as per my comments above and I state which specific version I'm using.

What are you using when you say "latest version of Flutter" specifically?
I'm seeing issues as per my comments above and I state which specific version I'm using.

The master channel being the latest. If you have sample code that has the pixelated issues that I can clone locally I'd love to take a look.

That's disappointing that you can't seem to determine what changed in Dev to confirm this was fixed 😞
I'll try to put together a repeatable example, but what for, how is that likely to help if the dev release has it fixed?? Are you  confirming you will be able to dig    out the fix and push it into stable or beta??
Seems like a waste of time - if you're sure it's working in Dev then close this (note that this was stated before and it was wrong then too)

That's disappointing that you can't seem to determine what changed in Dev to confirm this was fixed 😞

that's the hard part, as mentioned above this issue keeps coming back

if you're sure it's working in Dev then close this (note that this was stated before and it was wrong then too)

@gslender I disagree on that!
@exaby73
even if the issue is fixed in master
a test (maybe golden) should be added to prevent this issue from re-emerging
(thing that happened consistently, see link above)

if you're sure it's working in Dev then close this (note that this was stated before and it was wrong then too)

@gslender I disagree on that!

Sorry... I must have misunderstood the point of this post/comment then !?

The issue won't be closed yet. Still actively investigating what the issue could be. Yes, a golden test of some kind should be added regardless if the issue has been fixed or not so that we don't regress in the future.

Running the gallery app locally on the latest master does not reproduce the issue, but the one hosted on gallry.flutter.dev, still does. @alanwutang11 If this issue is fixed, maybe raising another issue to include the golden files would be more suitable than clubbing it with this issue?

Running the gallery app locally on the latest master does not reproduce the issue, but the one hosted on gallry.flutter.dev, still does. @alanwutang11 If this issue is fixed, maybe raising another issue to include the golden files would be more suitable than clubbing it with this issue?

there you go

https://github.com/flutter/flutter/issues/118883

cc @exaby73 @alanwutang11

I am closing this issue as it seems to be fixed. Thanks @iapicca for raising the issue for adding the golden files. @alanwutang11 If you disagree with closing this issue, let me know and I can reopen the issue. Thank you


xtensa/esp32s3: Add USB OTG device driver

Summary
Add ESP32-S3 USB OTG device driver.
Impact
Testing
apps/examples/usbserial.

@donghengqaz hi, sorry for commenting on this old commit.
It seems this defconfig is not working. Do you have any idea why this could be?
I would like to get this up and running, any help would be appreciated!

@fdcavalcanti did you try to use an old version from the time @donghengqaz submitted it?

I did not. But I know its failing for some time since the first internal failure report is from Aug/23.


Add dependabot configuration for keeping dependencies current

This change adds a Dependabot[1] configuration to keep dependencies up-to-date. The routine will check files like https://github.com/amazon-ion/ion-python/blob/master/requirements.txt for any updates.
[1] https://docs.github.com/en/code-security/dependabot
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

one option is to temporarily ungroup the updates until they're mostly updated. could help move https://github.com/amazon-ion/ion-python/pull/344 forward


Add HNSW PQ configuration to perf tests

Description
Adds HNSW PQ perf test config to release configs.
Ran on my own machine and got the following results:
  "results": {
    "test_took":<PHONE_NUMBER>9604654,
    "delete_model_took_total": 74.6723180054687,
    "delete_index_took_total": 91.5388090070337,
    "train_model_took_total": 98810.09195902152,
    "create_index_took_total":<PHONE_NUMBER>066527,
    "ingest_took_total": 67643.88565102126,
    "refresh_index_store_kb_total":<PHONE_NUMBER>96875,
    "refresh_index_took_total": 25093.700887984596,
    "query_took_total": 101655.0,
    "query_took_p50": 10.0,
    "query_took_p90": 11.0,
    "query_took_p99": 12.0,
    "query_took_p99.9": 13.0,
    "query_took_p100": 99.0,
    "query_client_time_total": 121304.0,
    "query_client_time_p50": 12.0,
    "query_client_time_p90": 13.0,
    "query_client_time_p99": 14.0,
    "query_client_time_p99.9": 41.0,
    "query_client_time_p100": 228.0,
    "query_memory_kb_total": 172384.0,
    "query_recall@K_total": 0.629275,
    "query_recall@1_total": 0.9948
  }

Check List

[X] Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Let make sure that we add the configuration for filters also with HNSW-PQ for faiss. Not in this PR, we can take it as a separate PR.


Refactoring the cli interface and implementing methods corresponding to the String data structure

At present, the put, get, and del methods in the cli are all forced by string. These methods have limitations and cannot be connected to subsequent different data structures.

We need a unified interface now, and then implement these interfaces using these different kinds of data structures.
Now that the basic methods of the String data structure have been built, you need to complete the various methods in the String data structure in the cli.
See #135 for the method corresponding to the String data structure

it may can be closed?


catchfilebeetweentags

I tried "example.tex" in your export folder, in order to provide a paper from selected "Zetteln". The package catchbeetweentags swallows some "eol", at least with texlive 2021 on my Mac.
\usepackage{catchfilebetweentags}
\usepackage{etoolbox}
\makeatletter
\patchcmd{\CatchFBT@Fin@l}{\endlinechar\m@ne}{}
{}{\typeout{Unsuccessful patch!}}
\makeatother
works for me.

Thanks for pointing this out, I hadn't noticed - I'll add that to example.tex.


Generated CHANGELOG.md

Generated via https://github.com/skywinder/github-changelog-generator
Important: the latest released version does not have support for "Breaking changes", so I compiled the gem from master branch.
Usage:
$ export CHANGELOG_GITHUB_TOKEN="...."   # (optional, see their README.md)
$ github_changelog_generator

What do you think?
#319.

@salmanulfarzy you've published a few releases by now, all of them have a quite good changelog - how was the experience? Does it consume a lot of time composing the changelog? Are you willing to continue doing this?
Manual changelog will always have higher quality than an autogenerated one, so if you are going to continue taking care of changelog we can close this PR.

I'm just adding release notes, We still need changelog. Also, It would be better without unreelased section.

@maximbaz would you be interested in updating the CHANGELOG ? It would be great if we keep don't keep GitHub releases as single source of changelog and have one local copy.
Seems like there are some active development with github-changelog-generator, Would it possible for use if for our use case ?

CHANGELOG.md

This one was modified from GH release notes with some manual changes, headers to titles added manually.
➜ GET https://api.github.com/repos/denysdovhan/spaceship-prompt/releases > changelog.json
➜ jq -r '.[] | .name, .body' changelog.json >  CHANGELOG.md
➜ sed -i -e 's/:.*: /g' CHANGELOG.md

Agree that this was ugly way and there are tools like IonicaBizau/json2md which can be used for converting json to md. Issue references were  relative and won't work locally.

Honestly I lost interest in this, with awesome and descriptive Github releases you are making I don't see the need to bother with additionally generating changelog (also, having different changelogs will be confusing 😉 ). You can simply copy-paste the text from release to a CHANGELOG.md file if you want to have a copy.
Feel free to overtake this PR if you want to continue, otherwise just close this.

In that case closing this now, I was thinking about something that reflects our releases page.


Fix DexUnreachable raising an exception when being evaluated

Cooouuulllld we have a little more in the commit message about what's actually wrong? (It's non-obvious to me, would be good for everyone else to see)

Amended the message.

Cheers, LGTM.


Change font from comic sans in diagrams

Would it be possible to change the font away from comic sans in the architecture diagrams?
e.g. https://github.com/apache/incubator-mxnet/blob/master/docs/architecture/note_engine.md
The docs are otherwise excellent and very helpful.
Thank you!

This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this is still an active issue. Thanks!
Also, do please check out our forum (and Chinese version) for general "how-to" questions.

Reopening - this is still an issue:
https://mxnet.incubator.apache.org/api/python/symbol_in_pictures.html

@altosaar this was most likely a design choice to avoid discrimination against dyslexics.
As a British Dyslexic myself I fully support the recommendation from the The British Dyslexia Association to use this typeface for ease of reading: see https://bdatech.org/what-technology/typefaces-for-dyslexia.
As a matter of fact, my code editor's font is set to comic sans. :)

Wow, I had no idea about that - fascinating. Thanks for sharing @thomelane :) I'll keep that in mind for inclusive design moving forward.


⚠ Samsung has degraded performance

In a4efb35, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2706 ms

Resolved: Samsung performance has improved in 73ed4fb.


Add more Split Column

Hi,
tanks for your excellent plugin,
is there any way to split a bucket by more than one column?
for example, I have a bucket consist of College, field, sex, grade, I want to have a column of the count of students by their sex and another column of the count of students by their grade while my rows are split by College.

Hi,
The plugin does not allow to have multiple "split cols" bucket.
That said, given your case, you could achieve your need, doing so:

a split rows bucket by college
a split cols bucket by grade
a "Max bucket" metric, with "filters" bucket aggregation and one filter "sex:male", count metric aggregation and custom label "male"
another "Max bucket" metric, with "filters" bucket aggregation and one filter "sex:female", count metric aggregation and custom label "female"

You should get one column per duo "grade - sex"

Hi,
Thanks a lot,
You're amazing.

Nice to see that you appreciate my work ;)
If your issue is solved, thanks to close the issue.

i'm very happy to know you , and thank you for your excellent knowledge sharing. can you give me your email address ? i'm very thankful if i ask some more questions. could we being in touch in Persian ? i am waiting for your response.

Hi,
<EMAIL_ADDRESS>


Appengine SDK install warning opaque

The Eclipse preferences for Google Cloud tools, google cloud sdk directions to install app engine sdk are opaque.
Installing the cloud tools, with app engine sdk is needed. The url endpoint is a bit generic.

What I had to do to install the app engine sdk. New users that find instructions like this might find it helpful.

This duplicates: https://github.com/GoogleCloudPlatform/cloud-tools-for-eclipse/issues/367


[Feat]: Refactor: Add Form Tags and Modularize Repeated Components in Login and Signup

Feature Description
Add Form Tags and Modularize Repeated Components in Login and Signup
Provide a detailed description of the feature

Code Duplication: The login and signup components had repeated input field code, making the codebase harder to maintain and scale.
Lack of Proper Form Structure: The login and signup components did not use <form> tags, which are essential for proper form submission and validation handling.

Put here any screenshots or videos (optional)
No response
By submitting this issue, you agree to

[X] I agree to follow this project's Code of Conduct
[X] I’ve read all the issues in the issue section, and this feature has not been proposed yet.

Other Information

[X] I'm a GSSoC '24 Extended contributor
[X] I'm a Hacktoberfest '24 contributor
[X] I am willing to work on this feature (optional)

#197 Completed. Please Check. Thank you


Dropped service.remove events from k8s-agent can result in duplicated service entries

Steps to reproduce:

Create k8s environment
Create k8s service using kubectl. Wait till k8s-agent propagates the service to rancher (you can list it by name in the DB)
Stop k8s-agent service (Infrastructure stacks -> Kubernetes -> k8s-agent service stop)
remove k8s service using kubectl.
Start k8s-agent service
recreate the k8s service

Bug: service with duplicated name gets created in rancher. It happens because there was no service.remove event due to k8s agent being down (stop/start is just one way of putting it down, but there can be other reasons like network being down, etc). And when next create event comes to rancher for the service having the same name, but diff external id, we create the duplicate.
The fix should go to this file: https://github.com/rancher/cattle/blob/597aacb15d649d9b43494ef82ac74b3c4c10f5aa/code/iaas/logic/src/main/java/io/cattle/platform/process/externalevent/ExternalServiceEventCreate.java
On create/update event, first check if there is a service in the stack, having the same name, and kind=kubernetesService. Then validate its external id. If different, remove the old record and create a new one.

Tested with v1.6.18-rc1 using the below steps:

Create k8s environment
Create k8s service using kubectl. Wait till k8s-agent propagates the service to rancher (you can list it by name in the DB)
Stop k8s-agent service (Infrastructure stacks -> Kubernetes -> k8s-agent service stop)
remove k8s service using kubectl.
Start k8s-agent service
recreate the k8s service

When the service is recreated, there are no duplicate entries in the database.
But, creating an ingress pointing to this recreated service, does not get programmed. Ingress is not functional.

Workaround
Delete all duplicate services
SET FOREIGN_KEY_CHECKS=0;
select id, name, created, removed, remove_time from service where removed IS NOT NULL AND remove_time IS NOT NULL ORDER BY name;
select id, description, created, end_time, event_type, service_id from service_log where service_id in (select id from service where removed IS NOT NULL AND remove_time IS NOT NULL);
DELETE from service_log where service_id in (select id from service where removed IS NOT NULL AND remove_time IS NOT NULL);
DELETE from service where removed IS NOT NULL AND remove_time IS NOT NULL;
select id, name, created, removed, remove_time from service where removed IS NOT NULL AND remove_time IS NOT NULL;
SET FOREIGN_KEY_CHECKS=1;

Delete duplicate services by app
example app = k8test1
example app = k8test1
SET FOREIGN_KEY_CHECKS=0;
select id, name, created, removed, remove_time from service where name like "k8test1" AND removed IS NOT NULL AND remove_time IS NOT NULL ORDER BY name;
select id, description, created, end_time, event_type, service_id from service_log where service_id in (select id from service where name like "k8test1" AND removed IS NOT NULL AND remove_time IS NOT NULL);
DELETE from service_log where service_id in (select id from service where name like "k8test1" AND removed IS NOT NULL AND remove_time IS NOT NULL);
DELETE from service where name like "k8test1" AND removed IS NOT NULL AND remove_time IS NOT NULL;
select id, name, created, removed, remove_time from service where name like "k8test1" AND removed IS NOT NULL AND remove_time IS NOT NULL;
SET FOREIGN_KEY_CHECKS=1;


How to use Whisper for multiple language STT or ASR in Android and IOS

I wanted to use Whisper model for the STT but as I look into the code written for android and ios, I can't find the needed function to init the Whisper Model. I already see that the Whisper is supported for OfflineModel, btw I don't understand the concept of 2Pass, it would be great to get to know it better.

please see
https://github.com/k2-fsa/sherpa-onnx/blob/de655e838e7e1cc073275be119e7cdf0bd5d4108/android/SherpaOnnx2Pass/app/src/main/java/com/k2fsa/sherpa/onnx/SherpaOnnx.kt#L351-L373

https://github.com/k2-fsa/sherpa-onnx/blob/de655e838e7e1cc073275be119e7cdf0bd5d4108/android/SherpaOnnx2Pass/app/src/main/java/com/k2fsa/sherpa/onnx/MainActivity.kt#L209-L212
You can either see secondType to 2 or 3.
Remember to place the corresponding files to assets.

You can find pre-built ASR APKs with Whisper at
https://github.com/k2-fsa/sherpa-onnx/releases/tag/v1.9.14

Similarly, for iOS, please see
https://github.com/k2-fsa/sherpa-onnx/blob/de655e838e7e1cc073275be119e7cdf0bd5d4108/ios-swiftui/SherpaOnnx2Pass/SherpaOnnx2Pass/Model.swift#L96
https://github.com/k2-fsa/sherpa-onnx/blob/de655e838e7e1cc073275be119e7cdf0bd5d4108/ios-swiftui/SherpaOnnx2Pass/SherpaOnnx2Pass/SherpaOnnxViewModel.swift#L94
You need to place the corresponding model files in your project.

Can I only use Whisper since when I test with 2Pass, it can only detect English. Ask I understand in the 2Pass code, I have to have 2 models, sherpa-onnx-streaming-zipformer-en-20M-2023-02-17 and sherpa-onnx-whisper-base.en. I cannot use only Whisper.

whisper is a non-streaming ASR model, you cannot use it for real-time streaming ASR.
We don't provide an APK or an example to use Whisepr alone for non-streaming ASR in Android/iOS, but we do provide APIs.
So the answer is yes; you can use Whisper alone in Android/iOS.

can you guide me on how to use the APIs or at least where's the APIs at. I'm new to mobile and AI, so I appreciate your help a lot

You can find all the required APIs in our two-pass example, which I have already posted in the first comment.
If you are new to Android and iOS and are also new to Kotlin and Swift, then it may be difficult for you.

I was able to get the whisper small multilingual to work with this 2Pass code for Romanian language:
func getNonStreamingWhisperSmall() -> SherpaOnnxOfflineModelConfig {
  let encoder = getResource("small-encoder.int8", "onnx")
  let decoder = getResource("small-decoder.int8", "onnx")
  let tokens = getResource("small-tokens", "txt")

  return sherpaOnnxOfflineModelConfig(
    tokens: tokens,
    whisper: sherpaOnnxOfflineWhisperModelConfig(
      encoder: encoder,
      decoder: decoder,
      language: "ro"
    ),
    numThreads: 1,
    modelType: "whisper"
  )
}

Then in the SherpaOnnxViewModel:initOfflineRecognizer()
change
        let modelConfig = getNonStreamingWhisperTinyEn()
to
        let modelConfig = getNonStreamingWhisperSmall()
I think it would make more sense for my use case to use VAD instead (as in SherpaOnnxVadAsr).  I will try that next.
@csukuangfj how hard is it to make the code changes for the whisper model to take the language param at runtime, not while the model is loaded?  Could you please point me to the general code area?
Thank you!

@csukuangfj I noticed that if I just set the language to en, whisper will switch from transcribe to translate mode.

@csukuangfj I think this is what I am looking for if I want to pass the language to decoder at runtime:
https://github.com/k2-fsa/sherpa-onnx/blob/master/sherpa-onnx/csrc/offline-whisper-greedy-search-decoder.cc#L30

how hard is it to make the code changes for the whisper model to take the language param at runtime

I'm sorry; unfortunately, we don't provide an API for users to do that.

@csukuangfj my bad, I misread the code.  Whisper multilingual model config with an empty language is working perfectly.  I tested it with VAD in iOS SherpaOnnx2Pass.  After init VAD, just doing something similar to Android's version:
          let array = convertedBuffer.array()
          if !array.isEmpty {
              self.vad.acceptWaveform(samples: [Float](array))
              while !self.vad.isEmpty() {
                  let s = self.vad.front()
                  self.vad.pop()
                  let lastSentence = self.offlineRecognizer.decode(samples: s.samples).text
                  self.sentences.append(lastSentence)
                  self.updateLabel()
              }
          }


Video/Audio call

Hello , Please can you guys help me how to reject call with agora in both side from caller/receiver in flutter app, I'm using flutter with agora rtc engine

Sorry, I don't understand, what means rejecting the call with agora?

Done

could you share your solution? It may help other developers. Thanks.


Middle-mouse closing unsaved files locks the desktop environment

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
'untitled' has changes, do you want to save them? prompt causes my Desktop Environment to freeze.
Steps to Reproduce

Create a new file: File > New file
Put some content in this new unsaved file
Middle mouse click to close the file; you will not be able to click anywhere; you need to use ALT+TAB functionality to move to a different window. If you click the x of the file tab you will not reproduce this issue

Expected behavior: I expect it to not freeze my DE
Actual behavior: It freezes my DE; forcing me to killall atom
Reproduces how often: 100% of the time
Versions
$ atom --version
Atom    : 1.19.1
Electron: 1.6.9
Chrome  : 56.0.2924.87
Node    : 7.4.0
$ lsb_release -a
No LSB modules are available.
Distributor ID:	LinuxMint
Description:	Linux Mint 18.2 Sonya
Release:	18.2
Codename:	sonya

This is with XFCE, not Cinnamon.

Thanks for taking the time to contribute!
We noticed that this is a duplicate of https://github.com/atom/atom/issues/15197. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


KAFKA-17564: Move BrokerFeatures to server module

Jira: https://issues.apache.org/jira/browse/KAFKA-17564
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@frankvicky thanks for your comments, addressed them.


Adjust landing page design

put logo in header bar
increase size of "Entdecken Sie..."

Putting logo in menu bar was discussed earlier and dismissed. Menu bar is static and therefore does not scroll with content. In smaller windows/screens, the menu bar would then occupy too much of the space. Therefore, the larger logo on the landing page is outside the menu bar.
We can add the logo also in the menu bar but then it is doubled and might look odd.

I would rather not double the logo. Can we move it to the right (including the "Entdecken Sie"- text) so it forms a vertical line with the white column of the menu bar?


Make Jquery Calx a bit more angular friendly

In order for angular to process changes to an input field that is set programmatically an 'input' event needs to be fired on the updated form item.
Example:

this will not trigger an angular update
$("#foo").val("x");

However:

this will
$("#foo").val("x");
$("#foo").triggerHandler('input');

In the
cell.fx.renderComputedValue
the following code can be changed to trigger angular changes with the addition of the bolded values:
if(isFormTag){
            if(this.isCheckbox){
                this.el.prop('checked', (originalVal == this.el.val()));
            }else if(tagName == 'select'){
                this.el.val(originalVal);
newline         this.el.triggerHandler('input');
            }else if(tagName == 'input' || tagName == 'textarea'){
                this.el.val(formattedVal);
newline         this.el.triggerHandler('input');
            }
}

PS: I realize this is an enhancement I was unsure how to tag it as such.

Sorry you need to use this for IE:
if (isFormTag) {                 if (this.isCheckbox) {                     this.el.prop('checked', (originalVal == this.el.val()));                 } else if (tagName == 'select') {                     this.el.val(originalVal);                     this.el.trigger('input');                     this.el.trigger('change');                 } else if (tagName == 'input' || tagName == 'textarea') {                     this.el.val(formattedVal);                     this.el.trigger('input');                     this.el.trigger('change');                 }             } else {                 this.el.html(formattedVal);             }

I am afraid that triggering change or input event will causing infinite-loop of calculation when calx is configured to depend on this event.
maybe its time to build calx for angular?

+1 please

@ikhsan017 - this would also help with the WP plugin i would think +1