Segmentation fault in SPMV benchmark on GPU target

Segmentation fault when running the spmv benchmark from npbench with dace_gpu framework
(.venv) epaone@nid01934:/scratch/snx3000/epaone/repo/npbench> python run_benchmark.py -b spmv -f dace_cpu
***** Testing DaCe CPU with spmv on the S dataset *****
NumPy - default - validation: 13ms
DaCe CPU - fusion - first/validation: 57ms
DaCe CPU - fusion - validation: SUCCESS
DaCe CPU - fusion - median: 55ms
DaCe CPU - parallel - first/validation: 2ms
DaCe CPU - parallel - validation: SUCCESS
DaCe CPU - parallel - median: 1ms
DaCe CPU - auto_opt - first/validation: 2ms
DaCe CPU - auto_opt - validation: SUCCESS
DaCe CPU - auto_opt - median: 1ms
(.venv) epaone@nid01934:/scratch/snx3000/epaone/repo/npbench> python run_benchmark.py -b gemm -f dace_gpu
***** Testing DaCe GPU with gemm on the S dataset *****
NumPy - default - validation: 111ms
DaCe GPU - fusion - first/validation: 423ms
DaCe GPU - fusion - validation: SUCCESS
DaCe GPU - fusion - median: 2ms
DaCe GPU - parallel - first/validation: 4ms
DaCe GPU - parallel - validation: SUCCESS
DaCe GPU - parallel - median: 2ms
DaCe GPU - auto_opt - first/validation: 4ms
DaCe GPU - auto_opt - validation: SUCCESS
DaCe GPU - auto_opt - median: 1ms
(.venv) epaone@nid01934:/scratch/snx3000/epaone/repo/npbench> python run_benchmark.py -b spmv -f dace_gpu
***** Testing DaCe GPU with spmv on the S dataset *****
NumPy - default - validation: 13ms
Segmentation fault (core dumped)

From gdb backtrace it is possible to locate the segmentation in access to A_row array:
Thread 1 "python" received signal SIGSEGV, Segmentation fault.
0x000015554fe9b27a in __program_fusion_internal (__state=0x5842880, A_col=0x1553e2a04200, A_row=0x1553e2a00000, A_val=0x1553e2a0c200, __return=0x1553e2a24200, x=0x1553e2a1c200, M=4096, N=4096, nnz=8192) at /scratch/snx3000/epaone/repo/npbench/.dacecache/fusion/src/cpu/fusion.cpp:25
25              __tmp4 = A_row[(i + 1)];
(gdb) bt
#0  0x000015554fe9b27a in __program_fusion_internal (__state=0x5842880, A_col=0x1553e2a04200, A_row=0x1553e2a00000, A_val=0x1553e2a0c200, __return=0x1553e2a24200, x=0x1553e2a1c200, M=4096, N=4096, nnz=8192) at /scratch/snx3000/epaone/repo/npbench/.dacecache/fusion/src/cpu/fusion.cpp:25
#1  0x000015551e3466dd in ?? () from /usr/lib64/libffi.so.7
#2  0x000015551e345bdf in ?? () from /usr/lib64/libffi.so.7
#3  0x000015551e5a0a41 in _call_function_pointer (argtypecount=<optimized out>, argcount=9, resmem=0x7fffffff3b20, restype=<optimized out>, atypes=<optimized out>, avalues=<optimized out>,
    pProc=0x15554fe9b570 <__program_fusion(fusion_t *, unsigned int * __restrict__, unsigned int * __restrict__, double * __restrict__, double * __restrict__, double * __restrict__, long long, long long, long long)>, flags=4353)
    at /home/users/mnusseibeh/rpmbuild/BUILD/cray-python-<IP_ADDRESS>-202108131723_038f7ca/Python-3.9.4/Modules/_ctypes/callproc.c:920
#4  _ctypes_callproc (pProc=pProc@entry=0x15554fe9b570 <__program_fusion(fusion_t *, unsigned int * __restrict__, unsigned int * __restrict__, double * __restrict__, double * __restrict__, double * __restrict__, long long, long long, long long)>, argtuple=argtuple@entry=0x1555501ecba0, flags=4353,
    argtypes=argtypes@entry=0x155555477040, restype=0x84ff70, checker=0x0) at /home/users/mnusseibeh/rpmbuild/BUILD/cray-python-<IP_ADDRESS>-202108131723_038f7ca/Python-3.9.4/Modules/_ctypes/callproc.c:1263
#5  0x000015551e595699 in PyCFuncPtr_call (self=self@entry=0x1555501d51c0, inargs=inargs@entry=0x1555501ecba0, kwds=0x0) at /home/users/mnusseibeh/rpmbuild/BUILD/cray-python-<IP_ADDRESS>-202108131723_038f7ca/Python-3.9.4/Modules/_ctypes/_ctypes.c:4201
#6  0x0000155554eafcc1 in _PyObject_Call (tstate=0x605ef0, callable=0x1555501d51c0, args=0x1555501ecba0, kwargs=<optimized out>) at Objects/call.c:281
#7  0x0000155554f26be0 in do_call_core (kwdict=0x0, callargs=0x1555501ecba0, func=0x1555501d51c0, tstate=<optimized out>) at Python/ceval.c:5120
#8  _PyEval_EvalFrameDefault (tstate=<optimized out>, f=<optimized out>, throwflag=<optimized out>) at Python/ceval.c:3580

and visualize some context:
(gdb) info locals
i = 0
__tmp3 = <optimized out>
__tmp4 = <optimized out>
(gdb) p A_row
$1 = (unsigned int * __restrict__) 0x1553e2a00000

The generated code looks like:
 13 void __program_fusion_internal(fusion_t *__state, unsigned int * __restrict__ A_col, unsigned int * __restrict__ A_row, double * __restrict__ A_val, double * __restrict__ __return, double * __restrict__ x, long long M, long long N, long long nnz)
 14 {
 15     long long i;
 16     unsigned int __tmp3;
 17     unsigned int __tmp4;
 18
 19
 20
 21
 22     for (i = 0; (i < M); i = (i + 1)) {
 23
 24         __tmp3 = A_row[i];
 25         __tmp4 = A_row[(i + 1)];

The environment looks like:
(.venv) epaone@nid01934:/scratch/snx3000/epaone/repo/npbench> nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2020 NVIDIA Corporation
Built on Mon_Nov_30_19:08:53_PST_2020
Cuda compilation tools, release 11.2, V11.2.67
Build cuda_11.2.r11.2/compiler.29373293_0
(.venv) epaone@nid01934:/scratch/snx3000/epaone/repo/npbench> python --version
Python 3.9.4
(.venv) epaone@nid01934:/scratch/snx3000/epaone/repo/npbench> pip list
Package             Version  Editable project location
------------------- -------- ------------------------------------
aenum               3.1.12
astunparse          1.6.3
blinker             1.6.2
certifi             2023.5.7
chardet             5.1.0
charset-normalizer  3.1.0
click               8.1.3
commonmark          0.9.1
contourpy           1.0.7
cupy-cuda11x        12.1.0
cycler              0.11.0
dace                0.14.2
dill                0.3.6
exceptiongroup      1.1.1
fastrlock           0.8.1
Flask               2.3.2
fonttools           4.39.4
idna                3.4
importlib-metadata  6.6.0
importlib-resources 5.12.0
iniconfig           2.0.0
itsdangerous        2.1.2
Jinja2              3.1.2
kiwisolver          1.4.4
llvmlite            0.40.0
MarkupSafe          2.1.2
matplotlib          3.7.1
mpmath              1.3.0
networkx            3.1
npbench             0.1      /scratch/snx3000/epaone/repo/npbench
numba               0.57.0
numpy               1.24.3
packaging           23.1
pandas              2.0.1
Pillow              9.5.0
pip                 23.1.2
pluggy              1.0.0
ply                 3.11
Pygments            2.15.1
pygount             1.5.1
pyparsing           3.0.9
pytest              7.3.1
python-dateutil     2.8.2
pytz                2023.3
PyYAML              6.0
requests            2.30.0
rich                12.6.0
scipy               1.10.1
setuptools          49.2.1
six                 1.16.0
sympy               1.9
tomli               2.0.1
tzdata              2023.3
urllib3             2.0.2
websockets          11.0.3
Werkzeug            2.3.4
wheel               0.40.0
zipp                3.15.0

The problem seems to be caused by the fact that array 'A_row' has storage set to StorageType.GPU_Global, so the generated host code will attempt to access a memory address on device memory:
(.venv) epaone@nid05267:/scratch/snx3000/epaone/repo/npbench> LD_LIBRARY_PATH=$CUDA_HOME/lib64:$LD_LIBRARY_PATH cuda-memcheck python run_benchmark.py -b spmv -f dace_gpu
========= CUDA-MEMCHECK
***** Testing DaCe GPU with spmv on the S dataset *****
NumPy - default - validation: 27ms
========= Error: process didn't terminate successfully
=========        The application may have hit an error when dereferencing Unified Memory from the host. Please rerun the application under cuda-gdb or Nsight Eclipse Edition to catch host
side errors.
========= No CUDA-MEMCHECK results found

It looks like all access nodes get GPU_Global allocation by default, no matter how they are used in the generated code:
        def copy_to_gpu(sdfg):
            for k, v in sdfg.arrays.items():
                if not v.transient and isinstance(v, dace.data.Array):
                    v.storage = dace.dtypes.StorageType.GPU_Global

I have skipped the fusion execution step of SPMV in npbench, and I can observe that parallel and auto_opt implementations work fine.
(.venv) epaone@nid01933:/scratch/snx3000/epaone/repo/npbench> LD_LIBRARY_PATH=$CUDA_HOME/lib64:$LD_LIBRARY_PATH python run_benchmark.py -b spmv -f dace_gpu
***** Testing DaCe GPU with spmv on the S dataset *****
NumPy - default - validation: 13ms
DaCe GPU - parallel - first/validation: 13ms
DaCe GPU - parallel - validation: SUCCESS
DaCe GPU - parallel - median: 2ms
DaCe GPU - auto_opt - first/validation: 4ms
DaCe GPU - auto_opt - validation: SUCCESS
DaCe GPU - auto_opt - median: 2ms

The reason is that the parallel optimisation converts the outer for loop into a map, then in the generated code A_row is accessed on device.
Therefore, this issue is limited to the fusion step and, in general, to for loops where inter-state symbols are defined based on array values.

I have tested it on GPU and it works. Thanks

However, I see now that the autoopt transformation step fails:
***** Testing DaCe GPU with spmv on the S dataset *****
NumPy - default - validation: 26ms
DaCe autoopt failed
DaCe GPU - fusion - first/validation: 3870ms
DaCe GPU - fusion - validation: SUCCESS
DaCe GPU - fusion - median: 148ms
DaCe GPU - parallel - first/validation: 30ms
DaCe GPU - parallel - validation: SUCCESS
DaCe GPU - parallel - median: 20ms

This is a workflow/ordering of transformations-related issue. We have introduced a workaround in the branch autoopt-device-validation-issues but we will discuss in the next meeting how to better address the problem. See also #1323

The issue is finally being fixed by utilizing the auto-optimizer's new use_gpu_storage flag. See also NPBench's #20

Thank you. I have also tested it and it works.


Generic tools Kalyan

[x] Create config_params_Kalyan for all common parameters

[x] Generate figure with image data and overlaid ground truth

[x] Include figure in paper

In Overleaf there is a dummy figure used. For the correct figure, comment out and un comment the 2 lines under %%%%%%%%%%%%%%%%%%%%%%%% change figure %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%


[CMake] Use modern CMake features for executables and their dependencies

Affected subprojects:

angelscript
authserver
dataextractor
mpq
realmserver
taskflow
worldnode

Thank you :)


[skip ci] iscsi: don't set default value for trusted_ip_list

It restricts access to the iSCSI API.
It can be left empty if the API isn't going to be access from outside the
gateway node
Even though this seems to be a limited use case, it's better to leave it
empty by default than having a meaningless default value.
We could make this variable mandatory but that would be a breaking
change. Let's just add a logic in the template in order to set this
variable in the configuration file only if it was specified by users.
Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1994930
Signed-off-by: Guillaume Abrioux<EMAIL_ADDRESS>

I guess this will break the iscsi integration within the ceph dashboard
https://github.com/ceph/ceph-ansible/blob/master/roles/ceph-iscsi-gw/tasks/common.yml#L31-L45

Not the best solution but at least it works
    - name: add mgr ip address to trusted list with dashboard - ipv4
      set_fact:
-        trusted_ip_list: '{{ trusted_ip_list }},{{ hostvars[item]["ansible_facts"]["all_ipv4_addresses"] | ips_in_ranges(public_network.split(",")) | first }}'
+        trusted_ip_list: '{{ trusted_ip_list | default("") }}{{ "," if trusted_ip_list is defined else "" }}{{ hostvars[item]["ansible_facts"]["all_ipv4_addresses"] | ips_in_ranges(public_network.split(",")) | first }}'
      with_items: '{{ groups[mgr_group_name] | default(groups[mon_group_name]) }}'
      when:
        - dashboard_enabled | bool
        - ip_version == 'ipv4'


Where does 0x3f0 come from ?

I cannot find any value 0x3f0 in the header file.
kLAPolicyDeviceOwnerAuthentication == 2
kLAPolicyDeviceOwnerAuthenticationWithBiometrics == 1
Where was this reverse engineered from ?

LocalAuthentication.framework ;)


Disable skip step verification in export mode.

This change greatly improves export mode performance on a large amount of tests. I have 5 vs. 13 seconds on 10K tests.

Also it would be nice to add a test for this change.


Type::convertToPHPValue($value) doesn't actually get called

Recently creating a custom type I ran across an issue regarding type conversions. I created a type that ciphers plain text, but I need to be able to decrypt the data after it's been pulled from the database. Turns out that convertToPHPValue() never actually gets called. This is different from the Doctrine ORM because I know that it definitely gets called for ORM types.
/**
 * Ciphered data type
 */
class CipheredStringType extends Type {

	protected $cipher;

	public function convertToDatabaseValue($value) {
		return $value !== null ? $this->getCipher()->encrypt((string) $value) : null;
	}

        //This never actually gets called, so I get the encrypted text after database pull
	public function convertToPHPValue($value) {
		 return $value !== null ? $this->getCipher()->decrypt((string) $value) : null;
	}

	public function getCipher() {
		//Cipher created here
	}

}

I know there are other ways to handle this but it would be nice if I could do it transparently via a type. This is confusing for new people to the Doctrine ODM because the documentation implies that convertToPHPValue() will actually get called at some point in the process.

In the ODM right now you need to implement these methods:
public function closureToMongo()
{
    return '$return = (string) $value;';
}

public function closureToPHP()
{
    return '$return = (string) $value;';
}

Hi @housebolt
Here is a temporary solution since there has been no progress on the subject
public function closureToPHP() :string
{
    return '$type = \Doctrine\ODM\MongoDB\Types\Type::getType(\'type_name\'); $return = $type->convertToPHPValue($value);';
}

Actually there's a \Doctrine\ODM\MongoDB\Types\ClosureToPHP which provides exactly that function. We recommend using it along with implementing convertToPHPValue method. It's also showcased in the docs: https://www.doctrine-project.org/projects/doctrine-mongodb-odm/en/latest/reference/custom-mapping-types.html. If there is still something off please do open a new issue instead of resurrecting an eight years old one.


Problems with the new version

Yesterday it was the block highlight. Today after update, it's instead the underline highlight. Have tried hi link illuminatedWord Visual, doesn't work.
Also, in the newer version, sometimes all the same words highlight, which is good. But sometimes, only the word under cursor is highlighted even if the same word is right next to it. Sometimes it just doesn't highlight at all. It's so inconsistent even within the same file. Come on.

Same problem, now it underlines words instead of highlight them

I think this is as intended see https://github.com/RRethy/vim-illuminate#highlight-groups ?
You could overwrite these highlight groups to your liking

I think this is as intended see https://github.com/RRethy/vim-illuminate#highlight-groups ?
You could overwrite these highlight groups to your liking

I think i tried that with stanout and strikethrough. But it doesn't seem to have any effect on it. What should I config?

I believe it should be -
vimscript
hi link IlluminatedWordText Visual
hi link IlluminatedWordRead Visual
hi link IlluminatedWordWrite Visual

lua
vim.api.nvim_set_hl(0, "IlluminatedWordText", { link = "Visual" })
vim.api.nvim_set_hl(0, "IlluminatedWordRead", { link = "Visual" })
vim.api.nvim_set_hl(0, "IlluminatedWordWrite", { link = "Visual" })

@krish-r I can confirm the lua part - was just writing something similar 😄

I updated the migration guide to make it clear the highlight groups changed. FWIW they changed because now highlights can have a type which wasn't the case before. What @krish-r posted is correct.

Also, in the newer version, sometimes all the same words highlight, which is good. But sometimes, only the word under cursor is highlighted even if the same word is right next to it. Sometimes it just doesn't highlight at all. It's so inconsistent even within the same file. Come on.

@iaurman You'll have to be a bit more specific here, are you using tree-sitter or LSP? If so, then the highlighting is dependant on what they provide. In the screenshots you showed you hovered on return and int, for return it's out of scope of the other return so having them both highlight wouldn't make sense, and for int it's a builtin type not an identifier so most LSP servers and tree-sitter won't highlight it. It sounds like you want the naive highlighting from only regex matching which you have achieve with this:
require('illuminate').configure({
    providers = {
        'regex',
    },
}

I updated the migration guide to make it clear the highlight groups changed. FWIW they changed because now highlights can have a type which wasn't the case before. What @krish-r posted is correct.

Also, in the newer version, sometimes all the same words highlight, which is good. But sometimes, only the word under cursor is highlighted even if the same word is right next to it. Sometimes it just doesn't highlight at all. It's so inconsistent even within the same file. Come on.

@iaurman You'll have to be a bit more specific here, are you using tree-sitter or LSP? If so, then the highlighting is dependant on what they provide. In the screenshots you showed you hovered on return and int, for return it's out of scope of the other return so having them both highlight wouldn't make sense, and for int it's a builtin type not an identifier so most LSP servers and tree-sitter won't highlight it. It sounds like you want the naive highlighting from only regex matching which you have achieve with this:
require('illuminate').configure({
    providers = {
        'regex',
    },
}

If you want something different, you'll need to provide more information than a couple of screenshots which aren't very useful to anyone, it's not even clear what language you are using (it could be C/C++/Java).
I'll leave this issue open for a day in case of follow-ups.

Sorry for the delay. Yes, krish-r's answer and the regex highlighting is my solution. My LSP and tree-sitter might not be working properly before thus causing me to think it's a problem here. Thanks for the solution.


Roadmap

Development

[x] Get all components into the client and render them
[x] Parse meta information with react-docgen
[x] Navigation
[ ] Homepage
[ ] Fuzz testing
[ ] Documentation

Design

[ ] Logo
[ ] Styleguide Client
[ ] Marketing Website

Things up for discussion:

allow to overwrite the Page component so people can customise it
generate various states based on comments examples/fixtures data (How does it work with ImmutableJS?)

I updated the Roadme to be split in Alpha dev & later

Closing as the remaining points were split across multiple issues.


[stable/several] CI/CD failing if dependencies updated using Helm 3

Describe the bug
There are some PRs failing in the CI/CD tests with the following error:
Error: requirements.lock is out of sync with requirements.yaml

for example this one: https://github.com/helm/charts/pull/20798
This PR is updating the images, the chart version, app version, etc. We did this kind of PRs automatically for a long time without any issue. In the past week, we update our internal pipeline to use Helm 3 instead of Helm 2 (as recommended).
Since this version bump, we are receiving the above error by the CI/CD in those charts with dependencies.
The issue is that the generated digest or generated in the requirements.lock is not the same if you run helm dep update with Helm 3 and Helm 2:
Helm 2
$ helm2 dependency update
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "stable" chart repository
Update Complete.
Saving 1 charts
Downloading mariadb from repo https://kubernetes-charts.storage.googleapis.com/
Deleting outdated charts

$ cat requirements.lock | grep -A 2 digest
digest: sha256:cd64413a4a697ccf85c0091e9c55cdc5876938ddced84c05d37c57ff9abc5864
generated: "2020-02-18T11:09:35.806304593Z"

Helm 3
$ helm dependency update
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "stable" chart repository
Update Complete. ⎈Happy Helming!⎈
Saving 1 charts
Downloading mariadb from repo https://kubernetes-charts.storage.googleapis.com/
Deleting outdated charts

$ cat requirements.lock | grep -A 2 digest
digest: sha256:54dca7bae6be7a5ba35fa733f407bb2960ea718c1e3f6afa622c4f934d22d22c
generated: "2020-02-18T11:07:45.185624819Z"

As you can see, the digest (and generated) are different without other changes apart from the Helm version.
In the mentioned PR, changes in the requirements.lock are just the autogenerated ones due to run helm dep update using Helm 3: https://github.com/helm/charts/pull/20798/files#diff-d52af004c1d687e2a87224fb352d5e99
It seems this issue is already reported and fixed in Helm 3 but there is no backward compatibility, so AFAIK Helm 2 is not able to recognize requirements.lock built using Helm 3:
https://github.com/helm/helm/issues/4011
https://github.com/helm/helm/issues/2731
In this case, as the CI/CD system is using Helm 2 all the PRs with dependencies updated with Helm 3 are going to fail:
readonly HELM_TARBALL=helm-v2.9.1-linux-amd64.tar.gz

Which chart:
All of the charts with dependencies

/assign @mattfarina @unguiculus @cpanato

For the record: CI uses https://github.com/helm/charts/blob/master/test/e2e.sh.
Indeed, CI still uses Helm 2. This will not change. See https://github.com/helm/charts#deprecation-timeline

Hi @unguiculus

This will not change

Could you please elaborate the reasons for this? Do you refer to this?

Given the deprecation plan, this project is intended for apiVersion: v1 Charts (installable by both Helm 2 and 3), and not for apiVersion: v2 charts (installable by Helm 3 only)

Closing this issue as we moved all the Bitnami maintained charts to the bitnami repo (https://github.com/helm/charts/issues/20969) and this issue is not going to happen again as there wouldn't be new releases.


Incremental Reading not working when not connected to internet.

The incremental reading queue completely disappears when you're not connected to the internet.

This is a limitation of the plugin system at the moment - plugins require an internet connection.


Fix issues with dataflow test

Simple fix for making sure generated job name meets dataflow's naming requirements
Use job_name in targetspec in dataflow integration test, not job_id. job_id is not
available for string expansion, as it is only known after submission
Validate the job names passed in to make sure they meet dataflow's requirements

Thanks a lot for making that more robust, @cisaacstern!


[Feature request] Work with smart doors synchronization group

This feature would allow linking multiple doors to open at once (or with a delay to allow for things like airlocks or a corridor of doors locking). Currently there is not a compatibility between smart doors synchronization and animated walls.

i don't plan to add that, but you can achieve that with macros pretty easly


Pass DOCKER_HOST to calico/node

See https://github.com/projectcalico/libnetwork-plugin/issues/90#issuecomment-266078037 for detail

I'm thinking of one of the two approaches here:

calicoctl node run command checks if ENV var DOCKER_HOST is set, if it is, then pass it to calico/node as -e DOCKER_HOST
add a flag for calicoctl node run --docker-host=<DOCKER_HOST>

I'm leaning more towards the first option.
WDYT? @tomdee @robbrockbank

I prefer to have things that we think are permanent features as command line options - they are easier to use and easier to document IMHO.   This feels like a permanent feature to me, so I'd err on having it as an option rather than an env


Run unit and validation test against Py based server

@vkuznet
I merged PR #22 .  Can you start to run DBS unit and validation tests against the python and Go servers using branch DBSClient4go?  I knew you already ran unit tests before, if you haven't run validation tests, here is the command to run it:
python setup_test.py test_system --host=https://cmsweb-testbed.cern.ch  --validation

Done.


Deletion rules for indicators

Terminology
1. Confirmation prompt: This is a dialogue that warns you of a destructive action and prompts you for confirmation that you want to proceed.
2. Rationale prompt: This is a dialogue that warns you that changes are being logged and prompts you to type text into a rationale field.
Summary

We have always had confirmation prompts when you delete just about anything.
With Partner Access, we've introduced rationale prompts that replace some of the old confirmation prompts.
We do NOT require a rationale for every type of deletion.
However, for any deletion that does NOT require a rationale, we still need a confirmation prompt.
Some of these confirmation prompts seem to have gone missing, allowing deletions to proceed with no warning. This epic documents expected behaviors for deleting indicators in a variety of scenarios. Other tickets will cover other types of deletions.
To keep the UX consistent with what's been built for Partner Access, we can apply the same style of notification even when we're only confirming and not prompting for a rationale.
Success messages should always appear after the prompt closes, whether it's a confirmation prompt or a rationale prompt.

Acceptance criteria
Scenario 1: Delete an indicator that was added without completing the setup process
Because there are no results linked to the indicator, we do not need a rationale for deleting it. However, we still need a confirmation prompt. At the moment, we have problematic behavior captured by Cameron in #1350. Here is the desired behavior:

[x] 1. I see a red PNotify warning in the lower left corner with the following text:
Warning
This action cannot be undone. All details about this indicator will be permanently removed.
Are you sure you want to delete this indicator?

[x] 2. There are two buttons on this notification: OK and Cancel.

[x] 3.  If I click Cancel or click the "X" in the upper right corner of the notification, it closes and nothing happens.

[x] 4. If I click OK, the following things happen:

[x] 4.1. The indicator is deleted.

[x] 4.2. I am returned to the program page.

[x] 4.3. In the lower left corner of the program page is a success message with the following text:
Success
The indicator was deleted.

[x] 4.3.1. The success message is auto-dismissing. It does not have an OK button, but does have an "X" button.

[x] 5. All of the above is true regardless of target frequency of the indicator I've deleted -- time aware, midline/endline, event, LoP-only.

Scenario 2: Indicator with no results -- regardless of whether or not there are targets in place
As specified in the PA wiki, when there are no results linked to an indicator, we do not need a rationale for deleting the indicator. However, we still need a confirmation prompt. Following is the desired behavior if you click "Delete this indicator" on the Summary tab of the indicator setup modal:

[x] 1. I see a red PNotify warning in the lower left corner of the indicator setup modal (not the screen):
Warning
This action cannot be undone. All details about this indicator will be permanently removed.
Are you sure you want to delete this indicator?

[x] 2. There are two buttons on this notification: OK and Cancel.

[x] 3.  If I click Cancel or click the "X" in the upper right corner of the notification, it closes and nothing happens.

[x] 4. If I click OK, the following things happen:

[x] 4.1. The indicator is deleted.

[x] 4.2. The indicator setup modal closes.

[x] 4.3. In the lower left corner of whatever page was behind the modal, is the following success message:
Success
The indicator was deleted.

[x] 4.3.1. This is true if I'm coming from the Program page.
[x] 4.3.2. This is true if I'm coming from the IPTT.

[x] 4.4. The success message is auto-dismissing. It does not have an OK button, but does have an "X" button.

[x] 5. All of the above is true regardless of target frequency of the indicator I've deleted -- time aware, midline/endline, event, LoP-only.

Scenario 3: Indicator with results / with targets
As specified in the PA wiki, when an indicator has results linked to it, we must always prompt for a rationale if you click "Delete this indicator." The interaction details are mostly right. To make it fully consistent with the patterns we're establishing for Satsuma and beyond, it needs some tweaking.

[x] 1. We prompt for a rationale. (This is fine.)

[x] 2. The rationale field is required, and I get an error message if I submit without typing anything in. (This is fine.)

[x] 3. If I type something in and click OK, the following should happen, in the following order.

[x] 3.1. The rationale prompt closes.

[x] 3.2. In the lower left corner of whatever page was behind the modal, is the following success message:
Success
The indicator was deleted.

[x] 3.2.1. This is true if I'm coming from the Program page.
[x] 3.2.2. This is true if I'm coming from the IPTT.

[x] 3.3. The success message is auto-dismissing. It does not have an OK button, but does have an "X" button.

[x] 4. All of the above is true regardless of target frequency of the indicator I've deleted -- time aware, midline/endline, event, LoP-only.

Scenario 4: Indicator with results / no targets
This is an edge case that isn't so edgy; it's commonly found in older, pre-Winterhawks programs, and can happen anytime you delete your targets after adding results. Because there ARE results linked to the indicator that will be destroyed, we should absolutely prompt for a rationale and log changes.

[x] 1. The prompt/OK/Success message sequence is exactly the same as described in Scenario 3.

Housekeeping

[ ]  Unit test/s written wherever applicable
[ ]  i18n tags included wherever applicable
[ ]  Functionality is accessible to people using screen readers

QA - PASSED
Scenario 1: Delete an indicator that was added without completing the setup process

[x] 1. I see a red PNotify warning in the lower left corner with the following text:
Warning
This action cannot be undone. All details about this indicator will be permanently removed.
Are you sure you want to delete this indicator?

[x] 2. There are two buttons on this notification: OK and Cancel.

[x] 3.  If I click Cancel or click the "X" in the upper right corner of the notification, it closes and nothing happens.

[x] 4. If I click OK, the following things happen:

[x] 4.1. The indicator is deleted.

[x] 4.2. I am returned to the program page.

[x] 4.3. In the lower left corner of the program page is a success message with the following text:
Success
The indicator was deleted.

[x] 4.3.1. The success message is auto-dismissing. It does not have an OK button, but does have an "X" button.

[x] 5. All of the above is true regardless of target frequency of the indicator I've deleted -- time aware, midline/endline, event, LoP-only.

Scenario 2: Indicator with no results -- regardless of whether or not there are targets in place

[x] 1. I see a red PNotify warning in the lower left corner of the indicator setup modal (not the screen):
Warning
This action cannot be undone. All details about this indicator will be permanently removed.
Are you sure you want to delete this indicator?

[x] 2. There are two buttons on this notification: OK and Cancel.

[x] 3.  If I click Cancel or click the "X" in the upper right corner of the notification, it closes and nothing happens.

[x] 4. If I click OK, the following things happen:

[x] 4.1. The indicator is deleted.

[x] 4.2. The indicator setup modal closes.

[x] 4.3. In the lower left corner of whatever page was behind the modal, is the following success message:
Success
The indicator was deleted.

[x] 4.3.1. This is true if I'm coming from the Program page.
[x] 4.3.2. This is true if I'm coming from the IPTT.

[x] 4.4. The success message is auto-dismissing. It does not have an OK button, but does have an "X" button.

[x] 5. All of the above is true regardless of target frequency of the indicator I've deleted -- time aware, midline/endline, event, LoP-only.

Scenario 3: Indicator with results / with targets

[x] 1. We prompt for a rationale. (This is fine.)

[x] 2. The rationale field is required, and I get an error message if I submit without typing anything in. (This is fine.)

[x] 3. If I type something in and click OK, the following should happen, in the following order.

[x] 3.1. The rationale prompt closes.

[x] 3.2. In the lower left corner of whatever page was behind the modal, is the following success message:
Success
The indicator was deleted.

[x] 3.2.1. This is true if I'm coming from the Program page.
[x] 3.2.2. This is true if I'm coming from the IPTT.

[x] 3.3. The success message is auto-dismissing. It does not have an OK button, but does have an "X" button.

[x] 4. All of the above is true regardless of target frequency of the indicator I've deleted -- time aware, midline/endline, event, LoP-only.

Scenario 4: Indicator with results / no targets
This is an edge case that isn't so edgy; it's commonly found in older, pre-Winterhawks programs, and can happen anytime you delete your targets after adding results. Because there ARE results linked to the indicator that will be destroyed, we should absolutely prompt for a rationale and log changes.

[x] 1. The prompt/OK/Success message sequence is exactly the same as described in Scenario 3.


read_sql from SQL Server to Arrow truncates datetime

I'm working with a MS SQL Server 2019. When reading a datetime field, read_sql correctly captures it in a pandas dataframe. When I convert that dataframe to an arrow table, datetimes are also retained correctly. However, loading directly to an arrow table truncates the datetime fields to midnight. I'd like to remove the pandas dependency and load directly to an arrow table. Is there a way to do this without truncating the datetime?
Example Code:
import connectorx as cx
import pyarrow as pa
con_string = 'mssql://user:pwd@hostname.de%5CG:1439/database'
print('------- Pandas Table -------')
query = 'SELECT top 5 Datum FROM Termine WHERE Datum>getdate()'
pandas_table = cx.read_sql(con_string, query, return_type='pandas')
print(pandas_table)
print('------- Arrow table from pandas -------')
arrow_table_from_pandas = pa.Table.from_pandas(pandas_table)
print(arrow_table_from_pandas)
print('------- Arrow Table -------')
arrow_table = cx.read_sql(con_string, query, return_type='arrow')
print(arrow_table)

Example Output:
------- Pandas Table -------
                Datum
0 2022-02-06 07:30:00
1 2022-02-07 00:00:00
2 2022-02-07 00:00:00
3 2022-02-07 07:00:00
4 2022-02-07 07:30:00
------- Arrow table from pandas -------
pyarrow.Table
Datum: timestamp[ns]
----
Datum: [[2022-02-06 07:30:00.000000000,2022-02-07 00:00:00.000000000,2022-02-07 00:00:00.000000000,2022-02-07 07:00:00.000000000,2022-02-07 07:30:00.000000000]]
------- Arrow Table -------
pyarrow.Table
Datum: date64[ms]
----
Datum: [[2022-02-06,2022-02-07,2022-02-07,2022-02-07,2022-02-07]]

Hi @t-alex-fritz , there is some issue when we dealing with datetime for arrow and we are currently shifting from arrow to arrow2. Can you set the return_type="arrow2" to see whether it solves the problem?

Hi @wangxiaoying, thank you! This throws a ValueError: arrow2

@t-alex-fritz , can you update to the latest alpha version 0.2.4a6?

On 0.2.4a6 I do get non-truncated timestamp[ns] - that worked! However, pyarrow then fails to write the table to parquet whenever datetimes are involved that aren't rounded.
import connectorx as cx
import pyarrow.parquet as pq
con_string = 'mssql://db:0b2c4c90-c53d-425f-82da-beefd7ff0f99@kirschbaum.work4allcloud.de%5CG:1439/workm001'
query = 'SELECT top 5 Datum FROM Termine WHERE Datum>getdate()'
arrow_table = cx.read_sql(con_string, query, return_type='arrow2')
print(arrow_table)
pq.write_table(arrow_table, 'output.parquet')

pyarrow.Table
ModificationDate: timestamp[ns]
----
ModificationDate: [[2021-04-14 00:13:02.216666666,2021-07-22 10:11:13.310000000,2021-08-24 15:59:44.463333333,2021-05-12 00:04:40.973333333,2021-08-24 16:14:56.170000000]]
Traceback (most recent call last):
  [...]
    pq.write_table(arrow_table, 'output.parquet')
  File "...", line 2092, in write_table
    writer.write_table(table, row_group_size=row_group_size)
  File "...", line 754, in write_table
    self.writer.write_table(table, row_group_size=row_group_size)
  File "pyarrow/_parquet.pyx", line 1506, in pyarrow._parquet.ParquetWriter.write_table
  File "pyarrow/error.pxi", line 99, in pyarrow.lib.check_status
pyarrow.lib.ArrowInvalid: Casting from timestamp[ns] to timestamp[us] would lose data:<PHONE_NUMBER>216666666

@t-alex-fritz looks like it is the issue of type conversion of pyarrow. Can you check whether this allow_truncated_timestamps answers your question? https://stackoverflow.com/questions/53893554/transfer-and-write-parquet-with-python-and-pandas-got-timestamp-error

Oh, you are right, my bad! That last part was a pyarrow thing and works now after setting coerce_timestamps and allow_truncated_timestamps. Thanks a lot @wangxiaoying. And great project! Will be using this a lot.


Create 0000-01-02-KarolinaTechDev.md

created a branch, created a file and made a commit, and opened a pull request

merged pul request

deleted my-slide


Title: 5 Ways to Use Column Level Data Lineage / Montecarlo Data Blog

URL: https://www.montecarlodata.com/title-5-ways-to-use-column-level-data-lineage/?utm_source=rss&utm_medium=rss&utm_campaign=title-5-ways-to-use-column-level-data-lineage
Review (1-2 sentences): Have you ever thought of implementing column level lineage? If not, just read why it can be extremely helpful to have.

Published in Digest #28.


what is the command to convert .mib to file format for simulator?

what is the command to convert TmrSys1Data.mib to file format for simulator?

You should spend enough time on the documentation, as such has been well documented,
https://docs.lextudio.com/snmpsim/quick-start#simulate-from-mib

You should spend enough time on the documentation, as such has been well documented,
https://docs.lextudio.com/snmpsim/quick-start#simulate-from-mib

thanks. i wasn't aware of this documentation.
I am in Windwos cmd. and this command does not works.
pipenv run snmpsim-command-responder
I saw some .py files installed at here C:\Program Files\Python311\Scripts
should i call them individually? filename.py follow by command....

Error:
C:\Windows\system32>pipenv run snmpsim-command-responder --data-dir=./data
Creating a virtualenv for this project...
Pipfile: C:\Windows\System32\Pipfile
Using default python from C:\Program Files\Python311\python.exe (3.11.4) to create virtualenv...
[  ==] Creating virtual environment...created virtual environment CPython3.11.4.final.0-64 in 6018ms
creator CPython3Windows(dest=C:\Users\gan.virtualenvs\System32-eU24dAeB, clear=False, no_vcs_ignore=False, global=False)
seeder FromAppData(download=False, pip=bundle, setuptools=bundle, wheel=bundle, via=copy, app_data_dir=C:\Users\a\AppData\Local\pypa\virtualenv)
added seed packages: pip==24.0, setuptools==69.5.1, wheel==0.43.0
activators BashActivator,BatchActivator,FishActivator,NushellActivator,PowerShellActivator,PythonActivator
Successfully created virtual environment!
Virtualenv location: C:\Users\a.virtualenvs\System32-eU24dAeB
Creating a Pipfile for this project...
'snmpsim-command-responder' is not recognized as an internal or external command,
operable program or batch file.

ok it seems works after correcting python env


Enable LTO

As shown in #4 it may improve perf and reduce executable binary size.
Closes #4.

The technical reason is that I consider you a nuisance and your PR a waste of time, and wish to discourage your behavior.


[DSSv3] Fix issue 13685 - add function arrayShape

This PR depends on PR #8175

Closing this as per the comment in #8175


feat(install_util): add support for DENO_DIR

This PR changes install_util.ts to install esbuild.wasm into DENO_DIR if it is defined.

@uki00a
I'm ok with this addition but can you elaborate a little bit on the motivation of this change?
Do you usually specify DENO_DIR somewhere different from the default?

@kt3k
I'm using asdf-deno to manage multiple Deno environments.
asdf-deno is dynamically switching DENO_DIR depending on the currently enabled Deno version.
For example, if the currently enabled version of Deno is v1.16.4, the following directory will be set to DENO_DIR:
~/.asdf/installs/deno/1.16.4/.cache/deno

To avoid this behavior by asdf-deno, I've explicitly set DENO_DIR in ~/.zshrc as follows:
export DENO_DIR=$HOME/.cache/deno

Therefore, I wanted esbuild.wasm used by packup to also be installed in ~/.cache/deno instead of ~/.deno :slightly_smiling_face:

published as v0.1.10

Thanks for the review and release!


Expose SyncPeriod To Force Reconciliation

This PR contains a small set of changes which exposes a new environment variable in the knative-kafka Controller's Deployment YAML called SYNC_PERIOD_MINUTES.  This variable can be used to force the controllers to re-reconcile ALL resources periodically.  This is intended as a short term recovery enhancement to help minimize the impact of the scenario described in #9758.
Prior to exposing this configuration, the default value (from K8S) was every 10 hours.  The default value included in this PR is now every 4 hours.  It is important to set this value conservatively so as not to unnecessarily or endlessly perform reconciliation.  The trade-off is between churn and downtime, and will depend on the number of resources in the system in question.  Probably taking the period down to ~30 minutes would be a reasonable minimum (aggressive) setting.
These changes have NOT been tested!!!   A minimal unit test exists for the environment variable processing, and the configuration changes were only very quickly verified (one time outside a Kyma cluster) to prove they had the intended effect of forcing the re-reconciliation.  This PR is being created so that someone in the Kyma organization can take them and perform actual QA testing in a Kyma cluster with a large number of KafkaChannels / Subscriptions to verify the following...

What a reasonable Sync Period duration would be for given cluster sizes.
That no unintended side-effects occur from the more frequent re-reconciliation.

If the changes are shown to be desirable, then the Kyma organization can decide whether to perform another release of the deprecated knative-kafka project, and to determine how/where/when to deploy the update.

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


React-Native version

Hello, I'm trying to convert this library ES6 syntax. I want to run this library in client side. Is this possible. I also wonder that is this library use socks5-https-client on backside. This is what i did: https://github.com/kaanoner3/instagram-private-api
this is what i got when i try to import instagram-private-api:

Hey @kaanoner3, did you manage to port this into react-native library? I am doing the exact same at the moment and i came across your fork of the repository...

Hey, has anyone successfully accomplished this? I'm trying to do the same right now.

Man you asking about 2 completely non related things. React-native is front end stuff, and this repo can be build as rest


Add IdentityUser POCO navigation properties - issue with generic value int

Hi,
for me the example of the navigation properties did not work when I tried to use Entity Framework Core Add-Migration along with it.
When calling
Add-Migration MyMigrationName -Context "MyAppIdentityDbContext"
I received an error message similar to the following:
The entity type 'IdentityUserLogin' requires a primary key to be defined.
The issue was that the properties have been defined as follows:
///
/// Navigation property for the roles this user belongs to.
///
public virtual ICollection<IdentityUserRole> Roles { get; } = new List<IdentityUserRole>();
public virtual ICollection<IdentityUserClaim> Claims { get; } = new List<IdentityUserClaim>();
public virtual ICollection<IdentityUserLogin> Logins { get; } = new List<IdentityUserLogin>();
As a generic type there is used an int. I changed that to string as the primary key in the AspNetUsers table is of type nvarchar.
Because of that it seems that the int considered as foreign key does not successfully map.
Change the properties from e.g.
public virtual ICollection<IdentityUserClaim> Claims { get; } = new List<IdentityUserClaim>();
to
public virtual ICollection<IdentityUserClaim> Claims { get; } = new List<IdentityUserClaim>();
and it worked.
This might be an issue with the example?
Best Regards,
Lars

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

ID: 1f4b263b-e8f9-8b44-8c13-5bc0128fb1c9
Version Independent ID: 985774b6-a165-f22c-c354-6c12a6783063
Content: Migrate authentication and Identity to ASP.NET Core 2.0
Content Source: aspnetcore/migration/1x-to-2x/identity-2x.md
Service: asp.net-core
GitHub Login: @scottaddie
Microsoft Alias: scaddie

Unfortunately this is very confusing as evidenced by the very long thread at https://github.com/aspnet/Identity/issues/1361.  Hopefully these navigation properties will be added to the new project templates when selecting individual user accounts.  Below is what I ended up using in my project:
public virtual ICollection<IdentityUserRole<string>> Roles { get; set; }
public virtual ICollection<IdentityUserClaim<string>> Claims { get; set; }


links is bad

[Enter feedback here]
July 12, 2022 HoloLens 2 Emulator Links =June 14
The link refers to last month's version, please update the link quickly, by the way, this is the case for every monthly update, I hope it can reduce feedback
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: bf19b222-f79a-c16b-ab29-f92c05a14651
Version Independent ID: bf19b222-f79a-c16b-ab29-f92c05a14651
Content: HoloLens Emulator archive - Mixed Reality
Content Source: mixed-reality-docs/mr-dev-docs/develop/advanced-concepts/hololens-emulator-archive.md
Product: mixed-reality
Technology: mr-dev
GitHub Login: @hamalawi
Microsoft Alias: moelhama

Hi @a11303, I just checked the link and it looked fine to me. Please reopen if you are still having problems.


Better query-based filtering

It would be nice if all GET routes could support query-based filtering ?field=value for every field (including for /user).

More or less done in branch develop.
Could be improved, though.


Pipenv install fails with traceback on Ubuntu 16.04.4

I am converting an already-running django project on an Ubuntu server over to pipenv. I converted my requirements.txt file over to Pipfile and Pipfile.lock on my local system, and it's running fine. When I ssh into the server, pull down my code, and run pipenv install, the system can hang for over 30 minutes (usually I stop the VM after a few minutes).
Please run $ python -m pipenv.help, and paste the results here.
$ python -m pipenv.help output
Pipenv version: '11.10.0'
Pipenv location: '/home/innovationuser/.pyenv/versions/3.6.4/lib/python3.6/site-packages/pipenv'
Python location: '/home/innovationuser/.pyenv/versions/3.6.4/bin/python'
Other Python installations in PATH:

2.7: /usr/bin/python2.7
2.7: /usr/bin/python2.7
Traceback (most recent call last):
File "/home/innovationuser/.pyenv/versions/3.6.4/lib/python3.6/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/home/innovationuser/.pyenv/versions/3.6.4/lib/python3.6/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/innovationuser/.pyenv/versions/3.6.4/lib/python3.6/site-packages/pipenv/help.py", line 89, in
main()
File "/home/innovationuser/.pyenv/versions/3.6.4/lib/python3.6/site-packages/pipenv/help.py", line 24, in main
found = find_python_in_path(python_v)
File "/home/innovationuser/.pyenv/versions/3.6.4/lib/python3.6/site-packages/pipenv/core.py", line 355, in find_python_in_path
version = python_version(p)
File "/home/innovationuser/.pyenv/versions/3.6.4/lib/python3.6/site-packages/pipenv/utils.py", line 218, in python_version
return u'{major}.{minor}.{micro}'.format(**version)
TypeError: format() argument after ** must be a mapping, not NoneType

Expected result
Create a new .venv with dependencies.
Actual result
Hangs the server for 30+ minutes with what seems to be pegged disk read bytes (nearly 200MB/min) (minimal network activity and <10% CPU usage).
The console returned this: https://pastebin.com/Rn5v7T01
I piped pipenv install into a log file and got this: https://pastebin.com/GUsEA11b
Steps to replicate
Provide the steps to replicate (which usually at least includes the commands and the Pipfile).
> pipfile install

Pipfile:
[[source]]
url = "https://pypi.python.org/simple"
verify_ssl = true
name = "pypi"

[packages]
"Fabric3" = "*"
wagalytics = "*"
wagtail = "*"
wagtailfontawesome = "*"
wagtailmenus = "*"
django-debugtools = "*"

[dev-packages]

[requires]
python_version = "3.5.2"

@Chris-May I think you generated at least part of this pipfile by hand, because python_version is only ever allowed to be a short version (e.g. 3.5 in this case) -- we have better messaging in the coming point release but this is likely part of the problem.

As for the other bug we have already pushed a fix and will have a release out very soon, just need to rerun tests now that pip 10.0.1 is out

Awesome! Thanks!


keybase login issue

[write something useful and descriptive here]
my log id: c5db3651de6745e29d73cb1c

How I will do that I dont know

Where I will type this verification code? I am using only one device and I want to use keybase on terminal.


ENH: Test itk::ImageFileReader image IO to requested region mismatch

ENH: Test itk::ImageFileReader image IO to requested region mismatch
STYLE: Break ImageCompare module test command into multiple lines

PR Checklist

[X] No API changes were made (or the changes have been approved)
[X] No major design changes were made (or the changes have been approved)
[X] Added test (or behavior not changed)

This now has a merge conflict. Please update.

This now has a merge conflict. Please update.

Done.


[BUG] Enabling nullable reference types causes ReactiveContentPage to error

Nullable causes ReactiveContentPage to error
Reopening #2330
Created a ReactiveContentPage and enabled Nullable reference types causes
MainPage.xaml.cs(5, 26): [CS0263] Partial declarations of 'MainPage' must not specify different base classes
Steps To Reproduce

Created sample project to reproduce issue. https://github.com/grierson/Sandbox
Try to build the project.
Error

Expected behavior
For the project to run.
Environment

ReactiveUI: 11.5.35
ReactiveUI.XamForms: 11.5.35
Xamarin.Forms: 4.8.0

Additional context
Use ReactiveContentPage for ReactiveUI.Validation binding

This one is likely a wpf or dotnet bug. You'll have to lodge the bug with them about the nullability issue.

One suggestion
https://github.com/grierson/Sandbox/blob/99acf158da82e070602cdd75b8ba35666a0c5911/Sandbox/Sandbox/MainPage.xaml.cs#L5
Remove the base type declaration in your c# file. It's not needed and redundant due to it being a partial class.
It's likely due to the view model being nullable which makes sense since you won't have the view model always set all the time.

This is by design and not a bug. We make the view models property nullable.


Use try_insert to prevent panics

When using insert instead of try_insert on EntityCommand, it can happen that the system panics, when the entity does not exist, even when the EntityCommand has been accessed with get_entity(...). The following example demonstrates the problem:
if let Some(mut entity_commands) = commands.get_entity(entity) {
    // at this point some other system despawns `entity` concurrently
    entity_commands.insert(bundle);
    //              ^^^^^^ this will panic
}

This is also documented in the official bevy docs: https://docs.rs/bevy/0.12.1/bevy/ecs/system/struct.Commands.html#method.get_entity
Other occurrences of .insert (where probably no change is necessary)
I've also looked at other occurrences of .insert on EntityCommand in the code, but I don't think we will have problems there, when .insert is used, because it is directly retrieved from the Query of the same system, but I'm not sure. What do you think?
Here are other locations of .insert:
https://github.com/aevyrie/bevy_mod_picking/blob/1471a9f83435f3fdd43829b90391e6ab965e0e73/backends/bevy_picking_egui/src/lib.rs#L64-L71
https://github.com/aevyrie/bevy_mod_picking/blob/1471a9f83435f3fdd43829b90391e6ab965e0e73/crates/bevy_picking_highlight/src/lib.rs#L287-L296

Thank you for merging and also adding it to the other call sites!❤


docs: Fixed typos

Fixed typos inside envoy/docs/root/start/quick-start/securing.rst
Signed-off-by: Rajdeep Roy Chowdhury<EMAIL_ADDRESS>Commit Message: Fixed typos inside envoy/docs/root/start/quick-start/securing.rst
Additional Description: N/A
Risk Level: Low
Testing: N/A
Docs Changes: envoy/docs/root/start/quick-start/securing.rst

/retest


TST refine widget fs API unit tests

Continuation of #13.

@Charlie-XIAO I have refactored the unit test for fs api. Could you write a review for it?

LGTM, here is the coverage report:

I have just noticed that it was me who opened the PR, so I cannot approve it myself. However, the main works are done by @Xinyu-Li-123 and I just modified on top of that, so I will override the requirements and merge (especially due to the deadline).


test_results_path as a directory

First of all thanks a lot for this awesome action!
We are using multi-stage docker builds so our tests run inside docker.
I was trying to use the test_restults_path variable to point to a directory because we are writing all the tests to a directory.
It seems like test_results_path is expected to be an absolute path to a single file.
Is there a possibility to point to a directory and then generate a test report for all the trx files present in the directory?
If I do provide an absolute path, I am getting this error
Export-Clixml: /home/runner/work/_actions/zyborg/dotnet-tests-report/v1.0.0/action.ps1:279
Line |
 279 |      Export-Clixml -InputObject $testResult -Path $result_clixml_path
     |      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Could not find a part of the path
     | '/home/runner/work/auditlogas/auditlogas/_TMP/dotnet-test-result.clixml'.

The above error seems to be happening because the _TMP directory doesn't exist.
If I create $PWD/_TMP prior to running this action then it works as expected.
I would really appreciate some guidance
Thanks!

@ebekker I have opened a PR to allow failing the check if there is a failing test.

Closing this issue as I have created a new action to get the desired output.
https://github.com/marketplace/actions/trx-parser

Related:  https://github.com/github/docs/issues/2403

Related:  https://github.com/github/docs/issues/2403


TextLayer: support text auto wrapping

Background
For #3299

Change List

add wordBreak and width options to TextLayer
add test
update doc

Coverage decreased (-0.1%) to 79.846% when pulling 82a335e665550ce1c6dc4405b79055e2542f889d on xx/auto-wrap into 9cc5501b763cfa7f914d0dce45d5968a1153c0f8 on master.


GPG error: https://developer.download.nvidia.com/compute/cuda/repos/ubuntu2004/x86_64  InRelease: The following signatures couldn't be verified because the public key is not available:

trying to build the docker image for AlphaFold v2.2.0 on CentOS 7 using:
docker build -f docker/Dockerfile -t alphafold .
But I fail with this message:
W: GPG error: https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64  InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY A4B469963BF863CC
E: The repository 'https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64  InRelease' is not signed.
Could you assist me here ?
(alphafold) [root@localhost alphafold-2.2.0]# docker build -f docker/Dockerfile -t alphafold .
Sending build context to Docker daemon  7.203MB
Step 1/18 : ARG CUDA=11.1
Step 2/18 : FROM nvidia/cuda:${CUDA}-cudnn8-runtime-ubuntu18.04
---> 81a7b51c961c
Step 3/18 : ARG CUDA
---> Using cache
---> 7b4ea3583467
Step 4/18 : SHELL ["/bin/bash", "-c"]
---> Using cache
---> 821d4ba2ec1f
Step 5/18 : RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -y       build-essential       cmake       cuda-command-line-tools-${CUDA/./-}       git       hmmer       kalign       tzdata       wget     && rm -rf /var/lib/apt/lists/*
---> Running in 131c8c2a7422
Get:1 http://security.ubuntu.com/ubuntu bionic-security InRelease [88.7 kB]
Get:2 https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64  InRelease [1575 B]
Err:2 https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64  InRelease
The following signatures couldn't be verified because the public key is not available: NO_PUBKEY A4B469963BF863CC
Get:3 http://archive.ubuntu.com/ubuntu bionic InRelease [242 kB]
Ign:4 https://developer.download.nvidia.com/compute/machine-learning/repos/ubuntu1804/x86_64  InRelease
Get:5 https://developer.download.nvidia.com/compute/machine-learning/repos/ubuntu1804/x86_64  Release [564 B]
Get:6 https://developer.download.nvidia.com/compute/machine-learning/repos/ubuntu1804/x86_64  Release.gpg [833 B]
Get:7 http://security.ubuntu.com/ubuntu bionic-security/multiverse amd64 Packages [21.1 kB]
Get:8 http://security.ubuntu.com/ubuntu bionic-security/main amd64 Packages [2747 kB]
Get:9 https://developer.download.nvidia.com/compute/machine-learning/repos/ubuntu1804/x86_64  Packages [73.8 kB]
Get:10 http://security.ubuntu.com/ubuntu bionic-security/restricted amd64 Packages [921 kB]
Get:11 http://security.ubuntu.com/ubuntu bionic-security/universe amd64 Packages [1498 kB]
Get:12 http://archive.ubuntu.com/ubuntu bionic-updates InRelease [88.7 kB]
Get:13 http://archive.ubuntu.com/ubuntu bionic-backports InRelease [74.6 kB]
Get:14 http://archive.ubuntu.com/ubuntu bionic/universe amd64 Packages [11.3 MB]
Get:15 http://archive.ubuntu.com/ubuntu bionic/main amd64 Packages [1344 kB]
Get:16 http://archive.ubuntu.com/ubuntu bionic/restricted amd64 Packages [13.5 kB]
Get:17 http://archive.ubuntu.com/ubuntu bionic/multiverse amd64 Packages [186 kB]
Get:18 http://archive.ubuntu.com/ubuntu bionic-updates/universe amd64 Packages [2275 kB]
Get:19 http://archive.ubuntu.com/ubuntu bionic-updates/main amd64 Packages [3181 kB]
Get:20 http://archive.ubuntu.com/ubuntu bionic-updates/multiverse amd64 Packages [29.8 kB]
Get:21 http://archive.ubuntu.com/ubuntu bionic-updates/restricted amd64 Packages [954 kB]
Get:22 http://archive.ubuntu.com/ubuntu bionic-backports/main amd64 Packages [12.2 kB]
Get:23 http://archive.ubuntu.com/ubuntu bionic-backports/universe amd64 Packages [12.9 kB]
Reading package lists...
W: GPG error: https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64  InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY A4B469963BF863CC
E: The repository 'https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64  InRelease' is not signed.
The command '/bin/bash -c apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -y       build-essential       cmake       cuda-command-line-tools-${CUDA/./-}       git       hmmer       kalign       tzdata       wget     && rm -rf /var/lib/apt/lists/*' returned a non-zero code: 100

I have the same problem.
It's related to:
https://developer.nvidia.com/blog/updating-the-cuda-linux-gpg-repository-key/
https://forums.developer.nvidia.com/t/notice-cuda-linux-repository-key-rotation/212771
Most solutions that were suggested did not work for me.

remove /etc/apt/sources.list.d/cuda.list and /etc/apt/sources.list.d/nvidia-ml.list before updating the key.
RUN rm /etc/apt/sources.list.d/cuda.list
RUN rm /etc/apt/sources.list.d/nvidia-ml.list
RUN apt-key del 7fa2af80
RUN apt-get update && apt-get install -y --no-install-recommends wget
RUN wget https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64/cuda-keyring_1.0-1_all.deb
RUN dpkg -i cuda-keyring_1.0-1_all.deb

I got the same errors and none of the solutions suggested by nvdia-cuda worked for me.

I was able to build the Docker container from the Dockerfile last week by adding this line:
RUN apt-key adv --fetch-keys https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64/3bf863cc.pub

I have exactly the same problem. None of the suggestions above helped. Is there an update from the developers on this issue yet?
docker build -f docker/Dockerfile -t alphafold .
Sending build context to Docker daemon  13.31MB
Step 1/18 : ARG CUDA=11.1
Step 2/18 : FROM nvidia/cuda:${CUDA}-cudnn8-runtime-ubuntu18.04
---> 81a7b51c961c
Step 3/18 : ARG CUDA
---> Using cache
---> a45fc778b185
Step 4/18 : SHELL ["/bin/bash", "-c"]
---> Using cache
---> 226f7f1a04ce
Step 5/18 : RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -y       build-essential       cmake       cuda-command-line-tools-${CUDA/./-}       git       hmmer       kalign       tzdata       wget     && rm -rf /var/lib/apt/lists/*
---> Running in 4cba039f6292
Get:1 https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64  InRelease [1581 B]
Err:1 https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64  InRelease
The following signatures couldn't be verified because the public key is not available: NO_PUBKEY A4B469963BF863CC
Get:2 http://archive.ubuntu.com/ubuntu bionic InRelease [242 kB]
Get:3 http://security.ubuntu.com/ubuntu bionic-security InRelease [88.7 kB]
Get:4 http://archive.ubuntu.com/ubuntu bionic-updates InRelease [88.7 kB]
Get:5 http://archive.ubuntu.com/ubuntu bionic-backports InRelease [74.6 kB]
Ign:6 https://developer.download.nvidia.com/compute/machine-learning/repos/ubuntu1804/x86_64  InRelease
Get:7 https://developer.download.nvidia.com/compute/machine-learning/repos/ubuntu1804/x86_64  Release [564 B]
Get:8 http://archive.ubuntu.com/ubuntu bionic/multiverse amd64 Packages [186 kB]
Get:9 https://developer.download.nvidia.com/compute/machine-learning/repos/ubuntu1804/x86_64  Release.gpg [833 B]
Get:10 http://archive.ubuntu.com/ubuntu bionic/main amd64 Packages [1344 kB]
Get:11 http://archive.ubuntu.com/ubuntu bionic/restricted amd64 Packages [13.5 kB]
Get:12 http://archive.ubuntu.com/ubuntu bionic/universe amd64 Packages [11.3 MB]
Get:13 https://developer.download.nvidia.com/compute/machine-learning/repos/ubuntu1804/x86_64  Packages [73.8 kB]
Get:14 http://archive.ubuntu.com/ubuntu bionic-updates/restricted amd64 Packages [966 kB]
Get:15 http://archive.ubuntu.com/ubuntu bionic-updates/multiverse amd64 Packages [29.8 kB]
Get:16 http://archive.ubuntu.com/ubuntu bionic-updates/main amd64 Packages [3199 kB]
Get:17 http://archive.ubuntu.com/ubuntu bionic-updates/universe amd64 Packages [2277 kB]
Get:18 http://security.ubuntu.com/ubuntu bionic-security/main amd64 Packages [2765 kB]
Get:19 http://archive.ubuntu.com/ubuntu bionic-backports/universe amd64 Packages [12.9 kB]
Get:20 http://archive.ubuntu.com/ubuntu bionic-backports/main amd64 Packages [12.2 kB]
Get:21 http://security.ubuntu.com/ubuntu bionic-security/restricted amd64 Packages [932 kB]
Get:22 http://security.ubuntu.com/ubuntu bionic-security/multiverse amd64 Packages [22.8 kB]
Get:23 http://security.ubuntu.com/ubuntu bionic-security/universe amd64 Packages [1503 kB]
Reading package lists...

@0Bernhard Have you tried adding the following just below the SHELL ["/bin/bash", "-c"] line in the Dockerfile?
SHELL ["/bin/bash", "-c"]

RUN apt-key adv --fetch-keys https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64/3bf863cc.pub

The issue is also known to Nvidia and they will hopefully resolve it soon (this week) by releasing an updated Docker image, see https://gitlab.com/nvidia/container-images/cuda/-/issues/158 for more details.

FYI, there are two PRs that fix this by running apt-key command, so you can use those until Nvidia fixes this if you want

fbartusch's answer seemed to work for me. Thanks.

Fixed in https://github.com/deepmind/alphafold/commit/55c948c1ae51a17829d7a59bd1f01eaacdf9566e by bumping the CUDA version to 11.1.1.


white screen on ios with -- --livereload

Hi , i got a white screen on my emulator, if i run with :
cordova run ios -- --livereload
someone got a idea ?
thx :-)

haaa, I know what's going on here. the ios emulator tries to connect to 'localhost' instead of 'external', I will work on a fix soon.

Hey @istrion, this issue has been fixed with the following PR: https://github.com/omefire/cordova-plugin-livereload/pull/53
Could you please try it out and let me know if it now works for you ?


chore: move channels into impl classes

This broke implementation:
 { "chromiumSandbox", options.ChromiumSandbox },                 { "slowMo", options.ChromiumSandbox },


feat: Enable user to enable mirroring collector capability.

@ethicalmohit thx for the PR. Can you plz fix lint issues? follow this doc

/gcbrun


New Realtime GVT Scheduler, synch=5

Hi all;
A new ROSS "realtime" scheduler is available that computes GVT at a set realtime interval. This is for applications that have either wild rollback behaviors or a good amount of variance in the time to compute a single event.  An example PHOLD run command line is as follows:
mpirun -np 4 ./phold --synch=5 --extramem=65536 --nlp=1024 --batch=32 --gvt-interval=256 --gvt-realtime-interval=3600000 --clock-rate=3600000000 --start-events=16  --lookahead=0.10 --end=2000 > results-sync-5
Note, above you are using the new Real Time scheduler with --synch=5. Here, the --gvt-realtime-interval is the number of clock cycles between successive GVTs. So, you need to set the clock rate of
the processor you are using via --clock-rate.
To which back to using the original ROSS optimistic schedule, just use:
mpirun -np 4 ./phold --synch=3 --extramem=65536 --nlp=1024 --batch=32 --gvt-interval=256 --gvt-realtime-interval=3600000 --clock-rate=3600000000 --start-events=16  --lookahead=0.10 --end=2000 > results-sync
Notice, they only thing that chance is the --sync option. This is because you can keep realtime
and original GVT interval settings for both command lines and the scheduler determines which set of options to us (either gvt-interval or gvt-realtime-interval).
Enjoy!,
Chris

Current coverage is 72.23%

Merging #79 into master will decrease coverage by -2.42% as of 15b268f

@@            master     #79   diff @@
======================================
  Files           25      25
  Stmts         2438    2536    +98
  Branches         0       0
  Methods
======================================
+ Hit           1820    1832    +12
  Partial          0       0
- Missed         618     704    +86

Review entire Coverage Diff as of 15b268f

Powered by Codecov. Updated on successful CI builds.

CMake needs to test if the environment variable $CLOCK_SPEED is actually set...

This might be a dumb question, but why is the time in terms of clock cycles instead of "real" time? If I'm reading the example correctly, the interval is set to 1 millisecond<PHONE_NUMBER>/3600000) - is there an issue with just using that instead?

@JohnPJenkins I completely agree and I've made the changes to support that. The --gvt-interval command line option now sets the number of milliseconds between GVTs when used with --synch=5.

@gonsie  Thanks! Just to be sure, do we need to still worry about having the right clock speed set, and take DVFS and thread affinities into account (ie is rtdsc still being used)?

We may wish to take advantage of the Precision Time Protocol.  From the article, "... it achieves clock accuracy in the sub-microsecond range, making it suitable for measurement and control systems."
ptpd contains a PTP implementation.

Checklist for merging:

[X] Tested with Travis
[X] Coverage increases
[X] Documentation exists: blog post


[android_intent] Added 'action_app' action and getIntentExtras, getIntentData

The "action_app" special action allows to create an Intent that launches the app's launcher activity (which for most Flutter apps, it's the Activity handling the Flutter app itself).
Also added methods getIntentExtras and getIntentData which respectively let you retrieve the current Intent's extras [arguments] and data.
These two features combined are useful when your app is divided between a foreground component (the main app running the Flutter view) and a background component (a service that runs in the background).
With these features the background component can easily trigger the main app to open (whether it was running or not) and pass extra arguments to the Intent. The main app can then load up and use these extra arguments to do some custom processing.
Please note that if your app is already opened and you launch a "action_app" intent, you need a bit of extra logic in your MainActivity to ensure that arguments are refreshed from the new Intent, something like this:
  @Override
  protected void onNewIntent(Intent intent) {
      super.onNewIntent(intent);
      Bundle bundle = intent.getExtras();
      getIntent().putExtras(bundle);
  }

Re-implemented in https://github.com/flutter/plugins/pull/2308


[Gally]: master <- dev

Automatically created by Git-Ally

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


On Android, don't resolve the asset directory dynamically

Thanks for supplying a fix for this. I take it this fixes the startup error when you ran it in an Android emulator?

Yes, exactly. Sorry for for the lack of details.


Uploader does not send data if no Guardian3 sensor used

If I'm just using the 670G without the guardian3 sensor, the uploader doesn't upload data. It would be good for basal,temp basal, CAGE,Carb,Bolus data to be sent to nightscout even if there's no sensor being used (Full disclosure, I used guardian3, got tired of it, and am in the process of switching to dexcom while keeping the 670G).
Comments welcome
Yaron

Your Environment

Uploader Version Number:
Your Android Phone Model Name (e.g. Samsung J1 Ace):
Android Version Number (e.g. use 4.4 for 4.4.4):
Network Connection at the time the issue occurred (Wi-Fi, Mobile):
(if relevant) Nightscout website version (e.g. 0.8.4 Funnel cake, master / dev branch):

Brief Explanation of Issue

Uploader Error Code

Steps to Reproduce (for bugs)

Severity Score

It should be working and uploading pump events even when the pumps cgm is not in use. Are you using version 0.7.1 of the uploader?

Thanks! I'll verify my version
-------- Original message --------From: Pogman<EMAIL_ADDRESS>Date: 12/22/19  9:16 PM  (GMT-08:00) To: pazaan/600SeriesAndroidUploader<EMAIL_ADDRESS>Cc: kretch<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [pazaan/600SeriesAndroidUploader] Uploader does not send data if no Guardian3 sensor used (#263) It should be working and uploading pump events even when the pumps cgm is not in use. Are you using version 0.7.1 of the uploader?
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.
[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/pazaan/600SeriesAndroidUploader/issues/263?email_source=notifications\u0026email_token=AB24KMHGFXAMZTKNM3BFPTTQ2BCUNA5CNFSM4J6PJO3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHQHLZY#issuecomment-568358375",
"url": "https://github.com/pazaan/600SeriesAndroidUploader/issues/263?email_source=notifications\u0026email_token=AB24KMHGFXAMZTKNM3BFPTTQ2BCUNA5CNFSM4J6PJO3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHQHLZY#issuecomment-568358375",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]

Works perfectly with 0.7.1
Thanks
Yaron
On Sun, Dec 22, 2019, 9:54 PM yaronkretchmer<EMAIL_ADDRESS>wrote:

Thanks! I'll verify my version
-------- Original message --------
From: Pogman<EMAIL_ADDRESS>Date: 12/22/19 9:16 PM (GMT-08:00)
To: pazaan/600SeriesAndroidUploader <
<EMAIL_ADDRESS>Cc: kretch<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [pazaan/600SeriesAndroidUploader] Uploader does not send data
if no Guardian3 sensor used (#263)
It should be working and uploading pump events even when the pumps cgm is
not in use. Are you using version 0.7.1 of the uploader?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pazaan/600SeriesAndroidUploader/issues/263?email_source=notifications&email_token=AB24KMHGFXAMZTKNM3BFPTTQ2BCUNA5CNFSM4J6PJO3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHQHLZY#issuecomment-568358375,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB24KMATGR6UPKULOGYIJWDQ2BCUNANCNFSM4J6PJO3A
.

Ok great, thanks for letting us know.


Split and documment

Split the types and functions in  0519a3e2f84a9bc12d012b27ab1464184484441e

I already splitted the code and docummented some, but i still have to check if this is working


iOS compilation issues

solves issues reported in issue #1

looks good to me, thanks for the pull


Use ignored files

The Template.@ignore set was not being used to filter out undesired template files

Merged in Pull Request #46.


Setting internal ip to pod ip.

Fixes #2044

@googlebot I signed it!

I also had the same problem. This fix solved it.

I also had the same problem. This fix solved it.

This fix will only work when we are running perfkit from within the K8s cluster (since perfkit does SSH via CNI IP).
I am exploring ways to execute perfkit from a VM that is outside my K8s cluster.

@marcomicera After this fix, I am also facing issue with SSH. The dockerfile provided in this repo doesn't works. Did you do any modifications in your Dockerfile for running this. I'd be glad if you can share.

@jerlawson Can you please take a look at this PR and approve it.

Please rebase from head so we can see a cleaner diff.

Please rebase from head so we can see a cleaner diff.

Done


[PECO-964] Initial Staging Ingestion implementation

PECO-964

Codecov Report
Patch coverage is 13.20% of modified lines.

Files Changed
Coverage

lib/DBSQLClient.ts
ø

lib/DBSQLOperation/index.ts
0.00%

lib/errors/OperationStateError.ts
0.00%

lib/DBSQLSession.ts
12.50%

lib/errors/StagingError.ts
100.00%

:loudspeaker: Thoughts on this report? Let us know!.


add new GANs: Bayesian GAN, CcycleGAN, DSP-GAN, EGAN, GANCS, SBADA-GAN

Added 6 new GANs and updated the cumulative GAN plot.
There is a new name collision on Bayesian GAN. The second paper is rebranding the old method as LFVI GAN (Likelihood-free Variational Inference GAN) and it's actually a different approach, because it marginalizes the latent variables in Bayesian inference, while the former method marginalizes only on the parameters.
I kept for each method the name used in the proposing paper for consistency.
The new GANs:

Bayesian GAN
Conditional cycleGAN
Depth Structure Preserving GAN (DSP-GAN)
Enhanced GAN (EGAN)
GAN Compressive Sensing (GANCS)
Symmetric Bi-directional ADAptive GAN (SBADA-GAN)

Okay, we can refer the original Bayesian GAN as LFVI GAN if further papers refer it as such perhaps. Will keep an eye.


fix: scroll_preview maps not deleted after docs window closed

Hi there, i was getting this weird error from scroll preview window when trying to use the  in buffer because these up and down keys were not getting deleted after the preview window was closed.
I am new programmer so lurking around in a big code base is not my greatest power so what i had in my knowledge tried to use and came up with this messy solution which isn't great but you can fix it.
   E5108: Error executing lua: .../share/nvim/lazy/lspsaga.nvim/lua/lspsaga/diagnostic.lua:125: Invalid window id: 1061
stack traceback:
	[C]: in function 'nvim_win_call'
	.../share/nvim/lazy/lspsaga.nvim/lua/lspsaga/diagnostic.lua:125: in function 'scroll_with_preview'
	.../share/nvim/lazy/lspsaga.nvim/lua/lspsaga/diagnostic.lua:147: in function <.../share/nvim/lazy/lspsaga.nvim/lua/lspsaga/diagnostic.lua:146>

sorry for this bug. can you try the latest commit.

fixed thanks for this amazing plugin. 😍


Get ROCK image right tag considering the latest merged rock files

Applicable spec:
Overview
What happened in Synapse:
24 Nov: I submitted a WIP PR that changes the ROCK images https://github.com/canonical/synapse-operator/pull/111
27 Nov: Publish to Edge workflow sucessfully ran with merged changes (update jsonschema)
27 Nov: I refreshed Synapse charm in production to apply the fix regarding nginx pebble ready event (23 Nov) + this last one regarding jsonschema
For my huge surprise: Synapse image is running the latest Synapse version as was changed in the WIP PR
Rationale
Make publish charm get the latest image from the latest merged PR and not the latest uploaded.
Workflow Changes

Checklist

[X] The contributing guide was applied
[X] The PR is tagged with appropriate label (urgent, trivial, complex)

The problem of using head_sha is that we are using the squash merge, which causes the commit in main branch is different with the commit in the pull request.
For example, the commit in main branch is https://github.com/canonical/synapse-operator/commit/7defdb47bea9db89844f9dd31db6a44a8ac443bb, but the corresponding commit in pull request is https://github.com/canonical/synapse-operator/pull/111/commits/1e13c045a5bfdf7e851561f98d66475767e3fc82.
Unfortunately, tests inside this repository don't reflect that situation.

The problem of using head_sha is that we are using the squash merge, which causes the commit in main branch is different with the commit in the pull request.
For example, the commit in main branch is canonical/synapse-operator@7defdb4, but the corresponding commit in pull request is canonical/synapse-operator@1e13c04.
Unfortunately, tests inside this repository don't reflect that situation.

I think something like this should work:
# Get commit info
TREE_SHA=$(gh api \
  -H "Accept: application/vnd.github+json" \
  -H "X-GitHub-Api-Version: 2022-11-28" \
  /repos/${{ github.repository }}/commits/${GITHUB_SHA} \
   --jq '.commit.tree.sha')
# Get workflow from this specific tree id
RUN_ID=$(gh api \
  --method GET \
  -H "Accept: application/vnd.github+json" \
  -H "X-GitHub-Api-Version: 2022-11-28" \
  /repos/${{ github.repository }}/actions/runs \
  -f path=.github/workflows/integration_test.yaml \
  -f status=completed \
  -f event=pull_request \
  --jq "[
          .workflow_runs[]
          | select(.path == \".github/workflows/integration_test.yaml\")
          | select(.head_commit.tree_id == \"$TREE_SHA\")
        ] | max_by(.updated_at) | .id")

The problem of using head_sha is that we are using the squash merge, which causes the commit in main branch is different with the commit in the pull request.
For example, the commit in main branch is canonical/synapse-operator@7defdb4, but the corresponding commit in pull request is canonical/synapse-operator@1e13c04.
Unfortunately, tests inside this repository don't reflect that situation.


Feature request: fast failure when a template cannot be filled

When using --var-files, we accidentally forgot to add a variable to our file.  We should have added the line some_variable_name: correct-value in our variable file, but missed it.
The manifest got deployed with a property value set to the literal string ((some_variable_name)), and we had to look through logs to discover that.  This was surprising.
It would be great if the deploy command gave us fast feedback that our template couldn't be completely filled in, because a variable wasn't set.
cc: @markstgodard

@rosenhouse we have build-manifest command with --var-errs flag that will force CLI to list any missing variables. the reason why deploy cmd doesnt do it is that we are planning for the Director to do interpolation as well, so soon enough it will raise errors. In summary: two levels of interpolation, one in CLI, and one is in the Director and the Director part isn't complete yet.

makes sense, thanks for the quick response!


Better UI

Make it were you can put the web UI inside of a Webpage willing to pay for it to get done!

Closing as this can be done via an


parceler not support List Object

Hi, help me please!
in kotlin parceler not support list ?
this field in class

in the build process out error :(
Parceler: Unable to find read/write generator for type java.util.List<? extends com.merqueo.banners.model.Product> for java.util.List<? extends com.merqueo.banners.model.Product> products     java.util.List<? extends com.merqueo.banners.model.Product> products, @org.jetbrains.annotations.Nullable()
I have a different error :(
69: error: incompatible types: inference variable T has incompatible bounds             new com.merqueo.utils.ItemListParcelConverter().toParcel(InjectionUtil.getField(new InjectionUtil.GenericType<List<?>>(), com.merqueo.banners.model.Banner.class, banner$$1, "products"), parcel$$1);                                                                                ^     equality constraints: List<?>     upper bounds: List<? extends Product>,Object   where T is a type-variable:     T extends Object declared in method <T>getField(GenericType<T>,Class<?>,Object,String) Note: Some input files use or override a deprecated API. Note: Recompile with -Xlint:deprecation for details. Note: Some input files use unchecked or unsafe operations. Note: Recompile with -Xlint:unchecked for details. Note: Some messages have been simplified; recompile with -Xdiags:verbose to get full output 1 error

See #264 it basically depends on your converter. I managed to get it working with List in Kotlin, though it will create an ArrayList behind the scenes and assign it to the List property.

Closing due to inactivity.  Feel free to reopen.


Connection Oracle DB

Hi,
I set an oracle connection as your instructions at https://superset.apache.org/docs/databases/oracle, but it fails, I must to use cx_oracle connection as https://docs.sqlalchemy.org/en/12/core/engines.html#oracle (the second option) because sqlalchemy gets error with listener if you use SIDNAME.
Best regards!

Thanks for reporting! My understanding is our docs needs to be updated. do you @leire23 mind opening a PR for the update? 🙏
@srinify


Automated cherry pick of #8850: fix: add host-joint resources into domain level resources

Cherry pick of #8850 on release/3.5.
#8850: fix: add host-joint resources into domain level resources

Codecov Report

Merging #8851 (862a53d) into release/3.5 (e3fe6c4) will increase coverage by 11.96%.
The diff coverage is 0.00%.

@@               Coverage Diff                @@
##           release/3.5    #8851       +/-   ##
================================================
+ Coverage         6.03%   18.00%   +11.96%
================================================
  Files              795       84      -711
  Lines           124282     8850   -115432
================================================
- Hits              7503     1593     -5910
+ Misses          116114     7088   -109026
+ Partials           665      169      -496

Flag
Coverage Δ

aFlag
18.00% <0.00%> (+11.96%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/apigateway/handler/auth.go
0.00% <0.00%> (ø)

pkg/apigateway/handler/misc.go
0.00% <0.00%> (ø)

pkg/apigateway/handler/proxy.go
0.00% <0.00%> (ø)

pkg/keystone/driver/cas/cas.go
11.11% <0.00%> (+0.52%)
:arrow_up:

pkg/compute/models/guest_template.go

pkg/util/s3auth/v4.go

pkg/compute/models/cloudproviderregions.go

pkg/keystone/tokens/payloads.go

pkg/cloudcommon/db/quotas/quotas.go

pkg/cloudcommon/db/statusdomainresource.go

... and 603 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 55dd336...862a53d. Read the comment docs.

/lgtm
/approve


[distributed]: allow for .ts.update in DistributedMLForecast

Description
It is mentioned here that we can incorporate new data to our MLForecast models by updating their time-series object. However, this feature is not valid in DistributedMLForecast.
Use case
I tried using mlforecast_model._base_ts.update(mlforecast_data), but I got this error:
File /alloc/data/fury_cemld-kubesizer/notebooks/my-env/lib/python3.9/site-packages/mlforecast/core.py:773, in TimeSeries.update(self, df)
    771 def update(self, df: DataFrame) -> None:
    772     """Update the values of the stored series."""
--> 773     validate_format(df, self.id_col, self.time_col, self.target_col)
    774     uids = self.uids
    775     if isinstance(uids, pd.Index):

AttributeError: 'TimeSeries' object has no attribute 'id_col'

So I also tried:
mlforecast_model._base_ts.id_col = "unique_id"
mlforecast_model._base_ts.time_col = "ds"
mlforecast_model._base_ts.target_col = "y"
mlforecast_model._base_ts.update(mlforecast_data)

and this time the error was:
File /alloc/data/fury_cemld-kubesizer/notebooks/my-env/lib/python3.9/site-packages/mlforecast/core.py:774, in TimeSeries.update(self, df)
    772 """Update the values of the stored series."""
    773 validate_format(df, self.id_col, self.time_col, self.target_col)
--> 774 uids = self.uids
    775 if isinstance(uids, pd.Index):
    776     uids = pd.Series(uids)

AttributeError: 'TimeSeries' object has no attribute 'uids'

Hey @maxi-marufo, thanks for using mlforecast. Are you performing distributed inference? I think it may be easier to set keep_last_n during fit (to reduce the history that is kept), use to_local and then update the local forecast object. If that wouldn't work for you I think we could add an update method that would broadcast the new dataframe to all workers and update each partition locally.

Yes, I'm performing distributed inference. I could try to use to_local to be able to then use .ts.update, but that would also require transforming the model back to a distributed version. Is there a method or way to do that? Also, this would actually be a workaround to the update method in DistributedMLForecast.
Btw, another possible feature related to this would be to be able to continue a previous training, by using the init_model  param in LightGBM and xgb_model param in XGBoost. I guess the continuation of an MLForecast training would mean to update the time-series and then refit the LightGBM/XGBoost model, right?

Just added the DistributedMLForecast.update. Note that it takes a pandas dataframe, so that'll be broadcasted to all the workers and each one will update its corresponding series.
For continuing training, yes. You could call update and then train the models for a few more iterations. One important thing to note here is that preprocess and fit modify the internal state of the forecast object (saves the series values and the last dates and so on) so you have to be careful, I suggest using preprocess/fit on a different forecast object that gets the models with the arguments to continue training and then overriding the models_ attribute of your original object, so something like:
orig_fcst = DistributedMLForecast.load(...)
orig_fcst.update(new_df)
new_fcst = DistributedMLForecast(
    models={'lgb': SparkLGBMForecast(init_model=orig_fcst.models_['lgb'], ...)
)
new_fcst.fit(some_df)  # doesn't necessarily have to be new_df, could potentially have a bit more history to compute your features
orig_fcst.models_ = new_fcst.models_
orig_fcst.save(...)

Feel free to open a new issue if you run into problems trying to continue training.


Deprecation.

Note that I've not had time to spend on this project and I think the dependencies are increasingly falling out of date.
If anyone is interested in taking over the project, please do! (@tobkle, @lorensr?)
Otherwise I've added a note: https://github.com/tmeasday/create-graphql-server/blob/master/README.md#create-graphql-server

No, I found it very useful, though—thanks a lot for making it! Wonder in
future how much the "graphql database" idea of Prisma will take hold
On Wed, Feb 28, 2018 at 8:22 PM, Tom Coleman<EMAIL_ADDRESS>wrote:

Note that I've not had time to spend on this project and I think the
dependencies are increasingly falling out of date.
If anyone is interested in taking over the project, please do! (@tobkle
https://github.com/tobkle, @lorensr https://github.com/lorensr?)
Otherwise I've added a note: https://github.com/tmeasday/
create-graphql-server/blob/master/README.md#create-graphql-server
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tmeasday/create-graphql-server/issues/55, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAPVmD190N00Pa-3S1_j2n7m2kG2xcxXks5tZ01LgaJpZM4SXnYJ
.

Thanks Tom for developing this. I still like the idea of having a NodeJS GraphQL generator and I also learnt a lot while working with it. Recently I was working with prisma, as there is a lot activity and they solved already a lot of topics, which this project would have to solve. What I don't like on prisma is, that it is in Scala, would have preferred a NodeJS solution. However, I cannot spend the required time on this project. Thanks for asking.


K8s: use stucts in-place of maps for node meta

NOTE: WIP as there is still a bug in k8s.Extra object serialization - @safchain / @lebauce  - can you check why this is happening?
This PR replaced the K8s field which was a map[string]interface{} with a real structure. So as to support filters/gremlin I use a set of generic methods which traverse the structure and according to content of the skydive tag ("string", "int" or empty) implement the common.Getter interface.
Using this technique maybe more elegant that implementing specific GetFieldString() GetFieldInt64() etc. methods as for example is the case in netlink/neighbor.go so going forward methods I implemented in k8s/graph.go can be moved to common.
I assume we could consider later on relaying directly on the skydive tags thus enabling to cancel all-together the need to implement the common.Getter interface, but this can only be done once all searchable struct fields are tagged (using skydive tag) or if we agree to some convention as to which fields are exposed as string or int64.
In any case - code is ready for review.

@hunchback With a decoder on the Type, we are not protected against specifying conflict types for the same attribute (like Ports). I don't understand why we need such a change as a structure embedding other structure should work
type Metadata struct {
    *JobMetadata
    *ServiceMetadata
    ...
}

run skydive-k8s-tests

run skydive-functional-tests-backend-orientdb

@safchain - do you want us to move forward with this PR? the last was that the k8s Extra data was munged  when viewed in the UI - indicating some the generic serialization of arbitrary objects was not working correctly.

the serialization of the K8s.Extra does not work properly:

@lebauce / @safchain - is ready for final review


Multi Selection clear ng-model doesn't work

I am looking at your demo here : http://plnkr.co/edit/juqoNOt1z1Gb349XabQ2?p=preview
When I click on the button 'Clear ng-model', nothing happens. It does work on the simple select.
I also tried on my application but nothing happens either. Is it just me?

Nevermind, it was only the demo that was not up to date. I made a pull request to fix it. #948
All good for me.


Forcing the response into modal

I  know that the opposite has already been discussed (and I struggled with that too) - namely getting non-inertia responses not to load into a modal. However, I wonder if it would be possible to have an Inertia Response load into a modal? Is there any way of triggering this (other than re-implementing Response and reneder to fiddle with the x-inertia on my side?
Thank you.

I am closing this as I see no interest. I've created a helper that handles the response in way so that I can force a modal. Thanks.

Hi @pdiveris !
There's usually a pretty slow response when it comes to issues, unfortunately.
You can re-open the issue if you feel that this would be a feature you'd like to see in the adapter. 🙂

Sorry folks. I work on this project as I have time. I often don't respond to issues the day they are opened, but rather when I get a moment to sit down and work through them.
For what it's worth, I wouldn't even know how to get an Inertia response to load in a modal instead of the root layout as it does now. The way Inertia works is by utilizing an existing React/Vue.js/Svelte instance, so loading it in an entirely separate part of the DOM doesn't sound easy. That said, modals are totally doable in Inertia, I use them all the time. I simply add them as child components to my page components.
Good luck! 👍

No worries, I gathered that you are thin on time so I closed the issue after finding a way of achieving just that. My comment was not meant to sound as negative or a complain. I appreciate your work with inertiajs, I was trying to something along these lines too, and I am using it heavily. Keep up the good work!

Thanks @pdiveris, I appreciate that! 💪

@pdiveris could you post that solution of yours please? I'm interested into something similar.

@reinink / @pdiveris could you post your solutions to modals?  I am trying to make a "confirm" modal when a user tries to delete something.  I was thinking of something like the error display modals but don't know if inertia has a clever way to do this built in.
Thanks!

@Neophen and @stygiansabyss you can easily load modal on your component as follows;
on the main component where you want the modal to stay; you do something like this.
`

       .... Your Component content goes here!
      Show Modal
<modal-component :prop-show="showDetails" @closeModal="closeModal()">

.... Modal Content Goes here!
<button @click="confirmMethod"> Confirm
<button @click="closeModalMethod"> Cancel

`
then your script can then look like this;
``
Lastly, the content of you modal component can be like this;
`

`
I don't know if this is still helpful cos it is been long this issues was raised. I believe it might help someone though.


New user control doesn't sanitize namespace name

Create a new CppWinRT XAML user control in a project that has a dot in its name (e.g. Microsoft.ReactNative).
The resulting generated code includes the dots in the namespace which is not valid:

namespace winrt::Microsoft.ReactNative::implementation
{
    struct DevMenu : DevMenuT<DevMenu>
    {
        DevMenu();

        int32_t MyProperty();
        void MyProperty(int32_t value);

        void ClickHandler(Windows::Foundation::IInspectable const& sender, Windows::UI::Xaml::RoutedEventArgs const& args);
    };
}

namespace winrt::Microsoft.ReactNative::factory_implementation
{
    struct DevMenu : DevMenuT<DevMenu, implementation::DevMenu>
    {
    };
}

This value is coming from the RootNamespace property which for our project is set to Microsoft.ReactNative (same thing as the project name). Note however that this property cannot be set to a C++ friendly namespace name like Microsoft::ReactNative since winmdexp uses the root namespace value to construct file names which cannot include colons.
So I think CppWinRT needs to be smart here and convert dots to colons if it encounters them in the RootNamespace variable

DevComm issue: https://developercommunity.visualstudio.com/content/problem/1151282/vsix-does-not-correctly-provide-rootnamespace-and.html

Unfortunately, the team does not have the capacity or skills to deal with bugs outside of the core C++/WinRT library and compiler. Feel free to keep the conversation going and contribute a PR as needed.