register offset addition results in >32 bits and -Wc++11-narrowing

Some registers sum the base address and register offset to a result that requires more than 32 bits/4 bytes, which is invalid for a 32-bit address space. For example:
https://github.com/kvasir-io/Kvasir/blob/master/Lib/Chip/Unknown/STMicro/STM32L15xxE/GPIOE.hpp#L6
This results in -Wc++11-narrowing when trying to compile code that includes the header, since the type of all register field values is unsigned.
This is the section from the SVD that results in the addition:
https://github.com/posborne/cmsis-svd/blob/master/data/STMicro/STM32L15xxE.svd#L4172-L4183
As you can see, 0x40021000 + 0xFFFFF800 = 0x140020800
I came across this while compiling tests for the new generator. I am not sure how to fix it; it seems like there's either a mistake in the SVD file or we're missing a step in how to sum the offset and the base address. Do we need to apply a different operator? Do we need to wrap around? Am I missing something with my assumption about the size of the address space?

@odinthenerd Do you know how this works?

check out http://www.st.com/content/ccc/resource/technical/document/reference_manual/cc/f9/93/b2/f0/82/42/57/CD00240193.pdf/files/CD00240193.pdf/jcr:content/translations/en.CD00240193.pdf page 47.


uv loop not supporting PEP 517

apologies if this is obvious -- novice here
im on windows 11 and have been struggling to get this set up for a while
I am wondering if uv loop is necessary for everything to run or is it there solely for optimization
should I consider installing Docker or some type of linux environment?
Thank you in advance! :)
the error in terminal
Note: This error originates from the build backend, and is likely not a problem with poetry but with uvloop (0.17.0) not supporting PEP 517 builds.

I'll look into whether we can drop the uvloop dependency, but in the meantime I recommend doing development on Windows using Windows Subsystem for Linux (WSL).

Thank you, ill look into that--
Running it on my mac now and it works just fine!


Make TLS enabled by default and add default config for Docker Hub

For the ease of pulling images from public Docker Hub repos, a default config is added so makisu would use https + basic auth with empty username and password as required by Docker.
TLS is enabled by default as a common practice.

 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.


Go to defination very very slow to to response in ver 1.33

Issue Type: Bug
[Window Title]
Go to defination very very slow to to response in ver 1.33
[Main Instruction]
Visual Studio Code
When I press ctrl + F12 (ctrl + click) it loading and very very slow go to defination
I tried to disabled all extensions but it still very very slow.
For the same source code, I used version 1.32.3, it worked well.
Please let me know, how to fix it?
Thank you!
Version: 1.33.0 (user setup)
Date: 2019-04-04T15:14:28.026Z
OS: Windows_NT x64 10.0.17134

/duplicate of https://github.com/Microsoft/vscode/issues/71779


register_buffer if not required gradients

I wonder if it makes more sense to register_buffer here when no gradient required?
this will allow to avoid issues when switching on/off gradients for the parent model when fine-tuning for example
https://github.com/lucidrains/light-recurrent-unit-pytorch/blob/25da1a0838809191a0a9094464e019bc9934b875/light_recurrent_unit_pytorch/light_recurrent_unit.py#L36

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx

@inspirit yup, makes sense 👍


Adding community SQL Server adapter. Recommending old MSSQL adapter f…

…or older sails versions as that package has not been updated in two years at the time of this commit (not including their README.md).

Thanks @odnarb.  Minor thing, looks like we'd want to change that to <=0.9.x since swelham's adapter is using the old version of the adapter spec from pre 0.10.  But realistically, we can just switch this out, since I think it's a fair assumption that folks using 0.9 are unlikely to be swapping database adapters without upgrading Sails core as well.

Ok, I wasn't sure. It's been a while since I used 0.9.16!
-brandon

:+1: Looks good :)

Thanks!


Can't get any $TSTRAT through the faucet

I tried multiple addresses, and waited more than 8 hours but I don't get any $TSTRAT through the faucet

And it also accepts accounts from the main network.

@babgev can you confirm whether this is still the case?

Closing as no reply from user.


Getting Embeddings from the base model

Hey everyone!
I've been learning Torchensemble, and I wanted to know how you can extract the embeddings before the final classification layer.
Here is my ensemble model:
VotingClassifier(
 (base_estimator_): CCT(
  (tokenizer): Tokenizer(
   (conv_layers): Sequential(
    (0): Sequential(
     (0): Conv2d(3, 64, kernel_size=(7, 7), stride=(2, 2), padding=(3, 3), bias=False)
     (1): ReLU()
     (2): MaxPool2d(kernel_size=3, stride=2, padding=1, dilation=1, ceil_mode=False)
    )
    (1): Sequential(
     (0): Conv2d(64, 256, kernel_size=(7, 7), stride=(2, 2), padding=(3, 3), bias=False)
     (1): ReLU()
     (2): MaxPool2d(kernel_size=3, stride=2, padding=1, dilation=1, ceil_mode=False)
    )
   )
   (flattener): Flatten(start_dim=2, end_dim=3)
  )
  (classifier): TransformerClassifier(
   (attention_pool): Linear(in_features=256, out_features=1, bias=True)
   (dropout): Dropout(p=0.0, inplace=False)
   (blocks): ModuleList(
    (0): TransformerEncoderLayer(
     (pre_norm): LayerNorm((256,), eps=1e-05, elementwise_affine=True)
     (self_attn): Attention(
      (qkv): Linear(in_features=256, out_features=768, bias=False)
      (attn_drop): Dropout(p=0.1, inplace=False)
      (proj): Linear(in_features=256, out_features=256, bias=True)
      (proj_drop): Dropout(p=0.0, inplace=False)
     )
     (linear1): Linear(in_features=256, out_features=512, bias=True)
     (dropout1): Dropout(p=0.0, inplace=False)
     (norm1): LayerNorm((256,), eps=1e-05, elementwise_affine=True)
     (linear2): Linear(in_features=512, out_features=256, bias=True)
     (dropout2): Dropout(p=0.0, inplace=False)
     (drop_path): Identity()
    )
    (1-6): 6 x TransformerEncoderLayer(
     (pre_norm): LayerNorm((256,), eps=1e-05, elementwise_affine=True)
     (self_attn): Attention(
      (qkv): Linear(in_features=256, out_features=768, bias=False)
      (attn_drop): Dropout(p=0.1, inplace=False)
      (proj): Linear(in_features=256, out_features=256, bias=True)
      (proj_drop): Dropout(p=0.0, inplace=False)
     )
     (linear1): Linear(in_features=256, out_features=512, bias=True)
     (dropout1): Dropout(p=0.0, inplace=False)
     (norm1): LayerNorm((256,), eps=1e-05, elementwise_affine=True)
     (linear2): Linear(in_features=512, out_features=256, bias=True)
     (dropout2): Dropout(p=0.0, inplace=False)
     (drop_path): DropPath()
    )
   )
   (norm): LayerNorm((256,), eps=1e-05, elementwise_affine=True)
   (fc): Linear(in_features=256, out_features=5013, bias=True)
  )
 )
 (estimators_): ModuleList()
 (_criterion): CrossEntropyLoss()
)

You see, there is a fc (fully connected layer) in the last of the base model. I want to delete it or replace it with Identity. How to do it? It seems removing the layer in the base estimator does not do anything.

Hi, @desertSniper87, estimators are stored in the module estimators_, but it looks like there is nothing in your classifier, could you show us how you initialized the model ?

Created the ensemble using this
from torchensemble import VotingClassifier

ensemble = VotingClassifier(
    estimator=cct_model,
    n_estimators=5,
    cuda=True
)

Took the cct_model from here:
from src import cct_7_7x2_224
model = cct_14_7x2_224(**model_args)

OK, after fitting your ensemble model, you can simply index the target estimator using model = ensemble.estimators_[i], which should be a object of cct_model, you can then extract the embeddings using the corresponding model parameter.

Thank you for your response.


Issue with tasks Manage profile NICs and virtual machines NICs

Hi,
Using this role, I cannot correctly create any NIC by setting a specific network.
I wonder why I don't find any example of playbook using this role and doing what I need on the Internet. Is it really working or I do not use the right syntax in my playbook ?
This is my playbook using this role. Each time I run it, it works, bu the network is not set on my NIC eth0
---
- hosts: localhost
  connection: local
  gather_facts: false

  vars:
    engine_url: <censored>
    engine_user: <censored>
    engine_password: "<censored>"
    engine_cafile: <censored>

    debug_vm_create: yes

    classique_vm:
      cluster: <censored>
      datacenter: <censored>
      state: running
      storage_domain: <censored>
      template: <censored>
      clone: yes
      memory_guaranteed: 1MiB
      nics:
        - name: eth0
          network: <censored>
      cloud_init:
        host_name: <censored>
        root_password: '<censored>'
        authorized_ssh_keys: <censored>
        regenerate_ssh_keys: true
        nic_name: eth0
        nic_ip_address: <censored>
        nic_on_boot: true
        nic_boot_protocol: static
        nic_gateway: <censored>
        nic_netmask: <censored>
      cloud_init_persist: true

    vms:
      - name: <censored>
        profile: "{{ classique_vm }}"

  roles:
    - ovirt-vm-infra

These two tasks seem very similar :
https://github.com/oVirt/ovirt-ansible-vm-infra/blob/969da98c4b85659956fd3b93c64cffefba33ab9a/tasks/vm_state_present.yml#L34-L47
https://github.com/oVirt/ovirt-ansible-vm-infra/blob/969da98c4b85659956fd3b93c64cffefba33ab9a/tasks/vm_state_present.yml#L49-L62
Does the 2nd one even make sense since it seems to rely on a nics property of the vms item itself, rather than the one inside its profile ?

Hello friend,
This is normal I think. First task will create VNIC defined in your vm's profile (describe in your playbook), the second will create VNIC defined in your vm (describe in your playbook too)
This is the result of execution of these 2 tasks.
It seems to be done, but the VNIC's network is not selected in RHV. Even if its name is !
TASK [ovirt-vm-infra : Manage profile NICs] ************************************
task path: /var/lib/awx/projects/rhv_installation_vm/roles/ovirt-vm-infra/tasks/vm_state_present.yml:34
<localhost> ESTABLISH LOCAL CONNECTION FOR USER: <censored>
<localhost> EXEC /bin/sh -c 'echo ~awx && sleep 0'
<localhost> EXEC /bin/sh -c '( umask 77 && mkdir -p "` echo /var/lib/awx/.ansible/tmp/ansible-tmp-1540451713.68-227700303933587 `" && echo ansible-tmp-1540451713.68-227700303933587="` echo /var/lib/awx/.ansible/tmp/ansible-tmp-1540451713.68-227700303933587 `" ) && sleep 0'
Using module file /usr/lib/python2.7/site-packages/ansible/modules/cloud/ovirt/ovirt_nics.py
<localhost> PUT /var/lib/awx/.ansible/tmp/ansible-local-3ZTM4h3/tmpeLpu_p TO /var/lib/awx/.ansible/tmp/ansible-tmp-1540451713.68-227700303933587/ovirt_nics.py
<localhost> EXEC /bin/sh -c 'chmod u+x /var/lib/awx/.ansible/tmp/ansible-tmp-1540451713.68-227700303933587/ /var/lib/awx/.ansible/tmp/ansible-tmp-1540451713.68-227700303933587/ovirt_nics.py && sleep 0'
<localhost> EXEC /bin/sh -c '/usr/bin/python /var/lib/awx/.ansible/tmp/ansible-tmp-1540451713.68-227700303933587/ovirt_nics.py && sleep 0'
<localhost> EXEC /bin/sh -c 'rm -f -r /var/lib/awx/.ansible/tmp/ansible-tmp-1540451713.68-227700303933587/ > /dev/null 2>&1 && sleep 0'
changed: [localhost] => (item=[{u'profile': {u'datacenter': u'<censored>', u'storage_domain': u'<censored>', u'clone': True, u'memory_guaranteed': u'1MiB', u'cluster': u'<censored>', u'state': u'running', u'template': u'<censored>', u'cloud_init_persist': True, u'cloud_init': {u'authorized_ssh_keys': u'<censored>', u'nic_netmask': u'<censored>', u'nic_gateway': u'<censored>', u'regenerate_ssh_keys': True, u'nic_name': u'eth0', u'nic_ip_address': u'<censored>', u'host_name': u'<censored>', u'nic_boot_protocol': u'static', u'nic_on_boot': True, u'root_password': u'<censored>'}}, u'name': u'<censored>'}, {u'name': u'eth0', u'network': u'VLAN748/VLAN748'}]) => {
    "changed": true,
    "diff": {
        "after": {},
        "before": {}
    },
    "id": "0ca50c35-247f-4398-9970-dba39777dbe9",
    "invocation": {
        "module_args": {
            "fetch_nested": false,
            "interface": null,
            "mac_address": null,
            "name": "eth0",
            "nested_attributes": [],
            "network": "VLAN748/VLAN748",
            "poll_interval": 3,
            "profile": null,
            "state": "present",
            "template": null,
            "timeout": 180,
            "vm": "<censored>",
            "wait": true
        }
    },
    "item": [
        {
            "name": "<censored>",
            "profile": {
                "clone": true,
                "cloud_init": {
                    "authorized_ssh_keys": "<censored>",
                    "host_name": "<censored>",
                    "nic_boot_protocol": "static",
                    "nic_gateway": "<censored>",
                    "nic_ip_address": "<censored>",
                    "nic_name": "eth0",
                    "nic_netmask": "<censored>",
                    "nic_on_boot": true,
                    "regenerate_ssh_keys": true,
                    "root_password": "<censored>"
                },
                "cloud_init_persist": true,
                "cluster": "<censored>",
                "datacenter": "<censored>",
                "memory_guaranteed": "1MiB",
                "state": "running",
                "storage_domain": "<censored>",
                "template": "<censored>"
            }
        },
        {
            "name": "eth0",
            "network": "VLAN748/VLAN748"
        }
    ],
    "nic": {
        "href": "/ovirt-engine/api/vms/6efb4975-baac-4fc4-bbba-3162dc945c28/nics/0ca50c35-247f-4398-9970-dba39777dbe9",
        "id": "0ca50c35-247f-4398-9970-dba39777dbe9",
        "interface": "virtio",
        "linked": true,
        "mac": {
            "address": "00:1a:4a:16:01:9d"
        },
        "name": "eth0",
        "network_filter_parameters": [],
        "plugged": true,
        "reported_devices": [],
        "statistics": [],
        "vm": {
            "href": "/ovirt-engine/api/vms/6efb4975-baac-4fc4-bbba-3162dc945c28",
            "id": "6efb4975-baac-4fc4-bbba-3162dc945c28"
        }
    }
}
Read vars_file 'xorc_password.yml'

TASK [ovirt-vm-infra : Manage virtual machines NICs] ***************************
task path: /var/lib/awx/projects/rhv_installation_vm/roles/ovirt-vm-infra/tasks/vm_state_present.yml:49

Read vars_file 'xorc_password.yml'


nacl: use get_random_bytes syscall instead of SRPC

Native Client has a get_random_bytes syscall supported since pepper_39 that can be used to generate random bytes:
https://code.google.com/p/chromium/codesearch#chromium/src/native_client/src/trusted/service_runtime/include/bits/nacl_syscalls.h&sq=package:chromium&type=cs&l=128&rcl=1423805234
The nacl IRT calls it thusly:
https://code.google.com/p/chromium/codesearch#chromium/src/native_client/src/untrusted/irt/irt_random.c&sq=package:chromium&type=cs&q=get_random_bytes&l=20
If we're OK with depending on this (which means running on sel_ldrs from pepper_39 or newer) then we can call this directly instead of going through SRPC to look up the SecureRandom service.  It appears that this is the only use of srpc in go, so we could additionally remove the SRPC support entirely as newer nacl features are not exported in this manner.

I think this is a good idea. I'm almost certain that the nacl/arm builder
needs pepper_39 at least, so this isn't a big imposition.
On Sat, Feb 14, 2015 at 9:40 AM, James Robinson<EMAIL_ADDRESS>wrote:

Native Client has a get_random_bytes syscall supported since pepper_39
that can be used to generate random bytes:
https://code.google.com/p/chromium/codesearch#chromium/src/native_client/src/trusted/service_runtime/include/bits/nacl_syscalls.h&sq=package:chromium&type=cs&l=128&rcl=1423805234
The nacl IRT calls it thusly:
https://code.google.com/p/chromium/codesearch#chromium/src/native_client/src/untrusted/irt/irt_random.c&sq=package:chromium&type=cs&q=get_random_bytes&l=20
If we're OK with depending on this (which means running on sel_ldrs from
pepper_39 or newer) then we can call this directly instead of going through
SRPC to look up the SecureRandom service. It appears that this is the only
use of srpc in go, so we could additionally remove the SRPC support
entirely as newer nacl features are not exported in this manner.
—
Reply to this email directly or view it on GitHub
https://github.com/golang/go/issues/9870.

Great! Do you know if the other builders require something that new?

I had a quick look at the scripts which generate the base vm image for the
linux-nacl-{386,amd64} builders but it wasn't clear
https://github.com/golang/build/blob/master/env/linux-x86-nacl/scripts/build-go-builder.sh
@bradfiz which pepper version were those components taken from ?
As for the nacl/arm build, that runs on a dev board on my desk, I can
adjust it as necessary.
On Sat, Feb 14, 2015 at 10:00 AM, James Robinson<EMAIL_ADDRESS>wrote:

Great! Do you know if the other builders require something that new?
—
Reply to this email directly or view it on GitHub
https://github.com/golang/go/issues/9870#issuecomment-74342471.

I wrote a patch: https://go-review.googlesource.com/#/c/4860/ but gerrit gold me somebody already did this: https://go-review.googlesource.com/#/c/1755/1 (probably better)

And this issue is a dupe of issue 9261 (which I didn't find since it doesn't have the os-nativeclient label).

Yes, it's a dupe of #9261.


Slick js doesn't work on mobile, only on desktop

I don't know why, but when I access my website through my pc the slider works correctly, now on the phone only the main slider is left, why is that?
My css code:
.carousel.slick-initialized{
	visibility:visible;
	opacity:1;
}

.carousel .slick-slider{
	display:none ;
}

.carousel .slick-slider:first-child{
	display:block; visibility:hidden;
}

.carousel.slick-initialized .slick-slider{
	display:block!important;visibility:visible!important;
}

My requests and JS
<link rel="stylesheet" type="text/css" href="https://cdnjs.cloudflare.com/ajax/libs/slick-carousel/1.7.1/slick-theme.css" />
<link rel="stylesheet" type="text/css"<EMAIL_ADDRESS><script type="text/javascript"<EMAIL_ADDRESS><script>
$(document).ready(function() {
    $('.carousel').slick({
		infinite: true,
    	dots: true,
		arrows:false,
   		autoplay: true,
    	autoplaySpeed: 6000,
		arrows:false,

  });

@ChristianoBourguignon if I've understood your question correctly it sounds like you might need to use the responsive option, where you can set how many slides you want to display at different breakpoints.
E.g.
$('.responsive').slick({
  dots: true,
  infinite: false,
  speed: 300,
  slidesToShow: 4,
  slidesToScroll: 4,
  responsive: [
    {
      breakpoint: 1024,
      settings: {
        slidesToShow: 3,
        slidesToScroll: 3,
        infinite: true,
        dots: true
      }
    },
    {
      breakpoint: 600,
      settings: {
        slidesToShow: 2,
        slidesToScroll: 2
      }
    },
    {
      breakpoint: 480,
      settings: {
        slidesToShow: 1,
        slidesToScroll: 1
      }
    }
  ]
});

From the 'Responsive display' section here: https://kenwheeler.github.io/slick/

@ChristianoBourguignon se entendi corretamente sua pergunta, parece que você pode precisar usar a opção, onde você pode definir quantos slides você deseja exibir em diferentes pontos de interrupção.responsive
E.g.
$('.responsive').slick({
  dots: true,
  infinite: false,
  speed: 300,
  slidesToShow: 4,
  slidesToScroll: 4,
  responsive: [
    {
      breakpoint: 1024,
      settings: {
        slidesToShow: 3,
        slidesToScroll: 3,
        infinite: true,
        dots: true
      }
    },
    {
      breakpoint: 600,
      settings: {
        slidesToShow: 2,
        slidesToScroll: 2
      }
    },
    {
      breakpoint: 480,
      settings: {
        slidesToShow: 1,
        slidesToScroll: 1
      }
    }
  ]
});

Da seção 'Exibição responsiva' aqui: https://kenwheeler.github.io/slick/

Sorry if it was not very well explained, I am horrible at that ..
But anyway, the slider works perfectly on the desktop, but on the phone it only shows one image, it does not change the banner
you can see this by accessing makobaby.com.br on the pc (simulated mobile for you to see how it would look on the phone) and on the phone (it does not appear the other sliders)
note: all my sliders are in the slick-slider class and only the main one (the only one shown on the phone) is missing, so I believe it is something in the css, but this CSS is to hide the sliders during loading

@ChristianoBourguignon se entendi corretamente sua pergunta, parece que você pode precisar usar a opção, onde você pode definir quantos slides você deseja exibir em diferentes pontos de interrupção.responsive
E.g.
$('.responsive').slick({
  dots: true,
  infinite: false,
  speed: 300,
  slidesToShow: 4,
  slidesToScroll: 4,
  responsive: [
    {
      breakpoint: 1024,
      settings: {
        slidesToShow: 3,
        slidesToScroll: 3,
        infinite: true,
        dots: true
      }
    },
    {
      breakpoint: 600,
      settings: {
        slidesToShow: 2,
        slidesToScroll: 2
      }
    },
    {
      breakpoint: 480,
      settings: {
        slidesToShow: 1,
        slidesToScroll: 1
      }
    }
  ]
});

Da seção 'Exibição responsiva' aqui: https://kenwheeler.github.io/slick/

Summary: it is a banner using  to serve different image sizes, but on mobile it seems that I did an "unslick" and left the others hidden


[bug] pouch exec doesn't show the error if the command is not found

Ⅰ. Issue Description
pouch exec doesn't show any information if the command is not found.
Ⅱ. Describe what happened
When you use non-found command to the container, it should return the error, like pouch run does
➜  ~ pouch run registry.hub.docker.com/library/busybox:1.28 la
Error: failed to run container 922322: {"message":"failed to create task, container id: 9223220c840b008bb4577e87919e64cd0fcff9f5e065fe407764f14002c80595: OCI runtime create failed: container_linux.go:265: starting container process caused \"exec: \\\"la\\\": executable file not found in $PATH\": unknown"}

pouch exec doesn't shows error
➜  ~ pouch exec 2d1a47 la

But pouch exec -it shows error and acts like pouch run
➜  ~ pouch exec -it 2d1a47 la
OCI runtime exec failed: exec failed: container_linux.go:265: starting container process caused "exec: \"la\": executable file not found in $PATH": unknown

Ⅲ. Describe what you expected to happen
pouch exec should act like pouch exec -it when the command doesn't exist.
Ⅳ. How to reproduce it (as minimally and precisely as possible)

Ⅴ. Anything else we need to know?
Ⅵ. Environment:

pouch version (use pouch version):
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

this issue has been fixed.


Add support for a defaultFolderPath prop for the

Q
A

New feature?
[ x ]

Related issues?
fixes #936

What's in this Pull Request?
Added a property (defaultFolderAbsolutePath) to the <FilePicker/> to set a default selected library/folder for the <SiteFilePickerTab/>.

Thank you @Katli95 for the enhancement!
It has been merged and will be included in the next release!


Incorrect Content-Type for attachments

Suppose I attach an html file, like this:
< file.html curl -X PUT ntfy.sh/topic -H "Filename: file.html" --data-binary @-

I get a response like (some parts left out):
{
  "event":"message",
  "topic":"topic",
  "attachment": {
    "name":"file.html",
    "type":"text/html; charset=utf-8",
    "url":"https://ntfy.sh/file/ABCDEFGHI.html"
  }
}

However, when I visit https://ntfy.sh/file/ABCDEFGHI.html, I expect that the "Content-Type" header in the response would be "text/html; charset=utf-8". Instead, it's "text/plain; charset=utf-8".
So, if I'm using the web app, I would expect that it would render correctly in the browser, but it doesn't.
Additionally, I don't know if this is a different bug, or caused by the same thing, but if I open that file from an Android notification, I would expect to be able to open that file in my browser, or "HTML Viewer". But only text editors are listed. I end up having to save the attachment to another location, and open it from a file manager.

Of course. That makes a lot of sense.
I'll open a new issue did the Android thing with more details.
Thanks!


Merge the split cohort after the non-coded addresses are geocoded

This might necessitate a higher-level function that executes the three steps (getCohortAddresses, geocodeAddresses, importCohortTable), and does this merge between steps 2 and 3.
enableCohort()?

Depends on #183


Update dev dependencies.

Updating dev. dependencies to fix
Dependabot alerts
Resolves: OLPEDGE-2415
Signed-off-by: Oleksii Zubko<EMAIL_ADDRESS>

Codecov Report

Merging #408 (b5f23a9) into master (afdd2a7) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master       #408   +/-   ##
===========================================
  Coverage   88.2234%   88.2234%
===========================================
  Files            72         72
  Lines          2471       2471
  Branches        271        271
===========================================
  Hits           2180       2180
  Misses          161        161
  Partials        130        130

Impacted Files
Coverage Δ

@here/olp-sdk-dataservice-api/lib/config-api.ts
95.6990% <ø> (ø)

@here/olp-sdk-dataservice-api/lib/lookup-api.ts
59.2593% <ø> (ø)

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 afdd2a7...b5f23a9. Read the comment docs.


Added SKU for "Ordered Product" event.

Added SKU for Ordered Product event so that it's consistent with Placed Order event. Also, Bonobos is asking for this - https://segment.zendesk.com/agent/tickets/47818
Klaviyo says it's recommended to do this too.

Lgtm


feature: add camera type 'brown' to account for Brown-Conrady optical…

Following the discussion in PR #225 here is a possible implementation of the 5 distortion parameters (Brown-Conrady) model that OpenCV also supports: K1, K2, P1, P2, K3.
This addresses Issue #165.
@lamerman @paulinus There is a problem somewhere in the bundle adjustment that causes this not to converge to a solution.  I've spent some time debugging it bug I can't seem to find the issue so I thought to put it up and ask because you both seem to have some experience with Ceres.

I have a small dataset with distortion coefficients which could be helpful to verify this PR. What's the best way to reference it? Shall I add it to data/ and update the PR ?

@bdholt1 it would be great if you can upload the data to a server and add the link here. Thanks!
About the convergence, it is likely that adding more parameters renders the problem ambiguous and hard to optimize. I think the main use case for using all of the brown parameters is to precalibrate the camera and then fix the parameters during bundle adjustment.
We can do that by setting a very small standard deviation on the parameters in config.yaml. We might want to add the possibility of not optimizing the parameters at all.

@bdholt1, I finally took the time to look at this. I think everything was good except for the copy/paste error above. It works well in my limited experiments.
Since the master has diverged a little since this PR I've merged it manually, and it lives in this branch now https://github.com/mapillary/OpenSfM/compare/feature-brown-camera-model
I'll do some more tests and merge to master. Thanks for your effort!

Thanks for reviewing @paulinus.  I feel a bit unlucky missing that one but
I'm glad you found it.  Do you need me to do anything else?  What's the
plan with the perspective camera in the future? Will it be removed seeing
as it's going to be redundant?
On 22 December 2017 at 10:04, Pau Gargallo<EMAIL_ADDRESS>wrote:

@bdholt1 https://github.com/bdholt1, I finally took the time to look at
this. I think everything was good except for the copy/paste error above. It
works well in my limited experiments.
Since the master has diverged a little since this PR I've merged it
manually, and it lives in this branch now https://github.com/mapillary/
OpenSfM/compare/feature-brown-camera-model
I'll do some more tests and merge to master. Thanks for your effort!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mapillary/OpenSfM/pull/239#issuecomment-353565419,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA5N5A59tG3qoW8ouxVPQmE8qZ7oy-wqks5tC36UgaJpZM4QbnOU
.

The simpler perspective model will not be redundant unless there is a simple way of locking a subset of the camera parameters. During bundle adjustment, if we let the parameters free, the result will not be the same with the two models since one has more free parameters than the others.
In general, optimizing all parameters of the Brown model during bundle adjustment is not a good idea since are generally underconstrained.
Thus, I see the following two scenarios

The exact calibration is unknown. In this case, the best is to use the simple perspective model and optimize its 3 parameters during bundle adjustment.

The exact calibration of the camera is known (for example using a chessboard calibration). In this case, the best is to use the Brown camera model and fix the parameters during bundle (using optimize_camera_parameters: no)

By the way, I've added the principal point parameters to the Brown model to make it equal to the one in OpenCV.

OK, that makes sense to use the Brown model only when parameters are provided and use the perspective otherwise.  Is there anything else you need me to do?  I can have another go with the dataset to see how well it performs now that you've found the bug.

It would be great if you could try your dataset with the branch https://github.com/mapillary/OpenSfM/tree/feature-brown-camera-model
thanks!


Pause/Play button somewhere

render.audio.volume(0)  && render.audio.volume(1)

@skftn This is added in pull request:  #8


Location of ProjectTemplate used for new projects

Hi,
The readme states that "ProjectTemplate is an empty project ..." but I cannot find it in this repo. I'm proposing to update the readme to be more precise as to where to find said template.
regards.

All fixed. The ProjectTemplate was removed in the previous commit and the README was never updated to indicate this.


Resumes testing

Reactivates testing. For now the tests concentrate on the orbital.py functions. Indirectly this covers also complex_fcn.py and a bit nuclear_potential.py. The next batch of tests will have to cover two_electron.py.

All changes from @robertodr accepted and then some... :-)


API key generation should highlight 'owner' selection

We have seen many support requests where authors were not able to push packages using API keys as those API keys were not scoped to the right owner. This happens when there are one or more orgs, the author belong to and want to publish the package against an organization account.
We would need to highlight the owner selection when there are ore than one owner to select from.
My recommendation is to make it an explicit selection without defaulting to the current owner. This way the author will be aware of the accounts options they want to select.

We should empty out the default account selection:

@skofman1 can you help prioritizing this issue as this may help in reducing support cost.


Test Planning and Management with Azure Test Plans

In the following lab:
In the step 5:
On the right side you can see that this test suite has three test cases designed to confirm expected behavior of the feature implementation. Double-click the Verify that user is allowed to save his credit card detail test case.
When i double-click a window pops up saying:
No test results found
Results will appear here when you run the test case
Thefore, i can´t proceed with Step 6, as i can´t find anywhere the "Related Work" panel

Just ran into the same problem.   There are no test plans, and teams have no users, message at top 'You are currently logged in as a user with basic access level and you are seeing this page as you don't have any test plans and test suites created for the selected team.'
Is it just the 'Test Planning and Management with Azure Test Plans' that is no longer maintained or is it all the labs?
Very disappointing.  Waste of time, to have set this up, and then not able to continue.


/admin/prizes thinks that treeek22 earned 2 tix Penny Dreadful Prizes for Week Ending Mar 2 but he only earned one - why? Total is also one out even after accounting for that but i can't work out why

This seems to be (at least in the case of https://pennydreadfulmagic.com/competitions/430/ ) the case of the person that comes third being credited as if they came 2nd in a top 4 situation.

Deck was incorrectly set as finish=8 after drop in swiss. Should have been finish=9. This is fixed in the scraper now and I manually fixed up prod db.
This made me think of running:
SELECT d.id AS deck_id, d.competition_id, d.finish, c.top_n FROM deck AS d INNER JOIN competition AS c ON d.competition_id = c.id WHERE (d.finish IN (4, 6,7,8) AND c.top_n = 8) OR (d.finish = 4 AND c.top_n = 4)
    -> ;
+---------+----------------+--------+-------+
| deck_id | competition_id | finish | top_n |
+---------+----------------+--------+-------+
|    1388 |             58 |      8 |     8 |
|    1466 |            140 |      8 |     8 |
|    1830 |            157 |      6 |     8 |
|    2349 |            174 |      8 |     8 |
|    2494 |            180 |      8 |     8 |
|    2504 |            180 |      6 |     8 |
|    2507 |            180 |      7 |     8 |
|    3481 |            188 |      7 |     8 |
|    3994 |            201 |      8 |     8 |
|    4286 |            210 |      6 |     8 |
|    4321 |            212 |      7 |     8 |
|    5872 |            253 |      6 |     8 |
|   10411 |            425 |      8 |     8 |
|   12017 |            453 |      8 |     8 |
|   12332 |            456 |      7 |     8 |
|   12836 |            472 |      7 |     8 |
|   12853 |            472 |      8 |     8 |
+---------+----------------+--------+-------+
17 rows in set (0.04 sec)

Hmm.

Scraper theoretically puts everyone on the correct finish value now, and I've fixed all these by hand. I'll check in a couple of weeks that this issue hasn't reoccurred.


GUI Interface

Is anyone aware of a GUI interface for git-tfs?
Perhaps it may work already in a git gui application?

Sorry for the late reply. I did not get this running on a mac, in fact I got confused between this project and mono git-tf. Sorry about that :(

in fact I got confused between this project and mono git-tf.

git-tf is a Microsoft project that has no GUI and is developed in java ;)


FLW-833 iOS. Fix Empty state on input field in Contacts

fix empty state top insets on contacts screen
include fee for outgoing transfers

Codecov Report

Merging #251 (5759251) into develop (7920849) will increase coverage by 0.99%.
The diff coverage is 55.98%.


Add platformLocation to navigate back when course not found

Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING and other relevant documents.

Description

add PlatformLocation to navigate back when course not found in apps/course/src/app/layouts/course-layout/course-layout.component.ts
closes danieljancar/chorizo#54

Hey @jibon0070,
Why did you close the PR? It was good, just needed some last changes! I'm thankful for your interest in contributor and would be happy to see you further helping us. 😄

Sorry bro I was free, my boss just dumped a pile of issue for me in my day job, really sorry


Modbus broken on sam0 due to DE race condition.

Describe the bug
When using modbus on the atsamd21 sam0 platform, the modbus uart driver de-asserts the DE pin on re-entry into the cb_handler_tx after the last byte has been written to uart_fifo_fill here.
However, this creates a race condition since after the last byte is written to the TX register and begins clocking out, the isr re-entry and modbus_serial_tx_off causes DE is de-asserted before the byte is fully transmitted.
This results in truncation of data on the last byte when using rs495 half duplex transceivers on the differential output.
The following output on low baudrates first identified in issue #38264.

Since the implementation of modbus expects data to be fully transmitted before modbus_serial_tx_off is called, modbus should explicitly check if tx has completed. The following code snippit along with the sam0 uart changes outlined in PR #38311 were able to successfully resolve the issue however others have reported on different platforms that this fix does not work.
diff --git a/subsys/modbus/modbus_serial.c b/subsys/modbus/modbus_serial.c
index 06e44bd241..9f5dddac5f 100644
--- a/subsys/modbus/modbus_serial.c
+++ b/subsys/modbus/modbus_serial.c
@@ -393,7 +393,7 @@ static void cb_handler_tx(struct modbus_context *ctx)
                                   cfg->uart_buf_ctr);
                cfg->uart_buf_ctr -= n;
                cfg->uart_buf_ptr += n;
-       } else {
+       } else if (uart_irq_tx_complete(cfg->dev) != 0) {
                /* Disable transmission */
                cfg->uart_buf_ptr = &cfg->uart_buf[0];
                modbus_serial_tx_off(ctx);

To Reproduce
Attach oscilloscope to TX and DE pins on the sam0 platform with 115200 baudrate.
Trigger a data transmission via the modbus driver.
observe on oscilloscope DE is de-asserted before the last byte is transmitted as shown in the image above.
Expected behavior
DE should de-assert after the last byte has finished being transmitted as shown below with the changes outlined above in this issue on sam0.

Impact
Modbus is non functional on hardware platform implementations that differentiate between uart_irq_tx_ready and uart_irq_tx_complete. This is due to modbus checking uart_irq_tx_ready before calling the cb_handler_tx funciton. cb_handler_tx is entered when the uart TX register is empty but the uart peripheral shift register has not finished shifting out the bits yet resulting in the issue described above.
Platforms which utilize the same underlying driver layer such as the nRF52 for both  irq_tx_ready and irq_tx_complete  methods are unaffected.
Logs and console output
Pictures shown above.
Environment (please complete the following information):

OS: Windows
Toolchain Zephyr SDK
Commit SHA or Version used:
Original commit fix on my fork 9d2758b8a7
From zephyr RTOS base commit d33017b458fbdb9974f70939f099be5bfcf35794

Additional context
The fix described in this issue was identified as broken on other platforms here https://github.com/zephyrproject-rtos/zephyr/pull/38311#discussion_r754478074 which might have similar hidden bugs in the underlying uart implementation to be found.
Similarly, when first diagnosing this issue, I also had a similar problem where the RTOS would hang infinitely in isr checking for tx_complete and it would never be set.
This was due to the INTFLAG register being |= with a bitmask  in the uart_sam0_irq_update function. Per the datasheet writing a '1' to any register in INTFLG clears it therefore |= resolving into INTFLAG = INTFLAG | bitmask would clear all pending interrupts resulting in any later checks with regs->INTFLAG.bit.TXC != 0 to always return 0. This fix for the sam0 was also included in PR #38311
This issue affecting modbus being at a higher level would need to be checked against multiple uart hardware implementations to ensure a similar bug does not exist which might go un-noticed otherwise.

I would like this issue to stay open


variation number

When I use PanGenie, the actual number of output variations is much smaller than the number of input variations. Why is this?

Please provide more information. How did you run PanGenie (commands / parameters)? How does your input VCF look like? What does the log say?


Dedup process_chart_trends from 4 copy/paste locations

@martinpovolny @h-kataria @dclarizio please review.

@chessbyte : please, see https://github.com/ManageIQ/manageiq/pull/6053
I was keeping it as WIP for 2 weeks because the backend was broken and the charts did not render -- see https://github.com/ManageIQ/manageiq/pull/6272

@martinpovolny  The smaller the PR, the easier it is to merge.  Although #6053 includes the refactorings in this PR, that one is much bigger and harder to merge.  This PR is easier to merge because it is a straight de-dup of 1 copy/paste that can be merged by eyeballing the code, even if the backend does not work properly.

Thx @chessbyte for the simple clean up.


RenameVariables does not work on variable in EXCEPT arguments

I recently decided to enable RenameVariables transformer and noticed it missed variables in the EXPECT arguments.
BEFORE

AFTER

Code to reproduce
My Keyword
    [Arguments]  ${expected error}
    TRY
        No Operation
    EXCEPT  ${expected error}
        No Operation
    END

It does not seem to update the variables used in WHILE's limit option either.
BEFORE
My Keyword
    [Arguments]    ${my condition}    ${my limit}
    WHILE    ${my condition}    limit=${my limit}
        No Operation
    END

AFTER
My Keyword
    [Arguments]    ${my_condition}    ${my_limit}
    WHILE    ${my_condition}    limit=${my limit}
        No Operation
    END


Remove slider widget from attendance count & invoice amount

Just let it accept a inputted number.
I've never used it, and actually had it cause problems when my scroll wheel activates it and changes the value.

@maneesha Can you show me a screenshot? I have never seen such thing in AMY, so it may be browser-dependend.

Sorry... meant "spinner" not "slider"

@maneesha is this still an issue?
Unfortunately & probably, it's not possible to switch only one field to non-HTML5 mode – I think it's "all or nothing". But I'll check.

@maneesha ping on this issue

See the image in my comment on Oct 29.  The spinner (up/down arrows) mean that if I hit my mouse scroll wheel while in that field, I change the value.
I'd like to disable the spinner so the only way to enter a value in any numeric field (attendance, admin fee, etc) is to key it in.


ci: Add buildcache to all CI builds

It's way fast:

Also cleaned up our workflows a bit

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.005%) to 64.398%

Files with Coverage Reduction
New Missed Lines
%

goalc/listener/Listener.cpp
3
84.29%

Totals

Change from base Build<PHONE_NUMBER>:
-0.005%

Covered Lines:
39864

Relevant Lines:
61903

💛  - Coveralls


Add a note regarding play connectivity.

What changed, and why it matters
Since the HTTPS link to play will not work with VPC and filtered networks, I'm adding a note/warning per @kmichel-aiven
for services that are either restricted by IP addressed or are within a VPC without public access.

I'll be adding another commit to use relative path for the links.


Merge append and delete actions on indexes on modified source data

NOTES TO REVIEWERS:

Index metadata changes from another PR #170 by @pirz have been handpicked without which correctness cannot be guaranteed for this PR.
Changes include:

add "appended" list to index metadata

add "deleted" list to index metadata
Reviewers are NOT required to review PR #170 to review this PR.

What is the context for this pull request?
Hyperspace currently handles append support and delete support on modifiable source data separately. Users can either allow deletion of some source files and refresh their index, or they can allow adding files to data and refresh the index.
This PR merges these operations which will enable users to have a fully modifyable data source. Users can delete some of the indexed source files and also append some new source files. A refreshIndex(indexName, mode = "incremental") operation will allow users to update their indexes to accommodate for all these changes.

Tracking Issue: #149 , also Fixes #179, #183
Parent Issue: #136
Dependencies: NA

What changes were proposed in this pull request?
1
Merge the behavior of RefreshDeleteAction and RefreshAppendAction in a single user facing api refreshIndex(indexName, mode = "incremental"). When the user calls this api, hyperspace should update the index by removing records from all the deleted source files, and adding records from the newly added source files.
2 PR #188 MERGED
IndexLogEntry will support storing "appended" and "deleted" files (these changes were picked from #170 for correctness).
3
Update RefreshDeleteAction and RefreshAppendAction to also update the "appended" and "deleted" files list in index metadata.
4 PR #189 OPEN
If no updates to data source are found, it will NOT throw an exception anymore. This means:

If no deleted files are found, log a warning message and exit the RefreshDeleteAction without throwing exception.
If no appended files are found, log a warning message and exit the RefreshAppendAction without throwing exception.

Does this PR introduce any user-facing change?
Yes.

This PR introduces refreshIndex(indexName, mode = "incremental") for the users. Users can choose this over mode="full" for a faster refresh of index files.
index metadata files now contain "appended" and "deleted" list of files from original source data. These files are those for which index has not been updated.

How was this patch tested?
added unit tests where we delete some files from original indexed data, followed by appending new data to original data. A refresh(indexName, mode = "incremental") will update the indexes such that newly created index files will reflect the most recent version of the data.

Need to be consistent the lineage column config of each index entry when RefreshAppendAction -
https://github.com/microsoft/hyperspace/blob/1d3ac492b0445e2c64aa8798945ea29d9e8b4cd9/src/main/scala/com/microsoft/hyperspace/actions/CreateActionBase.scala#L171

Need to be consistent the lineage column config of each index entry when RefreshAppendAction -
https://github.com/microsoft/hyperspace/blob/1d3ac492b0445e2c64aa8798945ea29d9e8b4cd9/src/main/scala/com/microsoft/hyperspace/actions/CreateActionBase.scala#L171

oh nice catch :) ! thanks!

2 PR #188 MERGED
IndexLogEntry will support storing "appended" and "deleted" files (these changes were picked from #170 for correctness).
3
Update RefreshDeleteAction and RefreshAppendAction to also update the "appended" and "deleted" files list in index metadata.

Just for clarification: this is for quick refresh, but I think we could utilize this for RefreshDeleteAction - in case RefreshAppendAction failed.

There are deleted and appended files
RefreshDeleteAction => success / deletedFiles should be empty, appendedFiles should contain the list of appended files.
RefreshAppendAction

fail => non empty appendedFile should be handled properly in query time => hybrid scan
success => appendedFiles should be cleared.

After processing the appended/deleted source files, we should clear the lists.

2 PR #188 MERGED
IndexLogEntry will support storing "appended" and "deleted" files (these changes were picked from #170 for correctness).
3
Update RefreshDeleteAction and RefreshAppendAction to also update the "appended" and "deleted" files list in index metadata.

Just for clarification: this is for quick refresh, but I think we could utilize this for RefreshDeleteAction - in case RefreshAppendAction failed.

There are deleted and appended files

RefreshDeleteAction => success / deletedFiles should be empty, appendedFiles should contain the list of appended files.

RefreshAppendAction

fail => non empty appendedFile should be handled properly in query time => hybrid scan
success => appendedFiles should be cleared.

After processing the appended/deleted source files, we should clear the lists.

Thanks @sezruby , could you please check RefreshActionBase, RefreshAppendAction, RefreshIncrmentalAction, I think I am handling everything you mentioned here for refresh mode "incremental" as well, unless I missed any specific case, i think these changes are part of this PR.
Please Refer def logEntry, def appendedFiles, def deletedFiles in these classes.
Please let me konw if I missed any case.

Need to be consistent the lineage column config of each index entry when RefreshAppendAction -
https://github.com/microsoft/hyperspace/blob/1d3ac492b0445e2c64aa8798945ea29d9e8b4cd9/src/main/scala/com/microsoft/hyperspace/actions/CreateActionBase.scala#L171

oh nice catch :) ! thanks!

Just to be sure, bucket number also should be consistent with the previous log entry:
https://github.com/microsoft/hyperspace/blob/1d3ac492b0445e2c64aa8798945ea29d9e8b4cd9/src/main/scala/com/microsoft/hyperspace/actions/CreateActionBase.scala#L121

Please don't forget to handle this in 0.4: #187 (comment)

Thanks for the reminders. I prefer not to take this in this PR

Please don't forget to handle this in 0.4: #187 (comment)

Thanks for the reminders. I prefer not to take this in this PR

Could you please create an issue?

Can you or @thrajput also update the Python binding?
@AFFogarty can help on the C# side once 0.3 is released. (I will ping you @AFFogarty 😄 when it's ready)

Merging this as no responses from @sezruby and @pirz. Feel free to make comments if you have any.


how to develop with it

sorry, i am a java coder, so i am confused by how it work.
could you prompt me how to add a page and how to get data from DB or ES.
or what type of knowledge should i learn. thank you.

It doesn't get data from DB or ES, actually. This is the official SkyWalking query protocol for UI, https://github.com/apache/incubator-skywalking/blob/master/docs/en/protocols/README.md#query-protocol


Turn off https default for development environment

While I agree that https is a sensible default for production, I think that normal http makes more sense in development environments. Feel free to reject this PR if you disagree.

I don't really have an opinion one way or another on this. I run a self-signed cert locally just to not have to worry with things like this. The only caveat here would be that if the work in #211 is merged, the default environment is development. If someone only copies .htaccess.dist into it's production location and neglects to specify the production environment, by default we wouldn't be enforcing SSL.
All that said... there's a lot of FUD / "what-ifs" there that could be just as easily cleared up with documentation. I think it's a wash either way.

Thanks @zymsys


add local Reproducible Builds timestamp

this component is already reproducible in release 12 thanks to values inherited from parent
https://github.com/jvm-repo-rebuild/reproducible-central#org.apache.httpcomponents:httpcomponents-parent
but having a component-specific timestamp to mark component's release gives more natural result

@michael-o

@michael-o


Meaning of grasp labels

Dear author,
I saw there are some grasp annotations in the label files. What is the meaning of translation_d, rotation_d and width_d?
{"numgrasp": 146, "translation_d": 6.7442166274596925e-09, "rotation_d": 5.012725293206674e-05, "width_d": 0.0028016255560493948}
Are they used in the training process?
Best regards,
Shiyu Li

I have another question regarding to the paper. In the paper, it is mentioned "d denotes the depth offset from the grasp center to corresponding point cloud surface point. Refer to appendices for more details of our data preparation".
However I cannot find any appendices to the paper. Can you explain how to calculate "center_z_depth" in the grasping labels? Thank you very much.

Dear author,
I saw there are some grasp annotations in the label files. What is the meaning of translation_d, rotation_d and width_d? {"numgrasp": 146, "translation_d": 6.7442166274596925e-09, "rotation_d": 5.012725293206674e-05, "width_d": 0.0028016255560493948}
Are they used in the training process?
Best regards, Shiyu Li

Thank you for your interest of our work. These are some log files output during our grasp label processing scripts to check the validity of the processed labels. You can just ignore them.

I have another question regarding to the paper. In the paper, it is mentioned "d denotes the depth offset from the grasp center to corresponding point cloud surface point. Refer to appendices for more details of our data preparation".
However I cannot find any appendices to the paper. Can you explain how to calculate "center_z_depth" in the grasping labels? Thank you very much.

You can refer to Fig. 2 in our paper to get how to get the z offset. It's the distance of the grasp centers to the corresponding point in the pointcloud (oftenly the object surface) along z-axis.

Thank you very much


CSS differences of ix-icons in productive and dev build due to webpack optimization

What happened?
I am working on a docusaurus project and make use of ix-icons. When using the ix-icon tag inside an mdx-file and specifiying the size inside the tag (e.g. size = "16") this works fine for dev build. But when building the project for productive, webpack optimizes the css files and bundels them. In my case the .glyth-16 class gets bundeled with the h6 tag and therefore shiftet upwards inside the bundled css file. Due to the upshift the .glyph-16 class is not able to overwrite the font-size from the original .glypth class and therefore i can not set the font-size properly.
One possible solution that worked for my build is to define the size classes like .glyph-16 as nested classes like:
ix-icon > .glyph-16
What type of frontend frameware are you seeing the problem on?
React
Which version of iX do you use?
v1.4.1
Code to produce this issue.
Make use of the ix-icon tag with size="16" inside a mdx-file of a docusaurus project where the h6 tag is defined with a font-size of 1rem (same as font-size from .glyph-16). Webpack should bundle the classes in productive build and the font-size should not be set accordingly to size 16.

Please provide a repository to reproduce. You can take one of the Stackblitz Demo -> https://docusaurus.io/docs/playground

This issue depended on an older version and is resolved in the current version.


[Question] Encryption key type.

Hi, do you have any information regarding the encryption key? Like what parts of the file are encrypted? Is it everything starting after the data/payload offset, or is it only the chunk/keyframe information? Do you know what type of encryption format it is? I was guessing at AES or BigCrypt but I'm not sure? Is it possible to even decrypt the data?

There isn't a whole lot of information out about the rest of the information in the replay files. No one really knows what encryption they use, but I think I saw someone claim it was Blowfish. Additionally, the actual packet data in the replay file is obfuscated in the same way live packet data is obfuscated in live games. The obfuscation "key" changes each patch which is what makes old replays incompatible.


chain-registry: update chain.json to 0.8.0

someone needs to bump the version 0.4.0 to 0.8.0 here https://github.com/cosmos/chain-registry/blob/11f0b018eed478e4b238ac3e0aafc672df53a67c/testnets/elystestnet/chain.json
FWIW, I'm currently using this chain.json => https://gist.githubusercontent.com/andy108369/727bb9d8612fc66e9db35eebf183afe3/raw/51b703b3f895a2ec5e492dbc3ae8d7bf2a0c19d2/elys-chain.json
cc @cosmic-vagabond
refs
https://github.com/elys-network/elys/issues/39#issuecomment-1557259870

good call out @andy108369 I created this PR to update to the latest version as well as include the previous versions that were not added yet https://github.com/cosmos/chain-registry/pull/2498


Change workspace numbering

I know we all in the world of software like to start our lists at a 0 index, however this is a bit counterintuitive in the context of window managers. Starting at workspace 1 would make more sense and be more intuitive in most scenarios I think (in the top of the keyboard the first number to the left is 1, so for my first workspace I would look at 1), or at least include the possibility to declare them, like if a workspace is not associated with a shortcut in the config file then it is not enabled.
I hope this is clear, and loving the window manager so far!

like if a workspace is not associated with a shortcut in the config file then it is not enabled.

That's exactly how it works right now (if it doesn't then it's a bug, and I'd like to see your config)

Sure, here it is how it is active right now:
# Reference: https://github.com/i3/i3/blob/next/etc/config

enable-normalization-flatten-containers = true
enable-normalization-opposite-orientation-for-nested-containers = true

gaps.inner.horizontal = 4
gaps.inner.vertical   = 4
gaps.outer.left       = 4
gaps.outer.bottom     = 4
gaps.outer.top        = 4
gaps.outer.right      = 4

[mode.main.binding]
alt-enter = 'exec-and-forget open /Applications/Warp.app'

ctrl-h = 'focus left'
ctrl-j = 'focus down'
ctrl-k = 'focus up'
ctrl-l = 'focus right'

alt-shift-h = 'move-through left'
alt-shift-j = 'move-through down'
alt-shift-k = 'move-through up'
alt-shift-l = 'move-through right'

ctrl-alt-l = 'join-with right'
ctrl-alt-h = 'join-with left'
ctrl-alt-j = 'join-with down'
ctrl-alt-k = 'join-with up'

alt-f = 'fullscreen'

alt-s = 'layout v_accordion' # 'layout stacking' in i3
alt-w = 'layout h_accordion' # 'layout tabbed' in i3
alt-a = 'layout tiles horizontal vertical' # 'layout toggle split' in i3

alt-shift-space = 'layout floating tiling' # 'floating toggle' in i3

# Not supported, because this command is redundant in AeroSpace mental model.
# See: https://nikitabobko.github.io/AeroSpace/guide.html#floating-windows
#alt-space = 'focus toggle_tiling_floating'

# `focus parent`/`focus child` are not yet supported, and it's not clear whether they
# should be supported at all https://github.com/nikitabobko/AeroSpace/issues/5
# alt-a = 'focus parent'

cmd-1 = 'workspace 1'
cmd-2 = 'workspace 2'
cmd-3 = 'workspace 3'
cmd-4 = 'workspace 4'
cmd-5 = 'workspace 5'
cmd-6 = 'workspace 6'
cmd-7 = 'workspace 7'
cmd-8 = 'workspace 8'
cmd-9 = 'workspace 9'
cmd-0 = 'workspace 10'

cmd-shift-1 = 'move-node-to-workspace 1'
cmd-shift-2 = 'move-node-to-workspace 2'
cmd-shift-3 = 'move-node-to-workspace 3'
cmd-shift-4 = 'move-node-to-workspace 4'
cmd-shift-5 = 'move-node-to-workspace 5'
cmd-shift-6 = 'move-node-to-workspace 6'
cmd-shift-7 = 'move-node-to-workspace 7'
cmd-shift-8 = 'move-node-to-workspace 8'
cmd-shift-9 = 'move-node-to-workspace 9'
cmd-shift-0 = 'move-node-to-workspace 10'

alt-shift-c = 'reload-config'

alt-r = 'mode resize'

[mode.resize.binding]
h = 'resize width -50'
j = 'resize height +50'
k = 'resize height -50'
l = 'resize width +50'
enter = 'mode main'
esc = 'mode esc'

I just installed it, and used mostly the i3-like config as a base. And here is proof of the space 0 being active:

Can't reproduce

What happens if you relaunch AeroSpace?
What happens if you do aerospace workspace next && aerospace workspace prev in CLI?

Yep, fully relaunching AeroSpace fixed the issue, I was just using the reload config command. Perhaps spaces persist through reloads? I tried relaunching before the command you specified so I couldn't tell what happened then.

Reloading the config doesn't change the current layout, and currently focused workspace. If AeroSpace did that, it could mess up your active layout
You could switch away from workspace 0 after the config reload, and the workspace would go away
So it all works as designed

Thanks for the explanation!


Variables with default values don't work for image tags

Trying to use {env.VERSION:v1.0} (or anything else for the default value) results in an error.
[ERROR] failed to load config from "dobi.yaml": Error at myimage.tags: failed validation: the first tag "v1.0}" may not include an image name

I'm guessing it's trying to validate the tag before the variable is evaluated.

Thank you for the bug report!
One way to fix this might be to have ValidateTags create a call something like this https://github.com/dnephin/dobi/blob/master/execenv/environment.go#L45-L51 on the tag value, before parsing it, to remove the template.


Write and parse address in url

fixes #10
The addresses are written into the url inside the 'point' param. Separated from the coordinate with a '_' . I expect this character to appear in adresses not very often. The current approach handles a case where '_' would be part of an address by only parsing the part before the '_' character.
Example URL: http://localhost:3000/?point=52.464846%2C13.443016_Karl-Marx-Stra%C3%9Fe%2C+12057+Berlin%2C+Deutschland&profile=car&layer=OpenStreetMap

I didn't tag you before, but maybe @karussell should also look at the url format, whether you are fine with this solution.

I expect this character to appear in adresses not very often.

But even if it would be it wouldn't matter (?)

The current approach handles a case where '' would be part of an address by only parsing the part before the '' character

I like it :+1:
We might later introduce some kind of compression as for many points we will likely blow up the max URL length. (for current GH maps 1.0 we do not include the coordinates in the URL (only addresses), but this has other limitations like the need to wait for geocoding on the first route request and also in rare cases this produces different geocodes over time)

But even if it would be it wouldn't matter (?)

The address wouldn't be properly added to the textbox. The coordinate would still be correct.


Fixed #2416

Thanks your PR!
But commits and changes look very confusing?

I think this PR was a mistake?

Yes it was. Somehow the company intern files got pushed in here T_T

Not the first time :joy:


Readme Invoke-RemoteDesktopServer should be Invoke-RemoteDesktopViewer

In the readme Invoke-RemoteDesktopServer should be Invoke-RemoteDesktopViewer.

I will take a look and fix that thanks ;-)


Better api error handling

All I wanted to fix is #47 .. suddently it ended up in a short rewrite 😅
fixes #47
Main fix is here:
Instead of throwing a GradleException in case API return != 200 status code, we return an Result.failure (Kotlin builtin).
This is how it look before this PR:
https://github.com/gradle/foojay-toolchains/blob/15211a0a6958f43ca711a5ffdf3013c449effb4c/foojay-resolver/src/main/kotlin/org/gradle/toolchains/foojay/FoojayApi.kt#L107-L109
Now it looks like this
        if (status != HttpURLConnection.HTTP_OK) {
            return Result.failure(GradleException("Requesting vendor list failed: ${readContent(con.errorStream)}"))
        }

The result will be bubbled up until it reaches the Resolver.
The Resolver will then do a logging (warn level) and return an Optional.empty.
The resulting message will then instead of one of the linked issue be like:
* What went wrong:
A problem occurred configuring project ':domain:map'.
> Failed to calculate the value of task ':domain:map:compileJava' property 'javaCompiler'.
   > Cannot find a Java installation on your machine matching this tasks requirements: {languageVersion=18, vendor=AZUL, implementation=vendor-specific} for MAC_OS on aarch64.
      > No locally installed toolchains match and the configured toolchain download repositories aren't able to provide a match either.

(Ignore the case that a java 18 Zulu package is available, for testing purpose I modified the result.success case to be an error 😉 )
This is the "default" error message that will be displayed by the Gradle build tool itself and I think it is more obvious (but more generic) than the one before.
Furthermore, if running with --info (or something) the output includes Failed to resolve Java toolchain.
Not sure if we want to enhance this? 🤔
We can 🤷 😁
This doesn't "fix the problem" in case of foojay api is down AND there is no local jdk installed.
In this case users are "locked in" and can't work until foojay api is up again.
But I guess providing a fallback is out of scope of this plugin ...
Maybe we can enhance the error message with "If foojay api is down, use one of the locally installed java versions instead: [listOfLocallyAvailableJdks]"
What do you think? 🤔
Whatever, about the rewrite.. 😁
I extracted a FoojayService out of FoojayApi.
FoojayApi has only the responsibility to make the connection and return an Result<JsonString>.
Thats it.
FoojayService will handle the findMatchingDistribution, followed by the findMatchingPackag logic...
I have the feeling this makes things a bit nicer ...

Thanks @StefMa. I keep wondering if it wouldn't be a better fix to catch the original exception on the Gradle side and provide a better error message... Because with or without your changes, the build has to fail; it can't really continue (it attempted to download a toolchain because there is no local alternative). I will get a second opinion internally.

Currently it will fail.
But not with the Exception error message thrown by the foojay-toolchain plugin , but with the Gradle error message

No locally installed toolchains match and the configured toolchain download repositories aren't able to provide a match either.

And I guess this is the right approach.
Because, in fact, the user don't have locally installed toolchains and the toolchain repositores aren't able to provide a match either (in the reported issue case because the API returns status code).
So this behavior is... correct?!
Why should Gradle itself catch an exception in case an toolchain repository throws an error?
In case a toolchain repo can't provide a matching URL (for whatever reasons, backend down, requested jdk is not available anymore, ... 🤷 ) it should return an Optional.empty.
I think Gradle shouldn't catch exceptions happen in that toolchain repos because it also don't catch exceptions in (normal) plugins. They will also be just thrown.
Just my two cents on that 🙃

@StefMa after thinking a bit more about it, I propose we do the following:

NOT do this change to the plugin
Do this change on the Gradle side instead
Optionally, if you are interested, add some exponential backoff type of retry mechanism to the plugin, to reduce the number of failures

I think the Gradle side change is necessary, because, if one resolver fails, then we still want to try subsequent ones, if they have been set.
This fix will also offer a solution to build authors: they will now be able to set multiple resolvers (assuming the community writes some more) to have a backup when Foojay backend failures happen.
WDYT?

Hey @jbartok,
I guess the propsed change on Gradle side makes a lot of sense.
As you mentioned, throwing of one toolchain provider shouldn't prevent Gradle to check additional registered tolchain providers 👍
I also think that adding a retry mechanism to this plugin makes sense.
Just two or three times should be enough to fetch some hiccups.
However, I also think it make sense to still add this fix to the plugin. Even though Gradle will catch exceptions, the goal of a toolchain provider should be to provide an Optional.empty in case it can't provide a toolchain URI. Other toolchain providers, if there any 😁 , shouldn't think that, taking this plugin as a reference, throwing in the middle of the process is a "good"/"accepted"/"fine" pattern.
This plugin throw a exception by intention. I guess this shouldn't be the case.
As long as toolchain provider authors can catch/avoiding throwing exceptions, they should doing that.
I guess the fix here and the fix in Gradle are separated things.
The fix in Gradle is needed in case a toolchain provider does a mistake and throws for "edge cases"...
Both fixes strengthen the toolchain support in Gradle.
But I guess both are desired to work separated and intepnend from each other... 🤔

I'm not sure we want to blur the distinction between "Fojjay has no matching toolchains in their repertoire" and "Foojay has crashed and burned". Will think about it.

@StefMa, your reasoning makes sense to me, and I'm not 100% sure of myself when I say this, but let's go with JUST the Gradle side fix for now. We can always revisit this later if we change our minds.
I will open a separate issue for the exponential back-off improvement to the plugin, in case somebody decides to implement it.


Fixed SSH Keys Permission Denied

The issue reproduce for domain users ( domain\username, actually code uses only username )

Nice :+1:
@allcontributors add @dimitory for code


Fix git login problem for purescript-halogen dep.

I was getting the following error when using the original url:
fatal: Could not read from remote repository.
Please make sure you have the correct access rights and the repository exists.

Awesome, thanks :D

Thanks for this project! :) :+1:


Add license

This PR adds a missing license 🎉
Could you please publish this change to npm after merging?

Yes pls, without an explicit license file this repository is proprietary code and cannot be used. MIT requires full license text to be present alongside the code.


VERY high CPU and GPU usage for no reason

I have a 7700k and a 2070 SUPER. I don't think I should be getting these numbers at all let alone in a blank chart.

You probably have the framerate settings set to unlimited or a very high number. Reducing that should fix this issue.

Ah yeah my bad.
I don't remember changing it to unlimited, but if that's the default maybe consider setting it to something else. Also maybe move that option to the graphics tab in settings?


Xamarin.Forms - ListViewCachingStrategy.RecycleElement issue

Problem:
If we use Bindings for setting Source property in a ViewCells which are used in a ListViews with ListViewCachingStrategy.RecycleElement enabled - images get loaded into CachedImage not correctly (they are mixed).
It's because:

Note: When devolopers specify ListViewCachingStrategy.RecycleElement, OnElementChanged events are not raised when cells are recycled. Instead, the cell is retained and its property values change when the binding context is updated to that of an available cell, OnElementPropertyChanged events are raised. Application developers should remember to listen for the correct events, and should note that their renderers will need to be updated if the default behavior changes to ListViewCachingStrategy.RecycleElement in a future release.

To avoid it:
Instead using bindings for setting Source property - please use ViewCell's OnBindingContextChanged override to update it avoiding loading images to wrong views.
Any ideas how can we overcome that? Or should we leave it as is as maybe it's intended behavior when using this kind of ListViewCachingStrategy?

I would suggest writing about it in XF Slack room: https://xamarinchat.slack.com
Maybe some XF guys is there

@daniel-luberda What are the symptoms of this exactly? We occasionally have an issue where the wrong image gets cached - eg: user 1's profile image is actually displaying user 3's profile image and it stays that way throughout the app (so the wrong image is being attached to the cache key somehow). Haven't looked into it too much but it could always be occurring on the main page which uses recycled cells.
If this isn't a symptom of this I may need to open a new issue.

@ahohns Hi, sorry for answering so late. I've been quite busy.
Symptoms are that images get mixed between ListViews rows (eg. image from third row shows image from first row, etc). Did you try to use OnBindingContextChanged override instead using bindings?

mm completely different issue then, thanks.

@ahohns would it be possible to open an issue about it and add some sample there?

Possibly - it's one of those "it very rarely happens but it does happen" things so there's no reliable way to repro and I'd need to make sure it isn't our server causing the problem first.

Hi all, I was seeing mixed images as I scrolled fast. I'm setting properties in OnBindingContextChanged. Please see here http://forums.xamarin.com/discussion/59402/implementation-of-custom-view-cells-for-a-listview-with-recycleelement#latest
This seems to fix the issue.

I think it's an intended behaviour. As Xamarin states, do not use bindings. This problem otherwise occurs on many other controls, too.

@coonmoo @daniel-luberda  John Miller from Xamarin confirmed it

Note: When developers specify ListViewCachingStrategy.RecycleElement, OnElementChanged events are not raised when cells are recycled. Instead, the cell is retained and its property values change when the binding context is updated to that of an available cell, OnElementPropertyChanged events are raised. Application developers should remember to listen for the correct events, and should note that their renderers will need to be updated if the default behavior changes to ListViewCachingStrategy.RecycleElement in a future release.

It's mentioned in the API documentation: https://developer.xamarin.com/api/type/Xamarin.Forms.ListViewCachingStrategy/

@daniel-luberda Do we need to do anything in the renderer? If not then let's close this

@molinch We don't need to change anything in renderers. I'll update WIKI.


Will you release different packages for different rules or modify the default preset?

For example, jsx/no-array-index-key corresponds to @eslint-react/jsx.
Currently, it is not possible to use the @eslint-react/eslint-plugin package to lint as described in the readme, because jsx-related rules cannot be correctly mapped.
TypeError: Key "rules": Key "@eslint-react/jsx/no-array-index-key": Could not find plugin "@eslint-react/jsx".

I'm using eslint flat config.

If you want to change the preset to fix it, i can make a PR for it.

Hi There.
Actually all rules belong to @eslint-react/eslint-plugin package, while eslint-react-jsx, eslint-react-component, etc. are internal utilities that do not contain any rules, and these utilities will be released separately for projects that need them.
In addition, there is another eslint plugin that uses the same rule names and preset format, https://github.com/DimensionDev/eslint-plugin See also
The correct configuration for manually specifying and using @eslint-react/jsx/no-array-index-key in a project is as follows
module.exports = {
    root: true,
    env: { browser: true, es2020: true },
    extends: [
        'eslint:recommended',
        'plugin:@typescript-eslint/recommended',
        // 'plugin:eslint-react/recommended',
    ],
    plugins: ['@eslint-react'],
    parser: '@typescript-eslint/parser',
    rules: {
        // The `jsx/` is part of the rule name, not a sub-package's name of @eslint-react
        "@eslint-react/jsx/no-array-index-key": "error",
    },
}

I've made a repo to show you how to do this https://github.com/Rel1cx/eslint-react-issue-8 , hopefully this helps!

Sorry for not providing a minimal reproduction right away, in fact I encountered the problem using eslint flat config.
https://github.com/hyoban/eslint-react-reproduction

Nice Reproduction! We do need to fix the mapping.

If you want to change the preset to fix it, i can make a PR for it.

That's great. I'd really appreciate it.

Also, if it's hard to fix this by changing the default presets, I'd consider splitting the @eslint-react/eslint-plugin package into multiple packages, as you suggested at the beginning.
(@eslint-react/eslint-plugin-jsx for the jsx rules, @eslint-react/eslint-plugin-hooks for the hooks rules, and so on.)
Then use a @eslint-react/eslint-plugin package to depend on all these rules.


[AutoDiff] Revamp and fix shouldBeDifferentiated across JVP and VJP.

shouldBeDifferentiated existed as three different versions in LinearMapInfo, JVPEmitter and VJPEmitter. What's worse is that the conditions were wrong in that we weren't checking for both active results and active arguments when deciding whether to differentiate an instruction. What's even worse is that JVPEmitter::visitApplyInst and VJPEmitter::visitApplyInst are duplicating almost the entire shouldBeDifferentiated logic, causing huge inconsistency and maintenance overhead (terrible).
This patch revamps all three shouldBeDifferentiated implementations into two APIs in LinearMapInfo:

bool LinearMapInfo::shouldDifferentiateApplyInst(ApplyInst *ai)
Whether the apply should be differentiated is determined sequentially from the following conditions:

The instruction has an active inout argument.
The instruction is a call to the array literal initialization intrinsic ("array.uninitialized_intrinsic"), where the result is active and where there is a store of an active value into the array's buffer.
The instruction has both an active result (direct or indirect) and an active argument.

bool LinearMapInfo::shouldDifferentiateInstruction(SILInstruction *inst)
Whether the instruction should be differentiated is determined sequentially from the following conditions:

The instruction is an apply and shouldDifferentiateApplyInst returns true.
The instruction has an active operand and an active result.
The instruction has side effects and an active operand.

This patch also replaces the custom activity-checking logic in JVPEmitter::visitApplyInst and VJPEmitter::visitApplyInst with calls to LinearMapInfo::shouldDifferentiateInstruction.
Misc changes:

Move the check for the "array.uninitialized_intrinsic" intrinsic to a helper function.
Rename linearMapInfo in VJPEmitter to pullbackInfo for consistency with JVPEmitter.

Thanks @bartchr808 for the pair-programming.

@swift-ci please test tensorflow


(PUP-9928) Cleanup puppetres.dll

New installers no longer use puppetres.dll, so the file won't be copied
when installing puppet-agent.

Puppetres.dll is no longer present on a fresh installation of puppet-agent, so when uninstalling puppet agent, %PROGRAMDIR%\Puppet Labs\.. is deleted entirely.

If puppetres.dll is present from an older version of puppet-agent, the uninstaller will not delete the file.
Investigating if a cleanup script can delete the file without any issues.

As the new version no longer overrides puppetres.dll, even if puppetres.dll is loaded and locked, the installer won't trigger the DHCP restart, but the file still cannot be deleted(maybe moved using the workaround solution).

A simple solution would be to keep the file there and not interfere with it if it is loaded/locked.
Another solution would be that upon install/uninstall to delete the file if it is not loaded, but if it is loaded,do nothing.

Using the first solution, so the installer will not interfere with puppetres.dll for the moment.

jenkins please test this on windows2012r2-64a,windows10ent-32a

Putting DNM back on as this must not merge until https://github.com/puppetlabs/puppet-agent/pull/1749 is merged.


Nested sandbox.zarr in 00_xenium_and_visium.ipynb

The LANDMARKS_SDATA_PATH is set to GENERATED_DATA_PATH / "sandbox.zarr" in the notebook. However when downloading and unzipping the data the sandbox.zarr directory contains sandbox.zarr itself. This can cause users who don't know this to get an error as the upper level is not a valid .zarr.

@LucaMarconato , what is your preferred fix here? I think just adding a note about this in a markdown cell would be sufficient. Thoughts?

@melonora thanks for reporting.
This is the intended behavior, the user should place the sandbox.zarr file in the path GENERATED_DATA, which is spatialdata-sandbox/generated_data/xenium_visium_integration.
In the notebooks it's written to make this path discoverable via symlinks, so as a user I think I would be aware that I need to place the unzipped file to the right place.
Anyway, to avoid confusion, I will add another note in the notebook when defining GENERATED_DATA_PATH / "sandbox.zarr" saying to the user that the files needs to be placed to the right location.

But just to be sure that I understood your concern correctly, here also the BC_atlas_we.h5ad and visium_copyKat.h5ad needs to be placed to the right location.

Why do you think that extra care needs to be placed to sandbox.zarr?

Ah! I think I get it! One may expect that sandbox.zarr would expand to spatialdata-sandbox, but sandbox.zarr was a bad chosen name that I never changed thereafter. A better name is manual_annotations.zarr. I will open an issue to rename the file.

Open the issue here: https://github.com/scverse/spatialdata-notebooks/issues/39, I close this issue because the problem is not about nesting but about a poorly chosen name.

it might be poorly choosen as well, but right now also when downloading you get this:
sandbox.zarr/sandbox.zarr where the top level sandbox.zarr is not a valid zarr and you indicate in the notebook that this is the level that will be read as zarr file, which causes an error.

For the record this is what you get:
Yeah, I've seen that as well. For now, I think we just need a simple comment in the notebook explaining the issue.

Ah ok I understand now. I can't reproduce this with macOS (both extracting using the finder or using unzip). I think it's Windows specific but I will now add a note in the notebook about that.


ERROR: Invalid option in build() call: "jsxSideEffects" Vite v4

Hey folks,
Great plugin! Been trying to get it to work with my custom Vite development server without much success so far. While trying to figure that out however, I upgraded to the latest version of Vite (4.0) and realized that this breaks the plugin in a different way:
error when starting dev server:
Error: Build failed with 1 error:
/Users/alexturpin/src/vpc-test/node_modules/vite-plugin-cloudflare/node_modules/esbuild/lib/main.js:244:12: ERROR: Invalid option in build() call: "jsxSideEffects"
    at failureErrorWithLog (/Users/alexturpin/src/vpc-test/node_modules/vite-plugin-cloudflare/node_modules/esbuild/lib/main.js:1624:15)
    at /Users/alexturpin/src/vpc-test/node_modules/vite-plugin-cloudflare/node_modules/esbuild/lib/main.js:1143:18
    at /Users/alexturpin/src/vpc-test/node_modules/vite-plugin-cloudflare/node_modules/esbuild/lib/main.js:1138:9
    at /Users/alexturpin/src/vpc-test/node_modules/vite-plugin-cloudflare/node_modules/esbuild/lib/main.js:678:9
    at handleIncomingPacket (/Users/alexturpin/src/vpc-test/node_modules/vite-plugin-cloudflare/node_modules/esbuild/lib/main.js:775:9)
    at Socket.readFromStdout (/Users/alexturpin/src/vpc-test/node_modules/vite-plugin-cloudflare/node_modules/esbuild/lib/main.js:644:7)
    at Socket.emit (node:events:513:28)
    at Socket.emit (node:domain:489:12)
    at addChunk (node:internal/streams/readable:324:12)
    at readableAddChunk (node:internal/streams/readable:297:9)

Also if anyone has successfully integrated this with a custom dev server, any tips? Even on Vite 3. Getting this error:
ReferenceError: Cannot access 'seenIds' before initialization
    at Object.delayDepsOptimizerUntil (file:///Users/alexturpin/src/vpc-test/node_modules/vite/dist/node/chunks/dep-5605cfa4.js:41492:55)
    at doTransform (file:///Users/alexturpin/src/vpc-test/node_modules/vite/dist/node/chunks/dep-5605cfa4.js:36846:36)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

Hey Alex, Sure, I'll work on it.

I updated one of the examples (basic) to vite@4 and it works without no problem. So it'd be good to try again (or the latest version), anyways, if it still does not work, I'd_ like to see a reproduction!
Same for the custom server, I'd like to see a reproduction.

@Aslemammad thank you so much for taking the time 🙏
After looking into it some more, the v4 problem seems to be related to using the Vite React plugin too, hence why it doesn't show up in the basic example. Here's a repro: https://github.com/alexturpin/vpc-test
Here's a repro for a custom server, back with Vite v3: https://github.com/alexturpin/vpc-test/tree/custom-server

Thank you so much Alex, I'll take a look soon.

@Aslemammad you rock!

Hey @alexturpin, Fixed in v0.3.0! Could you please check if it's fixed? Thank you so much.

Absolutely will try next week!

Sure, thanks.

@Aslemammad now getting TypeError: esbuild.context is not a function 🤔
Updated my example here: https://github.com/alexturpin/vpc-test

Looks like maybe esbuild was moved from dependencies to devDependencies?

I guess you need to update your esbuild version, and if possible, share the stack.

Sorry, I'm not sure I understand. Do I have to install my own esbuild peer dependency? It was my understanding that vite-plugin-cloudflare ran esbuild under the hood. Can you elaborate on what information I can provide about the stack? It's a simple vite app https://github.com/alexturpin/vpc-test

sure, I meant the error stack! let me run the app again and see, because last time it worked for me.

And in the server.ts file (for custom server), you also need to add this line of code, so the middlewares get applied.
  app.use(vite.middlewares)

I've been able to reproduce the error, and it is because Vite's using an older version of esbuild, so to fix the error, you need to update esbuild:
npm add -D esbuild@latest

Ah yes sorry my bad, I had put the full error in my repo README but not here 😅 Installing esbuild ourselves does seem to fix it, thank you. 🙏
Shouldn't esbuild be a dependency of vite-plugin-cloudflare in this case, just like in 0.2.0? If it makes its own use of it, and requires a specific version as well. Or at the very least a peer dependency? It seems to me this would happen to new users on a fresh install without too much warning otherwise.

sure, i'll add this to the docs.

Thanks for all the hard work on this plugin, it makes for a very slick setup

Sure, feel free to create more issues and suggestions, it'd help so much.


about api update

Hello,i want to update a project using mmdet 2.2.0 to mmdet 3.3.0.
so how to update the old import  to mmdet 3.3.0
# in mmdect 2.2.0
from mmdet.datasets import build_dataset, replace_ImageToTensor

@RangiLyu It seems that many APIs have changed from mmdet 2.x to 3.x, and the same goes for mmcv. Is there a place where we can check the new API?


#154223 fix width from desktop bug for character count fields

Changes:

Allows the width of a character count component to be changed with the column size property

You also need to increment the package version to 3.0.3


pynitrokey: 0.4.40 -> 0.4.41

Description of changes

Update pynitrokey to 0.4.41
Add shell completion to nitropy
Add new dependency: nethsm at 0.4.0

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Thanks for the review @panicgh, all changes applied.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1256


Ability to edit left sidebar text strings

Any chance we will see the ability to edit the (currently hard-coded) left sidebar text strings?
Examples:
1 I'd like to replace the word "Ignored" with "Deferred".  'Ignore' is not the right choice of verbiage at my institution.
2 The left sidebar isn't consistent with upper-case and lower-case words. Look carefully you will see what I mean.
(This seems like nitpicky stuff, I know, but I work with people who take language/spelling/grammar seriously and I want Nudge to look as professional and consistent as possible.)

I can fix point 2 but it is 99% chance I will not change the left hand and make it user definable. The code is already too complex, especially when you add customizable localization.

Sorry I meant point 1/2 but not your initial question

Thanks Erik. Much appreciated.
Keep up the good work. Looking for to rolling Nudge into production this spring.


Use esbuild metafile. Add support for 'entryNames' esbuild option.

Description
Currently (before this PR) the esbuild plugin would always output JS asset files as options.outdir + replaceExtension(page.data.url). Which means The esbuild option entryNames would not work.
Additionally the esbuild plugin tries to map output files from esbuild to lume pages (JS eintry points). This looks similar to this:
const page = pages.find((page) => withoutExtension(page.sourcePath) === withoutExtension(outfile.path));

When entryPoints is set/changed this no longer works. The plugin then can not find a page for the outfile and adds it as a chunk.
When using the entryNames option like this:
site.use(esbuild({
  options: {
    entryNames: "js/[name].[hash]",
  },
}));

Given a source file scripts/index.ts you would get something like the following output files:

js/index.WSRTVQIK.js

No src connection
Bundled and transformed file

scripts/index.ts

Src connection: scripts/index.ts
No transforms - raw source file

This PR adds support for the entryPoints option. It now uses the esbuild metafile which contains information about input and output files and their relation. Using this metafile we can properly map esbuild outfiles to lume pages. It now also uses the esbuild outfile filename as final data.url:
page.data.url = outfile.path.replaceAll(
  dirname(page.sourcePath),
  dirname(page.data.url),
)

The replace is there to support url overwrites via basename. Using lume url overwrites in combination with the entryNames esbuild option works generally. However as we only write the file to a different location to what esbuild intended, JS imports to this file (e.g. via dynamic imports) are not rewritten! Esbuild currently does not offer any functionality to dynamically rewrite an outfile path per file (as far as I know). So this is as much as we can do.
I have also mostly reverted the changes from commit https://github.com/lumeland/lume/commit/13796a7fe66c007aaca6a23084233247a46235fb as that workaround is no longer necessary.
Related Issues
None
Check List

[x] Have you read the
CODE OF CONDUCT
[x] Have you read the document
CONTRIBUTING

[x] One pull request per feature. If you want to do more than one thing,
send multiple pull request.
[x] Write tests.
[x] Run deno fmt to fix the code format before commit.
[x] Document any change in the CHANGELOG.md.

Note that the previous implementation completely ignored the esbuild output path and just wrote it to page.data.url - which also breaks dynamic imports if we use lume url overwrites via basename.

Yeah, to be honest, I hadn't thought about that situation until now! :)
Thanks for your contribution. The esbuild plugin is much stronger now!


Support for schema.org extensions

At the moment there is no for schema.org extensions.  Thanks to Paxa for the heads-up.
Currently if you have:
itemtype="http://schema.org/WebPageElement/ItemType"
Mida ignores this item, and push name attributes to parents' item.

No, I was mistaken (invalid layout). It works well
I test it on http://microdata.realitysimple.com/projects
doc.items[0].properties['mentions'][0].type
"http://schema.org/WebPageElement/ItemType"
doc.items[0].properties['mentions'][0].vocabulary
Mida::SchemaOrg::WebPageElement

But If I add new property to extended item it will be ignored, because extended item is validating as known found (http://schema.org/WebPageElemen in my case)


Chat with a specific user is not loading

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

Version Number:
Reproducible in staging?: needs reproduction
Reproducible in production?: needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @jamesdeanexpensify
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1718212058167969
Action Performed:

Open app and login<EMAIL_ADDRESS>Search for<EMAIL_ADDRESS>with who already have a chat
Select the user

Expected Result:
Chat report loading with past messages
Actual Result:
Skeleton is loading and in the console showing the updates, please watch the full video
Workaround:
unknown
Platforms:

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

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

Screenshots/Videos
https://github.com/Expensify/App/assets/38435837/6ff760ce-faf5-4691-a446-511fe7229f11

View all open jobs on GitHub

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

@jamesdeanexpensify to reproduce this I'd need to supportal into your account, are you OK with that?

I'd be fine with that but I just checked again today and all things seem to work fine. I'll close for now. Thank you!


WIP: Onboarding Guide

Working on documentation to help enterprise on-boarding processes

Closing this as this has been open with no activity for a while - it can always be re-opened if needed.

Re-opening this. I think it's important to document on-boarding. Need an owner.

@it33 I can own this guide.  I can see the benefit of having a document such as this and there are some things I'd like to add.

Thanks @wiersgallak!

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

@wiersgallak are you okay if we close this PR and move to GitLab feature proposal repo to discuss? https://gitlab.com/mattermost/proposals/issues/44
I was talking to Stu about this as well, and he's working on something here too.


How to get attachmented files on rainloop

RainLoop version, browser, OS:
V1.11.1
Chrome 61
Wdindows10
Expected behavior and actual behavior:
What I want to do is to get all files which are selected in a message view.
presumably this action is done in the messageView.js file.
So there is a function called getAttachmentsHashes() which it's returned value is in hashes.
How can I get files from these hashed so I can upload them with an ajax request?

What do you mean by "get all files"? Download all of them at once?

nope I mean the actual files, so I can upload asynchronously to another source.

Interesting because I'm looking for something similar. So far I have found you need to fetch attachments from message itself and store temporarily on the server. To do that there are two ways - to create download link and access it or to use DoMessageUploadAttachments() to store attachments.  But for now I'm somehow stuck on both. First requires web session/cookie to have it done and for second endpoint is a web part (JavaScript) but I need to get that done on the server part (PHP).

Another thing is that downloading selected attachments gives the link which is a zip file, not separate links

@Med-All  I have seen your previous issues with rainloop and i think you might be able to help. I am beginner and a intern right now. i am trying to integrate a calendar in which i want it to intercommunicate between rainloop and the calendar app to create a event automatically once i receive a mail with .ics or .iCal(just like google calendar). I am not able to find out how the attachments in rainloop are being handled. Any help for where to look will be highly appreciated.
i am looking inside attachment.php inside /app/libraries/MailSo/Mail/Attachment.php and mailclient.php

@mehi-shokri  were you able to to find out how attachments are being handled?

@danishsultan007 Nope.


A validation for numeric value limit

Hi!
Another suggetion for your lib is validation for numeric valeu limit.
example:
min_value = 1
max_value = 100

Good idea!

It has been implemented and it will be available in release 2.2.0.


gsoler2 Acrolinx quality push: bitsadmin-peers-and-clear.md

#assign: @robinharwood, @Xelu86
@robinharwood @Xelu86
Can you review the proposed changes?
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Looking good, merging now,


Prevent static libraries from being suggested as a path

$ find_libpython --candidate-names
libpython3.8.dll.a
libpython3.8.dll
python3.8.dll
python3.dll
python.dll

Additionally, candidate names contains libpython38.dll.a. AFAIK this is not a viable dynamic library name and should not be considered. get_config_var("LDLIBRARY") and get_config_var("LIBRARY") return libraries that are archives due to the python in question not being compiled with --enable-shared.
Originally posted by @ktbarrett in https://github.com/ktbarrett/find_libpython/issues/1#issuecomment-770097103

Closed by #16.